Skip to content

Fix trigonometry alias problem#1891

Merged
adrianVmariano merged 20 commits intoBelfrySCAD:masterfrom
RAMilewski:master
Jan 30, 2026
Merged

Fix trigonometry alias problem#1891
adrianVmariano merged 20 commits intoBelfrySCAD:masterfrom
RAMilewski:master

Conversation

@RAMilewski
Copy link
Contributor

@RAMilewski RAMilewski commented Jan 29, 2026

Moved former "aliases" to full function entries.

Resolves #1805

@RAMilewski RAMilewski changed the title Fix triganometry alias problem Fix trigonometry alias problem Jan 29, 2026
Add new entries to See Also lines.
@RAMilewski RAMilewski marked this pull request as ready for review January 29, 2026 18:00
@adrianVmariano
Copy link
Collaborator

Maybe opp_hyp_to_adj the synopsis should flip order to match

The terms "primary angle" and "primary corner" appear throughout. There were there before, so not your fault....but I've never heard them and neither has the Claude AI, or google. So maybe not a term to use...or not without definition, at least. (And "primary" suggests biggest, maybe, which isn't right.)

Maybe "reference angle"? I came up with that on my own, but Claude also suggested it.

Instead of:

 ang = The angle in degrees of the primary corner of the right triangle.

maybe just

ang = reference angle of the right triangle in degrees

in all the places where it appears?

ang_adj_to_opp()

That's all I noticed.

replace primary angle and primary corner with reference angle.
@RAMilewski RAMilewski marked this pull request as draft January 30, 2026 16:47
@RAMilewski RAMilewski marked this pull request as ready for review January 30, 2026 17:06
@adrianVmariano adrianVmariano merged commit ffa493b into BelfrySCAD:master Jan 30, 2026
3 checks 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.

[BUG] Various trig functions have docs aliases when they should not.

2 participants