-
Notifications
You must be signed in to change notification settings - Fork 19
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
Dynamic levels directory #151
Closed
assertivist
wants to merge
81
commits into
avaraline:master
from
assertivist:dynamic-levels-directory
Closed
Dynamic levels directory #151
assertivist
wants to merge
81
commits into
avaraline:master
from
assertivist:dynamic-levels-directory
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…xt y values on older levels, ryan levels work though
…r ones do not seem to work as well
This thing is so dumb that it tried to greet Head who has committed to the repository and made numerous pull requests and comments. If it can't even tell that he doesn't need to be greeted then it sucks
…happening during a hang
Trying to re-create some hangs. This might be useful for other tests as well.
This fixes a bug where a client with PACKET_DEBUG on who receives a message from connection that is no longer valid (client was kicked).
The dreaded "Waiting for <Player>" message has ruined many a game. This fix simply kicks the player who is being waiting on so the game can continue. The booted player can log in again.
…section_point function
…xt y values on older levels, ryan levels work though
…r ones do not seem to work as well
…section_point function
…evel not found status, template level files that KIND OF work
…ize all the static string bits
…u scale something in inkscape and it becomes .9999999 instead of 1.0 etc)
Closed as this is getting combined with ALF. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #150
Avara now reads the sets available from the
levels
directory. Additionally, it detects whether a given set is a resource file or the updated format of a directory with JSON and SVGs. That means that the PR alsoFixes #27 🎉
as with these patches Avara supports both resource files and SVGs without the use of a command line switch. There is now a function in
LevelLoader
that (crudely) gives a version (rsrc or svg) for a given set, and this is used to trigger the alternate level loading code.I was also able to improve the failure state for both types of sets--instead of causing crashes, loading missing levels should now properly trigger the "level not found" status.
Added two SVG-only sets,
20XX
(for testing and template levels) andtyyli
(an in-progress Piet Mondrian inspired set that anyone is welcome to contribute to)If you can bear to read it, please review and give feedback on implementation. I'm open to ideas if you see something particularly bad. This also needs a network play test before it can be merged.