Skip to content

fix: meaning of start and stop in bin-edge widget with log scale#909

Merged
SimonHeybrock merged 2 commits into
mainfrom
replace-log-by-geom
May 1, 2026
Merged

fix: meaning of start and stop in bin-edge widget with log scale#909
SimonHeybrock merged 2 commits into
mainfrom
replace-log-by-geom

Conversation

@jokasimr
Copy link
Copy Markdown
Contributor

@jokasimr jokasimr commented Apr 30, 2026

Fixes #908

Bug in the edge-widget in the workflow configuration page:

When the scale was set to scale.LOG the start and stop entries in the selector represented log10(...) of the start and stop of the desired range.

With this fix the start and stop entries in the widget represent the start and stop value of the range, which is the behavior users would expect.

@SimonHeybrock
Copy link
Copy Markdown
Member

SimonHeybrock commented May 1, 2026

Can you include some before/after screenshots in the PR description?

Got confused due to the lack of description, and because my brain read your issue title and said "Logscale" -> it is about plotting. But this is not about logscale, right? It is about how define define/configure edges to use in a workflow.

Can you please clarify issue and PR titles and add a description in the PR itself? We are using those to create user-facing docs about changes.

@jokasimr jokasimr changed the title fix: use geomspace fix: meaning of start and stop in bin-edge widget May 1, 2026
@jokasimr jokasimr changed the title fix: meaning of start and stop in bin-edge widget fix: meaning of start and stop in bin-edge widget with log scale May 1, 2026
@jokasimr
Copy link
Copy Markdown
Contributor Author

jokasimr commented May 1, 2026

Got confused due to the lack of description

There is a description in the issue?

Can you please clarify issue and PR titles and add a description in the PR itself? We are using those to create user-facing docs about changes.

I've added the description to the PR and clarified the title.

@SimonHeybrock SimonHeybrock merged commit 475ef45 into main May 1, 2026
13 checks passed
@SimonHeybrock SimonHeybrock deleted the replace-log-by-geom branch May 1, 2026 07:56
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.

Logscale should use np.geomspace instead of np.logspace

2 participants