[BUG]: Fix SpaceWarper to handle native space data conversion in "auto" mode #482

Merged
synchon merged 5 commits from fix/native-data-with-auto-space-warper into main 2026-03-12 13:32:01 +00:00
synchon commented 2026-01-28 15:07:28 +00:00 (Migrated from github.com)
  • description of feature/fix
  • tests added/passed
  • add an entry for the latest changes

This PR fixes the logic for SpaceWarper native space data conversion when using="auto".

* [x] description of feature/fix * [x] tests added/passed * [x] add an entry for the latest changes This PR fixes the logic for `SpaceWarper` native space data conversion when `using="auto"`.
fraimondo (Migrated from github.com) reviewed 2026-01-28 15:07:28 +00:00
github-actions[bot] commented 2026-01-28 15:53:12 +00:00 (Migrated from github.com)
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://juaml.github.io/junifer/pr-preview/pr-482/

Built to branch gh-pages at 2026-01-29 12:46 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

[PR Preview Action](https://github.com/rossjrw/pr-preview-action) v1.8.1 :---: | <p><img src="https://qr.rossjrw.com/?color.dark=0d1117&url=https://juaml.github.io/junifer/pr-preview/pr-482/" height="100" align="right" alt="QR code for preview link"></p> :rocket: View preview at <br> https://juaml.github.io/junifer/pr-preview/pr-482/ <br><br> | <h6>Built to branch [`gh-pages`](https://github.com/juaml/junifer/tree/gh-pages) at 2026-01-29 12:46 UTC. <br> Preview will be ready when the [GitHub Pages deployment](https://github.com/juaml/junifer/deployments) is complete. <br><br> </h6> <!-- Sticky Pull Request Commentpr-preview -->
codecov[bot] commented 2026-03-12 12:29:37 +00:00 (Migrated from github.com)

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.55%. Comparing base (bb6de25) to head (6c48e06).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #482   +/-   ##
=======================================
  Coverage   91.55%   91.55%           
=======================================
  Files         146      146           
  Lines        5744     5744           
  Branches      929      929           
=======================================
  Hits         5259     5259           
  Misses        313      313           
  Partials      172      172           
Flag Coverage Δ
docs 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
junifer/preprocess/warping/space_warper.py 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
## [Codecov](https://app.codecov.io/gh/juaml/junifer/pull/482?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) Report :white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 91.55%. Comparing base ([`bb6de25`](https://app.codecov.io/gh/juaml/junifer/commit/bb6de25adac46a529926dbe2a2c939a7ce278c29?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml)) to head ([`6c48e06`](https://app.codecov.io/gh/juaml/junifer/commit/6c48e06369325dc0797869cbbe8f142dcdb500c0?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml)). :warning: Report is 6 commits behind head on main. <details><summary>Additional details and impacted files</summary> [![Impacted file tree graph](https://app.codecov.io/gh/juaml/junifer/pull/482/graphs/tree.svg?width=650&height=150&src=pr&token=5H21JuZXMw&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml)](https://app.codecov.io/gh/juaml/junifer/pull/482?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) ```diff @@ Coverage Diff @@ ## main #482 +/- ## ======================================= Coverage 91.55% 91.55% ======================================= Files 146 146 Lines 5744 5744 Branches 929 929 ======================================= Hits 5259 5259 Misses 313 313 Partials 172 172 ``` | [Flag](https://app.codecov.io/gh/juaml/junifer/pull/482/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [docs](https://app.codecov.io/gh/juaml/junifer/pull/482/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | `100.00% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files with missing lines](https://app.codecov.io/gh/juaml/junifer/pull/482?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml) | Coverage Δ | | |---|---|---| | [junifer/preprocess/warping/space\_warper.py](https://app.codecov.io/gh/juaml/junifer/pull/482?src=pr&el=tree&filepath=junifer%2Fpreprocess%2Fwarping%2Fspace_warper.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=juaml#diff-anVuaWZlci9wcmVwcm9jZXNzL3dhcnBpbmcvc3BhY2Vfd2FycGVyLnB5) | `100.00% <ø> (ø)` | | </details> <details><summary> :rocket: New features to boost your workflow: </summary> - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems. </details>
synchon commented 2026-03-12 13:31:43 +00:00 (Migrated from github.com)

CI passes on juseless.

CI passes on juseless.
Sign in to join this conversation.
No reviewers
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
juaml/junifer!482
No description provided.