VEVO videos only available in 25p? VEVO on YouTube are true 23.976p/25p/29,97p

If I download a video direct from VEVO.com, the files always have 25fps. But that is wrong!
It's only OK, if the original video also has also 25.000 fps, but over 90% of the videos have 23.976 fps or 29.970 fps.
The files from YouTube are fine but quality isn't as good as from VEVO.com itself. I checked several videos.
I want to download the "native" files from VEVO.com. Also when audio and video files are separate. That's not a problem, but it looks like the files are reencoded or (re)muxed in any way. MediaInfo Lite told me that :)
The problem is that the videos stutter while playing them with MPC-HC with auto refresh rate change. I use the correct matching refresh rate for playing every video.
If you need more information, let me know.

Post the full output you get when running with --verbose option.

Microsoft Windows [Version 6.1.7601]
Copyright (c) 2009 Microsoft Corporation. Alle Rechte vorbehalten.
C:\Users\Klaus>Downloads\youtube-dl http://www.vevo.com/watch/jennifer-l
opez/on-the-floor/USUV71100181 -F --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['http://www.vevo.com/watch/jennifer-lopez/on-the-flo
or/USUV71100181', '-F', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2015.02.06
[debug] Python version 2.7.8 - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Proxy map: {}
[Vevo] Retrieving oauth token
[Vevo] USUV71100181: Downloading JSON metadata
[Vevo] USUV71100181: Downloading HLS formats
[Vevo] USUV71100181: Downloading m3u8 information
[Vevo] USUV71100181: Downloading SMIL info
[info] Available formats for USUV71100181:
format code extension resolution  note
m3u8-meta   mp4       multiple    Quality selection URL  (worst)
Low         mp4       176x144     x264@  56k, quicktime@ 24k
Med         mp4       480x360     x264@ 500k, quicktime@128k
High        mp4       1280x720    x264@2048k, quicktime@192k
SMIL_564    mp4       512x288     x264@ 500k, quicktime@ 64k
SMIL_864    mp4       512x288     x264@ 800k, quicktime@ 64k
SMIL_1328   mp4       768x432     x264@1200k, quicktime@128k
SMIL_1728   mp4       768x432     x264@1600k, quicktime@128k
SMIL_2528   mp4       1280x720    x264@2400k, quicktime@128k
SMIL_3328   mp4       1280x720    x264@3200k, quicktime@128k
SMIL_4392   mp4       1920x1080   x264@4200k, quicktime@192k
SMIL_5392   mp4       1920x1080   x264@5200k, quicktime@192k
m3u8-278    mp4       416x234      278k , avc1, mp4a
m3u8-427    mp4       416x234      427k , avc1, mp4a
m3u8-651    mp4       480x270      651k , avc1, mp4a
m3u8-776    mp4       640x360      776k , avc1, mp4a
m3u8-1105   mp4       640x360     1105k , avc1, mp4a
m3u8-1555   mp4       960x540     1555k , avc1, mp4a
m3u8-2894   mp4       960x540     2894k , avc1, mp4a
m3u8-3783   mp4       1280x720    3783k , avc1, mp4a
m3u8-4934   mp4       1920x1080   4934k , avc1, mp4a
m3u8-6047   mp4       1920x1080   6047k , avc1, mp4a  (best)
C:\Users\Klaus>Downloads\youtube-dl http://www.vevo.com/watch/jennifer-l
opez/on-the-floor/USUV71100181 -f m3u8-6047 --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['http://www.vevo.com/watch/jennifer-lopez/on-the-flo
or/USUV71100181', '-f', 'm3u8-6047', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2015.02.06
[debug] Python version 2.7.8 - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Proxy map: {}
[Vevo] Retrieving oauth token
[Vevo] USUV71100181: Downloading JSON metadata
[Vevo] USUV71100181: Downloading HLS formats
[Vevo] USUV71100181: Downloading m3u8 information
[Vevo] USUV71100181: Downloading SMIL info
[debug] Invoking downloader on u'http://hls-aws.vevo.com/v3/hls/2014/09/USUV7110
0181/7a0d8876-e768-4611-b2ff-b2041a71c497/5200/usuv71100181_5200k_1920x1080_h264
_5200_aac_128.m3u8'
[download] Destination: On The Floor-USUV71100181.mp4
[hlsnative] USUV71100181: Downloading m3u8 manifest
[hlsnative] USUV71100181: Downloading segment 1 / 27
...

File is still downloading a second time, so I thought it be enough to post here?

(For reference, the youtube video is https://www.youtube.com/watch?v=t4H_Zoh7G5A)
If Youtube and/or vevo.com reencode the original video we can't do anything, unless they provide a link to the original file.
Note that on vevo.com if you select the format SMIL_5392, it will download a video with the same framerate as bestvideo youtube.com: 23.98 (according to ffprobe).

MPC-HC plays it as 25p.
General
ID                          : 1 (0x1)
Complete name               : C:\Users\Klaus\On The Floor-USUV71100181.mp4
Format                      : MPEG-TS
File size                   : 179 MiB
Duration                    : 4mn 26s
Overall bit rate mode       : Variable
Overall bit rate            : 5 627 Kbps
Video
ID                          : 256 (0x100)
Menu ID                     : 1 (0x1)
Format                      : AVC
Format/Info                 : Advanced Video Codec
Format profile<EMAIL_ADDRESS>Format settings, CABAC      : Yes
Format settings, ReFrames   : 4 frames
Codec ID                    : 27
Duration                    : 4mn 26s
Nominal bit rate            : 5 200 Kbps
Width                       : 1 920 pixels
Height                      : 1 080 pixels
Display aspect ratio        : 16:9
Frame rate mode             : Variable
Color space                 : YUV
Chroma subsampling          : 4:2:0
Bit depth                   : 8 bits
Scan type                   : Progressive
Writing library             : x264 core 142 r2479 dd79a61
Encoding settings           : cabac=1 / ref=4 / deblock=1:0:0 / analyse=0x1:0x111 / me=umh / subme=8 / psy=1 / psy_rd=1.00:0.00 / mixed_ref=1 / me_range=16 / chroma_me=1 / trellis=1 / 8x8dct=0 / cqm=0 / deadzone=21,11 / fast_pskip=1 / chroma_qp_offset=-2 / threads=12 / lookahead_threads=2 / sliced_threads=0 / nr=0 / decimate=1 / interlaced=0 / bluray_compat=0 / constrained_intra=0 / bframes=3 / b_pyramid=2 / b_adapt=1 / b_bias=0 / direct=1 / weightb=1 / open_gop=0 / weightp=2 / keyint=250 / keyint_min=25 / scenecut=40 / intra_refresh=0 / rc_lookahead=50 / rc=cbr / mbtree=1 / bitrate=5200 / ratetol=1.0 / qcomp=0.60 / qpmin=0 / qpmax=69 / qpstep=4 / vbv_maxrate=5200 / vbv_bufsize=5200 / nal_hrd=none / filler=0 / ip_ratio=1.40 / aq=1:1.00
Audio
ID                          : 257 (0x101)
Menu ID                     : 1 (0x1)
Format                      : AAC
Format/Info                 : Advanced Audio Codec
Format version              : Version 4
Format profile              : LC
Muxing mode                 : ADTS
Codec ID                    : 15
Duration                    : 4mn 26s
Bit rate mode               : Variable
Channel(s)                  : 2 channels
Channel positions           : Front: L R
Sampling rate               : 44.1 KHz
Compression mode            : Lossy
Delay relative to video     : -36ms
Menu
ID                          : 4096 (0x1000)
Menu ID                     : 1 (0x1)
Duration                    : 4mn 26s
List                        : 256 (0x100) (AVC) / 257 (0x101) (AAC)
Service name                : USUV71100181
Service provider            : FFmpeg
Service type                : digital television

As I said, you have to download the SMIL_5392 format (you may need to remove the existing video file or use the --output option).
Maybe we should make it the default.

Please check this sample:
VEVO: 25fps
http://www.vevo.com/watch/katy-perry/dark-horse-official/USUV71400083
and youtube version: 23.98 fps
https://www.youtube.com/watch?v=0KSOMA3QBU0
That's the problem.

Can you reproduce that?

All seem to play fine using mpv, ffplay and vlc, but:

Quicktime can't play the m3u8 video, unless you convert it with ffmpeg -i Dark\ Horse\ \(Official\)-USUV71400083.mp4 -bsf:a aac_adtstoasc  -c copy fixed.mp4 (could be more or less the same problem as #4776).
I was playing with a ffmpeg filter to play the two files at the same time and the m3u8 (and only that) doesn't play right.

Downloading the video without the native downloader (requires changing the code) produces a file without the problems I listed, so I guess that's the problem.
To clarify things: if you play the SMIL_5392 (make sure it's redownloaded), does it show the same problems as the m3u8? (if you could tried to convert the file with the ffmpeg command, that would be great).

SMIL_539 is downloading right now. I must wait ...
OK, how do I convert the file? How do I use
ffmpeg -i Dark\ Horse\ (Official)-USUV71400083.mp4 -bsf:a aac_adtstoasc -c copy fixed.mp4
What ffmpeg.exe? Where do I get the same one you are using, just to be sure.
IMPORTANT: Are both m3u8 AND SMIL reencoded or remuxed in any way?

Is it possible to add support for the direct audio and video streams from the VEVO servers?
I know that the VEVO-videos are also on youtube, but yesterday I discovered that the streams from VEVO must be of higher quality compared to youtube!
VEVO:
http://www.vevo.com/watch/kelly-clarkson/Heartbeat-Song/GBE431500003
VEVO-video on youtube:
https://www.youtube.com/watch?v=d4_6N-k5VS4
I made screenshots of the same frames of the videos and you can compare.
I know only hair ;) but you see very clear what I am talking about:

Download page: http://ffmpeg.org/download.html
About m3u8 (or HLS): they split the video in multiple fragments. I would guess they convert it, specially since some qualities are probably used for mobile devices.
About the smil, I don't know.
I don't know why you care so much, it's up to the services to provide the original file.

"I don't know why you care so much, it's up to the services to provide the original file."
I know, but I think there is something in youtube-dl or between me and VEVO that muxes a onlyvideo.mp4 and a onlyaudio.aac to an video+audio.mp4
But exactly here is the problem, I don't know who encodes the videos with x264:
Example: Katy Perry - Dark Horse (SMIL)
MediaInfo Lite:
General
Complete name               : C:\Users\Klaus\Dark Horse (Official)-USUV71400083.mp4
Format                      : MPEG-4
Format profile              : Base Media
Codec ID                    : isom
File size                   : 120 MiB
Duration                    : 3mn 45s
Overall bit rate mode       : Variable
Overall bit rate            : 4 491 Kbps
Writing application         : Lavf55.22.102
Video
ID                          : 1
Format                      : AVC
Format/Info                 : Advanced Video Codec
Format profile<EMAIL_ADDRESS>Format settings, CABAC      : Yes
Format settings, ReFrames   : 5 frames
Codec ID                    : avc1
Codec ID/Info               : Advanced Video Coding
Duration                    : 3mn 44s
Bit rate                    : 4 358 Kbps
Nominal bit rate            : 5 200 Kbps
Width                       : 1 920 pixels
Height                      : 1 080 pixels
Display aspect ratio        : 16:9
Frame rate mode             : Constant
Frame rate                  : 25.000 fps
Color space                 : YUV
Chroma subsampling          : 4:2:0
Bit depth                   : 8 bits
Scan type                   : Progressive
Bits/(Pixel*Frame)          : 0.084
Stream size                 : 117 MiB (97%)
Writing library             : x264 core 140 r2377 1ca7bb9
Encoding settings           : cabac=1 / ref=5 / deblock=1:0:0 / analyse=0x1:0x111 / me=umh / subme=8 / psy=1 / psy_rd=1.00:0.00 / mixed_ref=1 / me_range=16 / chroma_me=1 / trellis=1 / 8x8dct=0 / cqm=0 / deadzone=21,11 / fast_pskip=1 / chroma_qp_offset=-2 / threads=24 / lookahead_threads=4 / sliced_threads=0 / nr=0 / decimate=1 / interlaced=0 / bluray_compat=0 / constrained_intra=0 / bframes=3 / b_pyramid=2 / b_adapt=1 / b_bias=0 / direct=3 / weightb=1 / open_gop=0 / weightp=2 / keyint=250 / keyint_min=25 / scenecut=0 / intra_refresh=0 / rc_lookahead=50 / rc=cbr / mbtree=1 / bitrate=5200 / ratetol=1.0 / qcomp=0.60 / qpmin=0 / qpmax=69 / qpstep=4 / vbv_maxrate=5200 / vbv_bufsize=5200 / nal_hrd=none / filler=0 / ip_ratio=1.40 / aq=1:1.00
Language                    : English
Audio
ID                          : 2
Format                      : AAC
Format/Info                 : Advanced Audio Codec
Format profile              : LC
Codec ID                    : 40
Duration                    : 3mn 45s
Duration_LastFrame          : -10ms
Bit rate mode               : Variable
Bit rate                    : 128 Kbps
Maximum bit rate            : 5 200 Kbps
Channel(s)                  : 2 channels
Channel positions           : Front: L R
Sampling rate               : 44.1 KHz
Compression mode            : Lossy
Stream size                 : 3.43 MiB (3%)
Language                    : English
And youtube version:
General
Complete name               : D:\Videos\Musikvideos MP4-AAC-256 WebM-OGG-192\Katy Perry - Dark Horse (Official) ft. Juicy J [0KSOMA3QBU0]\MP4\Katy Perry - Dark Horse (Official) ft. Juicy J.mp4
Format                      : dash
Codec ID                    : dash
File size                   : 89.7 MiB
Duration                    : 3mn 44s
Overall bit rate            : 3 347 Kbps
Encoded date                : UTC 2014-06-18 06:05:55
Tagged date                 : UTC 2014-06-18 06:05:55
Video
ID                          : 1
Format                      : AVC
Format/Info                 : Advanced Video Codec
Format profile<EMAIL_ADDRESS>Format settings, CABAC      : Yes
Format settings, ReFrames   : 3 frames
Codec ID                    : avc1
Codec ID/Info               : Advanced Video Coding
Duration                    : 3mn 44s
Bit rate                    : 3 343 Kbps
Width                       : 1 920 pixels
Height                      : 1 080 pixels
Display aspect ratio        : 16:9
Frame rate mode             : Variable
Frame rate                  : 23.976 fps
Minimum frame rate          : 23.974 fps
Maximum frame rate          : 1 046.512 fps
Color space                 : YUV
Chroma subsampling          : 4:2:0
Bit depth                   : 8 bits
Scan type                   : Progressive
Bits/(Pixel*Frame)          : 0.067
Stream size                 : 89.7 MiB (100%)
Encoded date                : UTC 2014-06-18 06:05:55
Tagged date                 : UTC 2014-06-18 06:05:55
Video only!!
MPC-HC adds audio automatically.
You see framerate doesn't match.
And that is causing stuttery playback, but you oly see that if you use the correct refresh rate of your monitor, not every supports all, mostly they only support 60.000Hz

Please note the file is constant frame rate not variable as detected by MediaInfo, I already told them that detcts it wrong, but they told me to pay if I want that be fixed. Strange people. I thought free and open source is free and open source. Anyway that's not the point here.
I made clips with my mobile phone to show you how to play the files the right way. -> MPC-HC uses the correct refresh rate or double (it is for interlaced videos) But 25 fps can be watched without problems at doubled refresh rate.
Here are the links to the files:
https://drive.google.com/file/d/0B4y5u0gBKYqbUm5WX2toOFhIb2c/view?usp=sharing
https://drive.google.com/file/d/0B4y5u0gBKYqbN1kzamV0dVRxR0k/view?usp=sharing

I'm sure we find a solution for my desire :)

I can't see any obvious difference from the videos you posted, could you describe the problem better?

I know, but I think there is something in youtube-dl or between me and VEVO that muxes a onlyvideo.mp4 and a onlyaudio.aac to an video+audio.mp4

Why do you think that?

But exactly here is the problem, I don't know who encodes the videos with x264:

I think I don't understand this part, vevo is encoding their videos with x264, that's all.

"I can't see any obvious difference from the videos you posted, could you describe the problem better?"
The 23.976 video is playing smooth, the 25.000 video has microstutters every 4 or 5 seconds. It is possible that you can't see these in my clips from my mobile phone. I'm sure it can't be seen here. My intention with the clips is, that you see, the display has to switch to another refresh rate in order to display the files right. And the red and green line must be parallel in display stats CTRL + J.
"I think I don't understand this part, vevo is encoding their videos with x264, that's all."
I inspected the files with MediaInfo and saw that the files are encoded or at least muxed with:
"Writing application         : Lavf55.21.100"
-> This is open source software and I really can't imagine that VEVO uses this. Normally download tools use ffmpeg to combine audio and video to merge it into one file so average user can play it with VLC media player which doesn't support external audio like MPC-HC does (there is only a really bad support and a very complicated way -< doesn't work properly)
That is the reason why I said that I want the files straight from vevo without any muxing with Lavf55.21.100. The other filestype is at least encoded with x264.
But if you say there is no muxing with ffmpeg during I download this file from VEVO.com with youtube-dl, there is nothing we can do.  But I really can't believe that they did framerate conversions, and really such bad ones
The youtube files are fine, but I am interesed in higher bitrate videos, because youtube is continuously going down with bitrate and picture quality is suffering from this for month now.
Youtube also did framerate conversions in the past, like 23.976 to 30.000. But they learned what they did wrong :)
Just for completion: Is there really no ffmpeg or any other tool used for compining several segments of the video. And also not to combine audio with video?
Youtube also does segment, right? But here you see there is so ffmpeg used to combine the segments: UTC is written by Google a time before the video is published.
!!! Keep in mind that Google/YouTube encodes much of their videos new (with of course lower bitrate, so the video do look ugly) so there is probably a new UTC written in the files !!!
For example the file was encoded before it was published on YouTube:
Compare date on https://www.youtube.com/watch?v=BfgoiYu9EUE with following UTC:
General
Complete name               : D:...\Jennifer Lopez - First Love (Official Video).mp4
Format                      : dash
Codec ID                    : dash
File size                   : 105 MiB
Duration                    : 5mn 2s
Overall bit rate            : 2 905 Kbps
Encoded date                : UTC 2014-05-28 14:12:51
Tagged date                 : UTC 2014-05-28 14:12:51
Video
ID                          : 1
Format                      : AVC
Format/Info                 : Advanced Video Codec
Format profile<EMAIL_ADDRESS>Format settings, CABAC      : Yes
Format settings, ReFrames   : 2 frames
Codec ID                    : avc1
Codec ID/Info               : Advanced Video Coding
Duration                    : 5mn 2s
Bit rate                    : 2 902 Kbps
Width                       : 1 920 pixels
Height                      : 1 080 pixels
Display aspect ratio        : 16:9
Frame rate mode             : Variable
Frame rate                  : 23.976 fps
Minimum frame rate          : 23.974 fps
Maximum frame rate          : 24.220 fps
Color space                 : YUV
Chroma subsampling          : 4:2:0
Bit depth                   : 8 bits
Scan type                   : Progressive
Bits/(Pixel*Frame)          : 0.058
Stream size                 : 105 MiB (100%)
Encoded date                : UTC 2014-05-28 14:12:51
Tagged date                 : UTC 2014-05-28 14:12:51
And that's a newer coded file from https://www.youtube.com/watch?v=BfgoiYu9EUE
General
Complete name               : D:...\Jennifer Lopez - First Love (Official Video).mp4
Format                      : dash
Codec ID                    : dash
File size                   : 78.1 MiB
Duration                    : 5mn 2s
Overall bit rate            : 2 167 Kbps
Encoded date                : UTC 2014-05-31 15:08:09
Tagged date                 : UTC 2014-05-31 15:08:09
Video
ID                          : 1
Format                      : AVC
Format/Info                 : Advanced Video Codec
Format profile<EMAIL_ADDRESS>Format settings, CABAC      : Yes
Format settings, ReFrames   : 3 frames
Codec ID                    : avc1
Codec ID/Info               : Advanced Video Coding
Duration                    : 5mn 2s
Bit rate                    : 2 164 Kbps
Width                       : 1 920 pixels
Height                      : 1 080 pixels
Display aspect ratio        : 16:9
Frame rate mode             : Variable
Frame rate                  : 23.976 fps
Minimum frame rate          : 23.974 fps
Maximum frame rate          : 775.862 fps
Color space                 : YUV
Chroma subsampling          : 4:2:0
Bit depth                   : 8 bits
Scan type                   : Progressive
Bits/(Pixel*Frame)          : 0.044
Stream size                 : 78.0 MiB (100%)
Encoded date                : UTC 2014-05-31 15:08:09
Tagged date                 : UTC 2014-05-31 15:08:09
Three days later much smaller videoonly filesize !!! I bet not so many know this :)
If I should explain anything better let me know -> I'm not native english, I'm from bavaria :D

Is ffmpeg used while downloading from VEVO.com?

youtube-dl doesn't use ffmpeg when downloading from VEVO.com, they are using it (ffmpeg can produce m3u8 manifest and segments from a video) themselves.
I think there's nothing that youtube-dl can do, so we should close the issue.

Yeah, I agree with you.
OK, so I stay downloading VEVO videos from YouTube. Their videos have the original framerate, no converted framerate from 23.976 (24.000/1.001) to 25.000 or similar.
To everyobdy, who reads this thread and is interested:
To watch the jerkyness of the converted files, play a 25.000fps VEVO file which is 23.976fps at YouTube. I recommend MPC-HC, and also MPC-BE.
The refreshrate of your monitor must match the framerate of your playing file.
Please note: NOT every monitor does support all refresh rates like 23/24/25/29/30/50/59/60Hz.. 23Hz mode is for 23.976, 59Hz for 59.970, 29Hz for 29.970. Mine does all :D
Most ones only do 60 Hz. LOL
Maybe anybody could tell the guys from VEVO that their videos are encoded at wrong framerate!
I'm very dissapointed from the video encoding guys at VEVO.
@ VEVO guys: SHAME ON YOU!!!

Im not sure what the rendering engine in MPC is or does, but if you play a segment or the whole HLS stream in something like VLC then non of these issues exists because the frame rate is available in the TS header which in turn defines how the timing is rendered in VLC.  There are technical reasons why we use 25fps and a proper frame rate conversion is done to achieve this which is nothing extraordinary or unusual.  Im unable to see any issues with rendering the HLS encoded content in VLC or QT or our great new vevo.com website.  For the record we encode our own content as do YT from our source mezzanine files - for some additional light reading please checkout our blog http://blog.vevo.com/smart-content-encoding-at-vevo/.  Do let me know if there is more we can do to help answer your questions, but i suspect this is MPC unable to tolerate our frame rate choices.

Thank you for your reply.

There are technical reasons why we use 25fps...

What's exactly the reason for the conversion?

...a proper frame rate conversion is done...

And that's exaclty the point, what is wrong with most files
What refresh rate do you use for this test with VLC?
A very annoying kind of framerate conversion is from 23.976 fps (24.000/1.001) to 25fps by adding a frame.

Don't you help me?

files are coded wrong, because there IS a framerate conversion
Still no help?

By @nickvh's description, it seems a Vevo's internal decision to use 25fps. Ask @nickvh or other Vevo people for their problems. youtube-dl just downloads the file(s) served by Vevo, and we can't provide any help here.

OK, I asked @nickvh for help, but I think I don't get any from him and I don't want to start a new thread anywhere, because I know they are coding their files wrong, I wouldn't know where to post it if I would do.

Or you may want to forward this bug to https://trac.mpc-hc.org/, MPC's bug tracker. If a video can be played fluently in many players except MPC, it might be MPC's fault.

Ok, I explain it short, a music video which is shot at 23.976 fps is reconverted so every seconds a frame is duplicated when framerate is converted to 25 fps, a very good or bad example is the one mentioned already above:
https://www.youtube.com/watch?v=0KSOMA3QBU0
coded right from youtube
http://www.vevo.com/watch/USUV71400083
bad encoding with 25fps
but keep in mind you have to use the mathcing refresh rate of your Monitor or TV.
VLC can't switch it on its own, but MPC-HC and MPC-BE can.
But only some can display 23 Hz / 24 Hz / 25 Hz / 29 Hz / 30 Hz / 50 Hz / 59 Hz / 60 Hz, most devices only can 60 hz so 3:2 pull down is applied which I don'T ant to see. I only want native 24p. But there is 23 Hz setting for 23.976 (24.000/1.001) and 24.000p. And VEVO also does a framerate conversion from 59.940 (59 Hz mode) to 25 fps which is very stuttery. They also do upscaling 720p to 1080p which is a waste.

OK by this example problems are from videos but not players. Only Vevo can fix it.

+1 for yan12125, if the content cant be played in MPC but plays at good quality via HTML 5 and our 12 other platform players - id look to MPC to resolve this dynamically..
We encode to 25fps for business reasons.

It is not a bug by MPC HC or MPC BE. WHAT DO I HAVE TO WRITE SO YOU GET IT?
Do you know what a refresh rate is? You can only play a 60 hz video like this one smooth when using a standard refresh rate of 60 Hz:
https://www.youtube.com/watch?v=_fC53l3jGAc
The refresh rate of the graphic card must match the framerate of the video. The standard user has a monitor which only supports 60 Hz.
If you live in PAL land like me in the deep bavarian forrest, you can also play 50p videos, but only smooth if you set the refresh rate in the grafic card to 50 hz.
Do you now get what Vevo is doing wrong? It is not a bug by MPC.
What are business reasons? JUST LEAVE THE FRAMERATE AS IT IS.
The more I need to write here and have to repeat Me several times, I personally think you want me to get angry by ignoring my posts.
What do I need to explain now?

By this example I guess there's a duplicated frame in each second, as described previously by @Klaus1189. If this is really the case, Vevo should check their video encoders.
Need more time to analyze the downloaded file.

HTML 5 is the same crap in the brower, but it should be fine if the refresh rate is set to the matching framerate of the video. A clever videoplayer like MPC can do this on its own.
WATCH THE CANYON 60 hz video and you know what I mean by matching refreshrate.
Yeah, @yan12125 understands my point now. Vevo encoders are silly.
And they also upscale 720p hd to 1080p fullhd. What a waste. And if the video of live concerts is shot at 59.940p (59 hz) mode in grafic driver, a framerate conversion to 25 is more than silly. It ruins complete video experience.

Can you decide between technical reasons and business reasons for the framerate conversion to 25 fps regardless of the source framerate? I'm speechless ...


faster zeroing of .bss

This PR updates the code for zeroing the .bss section.
The updated version uses core::ptr::write_bytes() which invokes a single memset,
compared to the current code which initializes one byte at a time.
The updated version would initialize the .bss section faster than the current version.
Thank you for reviewing this PR :) :bowing_man:

bors d+
General note: it is more important for libtock-rs to be small than fast. This PR reduces code size, with no significant adverse effects, so I think it is an improvement.

I organized a table from the CI's size-diff results. There are cases where the .text size rather increase..

Risc-V: (out of 16 cases) binary size decreased in 10 cases, increased in 6 cases.
Armv7: (out of 23 cases) binary size decreased in 18 cases, increased in 5 cases.

I'll later look at the programs that increased in size, to get an understanding of why it's happening.
 It could also make LLVM make a different choice early on during optimization, causing large changes to the layout of the generated code.
Most examples shrunk, and none of the remaining examples grew precipitously, so this is an improvement.

bors r+


Broken FAQ link on the main website

At pockethost.io/, the FAQ button links to https://pockethost.gitbook.io/manual/daily-usage/faq, which is a broken link. I think this is supposed to be linking to https://pockethost.gitbook.io/manual/overview/faq instead.

Great catch! Fixed.


HHH-12622 JdbcResourceLocalTransactionCoordinatorImpl#markRollbackOnly should be ignored if there is no TX

https://hibernate.atlassian.net/browse/HHH-12622

@scottmarlow In case you didn't see my comment on HipChat, could we get a test case for this?

Test case is now provided and for those not on hipchat, Steve pushed a change that didn't help with this issue, so my fix is still needed.

Fix is no longer needed.  Steve fixed it at a higher level.


Improved style property setting

From https://github.com/jamwaffles/embedded-graphics/issues/139#issuecomment-515026696:

I'd be happy to explore a solution using T: Into<Option<C>> as suggested, as I think being able to pass None for a style property keeps the API cleaner and smaller than having no_*() methods.

To set a style:
thing.stroke(Rgb565::RED);
// or
thing.stroke(Some(Rgb565::RED));

To clear a style:
thing.stroke(None)

Using Into<OptionC>> keeps the API clean with one method per property, allows ergonomic use without Some() peppered everywhere but also supports un-setting the style by passing None in.

I came across the gtk-rs release notes yesterday, which contained some interesting points:
https://gtk-rs.org/blog/2019/06/22/new-release.html#stepping-back
The essentially just reverted back to using Some and None.

Hmm. Their API surface is a lot larger and more complex than e-g's. Perhaps we see how ergonomic it is for e-g and change it if there are lots of confusing errors from usage in the wild?

OK. The link was primarily a reminder that we should check that there are no side effects we didn't consider.
Should we implement a similar change for Points, by using impl Into<Point>? Example from #158:
display.draw(
    Rectangle::new(point(0, 0), point(64, 64))
        .translate(point(96, 0))
        .stroke(Some(BinaryColor::On)),
);

display.draw(
    Rectangle::new((0, 0), (64, 64))
        .translate((96, 0))
        .stroke(BinaryColor::On),
);

Having thought about it some more, the gtk-rs post makes a very good point:

The second one is to go back to some Rust fundamentals: being explicit.

I agree. It's non-obvious to the programmer that a style can be un-set by passing None (unless they read the docs). The ergonomics of not having to type Some() or Point::new() don't really outweigh this fact IMO. I'd suggest we leave the API as is for now. There might be a better solution come up with later on. I'll close this issue for now but I'm happy to reopen if there's a strong desire for a more concise API.

I don't think I saw anything, but can you confirm that #158 and #152 don't use Into<Point> anywhere?

No, they don't.
I suggested this because the code example above looked strange with the point function. But using Point::new instead does make it look better.

Yeah it does look odd I agree. Slightly better with macros, but now that the names are shorter I don't think these helpers are required at all :slightly_smiling_face:.


Folder Structure

Hello, I saw KLVTZ (Isolating frontend dependencies. #396) and i think that the folder structure is scattered too. So i made the changes that i like but keeping the same as KLVTZ idea. There are no issues with the update as i have done the required changes. Here are the changes that I have made:
Folder Structure
assets
├── components
├── stylesheets
├── images
├── javascript
│   ├── custom
│   └── vendor
├── scss
│   ├── config
│   └── site
└── fonts

Files Changes

Bowers components are location in assets/components
Grunt.js
bower.json (Updated Font Awesome)
Images
Foundation.scss (Is normalize and Font Awesome now coming from components)
settings.scss
enqueue-scripts.php (New locations)
Many more..

Images

Installing

Thank you for making FoundationPress as it makes Wordpress themes easy to develop. :)
Brandon

Good job :+1:


Exception thrown when debugging tests on latest Main

If I attempt to debug tests on latest VSMain I get the following error which causes the test to fail before I can see anything "interesting":
System.Runtime.InteropServices.COMException: 'Exception from HRESULT: 0x8971001E'

This is thrown from https://github.com/microsoft/vs-extension-testing/blob/main/src/Microsoft.VisualStudio.Extensibility.Testing.Xunit.Shared/Harness/VisualStudioInstance.cs#L45. This replicates both against the tests in this repo and those in razor-tooling.

I'm able to get this working for the tests in this repo by changing from Attach2 to Attach. @sharwell I know that code is from awhile ago but do you recall why we chose Attach2("Managed") over Attach() or in general have context on if the above would be an acceptable change or what alternatives this implies?

The work-around for this I've found is to launch the test with "Run" and then manually attach the debugger once the new VS instance launches. You've got to be paying attention but it's pretty easy to jump in on time.


Layer not showing on top after dragging an analysis-node layer below it

Metro Madrid use case - step 4
1-Add AOI analysis to Layer 1.
2-Drag AOI analysis node (Layer 2) below Layer 1.

The expected view should be Layer 1 on top of Layer 2. This is solved after refreshing the browser:

The problem is hitting again, could you (any of both @viddo or @alonsogarciapablo) take care of it?

Not even refreshing works, because the point layer that is on top in the last screenshot is from the analysis node.

The problem is hitting again, …

Just to confirm what problem we're talking about:
@ramiroaznar, from the screenshots it looks like the layers appear in correct order in the layers list on the left, correct? Assuming that's the case:
It looks like it's the same problem as described in issue #8245 and supposedly fixed in PR #8292. @alonsogarciapablo could there be something that changed related to this in cartodb.js since the last fix that provokes this from happening again?

@viddo it is the same problem, but I think it is not solved. Check the following screenshots:
The Layer 1 (black points) should be displayed on top of Layer 2 (orange points and buffers):

Because when I disable the Layer 2 view, Layer 1 is shown:

@alonsogarciapablo could there be something that changed related to this in cartodb.js since the last fix that provokes this from happening again?

We haven't changed anything lately that could have broken this as far as I know.

Hey, that happened to me also a few days ago (before releasing). Hard to replicate since it doesn't not happen always, but looks something related with the cache @rochoa

I submitted this PR to ensure the order of layers is the same in both the builder and CDB.js. But, that wasn't actually the real issue why this was failing.
Here's what I discovered. When you add an analysis to a layer, the CartoCSS of that layer is reseted to:
// points
 [mapnik-geometry-type=point] {
    marker-fill: #FF6600;
    marker-opacity: 1;
    marker-width: 12;
    marker-line-color: white;
    marker-line-width: 3;
    marker-line-opacity: 0.9;
    marker-placement: point;
    marker-type: ellipse;marker-allow-overlap: true;
  }

 //lines
 [mapnik-geometry-type=linestring] {
    line-color: #FF6600;
    line-width: 2;
    line-opacity: 0.7;
  }

 //polygons
 [mapnik-geometry-type=polygon] {
    polygon-fill:#FF6600;
    polygon-opacity: 0.7;
    line-opacity:1;
    line-color: #FFFFFF;
   }

AFAIK, since these selectors don't have an ID (eg: #layer), they affect all of the layers in the map. In the particular situation that @ramiroaznar described in this comment. The layer with the black points, was being affected by this generic cartocss, and that's why he was seeing the orange points inside of the orange polygons.
Changing that default CartoCSS to the following seems to fix the problem:
// points
#layer[mapnik-geometry-type=point] {
    marker-fill: #FF6600;
    marker-opacity: 1;
    marker-width: 12;
    marker-line-color: white;
    marker-line-width: 3;
    marker-line-opacity: 0.9;
    marker-placement: point;
    marker-type: ellipse;marker-allow-overlap: true;
  }

 //lines
#layer[mapnik-geometry-type=linestring] {
    line-color: #FF6600;
    line-width: 2;
    line-opacity: 0.7;
  }

 //polygons
#layer[mapnik-geometry-type=polygon] {
    polygon-fill:#FF6600;
    polygon-opacity: 0.7;
    line-opacity:1;
    line-color: #FFFFFF;
   }

I'm trying to find out where that cartocss is coming from.

Ahhhh that's a good one man! Well spotted!

@alonsogarciapablo BRAVO!

Default CartoCSS seems to be coming from https://github.com/CartoDB/cartodb/blob/master/lib/assets/javascripts/cartodb3/editor/style/style-converter.js#L5. But, we also have these default styles (https://github.com/CartoDB/cartodb/blob/master/lib/assets/javascripts/cartodb3/data/camshaft-reference.js#L121) which would totally fix the issue.... @xavijam I'd love your advice here since you're more familiar with stylin'.

adding @javisantana, he will take care of it.

@alonsogarciapablo I'm changing the default style, I'll ping you in the PR ok?

@javisantana: 👍

@alonsogarciapablo https://github.com/CartoDB/cartodb/pull/8713/files#diff-1b2da36fb3e2b1fd1420c14e984d4fe3R121

After first bunch of changes about that, could we close this one?

We can close it if @javisantana fixed this.


Select the option

how to choose option 1 or others on black circle, and why be "undefined" on the other circle?
I have this kind of problem on another website, maybe someone can help me.
thank you..

Currently our Lite version works only for clicking element and not interact with Form fields like input, textarea, or dropdown. We are working on it to create PRO version of same which can do all short of stuff you want. So stay connected with us. We will publish PRO version soon in Jan 2018

my problem about "undefined" already clear, great thanks to you. it's really helpful, and sure i'll stay connected..

Thanks


Gui

Creating a gui.py file under gui folder and removing the code present in readme file.

But it is created in readme for better understanding.

But readme can be edited and instructions can be added how to run, readme should not contain code I guess

Ok.

Solved in #6


⚠ Jenkins Plugins Artifactory has degraded performance

In 86c4630, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1234 ms

Resolved: Jenkins Plugins Artifactory performance has improved in ffa29ea after 12 minutes.


[WIP] Update Blueprint edit API

This is a WIP as it is dependent on both #13352 and #13456. When a Blueprint is "Edited" it should automatically become a new version. This will use the new_version method and return the newly created version of the Blueprint.
Links

Pivotal Ticket

@miq-bot add_label wip, enhancement, api
@miq-bot assign @abellotti

Not really crazy about this, from an API perspective, an edit shouldn't be changing a resource id.  I prefer exposing this as a different action, if the underlying method is new_version, then let's expose it as new_version.

