search

DeltaRCM / pyDeltaRCM

Delta model with a reduced-complexity approach
https://deltarcm.org/pyDeltaRCM/
MIT License
18 stars 10 forks source link

add two more "after" hooks #243

Closed amoodie closed 2 years ago

amoodie commented 2 years ago

This PR adds two additional "after" style hooks. These are occurring after the domain setup hook_after_create_domain and after the last step of the timestep before saving the outputs hook_after_finalize_timestep.

codecov[bot] commented 2 years ago

Codecov Report

Merging #243 (d9b33cc) into develop (2171c99) will increase coverage by 0.04%. The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #243      +/-   ##
===========================================
+ Coverage    78.91%   78.96%   +0.04%     
===========================================
  Files           12       12              
  Lines         2590     2596       +6     
===========================================
+ Hits          2044     2050       +6     
  Misses         546      546
Impacted Files Coverage Δ
pyDeltaRCM/_version.py 100.00% <100.00%> (ø)
pyDeltaRCM/hook_tools.py 98.30% <100.00%> (+0.12%) :arrow_up:
pyDeltaRCM/model.py 89.45% <100.00%> (+0.03%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2171c99...d9b33cc. Read the comment docs.