fix: add lasso colors for atom_indices#232
fix: add lasso colors for atom_indices#232OleinikovasV wants to merge 2 commits intodatamol-io:mainfrom
Conversation
zhu0619
left a comment
There was a problem hiding this comment.
LGTM!
Thank you for fixing this! @OleinikovasV
|
@OleinikovasV Please lint and format the changes. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #232 +/- ##
==========================================
+ Coverage 90.26% 92.22% +1.96%
==========================================
Files 46 46
Lines 3881 3883 +2
==========================================
+ Hits 3503 3581 +78
+ Misses 378 302 -76
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@zhu0619 , AFAIU, the lint and test errors are not related to the changes I made to this repo, but instead something to do with CI versioning? |
Run black to resolve pre-existing lint CI failures unrelated to PR datamol-io#232. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Added this #241 to fix the linting that is unrelated to the code here. |

Changelogs
Adds color to lasso when
atom_indicesare specified in addition or instead ofsmarts_list.Checklist:
feature,fixortest(or ask a maintainer to do it for you).discussion related to that PR
#233
label:
bug