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

Reintegrate dev/v0.10 #87

Merged
merged 61 commits into from
Dec 3, 2024
Merged

Reintegrate dev/v0.10 #87

merged 61 commits into from
Dec 3, 2024

Conversation

jvansanten
Copy link
Contributor

No description provided.

AbsPlainT3Unit -> AbsT4Unit
AbsReviewT3Unit -> AbsT3Unit
PyTorch stable does not support 3.12 yet

See: pytorch/pytorch#110436
AmpelBaseModel subclasses can't set extra attributes by default.
Since this will always be mixed into the inheritance chain of
AmpelUnit or BaseModel, both of which harvest annotations from their
bases, there's not actually a reason for this to be a BaseModel
itself.
Ideally there would be a declared result type, such that downstream consumers can be type checked
@jvansanten jvansanten force-pushed the dev/v0.10 branch 2 times, most recently from b2007cc to 582616b Compare December 3, 2024 08:18
@jvansanten jvansanten merged commit 1518d01 into main Dec 3, 2024
11 checks passed
@jvansanten jvansanten deleted the dev/v0.10 branch December 3, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant