Get-MsalToken fails for some users, references a .NET in error message.

We are using the same version of PoweShell (5.1.19041.2673) and MSAL.PS (<IP_ADDRESS>).  Also, referencing the same Azure Tenant, Azure App Registration and certificate.  All laptops connected to the same network and have similar age and configuration.
Executed the following script for three of us.  It worked for me but failed for 2 of my co-workers.
$subject=""
$Tenant = ""
$ClientID = ""
$subjectName = "CN=$($subject)"
$cert = Get-ChildItem Cert:\CurrentUser\My  | where-object { $_.Subject -eq $subjectName } | Select-Object * | Sort-Object -desc NotAfter | Select-Object -first 1
$ThbPrint = $cert.Thumbprint
$connectionDetails = @{
'TenantId' = $Tenant
'ClientId' = $ClientID
}
$connectionDetails += @{'ClientCertificate' = Get-Item -Path "Cert:\CurrentUser\My\$($ThbPrint)" }
$token = Get-MsalToken @connectionDetails
They receive the following error message:
"Get-MsalToken : Could not use the certificate for signing. See inner exception for details. Possible cause: this may be a known issue with apps build against .NET Desktop 4.6 or lower. Either target a higher  version of .NET desktop - 4.6.1 and above, or use a different certificate type (non-CNG) or sign your own assertion as described at https://aka.ms/msal-net-signed-assertion.  At C:\PShell\Scripts - Outlook Rooms\testroom.ps1:137 char:18 +         $token = Get-MsalToken @connectionDetails +                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~     + CategoryInfo          : AuthenticationError: (Microsoft.Ident...arameterBuilder:AcquireTokenForClientParameterBuilder) [Write-Error], MsalClientException     + FullyQualifiedErrorId : GetMsalTokenFailureAuthenticationError,Get-MsalToken"
We tried installing https://dotnet.microsoft.com/en-us/download/dotnet/thank-you/runtime-desktop-6.0.15-windows-x64-installer
List of .NET versions:
dotnet --list-runtimes
Microsoft.AspNetCore.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.28 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.4 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

The reason it worked for me is because I had MicrosoftTeams module loaded where others did not. MicrosoftTeams loaded first so I am using an MSTeams version.  Replaced the Microsoft.Identity.Client. and  Microsoft.Identity.Client.Desktop in ...WindowsPowerShell\Modules\MSAL.PS\<IP_ADDRESS>\Microsoft.Identity.Client.4.37.0\net45 and .. Desktop\net45 with MSTeams version.  For PowerShell 5 also required "authority cannotbe found fix", https://github.com/AzureAD/MSAL.PS/issues/45.

This is a work around not a fix.  Keeping it open.

This is because MSAL.PS targets MSAL.NET for net45 instead of net461. net45 has poor crypto support.


[Event Request] Report 12 VAT Statement OnCalcLineTotalOnCaseElse

Please add an event OnCalcLineTotalOnCaseElse in Report 12 VAT Statement
procedure CalcLineTotal(VATStmtLine2: Record "VAT Statement Line"; var TotalAmount: Decimal; Level: Integer): Boolean
    begin
        if Level = 0 then
            TotalAmount := 0;
        case VATStmtLine2.Type of
            VATStmtLine2.Type::"Account Totaling":
                begin
                    ...
                end;
            VATStmtLine2.Type::"VAT Entry Totaling":
                begin
                    ...
                end;
            VATStmtLine2.Type::"Row Totaling":
                begin
                    ....
                end;
            VATStmtLine2.Type::Description:
                ;
            else
                OnCalcLineTotalOnCaseElse(VATStmtLine2, Amount, TotalAmount, Level, PeriodSelection, StartDate, EndDate, EndDateReq, PrintInIntegers, UseAmtsInAddCurr);  <---- NEW EVENT

        end;

        exit(true);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnCalcLineTotalOnCaseElse(var VATStmtLine2: Record "VAT Statement Line"; var Amount: Decimal; var TotalAmount: Decimal; Level: Integer; PeriodSelection: Enum "VAT Statement Report Period Selection"; StartDate: Date; EndDate: Date; EndDateReq: Date; PrintInIntegers: Boolean; UseAmtsInAddCurr: Boolean)
    begin
    end;

Thank You!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Close Window or Frame

Hello,
simple question:). How can i close an windows on eFrame::App::update{}? As an example with if btn_quit.clicked() {????};?
thx

Not really experienced enough to be sure. But have you tried using Context::send_viewport_cmd() to send a ViewportCommand::Close command?
Maybe using the Discussions github tab to ask for help before sending a bug report would also be better for this kind of questions.

thats is it,thx


Failed assertions and return values in samples/socket-api/wasm-src/socket_opts.c

@loganek  we found something in samples/socket-api/wasm-src/socket_opts.c:
If run with ./socket_opts(the native version) and iwasm ./socket_opts(the Wasm version), four failures will be raised
Below assertions are failed

[ ] https://github.com/bytecodealliance/wasm-micro-runtime/blob/main/samples/socket-api/wasm-src/socket_opts.c#L85
[ ] https://github.com/bytecodealliance/wasm-micro-runtime/blob/main/samples/socket-api/wasm-src/socket_opts.c#L94

Below return values are failed values and perror() finds errno are both Invalid argument

[ ] https://github.com/bytecodealliance/wasm-micro-runtime/blob/c505da7464b70e009a44c3c7d0a6645f7cc44166/samples/socket-api/wasm-src/socket_opts.c#L251
[ ] https://github.com/bytecodealliance/wasm-micro-runtime/blob/c505da7464b70e009a44c3c7d0a6645f7cc44166/samples/socket-api/wasm-src/socket_opts.c#L255

@lum1n0us Thanks for raising, what OS are you using? I can setup the same environment

I'm going to remove the IPV6_JOIN_GROUP examples from this and add to its own multicast IPV4/6 sample as part of this PR  (revision in progress) https://github.com/bytecodealliance/wasm-micro-runtime/pull/1519

@lum1n0us I've updated https://github.com/bytecodealliance/wasm-micro-runtime/pull/1519 so that there's a separate multicast ipv4/6 examples which resolves the join/leave group issue. I think the other assertion failures are OS-specific. Eg on linux the send buffer size you request is doubled, but on mac it isn't.
I'm thinking it might be worth removing the assertions and printing what is received from getting the socket option back, but this example has been useful in testing socket opt changes in WAMR and the assertion easily demonstrates a failure.


OMMkiller terminated backup process

I'm trying use cron job to copy files from kubernetes to S3 and in kubernetes and got pod error OMMKilled
Looks like it slowly increase virtual memory and reaching pod limits.
Container limits set to: cpu 3. Memory: 5Gb
amount of copy files: 3401311
changing buffer memory didn't help
Am i doing something wrong?
or its memory leak issue?

pprof.pdf

I've had to play with the parallel argument. Setting parallel to 0 has caused me problems. In my tests setting --parallel to 20 uses about 500Mi


Dicsuss global visual representation of challenges over time

I was looking at this TV program page when thinking that maybe we could do something similar for challenges. x-axis would be time and y-axis challenges?

Source
Tagging @vpchung and @rrchai for ideas

It is a great visualization and thanks for sharing.

what will the categorical group of challenges for y-axis: platform?

Most mouses have vertically scroll wheel, but not horizontal scroll wheel. I am not sure if users would prefer to scroll horizontally over scroll vertically. This page may consider this and only allow to filter and show all programs in one day. In addition, I think most users don't the exact date of challenges.
If we would like to go for this visualization, maybe we could just show the all challenges in a year. In the future, we can narrow to month if we have more challenges.


[chore]: Investigate and fix "Deploy to Cloudflare workers" job

Made an issue + PR in the wrangler-action github repo to help debug the problem - https://github.com/cloudflare/wrangler-action/issues/226

With wrangler-action now updated to show the inner exception, I have a new lead.
🔑 Uploading secrets...
  Error: Invalid Version: latest
  ##[debug]TypeError: Invalid Version: latest
  ##[debug]    at new SemVer (file:///home/runner/work/_actions/cloudflare/wrangler-action/v3/dist/index.mjs:3094:13)
  ##[debug]    at compare (file:///home/runner/work/_actions/cloudflare/wrangler-action/v3/dist/index.mjs:3368:32)
  ##[debug]    at gt (file:///home/runner/work/_actions/cloudflare/wrangler-action/v3/dist/index.mjs:3379:29)
  ##[debug]    at semverCompare (file:///home/runner/work/_actions/cloudflare/wrangler-action/v3/dist/index.mjs:28453:24)
  ##[debug]    at uploadSecrets (file:///home/runner/work/_actions/cloudflare/wrangler-action/v3/dist/index.mjs:28653:13)
  ##[debug]    at main (file:///home/runner/work/_actions/cloudflare/wrangler-action/v3/dist/index.mjs:28561:15)
  ::endgroup::
Error: Failed to upload secrets.
Error: 🚨 Action failed


Feature1

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: vkadia sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


remember map selections and keep selections on durning tarkov monitor screenshot feature

Describe the Feature Request
It would be nice to have a way to remember the selections when you go map to map, for example keep the loot stashes shown instead of resetting the selection on reloading the page.  also when you take a screenshot on tarkovmonitor app it will keep those selections and show player position rather than resetting the selections and just showing player marker.

The map settings should be saved as you navigate to different maps and when you reload a page. If that's not happening, there's something wrong with your browser's local storage. Your browser may be configured to not allow the website to store data or something.

@djperrotti which browser are you using?

Pushed out some changes today that should this work better.


Instructions crowd up on the titles

Can we get some extra margin between the titles and the instructions:
From

to

@jwunderl this is fixed right?

Ah yes fixed in https://github.com/microsoft/pxt/pull/9050


Add release-note block to PR template

What this PR does / why we need it:
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:
NONE

/lgtm

@mrIncompetent unfortunately you have to re-lgtm :disappointed:

/lgtm


Add comment that Refresh does not work on a custom page in model driven app.

https://powerusers.microsoft.com/t5/Building-Power-Apps/Refresh-function-not-working-on-custom-page-model-driven-app/td-p/1384587

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4face5c1-c211-72f3-6f94-f0f858be931a
Version Independent ID: 4face5c1-c211-72f3-6f94-f0f858be931a
Content: Refresh function in Power Apps - Power Platform
Content Source: power-platform/power-fx/reference/function-refresh.md
Service: power-platform
Sub-service: power-fx
GitHub Login: @gregli-msft
Microsoft Alias: gregli

