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

ENH: Add grc structure based on MJA dev #313

Merged
merged 2 commits into from
Sep 10, 2024

Conversation

inoelloc
Copy link
Member

@inoelloc inoelloc commented Sep 9, 2024

  • Add a new hydrological module: grc
  • Update grd structure image (remove percolation)

Should replace #305

- Add a new hydrological module `grc`
- Update `grd` structure image (remove percolation)
@inoelloc inoelloc added the enhancement New feature or request label Sep 9, 2024
@inoelloc inoelloc added this to the Release v1.1.0 milestone Sep 9, 2024
@inoelloc
Copy link
Member Author

inoelloc commented Sep 9, 2024

@pag13, @nghi-truyen, je viens de voir que dans grc et gr6 le flux d'echange est aussi appliqué sur le second réservoir. Je corrigerai les graphs d'ici peu !

Copy link
Member

@nghi-truyen nghi-truyen left a comment

Choose a reason for hiding this comment

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

Bon pour moi !

doc/source/math_num_documentation/forward_structure.rst Outdated Show resolved Hide resolved
Copy link
Collaborator

@pag13 pag13 left a comment

Choose a reason for hiding this comment

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

Well done thanks.
I would remove sentence on Reunion island that is not appropriate in doc standard.
I would say delayed transfer for the branch with two reservoirs.
Thx
PAG

doc/source/_static/grc_structure.svg Outdated Show resolved Hide resolved
doc/source/math_num_documentation/forward_structure.rst Outdated Show resolved Hide resolved
@inoelloc
Copy link
Member Author

@pag13, @nghi-truyen Ca devrait etre ok comme ca. J'ai préféré garder first and second plutot que delayed branch si c'est ok pour vous.

@pag13
Copy link
Collaborator

pag13 commented Sep 10, 2024 via email

@inoelloc
Copy link
Member Author

@maximejay, je te ping si jamais tu veux verifier par rapport à ton PR

@inoelloc inoelloc merged commit d5c92c6 into DassHydro:main Sep 10, 2024
22 checks passed
@inoelloc inoelloc deleted the enh-grc-structure branch September 10, 2024 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants