Skip to content

Fix calculus units for unit-sphere derivatives - #1876

Draft
rajeeja wants to merge 2 commits into
mainfrom
rajeeja/calculus-units-1856
Draft

rajeeja wants to merge 2 commits into
mainfrom
rajeeja/calculus-units-1856

Conversation

@rajeeja

@rajeeja rajeeja commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Closes #1856. Gradient, curl, and divergence now label radius-one results per radian and strip surrounding whitespace from input units, without changing numerical scaling. Adds regression coverage and expands the existing vector calculus guide with a unit-sphere versus physical-radius example.

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

Units in calculus methods: sphere_radius=1 causes "/m" not "/rad"; whitespace isn't stripped

1 participant