search

mila-iqia / milatools

Tools to connect to and interact with the Mila cluster
MIT License
62 stars 12 forks source link

Add X forwarding to compute nodes in SSH config #80

Closed lebrice closed 10 months ago

satyaog commented 10 months ago

What are the benefits of this PR?

lebrice commented 10 months ago

What are the benefits of this PR?

This makes it possible to view interactive windows on the client machine (matplotlib plots and such).

codecov-commenter commented 10 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (987a516) 55.74% compared to head (c7128cd) 55.74%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #80 +/- ## ======================================= Coverage 55.74% 55.74% ======================================= Files 9 9 Lines 1202 1202 ======================================= Hits 670 670 Misses 532 532 ``` | [Flag](https://app.codecov.io/gh/mila-iqia/milatools/pull/80/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mila-iqia) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/mila-iqia/milatools/pull/80/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mila-iqia) | `55.74% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=mila-iqia#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.