Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

GE_DMI3_Torso dataset #111

Merged
merged 1 commit into from
Sep 25, 2024
Merged

GE_DMI3_Torso dataset #111

merged 1 commit into from
Sep 25, 2024

Conversation

KrisThielemans
Copy link
Member

Files for GE_DMI3_Torso. I also made an attempt to adjust the examples to cope with non-TOF multfactors for TOF data, but there are remaining problems in the BSREM set-up, so instead, I've chosen to create the data with TOF multfactors as well.

@KrisThielemans
Copy link
Member Author

Output of plot_BSREM_metrics.py

BSREM objective values

GE_DMI3_Torso_BSREM_objectives_last
GE_DMI3_Torso_BSREM_objectives

metrics

GE_DMI3_Torso_metrics_BSREM
GE_DMI3_Torso_metrics_BSREM_OSEM

pass at 1200 updates

images

at pass index
GE_DMI3_Torso_image_at_0 01_0 005
diff with OSEM
GE_DMI3_Torso_OSEM_diff_image_at_0 01_0 005
diff with reference
GE_DMI3_Torso_ref_diff_image_at_0 01_0 005

@KrisThielemans
Copy link
Member Author

@casperdcl could be interesting to add this to our test runs in the leaderboard

@KrisThielemans
Copy link
Member Author

@KrisThielemans
Copy link
Member Author

@casperdcl I've removed the commits trying to cope with nonTOF multfactors for TOF data (and moved it to #114). This should now be to your liking 😄 .

@KrisThielemans KrisThielemans merged commit ed633e6 into main Sep 25, 2024
2 checks passed
@KrisThielemans KrisThielemans deleted the GE_DMI3_Torso branch September 25, 2024 21:41
@KrisThielemans KrisThielemans restored the GE_DMI3_Torso branch September 25, 2024 21:41
@KrisThielemans KrisThielemans deleted the GE_DMI3_Torso branch September 25, 2024 21:42
casperdcl added a commit that referenced this pull request Sep 25, 2024
@casperdcl
Copy link
Member

casperdcl commented Sep 25, 2024

could be interesting to add this to our test runs in the leaderboard

added to #91...

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants