-
Notifications
You must be signed in to change notification settings - Fork 559
docs: Add documentation for integrated vectorization. #834
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed can we add a section about the inability to switch between push and pull.
This is great. Just a couple of things we can add
Also, shall we hold on this merge till the time all IV changes are pushed in? |
Looks great! Once the IV changes are in, I'd be happy to go through the docs/steps to test it out as a user fresh to it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
(`-,-,
('(_,( )
_ `_'
__|_|__|_|_
_|___________|__
|o o o o o o o o/
~'`~'`~'`~'`~'`~'`~
Co-authored-by: frtibble <32080479+frtibble@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Purpose
This PR adds a section to the README on Integrated Vectorization that links to a page that describes how to deploy it.
Required by #761.
Does this introduce a breaking change?
Pull Request Type
What kind of change does this Pull Request introduce?
How to Test
n/a