Skip to content

Customizable clustergram line width#434

Merged
hendraet merged 2 commits into
devfrom
customizable-clustergram-line-width
May 22, 2026
Merged

Customizable clustergram line width#434
hendraet merged 2 commits into
devfrom
customizable-clustergram-line-width

Conversation

@hendraet
Copy link
Copy Markdown
Collaborator

Description

related to parts of #341
Adds a form element that lets users control the line width of the dendrograms that are part of a clustergram. Also updates the example workflow accordingly.

Testing

Create a run based on the "example dataset" workflow and run until clustergram step. Line width should be customizable in the form.

PR checklist

Development

  • If necessary, I have updated the documentation (README, docstrings, etc.)
  • If necessary, I have created / updated tests.

Mergeability

  • main-branch has been merged into local branch to resolve conflicts
  • The tests and linter have passed AFTER local merge
  • The backend code has been formatted with black
  • The frontend code has been formatted with pnpm format and checked with pnpm lint

Code review

  • I have self-reviewed my code.
  • At least one other developer reviewed and approved the changes

@hendraet hendraet changed the base branch from main to dev May 22, 2026 11:32
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 22, 2026

Coverage report

This PR does not seem to contain any modification to coverable code.

Comment thread backend/user_data/workflows/example_dataset.yaml Outdated
Copy link
Copy Markdown
Collaborator

@jorisfu jorisfu left a comment

Choose a reason for hiding this comment

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

LGTM, but change the current_step in the workflow before merging

@hendraet
Copy link
Copy Markdown
Collaborator Author

Noice, thx for the quick review

@hendraet hendraet merged commit f89b6fb into dev May 22, 2026
1 check passed
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.

2 participants