Closed savinoda closed 7 years ago
I removed the conflicts with develop so that this branch can safely be merged. 👍 But I haven't spend time to test it or review it, so I won't do the merge myself. If Romaric can't, I will take a look at it next week.
I will probably have a look at that on monday morning. Thanks for the modification Giuseppe !
Le jeu. 13 juil. 2017 à 16:16, wasp898 notifications@github.com a écrit :
I removed the conflicts with develop so that this branch can safely be merged. 👍 But I haven't spend time to test it or review it, so I won't do the merge myself. If Romaric can't, I will take a look at it next week.
— You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub https://github.com/UCA4SVR/TOUCAN-VR/pull/77#issuecomment-315091106, or mute the thread https://github.com/notifications/unsubscribe-auth/AAYJWZNC2qN1kJSy0pgXrdIYAcKVYXyCks5sNibPgaJpZM4OWn5o .
This feature can be split in two parts:
Logging of the quality that each segment of each tile has when is removed from the buffer by the decoder. This will result in logging the quality that the user will see for that segment of that tile.
Logging of the picked tiles: when the function onPick() of the picker is called, a record in the log file is created. It contains the system time and a number of columns equals the number of tiles with a boolean value inside: True means the tile is picked in that moment, False otherwise. This second logger may have problems due to the fact that it's included in a singleton class: the picker is not a new object when a new intent arrives but we need a fresh logger. I implemented a refreshing strategy and I tested it with different videos: it seems to work properly but please check it when you review my code.