-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fix ping service #21
Comments
I failed to apply the same obsolete=True filter on the ping module that I was using to create the list of agents. I have now applied that an both RO and the deprecated amigo2 services are no longer being pinged, and no longer in the list.. |
Great. The unresolvable purls in bioportal are the only real sticky wickets then. |
Here is the updated list; still not getting a response from HP and PR.
|
But your right, http://pir.georgetown.edu/cgi-bin/pro/entry_pro?id=PR:000000024 is resolving when i paste in my browser. Not sure why it keeps coming back 404 when I do it from the python script |
Super, thanks, none of these are failing due to poorly composed URLS, so that's the real thing to avoid. Not sure what to do about bioportal. Perhaps actually remove them from resulting merged set? |
Another case where I should have studied the xml better. Both have a state=down attribute. I can filter those out as well |
Well, those can be transient. I wouldn't necessarily filter them out. We can preserve the state=down flag in our merged record if we want, but we're essentially doing our own test, which is more up-to-date by definition. |
Ah, now I understand what the reliability scores mean. Well, would it be better to capture that from id.org, or generate our own? |
I think it is more important to do our own because it tests the uptime while also exposing situations in which we have composed the URL poorly, whether due to omitted prefix, duplicated prefix, casing issues, etc. |
The text was updated successfully, but these errors were encountered: