Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"makepretty" is nonexistent #23

Closed
lukebun opened this issue Aug 18, 2023 · 2 comments
Closed

"makepretty" is nonexistent #23

lukebun opened this issue Aug 18, 2023 · 2 comments

Comments

@lukebun
Copy link

lukebun commented Aug 18, 2023

Hello,

Thank you for sharing this tool I think that it will be incredibly useful and I've already started to apply it to my own data. However, I ran into a few problems.

The first problem is that the function "makepretty" appears to be nonexistent. It's called in multiple files, but I do not believe it is declared. I was able to produce figures by commenting all "makepretty" lines out. I would appreciate it if these files could be included.

2 other minor problems: (1) the GUI at the bottom of "bc_qualityMetric_pipeline" seems to expect data for raw waveforms, but by default param.extractRaw in "bc_qualityParamValues" is set to 0 so these data don't exist. Changing the variable and rerunning solves the problem. (2) in "bc_qualityMetric_pipeline" the comment says that ephysMetaDir should have a ".meta" file but OpenEphys only outputs a ".oebin" file. Using the ".oebin" file instead works, but I had some confusion hunting for the nonexistent ".meta" file.

@Julie-Fabre
Copy link
Owner

Julie-Fabre commented Aug 19, 2023

Hi lukebun,

Thank you for your issue!

To answer your comments in order:

  1. Good catch for the makepretty function. It was in another repo of mine, and I've moved it to bombcell.
  2. Yes, you're right. I've changed the default to param.extractRaw=1, which is what it should be set to. I changed the GUI to accommodate for when param.extractRaw=0.
  3. Yes, good point. Bombcell accepts both .meta and .oebin meta files. I've updated the comment in the pipeline.

Let me know if you run into more problems,
Cheers

@Julie-Fabre
Copy link
Owner

FYI - Since this function is shared and used in several if my repos, I ending up moving the makepretty function to a different repo: prettify-matlab. It's now called prettify-plot and has extended "prettifying" capabilities.
Without this repo, bombcell will still run and will use the old makepretty function, but output a warning text.

Marking this issue as closed now. Thank you for comments! If you have more issues or comments, feel free to open it up again/ up a new one or email me.
Cheers

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

No branches or pull requests

2 participants