Add AOC 2024 day2 part2, day3, day4, day5 #68
Conversation
There was a problem hiding this comment.
Looking over it, the runtimes for all the algorithms look good, and everything passes.
There is a single console.log that we would want to remove before committing.
Aswell, I feel the specialization of the UDFs could be decreased (e.g. the udf.reduce function in day3-2). However, I would believe it is fine to merge for now, and revisit later.
test/aoc/aoc_2024.test.js
Outdated
| arrMid: (arr) => arr[Math.floor(arr.length / 2)], | ||
| slice: a => array => array.slice(a), | ||
| customSort: (rules, arr) => { | ||
| console.log(arr) |
There was a problem hiding this comment.
Before pulling into the upstream repo, could we get rid debug print statement removed?
| MAMMMXMMMM | ||
| MXMXAXMASX`; | ||
| let match = rh`.input | udf.matchAll "(mul\\\\((\\\\d{1,3}),(\\\\d{1,3})\\\\)|do\\\\(\\\\)|don't\\\\(\\\\))" "g"` | ||
| let query = rh`udf.reduce ${match}` |
There was a problem hiding this comment.
It seems to me as though essentially almost all of this problem is done inside this udf.
We can commit it for now, but I think it would be beneficial to come back and try to rewrite it moreso in rhyme.
No description provided.