Skip to content

Filter transform instances#2935

Closed
0HyperCube wants to merge 1 commit into
masterfrom
filter-transform-instances
Closed

Filter transform instances#2935
0HyperCube wants to merge 1 commit into
masterfrom
filter-transform-instances

Conversation

@0HyperCube
Copy link
Copy Markdown
Contributor

image image

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) July 25, 2025 00:42 Inactive
Comment thread node-graph/preprocessor/src/lib.rs
Comment thread node-graph/gcore/src/transform_nodes.rs Outdated
Comment thread node-graph/gcore/src/transform_nodes.rs Outdated
@0HyperCube 0HyperCube force-pushed the filter-transform-instances branch from 137c24e to 9f6ea66 Compare August 4, 2025 13:45
@0HyperCube
Copy link
Copy Markdown
Contributor Author

@Keavon closing because I'm not rebasing this yet again, I only just finished rebasing it the first time. Thanks.

@0HyperCube 0HyperCube closed this Aug 4, 2025
@Keavon
Copy link
Copy Markdown
Member

Keavon commented Aug 4, 2025

I'm always happy to rebase things, just send me a message. I'll do this one now.

@Keavon Keavon reopened this Aug 4, 2025
@Keavon Keavon force-pushed the filter-transform-instances branch from 9f6ea66 to 0508da1 Compare August 4, 2025 20:03
@Keavon
Copy link
Copy Markdown
Member

Keavon commented Aug 4, 2025

Rebase is done. That only affected about 5 small conflicts, so I'm glad to see my name updates aren't causing too many nasty conflicts.

@github-actions github-actions Bot temporarily deployed to graphite-dev (Preview) August 4, 2025 20:07 Inactive
@0HyperCube
Copy link
Copy Markdown
Contributor Author

@Keavon if you rebase, please make sure the code actually compiles…

@Keavon
Copy link
Copy Markdown
Member

Keavon commented Dec 20, 2025

This will instead be solved by the more generalized approach of #2988. But this was a really cool prototype, thanks for exploring this approach, it was valuable to learn from.

@Keavon Keavon closed this Dec 20, 2025
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.

3 participants