consistently use patch to correctly detect Slurm job environment for jax v0.6.2 and v0.7.0#24447
Conversation
|
@pavelToman, can you sync the PR with develop? Now that the PR with the patch is merged, the CI should hopefully work. |
done |
|
@boegelbot please test @ jsc-zen3 |
|
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3654706479 processed Message to humans: this is just bookkeeping information for me, |
|
I've manually cancelled job 9158 at |
|
@boegelbot please test @ jsc-zen3 |
|
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3655767109 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @boegel |
Locks were still present. |
Locks removed... |
|
@boegelbot please test @ jsc-zen3 |
|
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3656878771 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen3 |
|
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3657838860 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen3 |
|
@pavelToman: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3659315197 processed Message to humans: this is just bookkeeping information for me, |
|
I don't think the problem is the locks... I'm testing an interactive installation in the bot account to figure out what's going on. |
Looks like running the |
|
Test report by @Thyre |
|
I canceled job It looks like 4 GB of RAM per core is insufficient for the 'jax' test suite. Using fewer cores but more RAM per core seems to work in my tests. |
|
@boegelbot please test @ jsc-zen3 |
|
@SebastianAchilles: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3677752683 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Going in, thanks @pavelToman! |
The patch from: jax-ml/jax#32799
Requires: