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

DNA copying challenge #1031

Merged
merged 2 commits into from
Oct 14, 2024
Merged

DNA copying challenge #1031

merged 2 commits into from
Oct 14, 2024

Conversation

kostmo
Copy link
Member

@kostmo kostmo commented Jan 23, 2023

A challenge that utilizes a stack.
Closes #932.

Demo:

scripts/play.sh --scenario data/scenarios/Challenges/dna.yaml --autoplay

Screenshot from 2024-10-13 17-06-06

@kostmo kostmo force-pushed the dna-copy branch 2 times, most recently from 6077349 to 2eb96d2 Compare January 28, 2023 06:06
@xsebek
Copy link
Member

xsebek commented Feb 11, 2023

@kostmo could you finish this using the let binding workaround suggested in #1032 (comment)?

This challenge looks really good and the workaround is only one extra let that we can remove once the issue is fixed.

@xsebek
Copy link
Member

xsebek commented Sep 1, 2024

@kostmo, is this ready now that the bug has been fixed? 🙂

I think this was before rec, so the base might need a device for that.

@kostmo kostmo force-pushed the dna-copy branch 9 times, most recently from e50e4fb to eeb8d54 Compare October 13, 2024 23:53
@kostmo kostmo added the G-Scenarios An issue having to do with scenario design, the way scenarios are described and loaded, etc. label Oct 14, 2024
@kostmo kostmo requested review from byorgey and xsebek October 14, 2024 00:07
@kostmo kostmo marked this pull request as ready for review October 14, 2024 00:07
Copy link
Member

@byorgey byorgey left a comment

Choose a reason for hiding this comment

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

Love it!! My solution can be seen here: https://github.com/byorgey/swarm-defs/blob/main/dna.sw

I note only that it was slightly confusing that e.g. the siphon says it enables the drill command, but in fact one must say use "siphon" rather than drill. That's not something that can/should be solved in this PR but I thought it worth mentioning.

Once I had what I thought was sure to be a working solution, I enjoyed watching in horror as the mushrooms spread uncontrollably and ruined it, requiring me to switch the cloning order. 🍄 🍄 🍄 😁

Shall we close #932 when this is merged?

@kostmo kostmo added the merge me Trigger the merge process of the Pull request. label Oct 14, 2024
@mergify mergify bot merged commit 0fda99f into main Oct 14, 2024
14 checks passed
@mergify mergify bot deleted the dna-copy branch October 14, 2024 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
G-Scenarios An issue having to do with scenario design, the way scenarios are described and loaded, etc. merge me Trigger the merge process of the Pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Challenge scenario requiring the use of a stack
3 participants