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.

Sign up for GitHub

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

Jump to bottom

IfcThirdOrderPolynomialSpiral #550

Merged
merged 4 commits into from
Aug 1, 2022
Merged

IfcThirdOrderPolynomialSpiral #550

merged 4 commits into from
Aug 1, 2022

Conversation

Elvira2227
Copy link
Collaborator

@Elvira2227 Elvira2227 commented Jun 13, 2022 •
edited
Loading

In this pull request I implemented functions for reading the IfcThirdOrderPolynomialSpiral.
Now we can implement "Bloss curve" and this pull request can be addressed to issue #524.
In the CurveConverter.h I added:

  1. getPointOnCurve()
  2. getDirectionOfCurve()
  3. additional statement in convertIfcCurveSegment()

Here you can see the screen shot (it presents BlossCurve_100.0_300_1000_1_Meter) :
image

@Elvira2227 Elvira2227 requested review from pjanck and jschlenger June 13, 2022 14:57
@Elvira2227 Elvira2227 self-assigned this Jun 13, 2022
@Elvira2227 Elvira2227 added the IFC Content related to Industry Foundation Classes (IFC) functionalities label Jun 13, 2022
@Elvira2227 Elvira2227 mentioned this pull request Jun 13, 2022
Open
6 tasks
pjanck
pjanck approved these changes Jul 19, 2022
View reviewed changes
Copy link
Contributor

@pjanck pjanck left a comment

Choose a reason for hiding this comment

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

I assume that unit tests work correctly.

jschlenger
jschlenger reviewed Jul 27, 2022
View reviewed changes
Copy link
Collaborator

@jschlenger jschlenger left a comment

Choose a reason for hiding this comment

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

Have you checked that you are not only visualizing the curve that Peter added to the IFC file to visually check if the calculation is done right?
(as was the case for IfcSine in the beginning)

@jschlenger
Copy link
Collaborator

Everything seems fine. Let's merge this.

@jschlenger jschlenger merged commit 9cb2cce into tumcms:development Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jschlenger jschlenger jschlenger left review comments

@pjanck pjanck pjanck approved these changes

Assignees

@Elvira2227 Elvira2227

Labels
IFC Content related to Industry Foundation Classes (IFC) functionalities
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Elvira2227 @jschlenger @pjanck