datamol-io / datamol

Molecular Processing Made Easy.
https://docs.datamol.io
Apache License 2.0
452 stars 47 forks source link

Fix bugs and support atom indices as inputs in the lasso viz function #198

Closed hadim closed 1 year ago

hadim commented 1 year ago

Changelogs


Checklist:


discussion related to that PR

codecov[bot] commented 1 year ago

Codecov Report

Merging #198 (c6c656b) into main (55c190c) will increase coverage by 0.00%. The diff coverage is 91.66%.

@@           Coverage Diff           @@
##             main     #198   +/-   ##
=======================================
  Coverage   91.47%   91.48%           
=======================================
  Files          46       46           
  Lines        3672     3664    -8     
=======================================
- Hits         3359     3352    -7     
+ Misses        313      312    -1     
Flag Coverage Δ
unittests 91.48% <91.66%> (+<0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
datamol/viz/_lasso_highlight.py 90.95% <91.66%> (+0.12%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more