search

biolab / orange3-timeseries

🍊 :chart_with_upwards_trend: Orange add-on for analyzing, visualizing, manipulating, and forecasting time series data.
Other
62 stars 41 forks source link

Periodogram: Use appropriate units for x-axis #214

Closed mcapurso closed 2 years ago

mcapurso commented 2 years ago
Timeseries version
Orange version
Expected behavior
Actual behavior
Steps to reproduce the behavior
Additional info (worksheets, data, screenshots, ...)
mcapurso commented 2 years ago

In Orange 3.32 , Timeseries 0.4.1 on x axis the numbers for periods are wrong. If you get Yahoo finance widget, download Amazon quotes from October 2019 to June 2022 , open with line chart you see dates correctly. If you see correlogram , you see not dates but integer number of period. BUT if you open periodogram you see a strange VERY HIGH number on x axis. Probably the date is wrongly interpreted as a very high number. Please use period number or correct date. Screenshot from 2022-06-30 16-17-29 Screenshot from 2022-06-30 16-17-10 Screenshot from 2022-06-30 16-16-35

ajdapretnar commented 2 years ago

Perhaps I have misunderstood the issue, but Periodogram does not show datetime on x axis. It shows periods.

janezd commented 2 years ago

Perhaps I have misunderstood the issue

Yes and no. It cannot show dates. e.g. July 5, 2022. It shows time periods ... in seconds. 2.5 megaseconds is one month.

I changed the title to "Periodogram: Use appropriate units for x-axis". That is, if the span of values is above around 500, use minutes; if above 15000 seconds, it's hours; if above ~1M days, if ~10M, months; ~150M it's years. Roughly. I guess.

ajdapretnar commented 2 years ago

A, right, I understand now. I was confused by the comparison with Line Chart. I agree the units should make more sense.