-
Notifications
You must be signed in to change notification settings - Fork 556
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
[SSD] script to convert weights to a pickled dict #436
Conversation
recheck |
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
recheck |
Hi @petermattson , could someone from google review this old PR so we can merge? |
Will do. +Elias Mizan <emizan@google.com> could you please take a quick
look at this PR?
…On Fri, Mar 5, 2021 at 8:31 PM johntran-nv ***@***.***> wrote:
Hi @petermattson <https://github.com/petermattson> , could someone from
google review this old PR so we can merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#436 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIIVUHLEOW2BBV3YU6ULH2TTCGV2HANCNFSM4V4YMYIA>
.
|
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, please merge
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.
Re-approving after a trivial conflict resolution.
Hi @petermattson , the CLA appears to be re-broken here. Do you have any ideas on how to fix it again? |
+Elias Mizan ***@***.***>, any ideas? Thanks for looking into it. :-)
…On Wed, Apr 14, 2021 at 9:26 AM johntran-nv ***@***.***> wrote:
Hi @petermattson <https://github.com/petermattson> , the CLA appears to
be re-broken here. Do you have any ideas on how to fix it again?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#436 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIIVUHL7H2MINJYYUMHOCNLTIW647ANCNFSM4V4YMYIA>
.
|
I really have no idea, also I do not have access to the list of people who have signed the CLA. Can ahmadki sign the CLA again? In my case I had a CLA problem because I switched my username, but I doubt this is the case here. |
Added a script to convert PyTorch weights (.pth) to a pickled dictionary of numpy arrays.
Closes mlcommons/training_policies#391