Repository navigation
Fix open issues #205–#212 - #213
Merged
Merged
Conversation
…overlay (#209) Velocity stays: it is mean step length over the trail, and the analysis side reads velocity_px_frame. The behavioral_state column is gone from new tracking CSVs; readers already treat it as optional, so old files still load.
…load (#205) refresh_plugin_nodes() ran once, when the panel was built, so a node a freshly installed plugin registered stayed off the list until a restart.
The card's Install button installs the requirements GLIDER's own metadata lists for the extra (not glider[extra] from PyPI, a different project), then offers a restart, since the availability probes and tool stacks load once.
…ight (#212) An ellipse is the affine image of a circle, so a round arena reuses the homography: its corners are the ellipse's bounding parallelogram, and only containment (the gate) and the centre zone know it is round. The dialog gains a Rectangle/Circle choice and separate width and height; reopening an arena now keeps its height.
…p retrieval (#212) Camera Settings > Arena… outlines the live camera's floor and is remembered between sessions; ArenaCalibration carries the scalar interface the tracking logger read off a line calibration, so the CSV headers and distance_mm are unchanged. Set scale… in pup retrieval opens the same dialog. Scale borrowing and the spread check now see arena-only calibration files. The line-drawing dialog is gone; CameraCalibration stays so old master files still load.
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.
Fixes all eight open issues, one commit per issue (four for #212).
Closes #205
Closes #206
Closes #207
Closes #208
Closes #209
Closes #210
Closes #211
Closes #212
Changes
plugins_changedafter an install, enable or reload, and the main window rebuilds the library on it.installer.extra_requirements), notglider[extra], which would resolve the unrelatedgliderproject on PyPI. Then it offers a restart, because the availability probes are memoized. The relaunch runs at interpreter exit, so serial ports are released first. Frozen builds get no button.BehaviorAnalyzer, its overlay, its settings and thebehavioral_statecolumn from new tracking CSVs.velocity_px_frameis kept, now computed as the mean step length over the trail. Analysis already treatsbehavioral_stateas optional, so old recordings still load.ArenaCalibrationgainsshape="ellipse". An ellipse is the affine image of a circle, so the existing homography is reused: the corners are the ellipse's bounding parallelogram. Only containment (outside_cm, used by the arena gate) and the centre zone know the arena is round.ArenaCalibrationprovides the scalar interface the logger reads, so the CSV headers anddistance_mmare unchanged._borrow_from_folder_mateandcalibration_spread_warningnow also read arenas.CalibrationDialogis deleted.CameraCalibrationstays, so old master files still load.Docs are updated for the Builder window, tracking, the camera arena section and pup retrieval.
Testing
QT_QPA_PLATFORM=offscreen uv run --no-sync pytest -q -m "not slow"gives 6674 passed, 4 skipped, 7 deselected. That is below the pre-Arena calibration #212 run on this branch (6696) because the deleted calibration dialog's test file and the seven line-copy tests went with it.ruff checkandblack --checkare clean onsrc tests plugins tools.