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

Partial lerp() implementation #2237

Merged

Conversation

ike709
Copy link
Collaborator

@ike709 ike709 commented Mar 5, 2025

#2203

Implements lerp() for nums. Other arg types like vectors are a future PR's problem.

@boring-cyborg boring-cyborg bot added Compiler Involves the OpenDream compiler Runtime Involves the OpenDream server/runtime labels Mar 5, 2025
@github-actions github-actions bot added the size/M label Mar 5, 2025
@ike709
Copy link
Collaborator Author

ike709 commented Mar 5, 2025

Requires vgstation-coders/vgstation13#37461 for CI to pass

@ike709 ike709 requested a review from wixoaGit March 6, 2025 13:34
@wixoaGit wixoaGit enabled auto-merge (squash) March 12, 2025 15:29
if (valA.TryGetValueAsFloatCoerceNull(out var floatA) && valB.TryGetValueAsFloatCoerceNull(out var floatB)) {
return new DreamValue(floatA + (floatB - floatA) * factor);
}
// TODO: Change this to a type mismatch runtime once the other valid arg types are supported

Check warning

Code scanning / InspectCode

Incorrect blank lines: Blank lines are missing elsewhere Warning

Blank lines are missing, expected minimum 1 instead of 0
@wixoaGit wixoaGit merged commit 7451f18 into OpenDreamProject:master Mar 12, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Compiler Involves the OpenDream compiler Runtime Involves the OpenDream server/runtime size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants