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

rel/v0.7.0 #216

Merged
merged 122 commits into from
Sep 7, 2023
Merged

rel/v0.7.0 #216

merged 122 commits into from
Sep 7, 2023

Conversation

alexdaniel654
Copy link
Member

@alexdaniel654 alexdaniel654 commented Sep 7, 2023

Release Checklist

Pull Request

  • Branch is from dev to rel/v[x.y.z]
  • Update the CHANGELOG.md file with the new version number, release date and add the new features/fixes along with any tags of PRs/issues
  • Bump the version number in setup.py
  • Add any new contributors and bump the version number in CITATION.cff
  • Do a final check if anything in readme.MD needs updating
  • The pull requests is from rel/v[x.y.z] to master
  • All tests pass
  • Merge rel/v[x.y.z] into master
  • Merge rel/v[x.y.z] into dev
  • Create a tag on the merge commit on master with the same version number as the release and push to upstream

Post Pull Request Checks

  • Close any issues from the milestone that didn't automatically close on merge
  • Close the milestone itself
  • Check the new version has appeared in the releases list
  • Check the new version has appeared on PyPI
  • Check a now DOI has been minted on Zenodo

This is a slight change from previous method of filtering out values within 1 ms of the main map upper limit, in this case T2.
Confusing having R2 (1/T2) and R2 (R squared measure of model fit). Given 1/T2 is rarely used, this option has been removed.
Means lots of the code can be shared between relaxometry methods. 
Also added documentation
Ended up being too specific to each method
Already had decent coverage from mapping tests, but this means its easier to isolate issues further down the line
Was causing tests to fail at the very end (upload of coverage)!
…misation

Curve Fit Optimisation and R-Squared
Only runs for GE at the moment and results aren't verified!
Done at 30,000 ft so probably terrible.
Could it be that this is using too much memory for the Actions runner?
Includes skimage label function but none of the region props stuff
Also upgrade to codecov v3 in the hop that actually uploads more often
Hopefully it'll be added again in the future.
T2 StimFit and Shape Features Compatibility
Kept core packages that will effect results with more tight requirements though.
@alexdaniel654 alexdaniel654 self-assigned this Sep 7, 2023
@alexdaniel654 alexdaniel654 added this to the v0.7.0 milestone Sep 7, 2023
@codecov
Copy link

codecov bot commented Sep 7, 2023

Codecov Report

Merging #216 (2414dab) into master (a7126d0) will increase coverage by 0.28%.
The diff coverage is 98.50%.

@@            Coverage Diff             @@
##           master     #216      +/-   ##
==========================================
+ Coverage   97.62%   97.91%   +0.28%     
==========================================
  Files          41       48       +7     
  Lines        3581     4311     +730     
==========================================
+ Hits         3496     4221     +725     
- Misses         85       90       +5     
Files Changed Coverage Δ
ukat/mapping/b0.py 100.00% <ø> (ø)
ukat/mapping/tests/test_b0.py 98.41% <ø> (ø)
ukat/qa/tests/test_snr.py 97.85% <ø> (ø)
ukat/utils/arraystats.py 100.00% <ø> (ø)
ukat/utils/tests/test_ge.py 100.00% <ø> (ø)
ukat/utils/tests/test_tools.py 100.00% <ø> (ø)
ukat/mapping/fitting/relaxation.py 93.44% <93.44%> (ø)
ukat/mapping/diffusion.py 97.12% <94.11%> (-0.81%) ⬇️
ukat/mapping/fitting/tests/test_relaxation.py 98.11% <98.11%> (ø)
ukat/mapping/t2_stimfit.py 98.13% <98.13%> (ø)
... and 17 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@alexdaniel654 alexdaniel654 merged commit 38c00d1 into master Sep 7, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant