Skip to content

Conversation

hoxbro
Copy link
Member

@hoxbro hoxbro commented May 28, 2025

Adds radius and radius_dimension to the Bokeh PointPlot. Because radius works with Circle, we update the _plot_methods before calling _init_glyph.

Copy link

codecov bot commented May 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.82%. Comparing base (311d7f9) to head (397d12a).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6599   +/-   ##
=======================================
  Coverage   88.81%   88.82%           
=======================================
  Files         326      326           
  Lines       69530    69550   +20     
=======================================
+ Hits        61756    61776   +20     
  Misses       7774     7774           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hoxbro hoxbro marked this pull request as ready for review June 3, 2025 13:37
def _init_glyph(self, plot, mapping, properties):
if "radius" in properties:
self._plot_methods = dict(single='circle', batched='circle')
properties.pop("size", None)
Copy link
Member Author

Choose a reason for hiding this comment

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

This will make it so radius always has priority over size.

@hoxbro hoxbro requested a review from philippjfr June 3, 2025 13:42
@hoxbro hoxbro changed the title enh: Add radius and radius_dimension to PointPlot enh(bokeh): Add radius and radius_dimension to PointPlot Jun 3, 2025
@philippjfr
Copy link
Member

Looks good!

@hoxbro hoxbro merged commit 634e0b1 into main Jun 4, 2025
16 checks passed
@hoxbro hoxbro deleted the point_radius branch June 4, 2025 14:57
@droumis droumis added this to NIH-NCI Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants