-
Notifications
You must be signed in to change notification settings - Fork 9
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
add ccd=0 #12
base: main
Are you sure you want to change the base?
add ccd=0 #12
Conversation
I changed the tab problem. |
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.
This looks good to me, Simone
If this changes results then the baselines stored in the stand-alone carma repo will need updated. |
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.
I am not sure this change gives you the right granularity for turning it on and off. If you had a mixed cloud and aerosol model, wouldn't you want to include this in the cloud-aerosol interactions, but not in the aerosol-aerosol or cloud-cloud interactions. This flag always has them either on or off for all groups (particle types). Perhaps it should be a flag on AddCoagulation() so that it can be kept per coagulation interaction. Perhaps something similar to use_vw(group1, group2). For example, if (use_ccd(j1,j2))then ...
should I define it as cstate%f_ variable or carma%f_ variable? |
It can be part of CARMA, since in is known at configuration time and is independent of the model state. Sent from my iPhoneOn Sep 4, 2024, at 4:08 PM, zywshoon ***@***.***> wrote:
I am not sure this change gives you the right granularity for turning it on and off. If you had a mixed cloud and aerosol model, wouldn't you want to include this in the cloud-aerosol interactions, but not in the aerosol-aerosol or cloud-cloud interactions. This flag always has them either on or off for all groups (particle types). Perhaps it should be a flag on AddCoagulation() so that it can be kept per coagulation interaction. Perhaps something similar to use_vw(group1, group2). For example, if (use_ccd(j1,j2))then ...
should I define it as cstate%f_ variable or carma%f_ variable?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because your review was requested.Message ID: ***@***.***>
|
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.
Is there a paper we can reference here?
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.
I haven't found any
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.
It doesn't seem correct for use_ccd to be dimensioned(NGROUP, NGROUP) and be part of the element data. Element data should be dimensions NELEMENT by something. If this is really decided at the group level, then it should be part of the group section above. If it is part of the element, then it should have NELEMENT dimensions. I assume you should just move it up to the group section.
add ccd =0 will cause the coagulation slightly different than before.
these are the tests didn't pass regression test because they all have coagulation processes on. Let me know how you want me to process.
** Starting FRACTALMICROTEST.exe at Wed Aug 14 14:51:45 MDT 2024 **
Fractal Test
Done
** Finished at Wed Aug 14 14:51:55 MDT 2024 **
** Starting SULFATETEST.exe at Wed Aug 14 14:52:02 MDT 2024 **
Sulfate Test
** Finished at Wed Aug 14 14:52:02 MDT 2024 **
** Starting SULFATE_VEHKAMAKI_TEST.exe at Wed Aug 14 14:52:02 MDT 2024 **
Sulfate Test
** Finished at Wed Aug 14 14:52:02 MDT 2024 **
** Starting SULFHETTEST.exe at Wed Aug 14 14:52:02 MDT 2024 **
Sulfate Heterogeneous Test
** Finished at Wed Aug 14 14:52:02 MDT 2024 **
** Starting SULFHET_VEHKAMAKI_TEST.exe at Wed Aug 14 14:52:02 MDT 2024 **
Sulfate Heterogeneous Test
** Finished at Wed Aug 14 14:52:02 MDT 2024 **