Skip to content

Parallax #445

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Parallax #445

wants to merge 7 commits into from

Conversation

hannalee2
Copy link

Brief summary of changes

On the Parallax side, it uses an HTTP server, so the Parallax binding is very similar to the pathfinder-mpm code.

One request is to add a CLI option for the port number, similar to what pathfinder-mpm provides. The default port for Parallax is currently 8081.

  def _get_binding_instance(self, options: EphysLinkOptions) -> BaseBinding:
    ... 
            if binding_cli_name == options.type:
                # Pass in HTTP port for Pathfinder MPM.
                if binding_cli_name == "pathfinder-mpm":
                    return MPMBinding(options.mpm_port)
                
                if binding_cli_name == "parallax":
                    return ParallaxBinding() # It would be great to have port number option for the parallax 

Notes

@hannalee2
Copy link
Author

Update SERVER_DATA_UPDATE_RATE.
Checked the functionality with @kjy5

@kjy5
Copy link
Member

kjy5 commented May 6, 2025

Let me know when this PR is ready for me to review and integrate!

@hannalee2
Copy link
Author

Hi, Kenneth,
PR is ready to be reviewed. Thanks!

@kjy5 kjy5 self-requested a review May 6, 2025 18:19
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.

2 participants