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

Human experiments #29

Open
wants to merge 75 commits into
base: comm_state
Choose a base branch
from
Open

Human experiments #29

wants to merge 75 commits into from

Conversation

wwongkamjan
Copy link
Contributor

No description provided.

@wwongkamjan wwongkamjan changed the base branch from main to deceptive_cicero October 30, 2023 03:27
@wwongkamjan wwongkamjan changed the base branch from deceptive_cicero to comm_state October 30, 2023 03:28
wwongkam and others added 30 commits March 20, 2024 16:57
This is required for the unit tests that I fixed on another branch and
cherry picked to this one to pass.
This branch does not install `daide2eng`, so the tests would fail if
they were run.
I based this commit on a draft Dockerfile from Joy and existing
documentation and `Dockerfile`s in this repository.
The server I've been testing with uses TLS, so we should be able to
connect using it.
This can be called within `milaWrapper` to use the API in
`chiron-utils`. `milaWrapper` would ideally be replaced with a subclass
of `BaselineBot`, but that will take more effort than it's worth at the
moment.
I was initially testing without a deadline, and the bot would raise
exceptions or skip messaging when no deadline was set.
We are only playing with a single game type (no DAIDE), so the
associated logic is redundant.
This does not appear to be used anymore.
I removed code that set variables that were never used, as well as some
commented-out code.
Mixing the two styles was making reading the console output difficult.
Unlike using `logging.info()`, this ensures the full stack trace is
being output.
Previously, our bot was being treated as a human player.
Incorporate `chiron-utils` into CICERO player
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