Skip to content

Conversation

@aacuevas
Copy link
Contributor

@aacuevas aacuevas commented Oct 2, 2025

Requires clroni compiled with net8 support
@aacuevas aacuevas requested a review from jonnew October 2, 2025 15:43
@jonnew
Copy link
Member

jonnew commented Nov 6, 2025

One thing I noticed. If I have a context opened in some other software (e.g. Bonsai or oni-repl) and attempt to program I get this exception:

image

Nothing crashes or anything, but perhaps the exception text could be a bit clearer.

@aacuevas
Copy link
Contributor Author

aacuevas commented Nov 7, 2025

Fixed. Now it passes the inner oni exception through.
Which when using the current liboni branch with support for NET8 is the all too familiar "Already triggered operation, but that will change to the new we made once liboni is properly merged.

@jonnew
Copy link
Member

jonnew commented Nov 7, 2025

Another non-breaking but odd behavior:

If a connect to a headstage by pressing the "program" button on the main form, the little modal GUI pops up with an hourglass indicator. The fields then populate with the detected headstage metadata. Great.

The odd behavior is that if the mouse is not moved during this time period, the "program" and "Cancel" buttons will remain greyed out and the hourglass indicator will remain forever, until the mouse is touched.

@aacuevas
Copy link
Contributor Author

aacuevas commented Nov 7, 2025

I'll look into that but I'm not sure that is something we have control over or it's a windows forms quirk

Copy link
Member

@cjsha cjsha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've got some questions/nit picks after writing some documentation:


image

Is it true that the highlighted fields only show minor version i.e. this corresponds to v0.4.0? Would the a headstage have a major version update and would that information be indicated here?


image

Since device is a specific term in the context of ONIX, I feel that the highlighted part should be labelled "Connected Hub:" or something?


Is this software versioned? The other controller & breakout board updaters are built, zipped, and hosted in the onix-docs repo, with their version indicated in the zip file name. I figured I would follow the same pattern for this software for now.

@aacuevas
Copy link
Contributor Author

Ah, good catches.
We were testing functionality and I forgot about these things. I will add functionality so the text versions are converted from number to proper version numbers
Same as the label name, thanks!

It is not versioned yet, but I will add some version number somewhere, that's a good idea

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.

Create a C# GUI programmer for hubs

4 participants