Skip to content

Kpca updates#3

Open
melisirfan wants to merge 6 commits intomainfrom
kpca_updates
Open

Kpca updates#3
melisirfan wants to merge 6 commits intomainfrom
kpca_updates

Conversation

@melisirfan
Copy link
Collaborator

Updates after the KPCA paper review: changes have been made in FastBox.box and FastBox.foregrounds as well as the old foreground removal script being deleted and replaced with a new Jupyter notebook.

@codecov-commenter
Copy link

codecov-commenter commented Sep 22, 2021

Codecov Report

Merging #3 (5491941) into main (a260184) will decrease coverage by 0.64%.
The diff coverage is 3.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
- Coverage   33.20%   32.55%   -0.65%     
==========================================
  Files          14       14              
  Lines        1253     1281      +28     
==========================================
+ Hits          416      417       +1     
- Misses        837      864      +27     
Flag Coverage Δ
unittests 32.55% <3.50%> (-0.65%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
fastbox/foregrounds.py 17.96% <3.33%> (-0.09%) ⬇️
fastbox/box.py 76.32% <3.70%> (-6.67%) ⬇️

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 a260184...5491941. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants