-
Notifications
You must be signed in to change notification settings - Fork 17
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
Fix utils module not found #96
base: main
Are you sure you want to change the base?
Conversation
PTAL @SeaOfOcean |
@BeingGod thanks for the pr, you need to sign the license/cla first |
It looks like that I meet a problem. CLA signed page show that I have already signed CLA. What should I do now? |
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
|
Done. |
what‘s the docker image you use to reproduce the error? @BeingGod |
My docker was built by https://github.com/alibaba/ChatLearn/blob/main/docker/torch/Dockerfile.torch2.3.0 . |
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. Sorry for the late review.
Fix utils module not found bug when doing huggingface to megatron-lm model convert.