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

cogment run generate error #1

Open
Crazyalltnt opened this issue Oct 26, 2021 · 2 comments
Open

cogment run generate error #1

Crazyalltnt opened this issue Oct 26, 2021 · 2 comments
Assignees

Comments

@Crazyalltnt
Copy link

Crazyalltnt commented Oct 26, 2021

Hello AIR sir:
When I tested after installing cogment according to the document (https://docs.cogment.ai/introduction/installation/#test-your-installation), some errors occurred. The red box in the picture indicates that the proto_path is incorrect. One more &nbsp and one less /.
The correct path should be xxx/cogment/cogment-tutorial-rps/5-human-player/data.proto
But here is xxx/cogment/cogment-tutorial-rps/5-human-player data.proto

I check the code in generate.go of cogment-cli, and it seems that the path is connected with spaces. Is this the cause of the splicing error between the parent directory and the proto file path? How can I solve this problem?

image
image

@cloderic cloderic self-assigned this Nov 1, 2021
@cloderic
Copy link
Member

cloderic commented Nov 1, 2021

Thank you for your report we will investigate!

@air-sara
Copy link
Contributor

air-sara commented Nov 5, 2021

Hello, this has now been fixed and will be available in the next version of this repository. Thank you for bringing this to our attention ^-^

@cloderic cloderic assigned air-sara and unassigned cloderic Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants