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.

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

Update tropomi_l2 reader with "_reduced" file patterns #2660

Merged
merged 2 commits into from
Dec 4, 2023

Conversation

Kexin828
Copy link
Contributor

@Kexin828 Kexin828 commented Dec 3, 2023

The Tropomi l2 reader can read ’S5P_OFFL_L2__NO2____‘ files but cannot read 'S5P_RPRO_L2__NO2____' files. Because 'S5P_RPRO_L2__NO2____' files have reduced in their names, which means the pattern defined here doesn't match. Adding a new pattern to the pattern list with the reduced added to the end can work.

  • Closes #xxxx
  • Tests added
  • Fully documented
  • Add your name to AUTHORS.md if not there already

The Tropomi l2 reader can read ’S5P_OFFL_L2__NO2____‘ files but cannot read 'S5P_RPRO_L2__NO2____' files. Because 'S5P_RPRO_L2__NO2____' files  have reduced in their names, which means the pattern defined here doesn't match. 
Adding a new pattern to the pattern list with the reduced added to the end can work.
@djhoese djhoese self-assigned this Dec 3, 2023
@djhoese djhoese added enhancement code enhancements, features, improvements component:readers labels Dec 3, 2023
@djhoese
Copy link
Member

djhoese commented Dec 3, 2023

Closes #2659

Copy link

codecov bot commented Dec 4, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (25786b6) 95.29% compared to head (2c83c70) 95.29%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2660   +/-   ##
=======================================
  Coverage   95.29%   95.29%           
=======================================
  Files         369      369           
  Lines       52049    52049           
=======================================
  Hits        49600    49600           
  Misses       2449     2449           
Flag Coverage Δ
behaviourtests 4.19% <ø> (ø)
unittests 95.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 7089468775

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.863%

Totals Coverage Status
Change from base Build 7057184699: 0.0%
Covered Lines: 49726
Relevant Lines: 51872

💛 - Coveralls

@djhoese djhoese changed the title Update tropomi_l2.yaml Update tropomi_l2 reader with "_reduced" file patterns Dec 4, 2023
@djhoese djhoese merged commit cc53903 into pytroll:main Dec 4, 2023
17 of 19 checks passed
@djhoese
Copy link
Member

djhoese commented Dec 4, 2023

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:readers enhancement code enhancements, features, improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants