Year 2022 Day 20 regression #6

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

Jump to bottom

Closed

dfarnham wants to merge 1 commit into maneatingape:main from dfarnham:dfarnham-patch-1

Closed

Year 2022 Day 20 regression #6

dfarnham wants to merge 1 commit into maneatingape:main from dfarnham:dfarnham-patch-1

Conversation

dfarnham

Copy link

Contributor

@dfarnham dfarnham commented Jan 14, 2025

No description provided.

@dfarnham


 Factor out a multiplication

ab0931c

@dfarnham

Copy link

Contributor Author

dfarnham commented Jan 14, 2025

This current algorithm is failing my puzzle input (a regression from your previous algorithm)

@maneatingape

Copy link

Owner

maneatingape commented Jan 14, 2025

Thanks for the bug finding!

Does this change fix the issue? I'm not sure how it would produce a different result.

@dfarnham

Copy link

Contributor Author

dfarnham commented Jan 14, 2025 via email

No, sorry for the confusion. Factoring out the multiplication just saves an instruction is all. If you navigate to my AOC repo you can test my puzzle input across your previous tree algorithm which produces the correct answer.

...

On Jan 14, 2025, at 10:27 AM, maneatingape ***@***.***> wrote: Thanks for the bug finding! Does this change fix the issue? I'm not sure how it would produce a different result. — Reply to this email directly, view it on GitHub <#6 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAOMDBHW3W5L73XZG6MYJLT2KVCIRAVCNFSM6AAAAABVFK6UDOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOJQGYZTQMJWHA>. You are receiving this because you authored the thread.