-
Notifications
You must be signed in to change notification settings - Fork 2
Draft 6 #2
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
Open
andybee
wants to merge
19
commits into
bbc:master
Choose a base branch
from
andybee:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Draft 6 #2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ETSI standardisation of RadioEPG in to an ETSI specification introduced a versioned path for locating IP-based documents. The RadioVIS part of the Slideshow specification also has a similar path for it's endpoint. By using a path we avoid the potential of conflicting with other endpoints on a multi-purpose server. The inclusion of a version number also makes future development of the specification easier to manage without breaking legacy implementations.
RadioDNS and ETSI specifications commonly refer to radio stations as "radio services". The specific "station" param in HTTP POSTs is actually a Bearer definition, therefore it has been changed to "bearer".
There are some concerns about how reliable the time attribute will be from differing clients with differing "clock" sources. It will be difficult to tackle this within the timescales to deliver a finalised 1.0 specification, so as a compromise and to aide future investigation, we have added an optional 'time_source' attribute which allows a client to inform the broadcaster of where they sourced the time from. This will allow a broadcaster to make a better decision about how reliable that time might be and therefore potenitally vary their response to the incoming tag to increase or decreate the range/scope of returned events.
The use of a RadioDNS FQDN format (without the .radiodns.org ending) to define a service is unusual in the wider scope of RadioDNS applications. Changed to the ETSI Hybrid Radio spec-defined bearerURI, which is intended to be a universal identifier for Bearers.
…sing bearerURI format. Term sid was incorrect. Changing sid to bearer leaves us with 2x elements representing a service and it's relevant "location", therefore merging in to a single element with text body as the display name and attribute 'uri' with the bearer URI.
The RadioTAG specification should not be involved in the specification of display restrictions. The 100x100 restriction was placed in the spec due to the targeting of a specific device. Since inclusion, display technologies have improved significantly to the point that 100x100 may look poor on modern displays.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.