closing, cc: @chriskacerguis


Switch to new go version

go1.10 is out there. Use it!

@sergefdrv Let me clarify a goal of this Issue.
Since Go 1.10 has no significant changes to the language specification, is updating Dockerfiles enough for this?

@ynamiki Exactly, that is only about updating Dockerfiles and Readme. That means this issue is to be concerned after #7 is resolved.


Percentage bar

Progress bad is calculating overall percentage rather than the percentage of the amount you need from 1 to the next.
For example: I have 4 legendaries, and and i have 2940 kill points right now. 2940 is 61% of 4800 (the 5th legendary) which is what your calculator is doing.
What it needs to be doing is calculating the amount you need from one to the next. So from 4 to 5 I would need 2619 points. I have 759 out of 2619 which is 28.9%.

I'll take a look. The progressbar was added as a joke though, since the dropchance is random anyway. That's why there's a Sephuz at the end as well :)

Progress bar removed due to significant changes in legendary drops / crafts.


How to setup Namespaced-scope Helm Operator with 1.0.0?

Type of question
How to
Question
Is changing the RBAC files to Roles from ClusterRoles enough to make the Helm operator watch a namespace only?
Also, is it possible to add WATCH_NAMESPACE to the manager YAML so a namespace doesn't need to be specified? It's unclear if this is possible or what needs to be done for Helm since the doc discusses Go Manager options.
Environment

operator-sdk version: 1.0.0

Hi @pgerv12,
The steps are quite similar, however, we need doc it per type as you suggested. We have a doc per type.  See that we have a task for it already.  https://github.com/operator-framework/operator-sdk/issues/3688
Regards your questions:
If you look here it is the implementation of your main.go when you are using a Helm-based Operator.
So, you can use the WATCH_NAMESPACE as it was in the legacy layout in the config/manager/manager.yaml which allows you inform 1 namespace or a set of them ( multi-namespace scenario that you can see in the Operator Scope doc for Go).
Also, feel free to contribute to it. Your PR's are very welcome.   🥇
I will supplement the info in #3688 as well just in case someone decides to collab with it.
I am closing this one as sorted out. However, please feel free to ping us if you think that it should be re-opened.


Invalid schema still stored

Hi!
When I tried to add a new schema to the registry, I had a typo and I got an "Given schema is invalid" error. I corrected the typo in my schema and clicked "Save", and this time a new error showed up: "[Storing the given SchemaMetadata [SchemaMetadata{...}] is failed]". This is caused because even though the first error is prompted, the schema is still saved.
In addition, taking into account that there is no "direct" way to delete an existing schema, I have to deal with a schema version that will never work nor be used.
Bye!

@raju-saravanan It's 0.3.0

@unaipme There are two steps involved when you add new schema version with metadata config like name, description, compatibility etc

create schema metadata like name, description, compatibility etc.
add new schema version for the above schema metadata

It may be possible that first step is successful and second step is failed because of wrong schema text. But when you correct that text and try to create again, UI may be trying to create metadata which already exists and it may be adding new schema version.
@shahsank3t Can you look into this?

Hi, @satishd . It's actually only one step, at least for me it shows up like in the screnshot below:

@unaipme UI shows as one step but there are two steps to the backend.

I'll take a look at this today.

It may be possible that first step is successful and second step is failed because of wrong schema text. But when you correct that text and try to create again, UI may be trying to create metadata which already exists and it may be adding new schema version.

@satishd Yes, that's exactly what's happening. I am adding a fix where if the schema has been created but adding version(second step) fails, only that will be called again by clicking save.
@unaipme Thanks for reporting the bug, I'll fix and raise a PR soon.

Raised PR: https://github.com/hortonworks/registry/pull/318


Timer for cleaning clipboard history

Add option to clear clipboard history automatically after specific time period set in extension settings.

Thanks for the suggestion, but this isn't going to happen. You can already:

Delete non-favorited items on reboot by choosing the "Only save favorites to disk" option.
Delete items after they exceed a count or storage size though the "Max number of items" and "Max clipboard history size" options.

If #40 ends up happening, then perhaps this would make sense, but I currently don't the the value justifying the complexity.


Try to make gazebo-sitl compatible with Parrot Sphinx Gazebo 7 (libignition-math2.so.2.2.2)

Parrot Sphinx currently only supports Parrot based products. There is no reason the gazebo-sitl plugin for ArduCopter can't be added as well. Parrot uses Gazebo 7.0.1, which is actually version 7.0.0, so the current gazebo-sitl needs to be a bit more backwards compatible.
http://developer.parrot.com/docs/sphinx/index.html
There is currently a compile error related to the Target Pose that needs fixed, the offending code is commented out in this commit so that things compile cleanly.  https://github.com/MAVProxyUser/gazebo-sitl/blob/dc8d20c531500f6d0f9a7795998ea847023dbf44/gzsitl/gzsitl_plugin.cc#L128
[ 87%] Building CXX object gzsitl/CMakeFiles/gzsitl_plugin.dir/gzsitl_plugin.cc.o
/opt/mesmer/log/gazebo-sitl/gzsitl/gzsitl_plugin.cc: In member function ‘void gazebo::GZSitlPlugin::OnUpdate()’:
/opt/mesmer/log/gazebo-sitl/gzsitl/gzsitl_plugin.cc:130:28: error: no match for ‘operator=’ (operand types are ‘ignition::math::Pose3d {aka ignition::math::Pose3}’ and ‘const gazebo::math::Pose’)
this->perm_target_pose = this->perm_target->GetWorldPose();
^
/opt/mesmer/log/gazebo-sitl/gzsitl/gzsitl_plugin.cc:130:28: note: candidate is:
In file included from /usr/include/ignition/math2/ignition/math/Frustum.hh:22:0,
from /usr/include/ignition/math2/ignition/math.hh:7,
from /usr/include/sdformat-4.3/sdf/Param.hh:35,
from /usr/include/sdformat-4.3/sdf/Element.hh:24,
from /usr/include/sdformat-4.3/sdf/sdf.hh:5,
from /usr/local/include/gazebo-7/gazebo/common/Battery.hh:25,
from /usr/local/include/gazebo-7/gazebo/common/common.hh:8,
from /opt/mesmer/log/gazebo-sitl/gzsitl/gzsitl_plugin.hh:20,
from /opt/mesmer/log/gazebo-sitl/gzsitl/gzsitl_plugin.cc:20:
/usr/include/ignition/math2/ignition/math/Pose3.hh:222:25: note: ignition::math::Pose3& ignition::math::Pose3::operator=(const ignition::math::Pose3&) [with T = double]
public: Pose3 &operator=(const Pose3 &_pose)
^
/usr/include/ignition/math2/ignition/math/Pose3.hh:222:25: note:   no known conversion for argument 1 from ‘const gazebo::math::Pose’ to ‘const ignition::math::Pose3&’
make[2]: *** [gzsitl/CMakeFiles/gzsitl_plugin.dir/gzsitl_plugin.cc.o] Error 1
make[1]: *** [gzsitl/CMakeFiles/gzsitl_plugin.dir/all] Error 2
make: *** [all] Error 2

My initial attempt to use:
this->perm_target_pose = this->perm_target->GetWorldPose();

is wrong because it returns math::Pose, we expect  ignition::math::Pose3d& WorldPose
http://osrf-distributions.s3.amazonaws.com/gazebo/api/dev/classgazebo_1_1physics_1_1Entity.html
try InitialRelativePose, DirtyPose, RelativePose, WorldPose from <Collision.hh> Collision Class
The original code was DEPRECATED from the Physics Entity Class and "Reimplemented in Collision"
http://gazebosim.org/api/dev/classgazebo_1_1physics_1_1Entity.html#a736a629303b996ed3fd62a90ede62d0b
we are currently in ‘class gazebo::physics::Model’ class... and as such as using a physics entity.


cask: Delete package dir if it is symlink on the uninstall

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

When a Formula is converted into a Cask (e.g. osxfuse),
a symlink remains at the place of package dir.
This change ensure to remove such leftovers.

Makes sense to me. @jawshooah @reitermarkus?

👍 See https://github.com/Homebrew/homebrew-fuse/issues/86 for the symptom.

Strange, seems like this bit should take care of symlinks. Is the problem that these symlinks are listed as directories in the BOM?

@jawshooah Yes, symlinks which links to a directory is not handled.
This situation is caused when the package moved from Formula to Cask, and then install the Cask without uninstalling the Formula, and then uninstall the Cask.
Example
$ ls -ld /usr/local/include/osxfuse
lrwxr-xr-x  1 umireon  staff  26 Nov 24 02:06 /usr/local/include/osxfuse -> /usr/local/Cellar/osxfuse/x.x.x/include
$ brew cask install osxfuse
🍺  osxfuse was successfully installed!
$ ls /usr/local/include/osxfuse/
fuse	fuse.h
$ brew cask uninstall osxfuse
==> Running uninstall process for osxfuse; your password may be necessary
==> Unloading kernel extension com.github.osxfuse.filesystems.osxfusefs
==> Removing files from pkgutil Bill-of-Materials
==> rmdir: /usr/local/include/osxfuse: Not a directory
Error: Command failed to execute!

==> Failed command:
/usr/bin/sudo -E -- /bin/rmdir -- #<Pathname:/usr/local/include/osxfuse>

==> Standard Output of failed command:

==> Standard Error of failed command:
rmdir: /usr/local/include/osxfuse: Not a directory

==> Exit status of failed command:
#<Process::Status: pid 68354 exit 1>

Got it, thanks for the explanation.


Cypress workflow improvements

Description
This PR has 2 improvements:

Update glob to include subdirectories.  Test specs have been added under vis_augmenter subdirectory as part of this PR and are currently not running in this workflow.
Remove env vars to just fetch the same branch that the workflow is running from. For example, in main, this will fetch main branches of all the repos. This follows what is done in OSD repo. It also helps prevent conflicts and lessens the differences between main/2.x.

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

CI will fail to run 3.0 cluster. Confirmed on my local this runs all 11 test suites (8 existing + 3 new under vis_augmenter)
ohltyler@b0f1d86ddbb0 opensearch-dashboards-functional-test % yarn cypress:run-without-security --spec 'cypress/integration/plugins/anomaly-detection-dashboards-plugin/**/*'
yarn run v1.22.19
$ env TZ=America/Los_Angeles NO_COLOR=1 cypress run --headless --env SECURITY_ENABLED=false --spec 'cypress/integration/plugins/anomaly-detection-dashboards-plugin/**/*'
Couldn't determine Mocha version

====================================================================================================

  (Run Starting)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Cypress:        9.5.4                                                                          │
  │ Browser:        Electron 94 (headless)                                                         │
  │ Node Version:   v14.20.0 (/Users/ohltyler/.nvm/versions/node/v14.20.0/bin/node)                │
  │ Specs:          11 found (plugins/anomaly-detection-dashboards-plugin/create_detector_spec.js, │
  │                  plugins/anomaly-detection-dashboards-plugin/dashboard_spec.js, plugins/anomal │
  │                 y-detection-dashboards-plugin/detector_configuration_spec.js, plugins/anomaly- │
  │                 detection-dashboards-pl...)                                                    │
  │ Searched:       cypress/integration/plugins/anomaly-detection-dashboards-plugin/**/*           │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘

@amitgalitz can you look into the test failures? Looks related to associated detectors page.


Picking a date from DatePicker causes date to be shifted by one day backwards.

Repro. Steps:

Choose a day from DatePicker
Date is shifted by one day backwards.

Fixed in 4ecc882e10c4d6f9dd22a985fd4c9868467a0516

Fixed in 4ecc882e10c4d6f9dd22a985fd4c9868467a0516.


Issue with BindView annotation and IDs[core]

Checks before report
Verified Everything.
Describe error

Take a look at the above image. The BindView annotation have resource ID's integer value from R class instead of exact resource id like @BindView(R.id.myId) not like @BindView(2131....)
Is there any workaround for this?

The generated R classes usually have all resource IDs as public static final int fields. The Java compiler treats them as constants and replaces every usage of this constant by it's value to speed-up the class loading process.
Therefore the compiled class (and hence the decompiled class) only contains the value and not the reference to the field of the R class the value comes from.
The only chance would be to build a mapping table for all R entries and the guess for each number if it is a resource id or not.

@redmi111 thanks! Fixed, please check latest unstable build.

The same issue with the older butterknife library.

@redmi111 fixed, thanks!


Update Polarion ID into metris auto test case

Signed-off-by: quchangl-github<EMAIL_ADDRESS>used to fix the issue open-cluster-management/acm-qe#372

/assign @morvencao

@morvencao Thanks for your comments. We are requested to add Polarion ID into automation test case, you can see some other components auto cases have been added with it - https://github.com/open-cluster-management/canary/blob/2.3-integration/tests/2.3.2-DOWNSTREAM-2021-08-12-05-36-56/results.md, you can search keyword RHACM to find them.


Attempting to link Venice

For whatever reason, the library is using a venice dependency which is causing problems.  I forked and updated tag to latest commit on master which fixed it.
I can't really PR a tag, but if you updated, would be great.  The dependency appears to not be necessary and is not declared in the package.swift

Sorry, this was probably pushed too early in error, with preparation for connection pooling, which uses Venice: https://github.com/stepanhruda/PostgreSQL-Swift/pull/15
I'm hoping we'll be getting that merged soon (unfortunately Venice was crashing Xcode and I'd really like a functional IDE integration).

Ah, sorry again, I just figured out what's going on and fixed it. Because swiftpm can currently only point to tags and not branches, the highest tag is on a branch that's not merged yet.
The dependency was of course properly specified there (otherwise your swiftpm wouldn't know about it), but it probably collided with a different dependency, assuming you are using Epoch in your project.
Blow away the directory in Packages and swift build --clean && swift build, you should now get version 0.0.6 and link fine.


UsingRegistry should check that registry address is not zero in setRegistry

Expected Behavior
Please describe the behavior you are expecting
Current Behavior
What is the current behavior?

reserved for evgeni@limechain


Lexial Block dropdown hidden behind following field.

Link to reproduction

Describe the Bug
I have a Lexical RichText field that contains the BlocksFeature embedding one Block called NaviationActionBlock.
The NaviationActionBlock contains as last field a relationship field.
🐛 Now when this relationship field opens, its drop-down is hidden.
It gets partially hidden behind the first input field after the RichText field.
Screenshot

To Reproduce
Field setup for  RichText field
Setup of RichText field and following one that is hiding the Blocks dropdown
{
    name: 'text',
    label: 'Text',
    type: 'richText',
    editor: lexicalEditor({
      features: () => [
        HeadingFeature({
          enabledHeadingSizes: ['h1', 'h2', 'h3'],
        }),
        ParagraphFeature(),
        BoldTextFeature(),
        ItalicTextFeature(),
        StrikethroughTextFeature(),
        UnderlineTextFeature(),
        SubscriptTextFeature(),
        SuperscriptTextFeature(),
        LinkFeature({}),
        IndentFeature(),
        AlignFeature(),
        SsmlSubFeature(),,
        UploadFeature({
          collections: {
            'simulation-images': { fields: [] },
          },
        }),
        BlocksFeature({ blocks: [NaviationActionBlock] }),
      ],
    }),
    localized: true,
},
{
    name: 'voiceActor',
    label: 'Voice Actor',
    type: 'relationship',
    relationTo: VoiceActors.slug,
    required: true,
}

The last field of the NaviationActionBlock
export const NaviationActionBlock: Block = {
  slug: 'navigation',
  fields: [
...
 {
      name: 'text',
      label: 'Text',
      type: 'relationship',
      relationTo: SpeakableTexts.slug,
    },
  ],
};

Payload Version
"payload": "^2.1.1"
Adapters and Plugins
"@payloadcms/bundler-webpack": "^1.0.5",
"@payloadcms/db-mongodb": "^1.0.7",
"@payloadcms/plugin-cloud-storage": "^1.1.1",
"@payloadcms/plugin-sentry": "^0.0.6",
 "@payloadcms/richtext-lexical": "^0.1.17",

This has been fixed by #4113 and will be available in the next version :)


How we are planning to include the fixes made to m104_release into m114_release

I can see lot of commits to fix / overcome issues are available on branch m104_release but not on m114_release.
I would like to know about how we are planning to include them.

We have actually merged m104 and all previous patches into m114 through 8 big patches. Let me know if I missed anything
https://github.com/webrtc-sdk/webrtc/pull/76

Ok.
I see that all the changes are included in the patches but we missed #78.

Ah, sorry, I forgot this PR,  okay, let me cherry-pick to m114 first, the next upgrade to the m version will be merged into Android improvements

Ah, sorry, I forgot this PR, okay, let me cherry-pick to m114 first, the next upgrade to the m version will be merged into Android improvements

Okay, Thanks.

Ok, just merged into https://github.com/webrtc-sdk/webrtc/commit/61fc7d615ac87d67762a94b4c700aa4c33f79a84


doc(README) : remove $

Removing $ from script would make it runnable much easier

Fixes #4
Checklist

[x] Review Contribution Guidelines.
[x] Make sure all contributed code can be distributed under the terms of the
Apache License 2.0, e.g. the code was written by
you or the original code is licensed under a license compatible to Apache License 2.0.

Thanks! However I'll decline for the moment as the README is not even close to what I'd like to see once the 1st release is consumable. Everything is in flux at the moment. Stay tuned.

Stay tuned.

Sure :sunglasses:


Some of the language_2/async_star_test tests are failing in --strong kernel mode after a recent change to the type inferencing code

The following tests have started getting an assert failure in kernel mode after the recent change to the type inference code in the FE https://dart-review.googlesource.com/c/sdk/+/29744
language_2/async_star_test/none
language_2/async_star_test/01
language_2/async_star_test/02
language_2/async_star_test/03
language_2/async_star_test/04
language_2/async_star_test/05
The backtrace from the failures is
../../runtime/vm/object.cc: 15721: error: expected: !other.IsDynamicType()
Dumping native stack trace for thread 6544
[0x0000000000ae4847] dart::Profiler::DumpStackTrace(bool)
[0x0000000000ae4847] dart::Profiler::DumpStackTrace(bool)
[0x0000000000db8126] dart::DynamicAssertionHelper::Fail(char const*, ...)
[0x0000000000a2d85c] dart::Instance::IsInstanceOf(dart::AbstractType const&, dart::TypeArguments const&, dart::TypeArguments const&, dart::Error*) const
[0x0000000000a2e908] dart::Instance::IsFutureOrInstanceOf(dart::Zone*, dart::AbstractType const&, dart::Error*) const
[0x0000000000a2dae4] dart::Instance::IsInstanceOf(dart::AbstractType const&, dart::TypeArguments const&, dart::TypeArguments const&, dart::Error*) const
[0x0000000000b5e463] dart::DRT_TypeCheck(dart::NativeArguments)

cc @stereotype441

A status file update to account for this was made in https://dart-review.googlesource.com/c/sdk/+/30060


Revert "Update Python worker imports to use proper paths (#39)"

This reverts commit e13f4a2f75ad00ac173335a04c1acd611e722587.

Opened an issue in the python worker
https://github.com/Azure/azure-functions-python-worker/issues/549

Looks like these import statements apply to all languages. @maiqbal11 we need to figure out another way to handle case for python.


Fix animation race condition on Safari

Fixes #192 :
On Safari, there is a race condition, in which the @keyframes are added to the <StyleSheet> on the page after the animationName style property is added to the styled element. Safari doesn't find the animation, and doesn't refresh those elements when the animation is added to the stylesheet.
Looking at the code, the update of the global stylesheet (src/components/style-sheet.js:43) is deferred by 0 milliseconds, while the addition of the style rule to the element is synchronous. This would cause the race condition described above.
This PR removes the debouncing from the CSS change listener, but leaves the debounce in componentDidMount intact.

Coverage remained the same at 94.616% when pulling a2e30985e4a3ace9c3e2d6f609181a8ce53d040d on apiv:master into da33b8f13cadda9b421c0a6c567fca7361c4e924 on FormidableLabs:master.

Coverage remained the same at 94.616% when pulling a2e30985e4a3ace9c3e2d6f609181a8ce53d040d on apiv:master into da33b8f13cadda9b421c0a6c567fca7361c4e924 on FormidableLabs:master.

Coverage remained the same at 94.616% when pulling a2e30985e4a3ace9c3e2d6f609181a8ce53d040d on apiv:master into da33b8f13cadda9b421c0a6c567fca7361c4e924 on FormidableLabs:master.

Closing because the same problem is solved by #989


add extension support

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
---we need extension support , extension manager could build like microsoft store , like the utools solution (utools:linkdeveloped by Chinese Developers )
If you'd like to see this feature implemented, add a 👍 reaction to this post.

@songtao1873
For which utility do you want to have extension support.

Just like Chrome extension plugins, translation, dictionary and development language document retrieval, all kinds of interesting extension plugins.

@songtao1873
For which utility do you want to have extension support.

Just like Chrome extension plugins, translation, dictionary and development language document retrieval, all kinds of interesting extension plugins.

@songtao1873 we have about 10 utilities, are you talking about plugin support for PT Run?

@songtao1873 we have about 10 utilities, are you talking about plugin support for PT Run?
no, I means that support third-party extension or add-ons support, thanks a lot.

@songtao1873 Sorry, I'm not following here what you're trying to explain.  Can you give a bit more concrete scenario here?  Which utilities do you want that add-on support.  The most clear example would have been PowerToys Run but you just said no to that.

@songtao1873 Sorry, I'm not following here what you're trying to explain. Can you give a bit more concrete scenario here? Which utilities do you want that add-on support. The most clear example would have been PowerToys Run but you just said no to that.

This is the case. I hope that powertoys will add some extensibility support, such as opening up some interfaces, allowing third-party developers to develop some interesting plug-ins for powertoys (such as javascript plug-ins) to make up for the lack of powertoys functions, and upload them to the cloud for Users who need it can download and use.

For PowerToys run, correct?  Or just utilities in general?  If in general, just follow the contribution guidelines.

For PowerToys run, correct? Or just utilities in general? If in general, just follow the contribution guidelines.

That is to say, there is no need to merge into the master source code of powertoys. If there is a plan to expand the store in the future, you only need to submit the extension plug-in developed by yourself to the extension store. No need to consider powertoy's source code tree. If this is the case, powertoys is only a platform for functional realization, working at a relatively low level, and extension plug-ins work on powertoys.

What is the benefit then versus just releasing your own app or to the store? We are a suite of apps that have a centralized runner and settings

What is the benefit then versus just releasing your own app or to the store? We are a suite of apps that have a centralized runner and settings


Add support for LISTEN.moe

Right now I can play audio from https://listen.moe/opus as expected, but no audio track info is shown.
This can be implemented through their websocket thing(tm): https://docs.listen.moe/ws/spec.html#receiving-data

If you want you can use a library i made for that. Its really simple to use


PECL install fails when extending the runtime

I get the following error when trying to run pecl install:
Step 3/4 : RUN pecl install pthreads
 ---> Running in 77064be783ca
// ...
Cannot find autoconf. Please check your autoconf installation and the
$PHP_AUTOCONF environment variable. Then, rerun this script.

Here is my Dockerfile:
FROM gcr.io/google_appengine/php

# Custom PECL installation
RUN pecl install pthreads

It's fixable if I add the following right before it:
RUN apt-get install -y autoconf build-essential

Is there a way users can quickly install unsupported extensions and specific versions of supported extensions?

We don't use --enable-maintainer-zts for compile PHP, which means you can not use pthreads extension anyways.
To answer the question, you're right. Now you have to have a custom Dockerfile to install extensions which we don't offer. We'd like to support custom build step in the near future though.
FWIW, we started to compile almost all stable extensions (see the list at https://cloud.google.com/appengine/docs/flexible/php/runtime), so I hope the issue is somewhat mitigated.

Yes, I was just using pthreads to test PECL. I eventually hit the ZTS issue, and that's ok because that's specific to pthreads. I am assuming at this point, other compatible extensions and extension versions will work.


New provider: City of Austin Utilities

Hello!
Currently working on adding a new provider,  City of Austin Utilities (https://github.com/max2697/opower/pull/1). It's also working on opower but I found some differences from this repos code.
First of all it's failing when it try to call _async_get_customers. This endpoint just redirecting to some html page via 302 code.
I made some research and found different endpoint with the same data:
https://dss-coa.opower.com/webcenter/edge/apis/multi-account-v1/cws/coa/customers
Some differences:

dss-{utilityCode} as a subdomain, {utilityCode} as a part of URI
/webcenter/ root instead of /ei/
This API required not only Bearer token but also header opower-selected-entities: ["urn:session:account:REDACTED"]

Found all 4 endpoints in my version of portal:
       /ei/edge/apis/DataBrowser-v1/cws/cost/utilityAccount/...
/webcenter/edge/apis/DataBrowser-v1/cws/cost/utilityAccount/...

       /ei/edge/apis/DataBrowser-v1/cws/utilities/{subdomain}/utilityAccounts/.../reads
/webcenter/edge/apis/DataBrowser-v1/cws/utilities/{utilityCode}/utilityAccounts/.../reads

       /ei/edge/apis/bill-forecast-cws-v1/cws/{subdomain}/customers/.../combined-forecast
/webcenter/edge/apis/bill-forecast-cws-v1/cws/{utilityCode}/customers/.../combined-forecast

       /ei/edge/apis/multi-account-v1/cws/{subdomain}/customers
/webcenter/edge/apis/multi-account-v1/cws/{utilityCode}/customers

So basically  City of Austin Utilities is using different version of a Digital Self Servicedescribe on this page: https://docs.oracle.com/en/industries/energy-water/digital-self-service/transactions-seamless/Content/Transactions/SeamlessIntegration/Introduction_Seamless.htm
Have anybody heard about this different version of Opower web interface?
What's the address of home page for Opower for other providers? Mine is https://dss-coa.opower.com/dss/overview

Well, I found an endpoint to get data about user accounts (not utilities accounts) and changed the code to allow it work with DSS. It's working now so I'll think about better way to integrate both portal version into one solution and will send a PR for this.


OR operator in query

We have AND in SB's query language. However there is no OR.  Sometimes you want to write OR queries such as follow:
<!-- #query task where deadline <= '{{today}}' OR tags=["p0", "p1"] where done=false render [[template/task]] -->

This lists tasks which needs urgent attention: with high priority or with today's deadline.
The closest thing to OR we have right now is overloaded equality check when rhs is a list (as in the example above). The semantics of tags=["p0", "p1"] that it's equivalent to tags="p0" OR tags="p1"
Note that multiple where clauses are allowed, but they still AND the results

This is now implemented


"Show text" node doesn't saved with metadata anymore

It seems the text grabed by the "show text node" isn't save in metadata of the image anymore. When we load an image with a show text node in the network, there is no text inside.

Should be fixed


Allow to filter workloads by namespace

What kind of request is this (question/bug/enhancement/feature request):
feature request
Steps to reproduce (least amount of steps as possible):
go to workload list of a project and try to search by namespace
Result:
no result found
Other details that may be helpful:
When you have lots of workloads in a project it's impossible to find something without scroll all the page or use the CTRL-F / CMD-F to find something.
if you have more than 50 workloads you have also to switch between pages.
It should be very helpful if the search box can work also with the namespace name.
Also please add an option to see the namespace name in the "node view" of the workload.
if you have multiple workspaces with a service named "web" and you try the node view you cannot understand every workload what it is.
Environment information
Rancher HA, v 2.2.2

This would be very useful also for situations like these:

Otherwise it's impossibile to choose the correct option.

the original issue has been fixed in master (2.3).
For the Alias of another DNS records's issue will be addressed in https://github.com/rancher/rancher/issues/20484


downlit and xml2 packages

@cansavvy
When running GHA for Quarto-based OTTR, I keep on seeing this warning message: The downlit and xml2 packages are required for code linking, here and here. To fix this, do I need to add these packages on this line?

@cansavvy
When running GHA for Quarto-based OTTR, I keep on seeing this warning message: The downlit and xml2 packages are required for code linking, here and here. To fix this, do I need to add these packages on this line?

Yes that seems like a good first bet.
If you open a pull request with that add the gha's in this repo will test that build for you.


About SAC implementation

Hi, in your implementation, SAC is used but V is estimated by Q-function when updating critic and calculating target Q, instead of a separated value network in the original SAC paper. Would you please explain it or give some references? Thanks

Hi,
Since we're using discrete action spaces our Q-function outputs a value for each possible action. As such, we can marginalize Q in order to get V instead of estimating V separately.


fix: force import protobufjs/ext/descriptor package

We are using the protobufjs/ext/descriptor extension to be able to dynamically load/export protobuf definitions, but this never played nicely with the Typescript compiler and by importing it from google-gax makes it even harder to monkey patch some methods.
Same issue was reported on the protobufjs repo a while ago a seems like there is solution on fixing for that in the short term.

https://github.com/protobufjs/protobuf.js/issues/1499
https://github.com/protobufjs/protobuf.js/issues/1149

This PR changes things so we import protobufjs directly, add a TS file to manually declare the monkey patched methods and add protobufjs as a peer dependency.

Merging with system-check test bypassed.


[CORE] Add fallback summary for subqueries

What changes were proposed in this pull request?
Currently, the node fallback reasons for subqueries are missing from the Fallback Summary, this pr will supplement it.
before:

atfer

How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

cc @ulysses-you could you please take a look?

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_4228_time.csv
log/native_master_01_14_2024_00db209eb_time.csv
difference
percentage

q1
35.48
33.95
-1.531
95.68%

q2
24.51
24.88
0.366
101.50%

q3
39.16
38.01
-1.148
97.07%

q4
36.00
38.25
2.246
106.24%

q5
69.53
71.46
1.930
102.78%

q6
7.28
7.30
0.019
100.26%

q7
84.83
85.49
0.662
100.78%

q8
86.29
85.37
-0.919
98.94%

q9
126.20
122.58
-3.621
97.13%

q10
46.34
44.98
-1.361
97.06%

q11
21.54
19.89
-1.653
92.33%

q12
26.34
28.31
1.971
107.48%

q13
47.12
47.25
0.134
100.28%

q14
18.94
19.45
0.513
102.71%

q15
30.18
29.55
-0.627
97.92%

q16
12.63
14.42
1.791
114.18%

q17
102.42
102.43
0.006
100.01%

q18
148.12
148.49
0.370
100.25%

q19
13.62
13.56
-0.063
99.54%

q20
27.27
27.19
-0.081
99.70%

q21
226.84
228.47
1.632
100.72%

q22
14.83
13.86
-0.978
93.41%

total
1245.48
1245.14
-0.340
99.97%


Add community SDKs section

What does this PR do?
Add community SDKs section

How should this be manually tested?

 Snippets tests reports   SDK Version Tests overview      v6        v5                           :heavy_check_mark:                            v1        v1        v1        v1


docs: Add conda-forge install instructions

Resolves #40

Add install instructions from conda-forge for pixi and conda.

c.f. https://github.com/conda-forge/pyknos-feedstock

Add conda-forge badge.

@janfb this is ready for review.


removing a user's messageReaction seems to result in error.

my code is literally
client.on('messageReactionAdd', (messageReaction, user) => {     messageReaction.remove(user).catch(console.error); });
always results in an error '{"user_id": ["Value \\"undefined\\" is not snowflake."]}'

fetchUsers(limit = 100) {     const message = this.message;     return message.client.rest.methods.getMessageReactionUsers(       message, this.emoji.identifier, limit     ).then(...);   }
also returns this.emoji.identifier

Made a rash assumption, its rather the user id that is undefined....
so the remove() method in src/structures/MessageReaction.js
remove(user = this.message.client.user) {     const message = this.message;     user = this.message.client.resolver.resolveUserID(user);     if (!user) return Promise.reject('Couldn\'t resolve the user ID to remove from the reaction.');     return message.client.rest.methods.removeMessageReaction(       message, this.emoji.identifier, user     );   }
resolves user as an ID, and it then sends that user, which is an ID to the removeMessageReaction method. (src/client/rest/RESTMethods.js )
However, the removeMessageReaction method assumes that user is an user object, so it tries to get user.id, but that is undefined in this case....

I still have this issue. Version `11.0.0``.
msg.react(getEmoji('ear')).then(reaction => {
      const youtubeStream = ytdl(songUrl);
      const dispatcher = connection.playStream(youtubeStream, { volume: 0.5 });
      this.set('currentSong', dispatcher);

      dispatcher.on('error', err => {
        console.log(err);
        reaction.remove().then(() => resolve());
      });

      dispatcher.once('end', () => {
        reaction.remove().then(() => resolve()).catch(err => console.log(err));
      });
    }).catch(err => console.log(err));

This bugfix has not been deployed to the stable branch yet, because for some reason we haven't been releasing patch versions... cough
If you want the fix, you'll have to npm i hydrabolt/discord.js

Thanks @Programmix!

Could you explain further why this hasn't been fixed? It seems weird to me that there's a bug in a necessary feature which has been fixed, but it hasn't been deployed yet.

It's because the master branch has lots of changes worthy of a minor patch. Also, the developers can't agree on a release date for v11.1.0, which is necessary to get this patch out...

Thank you so much @Programmix! I don't have time to sit around waiting for the devs to figure out a release date. That did the trick and the bugs are no longer present. 👍


Bug

Does it happen to have a bug with buying items? with me I don't get any items in my inventory....

Actually i tested now, and it's working for me, remember that the second parameter of the config items need to be the same as sql item

Does it happen to have a bug with buying items? with me I don't get any items in my inventory....

Maybe that item is not existing in your sql database.


Error removing AzureML workspace service

Describe the bug
I wanted to remove the AzureML workspace service from a newly created workspace. Disabling the service already fails with the following error message:
Error message: Error: No value for required variable on variables.tf line 41: variable "azure_environment"
The root module input variable "azure_environment" is not set, and has no default value.
Use a -var or -var-file command line argument to provide a value for this variable.

Steps to reproduce

Fresh deployment of AzureTRE
Create the default workspace bundle following the documentation
Create a new workspace based on the default bundle
Create the AzureML bundle with make workspace_service_bundle BUNDLE=azureml
Deploy the AzureML workspace service in the new workspace with the default settings
Disable the AzureML service.

Another option is change the TRE version to main as per https://microsoft.github.io/AzureTRE/v0.11.0/tre-admins/upgrading-tre/#deploying-a-specific-version-of-azure-tre to:
"OSS_VERSION": "main"

@marrobi thanks for the quick follow up!


feat: added display settings for windowed mode

What does this PR change?
Fixes https://github.com/decentraland/unity-renderer/issues/411
...
How to test the changes?
TBD
Our Code Review Standards
https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

@kuruk-mm you have all available resolutions in the dropdown menu, its funny because the first thing I do is to scroll down but the scroll bar is disabled, maybe we should enable it.
Should full screen force the maximum resolution available?


What date to next release ?

I send pull request (is marged) fixing many bugs and this not go to actual release, what date to next release ?

There will still be lots of people using Foundation 5 even after Foundation 6 launches because they already built or started building on 5. So any PR's for 5 will still be valuable for lots of people.
Foundation for Sites 6 is a new codebase with many updates. We're doing a private release now and will announce the release date soon.


best way to make a 2d array immutable?

help me

Immutable.fromJS([[0, 1], [1, 0]]) will give you a List of Lists.

var a = Immutable.fromJS([[0, 1], [1, 0]]);
console.log(a.toArray());

Doesn't give you what it's supposed to! It prints the internals of the Immutable List object instead...

toArray will not alter the items in the list, in this case more Immutable List, since this is a 2D List. Use toJS() to deeply convert Lists to Arrays. Otherwise what you are seeing is fully expected

How can I do deduplication?
fromJS([[1,1],[1,1]]) to  List {size:1} List.toJS() === [[1,1]]


Remove list from EngineConfig.producers

From:
new Engine({
  producers: {
    list: ProducerConfig[]
  }
})

to:
new Engine({
  producers: ProducerConfig[]
})

Initially this was thought to give more control over the configuration/run-time setting for the producers. But so far no use case was ever found, and the configuration object can host any producer config that would be needed. Also the ProducerConfig can have an individual settings property if needed.

Solved by Engine Modules


Restore full state on termination of Windows process

Request from newsgroups to reset full console state on exiting on Windows.

As of v1.5.0, asciimatics uses a separate console buffer so that it can restore the original on termination.


Onefile finding files documentation suggestion

For this part of the documentation:
https://nuitka.net/doc/user-manual.html#use-case-4-program-distribution
Can we make it include a warning about how we can make onefile properly access files? Preferably referencing this:
https://nuitka.net/doc/user-manual.html#onefile-finding-files
It took me quite a bit of time to actually learn what was causing this problem, and about to submit an issue- only to find the answer further down the docs under a different section!

Yes, you are correct, going from standalone onefile is a no-brainer except for this one thing, and it should be pointed out.

I added that reference, and also updated a few information pieces there, it is much less Windows specific for many things now. I am closing the issue early due to lack of general relevance. You can see in on branch factory.
Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback if this is working is very welcome, just please do not share plans of doing it, but rather confirmations or denials of it working.

Now on the develop branch, will be in 1.3 shortly

This is part of the stable release 1.3 that I just made.


when first  exec invalidateQueries is not work with  post method

Describe the bug
// refresh page , gameId  is '' , send request
// changed gameId, nothing happend ❌
useDidUpdate(() => {
console.log(gameId)// // here is correct  ✅
    queryClient.invalidateQueries({
      predicate: (query) => {

        return ['/report_data'].some((str) =>
          query.queryKey[0]?.startsWith?.(str)
        )
      },
    })
  }, [gameId, timezone, attribution])

const {

  } = useQuery({
    queryKey: ['/report_data/table', fetchDataOptions],
    queryFn: () => axios.post('/report_data/table', { ...fetchDataOptions }),
    keepPreviousData: true,
    enabled: gridReady && !isEmpty(fetchDataOptions.data_table_options.columns),
  })

Your minimal, reproducible example
https://codesandbox.io/s/new
Steps to reproduce
for invalidateQueries , The post request will not be re-executed
Expected behavior
exec report_data/table request, even if it ie post method
How often does this bug happen?
None
Screenshots or Videos
No response
Platform
macos chrome
Tanstack Query adapter
None
TanStack Query version
4.29.3
TypeScript version
No response
Additional context
No response

I don't understand the question / problem. Please provide a reproduction. I hope the link to https://codesandbox.io/s/new is just a mistake :)

replace  queryFn: () => axios.post() to  queryFn: () => axios.get()
I write a demo to reproduce and I tell you the steps to reproduce. The difference is only in who writes demo. You must have a lot of templates over there, which can be used as soon as possible. It's much easier than me.
Besides, the cost I paid was to find this bug and make it come back. Isn't that enough?

this simple to reproduce it   if you  try and actually  not reproduce  i can  provide  demo .do you  agree?

the deal is always that if you want a maintainer to look into an issue that you have, you write the demo, because you know what the bug is.
if you want to start from a template, look at: https://tanstack.com/query/v4/docs/react/examples/react/simple

feel free to re-open with a reproduction


Maximum limit of Role Claims

I have used Role Claims for authorize our system, but, problem is role claim table growing rapidly and sometimes there are more than 200 claims for some roles. and it has given error (HTTP Error 400. The size of the request headers is too long.) when a user going to login the site. HOW CAN I FIX THIS ????

That's a really really big number of roles. So aside from reducing those you could not store them in the cookie, and have your policies go off to your role store instead when authorization happens., Would that work for you?

HOW CAN I FIX THIS

Perhaps with a separation of concerns: http://policyserver.io/. Consider the identity system separate from the authorization system. The video in the link above elaborates on this design.

Closing as the only answers that work are given. Please reopen if you need further help.


⚠ Processors has degraded performance

In d6ef334, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 85 ms

Resolved: Processors performance has improved in 73cda03.


fix TestKubeOnly flake

fixes: #20029

@hzxuzhonghu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
e3eba214d6b35a06cd6a638580c15105583c6386
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Fixed namespace checker to use the correct name

ISSUE TYPE

Bug fix Pull Request

SUMMARY
Apparently the wrong member was used to check the resource name. I don't
know how resillient this patch is, but it works for most of the part.

Fixes #419

Done, but this doesn't work in case of a delete event.

Done, but this doesn't work in case of a delete event.

@hellozee do you mind adding the e2e test for the namespace checker filter (https://github.com/infracloudio/botkube/blob/develop/test/e2e/filters/filters.go#L44)? I am fine if we address that as a separate task.
Please file an issue if you can't add that in this PR.
Thank you!

@hellozee do you mind adding the e2e test for the namespace checker filter (https://github.com/infracloudio/botkube/blob/develop/test/e2e/filters/filters.go#L44)? I am fine if we address that as a separate task.
Please file an issue if you can't add that in this PR.
Thank you!

It would be better if someone else picks it. I cant promise anything this week, sorry. If no one picks it up this week, I would put up a patch next week, 🐱

It would be better if someone else picks it. I cant promise anything this week, sorry. If no one picks it up this week, I would put up a patch next week, 🐱

Sounds good @hellozee

Sounds good @hellozee

Filed https://github.com/infracloudio/botkube/issues/443 to track. Merging this PR

Filed https://github.com/infracloudio/botkube/issues/443 to track. Merging this PR

@hellozee this doesn't seem to be working. I've added a fix https://github.com/infracloudio/botkube/pull/449. PTAL

@hellozee this doesn't seem to be working. I've added a fix https://github.com/infracloudio/botkube/pull/449. PTAL


lake build --require-silence

Many libraries require via CI that there is no output (e.g. to detect stray #check, or worse). Currently this is done by various adhoc methods piping output into fragile shell scripts.
I'd like a lake build --require-silence that errors if any build job writes to stdout or stderr.
I'm happy if this is a synonym for something in a more generic family (e.g. like the existing --wfail, turning warnings into failures), but I would like if the name is indicates the user intent (i.e. not --ifail).

The name is a bit odd, it wasn't clear to me if it might mean “seriously try hard to hide output”. --error-on-output or --check-no-output maybe? 🚲🏚

Oh, I did not notice you had written this issue. I (sadly?) already addressed this in #4127 with the name --iofail. What do you think of that name? It is a bit clearer than --ifail, but maybe not enough?

It'll do!


random bug

General information:
Project issue is for:
3.8.0
Expected result:
not sure i was eating food just sitting on main menu and game throws random error so took a screenshot for someone to look at sorry i cant tell you more i just dont know what happened
also got a random blank message from ragman could be related
How to reproduce:

not sure

Attachments

2024-01-09[15-58] (0).png (4019 KiB)
2024-01-09[15-59] (0).png (4034 KiB)
server-2024-01-09.log (550 KiB)
LogOutput.log (53 KiB)
2024.01.09_15-20-25_<IP_ADDRESS>.28375 traces.log (96 KiB)

Originally written by hughz001

okie just noticed my daily quest have just reset so i think its highly likely to do with that so repo just wait for dailys to refresh and be on the game when they do

Originally written by hughz001

The empty message is related to another issue: https://dev.sp-tarkov.com/SPT-AKI/Issues/issues/261.
There was a change in: https://dev.sp-tarkov.com/SPT-AKI/Server/commit/0b3ad5bbd8d6d8351beb6d452d09dca6f056a77c to make the mails sent on quest failure only send for non-repeatable quests.
Not sure why you got a message for a daily, do you know if you had the above fix from 2 days ago?

Originally written by chomp

100% iv being updating all commit so that i dont do that so not sure there

Originally written by hughz001

Should be resolved with: https://dev.sp-tarkov.com/SPT-AKI/Server/commit/443d8da7f79a7bc54e36c80d7ad5b0ed83f7a15b

Originally written by chomp


feat/ massmint-mobile-disclaimer

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Feature

Needs Design check

@exezbcz please review

Context

[x] Closes #8012

Did your issue had any of the "$" label on it?
yes $

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

Copilot Summary

🤖[deprecated] Generated by Copilot at e6f5e08
Added a new modal component and color variable to warn mobile users about the mass minting feature. The modal shows a warning-yellow message and two buttons to either leave or continue the feature. The modal and its translations are imported from the components/massmint/modals folder and the locales/en.json file respectively.

🤖[deprecated] Generated by Copilot at e6f5e08

To warn mobile users of mass minting
A modal was added with hinting
It uses NeoModal
And warning-yellow
And emits events for back and quitting

Thanks!
pay 50 usd

@yangwao
Bot didn't execute payment


Updated remrem-shared version to 0.3.2

Implemented base64 encryption functionality for ldap manager password in remrem-shared

added rootDn property in ldap configurations

variable name changed from ldapPassword to managerPassword

Coverage remained the same at 75.41% when pulling 609e0aa858ecc4e17806ad62d98b97866c2c1815 on xdurvak:base64 into 67df4949b35172e8bad9fb7a31a5a09fb96ed726 on Ericsson:master.

Coverage remained the same at 75.41% when pulling 716c92f2efd18e208161d73d9c08c3c2ec05342a on xdurvak:base64 into 67df4949b35172e8bad9fb7a31a5a09fb96ed726 on Ericsson:master.


cleanup hierarchies dependant on deleted element

this is an attemp to fix issue #275

i don't understand why the build is failing

everything seems resolved now

@seuros I'll leave this to you. @kazalt thanks for the PR.


CHEMBL nodes - categories value and attribute "biolink:category" don't agree on some nodes - attribute value missing on others

Looks like the 'categories' value of all CHEMBL.TARGET nodes were set to "biolink:ChemicalEntity"
For some "biolink:ChemicalEntity" is not in the "biolink:category" attribute.
Example
CHEMBL.TARGET:CHEMBL2023
{
"categories": [
"biolink:ChemicalEntity"
],
"name": "Catechol O-methyltransferase",
"attributes": [
{
"attribute_type_id": "biolink:publications",
"value": [
"PMID:762061",
"PMID:13575440",
"PMID:438821"
],
"value_type_id": "biolink:Uriorcurie",
"original_attribute_name": null,
"value_url": null,
"attribute_source": null,
"description": null,
"attributes": null
},
{
"attribute_type_id": "biolink:category",
"value": [
"biolink:Polypeptide",
"biolink:Protein",
"biolink:MolecularEntity"
],
"value_type_id": "metatype:Uriorcurie",
"original_attribute_name": null,
"value_url": null,
"attribute_source": null,
"description": "Categories of all nodes in this synonym set in RTX-KG2.",
"attributes": null
},
{
"attribute_type_id": "biolink:synonym",
"value": [
"Catechol O-Methyltransferase",
"catechol O-methyltransferase",
"Catechol O-methyltransferase"
],
"value_type_id": "metatype:String",
"original_attribute_name": null,
"value_url": null,
"attribute_source": null,
"description": "Names of all nodes in this synonym set in RTX-KG2.",
"attributes": null
},
{
"attribute_type_id": "biolink:IriType",
"value": "https://identifiers.org/chembl.target:CHEMBL2023",
"value_type_id": "metatype:Uri",
"original_attribute_name": null,
"value_url": "https://identifiers.org/chembl.target:CHEMBL2023",
"attribute_source": null,
"description": null,
"attributes": null
},
{
"attribute_type_id": "biolink:description",
"value": "Catechol O-methyltransferase (271 aa, ~30 kDa) is encoded by the human COMT gene. This protein is involved in the metabolism of neurotransmitters.; UMLS Semantic Type: STY:T126; UMLS Semantic Type: STY:T116",
"value_type_id": "metatype:String",
"original_attribute_name": null,
"value_url": null,
"attribute_source": null,
"description": null,
"attributes": null
},
{
"attribute_type_id": "biolink:has_count",
"value": 0,
"value_type_id": "EDAM:data_0006",
"original_attribute_name": "omnicorp_article_count",
"value_url": null,
"attribute_source": null,
"description": null,
"attributes": null
},
{
"attribute_type_id": "biolink:xref",
"value": [
"KEGG.ENZYME:<IP_ADDRESS>",
"NCIT:C16387",
"PathWhiz.ProteinComplex:10123",
"CHEMBL.TARGET:CHEMBL2372",
"UMLS:C0007407",
"PathWhiz.ProteinComplex:9192",
"MESH:D002394",
"PathWhiz.ProteinComplex:9898",
"PR:000005731",
"PathWhiz.ProteinComplex:8538",
"CHEMBL.TARGET:CHEMBL2176837",
"PathWhiz.ProteinComplex:1075",
"CHEMBL.TARGET:CHEMBL2023",
"PathWhiz.ProteinComplex:7324",
"CHEMBL.TARGET:CHEMBL3286068"
],
"value_type_id": "metatype:Nodeidentifier",
"original_attribute_name": null,
"value_url": null,
"attribute_source": null,
"description": "Identifiers of all nodes in this synonym set in RTX-KG2.",
"attributes": null
}
]
}
Example of missing biolink:categories attribute:
CHEMBL.COMPOUND:CHEMBL4650397
{
"categories": [
"biolink:ChemicalEntity"
],
"name": "GV-971",
"attributes": [
{
"attribute_type_id": "biolink:xref",
"value": [
"CHEMBL.COMPOUND:CHEMBL4650397"
],
"value_type_id": null,
"original_attribute_name": null,
"value_url": null,
"attribute_source": null,
"description": null,
"attributes": null
},
{
"attribute_type_id": "biolink:has_count",
"value": 0,
"value_type_id": "EDAM:data_0006",
"original_attribute_name": "omnicorp_article_count",
"value_url": null,
"attribute_source": null,
"description": null,
"attributes": null
},
{
"attribute_type_id": "biolink:synonym",
"value": [
"GV-971"
],
"value_type_id": null,
"original_attribute_name": null,
"value_url": null,
"attribute_source": null,
"description": null,
"attributes": null
}
]
}

Can you post the query(ies) used to produce the above nodes? I suspect that the query asked for biolink:ChemicalEntity; our system currently overrides Node.categories with the specific category(ies) the query asked for. We've considered changing this behavior (RTXteam/RTX#1360), but I believe we decided to wait until there was consensus from the consortium on what the ideal behavior here is.


Composes broken after v2.5.4

I just updated to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Upon initial startup, I get approximately 60 lines of this while CSS is being mangled:
[TypeError: Cannot read property 'toString' of undefined]
[TypeError: Cannot read property 'toString' of undefined]
[TypeError: Cannot read property 'toString' of undefined]
...

Also, most of my CSS styles are missing. Weirdly, some are not. After initial startup, the error messages are gone, until I run meteor reset.
The project used to work fine with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Further study showed that the class names get properly generated and even the styles defined in CSS files work, but composes is broken. It doesn't matter if I use relative or absolute paths.

I did some more research and pinpointed the issue to have emerged at v2.5.4. Before that it seems to work fine, also with Meteor 1.5-rc.6.

Thanks, I'll take a look ASAP.

I've been checking this in my test app, but can't reproduce it so far. I'll try it with the sample app tomorrow.
Could you post the contents of your package.json?
Thanks.

Sure! Here are the relevant bits:
"devDependencies": {
    ...
    "autoprefixer": "^6.5.3",
    "node-sass": "^4.5.3",
    "postcss-color-function": "^3.0.0",
    "postcss-modules-extract-imports": "1.0.0",
    "postcss-modules-local-by-default": "1.0.0",
    "postcss-modules-scope": "1.0.0",
    "postcss-modules-values": "1.1.1",
    "postcss-nested": "1.0.0",
    "postcss-simple-vars": "1.1.0",
    "postcss-utilities": "^0.6.1"
  },
  "cssModules": {
    "ignorePaths": [
      "node_modules",
      ".meteor",
      "server",
      "lib",
      "packages"
    ],
    "globalVariables": [
      "client/styles/global/fonts.m.css",
      "client/styles/global/colors.m.css",
      "client/styles/global/variables.m.css"
    ],
    "extensions": [
      "m.css"
    ],
    "enableSassCompilation": [
      "m.css"
    ],
    "postcssPlugins": {
      "postcss-simple-vars": {
        "fileOptions": [
          "client/styles/global/variables.m.css"
        ]
      },
      "postcss-modules-values": {},
      "postcss-nested": {},
      "postcss-utilities": {},
      "postcss-color-function": {},
      "postcss-modules-local-by-default": {},
      "postcss-modules-extract-imports": {},
      "postcss-modules-scope": {}
    }
  },
  "postcss": {
    "plugins": {
      "autoprefixer": {
        ...
      }
    }
  }
...

And a sample of a CSS definition that failed:
// file 1
.header {
  composes: pageHeader from '../../../styles/shared/typography.m.css';
}

// file 2
.base {
  font-family: $headerFont;
  font-weight: 100;
}

.pageHeader {
  composes: base;
  text-align: center;
  font-size: 3rem;
}

This was fixed in v2.7.2. It only occurred if the file being composed hadn't already been processed on it's own (so it was file load order dependent).

Thanks for the fix!


Move feedback form from My Mentor to one of the other pages.

From the user testing conducted this week, users said it is more appropriate to integrate the feedback form to one of the other pages. Also see issue #27.

Where?

Nikil has add the mentor details on top of the rating form. I guess this issue can be closed. What does anyone think?

Maybe make a dropdown in My Mentor which contains the details about the mentor and the feedback form