Bugfix: target power assert in ParlAIAllOrderIndependentWrapper

facebookresearch / diplomacy_cicero

Code for Cicero, an AI agent that plays the game of Diplomacy with open-domain natural language negotiation.

Other

1.3k stars 161 forks source link

Bugfix: target power assert in ParlAIAllOrderIndependentWrapper #22

Closed c-flaherty closed 1 year ago

c-flaherty commented 1 year ago

Fix for bug found by @ML-Chen:

  File "/home/ubuntu/diplomacy/diplomacy_cicero/parlai_diplomacy/wrappers/orders.py", line 1110, in _format_input_seq
    assert target_power is not None
AssertionError

We never ported this from private branch to release, but we should have.