cal/paper-dynasty-database
1
0
Fork 0
Code Issues 7 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

fix: use Field(default_factory) for offense_col random default (#24)
All checks were successful
Build Docker Image / build (pull_request) Successful in 3m10s
Details

Browse Source 
Pydantic evaluates bare `random.randint(1, 3)` once at class definition
time, so every PlayerModel instance shared the same value. Replaced with
`pydantic.Field(default_factory=...)` so a new random value is generated
per instance.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Cal Corum 2026-03-03 14:05:21 -06:00
parent 41c50dac4f
commit d26bc0dc7b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/routers_v2/mlbplayers.py
View File

@ -37,7 +37,7 @@ class PlayerModel(pydantic.BaseModel):
key_fangraphs: int = None
key_bbref: str = None
key_retro: str = None
offense_col: int = random.randint(1, 3)
offense_col: int = pydantic.Field(default_factory=lambda: random.randint(1, 3))
class PlayerList(pydantic.BaseModel):