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

185999014 - Fix single process simulator #877

Merged
merged 14 commits into from
Oct 24, 2023

Conversation

iduartgomez
Copy link
Collaborator

  • Fixes to simulated network implementation so it works correctly.
  • Now simulated network noise can be turned off per simulation. (Noise implementation has some errors that will be tackled in other PR.)
  • Fixed multiple errors in the op implementations (join ring, which now is connect).
  • Improved and fixed the event log (was necessary so checks can be done correctly on tests).
  • Added some missing todos functionality.
  • Handle concurrent messages for a single transaction gracefully.
  • Some preliminary work to takle #184207118, this fixed some of the issues with the ops.

@CLAassistant
Copy link

CLAassistant commented Oct 24, 2023

CLA assistant check
All committers have signed the CLA.

@iduartgomez iduartgomez force-pushed the 185999014-fix-simulation branch from 3021034 to 25e0cd8 Compare October 24, 2023 09:14
@iduartgomez iduartgomez force-pushed the 185999014-fix-simulation branch from b160dd0 to 368d003 Compare October 24, 2023 13:31
@iduartgomez iduartgomez merged commit cb51219 into freenet:main Oct 24, 2023
3 checks passed
@iduartgomez iduartgomez deleted the 185999014-fix-simulation branch October 24, 2023 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants