Skip to content

Conversation

@ncoghlan
Copy link
Collaborator

@ncoghlan ncoghlan commented Sep 8, 2025

Ensure annotations are eagerly populated when defining lmstudio.BaseModel and lmstudio.LMStudioStruct subclasses

Works around #153

Ensure annotations are eagerly populated when defining
`lmstudio.BaseModel` and `lmstudio.LMStudioStruct` subclasses

Works around #153
@github-actions github-actions bot added the CLA signed Indicates that all contributors have signed label Sep 8, 2025
@ncoghlan
Copy link
Collaborator Author

ncoghlan commented Sep 8, 2025

My local testing gave a misleading result, as I hadn't recreated the Python 3.14 tox environment since testing c20473c, so it still had the upstream fix included.

CI didn't have that problem, so it correctly showed that __init_subclass__ isn't an adequate workaround here.

@ncoghlan ncoghlan closed this Sep 8, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Sep 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA signed Indicates that all contributors have signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants