Skip to content

Conversation

@LizaShch
Copy link
Collaborator

@LizaShch LizaShch commented Apr 14, 2025

Added functionality for user to add selected colormapping dictionary (string) and will color the histogram based of it. Tested for group_by, together, and other various plots.

Copy link
Collaborator

@Ahmad8864 Ahmad8864 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small changes

group_data = plot_data[plot_data[group_by] ==
groups[i]][data_column]
group = groups[i]
group_data = plot_data[plot_data[group_by] == group][data_column]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line too long, <80 chars

hist_data = calculate_histogram(group_data, kwargs['bins'])


# Retrieve the specific color for this group if defined_color_map is provided
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line too long

fangliu117 and others added 23 commits May 12, 2025 14:31
…dataset

fix(summarize_dataframe): remove duplicated missing index
…ncolor

feat(visualize_nearest_neighbor): add pin-color and corresponding uni…
merge updated summarize_dataframe and visualize_nearest_neighbor with pin-color from dev to release_dev
…and helpers and unit test files individually
…nearest_neighbor

feat(pin_color): add pin color feature to visualize nearest neighbor …
merge dev into release_dev with updated visualize_nearest_neighbor by adding pin-color feature
…nearest_neighbor

fix(visualize_nearest_neighbor): rewrite unit tests and move up  libr…
merge the updated visualize_nearest_neighbor function and unit tests from dev to dev_release
step: bumping minor version from 0.8 to 0.9
step: bumping minor version from 0.8 to 0.9
Minor version bumping from v0.8.0 to v0.9.0
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.