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

thebe #267

Merged
merged 1 commit into from
Apr 17, 2024
Merged

thebe #267

merged 1 commit into from
Apr 17, 2024

Conversation

rkdarst
Copy link
Member

@rkdarst rkdarst commented Nov 5, 2023

  • content/pandas: Basic revision, no major changes.
  • Thebe: demonstrate Thebe to run code inline

@rkdarst
Copy link
Member Author

rkdarst commented Nov 5, 2023

@bast
Copy link
Collaborator

bast commented Nov 6, 2023

Seems to work well.

@bast
Copy link
Collaborator

bast commented Apr 17, 2024

Is this ready to go?

- Don't merge: this is a demo.
  - This includes the Pandas pull request, but is unrelated.  Diff
    will look better once that is merged.
- A "Launch" button appears at the top
- Then, it launches a kernel on mybinder.org in the default jupyter
  data sci repo (todo: does this have everything this course needs?).
- Then, you can click "run" on any cell and see the output (just like
  Jupyter would look), also edit the code and re-run.
- Formatting is a bit off (line spacing is too much, probably since
  this uses sphinx_rtd_theme).
- I don't think we should use this for 2023 (not unless there is more
  testing and maybe an own mybinder instance...), but hopefully it's a
  good demo.
@rkdarst rkdarst marked this pull request as ready for review April 17, 2024 17:52
@rkdarst
Copy link
Member Author

rkdarst commented Apr 17, 2024

I guess no reason not to? Can be used as an example and tested until next time...

@rkdarst rkdarst merged commit 188a462 into master Apr 17, 2024
4 checks passed
@rkdarst rkdarst deleted the thebe branch April 17, 2024 17:53
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.

2 participants