Issue metadata
Sign in to add a comment
|
No data received for isac_fix_test from webrtc-linux-large-tests since 17000 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Apr 7 2017
hlundin@: Could you please take a look to see whether this increase makes sense.
,
Apr 7 2017
This is not an increase, it is a data stoppage alert.
,
Apr 7 2017
kjellander@, can you help me figure out why this data stopped? The printout of the metric "time_per_10ms_frame" happens in webrtc/modules/audio_coding/codecs/isac/fix/test/kenny.cc, which is included in the isac_fix_test binary. Do we not run that on but any more? Who killed Kenny?
,
Apr 7 2017
Also, I cannot access the bot stdout from the data points in the graph.
,
Apr 7 2017
This has been reported in https://bugs.chromium.org/p/webrtc/issues/detail?id=7307 since March 8. Still not fixed though.
,
Apr 7 2017
,
Apr 7 2017
,
Apr 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/external/webrtc.git/+/49cad02cf38dc54f8ba0bdab28c3dbd21f1d4e81 commit 49cad02cf38dc54f8ba0bdab28c3dbd21f1d4e81 Author: kwiberg <kwiberg@webrtc.org> Date: Mon Apr 10 09:29:33 2017 Ignore some UBSan errors They proved to be too difficult to fix properly, so we revert the saturation fixes that were done in https://codereview.webrtc.org/2729573002 and https://codereview.webrtc.org/2746903002, and ignore them instead. BUG= webrtc:7307 , chromium:709364 , chromium:693868 Review-Url: https://codereview.webrtc.org/2809483002 Cr-Commit-Position: refs/heads/master@{#17612} [modify] https://crrev.com/49cad02cf38dc54f8ba0bdab28c3dbd21f1d4e81/webrtc/common_audio/signal_processing/include/spl_inl.h [modify] https://crrev.com/49cad02cf38dc54f8ba0bdab28c3dbd21f1d4e81/webrtc/modules/audio_coding/codecs/isac/fix/source/entropy_coding.c
,
Apr 12 2017
This should be fixed as of the CL in comment #9.
,
Apr 12 2017
Checked the graph, and it is now alive again. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by peah@chromium.org
, Apr 7 2017