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

Added a callback mechanism for capturing logs and updates during DI execution. Can be extended to other roles too. #1589

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

krish240574
Copy link

Features

Logs capture and log update mechanism for the DataInterpreter, Role, Team and Env classes, which can get displayed on a webpage using websockets.

  • xx
  • yy

Feature Docs

Influence

Result

Other

@krish240574 krish240574 marked this pull request as draft November 11, 2024 06:55
@davidleon
Copy link

team.py change contains space tab formatting change. Better retain the original formatting style. and minimize formatting change. Any example of using the gui_callback available? a small example would be great to illustrate how this enriches the frontend experience.

@krish240574
Copy link
Author

Sorry about that, David, I'll change back to spaces - let me put up an example of the usage , in a bit. Thanks for pointing that out.

Replaced tab formatting with spaces - 4.
@krish240574
Copy link
Author

krish240574 commented Nov 14, 2024

@davidleon - have fixed the formatting in team.py and added an example inside examples/gui_update_example.py. PLease note that I am actively working on improving the mesagefs and the quality of information communicated to the GUI, so I'll update this code periodically.

Included logic to use the gui update mechanism from a user program - much better logic, faster updates.
@krish240574
Copy link
Author

@davidleon - updated the user level example in examples/gui_update_example.py - better updates, better handling of the messages. Do take a look.

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