-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore: logging 1 - remove stdout suppression in the rest of the app scripts #175
Conversation
@hollandjg Does this one need reviewing? On the surface looks the same as #176. |
This PR replaces the stdout-suppression in the analysis_db files, but doesn't make any changes to the logging. When reviewing this one, I recommend hiding whitespace changes: |
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.
👍
…ession # Conflicts: # src/icesat2waves/analysis_db/B02_make_spectra_gFT.py # src/icesat2waves/analysis_db/B04_angle.py
…ession # Conflicts: # src/icesat2waves/analysis_db/A02c_IOWAGA_thredds_prior.py # src/icesat2waves/analysis_db/B02_make_spectra_gFT.py # src/icesat2waves/analysis_db/B03_plot_spectra_ov.py # src/icesat2waves/analysis_db/B04_angle.py # src/icesat2waves/analysis_db/B05_define_angle.py # src/icesat2waves/analysis_db/B06_correct_separate_var.py
This PR replaces the stdout-suppression in the
analysis_db
files, but doesn't make any changes to the logging.(These two changes are separated into separate PRs because the change in this PR is so noisy – it changes almost every line, but only really removes one statement – the context manager.)
builds on