Skip to content

Add Datagen providers for physics properties - #1630

Open
pouffy wants to merge 4 commits into
ryanhcode:mainfrom
pouffy:main
Open

pouffy wants to merge 4 commits into
ryanhcode:mainfrom
pouffy:main

Conversation

@pouffy

@pouffy pouffy commented Oct 14, 2026 •

Copy link
Copy Markdown

I think these would be useful for mod devs utilising Sable to easily generate properties instead of manually writing them up.

The providers are all abstract and should be fully functional :3

Here are the changes:

  • PhysicsBlockPropertyTypes#REGISTRY is now protected instead of private. (just so I could get the id of property types in the builder)
  • Created BlockStateConditionSet.Builder.
  • Created BezierResourceFunction.Builder.
  • Created DimensionPhysicsProvider.
  • Created BlockPhysicsPropertiesProvider.
  • Created FloatingBlockMaterialProvider.

- `PhysicsBlockPropertyTypes#REGISTRY` is now protected instead of private.
- Created `BlockStateConditionSet.Builder`.
- Created `DimensionPhysicsProvider`.
- Created `PhysicsBlockPropertiesProvider`.
- Added some Javadoc over builder methods
- Created `BezierResourceFunction.Builder`
- `DimensionPhysicsProvider.Builder#pressureFunc` now takes in a builder function instead of a built `BezierResourceFunction`.
@CLAassistant

CLAassistant commented Oct 14, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

- Created `FloatingBlockMaterialProvider`
- Renamed `PhysicsBlockPropertiesProvider` to `BlockPhysicsPropertiesProvider`
- Cleared out unnecessary imports in providers.
@pouffy pouffy changed the title Add Datagen providers for Block & Dimension Physics properties Add Datagen providers for physics properties Oct 14, 2026
@pouffy

pouffy commented Oct 14, 2026

Copy link
Copy Markdown
Author

I think this is fine

@pouffy
pouffy marked this pull request as ready for review October 14, 2026 00:51
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