Skip to content
This repository was archived by the owner on Nov 13, 2023. It is now read-only.

Add ModalDownsampler#23

Open
superdump wants to merge 1 commit into
bonsairobo:mainfrom
superdump:modal-downsampler
Open

Add ModalDownsampler#23
superdump wants to merge 1 commit into
bonsairobo:mainfrom
superdump:modal-downsampler

Conversation

@superdump

Copy link
Copy Markdown
Contributor

Something like this I think. In the test I was trying to use 1x1x1 lod0 chunks and a 2x2x2 superchunk and lod0 extent, thinking that it would downsample the 2x2x2 extent of lod0 to a 1x1x1 lod1, but I hit the assertion on the dst shape being Point3i::ZERO. I don't understand why.

@superdump

Copy link
Copy Markdown
Contributor Author

I've tried it for my blocky voxel terrain stuff and it's really slow. Perhaps it's useful for someone?

@bonsairobo

Copy link
Copy Markdown
Owner

Maybe you could add a benchmark in building_blocks_storage/benches/sampling.rs so we can compare actual numbers?

I also suspect that using the SmallKeyHashMap would help. It would be even better if we could assume that the voxel type was an index into an array.

@bonsairobo bonsairobo force-pushed the main branch 2 times, most recently from 1bd1c6c to af0582a Compare August 10, 2021 23:03
@bonsairobo bonsairobo force-pushed the main branch 2 times, most recently from 5dfcb24 to 790ac87 Compare August 29, 2021 21:04
@bonsairobo bonsairobo force-pushed the main branch 4 times, most recently from 3199918 to 4e9e0c1 Compare September 6, 2021 07:18
@bonsairobo bonsairobo force-pushed the main branch 7 times, most recently from 8645945 to a69d051 Compare September 16, 2021 05:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants