deephaven / deephaven-core

Deephaven Community Core
Other
257 stars 80 forks source link

Deephaven Tutorial - Chart Builder IllegalArgumentException #3507

Open bmingles opened 1 year ago

bmingles commented 1 year ago

Description

In the Deephaven tutorial, configuring a line chart for the crypto_btc_2021 via the Chart Builder throws an ILLegalArgumentException

image

Steps to reproduce

  1. Run sections 1-5 in the tutorial
  2. In section 6 (Plot data via query or the UI), create a line chart from the crypto_btc_2021 table using the Chart Builder

image

Expected results

A line chart should be shown

Actual results

An IllegalArgumentException is thrown with the message "Interval is not positive"

Versions

Engine Version: 0.22.0 Web UI Version: 0.30.0 Java Version: 17.0.6 Barrage Version: 0.5.0

vbabich commented 1 year ago

This looks like the api bug, can you narrow this down to fewer steps to reproduce?

bmingles commented 1 year ago

I'm not yet sure how to narrow down the setup steps, but I was able to reproduce this in code (not just the chart builder)

# This works
simple_line_plot = Figure().plot_xy(
    series_name="BTC Price", 
    t=crypto_btc_2021.tail(2_000), # NOTE: how this is limited to 2000 records
    x="Timestamp", 
    y="Price").show()

# This fails    
simple_line_plot = Figure().plot_xy(
    series_name="BTC Price", 
    t=crypto_btc_2021, # NOTE: this is unbounded
    x="Timestamp", 
    y="Price").show()