Skip to content

A REALLY MASSIVE UPDATE#7

Open
rerafyaw wants to merge 29 commits intosd16fall:masterfrom
rerafyaw:master
Open

A REALLY MASSIVE UPDATE#7
rerafyaw wants to merge 29 commits intosd16fall:masterfrom
rerafyaw:master

Conversation

@rerafyaw
Copy link

@rerafyaw rerafyaw commented Nov 7, 2016

This pull request creates 5 major revisions:
V1.0 (Base game)
V1.1 (Basic functionality and interaction)
V1.2 (Two player visuals)
V2.0 (Major bug fixes, sound support and multiplayer functionality)
V2.1 (Splash screen preface created, more bug fixes, fair play mechanisms (randomization of trajectory))

rerafyaw and others added 28 commits October 20, 2016 16:37
…s resolved, spawning is now fully randomized for all object geometries
…nabled, some fair-play mechanic fixes, and competitive play completed
… balances introduced (e.g. movespeed of players)
… more gameplay balances introduced (e.g. movespeed of players)
… more gameplay balances introduced (e.g. movespeed of players)
Copy link

@runnersaw runnersaw left a comment

Choose a reason for hiding this comment

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

You are missing instructions how to start the game, but you have a very good explanation of gameplay.

Your code to check for the restart key after the game is over doesn't work occasionally.

Beautiful documentation, great comments, nice docstrings, everything is there.

Implements the models well. Your code might have been a little more effective if you had also implemented a controller and a view. You have several functions that would be more effective if they were on a view object, such as the functions to draw the start screen and the restart prompt.

There are many variables that seem to be unused (like x_position, xx_position, and so on). It seems like you moved the x and y position information into a model and then didn't remove the initialization of these variables elsewhere.

Overall, good job. This is a large and ambitious project.

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

Comments