SDL 0213 - Remote Control - Radio and Climate Parameter Update

[jordynmackool]

Hello SDL community,

The review of "SDL 0213 - Remote Control - Radio and Climate Parameter Update" begins now and runs through December 18, 2018. The proposal is available here:

https://github.com/smartdevicelink/sdl_evolution/blob/master/proposals/0213-rc-radio-climate-parameter-update.md

Reviews are an important part of the SDL evolution process. All reviews should be sent to the associated Github issue at:

#643

What goes into a review?

The goal of the review process is to improve the proposal under review through constructive criticism and, eventually, determine the direction of SDL. When writing your review, here are some questions you might want to answer in your review:

• Is the problem being addressed significant enough to warrant a change to SDL?
• Does this proposal fit well with the feel and direction of SDL?
• If you have used competitors with a similar feature, how do you feel that this proposal compares to those?
• How much effort did you put into your review? A glance, a quick reading, or an in-depth study?
  Please state explicitly whether you believe that the proposal should be accepted into SDL.

More information about the SDL evolution process is available at

https://github.com/smartdevicelink/sdl_evolution/blob/master/process.md

Thank you,
Jordyn Mackool
Program Manager - Livio
[email protected]

[joeljfischer]

Can availableHDs be deprecated entirely if the functionality is subsumed by availableHDChannels? availableHDs and the array size of availableHDChannels should be the same, right?

As a meta point, it would be nice if proposals didn't package all the changes together into one proposal, but that climate, radio, etc. changes were separated into separate proposals.

[joeygrover]

I agree with @joeljfischer we should not be submitting/reviewing proposals that update multiple items if they are not specifically related. For example, all remote control proposals should only include a single module type. This point was actually brought up in a previously submitted proposal for RC changes #296.

[...] . It was also noted that limiting the scope of proposals, especially those related to Remote Control, in the future will help to make for easier understanding and discussion among SDLC Members.

[yang1070]

@joeljfischer I agree that we can deprecated parameter availableHDs, as the count of availableHDChannels shall be the same as availableHDs.

@joeygrover I think it is really a simple change to update the parameters in this proposal, even it has two remote control modules updated. I think the scope is very limited and it shall not have anything prevent the reviewers to understand the proposal.

The point brought up before is saying limiting the scope of proposal will make it easier understanding and discussion, but it does not say it is a pre-condition of submitting/reviewing proposals.

[yang1070]

@joeljfischer @joeygrover we will consider separate proposals per rc module moving forward with new proposals but not for this proposal

[joeljfischer]

@yang1070 To be clear, I think availableHdsAvailable could also be deprecated, right?

[yang1070]

@joeljfischer Yes, you are correct. Basically, for a parameter in module control data, there is a corresponding data in the capabilities. Both shall be deprecated.

[jordynmackool]

The Steering Committee voted to accept this proposal with revisions. The author is to update this proposal to state that availableHDs and availableHdsAvailable are deprecated entirely if the functionality is subsumed by availableHDChannels as finalized in this comment.

[jordynmackool]

The accepted revisions have been made and issues have been entered:
iOS
Android
Core
RPC Spec
SDL HMI