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

Clarify orientation requirements for base stations #424

Open
jonnew opened this issue Mar 12, 2025 · 1 comment · May be fixed by #425
Open

Clarify orientation requirements for base stations #424

jonnew opened this issue Mar 12, 2025 · 1 comment · May be fixed by #425
Assignees
Labels
documentation Improvements or additions to documentation
Milestone

Comments

@jonnew
Copy link
Member

jonnew commented Mar 12, 2025

The following text

https://github.com/open-ephys/bonsai-onix1/blob/cdc835e882bc61d5f3121f3d56020401e5c298bd/OpenEphys.Onix1/TS4231V1PositionData.cs#L27C60-L27C80

says the following:

/// <item><description>The base stations are pointed in the same direction.</description></item>

This should changed to:

/// <item><description>The base stations are in the same 3D orientation at an arbitrary 3D offset.</description></item>
@jonnew jonnew added the documentation Improvements or additions to documentation label Mar 12, 2025
@jonnew jonnew added this to the 0.5.0 milestone Mar 12, 2025
@cjsha cjsha changed the title Clarify oientation requirments for base stations Clarify orientation requirments for base stations Mar 12, 2025
@cjsha cjsha changed the title Clarify orientation requirments for base stations Clarify orientation requirements for base stations Mar 12, 2025
@cjsha
Copy link
Member

cjsha commented Mar 12, 2025

This is the most extensive explanation of lighthouse positioning/orienting I've seen. Are there are any more details? I'm going to port this information to onix hardware docs per open-ephys/onix-docs#18

@cjsha cjsha linked a pull request Mar 12, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants