allow lidarview to show multiple streams within titlebar and multiple bounding boxes within images#572
Conversation
…s comma-separated list), also display all bounding boxes reported between two consecutive images
|
I did a single test with |
| self.keyframes_id = names.index(keyframes_name) + 1 | ||
| if title_name is not None: | ||
| self.title_id = names.index(title_name) + 1 | ||
| title_list = title_name.split(",") |
There was a problem hiding this comment.
I was going to argue, that argparse can handle multiple arguments, but this modification seems to be reasonable, that I would accept even comma separated names of streams ...
|
I guess there were multiple 'rot' entries between the two consecutive images (view steps). Which of them would you like to be displayed? Or should we step by each log entry (you only see one 'rot' but the image will be the same for several steps)? Or would we attempt to aggregate/group such multiple entries (eg average)? Then the viewer would need to know the meaning of the stream so it can choose the proper aggregation function. |
|
well, you are right, that it is not obvious which one is correct. |
| if title is not None: | ||
| caption += ' (' + str(title) + ')' | ||
| for t in title: | ||
| caption += ' (' + str(t) + ')' |
There was a problem hiding this comment.
we could reduce the amount of brackets for the list, but ... that is just a detail, now separation is clear
2 participants
--title takes a comma-separated list, will display info from all listed streams within the titlebar
also, all bboxes reported between two consecutive images will be shown within the next image