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

Discard occurrences with unverified identificationVerificationStatus #24

Closed
damianooldoni opened this issue Jun 4, 2020 · 5 comments
Closed
Assignees
Labels
enhancement New feature or request

Comments

@damianooldoni
Copy link
Contributor

damianooldoni commented Jun 4, 2020

Based on issue trias-project/indicators#84:

  1. Add column identificationVerificationStatus to the cols_to_use (https://github.com/trias-project/occ-cube-alien/blob/master/src/belgium/2_create_db.Rmd#L183-L193)
  2. Discard occurrences with identificationVerificationStatus = "unverified". This is very similar to what we already do for occurrenceStatus (https://github.com/trias-project/occ-cube-alien/blob/master/src/belgium/2_create_db.Rmd#L234-L241)

Apply these changes in both pipelines (BE and modelling taxa at EU level).

@damianooldoni
Copy link
Contributor Author

As mentioned by @amyjsdavis in trias-project/indicators#84 (comment) we have more values of field identificationVerificationStatus to filter out. While making the European occurrence cube I found observations with the following values:

  • unverified,
  • unvalidated,
  • not able to validate
  • control could not be conclusive due to insufficient knowledge

After briefing with @amyjsdavis, we decided to remove them.

@damianooldoni
Copy link
Contributor Author

While making occurrence cube for risk assessment species as defined in references/modelling_species.tsv, I found some other suspect values which I think we should include in the list of values to discard:

  • Unconfirmed
  • Unconfirmed - not reviewed
  • Unconfirmed - plausible
  • Validation requested

@qgroom, @amyjsdavis: what do you think about it?

@qgroom
Copy link
Contributor

qgroom commented Nov 29, 2020

I would certainly include "Unconfirmed - plausible"

I would perhaps exclude the others. There are better ways to make this decision, but they are not so straightforward.

@damianooldoni
Copy link
Contributor Author

Thanks @qgroom. Yes, Unconfirmed - plausible can be left. I agree.

Up to now it seems this field is free: it would be much better with a specific dictionary. For example, unverified and unconfirmed, I think could be surely merged.

@damianooldoni
Copy link
Contributor Author

Done. I will close this issue again until new values to discard will be found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants