Skip to content

docs(changelog): describe the circulation velocity as the effective velocity - #172

Merged
ocayon merged 1 commit into
developfrom
docs/changelog-effective-velocity
Nov 4, 2026
Merged

ocayon merged 1 commit into
developfrom
docs/changelog-effective-velocity

Conversation

@ocayon

@ocayon ocayon commented Nov 4, 2026

Copy link
Copy Markdown
Member

Rewords the circulation entry of the 2.3.0 changelog so it says which velocity is meant: the span-perpendicular effective velocity, including induction. It also states the old formula in the same terms. Documentation only, no effect on results.

Written by Claude, reviewed by Oriol Cayon

…elocity

Say that the circulation uses the span-perpendicular effective velocity,
including induction, and state the old formula in the same terms.

Documentation only, no effect on results.

Generated with Claude Code
Copilot AI balanced review requested due to automatic review settings November 4, 2026 11:56
@ocayon ocayon mentioned this pull request Nov 4, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
5092 3943 77% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: a132267 by action🐍

@ocayon
ocayon merged commit a0fe3aa into develop Nov 4, 2026
1 check passed
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.

2 participants