Skip to content

Conversation

@rozyczko
Copy link
Member

@rozyczko rozyczko commented Feb 21, 2025

This is a deceptively large PR, but the bulk of the changes consists of simple string replacements for the new parameter syntax.
raw_value -> value
error -> variance
units -> unit

The changes related to EDA are mostly in terms of parsing the CIF and setting up the job (fixing a few bugs in the process).
Please pay attention to changes in
calculators/cryspy/parser.py
src/easydiffraction/io/cif.py
src/easydiffraction/io/cif_reader.py
and minor fixes in job.py and phase.py

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: ['enhancement', 'bug', 'chore', 'documentation', 'refactor']

@rozyczko rozyczko added the [scope] enhancement Adds/improves features (major.MINOR.patch) label Feb 21, 2025
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

1 similar comment
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

fix phase generation for cases when arguments don't contain atoms/cell
adjusted fitting results
Updated dependency for easyScience
@rozyczko rozyczko marked this pull request as draft March 4, 2025 08:40
@rozyczko
Copy link
Member Author

rozyczko commented Mar 4, 2025

Back to draft, pending EasyScience and EasyCrystallography releases

@rozyczko rozyczko marked this pull request as ready for review April 4, 2025 07:59
@rozyczko rozyczko merged commit 8c8b15f into develop Apr 4, 2025
3 checks passed
@rozyczko rozyczko deleted the new_param branch June 19, 2025 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[scope] enhancement Adds/improves features (major.MINOR.patch)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants