Skip to content
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.

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

WIP: Live in-situ experiment with DENS Impulse #33

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

uellue
Copy link
Member

@uellue uellue commented Oct 14, 2021

To be tested with real hardware, example with a real dataset!

Contributor Checklist:

  • I have added or updated my entry in the creators.json file
  • I have added a changelog entry for my contribution
  • I have added/updated documentation for all user-facing changes
  • I have added/updated test cases

Reviewer Checklist:

  • /azp run libertem.libertem-live-data passed

To be tested with real hardware, example with a real dataset!
@uellue uellue added the documentation Improvements or additions to documentation label Oct 14, 2021
@codecov
Copy link

codecov bot commented Oct 14, 2021

Codecov Report

Merging #33 (d1229c0) into master (e0f3745) will decrease coverage by 48.99%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #33       +/-   ##
===========================================
- Coverage   74.37%   25.38%   -49.00%     
===========================================
  Files          15       15               
  Lines        1245     1245               
  Branches      146      146               
===========================================
- Hits          926      316      -610     
- Misses        282      927      +645     
+ Partials       37        2       -35     
Impacted Files Coverage Δ
src/libertem_live/detectors/merlin/control.py 23.94% <0.00%> (-69.02%) ⬇️
src/libertem_live/detectors/merlin/acquisition.py 38.61% <0.00%> (-55.45%) ⬇️
src/libertem_live/detectors/merlin/data.py 17.67% <0.00%> (-54.75%) ⬇️
src/libertem_live/detectors/merlin/sim.py 20.00% <0.00%> (-52.61%) ⬇️
src/libertem_live/detectors/common.py 52.00% <0.00%> (-24.00%) ⬇️
src/libertem_live/api.py 90.32% <0.00%> (-9.68%) ⬇️

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 e0f3745...d1229c0. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant