aimclub / FEDOT

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

Version variable added #1118

Closed nicl-nno closed 1 year ago

nicl-nno commented 1 year ago

Version variable is added to support the following snippet:

from fedot import __version__
print(__version__)

Сaused by:

image

aim-pep8-bot commented 1 year ago

Hello @nicl-nno! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 3:1: F403 'from fedot import ' used; unable to detect undefined names Line 3:1: F401 'fedot.' imported but unused Line 4:1: F401 'fedot.version.version' imported but unused

Comment last updated at 2023-07-13 13:32:45 UTC
codecov[bot] commented 1 year ago

Codecov Report

Merging #1118 (b55b766) into master (cda892d) will increase coverage by 0.24%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1118      +/-   ##
==========================================
+ Coverage   78.32%   78.56%   +0.24%     
==========================================
  Files         130      131       +1     
  Lines        9321     9323       +2     
==========================================
+ Hits         7301     7325      +24     
+ Misses       2020     1998      -22     
Impacted Files Coverage Δ
fedot/__init__.py 100.00% <100.00%> (ø)
fedot/version.py 100.00% <100.00%> (ø)

... and 6 files with indirect coverage changes

MorrisNein commented 1 year ago

Можно создать version.py, в котором ничего не импортировать. Там задать переменную __version__. После этого импортировать эту переменную и в init, и в setup (только в setup импортировать по-хитрому, как в варианте 3 из примера выше).

Насколько я понимаю, в setup нужно, чтобы никакие компоненты фреймворка не импортировались. Поэтому для хранения версии нужен отдельный модуль, либо вообще текстовый файл.

nicl-nno commented 1 year ago

Вынес как ты предложил. Надеюсь это в pypi сработает.