Skip to content
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 1403 #122

Merged
merged 1 commit into from
Sep 10, 2024
Merged

Tickets/sitcom 1403 #122

merged 1 commit into from
Sep 10, 2024

Conversation

Garavena89
Copy link
Contributor

@Garavena89 Garavena89 commented Aug 1, 2024

Hello Alysha, Ioana and Eric
Could you please review this document about the safety entry to AuxTel?
It is very general but I think could be ok to start
Thanks!

Copy link
Contributor

@Slimleashma Slimleashma left a comment

Choose a reason for hiding this comment

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

I left a comment to add more data in a section, but I think this looks great! Thanks!

@isotuela isotuela changed the base branch from tickets/SITCOM-1443 to main August 12, 2024 13:38
@isotuela isotuela removed their request for review August 19, 2024 14:25

2. **Entry door key**

- The key to access the main entrance door to the telescope will be hanging on the right side of the door just below the fire extintor sign. It is strictly requested that the keys must be always left in the same place when leaving the telescope.

Choose a reason for hiding this comment

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

"fire extinguisher" is the word in English


.. tip::

Red light flashlights can be a good choice and they interfeer much less than regular white light.

Choose a reason for hiding this comment

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

interfere


- Being close to a radio (or always carrying one) and communicating your intentions to summit staff and on Slack.

- Following the safety gate procedure.

Choose a reason for hiding this comment

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

Is there a link to the safety gate procedure?

Safety/Enter-Observatory-Spaces/Enter-AuxTel-Telescope.rst Outdated Show resolved Hide resolved
Copy link

@PaulinaLSST PaulinaLSST left a comment

Choose a reason for hiding this comment

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

It's a great document; it is clear, readable, concise, and well-detailed. I only have one comment to review on line 90 ( a suggestion).

Thank you very much for this document.

Remember to always keep the list of :ref:`Emergency Numbers <summit-contact-numbers-emergency-numbers>` handy for any urgent situations.


This procedure was last modified on |today|.

Choose a reason for hiding this comment

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

It's a great document; it is clear, readable, concise, and well-detailed. I only have one comment to review on line 90 ( a suggestion).

Thank you very much for this document.

Copy link
Contributor

@isotuela isotuela left a comment

Choose a reason for hiding this comment

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

Thank you for this work! It looks good.

I have added some requests to include for completion: internal links to other safety pages, Safety Gate, LOTO and e-stop procedures.

Also, as the last step please squash your commits into a single commit.

Many thanks!
Ioana

Safety/Enter-Observatory-Spaces/Enter-AuxTel-Telescope.rst Outdated Show resolved Hide resolved

- Being close to a radio (or always carrying one) and communicating your intentions to summit staff and on Slack.

- Following the `safety gate procedure <https://obs-ops.lsst.io/v/SITCOM-1360/Safety/Safety-Systems/AuxTel-Safety-Gate-Introduction.html>`__.
Copy link
Contributor

Choose a reason for hiding this comment

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

Replace this external reference with an internal tag to the Safety Gate document. It will start working once both PR get merged.

Copy link
Contributor

Choose a reason for hiding this comment

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

This link won't work once SITCOM-1360 is merged nor do we want to link to an https, that might in the future give make linkcheck problems ;)


- Following the `safety gate procedure <https://obs-ops.lsst.io/v/SITCOM-1360/Safety/Safety-Systems/AuxTel-Safety-Gate-Introduction.html>`__.

- Locking out movement when necessary.
Copy link
Contributor

Choose a reason for hiding this comment

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

Link here to the LOTO procedure for AuxTel using internal referencing.

Also please link here the e-stop procedure that might be needed in case of emergency and one needs to press the e-stop. Thank you

Copy link
Contributor

@Slimleashma Slimleashma left a comment

Choose a reason for hiding this comment

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

I added one comment near line 111 to ensure that we remember to check weather conditions before ventilating. Everything else looks good.

@Garavena89
Copy link
Contributor Author

changes done! ready for merging

Copy link
Contributor

@isotuela isotuela left a comment

Choose a reason for hiding this comment

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

Dear Gonzalo,
Thank you for all the changes! Approved.
Ioana

Copy link
Contributor

@Slimleashma Slimleashma left a comment

Choose a reason for hiding this comment

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

No additional comments to make. Thank you for preparing this procedure!

@Garavena89 Garavena89 merged commit 1f3db22 into main Sep 10, 2024
2 checks passed
@Garavena89 Garavena89 deleted the tickets/SITCOM-1403 branch September 10, 2024 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants