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

Ef day 19 #77

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Ef day 19 #77

wants to merge 9 commits into from

Conversation

efatsi
Copy link
Contributor

@efatsi efatsi commented Dec 20, 2021

⚠️ Day 19 Spoilers ⚠️

Thoughts

Good heavens. These are getting hard. And unfortunately I need to stop spending so much time thinking about them (I'd peg this one at ~7 hours of hands-on-keyboard or hands-on-paper, and ~2 hours of me thinking about 3d space while my family eats dinner around me)

But fucking fuck they're fun. Because I know they're possible, and at the beginning I can just barely imagine a solution that doesn't have holes, and then I just keep taking baby steps, and every step solves 1 little problem but exposes another. And the puzzles are so well built that even when your stuff works on the examples you still have a little logic bug in your code to hunt down.

Takeaways

I can finally use pointers confidently, and it makes things a lot easier.

The Good

  • I started with all the Translation functions and data structs. Those held up well
  • Just being able to make the translations/offset between a sensor and an arbitrary base sensor was a feat. Maybe I'll do a labshare on this one one day though, sure was fun to figure out.

The Bad

I didn't bother with much refactoring beyond pushing things down out of main.

@efatsi efatsi requested a review from solomonhawk December 20, 2021 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant