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

Corrects the instructions so that the binary representation is accurate #1044

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

armahillo
Copy link

In the existing instructions, it indicates that the word 0000 1000 in binary would be 16 in decimal. This is incorrect. The correct value for that word is 8.

Since the binary representation is also in an SVG, it is easier to just change the decimal value. The solution is unchanged with this value, but understanding the binary representation is key to solving the exercise.

In the existing instructions, it indicates that the word 0000 1000 in
binary would be 16 in decimal. This is incorrect. The correct value for
that word is 8.

Since the binary representation is also in an SVG, it is easier to just
change the decimal value. The solution is unchanged with this value, but
understanding the binary representation is key to solving the exercise.
@ryanplusplus
Copy link
Member

Hey @armahillo, these instructions are generated from the upstream problem specifications. Specifically, they come from this file:
https://github.com/exercism/problem-specifications/blob/main/exercises/eliuds-eggs/introduction.md

I agree that this is wrong. Can you file a PR against the problem specifications repo so that this gets fixed for all tracks?

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.

2 participants