Commit eb22672f authored by Lluis Gifre Renom's avatar Lluis Gifre Renom
Browse files

Forecaster component:

- Updated TODO.txt
parent 0f64d1f0
Loading
Loading
Loading
Loading
+2 −23
Original line number Diff line number Diff line
@@ -3,30 +3,9 @@ Use a smaller network:
INFO     forecaster.tests.Tools:Tools.py:75 Discovering Devices and Links...
INFO     forecaster.tests.Tools:Tools.py:104   Found 22 devices and 462 links...

ERROR    grpc._server:_server.py:453 Exception calling application: The truth value of a Series is ambiguous. Use a.empty, a.bool(), a.item(), a.any() or a.all().
Traceback (most recent call last):
  File "/home/tfs/.pyenv/versions/3.9.13/envs/tfs/lib/python3.9/site-packages/grpc/_server.py", line 443, in _call_behavior
    response_or_iterator = behavior(argument, context)
  File "/home/tfs/tfs-ctrl/src/common/tests/MockServicerImpl_Monitoring.py", line 99, in QueryKpiData
    df_samples = self.ts_db.filter(kpi_uuids, start_timestamp=start_timestamp, end_timestamp=end_timestamp)
  File "/home/tfs/tfs-ctrl/src/common/tests/InMemoryTimeSeriesDatabase.py", line 30, in filter
    if len(kpi_uuids) > 0: data = data[data.kpi_uuid in kpi_uuids]
  File "/home/tfs/.pyenv/versions/3.9.13/envs/tfs/lib/python3.9/site-packages/pandas/core/generic.py", line 1527, in __nonzero__
    raise ValueError(
ValueError: The truth value of a Series is ambiguous. Use a.empty, a.bool(), a.item(), a.any() or a.all().


Seems mock_context does not add device and link ids to the topology
    def test_forecast_link(
        context_client : ContextClient,
        forecaster_client : ForecasterClient,
    ):  # pylint: disable=redefined-outer-name
        topology = context_client.GetTopology(ADMIN_TOPOLOGY_ID)
>       link_id = topology.link_ids[0]
E       IndexError: list index (0) out of range


In topology forecaster, the data file should contain more than 1 row per link, otherwise, it crashes with no enough samples.

- implement integration with pathcomputation


test case: