Skip to content

Wrap thunk functions in Chunk #271

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
Merged
merged 7 commits into from
Sep 22, 2021
Merged

Wrap thunk functions in Chunk #271

merged 7 commits into from
Sep 22, 2021

Conversation

jpsamaroo
Copy link
Member

@jpsamaroo jpsamaroo commented Aug 26, 2021 •
edited
Loading

Closes #270, closes #148

TODO

  • Implement function awareness in can_use_proc and schedule!
  • Fix Thunk printing
  • Implement support in delayed
  • Handle function chunks in eviction
  • Fix any breakage in DaggerWebDash
  • Add basic tests for callable structs
  • Add tests for correct processor and scope
  • Add docstring for new kwargs
  • Add docs on using callable structs

@jpsamaroo jpsamaroo mentioned this pull request Sep 21, 2021
Merged
@jpsamaroo jpsamaroo force-pushed the jps/function-chunks branch 3 times, most recently from 008ba4d to 343ab2c Compare September 22, 2021 21:08
@jpsamaroo jpsamaroo marked this pull request as ready for review September 22, 2021 21:08
@jpsamaroo
Copy link
Member Author

Now also partially addresses #150 by requiring a name (as String or Symbol) when adding processor callbacks.

@jpsamaroo jpsamaroo merged commit 2fdaa89 into master Sep 22, 2021
@jpsamaroo jpsamaroo deleted the jps/function-chunks branch September 22, 2021 22:35
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
data movement performance scheduler
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Treat thunk functions as Chunks Cache serialized functions and closures
1 participant
@jpsamaroo