Skip to content

Create snellius_example.sh - #40

Closed
xinan1911 wants to merge 1 commit into
masterfrom
xinan1911-snellius-example
Closed

Create snellius_example.sh#40
xinan1911 wants to merge 1 commit into
masterfrom
xinan1911-snellius-example

Conversation

@xinan1911

Copy link
Copy Markdown
Contributor

You can follow the instructions in the script to run an example on Snellius. The tokens on the Picas token server will be picked up by Snellius worker nodes.

You can follow the instructions in the script to run an example on Snellius. The tokens on the Picas token server will be picked up by Snellius worker nodes.
@lnauta

lnauta commented Jul 21, 2025

Copy link
Copy Markdown
Member

Check the general example for the passed arguments with --export that may be needed.
https://github.com/sara-nl/picasclient/blob/master/examples/slurm_example.sh

Also, it seems very similar to the general example, I am unsure if we should add this. We can discuss in person.

@xinan1911

Copy link
Copy Markdown
Contributor Author

@lnauta Could you explain what the #SBATCH --export=ALL,DESIGN_DOC=Monitor,VIEW=todo does?

The lines added are necessary for the pilot job being allocated in Snellius. For detailed reasoning, please check link. We can discuss this further when Haili is back.

@xinan1911

Copy link
Copy Markdown
Contributor Author

@mherkazandjian Please take a look of this PR

#git clone https://github.com/sara-nl/picasclient.git
#cd picasclient
#2.install packages
#pip install picas

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xinan1911 i suggest to remove the installation part from the comment here since it is covered below and in any case for non python 3.12 "pip install picas" will install an old version of picas (i think 0.2.1, when i tested without module loading the stuff below). so it might cause some confusion for some.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i ran the script on snellius and it worked (just an extra confirmation).

looks good. tnx.

@lnauta

lnauta commented Sep 19, 2025

Copy link
Copy Markdown
Member

Similar functionality was included in #42, check if this one is still needed.

@mherkazandjian
mherkazandjian deleted the xinan1911-snellius-example branch October 1, 2025 13:04
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.

3 participants