-
Notifications
You must be signed in to change notification settings - Fork 1
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
Gameserver update #66
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Сейчас в запуске обучения подразумевается, что всегда будет использоваться символьная машина. Это не так. Важно иметь возможность запускать классическое обучение.
Информация о символьной машине нужна только в play_game
. На клиенте она не нужна.
Кажется, отдельная функция run_training
лишняя.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
тэгни еще раз пожалуйста, когда Анины штуки поправишь
6371702
to
5b6c5e4
Compare
Нужно переписать коллектор статистики, чтобы не приходилось прокидывать инфу о названии SVM. Сейчас пока занес SVM в карту, далее еще валидацию причешу.
Сейчас в main происходит лишь чтение конфига, а полезная работа делегируется run training. Мне кажется такое разделение более логичным. |
cc6e86a
to
a9536ed
Compare
a9536ed
to
bf25921
Compare
@emnigma, я попытался с процесс пулом, но при ловле ошибок он зависает. Полагаю, что основным бутылочным горлышком при распараллеливании карт будет общение c сервером, поэтому даже c учетом GIL не должно стать на порядок медленнее. |
bf25921
to
f03e624
Compare
f03e624
to
dd49fa4
Compare
dd49fa4
to
29a9358
Compare
Поправил мелочи (или прокомментировал) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💪
95c0330
to
e93c530
Compare
Co-authored-by: Max Nigmatulin <40598909+emnigma@users.noreply.github.com>
8b4a10f
to
a81cd15
Compare
…rm. One SVM for one platform.
* Remove parallel training * Rename output of training * Add .mono to gitignore * Add handling of errors while training * Add `svm_info` to maps & refactor of training * Edit specification of config * Update statistics collector & refactor of training * Replace thread pool with process pool * Remove excess field from TrainingConfig * Refactor of validation * Refactor of epochs_statistics * Add SingleSVMInfo class * Remove excess import * Add config validation & revised training process * Add `pydantic` to project * Edit run_training: make the parameter "server_count" explicitly set * Minor fixes * Remove excess `svms_count` field * Add ignoring of extra fields in `SingleSVMInfo` * Fix `EpochsInfo`: make it a dataclass * Edit interface of model saver * Create single dataset. Merge configs. * Change optuna studies path. * Small fixes from comments. Co-authored-by: Max Nigmatulin <40598909+emnigma@users.noreply.github.com> * Fix bug with SVMInfo adding. * Refactor config. Add support of miltiple json files for single platform. One SVM for one platform. * Remove comments. * edit 'launch_servers': Add support of usvm * Fix statistics collector --------- Co-authored-by: Anya497 <chi.vinny0702@gmail.com> Co-authored-by: Anya Chistyakova <57658770+Anya497@users.noreply.github.com> Co-authored-by: Max Nigmatulin <40598909+emnigma@users.noreply.github.com>
svm_info
, теперь информация о svm хранится в картеpydantic
. Парсинг конфига стал приятнее.Пример нового лог файла: