Skip to content

Fix/benchmarks#183

Draft
kosack wants to merge 3 commits into
cta-observatory:masterfrom
kosack:fix/benchmarks
Draft

Fix/benchmarks#183
kosack wants to merge 3 commits into
cta-observatory:masterfrom
kosack:fix/benchmarks

Conversation

@kosack
Copy link
Copy Markdown
Member

@kosack kosack commented Mar 29, 2022

Fixing/improving a few benchmark notebooks

  • MODEL/classification: added plots showing energy and impact distributions for gammas/protons

kosack added 2 commits March 24, 2022 17:44
- PrecisionRecallDisplay crashed due to bins with no data
- re-ran black and isort to reformat all cells
@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 29, 2022

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.05%. Comparing base (1f392ea) to head (019314f).
⚠️ Report is 79 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #183   +/-   ##
=======================================
  Coverage   63.05%   63.05%           
=======================================
  Files          28       28           
  Lines        2885     2885           
=======================================
  Hits         1819     1819           
  Misses       1066     1066           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@HealthyPear HealthyPear added fix A PR that fixes a bug or a wrong behaviour. benchmarks labels Apr 23, 2022
@HealthyPear HealthyPear added this to the v0.5.1 milestone Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmarks fix A PR that fixes a bug or a wrong behaviour.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants