Skip to content

Add an Article for Day 17 of 2023 #513

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.

Sign up for GitHub

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
Merged
merged 3 commits into from
Dec 29, 2023
Merged

Add an Article for Day 17 of 2023 #513

merged 3 commits into from
Dec 29, 2023

Conversation

stewSquared
Copy link
Contributor

No description provided.

@stewSquared
Copy link
Contributor Author

stewSquared commented Dec 21, 2023 •
edited
Loading

This has a little errata. See #514
I'll handle it when I have a sec
edit: handled!

Philippus
Philippus suggested changes Dec 27, 2023
View reviewed changes
Copy link
Contributor

@Philippus Philippus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A typo in Dijkstra's name. :)

@stewSquared @Philippus
Fix spelling if "Dijkstra"
fb601dd 
Co-authored-by: Philippus Baalman <[email protected]>
bishabosha
bishabosha approved these changes Dec 29, 2023
View reviewed changes
Copy link
Member

@bishabosha bishabosha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really nice, thank you! I did some refactoring so that the grid isn't stored as a top level global

@bishabosha bishabosha merged commit 4c40060 into scalacenter:website Dec 29, 2023
@bishabosha
Copy link
Member

Throwing out ideas, we do have this requirement of part1 and part2 not using IO to load the input so its scala.js compatible but perhaps we can add an extra section to each page that covers how to load the input from file that then calls the part1, part2 methods? it could even be tabbed for scala.js/scala native/jvm

@stewSquared
Copy link
Contributor Author

@bishabosha Thanks! I like the change removing global vars with Using Grid. I'll use that pattern in future submissions (and day 23, which I'm writing up now)

re: not using IO: Yes! I think that's a great idea. Should we make it work with pastable input in 2024?

@stewSquared stewSquared deleted the website-2023-day-17 branch December 29, 2023 23:40
@bishabosha
Copy link
Member

re: not using IO: Yes! I think that's a great idea. Should we make it work with pastable input in 2024?

That's what the solver widget is meant to be if I take the time to set them up

@itsjoeoui itsjoeoui mentioned this pull request Dec 3, 2024
Merged
@SethTisue SethTisue mentioned this pull request Dec 11, 2024
Merged
@SethTisue SethTisue mentioned this pull request Dec 19, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Philippus Philippus Philippus requested changes

@bishabosha bishabosha bishabosha approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stewSquared @bishabosha @Philippus