Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review: Datasets from EU-legislation #89

Open
14 of 15 tasks
han-f opened this issue Jan 7, 2022 · 25 comments · May be fixed by #100
Open
14 of 15 tasks

Review: Datasets from EU-legislation #89

han-f opened this issue Jan 7, 2022 · 25 comments · May be fixed by #100
Assignees
Labels
review Data and Metadata review SIROP v1.5.1 metadata created with version 1.5.1

Comments

@han-f
Copy link
Contributor

han-f commented Jan 7, 2022

Issue description

I am submitting a set of metadata for review. The metadata is attached to a series of tables and already available on the OEP.
These are projections from European Member States submitted to the European Commission according to EU-legislation.

Workflow checklist

  1. GitHub
  • I have submitted this issue to have metadata and data review documented (Issue Feature/eu leg#89 #90)
  • Create a new review-branch and push OEMetadata to new branch (feature/eu_leg#89). If this step is too difficult, attach a file with the metadata as a comment in this issue and let the reviewer know.
  1. OEP
  • Upload data to the OEP in schema model_draft (see upload tutorial)
  • Link URL of data in this issue (model_draft.project_nameofdata)
  1. Start a Review
  • Start a pull request (PR) from review-branch to master
  • Assign a reviewer and get in contact
  1. Reviewer section
  • A reviewer starts working on the issue
  • Review data license
  • A reviewer finished working on this issue (and awarded a badge)
  • Update metadata on table
  • Data moved to its final schema
  • Add OEP tags to table
  • Merge PR and delete review-branch
  • Document final links of metadata and data in issue description
  • Close issue

Metadata and data for review

Here are the links to my data and metadata:

Reviewed and published metadata and data

Final naming and location of the data and metadata after the review are as follows:
schema.tablename

eio_ir_article23_t1 (2014-2020)

eio_ir_article23_t3 (2015-2020)

eaa (2016-2020)

others

@han-f han-f added the review Data and Metadata review label Jan 7, 2022
@han-f
Copy link
Contributor Author

han-f commented Jan 7, 2022

Data tables including metadata are here: https://openenergy-platform.org/dataedit/view/model_draft?query=EU-legislation
cc @wingechr - as info that review process has started.

@steull steull assigned steull and unassigned jh-RLI and christian-rli Jan 31, 2022
@steull steull added the SIROP label Jan 31, 2022
Ludee added a commit that referenced this issue Jan 31, 2022
Ludee added a commit that referenced this issue Jan 31, 2022
@Ludee
Copy link
Member

Ludee commented Jan 31, 2022

We added the metadata for testing the review process and documenting changes.

Ludee added a commit that referenced this issue Feb 1, 2022
Ludee added a commit that referenced this issue Feb 1, 2022
Ludee added a commit that referenced this issue Feb 1, 2022
Ludee added a commit that referenced this issue Feb 1, 2022
@steull
Copy link
Contributor

steull commented Feb 1, 2022

@han-f, according to the licence, it is originally version 2.5 and your version is version 4.0. Are you aware that you are relicensing?

@han-f
Copy link
Contributor Author

han-f commented Feb 1, 2022

thanks for this - as we created a new database by harmonising terminology acorss all the submission years (they were different( and by introduced foreign key tables etc, does this not allow for a new license as this is very different to how the original data comes about?
Also the original data does not come as a table that holds all European Member States but as one table for each Member State for each submission.

Please note - the orginal licences also differ between data with source eionet (up to 2019 submissions) and source reportnet (starting 2021 submission)
cc @wingechr

steull added a commit that referenced this issue Feb 4, 2022
steull added a commit that referenced this issue Feb 7, 2022
@steull
Copy link
Contributor

steull commented Feb 7, 2022

We realised that the metadata doesn't follow the latest v.1.5.0. We will add new files.
Furthermore, we realised, that the table names are difficult to understand (eio_ir_art...). We suggest to name the files: scenario.eea_eionet_eu_legislation_data_... Perhaps, it would be useful to include some parameter names for better understanding.

steull added a commit that referenced this issue Feb 7, 2022
@han-f
Copy link
Contributor Author

han-f commented Feb 8, 2022

@wingechr what do you think?

@wingechr
Copy link
Contributor

wingechr commented Feb 8, 2022

I think the somewhat cryptic table names are ok, otherwise they become too long. The human readable title exists for that very reason.

About the metadata: what is missing for v.1.5.0?

And thanks for checking

@Ludee
Copy link
Member

Ludee commented Feb 8, 2022

The cryptic names are OK but we were not able to identify what it means. It would make sense to add additional info to the "description" or explain it here.
I suggest an alternative name that is closer to the OEP naming convention:
oekoinstitut_eea_eionet_eu_emissions_per_country_2020

I would also suggest to add some text about which data has been collected from the eionet website.
A comment of the method of collection would be good as well. Was it by hand or scripted?

Ludee added a commit that referenced this issue Feb 8, 2022
Ludee added a commit that referenced this issue Feb 8, 2022
Ludee added a commit that referenced this issue Feb 8, 2022
Ludee added a commit that referenced this issue Feb 8, 2022
@steull steull closed this as completed Apr 7, 2022
@Ludee Ludee reopened this Apr 7, 2022
@Ludee
Copy link
Member

Ludee commented Apr 7, 2022

It seems like we had a misunderstanding.
The tables we just moved were not ready for review yet.
Please move them back to model_draft.

@Ludee Ludee assigned wingechr and unassigned Ludee Apr 7, 2022
@Ludee
Copy link
Member

Ludee commented Apr 7, 2022

@wingechr @han-f please update the first comment with all tables that should stay in scenario.

@wingechr
Copy link
Contributor

wingechr commented Apr 7, 2022

it's easier for us if i move the tables back into model_draft (which I will do if no one objects). these are:

  • parameter dimension table eu_leg_dim_parameter
  • eionet parameter data tables eu_leg_data_****_eio_ir_article23_t3
  • reportnet parameter table: eu_leg_data_2021_rep_table_3

@wingechr
Copy link
Contributor

I moved the tables.

NOTE: for some reason, I got 500 Errors, although it seemed to have worked.

@wingechr wingechr reopened this Apr 21, 2022
@wingechr
Copy link
Contributor

wingechr commented Apr 21, 2022

moving tables from scenario to model_draft via API used to work.
now it still worked, but threw 500 server side errors and left table artefacts:

grafik

@MGlauer: could you at some point run the mirror task on the producion server? thx

@Ludee I have a hunch that the error is related to the fact that you moved the tables, but you were not the owner?

@wingechr wingechr assigned Ludee and unassigned wingechr Apr 21, 2022
@Ludee
Copy link
Member

Ludee commented May 31, 2022

I'm not sure if it is due to the permissions. I rather think it is some kind of bug when moving tables with existing metadata.
The tables are still in scenario and the suggested renaming has not been implemented. I'm not sure how to best rename existing tables. This can cause a lot of other problems.

@han-f
Copy link
Contributor Author

han-f commented May 31, 2022

I think the tables are back ins scenario, since @MGlauer had deleted the table artefacts and @wingechr had then moved the tables again.

I also think we can keep the table names as they are now and apply new names for tables that we subsequently add. Like this we won't produce further problems as these tables were now already shared with external people and I would be hesitant to do anything that could throw errors.

@Ludee
Copy link
Member

Ludee commented May 31, 2022

OK, I totally agree with the renaming. Let's keep it like this.

I updated the comment above to included all related tables.
Please check if the list is complete.

@Ludee
Copy link
Member

Ludee commented Jun 1, 2022

As discussed today @steull will update all metadata to v1.5.1, add it to the repo and included the review information (issue link and badge). Thanks to everybody for the constructive feedback!

steull added a commit that referenced this issue Jun 10, 2022
steull added a commit that referenced this issue Jun 10, 2022
steull added a commit that referenced this issue Jun 10, 2022
@steull
Copy link
Contributor

steull commented Jun 10, 2022

I have updated all tables on the OEP with the new version of the metadata

@steull steull linked a pull request Jun 13, 2022 that will close this issue
@Ludee Ludee linked a pull request Jun 13, 2022 that will close this issue
@Ludee
Copy link
Member

Ludee commented Jun 16, 2022

Please add all metadata strings from the linked OEP tables to the repo. @steull
In addition revert the changes by reviewers in the metadata -> remove "Öko-Institut" from title.

steull added a commit that referenced this issue Jun 16, 2022
@steull
Copy link
Contributor

steull commented Jun 16, 2022

All metadata are available under the updated link in the description.
I also changed the title and removed "Öko-Institut".

@han-f
Copy link
Contributor Author

han-f commented Jun 17, 2022

I still see the "Öko-Institut" prefix at some tables it, even after refreshing:

https://openenergy-platform.org/dataedit/view/scenario?query=&tags=180&tags=326

cc @wingechr

@han-f
Copy link
Contributor Author

han-f commented Jun 17, 2022

That said - now all parameter table titles also refer to the Monitoring Mechanism Regulation (MMR), which is not the case for 2021 and also the description seems to have been copied over and is now not reflecting the data sources.
grafik

I will edit that back by hand now.

Good the review process will be made easier. Sometimes data owners use the functionality on the OEP to edit and add metadata, and then it may not be reflected in github.

@steull steull added v1.5.1 metadata created with version 1.5.1 and removed v1.4 metadata created with version 1.4 labels Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review Data and Metadata review SIROP v1.5.1 metadata created with version 1.5.1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants