-
Notifications
You must be signed in to change notification settings - Fork 95
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
listmode files can start with non-zero time-tag #70
Comments
example script: first_tag=`list_lm_events --time 1 --coincidence 0 --num-events-to-list 1 $list_filename`
echo "0 $first_tag" > $fdef_filename
# now have the rest of the fdef info, e.g.
1 $frame_duration |
@KrisThielemans How is the skip frame defined then? will STIR know to ignore a frame that contains zero events in the listmode, or will it actually attempt to recon/process a PET image with all zeroes in the sinos? |
an fdef with a line |
Ah, my question was more: Does it skip it because it skips whatever is in the first line? Skips whatever starts with 0? or checks if there are events, and skips if there are no events? What is the logic? |
it advances the start of the next time frame without storing data because of the first STIR/src/include/stir/TimeFrameDefinitions.h Lines 65 to 67 in aae4c92
the fdef doesn't check events itself at all. |
In any case, a more complete example script would be:
Just removes newlines and grabs the last word in the output of |
@KrisThielemans your proposed solution does not seem to work for GE data. the first prompt in my example dataset is at Time 7467, but when I do Not sure if this is a problem with how |
Timing is currently in terms of the value of the time tag. For Siemens files, the first time tag is usually zero, but in ~1% of the cases not. GE files never start with zero. It would therefore be better to have timing relative to the value of the first time tag. Current work-around: use list_lm_events to find first time tag and create initial “skip-frame” in the frame-defs file.
The text was updated successfully, but these errors were encountered: