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

links in jobfs documentation broken? #549

Open
briochemc opened this issue Jan 17, 2025 · 3 comments · May be fixed by #551
Open

links in jobfs documentation broken? #549

briochemc opened this issue Jan 17, 2025 · 3 comments · May be fixed by #551

Comments

@briochemc
Copy link

The CLEX blog and NCI links in the documentation for jobfs below are broken for me:

See `NCI jobfs documentation`_ and the `CLEX blog for details`_.

They point to:

and are defined here:

payu/docs/source/config.rst

Lines 121 to 122 in 20a8e76

.. _`NCI jobfs documentation`: https://opus.nci.org.au/display/Help/PBS+Directives+Explained#PBSDirectivesExplained--ljobfs=%3C10GB%3E
.. _`CLEX blog for details`: https://climate-cms.org/posts/2022-11-10-jobfs.html#what-is-pbs-jobfs

I'm not sure what happened to the CLEX blog, and I'm not sure if links to NCI's docs are persistent, so what about simply copying the content of the current NCI docs' on jobfs, which is only a couple lines anyway. That is, directly add

The maximum amount of local disk available to the job on the hosting compute nodes. If this is missing in the submission, the value is set to 100 MiB. The jobfs allocation in a multiple-node jobs will be distributed equally among every nodes.

to the payu docs. I can submit a PR if needed.

@jo-basevi
Copy link
Collaborator

Thanks for raising this issue!

I think the CLEX blog has been moved to Github Pages - so the link for the above post is now found at: https://coecms.github.io/posts/2022-11-10-jobfs.html

The links to the NCI docs seems to be working currently so I wonder if the website was down for maintenance earlier

@briochemc
Copy link
Author

The links to the NCI docs seems to be working currently so I wonder if the website was down for maintenance earlier

Ah yes, it's working for me today. But either way, the link currently points to the "to be archived" version of the NCI docs on Gadi. Maybe pointing to the current NCI docs would be better?

Do you want me to submit a PR with updated links?

Also do you want me to add a little bit (copied from the NCI docs)?

@jo-basevi
Copy link
Collaborator

Oops I missed that it was the "to be archived" version! Yes if you are happy to update the links in a PR, I'll be happy to review it for you. I think it'll be OK to add that little bit from the NCI docs.

@briochemc briochemc linked a pull request Jan 20, 2025 that will close this issue
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 a pull request may close this issue.

2 participants