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

About the compatibility version #138

Closed
Anm-pinellia opened this issue May 30, 2024 · 2 comments
Closed

About the compatibility version #138

Anm-pinellia opened this issue May 30, 2024 · 2 comments

Comments

@Anm-pinellia
Copy link

Thanks for your kind work, i found that current version may raise error in my environments. I tried to find out the source of this issue usually comes from timm and pytorch-lightning. Though I tried to fix them by modifying some codes, but i am afraid of some other issues. Can you give more information about the current OpenSTL framework, like timm, pytorch-lightning, torch version?

@Anm-pinellia
Copy link
Author

For example, my timm version is 1.0.3, pytorch-lightning 2.1.0, OpenSTL 1.0.0. I found that current OpenSTL use timm's lrscheduler, and it will raise error "TypeError: step() missing 1 required positional argument: 'epoch'". This is cause by OpenSTL try to call scheduler.step() with its pytorch-lightning process flow, but cosinescheduler timm in this version must receive a params of epoch.

@chengtan9907
Copy link
Owner

The issue is not related to the version. There are some conflicts between timm and lightning in the scheduler. This bug was addressed in the latest commit; you can refer to commit#96a1c77.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants