tlambert03 / microvis

Other
2 stars 2 forks source link

refactor: Backend -> Protocol for protocols #25

Closed alisterburt closed 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

Merging #25 (e48a994) into main (f1c7bde) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   67.24%   67.24%           
=======================================
  Files          26       26           
  Lines         922      922           
=======================================
  Hits          620      620           
  Misses        302      302           
Impacted Files Coverage Δ
src/microvis/backend/vispy/_camera.py 51.61% <100.00%> (ø)
src/microvis/backend/vispy/_canvas.py 62.85% <100.00%> (ø)
src/microvis/backend/vispy/_node.py 60.00% <100.00%> (ø)
src/microvis/backend/vispy/_view.py 70.00% <100.00%> (ø)
src/microvis/core/canvas.py 100.00% <100.00%> (ø)
src/microvis/core/nodes/_data.py 86.00% <100.00%> (ø)
src/microvis/core/nodes/camera.py 100.00% <100.00%> (ø)
src/microvis/core/nodes/image.py 71.25% <100.00%> (ø)
src/microvis/core/nodes/node.py 70.32% <100.00%> (ø)
src/microvis/core/view.py 98.21% <100.00%> (ø)

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

tlambert03 commented 1 year ago

conflicts now, I think from #24

alisterburt commented 1 year ago

thanks - updated to AdaptorProtocol everywhere after discussion in #23, will merge here