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

Implement automation mechanism #75

Closed
wants to merge 0 commits into from
Closed

Conversation

Wiguwbe
Copy link
Contributor

@Wiguwbe Wiguwbe commented Aug 17, 2023

An external script can be used to automate the scheduling.

An event manager receives events from processes, pages and IO queue, and dispatches them to the external script.

The external script then processes the events and returns a list of actionable events (simulating clicks).

The game scene dispatches the received events to the respective receivers.

Main changes are:

  • event manager;
  • changes to provide events to event manager;
  • compile and execute an external script;
  • _on_click/_onClick is now public;
  • game's scene dispatches events from script before calling the update function

An example automated script is provided, mostly to explain the exposed API, it has a (not so) simple example implementation.

@Wiguwbe
Copy link
Contributor Author

Wiguwbe commented Aug 17, 2023

To run, provide a python script as argument (I shall update the Readme)

@wviana
Copy link
Contributor

wviana commented Aug 17, 2023

Maybe we could create another PR just removing those unnecessary white spaces. So it will make cleaner these commits.
This time not only in files you've touched, but the entire project.

@Wiguwbe
Copy link
Contributor Author

Wiguwbe commented Aug 17, 2023

I did notice that a lot of whitespace was trimmed, just haven't actually looked at the diff here

It should be easy to make, should @plbrault be okay with it

@plbrault
Copy link
Owner

Hey! This looks amazing! It will take me a little while to review though and I will be quite busy over the next few days, so I will get back to you as soon as I can.

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.

3 participants