Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add rasterize as an actual attribute - #5783

Open
ffreyer wants to merge 8 commits into
masterfrom
ff/fix-rasterize
Open

Add rasterize as an actual attribute#5783
ffreyer wants to merge 8 commits into
masterfrom
ff/fix-rasterize

Conversation

@ffreyer

@ffreyer ffreyer commented Sep 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Description

Summary:

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • Added an entry in CHANGELOG.md (for new features and breaking changes)
  • Added or changed relevant sections in the documentation
  • Added unit tests for new algorithms, conversion methods, etc.
  • Added reference image tests for new plotting functions, recipes, visual options, etc.

@github-project-automation github-project-automation Bot moved this to Work in progress in PR review Sep 11, 2026
@asinghvi17

asinghvi17 commented Sep 11, 2026

Copy link
Copy Markdown
Member

Sounds good to me but note that at some point we did want to allow rasterize = (GLMakie, px_per_unit::Int) so I might want to bring that back too. I assume that would be a simple patch after.

@jkrumbiegel

jkrumbiegel commented Sep 12, 2026

Copy link
Copy Markdown
Member

Probably not so necessary anymore when the cairomakie mesh drawing PR goes in

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

PR review
Status: Work in progress

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

rasterize can't be switched from a Bool to an Int? Warning message of arrows3d with kwargs rasterize using CairoMakie

3 participants

@ffreyer @asinghvi17 @jkrumbiegel