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

[FIX] Projections: Fix color density for continuous color palettes #5665

Merged
merged 1 commit into from
Oct 25, 2021

Conversation

janezd
Copy link
Contributor

@janezd janezd commented Oct 22, 2021

Issue

Fixes #5638.

The obvious symptom was that colors density was sometimes not shown. In reality, even when shown, it was sometimes incorrect.

The code ignores the points that are not colored because the corresponding "color index" exceeds the number of availiable colors. This filter should only be applied when coloring according to discrete palettes, not continuous ones.

Description of changes

Check that colors are discrete before filtering them.

Includes
  • Code changes
  • Tests
  • Documentation

@codecov
Copy link

codecov bot commented Oct 22, 2021

Codecov Report

Merging #5665 (d76b45f) into master (45f17d5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #5665   +/-   ##
=======================================
  Coverage   86.01%   86.02%           
=======================================
  Files         315      315           
  Lines       65948    65950    +2     
=======================================
+ Hits        56726    56731    +5     
+ Misses       9222     9219    -3     

@VesnaT VesnaT merged commit 8ef0aec into biolab:master Oct 25, 2021
markotoplak pushed a commit to markotoplak/orange3 that referenced this pull request Oct 27, 2021
[FIX] Projections: Fix color density for continuous color palettes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scatter Plot: 'Show color regions' not working
2 participants