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

PointingXds: POINTING_BEAM #334

Closed
kettenis opened this issue Nov 25, 2024 · 1 comment · Fixed by #345
Closed

PointingXds: POINTING_BEAM #334

kettenis opened this issue Nov 25, 2024 · 1 comment · Fixed by #345
Assignees

Comments

@kettenis
Copy link
Contributor

The description says "optionally expressed as polynomial coefficients" but I don't see how the specified dimensions allows this.

@FedeMPouzols
Copy link
Collaborator

Yes, thanks for catching this. The optional dimension n_polynomial is missing in the table.
The optional dimension n_polynomial is included in the schema but it is not being propagated to the spec tables by the code that pulls that information. It seems we never listed n_polynomial as a coordinate and that seems to confuse the code. I suspect we'll have to define a PointingBeamArray class or similar, also to keep the ordering of dimensions, as n_polynomial is in principle the last one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants