Skip to content
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.

Sign up for GitHub

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

Jump to bottom

The description for uql_bucket_size and uql_quantize_all_layers is the same in the uniform quantization doc. #42

Closed
qingqing01 opened this issue Nov 11, 2018 · 2 comments
Closed

The description for uql_bucket_size and uql_quantize_all_layers is the same in the uniform quantization doc. #42

qingqing01 opened this issue Nov 11, 2018 · 2 comments
Assignees
@jiaxiang-wu
Labels
documentation Documentation needs revision

Comments

@qingqing01
Copy link

qingqing01 commented Nov 11, 2018 •
edited
Loading

See the doc https://pocketflow.github.io/uq_learner/#uniform-quantization-learner

--uql_bucket_size 256 quantize the first and last layers of the network or not
--uql_quantize_all_layers False quantize the first and last layers of the network or not
@jiaxiang-wu
Copy link
Contributor

Thanks for pointing this out. We will fix this ASAP.

@jiaxiang-wu jiaxiang-wu self-assigned this Nov 12, 2018
@jiaxiang-wu jiaxiang-wu added the documentation Documentation needs revision label Nov 12, 2018
jiaxiang-wu added a commit to jiaxiang-wu/PocketFlow that referenced this issue Nov 15, 2018
@jiaxiang-wu
update UQ-learner's hyper-param description; issue Tencent#42 fixed
de338ba
@jiaxiang-wu jiaxiang-wu mentioned this issue Nov 15, 2018
Merged
jiaxiang-wu added a commit that referenced this issue Nov 15, 2018
@jiaxiang-wu
Fix a few documentation issues (#66)
766877d 
* add source files for documentation

* add Python package dependencies

* add README.md

* issue #48 fixed

* update learners' description; issue #50 fixed

* update UQ-learner's hyper-param description; issue #42 fixed

* update docs for UniformQuantLearner and NonUniformQuantLearner

* typo fixed; remove trailing whitespaces
@jiaxiang-wu
Copy link
Contributor

Resolved in the pull request #66

jiaxiang-wu added a commit that referenced this issue Nov 16, 2018
@jiaxiang-wu
Update the command for model conversion (#70)
384f07d 
* add source files for documentation

* add Python package dependencies

* add README.md

* issue #48 fixed

* update learners' description; issue #50 fixed

* update UQ-learner's hyper-param description; issue #42 fixed

* update docs for UniformQuantLearner and NonUniformQuantLearner

* typo fixed; remove trailing whitespaces

* UniformQuant(TF)Learner's doc revised; issue #61 fixed
@GoldenSpark GoldenSpark mentioned this issue Nov 21, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jiaxiang-wu jiaxiang-wu

Labels
documentation Documentation needs revision
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@qingqing01 @jiaxiang-wu