-
Notifications
You must be signed in to change notification settings - Fork 3
Data view : Display tiny label with emoji #12
Comments
RE: need to map all the COCO classes to an emoji in the config.json |
yep, need to also update to the latest OpenMoji SVGs there is not config.json in this repo, we use the one from the main repo. Just need to complete the DISPLAY_CLASSES https://github.com/opendatacam/opendatacam/blob/master/config.json#L24 The coco classes: "person", "bicycle", "car", "motorcycle", "airplane", "bus", "train", "truck", "boat", "traffic light", |
Will do ... tomorrow. |
Ok. Mapped all COCO classes to hexcodes (and OpenMoji) ... see https://docs.google.com/spreadsheets/d/1PuwHMTkRK5TMgS-xrN99U6Wnv8CoFT8nmvJW5sOIiWg/edit?usp=sharing (it might be a good idea to link this doc somewhere in the readme, as it gives a nice overview, where?). What to do with the one e.g. parking meter, where there is no hexcode / no OpenMoji extra available? Maybe "missing font/char" https://openmoji.org/library/#search=missing&emoji=25A1? |
PS. the config.json might hold a small bug. Check motorcycle vs. motorbike. |
nice thanks ! working on including this for next beta... and yes I think "missing font/char" is ok... yes will add on the documentation here: https://github.com/opendatacam/opendatacam/blob/master/documentation/CONFIG.md#display-custom-classes |
got this implemented.. currently displaying the most counted classes, and if there is just one, display just one... makes things a bit clearer.. now I think should also add the label as the emoji sometimes isn't available + not necessary very clear |
Yes to tiny labels. In case the the counter detail view is big enough I would also add a hint in the title e.g. "{counter-name} (top 6 classes)" |
Right now we are displaying 6 mobility classes, but ideally would be better to just display the 6 most counted classes depending on the data recorded... I'm testing on my desk and would love to see some laptop / coffee / chair OpenMojis ;-)
TODO:
The text was updated successfully, but these errors were encountered: