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

Fix typo in training/d22278a0.json task #4

Merged
merged 1 commit into from
Nov 17, 2019

Conversation

wizeman
Copy link
Contributor

@wizeman wizeman commented Nov 16, 2019

There seems to be a typo in this task's test output grid at (zero-based) index (x=8, y=7), where it should be yellow (color 4) rather than empty (color 0).

Note that both me and my wife solved the problem separately and arrived at the same answer, which only differs from the current answer by this one pixel (although, we arrived at the same conclusion through different thought processes!). That said, it's certainly possible that we're both mistaken, of course, in which case feel free to close this PR.

Thanks!

There seems to be a typo in the test output grid at (zero-based)
index (x=8, y=7), where it should be yellow (color 4) rather than
empty (color 0).
@leapingllamas
Copy link

@fchollet can I suggest that a formatter is applied to the JSON files. That way, with multiline files, diffs in a PR will more easily show and be understood.

Copy link
Owner

@fchollet fchollet left a comment

Choose a reason for hiding this comment

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

Yes, you are correct. Thanks for the fix.

@fchollet fchollet merged commit c687d93 into fchollet:master Nov 17, 2019
@fchollet
Copy link
Owner

@leapingllamas: formatted or not, the contents are impossible to interpret without a visualizer.

@petegordon petegordon mentioned this pull request Jul 3, 2024
nesco pushed a commit to nesco/ARC-AGI that referenced this pull request Jul 24, 2024
Fix typo in training/d22278a0.json task
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.

3 participants