Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feature/models/cngrxj #112

Merged
merged 486 commits into from
Dec 20, 2024

Conversation

sivogel
Copy link
Member

@sivogel sivogel commented Apr 21, 2023

Here is my first draft of the Cema-Neige and the grxj-models based on the work of Bastian Klein. There are some open topics I would like to discuss, but maybe you can have a first look at it:

Open topics:

  • Doctests of grxjland application models: Do we need all doctests? -> If so, we should clarify the purpose of each doctest
  • gr5j in the air package airGR splits the output of the unit hydrograph, but in some publications gr5j (like gr4j and gr6j) the input to the unit hydrograph is split and two seperate unit hydrographs are applied. Are we sure that we want to stay with the airGR version?
    Lavenne:
    image
    airGR:
    image
  • Some minor open todos
  • Rename grxj_land. I propose to rename the model group grxjland to gr_land (gr for genie rurale).
  • Delete unnecessary comments
  • die cn_minmax Modelle machen eigentlich nur in Tagesschrittweite Sinn. Rein faktisch kann man sie aber in allen Schrittweiten anwenden

@sivogel sivogel requested a review from tyralla April 21, 2023 12:26
@sivogel
Copy link
Member Author

sivogel commented Apr 21, 2023

I cancelled the Travis jobs, because the last build of the master branch on which it is rebased was not successful.

@tyralla tyralla mentioned this pull request Dec 4, 2023
30 tasks
tyralla and others added 27 commits February 2, 2024 12:46
… by the site-package `strenum` (fixes commit f69c9d2 for Python 3.9 and 3.10).
… `netcdftools` reads all available data even if the time series of some sequences are missing.
…ssages created by the decorator `print_progress` of module `printtools`.
…nttools` to method `read` and `write` of class `NetCDFInterface` of module `netcdftools`.
…the relevant element as soon as possible (class `SubmodelAdder` of module `importtools`).

This change should only be relevant when creating a model "by hand" for testing or other specific purposes.
…`evap_morsim` to the main model `lland_v3` to define (the same) input time series values for the equally named wind speed sequences.
…re clearly by splitting class `NetCDFInterface` into `NetCDFInterfaceReader`, `NetCDFInterfaceWriter`, and `NetCDFInterfaceJIT` and class `NetCDFVariableFlat` into `NetCDFVariableFlatReader` and `NetCDFVariableFlatWriter` (module `netcdftools`).

This change is not only a refactoring but also allows the reading of time series data from input sequence objects (with the same standard name) that belong to different (sub)models of the same element (relevant when working with the new HydPy standard naming convention for input time series files).
…add the property `seriesmode` to the class `IOSequence` of module `sequencetools`.
…submodel" (class `SubmodelAdder` of module `importtools`).

Otherwise, the flag would stay true when replacing a "main submodel" with a "real submodel".
…s (with the same standard name) that belong to different (sub)models of the same element (module `netcdftools`, completes commit f21c00b).
…er level to the required water depth (fixes commit 19dd0e1).
…and its application models to allow for an optional stabilisation of the flow between surface water and groundwater.
…odel `dam` from 0.01 to 0.0001 to increase the minimal numerical accuracy all dam application models achieve by default.
… not risk performance losses when working with Cython.
…l evaporation via option submodels that follow the `PrecipModel_V2` and `PETModel_V1` interface.
…Inz`, and `SInz` of base model `lland` (see issue hydpy-dev#79).
…ater` and `SoilMoistureLimit` instead of `FieldCapacity` and `WiltingPoint` and thereby change the way soil surface resistance is calculated for extremely shallow soils.

This commit is more of a refactoring and conceptual fix.  The resulting changes in the calculation results should be of minor relevance.
…t of wind speed and the leaf area index (if the information is available/required) when connecting main models and submodels.
…ng floats, ints, and bools to module `objecttools`.
…ition lines of parameters which a main model has prepared are not skipped if the related aspects of the main model or the submodel have been changed in the meantime.
…` and `evap_mlc` so that the adjustment of (grass) reference evapotranspiration to the more delayed evaporation from large water bodies becomes usable for different model combinations.
…sts by increasing the aerodynamic resistance (and remove the wind speed adjustment for forests of Oliver, 1979, for consistency).
tyralla and others added 29 commits September 25, 2024 10:17
…e `projectname` property of the `pub` module.
…les` of class `NetworkManager` of module `filetools` a little.
…e the "complete" selection based on its new property `complete`.
…dule `hydpytools` and class `Elements` of `devicetools`.
… data variable names (especially with respect to node sequences) in the User Guides Project Structure section.
New GH token
…nment` context manager after updating all dependencies.

Fixes hydpy-dev#149
…dding submodels (class `SubmodelAdder` of module `importtools`).

This change is necessary due to the new `FrameLocalsProxy` approach of Python 3.13.  However, it is also a general improvement because it should help to minimise conflicts between already available and newly introduced variables.
# Conflicts:
#	hydpy/docs/rst/modelcollection.rst
@tyralla tyralla merged commit 93d9816 into hydpy-dev:feature/models/cngrxj Dec 20, 2024
0 of 2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants