-
Notifications
You must be signed in to change notification settings - Fork 6
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
Make species info conditionally required on observation_type=Animal #25
Comments
In GitLab by @kbubnicki on Aug 28, 2020, 15:45 And what about the proposed |
In GitLab by @peterdesmet on Aug 28, 2020, 15:51 From my experience with Darwin Core, only Note: not all observations (empty, etc.) will have a species observation, so I don't know how we can make any of these 3 terms required, while the it is not required that they have content. The same is true for e.g. |
In GitLab by @yliefting on Aug 28, 2020, 15:56 I still think mentioning and using a documented species list is essential for the standard. Species names change all the time and it's a complicated task to figure out how names changed if you have a 10 year old package without any information on the species list used. It becomes even worse when you allow subspecies, which is relevant to keep track of for many endangered species. |
In GitLab by @kbubnicki on Aug 28, 2020, 17:08 Thats right! None of them can be actually required. Maybe we could use some derived rule e.g. if But I do not know if this is possible with json schema specification. |
In GitLab by @kbubnicki on Aug 28, 2020, 17:11 I agree but then we would probably also need to define a list of supported taxonomy providers? |
In GitLab by @peterdesmet on Aug 29, 2020, 15:20 I feel like we are going quite far here.
I therefore think there it is already quite a requirement (which I support) to have |
In GitLab by @peterdesmet on Sep 2, 2020, 15:23 Addressed in #32: it is possible. |
In GitLab by @peterdesmet on Sep 2, 2020, 15:24 Consensus is:
|
In GitLab by @peterdesmet on Sep 8, 2020, 12:04 This issue meandered a bit, and in taxonomic coverage only |
In GitLab by @peterdesmet on Sep 8, 2020, 12:04 changed title from Make species{-_common/vernacular_name option-}al to Make species{+ info conditionally required on observation_type=Anim+}al |
In GitLab by @kbubnicki on Sep 18, 2020, 12:56 This does not seem to be as easy as I thought. I am not sure if the frictionless table schema specification supports cases like this one. Probably we would have to add a new custom constraint: Can you also look at this @peterdesmet? |
In GitLab by @peterdesmet on Sep 25, 2020, 14:18 Indeed, it is not supported in Table Schema. It was suggested as an issue in 2015, but marked as WONTFIX. See #32 for a potential solution (extended the schema), but for now I would propose not to implement this and keep scientificName not required. @kbubnicki is that ok for you? |
In GitLab by @kbubnicki on Sep 26, 2020, 12:51 I agree. We can (re)consider adding this when there will be support for such custom constraints in Table Schema. |
In GitLab by @peterdesmet on Aug 21, 2020, 2:31
taxonomic_coverage has 3 properties:
species_latin
,species_common
andcount
(see also #14 to update these names). The first 2 are required, but I think it cannot always be guaranteed that aspecies_common
/vernacular_name
is available. I would make that term optional.Issue scope is now this: https://gitlab.com/oscf/camtrap-package-schemas/-/issues/25#note_408998361
The text was updated successfully, but these errors were encountered: