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

Drop games with non-human players from model input #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion 02 - Feature engineering.ipynb
Original file line number Diff line number Diff line change
Expand Up @@ -794,7 +794,7 @@
"cell_type": "markdown",
"metadata": {},
"source": [
"Finally, let's exclude unrated games, because people play differently when it \"doesn't count.\" We'll also throw out untimed (\"Correspondence\") games, because we've mentioned that we're especially interested in the effect of time pressure on live win probability, and there is no real time pressure (or clock) when players have unlimited time."
"Finally, let's exclude unrated games, because people play differently when it \"doesn't count.\" We'll also throw out untimed (\"Correspondence\") games, because we've mentioned that we're especially interested in the effect of time pressure on live win probability, and there is no real time pressure (or clock) when players have unlimited time. Last but not least, we'll also remove all games involving non-human players."
]
},
{
Expand All @@ -806,6 +806,8 @@
"model_input_table = model_input_table.filter(\n",
" (model_input_table.is_rated)\n",
" & (model_input_table.lichess_time_control_type != \"Correspondence\")\n",
" & (model_input_table.white_title.fill_null(\"\") != \"BOT\")\n",
" & (model_input_table.black_title.fill_null(\"\") != \"BOT\")\n",
")"
]
},
Expand Down