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

[BUG] all_objects returns variable name and not class name if these are different #112

Open
fkiraly opened this issue Jan 14, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@fkiraly
Copy link
Contributor

fkiraly commented Jan 14, 2023

From this refactor PR in sktime: sktime/sktime#3777

It would appear that all_objects returns the variable name, not the class name, as the string (first tuple element) when the two are unequal.

I'm not entirely sure about the "should" here and whether that's indeed the root cause of the test failure, but it seems to be discrepant with behaviour in sktime.

@fkiraly fkiraly added the bug Something isn't working label Jan 14, 2023
@RNKuhns
Copy link
Contributor

RNKuhns commented Feb 5, 2023

@fkiraly can you confirm this was fixed by #115.

@fkiraly
Copy link
Contributor Author

fkiraly commented Feb 5, 2023

It's not released yet, is it?
Since 0.3.0 predates #115.

I've opened a version of the refactor branch with install from current skbase main branch, if the tests pass here, it should tell us whether it's fixed: sktime/sktime#4208

@RNKuhns
Copy link
Contributor

RNKuhns commented Feb 5, 2023

@fkiraly looks like sktime/sktime#4208 is failing, but not for the same reasons as the prior test. I think they might be un-related to BaseObject, but I haven't looked to closely (seems like errors had to do with missing parameters in construction in a few estimators and a test case).

@fkiraly
Copy link
Contributor Author

fkiraly commented Feb 6, 2023

yes, indeed. I'm not sure what causes these, seems like default parameter generation.
I would put the functions that retrieve the set of parameters (like get_param_names) under suspicion.

This is the most recent change to this kind of function, in skbase, that I remember:
#65

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants