-
Notifications
You must be signed in to change notification settings - Fork 9
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
First draft for bot documentation #14
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Someone is attempting to deploy a commit to a Personal Account owned by @asadm on Vercel. @asadm first needs to authorize it. |
Added - sample joystickConfig object - sample DQN Specs object
pages/bots/custombot.mdx
Outdated
// Implement your custom bot logic and methods here | ||
} |
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.
Start with a very simple example instead of empty class
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.
Resolved in separate PR
pages/bots/custombot.mdx
Outdated
Subsequently, extend this base Bot class for your custom bot. | ||
This class is devoid of any predefined methods; its primary role | ||
is to ascertain that your custom bot is identified as a Bot type within the SDK. |
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.
Simplify, too complex words
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.
Resolved in separate PR
pages/bots/custombot.mdx
Outdated
|
||
// OPTIONAL: If you want botParams object for the initialization of custom bot class. | ||
botParams: { | ||
... YourCustomBot parameters, |
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.
Also give an example and explain this goes to constructor as object in first parameter
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.
Resolved in separate PR
pages/bots/custombot.mdx
Outdated
... YourCustomBot parameters, | ||
} | ||
}, | ||
}).then(() => { |
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.
Use await
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.
Resolved in separate PR
pages/bots/custombot.mdx
Outdated
|
||
```js | ||
if (player.isBot()) { | ||
// Logic to make the bot act within the game loop |
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.
Call the custom method you defined in above class as example
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.
Resolved in separate PR
pages/bots/dqnbot.mdx
Outdated
@@ -0,0 +1,90 @@ | |||
# DQN Bot Integration with Playroom |
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.
can you move dqn and dqnjoystick in a separate PR. Let's first launch custom / basic bot only.
Does this need to be updated and pushed for the latest docs version or are we just leaving it? (I can make updating changes if needed on a separate pull req.) |
Added Documentation for Custom Bot, DQN Bot, DQN Bot with Joystick
Updated InsertCoin with parameters like 'enableBots' and 'botOptions'