Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Notebook updates #47

Merged
merged 1 commit into from
Dec 11, 2019
Merged

Notebook updates #47

merged 1 commit into from
Dec 11, 2019

Conversation

bmorcos
Copy link
Contributor

@bmorcos bmorcos commented Dec 11, 2019

Motivation and context:

  • Make speed plot look better for controlled oscillator
  • Add note that animation may take some time to build

How long should this take to review?

  • Quick (less than 40 lines changed or changes are straightforward)

Types of changes:

  • Non-code change (touches things like tests, documentation, build scripts)

Checklist:

  • I have read the CONTRIBUTING.rst document.
  • N/A I have updated the documentation accordingly.
  • I have included a changelog entry.
  • I have tested this with all supported devices.
  • N/A I have added tests to cover my changes.
  • N/A I have run the test suite locally and all tests passed.

- Make speed plot look better for controlled oscillator
- Add note that animation may take some time to build
@bmorcos bmorcos merged commit 3785c29 into master Dec 11, 2019
@bmorcos bmorcos deleted the nb-speed branch December 11, 2019 20:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants