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

Final review #13

Closed
4 tasks done
damianooldoni opened this issue Feb 20, 2023 · 7 comments
Closed
4 tasks done

Final review #13

damianooldoni opened this issue Feb 20, 2023 · 7 comments
Assignees

Comments

@damianooldoni
Copy link
Contributor

damianooldoni commented Feb 20, 2023

To be reviewed:

  • the dates in field entry_into_force in raw data (tsv file) some are in the future, e.g. for Celastrus orbiculatus is this field set to 2027-08-02, while for Fundulus heteroclitus and_Xenopus laevis_ is 2024-08-02. I see also that in 2022 we have two very near dates: 2022-07-12 and 2022-08-02. Maybe is correct, but still worth a final check
  • the link to the most actual consolidated pdf version (I modified the link to point the 2022 list, please @timadriaens double check this) in Workflow section of README.
  • The webpage with English common names is outdated. I couldn't find any other similar webpage. The only link with common names is the PDF brochure to download on circabc but I don't find it good enough.
  • metadata. I went through the document and seems good, but still, @timadriaens, we need a sentence mentioning the issues encountered while harvesting data from EASIN API, see Communicate to JRC and solve missing data in JRC catalouge #9 (+ Add distribution #10), I think, to justify our approach.
@timadriaens
Copy link
Contributor

  • future dates are correct, this is because of prolonged transitionary measures foreseen in the Regulation. Still, I would already list those species as Member States are already working on them.
  • Link to the consolidated list in the README is fine
  • Common names document: the IAS webpage of IUCN was refurbished recently (and the resources page for the EU Reg you find here) and doesn't seem to have a link to the file anymore. I filed a request with IUCN colleagues.
  • I have mentioned EASIN in the additional information sentence. We will communicate with them after we published the checklist to see if we can indeed in the future harvest it from their api instead (but for now I would refrain from mentioning any data gaps there).

@damianooldoni
Copy link
Contributor Author

@timadriaens: I will add the link to those three files in README. Could you please check that these names are the same as the ones in our tsv? Or did you already done it? Thanks.

@timadriaens
Copy link
Contributor

should we not include the common names in other languages? If you want I can do that.

@damianooldoni
Copy link
Contributor Author

yes, we can. If you do, I am happy to check and map it further to DwC 👍 I also noticed that the names in the spreadsheets are not always correct as we think they should be, e.g. capital letters in French and English common names should be removed unless referring to a country, sin't? I also see that we are not always applying this rule: see e.g. "Pumpkinseed" which probably shoul be "pumpkinseed"? I let this issue to you as it is not my domain of expertise 😄

@damianooldoni
Copy link
Contributor Author

We can discuss this further if you need in #14. We leave this issue for final review.

@damianooldoni
Copy link
Contributor Author

Data and metadata have been reviewed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants