Skip to content

Conversation

@cadenmyers13
Copy link
Contributor

closes#291

@cadenmyers13cadenmyers13 marked this pull request as draft December 21, 2025 19:34
@codecov
Copy link

codecovbot commented Dec 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (9c04eb9) to head (8d78856).

Additional details and impacted files
@@ Coverage Diff @@## main #371 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 9 9 Lines 516 520 +4 ========================================= + Hits 516 520 +4 
Files with missing linesCoverage Δ
tests/test_tools.py100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cadenmyers13
Copy link
ContributorAuthor

@sbillinge marking this as a draft for now. I need data for the z-scan measurement. I reached on to Till the other day and havent heard anything. Once we have some data for this we should be good to go. Do you have any data for this?

This is frequently the case for high-energy X-rays. However,
this must be corrected for when using low energy X-rays, such
as those from a laboratory source. To correct for X-ray absorption,
the X-ray absorption coefficient, μ, must be known.
Copy link
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please double check this

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that is called the "linear absorption coefficient" but apart from that it looks good.

@github-actions
Copy link

Warning! No news item is found for this PR. If this is a user-facing
change/feature/fix,
please add a news item by copying the format from news/TEMPLATE.rst.
For best practices, please visit
https://scikit-package.github.io/scikit-package/snippets/news-file-format.html#how-do-i-create-a-news-file.

@cadenmyers13
Copy link
ContributorAuthor

closing to redo on clean branch. reopening a new PR here: #375

Sign up for freeto 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.

docs: add docs for mu / muD calculator

2 participants

@cadenmyers13@sbillinge