Skip to content

Comments

OSIRIS Silicon Analyser IDF Updates#40747

Draft
MohamedAlmaki wants to merge 4 commits intomainfrom
35777_update_si_idf
Draft

OSIRIS Silicon Analyser IDF Updates#40747
MohamedAlmaki wants to merge 4 commits intomainfrom
35777_update_si_idf

Conversation

@MohamedAlmaki
Copy link
Contributor

Description of work

Closes #xxxx.

To test:


Reviewer

Your comments will be used as part of the gatekeeper process. Comment clearly on what you have checked and tested during your review. Provide an audit trail for any changes requested.

As per the review guidelines:

  • Is the code of an acceptable quality? (Code standards/GUI standards)
  • Has a thorough functional test been performed? Do the changes handle unexpected input/situations?
  • Are appropriately scoped unit and/or system tests provided?
  • Do the release notes conform to the guidelines and describe the changes appropriately?
  • Has the relevant (user and developer) documentation been added/updated?
  • If the PR author isn’t in the mantid-developers or mantid-contributors teams, add a review comment rerun ci to authorize/rerun the CI

Gatekeeper

As per the gatekeeping guidelines:

  • Has a thorough first line review been conducted, including functional testing?
  • At a high-level, is the code quality sufficient?
  • Are the base, milestone and labels correct?

@MohamedAlmaki MohamedAlmaki added this to the Release 6.16 milestone Jan 28, 2026
@MohamedAlmaki MohamedAlmaki added ISIS: Spectroscopy Issue and pull requests relating to Muons, Indirect and Inelastic at ISIS Epic Used for issues and PRs that are managed under the ISIS Epic Workstream labels Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Epic Used for issues and PRs that are managed under the ISIS Epic Workstream ISIS: Spectroscopy Issue and pull requests relating to Muons, Indirect and Inelastic at ISIS

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant