Closed sizmailov closed 2 years ago
Merging #161 into master will increase coverage by
1.74%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #161 +/- ##
===========================================
+ Coverage 98.25% 100.00% +1.74%
===========================================
Files 27 1 -26
Lines 6635 353 -6282
Branches 44 44
===========================================
- Hits 6519 353 -6166
+ Misses 116 0 -116
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 3e1051f...156ab3d. Read the comment docs.
Finally this PR starts looking like something (check out spoiler in first comment)! :fireworks: :tada:
Two (minor) remaining problems pointed out in first comment still exist and second one is worth discussion.
Similarly as with #145 I have reconsidered my position on this PR -- since this is a rather niche feature and the functionality is quite standalone, I think it could live on its own outside of the m.css repository, maintained by a third party instead. M.css itself will only contain the m.plots
plugin with functionality reduced to a subset where I can have full control over how the output looks like.
I hope that's understandable.
Generated images stored in global output directory under
matplotlib-figures
sub-directory.TODO:
tune html/css to make it look prettyname output images similarly to source filesDepends on #145
test page render
![image](https://user-images.githubusercontent.com/2975991/85633090-3259be00-b681-11ea-814e-dbd2fd4baa2e.png)