Skip to content

{data}[gfbf/2023a] HF-Datasets v2.18.0#20166

Merged
casparvl merged 6 commits intoeasybuilders:developfrom
VRehnberg:20240320135109_new_pr_Datasets2180
Mar 21, 2024
Merged

{data}[gfbf/2023a] HF-Datasets v2.18.0#20166
casparvl merged 6 commits intoeasybuilders:developfrom
VRehnberg:20240320135109_new_pr_Datasets2180

Conversation

@VRehnberg
Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

@VRehnberg
Copy link
Copy Markdown
Contributor Author

Based on #19795

@VRehnberg
Copy link
Copy Markdown
Contributor Author

Test report by @VRehnberg
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
alvis-skylake-build - Linux Rocky Linux 8.9, x86_64, Intel Xeon Processor (Skylake, IBRS, no TSX), Python 3.6.8
See https://gist.github.com/VRehnberg/3f8a040bf15ef3b028ed42291c4322ab for a full test report.

@casparvl casparvl added the new label Mar 20, 2024
@casparvl
Copy link
Copy Markdown
Contributor

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

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

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

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 2009914375 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).

@casparvl
Copy link
Copy Markdown
Contributor

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Copy Markdown
Collaborator

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

@VRehnberg
Copy link
Copy Markdown
Contributor Author

Considering adding a patch to enable using SoftFileLock instead of FileLock. Is a problematic package on some file systems otherwise. huggingface/datasets#6744

@casparvl
Copy link
Copy Markdown
Contributor

Ok, I'll hold off on further testing. Maybe put your PR in Draft mode for now, you can put it back on waiting for review once you've decided / implemented that change.

Comment thread easybuild/easyconfigs/d/Datasets/Datasets-2.18.0-gfbf-2023a.eb Outdated
@VRehnberg VRehnberg changed the title {data}[gfbf/2023a] Datasets v2.18.0 {data}[gfbf/2023a] HF-Datasets v2.18.0 Mar 20, 2024
@VRehnberg
Copy link
Copy Markdown
Contributor Author

Test report by @VRehnberg
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
alvis-skylake-build - Linux Rocky Linux 8.9, x86_64, Intel Xeon Processor (Skylake, IBRS, no TSX), Python 3.6.8
See https://gist.github.com/VRehnberg/ce76b268bdfc6d946c54fb304481aae7 for a full test report.

@VRehnberg
Copy link
Copy Markdown
Contributor Author

Test report by @VRehnberg
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
alvis-skylake-build - Linux Rocky Linux 8.9, x86_64, Intel Xeon Processor (Skylake, IBRS, no TSX), Python 3.6.8
See https://gist.github.com/VRehnberg/13d9a03a453d4c01835094dd916ee01a for a full test report.

@VRehnberg
Copy link
Copy Markdown
Contributor Author

@casparvl I think I'm satisfied now.

@VRehnberg
Copy link
Copy Markdown
Contributor Author

Test report by @VRehnberg
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
alvis-skylake-build - Linux Rocky Linux 8.9, x86_64, Intel Xeon Processor (Skylake, IBRS, no TSX), Python 3.6.8
See https://gist.github.com/VRehnberg/2e26f21ace58b75ff311f5feea01ad5e for a full test report.

@casparvl
Copy link
Copy Markdown
Contributor

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

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

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

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 2011684776 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).

@casparvl
Copy link
Copy Markdown
Contributor

Test report by @casparvl
FAILED
Build succeeded for 1 out of 2 (1 easyconfigs in total)
tcn1.local.snellius.surf.nl - Linux RHEL 8.6, x86_64, AMD EPYC 7H12 64-Core Processor, Python 3.6.8
See https://gist.github.com/casparvl/826725dfc2838379fb3bdd2e20e888fa for a full test report.

@boegelbot
Copy link
Copy Markdown
Collaborator

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

@casparvl
Copy link
Copy Markdown
Contributor

crap, I still have this version issue on my system...

datasets 2.18.0 has requirement pyarrow>=14.0.1, but you have pyarrow 0.0.0.

The following Python packages were likely not installed correctly because they show a version of '0.0.0':

I don't get it. It should have been resolved by rebuilding #19777 , which I did... strange :\ Well, as long as it passes on the other systems, I'm going to assume it is my own problem, there is probably something else I should have rebuilt...

@casparvl
Copy link
Copy Markdown
Contributor

@boegelbot please test @ jsc-zen3

@boegelbot
Copy link
Copy Markdown
Collaborator

@casparvl: 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=20166 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_20166 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 2011695049 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
Copy Markdown
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 3 out of 3 (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/00299ea42da435e2c443c46aec828cad for a full test report.

@casparvl
Copy link
Copy Markdown
Contributor

For my own system, I think maybe I need to rebuild #19758 too. I'll try. If that works, it's nice confirmation that this PR can be merged. If I keep hitting the same issue, I'll just ignore the failure on my system - it is clearly not related to this PR.

@casparvl
Copy link
Copy Markdown
Contributor

Test report by @casparvl
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
tcn1.local.snellius.surf.nl - Linux RHEL 8.6, x86_64, AMD EPYC 7H12 64-Core Processor, Python 3.6.8
See https://gist.github.com/casparvl/4561a90331302f193367cfe9807ca315 for a full test report.

@casparvl
Copy link
Copy Markdown
Contributor

Ok, so, indeed:

datasets 2.18.0 has requirement pyarrow>=14.0.1, but you have pyarrow 0.0.0.

The following Python packages were likely not installed correctly because they show a version of '0.0.0':

was resolved for my by rebuilding #19758 . With that, this PR now also succeeds on my system.

@casparvl
Copy link
Copy Markdown
Contributor

Going in, thanks @VRehnberg!

@casparvl casparvl merged commit ab0665b into easybuilders:develop Mar 21, 2024
WilleBell added a commit to WilleBell/easybuild-easyconfigs that referenced this pull request Apr 9, 2024
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.

4 participants