randyzwitch / ECharts.jl

Julia package for the Apache ECharts v4 visualization library
https://randyzwitch.com/ECharts.jl/
Other
84 stars 9 forks source link

Refactor series #41

Closed randyzwitch closed 6 years ago

randyzwitch commented 6 years ago

Series are broken into their individual types, except for XY which makes sense to combine. Next step is to find the common pieces between series, to reduce code into more rollups

codecov[bot] commented 6 years ago

Codecov Report

Merging #41 into master will not change coverage. The diff coverage is 95%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   84.17%   84.17%           
=======================================
  Files          29       29           
  Lines         493      493           
=======================================
  Hits          415      415           
  Misses         78       78
Impacted Files Coverage Δ
src/theme.jl 100% <ø> (ø) :arrow_up:
src/definetypes.jl 100% <ø> (ø) :arrow_up:
src/ECharts.jl 38.88% <ø> (ø) :arrow_up:
src/plots/box.jl 87.09% <100%> (ø) :arrow_up:
src/plots/scatter.jl 100% <100%> (ø) :arrow_up:
src/plots/candlestick.jl 100% <100%> (ø) :arrow_up:
src/plots/xy_plot.jl 94.73% <100%> (ø) :arrow_up:
src/plots/circular_plot.jl 100% <100%> (ø) :arrow_up:
src/plots/heatmap.jl 100% <100%> (ø) :arrow_up:
src/plots/gauge.jl 100% <100%> (ø) :arrow_up:
... and 7 more

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 bf39d5c...34960f1. Read the comment docs.