-
Notifications
You must be signed in to change notification settings - Fork 49
Description
Hi,
I wanted to prepare a pull request for integrating Temporal Binary Representation in Tonic, when I discovered BinaRep among your event encoding methods: https://tonic.readthedocs.io/en/latest/auto_examples/representations/plot_tobinarep.html#sphx-glr-auto-examples-representations-plot-tobinarep-py
Temporal Binary Representation is an encoding strategy that I published in 2020 in the proceedings of the 25th International Conference on Pattern Recognition (ICPR): https://www.computer.org/csdl/proceedings-article/icpr/2021/09412991/1tmhW5YGUSY
A preprint of the paper was also made available in October 2020: https://arxiv.org/pdf/2010.08946.pdf
The main idea of the paper was to collect binary frames with an arbitrarily small accumulation time and then merge N consecutive binary frames into a single frame just by performing a binary-to-decimal conversion. In this way, we convert a sequence of binary values for a certain pixel into a single decimal integer describing its temporal evolution.
The BinaRep paper, was published in 2022 at the International Conference on Image Processing, that is two years after TBR was published:
https://ieeexplore.ieee.org/document/9898061
The thing is that BinaRep proposed exactly the same approach as TBR.
The similarity can be easily seen by comparing a few lines of code in our implementation and the one integrated here in Tonic.
Original implementation of our approach:
https://github.com/fedebecat/tbr-event-object-detection/blob/master/src/tbe.py#L29
Implementation of BinaRep in Tonic:
tonic/tonic/functional/to_bina_rep.py
Line 61 in debdd2e
return np.sum(mask * frames, 0) / (2 ** mask.shape[0] - 1) |
I have already informed IEEE, which hosts the proceedings of both papers. I would therefore ask you to remove references to the plagiarizing paper from the Tonic website or at least reference the correct paper (TBR) for the method instead of BinaRep.
Thank you for your understanding.
Federico