optimization of encodedBitmap_Bytes_withnumpy

[fookatchu]

I recently wrote a library to read/write Photon files for my SL1-to-Photon-Converter and largely used PhotonFile.py as a reference. Unfortunately the code is not very compatible any more, so I thought I would at least give something back, which is the optimized code for the rle encoding. In my tests the new function is roughly 10 times faster than the orignal numpy version. I did some tests and the resulting binarys are identitical. If you need any further changes, please let me know and I will try my best to implement them.

p.s.
This is my first pull request on github, so please excuse any mistakes.

[NardJ]

Nice job! I will integrate this in the next release. thx, Nard Op wo 24 apr. 2019 om 19:01 schreef fookatchu <notifications@github.com>:

…

I recently wrote a library to read/write Photon files for my SL1-to-Photon-Converter and largely used PhotonFile.py as a reference. Unfortunately the code is not very compatible any more, so I thought I would at least give something back, which is the optimized code for the rle encoding. In my tests the new function is roughly 10 times faster than the orignal numpy version. I did some tests and the resulting binarys are identitical. If you need any further changes, please let me know and I will try my best to implement them. p.s. This is my first pull request on github, so please excuse any mistakes. ------------------------------ You can view, comment on, or merge this pull request online at: #99 Commit Summary - optimization of encodedBitmap_Bytes_withnumpy File Changes - *M* PhotonFile.py <https://github.com/Photonsters/PhotonFileEditor/pull/99/files#diff-0> (26) Patch Links: - https://github.com/Photonsters/PhotonFileEditor/pull/99.patch - https://github.com/Photonsters/PhotonFileEditor/pull/99.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#99>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACXNXGFKAZYPSCHZ32APGXLPSCHAHANCNFSM4HIGECRA> .

[X3msnake]

@NardJ you forgot this :)