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

Diamond iso #925

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

Diamond iso #925

wants to merge 18 commits into from

Conversation

airza
Copy link
Contributor

@airza airza commented Jul 11, 2015

I think the documentation might need fixing, but you can do lots of cool new things. In particular, pathfinding (there is kind of an inefficient part of it that is O(n) in the size of the grid instead of O(1) though.). It checks for both empty tiles on the lower part of the grid as well as objects with the component "Obstacle" (There might be some way this can better conform with the rest of the library, but since 2d collision and grid collision don't work the same I wasn't sure.

Also, i made the simplifiying assumption that grids would be 1 ground level and 1 upper level for the purposes of placement and pathfinding - most of the code probably works in 3D for placement but that gets into some deep issues if we have no rotation method.

I've tested it a lot on my game in development, but that game doesn't really try to move the grid or do anything different than the square tiles.

starwed and others added 15 commits July 11, 2015 19:00
Since we changed the method signature of Crafty.load, we should warn when people try to use the old syntax.
This adds a new grunt task `grunt api-server` that will serve the api documentation directly from `api.json`.

Use `grunt view-api` to build the docs and then view them in your default browser.

This copies over some scripts from the craftyjs.com repo, but perhaps there's a better way to share them.
…onal placement of tiles with intelligent Z calculation is now in place. Also methods for choosing tile based on pixel and pixels based on tile work better.
We had inconsistent conventions regarding @param and @returns tags

- In some places, the description of the param/return value was continued across multiple lines
- In many other places, the description of the method as a whole was place directly after such tags

While the latter outnumbered the former, I think its useful to be able to have longer param/return descriptions split across multiple lines.  So to allow this while not breaking the docs, this fixes all incidences of the second style.  (This is probably also more readable when looking directly at the source.)
- better support for internal linking
- @warning and @note tags
- In the intial parse, assume that a block of text doesn't end until *two* linebreaks, rather than one, even if its the value of a tag.
I went through and fixed a large number of small issues in our documentation.  This includes typos, formatting problems, and some places where docs were obviously wrong or missing.
- Fixes a rounding issue when calculating the sign of negative fractions
- Triggers NewDirection in the frame where the direction changes, instead of the next frame
- Renames NewRotation to NewAngularDirection and fixes the same issues
- Adds/fixes related tests
Add some in-line documentaiton, and tweak the wording of the API docs for the Motion component
Instead of adjusting the speed based on the frame rate, try to saty in sync with the game clock by using frame.dt.  Note that speeds are in pixels per second.

This is a breaking change, because it changes how we interpret speeds.  (Pixels per second instead of pixels per frame.)
"Jumpway" is probably a bit confusing, even though it kind of makes sense in the context of Twoway and Fourway.
@airza
Copy link
Contributor Author

airza commented Jul 11, 2015

I don't think I squashed the commits right :s

@airza
Copy link
Contributor Author

airza commented Jul 11, 2015

Okay, now the documentation at least isn't lying anymore.

@airza
Copy link
Contributor Author

airza commented Oct 5, 2015

I pulled the latest changes, tested against the game I have in development. Doesn't break centerAt, has documentation, and adds new functionality!

@mucaho mucaho mentioned this pull request Oct 21, 2015
3 tasks
@airza
Copy link
Contributor Author

airza commented Jun 8, 2016

Is there anything I need to do in order to get this merged in? Haven't had the time recently to look at the newsgroup.

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.

4 participants