aimclub / FEDOT

Automated modeling and machine learning framework FEDOT
https://fedot.readthedocs.io
BSD 3-Clause "New" or "Revised" License
619 stars 84 forks source link

Fix gensim for PyPi release #1287

Closed DRMPN closed 2 months ago

DRMPN commented 2 months ago

This is a 🐛 bug fix

Summary

Fix gensim problem with pinned dependencies versions

Context

PyPi intermediate release 0.7.3.2 doesn't publish:

Can't have direct dependency: gensim@                                  
git+https://github.com/piskvorky/gensim.git@ad68ee3 ; extra == "extra".

https://github.com/aimclub/FEDOT/actions/runs/8849483743/job/24301597093

docu-mentor[bot] commented 2 months ago

👋 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] commented 2 months ago

All PEP8 errors has been fixed, thanks :heart:

Comment last updated at
codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.90%. Comparing base (cd915a8) to head (f3dbbd8). Report is 5 commits behind head on master.

:exclamation: Current head f3dbbd8 differs from pull request most recent head 26ce738. Consider uploading reports for the commit 26ce738 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1287 +/- ## ========================================== + Coverage 79.88% 79.90% +0.02% ========================================== Files 146 146 Lines 10065 10066 +1 ========================================== + Hits 8040 8043 +3 + Misses 2025 2023 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Lopa10ko commented 2 months ago

gensim редко релизится, версию специально брали с определенного коммита из-за падения интеграционников

подробнее проблема описана здесь

pypi не поддерживает такие зависимости, но должны быть workaround'ы

или нужно, наверное, менять версию scipy

DRMPN commented 2 months ago

было бы замечательно еще несколько раз локально проверить модульные тесты и конкретный интеграционник, который падал раньше + запустить интеграционники в github actions пару раз, чтобы убедиться, что нет с ними проблем

Да, так и планировал делать. Не нравится, что для 3.8 срабатывает таймаут на установке, так что еще попробую другие версии, мб только для этой такая проблема. Есть идея изменить версию scipy в Големе, не знаю насколько адекватная.

DRMPN commented 2 months ago

✅ Интеграционные - https://github.com/aimclub/FEDOT/actions/runs/8866144787

DRMPN commented 2 months ago

❌ Интеграционные - https://github.com/aimclub/FEDOT/actions/runs/8866585903

DRMPN commented 2 months ago

✅ Интеграционные - https://github.com/aimclub/FEDOT/actions/runs/8866859667

DRMPN commented 2 months ago

✅ Интеграционные - https://github.com/aimclub/FEDOT/actions/runs/8867155981

DRMPN commented 2 months ago

✅ Интеграционные - https://github.com/aimclub/FEDOT/actions/runs/8937098133

DRMPN commented 2 months ago

Тесты проходят и для Голема - https://github.com/aimclub/GOLEM/pull/274 Можно оставить зависимости в таком виде или же сделать новый релиз Голема с измененной версией scipy. Тут я уже не знаю как лучше.