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

remove smartapi infores #1245

Closed
cbizon opened this issue Mar 21, 2023 · 4 comments
Closed

remove smartapi infores #1245

cbizon opened this issue Mar 21, 2023 · 4 comments

Comments

@cbizon
Copy link
Collaborator

cbizon commented Mar 21, 2023

There are a number of infores entries that use the smartapi registration for their URL. This url is now being displayed in the translator UI as a place for users to find more information. I think most of these sources should have a better URL. For instance the SRI reference KG could maybe point at monarch or somewhere. For tools/aggregators, I think that the github page would even be more useful than the smart api registration. Note that the registration IDs are not stable!

@sierra-moxon
Copy link
Member

@cbizon - where should I point the automat infores's to?
e.g.: Automat Chemical normalization (trapi v-1.1.0)

@cbizon
Copy link
Collaborator Author

cbizon commented Mar 27, 2023

@EvanDietzMorris can you check the infores & see if there are some that should be removed? e.g. obsolete automats

@sierra-moxon sierra-moxon reopened this Mar 27, 2023
@EvanDietzMorris
Copy link
Collaborator

I made a PR for removing the obsolete/inactive automats. I wasn't sure what to put for the URLs for the other automats so those are still blank though.

@sierra-moxon
Copy link
Member

closing as complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants