Skip to content

Conversation

@bkeryan
Copy link
Collaborator

@bkeryan bkeryan commented May 8, 2025

What does this Pull Request accomplish?

Add a few test cases for waveforms backed by memory-mapped files.

Why should this Pull Request be merged?

Help to understand what else needs to be done for this use case to be usable.

What testing has been done?

Ran new tests.

@bkeryan bkeryan requested a review from csjall as a code owner May 8, 2025 23:10
@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2025

Test Results

   10 files  ±0     10 suites  ±0   28s ⏱️ -1s
  532 tests ±0    532 ✅ ±0  0 💤 ±0  0 ❌ ±0 
5 320 runs  ±0  5 320 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 93dc0ee. ± Comparison against base commit b20757f.

@bkeryan bkeryan merged commit b69bae7 into main May 9, 2025
14 checks passed
@bkeryan bkeryan deleted the users/bkeryan/memmap-tests branch May 9, 2025 15:45
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.

3 participants