Conversation
|
All PEP8 errors has been fixed, thanks ❤️ Comment last updated at Fri, 27 Feb 2026 16:28:05 |
There was a problem hiding this comment.
почему табуляция поехала в steps?
| @@ -0,0 +1,67 @@ | |||
| name: Integration Tests | |||
There was a problem hiding this comment.
| name: Integration Tests | |
| name: Integration (Industrial) |
|
|
||
| on: | ||
| schedule: | ||
| - cron: '0 12 1,15 * *' |
There was a problem hiding this comment.
подумать о том, как часто мы хотим запускать интеграционное тестирование для industrial?
подкорректировать cron expression при необходимости
There was a problem hiding this comment.
Тесты для индастриал-модуля поставил раз в неделю на восемь часов позже основного модуля
name: Integration (Industrial)
on:
schedule:
- cron: '0 20 * * 1'
| cd fedot/industrial | ||
| uv run pytest --cov=fedot_ind tests/integration -vvv --cov-report=xml |
There was a problem hiding this comment.
сейчас в fedot/industrial тестов вообще нет (этот модуль не будет использоваться в качестве склонированнного репозитория, а будет достаточно сильно упрощен и адаптирован под текущую FEDOT инфру)
сейчас выходит, что этот gh actions никакие тесты вообще не гоняет
подумать, куда перенести тесты на fedot/industrial из https://github.com/aimclub/Fedot.Industrial
как вариант - добавить в директорию test субдиректорию test/industrial и в воркфлоу проваливаться в нее
.github/workflows/unit-build.yml
Outdated
| - name: Codecov-coverage | ||
| uv run pytest test/unit --cov=fedot -s | ||
|
|
||
| - name: FEDOT.Industrial tests |
There was a problem hiding this comment.
| - name: FEDOT.Industrial tests | |
| - name: Industrial tests |
убрать упоминание Fedot.Industrial фреймворка здесь и во всех похожих местах, fedot/industrial - полноценный модуль FEDOT
.github/workflows/unit-build.yml
Outdated
| cd fedot/industrial | ||
| uv run pytest test/unit --cov=fedot_ind -s |
There was a problem hiding this comment.
см. комментарий выше - в fedot/industrial тесты не содержатся, и их там никогда и не будет
| {name = "NSS Lab", email = "itmo.nss.team@gmail.com"} | ||
| ] | ||
| classifiers = [ | ||
| "License :: OSI Approved :: BSD License", |
There was a problem hiding this comment.
уточнить по поводу лицензии BSD-3-Clause
There was a problem hiding this comment.
был ли протестирован этот pyproject.toml для всех представленных версий python в unix-based (e.g. Linux, macOS) системах и на Windows?
There was a problem hiding this comment.
Протестировано на Windows и Linux
There was a problem hiding this comment.
на всех версиях python 3.10+ без ошибок?
There was a problem hiding this comment.
на всех версиях python 3.10+ без ошибок?
…b/FEDOT into refactor/cicd-dependecies
Summary
|
| from fedot.core.pipelines.tuning.tuner_builder import TunerBuilder | ||
| from golem.core.tuning.sequential import SequentialTuner | ||
|
|
||
| from fedot_ind.api.main import FedotIndustrial |
There was a problem hiding this comment.
здесь и во всех добавленных тестах fedot_ind -> fedot.industrial
2 participants
Summary
pyproject.tomlto maintain dependencies/testfolder for Fedot.Industrial will be placed into./fedot/industrial./other-requirementsinto[dependecy-groups]Context
closes #1414