Skip to content

Conversation

@c0d3st0rm
Copy link
Contributor

I re-created the XKCD antigravity comic (http://xkcd.com/353/) in ASCII form, and added it to the antigravity module to simulate the classic Python antigravity. There's two versions, the micro one (default), and the full one (optionally swappable for the micro one at build time). It uses the RLE as suggested by @whaleygeek and @dpgeorge to encode the strings. The micro comic, both text and RLE printing code, is 288 bytes. The larger comic, both text and RLE code, is just over 1KB (1053 bytes).

@dpgeorge
Copy link
Member

@JoeGlancy really great! Just a few code style issues to fix up, if you don't mind: 4 spaces for tab, curly brackets on the line with the statement, and curly brackets even for 1-liner. Also please squash all commits for the final merge (and rebase against master: git rebase -i master).

Copy link
Member

Choose a reason for hiding this comment

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

Please put different names on different lines, ie add the following for you:

Copyright (c) 2016 Joe Glancy

@c0d3st0rm
Copy link
Contributor Author

I think that's all.

@ntoll
Copy link
Contributor

ntoll commented Jan 31, 2016

@JoeGlancy you need to add yourself to the AUTHORS file. ;-)

@dpgeorge
Copy link
Member

dpgeorge commented Feb 1, 2016

Yes, looks good, but please combine commits into one.

@ntoll
Copy link
Contributor

ntoll commented Feb 1, 2016

@c0d3st0rm
Copy link
Contributor Author

I was sure I already did. I'll do both tonight when I find time.

@ntoll
Copy link
Contributor

ntoll commented Feb 1, 2016

Great stuff and thanks! Looks like this'll make it into the imminent 1.0 release. 😄

@c0d3st0rm
Copy link
Contributor Author

Sweet! I look forward to it :)

@ntoll
Copy link
Contributor

ntoll commented Feb 1, 2016

Also, make sure you add yourself to the AUTHORS file before you roll up all the commits..!

@c0d3st0rm
Copy link
Contributor Author

Will do!

@c0d3st0rm
Copy link
Contributor Author

Done?

…) version and the full version (also updated AUTHORS file).
@dpgeorge
Copy link
Member

dpgeorge commented Feb 1, 2016

Ok, great, it's now 256 bytes (the small version of the comic with 1 pane).

There are still 6 commits that need squashing, but I can do this with a rebase myself, unless there are objections?

@c0d3st0rm
Copy link
Contributor Author

Finally, all into one commit!
No, sorry, I attempted 3 times but it finally rebased successfully :)

@dpgeorge dpgeorge merged commit a50f60c into bbcmicrobit:master Feb 1, 2016
@dpgeorge
Copy link
Member

dpgeorge commented Feb 1, 2016

Great work, thanks for your contribution! And hopefully you learned a bit more about using git :)

@c0d3st0rm
Copy link
Contributor Author

Yup 👍 :)

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