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

Correct polynomial spiral equations #144

Closed
RickBrice opened this issue Feb 15, 2024 · 2 comments
Closed

Correct polynomial spiral equations #144

RickBrice opened this issue Feb 15, 2024 · 2 comments
Assignees

Comments

@RickBrice
Copy link

Problem
The $\lambda(u)$ equations for IfcSecond, IfcThird, and IfcSeventhOrderPolynomialSpiral are incorrect. The equations are of the form:

$$\lambda(u) = C + \int_{0}^{u}\cos \theta(t) dt x \int_{0}^{u}\sin \theta(t) dt y$$

The integrals yield scalar values while C is a location with length units.

Solution(s)
I think the equation should be
$$\lambda(u) = C + L ( \int_{0}^{u}\cos \theta(t) dt x \int_{0}^{u}\sin \theta(t) dt y )$$
where $L$ is the arc length when $u=1.0$.

The documentation might also benefit from stating $u=1.0$ when $\theta=\frac{\pi}{2}$

Require schema changes?

  • yes
  • no
  • don't know

Require documentation changes?

  • yes
  • no
  • don't know
@RickBrice
Copy link
Author

The hand calculations in issue #146 suggest that the integral equations may be correct, but there are likely other inaccuracies in the documented equations for the Ifc___PolynomialSpiralCurve types.

@RickBrice
Copy link
Author

After learning of the correct repository for ifc rail unit tests, I am able to exactly match horizontal and vertical for the polynomial spirals. Have not yet compared unit test results with cant. I’ll close this issue with no changes required.

@peterrdf @apinzenoehler @evandroAlfieri

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

No branches or pull requests

3 participants