Skip to content

use tforeach in vomps - #542

Merged
lkdvos merged 1 commit into
mainfrom
ksh/vomps
Oct 26, 2026
Merged

lkdvos merged 1 commit into
mainfrom
ksh/vomps

Conversation

@kshyatt

@kshyatt kshyatt commented Oct 20, 2026

Copy link
Copy Markdown
Member

A few other algos replaced spawn and sync with tforeach. One nice thing about it is it respects the scheduler choice in defaults. However VOMPS wasn't yet using this.

@kshyatt
kshyatt requested a review from lkdvos October 20, 2026 12:03
@codecov

codecov Bot commented Oct 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/algorithms/approximate/vomps.jl 0.00% 2 Missing ⚠️
Files with missing lines Coverage Δ
src/algorithms/approximate/vomps.jl 66.17% <0.00%> (+0.95%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lkdvos
lkdvos merged commit 189793d into main Oct 26, 2026
137 of 140 checks passed
@lkdvos
lkdvos deleted the ksh/vomps branch October 26, 2026 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants