Skip to content
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

1119 fix data split #1129

Closed
wants to merge 13 commits into from
Closed

1119 fix data split #1129

wants to merge 13 commits into from

Conversation

kasyanovse
Copy link
Collaborator

@kasyanovse kasyanovse commented Jul 31, 2023

Fix issue with wrong train sample for classification problem. Issue consists in difference between label sets for the train and test samples. Some labels are found only in the test. The problem arises from unstratified sample for classification in some cases.

Now the sample for classification is always stratified.

nicl-nno and others added 10 commits July 13, 2023 16:55
* Version variable added to fedot module
* Version incremented, README updated

* Golem version upd
* Minor fixes of documentation
* Fix params setting

* Minor

* PEP 8

* Correct test

* Add comment
* Version variable added to fedot module
* Version incremented, README updated

* Golem version upd
* Minor fixes of documentation
* Fix params setting

* Minor

* PEP 8

* Correct test

* Add comment
@kasyanovse kasyanovse linked an issue Jul 31, 2023 that may be closed by this pull request
@aim-pep8-bot
Copy link

aim-pep8-bot commented Jul 31, 2023

Hello @kasyanovse! 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-08-01 06:47:37 UTC

@nicl-nno
Copy link
Collaborator

Кажется куча лишних коммитов прилипла.

@kasyanovse kasyanovse changed the base branch from master to ts-opt-analytics July 31, 2023 14:53
@kasyanovse kasyanovse closed this Aug 1, 2023
@kasyanovse kasyanovse deleted the 1119-fix-data-split branch August 1, 2023 06:58
@kasyanovse kasyanovse removed a link to an issue Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants