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

Usvm's gameserver #65

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from
Draft

Usvm's gameserver #65

wants to merge 2 commits into from

Conversation

Parzival-05
Copy link
Collaborator

Добавил сервер для общения c usvm. Вместо какого-то explorer сейчас работает рандомный, но пообщаться c таким получилось.

Copy link
Owner

@gsvgit gsvgit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Думаю, надо сделать мне форк USVM, чтобы все напротив него работали и он был в подмодулях.
  2. Кажется, каборы карт на уровне json должны быть по языкам. То есть отдельно лежит описание (и набор методов) для .NET, отдельно для Java. Это, видимо, ещё и с Gameserver update #66 связано.
  3. Ну и комментарии.

DatasetConfig:
dataset_base_path: ../maps/DotNet/Maps/Root/bin/Release/net7.0 # a patch
dataset_description: ../workflow/dataset_for_tests.json # a patch
OptunaConfig:
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А почему эти секции дублируются?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправлено в 66

AIAgent/launch_servers.py Outdated Show resolved Hide resolved
@emnigma
Copy link
Collaborator

emnigma commented May 21, 2024

@Parzival-05 , пока не смотреть? Надо бы какие-нибудь простенькие тесты добавить, чтобы завелось чуть-чуть. В #56 по-моему хорошие шаги

Можно даже наверное с рандомным серчером для начала

@Parzival-05
Copy link
Collaborator Author

@Parzival-05 , пока не смотреть? Надо бы какие-нибудь простенькие тесты добавить, чтобы завелось чуть-чуть. В #56 по-моему хорошие шаги

Можно даже наверное с рандомным серчером для начала

Сейчаc можно позапускать и c картами VSharp.

@Parzival-05
Copy link
Collaborator Author

Parzival-05 commented May 22, 2024

2. Кажется, каборы карт на уровне json должны быть по языкам. То есть отдельно лежит описание (и набор методов) для .NET, отдельно для Java. Это, видимо, ещё и с Gameserver update #66 связано.

Исправлено в Gameserver update #66

@Parzival-05 Parzival-05 marked this pull request as draft May 22, 2024 18:18
@Parzival-05
Copy link
Collaborator Author

Тут пока падает, т.к. нужно синхронизироваться c 66.

Copy link
Collaborator

@emnigma emnigma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Давай тогда дождемся #66 , тогда можно будет CI прогнать и влить

Comment on lines +4 to +6
[submodule "GameServers/usvm"]
path = GameServers/usvm
url = https://github.com/Parzival-05/usvm.git
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

вроде как Семен уже сделал форк

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.

None yet

3 participants