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

Make parameter and objective classes slotted #74

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

AdrianSosic
Copy link
Collaborator

@AdrianSosic AdrianSosic commented Jan 4, 2024

Now that attrs==24.1.0 supports cached_property with slotted classes, we can finally enable slots for our parameter and objective classes.

@AdrianSosic AdrianSosic self-assigned this Jan 4, 2024
@AdrianSosic AdrianSosic added the enhancement Expand / change existing functionality label Jan 4, 2024
@AdrianSosic
Copy link
Collaborator Author

Note: there was another issue which messed with our subclass detection mechanism, which has already be resolved but the fix has not yet been released. However, the checks should automatically pass once attrs 24.1.0 is available.

Copy link
Collaborator

@AVHopp AVHopp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess there is not really a need for a proper and full review, this is just some maintenance right?

Copy link
Collaborator

@Scienfitz Scienfitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deleted

@Scienfitz Scienfitz added the on hold PR progress is awaiting for something else to continue label Apr 3, 2024
@AdrianSosic
Copy link
Collaborator Author

@AVHopp @Scienfitz, FYI: I rebased and added one more commit since in the meantime we also had to make our Objective classes slotted ... a change that can be similarly reverted by this PR once the attrs support has been released

@AdrianSosic AdrianSosic changed the title Make parameter classes slotted Make parameter and objective classes slotted Apr 18, 2024
@Scienfitz Scienfitz marked this pull request as draft June 21, 2024 14:30
@AdrianSosic AdrianSosic removed the on hold PR progress is awaiting for something else to continue label Aug 3, 2024
@AdrianSosic AdrianSosic marked this pull request as ready for review August 3, 2024 19:44
CHANGELOG.md Show resolved Hide resolved
@AdrianSosic
Copy link
Collaborator Author

Again on hold until the following blocking issue gets resolved:
python-attrs/attrs#1325

@AdrianSosic AdrianSosic added the on hold PR progress is awaiting for something else to continue label Aug 13, 2024
@AdrianSosic AdrianSosic marked this pull request as draft August 13, 2024 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Expand / change existing functionality on hold PR progress is awaiting for something else to continue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants