Skip to content

Handle partial STB data selections in averaging UIs - #38

Draft
ronangiss wants to merge 88 commits into
mainfrom
Calimero
Draft

Handle partial STB data selections in averaging UIs#38
ronangiss wants to merge 88 commits into
mainfrom
Calimero

Conversation

@ronangiss

Copy link
Copy Markdown
Collaborator

Summary

  • Make phaseAvg_UI and timeAvg_UI derive available Calimero distance/type options from files present on disk.
  • Allow non-Calimero types such as turbine to appear as selectable type labels instead of requiring all legacy downstream Calimero types.
  • Refresh dependent frequency/amplitude dropdowns from the selected type so partial datasets remain selectable.

Testing

  • git diff --check HEAD~1 HEAD
  • python3 static selection-mapping validation for turbine-only, single-Calimero, and mixed datasets

Note: MATLAB/Octave is not installed in the container, so constructor/UI runtime validation could not be executed here.

Open in Web Open in Cursor 

ronangiss and others added 30 commits June 4, 2026 11:29
using measured freq for time phasing rather than the desired freq
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
phase avg via measured freq rather than desired freq had some lingering bugs to fix, working now
* Add shift mode to time average UI

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Skip zero frequency shifts in time average UI

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
* Add shift mode to time average UI

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Skip zero frequency shifts in time average UI

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Apply convection shift before time average alignment

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
* Add time average x-axis selector

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Clean up time average selector state

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Tidy time average plot branch formatting

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Remove time average trailing whitespace

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
* Add time-average files to flow field UI

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Lock plot types for selected time-average files

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
* Add RK4 particle tracking visualization

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Simplify RK4 particle validity check

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Document trilinear interpolation helpers

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
* Add RK4 particle tracking visualization

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Simplify RK4 particle validity check

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Document trilinear interpolation helpers

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Remove unused FTLE plot env override

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
ronangiss and others added 30 commits June 23, 2026 09:18
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
These are some changes that I have made
* Add selectable benchtop force plot source

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

* Support all benchtop force plot arrays

Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: ronangiss <ronangiss@users.noreply.github.com>
This code works now without throwing an error when running. It does not substantially improve the control (makes it a little worse actually), perhaps because it replays the OF commands in time rather than corresponding to motor position
I realized during these tests that my dmc file was erroring out quietly so it had never been implementing the more advanced control I thought I was implementing.
plot bin distribution of laser pulses immediately and cleaning of laser pulse code
No errors thrown and error reduced compared to baseline but not better than P or D alone
added handle for rising/falling edge since not all wheels have magnet oriented same way
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants