-
Notifications
You must be signed in to change notification settings - Fork 6
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
Interactive sessions on BluePebble for users from Maths #22
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @martonbalazs , this is great information! I always end up having to look up how to create an interactive session, so this will be useful to me personally.
I've made a few suggestions where I have spotted small issues or think the meaning of the text could be clarified. Could you review these and (if you are happy with them) commit them as changes to your PR? I'm open to discussion about the suggestions if you think the text should be written otherwise.
Once we are happy that the suggestions have been dealt with, I will ask a member of the HPC team to take a quick glance at the text, just to make sure this aligns with their recommendations for running interactive jobs. Provided they are happy with the text, I will then be happy to merge the PR.
Thanks again for your contribution 🙂
Thanks, will do the changes next week when I'm back to Bristol. (Simpler than setting up the whole environment on my laptop.) |
That would be great, thanks! |
I've pushed a new commit, I hope that still works with the pull request. All suggestions implemented, I've added a comment on using the magma partition in case the Magma software itself is needed for longer runs. |
Thanks! I've activated the build workflow on the PR. I'll aim to get this quickly checked over (by me and a member of the HPC team) soon so we can merge it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making these changes @martonbalazs , I think this is ready to merge!
Requested by HPC team
Based on feedback from HPC team
Improve wording
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added a warning message requested by the HPC team to the page on interactive sessions, describing the circumstances in which interactive sessions are supported.
No description provided.