Convert pressure variables in RAQMS reader

zmoon commented 1 year ago

zmoon commented 1 year ago

@mebruckner @rschwant does this look ok? I based the renames on the ones of @rschwant .

mebruckner commented 1 year ago

@zmoon It looks good to me. I can test it and update the raqms melodies-monet commit to reflect this change.

zmoon commented 1 year ago

@mebruckner that would be great, thanks

rschwant commented 1 year ago

Yes, @zmoon this looks good to me.

zmoon commented 1 year ago

@mebruckner I'm going to merge this, if you find an issue we can address in another PR.

zmoon commented 1 year ago

CI failures upon merge (though passing before) due to:

pytest 7.2 release breaking pytest-xdist
- this has been fixed (https://github.com/pytest-dev/pytest-xdist/issues/246) but conda-forge pytest-xdist is still an old version
EPA AQS URLs erroring in link check apparently due to their server not supporting OpenSSL 3
- e.g.
  (tutorial/aqs_pams: line 27) broken https://aqs.epa.gov/aqsweb/airdata/download_files.html - HTTPSConnectionPool(host='aqs.epa.gov', port=443): Max retries exceeded with url: /aqsweb/airdata/download_files.html (Caused by SSLError(SSLError(1, '[SSL: UNSAFE_LEGACY_RENEGOTIATION_DISABLED] unsafe legacy renegotiation disabled (_ssl.c:1129)')))
- seems this is affecting the AQS test as well (232e0e4): FAILED tests/test_aqs.py::test_aqs - requests.exceptions.SSLError: HTTPSConnectionPool(host='aqs.epa.gov', port=443): Max retries exceeded with url: /aqsweb/airdata/daily_PM10SPEC_2019.zip (Caused by SSLError(SSLError(1, '[SSL: UNSAFE_LEGACY_RENEGOTIATION_DISABLED] unsafe legacy renegotiation disabled (_ssl.c:1131)')))

noaa-oar-arl / monetio