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

fix JULIA_DEPOT_PATH for Julia #19650

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

smoors
Copy link
Contributor

@smoors smoors commented Jan 19, 2024

(created using eb --new-pr)

currently the Julia root is appended to JULIA_DEPOT_PATH, which is obviously wrong

workaround until we have a solution for easybuilders/easybuild-framework#4442

@smoors smoors added the bug fix label Jan 19, 2024
@smoors smoors added this to the release after 4.9.0 milestone Jan 19, 2024
@smoors
Copy link
Contributor Author

smoors commented Jan 19, 2024

Test report by @smoors
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node302.hydra.os - Linux Rocky Linux 8.8, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/smoors/cbbd0b911bef1bc3ae80f2802d574de4 for a full test report.

@smoors
Copy link
Contributor Author

smoors commented Jan 30, 2024

@boegelbot: please test @ generoso

@boegelbot
Copy link
Collaborator

@smoors: Request for testing this PR well received on login1

PR test command 'EB_PR=19650 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs /opt/software/slurm/bin/sbatch --job-name test_PR_19650 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 12769

Test results coming soon (I hope)...

- notification for comment with ID 1916237534 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns2 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/boegelbot/ede04e07112c40fd896e4c4a2ad83725 for a full test report.

@smoors
Copy link
Contributor Author

smoors commented Jan 30, 2024

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Collaborator

@smoors: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=19650 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_19650 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 3515

Test results coming soon (I hope)...

- notification for comment with ID 1916286617 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.3, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.18
See https://gist.github.com/boegelbot/9d28c7e1dadc6f6491b832e209d93da0 for a full test report.

@boegel
Copy link
Member

boegel commented Jan 31, 2024

@lexming mentioned that using JULIA_LOAD_PATH could be an alternative instead...

@smoors
Copy link
Contributor Author

smoors commented Jan 31, 2024

it seems that JULIA_LOAD_PATH is only for source code loading, not for compiled code loading.
i did not find any way to load the compiled code using that environment variable.

  • here, using JSON loads without precompiling
module load IJulia/1.24.0-Julia-1.8.5
echo JULIA_LOAD_PATH  # :/apps/brussel/RL8/skylake-ib/software/IJulia/1.24.0-Julia-1.8.5
  • here, using JSON fails:
unset JULIA_DEPOT_PATH
export JULIA_LOAD_PATH=:/apps/brussel/RL8/skylake-ib/software/IJulia/1.24.0-Julia-1.8.5
  • here, using JSON precompiles the JSON package in the default JULIA_DEPOT_PATH, which is not what we want:
unset JULIA_DEPOT_PATH
export JULIA_LOAD_PATH=:/apps/brussel/RL8/skylake-ib/software/IJulia/1.24.0-Julia-1.8.5/packages

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants