add validation on _window_size param in topo extractor

[Lopa10ko]

This is a 🐛 bug fix.

Summary

• [x] add mandatory non-negative and non-zero validation on window size param inside a TopologicalFeaturesImplementation fit (1c04aad)
• [x] 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[bot]]

👋 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[bot]]

All PEP8 errors has been fixed, thanks :heart:

Comment last updated at

[nicl-nno]

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

[Lopa10ko]

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

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

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