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

Switch theme to export notebooks #1637

Closed
wants to merge 3 commits into from

Conversation

nathanshammah
Copy link
Member

Description

Close #1619.

License

  • I license this contribution under the terms of the GNU GPL, version 3 and grant Unitary Fund the right to provide additional permissions as described in section 7 of the GNU GPL, version 3.

Before opening the PR, please ensure you have completed the following where appropriate.

@github-actions
Copy link

Binder 👈 Launch a binder notebook on branch nathanshammah/mitiq/download-docs

@codecov
Copy link

codecov bot commented Dec 16, 2022

Codecov Report

Merging #1637 (570cedf) into master (971b9a4) will decrease coverage by 0.01%.
The diff coverage is n/a.

❗ Current head 570cedf differs from pull request most recent head 1b51284. Consider uploading reports for the commit 1b51284 to get more accurate results

@@            Coverage Diff             @@
##           master    #1637      +/-   ##
==========================================
- Coverage   98.32%   98.31%   -0.01%     
==========================================
  Files          68       65       -3     
  Lines        3277     3026     -251     
==========================================
- Hits         3222     2975     -247     
+ Misses         55       51       -4     
Impacted Files Coverage Δ
mitiq/executor/executor.py 98.03% <0.00%> (-0.04%) ⬇️
mitiq/rem/rem.py 100.00% <0.00%> (ø)
mitiq/__init__.py 100.00% <0.00%> (ø)
mitiq/pec/representations/optimal.py 97.29% <0.00%> (ø)
mitiq/rem/inverse_confusion_matrix.py 100.00% <0.00%> (ø)
mitiq/typing.py
mitiq/calibration/calibrator.py
mitiq/calibration/settings.py
mitiq/calibration/__init__.py
mitiq/_typing.py 100.00% <0.00%> (ø)
... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Misty-W Misty-W modified the milestone: 0.24.0 Jan 31, 2023
@natestemen
Copy link
Member

@nathanshammah if my memory serves correct, you had this working where the option to download was with the plain markdown file (instead of what would be ideal which is the ipynb file). If so, let's just do that for now, otherwise I fear this will be forgotten about, and I think having an md download is better than nothing!

WDYT?

@natestemen
Copy link
Member

@nathanshammah should we close this?

@natestemen natestemen closed this Jul 26, 2023
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.

Add download option button to tutorials in docs
3 participants