Skip to content

Argos collector (map_v3)#185

Open
colby-nyce wants to merge 339 commits into
mainfrom
colby-nyce/collector-v3
Open

Argos collector (map_v3)#185
colby-nyce wants to merge 339 commits into
mainfrom
colby-nyce/collector-v3

Conversation

@colby-nyce
Copy link
Copy Markdown
Collaborator

@colby-nyce colby-nyce commented May 19, 2026

@colbynyce-mips
Copy link
Copy Markdown
Collaborator

The UI performance (especially Scheduling Lines widget) has been greatly improved and the most annoying bugs have been fixed. I think these changes are ready for the initial map_v3 release.

Subsequent releases will come later, roughly in this order:

  • Move Sparta pair collector / pair definition / macros to SimDB
  • Refactor Sparta Collectable / IterableCollector classes so the only thing left in Sparta is timing (e.g. DelayedCollectable, collectWithDuration, etc.) and SimDB gets the nuts and bolts (Pair code, BitBucket, constexpr helpers, container collection logic, etc.)
  • Enable minifiers to save lots of disk space (we are doing full byte dumps every time now, but it is easiest to re-enable minifiers after the Sparta/SimDB merged code above)
  • Delete all old Argos code / doc
  • Write up new Argos doc
  • Backend performance (BitBucket logic, how we collect enums, and maybe some other things are not maximally performant, but it's much easier to wait for Sparta/SimDB code to merge first)

Some of the python UI code was written by Cursor, and while I don't love some of it, we'll likely just accept it as-is unless we decide wxPython is the way forward (might get rewritten to Qt). This is all Cursor code:

  • AuiNotebook / inspector.py changes (fixes some UI bugs)
  • dtype_inspector.py is entirely from Cursor and likely contains "cruft" leftover from previous designs (not a fan of this code, but also don't want to change it since it's working fine)

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