Skip to content

Feature/copy from oedb to local db#362

Merged
IlkaCu merged 2 commits intodevfrom
feature/copy-from-oedb-to-local-db
Aug 26, 2020
Merged

Feature/copy from oedb to local db#362
IlkaCu merged 2 commits intodevfrom
feature/copy-from-oedb-to-local-db

Conversation

@eosram
Copy link

@eosram eosram commented May 8, 2020

  • Replace ego.io dependency (!)
  • Fix WKBElement workaround
  • Copy metadata JSON string
  • Hide configuration / use dataprocessing.tools.io.oedb_session
  • Provide some logging
  • Provide CLI

This pull request relies for the time being on small, but significant changes in ego.io (See PR openego/ego.io#84). I think these are the most relevant commits so far:
openego/ego.io@de46f50
openego/ego.io@382e312
openego/ego.io@370deb1

eosram added 2 commits May 8, 2020 15:05
In order for it to work the variable `LOCAL` in `main` has to be set to
the local database URL. `INPUT` specifies which tables should be copied
from the oedb.
Atm oedialect requires `geoalchemy < 0.7.0`.
@eosram eosram marked this pull request as draft May 23, 2020 15:49
@gplssm
Copy link
Contributor

gplssm commented May 26, 2020

Nice stuff 👏 we'll come back to it, when we implement the new DP workflow

@gplssm gplssm marked this pull request as ready for review August 26, 2020 07:36
@gplssm
Copy link
Contributor

gplssm commented Aug 26, 2020

@IlkaCu and @gplssm decided that this - even if unfinished - is valuable! We merge it, to include it in the code base for the fork for the new DP for eGo^n

@gplssm gplssm self-requested a review August 26, 2020 07:37
Copy link
Contributor

@gplssm gplssm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gplssm gplssm assigned gplssm and IlkaCu and unassigned gplssm Aug 26, 2020
@IlkaCu IlkaCu merged commit ed5afae into dev Aug 26, 2020
@IlkaCu IlkaCu deleted the feature/copy-from-oedb-to-local-db branch August 26, 2020 09:24
@IlkaCu IlkaCu mentioned this pull request Aug 26, 2020
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants