Skip to content

Various Doc update following 2.0 change #476

Merged
kerautret merged 29 commits intoDGtal-team:masterfrom
kerautret:post2.1
Dec 29, 2025
Merged

Various Doc update following 2.0 change #476
kerautret merged 29 commits intoDGtal-team:masterfrom
kerautret:post2.1

Conversation

@kerautret
Copy link
Member

@kerautret kerautret commented Dec 27, 2025

PR Description

Following the new display introduced from polyscope viewer, the illustrations of tools in the documentations was obsolete.
It also add some fix when reproducing examples.
Some links were also broken from the new organization.

By the way the tools update is also given :DGtal-team/dgtal.github.io#26

Checklist

  • Doxygen documentation of the code completed (classes, methods, types, members...).
  • Main tool doxygen documentation (following existing documentation of DGtalTools documentation.
  • Check if it follows the tools structure described in CONTRIBUTING.md
  • New entry in the ChangeLog.md added.
  • Update the readme with potentially a screenshot of the tools if it applies.
  • No warning raised in Debug cmake mode (otherwise, Github Actions C.I. will fail).
  • All continuous integration tests pass (Github Actions).

@kerautret kerautret requested a review from dcoeurjo December 27, 2025 21:16
Copy link
Member

@dcoeurjo dcoeurjo left a comment

Choose a reason for hiding this comment

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

whaou, impressed by the fact that you rerender all snaphots... 👏

@dcoeurjo
Copy link
Member

thanks for the great PR

@kerautret
Copy link
Member Author

whaou, impressed by the fact that you rerender all snaphots... 👏

Thank you, yes, the old screenshots were starting to look outdated from QT 🤣 looks more modern with polyscope ;)

@kerautret kerautret merged commit 619a8c5 into DGtal-team:master Dec 29, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants