nytimes / ios-360-videos

NYT360Video plays 360-degree video streamed from an AVPlayer on iOS.
https://github.com/nytimes/ios-360-videos
Other
274 stars 38 forks source link

Quality of the video #51

Open kevincador opened 7 years ago

kevincador commented 7 years ago

Hi,

We're playing locally stored videos. We tested with good quality video (found on VLC's website and on the web). Some are 400mb 4K video.

Compared to the experience on VLC on a Mac with the same videos, the quality of the rendering seems bad.

Tested on an iPhone 7 Plus and on an iPad Mini 2. It may have something to do with the @2x or @3x scale.

Thanks for your feedback.

Kevin

nachonet commented 7 years ago

Hi @kevincador,

I was having the same issue so I decided to make some changes that you can see here (in my pull request): https://github.com/NYTimes/ios-360-videos/pull/53

Let me know if you find any trouble. Hope it can be useful!

Nacho

kevincador commented 7 years ago

Wow this is perfect. A huge improvement here. Thanks a lot for this pull request. Hope it gets merged and pushed to Cocoapods soon.

kevincador commented 7 years ago

@nachonet we tested other 360 player and we found out that ios-360-videos is still rendering the video with a quality loss. This is due to the zoom level. Seems that with other 360 player, we are a bit farther from the scene. Since we have a small zoom, the quality decrease.

How would one play with the zoom level of the "camera" to improve the quality of the image?

rhoudinet commented 7 years ago

There are a number of ways the quality of the rendering can be improved: