biodiversitydata-se / biologging-sensor-client

SBDI Biologging Sensor project - Client
GNU General Public License v3.0
2 stars 0 forks source link

Maps : add legends #92

Closed mathieuLU closed 1 week ago

mathieuLU commented 4 months ago

from jira https://biologging-data-lu.atlassian.net/jira/software/projects/BIOSENSOR/boards/1/backlog?selectedIssue=BIOSENSOR-127

To add a legend

mathieuLU commented 3 months ago

ok so i need more details about how to display these information. Do we need a generic text block ? Displayed for all the maps ? do we want it to be visible all the time ?

Some information are common to all the maps display :

And then about the legend :

Screenshot from 2024-08-08 16-21-42

mathieuLU commented 3 months ago

format to be determined for the mouseover event. Is it something we want only for the map or anywhere ? and which format ? here is a first suggestion for the map mouseover Screenshot from 2024-08-08 16-59-37

DeboraArlt commented 2 months ago

for det legend of things that are always the same I guess we need what you call a generic text block, e.g.: Map displays a maximum of 5 tracks, each track is from one animal with a maximum of 20 position points displayed. (here comes a follow up question: which tracks? which points? who/what decides which tracks are displayed, and which points (which part of a track?)? maybe don't need to explain in legend but would be good to know if someone asks) [print black circle] last position (on the WRAM pages for example it shows bild

At the moment the figure headers are not very user friendly. I think they display the sensor type or similar, e.g. the map has title "Geographic spherical coordinate system" - I think it would be clerar with a specific image title (or use as subtitle) like e.g. "animal tracks" - with "animal tracks" in the title we do not any specific legend text. same gors for e.g. the temperature diagram - has now title "thermometer" but more user friendly would be something like "body temperature"? may have to carefully select the title text so it is representative for different types of thermometer sensor data.

For the colours of the different tracks - yes, have the same kind of legend as for altimeter/thermometer. Ideally use the same colours for a track and as the temperature lines if this is data from the same animals (I assume it is).

map mouseover positions: looks good. BUT - is it possible to add also the date and time for the position? Similar to the temperature diagram: there it is date and time and animal now that is dipslayed at map mouseover, ideally would also be nice to see the exact temperature value.

Let's look at this together during the next meeting.

mathieuLU commented 2 months ago

same comment here as what was decided here same comment as here https://github.com/biodiversitydata-se/biologging-sensor-client/issues/93#issuecomment-2334221020

of course the mouse over for the map can display date+time !

mathieuLU commented 1 month ago

status of the discussion regarding the maps

Screenshot from 2024-09-13 17-49-38 Screenshot from 2024-09-13 17-49-43

mathieuLU commented 1 month ago

OMG what a mess it was only to change this tooltip for the map... quite tough to deal with this Coordinate object and to send extra info to the map (tag id, date, ..) https://github.com/biodiversitydata-se/biologging-sensor-client/commit/1c5156e6f39add0c841fdea4cb7229899f3d7039 and https://github.com/biodiversitydata-se/biologging-sensor-client/commit/ada0965486ae501006661b87a0e1662f9d586fe3 and

looks like we have something ok now Screenshot from 2024-09-26 14-25-10 Screenshot from 2024-09-26 14-25-32

the legend is still missing

DeboraArlt commented 1 month ago

looks good enough! very acceptable! just the legend now ;-)

mathieuLU commented 1 month ago

ok so we now have an embedded legend box. last commit => https://github.com/biodiversitydata-se/biologging-sensor-client/commit/9d0188d7a82256bf94a9553348ae8290a08a2895

it works well to have the text "scientific name + tagID" for the moose project. But unfortunately it does not work for the tracking radar. Since there is no tagID attached to the animal. See the screenshots below

Moose : Screenshot from 2024-10-02 11-17-05

Tracking radar : Screenshot from 2024-10-02 11-17-16

Since i have the same text without tagID, the legend does not know how to solve it and displays only the first one. Several options :

DeboraArlt commented 1 month ago

I vote for option 1: attaching the animaID instead of the tagID when no unique tagID

mathieuLU commented 1 month ago

done Screenshot from 2024-10-08 15-01-53

DeboraArlt commented 1 month ago

need prabably also to change in the text explaining the graph to "animal ID" replacing the "tag ID"

mathieuLU commented 4 weeks ago

fixed as well with https://github.com/biodiversitydata-se/biologging-sensor-client/commit/68863d19545abc5806d662bb67089838d593c974

mathieuLU commented 3 weeks ago

new legend left-aligned (as well for the embedded) => https://github.com/biodiversitydata-se/biologging-sensor-client/commit/421ad4909486c1c89d3236336da5d8e5405aa39f

Screenshot from 2024-10-11 14-47-38

mathieuLU commented 1 week ago

Åke made me realize that we have only one legend for the maps, when you suggested 2 different texts. So right now I always display the text you suggested for the tracking radar, even for the moose : https://biologging.biodiversitydata.se/visualisation/dataset_wram_moose_2003

Since this is the same tool, i think it's easier to keep the same text all the time. Is it ok if we use all the time the text you wrote for the moose tracking data ?

DeboraArlt commented 1 week ago

I see. We missed that there should be a common text for the maps independent of sensor type. So we need a general text that can cover different sensor types. Right now the texts that I suggested include bits that are specific and mention "radar" pr "GPS".

suggestion for a new general text, replacing "radar" with "sensor": Movement data for a random selection of 5 animals (animal IDs in legend). Each trajectory displays the first 50 positions derived from sensor measurements, starting at black circle (sensor deployment). Use mouse over a position to read GPS coordinates, animal ID, date/time.

question - is it the same number of positions (50) we display for GPS and radar? Or can you make this element dynamic?

mathieuLU commented 1 week ago

yes we display the same number of position

of course we can configure a different sensor configuration for each if needed.

yes right now it's the same number of positions for any map item

DeboraArlt commented 1 week ago

and regarding the display of data for the data points in the map: I realise that the decimals shown for Lat/Lon seem to be the same as what is contained in the database. Is there an easy way to limit the number of decimals shown in the labels, to lets say 4 or decimals?

This is how it looks for snipe now: bild =5 decimals, looks ok

moose: bild =7decimals, ok-ish, starts to get a bit much

radar: bild =10+ decimals, hard to read

This is more for readability, simlar as the suggestion to change to a more readable date-time format. can be done after launch if time is limiting.

mathieuLU commented 1 week ago

yes ! fixed with https://github.com/biodiversitydata-se/biologging-sensor-client/commit/76b6554f1cb4a39f0abc9b2680a57e834147a30e and https://github.com/biodiversitydata-se/biologging-sensor-client/commit/913bfdf841764cfae77a030c0edf624a71be1817

DeboraArlt commented 1 week ago

let's keep it simple for now, the text works for any "sensor" :) We can always get more dynamic later ;-)

nice decimal fix :)

can be closed :)