Skip to content

Commit

Permalink
Merge pull request #484 from JamesWrigley/undefined-variable
Browse files Browse the repository at this point in the history
Reference Dagger.EAGER_THUNK_STREAMS explicitly
  • Loading branch information
jpsamaroo authored Mar 14, 2024
2 parents 6b9a1af + 73c62dc commit b4f41c8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/sch/eager.jl
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ function eager_cleanup(state, uid)
delete!(state.thunk_dict, tid)
end
remotecall_wait(1, uid) do uid
lock(EAGER_THUNK_STREAMS) do global_streams
lock(Dagger.EAGER_THUNK_STREAMS) do global_streams
if haskey(global_streams, uid)
delete!(global_streams, uid)
end
Expand Down
2 changes: 1 addition & 1 deletion src/submission.jl
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ end
function EagerThunkMetadata(spec::EagerTaskSpec)
f = chunktype(spec.f).instance
arg_types = ntuple(i->chunktype(spec.args[i][2]), length(spec.args))
return_type = Base._return_type(f, Base.to_tuple_type(arg_types))
return_type = Base.promote_op(f, arg_types...)
return EagerThunkMetadata(return_type)
end
chunktype(t::EagerThunk) = t.metadata.return_type
Expand Down

0 comments on commit b4f41c8

Please sign in to comment.