Thanks for your comment, @kayetter. This issue has been assigned to the doc author for review.


cert-manager addon

Motivation
Contribution to the security add-on:
This security add-on draft version installs cert-manager and automates TLS certificate provisioning for the basic security mode, in combination with one of the ingress add-ons: `ingress-nginx'  and ' traefik'.
How to Demonstrate (works also with traefik instead of ingress-nginx):
k2s addons enable ingress-nginx
k2s addons enable dashboard
k2s addons enable security

This will cause:

installation of cert-manager
creation of a cluster-wide CA Issuer, with the name k2s-ca-issuer
the public certificate of this issuer gets imported in the window computer certificate store's trusted root authorities
the dashboard ingress on https://k2s-dashboard.local is terminated with a TLS Certificate issued by K2s Self-Signed CA for the host k2s-dashboard.local. The browsers will show no security warning.
README.md shows how products can use the feature to terminate their Ingresses.

TODO
gateway-nginx: multiple issues to solve first:

dashboard not working with gateway yet, regardless of cred-manager
cred-manager needs gateway CRDs to be deployed, else it crasches. Currently these CRDs come with gateway-nginx...

Observations:

dashboard traefik is using now Kind: Ingress. cert-manager is not working with traefik CRD IngressRoute:   https://doc.traefik.io/traefik/providers/kubernetes-crd/

Contribution to the https://github.com/Siemens-Healthineers/K2s/issues/258:
This security add-on draft version installs cert-manager and automates TLS certificate provisioning for the basic security mode, in combination with one of the ingress add-ons: `ingress-nginx' and ' traefik'.


Fix missing void argument declarations #678

This is my first contribution so apologies in advance if I am violently ignoring any guidelines.

Thanks for the contribution! To accept this or any future contributions we will need a Contributor’s Licence Agreement (CLA) signed or authorised by yourself. The easiest way to do this is if you create an mbed account and accept this click through agreement which is applicable if this is a personal contribution. Alternatively, you can find a [slightly different agreement to sign here] (https://www.mbed.com/en/about-mbed/contributor-license-agreements/), which can be signed and returned to us, and is applicable if you don't want to create an mbed account or alternatively if this is a corporate contribution.
Thanks for your understanding and again, thanks for the contribution!

I already did this, but my mbed account was not linked to my github one yet (which I just did), could that have been the problem?

@sbutcher-arm PING :ping_pong:

@sbutcher-arm can you by any chance confirm my CLA status and merge this?

Hi @jorisa. Can you please confirm that this is your mbed account?

Yes that is my mbed account.

Thanks for confirming!

Approved by me, but needs a second reviewer, so assigning to @gilles-peskine-arm.
@gilles-peskine-arm - once (or if) you approve, please mark as 'ready to merge'.


Clustering of a variable returns a bad error message

See https://cartodb.github.io/carto-vl/examples/editor/index.html#eyJhIjoibWFkcmlkX2xpc3RpbmdzICIsImIiOiIiLCJjIjoiY2FydG92bCIsImQiOiJodHRwczovL3t1c2VyfS5jYXJ0by5jb20iLCJlIjoiICAgIEBjYXRlZ29yaWVzOiB2aWV3cG9ydEhpc3RvZ3JhbSgkbmVpZ2hib3VyaG9vZF9ncm91cCwgMSwgMTIpLFxuICAgIEByb29tVHlwZTogKCRyb29tX3R5cGUpIFxuICAgIEBhdmVyYWdlUHJpY2U6IHZpZXdwb3J0QXZnKCRwcmljZSlcbiAgICBcbiAgICB3aWR0aDogNSwgIFxuICAgIGNvbG9yOiByYW1wKGNsdXN0ZXJNb2RlKEByb29tVHlwZSksIFBSSVNNKSxcbiAgICBzdHJva2VXaWR0aDogMC4xLCBcbiAgICAiLCJmIjp7ImxuZyI6LTMuNzAyODYyNjMzOTc0MTc3NywibGF0Ijo0MC40MTc5Mjc2NDIzNDM0N30sImciOjEyLjY3MTMxNTgxMTY2MTM1NSwiaCI6IkRhcmtNYXR0ZXIiLCJpIjoiZGF0YXNldCJ9
Keep in mind that this is just a lack of a proper check and error throwing, the case cannot be supported logically because variables must be accessible from JS and you cannot mix clustered and unclustered accesses to the same property
As reported by @ramiroaznar

If instead of using the variable, you use the column again:
@categories: viewportHistogram($neighbourhood_group, 1, 12)
@roomType: $room_type
@averagePrice: viewportAvg($price)

width: 5,
color: ramp(clusterMode($room_type), PRISM)
strokeWidth: 0.1

You get the right error message: CartoValidationError: [Incorrect value]: Incompatible combination of cluster aggregation usages ([{"type":"aggregated","op":"mode"}]) with unaggregated usage for property 'room_type'.

Closing cause it's been fixed in this PR https://github.com/CartoDB/carto-vl/pull/1281


Implement vote program

https://docs.solana.com/apps/builtins#vote-program

Going to close this for now and re-open if someone requests for this.


ResizableElement: touch support + relative width

[ ] Touch support
[x] Support for relative sizing css units, like rem, for min- and maxwidth

I'm closing this issue as ResizableElement is not being used in a mobile view, thus touch support is not needed for now.


update sd main function

Modified the main function, the stable_diffusion.py now can accept four arguments given by user: number of iteration, number of prompts, comment and previous prompts.


GetMedia needs auth

I think all calls need auth these days: See
https://www.instagram.com/developer/endpoints/
The Instagram API requires an access_token from authenticated users for each endpoint. We no longer support making requests using just the client_id.
This change fixed this for the getmedia call, but I think in the new situation auth is always true. If you agree I can create a pull req to remove the auth option (as it will always be true).

Thank you! Looks like I will update package to sign all requests by default

Thanks for the super quick merge! :)


[SPARK-18364][YARN] Expose metrics for YarnShuffleService

Produces metrics that look like this:
[user@host ~]$ curl -sk -XGET https://`hostname -f`:8042/jmx | jq . | grep 'shuffleservice' -B 1 -A 18
    {
      "name": "Hadoop:service=NodeManager,name=shuffleservice",
      "modelerType": "shuffleservice",
      "tag.Hostname": "<redacted>",
      "openBlockRequestLatencyMillis_count": 1,
      "openBlockRequestLatencyMillis_rate15": 0.0011080303990206543,
      "openBlockRequestLatencyMillis_rate5": 0.0033057092356765017,
      "openBlockRequestLatencyMillis_rate1": 0.015991117074135343,
      "openBlockRequestLatencyMillis_rateMean": 0.003843993699021382,
      "blockTransferRateBytes_count": 118,
      "blockTransferRateBytes_rate15": 0.1307475870844372,
      "blockTransferRateBytes_rate5": 0.39007368980982715,
      "blockTransferRateBytes_rate1": 1.8869518147479705,
      "blockTransferRateBytes_rateMean": 0.45359183094454836,
      "registeredExecutorsSize": 2,
      "registerExecutorRequestLatencyMillis_count": 2,
      "registerExecutorRequestLatencyMillis_rate15": 0.001697343764758814,
      "registerExecutorRequestLatencyMillis_rate5": 0.002970701813078509,
      "registerExecutorRequestLatencyMillis_rate1": 0.0005857750515146702,
      "registerExecutorRequestLatencyMillis_rateMean": 0.007687995987242345
    },
[user@host ~]$

I'd happily get rid of these two lines if anyone has suggestions for how to do that:
      "modelerType": "shuffleservice",
      "tag.Hostname": "<redacted>",

I'd like for other Palantirians to review this and then send it upstream.

Addressed comments in new commits

@robert3005 @mccheah good to send upstream now?
Happy to merge once the build passes also so we can start rolling this out

Yeah, I think this is as good as it can be

Upstreamed with link at https://issues.apache.org/jira/browse/SPARK-18364

Tests flaked with GC limit exceeded, restarted..


Read EXR files from memory

Is your feature request related to a problem? Please describe.
Some file formats like USD can read textures as EXR file format. For reading them properly, we need to read the file from memory.
Describe the solution you'd like
Handle SetMemoryBuffer in the vtkF3DEXRReader

I can work on this issue.

Nice!

@yigitako any news on this ?

Any news on this feature @yigitako ?

with no feedback, I'm unassainging this @yigitako

@mwestphal I am starting to look into this

hey @nabielkandiel , what is the status of this ?

@gapry it's almost done here: https://github.com/f3d-app/f3d/pull/1448
The previous contributor doesn't seem active anymore, so feel free to take a look and let us know if you want to take over and finalize it.

@gapry any news on this ?

No worries!

How can I contribute my changes to the original pull request? Thanks

The original branch is not in our (the maintainers) control, so the second approach is indeed correct. Once you have created a new PR with the code, I'll close the original PR and development will continue  in your branch.


Mixed dtype: concatenate

Concatenation of arrays of different dtypes are not supported in ChainerX.

Should not the stack function be fixed as well? 🤔


rpiCamera source: encoder_encode(): ioctl() failed

Which version are you using?
v0.20.0
Which operating system are you using?

[ ] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[x] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
After a while, encoder_encode(): ioctl() failed will appear, and the stream will stop working. rtsp-simple-server is still running, but the stream will never recover. Restarting rtsp-simple-server makes the stream work again until the next ioctl() failed.
Describe how to replicate the issue

I don't think it is relevant that I am using HLS, but this is the configuration that I have used.

Start with default config. Enable HLS and disable other protocols.
Add the following path to the configuration file.

stream:
    source: rpiCamera
    rpiCameraWidth: 1920
    rpiCameraHeight: 1080
    rpiCameraFPS: 30
    rpiCameraIDRPeriod: 10
    rpiCameraProfile: main
    rpiCameraLevel: '4.1'
    rpiCameraBitrate: 8000000

Play the stream in a browser at http://<ip>:8888/stream.

Did you attach the server logs?

no
Did you attach a network dump?

no

Looking at the code, I can see that it happens in encoder.c:310 when trying to enqueue a buffer. It would be nice if rtsp-simple-server tried to recover instead of just exiting, which results in the entire process needing to be restarted to start the stream again.

Hello, i'm willing to add a recovery mechanism to the server, but from my experience ioctl errors appear only when the Raspberry Pi has issues related to power supply or faulty links between the board and the camera, and in this case trying to perform a software recovery is useless, you have to reboot the device.
Are you sure that the error happens normally and is not related to faulty hardware?

I can't rule out that it's hardware related, but I can say for sure that when it happens, restarting rtsp-simple-server works as a method of recovery. My best guess is that is is related to noise, since it seems to happen more often when multiple Raspberry Pis with cameras attached are placed next to each other.
In libcamera, ioctl() calls are wrapped in xioctl that retries 10 times. Maybe that's a possible (partial) solution? But I think rtsp-simple-server should also try to restart the camera process.

Example of restart of rtsp-simple-server working:
pi@picam:~/rtsp-release $ ./rtsp-simple-server
[22:02:19.837038036] [3271]  INFO Camera camera_manager.cpp:293 libcamera v0.0.0+3730-67300b62
[22:02:19.877092398] [3272]  WARN CameraSensorProperties camera_sensor_properties.cpp:174 No static properties available for 'imx519'
[22:02:19.877138656] [3272]  WARN CameraSensorProperties camera_sensor_properties.cpp:176 Please consider updating the camera sensor properties database
[22:02:19.901427028] [3272]  WARN RPI raspberrypi.cpp:1274 Mismatch between Unicam and CamHelper for embedded data usage!
[22:02:19.901913845] [3272] ERROR DelayedControls delayed_controls.cpp:87 Delay request for control id 0x009a090a but control is not exposed by device /dev/v4l-subdev0
[22:02:19.902157170] [3272]  INFO RPI raspberrypi.cpp:1398 Registered camera /base/soc/i2c0mux/i2c@1/imx519@1a to Unicam device /dev/media3 and ISP device /dev/media0
[22:02:19.903009012] [3271]  INFO Camera camera.cpp:1029 configuring streams: (0) 1920x1080-YUV420 (1) 3840x2160-SGBRG10_CSI2P
[22:02:19.903360649] [3272]  INFO RPI raspberrypi.cpp:763 Sensor: /base/soc/i2c0mux/i2c@1/imx519@1a - Selected sensor format: 3840x2160-SGBRG10_1X10 - Selected unicam format: 3840x2160-pGAA
encoder_encode(): ioctl(VIDIOC_QBUF) failed
^Cpi@picam:~/rtsp-release $ ./rtsp-simple-server
[22:55:01.630182692] [3387]  INFO Camera camera_manager.cpp:293 libcamera v0.0.0+3730-67300b62
[22:55:01.673640553] [3388]  WARN CameraSensorProperties camera_sensor_properties.cpp:174 No static properties available for 'imx519'
[22:55:01.673687534] [3388]  WARN CameraSensorProperties camera_sensor_properties.cpp:176 Please consider updating the camera sensor properties database
[22:55:01.702473714] [3388]  WARN RPI raspberrypi.cpp:1274 Mismatch between Unicam and CamHelper for embedded data usage!
[22:55:01.702981453] [3388] ERROR DelayedControls delayed_controls.cpp:87 Delay request for control id 0x009a090a but control is not exposed by device /dev/v4l-subdev0
[22:55:01.703245489] [3388]  INFO RPI raspberrypi.cpp:1398 Registered camera /base/soc/i2c0mux/i2c@1/imx519@1a to Unicam device /dev/media3 and ISP device /dev/media0
[22:55:01.704083911] [3387]  INFO Camera camera.cpp:1029 configuring streams: (0) 1920x1080-YUV420 (1) 3840x2160-SGBRG10_CSI2P
[22:55:01.704494076] [3388]  INFO RPI raspberrypi.cpp:763 Sensor: /base/soc/i2c0mux/i2c@1/imx519@1a - Selected sensor format: 3840x2160-SGBRG10_1X10 - Selected unicam format: 3840x2160-pGAA

After encoder_encode(): ioctl(VIDIOC_QBUF) failed no more segments were added to stream.m3u8 (of course). After stopping and starting rtsp-simple-server, segments are produced as normal.

I also encountered this issue randomly, the problem is that it's really difficult to replicate. We can either increase buffer_count or add a wrapper around ioctl(), i'm trying to test both solutions.

Since v0.20.2, the camera reader doesn't freeze anymore in case of ioctl() errors. If this doesn't solve the problem, feel free to reopen the issue.

@giano574 that ioctl() call tries to put a frame buffer into the encoder queue. It usually fails when the encoder queue is full or there's not enough memory. It doesn't allocate anything, and generally, there's nothing allocated dynamically in the camera module, everything is allocated once, during startup.
The memory probably explodes due to the fact that video is corrupt after the ioctl() fail and HLS segments grow in size until they reach hlsSegmentMaxSize.
In order to debug memory leaks i usually set pprof: yes in the configuration file and then generate a report of the memory usage with
docker run --rm -it --network=host golang:1.15 go tool pprof -text http://localhost:9999/debug/pprof/goroutine

I haven't been able to reproduce it lately, and I also don't know what to look for the the pprof reports. Do you think anything can be done in order for the memory usage to not increase when this happens?

yes, decrease hlsSegmentMaxSize

Maybe I don't understand, but why does the segment grow in size when queueing fails? Should the frames that couldn't be encoded not just be ignored?
And what will happen if hlsSegmentMaxSize is exceeded? Will the segment be completed? Discarded? Something else?

segments grow in size since segments are splitted by using IDR frames, as a segment must contain at least a IDR frame. If the stream gets corrupted, IDR frames are lost and segments grow until they reach hlsSegmentMaxSize, then they are forcefully completed.

Ah, I see. So the actual frames that aren't encoded are just lost, but then no new IDR's are added to the segment? So non-IDR frames are dequeued and put into the segment until the maximum size is reached? Does this mean that no new IDR frames will ever be generated again, or can/does it recover?

If I set hlsSegmentMaxSize and hlsSegmentCount so that hlsSegmentMaxSize * hlsSegmentCount < available RAM - some margin then it shouldn't be possible to exhaust the RAM, correct?

This is happening to me with a Pi camera as well. I'm using almost entirely default settings for the Raspberry Pi Camera

This happens to me on my Pi Zero 1 without HLS.  get that it's a low-memory device. In order to try to adjust, I'd like to ask: is it system or GPU RAM that affects this issue? Based on the answer, I'll reduce/expand the memory allocated to GPU to try to resolve.

@alexfornuto from my experience it's the system RAM. The camera also need the GPU RAM but in a fixed quantity, if you perform a quick search you'll find the minimum GPU needed to start the camera. Use that and give the remaining RAM to the system.


Don't support ESlint v9.x

log as below:
Error: Could not warm up worker. Formatting a file for the first time may take longer than usual.
Message: Invalid Options:
- Unknown options: ignorePath, resolvePluginsRelativeTo, rulePaths, useEslintrc
- 'resolvePluginsRelativeTo' has been removed.
- 'ignorePath' has been removed.
- 'rulePaths' has been removed. Please define your rules using plugins.
Stacktrace: Error: Invalid Options:
- Unknown options: ignorePath, resolvePluginsRelativeTo, rulePaths, useEslintrc
- 'resolvePluginsRelativeTo' has been removed.
- 'ignorePath' has been removed.
- 'rulePaths' has been removed. Please define your rules using plugins.
    at processOptions (e:\eslint-test\node_modules\eslint\lib\eslint\eslint-helpers.js:853:15)
    at new ESLint (e:\eslint-test\node_modules\eslint\lib\eslint\eslint.js:688:34)
    at getESLint (e:\eslint-test\node_modules\prettier-eslint\dist\utils.js:337:12)
    at getESLintConfig (e:\eslint-test\node_modules\prettier-eslint\dist\index.js:268:39)
    at analyze (e:\eslint-test\node_modules\prettier-eslint\dist\index.js:80:77)
    at format (e:\eslint-test\node_modules\prettier-eslint\dist\index.js:50:13)

Bump?

+1

+1

For those still having issues it might be worth looking into
https://github.com/idahogurl/vs-code-prettier-eslint/issues/222

:(

same


The newest version doesn't compile on Windows.

Hey guys, I have all the dependencies installed. and all the paths set so that these things are found..

yarn
rimraf
mage
npm
go

Yarn build however doesnt compile and throws a Type A not assignable to Type B error on my windows machine..
PS C:\Users\kib\Desktop\Projects\mqtt datasource for grafana\grafana-plugins\mqtt-datasource> yarn build
yarn run v1.22.15
warning ..\..\..\..\..\package.json: No license field
$ rm -rf dist && grafana-toolkit plugin:build && mage build:backend
'rm' is not recognized as an internal or external command,
operable program or batch file.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
PS C:\Users\kib\Desktop\Projects\mqtt datasource for grafana\grafana-plugins\mqtt-datasource> yarn build
yarn run v1.22.15
warning ..\..\..\..\..\package.json: No license field
$ rimraf dist && grafana-toolkit plugin:build && mage build:backend
  Using Node.js v16.5.0
  Using @grafana/toolkit v8.2.0-beta.2
√ Preparing
√ Linting
 PASS  src/handleEvent.test.ts

Test Suites: 1 passed, 1 total
Tests:       2 passed, 2 total
Snapshots:   2 passed, 2 total
Time:        5.271 s
Ran all test suites with tests matching "".
√ Running tests
- Compiling...  Starting type checking service...
  Using 1 worker with 2048MB memory limit
- Compiling...  ERROR in C:/Users/kib/Desktop/Projects/mqtt datasource for grafana/grafana-plugins/mqtt-datasource/src/ConfigEditor.tsx(53,17):
  TS2322: Type '{ name: string; required: true; value: string; css: string; autoComplete: string; onChange: (event: ChangeEvent<HTMLInputElement>) => void; }' is not assignable to type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
    Property 'css' does not exist on type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
  ERROR in C:/Users/kib/Desktop/Projects/mqtt datasource for grafana/grafana-plugins/mqtt-datasource/src/ConfigEditor.tsx(64,17):
  TS2322: Type '{ type: string; name: string; required: true; value: number; css: string; autoComplete: string; onChange: (event: ChangeEvent<HTMLInputElement>) => void; }' is not assignable to type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
    Property 'css' does not exist on type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
  ERROR in C:/Users/kib/Desktop/Projects/mqtt datasource for grafana/grafana-plugins/mqtt-datasource/src/ConfigEditor.tsx(76,17):
  TS2322: Type '{ name: string; value: string | undefined; css: string; autoComplete: string; onChange: (event: ChangeEvent<HTMLInputElement>) => void; }' is not assignable to type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
    Property 'css' does not exist on type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
  ERROR in C:/Users/kib/Desktop/Projects/mqtt datasource for grafana/grafana-plugins/mqtt-datasource/src/ConfigEditor.tsx(85,17):
  TS2322: Type '{ type: string; name: string; css: string; autoComplete: string; placeholder: string; value: string; onChange: (event: ChangeEvent<HTMLInputElement>) => void; onReset: () => void; }' is not assignable to type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
    Property 'css' does not exist on type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
  ERROR in C:/Users/kib/Desktop/Projects/mqtt datasource for grafana/grafana-plugins/mqtt-datasource/src/QueryEditor.tsx(22,13):
  TS2322: Type '{ name: string; required: true; value: string | undefined; css: string; autoComplete: string; onChange: (event: ChangeEvent<HTMLInputElement>) => void; }' is not assignable to type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
    Property 'css' does not exist on type 'IntrinsicAttributes & Pick<Props, "onSubmit" | "accept" | "acceptCharset" | "action" | "allowFullScreen" | "allowTransparency" | "alt" | "as" | ... 354 more ... | "addonAfter"> & RefAttributes<...>'.
× Build failed
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Faulty files

src\configEditor.tsx

src\QueryEditor.tsx

Versions

Yarn :  v1.22.15
Go : go1.17.1 windows/amd64
Mage:  Mage Build Tool v1.11.0-4-gdd94424, Commit: dd94424
Node: v16.5.0

@KirannBhavaraju i just tried building the datasource from source on my windows machine with the following version:

yarn: 1.22.17
Go: go version go1.17.6 windows/amd64
Mage: Mage Build Tool v1.12.1; Commit: 2f1ec40
Node: v16.13.1

it compiles fine on my end - do you mind updating your local tools and then trying again? maybe its just a version issue

@atifali Yeah, you use newer versions of every dependency almost. I wrote the issue 3 mo ago.. which was the case then. I did not try it recently..
but nice to see it is fixed.

@KirannBhavaraju cool yeah if you do want to give this another shot i would suggest using those versions and report back if any issues - closing this now


Fastlane Snapshot spans over an eternity before completing

New Issue Checklist

[X] Updated fastlane to the latest version
[X] I read the Contribution Guidelines
[X] I read docs.fastlane.tools
[X] I searched for existing GitHub issues

Issue Description

Fastlane Snapshot takes an eternity to complete capturing screenshots for languages listed in Snapfile. After an increase in Localizable.strings embedded in app/referenced in Xcode project file which also resulted in an increase of the Localizable.strings copied by Fastlane Snapshot (all of them, e.g. 7 languages defined in Xcode => 7 Localizable.strings copied), the time required for the test to complet finish increases drastically. For only two languages with a base of 8 Localizable.strings copied (8 languages in Xcode), the average time for tehuse two languages was about under a hour.
Another observation:
For some reason, it goes copies all Localizable.strings it finds in the project and runs through them after the inititial test for a given language is done, although only the first run has the launch argumetns and capturing of screenshots, while the rest is without and just launching the app, trying to do the test and then failing, rinse and repeat. As such, I'm forced to hardcode specific variables for the test on Fastlane simulators to pass consistently.
Command executed

bundle exec fastlane snapshot --skip_open_summary
Complete output when running fastlane, including the stack trace and command used

Example of start-end of test for one language
[20:39:54]: ▸ Processing Info.plist
[20:39:54]: ▸ Linking Savings
[20:39:54]: ▸ Linking SavingsFastlaneSnapshot
[20:39:54]: ▸ Processing empty-SavingsFastlaneSnapshot.plist
[20:39:54]: ▸ Build Succeeded
[20:39:54]: ▸ --- xcodebuild: WARNING: Using the first of multiple matching destinations:
[20:39:54]: ▸ { platform:iOS Simulator, id:A4B53192-9506-4271-9819-F21B579D7080, OS:16.4, name:iPhone 14 Plus }
[20:39:54]: ▸ { platform:iOS Simulator, id:A4B53192-9506-4271-9819-F21B579D7080, OS:16.4, name:iPhone 14 Plus }
[20:39:55]: ▸ Linking Savings
[20:39:55]: ▸ Linking SavingsFastlaneSnapshot
[21:30:23]: ▸ 2023-07-03 21:30:23.500 xcodebuild[25969:1640190] [MT] IDETestOperationsObserverDebug: 3027.958 elapsed -- Testing started completed.
[21:30:23]: ▸ 2023-07-03 21:30:23.500 xcodebuild[25969:1640190] [MT] IDETestOperationsObserverDebug: 0.000 sec, +0.000 sec -- start
[21:30:23]: ▸ 2023-07-03 21:30:23.500 xcodebuild[25969:1640190] [MT] IDETestOperationsObserverDebug: 3027.958 sec, +3027.958 sec -- end
[21:30:25]: ▸ Test Succeeded
[21:30:28]: Clearing Status Bar Override
+----------------+-------+-------+
|        snapshot results        |
+----------------+-------+-------+
| Device         | fr-FR | it-IT |
+----------------+-------+-------+
| iPhone 14 Plus |  💚   |  💚   |
Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
13.4.1

Ruby
3.2.1

Bundler?
false

Git
git version 2.38.1

Installation Source
~/.rbenv/versions/3.2.1/bin/fastlane

Host
macOS 13.4.1 (22F82)

Ruby Lib Dir
~/.rbenv/versions/3.2.1/lib

OpenSSL Version
OpenSSL 3.0.8 7 Feb 2023

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
14.3.1

Swift Version
5.8.1

System Locale

Variable
Value

LANG
da_DK.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane

def environment_variables
  ENV["SNAPSHOT_SIMULATOR_WAIT_FOR_BOOT_TIMEOUT"] = 15
end

default_platform(:ios)

platform :ios do
  desc "Generate new localized screenshots"
  lane :screenshots do
    #environment_variables
    capture_screenshots(scheme: "SavingsFastlaneSnapshot")
  end
end

`./Appfile`
app_identifier("APP_IDENTIFIER") # The bundle identifier of your app

# For more information about the Appfile, see:
#     https://docs.fastlane.tools/advanced/#appfile

# fastlane snapshot --reinstall_app --app_identifier "APP_IDENTIFIER"

fastlane gems

Gem
Version
Update-Status

fastlane
2.213.0
✅ Up-To-Date

Loaded fastlane plugins:
*No plugins Loaded
Loaded gems

Gem
Version

error_highlight
0.5.1

did_you_mean
1.6.3

syntax_suggest
1.0.2

atomos
0.1.3

rexml
3.2.5

CFPropertyList
3.0.6

claide
1.1.0

colored2
3.1.2

nanaimo
0.3.0

xcodeproj
1.22.0

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.8.0

terminal-table
1.8.0

word_wrap
1.0.0

optparse
0.1.1

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

artifactory
3.0.15

babosa
1.0.4

colored
1.2

highline
2.0.3

commander
4.6.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.5

faraday-cookie_jar
0.0.7

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-httpclient
1.0.1

faraday-multipart
1.0.4

faraday-net_http
1.0.1

faraday-net_http_persistent
1.2.0

faraday-patron
1.0.0

faraday-rack
1.0.0

faraday-retry
1.0.3

ruby2_keywords
0.0.5

faraday
1.10.3

faraday_middleware
1.2.0

gh_inspector
1.1.3

json
2.6.3

mini_magick
4.12.0

naturally
2.2.1

rubyzip
2.3.2

security
0.1.3

xcpretty-travis-formatter
1.0.1

dotenv
2.8.1

simctl
1.6.10

google-cloud-env
1.6.0

google-cloud-core
1.6.0

google-apis-storage_v1
0.19.0

rake
13.0.6

digest-crc
0.6.4

mini_mime
1.1.2

google-cloud-storage
1.44.0

emoji_regex
3.2.3

set
1.0.3

plist
3.7.0

public_suffix
5.0.1

addressable
2.8.4

multipart-post
2.3.0

excon
0.100.0

fastimage
2.2.7

bundler
2.4.6

jwt
2.7.1

webrick
1.8.1

httpclient
2.8.3

multi_json
1.15.0

signet
0.17.0

os
1.1.4

memoist
0.16.2

googleauth
1.6.0

retriable
3.1.2

trailblazer-option
0.1.2

declarative
0.0.20

uber
0.1.0

representable
3.2.0

google-apis-core
0.11.0

google-apis-playcustomapp_v1
0.13.0

google-apis-androidpublisher_v3
0.44.0

google-cloud-errors
1.3.1

google-apis-iamcredentials_v1
0.17.0

aws-eventstream
1.2.0

aws-sigv4
1.5.2

aws-partitions
1.781.0

jmespath
1.6.2

aws-sdk-core
3.175.0

aws-sdk-kms
1.67.0

aws-sdk-s3
1.126.0

forwardable
1.3.3

logger
1.5.3

pathname
0.2.1

shellwords
0.1.0

cgi
0.3.6

date
3.3.3

timeout
0.3.1

securerandom
0.2.2

uri
0.12.0

openssl
3.1.0

digest
3.1.1

ipaddr
1.2.5

resolv
0.2.2

time
0.2.1

stringio
3.0.4

open-uri
0.3.0

mutex_m
0.1.2

net-http
0.3.2

net-protocol
0.2.1

ostruct
0.5.5

english
0.7.2

erb
4.0.2

abbrev
0.1.1

tempfile
0.1.3

delegate
0.3.0

fileutils
1.7.0

tmpdir
0.1.3

base64
0.1.1

singleton
0.1.1

open3
0.1.2

nkf
0.1.2

racc
1.6.2

nokogiri
1.14.2

prettyprint
0.1.1

pp
0.4.0

find
0.1.1

yaml
0.2.1

psych
5.0.1

generated on: 2023-07-08

Update: After running for a little under 24 hours, it has only finished 4 out of 30 languages.


It is not letting me deploy, it is giving an error every time

Describe the bug
I am not sure what is giving the error, this is what it is shown in the prompt area:

The development server is running, and you can now test the new features in the Alerts tab.
Rollback to Checkpoint
Deploy this application.
I'll help you deploy the application to Netlify.
Deploy to Netlify
netlify
The application is being deployed to Netlify. Let me check the deployment status.
Something went wrong while creating your site on Netlify.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-fdawdj
Steps to reproduce

Expected behavior
I expected it to deploy, allow me to claim and view
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 50335817,
  "usedJSHeapSize": 46617305,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = SantiagoH
Chat ID = a2ec0ead3f3f

Additional context
Error:
Something went wrong while creating your site on Netlify.

@SanPlayground thanks for the report, seems like something is up with our Netlify endpoint! I have escalated to engineering and updates will come in issue #1483

Update: We have identified the core issue and fixed it. Happy coding!


Spatial lock status not saved

Godot version:
3.1.1
OS/device including version:
N/A
Issue description:
I tried to make a MeshInstance "locked" so that I could not move it accidentally. When I reopen the scene, the node is again unlocked. Git doesn't show any changes related to locking.
Might be important: the mesh instance is inherited from a base scene (the whole .tscn was created as 'make new inherited' from an imported .dae file)
Steps to reproduce:
Attempt to mark a mesh instance as locked.
Minimal reproduction project:

Related to #12990 and #12838

This is basically the same as #12838, so closing.


Register provider via policy For Kusto.

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run java - kusto - ci

API change check
API changes are not detected in this pull request.

/azp run java - kusto - mgmt - tests


'DetDataSample' object has no attribute '_gt_sem_seg'

could someone help me ? how to slove this problem?

could someone help me ? how to slove this problem?

Please follow the issue template to provide more information such as what command did you run and what config did you use?


Using context manager to control external processes

Adds possibility to work on different port numbers, and add a better control of processes removing the need of pkill  View pull request

Hello my friends ... I need help ... Please .. I need to penetrate Emil but I did not appreciate .... Can you help me


[BUG]Need to clean failed to scheduled extra replica if there is no eviction.

Describe the bug
A clear and concise description of what the bug is.
After increase the replica count number and decrease it back, the extra failed to scheduled replica stays in longhorn. We need to clean it up.
To Reproduce
Steps to reproduce the behavior:

Deploy a longhorn with 3 nodes in k8s cluster.
Create a volume with 3 replicas, everything should be done correctly.
Increase the replica number on this volume to 4. There should be a failed scheduled replica.
Decrease the replica number to 3, the extra failed scheduled replica stays there.

Expected behavior
The extra failed scheduled replica should be cleaned up. If there is no eviction on the disks or nodes.
Log
[longhorn-manager-5m4wj] time="2020-08-13T00:22:25Z" level=error msg="There's no available disk for replica test-r-cae0f985, size<PHONE_NUMBER>0"
[longhorn-manager-5m4wj] time="2020-08-13T00:22:25Z" level=error msg="unable to schedule replica test-r-cae0f985 of volume test"
[longhorn-manager-5m4wj] time="2020-08-13T00:22:25Z" level=info msg="Event(v1.ObjectReference{Kind:\"Volume\", Namespace:\"longhorn-system\", Name:\"test\", UID:\"e30f8fa2-2ec8-464e-9310-a83ed1d1c25f\", APIVersion:\"longhorn.io/v1beta1\", ResourceVersion:\"12395182\", FieldPath:\"\"}): type: 'Warning' reason: 'FailedEviction' Can not find resource for scheduling replica test-r-cae0f985."
[longhorn-manager-566fn] time="2020-08-13T00:22:25Z" level=debug msg="Updated volume test replica count from 4 to 3"

Environment:

Longhorn version:v1.0.1
Kubernetes version:v1.17.5
Node OS type and version:Ubuntu18.04

Additional context
The cleanup should be happen when there is no eviction, meaning the eviction has been cancelled if there is a previous one.
This is also related to 298 for auto-cleanup.
https://github.com/longhorn/longhorn/issues/298

Verified on Longhorn master - 09/10/2020
Validation - Pass
The failed replica is getting clean up.
Steps followed:

Deploy a longhorn with 3 nodes in the k8s cluster.
Create a volume with 3 replicas.
Increase the replica number on this volume to 4. There should be a failed scheduled replica.
Decrease the replica number to 3, the extra failed scheduled replica stays there.


Region pileup plots

It would be useful to have a script that can generate plots based on the pileups, similar to the plot in the supplementary figure here: https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5638069/
Probably best to implement this as a separate R script rather than making this part of the pipeline. Otherwise we'll have to add R as a requirement.

There is now code for a basic version of this in the FGFR2 tutorial: https://github.com/koelling/amplimap/blob/master/tutorials/tutorial_FGFR2.ipynb


Fix js typo (closing square brackets)

Simple typo fix in documentation. Close square brackets.

Thank you so much for your help! 😃


ENH: adding list of packages to exclude from cleanup

This is to add a file to host the agreed upon exceptions for the cleanups. Neither the number version nor the date based cleanup will run for these.
CI is expected to fail as
Fixes #34
cc @mattip

The reason the CI is failing is described in https://github.com/scientific-python/upload-nightly-action/pull/32#issuecomment-1678400638

Perhaps the retention policy should be mentioned in the README.md as well?

Given the one approval, and the discussion on the rest, I go ahead and merge this now.


Corrected version numbering for Django support

Corrected version numbers for Django support, as suggested in https://github.com/cockroachdb/docs/pull/6521#issuecomment-624269747.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/0efe08a44bac8e3d3584cb0ffb58b17a5721ddf3/
Edited pages:

v19.2/build-a-python-app-with-cockroachdb-django.md
v20.1/build-a-python-app-with-cockroachdb-django.md

There are mentions of "django 2" and django-cockroachdb 2 which should be 2.2.
You might want to edit python3 -m pip install django and python3 -m pip install django-cockroachdb to specify a version number like django-cockroachdb==3.0.* so that the instructions work if there's a slight delay between the release of Django 3.1 and django-cockroachdb 3.1.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/eb8d09c75d7a04c871617bab9a4762ecadb4da8b/
Edited pages:

v19.2/build-a-python-app-with-cockroachdb-django.md
v19.2/install-client-drivers.md
v20.1/build-a-python-app-with-cockroachdb-django.md
v20.1/install-client-drivers.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/faa9e270dddd93990571df4ea87952e65669a81f/
Edited pages:

v19.2/build-a-python-app-with-cockroachdb-django.md
v19.2/install-client-drivers.md
v20.1/build-a-python-app-with-cockroachdb-django.md
v20.1/install-client-drivers.md


ROX-20664: Compliance config cluster selection

Description
Adds the cluster selection step in the compliance scan configuration wizard.

Clusters are fetched in the ScanConfigWizardForm component to prevent unnecessary fetches every time the cluster selection step is mounted
Formik is used for managing selections across different steps
Cluster permissions are checked to verify user has read access for viewing/selecting clusters, as well as write access if there are no clusters installed
The useEffect inside ClusterSelection is required to make sure formik state is synchronized with user selections

Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Verified the following:

Formik state is updated when a user selects a single cluster as well as the select all
Clusters are preselected when navigating to/from different wizard steps
Clusters don't display if user doesn't have read access
Empty template state is display with call to action if there are no clusters

/retest-required

/retest


cmd/go: go get go 1.13 cannot fetch vX.Y.Z if tag X.Y.Z is defined

What version of Go are you using (go version)?

$ go version
go version go1.13beta1 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/andersonqueiroz/Library/Caches/go-build"
GOENV="/Users/andersonqueiroz/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/andersonqueiroz/devel/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/andersonqueiroz/sdk/go1.13beta1"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/andersonqueiroz/sdk/go1.13beta1/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/tmp/test/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/47/f0fcm9lx2f7bs36r73tnmlpc0000gn/T/go-build668830074=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
In an empty folder:

go1.13beta1 mod init bla
go1.13beta1 get<EMAIL_ADDRESS>

What did you expect to see?
the module been added to go.mod:

go get<EMAIL_ADDRESS>
Using go version go1.12.6 darwin/amd64 it just works. I know it's odd to a repo to have both tags 1.1.0 and v1.1.0. But ~for no good reason~ we have in my company and as I was trying go 1.13 beta1 I run into this problem
I created the repo AndersonQ/go1_13_mod_test to be able to reproduce it with a public repo.
What did you see instead?

$ go1.13beta1 get<EMAIL_ADDRESS>                                                                                ‹ruby-2.4.1›
verifying<EMAIL_ADDRESS><EMAIL_ADDRESS>reading<EMAIL_ADDRESS>410 Gone

However if I use 1.1.0 (go1.13beta1 get github.com/AndersonQ/go1_13_mod_test@1.1.0), it works adding the dependency with pseudo-version
cat go.mod Output
module bla
go 1.13
require github.com/AndersonQ/go1_13_mod_test v0.0.0-20190701100952-ced54b8ce3d7 // indirect

I am unable to reproduce this.
$cd `mktemp -d`
$go1.13beta1 mod init bla
$go1.13beta1 get<EMAIL_ADDRESS>go: finding github.com/AndersonQ/go1_13_mod_test v1.1.0
go: downloading github.com/AndersonQ/go1_13_mod_test v1.1.0
go: extracting github.com/AndersonQ/go1_13_mod_test v1.1.0
$cat go.mod
module bla

go 1.13

require github.com/AndersonQ/go1_13_mod_test v1.1.0 // indirect

@agnivade indeed, even I cannot reproduce it now. However it still happening on my private repo. I'll investigate more.

You mentioned this was a private repo, but the error message is from sumdb.golang.org. The Go module mirror and sum database won't be able to access this repo if it's not publicly visible. Could you try setting GOPRIVATE to exclude this repo? For example, export GOPRIVATE=github.com/AndersonQ/*.
In general, sumdb.golang.org may serve a 410 if it can't load the module at that version. There are many reasons that could happen (unfortunately, we don't have very descriptive error messages yet). You might also try temporarily setting GOPROXY=direct and GOSUMDB=off if that gives a clearer message.

@jayconrod GOPRIVATE=github.com/My_ORG/* did the trick!
So I believe my problem was reproducible earlier today because Go module mirror and Go checksum database had not indexed my example repo yet. Therefore go get behaved the same way it behaves with my private one.
Thanks guys!


Add interactive integration test for personalizing configuration with odo init

/area testing
There are no tests at the moment testing configuration personalization while using odo init.
$ odo init
  __
 /  \__     Initializing a new component
 \__/  \    Files: Source code detected, a Devfile will be determined based upon source code autodetection
 /  \__/    odo version: v3.9.0
 \__/

Interactive mode enabled, please answer the following questions:
Based on the files in the current directory odo detected
Language: JavaScript
Project type: Node.js
The devfile "nodejs:2.1.1" from the registry "StagingDevfileRegistry" will be downloaded.
? Is this correct? Yes
 ✓  Downloading devfile "nodejs:2.1.1" from registry "StagingDevfileRegistry" [3s]

↪ Container Configuration "runtime":
  OPEN PORTS:
    - 3000
    - 5858
  ENVIRONMENT VARIABLES:
    - DEBUG_PORT = 5858

? Select container for which you want to change configuration? runtime
? What configuration do you want change? Delete environment variable "DEBUG_PORT"
? What configuration do you want change? Add new environment variable
? Enter new environment variable name: DEBUG_PROJECT_PORT
? Enter value for "DEBUG_PROJECT_PORT" environment variable: 5858
? What configuration do you want change? NOTHING - configuration is correct

↪ Container Configuration "runtime":
  OPEN PORTS:
    - 3000
    - 5858
  ENVIRONMENT VARIABLES:
    - DEBUG_PROJECT_PORT = 5858

? Select container for which you want to change configuration? NONE - configuration is correct
? Enter component name: nodejs-starter

You can automate this command by executing:
   odo init --name nodejs-starter --devfile nodejs --devfile-registry StagingDevfileRegistry --devfile-version 2.1.1

Your new component 'nodejs-starter' is ready in the current directory.
To start editing your component, use 'odo dev' and open this folder in your favorite IDE.
Changes will be directly reflected on the cluster.

Also test this for adding/deleting ports.

@ritudes would you be interested in working on this? It seems like a good candidate to me for you.

Progress so far:
Ran the command and checked for the necessary test cases to be added on interactive_init_test.go file.


League Observer Tool not connecting

I am trying to connect to the observer tool I have LoL client open and everything, but it does not connect. I think it could be the server API key I used. For it I used the riot RGAP api key.

Can you please try to turn off the authentication at the prod toolkit? And if that fixes the issue, it is caused by the server API key.

I'm having a similar issue, idk if I'm doing anything wrong

Could you confirm you have a correct Toolkit key configured (starts with "RCVPT-")?
Does it work with the default admin key?
If you are only running the toolkit locally, you should be fine to turn of authentication anyways :)


chore: enable prereleases for 2022-04-release branch

In order to allow CI to release pre-releases for 2022-04-release, we need to merge this into master branch.
Part of https://github.com/asyncapi/spec/issues/735

/rtm


fix options table in quickstart tut

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add cxx animation example

@ajpmaclean : Please review.

The example never stops. Please have it stop after 360 degrees.
Remove #include <vtkAxesActor.h> and any other unsed includes.
Make the window a little bigger, say 640, 480.
Tnanks

Also in VTKExamples/src/Cxx there is a file called .clang-format that specifies the formatting for the Kitware standard formatting for C++. If you don't know how to use clang-format or it is not built in to your editor, I can fix the formatting once it is merged.

@lorensen . @ajpmaclean:

Sphere Rotation stops.
Unused Includes are removed
Window size changed to 640,480
Unwanted comments are removed.

I have enabled the built in clang-format in my editor.  but am not sure whether the formatting is as expected.
Please review.

For some reason your indentation seems to be 8 spaces instead of two spaces, so the formatting is not correct, I don' t think it picked up the .clang-format file. Please refer to Guidelines and look at existing examples.

@ajpmaclean :
I'll try again to fix the indentation issue.

Is this fine now ?

Why did you add #include <vtkAnimationScene.h>? I don't see it being used anywhere in the code.
Your formatting is still not correct, some of your lines are too long. You can download clang-format and use the supplied .clang-format to easily format your code if your editor is not supporting it. See Clang 9 documentation for how to use it. It is worth setting up!

Please review now. I've followed your inputs.

You need to re-run ctest as the images are different.
Image differencing failed to produce an image because images are different size:
Valid image: 230, 229, 1
Test image: 640, 480, 1
<DartMeasurement name="WallTime" type="numeric/double">0.35</DartMeasurement>
<DartMeasurement name="CPUTime" type="numeric/double">0.351</DartMeasurement>
<end of output>
Test time =   5.62 sec

There may be a problem with your formatting, when I run clang-format -i RotatingSphere.cxx I get:
#include <vtkActor.h>
#include <vtkNamedColors.h>
#include <vtkPolyDataMapper.h>
#include <vtkProperty.h>
#include <vtkRenderWindow.h>
#include <vtkRenderWindowInteractor.h>
#include <vtkRenderer.h>
#include <vtkSmartPointer.h>
#include <vtkSphereSource.h>

class vtkTimerCallback2 : public vtkCommand
{
public:
  int timerId;
  static vtkTimerCallback2* New()
  {
    vtkTimerCallback2* cb = new vtkTimerCallback2;
    cb->TimerCount = 0;
    return cb;
  }
  virtual void Execute(vtkObject* caller, unsigned long eventId,
                       void* vtkNotUsed(callData))
  {
    vtkRenderWindowInteractor* iren =
        dynamic_cast<vtkRenderWindowInteractor*>(caller);
    if (vtkCommand::TimerEvent == eventId)
    {
      ++this->TimerCount;
    }
    if (TimerCount < 36)
    {
      actor->RotateZ(5);
      iren->GetRenderWindow()->Render();
    }
    else
    {
      iren->DestroyTimer();
    }
  }

private:
  int TimerCount;

public:
  vtkActor* actor;
};

int main(int, char* [])
{
  vtkSmartPointer<vtkNamedColors> colors =
      vtkSmartPointer<vtkNamedColors>::New();

  // Create a sphere
  vtkSmartPointer<vtkSphereSource> sphereSource =
      vtkSmartPointer<vtkSphereSource>::New();
  sphereSource->SetCenter(0.0, 0.0, 0.0);
  sphereSource->SetRadius(1.0);
  sphereSource->SetThetaResolution(15);
  sphereSource->SetPhiResolution(15);
  sphereSource->Update();

  // Create a mapper and actor
  vtkSmartPointer<vtkPolyDataMapper> mapper =
      vtkSmartPointer<vtkPolyDataMapper>::New();
  mapper->SetInputConnection(sphereSource->GetOutputPort());
  vtkSmartPointer<vtkActor> actor = vtkSmartPointer<vtkActor>::New();
  actor->SetMapper(mapper);
  actor->RotateX(90);
  actor->GetProperty()->SetRepresentationToWireframe();

  // Create a renderer, render window, and interactor
  vtkSmartPointer<vtkRenderer> renderer = vtkSmartPointer<vtkRenderer>::New();
  vtkSmartPointer<vtkRenderWindow> renderWindow =
      vtkSmartPointer<vtkRenderWindow>::New();
  renderWindow->AddRenderer(renderer);
  renderWindow->SetSize(640, 480);

  vtkSmartPointer<vtkRenderWindowInteractor> renderWindowInteractor =
      vtkSmartPointer<vtkRenderWindowInteractor>::New();
  renderWindowInteractor->SetRenderWindow(renderWindow);

  // Add the actor to the scene
  renderer->AddActor(actor);

  renderer->SetBackground(colors->GetColor3d("Seashell").GetData());
  // Render and interact
  renderWindow->Render();

  // Initialize must be called prior to creating timer events.
  renderWindowInteractor->Initialize();

  // Sign up to receive TimerEvent
  vtkSmartPointer<vtkTimerCallback2> cb =
      vtkSmartPointer<vtkTimerCallback2>::New();
  cb->actor = actor;

  renderWindowInteractor->AddObserver(vtkCommand::TimerEvent, cb);
  int timerId = renderWindowInteractor->CreateRepeatingTimer(100);
  cb->timerId = timerId;

  // Start the interaction and timer
  renderWindowInteractor->Start();

  return EXIT_SUCCESS;
}

@ajpmaclean : Please review now.

Looking good! Thanks.
These examples showcase VTK and must inspire, so the images must look good and the code formatting has to be consistent.

@ajpmaclean: Thank you for your inputs and valuable time to review the examples.
Such great work by you and @lorensen.
VTK is helping a lot of users like me to learn visualization concepts very effectively.


Breaking changes in V4?

I see the major version has incremented but there doesn't seem to be any breaking changes listed in the release changelog or the release section.
Were there any breaking changes? If not, why was the major version changed?

Hi @nullberri,
Here is the migration guide from v3 to v4: https://docs.tss-react.dev/update-to-v4

@garronej  Thank you i see i missed the first line of the readme 😅.


Add missing http.Forbidden error

"401 Unauthorized" and "403 Forbidden" are two different HTTP errors. CouchDB uses the first one when the server requires authentication credentials, and the second one when the request requires an authorisation that the current user does not have.
For instance, using a validate_doc_update design document can result in "403 Forbidden" being returned. See the official documentation for more details and examples:
// user is not authorized to make the change but may re-authenticate
throw({ unauthorized: 'Error message here.' });
// change is not allowed
throw({ forbidden: 'Error message here.' });

This patch adds Forbidden to the list of HTTP errors raised by couchdb-python.

Good stuff, thanks!


Issue #251 - Swipeble openLeft / openRight

Motivation
Author of the #251 issue noticed that it would be extremely useful to use openLeft/openRight methods on Swipeable
Chages
Documented close() method which wasn't mentioned in docs before.
Added methods openLeft and openRight as @henrikra  requested with docs.

Great can you also update TS types? https://github.com/kmagiera/react-native-gesture-handler/blob/master/react-native-gesture-handler.d.ts#L385

yup, is it right, @henrikra?


Fixed memtier_benchmark-1key-zset-100-elements-zrangebyscore-all-elements-long-scores key and prepopulation stage

By analyzing the test we saw that the input key and the benchmark key where different.
We also saw that the number of elements did not matched.

Codecov Report
Patch and project coverage have no change.

Comparison is base (d9c42fc) 56.71% compared to head (1ec8ac2) 56.71%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #239   +/-   ##
=======================================
  Coverage   56.71%   56.71%
=======================================
  Files          24       24
  Lines        2234     2234
=======================================
  Hits         1267     1267
  Misses        967      967

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Able to start a container without passing any networkIds

Version -V0.12.1
Using APIs , I am able to start a container without passing  networkIds param( I passed only imageUuid and name). In this case container has no ipaddress assigned to it.
In such cases , we expect API to default to Managed network.
HTTP/1.1 POST /v1/containers
Host:
Accept: application/json
Content-Type: application/json
Content-Length: 330
{
"imageUuid": "docker:nginx",
"name": "test5",
"networkIds": [ ],
"ports": [ ],
"startOnCreate": true,
"commandArgs": [ ],
"publishAllPorts": false,
"privileged": false,
"capAdd": [ ],
"capDrop": [ ],
"dns": [ ],
"dnsSearch": [ ],
"stdinOpen": false,
"tty": false,
"entryPoint": [ ],
"restartPolicy": null,
"devices": [ ],
"dataVolumes": [ ],
"dataVolumesFrom": [ ],
}
HTTP Response:
HTTP/1.1 201
Expires: Thu, 01 Jan 1970 00:00:00 GMT
X-API-Schemas: http://:8080/v1/schemas
Server: Jetty(8.1.11.v20130520)
X-API-Client-IP: <IP_ADDRESS>
Content-Length: 2309
Content-Type: application/json; charset=utf-8
{
"id": "1i14",
"type": "container",
"links": {
"self": "…/v1/containers/1i14",
"account": "…/v1/containers/1i14/account",
"credentials": "…/v1/containers/1i14/credentials",
"hosts": "…/v1/containers/1i14/hosts",
"instanceLinks": "…/v1/containers/1i14/instancelinks",
"loadBalancerTargets": "…/v1/containers/1i14/loadbalancertargets",
"mounts": "…/v1/containers/1i14/mounts",
"ports": "…/v1/containers/1i14/ports",
"targetInstanceLinks": "…/v1/containers/1i14/targetinstancelinks",
"volumes": "…/v1/containers/1i14/volumes",
"stats": "…/v1/containers/1i14/stats",
},
"actions": {
"remove": "…/v1/containers/1i14/?action=remove",
"start": "…/v1/containers/1i14/?action=start",
"logs": "…/v1/containers/1i14/?action=logs",
},
"name": "test5",
"state": "creating",
"accountId": "1a1",
"agentId": null,
"allocationState": "inactive",
"capAdd": [ ],
"capDrop": [ ],
"command": null,
"commandArgs": [ ],
"cpuSet": null,
"cpuShares": null,
"created": "2015-03-18T22:39:48Z",
"createdTS":<PHONE_NUMBER>000,
"data": {
"fields": {
"imageUuid": "docker:nginx",
"publishAllPorts": false,
"startOnCreate": true,
"dns": [ ],
"dnsSearch": [ ],
"stdinOpen": false,
"ports": [ ],
"commandArgs": [ ],
"networkIds": [ ],
"entryPoint": [ ],
"capAdd": [ ],
"dataVolumesFrom": [ ],
"tty": false,
"capDrop": [ ],
"dataVolumes": [ ],
"restartPolicy": null,
"privileged": false,
"devices": [ ],
},
},
"dataVolumes": [ ],
"dataVolumesFrom": [ ],
"description": null,
"devices": [ ],
"directory": null,
"dns": [ ],
"dnsSearch": [ ],
"domainName": null,
"entryPoint": [ ],
"environment": null,
"firstRunning": null,
"hostname": null,
"imageUuid": "docker:nginx",
"kind": "container",
"lxcConf": null,
"memory": null,
"memorySwap": null,
"networkIds": [ ],
"ports": [ ],
"primaryIpAddress": null,
"privileged": false,
"publishAllPorts": false,
"registryCredentialId": null,
"removeTime": null,
"removed": null,
"requestedHostId": null,
"restartPolicy": null,
"startOnCreate": true,
"stdinOpen": false,
"systemContainer": null,
"token": null,
"transitioning": "yes",
"transitioningMessage": "In Progress",
"transitioningProgress": null,
"tty": false,
"user": null,
"uuid": "7b023fb1-eb7b-4db2-bea8-29c1a486660c",
}

This is now networkMode and not relevant..


Add miQC

Description
To add miQC to QC script. Please notice that new packages have been added. To install SeuratWrappers I provided the specific code to it as others were not working.
Fixes partially # 2149
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Environment:
Test files:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings
[ ] I have committed any related changes to the PR

Hmmm, after fighting through some install issues (especially seurat wrappers), getting error:

processing file: seurat_alignment_qc.Rmd
|............                 |  40% (miQC-plot)                             Quitting from lines 214-226 (seurat_alignment_qc.Rmd)
Error in FLXfit(model = model, concomitant = concomitant, control = control,  :
2 Log-likelihood: NaN
Calls:  ... flexmix -> flexmix -> flexmix -> flexmix -> FLXfit -> FLXfit

Will try and troubleshoot more tomorrow

@migbro This should have been addressed now, see d1029470cfad77e70c771f4b0ae723bddd615156. Please let me know if that works for you too.


Prescribe how values of Postgres data types should be encoded in the HTTP API

Following on from https://github.com/electric-sql/electric-next/issues/164, we've arrived at a consensus that column values should be included in JSON responses from our HTTP API as strings. But that's not the end of story, since there's both ambiguity and inefficiencies in saying "just use the stringified value".
Ambiguity comes from the fact that some data types can be formatted differently depending on external factors. Postgres docs on the interval type show an illustrative example of how the same interval value can be encoded differently depending on the intervalstyle setting:

Style Specification
Year-Month Interval
Day-Time Interval
Mixed Interval

sql_standard
1-2
3 4:05:06
-1-2 +3 -4:05:06

postgres
1 year 2 mons
3 days 04:05:06
-1 year -2 mons +3 days -04:05:06

postgres_verbose
@ 1 year 2 mons
@ 3 days 4 hours 5 mins 6 secs
@ 1 year 2 mons -3 days 4 hours 5 mins 6 secs ago

iso_8601
P1Y2M
P3DT4H5M6S
P-1Y-2M3DT-4H-5M-6S

Taking the venerable blob type as another example, both the hex and the escape encodings take up more space than base64, so it would be beneficial if Electric re-encoded values of this type into base64 when preparing a JSON response.
We need to prescribe the format to use for at least for some of the more ambiguous Postgres data types when encoding their values into JSON strings.
TBD a list of candidate types...

Because my in-Elixir implementation of the interval needs to be able to parse constants without hitting PG constantly, I've implemented a parser for each for those interval formats, but I'm always using ISO8601 output. I think for intervals we should settle on that, because it's postgres and sql_standard are rife with weird edge cases that I had to double-check by running weird mixed stuff like SELECT '1 year 2-1 3 days 2:2'::interval and those make writing a parser on JS side way more complicated for no benefit.

@icehaunter Do we actually need to be able to parse all PG formats?
My thinking was that we'd settle on one format that we want to receive values from PG in and then enforce the use of that format in ReplicationClient by setting display settings right after opening a connection. This helps reduce the complexity of consuming Postgres' replication stream.

The present issue is about the other problem: choosing the format to use in our JSON responses for those Postgres types that have more than one output format. I was suggesting that we use base64 for blobs as an example in the issue description, but I'm now reversing my position. Ultimately, Electric should try to stay out of the way between Postgres sending an operation and a client consuming it. To that end, a simpler framework for reasoning about types with multiple output formats is to say "hey, we've chosen the ISO format for dates and ISO format for intervals so you can rely on your values always using those when your client is consuming our JSON API. For info about the specifics of those formats, refer to the official Postgres documentation". This can also be extended to user-defined types and types coming from PG extensions: refer to the extension docs to understand the structure of the format it uses when encoding its custom types to strings. Electric will simply stream values in the same format you'd see when interacting with the extensions types in a psql session.
While it would have been marginally more efficient to encode a blob into base64 before sending it to a client, we're going to make life easier for us and our users by choosing to stick with one of Postgres' output formats for blobs.
Now, for those users who'll be prioritizing efficiency, we should eventually support using Postgres' binary format, both when consuming its replication stream in Electric and when serving operations to the client (using a binary format for HTTP responses in this case, as opposed to JSON). Then clients will be able to use a library of choice for their language that can decode Postgres' binary protocol into that language's native data types.

@icehaunter Do we actually need to be able to parse all PG formats?

@alco unfortunately yes, because of ::interval thing on a text column. This is a rare usecase, but a client might want a runtime cast to an arbitrary type (sometimes interval) from text in a where clause (or, more importantly, in a SELECT if we support mapping columns somehow

@icehaunter That's an interesting use case.
What about dates? If Electric supports all interval styles, it is reasonable to assume that all of Postgres data styles are also supported. And that's where it gets complicated - https://www.postgresql.org/docs/current/datetime-input-rules.html. On the other hand, the interpretation rules are fully documented so it is conceivable that we could implement them in Elixir.
But unlike with intervals, we'll need to consult Postgres' current setting of DateStyle to correctly parse a data that's not using ISO. The same string '10/11/1999" can be interpreted as 1999-10-11 or 1999-11-10, depending on DateStyle. And the setting can be changed globally, per database, per session.
I'd say we restrict the range of input formats that are accepted in shapes' where clauses. E.g. only ISO dates, ISO intervals, hex-encoded blobs, etc. And use the same formats when encoding values to JSON in the HTTP API.

In the spirit of worse is best, we should just start with a single unbreakable strategy and I feel that is to restrict the range of input formats that are accepted and document the choices. It's quite reasonable that people need to adhere the specific input formats that Electric accepts.
I don't see a great problem to settle with base64 for blobs, if we think that is the best option. My argument is that people should not have any expectation about the format that the data would be returned without reading the docs.
Now that we have schema information in the messages, we can enrich the information with the choices, so that 'documentation lives in code'. e.g. stating the used output format for the blob or the DateStyle for a data field -- I'd do this even if we have a single option for the moment.
Overtime we could allow people to specify the formats they want to have things over the wire and that would reflect across the stack and on the returned schema. I don't think we'd want that for a long time. It would also be possible to extend to WHERE clauses in requests if the developer additionally provided the formatting that he is using. I don't think we'd want this for even a longer time.

Agree 👍 options are for successful products — let's just pick the most common / best format for now and then later (assuming there's serious demand) add option support.

@balegas

I don't see a great problem to settle with base64 for blobs, if we think that is the best option. My argument is that people should not have any expectation about the format that the data would be returned without reading the docs.

Another perspective on the issue of chosing the best format for transfering PG values from the server to the client is "how much additional work is required?" In the example with blobs, the least amount of work can be required if we assume the following constraints:

Electric configures its replication connection with Postgres to use the hex encoding for blobs (see https://github.com/electric-sql/electric-next/issues/175 on how to achieve that)
any blob value in the replication stream will be hex-encoded and Electric can store and then deliver those values to clients verbatim
in our docs we'll say: "Electric enforces the use of Postgres' hex encoding for blobs. Read the official Postgres docs to learn more"

We have saved on any post-processing on the server and writing any custom format docs by just using Postgres' native format. This will be an even bigger benefit when we switch to using Postgres' binary format for column values, which I expect we'll do at some point to gain more throughput.
In the scenario of using binary encoding, we won't even need to specify which blob encoding to use for the logical replication stream since there's a single binary encoding for every type. And so we could transfer those same binary-encoded values to the client which would then be able to use a postgres library native to its langauge to decode column values. He we'll save on the need to write client-side decoding code by just deferring to community-built libraries for working with the Postgres binary protocol.

@kevin-dp has raised a couple of related points in a discussion we had in Discord:

Those advanced types [blobs, extension types, etc.] could be parsed by the developers themselves. In which case they will receive the values in the format based on how they configured PG.
They are the ones that configured PG that way, so they are the ones to handle those message formats.

I think those propositions aren't taking into account some practical complexities developers have to deal with:

when one uses hosted Postgres, they don't necessarily choose the configuration. At least, not the default values, which can be different between different managed Postgres providers
so one's client-side parsing code ends up depending on server-side choices that one my not even have control over
I think it's likely that Electric client implementations will do the parsing most of the time, not the client apps. Like in the case of our TS client, we provide native decoding to JS types and client apps will use those unchanged. It would actually put more work on our plate if we have to build support for multiple formattings of the same type into our client. The same reasoning can be applied to community-built clients in other languages.

Looking into IntervalStyle, i would opt for iso_8601 as it is a standard, it's compact but still explicit (in contrast to  sql_standard), and it can be parsed by libraries such as moment.js


How to deal with the generic GraphQL scalar value in response?

So for example,if the response data are like this:
{
  article :{
    id:"",
    title:"",
    content:{
      paragraph:[
        id:"",
        elements:[{tag: "p", text: "", id: "e099a1"}]
      ]
    }
  }
}

It supporsed to have so-called Article ,Content ,Paragraph entities in the GraphQL's schema, but as to the content node,it's already a GenericScalar type,so Apollo can't generate the corresponding class like:
class Content{
  List<Paragraph> paragraphs;
  class Paragraph{
    List<Element> elements;
    ....
  }
}

So I have to do extra work on those scalar value,but what's worse: because use Rx2Apollo,
the most origina response data I can get is like this:
Content: {paragraphs=[{id=587f6c, elements=[{tag=p, text=not ok, id=e099a1}]}]}

It's not  JSON data at all (even the value term is  not surrounded by " "),so there is certainly much work which is exceedingly troublesome if I want to convert it to POJO.
Now I've found  no way to get the raw JSON sent by the server in response, and Apollo
seems to lose its advantages and conveniences in that case.
Although I think our backend developers should take the blame for this ( the response is not well-formated,I mean), but what should I do in my shoes _(:ｪ 」∠)_
Any advices , thx in advance :D

Yeah that is totally issue with pure designed backend.
Because Content defined as a custom scalar type it is sent in the response as plain text String not as Json string. I believe first what you need is to check if this is true and ask backed guys to send you as Json string.


MAPS - Send Safe Samples or Send All Samples - Which is Correct?

Under "Other recommended configuration settings > Update endpoint configuration protection > Policy location: \Windows Components\Microsoft Defender Antivirus\MAPS > Send file samples when further analysis is required", the suggestion states to enable it and set it to "send safe samples".
However, lower on the page, under "Setup Defender AV policies > Configure Cloud Deliver Protection and send samples automatically > Policy location: \Windows Components\Microsoft Defender Antivirus\MAPS > Send file samples when further analysis is required", the suggestion states to enable it and set it to "send ALL samples".
Which setting is correct? They're both referring to the same policy.
Thank you again for the opportunity to provide feedback! I hope you can see why this would be confusing to a new user.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 523651c3-a318-2369-6892-d13e04991596
Version Independent ID: 624c5e05-f87d-1bd4-bc87-bd55dcac1d86
Content: Onboard Windows devices to Microsoft Defender for Endpoint via Group Policy
Content Source: microsoft-365/security/defender-endpoint/configure-endpoints-gp.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@lccctemp Thank you for your feedback. Both the options are safe
The "Send File samples when further analysis is required " should be set to 1 (Send safe samples) or 3(Send all samples). Setting to 0 and 2 will not lower the protection state of the device.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @lccctemp Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[libgd] Enable fontconfig support

This enables fontconfig support for libgd. Needed e.g. for building gnuplot.
Furthermore the spelling of the webp support option is corrected.

@AlexanderTaeschner,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@AlexanderTaeschner, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Works great!
Thanks for the PR and thanks for your work to make gnuplot build on vcpkg 😄!


Convert tserver module tests from JUnit4 to JUnit5

This PR converts all tests in the tserver module from JUnit4 to JUnit5 Jupiter tests.
All tests in the module pass before and after these changes.

I reviewed all the changes in this PR. They all look right to me.


added latest presentations

[ ] Tests added / passed
[ ] Passes black dask / flake8 dask

@javad94 thanks for adding these!  Any interest in also tackling this issue related to building a list of resources https://github.com/dask/dask/issues/5017 ?

@quasiben How I can help for that issue?

Looks good -- I verified most of the Youtube links and they all worked


Feature Request: url right click menu

In some webapps a "Save link as..." and a "Copy url" option might be needed. Right now, right clicking on   a link brings up a menu with "Open in default browser" and Open in new window".
For example, when there is an attachment in the Outlook Web App it instructs you to right click and save as. An option that is not possible at the moment.

Yeah, some work needs to go into adding more options into the right click menu. I find myself missing some of the basic options that I have with the right click menu from chrome at times

You might find electron-context-menu useful for this. It includes a Copy Link item by default.

This seems like a duplicate of https://github.com/jiahaog/nativefier/issues/95.

Yup! @gesquive please see the duplicate issue!


CLOUDSTACK-10138: Load br_netfilter in security_group management script

When setting
sysctl -w net.bridge.bridge-nf-call-arptables=1
sysctl -w net.bridge.bridge-nf-call-iptables=1
sysctl -w net.bridge.bridge-nf-call-ip6tables=1
/usr/share/cloudstack-common/scripts/vm/network/security_group.py
doesn't check that br_netfilter is load.

I suppose it also should be added to 4.10.1

What if module br_netfilter does not exist ? for example on Ubuntu 12.04 or CentOS 7 ?
As I know, the issue only happens on Ubuntu 16.04.
Actually we fixed it by chef
execute 'modprobe br_netfilter' do
  command 'if [[ $(modinfo br_netfilter 2>1 /dev/null) ]]; then /sbin/modprobe br_netfilter; fi'
  only_if { node['platform_version'] == '16.04' }
end

Hello, 14.04 is affected too.
Ok, generally, it either should be handled by

script (may be smarter) or
-s etup of agent or
documentation (which is absent).

I think the check should be imroved. Ok, I haven't realized that it may be absent, but this means that it also can be either compiled-in or disabled, and your check is also weak (for abstract user).

I also propose alternative approach based on installation script for debian/ubuntu:
https://github.com/apache/cloudstack/pull/2321

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1281

This PR may be closed since the other PR was merged.


Click after page reload has no effect

On eg http://tobireif.com/boomchat/ , when I click on the smiley button, the animation starts. But when I then leave the mouse pointer where it is and reload the page (on Mac OS using cmd-r), then click on the button again, nothing happens.

Likely due to the interaction manager using mousemove to track position and on click using that stored position.

Problem doesn't exist in latest; http://codepen.io/staff0rd/pen/XNZyrx?editors=0010


Add colored recipients and support spaced fingerprints

The following PR shows a colored output when listing vault recipients
using vault recipients command.
It also allows to add vault recipients using spaced fingerprints as
follows vault add ABCD EF123 ...
Here's an example for the colored output:

@xetorthio ping ping ping.

LGTM!


tracking of browser process / waiting for browser close / update start args / couple of example fixes

Hi,
I've added a couple of changes
First I've added tracking of the browser process's launched
this can be retrieved via get_browser_process()
the idea is to use this to determine when the browser process has closed
so the python side can then close as well at the same time
Next there's a method called wait_browser_close()
this can be used to wait for browser instances to close
Next I've added a update_startargs() method, just to avoid accessing _start_args directly
What I'm doing is just using eel to launch chrome in application mode, but using a flask app as the backend
something like this

# Run Flask first
flaskserv.get_free_port()
flaskserv.start_async()

eel.update_startargs(size=(300, 200), mode='chrome', port=flaskserv.Port)
eel.show('/')
eel.wait_browser_close()

So instead of running the inbuilt bottle server, I'm running flask in a thread
(marked as a daemon so that it terminates when the main thread finishes)
then once we get past wait_browser_close() the whole app closes threads and all

Hi @grbd - thanks for the contribution.
I'm not sure I understand the need for this yet - but I also think that supporting this interface is going to lead to a fair amount of extra complexity for Eel to manage.
The Eel server should shut down shortly after the last websocket connection is closed. I don't think we want to support tracking the browser process itself - just the connections between a browser and the Eel server. This should work even if your case where you use Flask instead of Eel's bottle. Does this not work for you? Can you explain why? What do you want to do with the tracked browser process (if not just waiting on them) and why is waiting for websockets to close insufficient?
Having an interface to update the start args seems reasonable to me - I'd be happy to accept this contribution separately.


Docs should note helpful commands/hotkeys

In particular, I found the generate-stubs feature difficult to use, not realizing there was a command View.ShowSmartTag (by default mapped to hotkeys Shift+Alt+F10 and Ctrl+.) to bring up that context menu.

Could you please send a PR for this? We have a list of key bindings at http://fsprojects.github.io/VisualFSharpPowerTools/#Default-Keymap, it would be great to keep it up-to-date.
The source of docs is at https://github.com/fsprojects/VisualFSharpPowerTools/blob/master/docs/content/index.fsx

Closing. Feel free to send a PR if you find a better way to document shortcut keys.