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

Cell height is defined incorrectly #17219

Closed
tovl opened this issue Oct 11, 2019 · 1 comment · Fixed by #17990
Closed

Cell height is defined incorrectly #17219

tovl opened this issue Oct 11, 2019 · 1 comment · Fixed by #17990
Assignees
@pchote
Labels
Bug Tiberian Sun
Milestone
Tiberian Sun firs...

Comments

@tovl
Copy link
Contributor

tovl commented Oct 11, 2019

Cell heights for isometric mods are currently only defined at the center of the cells. This is not enough to account for the possible slopes. We should either define the terrain heights at the vertices of the grid, or alternatively include a WRot with the cell normal direction in addition to the cell height.
@PunkPun
Copy link
Member

PunkPun commented Oct 12, 2019

related #17102

@pchote pchote self-assigned this Apr 28, 2020
@pchote pchote mentioned this issue Apr 28, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pchote pchote

Labels
Bug Tiberian Sun
Projects
None yet
Milestone
Tiberian Sun first 'full' (beta) release
Development

Successfully merging a pull request may close this issue.

Account for slopes in Map.DistanceAboveTerrain. pchote/OpenRA
4 participants
@pchote @abcdefg30 @tovl @PunkPun