-
Notifications
You must be signed in to change notification settings - Fork 129
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
return svg text in MolVisualizer #388
Conversation
No-compat: The returned value will change to raw When using on jupyter notebook, user need to write below explicitly.
|
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.
The change looks good other than the one on which I commented about the default value of the argument!
@@ -46,9 +46,24 @@ def __init__(self, logger=None): | |||
|
|||
def visualize(self, saliency, mol, save_filepath=None, | |||
visualize_ratio=1.0, color_fn=red_blue_cmap, | |||
scaler=abs_max_scaler, legend=''): | |||
scaler=abs_max_scaler, legend='', raise_import_error=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.
I think raise_import_error
should be False
by default if we follow the original behavior of saving png files.
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.
okay i will fix it to set False as default.
Codecov Report
@@ Coverage Diff @@
## master #388 +/- ##
==========================================
- Coverage 91.42% 86.64% -4.78%
==========================================
Files 222 222
Lines 10786 10786
==========================================
- Hits 9861 9346 -515
- Misses 925 1440 +515 |
No description provided.