Skip to content

Polyscope 2.6#479

Merged
dcoeurjo merged 1 commit intomainfrom
poly2.6
Feb 27, 2026
Merged

Polyscope 2.6#479
dcoeurjo merged 1 commit intomainfrom
poly2.6

Conversation

@dcoeurjo
Copy link
Member

PR Description

Also removing the local polyscope and use the version shipped with DGtal when DGTAL_WITH_POLYSCOPE_VIEWER is set. The doc should probably be updated.

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).

@dcoeurjo
Copy link
Member Author

At this point, it works for polyscope2.5 in DGTal (main) but not the 2.6 PR

@dcoeurjo
Copy link
Member Author

Hi @kerautret I am merging this one to unlock DGtal GitHub actions. We will fix the doc later

@dcoeurjo dcoeurjo merged commit b800e13 into main Feb 27, 2026
4 checks passed
@kerautret
Copy link
Member

Thanks @dcoeurjo that noticed

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