-
Notifications
You must be signed in to change notification settings - Fork 7
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
Handle dryad URLs #50
Comments
For some related issues on the structure of Dryad identifiers in DataONE, see https://redmine.dataone.org/issues/7896 |
@gothub sorry for the confusion. The idea is that scientists could also go on each data repository and get the URL from there. The KNB The DRYAD URL comes from this package https://datadryad.org/resource/doi:10.5061/dryad.7ns4pk2 for the dataset Side note: when I search on dataONE for this DOI (10.5061/dryad.7ns4pk2) I get 5 hits...more likely related to the problem Matt mentioned, but if I search for the "DRYAD" dataset identifier (dryad.181477) I get 0 hit. So we might have to understand the URL logic behind DRYAD if we want to support it. |
Here is the corresponding DataONE URL for the above Dryad id: https://cn.dataone.org/cn/v2/resolve/https://doi.org/10.5061/dryad.7ns4pk2/1/bitstream |
@gothub following our discussion I think it would make sense to add a rule to prioritize the DataONE URLs and then default to the current system if it fails to make the fct more efficient. This being said that does not solve the mapping problem between DRAYD URLs and corresponding DataONE ones. |
Currently, because of the way the data URL for dryad is constructed, it doesn't work with our function.
check_version
ends up looking for nonsensical results because it keeps chunking the URL and eventually looking for anything that matches1
. I've changed the breaking point to nchar(pid) > 5 (instead of 0) to account for this to some extent. 4163fb9Not sure what the logic of dryad URL's is, so more investigation is needed!
The text was updated successfully, but these errors were encountered: