Skip to content
This repository has been archived by the owner on Jan 3, 2023. It is now read-only.

Update fusion doc and add ONNX build flag to buildlb doc #1585

Merged
merged 7 commits into from Sep 12, 2018

Conversation

indie
Copy link
Contributor

@indie indie commented Sep 10, 2018

No description provided.

doc/sphinx/source/buildlb.rst Outdated Show resolved Hide resolved
doc/sphinx/source/buildlb.rst Outdated Show resolved Hide resolved
doc/sphinx/source/fusion/index.rst Outdated Show resolved Hide resolved
Optimization passes can be programmed ahead of time if you know what your graph
will look like when it's ready to be executed, or the optimization passes can
be figured out manually with *Interpreter* mode on a stateless graph.
Optimization passes can be programmed ahead of time if you know or can predict
Copy link
Contributor

Choose a reason for hiding this comment

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

Optimizations can be experimented without using any backend by registering a pass with pass manager(Manager) and calling run_passes on a Function and then inspecting a transformed graph.
Interpreter isn't different from any other backend, so optimization strategies are hard-coded and can't be changed dynamically.

Copy link
Contributor

@Krovatkin Krovatkin left a comment

Choose a reason for hiding this comment

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

:shipit:

@indie indie removed the request for review from rkimballn1 September 10, 2018 23:19
@postrational postrational added the ONNX Focus on ONNX label Sep 11, 2018
@indie indie dismissed diyessi’s stale review September 11, 2018 17:56

resolved comments

@rkimballn1 rkimballn1 added the Fully Reviewed PR is approved by all reviewers. label Sep 12, 2018
@rkimballn1 rkimballn1 merged commit 2d0721d into master Sep 12, 2018
@rkimballn1 rkimballn1 deleted the doc_debugging branch September 12, 2018 14:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Fully Reviewed PR is approved by all reviewers. ONNX Focus on ONNX
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants