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.

Sign up for GitHub

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

Jump to bottom

Next AIrsenal release [develop branch] #645

Merged
merged 52 commits into from
Mar 22, 2024
Merged

Next AIrsenal release [develop branch] #645

merged 52 commits into from
Mar 22, 2024

Conversation

jack89roberts
Copy link
Contributor

@jack89roberts jack89roberts commented Oct 6, 2023 •
edited
Loading

Before merging this, merge anguswilliams91/bpl-next#26 in bpl-next first and then change the bpl-next branch in pyproject.toml here back to main.

Closes #657

rchan26 and others added 30 commits December 30, 2022 16:07
@rchan26
type hints
926a131
@rchan26
get appropriate results_dict to fit neutral model
5084368
@rchan26
look at feature/neutral-dixon-coles branch of bpl
17d8491
@rchan26
update functions that use team model
b65453f
@rchan26
fix Dict type-hint
45b85e1
@rchan26
fix Dict type-hint when nested
d356d98
@rchan26
update tests and end_date retrieval bug
bd00417
@rchan26
change to check team_model type when computing outcome prob
6dcbc2f
@rchan26
type-hints for utils
a45c5ff
@rchan26
finish type-hints for utils
0f18788
@rchan26
type-hints for prediction utils
2158d22
@rchan26
allow for other player model class in type-hints
8aa118b
@rchan26
fix choices option in run_pipeline cli
d0c6e0c
@radka-j @rchan26
add time_diff to training data
1d9fb90 
Co-authored-by: rchan26 <[email protected]>
@radka-j
time_diff is number of days from max results date
3c674c2
@radka-j
add ratings as optional parameter to add_new_teams_to_model
6b06f18
@radka-j
Merge branch 'develop' into add_time_diff_to_data
d2b29d1
@radka-j
pass time_decay parameter to model.fit() method
91bbb14
@rchan26
lots of merge conflicts
1f018d7
@rchan26
Merge branch 'update-team-model' into add_time_diff_to_data
a0f1945
@rchan26
apply lint2
3b1b496
@rchan26
Merge pull request #591 from alan-turing-institute/add_time_diff_to_data
7170d7b 
Add time diff to data
@rchan26
remove 'model' arg and just use 'model_class' in create_and_fit_team_…
2dff338 
…model
@rchan26
bug fixes for fitting neutral model
d725173
@rchan26
Merge branch 'develop' into update-team-model
3429ad1
@rchan26
update tests
5331ee7
@rchan26
Merge branch 'develop' into update-team-model
37a4d9d
@rchan26
notebook using new model fitting update
e119e5b
@rchan26
update CLI to pass in model and epsilon properly
163b08c
@rchan26
do not pass in team_model and model arg to fixture_probabilities
f9f45b0
rchan26 and others added 17 commits August 8, 2023 00:04
@rchan26
fit requested model if not fit
df1fd6f
@jack89roberts
Merge branch 'develop' into update-team-model
567644a
@iansealy
Give --max_transfers option to airsenal_run_pipeline.
766cbb8
@iansealy
Limit allowable values for --max_transfers.
415099d
@iansealy
Add missing trailing comma.
705947e
@iansealy
Give --max_hit and --allow_unused options to airsenal_run_pipeline.
77e711d
@iansealy
Fix long line.
992317b
@jack89roberts
Merge pull request #637 from iansealy/max-transfers
1453b6d 
Allow --max_transfers, --max_hit and --allow_unused options for airsenal_run_pipeline
@rchan26
add function to parse team model from str and pass in modelinstance
2af6645
@rchan26
Merge branch 'main' into update-team-model
9ad9c5b
@rchan26
import models from bpl rather than airsenal.bpl_interface
3d00d7f
@rchan26
call the model instances
5c95e6b
@rchan26
change 'xdc' to 'extended'
f9ecbd1
@rchan26
Merge branch 'develop' into update-team-model
baf62d9
@rchan26
change wording to 'using the default epsilon'
3a9faf0
@rchan26
change back to use ratings by default
00c8b17
@jack89roberts
Merge pull request #561 from alan-turing-institute/update-team-model
8d404d4 
Updating team model
@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@jack89roberts jack89roberts mentioned this pull request Feb 24, 2024
Merged
@nbarlowATI nbarlowATI merged commit 9e567a5 into main Mar 22, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

airsenal_run_prediction throwing errors after new install on a new WSL2/WIN 11 install
5 participants
@jack89roberts @nbarlowATI @rchan26 @radka-j @iansealy