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.

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

Handle ClientApp exception simulation #3075

Merged
merged 48 commits into from
Mar 28, 2024

Conversation

jafermarq
Copy link
Contributor

@jafermarq jafermarq commented Mar 7, 2024

  • Upon message execution exception, return error_reply message.
  • Attempt loading ClientApp before launching simulation backend (mirroring more closely in this way what happens in the SupperNode)
  • Simplifications related to LoadClientAppError in the backend as a result.

@jafermarq jafermarq added the simulation An issue related to Flower's simulation label Mar 7, 2024
@jafermarq jafermarq changed the base branch from main to handle-flower-callable-exception March 7, 2024 12:09
Base automatically changed from handle-flower-callable-exception to main March 28, 2024 13:35
@jafermarq jafermarq marked this pull request as ready for review March 28, 2024 16:42
@danieljanes danieljanes enabled auto-merge (squash) March 28, 2024 18:01
@danieljanes danieljanes merged commit 531e0e3 into main Mar 28, 2024
28 checks passed
@danieljanes danieljanes deleted the handle-clientapp-exception-simulation branch March 28, 2024 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
simulation An issue related to Flower's simulation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants