Skip to content

Commit

Permalink
only enable b-frames and the "video" flag with fps>=10 and at least 2…
Browse files Browse the repository at this point in the history
…0 frames (same as what is used for the target quality when evaluating pipeline options

git-svn-id: https://xpra.org/svn/Xpra/trunk@13439 3bb7dfac-3a0b-4e04-842a-767bc560f471
  • Loading branch information
totaam committed Aug 23, 2016
1 parent efd3588 commit c71eb95
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/xpra/server/window/window_video_source.py
Original file line number Diff line number Diff line change
Expand Up @@ -1277,7 +1277,8 @@ def setup_pipeline_option(self, width, height, src_format,
ve = encoder_spec.make_instance()
options = self.encoding_options.copy()
vr = self.video_subregion
if vr and vr.enabled:
events_count = self.statistics.damage_events_count - vr.set_at
if vr and vr.enabled and vr.fps>=10 and events_count>=20:
options["source"] = "video"
if encoder_spec.encoding in self.supports_video_b_frames:
#could take av-sync into account here to choose the number of b-frames:
Expand Down

0 comments on commit c71eb95

Please sign in to comment.