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

Parsing the event logs relies on the order in which user input and the resulting task action are generated, this is prone to future bugs! #37

Open
BenedictWilkins opened this issue Nov 1, 2024 · 0 comments
Labels
bug Something isn't working priority:high A high priority issue

Comments

@BenedictWilkins
Copy link
Contributor

The event log parser makes use of the accidental fact that user input events proceed the mapped task related event to determine which events are triggered by a user. This dependence is BAD as it the ordering of logged events depends on the order of the actuator execute calls, if the actuator is ever reordered (or the event logging system changes slightly), then this will break.

A MUCH BETTER SOLUTION:

  • Record which actuators belong to which agent in the event log, and map the source field to the given agent to determine if its an avatar that produced the action.
@BenedictWilkins BenedictWilkins added the bug Something isn't working label Nov 1, 2024
@BenedictWilkins BenedictWilkins added the priority:high A high priority issue label Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority:high A high priority issue
Projects
None yet
Development

No branches or pull requests

1 participant