-
Notifications
You must be signed in to change notification settings - Fork 86
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
Rename graphs.py to visualizations.py #3439
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3439 +/- ##
=======================================
+ Coverage 99.7% 99.7% +0.1%
=======================================
Files 334 334
Lines 32951 32959 +8
=======================================
+ Hits 32821 32829 +8
Misses 130 130
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.
LGTM! Thanks a lot for completing this!
Final step of the code side of the
graphs.py
overhaul (docs updates coming next).graphs.py
has been renamed, and now only contains functions that are visualization-specific model understanding functions.Happy to move more functions around if parts of this organization don't make sense.