Skip to content

Bounded Dec: Utility functions to cap value#787

Merged
xmariachi merged 4 commits intodevfrom
diego/engn-3604-boundeddec-add-new-capped-functions
Apr 1, 2025
Merged

Bounded Dec: Utility functions to cap value#787
xmariachi merged 4 commits intodevfrom
diego/engn-3604-boundeddec-add-new-capped-functions

Conversation

@xmariachi
Copy link
Contributor

@xmariachi xmariachi commented Mar 31, 2025

Purpose of Changes and their Description

  • Utility function in bounded_dec to cap value to limits.

Link(s) to Ticket(s) or Issue(s) resolved by this PR

ENGN-3604

Are these changes tested and documented?

  • If tested, please describe how. If not, why tests are not needed. -- unit tests added
  • If documented, please describe where. If not, describe why docs are not needed. -- no need, just utility methods
  • Added to Unreleased section of CHANGELOG.md?

@xmariachi xmariachi changed the title Utility functions to cap value Bounded Dec: Utility functions to cap value Mar 31, 2025
@xmariachi xmariachi marked this pull request as ready for review March 31, 2025 16:50
amimart
amimart previously approved these changes Mar 31, 2025
Copy link

@felipead felipead left a comment

Choose a reason for hiding this comment

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

Nice testing!

@xmariachi xmariachi merged commit 85172d0 into dev Apr 1, 2025
9 checks passed
@xmariachi xmariachi deleted the diego/engn-3604-boundeddec-add-new-capped-functions branch April 1, 2025 12:04
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.

5 participants