Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

More tests #4999

Merged
merged 47 commits into from
Apr 5, 2023
Merged

More tests #4999

merged 47 commits into from
Apr 5, 2023

Conversation

mojtaba-komeili
Copy link
Contributor

@mojtaba-komeili mojtaba-komeili commented Mar 31, 2023

Patch description

  • Another try for turning them green.
  • Upgraded the deprecated CircleCI image that was running our tests.

@klshuster
Copy link
Contributor

which tests should this be turning green? i.e., cleaninstall and unittests_38 still red?

@mojtaba-komeili
Copy link
Contributor Author

which tests should this be turning green? i.e., cleaninstall and unittests_38 still red?

Yeah, I think cleaninstall is what causes the issue with the others too. There is some version conflict betweeen torch, torchvision and torchtext which looks impossible to solve.

@klshuster
Copy link
Contributor

Yeah, I think cleaninstall is what causes the issue with the others too. There is some version conflict betweeen torch, torchvision and torchtext which looks impossible to solve.

There is some logic in how to assign versions

can you try these? this is my current env:

torch                    1.13.1
torchaudio               0.13.1
torchvision              0.14.1

@mojtaba-komeili
Copy link
Contributor Author

OK, thanks. That was the magic combination. It cleaned up the clean install. Now four unit test are failing apex (1) and starspace (3). I would say let's merge these and work on those in another PR?

@klshuster
Copy link
Contributor

yep, so i already know the apex issue

pytorch 1.13.1 has an issue where torch.load doesn't respect custom pickle modules, so it's breaking when a model is loaded that we trained a while ago when apex was installed

for now, we may need to just mark that test as skippable if the pytorch version is 1.13.1

@mojtaba-komeili mojtaba-komeili merged commit cee9272 into main Apr 5, 2023
@mojtaba-komeili mojtaba-komeili deleted the more-tests branch April 5, 2023 17:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants