Skip to content

KiteModels.jl: a definition for KPS3 and KPS4 #322

Description

@1-Bort-1

KPS3 and KPS4 describe their system with a scalar segment count and a handful of settings. Build a SystemDefinition from those instead — points, segments, the tether, the winch — so a KPS4 run can be drawn by anything that reads a definition, which is the whole point of the plan and the thing KiteViewers cannot do today.

@reexport using KiteGeometry. The mirror removal of unit mirror applies here too where model structs carry positions that belong to the state.

sysstate is a need rather than a nicety: this repo writes aero_force_b and tether_induced_force per step, and those are the fields that unit renames. KiteModels #312 is the open frame pull request here, so it is your base while it is open.

The check is that an existing KPS4 example runs unchanged and its log carries a definition whose point count and connectivity match what the model actually integrated.

Unit kitemodels of plan plans/topology_state_split_plan.md, tracked in 1-Bart-1/Agents#294.
https://github.com/1-Bart-1/Agents/blob/4ee7e2b839cdd358bef56ef3b00097f4b5702f50/plans/topology_state_split_plan.md

Activity

  1. self-assigned this
    on Sep 26, 2026
  2. 1-Bort-1 commented on Sep 26, 2026

    @1-Bort-1
    MemberAuthor

    Adding system_definition(s) for KPS3 and KPS4. It builds a KiteGeometry SystemDefinition from the springs the model actually integrates, adds @reexport using KiteGeometry (pinned through [sources] until the package is registered), and puts the definition in the saved log's metadata under topology. Stacked on #312, which is now on KiteUtils 0.13.
    Not touching aero_force_b/tether_induced_force: this repo writes neither. The bin/install fix is in #323.

  3. 1-Bort-1 commented on Sep 26, 2026

    @1-Bort-1
    MemberAuthor

    Question from KiteModels.jl-322

    I can't stack on #312: frame-unification is 29 commits behind main, and agent stack 312 fails because of a conflict in CHANGELOG.md, the only file that conflicts (Project.toml and src/KiteModels.jl merge cleanly). How should I proceed?
    A) Merge origin/main into frame-unification (a merge, not a rebase), push it to #312, then stack on it. I'd resolve only the CHANGELOG hunk.
    B) You or #312's author update it, and I stack once it is green.
    C) Build on #321 or main and leave #312's frame changes out of this unit.
    I recommend A. #312 also has no CI run yet, so the stack would wait on its checks either way. One more thing: nothing in this repo writes aero_force_b or tether_induced_force, so the sysstate rename has nothing to change here. The log does need KiteUtils 0.13 for SysLog.metadata, and OpenSourceAWE/KiteViewers.jl#56 gates that.

  4. 1-Bart-1 commented on Sep 27, 2026

    @1-Bart-1
    Member

    #312 is getting updated

  5. 1-Bort-1 commented on Sep 30, 2026

    @1-Bort-1
    MemberAuthor

    The plan changed what this unit asks for, so the body above is now kitemodels as plans/topology_state_split_plan.md words it today. Read it again before you go on.

    https://github.com/1-Bart-1/Agents/blob/4ee7e2b839cdd358bef56ef3b00097f4b5702f50/plans/topology_state_split_plan.md

  6. 27 remaining items

  7. changed the title [-]One kite system definition across the Kite packages: KiteModels.jl: a definition for KPS3 and KPS4[/-] [+]KiteModels.jl: a definition for KPS3 and KPS4[/+] on Oct 1, 2026
  8. added
    plan:unitFiled from a plan the box is scheduling
    agent:ciAgent task state
    and removed
    agent:ciAgent task state
    on Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

agent:reviewAgent task stateplan:unitFiled from a plan the box is scheduling

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions