-
Notifications
You must be signed in to change notification settings - Fork 56
Data mix fix (backport #366) #368
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
Conversation
|
The code has changed between releases here, so this automatic backport failed. We'll need to manually adjust the code to work with the older version of SDG here. |
|
@aakankshaduggal I adjusted this backport of your PR to work with SDG 0.3.x - it's a pretty straightforward change to remove the |
|
Thanks @bbrowning, this looks good to me! |
|
Hmm, got caught by some other changes preventing CI from passing on this backport - will have to open a separate backport PR, get that one merged, and then maybe we can get this one green. |
4e10073 to
d2a179a
Compare
|
@Mergifyio rebase |
❌ Unable to rebase: user
|
|
@Mergifyio rebase |
Signed-off-by: Aakanksha Duggal <[email protected]> (cherry picked from commit 98c0fab)
Signed-off-by: Aakanksha Duggal <[email protected]> (cherry picked from commit 5667ac9)
…g_format Signed-off-by: Aakanksha Duggal <[email protected]> (cherry picked from commit cd0b73a)
Signed-off-by: Aakanksha Duggal <[email protected]> (cherry picked from commit 3040657)
Older versions of SDG did not have the `use_legacy_pretraining_format` parameter, so this just removes that from the backported code. Signed-off-by: Ben Browning <[email protected]>
✅ Branch has been successfully rebased |
d2a179a to
625d9ab
Compare
|
I will need to investigate why mergify didn't auto-merge this, but it looks like just a gap in the auto-merge rules at least for this release branch. Merging manually since all tests passed and it has multiple approvals. |
Addresses #365
This is an automatic backport of pull request #366 done by Mergify.