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

Scheduled workflow failed #2577

Closed
github-actions bot opened this issue May 22, 2022 · 4 comments · Fixed by #2581
Closed

Scheduled workflow failed #2577

github-actions bot opened this issue May 22, 2022 · 4 comments · Fixed by #2581
Assignees
Labels

Comments

@github-actions
Copy link

Oh no, something went wrong in the scheduled workflow PyTorch version tests with commit dc1d085.
Please look into it:

https://github.com/pytorch/ignite/actions/runs/2364687129

Feel free to close this if this was just a one-off error.

@github-actions github-actions bot added the bug label May 22, 2022
@vfdev-5
Copy link
Collaborator

vfdev-5 commented May 22, 2022

@sdesrozis can you please take a look

@sdesrozis
Copy link
Contributor

sdesrozis commented May 23, 2022

Something changed with dill.

import dill
from abc import ABCMeta


class ABCMetaClass(metaclass=ABCMeta):
    def __init__(self):
        pass

class Test(ABCMetaClass):
    def __init__(self):
        pass

if __name__ == "__main__":
    t = Test()
    t = dill.loads(dill.dumps(t))

The above code works fine with dill==0.2 and fails with the latest version dill==0.3.5.1...

See the issue uqfoundation/dill#332

The minimalist snippet from the issue

import dill as pickle

from abc import ABCMeta

class TestClass(metaclass=ABCMeta):
    pass

with open('test.pickle', 'wb') as f:
    pickle.dump(TestClass, f)

@sdesrozis
Copy link
Contributor

Please, read this uqfoundation/dill#450 (comment)

@vfdev-5
Copy link
Collaborator

vfdev-5 commented May 23, 2022 via email

@sdesrozis sdesrozis mentioned this issue May 25, 2022
3 tasks
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 a pull request may close this issue.

2 participants