-
Notifications
You must be signed in to change notification settings - Fork 21
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
fao_class description has link to a dead site #696
Comments
@turbomam could you query NCBI and see how/ if this slot is filled out much? |
looking at the FAO website and following links on there, it seems like they now use/recommend ISRIC world soil information list found here: https://www.isric.org/explore/world-soil-distribution |
select
"value",
count(1)
from
all_attribs aa
where
attribute_name = 'fao_class'
group by
"value"
order by
count(1) desc ; There were only 6242 NCBI Biosamples with a
|
A good chunk of what is in NCBI currently is some variation of the FAO classification.. variations in plurality and capitalization, but it's one of the closer ones I've seen! That said, I agree with @only1chunts. We should create an FAO enumeration and require it be one of the provided list. Thanks for the link Chris, I wasn't able to find that site. I'll work on the enum and making the updates to the description now. |
Updated, see #726 Note, the enum already existed so I just updated the available values |
Describe the bug
A clear and concise description of what the bug is.
the link provided to complete fao_class is broken (Soil classification from the FAO World Reference Database for Soil Resources. The list can be found at http://www.fao.org/nr/land/sols/soil/wrb-soil-maps/reference-groups)
Expected behavior
A clear and concise description of what you expected to happen.
The link should work, or we should provide a new link.
Additional context
Add any other context about the problem here.
Some things to consider... how often is this slot used? Does an updated link exist? Is this the same thing-> https://www.britannica.com/science/soil/FAO-soil-groups (i don't know what the previous site link looked like)
The text was updated successfully, but these errors were encountered: