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

Allow different parameterisation for thrust and power #65

Open
3 tasks
thclark opened this issue Mar 12, 2024 · 0 comments
Open
3 tasks

Allow different parameterisation for thrust and power #65

thclark opened this issue Mar 12, 2024 · 0 comments

Comments

@thclark
Copy link
Contributor

thclark commented Mar 12, 2024

Feature request

Use Case

We want to be able to use different parameterisation for thrust and power if necessary.

Current state

It's very common for power to be given as absolute values, while thrust is given as a coefficient. In our current setup, we have the same parameterisation for both power and thrust.

It's also awkward to have a 'cp' field containing power, with a boolean cp_is_coefficient.

Proposed Solution

{ 
  // ...
  "power": {
    "is_coefficient": false,
    "parameters": {
       // ...
    },
   "values": {
      // ...
    }
  },
  "thrust": {
    "is_coefficient": true,
    "parameters": {
       // ...
    },
   "values": {
      // ...
    }
  }
}

Supporting Octue

Octue is an Open-Source Software organisation helping you and many others to fight climate change. Providing support for your issues costs us c. 500GBP/day. Might your organisation be in a position to help sponsor this issue?

  • We already are (eg via technical partnership or GitHub Sponsors )
  • Yes (we'll prioritise this issue, and be in touch)
  • No (we'll work on this issue as and when it aligns with our core development activity)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: First draft
Development

No branches or pull requests

1 participant