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.

Sign up for GitHub

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

Jump to bottom

Indicate in query response body the level of results detail enabled #171

Closed
alyssadai opened this issue Sep 14, 2023 · 0 comments · Fixed by #193
Closed

Indicate in query response body the level of results detail enabled #171

alyssadai opened this issue Sep 14, 2023 · 0 comments · Fixed by #193
Assignees
@alyssadai
Labels
feat:improve Incremental, user facing improvements of an existing feature. type:feature Effort to deliver new features, feature changes & improvements

Comments

@alyssadai
Copy link
Contributor

alyssadai commented Sep 14, 2023 •
edited
Loading

Having an easily accessible string/dict key stating whether the query results were aggregated or not (for a given graph db) would make it much easier for tools using the API to decide how to handle the results.

We can make this a new header key called "records_protected" with values being true or false

See also: neurobagel/query-tool#218
@alyssadai alyssadai added feat:improve Incremental, user facing improvements of an existing feature. type:feature Effort to deliver new features, feature changes & improvements labels Sep 14, 2023
@surchs surchs mentioned this issue Sep 28, 2023
Closed
2 tasks
@alyssadai alyssadai self-assigned this Sep 28, 2023
@alyssadai alyssadai mentioned this issue Sep 28, 2023
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alyssadai alyssadai

Labels
feat:improve Incremental, user facing improvements of an existing feature. type:feature Effort to deliver new features, feature changes & improvements
Projects
Neurobagel
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[ENH] Update aggregate response body neurobagel/api
1 participant
@alyssadai