You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In #680, we've drastically simplified the former TaggedTable (now TabularDataset). Similarly, TimeSeries should be simplified, which currently duplicates a lot of things one can already do with a Table.
Desired solution
Create a new TimeSeriesDataset class in safeds.data.labeled.containers. It should be created from a Table or a Mapping[str, Sequence[Any]] (see TabularDataset) by specifying
time_name
target_name
extra_names
Remove the current TimeSeries class completely. Instead, integrate the limited additional functionality to work with temporal data (e.g. plots) into the normal Table class.
Add a to_time_series_dataset method to Table, as an additional means to create a TimeSeriesDataset
This greatly improves maintainability. Moreover, then there's only one way to manipulate tabular data.
Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
No response
The text was updated successfully, but these errors were encountered:
Is your feature request related to a problem?
In #680, we've drastically simplified the former
TaggedTable
(nowTabularDataset
). Similarly,TimeSeries
should be simplified, which currently duplicates a lot of things one can already do with aTable
.Desired solution
TimeSeriesDataset
class insafeds.data.labeled.containers
. It should be created from aTable
or aMapping[str, Sequence[Any]]
(seeTabularDataset
) by specifyingTimeSeries
class completely. Instead, integrate the limited additional functionality to work with temporal data (e.g. plots) into the normalTable
class.to_time_series_dataset
method toTable
, as an additional means to create aTimeSeriesDataset
This greatly improves maintainability. Moreover, then there's only one way to manipulate tabular data.
Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
No response
The text was updated successfully, but these errors were encountered: