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.

Sign up for GitHub

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

Jump to bottom

Loading map info from file. #16

Merged
merged 15 commits into from
Jul 1, 2015
Merged

Loading map info from file. #16

merged 15 commits into from
Jul 1, 2015

Conversation

langest
Copy link
Owner

@langest langest commented Apr 24, 2015

NOT READY!!!
This update will make it possible to load maps from file located in the info directory.
Todo:

  • Loading map from info file
  • Actually use the info loaded
  • Make sure environment is drawn
  • Environment blocks buildings

@langest
Implemented member function to load info from file.
249fd9b 
Implemented functionality to load info about a map from file.
Similar to how virus, tower and wave loads from file.
@langest langest added this to the 1.0 milestone Apr 24, 2015
@langest
Now loading map from file.
0b04c74 
Added last functionality to load the map from file.
Next is to make sure that the info actually is used.
@langest
Now loading info into the game
8796ad3 
Loading the info into gameboard and player.
Also renamed hp to cp since it is named control points.
@langest
Fixed bug where towers wasn't available
b6e4b01 
Also fixed type-o's of the available set.
@langest
Copy link
Owner Author

langest commented Apr 24, 2015

It seems to work now. Though I still have to test to draw out the environment and see if it can affect gameplay.

@langest
Now it draws environment though something is wrong
f976464 
I am going to debug it tomorrow.
@langest
Copy link
Owner Author

langest commented Apr 24, 2015

I found a bug while adding drawing of environment.
Going to debug it tomorrow.

@langest
Added range to direction projectile
fab30d6 
Added a range so they will not depend on the definition of the map
size.
@langest
Removed dependency of defined board size
fb48675 
Instead reads everything from file in info/<mapid>.

Found interesting bug: When drawing the cursor
the right side of the frame is erased.
But it only seems to appear when the board is large enough.
Issue is created on github.
@langest
Copy link
Owner Author

langest commented Apr 25, 2015

Now we can load map from file.
Also found a bug #17 but it seems to be unrelated to the updates here since it is located in GUI.

@langest langest mentioned this pull request Apr 25, 2015
Closed
@langest
Environment blocking buildings
a357239 
The environment now blocks buildings.
@langest
Copy link
Owner Author

langest commented Apr 26, 2015

Now all environment states != 0 blocks buildings.

@langest langest mentioned this pull request Apr 26, 2015
Open
@langest
Fixed bug in dir projectiles
2394914 
Where they did not travel their range properly.
@langest
fmt
83fbc19
@langest
Added two more towers for testing.
9a3368d 
They are pretty ugly but thet get the work done.
@langest
Projectile manager now removed projs that are oob.
62908bf 
oob = out of bounds
@langest
fmt
eaa02b2
@langest
Copy link
Owner Author

langest commented Apr 26, 2015

Fixed bug with direction projectiles not working properly with the variable size of the map.

@ThompZon
Copy link
Contributor

After aee9f9f towers can now spawn projectiles when standing along the edges, but projectiles can also hit the edge (resulting in the gfx be there after the projectile is destroyed). This must be fixed!

@langest
Copy link
Owner Author

langest commented May 1, 2015

I cannot recreate this bug.
Try recompiling everything and then check if it is still present.

langest added a commit that referenced this pull request Jul 1, 2015
@langest
Merge pull request #16 from langest/maps_from_file
9467e99 
Loading map info from file.
@langest langest merged commit 9467e99 into master Jul 1, 2015
@langest langest deleted the maps_from_file branch July 1, 2015 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ThompZon ThompZon

Labels
None yet
Projects
None yet
Milestone
1.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@langest @ThompZon