-
Couldn't load subscription status.
- Fork 22
Select formats based on index #2
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
Conversation
|
Does this also do what sccn/xdf#38 proposes? If not, it would be nice if you could integrate that into this PR. |
Yes, I didn't see that but it does everything sccn/xdf#38 does (and a tiny bit more). Plus, it's already in the correct repository :-) |
|
OK great, that's what I wanted to know - so we can safely close sccn/xdf#38 once this gets merged. |
| values[k, :] = struct.unpack(temp[StreamId].structfmt, raw) | ||
| # no fromfile(), see | ||
| # https://github.com/numpy/numpy/issues/13319 | ||
| values[k, :] = np.frombuffer(raw, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
frombuffer doesn't need a character format string and is about 30% faster in one of my tests
|
We really need to set up a CI service. I don't have the time to verify if these changes (especially the switch from |
|
I created an example file repository with a tiny (2kB) example file. It's not a replacement for a CI, but the most basic test ( |
As a precursor for the chunk7 reader, use the same format ordering as in lsl and read in the native datatype (line 293)