Skip to content

3D measuring tool - #397

Open
Pessimistress wants to merge 3 commits into
1.1-devfrom
x/measure-layer
Open

3D measuring tool#397
Pessimistress wants to merge 3 commits into
1.1-devfrom
x/measure-layer

Conversation

@Pessimistress

Copy link
Copy Markdown
Contributor

avs-measurment

To test locally:

git checkout x/measure-layer
yarn bootstrap
cd test/app/measure
yarn start-local

There are several ways we can package this. Changes to the core can be published as streetscape.gl@1.1.0-alpha (it depends on a non-production release of deck.gl). The TapeMeasureLayer (where majority of the logic is) can either be

  • ported into nebula.gl
  • published via @streetscape.gl/layers

ChangeList

  • Bump deck.gl to 7.4.0-alpha.1 (with 3d picking feature)
  • Upgrade PointCloudLayer to use deck.gl@7.2's shader hooks (required by 3d picking)
  • Add test app

@supersonicclay @marionleborgne @jlisee

@ibgreen

ibgreen commented Oct 3, 2019

Copy link
Copy Markdown
Contributor

This looks really neat! And as far as I can see this tool does not have any react dependencies which is great.

These are just longer term thoughts:

It would be great if for next release we could unbundle this from streetscape and put this in a helper module in e.g. deck.gl or nebula.gl.

Perhaps there is opportunity to create a "toolkit" for such measuring tools, e.g.

  • an area calculation tool (which we presumably would want to build on nebula polygon rendering).
  • a volume calculation tool
  • a point cloud volume point counter tool
  • a base class for user to create custom tools...

And perhaps the user could declaratively specify a selection of these tools, making them accessible in pydeck and the playground as well?

@ibgreen

ibgreen commented Oct 3, 2019

Copy link
Copy Markdown
Contributor

@romaxa FYI

@Pessimistress
Pessimistress changed the base branch from x/picking-fix to master October 3, 2019 03:10
@coveralls

coveralls commented Oct 3, 2019

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.3%) to 14.942% when pulling 43ade3d on x/measure-layer into cf5dd9a on master.

@supersonicclay

Copy link
Copy Markdown

I'd love to see this functionality in nebula.gl. I think we have a lot of the plumbing in place to be able to support this.

@Pessimistress
Pessimistress changed the base branch from master to 1.1-dev October 5, 2019 03:31
@Pessimistress Pessimistress changed the title [POC] 3D measuring tool 3D measuring tool Oct 5, 2019
@CLAassistant

CLAassistant commented Apr 16, 2020

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Xiaoji Chen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@supersonicclay

supersonicclay commented Jun 12, 2020

Copy link
Copy Markdown

We have distance and angle measurement tools in nebula now. Can we refactor this PR to use it?

@alexnaughtonjr

Copy link
Copy Markdown

Nice!

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.

7 participants