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

Add code to invoke annotation service #418

Closed
MarkDWilliams opened this issue Jun 7, 2023 · 3 comments
Closed

Add code to invoke annotation service #418

MarkDWilliams opened this issue Jun 7, 2023 · 3 comments
Assignees

Comments

@MarkDWilliams
Copy link
Collaborator

The ARS will need to call out to the annotation service with each new response received. This call should happen in the same place that the node normalizer is called and should be wrapped with error handling. In the event of an error, the status of the Message should be set to 206, a descriptive error should be output to the ARS log, and a new log object should be added to the Message.

This ticket depends on the implementation of the annotation service.

@sstemann
Copy link
Contributor

@MarkDWilliams to talk to Chunlei then we'll figure out when to do this. plans to use a batch version.

@sstemann
Copy link
Contributor

Chunlei sent over example queries, confirmed it supports bath queries. It's up but not in SmartAPI.

Will follow up to determine when service will be available in SmartAPI

@MarkDWilliams
Copy link
Collaborator Author

This should address this
#430

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

No branches or pull requests

2 participants