Skip to content

Conversation

@aknierim
Copy link
Member

@aknierim aknierim commented Dec 30, 2025

Adds a data converter class that allows conversion between data formats for pyvisgen.

With this we can skip doing simulations over and over to get the same data in different formats, thus saving us many hours (e.g. ~20 min of conversion vs ~25h of new simulations).

To-Do

  • PT support
  • Docstrings & type hints
  • Docs pages
  • CLI tool
  • Tests
  • Changelog

Note: I don't plan to include FITS for now, until we have fixed our FITS writer. Once that is taken care of, we can add FITS conversion, too.

Note 2: Draft for now until I have added the items from the To-Do list.

Merge after #134

…in pyvisgen

With this we can skip doing simulations over and over to get the same
data in different formats, thus saving us many hours.
@aknierim aknierim added the enhancement New feature or request label Dec 30, 2025
@codecov
Copy link

codecov bot commented Dec 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.67%. Comparing base (6c80d86) to head (409eb58).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
- Coverage   96.20%   95.67%   -0.53%     
==========================================
  Files          32       25       -7     
  Lines        2002     1619     -383     
==========================================
- Hits         1926     1549     -377     
+ Misses         76       70       -6     

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

@aknierim aknierim marked this pull request as ready for review January 5, 2026 16:15
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.

2 participants