Skip to content

Get depth API#285

Closed
nmichaud wants to merge 1 commit intoperspective-dev:masterfrom
nmichaud:get-depth-api
Closed

Get depth API#285
nmichaud wants to merge 1 commit intoperspective-dev:masterfrom
nmichaud:get-depth-api

Conversation

@nmichaud
Copy link
Contributor

Forgot to include this in #277

@texodus
Copy link
Member

texodus commented Oct 22, 2018

Thanks for the PR!

I think this patch needs a bit more work before we can merge it. We've updated our contributing.md to make our PR guidelines more explicit. Please take a look - specifically for this change, I think we'd need

  • An explanation of what this API does and why it needs to be written in C++; as it stands this information can be much more efficiently calculated from the view()s config object.
  • A Javascript API
  • Tests and JSDoc

@texodus texodus closed this Oct 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants