Optimize function - Githubissues

allora-network / allora-chain

Node software to run the Allora Network

https://www.allora.network/

Apache License 2.0

73 stars 67 forks source link

Optimize function #431

Closed ylmin closed 1 month ago

ylmin commented 1 month ago

Closes: #XXX

What is the purpose of the change

Add a description of the overall background and high level changes that this PR introduces

GetSortedKeys: Initialized the slice with make([]K, 0, len(m)) to avoid slice resizing.
LinearInterpolation: Simplified the interval search in linear interpolation using sort.Search, reducing the number of iterations.

Testing and Verifying

(Please pick one of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

Added unit test that validates ...
Added integration tests for end-to-end deployment with ...
Extended integration test for ...
Manually verified the change by ...

Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Allora documentation site docs.allora.network source code at: https://github.com/allora-network/docs
[ ] Code comments?
[ ] N/A

kpeluso commented 1 month ago

Thank you for the contribution!