Skip to content

Add getXxxPoints() Q_INVOKABLE methods to MViewStats and MSfMDataStats#95

Draft
Copilot wants to merge 2 commits intodevelopfrom
copilot/add-getxxxpoints-methods
Draft

Add getXxxPoints() Q_INVOKABLE methods to MViewStats and MSfMDataStats#95
Copilot wants to merge 2 commits intodevelopfrom
copilot/add-getxxxpoints-methods

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 20, 2026

  • Remove fillXxxSerie(QXYSeries*) method declarations from MViewStats.hpp and MSfMDataStats.hpp
  • Remove #include <QtCharts/QLineSeries> from MViewStats.hpp
  • Remove #include <QtCharts/QBoxSet> and #include <QtCharts/QLineSeries> from MSfMDataStats.hpp
  • Remove fillXxxSerie() implementations from MViewStats.cpp (and the now-unused QPen/QSGGeometryNode includes)
  • Remove fillXxxSerie() implementations from MSfMDataStats.cpp
  • Remove find_package(Qt6Charts REQUIRED) from root CMakeLists.txt
  • Remove find_package(Qt6 COMPONENTS Charts REQUIRED) and Qt6::Charts from src/qtAliceVision/CMakeLists.txt
  • Code review: no comments
  • CodeQL: 0 alerts

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: fabiencastan <153585+fabiencastan@users.noreply.github.com>
Agent-Logs-Url: https://github.com/alicevision/QtAliceVision/sessions/83f52c5f-b0f5-4978-b8b8-5d7845182888
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