-
Notifications
You must be signed in to change notification settings - Fork 61
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
AEA-608 Minor changes to dialogue API #1442
Conversation
…es except initial one; formatting;
…feature/dialogue
… protocols and updated skills with AgentRole -> Role change; Black now applies to _pb2.py files;
@@ -68,11 +68,11 @@ class TacDialogue(Dialogue): | |||
), | |||
} | |||
|
|||
class AgentRole(Dialogue.Role): | |||
class Role(Dialogue.Role): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The roles are not auto sorted by alphabet as you can see below
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please amend generator to do so
# Run protocol buffer compiler | ||
try_run_protoc( | ||
self.path_to_generated_protocol_package, self.protocol_specification.name | ||
) | ||
|
||
# Run black formatting | ||
try_run_black_formatting(self.path_to_generated_protocol_package) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Running black
after protoc
so it formats _pb2.py
files too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM; only one issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This is a minor PR with the following changes:
AgentRole
-->Role
in FipaDialogue.counterparty
field is automatically set for all messages except the initial one. This can be done for the initial message once a newcreate_dialogue
method is added which gets thecounterparty
as a field.Due to the changed names (item 1 and 2 above), skill codes no longer work and linters fail. Those were not changed because skills are being heavily changed in another branch.
Types of changes
What types of changes does your code introduce to agents-aea?
Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply.develop
branch (left side). Also you should start your branch off ourdevelop
.aea cli
tool works