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

Landcover order #3441

Merged
merged 3 commits into from
Oct 28, 2018
Merged

Conversation

matthijsmelissen
Copy link
Collaborator

This resolves #2782

Copy link
Collaborator

@kocio-pl kocio-pl left a comment

Choose a reason for hiding this comment

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

Unfortunately I'm not able to really review this code.

@matthijsmelissen
Copy link
Collaborator Author

Is anybody willing to review this change?

@sommerluk
Copy link
Collaborator

Haven’t tested it yet, but the code looks good to me.

@sommerluk sommerluk merged commit b3323b4 into gravitystorm:master Oct 28, 2018
@sommerluk
Copy link
Collaborator

Thanks for the patch. Sorry for the delay.

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.

Should landcover be ordered by osm_id in addition to way_area?
3 participants