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

Update Offline Plotting Tutorial with kwargs examples #1516

Conversation

WilliamHPNielsen
Copy link
Contributor

A user in Copenhagen asked me how to:

  • Set the colormap for a single call to plot_by_id
  • Set the coloraxis to a log scale in plot_by_id
  • Set the range for the colorscale in plot_by_id

This is all easily done by passing the appropriate kwargs to plot_by_id.

Changes proposed in this pull request:

  • Document how to do the three things mentioned above.

@QCoDeS/core

@jenshnielsen
Copy link
Collaborator

Looks good.

A few optional suggestions

  • Should the logscale be done on data that is always > 0
  • Should @Dominik-Vogel s autoscaling be referenced in the vmin/vmax example

@codecov
Copy link

codecov bot commented Mar 8, 2019

Codecov Report

Merging #1516 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1516   +/-   ##
=======================================
  Coverage   73.84%   73.84%           
=======================================
  Files          92       92           
  Lines       10459    10459           
=======================================
  Hits         7723     7723           
  Misses       2736     2736

@jenshnielsen
Copy link
Collaborator

jenshnielsen commented Mar 13, 2019

@WilliamHPNielsen I think you might need to increase the timeout for that notebook https://nbsphinx.readthedocs.io/en/0.4.2/timeout.html

Edit: I think that is wrong piplines was just slow for some reason and it timed out at the 60 min deadline

@jenshnielsen jenshnielsen merged commit f95029f into microsoft:master Mar 14, 2019
giulioungaretti pushed a commit that referenced this pull request Mar 14, 2019
Merge: 5482fb9 aaccc27
Author: Jens Hedegaard Nielsen <[email protected]>

    Merge pull request #1516 from WilliamHPNielsen/docs/extend_offline_plotting_tutorial
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants