andrewRowlinson / mplsoccer

Football pitch plotting library for matplotlib
MIT License
405 stars 82 forks source link

Update deprecated set indexer in statsbomb event reader #46

Closed tdy closed 2 years ago

tdy commented 2 years ago

As of pandas 1.4.0, indexing with sets now raises a deprecation warning and will eventually raise a full error, so technique_id_cols and technique_name_cols should be converted from sets into lists.

.venv/lib/python3.9/site-packages/mplsoccer/statsbomb.py:70: FutureWarning: Passing a set as an indexer is deprecated and will raise in a future version. Use a list instead.

andrewRowlinson commented 2 years ago

Thanks for this contribution, I have merged it now.