Skip to content

Fix model construction errors - #199

Merged
AnderBiguri merged 1 commit into
CambridgeCIA:mainfrom
cslht:fix/model-construction-crashes
Aug 6, 2026
Merged

Fix model construction errors#199
AnderBiguri merged 1 commit into
CambridgeCIA:mainfrom
cslht:fix/model-construction-crashes

Conversation

@cslht

@cslht cslht commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Fixes that let models be built from default parameters, without passing a parameter object:

  • dncnn.py: model built with the local model_parameters argument (None by default) instead of the parsed ones; now rebinds to self.model_parameters.
  • drunet.py: same rebind, plus a stale local reference inside an f-string; now uses self.model_parameters.
  • gs_drunet.py: imported DRUNet from a relative path that does not exist; now from CNNs.
  • ItNet.py / LG.py: inherited from nonexistent LIONmodel.LIONmodel; now from LIONmodel.
  • AR.py: read the undefined self.geo; now self.geometry.
  • LIONmodel.py: NoDataException was referenced but never imported in set_normalisation; now imported.
  • GaussianDenoiserSolver.py: called the non-callable numpy .size(0); now .shape[0].

@cslht cslht changed the title Fix construction errors in models and GaussianDenoiserSolver Fix model construction errors Aug 6, 2026
@AnderBiguri
AnderBiguri merged commit 432dc78 into CambridgeCIA:main Aug 6, 2026
1 check passed
cslht11 pushed a commit to cslht11/LION that referenced this pull request Sep 6, 2026
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.

2 participants