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
Update sans.rst #18511
Update sans.rst #18511
Conversation
Just added a few fullstops and changed a few words
@AnthonyLim23 |
yes you are right, I will change that. I just put fixed to be consistent |
changed to fix for beam ...
@AnthonyLim23 When you make a change and push it, it is recommended to cancel the builds on previous commits. This can free up considerable resources. Thanks |
I am not sure how to do that from github. I do not have access to anything else from home. |
notes look fine. Once tests pass, please |
- Fix wrong initial position of LARMOR data in the beam centre finder | ||
- Allow loading of CanSAS data without error data | ||
- Fix saving CanSAS with transmission data from the ISIS SANS GUI | ||
- Fix for beam center finder. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
instead of Fix
? Just to be consistent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I did not carefully read the previous comments
Just added a few fullstops and changed a few words
Description of work.
To test:
Fixes #xxxx.
Reviewer
Please comment on the following (full description):
Code Review
Functional Tests
Do changes function as described? Add comments below that describe the tests performed?
How do the changes handle unexpected situations, e.g. bad input?
Has the relevant documentation been added/updated?
Is user-facing documentation written in a user-friendly manner?
Has developer documentation been updated if required?
Does everything look good? Comment with the ship it emoji but don't merge. A member of
@mantidproject/gatekeepers
will take care of it.