-
Notifications
You must be signed in to change notification settings - Fork 87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix integration tests (v 0.7.1) #1110
Comments
Проблема, которая препятствует разрешению, связана с GOLEM и находится в следующем методе Dispatcher'а: Проблема всплыла после коммита 4c58deda3646f91ce4053b40571f2fd08a9f1bb8, до него всё норм. Ошибка воспроизводится в FEDOT на тесте test/integration/api_params/test_main_api_params.py::test_timeout |
В тесте в конфиге указано что:
Есть ли вообще смысл в такой настройке - для эволюции это достаточно вырожденный случай. |
Можно в GOLEM добавить issue на корректную обработку краевых значений - но вроде это не тот сценарий, который есть смысл интеграционно тестировать в fedot-е. |
Согласен с Колей. Можно поставить 2 и закрыть вопрос. А вообще, даже при pop_size=1 должна передаваться популяция, то есть список, пусть и из 1 элемента. Почему это не так -- посмотрю |
@IIaKyJIuH попробуй пж прогнать с этим фиксом, у меня проходит |
A respectful issue.
As now tests badge is on the main project page -- they must be more predictable, don't fall stochastically.
And even if they must be stochastic -- let's run them multiple times and estimate required number of times they finish with positive results.
Failing tests from the last run on v0.7.1.
The text was updated successfully, but these errors were encountered: