Skip to content

[caffe2] pre-convert filter format in optimizeForIDEEP#9488

Closed
gujinghui wants to merge 1 commit intopytorch:masterfrom
gujinghui:weight_convert
Closed

[caffe2] pre-convert filter format in optimizeForIDEEP#9488
gujinghui wants to merge 1 commit intopytorch:masterfrom
gujinghui:weight_convert

Conversation

@gujinghui
Copy link
Collaborator

@gujinghui gujinghui commented Jul 17, 2018

Convert the filters to expected format before computation.

@gujinghui
Copy link
Collaborator Author

@yinghai pls help review
@jgong5

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

@gujinghui
Copy link
Collaborator Author

@yinghai rebased. pls review

@ezyang
Copy link
Contributor

ezyang commented Oct 12, 2018

ping @yinghai

@gujinghui
Copy link
Collaborator Author

@yinghai @ezyang
Just rebased this PR.
Can we merge this one at first?
Thanks.

@yinghai
Copy link
Contributor

yinghai commented Oct 20, 2018

Please split this into 3 PRs as you outlined above.

Signed-off-by: Gu, Jinghui <jinghui.gu@intel.com>
@gujinghui
Copy link
Collaborator Author

@yinghai
split to #12970 and #12971.
pls merge those two first. Then, merge this PR.

@yinghai
Copy link
Contributor

yinghai commented Feb 15, 2019

@gujinghui Is this still relevant?

Copy link
Contributor

@yinghai yinghai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No update for long.

@gujinghui
Copy link
Collaborator Author

@yinghai Thanks for reminder.
Not need now. Will submit new PR for this if necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants