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.

Sign up for GitHub

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

Jump to bottom

Version of Matlab #112

Closed
Alkohost opened this issue Sep 13, 2019 · 4 comments · Fixed by #595
Closed

Version of Matlab #112

Alkohost opened this issue Sep 13, 2019 · 4 comments · Fixed by #595

Comments

@Alkohost
Copy link

Hello.
You wrote in requirements:
Required toolboxes: parallel computing toolbox, signal processing toolbox, MATLAB >=R2016b

But it seems only since version 2017 actions like dataRAW = dataRAW - mean(dataRAW, 1); are acceptable.

In 2016a is not working.
@Alkohost
Copy link
Author

Sorry I just found this #74
Can you please note this in the ReadMe?

@marius10p
Copy link
Contributor

Thanks for pointing that out. It wouldn't be too difficult to replace all the broadcast instances with the previous Matlab equivalent (bsxfun), but it's probably best to update to a recent Matlab version anyway.

@marius10p
Copy link
Contributor

Actually, it says here that broadcasting should work in 2016b.

@Alkohost
Copy link
Author

Yes you are right. The limit is between 2016a and 2016b.
So the current description is correct.

@carsen-stringer carsen-stringer mentioned this issue Feb 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

adding rtd and badges MouseLand/Kilosort
2 participants
@marius10p @Alkohost