-
Notifications
You must be signed in to change notification settings - Fork 1
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
Tickets/sitcom 1444 #124
Tickets/sitcom 1444 #124
Conversation
3dfde74
to
7882526
Compare
7882526
to
c4c3e7b
Compare
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.
Thank you! It looks good. I have added some suggestions, mostly regarding the caption in figures and the position of the LOTO box note. I think if each step's key words is in bold, it would improve reading, just like in step 2 of Applying LOTO on OSS.
Also, please squash commits into a single one. Let me know if you need any help!
Thank you!
Simonyi/Non-Standard-Operations/MTCS/TMA/Simonyi-LOTO-procedure.rst
Outdated
Show resolved
Hide resolved
Simonyi/Non-Standard-Operations/MTCS/TMA/Simonyi-LOTO-procedure.rst
Outdated
Show resolved
Hide resolved
Simonyi/Non-Standard-Operations/MTCS/TMA/Simonyi-LOTO-procedure.rst
Outdated
Show resolved
Hide resolved
Simonyi/Non-Standard-Operations/MTCS/TMA/Simonyi-LOTO-procedure.rst
Outdated
Show resolved
Hide resolved
Simonyi/Non-Standard-Operations/MTCS/TMA/Simonyi-LOTO-procedure.rst
Outdated
Show resolved
Hide resolved
Simonyi/Non-Standard-Operations/MTCS/TMA/Simonyi-LOTO-procedure.rst
Outdated
Show resolved
Hide resolved
Simonyi/Non-Standard-Operations/MTCS/TMA/Simonyi-LOTO-procedure.rst
Outdated
Show resolved
Hide resolved
Simonyi/Non-Standard-Operations/MTCS/TMA/Simonyi-LOTO-procedure.rst
Outdated
Show resolved
Hide resolved
25a9fe0
to
727c18c
Compare
247e7de
to
73b1e0d
Compare
3f63376
to
c33cb18
Compare
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.
Thank you for the new photos and explicit instructions about the LOTO boxes. One note to make (and maybe there is hesitance to make this official) is that during the night, only ONE pump needs to be locked to consider the TMA fully under LOTO. This is helpful when one cannot find the LOTO boxes or there aren't multiple people around to move this process faster. It's something to note that LOTO at night with multiple locks and keys raises a risk of leaving lights on longer and losing these multiples. Can we consider this additional caveat for the document?
@Slimleashma I can add a note box with 'LOTO at night ' to highlight these caveats -- What do you think ? |
That could be good. Multiple locks is overkill when we are talking about a
two-person operation at night.
Alysha
…On Mon, Aug 19, 2024 at 10:07 PM Kshitija Kelkar ***@***.***> wrote:
@Slimleashma <https://github.com/Slimleashma> I can add a note box with
'LOTO at night ' to highlight these caveats -- What do you think ?
This note can be convereted to sub-section after we confirm the official
word on it.
—
Reply to this email directly, view it on GitHub
<#124 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ASLSVBYHWH2K67JET6FF5QDZSJGADAVCNFSM6AAAAABMAW75GKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJXGM2TMMJXHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@Slimleashma @isotuela- I added a note about "LOTO at night". Could you please check if it reads ok? |
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.
Only one minor grammar comment. Could you please squash your commits into one? Let me know if you encounter any issue.
Thanks!
Ioana
Simonyi/Non-Standard-Operations/MTCS/TMA/Simonyi-LOTO-procedure.rst
Outdated
Show resolved
Hide resolved
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.
Looks great! Thanks for the nighttime caveat.
@isotuela formatting corrected and commits squashed! |
ad3037d
to
d1569c3
Compare
d1569c3
to
97a3ea7
Compare
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.
All good and ready to merge.
Thanks!
I have migrated the procedure to Obs-Ops. I have also corrected a bit of English and the flow while keeping teh content exactly same.
There is several important things I identified as missing from the content ---
I was not involved in the content creation and the review of the original Confluence page.