TodePond / CellPond

surreal cellular automata
https://cellpond.cool
MIT License
280 stars 35 forks source link

Added rounding in conditionFunction and fixed merging #356

Closed Paalui closed 2 weeks ago

Paalui commented 2 weeks ago

I fixed #353 and #354. The rounding fixes #353 , but its only done in the condition so inaccuracies could still add up.

Multicells always got merged even if the right side is left empty. I changed it to be n times the nothing instruction instead in that case.

Also the cells get skiped one at a time after a merge instruction. This causes the next instructions to be applied on the already deleted/merged cells. Hope i didn't miss a case where you want to skip cells one at a time.

netlify[bot] commented 2 weeks ago

Deploy Preview for keen-boyd-1443aa ready!

Name Link
Latest commit a4dce1abefa95ea03b91f563e4ac502c1b810cdf
Latest deploy log https://app.netlify.com/sites/keen-boyd-1443aa/deploys/668c7abe1eea3500085f6e33
Deploy Preview https://deploy-preview-356--keen-boyd-1443aa.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

TodePond commented 2 weeks ago

Amazing! And you're correct with all your assumptions. I will merge without testing, but let me know if something goes wrong and I can merge more PRs again :)

also congratulations for being the second person in the world for understanding and writing Dragon code