Skip to content

Implements vSHARP model (2D)#271

Merged
jonasteuwen merged 8 commits into
mainfrom
feature/vsharp-model
Mar 27, 2024
Merged

Implements vSHARP model (2D)#271
jonasteuwen merged 8 commits into
mainfrom
feature/vsharp-model

Conversation

@georgeyiasemis

Copy link
Copy Markdown
Contributor
  • This PR introduces the variable Splitting Half-quadratic ADMM algorithm for Reconstruction of inverse-Problems (vSHARP) model as presented in https://arxiv.org/abs/2309.09954.
  • It includes experiment files as presented in the paper for the fastmri prostate dataset.
  • It implements tests for vSHARP
  • Contains minor changes that allow None for masking function.
  • Updates main README file.

@codecov

codecov Bot commented Mar 25, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 96.12403% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 84.34%. Comparing base (dd3beed) to head (1503e50).

Files Patch % Lines
direct/predict.py 0.00% 2 Missing ⚠️
direct/nn/vsharp/vsharp.py 98.79% 1 Missing ⚠️
direct/nn/vsharp/vsharp_engine.py 97.43% 1 Missing ⚠️
direct/train.py 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #271      +/-   ##
==========================================
+ Coverage   84.08%   84.34%   +0.25%     
==========================================
  Files          93       95       +2     
  Lines        6504     6631     +127     
==========================================
+ Hits         5469     5593     +124     
- Misses       1035     1038       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions github-actions Bot added documentation Improvements or additions to documentation python labels Mar 25, 2024

@jonasteuwen jonasteuwen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, only a few minor comments

Comment thread direct/nn/vsharp/vsharp.py
Comment thread direct/nn/vsharp/vsharp.py
Comment thread direct/nn/vsharp/vsharp.py
Comment thread direct/nn/vsharp/vsharp.py
Comment thread direct/nn/vsharp/vsharp.py Outdated
Comment thread direct/nn/vsharp/vsharp_engine.py
Comment thread direct/nn/vsharp/vsharp_engine.py
Comment thread direct/nn/vsharp/vsharp_engine.py
Comment thread direct/train.py

@jonasteuwen jonasteuwen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jonasteuwen jonasteuwen merged commit ad28ef4 into main Mar 27, 2024
@jonasteuwen jonasteuwen deleted the feature/vsharp-model branch March 27, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants