-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add support for sequence parallelism #812
Comments
Hi, do you have any progress on sequence parallelism ? |
Not yet. We haven't been able to find someone with the time to implement this yet. |
I'd be interested in taking this on, if help is still needed here. |
That would be awesome! Someone recently dropped by our discord to volunteer to tackle this, was that you? If not, you should drop by our discord server to coordinate. |
Yes that was me! NerevarTheCaptain on discord. |
@NeverNude How's this going? |
Apologies, I haven't been able to get to this yet, should have an update
within the next 7-10 days
…On Mon, May 1, 2023 at 9:47 PM Quentin Anthony ***@***.***> wrote:
@NeverNude <https://github.com/NeverNude> How's this going?
—
Reply to this email directly, view it on GitHub
<#812 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAMTVBSRQ4P2WVJSBNGHJKDXECGUVANCNFSM6AAAAAAVSZQMOA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Any update? Anything we can help with? |
Apologies, life got way busier than I expected. I am hoping to get to this in the next couple of weeks, but feel free to assign it to somebody else, I don't want to block progress unnecessarily here. |
@NeverNude -- Are you still interested in heading this up? Do you have an early version that you can put in a draft PR so that we can add other contributors? |
@Quentin-Anthony you should assign it to somebody else at this point. I am unlikely to find the time soon to be able to tackle this, apologies. |
@NeverNude -- No worries! All - we are re-opening this issue to contributors. Reach out if you're interested in tackling it! |
Add sequence parallelism from upstream megatron
The text was updated successfully, but these errors were encountered: