-
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
Handle partial node query successes when all nodes are selected #60
Comments
Blocked by neurobagel/planning#96 |
Unblocked since neurobagel/planning#96 has been addressed |
This would be a good deal easier if we had up to date examples of the expected responses. I made an issue to provide these here neurobagel/neurobagel_examples#31 |
@neurobagel/dev what would a partially successful federated API query response look like? I.e. how would I know that any nodes have failed to respond / which nodes didn't respond? edit: answering my own question with the current test from f-API: But want to reiterate: we need to have a reference example for these responses. Either just the static files / shapes / response objects, or - better - that and also a live staging deployment we can play with! |
🚀 Issue was released in |
See https://github.com/neurobagel/federation-api/blob/350469a3a72d9ac6278d8371116c195b95a1b276/app/api/models.py#L57-L62 for what the new response from the federation API should look like.
Note that node API responses have not changed their shape at all.
UI behaviour should be:
optional -> auto-remove all the nodes that have failed?
The text was updated successfully, but these errors were encountered: