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

Update bokeh.figure.plotting.circle calls to use radius keyword #1632

Open
stscijgbot-jwql opened this issue Aug 19, 2024 · 1 comment
Open

Comments

@stscijgbot-jwql
Copy link
Collaborator

stscijgbot-jwql commented Aug 19, 2024

Issue JWQL-250 was created on JIRA by Bryan Hilbert:

Starting with Bokeh 3.4 the circle plotting function requires either the "size" or "radius" keyword to be included in calls. If neither is specified, Bokeh returns a blank plot because they do not define a default value for these keywords.

Note that use of "size" is already deprecated and will be removed in an upcoming Bokeh release, so we should make sure we're using "radius" in all calls. 

A quick search of the codebase shows calls to the circle plotting function in:

bokeh_dashboard.py

engineering_database.py

bokeh_containers.py

monitor_bad_pixel_bokeh.py

dark_monitor.py

edb_telemetry_monitor.py

along with a handful of bokeh-templating related files that we can ignore.

@stscijgbot-jwql
Copy link
Collaborator Author

Comment by Bryan Hilbert on JIRA:

Fix in #1643

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

No branches or pull requests

1 participant