Skip to content
Snippets Groups Projects

Apply feedback from regroup

Merged Vaarre-Lamoureux, Kaysa S requested to merge kvl-apply-feedback into main
All threads resolved!

Context

This MR aims to apply some of the feedback received from the Findex regroup meeting. This feedback pertains specifically to the global maps showing weighted threat scores for major and sub-threat categories by basin (and their legends).

Summary of changes

This MR consists of two changes:

  • Rearrange color palettes to more closely resemble to colors used in the paper
  • Change the exploitation threat category to "Fishing pressure"

Expected results

Expected results should include a recolor of the existing maps and legends in src/assets/images/.

Verification of results

The expected results should consist of the following:

Habitat maps and legends change from yellow to green.

image.png --> Habitat_map.png

Pollution maps and legends change from blue to yellow.

image.png --> Pollution_map.png

Exploitation changes to fishing pressure, and the maps and legends change from pink to blue.

image.png--> Fishing_pressure_map.png

Invasive species maps and legends change from green to pink.

image.png --> Invasive_species_map.png

Requested Review

  1. Run the pipeline to make sure no errors pop up
  2. Check the output and verify the results.
    1. Are the maps the correct colors? Was exploitation changed to fishing pressure?
  3. I would like to know your opinion on if rearranging the colors is enough, or if we should edit the color palettes to match figure 2 in the paper.

Next Steps

Next steps include working on getting these maps into the planned tab structure, see merge request Findex global maps tabs

Merge request reports

Merged by Vaarre-Lamoureux, Kaysa SVaarre-Lamoureux, Kaysa S 3 weeks ago (Feb 24, 2025 2:40pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    • 96f7ce73 - edit map colors and push new weighted threats csv

    Compare with previous version

  • resolved all threads

  • made the edits, let me know if there's anything else to tweak :)

  • Hayley Corson-Dosch approved this merge request

    approved this merge request

  • Looks great, Kaysa! The maps and colors are updated as expected. Let's stick with these colors for now, as they fit in with the color palette of the site.

  • mentioned in commit 106df973

  • Hayley Corson-Dosch mentioned in merge request !90 (merged)

    mentioned in merge request !90 (merged)

  • Please register or sign in to reply
    Loading