8000 Add the formula for number of indexes per resolution by isaacbrodsky · Pull Request #69 · uber/h3 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add the formula for number of indexes per resolution #69

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

Merged
merged 2 commits into from
Jun 8, 2018

Conversation

isaacbrodsky
Copy link
Collaborator

This seems a little inefficient to put into its own file, but I'll put it there for the moment and later refactor the generators. Especially so the generators do not depend on the files whose contents they are partly generating (perhaps have a generated tables file, and a base tables/constants file it's based off of.)

@coveralls
Copy link
coveralls commented Jun 4, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 1215914 on isaacbrodsky:generate-num-hexagons into 12f467f on uber:master.

Copy link
Contributor
@isaachier isaachier left a comment

Choose a reason for hiding this comment

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

LGTM

printf("};\n");
}

int main(int argc, char* argv[]) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Alternatively, you could change this to int main(void) and ignore the arguments entirely.

Copy link
Collaborator

Choose a reason for hiding this comment

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

+1

Copy link
Collaborator Author
@isaacbrodsky isaacbrodsky Jun 8, 2018

Choose a reason for hiding this comment

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

If we make that change, I think it should be made to all the filters. edit: by which I mean all the apps its relevant for.

printf("};\n");
}

int main(int argc, char* argv[]) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

+1

@isaacbrodsky isaacbrodsky merged commit ff801e3 into uber:master Jun 8, 2018
@isaacbrodsky isaacbrodsky deleted the generate-num-hexagons branch June 8, 2018 16:20
mrdvt92 pushed a commit to mrdvt92/h3 that referenced this pull request Jun 19, 2022
Add the formula for number of indexes per resolution
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
0