Skip to content

Add Ceil/Floor/Round To Interval nodes and SlotToBodyNode#132

Open
Xlinka wants to merge 2 commits intomainfrom
Linka
Open

Add Ceil/Floor/Round To Interval nodes and SlotToBodyNode#132
Xlinka wants to merge 2 commits intomainfrom
Linka

Conversation

@Xlinka
Copy link
Copy Markdown
Collaborator

@Xlinka Xlinka commented Mar 3, 2026

  • Adds RoundToInterval, FloorToInterval, and CeilToInterval ProtoFlux nodes (float & double variants) under Obsidian/Math — rounds a value to the nearest multiple of a given interval, useful for grid snapping and value binning

  • Adds SlotToBodyNode under Obsidian/Avatar — resolves a slot to its BodyNode enum value by walking up to the nearest BipedRig

Xlinka added 2 commits March 3, 2026 16:00
Add six new ProtoFlux Obsidian math nodes (float and double variants): CeilToInterval, FloorToInterval, and RoundToInterval.

reference issue is Yellow-Dog-Man/Resonite-Issues#6289
@Nytra
Copy link
Copy Markdown
Collaborator

Nytra commented Mar 7, 2026

SlotToBodyNode looks good, but the Interval nodes aren't really doing anything that isn't already possible

The Interval nodes I think would work better as ProtoFlux functions which will have to wait until they get implemented in Resonite officially Yellow-Dog-Man/Resonite-Issues#564

@fulgenscode
Copy link
Copy Markdown

I was happy to see the interval nodes, but I can see a potential issue if Resonite implements nodes with the same name.

@Xlinka
Copy link
Copy Markdown
Collaborator Author

Xlinka commented Mar 8, 2026

I was happy to see the interval nodes, but I can see a potential issue if Resonite implements nodes with the same name.

this has happend before where we just remove the plugin node as its been replaced by first party.

@Nytra
Copy link
Copy Markdown
Collaborator

Nytra commented Mar 18, 2026

this has happend before where we just remove the plugin node as its been replaced by first party.

there would be no way to upgrade them, they would just become null

should think about future compatibility a bit

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