Skip to content

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

Open
Xlinka wants to merge 2 commits into
mainfrom
Linka
Open

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

Conversation

@Xlinka

@Xlinka Xlinka commented Mar 3, 2026

Copy link
Copy Markdown
Collaborator
  • 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

Nytra commented Mar 7, 2026

Copy link
Copy Markdown
Collaborator

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

Xlinka commented Mar 8, 2026

Copy link
Copy Markdown
Collaborator Author

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

Nytra commented Mar 18, 2026

Copy link
Copy Markdown
Collaborator

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