-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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] Fix nested_univ
converter inconsistent handling of index level names
#7026
Conversation
@fkiraly The failure in run notebooks is caused by this change, as we no longer return
from sktime.datasets import load_arrow_head
from sktime.datatypes import convert_to
df_panel = load_arrow_head(split="TRAIN")[0]
df_panel = convert_to(df_panel, "pd-multiindex").reset_index()
df_panel = df_panel.set_index(["level_0", "timepoints"]) Changing set_index to df_panel = df_panel.set_index(["level_0", "level_1"]) Fixes it because now the correct column name is being returned. |
nested_univ
converternested_univ
converter inconsistent handling of index level names
that seems like a reasonable fix - btw, I think the loader should just use the This highlights that this might break user code, although only under very brittle circumstances, so we may consider shipping it only with the full minor version. |
Do you mean something like this ?? df_panel = load_arrow_head(split="TRAIN", return_type="pd-multiindex")[0]
Is this good to go now?? so I will fix the failures in notebook too |
yes, that's what I meant - the notebook should of course run with this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see above, the notebook ought to run
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
made changes in the notebook |
…l names (sktime#7026) This PR fixes the `nested_univ` converter, to return the index name of series nested in the dataframe. Fixes sktime#7025
This PR fixes the
nested_univ
converter, to return the index name of series nested in the dataframe.Fixes #7025