Skip to content

fix(output): the geometric curvature and section moment of a stretched line - #99

Merged
SMI-Lab-Inha merged 1 commit into
devfrom
fix/curvature-stretch-definition
Nov 7, 2026
Merged

SMI-Lab-Inha merged 1 commit into
devfrom
fix/curvature-stretch-definition

Conversation

@SMI-Lab-Inha

Copy link
Copy Markdown
Owner

Summary

The node outputs of a finite-EI line read the conjugate of the tangent as the moment. The element energy is EI κ²/2 per unstretched length, so that conjugate has normal length EI κ/(1+ε)², which put Curv 2ε and BendMom ε below their definitions.

  • Curv (and CurvX, CurvY) is now the geometric curvature of the stretched line, the reciprocal of the bend radius.
  • BendMom is the moment that balances the forces, EI κ/(1+ε), and StfMom the sleeve's share of it.
  • BendMom (1+ε) = EI × Curv, so the two no longer agree exactly; the difference is the strain.

Result changes

Tension, positions and forces do not change. On the 12 example decks with finite-EI lines only the curvature and moment columns move, at most 0.024 % and 0.012 %, and at most 0.0026 % at peak curvature. On a soft line the change is about 2ε: 2.3 % at 1.17 % strain.

Checks

New test driver_curvature_stretch against the elastica of the same energy, at EA 2e6 (strain up to 1.17 %):

Quantity Before After
Curv 2.30 % 0.006 %
BendMom 1.15 % under 0.001 %
Difference from the exact trace 2.33 % 0.17 % (mesh error)
  • It fails on the old definition at all three EA values.
  • With a rotational spring at the end, BendMom equals the spring moment at 1.00000 (0.98944 before).
  • driver_bend_stiffener, its reference now on the same energy: cable moment 0.18 → 0.01 % of root, node positions 0.6 → 0.03 mm.
  • CTest 360 of 360 on the default kernel and with OPENBLAS_CORETYPE=Prescott; no new compiler warnings.

The CHANGELOG entry lands with the release batch.

… a stretched line

The element energy is EI kappa^2/2 per unstretched length with kappa the
curvature of the stretched line, so the recovered m, the conjugate of the
tangent r', has the normal length EI kappa/(1 + e)^2. The node outputs read
that length as the moment and divided it by EI, which put Curv 2e and
BendMom e below the values they are defined by.

The energy does not change under a rotation of the whole line, so the
moment that balances the forces is r' x m, of length M = EI kappa/(1 + e).
BendMom is now M, Curv is (1 + e) M/EI, the reciprocal of the bend radius,
and StfMom the sleeve's share of M, with the stretch |r'| of the node's
tangent. Tension, positions and forces do not change. Under OutputRecovery
element the exact traces and EI x curvature stand on unsleeved elements.

driver_curvature_stretch holds both against the elastica of this energy,
strain swept to 1.2 %: the curvature error falls from 2.30 % to 0.006 % and
the moment error from 1.15 % to under 0.001 %. driver_bend_stiffener takes
the same elastica: the cable moment moves from 0.18 % to 0.01 % of the root
value and the node positions from 0.6 to 0.03 mm.

Signed-off-by: Jae Hoon Seo <jaehoon.seo@inha.ac.kr>
@SMI-Lab-Inha

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 7, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-07T15:37:42.399112Z 71b9dd4 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 CableDyn | 🛠️ Build #35720871 | 📁 Comparing 71b9dd4 against latest (3d04008)

  🔍 Preview build  

18 files changed · ± 18 modified

± Modified

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: 71b9dd4bf2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@SMI-Lab-Inha
SMI-Lab-Inha merged commit 7d66c91 into dev Nov 7, 2026
21 checks passed
@SMI-Lab-Inha
SMI-Lab-Inha deleted the fix/curvature-stretch-definition branch November 7, 2026 17:35
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.

1 participant