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

[wip] introduces an ibis backend to hvplot #507

Merged
merged 2 commits into from Jan 14, 2021

Conversation

tonyfast
Copy link
Contributor

@tonyfast tonyfast commented Sep 2, 2020

This PR adds the ability to

import hvplot.ibis

Adds an ibis backend to hvplot following the merging of holoviz/holoviews#4517

  • add ibis docs to hvplot.
    • add a link and description to the overview page.
    • add Databases.ipynb to the examples/user_guide
      • demonstrate sqlite because that is standard lib.
      • demonstrate plotting from an omnisci database
  • add hvplot api for ibis
  • pass the tests

image (1)

@jbednar
Copy link
Member

jbednar commented Sep 2, 2020

Very cool. Can you show an example from scratch, illustrating how an Ibis user would use this interface in practice (not just the internal test case as shown above)?

@tonyfast
Copy link
Contributor Author

tonyfast commented Sep 2, 2020

i will indeed. @dharhas told me i need to be better pushing my code. i have some work to go through, but i'll add something more significant examples, documentation, and tests.

@coveralls
Copy link

coveralls commented Dec 9, 2020

Coverage Status

Coverage decreased (-0.3%) to 75.064% when pulling 460c897 on tonyfast:ibis-backend into 9320892 on holoviz:master.

@philippjfr
Copy link
Member

Thanks @tonyfast. I'm going to merge and will add docs later.

@philippjfr philippjfr merged commit f809535 into holoviz:master Jan 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants