Skip to content

Parametrized flows #2

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.

Sign up for GitHub

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

Jump to bottom
Open
cflinto wants to merge 158 commits into
base: master
Choose a base branch
from
Open

Parametrized flows #2

cflinto wants to merge 158 commits into from

Conversation

@cflinto
Copy link

@cflinto cflinto commented Oct 7, 2022

This PR is a baseline to work on the parametrized flow feature.

@cflinto
Copy link
Author

cflinto commented Oct 7, 2022

Task 1: make the G[...] available in the BODYs

@cflinto
Copy link
Author

cflinto commented Oct 7, 2022

Task 2: Create a variable number of flows in the structure (-> init function)

@cflinto
Copy link
Author

cflinto commented Oct 7, 2022

Task 3: After all the flows are created, fix the values (masks, links, ...).

@cflinto
Copy link
Author

cflinto commented Oct 7, 2022

Tasks 4: Allow the iterator (p=m..M) to be referenced in the expression in case of a dependency with a ternary (-> Check how it's handled in the broadcasts).

@cflinto
Copy link
Author

cflinto commented Oct 12, 2022

Task 5: allow specifying a flow without an iterator e.g.:
RW Dat[3] <- ....

cflinto added 25 commits February 14, 2023 18:13
@cflinto
revert make_key to the original version
1227a36
@cflinto
complete the previous revert
e0dbfa5
@cflinto
use parametrized flow id if needed in reshaping
c783593
@cflinto
use variable for flow_index in the reshaping from deps
137757c
@cflinto
correct flow id for affecting repo_entry->data to NULL
dbdc338
@cflinto
correct the number of flows if parametrized
dbad205
@cflinto
use data_dynamic_offset_of_parametrized_ instead of flow_id + only de…
1193e14 
…clare in/out masks if the flow is in/out
@cflinto
add asserts + function to find a flow_id + switch back to using spec_…
81fe29f 
…%s_%s.flow_id_of_flow_of_%s_%s_for_%s
@cflinto
change the flow_index affectation so that it works for READ/WRITE-onl…
26b807a 
…y flows
@cflinto
update dependencies_goal later
47bc442
@cflinto
correction
ebb8194
@cflinto
correct parsec_check_sanity_of_task_class
99d818f
@cflinto
update the parametrized iterator so that it can be used in conditions
676a5f2
@cflinto
declare the referrer iterators before normal locals so that it can be…
6ca5ced 
… used in their expressions
@cflinto
correct some mistakes in jdf2c + separate in/out flows in parsec_debu…
0f11901 
…g_dump_task_class_at_exec
@cflinto
fix error in parsec_helper_get_flow_index
6ce5803
@cflinto
construct dep_mask_out_of_flow_of_%s_%s_for_%s with dep_index instead…
e84e2be 
… of flow_id
@cflinto
correct brackets in iterate_successors
6b355e6
@cflinto
open brackets + increase the str size of the mask
f5aa873
@cflinto
correct flow_id for gpu tasks + correct dump flow id + some minor cha…
f32bf5a 
…nges
@cflinto
correct the filling ot the gpu_task structure
c6b7d85
@cflinto
switch to a 64-bit mask for dependencies
cce19f8
@cflinto
separate dep_index lifting from the dep shift
1697ae6
@cflinto
fix nb of specializations given to parsec_update_dep_index_after_dep
d952ca2
@cflinto
remove trash comments
92f8ddd
@bosilca bosilca self-requested a review as a code owner October 5, 2023 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bosilca bosilca Awaiting requested review from bosilca

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@cflinto