frser-sqlite-datasource v3.5.0 either does not exist or is not supported on your system linux-amd64

Hello Sergei,
when installing frser-sqlite-datasource v3.5 I get this error message.
Error: frser-sqlite-datasource v3.5.0 either does not exist or is not supported on your system linux-amd64
What am I doing wrong?
frser-sqlite-datasource Version: 3.5.0

Run via docker: yes
Browser: chrome
Grafana Version: 10.4.0
Plugin Version: 3.5.0
Plugin installed via grafana-cli or manually: grafana-cli

Kind regards sensej

Can you provide the exact command you are using?
Can you also check if the same thing happens for v3.4.0?

Command: grafana-cli plugins install frser-sqlite-datasource 3.5.0
this command works (grafana-cli plugins install frser-sqlite-datasource 3.4.0)

It is normal that this command fails.
Version 3.5.0 is not yet released on the Grafana webpage.
You can install version 3.5.0 but you need to follow the instructions here: https://github.com/fr-ser/grafana-sqlite-datasource/releases/tag/v3.5.0

ok, thank you for the info.

Hello Sergei,
Version 3.5.0 is now running for me.
Can you please show some examples of using the query variable with __text and __value?
How should the query be structured? Example: Select RowName1 AS __text, RowName2 AS __value From TableName;
How can I use __value? Example: ${GA:value}
How can I use __text? Example: ${GA:text}
I always get the error message with this query. -> Select RowName1 AS __text, RowName2 AS __value From TableName;
Error updating options: Received more than one (2) fields: __text,__value

Hello Sergej,
Version 3.5.0 is now running for me and using the query variables also works.
Thanks


Enable users to follow any list

Users should be able to follow any list by hitting some endpoint, probably api/list/{list_id}
Since the callback URL for Twitter authentication has to be registered with Twitter, we can't dynamically generate it. That means the flow will have to be homepage -> authenticate -> select list to follow (how?) -> process list as usual, unless I can pass some kind of context through the authentication process - ie, so that when Twitter returns the response, I know what list the user was trying to follow when they hit the authentication flow.
If I can't do that, there will have to be more of a frontend than there currently is, which represents a fairly chunky bit of work. Specifically, working out how a user selects which list they want to follow will be a tricky experience. There will also have to be a new bit of code at the backend.

User clicks a link --> server returns 200. HTML: "If you are not redirected in {n} seconds, click this link", potentially .js as well to automatically redirect user
Allows us to set a cookie in user browser so they only need to authenticate once as needed. Think about how long cookie should live for

Can we have something like "Click here to follow everyone in "[listname]" on Twitter" instead?

I've spent the week fighting with the web framework I'm using to return some HTML so that the user has something to click on. I'll keep working on it for now, I'm sure there's an easy way that I'm just missing.

Thank you for your perseverance 🙏

I think with the current architecture this isn't going to work. To get to something similar to https://megablock.xyz/, I am going to need to add a front-end application to deal with users. I'll put it in its own repo and name it appropriately, and it'll just be a small Flask app to manage the user session effectively. The Flask app will interact with this queueing system.


Scheduler is not using watch to monitor nodes

For some reason scheduler has a logic to watch nodes, but it is disabled:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/plugin/pkg/scheduler/factory/factory.go#L169
This is not very heavy, because there will be up to 100 nodes, but we should get rid of this.
@lavalamp @davidopp @wojtek-t

@piosz This is not a blocker for v1.0

@lavalamp
Is it possible to replace a poller with simple "watch with field selector" here:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/plugin/pkg/scheduler/factory/factory.go#L169
?
Basically pollMinions filters out all nodes that doesn't have condition NodeReady (plus are unschedulable, but this is easy to change to field selector). The problem is that "Node.Status.Conditions" is an array and IIUC there is currently no support for selectors on arrays.
Do we want to add it?
The alternative would be to wrap the watch interface + filtering function into another watch interface.

@wojtek-t I almost did this yesterday as part of #6546, but ended up not doing it. I think watch everything and filter in the client on request is probably easiest.

Fixed by #6717 and  #6857


Updated Abi struct for Base Rollup Circuit

The diagram https://miro.com/app/board/uXjVPlafJWM=/ have been altered in the way that data availability is ensured.
Instead of containing newNullifierHash, newCommitmentHash etc, it contains just a single calldataHash from two kernels.

Done.


[Broken] Viu Singapore not working

Checklist

[X] I'm reporting a broken site support
[X] I've verified that I'm running yt-dlp version 2021.04.03
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] Command-line config: ['-F', 'https://www.viu.com/ott/sg/en-us/vod/304335/Moon-Lovers-Scarlet-Heart-Ryeo', '--verbose']                                          [debug] Loading archive file None                       [debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8                                                [debug] yt-dlp version 2021.04.03                       [debug] Python version 3.9.2 (CPython 64bit) - Linux-4.14.113-21054248-aarch64-with-libc                        [debug] exe versions: none                              [debug] Proxy map: {}                                   [viu:ott] 304335: Downloading video info                [viu:ott] 304335: Downloading stream info               ERROR: An extractor error has occurred. (caused by KeyError('data')); please report this issue on https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/data/data/com.termux/files/usr/lib/python3.9/site-packages/yt_dlp/extractor/common.py", line 558, in extract
    ie_result = self._real_extract(url)
  File "/data/data/com.termux/files/usr/lib/python3.9/site-packages/yt_dlp/extractor/viu.py", line 226, in _real_extract
    stream_data = self._download_json(
KeyError: 'data'
Traceback (most recent call last):
  File "/data/data/com.termux/files/usr/lib/python3.9/site-packages/yt_dlp/extractor/common.py", line 558, in extract
    ie_result = self._real_extract(url)
  File "/data/data/com.termux/files/usr/lib/python3.9/site-packages/yt_dlp/extractor/viu.py", line 226, in _real_extract
    stream_data = self._download_json(
KeyError: 'data'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/data/data/com.termux/files/usr/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1026, in wrapper
    return func(self, *args, **kwargs)
  File "/data/data/com.termux/files/usr/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1047, in __extract_info
    ie_result = ie.extract(url)
  File "/data/data/com.termux/files/usr/lib/python3.9/site-packages/yt_dlp/extractor/common.py", line 571, in extract
    raise ExtractorError('An extractor error has occurred.', cause=e)
yt_dlp.utils.ExtractorError: An extractor error has occurred. (caused by KeyError('data')); please report this issue on https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.

Description

Viu Singapore seems to be broken

I already fixed this with C#, I don't know much about python. But I will try if I have time.

I already fixed this with C#

@zackmark29 Fixed what? The URL in this issue was fixed by https://github.com/yt-dlp/yt-dlp/commit/baa58739429cd04eb250036d90a7339b640aa932
$ yt-dlp https://www.viu.com/ott/sg/en-us/vod/304335/Moon-Lovers-Scarlet-Heart-Ryeo
[viu:ott] 304335: Downloading video info
[viu:ott] Downloading playlist 15237 - add --no-playlist to just download video
[download] Downloading playlist: Moon Lovers: Scarlet Heart Ryeo
[viu:ott] playlist Moon Lovers: Scarlet Heart Ryeo: Collected 20 videos; downloading 20 of them
[download] Downloading video 1 of 20
[viu:ott] 304335: Downloading video info
[viu:ott] 304335: Downloading stream info
[info] 304335: Downloading format(s) s1080p
[download] Destination: Traveling Back to Goryeo [304335].mp4
ffmpeg version 3.0.1 Copyright (c) 2000-2016 the FFmpeg developers
  built with gcc 5.3.0 (GCC)
  configuration: --enable-gpl --enable-version3 --disable-w32threads --enable-avisynth --enable-bzlib --enable-fontconfig --enable-frei0r --enable-gnutls --enable-iconv --enable-libass --enable-libbluray --enable-libbs2b --enable-libcaca --enable-libdcadec --enable-libfreetype --enable-libgme --enable-libgsm --enable-libilbc --enable-libmodplug --enable-libmp3lame --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libopus --enable-librtmp --enable-libschroedinger --enable-libsoxr --enable-libspeex --enable-libtheora --enable-libtwolame --enable-libvidstab --enable-libvo-amrwbenc --enable-libvorbis --enable-libvpx --enable-libwavpack --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxavs --enable-libxvid --enable-libzimg --enable-lzma --enable-decklink --enable-zlib
  libavutil      55. 17.103 / 55. 17.103
  libavcodec     57. 24.102 / 57. 24.102
  libavformat    57. 25.100 / 57. 25.100
  libavdevice    57.  0.101 / 57.  0.101
  libavfilter     6. 31.100 /  6. 31.100
  libswscale      4.  0.100 /  4.  0.100
  libswresample   2.  0.101 /  2.  0.101
  libpostproc    54.  0.100 / 54.  0.100
[h264 @ 000002379e63e1c0] non-existing SPS 0 referenced in buffering period
    Last message repeated 1 times
[h264 @ 000002379e455440] non-existing SPS 0 referenced in buffering period
Input #0, hls,applehttp, from 'https://d1k2us671qcoau.cloudfront.net/vodapi/viu_aws.m3u8?vid=76527ba3176f1d17db37afff62be49a1&layer=Layer4&ts=202104261848&Policy=eyJTdGF0ZW1lbnQiOlt7IlJlc291cmNlIjoiaHR0cHM6Ly9kMWsydXM2NzFxY29hdS5jbG91ZGZyb250Lm5ldC92b2RhcGkvKiIsIkNvbmRpdGlvbiI6eyJEYXRlTGVzc1RoYW4iOnsiQVdTOkVwb2NoVGltZSI6MTYxOTQ0ODQ4NH19fV19&Signature=iTUBJeu3uO0XAqsGffiYNh-J1GhnriW740gmK4U8p5hOfcGOpAW1KCTe8tMKbbg~DzzESRu9p0NHcdPkrWqrr-N3E9ijMvnl2PLSjjJWMNoWCDiEA~MHLFkCpvQBdfaf45MufFuoP73kxKwj8QG5g55xuIacfc~yN-VTNIfkjITWWVis6mRyaeYdh2-28fN1O6dM3XgYuiTN3thWTw43xo0pK7VJiQFryjTdISPakwHoaIxbrZosJQns4mYDzcCOKTXAOlbWLeCFEydaJGtQYMxQvxTwZ1U8AXVpR5FFngOm8IfuMI1SKKxyKraW3wRSJv5gjehW5oewsjyUigu-1w__&Key-Pair-Id=APKAJ6Z4RF5IYK7Y3SQQ':
  Duration: 00:59:45.08, start: 1.437344, bitrate: 0 kb/s
  Program 0
    Metadata:
      variant_bitrate : 0
    Stream #0:0: Video: h264 (High) ([27][0][0][0] / 0x001B), yuv420p, 1920x1080, 25 fps, 25 tbr, 90k tbn, 50 tbc
    Stream #0:1: Audio: aac (LC) ([15][0][0][0] / 0x000F), 48000 Hz, stereo, fltp, 98 kb/s
Output #0, mp4, to 'file:Traveling Back to Goryeo [304335].mp4.part':
  Metadata:
    encoder         : Lavf57.25.100
    Stream #0:0: Video: h264 ([33][0][0][0] / 0x0021), yuv420p, 1920x1080, q=2-31, 25 fps, 25 tbr, 90k tbn, 90k tbc
    Stream #0:1: Audio: aac (LC) ([64][0][0][0] / 0x0040), 48000 Hz, stereo, 98 kb/s
Stream mapping:
  Stream #0:0 -> #0:0 (copy)
  Stream #0:1 -> #0:1 (copy)
Press [q] to stop, [?] for help
frame=  116 fps=0.0 q=-1.0 Lsize=    2270kB time=00:00:04.56 bitrate=4075.2kbits/s speed=5.67x
video:2212kB audio:54kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: 0.187210%
Exiting normally, received signal 2.

ERROR: Interrupted by user


Differentiation

Add methods backward, derivative to problem for native differentiation. Add diffcp as an optional solver.

@SteveDiamond The build is passing; the failure is spurious (a Travis timeout).


deploy task should deploy from the default entry in packageDirectories in sfdx-project.json if the project is using sfdx format.

Feature Type
I'm interested in functionality that solves a current problem or limitation in how CumulusCI works.
Does your feature request solve a current problem?
Currently, CCI is designed for managing a single package per repository.
However, some commands could be made more flexible to help developers manually manage multiple packages in a single repo. For example, the retrieve_changes task has been improved (see #1335) and now writes retrieved changes to the default entry in packageDirectories in sfdx-project.json if the project is using sfdx format.
It would be great if the deploy task(s) would adopt the same behaviour : instead of deploying from the force-app folder by default, there should be an option (or default) for it to deploy from the default entry in packageDirectories in sfdx-project.json if the project is using sfdx format.
Describe the solution you'd like
When the deploytask is configured to deploy from path=force-app, I would like it to deploy, instead, from the default entry in packageDirectories in sfdx-project.json if the project is using sfdx format.
Describe alternatives you've considered
No response
Additional context
It would be great to change the current default behaviour of CCI, and not have to write custom task definitions for deploy to use this SFDX dynamic behaviour.

@RupertBarrow I like the idea of trying to make the deploy task be able to get a default package path from sfdx-project.json; thanks for the suggestion. We need to decide carefully how to handle backwards-compatibility since there are existing projects which still use Metadata API format packages in a src directory, with no sfdx-project.json. Possibly we could use logic like this:

If the path option is specified, use it.
Otherwise try to get the default package directory from sfdx-project.json and use it.
If not found, use src.

One option could also be to add an option "defaultpath: true" to explicitely activate this feature.
Or, better : if you want to deliver CCI with it activated, you could gives users the option to override it with "defaultpath: false".


Move link to self-service page

Based on the feedback from DAASI engineers we have to move the link to the self-service page as the current position isn't clear enough.
Maybe we should move it to the list of apps or structure the profile/settings section differently. TBD

Obsolete.


Request: QOI (Quite OK Image) Format

Just a suggestion of an additional codec to include. The QOI (Quite Okay Image) format is a single MIT C/C++ binary available here https://github.com/phoboslab/qoi. It has significantly faster encode/decode than PNG for a similar compression ratio. There is already python package, but it might make a nice addition to imagecodecs.

+1 for this one.

Hey
I have a question.
I used your tool for jpeg2000 lossless compression:
compressed_img = jpeg2k_encode(ds.pixel_array, level=0.0, reversible=1)  #2
One can immediately decompress the image with decode. Everything fine.
But after
from pydicom.encaps import encapsulate
from pydicom.uid import JPEG2000Lossless

ds.is_little_endian = True
ds.is_implicit_VR = False
ds.file_meta.TransferSyntaxUID = JPEG2000Lossless
ds.PhotometricInterpretation = 'RGB'
ds.PixelData = encapsulate([compressed_img])  #1
ds['PixelData'].is_undefined_length = True

ds.save_as("temp_test.dcm", write_like_original=False)

But after encapsulating the pixel data with pydicom (#1), I am not able to decompress with your tool.
Can you tell me why? How can I be sure, that the image is indeed reversible?
I found out, that the binary data (pixel data after encapsulating - see #1) has an additional byte-string in front of the compressed_image #2.
Am I missing something?
Thank you
Barbara

A QOI codec is included in v2022.7.27.


Improvement: add arguments to commands

This can be useful when running a command that needs specific arguments.
Simple example: Execute an Ansible Playbook, the command can be something like ansible-playbook -i inventory <playbook>, and when it is executed, clir asks for the argument <playbook>.

Done, arguments are identified by an underscore in front of them.


HALON-626: provide a way to run internal cluster commands.

Created by: qnikst
Update cluster presence command is introduced as a first
command.

Closing as an obsolete


Support custom rendering override for background image

Release
Renderer status
Tasks status

Not scheduled
🔄 .NET (#2539)🔄 Android (#2540)✔ iOS (#1609)🔄 TS (#2541)🔄 UWP (#2542)
🔄 Shared (#2537)✔ Designer (#2538)

The problem
While hosts can override rendering of text blocks, images, etc, they can't override background image

Auto-generated task status

[ ] Shared
[x] Designer
[ ] .NET
[ ] Android
[x] iOS
[ ] TS
[ ] UWP

Not quite sure what this is about or why it's needed, maybe an example would help.

If a host wants to override how the background image is drawn, they can't exactly do that today in many of the renderers. @nesalang you added this for iOS correct? Can you elaborate on the scenario that (Teams?) needed this ability for?

I understand the concept of custom rendering the background image, my question is why is that necessary? What's an example of a custom rendering that can't be accomplished with what we support out of the box, especially given we are introducing enhanced background image support?

Not currently under consideration, but will reactivate should we decide to pursue.


Change order of widget custom ramp and widget custom color fields and remove checkbox from the custom ramp one.

After using it for a while, I think it would be better to have the custom ramp before the custom colors of the bars.
Also, I don't think we should use the checkboxy type of component for the ramps, since it would be quite explanatory to see the ramp even before customizing it.

About last phrase, ok about that, but we will not provide the possibility to reset to the default auto-style ramp.

That's fine.

I've found a problem here, there isn't a default ramp for the widget, it depends on the data we are seeing in that moment. So that info is not "located" anywhere, it could change each time the user moves the map.

So what are we doing finally?

I'm waiting your thoughts from my comment...

Then, lets leave as it is right now for the time being. Closing!


Update but don't replace

If a component only cares about a certain set of query strings and wants to update them without affecting the others
E.g. ?q=search+term&sort=top
If the searchbox only updates the q parameter, and I have a separate dropdown component updating the sort parameter, I don't want them stepping on each other

I think the easiest route is just by decoding the query string, modifying it, and then encoding it before using url-set-query.
var setQuery = require('url-set-query')
var qs = require('query-string')
var assign = require('object-assign')
var input = 'http://foo.com/path?q=search+term&sort=top'

// extract
var query = qs.parse(qs.extract(input))

// update
assign(query, {
  q: 'new term'
})

// stringify
var url = setQuery(input, qs.stringify(query))
console.log(url)

A small increase in bundle size, but more robust:
var query = require('url').parse(input, true).query

:+1:


Semantic segmentation for binary masks (2 class: background/foreground object)

Hi @bodokaiser
Do you know how to adapt this code for binary masks? (background = black, foreground = white)
I changed the number of classes to 2:
#NUM_CLASSES = 22
NUM_CLASSES = 2
But some erros still occurs:
Traceback (most recent call last):
  File "main2.py", line 164, in <module>
    main(parser.parse_args())
  File "main2.py", line 139, in main
    train(args, model)
  File "main2.py", line 74, in train
    loss = criterion(outputs, targets[:, 0])
  File "/home/ubuntu/src/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/modules/module.py", line 224, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/ubuntu/pytorch/piwise/piwise/criterion.py", line 13, in forward
    return self.loss(F.log_softmax(outputs), targets)
  File "/home/ubuntu/src/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/modules/module.py", line 224, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/ubuntu/src/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/modules/loss.py", line 132, in forward
    self.ignore_index)
  File "/home/ubuntu/src/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/functional.py", line 674, in nll_loss
    return _functions.thnn.NLLLoss2d.apply(input, target, weight, size_average, ignore_index)
  File "/home/ubuntu/src/anaconda3/envs/pytorch/lib/python3.6/site-packages/torch/nn/_functions/thnn/auto.py", line 47, in forward
    output, *ctx.additional_args)
RuntimeError: weight tensor should be defined either for all or no classes at /pytorch/torch/lib/THCUNN/generic/SpatialClassNLLCriterion.cu:28
(pytorch) ubuntu@ip-172-31-85-122:~/pytorch/piwise$

Hi Andrew,
Can you change in main.py
if args.cuda:
    criterion = CrossEntropyLoss2d(weight.cuda())
else:
    criterion = CrossEntropyLoss2d(weight)

to
criterion = CrossEntropyLoss2d()

also you might want to read this for binary segmentation.

Thank you @bodokaiser !

Hi @bodokaiser again,
Just a small question, why the number of class is defined as 22 if PascalVoc has 20 classes (19 objects + background) ?

I think VOC before 2012 had less classes, however according to segmentation examples

pixel indices correspond to classes in alphabetical order (1=aeroplane, 2=bicycle, 3=bird, 4=boat, 5=bottle, 6=bus, 7=car , 8=cat, 9=chair, 10=cow, 11=diningtable, 12=dog, 13=horse, 14=motorbike, 15=person, 16=potted plant, 17=sheep, 18=sofa, 19=train, 20=tv/monitor)
For both types of segmentation image, index 0 corresponds to background and index 255 corresponds to 'void' or unlabelled.

So, if I have only 2 classes on my dataset (0 = background, 255 = foreground), I need to set NUM_CLASSES = 2 ?

NUM_CLASSES = 2 basically just says how much output channels to use in the last layer(s) of the chosen network architecture.
There is also another VOC specific transform which convert the color codes of the VOC images to class labels numbered from 1 to 22 so you might want to change this according to your dataset.

Solved!

I am trying to solve a binary mask segmentation as well for my dataset in this VOC format.. however getting a NaN value for segmentation loss.. could you please help me here??
Thanks!

Hello @numancelik34 ,
I am sorry for the late reply, and thank you for the contact!
Yes, my solution for this issue can be found here:
https://github.com/andrewssobral/deep-learning-pytorch/tree/master/segmentation
I created a git repository with some codes showing how to do binary segmentation with pytorch.
Please, let me know if it helps you, and feel free to contact me if you have any questions.
Best regards,
Andrews


Broken link in the docs

I finded this page https://github.com/sendgrid/sendgrid-nodejs/blob/main/docs/use-cases/README.md#email-use-cases after I click at the bottom link of the page "How to View Email Statistics" I went to the page and there is a broken link Event Webhook

Should be fixed now


[WIP]netpoll mode

https://github.com/mosn/easygo/commit/27b5582505b0975eb95d3299acd3abe5c086f311

add field to desc, avoid calling file.Fd which will set the conn to blocking mode

remove fd copy in NewDesc, prevent pre close file during GC, before the connection actually close

call ReadOnce again until readAgain == false, if there is still data to read when TLS is enabled.

Codecov Report

Merging #1511 (6b230f8) into master (2adbcbc) will increase coverage by 0.00%.
The diff coverage is 30.76%.

@@           Coverage Diff           @@
##           master    #1511   +/-   ##
=======================================
  Coverage   59.22%   59.23%
=======================================
  Files         338      338
  Lines       27046    27052    +6
=======================================
+ Hits        16019    16023    +4
- Misses       9247     9251    +4
+ Partials     1780     1778    -2

Impacted Files
Coverage Δ

pkg/network/connection.go
41.20% <0.00%> (ø)

pkg/network/eventloop.go
5.19% <33.33%> (+5.19%)
:arrow_up:

...aulttolerance/regulator/invocation_stat_factory.go
81.25% <0.00%> (-18.75%)
:arrow_down:

pkg/stream/stream.go
34.78% <0.00%> (-13.05%)
:arrow_down:

...ream/faulttolerance/regulator/default_work_pool.go
85.71% <0.00%> (-5.72%)
:arrow_down:

pkg/metrics/shm/zone.go
62.12% <0.00%> (-3.04%)
:arrow_down:

pkg/upstream/cluster/strict_dns_cluster.go
68.53% <0.00%> (-1.40%)
:arrow_down:

pkg/module/http2/transport.go
76.60% <0.00%> (ø)

pkg/upstream/servicediscovery/dubbod/bootstrap.go
59.52% <0.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2adbcbc...6b230f8. Read the comment docs.

GOMAXPROCS=4, use sofaload to benchmark

add testcase for OnReadTimeout

should be merged to develop branch, so deprecate this PR


support changing output device

Setting the default output device via /etc/pipewire/pipewire.conf or (temporarily) changing it using pw-cli is quite involved. Would you be open to adding both these options in pw-volume?

Yeah I'm considering adding this feature now. I think this will involve the ability to query sinks, and setting one. Probably the most straightforward way is to have it list output devices for pw-volume device output and set it via pw-volume device set-output <output>.


RLQS import protos

[!CAUTION]
Work in progress, do not merge.

This imports RLQS protos.
RLQS implementation prototype in #25

Implemented in https://github.com/grpc/grpc-java/pull/11418.


Clean-up all tables

They are currently not rendering very well, especially in the generated PDF.

I've started looking at this. Madoko tables are somewhat limited, so I'm replacing some tables with list when there is no easy way to fix them.


Join on empty string rather than IS NULL

I'm reasonably sure this is the fix for #176 but I don't have an editing environment set up to test it

The build failure seems to be something to do with coveralls:

Could not submit coverage: 504 Server Error: Gateway Time-out for url: https://coveralls.io/api/v1/jobs

Yes, looks good. That column had been made non-nullable to fix https://github.com/open-contracting/kingfisher-process/issues/235


Screen off with warning and alert

enable screen warning and alert if screen-off function is toggled
modification (ad74c314082909431fe8fd82e87faff2d1542517) to

performance tune for user not using screen-off function
prevent potential problem
less attraction impact period for driver (reduce period to 0.4 second)

Thanks!


MSID: 90996 Version: 2 DOI: 10.1101/2<IP_ADDRESS>8808

MSID: 90996
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>8808
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=90996
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>8808

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/90996v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>8808
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/90996/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

very minor reference formatting, otherwise fine to QC

Thanks Rebecca - should be solve din the next few hours


Notebooks should not use a hash in the url since new versions have new hashes. we should use name or path

/category implemented


Enables Disabling scroll to pan between modes

@boboxiaodd @NikKovIos
https://github.com/Yummypets/YPImagePicker/issues/148#issuecomment-397206799
Introduces isScrollToChangeModesEnabled in config. false will disable panning between modes

It could be used like workaround, but we should also correct the behavior when scroll is enabled, i think 🤔

@NikKovIos I didn't know that was a bug!? Panning an image in the library never accidentally scrolls sideways on my side, maybe I'm missing something^^

Hmm, there was an issue with scroll, but i can't find it..


remove parent relative path, which was incorrect

Our pom.xml files include a parent.relativePath declaration whose value is always ... That is not necessary and in fact it was incorrect (the correct value should have been ../pom.xml, which is in fact the default value).

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (59551e4) to head (c8a428e).
Report is 1060 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (c8a428e). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (c8a428e)

integration
7
1

integration2
3
1

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
1

skip-bytebuffers-false
7
0

unittests
5
0

unittests1
2
0

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #14037       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436        3     -2433
  Lines        133233        6   -133227
  Branches      20636        0    -20636
=============================================
- Hits          82274        0    -82274
+ Misses        44911        6    -44905
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
?

java-21
0.00% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
?

skip-bytebuffers-true
0.00% <ø> (-27.73%)
:arrow_down:

temurin
0.00% <ø> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

There is one more in pinot-timeseries-spi as ../../pom.xml. Is that correct?

It looks odd. I don't know why @ankitsultana used that reference. Anyway, that value may be semantically correct or not, but it is not the same as the ones I'm removing here that don't reference a pom.xml (but a folder) which is syntactically incorrect (and the semantic they want to implement is the one defined by default by Maven)


platform exception

No refresh token found.
flutter: An error occured during logging user in.
flutter: PlatformException(authorize_and_exchange_code_failed, Failed to authorize: unauthorized_client: Invalid client or Invalid client credentials, null, null)

That's not really an issue. The log just shows the error caught by the plugin when the user tried to login but failed, maybe due to a lost connection or invalid user credentials. Try to check with your backend regarding this issue, and feel free to reopen this issue if it persists.


allow for memset/memcpy alternatives during early boot

Zeroing the BSS and copying data to RAM with regular memset/memcpy may
cause problems when those functions are assuming a fully initialized
system for their optimizations to work e.g. some instructions require
an active MMU, but turning the MMU on needs the .bss section to be
cleared first, etc.
This is a more scalable and future-proof solution to the issue described
in #31321.

Out of curiosity... does the issue occur with the minimal libc's memset? That code doesn't seem to require BSS.

Out of curiosity... does the issue occur with the minimal libc's memset? That code doesn't seem to require BSS.

It's the newlib one. It uses the FP unit to go faster, but that
needs an addressing mode that isn't available without the MMU enabled.
The commit log for commit c5b898743a20 is worth reading.

Hey @npitre I think you also want to modify the scripts/gen_relocate_app.py content.

Good catch, done.


invalid models need a status identifier

so the admin can know which ones are invalid

ideally I can create this without adding a new field, via validating the models in admin and caching the result

better idea: models should be unable to be placed in an invalid state


Fix jshint errors

'const' is available in ES6
Missing semicolon.
'let' is available in ES6

Thanks, but this was resolved by dropping support for Node 6.


Package for Arch Linux, btw?

Hi guys,
there is a gb-studio package for Arch in AUR, but it is very outdated.
I'd like to request an up to date package, ideally in official community repo:
https://archlinux.org/packages/

AUR seems to had been updated : https://aur.archlinux.org/packages/gb-studio-git/
I can install 3.0.2 using it


[VT-8] Add support for "modify task"

Scott Kostyshak on 2013-04-06T21:34:33Z says:
It would be nice to have support for modifying a task in Vit.

I would prefer to have the binding be 'm'. I don't use priorities. I think that everyone uses 'modify' but only some use priorities. I would favor having a two-key binding for priorities. For example, capital 'P'. Thus, to get high priority, medium priority, low priority, one would do 'P-h', 'P-m', 'P-l', respectively.

Migrated metadata:
Created: 2013-04-06T21:34:33Z
Modified: 2014-02-09T01:14:58Z

David Patrick on 2013-04-06T21:53:28Z says:
+1

steve rader on 2013-04-06T21:55:31Z says:
Scott Kostyshak wrote:

It would be nice to have support for modifying a task in Vit.

what do you mean by "modify"?
there's already...
'e' for edit (task NNN edit)
'c' for change description (task NNN NEW DESCRIPTION)
":s/OLD/NEW/" for modify description (task NNN modify /OLD/NEW)

Scott Kostyshak on 2013-04-06T22:06:08Z says:
steve rader wrote:

Scott Kostyshak wrote:

It would be nice to have support for modifying a task in Vit.

what do you mean by "modify"?
there's already...
'e' for edit (task NNN edit)
'c' for change description (task NNN NEW DESCRIPTION)
":s/OLD/NEW/" for modify description (task NNN modify /OLD/NEW)

Ah, "change" is what I'm looking for. 'c' currently uses Taskwarrior's 'modify' command to change the description. But it can do more than that.
The following command changes the date so that it is due tomorow (and the description is left unchanged):

task  modify due:tomorrow

Similarly, one can change tags, UDAs, and more with the 'modify' command. Now that I know, I think that change does what I want. The question now is whether its good to keep the name as 'change' when most Taskwarrior users are used to 'modify'. At a minimum, I think that the help should say that 'change' can do more than change a description.

David Patrick on 2013-04-06T22:36:44Z says:
I like that line of thinking Scott, "modify" is sort of universal, powerful, and it's consistent with taskwarrior use. It might allow some kebinding decluttering too.. if hitting "m" resulted in a partially completed task command. As an example, suppose I was already looking at vit with a filter, and I hit "m" I would see
task proj:foo +tag modify >
from there it's easy-peasy to add/change/remove just about anything! description, project dates, whatever,
and in a way that re-enforces CLI usage, and not as an alternative to it.

steve rader on 2013-04-06T23:34:42Z says:
Oops!  I ahem don't really use stuff beyond descriptions and priorities, so
I had forgotten the functionality here: 'c' allows you to do "task NNN modify".
Scott - try 'c' and then...

 due:tomorrow

David - try 'c' and then...

 proj:foo +tag

Elegant, no?
The problem is, I think, is the doc...

 cmd   c STRING        change the current task's description

...should be...

 cmd   c STRING        modify (change) the current task

...or something?

David Patrick on 2013-04-06T23:44:54Z says:
or something! that's excellent!

Scott Kostyshak on 2013-04-06T23:49:08Z says:
Works well. I think the new help description is good.
Thanks Steve.

Scott Kostyshak on 2013-09-27T23:14:15Z says:
This should go into VIT 1.2.

David Patrick on 2013-09-28T15:22:46Z says:
is this not already working fine as keybind "c" (change)?
worksforme :)

Scott Kostyshak on 2013-09-28T15:50:10Z says:
David Patrick wrote:

is this not already working fine as keybind "c" (change)?
worksforme :)

Yes, the functionality is the same. I was keeping this issue to remember to change 'c' to 'm'.

Scott Kostyshak on 2013-09-28T15:52:02Z says:
David Patrick wrote:

I like that line of thinking Scott, "modify" is sort of universal, powerful, and it's consistent with taskwarrior use. It might allow some kebinding decluttering too.. if hitting "m" resulted in a partially completed task command. As an example, suppose I was already looking at vit with a filter, and I hit "m" I would see
task proj:foo +tag modify >
from there it's easy-peasy to add/change/remove just about anything! description, project dates, whatever,
and in a way that re-enforces CLI usage, and not as an alternative to it.

By the way, this can now be implemented with keybinds + %TASKARGS.

Scott Kostyshak on 2013-09-28T15:52:42Z says:
Done at 75bea863.


COSBETA-241: Redirect to business or Poison Centre pages based on domain

Business users and Poison Centre/MSA users will access the service via different domains. All user groups will use the same service (Keycloak) to log in to their accounts and should be correctly redirected back to the domain from which they started the login journey.

Pull Request Test Coverage Report for Build 5810

192 of 195 (98.46%)  changed or added relevant lines in 23 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.304%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cosmetics-web/app/controllers/errors_controller.rb
6
7
85.71%

cosmetics-web/app/controllers/concerns/domain_concern.rb
11
13
84.62%

Totals

Change from base Build 5794:
0.0%

Covered Lines:
10167

Relevant Lines:
10668

💛  - Coveralls


Feature request: ACME server

It would be really nice with support for acme / certbot clients to issue certificates.

The ACME protocol has a veeeery large footprint but we have a working beta here in our lab. As this is integrated into our commercial "protocol gateway" component its unlikely that it will become OpenSource in the near future - if you are interessted please contact me via PM.

ACME protocol has a veeeery large footprint
@oliwel
Maybe it would make sense to be part of this acme2certifier ca_handler.py - list?

ACME protocol has a veeeery large footprint

@oliwel Maybe it would make sense to be part of this acme2certifier ca_handler.py - list?

thank you for this pointer - as this project has a generic EST client and we have a generic EST server it should work out of the box, so feel free to integrate it.

@flybyray: Certificate enrollment via the generic est ca handler possible; we use OpenXPKI in our release regression since a few months.  We also released a dedicated ca handler integrating into OpenXPI via the RPC server API.  Feel free to test if you are interested...


Sending Push Notifications - Master key is invalid

I get Exception : "Master key is invalid, you should only use master key to send push"
I don't understand where i supposed to enter the master key
In Android :
androidQuery.whereEqualTo("deviceType", "android");
ParsePush androidPush = new ParsePush();
androidPush.setQuery(androidQuery);
androidPush.setData(androidData);
Log.i("androidData=", androidData + "");
Log.i("LOG", "Sending Android");
androidPush.sendInBackground(new SendCallback() {
						@Override
						public void done(ParseException e) {
							if (e == null) {

IOS:
PFPush *androidPush = [[PFPush alloc] init];
[androidPush setQuery:androidQuery];
[androidPush setData:androidData];
NSLog(@"Sending Android Data");
[androidPush sendPushInBackgroundWithBlock:^(BOOL succeeded, NSError *error){
if (succeeded) {
My Server Settings:
var api = new ParseServer({
databaseURI: databaseUri,
cloud: process.env.CLOUD_CODE_MAIN || __dirname + '/cloud/main.js',
appId: process.env.APP_ID,
masterKey: process.env.MASTER_KEY,
push: {
android: {
senderId: '12345678',
apiKey: 'My apikey'
},
ios: [
{
pfx: '/cer/developmentcertificate.p12',
bundleId: 'bundleId',
production: false // Dev
},
{
pfx: '/cer/distributioncertificate.p12',
bundleId: 'bundleId',
production: true // Prod
}
]
}
});

With parse-server (and hosted parse too), you should not sent push notifications from the client (because then anyone can spam all of your users with push notifications.)  Call a cloud function instead, and send them from cloud code by passing the { useMasterKey: true } option as a second parameter... See #564.

Thank you very much. Actually this what I thought but the reason for spending almost 10 hours to this and in the end open issue that ANY WORD OF THIS IN TUTORIAL HERE https://github.com/ParsePlatform/Docs/blob/master/en/android/push-notifications.mdown
and here
https://github.com/ParsePlatform/parse-server/wiki/Push
My app it's something like WhatsApp so i must send pushes from the client.
Anyway thank you !!!


Fix Cargo.toml warnings

warning: default_features is deprecated in favor of default-features and will not work in the 2024 edition
(in the keyboard-types dependency)
warning: default_features is deprecated in favor of default-features and will not work in the 2024 edition
(in the raw-window-handle dependency)
warning: default_features is deprecated in favor of default-features and will not work in the 2024 edition
(in the wayland-backend dependency)

We paused/stopped work on Glazier in January, and do not recommend its use.
Why do you need this change?

Ha - I missed that :)
I don't need glazier. I mistook it for a newer winit with some slightly nicer design decisions. I can move what I'm working on over to winit instead (a terminal backend).

P.s. this is still listed as active on https://linebender.org/


feature/67-give-the-storybook-a-custom-factiverse-logo

Added logo and img folder as described in #68
Closes #68
Closes #67
Screenshots (before and after the change):
added this logo

Can somebody check Storybook Publish and UI Tests? can not merge if these are not approved

I have approved it but still not reflecting. Maybe run again?

Chromatic was down yesterday when I tried to approve it, maybe something got messed up. Try doing a new commit to make the checks run again.


feat: async-node target support

Summary
Work in progress of adding the "async-node" target to rspack
ReadFileChunkLoading is based off this:
https://github.com/ScriptedAlchemy/webpack/blob/49dab4cc5ec8507ab4cad766e2a37a23d9f8a8d1/lib/node/ReadFileChunkLoadingRuntimeModule.js
and would require commonjs chunk plugin to be aware of if its async-node or node
https://github.com/ScriptedAlchemy/webpack/blob/2df826722016f844e7a618f5f787ffa6de43c9b7/lib/node/CommonJsChunkLoadingPlugin.js
this lets you use fs.readFile instead of require.
This is very useful as a modified version of this enabled module federation on node servers

@ScriptedAlchemy do you still want to work on this, if not I will work on this since it's needed for our test

since async-node target is supported in #3468  this pr could be closed


Adding regions dinamically

After audio is ready, I click on a button to add several regions but it doesn't work. Somebody knows what is happening?
Thanks in advance.
		var seek_regions = function(times){ 			for(i=0; i < times.length; i++){ 			    wavesurfer.addRegion({ 			        start: times[i], 			        end: times[i]+1, 			        color: 'hsla(400, 100%, 30%, 0.1)' 			    }); 			}

@manumora can you provide the code you are using on codepen?


App name should be removed from library project

Summary
Adding library to the project changing the app name as most of the app having app_name key default.
Code to reproduce

Just add image picker library in dependency

Android version
ALL
Impacted devices
All
SDK version
2.1

Thank you @HimanshuDudhat for the suggestion.
The changes will be reflected in the next release.


Add a "bundle" build target

Adds a supported way to invoke rollup to generate lit-element.bundled.js so I don't have to dig it out of ./node_modules/ any more.

We're going to publish bundles on lit.dev for Lit 2: https://github.com/lit/lit/issues/1257


Issue with Dynamic Waves

Hi,
I have an issue where when an object creates dynamic waves then after a period of time artefacts are produced on the waves causing large spikes.
I've recorded the unity file so that you can see what I mean:
https://youtu.be/rFcHJkGJwe0
Have you seen this happen before?

Here are the settings for the SimSettingsWave that I am using

After creating a new project and starting from scratch everything was fine :)


Bump minor version

This PR bumps the version numbers so they match the next release.
Please verify the version numbers are correct, and update the CHANGELOG manually. You
should merge this PR ASAP, but no later than before the next release cycle starts.
Checklist

[ ] Version numbers are correct
[ ] CHANGELOG.md has been updated

Only minor bump on main. Skipping


Question: Uploader - large(?) file sizes generate 502 errors

@evanchiu just wondering if you've run into any issues loading large (ish) files with uploader? I get a Failed to load resource: the server responded with a status of 502 () when trying to upload a file of 5mb or greater. Any experiences to share on this matter? The issues seems to be caused by the gateway rather than lambda.

Hello @i001962!
No I haven't personally run into this yet.
From the API Gateway docs it looks like the payload size limit is 10mb, with no way to increase it.  For uploading larger files, you might look at something like resumable.js, which chunks the files in client-side javascript, with metadata, to enable re-assembling them on the server-side.  For reassembly, each of the chunks could be stored in a separate S3 bucket, with another lambda watching for completion and reassembling into the main bucket.

I also was hoping to upload larger files (the 10mb stated payload would be enough) but I get an alert that reads "Error uploading [image.jpg]. Max upload size is ~4MB." when trying to upload a 6mb file. I was hoping to find some insight as to where this 4mb restriction is coming from.
While there is no way to self-change the limit, the doc page does say you can request an increase.

Hello @pr3sidentspence,
While the API Gateway payload size limit is 10mb, the Lambda invocation payload limit is 6mb.  I added a section about the uploader limitations to the readme.
I'd welcome a pull request to support the more complex multi-part upload.


Raspberry Pi & Chromium

Hi, I have script using Tampermonkey. Script runs for forms on website, but this website using turnstile. Your package will work with tampermonkey & chromium? I have no idea how this works technically, I understand that the code searches for an image on the screen, so it will find this image if it is in the Chromium window? If I understand correctly, the python script will run in the background and will always perform click actions if it finds a turnstile on the screen or I have to manually activate it somehow every time? Can you explain it to me?

@creativepl-php , It automatically scans the screen for turnstile checkboxes and clicks them. Therefore, it functions only in GUI. Cheers


[TEMPLATE] - Angular + Node.js/Express - FullStack

What domain will the project template be based on?
FullStack
If 'Others', please specify the preferred domain.
No response
Particular directory path for the template.
templates/FullStack/Angular+Backend/Angular+Nodejs/
What tech stacks do you want to include?
Angular, Node.js, Express, MongoDB(optional)
Brief Description
This project template combines Angular with Node.js and Express to create a full-stack web application using the MEAN stack. The frontend is developed using Angular for a dynamic and responsive user interface, while the backend utilizes Express to handle API requests and MongoDB for data storage. This setup is ideal for building modern web applications that require real-time data handling and seamless user experiences.
Relevant Links:
Angular Documentation
Node.js Documentation
Express.js Documentation
MongoDB Documentation
👀 Have you spent some time checking if this issue has been raised before?

[X] Yes
[ ] No

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Would you like to work on this issue?
No

Interested!
Also want to know, Can I use logging, code formatting and some other dev dependencies installed in this full stack template.

Sure thing .. you can use them .. just keep in mind that you are not using any such outdated packages or versions
Assigning it to you

Closed by: https://github.com/Abhishek-Mallick/universal-box/pull/108


golang: fix ErrRpcStatus.GetCode() stack overflow

#349
fix ErrRpcStatus.GetCode() stack overflow

Codecov Report

Merging #352 (c905f29) into master (70d4382) will increase coverage by 0.16%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #352      +/-   ##
============================================
+ Coverage     33.77%   33.94%   +0.16%
  Complexity      643      643
============================================
  Files           220      220
  Lines         11439    11439
  Branches        277      277
============================================
+ Hits           3864     3883      +19
+ Misses         7325     7306      -19
  Partials        250      250

Flag
Coverage Δ

golang
34.00% <0.00%> (+0.74%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

golang/error.go
0.00% <0.00%> (ø)

golang/client.go
40.24% <0.00%> (+4.57%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Throughput Testing: SB Test Upgrade

Upgrade .NET SB tests to latest SDKs and MI.

PR opened

Closing as complete


[3.8] Report driver version in handshake

PHPORM-177
Checklist

[ ] Add tests and ensure they pass
[ ] Add an entry to the CHANGELOG.md file
[ ] Update documentation for new features

Could you update the changelog please.


[KOGITO-2550] - Fixed vet test

See: https://issues.redhat.com/browse/KOGITO-2550
Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster

Because the steps were missed(due to that if condition) there are some changes in the files. I changed those in https://github.com/kiegroup/kogito-cloud-operator/pull/412. After that is merged I can rebase and this one will succeed

@Kaitou786 this one depends on #412?

@ricardozanini on the master branch itself if you run make test you'll see the changes on the CRDs, I first noticed this when sending #412 . So after the #412 is merged the lint stage here will pass (After Rebase)

can you verify why lint in failing?

@spolti yes, the lint is failing because the change is detected in the crds. Previously this check was not able to detect the changes as the operator-sdk command was not executing,  because of that some commit got merged in the master which should have also changed the crds. I changed added those crds in #412 After the rebase the lint over here will succeed.

@ricardozanini I made some changes can you review again?


[UI] Catalog service not loading properly

Rancher	v0.49.0
Cattle	v0.119.0
User Interface	v0.70.0
Rancher Compose	v0.6.0
I get the following error:
Mixed Content: The page at 'https://rancher-server/apps/catalog/library:elasticsearch' was loaded over HTTPS, but requested an insecure XMLHttpRequest endpoint 'http://rancher-server/v1-catalog/templates/library:elasticsearch:1'. This request has been blocked; the content must be served over HTTPS.

Also the Catalog Images at the catalog page give this warning:
Mixed Content: The page at 'https://rancher-mgmt.konetserv.io/apps/catalog' was loaded over HTTPS, but requested an insecure image 'http://rancher-mgmt.konetserv.io/v1-catalog/templates/library:sysdig-cloud?image'. This content should also be served over HTTPS.

+1
and kubernetes tab is missing...

+1

k8s is unrelated to this but also fixed in the next build.

Reopening to test .

Tested with rancher-server version - v0.49.1-rc2
Set up - Rancher server rancher-server with SSL behind Nginx container as https reverse proxy.
Able to list the catalog and create stack using "Elastic Search" template.


Feature request : add binding for method to be called when end of scrollbar is reached

I would like to load additional data when the end of the scrollbar is reached and there is additional data  available on the server. The method would be responsible for determining if additional data is available and to fetch the data and append it to the array displayed in the repeat.

This shouldn't be a part of vs-repeat. Write another directive and put it
on the same element as vs-repeat (or it's scrollable parent) and handle
scrolling behavior there)
On Sun, Dec 6, 2015 at 7:35 PM, rekna1<EMAIL_ADDRESS>wrote:

I would like to load additional data when the end of the scrollbar is
reached and there is additional data available on the server. The method
would be responsible for determining if additional data is available and to
fetch the data and append it to the array displayed in the repeat.
—
Reply to this email directly or view it on GitHub
https://github.com/kamilkp/angular-vs-repeat/issues/91.

I tried angular-scrollevents, but it seems to break the vs-repeat... I have put it on the same element as vs-repeat.

Take a look at this infinite scroll directive:
http://www.williambrownstreet.net/blog/2013/07/angularjs-my-solution-to-the-ng-repeat-performance-problem/


Inconsistent NuGet shield

Captain, we're seeing some strange NuGet readings on the shields:
The readings are fluctuating between 234K and 249K

It ought to match this result, which comes fresh from nuget.org:
curl -H 'Accept: application/json' "https://www.nuget.org/api/v2/Packages()?\$filter=Id%20eq%20'SimpleInjector'%20and%20IsLatestVersion%20eq%20true" | jq '.d.results[0].DownloadCount'

Right now, it seems to correspond. I'm tempted to attribute the inconsistence to a momentary NuGet instability?

I repeated every day for a week before raising the issue, I'll post here in the next few days if I see the issue again

Has this problem recurred?

Feel free to open a new issue if this problem recurs.


Currently the source reports are run directly on the checked out code.
Consider moving this to the GitHub search API:
https://developer.github.com/v3/search/
While there might be a reason for the source checkout (for example running existing tooling over the source), the current source reports all look like they can be implemented as search requests.
Downside - yet more running out of rate limited requests.

Plus side, searches don't use up the 5,000 request limit. Down side, they have a harsher(?) rate limit.

First step - write a source report that uses the GitHub search API. Will mean changing the API to include the GitHub client (or adding an alternative API).

Scratching this. Having the code local is simpler.


docs(api/rendering): instruction to disable nunjucks

Mainly to show example usage of hexo.render.renderer.get() method.
This also can be mentioned in this section https://github.com/hexojs/site/pull/1532.

https://github.com/think-in-universe/hexo-stop-tag-plugins/blob/5494a7f7a0820f61673cb06258743c7caa181c0d/lib/overrider.js#L20-L23

Check List
Please read and check followings before submitting a PR.

[ ] I want to publish my theme on Hexo official website.

[ ] I have read the theme publishing doc.
[ ] name is unique.
[ ] link URL is correct.
[ ] preview URL is correct.
[ ] preview URL web site is rendered correctly.
[ ] Add a screenshot to source/themes/screenshots.
[ ] Screenshot filename is same as value of name.
[ ] Screenshot size is 800 * 500.
[ ] Screenshot file format is png.

[ ] I want to publish my plugin on Hexo official website.

[ ] I have read the plugin publishing doc.
[ ] name is unique.
[ ] link URL is correct.

[x] Others (Update, fix, translation, etc...)

Languages:
[x] en English
[x] ko Korean
[x] pt-br Brazilian Portuguese
[x] ru Russian
[x] th Thai
[x] zh-cn simplified Chinese
[x] zh-tw traditional Chinese

We could add examples for that as well.

+1, but I'm not familiar with the syntax though.

We should add disableNunjucks for all hexo plugins.

Obviously except for markdown renderer (for anyone else reading this). I can add it and learn the syntax.


iostat graphs

Using version 2019-01-06
Command: iostat -Tsf 1 30 (AIX 7.1 and AIX 7.2)
The IOStat Logical and Physical graphs both show Kbps as zero.
The IOStat Logical and Physical graphs both show Kb_read for the actual write column and Kb_wrtn graph is the read column (in the iostat data file).  The tps graph is showing the Kbps column in the data.
Thanks
Dean Rowswell

Sounds like something is off by one with the parser. Can you attach a sample file?

iostat.txt

Somehow this never worked right and I didn't notice it either. I've fixed some array handling errors. Please try building manually and ensure everything looks ok now.

Hi Hunter - works great!  Thanks so much for the speedy fix and great tool by the way!   I use it mostly for nmon parsing but I can't find where/if nmon captures "iostat -f" data for filesystem I/O - so now I can simply run iostat -Tsf and then graph it with Visualizer.


Android fix

missing interface of the new exo-player library removed.
version updated

sorry for wrong place. closed.


The description of the metadata design document is not clear

Proposed changes
Issue Number: close #9179
Problem Summary:
leader 节点的日志条数达到阈值（默认 10w 条）并且满足checkpoint线程执行周期（默认六十秒）
The number of log entries of the leader node reaches the threshold (default 10w) and meets the checkpoint thread execution cycle (default 60 seconds)
Checklist(Required)

Does it affect the original behavior: Yes
Has unit tests been added: No Need
Has document been added or modified: Yes
Does it need to update dependencies: No
Are there any changes that cannot be rolled back: No

Further comments
No

I'm very sorray,The last PR was accidentally deleted by me..


Switch to different file format of manual

The manual currently is in DOCX format. But I don't have Microsoft Word and I don't want to have it. Is this really the only way to access the documentation?
Can you please switch to a format that is not proprietary?
OpenDocument Format is one option (.odt, used by e.g. LibreOffice and OpenOffice.org).
An export to PDF would also be fine.
A HTML manual would be even better.

https://github.com/erincatto/box2d/blob/ecatto/manual/docs/manual.md

I agree, I'm working on this currently. If you like, you can try building the manual in that branch and send me your feed back. Feel free to comment here: https://github.com/erincatto/box2d/pull/573

Ah thanks. manual.md looks good.

However, I find it very hard to navigate. There's no TOC, but that's a different issue …

manual.md is not supposed to be read directly. I put Doxygen build instructions in the pull request notes.

Documentation is now available online in HTML.
https://box2d.org/documentation/


grpc_credentials: remove well_known_names

Signed-off-by: Long Dai<EMAIL_ADDRESS>
Commit Message:
Remove weel_known_names in grpc_credentials field, part of #7238
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

source/extensions/grpc_credentials/well_known_names.h (and similar in other PRs) acted as a sort of docs for the format of factory names. Is this documented elsewhere (I didn't see at a quick glance)? Should the format be documented, perhaps here: https://github.com/envoyproxy/envoy/blob/main/include/envoy/config/typed_config.h#L22? There's a comment there, but I don't think it's sufficient...

source/extensions/grpc_credentials/well_known_names.h (and similar in other PRs) acted as a sort of docs for the format of factory names. Is this documented elsewhere (I didn't see at a quick glance)? Should the format be documented, perhaps here: https://github.com/envoyproxy/envoy/blob/main/include/envoy/config/typed_config.h#L22? There's a comment there, but I don't think it's sufficient...

+1 let's do a follow up PR somewhere to document expectations here. Perhaps also here: https://github.com/envoyproxy/envoy/blob/main/EXTENSION_POLICY.md


E2e testing: Minting/burning

[x] e2e tests fpr policy keys endpoints
[x] e2e tests for minting/burning

Comments
Can be merged after: https://github.com/input-output-hk/cardano-wallet/pull/3199
Some tests are currently pending due to issues with mint/burn.


Type Mapping Transformation: Autopopulate Keys Update

Bug Description
The "autopopulate keys" function during type mapping transformations uses an exact match and is case sensitive. However, the json payload typically uses syntax (e.g., data.description) that is different than the "cleaner" version of the ontology in Deep Lynx (e.g., Description). Also, the ontology (e.g., Control Center) may not be a perfect one to one match with the payload value (e.g., controlcenter), so a bag of words or closest match technique would greatly simplify large type mappings.
Steps to Reproduce
Using container 49 in Deep Lynx Acceptance, type mapping the Windchill data source.
Impact
This is not a blocker by any means, but a type mapping of 23 shapes took ~1 hour with all the button clicks whereas with a flexible autopopulate function, it probably would've been 10 minutes. This would be beneficial when we have larger data sources and type mappings.

this is a feature, not a bug


[stable/airflow] Only set Celery & Redis env variables when needed

The environment variables for Redis and Celery prevent
the helm chart to work in LocalExecutori mode.
They are now only exported if:

Redis is enabled
and Airflow executor is "Celery"

Signed-off-by: Olivier Monnier<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @gsemet please review

/lgtm


Docs.MS Release Updates for azure-security-keyvault-secrets

Update docs metadata and targeting for release of azure-security-keyvault-secrets

Docs Build status updates of commit c1309f4:
:clock10: Full build: the average full build time is 83 min(s) 42 sec(s), this is based on the last 695 build(s) for this repository.


Make installable package

Hi @juanmc2005 , here is an installable package as discussed in #2.
In order to make things work with the python packaging system, had to do the following changes

Assign a package name. I picked the rather long streaming_speaker_diarization, as that most closely matches the repository name. Feel free to pick something else. Right now, external code needs to import streaming_speaker_diarization.pipelines and similar, which is a lot of typing.
I had to group your individual modules (i.e. single files) into a package (folder with the files) that can be installed as one. That required introducing a separate folder with the package name. So your source packages are now in src/streaming_speaker_diarization/. An alternative setup could have been to replace the src/ folder by a folder named streaming_speaker_diarization/. I felt that keeping src/ was closer to your original layout.
I modified the imports. You had imported each module individually, which doesn't make sense anymore once they are grouped into a package. The imports now explicitly use local imports such as from . import sources as src. I import with the full package name only in the main.py so that you can still run main.py as a script. However, you would need to add the src folder to your PYTHONPATH. Once you installed the package, you can also run your main script as python -m streaming_speaker_diarization.main. I didn't mark the main script as an executable script as that would have probably meant picking a different name than main for it and I didn't want to interfere with your choices there.
I added the typical setup files (setup.cfg, setup.py and pyproject.toml). You can build a package (python wheel) by running python -m build --wheel. By having the setup.py you can probably also just pip-install from the git-repository itself. The setup files contain some classifiers. Let me know if those are appropriatse.

Please feel free to make any change that you see fit. If you are busy with other things, I'm also happy to apply any changes that you suggest.

Thanks for the feedback. If you don't mind, please take a look at what I did and if that meets your expectations.

Awesome, thank you for the quick changes. I'll make a test run on my end tomorrow morning and merge if everything goes well


[UWP] Switch the compiler to Clang for better performance

Description
Google has indicated that they only optimize for the clang compiler:

Windows: https://groups.google.com/forum/#!topic/skia-discuss/ZZ7Ol3alB-I
Linux: https://groups.google.com/forum/#!topic/skia-discuss/P4tg5eBVIiU

VS bug #730674

I see work to use clang on linux to get more performance - is it possible to do so on Windows or there are specific blockers?

It has been a long time, but we are moving Win32 to Clang. Noticing vast improvements already, just check out #758. Need a little more time for UWP, but that can come later as there are probably fewer users of that platform.
As soon as #1007 is merged, I'll get a preview release out for the world.

Hey Matt,
I am a user of UWP and would really appreciate Clang there too. I might be one of the few though 😄

Hi Matthew,
This landed recently, thought you might find it useful (it uses llvm-mingw).

Thanks! I must really get to this sometime.

I'm working on a feature for my UWP app that could really use this (rotated images). It's unfortunate how slow the rendering is at the moment.

I'm working on a feature for my UWP app that could really use this (rotated images). It's unfortunate how slow the rendering is at the moment.

I am also one of the few... But I am really having a hard time with the rendering performance of rotated images with Skia. Is this planned to be solved any time soon?
Thanks @mattleibow for all your research and work on this topic

I'm also still struggling with this on UWP. It's unfortunate that my desktop is slower than my Android phone, and users notice it too.
I'll eventually move to MAUI (hoping that it's better), but that's a long way off.


[Android] Please, please provide compiled binaries

It would be really good if someone can compile also for Android (Amd64) updated versions.
Hope someone can provide this. Thank you for your efforts!!!

Or add whatever has to be added to the Travis script so that they are built automatically.

I tried adding the build guide to .travis.yml and it works. Tested the arm64 binary on my phone. no issues.
https://github.com/Sporif/dnscrypt-proxy/releases
https://travis-ci.org/Sporif/dnscrypt-proxy/builds/344817293
I also tried using a docker container of the standalone android ndk (rhardih/stand) but it turned out much slower:
https://travis-ci.org/Sporif/dnscrypt-proxy/builds/344786625

Thank you very much for actual version!

Are android/x86_64 and android/i386 actually needed?
I thought that Android-x86 had been abandoned. Chromebooks maybe?

Android builds have been added, thanks a lot @Sporif !
Do you think we should upx them?

Just tried a upx version on my phone, says illegal instruction when I launch it :/

Ah, not a good start :(

I thought that Android-x86 had been abandoned. Chromebooks maybe?

http://www.android-x86.org/releases/releasenote-7-1-r1 :)


[Feature Request] Add FullDMD Info for Backglass Manager

leprinco — Today at 2:27 AM
I'd like to see in one place if my backglasses are full dmd. Can a full dmd column be added in the backglass manager

useful for sure

Fixed with 2.20.0.


Update project configuration files

This change updates/creates the following files

Add .gitignore file to filter undesired files (compiled, packaging and IDE)
Add requirement in requirements.txt
setup.py: Use setuptools instead of distrubutes as it is deprecated
setup.py: read requirements.txt

I tested these modifications.
It looks good to me.


Fix bug creating default config

Fixes NoMethodError: undefined method `create_config' when config file does not exist.

Came looking to see if anyone else had come across the same issue for the create_config and the second two methods not being in the class. Delighted to see this fit was posted earlier today and matched my own 😎


Warning about unused lfs_mlist_isopen

In the latest v2.3.0, there are build warnings about unused lfs_mlist_isopen() when asserts are disabled (-Wunused-function).  The build that gives this warning is running on GitHub Actions with a macos-11.0 container.

Oh interesting, thanks for reporting this. Out of curiousity, do you know which compiler/version reports this? GCC 7.3.1 doesn't warn since the function is static inline, as this is commonly used for header-only functions.
But it's easy enough to cover in a #ifndef LFS_NO_ASSERT, lfs_mlist_isopen involves a linked-list walk and probably shouldn't be inlined anyways.

Out of curiousity, do you know which compiler/version reports this?

No, sorry.  It's probably clang though because it's on Mac.  I can try to find out...

But it's easy enough to cover in a #ifndef LFS_NO_ASSERT

Yes, that's the change I made to get the build to pass without warnings.

Ah that makes sense, currently the only compiler in CI is Clang. I wonder if it does extra checks based on the file extension to avoid warning about header-only functions.
Anyway, here we go: https://github.com/littlefs-project/littlefs/pull/499

Looks like the compiler that gives the warning is:
Apple clang version 12.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin20.1.0

Hi, I am using the latest version of LittleFS 2.4.1 and I get the warning
error: 'lfs_mlist_isopen' defined but not used [-Werror=unused-function]
 static bool lfs_mlist_isopen(struct lfs_mlist *head,
             ^
cc1: all warnings being treated as errors

I get these errors with the following compiler flags -

LFS_NO_MALLOC
LFS_NO_DEBUG
LFS_NO_WARN
LFS_NO_ERROR
LFS_THREADSAFE

when I add the LFS_NO_ASSERT flag the error goes away.

Hmm, @Brute4cer skimming through lfs.c I'm not sure how that could happen.
Is it possible your system's implementation of assert does not use its value? Maybe only in release builds?
Maybe we should add __attribute__((unused)) to this function, though this would be compiler dependent...

I can confirm that on my system (stm32h7) the default assert impl will trigger this warning when NDEBUG is defined.
Adding the LFS_NO_ASSERT to our cmake build settings for the littlefs lib does limit the warnings.
Adding __attribute__((used))  or __attribute__((unused)) during compilation will also suppress the warning. with no change on final binary size.

Confirming this warning is generated with:
arm-none-eabi-gcc (15:9-2019-q4-0ubuntu1) 9.2.1 20191025 (release) [ARM/arm-9-branch revision 277599]
On this Pi Pico project: (Cortex-M0+)
https://github.com/rumbledethumps/rp6502


support TCP proxyapp plugin endpoint

Currently we communicate with plugin by stdin, stdout and stderr pipes.
Stdin and stdout are used to support RPC server. Stderr is used to route logs.
Kris asked to support TCP endpoints. Let's enable it.

TCP channel instead of stdin+stdout will be used for RPC calls.
Second optional TCP channel will be used for errors.

Minimal valid config specifies cmd OR RPC server host:port then.

@kalder I can't add you to reviewers.

Extracted logging to #3464 . This one is blocked by #3464 now.

@kalder I can't add you to reviewers.

@kalder I've added you as a collaborator to the repo (I think you need to accept the invitation before Taras can add you to reviewers).
Does the interface in this PR work for you?

Does the interface in this PR work for you?

Yes. I've accepted the invite and things seem to be working.


Restore the "home" screen with all accounts and mail counts

It seems that since 5.706 the home screen listing all accounts (with unread mails and stars count for each account) has disappeared.
This offered a really handy way to check for new mail on all accounts in one single refresh click.
Now, there's the need to go and select each account, switching views, or the unified inbox view. But I don't want to see at once all message subjects, only message counts for all accounts, in a single screen. The option to hide the unified inbox could have led back to the old screen, but no luck :-/
Please restore (optionally) the old feature.
Thanks in advance.

When displaying the list of accounts in the drawer you can swipe down to refresh all of them.
For the unread count see #4515.
We're not going to add back a dedicated accounts screen.

Too bad, but thanks for the quick feedback.

When displaying the list of accounts in the drawer you can swipe down to refresh all of them.
For the unread count see #4515.
We're not going to add back a dedicated accounts screen.

Swiping down does not work for me


yaxg: use ffmpeg instead of ffmpeg_3

Motivation for this change
https://github.com/NixOS/nixpkgs/issues/120705
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123474 at d496275b run on x86_64-linux 1

2 packages built successfully:

tests.trivial
yaxg

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/graphics/yaxg/default.nix:18:3:
   |
18 |   installPhase = ''
   |   ^

Tested and working
Why wasn't I pinged even though I'm the maintainer of yaxg?

@neonfuz You're not a member of the @NixOS organization.
I did mention you in https://github.com/NixOS/nixpkgs/issues/120705 however.


UDPIngress Routing Rules Support

This PR adds a v1beta1.UDPIngress API and removes the older v1alpha1.UDPIngress. In the new version multiple rules and service backends are available in the API.
This resolves #1080 and #1081

Reviewed and UATed config creation stuff. Didn't UAT actual proxying since that part isn't really user-facing and I don't see any reason why the integration test wouldn't cover everything necessary. There's one UX thing we may want to change (route naming) and one thing that's broken (TCPIngress and UDPIngress that use the same backend port, but different backend protocols, attempt to create a colliding Kong service, and somehow wipe out the TCPIngress route).

Good catch, thank you for being so thorough :bow:
I'll get this corrected.

We've left the UDPIngress controller disabled. Do we want to enable it by default now?

I think at this point we do, done in f098a76a8c56198bab5d21d71c897c3bd51f54c7

This does not handle UDPIngress SNI routing yet, correct? That was not quite in scope for the original issues, but we should add it for TCPIngress parity and coverage of Kong features. IMO that should be a follow-up issue since the foundation is solid and I don't see any reason the implementation here would block that off based on a cursory review of how we handle it for TCPIngress.

Right this would be out of scope for what we're trying to do here, and can be a follow up.

Before we merge: after talking a bit more with @rainest I'm going to add one more integration test that deploys same-named TCPIngress and UDPIngress to prove that we're not colliding.

@rainest ready for your re-review, new collision detection tests added in 5ebb9341658c44ce655ad5f45748ad95012a095d which I would appreciate if you could please look over. :bow:

https://github.com/Kong/kubernetes-ingress-controller/commit/0351511985111a1646444c6972494acc8428eee6 won't actually verify that the collision won't happen unfortunately, since the TCP and UDP Services are different.
The collision condition happens because the Kong service name, derived from the K8S Service name and port, is actually the same without the suffix. That only happens if both the UDP and TCP listens are exposed on the same Service with the same port (rather than simply having both TCPIngress and UDPIngress objects in general), whereas the test uses two different Services.
That's easy to see in the admin API (there's only one service where you'd expect two), but may be a bit more annoying to test completely by actually sending traffic, as you need something that actually does expose TCP and UDP ports and a client that can use them effectively. While CoreDNS does apparently support both TCP and UDP DNS, it doesn't look like the stock Golang resolver gives you a choice of protocol, and presumably always uses UDP. We'd probably need https://github.com/miekg/dns or similar instead.

0351511 won't actually verify that the collision won't happen unfortunately, since the TCP and UDP Services are different.
The collision condition happens because the Kong service name, derived from the K8S Service name and port, is actually the same without the suffix. That only happens if both the UDP and TCP listens are exposed on the same Service with the same port (rather than simply having both TCPIngress and UDPIngress objects in general), whereas the test uses two different Services.
That's easy to see in the admin API (there's only one service where you'd expect two), but may be a bit more annoying to test completely by actually sending traffic, as you need something that actually does expose TCP and UDP ports and a client that can use them effectively. While CoreDNS does apparently support both TCP and UDP DNS, it doesn't look like the stock Golang resolver gives you a choice of protocol, and presumably always uses UDP. We'd probably need https://github.com/miekg/dns or similar instead.

My bad, this new version of the test combines the TCP and UDP into a single service instead: https://github.com/Kong/kubernetes-ingress-controller/pull/1400/commits/7092d5fead3b9441dce76eda5228c12ce994673e
lmkwyt :+1:

After further discussion and troubleshooting with @rainest we verified that the service collision problem is no longer a problem when a service has two protocols sharing a port, the resulting kong configuration includes both of those services separately.
However we did find a potential new issue in that the kong Upstream didn't seem to be able to handle the protocol difference, and we'll be looking into that separately as something we may potentially need to report to upstream kong/kong.
For now, we're back to "ready to merge" this with new tests added proving that we've corrected for the kong.Service collision problem, and we have a follow up issue in https://github.com/Kong/kubernetes-ingress-controller/issues/1441

Since we're already adding a lot of tests here, I'm going to do some cleanup, refactor and expansion of all the existing integration tests here as it seems like now is as good a time to do it as any.

still wip, but most of the testing cleanup and refactoring is complete: the tests are consistently passing in places they were having trouble before: the next (hopefully last?) think to tackle is that there appears to be problems in pkg/parser/ with duplicate generated upstreams when TCP / UDP are created on the same cluster and I'm going to dig into that next.

There's several fixes to our integration tests present in this PR which would be of immediate help to other initiatives, so I'm moving the port collision issue into its own scope for now so that we can merge this and get the value present added to next sooner rather than later, and follow up on this in an upcoming iteration:
https://github.com/Kong/kubernetes-ingress-controller/issues/1446

Everything has been working solidly, but for hygiene I'm splitting the testing improvements here into their own PR:
https://github.com/Kong/kubernetes-ingress-controller/pull/1453
That way the commit history is clean and we can do one last review. I'll create a new followup/replacement PR for this once that merges.


Change BlockPuller logs to avoid string format penalty

 [BlockPuller] Change BlockPuller logs to avoid string format penalty


Datetime Problem on Where Clause Javascript

table.take(0).where(function (at) {
                                    return this.__updatedAt > new Date(at);
                                }, maxUpdatedAtSaved )
                        .includeTotalCount().read().then(function (results) { ...

Using this code to get the number of updates since i last updated my data returns a 400 Bad Request with the message :
{"message":"The query specified in the URI is not valid. A binary operator with incompatible types was detected. Found operand types 'Edm.DateTimeOffset' and 'Edm.DateTime' for operator kind 'GreaterThan'."}
Dunno how to get That dateTime switch to dateTime Offset...

I found on the js sdk the following code :

else if (_.isDate(value)) {
/*
* Dates are expected in the format
*   "datetime'yyyy-mm-ddThh:mm[:ss[.fffffff]]'"
* which JSON.stringify gives us by default
*/
text = JSON.stringify(value);
if (text.length > 2) {
text = text.slice(1, +(text.length - 2) + 1 || 9e9);
}
text = text.replace(/(T\d{2}:\d{2}:\d{2})Z$/, function(all, time) {
var msec;
msec = String(value.getMilliseconds() + 1000).substring(1);
return "" + time + "." + msec + "Z";
});
return "datetime'" + text + "'"; < ------------ Error!!!Should be datetimeoffset
}

This is something we are aware of, for iOS we ended up making a DateOffset class, that we knew to convert to datetimeoffset vs datetime.
For now, you can pass the raw oData query to your calls.  One option (for the short term) is to use a query, call query.toOData, and then replace datetime to datetimeoffset in the result.  Then pass that into the read call.  It should unblock you for now.
Another, if you don't need datetime support at all, is to copy the file from the cdn (http://ajax.aspnetcdn.com/ajax/mobileservices/MobileServices.Web-1.2.6.js) and tweak it / rebuild out of source.

Thx for the fast response :).
I have already changed the source code to datetimeoffset since i dont really need datetime yet.
Though i had problem minifing the source if you can help me here ?
Plz tell me if you intend to fix this in the short term cause we are building a commercial product and need the stable version.

I'll try to take a look at this tomorrow and see if there is a quick fix we can do to solve this now to get an official fix in quickly.
If you clone the JS repo, you should be able to use grunt to get the minified version, the default task should just work.  If not, let me know what issues you are seeing building.  I realize, we probably need to update the readme for ppl not familiar with using grunt.

This will require updating the queryJS repo first: https://github.com/Azure/queryjs

@nikatlas Did you close this as you have a workaround that works for you / no longer need it.  Or did you find a way to fix this w/o a code fix?


Changing parameters with Config file takes full day to take effect

Not sure this should be considered an issue.  But I noticed if changes are made during the dimming time, the mirror dimming doesn't reflect the changes until the next day.  Just something small I noticed.  Happy to clarify if needed.

If you're changing the config file, you should be reloading the MM to make the config changes take effect. If you're not doing that, I don't know how you expect it to know you changed the config. If you are doing that, I'm not sure what you mean and need clarification as to your exact process.

Yes.  I’ve been attempting to set up a long dim time.  Starting at 7pm and
reaching the max dim after 4 hours of dimming.
After I modify the max dim parameter to suit my needs, and restarting MM,
the mirror is exhibiting an incredibly dark dim.
I’ve noticed my adjustments go into effect, but not until I allow a full
day to pass and let the module “reset” the next day.
The module is working beautifully, but I’ve noticed this strange accordance
is all.
I should also note that I’m using this module in conjunction with
Mmm-remote control.  Not sure if that is having any influence
On Fri, Jul 22, 2022 at 4:58 PM Fifteen 15 Studios @.***>
wrote:

If you're changing the config file, you should be reloading the MM to make
the config changes take effect. If you're not doing that, I don't know how
you expect it to know you changed the config. If you are doing that, I'm
not sure what you mean and need clarification as to your exact process.
—
Reply to this email directly, view it on GitHub
https://github.com/Fifteen15Studios/MMM-AutoDimmer/issues/4#issuecomment-1192929981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV4YKP3IIC7YJF7FUOKOCH3VVMDORANCNFSM54MUT5XA
.
You are receiving this because you authored the thread.Message ID:
@.***>

If you start it during the schedule, it will start fully dim. So in your case, if you start your MM between 7pm and whenever the end of your schedule is, it will start fully dimmed. If, instead, you were to start it at 6:59, it will start bright and slowly dim.
I'm sure that with enough code I could start it partially dim, based on how far into the transition time you are, but that would be a lot of work for a very edge case.

Yeah.  I figured.  Thank you for your time!
On Fri, Jul 22, 2022 at 7:09 PM Fifteen 15 Studios @.***>
wrote:

If you start it during the schedule, it will start fully dim. So in your
case, if you start your MM between 7pm and whenever the end of your
schedule is, it will start fully dimmed. If, instead, you were to start it
at 6:59, it will start bright and slowly dim.
I'm sure that with enough code I could start it partially dim, based on
how far into the transition time you are, but that would be a lot of work
for a very edge case.
—
Reply to this email directly, view it on GitHub
https://github.com/Fifteen15Studios/MMM-AutoDimmer/issues/4#issuecomment-1192996067,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV4YKPZ2NXT3JRH73QXIJ43VVMS3FANCNFSM54MUT5XA
.
You are receiving this because you authored the thread.Message ID:
@.***>

This will be fixed in the next version. I just got it working, as I needed to do this for some other features I'm working on. Not sure when the next version will be released, but it should be within the next week or 2.

Also, it will work during the "brightening" time too (during the transition from dim to bright)

The new release properly handles this situation


Export Volunteers / Volunteer data

Create a CSV, a PDF and an Excel of the selected data of the backend giving also the option to download the related files in a zip/rar or accessible via URL to properly see them in the exported documents.
For this task, it is necessary to take the same structure/solution as the task developed in the week before.

Story Points: 8
See the discussion in Backend S2
Powered by Parabol

Story Points: 3
See the discussion in Backend S3
Powered by Parabol


TestCase decode url returns string in python3

When i create a test based on falcon.testing.TestCase or falcon.testing.TestBase and run simulate_request with:
url = urllib.parse.quote_plus('%C3%A4%C3%B6%C3%BC%C3%9F%E2%82%AC')
simulate_request(url, 'utf-8')

i get the following error:
  File "falcon/api.py", line 155, in falcon.api.API.__call__ (falcon/api.c:1733)
  File "falcon/request.py", line 265, in falcon.request.Request.__init__ (falcon/request.c:1784)
UnicodeEncodeError: 'latin-1' codec can't encode character '\u20ac' in position 13: ordinal not in range(256)

I think this is due to this https://github.com/falconry/falcon/blob/master/falcon/util/uri.py#L246
and this https://github.com/falconry/falcon/blob/master/falcon/request.py#L263

@jmvrbanac I would like to take this issue.

@masellfoodpanda I am unable to reproduce this error. I wrote a testing function
def test_xyz(self):
    import urllib
    url = urllib.parse.quote_plus('%C3%A4%C3%B6%C3%BC%C3%9F%E2%82%AC')
    self.simulate_request(url, 'utf-8')

It didn't threw any error.
I am using python3.4. Is there anything I am missing ?

Ah, sorry. Seems i made double encoding in the report.
here is a test:
#!/usr/bin/env python
# -*- coding: utf8 -*-

import falcon
from falcon.testing import TestBase, StartResponseMock
import urllib

class TestFalcon(TestBase):
    def setUp(self):

        class API(falcon.API):
            pass

        class Resource(object):
            def on_get(self, req, resp, data):
                pass

        self.api = API()
        self.srmock = StartResponseMock()

        self.api.add_route('/test/{data}', Resource())

        self.urls = [
            '/test/%E5%BB%B6%E5%AE%89',
            '/test/%C3%A4%C3%B6%C3%BC%C3%9F%E2%82%AC',
        ]

    def test_xyz(self):
        for url in self.urls:
            with self.subTest(data=urllib.parse.unquote_plus(url)):
                self.simulate_request(url, 'utf-8')

At the moment i need to monkey patch this like so:
from falcon.util import uri
orig_decode = uri.decode
uri.decode = lambda x: orig_decode(x).encode('utf-8').decode('latin1')

@yashmehrotra  If you aren't going to take this I'd like to take a look!

@ministrike3 Sure, have a go.

The problem is that create_environ() is not tunneling via ISO-8859-1 under p3k. See: https://github.com/falconry/falcon/blob/master/falcon/testing/helpers.py#L131
The fix should be as simple as checking six.PY3 and then performing path.encode('utf-8').decode('iso-8859-1').

Awesome, thank you @jmvrbanac and @kgriffs


Fix: Strenghen Crystal break on level up 👎 #50

Looks good to me, will test it out later.

Ran once and it strengthened the crystal :+1:
Ran one more time and it strengthened the crystal to max :+1:
Ran one more time and it didn't level up the crystal :-1:
When I tap "Level Up" button manually, sometimes it takes more than 2 seconds to load. I been manually timing it and it hasn't gone over 3 seconds so I think line 1159 to: sleep 3 will work from now on. I did that and it worked.


Add optional did replacement parameter to web_url_to_at_uri

This is the first bit needed to address https://github.com/snarfed/bridgy/issues/1575;  I wasn't sure if it was better to use the regex or simply match on provided handle parameter (currently unused)?

(You can see how that docstring ends up rendered here: https://granary.readthedocs.io/en/latest/source/granary.html#granary.bluesky.web_url_to_at_uri )

Looks good!


Add Custom init spawns for Icarus/Karutan2

Pls check if code is okay i am not used to C#.
Custom spawnpoint initalisation for

Icarus
Karutan2
Santa Village
Raklion Boss

close #304

If i die in Kanturu Ruins3 where should i get spawned Kanturu Ruins 1 or Kanturu Ruins 3 safe zone?
I think this needs an extra handeling because i dont think there is anything handling death in specific area of specific map to cause different spawn points to get used :)

Oh, you're right... I think we might just ignore that for now, until we come up with a good solution. As far as I know, only Kanturu is such a map.

I had a quick look at other server sources, and it seems that if you die in kanturu ruins 3, you'll get respawned at the safezone of kanturu 1, too - so all is good ;-)

okay haha :D

@sven-n Finished pls have a look <3

Okay i checked everything after your changes seems to be fine. Is something still missing? I think its mergable now :D

Yes, it looks good now. Thank you!

Yes, it looks good now. Thank you!

No THANK YOU <3


Better matcher usage have len be empty

What this PR does / why we need it:
This PR improves the multus e2e test readability by:

assert the list of VMs as empty rather than the sum of the lengths of both lists should be zero
using the HaveLen matcher rather than checking if the length of an array is equal to the expected number

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/approve


Run linkcheck

There are no calls to linkcheck in the template in CI. This should be added to the docs workflow but not in PRs.

Note that there is a tox env for this: https://github.com/scipp/copier_template/blob/ff7f76be4a02a021e522568892ab82c89d2aaf93/template/tox.ini.jinja#L41-L44

What was the reason for not running linkcheck in CI during PR? Was it only performance? In this case, should we just reenable it? Most projects don't have many links. And we tend to be slow to fix broken links when detected in main builds.

What was the reason for not running linkcheck in CI during PR? Was it only performance?

Not at all. It was due to frequent issues with flakyness, e.g., when a site is temporarily unreachable.


clifm: 1.1 -> 1.3

Motivation for this change
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I'm not using this package any more. Would you like to remove me and add yourself as a maintainer?

I don't really use this package, don't feel comfortable maintaining it, only wanted to try it and saw that it was outdated. I've removed you as a maintainer.


Registering plugin dll without steps

Hi,
I remember registering plugin .dll without the plugin steps. Is this an illusion in my mind or is there a way to configure it which I cannot recall now?

You can use the /e switch to exclude plugin-steps

Thank you very much!