Add feature to enable overlaying the atoms to the generated AFM images. - #364
Conversation
|
I'm not sure I like adding more plotting arguments to afm = afmulator(xyzs, Zs, ...)
afmulator.plot_images(afm, ..., atoms=atoms)We could also do this in a more generic way by passing all possible arguments to the underlying plotting function def plot_images(self, X, ..., **plot_kwargs):
...
plotImages(
...
**plot_kwargs
) |
711b9d1 to
aba8597
Compare
|
Indeed, passing all possible arguments to the plotting function |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #364 +/- ##
==========================================
- Coverage 55.61% 55.58% -0.04%
==========================================
Files 40 44 +4
Lines 6252 6268 +16
==========================================
+ Hits 3477 3484 +7
- Misses 2775 2784 +9
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:
|
04a6f12 to
dec9f08
Compare
NikoOinonen
left a comment
There was a problem hiding this comment.
Thanks, looks good now.
Fix #363
atomsto the methodevalin AFMulator.prefixcan be used when overlaying the atoms.