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

Serial Support #344

Open
wants to merge 5 commits into
base: serial
Choose a base branch
from
Open

Serial Support #344

wants to merge 5 commits into from

Conversation

Artucuno
Copy link
Contributor

@Artucuno Artucuno commented Sep 8, 2024

(Reopened #337)

  • Merged changes from master branch
  • Working implementation of Serial

Tested in:

  • Pokemon Yellow (WORKING)
    • Trading
    • Battle

Issues:

  • There is a small issue where the emulator will temporarily freeze when waiting for the other Gameboy to respond to data (Should be an easy fix for someone) - Fixed in 7878718 & fa3eaaa
  • Synchronization issues in some games, however Pokémon games work completely fine. Fixed in fa3eaaa
  • When compiled into Cython, PyBoy freezes after socket connection is established Fixed in fa3eaaa
  • GBs will only use the internal clock

@Baekalfen
Copy link
Owner

Very interesting work! Could you rebase this instead? I can't tell exactly which changes are yours and which are from the merge commit.

Also, you seem to change some of the formatting in the files. You should use pip install pre-commit && pre-commit install and let that take care of the formatting.

@beardyFace
Copy link

@Artucuno how do I test/use the serial connection through pyboy with these changes?

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