Skip to content

vtfpp: ImageConversion::alphaToDistance: init#114

Merged
craftablescience merged 8 commits into
craftablescience:mainfrom
partyvan:distancealpha-2026-cpp
May 18, 2026
Merged

vtfpp: ImageConversion::alphaToDistance: init#114
craftablescience merged 8 commits into
craftablescience:mainfrom
partyvan:distancealpha-2026-cpp

Conversation

@partyvan
Copy link
Copy Markdown
Contributor

i accidentally invented an extremely niche novel dithering algorithm too oops

putting this out there for style/sanity/etc checks while i come up with test cases and maretf flags

@craftablescience craftablescience added the enhancement New feature or request label May 17, 2026
@craftablescience
Copy link
Copy Markdown
Owner

Since all of this goes over my head and is relatively self-contained, if you can move the additions here to a new header/cpp/namespace I will merge as soon as you're done without my usual nitpicking and just pretend the files don't exist besides setting up language bindings. This is awesome btw and I'm glad you came back to this and sent it in, thanks!!!

@partyvan
Copy link
Copy Markdown
Contributor Author

there we go it's been put in its containment chamber with some snacks -- namespace name and so on seem all right to you?

@craftablescience
Copy link
Copy Markdown
Owner

Looks good thanks!

@partyvan partyvan marked this pull request as ready for review May 18, 2026 01:46
@partyvan
Copy link
Copy Markdown
Contributor Author

added some cursory tests (incl one that caught out a silly bug); should be g2g on this end. they're just procgen for now so no need to pull any new assets, though i might come up with some more comprehensive asset-based tests later

@craftablescience craftablescience force-pushed the distancealpha-2026-cpp branch from 9edd685 to 65f5507 Compare May 18, 2026 04:45
@craftablescience craftablescience merged commit 0022b72 into craftablescience:main May 18, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants