aws-samples / aws-eda-slurm-cluster

AWS Slurm Cluster for EDA Workloads
MIT No Attribution
23 stars 7 forks source link

Relax PyYAML version requirement #215

Closed cartalla closed 3 months ago

cartalla commented 3 months ago

Not sure why it was requiring a specific version, but doesn't seem to be necessary.

Update setup.sh

Don't deactivate virtual venv because it can be setting up newer python version.

Use return instead of exit so don't exit calling shell.

Use "which node" instead of "node -v" to detect if nodejs is in the path. "node-v" prompts to install nodejs, which I don't want because I want control over which version is installed.

Resolves #212

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.