-
Notifications
You must be signed in to change notification settings - Fork 4
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
Dyamond2 Example #118
base: main
Are you sure you want to change the base?
Dyamond2 Example #118
Conversation
@RobertPincus I added the dyamond example to a notebook and loaded using dask, I am just not sure about the data transformations that need to be done for I created a function to download the data and created this new PR as it rebases the latest changes. |
@sehnem I only have time for a quick peek but it seems the units for cloud optics should be g/m^2 for LWP/IWP and microns (10^-6 m) for particle sizes. I suspect that means you need to compute LWP = (DEPLP * QL)/sci.g * 1000 and pass RL*10e6 to the Python calls. |
@sehnem I've just pushed a commit to address units as per the comment above. You can see if this runs and give reasonable results? |
Seems like this supersedes #114 |
6aaeb8c
to
efb57bd
Compare
20ed5cf
to
02a56c0
Compare
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
…e' into new-dyamond2-example
for more information, see https://pre-commit.ci
@RobertPincus I changed all the calls to stack the dimensions that are not core, so you can run dyamond2 without renaming the dimensions I also made some tests running it with the full I think it should be good for merging. |
No description provided.