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

Angus/hips remote image load #1359

Draft
wants to merge 9 commits into
base: dev
Choose a base branch
from
Draft

Conversation

veggiesaurus
Copy link
Collaborator

@veggiesaurus veggiesaurus commented Feb 12, 2024

Description
Load files from the hips2fits server

Checklist

  • changelog updated / no changelog update needed
  • e2e test passing / corresponding fix added / new e2e test created
  • ICD test passing / corresponding fix added / new ICD test created
  • protobuf updated to the latest dev commit / no protobuf update needed
  • protobuf version bumped / protobuf version not bumped
  • added reviewers and assignee
  • added ZenHub estimate, milestone, and release

@veggiesaurus
Copy link
Collaborator Author

veggiesaurus commented May 8, 2024

@pford Do you think this is on the right track? this is for loading HIPS files from the hips2fits service using cfitsio's ability to load from URL. Some of the changes are required to avoid opening files multiple times

@confluence can we assign this to you or @confluence for cleanup/testing once the frontend is fleshed out? (it's only scripting at the moment)

@pford pford added requiring frontend this backend issue requires work in both frontend and backend requiring protobuf labels May 22, 2024
Copy link

Code Coverage

Package Line Rate Health
src.Cache 72%
src.DataStream 44%
src.FileList 67%
src.Frame 36%
src.HttpServer 42%
src.ImageData 28%
src.ImageFitter 83%
src.ImageGenerators 43%
src.ImageStats 75%
src.Logger 37%
src.Main 52%
src.Region 69%
src.Session 4%
src.Table 52%
src.ThreadingManager 67%
src.Timer 85%
src.Util 38%
Summary 46% (8620 / 18893)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requiring frontend this backend issue requires work in both frontend and backend requiring protobuf
Projects
No open projects
Status: New issues/PRs
Development

Successfully merging this pull request may close these issues.

Support for loading HiPS FITS images
2 participants