add validation on _window_size param in topo extractor

[Lopa10ko]

This is a 🐛 bug fix.

Summary

• add mandatory non-negative and non-zero validation on window size param inside a TopologicalFeaturesImplementation fit (1c04aad)
• add unit test for a non-negative window check and extraction method validation (77a9c78)

Context

fixed integration test failure with: ValueError: Found array with 0 feature(s) (shape=(3, 0)) while a minimum of 1 is required by check_pairwise_arrays.

[docu-mentor]

👋 Hi, I'm @docu-mentor, an LLM-powered GitHub app
powered by Anyscale Endpoints
that gives you actionable feedback on your writing.

Simply create a new comment in this PR that says:

@docu-mentor run

and I will start my analysis. I only look at what you changed
in this PR. If you only want me to look at specific files or folders,
you can specify them like this:

@docu-mentor run doc/ README.md

In this example, I'll have a look at all files contained in the "doc/"
folder and the file "README.md". All good? Let's get started!

[github-actions]

All PEP8 errors has been fixed, thanks ❤️

Comment last updated at

[nicl-nno]

А можно это модульным тестом покрыть для надежности?

[Lopa10ko]

А можно это модульным тестом покрыть для надежности?

да, можно и нужно - сделаю. (77a9c78)

Note

в другом PR планируется поработать в целом над этим тестом, потому что в последнее время только он и падает из интеграционников: по крайней мере прокидывание NaN или float.inf в оптимизатор явно не является ожидаемым - это вновь проблемы ETSModel #1285