Skip to content

Conversation

ShikharJ
Copy link
Contributor

@ShikharJ ShikharJ commented Aug 6, 2020

There's a lot to be done here:

  • Additional utility functions and refactoring some existing activation functions such as TanH, ReLU6 and Sigmoid
  • Adding support for variable demotion and usage of 8-bit and 16-bit quantized mixed operations
  • Implementing full pipeline as a model
  • Implementing quantized and floating-point numerical test-cases for the full pipeline
  • Adding the stripped down pipeline as a model
  • Adding complete documentation
  • Porting runtime-optimized code
  • Rewrite tests for runtime-optimized code and remove deprecated operators

@ShikharJ ShikharJ force-pushed the NetworkPO branch 4 times, most recently from 13caa3b to 1cc04b8 Compare August 7, 2020 19:12
@ShikharJ ShikharJ force-pushed the NetworkPO branch 4 times, most recently from f940815 to ec77746 Compare September 2, 2020 18:06
@ShikharJ ShikharJ changed the title [WIP] Stitched-Up RNNPool Face Detection Model Implementation Stitched-Up RNNPool Face Detection Model and Quantization Code Performance Refactor Sep 3, 2020
@ShikharJ
Copy link
Contributor Author

ShikharJ commented Sep 3, 2020

@harsha-simhadri This is ready for reviewing.

@harsha-simhadri
Copy link
Collaborator

@ShikharJ Is there a way to move the model params to a place outside github. I am concerned about bloat in the repo. How large are all the model files you are planning to check in?

@ShikharJ
Copy link
Contributor Author

@harsha-simhadri The collective size of the model params is around 2.2 MBs (1.6 MBs for the larger model and 0.6 MBs for the smaller model). I'm not aware if a modular method exists, apart from not committing the weight files into the master.

@harsha-simhadri
Copy link
Collaborator

@ShikharJ I am able to checkout git lfs and compile, but unclear what are the exec files and their commandline args. can you please add some notes to README, and print out a help line when the execs are run?

@harsha-simhadri harsha-simhadri merged commit 2c89850 into microsoft:master Oct 2, 2020
@ShikharJ ShikharJ deleted the NetworkPO branch October 5, 2020 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants