Conversation
|
I have tested it but I had to change a little thing ^^; BTW, always MemoryError, line 138 of EDIT : |
|
@psolyca I could look at a python generator that would stream maybe 1MB at a time, would that work? I can drop the type hinting, for python2 |
|
Also if you give me more details about the debugging output you're seeing and the changes you've had to made I can incorporate them in to an update to this PR. |
|
The review and the type hinting are the only modifications I have made.
After, it seems that the memory dump is the same as the other method. |
Are you passing |
* Formerly the data buffer was a list of integers. * This seems to more accurately reflect memory and is probably faster and more efficient. * Hopefully makes supporting 64-bit reads/writes easier * Format with `black` again.
* Make it simple to stream data to another program or write to a file.
* Otherwise there's a chance to mix the binary output with the debug output if piped to another program or redirected to a file.
* Some embedded platforms are still on Python 2.7 :(
* Clumsy but works.
0714c25 to
8f94a80
Compare
|
Yes I was using Last version is working as expected. |
|
Can you add something to the setup.py to explain which versions of python are supported? From there it appears that only 3.8 is supported. |
* Here we go.
f56a717 to
e58d998
Compare
* Flake8 meltsdown. Work around it.
e58d998 to
6cbf9bc
Compare
Yup, need to add some tests and CI via Github Actions since testing Python 2 is getting harder these days. My dev machine doesn't have Python 2 at all these days. @rhollen Are you running this on a sad Python 2 embedded device? |
I haven't tested yet, but I will be using python 3 |
This writes binary output to stdout to it can be piped to another program or to a file.
Closes #10
Quick demo: