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

Many small improvements to EXA-Parser #4

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

Conversation

gardenappl
Copy link

I had a bit of trouble getting EXA-Parser to work (since I haven't yet solved every level in the game, I wanted to commit them to the repository as I went, but the parser failed to work when there were missing solutions). So I started modifying the parser and ended up adding lots of changes.

  • Now works even if not all solutions are present
  • Print battles as a list instead of table (since the table's stats don't work anyway)
  • GIFs are attached to solutions instead of all levels (so that they can work even if not all levels have solutions)
  • Ignore solutions which don't work (those that have 0 cycles)
  • Slightly prettier messages
  • Print out the sorted GIFs
  • Error if not enough GIFs

You can see how it looks on the main branch here.

* Print battles as a list instead of table (since the table's stats
  don't work anyway)
* Now works even if not all solutions are present
* GIFs are attached to solutions instead of all levels
* Ignore solutions which don't work (those that have 0 cycles)
* Slightly prettier messages
* Print out the sorted GIFs
* Error if not enough GIFs
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.

1 participant