-
Notifications
You must be signed in to change notification settings - Fork 31
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
Develop satellite #182
Develop satellite #182
Conversation
Develop satellite
…IES-MONET into NOAA-CSL-develop_satellite
added logic needed for pairing aeronet and fv3raqms.
First pull of satellite read in code.
Develop satellite - merging in everything.
…ME in fnames. Should this be done across the driver module?
Co-authored-by: colin-harkins <[email protected]>
Fixed |
@dwfncar I went through and resolved the outstanding issues. The merge conflict I think should be resolved as follows:
I think once that is fixed this is good to go. |
Fixing final conflict.
Fixed final conflict - should be good to merge. If we remove "if obs_type == 'pt_sfc':" we could run into issues with xarray versus pandas dataframe. |
final final conflict resolved
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.
All of the recent changes look good to me.
Initial PR for develop_satellite to develop, to start review comments.