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

Plotting resolution matrix in msnoise-tomo #28

Open
Jacob-Alexander-Gochenour opened this issue Nov 29, 2022 · 2 comments
Open

Plotting resolution matrix in msnoise-tomo #28

Jacob-Alexander-Gochenour opened this issue Nov 29, 2022 · 2 comments

Comments

@Jacob-Alexander-Gochenour

I am posting about retrieving plots of the model resolution in msnoise-tomo. It appears this is done within the ANSWT.py script (Lines 464-532). However, these lines cannot be reached because they are under a if False: statement (Line 464) that does not appear to point to a variable being True or False. It also has a module/def that's not imported in (measure.find_contours, line 483) and a variable (pasgrille, line 523) that is not defined. Has this been corrected in any recent versions? or do you have any insight on how this should be corrected?

Also, it appears the lambda parameter input for the matrix H in the second iteration is lambda1, shouldn't it be lambda2 (line 306). This also leads me to ask, is there any built-in way in the MsNoise-tomo package to produce L-curves for the range of smoothing/damping parameter values?

@ThomasLecocq
Copy link
Owner

Hi!
I don't use msnoise tomo so much, I'll check the code & ask Aurélien or Raphael if they know what this is about. will update next week !

@Jacob-Alexander-Gochenour
Copy link
Author

Hi, Thomas. Are there any updates on this issue yet. Thanks.

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

No branches or pull requests

2 participants