Add all-in-one wrappers. #182
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Waiting for #181 to be merged before fixing the notebooks and finalizing things. |
* Begin clean of docs, setup notebook linking. * Ignore superop autogen * Linked examples notebooks to module page for rb, dfe. * Add tomography notebooks and do_tomography. * Add examples and clean up warnings * Clean up classical logic docs. * Add progress bar to adder, include notebook in docs. * Fix compilation errors and typos. * Clean up doc errors in dfe. * Revert accidental template change. * Cleanup dm docstrings. * Clean up operator_tools docs. * Clean up op_est docs. * Finish resolving build warnings. * Notebook spellcheck and format fix. * Resolve last batch of warnings besides notebook interlinks. * Finish cleaning up and evaluating notebooks. * Fill out examples in quick start, shorten header title for obs_est. * Add do methods to docs. * Move examples to docs. * Move examples to docs and tweak the nb links. * Add a link to the examples folder. * Update example nb test link. * Remove reference to nblink
stylewarning
approved these changes
Aug 9, 2019
|
@kylegulshen Fix the tests. |
|
Also, can you rebase this? |
|
Waiting for tests, then I'll merge if they pass. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This has a few API / signature changes. The main contribution is to add a
do_xwrapper that does all three stages of the experiment together with default parameters. The args list and order is standardized for all of these routines, and the args for similar functions was made to match this.closes #178
closes #81