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

Prepare for v4.0.0 release #573

Open
wants to merge 1,628 commits into
base: master
Choose a base branch
from
Open

Prepare for v4.0.0 release #573

wants to merge 1,628 commits into from

Conversation

wknoben
Copy link
Collaborator

@wknoben wknoben commented Jul 27, 2024

Updates master with the SUNDIALS and related developments. High level changes:

  • Addition option to use SUNDIALS library to solve model equation (optional, default compilation is without SUNDIALS).
  • BMI implementation.
  • Energy state calculation now based on enthalpy.
  • CMake build files for SUMMA.
  • CMake build files for Actors model for parallelization.
  • Various bugfixes.
  • Indentation fixes.

ashleymedin and others added 30 commits April 18, 2024 17:35
Update cmakelists to include c netcdf library for actors.
* ent_state: (77 commits)
  clean up aquiferIni decision
  change lyr_wght
  change to snow balance is realMissing if never a snow state in the data window, not just at end
  remove scaledBalance variable, decision name clarify-- now closedForm, enthalpyFormLU, or enthalpyForm
  banded enthalpy Jacobian needed different index
  account for missing vegetation in jacobian
  a bit more Brent speed up
  calculate snow merge detection outside of brent
  better bounds for brent function
  need to put snow merge condition on enthalpy as well as temperature
  need to change root finding to be on enthalpy
  ...

# Conflicts:
#	build/source/driver/summa_setup.f90
#	build/source/engine/coupled_em.f90
#	build/source/engine/enthalpyTemp.f90
#	build/source/engine/eval8summaWithPrime.f90
#	build/source/engine/layerMerge.f90
#	build/source/engine/systemSolv.f90
#	build/source/engine/varSubstep.f90
#	utils/bal_per_GRU.py
…parameters to match V3.

I have added some new files to Summa-Actors and now leverage the restart and setup files from summa.

I have also edited the order of paremeters for vegPhenlgy to match what is now a pull request in V3
Edit cmake lists file to reflect updates to summa-actors, and adjust …
…lit, can lose them in the middle of a run when snow buries the canopy
ashleymedin and others added 30 commits November 4, 2024 11:00
Simplifying the object-oriented methods used for stateFilter
Refactoring updates for summaSolve4homegrown
Change minstep to be read from the parameters file
Continuation of sundials branch merge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants