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

CarWindow api should allow app to manually set screen resolution #908

Closed
mrapitis opened this issue Mar 21, 2018 · 2 comments
Closed

CarWindow api should allow app to manually set screen resolution #908

mrapitis opened this issue Mar 21, 2018 · 2 comments
Assignees
Labels
bug A defect in the library
Milestone

Comments

@mrapitis
Copy link
Contributor

Bug Report

CarWindow based apps currently cannot manually set their screen resolution. The CarWindow api automatically sets the screen resolution of the projected app based on values provided by SDL cores RAI response or getsystemcapabilities response. Existing use cases require the app to project at a screen resolution that is lower than values provided by sdl core, however currently the api does not provide a way to allow the app to manually set a screen resolution.

OS & Version Information
iOS Version: n/a
SDL iOS Version: develop branch
Testing Against: n/a

@joeljfischer joeljfischer added the bug A defect in the library label Mar 26, 2018
@joeljfischer joeljfischer added this to the BugFixes milestone Mar 26, 2018
@mrapitis
Copy link
Contributor Author

mrapitis commented Mar 27, 2018

To add information to the original bug report, the code linked below should allow for the app to provide a user specified screen resolution, however the user defined resolution is stored in the preferredResolutions variable which is actually not used by the video encoder to set the screen resolution.

weakSelf.preferredResolutions = [weakSelf.dataSource resolutionFromHeadUnitPreferredResolution:weakSelf.preferredResolutions.firstObject];

Currently the VideoEncoder uses the “screenSize” variable to project the custom resolution as seen in the link below:

self.videoEncoder = [[SDLH264VideoEncoder alloc] initWithProtocol:self.videoFormat.protocol dimensions:self.screenSize properties:self.videoEncoderSettings delegate:self error:&error];

In addition, setting a user preferred screen resolution appears to be dependent on receiving a successful response for SDLVideoStreamingCapability from the head unit (see code linked below), however this also should not be the case as some head unit implementations do not support the getsystemcapabilities rpc.

@theresalech
Copy link
Contributor

Closed via #923

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A defect in the library
Projects
None yet
Development

No branches or pull requests

4 participants