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.

Sign up for GitHub

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

Jump to bottom

Change over to quality to determine picked "default" content types #64

Closed
fugu13 opened this issue Nov 25, 2020 · 1 comment · Fixed by #76
Closed

Change over to quality to determine picked "default" content types #64

fugu13 opened this issue Nov 25, 2020 · 1 comment · Fixed by #76
Assignees
@jam01
Labels
before-camel Things we want to complete before the Camel release goes out

Comments

@fugu13
Copy link
Contributor

fugu13 commented Nov 25, 2020

In case of ties, tiebreaker is order of appearance in header, first wins
@fugu13 fugu13 added the before-camel Things we want to complete before the Camel release goes out label Nov 25, 2020
@fugu13
Copy link
Contributor Author

fugu13 commented Nov 25, 2020

The main UX issue here is that the default quality is 1 and the range is 0 to 1, so making a particular content type a default involves modifying all the others. That said... this isn't a super common use case.

@fugu13 fugu13 assigned fugu13 and jam01 and unassigned fugu13 Nov 30, 2020
@jam01 jam01 mentioned this issue Dec 1, 2020
Merged
@jam01 jam01 closed this as completed in #76 Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jam01 jam01

Labels
before-camel Things we want to complete before the Camel release goes out
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

header default input/output through quality param datasonnet/datasonnet-mapper
2 participants
@fugu13 @jam01