added annotations for moveTo() to add compiler plugin support #1499
Conversation
… to add compiler plugin support
Jolanrensen
added
the
Compiler plugin
| * a path in the [DataFrame] to move columns. | ||
| */ | ||
| @Refine | ||
| @Interpretable("MoveInto1") |
There was a problem hiding this comment.
What use cases this would support?
There was a problem hiding this comment.
Ah I see the issue now... I assumed:
move { col }.into { pathOf("some", "new", "path") }, assuming the CS DSL in the compiler plugin supports pathOf() and "path"["of"] :)
And move { col }.into { someColGroup["newName"] }
but it also supports multiple cols with the new name depending on the column given...
It's like a rename.into + move.under
There was a problem hiding this comment.
but that's also the problem with rename.into {} isn't it? we don't know how the user uses the old name or any other column data to generate the new name...
There was a problem hiding this comment.
as long as user can write arbitraty code in the lambda such powerful operations are somewhat impossible to implement in the plugin =( shortcuts like rename { }.removePrefix could cover most popular usages though. Didn't think about move into yet
So let's remove these two for now
…hose cases well yet
Jolanrensen
changed the title
Jolanrensen
changed the title
Jolanrensen
changed the title
2 participants
No description provided.