Skip to content

Conversation

@LilithCole
Copy link
Contributor

@LilithCole LilithCole commented Nov 14, 2025

I think that covers it, if you need anything else get in touch.

There are probably formatting errors in this. If you are feeling petty you can tell me to do them when I'm back from thailand, otherwise it will be quicker for somebody else to look at them.

@LilithCole LilithCole added this to Tasks Nov 14, 2025
@github-project-automation github-project-automation bot moved this to Flash Review in Tasks Nov 14, 2025
@LilithCole LilithCole self-assigned this Nov 14, 2025
| FAST_RATE | The ramp rate to use for the fast ramps | 0.5 |
| FAST_PERSISTENT_SETTLETIME | The number of seconds to settle after a ramp fast to persist | 5 |
| PERSISTENT_SETTLETIME | The number of seconds to settle after a ramp to persist | 60 |
| FILTER_VALUE | The value to use to filter the target reached calculation | 0.1 |
Copy link
Member

Choose a reason for hiding this comment

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

The value to use to filter the target reached calculation

What does this mean?

| PERSISTENT_SETTLETIME | The number of seconds to settle after a ramp to persist | 60 |
| FILTER_VALUE | The value to use to filter the target reached calculation | 0.1 |
| FAST_FILTER_VALUE | The value to use to filter the target reached calculation when ramping fast to a target | 1 |
| NPP | The npp value when calculating whether or not the target has been reached | |
Copy link
Member

Choose a reason for hiding this comment

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

The npp value

What does this mean?

@Tom-Willemsen Tom-Willemsen merged commit bd0e5f3 into master Nov 14, 2025
2 checks passed
@Tom-Willemsen Tom-Willemsen deleted the add-most-of-myundocumented-cryomag-knowledge branch November 14, 2025 18:20
@Tom-Willemsen Tom-Willemsen removed this from Tasks Nov 14, 2025
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.

3 participants