Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix typo #69

Merged
merged 2 commits into from
Sep 4, 2023
Merged

Fix typo #69

merged 2 commits into from
Sep 4, 2023

Conversation

Adamantios
Copy link
Collaborator

@Adamantios Adamantios commented Sep 4, 2023

The following error was being raised from the core:

_check_matching_round_consistency
    round_to_behaviour[b.matching_round].append(b)

KeyError: <class 'packages.valory.skills.termination_abci.rounds.BackgroundRound'>

This indicates that the BackgroundRound did not have a matching behaviour specified. The reason was the typo on 1b1130e. As a result, the background_behaviours_cls attribute was never overridden to set the corresponding background behaviour.

ℹ️ Opened valory-xyz/open-autonomy#2042 to handle and show a more helpful error message on the core.

@DavidMinarsch DavidMinarsch merged commit fc9c1e5 into main Sep 4, 2023
@DavidMinarsch DavidMinarsch deleted the fix/typo branch September 4, 2023 09:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants