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

Maintain alternative pct: format for upscaling (^pct:)? #24

Open
mikeapp opened this issue Apr 17, 2019 · 1 comment
Open

Maintain alternative pct: format for upscaling (^pct:)? #24

mikeapp opened this issue Apr 17, 2019 · 1 comment
Assignees
Milestone

Comments

@mikeapp
Copy link
Member

mikeapp commented Apr 17, 2019

Issue
IIIF/api#1741

Pull Request
IIIF/api#1814
(line 236)

Preview
https://preview.iiif.io/api/1741_image_pct_n/api/image/3.0/#47-canonical-uri-syntax

Summary
Question of whether to maintain alternative upscaling form for pct:n. Keeping the ^pct:n form requires clients to be explicit that upscaling is intended and the status codes for the ^pct:n format allow a distinction between upscaling not supported and other syntax errors.

Resolution
Decision is to keep both forms and to clarify the status codes to be returned for a non-upscaling request that requires upscaling (e.g., pct:110), and a request for upscaling when upscaling is not supported (e.g., ^pct:110 sent to a server that does not support upscaling).

@mikeapp mikeapp self-assigned this Apr 17, 2019
@mikeapp mikeapp changed the title Include rules for upscaled sizes in Canonical URI section Maintain alternative pct format for upscaling? Apr 17, 2019
@mikeapp mikeapp changed the title Maintain alternative pct format for upscaling? Maintain alternative pct: format for upscaling (^pct:)? Apr 17, 2019
@azaroth42 azaroth42 added this to the April 2019 Call milestone Apr 17, 2019
@azaroth42
Copy link
Member

Issue 24 (Maintain alternative pct: format for upscaling (^pct:)?)

+1: 27 [Siani81 ahankinson aisaac andrewgunther awead azaroth42 beaudet dismorfo emulatingkat glenrobson hadro irv jbhoward-dublin jonhartzler joshuago78 jronallo jwd mattmcgrattan mcwhitaker mejackreed mikeapp mixterj regisrob scossu tomcrane tpendragon zimeon]
0: 2 [cubap jtweed]
-1: 0 []

Result: 27 / 29 = 0.93

Super majority is in favor, issue is approved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants