search

scikit-tda / kepler-mapper

Kepler Mapper: A flexible Python implementation of the Mapper algorithm.
https://kepler-mapper.scikit-tda.org
MIT License
629 stars 182 forks source link

fix gallery #220

Closed deargle closed 3 years ago

deargle commented 3 years ago

@sauln can you add me (deargle) to https://readthedocs.org/projects/kepler-mapper/ ? I have the gallery fixed locally, want to make sure it works live before merging.

codecov[bot] commented 3 years ago

Codecov Report

Merging #220 (8651e4e) into master (ac0c50e) will decrease coverage by 0.05%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
- Coverage   78.61%   78.56%   -0.06%     
==========================================
  Files          11       11              
  Lines         781      779       -2     
  Branches      161      161              
==========================================
- Hits          614      612       -2     
  Misses        138      138              
  Partials       29       29
Impacted Files Coverage Δ
kmapper/utils.py 90.00% <0.00%> (-0.48%) :arrow_down:
kmapper/plotlyviz.py 43.87% <0.00%> (-0.36%) :arrow_down:

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 ac0c50e...8651e4e. Read the comment docs.

sauln commented 3 years ago

@deargle you should have access now.

deargle commented 3 years ago

Hurray! See https://kepler-mapper.scikit-tda.org/en/fix-gallery/examples.html , gallery is fixed

deargle commented 3 years ago

@sauln I made the thumbnails on the gallery page (1) d3 vis instead of the default matplotlib capture, and (2) bigger, but still small enough for there to be two per row. In favor?

deargle commented 3 years ago

Closes #218