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

Fix example notebooks in section Parameter of Example of using Qcodes #1780

Merged
merged 19 commits into from
Oct 30, 2019

Conversation

lakhotiaharshit
Copy link
Contributor

Fixes the notebooks related to Parameters. Apart from fixing typing errors, the following important corrections are done:

  • make sure that all imports are proper
  • notebooks use plot_dataset and load_or_create_experiment instead of other functions,
  • suggest using ParameterWithSetpoints instead of ArrayParameters.
  • add links to other relevant notebooks wherever required
  • explain the meaning of setpoints

@astafan8 @GateBuilder

Copy link
Contributor

@astafan8 astafan8 left a comment

Choose a reason for hiding this comment

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

minor comments

docs/examples/Parameters/Complex_Parameters.ipynb Outdated Show resolved Hide resolved
docs/examples/Parameters/Parameters.ipynb Outdated Show resolved Hide resolved
docs/examples/Parameters/Parameters.ipynb Outdated Show resolved Hide resolved
docs/examples/Parameters/Parameters.ipynb Show resolved Hide resolved
docs/examples/Parameters/Parameters.ipynb Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Oct 22, 2019

Codecov Report

Merging #1780 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1780      +/-   ##
==========================================
+ Coverage   69.71%   69.71%   +<.01%     
==========================================
  Files         148      148              
  Lines       18572    18572              
==========================================
+ Hits        12947    12948       +1     
+ Misses       5625     5624       -1

@lakhotiaharshit lakhotiaharshit added the docs Related to docs improvements label Oct 23, 2019
Copy link
Collaborator

@jenshnielsen jenshnielsen left a comment

Choose a reason for hiding this comment

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

Looks good but please correct these links to be direct ipynb links

docs/examples/Parameters/Parameters.ipynb Outdated Show resolved Hide resolved
docs/examples/Parameters/Parameters.ipynb Outdated Show resolved Hide resolved
docs/examples/Parameters/Parameters.ipynb Outdated Show resolved Hide resolved
@jenshnielsen jenshnielsen merged commit a27cde5 into microsoft:master Oct 30, 2019
@lakhotiaharshit lakhotiaharshit deleted the update_notebook_w1 branch January 22, 2020 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Related to docs improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants