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

GridMap iterator col-major order or row-major order ? #80

Closed
rookiepig opened this issue Nov 28, 2016 · 2 comments
Closed

GridMap iterator col-major order or row-major order ? #80

rookiepig opened this issue Nov 28, 2016 · 2 comments

Comments

@rookiepig
Copy link

I am using the 1.4.1 version (apt-get install of ubuntu packages).

I run the iterator_demo, the iterator goes in column-major order, however the gif image of this github repo shows that the iterator goes in row-major order.

I am confused and want to ask that which one is correct ?

@pfankhauser
Copy link
Member

Well observed, @rookiepig =) the grid map iterator changed in 1.2.0 to be more efficient. This had the effect that the cell traversal order switched. So the iterator_demo is fine, I just haven't gotten around to update the GIF in the documentation.

@rookiepig
Copy link
Author

Glad to see it fix 👍 Indeed, I was mislead by the gif :)

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

No branches or pull requests

2 participants