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 luminescent #524

Closed
wants to merge 6 commits into from
Closed

update luminescent #524

wants to merge 6 commits into from

Conversation

paulxshen
Copy link
Contributor

@paulxshen paulxshen commented Dec 30, 2024

notebook now installs julia on test machine. lmk if you can now run cells. thanks :)

Summary by Sourcery

Documentation:

  • Add luminescent notebook to the table of contents.

Copy link
Contributor

sourcery-ai bot commented Dec 30, 2024

Reviewer's Guide by Sourcery

This pull request updates the documentation to include a new notebook that installs Julia on the test machine and allows users to run luminescent cells.

Sequence diagram for Julia installation and notebook execution

sequenceDiagram
    participant U as User
    participant N as Notebook
    participant J as Julia Environment
    U->>N: Open luminescent notebook
    N->>J: Check Julia installation
    alt Julia not installed
        J-->>N: Installation required
        N->>J: Install Julia
        J-->>N: Installation complete
    end
    N->>J: Initialize Julia kernel
    J-->>N: Kernel ready
    U->>N: Run notebook cells
    N->>J: Execute Julia code
    J-->>N: Return results
    N-->>U: Display output
Loading

File-Level Changes

Change Details Files
Added a new notebook for installing Julia and running luminescent cells.
  • Added a new section to the table of contents that points to the new notebook file.
  • Included the notebook twice in the table of contents file
docs/_toc.yml

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time. You can also use
    this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@joamatab joamatab self-requested a review December 30, 2024 02:16
Copy link
Contributor

@joamatab joamatab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still does not work on my end, see docs failing in this PR

docs/_toc.yml Show resolved Hide resolved
@joamatab joamatab closed this Jan 15, 2025
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