-
-
Notifications
You must be signed in to change notification settings - Fork 397
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
Add option to silence logger for MAST enable_cloud_dataset() #1536
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1536 +/- ##
==========================================
- Coverage 62.42% 62.42% -0.01%
==========================================
Files 176 176
Lines 14634 14635 +1
==========================================
Hits 9136 9136
- Misses 5498 5499 +1
Continue to review full report at Codecov.
|
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.
I think this is a good idea. 👍
@@ -1561,6 +1561,8 @@ def enable_cloud_dataset(self, provider="AWS", profile=None): | |||
though at the moment this argument is ignored. | |||
profile : str | |||
Profile to use to identify yourself to the cloud provider (usually in ~/.aws/config). | |||
verbose : bool | |||
Logger to display extra info and warning. |
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.
Add "Default True."
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.
Done.
@pllim - I'm on the verge whether this needs a changelog or not, leave it up to you, and will merge once CI is happy. |
I cancelled the CI because I plan to add a change log, squash, and then let it run. |
Address review comment by @ceb8 and add change log.
Would be nice if I can silence this without affecting the logger behavior as a whole. I don't need it to remind me about S3 pricing every time I run the function.
🙏 @ceb8 🙏