-
Notifications
You must be signed in to change notification settings - Fork 6
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(outputs): build outputs tree based on filesystem #2064
feat(outputs): build outputs tree based on filesystem #2064
Conversation
mc-all
and mc-ind
trees based on filesystem
antarest/study/storage/rawstudy/model/filesystem/common/area_matrix_list.py
Outdated
Show resolved
Hide resolved
antarest/study/storage/rawstudy/model/filesystem/common/area_matrix_list.py
Outdated
Show resolved
Hide resolved
antarest/study/storage/rawstudy/model/filesystem/common/area_matrix_list.py
Outdated
Show resolved
Hide resolved
antarest/study/storage/rawstudy/model/filesystem/common/area_matrix_list.py
Show resolved
Hide resolved
antarest/study/storage/rawstudy/model/filesystem/common/area_matrix_list.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quelques petits ajustements à faire, notamment pour documenter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci pour ce refactoring.
Une observation : à présent que la vérification de l’existence des fichiers et dossiers de sortie est effectuée, j'espère que cela ne ralentira pas le parsing d'une étude. Il est possible qu'on observe une dégradation des performances à l'ouverture d'une étude. Cela dit, si ça évite des anomalies côté utilisateur, c'est toujours bon à prendre.
antarest/study/storage/rawstudy/model/filesystem/root/output/simulation/mode/common/areas.py
Show resolved
Hide resolved
antarest/study/storage/rawstudy/model/filesystem/root/output/simulation/mode/common/utils.py
Outdated
Show resolved
Hide resolved
antarest/study/storage/rawstudy/model/filesystem/root/output/simulation/mode/common/utils.py
Outdated
Show resolved
Hide resolved
antarest/study/storage/rawstudy/model/filesystem/root/output/simulation/mode/common/set.py
Outdated
Show resolved
Hide resolved
antarest/study/storage/rawstudy/model/filesystem/root/output/simulation/mode/common/set.py
Show resolved
Hide resolved
@@ -56,17 +56,18 @@ def build(self) -> TREE: | |||
|
|||
class OutputSimulationTsGenerator(FolderNode): | |||
def build(self) -> TREE: | |||
children: TREE = { | |||
"hydro": OutputSimulationTsGeneratorCustomMatrixList( | |||
children: TREE = {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cette partie du code n'est pas couverte pas les tests unitaires.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe it's because the ts-generator part is not supported yet in AntaresWeb.
.../study/storage/rawstudy/model/filesystem/root/output/simulation/mode/common/binding_const.py
Show resolved
Hide resolved
.../study/storage/rawstudy/model/filesystem/root/output/simulation/mode/common/binding_const.py
Show resolved
Hide resolved
antarest/study/storage/rawstudy/model/filesystem/root/output/simulation/mode/mcind/mcind.py
Outdated
Show resolved
Hide resolved
...rest/study/storage/rawstudy/model/filesystem/root/output/simulation/ts_numbers/ts_numbers.py
Show resolved
Hide resolved
f9dce4f
to
31b69d7
Compare
c71ff8d
to
d7c0918
Compare
8db4c20
to
58cdeeb
Compare
cb37c6e
to
f11baf7
Compare
Solves [ANT-1306]