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

PSF and attenuation #33

Merged
merged 11 commits into from
Dec 16, 2024
Merged

Conversation

CHrlS98
Copy link
Contributor

@CHrlS98 CHrlS98 commented Dec 16, 2024

New PR compensate psf and attenuation (previously PR #21)

@pep8speaks
Copy link

pep8speaks commented Dec 16, 2024

Hello @CHrlS98! Thanks for updating this PR.

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-12-16 21:13:49 UTC

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lots of different models here, not all are used. Don't know what we want to do with that.

.gitignore Outdated
@@ -171,6 +171,10 @@ workflows/work/
# Macos
.DS_Store

# files
# Image files
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Data files

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done in 912bbb4

scripts/linum_compute_attenuation.py Show resolved Hide resolved
scripts/linum_compute_attenuationBiasField.py Outdated Show resolved Hide resolved
@@ -593,6 +594,95 @@ def getAttenuation_Vermeer2013(vol, dz=6.5e-6, mask=None, C=None):
return mu


def get_extendedAttenuation_Vermeer2013(vol, mask=None, k=10, sigma=5,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New issue #35 to solve units mismatch issue.

@CHrlS98 CHrlS98 added this to the Charles@UQAM milestone Dec 16, 2024
@joe-from-mtl joe-from-mtl merged commit d440587 into linum-uqam:main Dec 16, 2024
@CHrlS98 CHrlS98 deleted the psf-and-attenuation branch December 16, 2024 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants