Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API change first pass - Breaks loading region_offsets! #433

Merged
merged 3 commits into from
Sep 8, 2024

Conversation

TokisanGames
Copy link
Owner

@TokisanGames TokisanGames commented Jul 26, 2024

API Changes to existing functions (from main):

Breaking change: This PR currently breaks user scenes as region_offsets aren't converted to region_locations. It is a precursor to #374 which will have upgrade code.

Functions

Old Name New Name
Terrain3DStorage
set/get_region_offsets set/get_region_locations
get_region_offset get_region_location
get_region_offset_from_index get_region_location_from_id
get_region_index get_region_id
get_region_index_from_offset get_region_id_from_location
Terrain3DInstancer
clear_by_offset clear_by_location

Signals

Old Name New Name When emitted
Terrain3D
storage_changed - Will be dropped in #374
Terrain3DStorage
regions_changed maps_changed When any map/region regenerated
- region_map_changed When region map regenerated (new)
height_maps_changed height_maps_changed when height maps are regenerated (no change)
- control_maps_changed when control maps are regenerated (new)
- color_maps_changed when color maps are regenerated (new)
maps_edited(AABB) maps_edited(AABB) when a heightmap is modified with the AABB (no change)

@TokisanGames
Copy link
Owner Author

Updated Op with signal changes

@TokisanGames TokisanGames force-pushed the region-offsets-locations branch 2 times, most recently from e89a0a3 to 8d93422 Compare August 1, 2024 07:24
@TokisanGames TokisanGames marked this pull request as ready for review August 4, 2024 14:33
@TokisanGames TokisanGames self-assigned this Aug 28, 2024
@TokisanGames TokisanGames force-pushed the region-offsets-locations branch 5 times, most recently from 722b511 to 502dc0b Compare September 2, 2024 17:26
@TokisanGames TokisanGames merged commit efc447c into main Sep 8, 2024
13 checks passed
@TokisanGames TokisanGames deleted the region-offsets-locations branch September 8, 2024 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api API changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant