-
Notifications
You must be signed in to change notification settings - Fork 705
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}[GCCcore/12.3.0] PyTorch-bundle v2.1.2, SoX v14.4.2, libmad v0.15.1b, ... w/ CUDA 12.1.1 #19987
{lib}[GCCcore/12.3.0] PyTorch-bundle v2.1.2, SoX v14.4.2, libmad v0.15.1b, ... w/ CUDA 12.1.1 #19987
Conversation
…oX-14.4.2-GCCcore-12.3.0.eb, libmad-0.15.1b-GCCcore-12.3.0.eb, SentencePiece-0.2.0-GCC-12.3.0.eb
Test report by @jfgrimm |
@boegelbot: please test @ jsc-zen3 |
@jfgrimm: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1968569676 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot edit: succeeded apart from a sanity check that requires a GPU |
huh, jsc-zen3 should have that right? |
@boegelbot: please test @ jsc-zen3 |
@jfgrimm: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1969491560 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @SebastianAchilles |
@boegelbot: please test @ generoso |
@jfgrimm: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1970894376 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
…dle212 patch torchaudio to use external sox; enable CUDA support
Test report by @jfgrimm |
Test report by @jfgrimm |
easybuild/easyconfigs/p/PyTorch-bundle/PyTorch-bundle-2.1.2-foss-2023a-CUDA-12.1.1.eb
Outdated
Show resolved
Hide resolved
'" and not kaldi_io_test"' # requires kaldi_io | ||
'" and not test_dup_hw_acel"' # requires special render device permissions | ||
'" and not test_h264_cuvid"' # requires special render device permissions | ||
'" and not test_hecv_cuvid"' # requires special render device permissions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo, should be hevc
Test report by @VRehnberg |
Test report by @VRehnberg |
Test report by @VRehnberg |
Test report by @VRehnberg |
Most annnoying... I think I've rebuilt everything recently enough on both systems... I get this:
and the same in a bunch of other tqdm tests |
So the width padding whitespace of the progress bar from tqdm is different than expected. As far as failures go it's about as benign as I can imagine. But, I agree that it is annoying. I suppose that we can drop that with all the other loggers, probably is an issue with tqdm rather than ignite. But, it was nice to have some test that not all integration with third party loggers had failed. |
I don't care anymore. I'm just ignoring til this is merged. |
…ove some more tests
Gah.... inject-checksums messed things up... |
Test report by @akesandgren |
You also removed the --ignore=tests/ignite/contrib/handlers/test_tqdm_logger.py??? |
Test report by @akesandgren Running out of CUDAmemory.... not much to do then I guess. |
Yeah, I forgot to reload before adding my stuff... |
Test report by @akesandgren |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…asyconfigs into 20240228001016_new_pr_PyTorch-bundle212
Going in, thanks @Micket! |
(created using
eb --new-pr
)I have combined what I found from several conflicting PRs here to make what I think we want to have as the PyTorch-bundle.
I made some choices here:
(I don't really mind how this gets merged, if someone wants to update their PR instead, feel free to close this or just use it as a reference).