Skip to content

Fi proy substraction fixes #938

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
Merged
oierlauzi merged 49 commits into from
Nov 27, 2024
Merged

Fi proy substraction fixes #938

oierlauzi merged 49 commits into from
Nov 27, 2024

Conversation

@fede-pe
Copy link
Contributor

@fede-pe fede-pe commented Nov 13, 2024

This PR includes:

  • Fixes on projection subtraction a particle boosting both in MPI and single thread version
  • Improvements on projection subtraction a particle boosting both in MPI and single thread version

Test passing

Depends on: I2PC/xmippCore#210

fede-pe and others added 30 commits August 6, 2024 16:03
@fede-pe
remove unused variable
7afcc26
@fede-pe
refactor + comment code
c3f1755
@fede-pe
commit to checkout
53e8e66
@fede-pe
Merge branch 'devel' into fi_LigFit
1fc17ae
@fede-pe
uncomment code
fd603a0
@fede-pe
save statistics of subtracted particles (in progress...) commit to ch…
f2976f5 
…eckout due to config errors
@fede-pe
Merge branch 'devel' into fi_LigFit
1712c38
@fede-pe
add statistic of kept region in output log
99ea12f
@fede-pe
commit to checkout
336f0bb
@fede-pe
calculate avg and std weighted by mask + improve comments
4f34057
@fede-pe
debug output files flag!
36e2640
@fede-pe
remove --fmask_width unused option
f5e3e65
@fede-pe
remove limit freq substraction
f5cc407
@fede-pe
fix
c2f6012
@fede-pe
update default values
348d56e
@fede-pe
varaible reaming
3781151
@fede-pe
improve help and add debug code
73031dd
@fede-pe
fix max resol normalization + add debug code + prepare program to acc…
edd50aa 
…ept second mask
@fede-pe
add input mask as possible input (instead of circular mask)
d75a92e
@fede-pe
add code to debug input mask + fix freq map + conditional initializat…
c5ef554 
…ion of Fourier projectors
@fede-pe
apply (also) volume mask to input volume in provided
48b9e14
@fede-pe
set default maxRes=-1 to Nyquist/sqrt(2)
bec6c0b
@fede-pe
define independtly projection resolution and subtraction resolution +…
7534a54 
… debug code
@fede-pe
Merge branch 'devel' into fi_LigFit
3f5dead
@fede-pe
optmize code + comment unused methods
941e734
@fede-pe
refactor + add debug files + logic under PFourier0(0,0) = IiMFourier(…
58eb9f1 
…0,0) + enhancement
@fede-pe
Merge branch 'devel' into fi_LigFit
2888d46
@fede-pe
fix uninitialization of mask projector when mask is not provided in m…
1486320 
…pi version
@fede-pe
fixes, fixes everywhere + project mask in real space
584bf49
@fede-pe
project mask in real space in MPI version
3db0894
@fede-pe fede-pe mentioned this pull request Nov 13, 2024
Merged
@fede-pe fede-pe requested a review from oierlauzi November 13, 2024 14:23
@fede-pe fede-pe self-assigned this Nov 13, 2024
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 21, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

@oierlauzi oierlauzi merged commit ef09bea into devel Nov 27, 2024
6 checks passed
@oierlauzi oierlauzi deleted the fi_proySubstractionFixes branch November 27, 2024 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@oierlauzi oierlauzi oierlauzi approved these changes

Assignees

@fede-pe fede-pe

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fede-pe @oierlauzi