Feature: evaluate_labels function in Sequence
#164
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Changed `get_block` to always set the `label` field in the returned block (None if no labels)
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This implements the
evalLabelsfunction from upstream pulseq: https://github.com/pulseq/pulseq/blob/125c772180f324967091dc28a0e5a6b20b96faa9/matlab/%2Bmr/%40Sequence/Sequence.m#L1056Renamed to
evaluate_labelsto be consistent with the naming of other Sequence functions.This function tracks labels (or label evolutions) through the sequence. Either returns the final values, or label evolutions (e.g. per block, per ADC, or whenever the label changes).
A small supporting change in
event_lib.get_blockwas made to always set thelabelfield in the returned block (None if no labels).