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.

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

[quant][graphmode][fx] Scope support for call_method in QuantizationTracer #50173

Closed
wants to merge 8 commits into from

Commits on Jan 7, 2021

  1. [quant][graphmode][fx] Scope support for call_method in QuantizationT…

    …racer
    
    Summary:
    Previously we did not set the qconfig for call_method node correctly since it requires us to know
    the scope (module path of the module whose forward graph contains the node) of the node. This
    PR modifies the QuantizationTracer to record the scope information and build a map from call_method
    Node to module path, which will be used when we construct qconfig_map
    
    Test Plan:
    python test/test_quantization.py TestQuantizeFx.test_qconfig_for_call_method
    Reviewers:
    
    Subscribers:
    
    Tasks:
    
    Tags:
    
    [ghstack-poisoned]
    jerryzh168 committed Jan 7, 2021
    Configuration menu
    Copy the full SHA
    c35b966 View commit details
    Browse the repository at this point in the history
  2. Update on "[quant][graphmode][fx] Scope support for call_method in Qu…

    …antizationTracer"
    
    Summary:
    Previously we did not set the qconfig for call_method node correctly since it requires us to know
    the scope (module path of the module whose forward graph contains the node) of the node. This
    PR modifies the QuantizationTracer to record the scope information and build a map from call_method
    Node to module path, which will be used when we construct qconfig_map
    
    Test Plan:
    python test/test_quantization.py TestQuantizeFx.test_qconfig_for_call_method
    Reviewers:
    
    Subscribers:
    
    Tasks:
    
    Tags:
    
    Differential Revision: [D25818132](https://our.internmc.facebook.com/intern/diff/D25818132)
    
    [ghstack-poisoned]
    jerryzh168 committed Jan 7, 2021
    Configuration menu
    Copy the full SHA
    bf520b5 View commit details
    Browse the repository at this point in the history
  3. Update on "[quant][graphmode][fx] Scope support for call_method in Qu…

    …antizationTracer"
    
    Summary:
    Previously we did not set the qconfig for call_method node correctly since it requires us to know
    the scope (module path of the module whose forward graph contains the node) of the node. This
    PR modifies the QuantizationTracer to record the scope information and build a map from call_method
    Node to module path, which will be used when we construct qconfig_map
    
    Test Plan:
    python test/test_quantization.py TestQuantizeFx.test_qconfig_for_call_method
    Reviewers:
    
    Subscribers:
    
    Tasks:
    
    Tags:
    
    Differential Revision: [D25818132](https://our.internmc.facebook.com/intern/diff/D25818132)
    
    [ghstack-poisoned]
    jerryzh168 committed Jan 7, 2021
    Configuration menu
    Copy the full SHA
    a6ac7a2 View commit details
    Browse the repository at this point in the history
  4. Update on "[quant][graphmode][fx] Scope support for call_method in Qu…

    …antizationTracer"
    
    Summary:
    Previously we did not set the qconfig for call_method node correctly since it requires us to know
    the scope (module path of the module whose forward graph contains the node) of the node. This
    PR modifies the QuantizationTracer to record the scope information and build a map from call_method
    Node to module path, which will be used when we construct qconfig_map
    
    Test Plan:
    python test/test_quantization.py TestQuantizeFx.test_qconfig_for_call_method
    Reviewers:
    
    Subscribers:
    
    Tasks:
    
    Tags:
    
    Differential Revision: [D25818132](https://our.internmc.facebook.com/intern/diff/D25818132)
    
    [ghstack-poisoned]
    jerryzh168 committed Jan 7, 2021
    Configuration menu
    Copy the full SHA
    c16325c View commit details
    Browse the repository at this point in the history
  5. Update on "[quant][graphmode][fx] Scope support for call_method in Qu…

    …antizationTracer"
    
    Summary:
    Previously we did not set the qconfig for call_method node correctly since it requires us to know
    the scope (module path of the module whose forward graph contains the node) of the node. This
    PR modifies the QuantizationTracer to record the scope information and build a map from call_method
    Node to module path, which will be used when we construct qconfig_map
    
    Test Plan:
    python test/test_quantization.py TestQuantizeFx.test_qconfig_for_call_method
    Reviewers:
    
    Subscribers:
    
    Tasks:
    
    Tags:
    
    Differential Revision: [D25818132](https://our.internmc.facebook.com/intern/diff/D25818132)
    
    [ghstack-poisoned]
    jerryzh168 committed Jan 7, 2021
    Configuration menu
    Copy the full SHA
    835ae21 View commit details
    Browse the repository at this point in the history
  6. Update on "[quant][graphmode][fx] Scope support for call_method in Qu…

    …antizationTracer"
    
    
    Summary:
    Previously we did not set the qconfig for call_method node correctly since it requires us to know
    the scope (module path and type of the module whose forward graph contains the node) of the node. This
    PR modifies the QuantizationTracer to record the scope information and build a map from call_method
    Node to (module_path, module_type), which will be used when we construct qconfig_map
    
    Test Plan:
    python test/test_quantization.py TestQuantizeFx.test_qconfig_for_call_method
    Reviewers:
    
    Subscribers:
    
    Tasks:
    
    Tags:
    
    Differential Revision: [D25818132](https://our.internmc.facebook.com/intern/diff/D25818132)
    
    [ghstack-poisoned]
    jerryzh168 committed Jan 7, 2021
    Configuration menu
    Copy the full SHA
    46d4585 View commit details
    Browse the repository at this point in the history

Commits on Jan 9, 2021

  1. Update on "[quant][graphmode][fx] Scope support for call_method in Qu…

    …antizationTracer"
    
    
    Summary:
    Previously we did not set the qconfig for call_method node correctly since it requires us to know
    the scope (module path and type of the module whose forward graph contains the node) of the node. This
    PR modifies the QuantizationTracer to record the scope information and build a map from call_method
    Node to (module_path, module_type), which will be used when we construct qconfig_map
    
    Test Plan:
    python test/test_quantization.py TestQuantizeFx.test_qconfig_for_call_method
    Reviewers:
    
    Subscribers:
    
    Tasks:
    
    Tags:
    
    Differential Revision: [D25818132](https://our.internmc.facebook.com/intern/diff/D25818132)
    
    [ghstack-poisoned]
    jerryzh168 committed Jan 9, 2021
    Configuration menu
    Copy the full SHA
    de3d47a View commit details
    Browse the repository at this point in the history
  2. Update on "[quant][graphmode][fx] Scope support for call_method in Qu…

    …antizationTracer"
    
    
    Summary:
    Previously we did not set the qconfig for call_method node correctly since it requires us to know
    the scope (module path and type of the module whose forward graph contains the node) of the node. This
    PR modifies the QuantizationTracer to record the scope information and build a map from call_method
    Node to (module_path, module_type), which will be used when we construct qconfig_map
    
    Test Plan:
    python test/test_quantization.py TestQuantizeFx.test_qconfig_for_call_method
    Reviewers:
    
    Subscribers:
    
    Tasks:
    
    Tags:
    
    Differential Revision: [D25818132](https://our.internmc.facebook.com/intern/diff/D25818132)
    
    [ghstack-poisoned]
    jerryzh168 committed Jan 9, 2021
    Configuration menu
    Copy the full SHA
    3e9560f View commit details
    Browse the repository at this point in the history