Task hvmv_substation fails due to missing table

[nesnoj]

2 clean runs of continuous-integration/run-everything-over-the-weekend-2022-06-10 (head: 740f279) for DE lead to the same error:

psycopg2.errors.UndefinedTable: relation "grid.egon_hvmv_substation" does not exist

Apparently, you did some renaming/changed import of the HV/MV substation table in #860.
The table should have been created in osmtgmod substation but èxtract() may never been called? @ClaraBuettner

[nesnoj]

BTW, grid.egon_ehv_substation is not created either

[ClaraBuettner]

Sorry, I forgot to change the table name in two lines of the hvmv-substation script, this should be fixed in b2376c2.

I also merged this branch into CI again.

[ClaraBuettner]

BTW, grid.egon_ehv_substation is not created either

These tables are now created in the osmtgmod dataset. The substation-dataset creates and fills the grid.egon_hvmv_transfer_buses and grid.egon_ehv_transfer_buses tables. Before these changes, the tables were filled in one dataset and updated by another one which resulted into problems when one of them was skipped due to versioning.

[nesnoj]

Thanks for the quick fix, I'll give it another try

[nesnoj]

Seems to work out

[nesnoj]

Now I'm getting psycopg2.errors.UndefinedTable: table "egon_mv_grid_district" does not exist in the task mv_grid_districts.define-mv-grid-districts (HEAD: 5e0ab3b).

It is raised in these lines. @ClaraBuettner you added this drop statement recently. It does not check if the table actually exists. To my mind it can be removed as in L572 there's already a drop statement for this table (which, btw, checks for existence).

[nesnoj]

After removing those lines the task finishes successfully

[ClaraBuettner]

I changed this in #859. Dropping without CASCADE resulted into problems because the eMobility-dataset has a ForeignKey to the mv grids.
I can add IF EXISTS to the drop statement.

[nesnoj]

I see, yes that would be nice if you added IF EXISTS.

And could you please delete the ORM drop statement then?

PS: It would be also possible to keep the ORM statement and define the cascade in the foreign key relationship but I had troubles in the past and don't wanna mess around with this again.

[ClaraBuettner]

I see, yes that would be nice if you added IF EXISTS.

And could you please delete the ORM drop statement then?

Done in 8cab0bf

[nesnoj]

thx

[nesnoj]

Just a sec, you also dropped MvGridDistricts.__table__.create(bind=engine), please keep this ;)

[ClaraBuettner]

Just a sec, you also dropped MvGridDistricts.__table__.create(bind=engine), please keep this ;)

Sorry for my confusion, I fixed it again...