Skip to content

update torus.py to use absolute paths #288

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DininduSenanayake
Copy link

Running inference on a Slurm cluster with sbatch failed to create the cache file .score.npy on the expected path as torus.py was using relative paths which doesn't get inherited correctly during the submission. Plus torus.py failed to trigger a meaningful message as to why the cache file wasn't on the expected path. This modified version

  1. will look for and save the cache files in the same directory as the script itself, regardless of where the script is being run from
  2. added debugging prints will help you see if/when the cache files are being created
  3. script will explicitly check that both required cache files exist before trying to load them

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