xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.


Changes between Version 5 and Version 26 of Ticket #110


Ignore:
Timestamp:
07/23/12 18:24:15 (9 years ago)
Author:
Antoine Martin
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #110

    • Property Status changed from assigned to accepted
    • Property Owner changed from ahuillet to Antoine Martin
  • Ticket #110 – Description

    v5 v26  
    22
    33This ticket tracks remaining improvements which are not required for release but provide improvements we should deal with sooner or later:
    4 * [http://docs.cython.org/src/userguide/external_C_code.html#acquiring-and-releasing-the-gil releasing the gil]: automated testing revealed that x264 and vpx cause a jump in client and server latency. Unfortunately, because we use {{{PyObject_AsReadBuffer}}} to access the python data to encode/decode, we cannot simply add {{{with nogil:}}}. It may be worth trying out multiprocessing, so the encoder/decoder can take advantage of multicore cpus (at the cost of copying the data before processing it..)
     4* [http://docs.cython.org/src/userguide/external_C_code.html#acquiring-and-releasing-the-gil releasing the gil]: automated testing revealed that x264 and vpx cause a jump in client and server latency. Unfortunately, because we use {{{PyObject_AsReadBuffer}}} to access the python data to encode/decode, we cannot simply add {{{with nogil:}}}. It may be worth trying out multiprocessing, so the encoder/decoder can take advantage of multicore cpus (at the cost of copying the data before processing it..) - mostly done already.
    55* add hooks for rate control (set target bandwidth) - this can be in {{{update_batch_delay}}}
    66* add hooks to change color subsampling (currently hardcoded to 4x YUV420)
    77* check if automatic lossless refresh is needed or if ratecontrol does it for free
    8 * fix this warning during build:
     8* fix C compiler warning during build:
    99{{{
    1010xpra/x264\codec.c(1623) : warning C4013: 'change_encoding_speed' undefined; assuming extern returning int
    1111}}}
    12 * re-activate encoding latency detection with a smarter mechanism
    13 * update to newer versions of x264 since we are using the one build we know that does not crash (see r855)
     12(done)
     13* re-activate encoding latency detection with a smarter mechanism - partly done as part of #153
     14* update to newer versions of x264 since we are using the one build we know that does not crash (see r855) - done
     15* OpenGL - see #147 / #150
     16* swscale memalign warning - see #159
     17* ffmpeg win32 build - see #168