Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

fix buffer allocate and wrapper name confliction #3120

Closed
wants to merge 2 commits into from
Closed

fix buffer allocate and wrapper name confliction #3120

wants to merge 2 commits into from

Conversation

eedalong
Copy link
Contributor

#3119

  1. register zero_point and scale with a real value Tensor rather not None, cause latter value will cause state dict load failure

  2. avoid wrapper's name to be the same with module's name, cause it will cause some problems when load state dict

@ghost
Copy link

ghost commented Nov 24, 2020

CLA assistant check
All CLA requirements met.

@linbinskn
Copy link
Contributor

linbinskn commented Nov 24, 2020

You have to resolve the conflicts in unit test since you have changed the name of wrap module. For example, modify
the test_compressor.py in path-to-nni/test/ut/sdk.

@eedalong
Copy link
Contributor Author

You have to resolve the conflicts in unit test since you have changed the name of wrap module. For example, modify
the test_compressor.py in path-to-nni/test/ut/sdk.

what's the test command?

@eedalong eedalong closed this Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants