FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added for_each_index_nested ( do not merge yet ) by bradphelan · Pull Request #329 · taskflow/taskflow · GitHub

/ taskflow Public

Added for_each_index_nested ( do not merge yet ) - #329

Open
bradphelan wants to merge 1 commit into
taskflow:masterfrom
bradphelan:nested_loop
ConversationCommits (1)ChecksFiles changed
Open

Added for_each_index_nested ( do not merge yet )#329
bradphelan wants to merge 1 commit into
taskflow:masterfrom
bradphelan:nested_loop

Conversation

bradphelan commented Jun 24, 2021

Copy link
Copy Markdown
Contributor

Pull request for #328

I've added in for_each_index_nested however I cannot get the the test case to work. Have a look at the test case in

unittests/algorithm.cpp

if I use just for_each_index then the test case passes. If I use the implementation of for_each_index_nested as you reviewed yesterday then there is an error. Would you please have a look at and see what the problem may be.

XGZepto commented Feb 26, 2024

Copy link
Copy Markdown

Hi Brad,
came across this PR since it would be super helpful for the project I'm working out. I'll take a look at your code but do let us know if there's any update from your side since the PR was opened almost 3 years ago. Cheers.

bradphelan commented Feb 26, 2024

Copy link
Copy Markdown
Contributor Author

No we never updated this

avilleret commented Feb 6, 2025

Copy link
Copy Markdown

it seems this PR has never been reviewed while it contains useful improvement
Maybe @tsung-wei-huang could have a look ?

tsung-wei-huang commented Apr 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

The prospective v4.1 branch (current main) introduces a new N-dimensional IndexRange-based parallel algorithm that allow parallelization over a nested loop. See here - still experimental but features mostly complete.

This branch has not been deployed

No deployments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL