Skip to content

I/O helpers and real-data tests#132

Open
sk1p wants to merge 26 commits intoLiberTEM:masterfrom
sk1p:data-tests
Open

I/O helpers and real-data tests#132
sk1p wants to merge 26 commits intoLiberTEM:masterfrom
sk1p:data-tests

Conversation

@sk1p
Copy link
Copy Markdown
Member

@sk1p sk1p commented Apr 1, 2026

TODO

  • anisotropic pixel sizes? does the pixel size need to be a tuple always?
  • decide if we want to have a load_from_glob or load_from_list function, and how that should work

Changes...

  • tox config for tests w/ data and notebook tests
  • some test cases that use real input data
  • run cupy tests on a real GPU (and fixed some small cupy issues)

Contributor Checklist:

@sk1p sk1p changed the title Data tests continued I/O helpers and real-data tests Apr 1, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 1, 2026

Codecov Report

❌ Patch coverage is 93.33333% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.15%. Comparing base (2a2c050) to head (8179bfe).

Files with missing lines Patch % Lines
src/libertem_holo/base/io.py 93.15% 1 Missing and 4 partials ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #132       +/-   ##
===========================================
+ Coverage   58.48%   78.15%   +19.67%     
===========================================
  Files          13       14        +1     
  Lines         913      989       +76     
  Branches       85       93        +8     
===========================================
+ Hits          534      773      +239     
+ Misses        351      198      -153     
+ Partials       28       18       -10     

☔ 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.

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@sk1p sk1p mentioned this pull request Apr 1, 2026
4 tasks
@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 1, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 9, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 9, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 9, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

sk1p added 2 commits April 9, 2026 17:31
There should be no silent failure if cupy/cuda stops working
@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 9, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

give up and install the complete cuda toolkit
@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 9, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

see if it picks up the cuda installed via pypi
@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 9, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 9, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 9, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sk1p
Copy link
Copy Markdown
Member Author

sk1p commented Apr 9, 2026

/azp run LiberTEM.LiberTEM-holo

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant