Actually, I believe that the problem is not in the number of samples, but a missing detail.
If you specify that you want to save 3 samples, robometry will save a file with three entries, even if you push 10 times. It will simply save the last three items. This behavior may not be properly specified in the README.
Moreover, if you enable the autosave option, it will create a new file every time you push 3 samples. Hence, when pushing 10, you should expect three files with 3 samples each, and 1 file with the last item (triggering the manual "save").
Also this might not be properly specified in the README.
By forcing the for loops to run fr three elements does not improve the clarity in this sense.
Another issue with the README is that in this line
|
bufferConfig.n_samples = n_samples; |
n_samples should be substituted with
3.
Originally posted by @S-Dafarra in #187 (comment)
If you specify that you want to save 3 samples,
robometrywill save a file with three entries, even if you push 10 times. It will simply save the last three items. This behavior may not be properly specified in the README.Moreover, if you enable the
autosaveoption, it will create a new file every time you push 3 samples. Hence, when pushing 10, you should expect three files with 3 samples each, and 1 file with the last item (triggering the manual "save").Also this might not be properly specified in the README.
By forcing the for loops to run fr three elements does not improve the clarity in this sense.
Another issue with the README is that in this line
robometry/README.md
Line 84 in 29646dc
n_samplesshould be substituted with3.Originally posted by @S-Dafarra in #187 (comment)