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

check whether we wanna allow assigment on task (measure, etc) properties #1149

Open
berndbischl opened this issue Aug 29, 2024 · 1 comment

Comments

@berndbischl
Copy link
Sponsor Member

at least in the "weights PR" properties became an AB. where assignment was possible.
I don't like that. props should be read-only? as static as possible?

@mb706
Copy link
Collaborator

mb706 commented Aug 29, 2024

I agree that allowing assignment to properties may be a bad idea here; it is not entirely clear what 'removing' a property should do. E.g. for weights: remove the associated col roles? Or just signal that the relevant col role should be ignored? For missings: remove features with missing values?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants