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

specify velocity parameter to calculate scale of the time delay histo… #511

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

martynpclark
Copy link
Collaborator

Initial capabilities to use a velocity parameter to define the scale of the Gamma distribution. Hongli will complete testing.

@martynpclark
Copy link
Collaborator Author

@h294liu -- returning to an old pull request. Can you provide an update on the progress of these developments?

@h294liu
Copy link

h294liu commented Jul 5, 2023 via email

@martynpclark
Copy link
Collaborator Author

Yes, all is perfect! Thanks so much Hongli!!!

Copy link

@h294liu h294liu left a comment

Choose a reason for hiding this comment

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

I have tested and found that SUMMA run through these new codes without any problems. Yeah!


Test Setting Configurations

  • Set subRouting as constVel in the "modelDecisions.txt" file.
  • Add the following to "basinParamInfo.txt" file: routingVelocity | 1.00 | 0.1000 | 100.0000
    Note that the three numbers are the default, min, and max values of routingVelocity.

Test Results
The test domain is the Bow at Banff basin. The model includes 51 GRUs and runs for the year of 2008 (365 days). The model output is daily averageRoutedRunoff_mean.

Below is the model output comparison for one GRU (10 days).

  • The black line is for the timeDlay method (old method).
  • The other two lines are for the constVel method (newly added method).
  • The red line uses flow velocity=100m/s, and the blue line uses flow velocity=0.1m/s.

Screenshot 2023-08-03 at 10 31 51 PM

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