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

Surface.PointAtParameter #82

Open
vykrum opened this issue May 9, 2016 · 7 comments
Open

Surface.PointAtParameter #82

vykrum opened this issue May 9, 2016 · 7 comments
Labels
DesignScript DesignScript improvements Geometry Geometry improvements Nodes Nodal improvement

Comments

@vykrum
Copy link

vykrum commented May 9, 2016

Dynamo version

1.0.1.1250

Operating system

Windows 10

What did you do?

Surface.PointAtParameter

What did you expect to see?

Points confined to within the surface like on this other lofted surface..

20160509-1a

What did you see instead?

20160509-1

@andydandy74
Copy link

Sometimes this is actually useful. But there should be an easy way of eliminating points that are not loacted within a trimmed surface, perhaps a boolean toogle in the Surface.PointAtParameter node.

@vykrum
Copy link
Author

vykrum commented May 9, 2016

@andydandy74 Agree it could be useful in some cases as a sort of bounding grid
But in this case, the points beyond the surface are placed at elevations corresponding to part of some geometry that has been split and removed. Not sure if such phantom points that retain memory of discarded geometry are desirable.
In any case, the behavior of the node needs to be consistent, considering both the images attached above were generated from the same version.

@mccrone
Copy link

mccrone commented May 9, 2016

@vykrum if you want to discard the points that are past the surface, use the test: Geometry.DoesIntersect with the surface, then discard the points that return a false result with FilterByBoolMask.

@vykrum
Copy link
Author

vykrum commented May 9, 2016

@mccrone Earlier, had to use Point.Project in a similar way as a work around while using Surface.PointAtParameter. As of mid last month thought that wouldn't be necessary anymore. But seems like in some cases might still need to resort to that approach while in some other cases it won't be necessary as it works well.

@kronz
Copy link

kronz commented May 10, 2016

@aparajit-pratap

@riteshchandawar
Copy link

Here is the issue for internal tracking. http://adsk-oss.myjetbrains.com/youtrack/issue/MAGN-9320
Although the issue not similar to the above but while fixing that issue we can to consider this scenario as well.

Thanks,
Ritesh

@aparajit-pratap
Copy link

@riteshchandawar 9320 is an unrelated issue. That is related to the ASM scaling issue. Marking this as an improvement.

@Amoursol Amoursol transferred this issue from DynamoDS/Dynamo Apr 1, 2020
@Amoursol Amoursol added DesignScript DesignScript improvements Geometry Geometry improvements Nodes Nodal improvement labels Apr 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DesignScript DesignScript improvements Geometry Geometry improvements Nodes Nodal improvement
Projects
Status: Backlog
Development

No branches or pull requests

7 participants