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

ImportError: cannot import name 'get_args' from 'megatron' #181

Open
peter-fei opened this issue Dec 14, 2023 · 5 comments
Open

ImportError: cannot import name 'get_args' from 'megatron' #181

peter-fei opened this issue Dec 14, 2023 · 5 comments

Comments

@peter-fei
Copy link

No functions named get_args() in megatron
The funxitons in init doesn't include get_args

r"""
A set of modules to plugin into Megatron-LM with FastMoE
"""
from .utils import add_fmoe_args

from .layers import MegatronMLP
from .layers import fmoefy

from .checkpoint import save_checkpoint
from .checkpoint import load_checkpoint

from .distributed import DistributedDataParallel

from .balance import reset_gate_hook
from .balance import get_balance_profile
from .balance import generate_megatron_gate_hook
from .balance import add_balance_log

from .patch import patch_forward_step
from .patch import patch_model_provider

@laekov
Copy link
Owner

laekov commented Dec 14, 2023

What is the version of your Megatron-LM? They have being significantly changing the API in their recent versions.

@peter-fei
Copy link
Author

My version 1.1.0

@laekov
Copy link
Owner

laekov commented Dec 14, 2023

1.1.0 may be too old. We have verivied support for 2.2, 2.5 and 3.0.2 . If you have to use 1.1.0, you need to either modify megatron or fastmoe.

@peter-fei
Copy link
Author

Thanks. By the way, How can I get a higher version?

@laekov
Copy link
Owner

laekov commented Dec 14, 2023

Check out the tags here

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