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

{lib}[foss/2022a] PyTorch-Lightning v1.8.4 w/ Python 3.10.4 #16681

Conversation

surak
Copy link
Contributor

@surak surak commented Nov 18, 2022

(created using eb --new-pr)

@SebastianAchilles
Copy link
Member

Test report by @SebastianAchilles
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
bwd-rockylinux-90 - Linux Rocky Linux 9.0 (Blue Onyx), x86_64, Intel(R) Core(TM) i7-6900K CPU @ 3.20GHz (broadwell), Python 3.9.10
See https://gist.github.com/8bff5f56c6dfed92517818a027285eb6 for a full test report.

@SebastianAchilles
Copy link
Member

SebastianAchilles commented Nov 18, 2022

The error I get is:

ModuleNotFoundError: No module named 'hatchling'

termcolor >=2.0.0 requires hatchling: https://github.com/termcolor/termcolor/blob/2.1.0/pyproject.toml#L5

This PR requires #16402

@lexming lexming added the update label Dec 8, 2022
@lexming lexming added this to the 4.x milestone Dec 8, 2022
@lexming lexming changed the title Version bump. New dependencies of latest tensorboard and Lightning {lib}[foss/2022a] PyTorch-Lightning v1.8.2 w/ Python 3.10.4 Dec 8, 2022
@SebastianAchilles
Copy link
Member

Test report by @SebastianAchilles
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
bwd-rockylinux-91 - Linux Rocky Linux 9.1 (Blue Onyx), x86_64, Intel(R) Core(TM) i7-6900K CPU @ 3.20GHz (broadwell), 2 x NVIDIA NVIDIA GeForce GTX 1060 6GB, 515.86.01, Python 3.9.14
See https://gist.github.com/eb598f76714b94129aa88e2e7ec6a8fe for a full test report.

…8.2-foss-2022a-CUDA-11.7.0.eb

Co-authored-by: SebastianAchilles <achilles@aices.rwth-aachen.de>
@SebastianAchilles
Copy link
Member

Test report by @SebastianAchilles
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
bwd-rockylinux-91 - Linux Rocky Linux 9.1 (Blue Onyx), x86_64, Intel(R) Core(TM) i7-6900K CPU @ 3.20GHz (broadwell), 2 x NVIDIA NVIDIA GeForce GTX 1060 6GB, 515.86.01, Python 3.9.14
See https://gist.github.com/4907ad747a729306d03defc617bc4428 for a full test report.

@lexming
Copy link
Contributor

lexming commented Dec 8, 2022

Sorry to be a bother, but I'm still not convinced that termcolor and fire are dependencies. After looking into the dependencies a bit more, I have the impression that we even have some missing ones, like lightning_api_access. We should probably follow the list of requirements in https://github.com/Lightning-AI/lightning/blob/1.8.2/requirements.txt

@surak
Copy link
Contributor Author

surak commented Dec 8, 2022

Sorry to be a bother, but I'm still not convinced that termcolor and fire are dependencies. After looking into the dependencies a bit more, I have the impression that we even have some missing ones, like lightning_api_access. We should probably follow the list of requirements in https://github.com/Lightning-AI/lightning/blob/1.8.2/requirements.txt

Hey Alex, would you be so kind to make the edits incorporating these changes you propose? I will accept them right away!

@lexming
Copy link
Contributor

lexming commented Dec 10, 2022

@surak done in surak#4

I took the chance to also update to v1.8.4. All dependencies and extensions follow the requirements listed in https://github.com/Lightning-AI/lightning/blob/1.8.4/requirements.txt

update PyTorch-Lightning v1.8.2 to v1.8.4 and use dependencies following its requirements file
@surak
Copy link
Contributor Author

surak commented Dec 10, 2022

@surak done in surak#4

I took the chance to also update to v1.8.4. All dependencies and extensions follow the requirements listed in https://github.com/Lightning-AI/lightning/blob/1.8.4/requirements.txt

Thank you so much! You made a much better job than I did!

@boegelbot

This comment was marked as outdated.

@lexming
Copy link
Contributor

lexming commented Dec 11, 2022

@boegelbot: please test @ generoso

@lexming lexming changed the title {lib}[foss/2022a] PyTorch-Lightning v1.8.2 w/ Python 3.10.4 {lib}[foss/2022a] PyTorch-Lightning v1.8.4 w/ Python 3.10.4 Dec 11, 2022
@boegelbot
Copy link
Collaborator

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

PR test command 'EB_PR=16681 EB_ARGS= EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_16681 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

- notification for comment with ID 1345421521 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 3 out of 3 (3 easyconfigs in total)
cns1 - 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/5ef53551d8a8e225e727b14c4225eca4 for a full test report.

Copy link
Contributor

@lexming lexming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SebastianAchilles what do you think? is this ok on your side?

Copy link
Member

@SebastianAchilles SebastianAchilles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me as well.

Thanks a lot @lexming and @surak

@SebastianAchilles
Copy link
Member

Going in, thanks @surak!

@SebastianAchilles SebastianAchilles merged commit 999f334 into easybuilders:develop Dec 13, 2022
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