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
WIP: simplifies and tests #4
Conversation
and other enhancements in PR #4
and other enhancements in PR #4
BUG: rids Travis bugs
7dd9075
to
636726c
Compare
All the tasks are complete except one (detail below). This PR is nearly ready for merge and I feel the current implementation is fairly stable. @jakevdp can I hear your thoughts on naming? I only expose I implement 13 tests, some doctests and some testing interfacing with Altair (can't test UI yet, #5 is an example bug). I implemented the tests in a notebook because of an issue (jupyter-widgets/ipywidgets#13?) that means I can't run tests from the command line and only in Jupyter notebook. While not ideal, I've interfaced these notebook tests with pytest and Travis CI. Screenshots General interface (an older GIF can be seen on the README for this branch): When the encoding options are expanded: When the mark options are expanded: I have filed jupyter-widgets/ipywidgets#861 about the checkbox width. |
Quick question – would there be any way to make it so the ndims is not pre-specified? I imagine it being rather annoying if you're exploring a dataset, find you need one more dimension, and then have to re-run the cell and re-enter everything you've done. |
DOC: rewrites README to point to Vega
TST: adds some tests, UI: adds some notebook output
public components of Interface: - plot, settings - show, df MAINT: makes Interact.update "private"
BUG: resolves dockets bug
Work to be done before merge:
bin
,zero
, etc).applyColorToBackground
)implement command to get Altair command out to generate the shown plot