-
Notifications
You must be signed in to change notification settings - Fork 130
AlignAndFocusPowderSlim for NOMAD and POWGEN #40664
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.
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
Merged
peterfpeterson
merged 7 commits into
mantidproject:ornl-next
from
peterfpeterson:all_slim
Jan 21, 2026
Merged
AlignAndFocusPowderSlim for NOMAD and POWGEN #40664
peterfpeterson
merged 7 commits into
mantidproject:ornl-next
from
peterfpeterson:all_slim
Jan 21, 2026
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# Explicit dependencies |Dependency|Before|After|Change|Environments| |-|-|-|-|-| |mantidqt|6.14.20260114.2320|6.14.20260116.1839|Patch Upgrade|docs-build on linux-64| # Implicit dependencies |Dependency|Before|After|Change|Environments| |-|-|-|-|-| |[p11-kit](https://prefix.dev/channels/conda-forge/packages/p11-kit)|0.25.10|0.26.0|Minor Upgrade|{default, docs-build} on linux-64| |[liblzma](https://prefix.dev/channels/conda-forge/packages/liblzma)|5.8.1|5.8.2|Patch Upgrade|{default, docs-build, package-standalone, rattler-builder} on {osx-arm64, win-64}<br/>*all envs* on linux-64| |mantid|6.14.20260114.2320|6.14.20260116.1839|Patch Upgrade|docs-build on linux-64| |[gnutls](https://prefix.dev/channels/conda-forge/packages/gnutls)|he7cfd77_0|h18acefa_1|Only build string|{default, docs-build} on linux-64| |[nlohmann_json](https://prefix.dev/channels/conda-forge/packages/nlohmann_json)|h248ca61_1|h784d473_1|Only build string|{default, docs-build} on osx-arm64| [^1]: **Bold** means explicit dependency. [^2]: Dependency got downgraded.
<!--pre-commit.ci start--> updates: - [github.com/astral-sh/ruff-pre-commit: v0.14.11 → v0.14.13](astral-sh/ruff-pre-commit@v0.14.11...v0.14.13) <!--pre-commit.ci end--> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
# Explicit dependencies |Dependency|Before|After|Change|Environments| |-|-|-|-|-| |mantidqt|6.14.20260116.1839|6.14.20260119.1412|Patch Upgrade|docs-build on linux-64| |[hdf5](https://prefix.dev/channels/conda-forge/packages/hdf5)|nompi_h89f0904_104|nompi_h89f0904_105|Only build string|*all envs* on win-64| |[hdf5](https://prefix.dev/channels/conda-forge/packages/hdf5)|nompi_h1b119a7_104|nompi_h1b119a7_105|Only build string|*all envs* on linux-64| # Implicit dependencies |Dependency|Before|After|Change|Environments| |-|-|-|-|-| |mantid|6.14.20260116.1839|6.14.20260119.1412|Patch Upgrade|docs-build on linux-64| |[clang](https://prefix.dev/channels/conda-forge/packages/clang)|default_hf9bcbb7_16|default_hf9bcbb7_17|Only build string|*all envs* on osx-arm64| |[clang-18](https://prefix.dev/channels/conda-forge/packages/clang-18)|default_h73dfc95_16|default_h73dfc95_17|Only build string|*all envs* on osx-arm64| |[clangxx](https://prefix.dev/channels/conda-forge/packages/clangxx)|default_h36137df_16|default_h36137df_17|Only build string|*all envs* on osx-arm64| |[libclang-cpp18.1](https://prefix.dev/channels/conda-forge/packages/libclang-cpp18.1)|default_h73dfc95_16|default_h73dfc95_17|Only build string|*all envs* on osx-arm64| |[libclang-cpp20.1](https://prefix.dev/channels/conda-forge/packages/libclang-cpp20.1)|default_h99862b1_7|default_h99862b1_8|Only build string|*all envs* on linux-64| [^1]: **Bold** means explicit dependency. [^2]: Dependency got downgraded.
### Description of work There were several points in the code where EigenMatrix/EigenVector objects were converted into gsl objects to use in gsl functions. This PR removes these cases, swapping out the gsl functions for eigen implementations. Have enabled the system test which ref file I changed on windows and Mac, since the tests pass locally for me on Mac. Here's a windows test: https://builds.mantidproject.org/job/build_branch/1446/ ### To test: - Unit tests - Try out the relevant fitting functions if you want?
To make debugging the author mapping easier, add a main entrypoint to the authors tool. It also adds some flags to the `git log` call to make it more stable and skip merge commits. There is no associated issue. This is a trimmed down version of mantidproject#40396 because we decided that having a mailmap file required finding a place to store/share it. ### To test: This added the ability to get the authors directly from tools/DOI/authors.py and can be run without accidentally creating a DOI. Use tools/DOI/authors.py --help to find out how to use it and try the two modes. *This does not require release notes* because it modifies how authors are generated for the DOI tool.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pulls the following into
ornl-next