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

Exposing the low-level interface of hivemind. #31

Merged
merged 1 commit into from
Jul 1, 2020

Conversation

Quetzal2
Copy link
Collaborator

@Quetzal2 Quetzal2 commented Jul 1, 2020

This has originaly been discussed when the hivemind code was merged. #28 (comment)
And as the documentation says, there are different ways to use the framework. https://github.com/RLBot/rlbot-rust/blob/master/src/lib.rs#L19

I think that having a "low-level" and a "high-level" interface for hivemind, as we have for standard bots, would make things more coherent, and more flexible. Because right now, someone that is not using the high-level interface can't move over to a hivemind, due to the lack of access to the tools needed.
And finally, having access to the low-level interface doesn't change anything to the high-level one, so nothing's lost or changed ^^ .

What do you think?

The standard bot interface has a high-level and a low-level interface,
as said in the documentation. Having the same interfaces for hivemind
allows for using it in more complex situations.
Copy link
Contributor

@ViliamVadocz ViliamVadocz left a comment

Choose a reason for hiding this comment

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

I don't have any issues with this change. 👍

@Quetzal2 Quetzal2 merged commit 01bafee into RLBot:master Jul 1, 2020
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

2 participants