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

Improvements to Episode 2 (Working on a cluster) #79

Closed
Sabryr opened this issue Jan 25, 2019 · 15 comments
Closed

Improvements to Episode 2 (Working on a cluster) #79

Sabryr opened this issue Jan 25, 2019 · 15 comments
Assignees

Comments

@Sabryr
Copy link
Contributor

Sabryr commented Jan 25, 2019

Notes for possible PRs (I haven't gone through all the material yet, so some of the following may be addressed later).

  1. Why shouldn't we run long jobs on login nodes.
  2. A diagram to show the login process from laptop to login node via internet and how the login nodes are used to interact with the worker nodes.
  3. Elaborate more on, availability of common storage, place in one location and access from any worker node (much faster than through internet). A diagram to show all compute nodes are connected to each other and to the common storage.
  4. Photos or CPU and memory and disk (to show these are not mystical stuff but physical objects)
  5. When showing nproc, sinfo and free -n, show "df -h" to see available storage locations.
@psteinb
Copy link
Contributor

psteinb commented Jan 29, 2019

Hi Sabyr,
thanks for opening this issue. I fully support 1 & 2. On 3. you mean to discuss the shared file system, right? On 4. I've had mixed success with similar approaches, but still think that its a good idea. On 5. this part needs to be reworked as it currently is too SLURM and site specific. Further, there is quite a large possibility to talk about concepts that have not been touched.

@Sabryr
Copy link
Contributor Author

Sabryr commented Jan 30, 2019

Yes shared filesystem. I agree with your other comments. Your earlier material had a configuration option to easily change from SLURM to other. Is it OK to start a new issue to try to transfer that here, or was there a particular reason that approach was abandoned ? .

@psteinb
Copy link
Contributor

psteinb commented Jan 30, 2019

It's still the plan to implement the configuration trick you mentioned in this repo. Feel free to open an PR if you want. I already adopted lesson-outline.md in this direction, which might work as a guideline. However, at the last video call, people expressed strong interest to reiterate on this lesson-outline. So I am waiting for any more contributions until that discussion has come to a conclusion. @ChristinaLK @devbioinfoguy

@ChristinaLK
Copy link
Contributor

We should probably nail down who is adding the configuration trick. I thought @aturner-epcc might be doing it, but at this point, I think anyone who adds it would be a hero. :)

@devbioinfoguy
Copy link
Contributor

devbioinfoguy commented Jan 30, 2019 via email

@aturner-epcc aturner-epcc self-assigned this Jan 30, 2019
@aturner-epcc
Copy link
Contributor

It is in progress at my end - hope to have it in the next couple of weeks. Worst case by the end of Feb. I'll assign this to me.

@psteinb
Copy link
Contributor

psteinb commented Jan 30, 2019

You mean this in PR #73 ?

@Sabryr
Copy link
Contributor Author

Sabryr commented Jan 30, 2019

@aturner-epcc , I could help with this if you need. Is it possible to submit a issue specific for this (which issue did you assign it to yourself ? ) and a WIP PR. (sorry for pushing I would very much like to use this material in a workshop)

@aturner-epcc
Copy link
Contributor

I have stopped doing anything until #73 is merged and we make a plan for how to take this forwards. I do not want to tread on @psteinb and others toes and was only doing stuff to be useful, not to take over this, it was all Peter's ideas and he should get the credit.

I have a working version with auto-includes for PBS Pro on our system that required a few more customisation variables than in Peter's original implementation. Should we open a new issue specifically to discuss auto-includes so we can capture everything in one place going forward? I feel like we have struggled as discussion is spread across multiple threads....

@psteinb
Copy link
Contributor

psteinb commented Feb 4, 2019

@aturner-epcc thanks for stepping in. I totally agree. And yes, please open an issue on auto-includes. I am curious on what you found.

@aturner-epcc
Copy link
Contributor

Autoconfig now in via #101 . Need to reassess the original issues raised and split out into separate issues that we can work to address.

@tkphd tkphd mentioned this issue Jan 28, 2021
10 tasks
@tkphd
Copy link
Collaborator

tkphd commented Jan 28, 2021

Broke out item 1 as #250.

@tkphd
Copy link
Collaborator

tkphd commented Jan 28, 2021

Items 2 and 4 have been resolved by @Sabryr's contribution of original artwork.

@tkphd
Copy link
Collaborator

tkphd commented Jan 28, 2021

Item 5 was resolved separately (df -Th alongside nproc and sinfo exercises).

@tkphd
Copy link
Collaborator

tkphd commented Jan 28, 2021

Broke out item 3 as #251. This combined issue is deprecated, in favor of discussing the specific topics separately.

@tkphd tkphd closed this as completed Jan 28, 2021
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

No branches or pull requests

6 participants