Qt6 port + reproducible Windows build/installer + config & UX enhancements#80
Closed
fnsangiul wants to merge 1 commit into
Closed
Qt6 port + reproducible Windows build/installer + config & UX enhancements#80fnsangiul wants to merge 1 commit into
fnsangiul wants to merge 1 commit into
Conversation
…X enhancements" This reverts commit 492cf4b.
Contributor
|
ah, i see we reverted and reopened, left comments on the last one |
Member
Author
|
@sneakers-the-rat yes, my mistake. I merged by mistake and reverted to create a PR on a new branch. |
Contributor
|
totally understandable - the reason we use branch protections is because those kinda of mistakes are easy to make, common problem, not just a you thing :) |
Member
Author
|
Closing this draft in favor of a fresh PR that includes the Linux/Ubuntu port (libuvc capture backend, V4L2 device scanning, AppImage packaging + CI, udev rules) on top of the Qt6 migration and Windows build work. New PR opening now. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR ports the Miniscope DAQ QT software from Qt5 to Qt6, sets up a reproducible Windows build with a one-click installer, and adds a set of usability features on top. It's ~40 commits adding 2,135 / removing 399 lines across 44 files.
The work is grouped into four areas below. Each area was developed and merged on its own feature branch, so the history is easy to follow commit-by-commit.
1. Qt5 → Qt6 migration
The application now builds and runs on Qt6.
CMakeLists.txt) replacing the old setup.videodisplay,tracedisplay, andbehaviortracker.f-suffix literals,float/intcomparisons) — these were silently breaking trace plotting and the tracker overlay.QtQuickimports, null-traceDisplaycrash on Run, and a forced light color scheme fix for invisible white-on-white text.initNumpywarning (C4715) silenced.2. Reproducible build, packaging & release
environment.yml— a conda spec pinning the toolchain (Qt, OpenCV, Python) so the build is reproducible from scratch.tools/deploy.py+tools/launcher.cpp— produce a portable, self-contained folder: a double-clickableMiniscopeDAQ.exeat the top with all DLLs tucked away inbin/.installer/MiniscopeDAQ.iss, Inno Setup): per-user install (no admin rights), Start Menu / optional desktop shortcut, and clean uninstall via Add or Remove Programs..github/workflows/release.yml): on tag, builds and publishes both the portable zip and theSetup.exe.3. New features & UX
deviceID → camera nameso you can pick the right index.UserConfigExample-*.jsonsamples added (V3 / V4, single & dual webcam).4. Bug fixes
test.avibehind.Notes for review
qt6-portintegration branch (notmaster), so the work can be reviewed and integrated without landing directly onmaster.