Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace data classes with Linx #32

Open
kabasset opened this issue Feb 9, 2023 · 5 comments
Open

Replace data classes with Linx #32

kabasset opened this issue Feb 9, 2023 · 5 comments
Assignees
Labels
brand new feature Brand new feature not related to any existing service breaking Would break current API

Comments

@kabasset
Copy link
Collaborator

kabasset commented Feb 9, 2023

Use Linx' Raster and Sequence with associated views and iterators to simplify end-user API.

@kabasset kabasset added brand new feature Brand new feature not related to any existing service breaking Would break current API labels Feb 9, 2023
@kabasset kabasset self-assigned this Feb 9, 2023
@kabasset
Copy link
Collaborator Author

Bookkeeping:

  • Fits::Raster replaced with Linx::Raster and VecRaster with Raster
  • Fits::Subraster replaced with Linx::Patch but FileMemRegions not removed yet
  • EleFits' TypeUtils.h simplified thanks to Linx' SeqUtils
  • Column ctors being refactored

@kabasset
Copy link
Collaborator Author

  • FileMemRegions removed
  • Deprecated functions removed

@kabasset
Copy link
Collaborator Author

  • ProgramOptions replaced

TODO

  • Remove Euclid namespace
  • Remove Logger

@kabasset
Copy link
Collaborator Author

  • Euclid removed

@kabasset
Copy link
Collaborator Author

To be updated to new Linx => review already updated doc!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
brand new feature Brand new feature not related to any existing service breaking Would break current API
Projects
None yet
Development

No branches or pull requests

1 participant