Skip to content

JPEG2000 endpoints#34

Merged
dgarciabriseno merged 35 commits intoHelioviewer-Project:mainfrom
akash5100:jpeg2000_endpoints
Jul 8, 2022
Merged

JPEG2000 endpoints#34
dgarciabriseno merged 35 commits intoHelioviewer-Project:mainfrom
akash5100:jpeg2000_endpoints

Conversation

@akash5100
Copy link
Collaborator

@akash5100 akash5100 commented Jul 4, 2022

Fixes #6
Fixes #8
Fixes #5
Fixes #7

@akash5100 akash5100 self-assigned this Jul 4, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jul 4, 2022

Codecov Report

Merging #34 (c7d1bbd) into main (d83d2ce) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            6        10    +4     
  Lines           79       133   +54     
=========================================
+ Hits            79       133   +54     
Impacted Files Coverage Δ
hvpy/api_groups/jpeg2000/get_jp2_header.py 100.00% <100.00%> (ø)
hvpy/api_groups/jpeg2000/get_jp2_image.py 100.00% <100.00%> (ø)
hvpy/api_groups/jpeg2000/get_jpx.py 100.00% <100.00%> (ø)
...pi_groups/jpeg2000/get_jpx_closest_to_mid_point.py 100.00% <100.00%> (ø)
hvpy/api_groups/jpeg2000/get_status.py 100.00% <100.00%> (ø)
hvpy/io.py 100.00% <100.00%> (ø)
hvpy/utils.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d83d2ce...c7d1bbd. Read the comment docs.

@akash5100 akash5100 requested a review from wafels July 5, 2022 14:55
@nabobalis nabobalis marked this pull request as ready for review July 7, 2022 04:37
@nabobalis
Copy link
Member

I pushed a change for the reusable validators using the docs example from pydantic.

@nabobalis
Copy link
Member

The file conflict will need resolving before we can merge this.

Co-authored-by: Daniel Garcia Briseno <94071409+dgarciabriseno@users.noreply.github.com>
@dgarciabriseno dgarciabriseno merged commit 882c741 into Helioviewer-Project:main Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JPEG2000 - getStatus endpoint JPEG2000 - getJPXClosestToMidPoint endpoint JPEG2000 - getJPX endpoint JPEG2000 - getJP2Header endpoint

4 participants