Bug 2225518
| Summary: | PyMapVBVD assumes the platform is little-endian | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Ben Beasley <code> |
| Component: | python-pymapvbvd | Assignee: | Ben Beasley <code> |
| Status: | NEW --- | QA Contact: | |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 39 | CC: | code, neuro-sig |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | s390x | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | --- | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | Type: | Bug | |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | |||
| Bug Blocks: | 485231 | ||
This bug appears to have been reported against 'rawhide' during the Fedora Linux 39 development cycle. Changing version to 39. |
Description of problem: It looks like pymapvbvd assumes that both the binary file format and the host running the code are little-endian. Version-Release number of selected component (if applicable): 0.5.4-2.fc39 How reproducible: Steps to Reproduce: 1. Remove “ExcludeArch: s390x” 2. fedpkg scratch-build --srpm --arches s390x Actual results: =========================== short test summary info ============================ FAILED tests/test_flags.py::test_flagRemoveOS - IndexError: index -1 is out o... FAILED tests/test_flags.py::test_flagIgnoreSeg_flagDoAverage - IndexError: in... FAILED tests/test_flags.py::test_flagSkipToFirstLine - IndexError: index -1 i... FAILED tests/test_read.py::test_gre - IndexError: index -1 is out of bounds f... FAILED tests/test_read.py::test_epi - IndexError: index -1 is out of bounds f... FAILED tests/test_slicing.py::test_vb - IndexError: index -1 is out of bounds... FAILED tests/test_svs.py::test_vb - IndexError: index -1 is out of bounds for... FAILED tests/test_svs.py::test_ve - IndexError: index -1 is out of bounds for... FAILED tests/test_svs.py::test_vb_broken - IndexError: index -1 is out of bou... ======================== 9 failed, 17 warnings in 1.31s ======================== Expected results: All tests pass, as they do on other platforms. Additional info: If the file format is always little-endian, this should mostly be a matter of making endianness explicit when creating a numpy array that is to be copied bytewise in or out of the file: for example, np.dtype('<u4') instead of np.uint32. However, there are a lot of places this would need to be done, and it would require some time and study for me to offer a careful patch—probably more time than I want to spend on it. Upstream would be open to fixing this, but can’t dedicate the time currently if there aren’t a substantial number of potential users on s390x. https://github.com/wtclarke/pymapvbvd/issues/35