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.

Sign up for GitHub

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

Jump to bottom

Compute the mass and number weighted terminal velocity #222

Closed
Tracked by #215
trontrytel opened this issue Sep 7, 2023 · 1 comment · Fixed by #370
Closed
Tracked by #215

Compute the mass and number weighted terminal velocity #222

trontrytel opened this issue Sep 7, 2023 · 1 comment · Fixed by #370
Assignees
@trontrytel @anastasia-popova
Labels
P3
Milestone
O1.3.3 Property b...

Comments

@trontrytel
Copy link
Member

trontrytel commented Sep 7, 2023 •
edited
Loading

We probably want to use the formulation from Chen et al 2022, because it will be the easiest to integrate over the different P3 scheme regimes.

We should fix the kappa sign issue first. See #221
@trontrytel trontrytel mentioned this issue Sep 7, 2023
Open
@trontrytel trontrytel self-assigned this Sep 7, 2023
@trontrytel trontrytel added the P3 label Sep 7, 2023
@trontrytel trontrytel linked a pull request Sep 7, 2023 that will close this issue
P3 fall speed regime docs #191
Closed
1 task
@trontrytel trontrytel changed the title implement terminal_velocity(D) Compute the mass and number weighted terminal_velocity(D) Feb 2, 2024
@trontrytel trontrytel changed the title Compute the mass and number weighted terminal_velocity(D) Compute the mass and number weighted terminal velocity Feb 2, 2024
@trontrytel
Copy link
Member Author

The plan:

  • Add B4 and B5 tables from Chen2022 #332
  • aspect ratio + how to combine the P3 assumptions with kappa in Chen2022.describe in the docs what we do
  • (if you have the time then also look at integrals with Chen2022 for 1M scheme assuming kappa=0)

@trontrytel trontrytel linked a pull request Apr 18, 2024 that will close this issue
Add terminal velocities to P3 scheme #370
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@trontrytel trontrytel

@anastasia-popova anastasia-popova

Labels
P3
Projects
None yet
Milestone
O1.3.3 Property based snow/ice scheme
2 participants
@trontrytel @anastasia-popova