-
Notifications
You must be signed in to change notification settings - Fork 628
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
Refactor Repeat as a Macro? #4947
Comments
Implementing repeat as a macro of concat (not facet) might be reasonable. However, making repeat = pivot + facet is extremely unintuitive and would produce unnecessary data transforms, so it definitely should NOT be done. Also, I don't think this is a "help wanted" issue since it's not even clear that it should be done. It's also not an "enhancement" to the language, but rather internal refactor. |
I don't know how bad the produced code would be. Therefore I'm not sure I agree. |
A related enhancement of Vega-Lite would be having something like repeat but for layering. The idea would be that you can make for example a multi-series line chart with three lines ( |
Though I find the notion of |
|
You are right, I meant unpivot or fold. |
pretty much done in #6002, pending a few minor issues left. |
Our repeat should have the same behavior as using fold (unpivot) and then facet into row/column. We should investigate whether it makes sense to clean up the implementation and implement repeat as a macro.
The text was updated successfully, but these errors were encountered: