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

Check and update Folder 2 for v24.3.0 #225

Closed
hrobarts opened this issue Jan 29, 2025 · 6 comments · Fixed by #234
Closed

Check and update Folder 2 for v24.3.0 #225

hrobarts opened this issue Jan 29, 2025 · 6 comments · Fixed by #234
Assignees

Comments

@hrobarts
Copy link
Contributor

No description provided.

@hrobarts hrobarts self-assigned this Jan 29, 2025
@github-project-automation github-project-automation bot moved this to Todo in CIL work Jan 29, 2025
@hrobarts hrobarts moved this from Todo to In Progress in CIL work Jan 29, 2025
@hrobarts
Copy link
Contributor Author

hrobarts commented Jan 29, 2025

In folder 2
01_optimisation_gd_fista

  • objective function deprecation warning
  • rtol and atol deprecation warning GD raises deprecation warning if atol and rtol are not used CIL#2054
  • PDHG.py:406: UserWarning: Convergence criterion of PDHG for scalar step-sizes is not satisfied.
    warnings.warn(
  • PDHG seems to be much slower? 40 minutes for 4500 iterations - notebook says it should take 5 minutes. Maybe this is a problem with the cloud machine I'm using
  • PDHG dual gap is negative so doesn't show up on log plot

02_tikhonov_block_framework.ipynb

  • show1D labels -> dataset_labels

03_PDHG

  • show1D labels -> dataset_labels
  • PDHG.py:406: UserWarning: Convergence criterion of PDHG for scalar step-sizes is not satisfied.
    warnings.warn(

@lauramurgatroyd
Copy link
Member

1_optimisation
This cell should be code not markdown
Image

opened: #235

@hrobarts
Copy link
Contributor Author

hrobarts commented Jan 31, 2025

04_SPDHG

  • Same warning about change in behaviour for FGP_TV nonnegativity is now default False. Update to change nonnegativity=True?

This is with False
Image
This is with True
Image

  • SPDHG explicit renormalising section is failing with a TypeError: unsupported operand type(s) for *=: 'NoneType' and 'float' @MargaretDuff , do you have any insight on what's going on here? It looks like it's failing on the first iteration

Image

@hrobarts
Copy link
Contributor Author

05_Laminography_with_TV no problems :)

@MargaretDuff
Copy link
Member

04_SPDHG

  • Same warning about change in behaviour for FGP_TV nonnegativity is now default False. Update to change nonnegativity=True?

This is with False Image This is with True Image

Yes, set nonnegativity=True

  • SPDHG explicit renormalising section is failing with a TypeError: unsupported operand type(s) for *=: 'NoneType' and 'float' @MargaretDuff , do you have any insight on what's going on here? It looks like it's failing on the first iteration

Image

This seems to be a bigger issue - let me have a look

@MargaretDuff
Copy link
Member

I added a new issue in CIL: TomographicImaging/CIL#2058. I don't have a quick fix for this notebook though :(

@hrobarts hrobarts moved this from In Progress to Blocked in CIL work Feb 5, 2025
@github-project-automation github-project-automation bot moved this from Blocked to Done in CIL work Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants