Skip to content

Usability: submitting jobs #72

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

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Usability: submitting jobs #72

wants to merge 10 commits into from

Conversation

RobJY
Copy link
Contributor

@RobJY RobJY commented Mar 28, 2025

I reviewed the Submitting Jobs section for usability and have the following open questions:

  • In both slurm submitting jobs section and the slurm tutorial:
    • array job is using python2 in run-jobarray.s
    • python2_wordcount.py needs to remove the U option from the open file command
    • remove the 2 from python2_wordcountpy filename
  • in slurm submitting jobs:
    • xterm doesn't exist on (some/all) compute nodes. In the Interactive Bash Job Examples sections we ask users to verify forwarding is working by calling xterm. What should we use instead?
      • how about gnuplot test
    • make example prompts consistent
    • update GPU examples to our current GPUs
  • in slurm tutorial
    • tutorial says to read Slurm Main Commands before tutorial, so should the document order be changed?
    • compiling the MPI example with the intel compiler fails for me, but it works with gcc.
  • [ ] I also went through the old tutorials for linux and HPC and they have the same issues with quotes vs. admonitions and calling xterm when it doesn't exist on the server. Should I go ahead and make changes to them as well?
  • the following are bigger jobs than I originally thought, so I'll put them in a new PR
  • try adding Mermaid diagrams to tutorials (and to new tutorials that will become a new issue)

@s-sajid-ali can we make changes to the files getting copied in for the array job tutorials? Any thoughts on the other open questions or who I should contact about them?

Thanks for the help with these!

@RobJY RobJY requested a review from s-sajid-ali March 28, 2025 18:25
@RobJY RobJY self-assigned this Mar 28, 2025
Copy link

github-actions bot commented Mar 28, 2025

PR Preview Action v1.6.0

🚀 View preview at
https://NYU-RTS.github.io/rts-docs-dev/pr-preview/pr-72/

Built to branch gh-pages at 2025-04-03 19:06 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@RobJY
Copy link
Contributor Author

RobJY commented Apr 22, 2025

@s-sajid-ali I'm having trouble getting mermaid installed. I keep getting a conflict between eslint, @eslint/compat, and other packages. Were you able to get it installed and added to this project?

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.

1 participant