Skip to content

Comments

Source detection analysis#103

Draft
FeGeyer wants to merge 16 commits intomainfrom
source_detection
Draft

Source detection analysis#103
FeGeyer wants to merge 16 commits intomainfrom
source_detection

Conversation

@FeGeyer
Copy link
Member

@FeGeyer FeGeyer commented Nov 29, 2021

Add functionality from Paul's Master thesis

@FeGeyer FeGeyer requested a review from Kevin2 November 29, 2021 15:24
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@codecov-commenter
Copy link

codecov-commenter commented Nov 29, 2021

Codecov Report

Merging #103 (6826c54) into main (44331cd) will decrease coverage by 3.91%.
The diff coverage is 10.83%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #103      +/-   ##
==========================================
- Coverage   36.71%   32.79%   -3.92%     
==========================================
  Files          35       36       +1     
  Lines        4001     4098      +97     
==========================================
- Hits         1469     1344     -125     
- Misses       2532     2754     +222     
Impacted Files Coverage Δ
radionets/simulations/gaussians.py 50.58% <8.82%> (-14.78%) ⬇️
...ets/dl_framework/architectures/source_detection.py 10.63% <10.63%> (ø)
radionets/dl_framework/loss_functions.py 21.05% <25.00%> (+0.05%) ⬆️
radionets/dl_framework/architecture.py 100.00% <100.00%> (ø)
radionets/evaluation/scripts/start_evaluation.py 30.61% <0.00%> (-34.70%) ⬇️
radionets/evaluation/plotting.py 10.57% <0.00%> (-17.64%) ⬇️
radionets/evaluation/utils.py 58.95% <0.00%> (-12.14%) ⬇️
radionets/evaluation/train_inspection.py 15.23% <0.00%> (-10.16%) ⬇️
radionets/evaluation/jet_angle.py 90.74% <0.00%> (-3.71%) ⬇️

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 44331cd...6826c54. Read the comment docs.

@FeGeyer FeGeyer marked this pull request as draft July 18, 2022 10:31
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