[BUG] The argument --no-position-embedding should be fixed

[Hoonly]

Describe the bug from the file Megatron-LM/megatron/training/arguments.py

    group.add_argument('--no-position-embedding',
                       action='store_false',
                       help='Disable position embedding. Deprecated: use --position-embedding-type',
                       dest='add_position_embedding')

I can see that this argument is Deprecated, but if we only use --position-embedding-type=rope, the add_position_embedding will accept default value True, conflict to the rope

To Reproduce

Expected behavior

Stack trace/logs

Environment (please complete the following information):

• Megatron-LM commit ID 0d983e64afcd84cab83124e0b7ca89a3d8ec9655
• PyTorch version 2.3.0a0+ebedce2
• CUDA version V12.3.107
• NCCL version 2.19.4

Proposed fix If you have a proposal for how to fix the issue state it here or link to a PR.

Additional context Add any other context about the problem here.

[github-actions[bot]]

Marking as stale. No activity in 60 days.

[Hoonly]

这是来自QQ邮箱的自动回复邮件。   您好，您的邮件我已收到，谢谢。

[github-actions[bot]]

Marking as stale. No activity in 60 days.