Mentions légales du service

Skip to content
Snippets Groups Projects

Use CppyyKernel to run the notebooks instead of Xeus-cling.

Merged GILLES Sebastien requested to merge sgilles/gettingstartedwithmoderncpp:101_cppyy into master
All threads resolved!

The notebooks have been modified to go along this change; I tried as much as possible not to introduce changes in the actual content of the notebooks (minus some stylistic changes: initialization through braces replaced some initialization by parens, as the braces require the cppyy.cppdef magic the change seemed organic enough).

The main changes are:

  • Many references to Coliru have been suppressed and replaced with code run in a cell with clang magic.
  • Some cppyy related magics have been added for specific cells that require it.
  • The Docker image has been updated to reflect this.
  • Binder has been completely modified.
  • README and comments have been updated to suppress references to Xeus-cling and provide up-to-date information regarding installation.

Closes #101 (closed).

Merge request reports

Pipeline #1032925 passed

Pipeline passed for 551d5406 on sgilles:101_cppyy

Approved by

Merged by GILLES SebastienGILLES Sebastien 10 months ago (Sep 16, 2024 1:07pm UTC)

Merge details

Pipeline #1034967 passed

Pipeline passed for 9ca82e76 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • ROUVREAU Vincent
  • ROUVREAU Vincent
  • ROUVREAU Vincent
  • ROUVREAU Vincent
  • Ok if done on a separated PR, but changelog should also be updated

  • Thanks for the review! I'll answer your comment tomorrow :slight_smile:

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • ROUVREAU Vincent resolved all threads

    resolved all threads

  • ROUVREAU Vincent approved this merge request

    approved this merge request

  • mentioned in commit 9ca82e76

  • Please register or sign in to reply
    Loading