Skip to content

Conversation

@terraputix
Copy link
Collaborator

Repeated evaluation of p4nzenc128v16 might yield different results depending on the previous context. Not sure what is the reason, could be due to some uncleared registers or due to other things.

For the moment only happens on x86, ubuntu in docker.

* bump omfileformatc

* update fileformatc agan

* wip: implement c encoder

* reorganize tests

* no extra module named tests

* wip adopt c code

* lifetime fix

* fix offset write test

* fix write_3d test

* more tests

* old writer new reader test

* cleanup

* make old reader work again correctly

* all tests are working

* cleanup

* cleanup

* fix benches

* cleanup prints

* run checks on all pull requests
* test matrix

* run on any PR

* no mmap advice on windows

* fix for different types on msvc

* compression as well

* for now only remove files before test
* WIP

* more wip

* WIP: build basically working

* wip fix tests

* WIP buffered write

* wip remove json

* wip fix tests

* more tests

* all tests

* cleanup

* more cleanup

* error cleanup

* imports

* improve error handling

* rc for child backends

* remove arc

* reformat script and small bug fix in old writer log10

* streaming writes for old writer

* fix benchmarks

* cleanup imports
* bump submodule

* remove OmOffsetSize_t, but some things still don't align fully...

* binary asserts

* better type magix

* fix test

* remove test for debugging

* writer2 usize cleanup

* remove unneeded lifetime
@terraputix terraputix force-pushed the debugging-differences-on-x86-ubuntu-docker branch from 085cfe7 to f67a7aa Compare December 13, 2024 08:35
@terraputix terraputix self-assigned this Dec 13, 2024
Base automatically changed from multidim-file-format to main December 30, 2024 13:49
@codecov
Copy link

codecov bot commented Jul 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.54%. Comparing base (91c7a1b) to head (243ccb0).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #12   +/-   ##
=======================================
  Coverage   87.54%   87.54%           
=======================================
  Files          14       14           
  Lines        1373     1373           
=======================================
  Hits         1202     1202           
  Misses        171      171           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@terraputix terraputix force-pushed the debugging-differences-on-x86-ubuntu-docker branch from 9d3168e to c538fbe Compare July 24, 2025 09:03
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