Skip to content

Conversation

cjsha
Copy link
Member

@cjsha cjsha commented May 26, 2025

This is regarding #38, but this PR won't close the issue because there will likely be updates.

@cjsha cjsha requested a review from ChucklesOnGitHub May 26, 2025 22:04
@ChucklesOnGitHub
Copy link
Member

Before merging this PR, please merge open-ephys/bonsai-miniscope#37 otherwise ppl can't access the full GUI functionality (df/f)

Copy link
Member

@ChucklesOnGitHub ChucklesOnGitHub 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 made some suggestions for clarity for users, lmk if there's any you don't understand/wanna include. I called it an example GUI to make the point that it's one way of doing things.

Could you make the default filenames sth more descriptive than adaqdsfds.csv and c.avi?

Also, add links regarding subjects in Bonsai
- Make it easier to change FourCC and set to Y800
- provide more descriptive file names
- Save quaternion data as well as IO data
- file overwrite: false, file suffix: filenumber

- Also, add note in tutorial on how to change FourCC
@cjsha cjsha marked this pull request as draft May 27, 2025 17:06
@cjsha
Copy link
Member Author

cjsha commented May 27, 2025

Drafted until DF/F node is released.

@cjsha cjsha requested a review from aacuevas July 13, 2025 02:53
@cjsha
Copy link
Member Author

cjsha commented Jul 13, 2025

@aacuevas I requested your review to check the gui workflow and make sure it's sensible. Also, feel free to add the bobblehead 3d tracking visualizer. A nice UCLA Miniscope V4 STEP file is available here.

@ChucklesOnGitHub
Copy link
Member

Since the df/f is merged, can publish a first version of this GUI @cjsha ? We can add the boblehead later. We've started working on providing models for it.

@ChucklesOnGitHub
Copy link
Member

Include improvements done by cjsha in #52 (comment)

@ChucklesOnGitHub
Copy link
Member

User just mentioned that having a live estimated frame rate on screen helps monitor frame loss.

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.

2 participants