Skip to content

Crhm glacier infiltration module order correction 4 oct2023#434

Merged
jhs507 merged 8 commits intomasterfrom
CRHM_glacier_infiltration_module_order_correction_4Oct2023
Oct 10, 2023
Merged

Crhm glacier infiltration module order correction 4 oct2023#434
jhs507 merged 8 commits intomasterfrom
CRHM_glacier_infiltration_module_order_correction_4Oct2023

Conversation

@loganxingfang
Copy link
Collaborator

Correction to module order between infiltration modules Ayers, frozenAyers, frozen, crack, Greencrack, GreenAmpt, and PrairieInfiltration and glacier when constructing model project in GUI. The .h files of these modules Ayers, frozenAyers, frozen, crack, Greencrack, GreenAmpt, and PrairieInfiltration have been modified to fix the module order issue. More details are included in Issue#433.

Correction to module order between Ayers and glacier when constructing model project in GUI.
Correction to module order between frozenAyers and glacier when constructing model project in GUI. See Issue#433.
Correction to module order between frozen and glacier when constructing model project in GUI. See Issue#433.
Correction to module order between crack and glacier when constructing model project in GUI. See Issue#433.
Correction to module order between Greencrack and glacier when constructing model project in GUI. See Issue#433.
Correction to module order between GreenAmpt and glacier when constructing model project in GUI. See Issue#433.
Correction to module order between PrairieInfiltration and glacier when constructing model project in GUI. See Issue#433.
@loganxingfang loganxingfang requested a review from jhs507 October 4, 2023 20:07
@jhs507
Copy link
Collaborator

jhs507 commented Oct 4, 2023

I added a minor change to the way this change is commented to make it easier to read and associate the 1001 with peer rank.

If you are good with that I would say this is good to merge.

@jhs507 jhs507 merged commit 16e95bc into master Oct 10, 2023
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.

2 participants