hydrocode-de / integra-frontend

React frontend for INTEGRA
0 stars 1 forks source link

Abundant species #169

Closed mmaelicke closed 3 weeks ago

mmaelicke commented 3 weeks ago

closes #162

The visuals are not implemented yet, still need to check if the calculations are correct. It is not super easy to do the needed matrix manipulations in javascript. The insect bit-array is mapped to the species, and filtered by the currently planted tree species. This composes a matrix of insect abundance, given the used tree species. Then the column sums of a bitwise OR combination is the number of abundant insect species.

Can be merged, if correct, and the visuals are implemented.

Are there preferences, where to put this number?

github-actions[bot] commented 3 weeks ago

Visit the preview URL for this PR (updated for commit 849be4d):

https://integra-223bf--pr169-abundant-species-edzuczw2.web.app

(expires Wed, 19 Jun 2024 14:57:34 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: ccb6d271698eb667c2f8f1dbd17a4a78744476bc

noobla11 commented 3 weeks ago

We would like the title to be "Anzahl unterstützter Insektenarten" below the other two graphs.

Information pop-up text:

Dieser Wert bezeichnet die Anzahl an potentiell unterstützten Insektenarten, welche sich aus der in Ihrem System vorhandenen Baum- und Strauchartenzusammensetzung ergibt.

Nice would be an insect icon on the left and the number on the right, both centered in the tab. We will provide a picture soon.

mmaelicke commented 3 weeks ago

@noobla11 @karpaddel, I already made the plots even smaller, to fit a thrid component into the tab and added the layout as requested. I also added an icon, that is already part of the icon-set used in the application. Maybe you can use the current state of this branch as a basis for change requests.

karpaddel commented 3 weeks ago

Now the two graphs look very flat and wide, could they not be made a little slimmer?

Otherwise we like it:) as already discussed, colors might be changed again:)

mmaelicke commented 3 weeks ago

This is fine for now. If there are future changes needed in the width of the graphs, or the Card itself, we can open a new issue for that. I'll merge, to continue