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
Feature: IfcSweptDiskSolidPolygonal Informal Proposition #3
base: main
Are you sure you want to change the base?
Feature: IfcSweptDiskSolidPolygonal Informal Proposition #3
Conversation
Sandbox ok a1d0aa7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First comments. Did not look through the whole PR yet.
Then FilletRadius has to be smaller than or equal to the length of the start and end segments of the Directrix | ||
And FilletRadius has to be smaller than or equal to the length / 2 of the inner segments of the Directrix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Breadcrumbs: #20 - here is another section that should be amended accordingly if we strive for maximum reusability.
And FilletRadius = not null | ||
And Directrix forms a closed curve | ||
|
||
Then FilletRadius has to be smaller than or equal to the length of all segments of the Directrix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How come not length / 2
, similarly to the open curve example?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pjanck I think you're correct. A closed curve means all segments are the "inner segments" and hence the length / 2 rule should apply. Thank you.
There are also no test cases yet for closed curve directrices.
And FilletRadius = not null | ||
And Directrix forms an open curve | ||
|
||
Then FilletRadius has to be smaller than or equal to the length of the start and end segments of the Directrix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please change "shall" to "must" as per #20
And FilletRadius = not null | ||
And Directrix forms a closed curve | ||
|
||
Then FilletRadius has to be smaller than or equal to the length of all segments of the Directrix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
idem, change "shall" to "must" as per #20
This PR adds support for the Informal Proposition that compares FilletRadius to the length of the segments of the directrix.