Skip to content

[stdlib][math] implementation of log10 and bug fix#6181

Open
lamija-i wants to merge 3 commits intomodular:mainfrom
lamija-i:log10-bug-v2
Open

[stdlib][math] implementation of log10 and bug fix#6181
lamija-i wants to merge 3 commits intomodular:mainfrom
lamija-i:log10-bug-v2

Conversation

@lamija-i
Copy link
Copy Markdown
Contributor

Summary

Implemented log10 in math.mojo and fixed a related bug.

Changes

  • Added implementation of log10 in mojo/stdlib/std/math/math.mojo
  • Updated GPU math tests in mojo/stdlib/test/math/gpu/test_math.mojo
  • Fixed issue causing incorrect behavior in the previous implementation

Testing

  • All existing tests pass
  • Added/updated tests for log10

Notes

This PR ensures correct behavior and test coverage for log10.

#5856

@lamija-i lamija-i requested a review from a team as a code owner March 16, 2026 00:32
@github-actions github-actions bot added mojo-stdlib Tag for issues related to standard library waiting-on-review labels Mar 16, 2026
@lamija-i lamija-i requested a review from a team as a code owner March 16, 2026 01:12
@lamija-i lamija-i changed the title [BUG][mojo-lang] implementation of log10 and bug fix [stdlib][math] implementation of log10 and bug fix Mar 16, 2026
@abduld
Copy link
Copy Markdown
Contributor

abduld commented Mar 19, 2026

I do not see an implementation

@JoeLoser JoeLoser added waiting for response Needs action/response from contributor before a PR can proceed and removed waiting-on-review labels Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mojo-stdlib Tag for issues related to standard library waiting for response Needs action/response from contributor before a PR can proceed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants