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

Added cloud droplet size distribution dataset derived from specMACS polarization measurements #154

Merged
merged 2 commits into from
Nov 23, 2023
Merged

Added cloud droplet size distribution dataset derived from specMACS polarization measurements #154

merged 2 commits into from
Nov 23, 2023

Conversation

vpoertge
Copy link
Contributor

No description provided.

observingClouds
observingClouds reviewed Nov 15, 2023
View reviewed changes
Copy link
Collaborator

@observingClouds observingClouds left a comment

Choose a reason for hiding this comment

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

Hi @vpoertge looks like you finished a vast amount of work! Well done! Glad to see your contributions being added here.
I just had a quick look and have some minor questions.

HALO/specmacs/specMACS_polLL_cdsd.yaml Show resolved Hide resolved
HALO/specmacs/specMACS_polLL_cdsd.yaml Show resolved Hide resolved
@vpoertge
Copy link
Contributor Author

Hi @observingClouds,
I have processed the missing segments and added the links to the yaml-files. In case you are wondering: the transfer flights have not been processed yet and I decided to postpone this for a bit. Also, I only processed a few segments of HALO-0215 as HALO changed altitude quite often and I would have to adjust some things in the processing for that day.
Cheers,
Vero

observingClouds
observingClouds approved these changes Nov 23, 2023
View reviewed changes
Copy link
Collaborator

@observingClouds observingClouds left a comment

Choose a reason for hiding this comment

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

Hi @vpoertge ,
Thanks for adding the other files as well. It looks all good now. I will rebase your branch with the main to bring it up to date and merge it.

Would be great if you could add a short tutorial on how to use the data also in the howto.eurec4a.eu book 😀 I think that helps people a lot.

Great work @vpoertge !

@observingClouds observingClouds merged commit 6f87243 into eurec4a:master Nov 23, 2023
2 checks passed
@vpoertge
Copy link
Contributor Author

Hi @observingClouds ,

yes, that's on my to-do list. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@observingClouds observingClouds observingClouds approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vpoertge @observingClouds