search

crate / cratedb-examples

A collection of clear and concise examples how to work with CrateDB.
Apache License 2.0
9 stars 7 forks source link

AutoML: CI trips with `CellTimeoutError` #299

Closed amotl closed 8 months ago

amotl commented 9 months ago

Originally coming from an issue that mixed things up, GH-170, let's get things straight here.

Problem

CI on the AutoML job occasionally trips like this, failing the CI run.

E           nbclient.exceptions.CellTimeoutError: A cell timed out while it was being executed, after 300 seconds.
E           The message was: Cell execution timed out.
E           Here is a preview of the cell contents:
E           -------------------
E           s = setup(data, fh=15, target="total_sales", index="month", log_experiment=True)
E           -------------------

/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/site-packages/nbclient/client.py:801: CellTimeoutError

-- https://github.com/crate/cratedb-examples/issues/170#issuecomment-1837270337

Outlook

@andnig suggested at https://github.com/crate/cratedb-examples/issues/170#issuecomment-1937005557, that maybe the PYTEST_CURRENT_TEST environment variable, and what it is guarding, is not being evaluated correctly.

However, at https://github.com/crate/cratedb-examples/issues/170#issuecomment-1942262343, we have been able to confirm it works well.

amotl commented 9 months ago

We confirmed that PYTEST_CURRENT_TEST properly guards the compare_models() call. However, the cell timeout here happens within setup() already. Is there a chance to reduce its heavyness in one way or another?

amotl commented 8 months ago

Currently, we see no problems on CI in this regard. Therefore, I am closing the issue. Thanks for your support, @andnig!