Skip to content

Unify reference sequences #18

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Unify reference sequences #18

wants to merge 1 commit into from

Conversation

ckolbPTB
Copy link
Contributor

Just a few small things to make the reference sequences more similar and fix doc string issues. Leave open for a bit to collect more.

@ckolbPTB
Copy link
Contributor Author

ckolbPTB commented Feb 20, 2025 •
edited
Loading

@schuenke Here we allow the spoiler to be set:

mrseq/src/mrseq/scripts/t2_multi_echo_se_single_line.py

Line 30 in caec410

gz_spoil_duration: float,

Shall we also enable this for T1? -> Done

@github-actions GitHub Actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
/home/runner/.local/lib/python3.12/site-packages/mrseq
   _version.py770%2–9
/home/runner/.local/lib/python3.12/site-packages/mrseq/preparations
   t2_prep.py51198%195
/home/runner/.local/lib/python3.12/site-packages/mrseq/scripts
   t1_inv_rec_gre_single_line.py92990%161, 244, 287–292, 319, 325
   t1_inv_rec_se_single_line.py103991%212, 299, 352–357, 384, 390
   t2_multi_echo_se_single_line.py95991%163, 252, 305–310, 335, 341
TOTAL4053591% 

Tests Skipped Failures Errors Time
76 0 💤 0 ❌ 0 🔥 18.141s ⏱️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ckolbPTB