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

[ENH] Add back documentation about byte order / endianness documentation #6160

Open
maxnoe opened this issue Apr 24, 2024 · 3 comments
Open

Comments

@maxnoe
Copy link

maxnoe commented Apr 24, 2024

Is your feature request related to a problem? Please describe.

Older versions of the documentation contained mentions of endianness / byte order in the numpy and buffer parts.

These have gone away and I cannot find any mention of byte order / endian issues mentioned in the docs.

Describe the solution you'd like.

Add back information on what cython supports with respect to e.g. non-native byte order numpy arrays

Describe alternatives you've considered.

No response

Additional context

No response

@da-woods
Copy link
Contributor

Do you have a link (possibly to an archive site or anything like that)? I tried searching the commit history for a few likely words and didn't come up with anything.

Is it possible that it might have been a third-party document? I do remember seeing some useful unofficial guides to the finer details of some of the Numpy support.

@maxnoe
Copy link
Author

maxnoe commented Apr 24, 2024

I found this: https://cython-docs2.readthedocs.io/en/stable/src/tutorial/numpy.html

which looked like an old cython documentation, but seems like third party tutorials?

Anyways, the point is that cython code fails with errors like "big-endian buffer not supported on little-endian compiler" which is not documented anywhere now it seems.

We encounter this error regularly, since a data format that is extremely prevalent in astronomy mandates big-endian storage.

da-woods added a commit to da-woods/cython that referenced this issue May 4, 2024
@da-woods
Copy link
Contributor

da-woods commented May 4, 2024

I've added a little more detail in #6181. Mainly about options like cast rather than endianness specifically, but hopefully it's an improvement.

da-woods added a commit that referenced this issue May 11, 2024
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

No branches or pull requests

2 participants