Open nicktrem opened 8 months ago
As discussed today f2f with @GiulioRomualdi and @traversaro the remaining parts of this PR to be addressed are:
These points have all been addressed today. @GiulioRomualdi if you have any further questions feel free to ask me :smile:
I'm cleaning a bit the PR to move the blf deps optional so the feature can be ported to pip
Hi @nicktrem let me know when it's ready so I can start checking it :)
@nicktrem @GiulioRomualdi there is any update on this? Thanks!
It is good to go on my end @GiulioRomualdi let me know if you have any questions
According to @GiulioRomualdi he did some tests on the real robot, and he did not remember well, but when using the RT logger, something related to saving the data was not working.
That issue is on the BLF side, not the robot-log-visualizer side. I can investigate that separately, however, that issue is separate from this PR
This PR for the robot-log-visualizer is a smaller simpler version of the original PR for the robot-log-visualizer and just contains the bare-bone code needed to visualize data online. The point of breaking up the original PR was to simplify the merge process for the robot-log-visualizer.