Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

library/math: Added standard math functions#49

Closed
obscuren wants to merge 1 commit into
masterfrom
obscuren-patch-1
Closed

library/math: Added standard math functions#49
obscuren wants to merge 1 commit into
masterfrom
obscuren-patch-1

Conversation

@obscuren
Copy link
Copy Markdown

Added standard math library, including:

  • Two dimensional euclidean distance
  • Square root
  • Linear interpolation

Added standard math library, including:

* Two dimensional euclidean distance
* Square root
* Linear interpolation
Comment thread library/math.sol
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why lerp?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Not sure what you mean with why?

Are you asking why I needed it here or are you asking about the reasoning behind adding it to a math lib?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think he's asking why the function is called lerp. FYI lerp is the canonical shortened name (I believe) for the Linear Interpolation function.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yes I see now. I should have paid closer attention to the implementation when asked to rename the functions.

@obscuren
Copy link
Copy Markdown
Author

Closing this in favour of #50 @VoR0220 took ownership.

@obscuren obscuren closed this Nov 22, 2015
@obscuren obscuren deleted the obscuren-patch-1 branch November 22, 2015 16:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants