-
Notifications
You must be signed in to change notification settings - Fork 1
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
Refactor KiD driver, add precipitation analysis #108
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some buildkite jobs fail. Seems like there are some minor issues. Otherwise it looks good to me. Please squash you commits when it's ready to get merged.
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #108 +/- ##
=======================================
Coverage 84.00% 84.00%
=======================================
Files 19 19
Lines 1613 1613
=======================================
Hits 1355 1355
Misses 258 258
☔ View full report in Codecov by Sentry. |
e12e684
to
1e81745
Compare
…eptibility analysis
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Just update the version in Project.toml (increment the patch number) and squash your commits.
262076c
to
cbd27bc
Compare
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors try |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Build failed: |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
tryTimed out. |
Timed out. |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors try |
tryTimed out. |
bors r+ |
108: Refactor KiD driver, add precipitation analysis r=sajjadazimi a=mikhailmints Co-authored-by: Mikhail Mints <[email protected]>
Timed out. |
bors r+ |
Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
Purpose
To analyze precipitation susceptibility in the KiD model and compare it to expected results.
To-do
Content
KiD_driver.jl
inside a function so it can be more easily called from somewhere other than the command line.CloudMicrophysics.jl
ρq_rai
over time at the height of the cloud base.mm/command_line_args_types
, changingarg_type
of arguments toFloat64
instead ofReal
and adding the option to selectFloat64
orFloat32
as the float type.