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.

Sign up for GitHub

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

Jump to bottom

Add noise power estimation to particle subtraction #1006

Open
wants to merge 32 commits into
base: devel
Choose a base branch
from
Open

Add noise power estimation to particle subtraction #1006

wants to merge 32 commits into from

Conversation

fede-pe
Copy link
Contributor

@fede-pe fede-pe commented Apr 7, 2025 •
edited
Loading

This PR includes:

  • Changes in projection subtraction (and MPI version) to estimate the noise power of the particles focusing in the subtracted region.

fede-pe added 30 commits April 2, 2025 15:30
@fede-pe
method for noise estimation
8df6182
@fede-pe
method in .h
054ad61
@fede-pe
add params for noise estimation
6df2a2e
@fede-pe
global variables for noise estimatioN
6b2c84b
@fede-pe
new logic for noise estimation
fde286e
@fede-pe
remove unused variables
e02d0db
@fede-pe
make volume dimensions global
4ddcc97
@fede-pe
fixes, now compiling
30ebb1e
@fede-pe
write output estimation noise
b92685a
@fede-pe
fixes
bd8aa49
@fede-pe
fixes in parallel mpi
17a4e7f
@fede-pe
first trial with mpi, failing, commit to checkout
6d63fec
@fede-pe
first fixes in parallel mode + debug code. commit to checkout!
23360f4 
!
@fede-pe
commit to compare with devel
ee404a3
@fede-pe
commit to compare with devel
792b12d
@fede-pe
fixes in noise estimation
ce3da16
@fede-pe
generate side info noise estimation
3662f1b
@fede-pe
fixes, compuiling
bb0e63c
@fede-pe
fixes in noise estimation, now working
053e012
@fede-pe
first version of MPI versioN
6f443bc
@fede-pe
fixes in generateNoiseEstimationSideInfo + debug code
ffa1760
@fede-pe
fix MPI version and averagign of noise power
d4871e8
@fede-pe
remove logic of number of particles for noise estimation
5a03b33
@fede-pe
remove logic of number of particles for noise estimation II
d46b70b
@fede-pe
remove debug code2
0a5da10
@fede-pe
fixes
fffa784
@fede-pe
chages in params for new logic
785b96c
@fede-pe
comment debug code2
025f504
@fede-pe
fix write output
1dbbe7f
@fede-pe
adapt MPI version to new write outptu
68484a2
@fede-pe fede-pe self-assigned this Apr 7, 2025
@fede-pe fede-pe requested a review from oierlauzi April 7, 2025 09:41
@fede-pe fede-pe changed the title Fi noise estimation for subtraction Add noise power estimation to particle subtraction Apr 7, 2025
@fede-pe fede-pe added the enhancement New feature or request label Apr 7, 2025
@fede-pe fede-pe requested a review from Vilax April 7, 2025 09:54
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Apr 7, 2025

Quality Gate Passed Quality Gate passed

Issues
10 New issues
0 Accepted issues

Measures
2 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oierlauzi oierlauzi Awaiting requested review from oierlauzi

@Vilax Vilax Awaiting requested review from Vilax

At least 1 approving review is required to merge this pull request.

Assignees

@fede-pe fede-pe

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@fede-pe