Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@clatlan
Copy link
Owner

@clatlan clatlan commented Aug 21, 2024

This PR will be associated with a major release.
It intends to:

  • reorganise the package, in particular the pipeline.py module will become a subpackage. The process and pipeline must be separated. Other pipelines could be added (ptycho/simulation pipelines?). Related plotting functions will be moved there.
  • Better handling of cpu => gpu => cpu jumps. See pyslurmutils.
  • Improve the single-machine use-case (i.e. without jumps)
  • Make the main bcdi_pipeline.ipynb notebook lighter, simpler.
  • Better import management, lazy imports added.
  • Miscellaneous changes: addition of new features such as :
    • plotting of strain statistics in the pipeline
    • the function to check the twin image problem in the API
    • new utility functions (ex: energy_to_wavelength to avoid importing xrayutilities...)
    • better management of pipeline plots, remove redundancy.
    • hot pixels and background removal in the pipeline (already effective in step_by_step_bcdi_analysis.ipynb)
    • and many more...

All comments and suggestions are welcome.

@clatlan clatlan added enhancement New feature or request help wanted Extra attention is needed labels Aug 21, 2024
@clatlan clatlan self-assigned this Aug 21, 2024
@clatlan clatlan added this to the 0.2.0 release milestone Aug 21, 2024
@clatlan clatlan marked this pull request as ready for review February 19, 2025 10:39
@DSimonne DSimonne merged commit bd0e71a into master Feb 19, 2025
1 of 2 checks passed
clatlan added a commit that referenced this pull request Feb 19, 2025
Merge pull request #70 from clatlan/refacto
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Add incident flux normalisation parameter

3 participants