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

feat(api-study): add new endpoint to aggregate raw data #1981

Merged
merged 14 commits into from
Apr 19, 2024

Conversation

mabw-rte
Copy link
Contributor

@mabw-rte mabw-rte commented Mar 18, 2024

Context:
The R Scripts team has to do many API calls to perform aggregation of raw study output data w.r.t to mc years, areas ...

Issue:
The existing endpoints makes it possible to get only one file at a time. We need many queries to aggregate through data.

Solution:
Create a new endpoint to handle this task.

@mabw-rte mabw-rte force-pushed the feature/1149-add-output-data-endpoint branch from d42188a to 0a33d11 Compare March 19, 2024 20:01
@mabw-rte mabw-rte self-assigned this Mar 19, 2024
@mabw-rte mabw-rte force-pushed the feature/1149-add-output-data-endpoint branch from 0a33d11 to 847874b Compare April 5, 2024 18:11
@pull-request-size pull-request-size bot added size/XL and removed size/L labels Apr 5, 2024
@mabw-rte mabw-rte force-pushed the feature/1149-add-output-data-endpoint branch 2 times, most recently from 93fe2c1 to 1dc5075 Compare April 10, 2024 21:53
@mabw-rte mabw-rte force-pushed the feature/1149-add-output-data-endpoint branch 2 times, most recently from ebd1a24 to 1dfeeb2 Compare April 15, 2024 11:51
Copy link
Contributor

@laurent-laporte-pro laurent-laporte-pro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Peut-on éviter la duplication de code dans aggregate_areas_data et aggregate_links_data ?

antarest/study/common/default_values.py Show resolved Hide resolved
antarest/study/common/studystorage.py Outdated Show resolved Hide resolved
antarest/study/common/studystorage.py Outdated Show resolved Hide resolved
antarest/study/common/studystorage.py Outdated Show resolved Hide resolved
antarest/study/service.py Outdated Show resolved Hide resolved
antarest/study/storage/abstract_storage_service.py Outdated Show resolved Hide resolved
antarest/study/storage/abstract_storage_service.py Outdated Show resolved Hide resolved
antarest/study/storage/abstract_storage_service.py Outdated Show resolved Hide resolved
antarest/study/web/raw_studies_blueprint.py Show resolved Hide resolved
@mabw-rte mabw-rte force-pushed the feature/1149-add-output-data-endpoint branch 2 times, most recently from a441057 to e34ed15 Compare April 16, 2024 15:03
@mabw-rte mabw-rte force-pushed the feature/1149-add-output-data-endpoint branch from e34ed15 to a724248 Compare April 18, 2024 21:53
@mabw-rte mabw-rte merged commit 9ebe5c3 into dev Apr 19, 2024
6 of 7 checks passed
@mabw-rte mabw-rte deleted the feature/1149-add-output-data-endpoint branch April 19, 2024 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants