Skip to content

Conversation

@albertomercurio
Copy link
Member

Checklist

Thank you for contributing to Tutorials for Quantum Toolbox in Julia! Please make sure you have finished the following tasks before opening the PR.

  • Please read Contributing to Quantum Toolbox in Julia.
  • The (last update) date were modified for new or updated tutorials.
  • For new tutorials, a Version Information section was added at the end and displays the output of versioninfo().
  • All tutorials were able to render locally by running: make render.

Request for a review after you have completed all the tasks. If you have not finished them all, you can also open a Draft Pull Request to let the others know this on-going work.

Description

The current version of the website shows very low quality plots, everyone with a different width. Here I set the vector graphics, in order to show high-quality plots, and I also set a default witdth, common to all plots.

I also changed the date of every file to last-modified, as I think it is a smarter way to keep track of dates.

@github-actions
Copy link

github-actions bot commented May 22, 2025

PR Preview Action v1.6.1

🚀 View preview at
https://qutip.github.io/qutip-julia-tutorials/pr-preview/pr-43/

Built to branch gh-pages at 2025-05-22 08:24 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ytdHuang
Copy link
Member

I remember that setting Date as last-modified is incorrect. Since it records the render date.

@albertomercurio
Copy link
Member Author

It seems that it is only a problem of the GitHub Action

quarto-dev/quarto-cli#5899

They suggest to add the following script to the action, in order to restore information of the last modification time from git commits

https://stackoverflow.com/a/55609950

I'm trying to implement it

@albertomercurio
Copy link
Member Author

Also, I don't think we need the cardinalby/export-env-action action, as Quarto looks automatically at the _environment file when rendering.

@albertomercurio albertomercurio requested a review from ytdHuang May 22, 2025 09:29
@ytdHuang ytdHuang merged commit e921df0 into main May 22, 2025
2 checks passed
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.

3 participants