Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Heatmap correction #1017

Merged
merged 2 commits into from
Aug 22, 2023
Merged

Conversation

Zoophobus
Copy link
Contributor

Corrects the presentation of the heatmap values so that the plotted colours are set according to the maximum value per Impact Assessment Category.

Also adds the option to switch the axes x => y and y => x

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, feature, ui, change, documentation, breaking, ci
    as they show up in the changelog

… as a proportion of the maximum recorded value for the separate Impact Assessment methods). Also provides a checkbox for changing the axes of the heatmap plot.
@Zoophobus Zoophobus added the bug Issues/PRs related to bugs label Aug 22, 2023
@coveralls
Copy link

Coverage Status

coverage: 50.959% (-0.05%) from 51.011% when pulling 2048e25 on Zoophobus:heatmap_correction into 63cea10 on LCA-ActivityBrowser:master.

@Zoophobus Zoophobus merged commit 2b71e58 into LCA-ActivityBrowser:master Aug 22, 2023
9 checks passed
@bsteubing
Copy link
Member

Excellent that we have this now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues/PRs related to bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants