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

Regressors serialization #48

Closed
PabloKarpacho opened this issue Mar 23, 2023 · 0 comments · Fixed by #80
Closed

Regressors serialization #48

PabloKarpacho opened this issue Mar 23, 2023 · 0 comments · Fixed by #80
Assignees
Labels
accepted for consideration label for users to make them know that their issue has been read

Comments

@PabloKarpacho
Copy link
Contributor

Просматривая файл nodes.py, я заметил, что при обучении параметров модель регрессора сериализуется (причем используется достаточно длинная конструкция if/else). Далее упакованная модель помещается в словарь distributions и для дальнейшей работы требуется распаковать её.

Почему бы не оставить сериализацию модели только в том месте, где мы хотим сохранить нашу модель?
В этом случае можно помещать в словарь distribution регрессор в качестве объекта и использовать его напрямую, без распаковки.

@Roman223 Roman223 self-assigned this Sep 14, 2023
@Roman223 Roman223 added the accepted for consideration label for users to make them know that their issue has been read label Sep 14, 2023
@Roman223 Roman223 mentioned this issue Sep 22, 2023
1 task
@Roman223 Roman223 linked a pull request Sep 22, 2023 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted for consideration label for users to make them know that their issue has been read
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants