New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

don't make plots for particle extraction in estimate_roc #147

Merged

McHaillet merged 1 commit into SBC-Utrecht:main from sroet:change_estimate_roc_entry_point

Mar 26, 2024

Collaborator

sroet commented Mar 26, 2024

closes #143


          don't make plots for particel extraction in estimate_roc

11b9c66

sroet requested a review from McHaillet

March 26, 2024 14:22

github-actions bot commented Mar 26, 2024

File	Coverage		Missing
All files	`80%`	✅
src/pytom_tm/angles.py	`94%`	✅	`44`
src/pytom_tm/correlation.py	`85%`	✅	`94-95` `126`
src/pytom_tm/entry_points.py	`68%`	❌	`27` `104-121` `283-342` `431-446` `543-558` `792-859` `900-911`
src/pytom_tm/extract.py	`73%`	❌	`107-118` `170-176` `202-205` `277-295`
src/pytom_tm/io.py	`48%`	❌	`15-19` `25-28` `34-37` `43-46` `52-55` `62-66` `74-87` `93-100` `107-114` `129-131` `184` `218-219` `261-263` `279` `295` `312-321` `341-346`
src/pytom_tm/mask.py	`80%`	✅	`70` `85` `94-95`
src/pytom_tm/matching.py	`95%`	✅	`186-196` `340`
src/pytom_tm/parallel.py	`84%`	✅	`14-15` `81-87` `95-97` `104` `109` `148`
src/pytom_tm/plotting.py	`16%`	❌	`20-22` `25-32` `36-44` `48-51` `54-70` `73-74` `77-78` `98-103` `119-127` `133-150` `165-173` `177-194` `198-206` `210-217` `223` `229` `234` `248-398`
src/pytom_tm/template.py	`57%`	❌	`57-58` `67` `76-87` `92-97` `107-112` `114`
src/pytom_tm/tmjob.py	`96%`	✅	`170-171` `183` `189-191` `194` `327` `367`
src/pytom_tm/utils.py	`78%`	❌	`16-18`
src/pytom_tm/weights.py	`96%`	✅	`61` `292-293` `300` `479` `482` `485`

Minimum allowed coverage is 79%

Generated by 🐒 cobertura-action against 11b9c66

McHaillet approved these changes

View reviewed changes

Collaborator

McHaillet left a comment

Thanks for the fix!

McHaillet merged commit 84eba50 into SBC-Utrecht:main

2 checks passed

sroet deleted the change_estimate_roc_entry_point branch

March 26, 2024 16:02

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