Closed han-f closed 4 years ago
I made a few small changes to the strings. Everything seems plausible. Do you have a preferred schema where the tables should be moved to?
Thanks @christian-rli . @l-emele had a few further suggestions on how to improve the tables themselves. This may result in changes of the metadata, too. How do you think we should best proceed with this? Keep tables in model_draft, overwrite them there and resubmit metadata (if it did change)? If yes, then we could keep this ticket open until we know whether the metadata has changed and react accordingly once we know?
I'd suggest just keeping this issue open @han-f . You can change the tables in model_draft and let me know here when you're done. Then there is no need to resubmit the metadata :)
If there are any relevant changes, you/I can make the necessary updates to the metadata (just commit your changes and reference this issue) and I can upload them afterwards.
files have been updated. please review.
For clarification: We not only update the metadata but also the table structure and the data on the OEP.
Unfortunately this review is now pending for almost three months. Could someone please review?
Why don't we use PRs for the review?
So far this was the process...
We use PRs to merge fully reviewed metadata into the develop branch. But I suggest to always create a new branch and PR. The issue should then be linked to the PR. We are currently reworking this process the result will be documented in our wiki.
Review of metadata and move of data finished!
We have uploaded the two tables related to units/metrics and conversion to model_draft and submit them to the review process
Metadata and data for review:
https://github.com/OpenEnergyPlatform/data-preprocessing/blob/review/SzenarienDB_conversion_text/data-review/SzenDB_meta_units.jsonhttps://openenergy-platform.org/dataedit/view/model_draft/szenariendb_unitshttps://github.com/OpenEnergyPlatform/data-preprocessing/blob/review/SzenarienDB_conversion_text/data-review/SzenDB_meta_conversion.jsonhttps://openenergy-platform.org/dataedit/view/model_draft/szenariendb_conversionReviewed and published metadata and data: