-
Notifications
You must be signed in to change notification settings - Fork 19
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
processing TROPOMI L2 NO2 data and plot #225
base: develop
Are you sure you want to change the base?
Conversation
# Copyright (C) 2022 National Center for Atmospheric Research and National Oceanic and Atmospheric Administration | ||
# SPDX-License-Identifier: Apache-2.0 | ||
|
||
# calculate WRF-Chem NO2 trop. columns, for further pair with satellite swath data |
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 this only for WRF comparisons right now and you plan to generalize to more models later? Or should we call this python script cal_wrf_no2col.py?
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 is for WRF right now. I'm not sure if this can be applied to other models. It should work but with several modifications such as variable names, etc. I think it does make sense to change it to cal_wrf_no2col.py.
Hi Meng, |
@dwfncar : David I'm not sure what issues might be remaining in Meng's pull request. She harmonized the driver. Could you possibly take a look? Thanks! |
P.S. @mlirenzhenmayi : Meng it looks like these might be the issue: But thinking about these further... should this code be inside the: satellite_utilities.py file? |
@mlirenzhenmayi - could you pull another two updates two your branch? There's one for driver.py and one for satplots.py. |
Looks like AERONET web server is currently being updated, causing a CI failure. |
Hmmm, am wondering why we need the AERONET data for the TROPOMI satellite code? |
It's a test of the Note that the linter check is still failing since |
Thanks Zach! I see maybe you added those license lines now to those files? Or is there something else? |
Processing TROPOMI L2 NO2 data and plotting