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

Different units for trigonometric functions in PowerToys Run calculator plugin #36717

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

OldUser101
Copy link

Summary of the Pull Request

Added option to select angle unit for trigonometric calculations in the PowerToys Run calculator plugin.

PR Checklist

Detailed Description of the Pull Request / Additional comments

Added option in settings to allow the user to select between "Radians", "Degrees", or "Gradians".
Extra expression parsing used to add mathematical conversions for the units, as Mages does not natively support units other than radians.
Added suitable tests of the extra expression parsing, all pass.

Validation Steps Performed

Suitable tests added for extra expression parsing, all pass.
Tested feature mathematically.

@OldUser101
Copy link
Author

@microsoft-github-policy-service agree

This comment has been minimized.

This comment has been minimized.

@OldUser101 OldUser101 requested a review from htcfreek January 12, 2025 08:41
Copy link
Collaborator

@htcfreek htcfreek left a comment

Choose a reason for hiding this comment

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

Looks good to me. athank you for contribution.

@htcfreek htcfreek added the Needs-Review This Pull Request awaits the review of a maintainer. label Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs-Review This Pull Request awaits the review of a maintainer.
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants