Skip to content
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

Make VCE use Message and Flower callable #2783

Merged
merged 43 commits into from
Feb 18, 2024
Merged

Make VCE use Message and Flower callable #2783

merged 43 commits into from
Feb 18, 2024

Conversation

jafermarq
Copy link
Contributor

@jafermarq jafermarq commented Jan 10, 2024 •
edited
Loading

Changes:

  • Main change: Actors in the VCE operate with Messages.
  • Removed old proxy.
  • Client handler makes use of node_id when executing client_fn

To be merged after #2912

@jafermarq jafermarq added the simulation An issue related to Flower's simulation label Jan 10, 2024
@jafermarq jafermarq changed the title Make VCE use TaskIns/TaskRes Make VCE use Message and Flower callable Jan 29, 2024
@jafermarq jafermarq changed the base branch from main to add-node-id-to-metadata February 7, 2024 20:30
@jafermarq jafermarq marked this pull request as ready for review February 12, 2024 21:05
Base automatically changed from add-node-id-to-metadata to main February 13, 2024 17:16
@danieljanes danieljanes enabled auto-merge (squash) February 18, 2024 14:54
@danieljanes danieljanes merged commit ce2d970 into main Feb 18, 2024
27 checks passed
@danieljanes danieljanes deleted the vce-with-taskin-taskres branch February 18, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danieljanes danieljanes danieljanes approved these changes

@tanertopal tanertopal Awaiting requested review from tanertopal tanertopal is a code owner

Assignees
No one assigned
Labels
simulation An issue related to Flower's simulation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jafermarq @danieljanes @panh99