Skip to content

removes for loop improving runtime of baseline imputer#498

Merged
Advueu963 merged 1 commit intomainfrom
fix-481
Mar 12, 2026
Merged

removes for loop improving runtime of baseline imputer#498
Advueu963 merged 1 commit intomainfrom
fix-481

Conversation

@mmschlk
Copy link
Copy Markdown
Owner

@mmschlk mmschlk commented Mar 12, 2026

Motivation and Context


Public API Changes

  • No Public API changes
  • Yes, Public API changes (Details below)

How Has This Been Tested?

A new test.


Checklist

  • The changes have been tested locally.
  • Documentation has been updated (if the public API or usage changes).
  • An entry has been added to CHANGELOG.md (if relevant for users).
  • The code follows the project's style guidelines.
  • I have considered the impact of these changes on the public API.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Collaborator

@Advueu963 Advueu963 left a comment

Choose a reason for hiding this comment

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

Nice speedup!

@Advueu963 Advueu963 merged commit 62c10dd into main Mar 12, 2026
14 checks passed
@Advueu963 Advueu963 deleted the fix-481 branch March 12, 2026 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

remove the for loop in the value function of baseline imputer

2 participants