Cloud Shell terminal failed to initialize: The download of the specified resource has failed

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]

Microsoft Windows [Version 10.0.18363.836]

Windows Terminal version (if applicable):

Version: 0.11.1333.0

Any other software?

Steps to reproduce
Select "Azure Cloud Shell" in the pull down menu, select the tenant number

Expected behavior
Cloud Shell terminal should present

Actual behavior
Error is reported, the output as follows:
Requesting a cloud shell instance...
Succeeded.
Requesting a terminal (this might take a while)...
IXMLHttpRequest2Callback::OnError: -2146697208: The download of the specified resource has failed.
[process exited with code 1]

Wow, never seen that one before.

I do have the similar issue.
Please enter the desired tenant number.
Enter n to login with a new account
Enter r to remove the above saved connection settings.

0
Requesting a cloud shell instance...
Succeeded.
Requesting a terminal (this might take a while)...
IXMLHttpRequest2Callback::OnError: -2146697208: The download of the specified resource has failed.

Is it related to Firewall settings or NSG settings in Azure?.
Outbound and Inbound  Rules on my machine has all process allowed to communicate on port 443. NSG in azure has all access allowed.
Networking
Allow access from         All networks
Where I am missing the part?

Is there a work around for this problem?  I can't seem to get past no matter what I do. If I could vote up to get a fix into an iteration I certainly would do so.

I tried the same with some other laptops, on some of them it worked and for two I think it didn't. What I understood either it's related to lack of administration permission in order to execute or communicate with System32 file system or network policy blocking the request. What I am trying to find out what exactly causes this failure.

Received this error myself today too:
Tenant 0: Default Directory ({Redaced})
Please enter the desired tenant number.
Enter n to login with a new account
Enter r to remove the above saved connection settings.
> 0
Requesting a cloud shell instance...
Succeeded.
Requesting a terminal (this might take a while)...
IXMLHttpRequest2Callback::OnError: -2146697208: The download of the specified resource has failed.

[process exited with code 1]

This is on a host that has worked fined previously, for a couple years. Only thing that I know has changed today is that updated Powershell 7 to 7.2.2.


add spanish localization

Add Spanish localization to project.
Progress / Closed on issue #135
What it Does
This adds Spanish localization to the app.
How I Tested

Edit Scheme language to Spanish
Ran the BrainMarks app in Simulator and reviewed Spanish translations

Notes

The settings view was recently updated with section headers, so this was added for Spanish localization.  This will have to be updated for the other languages.

The names of the new app icons are  in English. I wasn't sure if those should be in Spanish as well or not, but I also did not figure out how to change them even if I wanted to.

Although there is a 'Latin America' subset of Spanish localization available (es-419), I do not think that it was needed for this app.  I ran the simulator with es-419 (vs es) and verified Spanish settings were still all there.

Screenshot
Added a screenshot of the settings view with Spanish translations in use.

yikes! found a (super small) issue after the PR was made.
TLDR;  "Settings" defined differently in two spots (lines 14/44), but app uses the later one (correct translation), so issue is not seen visually in the app.
I'm guessing we can just fix the typo on line 11 or perhaps get ride of the second settings? It appears that it takes the latest translation in a localization file for any duplicate translations. (I made a second "Tweets" at the bottom and gave it a random name which then got used, so I believe the latest of the duplicates is used for the app).
Regarding the PR - if I want to make an additional change, how does that work with an open PR? Do I need to re-squash or do anything special if any new commits are made?

I'm not able to check the issue you mentioned, but if you want to add stuff go ahead and fix it and push to your branch (the PR updates automatically)
(There will be 2 commits now rather than one, which is fine)
But if you push more changes (like 3 or more commits) can you re-squash them? [you'll have to force push, if you squash again]

Thanks for that info!
No more planned commits, should be good to go.


3.0.10 - GFX Settings synch between accounts

Reported by @ patreon by a supporter:
"Not sure if it started with this version, but Graphic Configuration changes seem to be global now whereas before I could change values for each account"
GFX settings seem to be synched between accounts in some sort of way.
First arrived at version 3.0.10.
Tested following workflows:

Launching 2 accounts (with loginfiles) with different settings. Closing the first --> GFX stays ok on acc nr 2. Closing first acc --> GFX stays ok for first and second account
Launching 1 account (with loginfile) closing it. Then launching 2nd account --> GFX settings did not synch
Launching 1 account (with loginfile) and launching a 2nd account which is a clone of account 1 --> GFX settings did not synch
Launching 2 accounts (without loginfiles) with different settings. Closing the first --> GFX stays ok on acc nr 2. Closing first acc --> GFX stays ok for first and second account

User reported working steps:
"I tested it a bit and it seems to only happen once you import a config under Account settings -> Graphic configuration. After you import on one account, the graphics settings get stuck for every account that you switch to, even if I make individual parameter changes. Only way to un-stuck it would be to close and re-open."

Fixed the reference which did set the GFX to the listview not the actual acc.
Should now work as intended


Not recognised as a launcher.

On lollipop there is no way to set this app as a launcher. Android version 5.0.2 on Moto G 2nd gen.

After lurking on github for a bit I've noticed that this functiinality was present but removed later on. If this is a feature and not a bug feel free to delete this issue but if you want to hear my personal opinion on that matter: what's the point of developing a launcher that can't be used as a launcher?

Thanks for your feedback.
I predicted that if I let that option a lot of people would try to use it by default and not realize that it can be used without replacing your home launcher.
I will add back that option in the next release.

Fixed in release v1.1.5.


Unable to interact with auth0 dialog on iOS

When logins are handled via an auth0 custom tab, a dialog like this opens up on iOS:

This dialog is displayed when logging out as well as it briefly opens up the link to log the user out and automatically comes back to the app, unfortunately Detox can't interact with this dialog so being able to open up the app logged in still won't allow to test logging out as Detox gets stuck here, unable to interact with any elements.
I've found that Detox can only interact with alert dialogs that the XCode hierarchy debugger can identify, this one isn't displayed in the debugger therefore the usual methods won't work in this scenario.
I've seen this issue also reproduced with the new Rate your App iOS dialog, with the difference that Detox can still interact with the app and even the elements behind it.
An ideal solution would be to work out a way to interact with the dialog, or perhaps being able to launch the app disabling it given that there's already some functionality for other notifications.

Not possible. These alerts are presented by different processes, and cannot be interacted with. Please lock your API so the alert isn’t displayed.


operator t8c-certified (42.64.0-beta.1)

t8c-certified-42.64.0-beta.1 bundle

/pipeline restart operator-hosted-pipeline


Error when running health check

See error on redid
Solution might be to play with socket timeout

Happens on strato instance only

fixed by setting redis read_timeout=-1. in config/database.php


fix: Update 'import' deprecation warning with a date.

Putting a date in the deprecation warning message allows users to plan migration more effectively.  The date chosen allows for plenty of time for users to migrate existing stacker.yaml files.
#571.

i'm not actually sure why  df405f7  failed and 2dd1369 succeeded. :-(


Add hw2 and hw4 😎

@xinbinhuang can you please check hw2 in this PR?
@mezzX can you please check hw4 in this PR?


Support alternative file extension for GQL files

Hello,
And thank you for this library!
It's nice to have a standardized way for handling configurations.
One little thing that bugged me today, is that I cannot directly use my existing *.gql schema files because they don't have the required file extension :(
Maybe we could change this?
I can provide a PR with the basic change (add case '.gql') but no need for a PR I guess :)
Or with something more advanced if you have some ideas of improvements?

Thanks a lot for bringing this up!
@asiandrummer @sashko @IvanGoncharov  what do you think about supporting .gql files out of the box? I'm not too much of a fan fostering ambiguity between .graphql and .gql files.

Instead of the switch on file extension, a very basic format detection can do the trick?

Where/how is .gql extension used? I'm also pessimistic with alternative file extensions, unless there's a clear need and a wide adoption for it.

@asiandrummer you're right, the .gql extension doesn't seems really used, so I'll rename my files and close this :)
Thanks


Change how trimmedPath is calculated

Fixes #34 . Also calculate cleanedSourceFolderPath and cleanedDestinationFolder once instead of for each file.

LGTM, the index position of the character is definitely more reliable than pattern matching. Thanks!


Create a base runtime image for Module Loader

When creating the Module Loader DaemonSet, KMM currently sets sleep infinity as the container's command.
sleep runs as the main process in the container (PID 1) and is therefore unkillable. That makes the container hang when terminated, because running as PID 1 means that the TERM signal that Kubernetes sends is ignored unless the binary specifically implements a TERM handler (which sleep doesn't).
The following solutions would mitigate that problem:

set the Module Loader pod's shareProcessNamespace property to true, but that implies that other containers in the pod may be able to interact with the Module Loader (athough there is a single container in that pod now);
introduce a requirement that pause be present in the Module Loader image - it explicitly handles the TERM signal which is the desired behaviour.

To facilitate option 2, we could release a slim image that includes both modprobe and pause - the two components that KMM requires in the image.

We should probably add find and xargs (findutils package) to the requirements, with regards to the firmware cleanup.
fwUnloadCommand = fmt.Sprintf(" && cd %s && find |sort -r |xargs -I{} rm -d %s/$(basename %s)/{}", fw, nodeVarLibFirmwarePath, fw)

We will not run ModuleLoader images in v2 but rather mount them and load kmods ourselves. @mresvanis @yevgeny-shnaidman OK to close this?

Maybe we should implement option 1 for V1?  We already have another bug that is probably dependent on this one


Allow controlling notifications at the topic level

At the moment, you can turn notifications on and off for a stream, and mute a topic. I would like to be able to enable notifications for certain topics within a stream, but not others.
Example usecase: a "status" stream where people or teams post updates in a topic with their name. I'd like to get notified of updates on my team's updates but not all updates.

(I assume there's already an issue for this, but I couldn't find it in the first couple of pages of issue search.)

@kamalmarhubi Does issue #6027 satisfy your use case? If so, this can be closed and marked as a duplicate.


Remove jcenter if it tries to download dependencies

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build


Consider making additive headers in CallSettings

We could do with some way of adding headers in a CallSettings without overwriting other headers.
This is needed - in some form - for https://github.com/GoogleCloudPlatform/google-cloud-dotnet/issues/341

This is now fixed.


Docstring fixes

Merge activity

Dec 24, 5:02 PM: @@anjor started a stack merge that includes this pull request via Graphite.


(AzureCXP) Update script

Based on customer feedback added missing wiggly brace with in the PowerShell script
https://github.com/MicrosoftDocs/azure-docs/issues/49195

@BharathNimmala-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

#sign-off


Setup AWS - Create a SageMaker Notebook

Tutorial URL: https://developers.sap.com/tutorials/hxe-aws-eml-02.html
Step 4 confirmation is broken. I'm successfully getting the tensorflow and Python versions but pasting the Python version into the confirmation box results in a "wrong answer".
I'm pasting in:
Python 3.6.5 :: Anaconda, Inc.
The terminal looks like this:
sh-4.2$ source activate tensorflow_p36
(tensorflow_p36) sh-4.2$ python -c 'import tensorflow as tf; print(tf.__version__)'
1.14.0
(tensorflow_p36) sh-4.2$ python --version
Python 3.6.5 :: Anaconda, Inc.
(tensorflow_p36) sh-4.2$

I see the tutorial shows tensorflow version 1.12.0 so possibly the Python version check is for an earlier version. The error that results is Sorry, that’s incorrect. Please try again.

im looking into the verification now.  Thank you.

@jarjarbentley any updates?


Login With MFA by skipping local username login

Basically we use singin policy to login with local accounts(either username or email). Additionally we have to enable MFA. So first user will feed his user name password and then he will get OTP to mobile. but in our case we have to use only MFA with out using local account user name or email. we have to skip login using local account and use only MFA configurations to login to B2C. Is this possible either using default policy or using custom policy?
refer link.
https://github.com/Azure-Samples/active-directory-b2c-advanced-policies/blob/master/wingtipgamesb2c/Policies/b2ctechready.onmicrosoft.com_B2C_1A_base.xml

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

ID: d8c394d1-e3b1-b88c-416c-3f8e0ab3ae3d
Version Independent ID: 8aafb9d8-72cb-c9b4-5a01-9e5633764f39
Content: Multi-Factor Authentication in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-reference-mfa.md
Service: active-directory
GitHub Login: @davidmu1
Microsoft Alias: davidmu

@vetrivelmp Thanks for the feedback! We are currently investigating and will update you shortly.

@vetrivelmp To discuss your specific implementation scenario, please open a new MSDN forum post and community will help address your query. Please refer to below link to MSDN forum to ask new question -
https://social.msdn.microsoft.com/Forums/azure/en-US/newthread?category=windowsazureplatform&forum=WindowsAzureAD&prof=required

@vetrivelmp We will now proceed to close this thread. Please paste your MSDN thread URL here and we will make sure your query in addressed in forums.


Test cluster provisioning and upgrading on Debian 10

broken beyond repair:
[<IP_ADDRESS>] The following packages have unmet dependencies:
[<IP_ADDRESS>]  containerd.io : Depends: libseccomp2 (>= 2.4.0) but 2.3.3-4 is to be installed
[<IP_ADDRESS>] E: Unable to correct problems, you have held broken packages.
ERRO[14:06:11 EEST] failed to install kubeadm: Process exited with status 100  node=<IP_ADDRESS>
[<IP_ADDRESS>] Reading package lists...

It was decided that we wouldn't support Debian10.


Snap and Lock functions only works when Assisted Flight is enabled

This is happening in version 1.3.1, where "Assisted Flight" was introduce. It couldn't be reproduced in the previous version 1.3.0.
Actually, I don't really know if it's a bug or not. I read your comment on #2 but didn't understand the real purpose of it. Can you explain it?

In the pull request you mentioned (#2), I disabled the keybindings for snap/lock by default with the toggle for assisted flight since those features are added functionality. I was worried that new players might accidentally lock their pitch, for example, without knowing that pitch locking was even a feature of a HUD mod. I see now how this might be confusing. I think a better solution would be to remove the toggle for assisted flight but not assign default keybindings for snap/lock, so if you want to use those features, you would need to set your own keybindings. If you think that will remove the confusion, let me know!

Oh, I see now. The main purpose was to be a simple HUD mod, I'm right?
So, thinking about what you said, I think it's easier for sure to do that. But that will delete your idea on implementing Assisted Flight! What do you think about adding the toggle for Assisted Flight into "Key Binds" menu inside the game options (along with the other options) and document what it does? I think it's a more straightforward thought.

I also thought about moving all the options about snapping/locking the screen to the Mod Menu like Assisted Flight, making it more minimalist and keeping the main idea of a simple QoL HUD, while maintaining its beautiful configuration.

I removed the assisted flight toggle and the default snap/lock keybindings (4248ded), so assisted flight is now disabled by default. If you want to use the snap/lock functionality (still called assisted flight), just assign your own keybindings. You'll see these changes in the next release (v1.4.0).


add rule (rrd): parameter count

max number of function parameteres should be 3

Hi @rrd108
I created this PR for this issue #60

Hi @rrd108 , #60  was merged so lets close this issue to not fill up the issues tab 👨💻


Blank page (Empty file)

Hello, when I try to acces to kanboard (after some server problems), I get a blank page, an empty file.
Apache logs:

[Thu Jul 09 09:55:16.614775 2015] [:error] [pid 26635] [client <IP_ADDRESS>:53069] PHP Fatal error:  Uncaught exception 'PDOException' with message 'could not find driver' in /var/www/kanboard/vendor/fguillot/picodb/lib/PicoDb/Driver/Sqlite.php:22\nStack trace:\n#0 /var/www/kanboard/vendor/fguillot/picodb/lib/PicoDb/Driver/Sqlite.php(22): PDO->__construct('sqlite:data/db....')\n#1 /var/www/kanboard/vendor/fguillot/picodb/lib/PicoDb/Database.php(81): PicoDb\Driver\Sqlite->__construct(Array)\n#2 /var/www/kanboard/app/ServiceProvider/DatabaseProvider.php(63): PicoDb\Database->__construct(Array)\n#3 /var/www/kanboard/app/ServiceProvider/DatabaseProvider.php(27): ServiceProvider\DatabaseProvider->getSqliteInstance()\n#4 /var/www/kanboard/app/ServiceProvider/DatabaseProvider.php(13): ServiceProvider\DatabaseProvider->getInstance()\n#5 /var/www/kanboard/vendor/pimple/pimple/src/Pimple/Container.php(273): ServiceProvider\DatabaseProvider->register(Object(Pimple\Container))\n#6 /var/www/kanboard/app/common.php(26): Pimple\Container->register(Object(ServiceProvider\DatabaseProvider))\n#7 /var/www/kanboard/index.php(4): require in /var/www/kanboard/vendor/fguillot/picodb/lib/PicoDb/Driver/Sqlite.php on line 22

Debian stable, and I have php5-sqlite installed. Also php5-mysql (if needed). I would like to get back all the data.
Thanks!

@theSouL
The error message includes the problem:

'PDOException' with message 'could not find driver'

The PDO driver for SQLite is missing.
This is not a Kanboard error!

Hi @froschdesign thanks for your answer.
I know isn't a Kanboard error, but I'm not into php and web applications, so I didn't know where to ask.
As I said, I have php5-sqlite installed. So, maybe I miss more packages?

So, maybe I miss more packages?

Yes, as I said: PDO driver for SQLite!

After installing the web server may need to be restarted.

@froschdesign Hahaha!
Sorry, I was using the sqlite3 package instead of sqlite. Thank you very much! I should be banned from this repo haha :P
Thanks, problem solved ^^

@theSouL
No problem. Please close this issue!


Improve readme instructions

@subatiq  hope you are doing well, will you merge this ?

Closing as dated


FAQ section and some other topics under Reference are not shown.

Description:
The link of the Reference tab is directed to Config Catalog and the other left menu items are not shown.
https://apim.docs.wso2.com/en/latest/reference/ConfigCatalog/
If we search for some content, it will display the correct menu.
https://apim.docs.wso2.com/en/latest/reference/faq/#why-cant-i-see-all-the-apis-that-i-published-on-the-developer-portal
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Fixed in

https://github.com/wso2/docs-apim/pull/1373
3.0.0

https://github.com/wso2/docs-apim/pull/1374
3.1.0

https://github.com/wso2/docs-apim/pull/1375
master


Video streaming app pod going to CrashLoopBackOff

Hello,
Describe the bug
When replicating the end-to-demo with MicroK8s (Ubuntu instance on GC) on my own, the pod of video streaming app goes to CrashLoopBackOff.
All other prior install steps were successful.
I added below the logs of failing pod
Didier
Output of kubectl get pods,akrii,akric -o wide
microk8s kubectl get all --all-namespaces -o wide
NAMESPACE     NAME                                              READY   STATUS             RESTARTS   AGE   IP            NODE            NOMINATED NODE   READINESS GATES
kube-system   pod/calico-kube-controllers-847c8c99d-65bbl       1/1     Running            0          18m   <IP_ADDRESS>    microk8s-akri
kube-system   pod/calico-node-xsvqv                             1/1     Running            1          18m   <IP_ADDRESS>   microk8s-akri
kube-system   pod/coredns-86f78bb79c-dbgm9                      1/1     Running            0          17m   <IP_ADDRESS>    microk8s-akri
default       pod/akri-agent-daemonset-c88pb                    1/1     Running            0          12m   <IP_ADDRESS>   microk8s-akri
default       pod/akri-controller-deployment-5b4bb5cbb5-8mlwp   1/1     Running            0          12m   <IP_ADDRESS>    microk8s-akri
default       pod/akri-video-streaming-app-fd5f4cb7d-fzpcq      0/1     CrashLoopBackOff   7          12m   <IP_ADDRESS>    microk8s-akri
NAMESPACE     NAME                               TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)                  AGE   SELECTOR
default       service/kubernetes                 ClusterIP   <IP_ADDRESS>             443/TCP                  18m
kube-system   service/kube-dns                   ClusterIP   <IP_ADDRESS>            53/UDP,53/TCP,9153/TCP   17m   k8s-app=kube-dns
default       service/akri-video-streaming-app   NodePort    <IP_ADDRESS>           80:31671/TCP             12m   app=akri-video-streaming-app
NAMESPACE     NAME                                  DESIRED   CURRENT   READY   UP-TO-DATE   AVAILABLE   NODE SELECTOR            AGE   CONTAINERS    IMAGES                                   SELECTOR
kube-system   daemonset.apps/calico-node            1         1         1       1            1           kubernetes.io/os=linux   18m   calico-node   calico/node:v3.13.2                      k8s-app=calico-node
default       daemonset.apps/akri-agent-daemonset   1         1         1       1            1           kubernetes.io/os=linux   12m   akri-agent    ghcr.io/deislabs/akri/agent:latest-dev   name=akri-agent
NAMESPACE     NAME                                         READY   UP-TO-DATE   AVAILABLE   AGE   CONTAINERS                 IMAGES                                                 SELECTOR
kube-system   deployment.apps/coredns                      1/1     1            1           17m   coredns                    coredns/coredns:1.6.6                                  k8s-app=kube-dns
kube-system   deployment.apps/calico-kube-controllers      1/1     1            1           18m   calico-kube-controllers    calico/kube-controllers:v3.13.2                        k8s-app=calico-kube-controllers
default       deployment.apps/akri-controller-deployment   1/1     1            1           12m   akri-controller            ghcr.io/deislabs/akri/controller:latest-dev            app=akri-controller
default       deployment.apps/akri-video-streaming-app     0/1     1            0           12m   akri-video-streaming-app   ghcr.io/deislabs/akri/video-streaming-app:latest-dev   app=akri-video-streaming-app
NAMESPACE     NAME                                                    DESIRED   CURRENT   READY   AGE   CONTAINERS                 IMAGES                                                 SELECTOR
kube-system   replicaset.apps/calico-kube-controllers-847c8c99d       1         1         1       18m   calico-kube-controllers    calico/kube-controllers:v3.13.2                        k8s-app=calico-kube-controllers,pod-template-hash=847c8c99d
kube-system   replicaset.apps/coredns-86f78bb79c                      1         1         1       17m   coredns                    coredns/coredns:1.6.6                                  k8s-app=kube-dns,pod-template-hash=86f78bb79c
default       replicaset.apps/akri-controller-deployment-5b4bb5cbb5   1         1         1       12m   akri-controller            ghcr.io/deislabs/akri/controller:latest-dev            app=akri-controller,pod-template-hash=5b4bb5cbb5
default       replicaset.apps/akri-video-streaming-app-fd5f4cb7d      1         1         0       12m   akri-video-streaming-app   ghcr.io/deislabs/akri/video-streaming-app:latest-dev   app=akri-video-streaming-app,pod-template-hash=fd5f4cb7d
Kubernetes Version: [e.g. Native Kubernetes 1.19, MicroK8s 1.19, Minikube 1.19, K3s]
snap list
Name              Version    Rev    Tracking         Publisher          Notes
core              16-2.47.1  10185  latest/stable    canonical✓         core
core18            20200929   1932   latest/stable    canonical✓         base
google-cloud-sdk  316.0.0    157    latest/stable/…  google-cloud-sdk✓  classic
helm              3.4.0      302    latest/stable    snapcrafters       classic
lxd               4.0.3      16922  4.0/stable/…     canonical✓         -
microk8s          v1.19.2    1769   1.19/stable      canonical✓         classic
snapd             2.47.1     9721   latest/stable    canonical✓         snapd
To Reproduce
I can publish my script in a repo with corresponding Github workflow to allow you to see full exec log (andreproduce by forking if needed)
Expected behavior
Get streaming app pod to Running status
Logs (please share snips of applicable logs)
microk8s kubectl logs akri-video-streaming-app-fd5f4cb7d-fzpcq
Traceback (most recent call last):
File "./streaming.py", line 33, in
grpc_port = os.environ[env_var_prefix + 'PORT_GRPC'] # instance services are using the same port by default
File "/usr/lib/python3.7/os.py", line 678, in getitem
raise KeyError(key) from None
KeyError: 'AKRI_UDEV_VIDEO_SVC_SERVICE_PORT_GRPC'
Additional context
GCE instance with Ubuntu LTS 20.04

Hello @didier-durand ,
The streaming app will crash with the default configuration if the configuration level service has not been created, which happens if no instances have been found. Can you please share how you deployed Akri, especially the configuration? Also, your output does not correspond to this command:
kubectl get pods,akrii,akric -o wide

Can you please share output of this command?

hi,
I did a full reinstall of my Ubuntu and can't get  gst-launch-1.0  commands to work again (video device not properly seen).
Please, have some patience until I understand the root cause and fix.
Sorry for the delay!
didier

I opened a ticket with the v4l2 team: I can't get GStreamer to work with the v4l2sink on Ubuntu 20.04 LTS
But, ffmpeg works fine with v4l2sink to generate similar videostreams
With those ffmpeg streams, the problem is the same : pod failing in CrashLoopBackoff
Below the outcome of the commands that you asked for. NB: no akrii or akric resources available
I also added the commannds and logs of the Akri helm chart install and of the install of the streaming application

get infos on pod, akrii, akric
microk8s kubectl get pods,akrii,akric
NAME                                              READY   STATUS             RESTARTS   AGE
pod/akri-agent-daemonset-2tslp                    1/1     Running            0          87m
pod/akri-controller-deployment-5b4bb5cbb5-b2tjk   1/1     Running            0          87m
pod/akri-video-streaming-app-fd5f4cb7d-9pzbj      0/1     CrashLoopBackOff   21         87m
Log of Akri helm chart install
helm --kubeconfig "$KUBE_CONFIG" repo add akri-helm-charts https://deislabs.github.io/akri/
export AKRI_HELM_CRICTL_CONFIGURATION="--set agent.host.crictl=/usr/local/bin/crictl --set agent.host.dockerShimSock=/var/snap/microk8s/common/run/containerd.sock"
helm install akri akri-helm-charts/akri-dev
$AKRI_HELM_CRICTL_CONFIGURATION
--set useLatestContainers=true
--set udevVideo.enabled=true
--set udevVideo.udevRules[0]='KERNEL=="video[0-9]*"'
--kubeconfig "$KUBE_CONFIG"
sudo microk8s kubectl wait --for=condition=available --timeout=50s deployment.apps/akri-controller-deployment -n default
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: kube-config
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: kube-config
"akri-helm-charts" has been added to your repositories
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: kube-config
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: kube-config
W1106 07:05:29.071185   35919 warnings.go:67] rbac.authorization.k8s.io/v1beta1 ClusterRole is deprecated in v1.17+, unavailable in v1.22+; use rbac.authorization.k8s.io/v1 ClusterRole
W1106 07:05:29.075364   35919 warnings.go:67] rbac.authorization.k8s.io/v1beta1 ClusterRole is deprecated in v1.17+, unavailable in v1.22+; use rbac.authorization.k8s.io/v1 ClusterRole
W1106 07:05:29.233020   35919 warnings.go:67] rbac.authorization.k8s.io/v1beta1 ClusterRole is deprecated in v1.17+, unavailable in v1.22+; use rbac.authorization.k8s.io/v1 ClusterRole
W1106 07:05:29.243928   35919 warnings.go:67] rbac.authorization.k8s.io/v1beta1 ClusterRole is deprecated in v1.17+, unavailable in v1.22+; use rbac.authorization.k8s.io/v1 ClusterRole
NAME: akri
LAST DEPLOYED: Fri Nov  6 07:05:28 2020
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:

Get the Akri Controller:
kubectl get -o wide pods | grep controller
Get the Akri Agent(s):
kubectl get -o wide pods | grep agent
Get the Akri Configuration(s):
kubectl get -o wide akric
deployment.apps/akri-controller-deployment condition met

Log of install for video streaming app
sudo microk8s kubectl apply -f "https://raw.githubusercontent.com/deislabs/akri/main/deployment/samples/akri-video-streaming-app.yaml"
sudo microk8s kubectl wait --for=condition=available --timeout=50s deployment.apps/akri-video-streaming-app -n default
deployment.apps/akri-video-streaming-app created
service/akri-video-streaming-app created
deployment.apps/akri-video-streaming-app condition met

Due to the recent changes re helm command on end-to-end-demo.md , this ticket is no longer valid: I get different results by applying those changes when I try to reproduce the demo. I'll re-open a new issue if needed.
So, closing for now.


Update sbt-mdoc to 2.2.19

Updates org.scalameta:sbt-mdoc from 2.2.13 to 2.2.19.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "sbt-mdoc" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #62.


Fix goto import/load on Windows + search for modules in local modules directory

Fix for goto import or load on Windows. Also the server will now look for the module in the local modules directory if it wasn't found in the compiler modules.

Thanks again!


In ConstrainedDecimal ge, gt, le, lt does not have any effect

Checks

[X] I added a descriptive title to this issue
[X] I have searched (google, github) for similar issues and couldn't find anything
[X] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
pydantic version: 1.8.2
pydantic compiled: True
install path: ...venvs/.venv38/lib/python3.8/site-packages/pydantic
python version: 3.8.10 (default, Jun  2 2021, 10:49:15)  [GCC 9.4.0]
platform: Linux-5.4.0-86-generic-x86_64-with-glibc2.29
optional deps. installed: ['typing-extensions']
...

from decimal import Decimal
from pydantic import ConstrainedDecimal

class MyConstrainedDecimal(ConstrainedDecimal):
    ge = Decimal("6")
    le = Decimal("6")

for test_value in (Decimal("5"), Decimal("6"), Decimal("7")):
    print(MyConstrainedDecimal.validate(Decimal(test_value)))

...

Output:
5
6
7
Expected: exception raised on value 5 and 7.
Same situation is for gt and lt.
It seems like constraints ge, gt, le, lt does not have any effect. I wasn't able to find much use case examples. But those I find shows that my case is correct. Also decimal_places constraint works OK.
I've seen a closed issue here but I haven't seen a specific decision regarding these constraints.

This is resolved in V2 with the V2 way of doing things:
from decimal import Decimal
from typing import Annotated

from annotated_types import Gt, Lt
from pydantic_core import ValidationError

from pydantic import TypeAdapter

MyConstrainedDecimal = Annotated[Decimal, Gt(Decimal('5')), Lt(Decimal('7'))]

MyConstrainedDecimalValidator = TypeAdapter(MyConstrainedDecimal)

try:
    MyConstrainedDecimalValidator.validate_python(Decimal('5'))
except ValidationError as e:
    print(e)
    """
    1 validation error for lax-or-strict[lax=function-after[validate(), union[json-or-python[json=function-after[Decimal(), union[int,float,constrained-str]],python=is-instance[Decimal]],int,float,constrained-str]],strict=custom-error[function-after[validate(), json-or-python[json=function-after[Decimal(), union[int,float,constrained-str]],python=is-instance[Decimal]]]]]
    Input should be greater than 5 [type=greater_than, input_value=Decimal('5'), input_type=Decimal]
        For further information visit https://errors.pydantic.dev/0.31.0/v/greater_than
    """

MyConstrainedDecimalValidator.validate_python(Decimal('6'))

try:
    MyConstrainedDecimalValidator.validate_python(Decimal('7'))
except ValidationError as e:
    print(e)
    """
    1 validation error for lax-or-strict[lax=function-after[validate(), union[json-or-python[json=function-after[Decimal(), union[int,float,constrained-str]],python=is-instance[Decimal]],int,float,constrained-str]],strict=custom-error[function-after[validate(), json-or-python[json=function-after[Decimal(), union[int,float,constrained-str]],python=is-instance[Decimal]]]]]
    Input should be less than 7 [type=less_than, input_value=Decimal('7'), input_type=Decimal]
        For further information visit https://errors.pydantic.dev/0.31.0/v/less_than
    """


Ignore case in hash comparison

Feedback from Google Play:
Hey, there is a bug where if you paste an MD5 hash with upper case it will not match a generated hash (which is lower case) even though they actually match. Please make case insensitive unless using a case sensitive hash (is there any?).

I have tried to reproduce this issue in android studio, but it seems that upper case and lowercase are matching?

Steps I have tried to replicate:
Added a text and generated hash.
Converted all the text to upper case and pasted into custom hash field.
Hit Compare and I get 'Match'

@alexander-kuruvilla, thanks for your help. I will try it on local build as soon as possible.

@fartem In case of different text inputs cases, the hashing does differ. For example in the MD5 hash for 'Hello'', "hello" and "HELLO" is different and its the default behavior of MessageDigest class. To resolve this case I can normalise all input cases to lowercase (or uppercase) before generating digest if you are ok with it.

@alexander-kuruvilla, we don't need to normalise input because it is valid case when user inputs text in different cases.

@alexander-kuruvilla, we don't need to normalise input because it is valid case when user inputs text in different cases.

True, its a valid case. Then ignoring that there isn't a bug.

@alexander-kuruvilla, yes, it is a valid case. I send additional info request to user with this issue, need time to check it again.
Thanks for your help.

Odd.  I can't seem to reproduce it now.  Perhaps I had accidentally selected too much and pasted with a space at the end in the first case?  (I tried adding a space in the custom hash box just now, and this does cause it to not match -- perhaps sanitize the input to disallow spaces?  I don't think any hash uses them.)

I don't have time to support the application, so the edits are unfortunately not relevant.


Add a Gitter chat badge to README.md

silverstripe/silverstripe-framework now has a Chat Room on Gitter
@sminnee has just created a chat room. You can visit it here: https://gitter.im/silverstripe/silverstripe-framework.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Wow, that's creepy. I was just testing it out!

creepy indeed

if @gitter-badger breaks the build, I'm going to rage

we aren't moving to gitter yet, and if we do, we'll likely promote a generic channel

For anyone who comes across this: we haven't made any decision to advocate Gitter as a comms channel, and if we do, we'll discuss it on http://groups.google.com/group/silverstripe-dev first. Sorry if I caused any alarm with my exploration! :-)


[Bug] [Module Name] Bug title

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating system information
Linux
Python version information
3.10
DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[X] Chat DB
[ ] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
GPU
Models information
LLM: QWEN2-max
What happened
在应用界面，能够成功连接MySQL数据库，但是在ChatDB或者自定义应用时，显示“暂无数据”。

What you expected to happen
已按照文档，在连接MySQL数据库时，运行相应版本的.sql文件。但是在界面中仍然无法查询到数据。
How to reproduce

在界面连接MySQL数据库
进入ChatDB应用，开始对话
选择参数 -> 暂无数据。

Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

@JackyMeow Thank for your pr, we have fixed this at #1953


Add confirmation message

💥 Proposal
Add a confirmation message when someone submits the contact us form.
Have you read the Contributing Guidelines on Pull Requests ?
Yes

Hii @rockingrohit9639, I would like to work on this issue. can you please assign me this issue.

Hii @rockingrohit9639, I would like to work on this issue. can you please assign me this issue.

Yes @pranabprohit, You can work on it.
Make sure you complete it in 4 days.


Clean up incorrect and stale docker volumes automatically

https://github.com/Sovereign-Labs/sovereign-sdk/issues/1159#issuecomment-1821947403
So it seems like docker-compose is not re-creating the volumes. This could continue impacting users especially if they switch between demo-rollup in sovereign repo and starter-rollup since both of them use different credentials paths for the volume.
one simple fix is to just have the makefile detect and delete volumes every time (the credentials folder is very light weight. < 1kb)
another approach is to just use different volume names for both

I'd lean toward just deleting the volumes. That way things don't break when someone has two forks of a package

I'm not sure what is the way to reproduce this, however the credentials persists because the sequencer address is hardcoded

is there any reproduction from non mac-os? I started to think it's a bug in compose local volume driver specific to this system

hmm indeed if you have 2 different projects with the same docker layout and volume names, they will never be recreated so you have to either name it differently or delete them on the startup


Debugger feature requests

Component
Forge
Describe the feature you would like
The debugger experience is pretty slick, but could be even better.

Making it smarter about where solidity structs are laid out and labelling them with variable names
visual storage/memory explorer with struct/variable knowledge
calling convention/ABI.encode awareness
get better at not losing where it is with weird assembly/reentrancy

Additional context
Was using the debugger last night, very exciting :) but I ended up falling back to "just logging a bunch" repeatedly in a way I wouldn't have with GDB.
Thanks for building this, excited to see where it goes!

Marking as duplicate, added your suggestions to the debugger meta issue: https://github.com/foundry-rs/foundry/issues/927


Switch install page from .html to .md

This fixes the "edit this page" feature. Follow-up to https://github.com/cockroachdb/docs/pull/2713.
Fixes #2157.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f4aaa4b57b33ac2f49cd3cc34c4a331ca446a9c0/


Update to ART 1.10.3

Description
This pull request merges the updates of ART 1.10.2 to branch main.

Codecov Report

Merging #1727 (a7fdb0e) into main (b0d86c4) will decrease coverage by 50.27%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main    #1727       +/-   ##
===========================================
- Coverage   88.17%   37.90%   -50.28%
===========================================
  Files         259      259
  Lines       21413    21414        +1
  Branches     3805     3805
===========================================
- Hits        18881     8117    -10764
- Misses       1588    12921    +11333
+ Partials      944      376      -568

Impacted Files
Coverage Δ

...adversarial_texture/adversarial_texture_pytorch.py
90.00% <100.00%> (+0.05%)
:arrow_up:

art/attacks/evasion/carlini.py
7.55% <0.00%> (-86.74%)
:arrow_down:

...rt/defences/detector/poison/clustering_analyzer.py
8.22% <0.00%> (-86.71%)
:arrow_down:

art/attacks/evasion/pe_malware_attack.py
14.28% <0.00%> (-84.80%)
:arrow_down:

...ces/detector/evasion/subsetscanning/scanningops.py
11.59% <0.00%> (-84.06%)
:arrow_down:

art/attacks/evasion/square_attack.py
10.32% <0.00%> (-84.04%)
:arrow_down:

art/attacks/evasion/wasserstein.py
11.94% <0.00%> (-83.28%)
:arrow_down:

art/metrics/privacy/membership_leakage.py
14.28% <0.00%> (-81.64%)
:arrow_down:

art/attacks/evasion/elastic_net.py
17.88% <0.00%> (-81.46%)
:arrow_down:

...cks/poisoning/perturbations/image_perturbations.py
8.97% <0.00%> (-80.77%)
:arrow_down:

... and 164 more


Backport 1.2: Add prereboot tag and workaround grub-common crashkernel config

Tag for items which should be updated before a reboot. Intention is that
this can be run from -master or another recent branch on an existing
stack before scheduled reboots occur on that stack.
In Precise grub-common includes an automatic crashkernel config, but it
isn't large enough for our updater kernels and configurations. Rather
than removing grub-common or backporting from a later release, just fix
the size of the crashkernel reservation to 256M.

This looks good. Test failed for unrelated reason, a repush could get them going again.


Model::incrementOrDecrement() not saving attributes set during updating event

Laravel Version
10.22
PHP Version
8.2
Database Driver & Version
No response
Description
Imagine we have a Product model.
We can listen for the updating event to always set the updated_by column like so;
Product::updating(function ($product) {
    $product->updated_by = Auth::id();
})

Now, when we try to update the Product stock using the increment() method, other attributes set by an observer during the updating event do not get saved to the database.
$product->increment('stock', 5); // `updated_by` is set but never saved to the db.

Note: In this example, the updating event is actually fired, and the observer code actually does set the value of updated_by.
https://github.com/laravel/framework/blob/00894b89e42a9d707c6038084bbeb25422eed5b5/src/Illuminate/Database/Eloquent/Model.php#L964
However, the other dirty attributes set during the updating event do not get saved to the database. I notice this happens because the save() method is never called (which actually saves all dirty attributes). The call is rather forwarded to the builder's own increment/decrement methods.
https://github.com/laravel/framework/blob/00894b89e42a9d707c6038084bbeb25422eed5b5/src/Illuminate/Database/Eloquent/Model.php#L968
The weirdest part is that all of the dirty attributes are being synced as changes to the model, even when they were never actually changed/saved in the database.
https://github.com/laravel/framework/blob/00894b89e42a9d707c6038084bbeb25422eed5b5/src/Illuminate/Database/Eloquent/Model.php#L969
This means, invoking $product->wasChanged('updated_by') returns true which is a false-positive result.
Steps To Reproduce
See description.

I believe this is expected since it's uses Fluent Query Builder to execute the query, which doesn't trigger any events:
https://github.com/laravel/framework/blob/00894b89e42a9d707c6038084bbeb25422eed5b5/src/Illuminate/Database/Eloquent/Builder.php#L1114-L1127

@crynobone I think you missed the method I am referring to. What you shared is a different increment method which is on the Builder. That is not the problem.
The issue is based on the increment method that exists on the eloquent Model.
https://github.com/laravel/framework/blob/00894b89e42a9d707c6038084bbeb25422eed5b5/src/Illuminate/Database/Eloquent/Model.php#L917-L929
This method calls the underlying incrementOrDecrement method which does triggers events, but doesn't acknowledge attributes that are set during the updating event, leading to false-positive results.
https://github.com/laravel/framework/blob/00894b89e42a9d707c6038084bbeb25422eed5b5/src/Illuminate/Database/Eloquent/Model.php#L952-L976

If you really look at the code, it doesn't use save() and instead create a new query builder for insert and update, you may be able to return false from updating and other stuff from updated but that's about it.

@crynobone are you saying that this is an expected behavior? and not a bug?
$product = Product::find(1);

$product->updated_by; // null

$product->increment('stock');

$product->wasChanged('updated_by'); // true
$product->updated_by; // 1

$product->refresh();

$product->updated_by; // null ???????????

@newtonjob yes this is not a bug. Replace your updating hook with this
Product::updating(function ($product) {
    $product->update(['updated_by' => Auth::id()]);
})

Or leave it as-is, but call $product->save(); before returning.

First, you should never execute another ->update(...) during an updating event, as that'll end up in an endless dispatch of the updating event and eventually timeout.
Second, invoking save() after increment() doesn't make any difference since the increment() method already invokes syncChanges() internally before you even get a chance to call save().
So, by the time you call save(), the "fake" changes have already been synced, thus save() doesn't see any "dirty" attributes to actually save.
Note that syncChanges() will sync all "dirty" attributes (even the ones that were never actually updated in db).
Moreso, doing a check with $product->wasChanged('updated_by') will return true when it actually never changed in db.
You still don't think this is a bug?

Interesting - I'll have to keep digging into this. But I was under the impression that mass assignment through ->update() doesn't trigger the updating and updated hooks


Graph shortest path with Floyd-Warshall removes explicit zeros.

My issue concerns scipy.sparse.csgraph.shortest_path, more specifically the Floyd-Warshall solver, which does not deal with explicit zeros as the other algorithms (they are removed here).
The following (slightly convoluted) example highlights the different handling of explicit zeros.
Reproducing code example:
import numpy as np
import scipy.sparse.csgraph
import sklearn.neighbors

examples = [
    # no infinity
    # (the undirected graph has one connected component)
    np.array([0, 1, 2, 3, 3])[:, None],
    # infinity only for 'FW' solver
    # (the undirected graph has one connected component)
    np.array([0, 1, 2, 3, 3, 3])[:, None],
    # infinity for all solvers
    # (the undirected graph has two connected components)
    np.array([0, 1, 2, 4, 4, 4])[:, None],
]

for X in examples:
    graph = sklearn.neighbors.kneighbors_graph(X, n_neighbors=2,
                                               mode='distance')
    print(scipy.sparse.csgraph.connected_components(graph)[0], end='\t')

    for method in ['FW', 'D', 'BF', 'J']:
        paths = scipy.sparse.csgraph.shortest_path(graph, method=method,
                                                   directed=False)
        print(np.any(np.isinf(paths)), end='\t')

    print('')

Print message:
1	False	False	False	False
1	True	False	False	False
2	True	True	True	True

Explanations:
Explicit zeros arise when you have duplicate points in your graph, thus having a connection with a distance of zero, which is different from the implicit zero (meaning no graph connection). When building a k-nearest neighbors graph, if some points are duplicated more than the number of neighbors k, then these points have only zeros in the set of explicit values, and using explicit zeros as if they were implicit zeros effectively remove the points from the connected graph. In particular, these points may still be connected to the rest if they are chosen as neighbors of another point (with directed=False).
Scipy/Numpy/Python version information:

1.2.1 1.16.4 sys.version_info(major=3, minor=7, micro=3, releaselevel='final', serial=0)

Yes, this is a bug and should be fixed. It seems that we would need to move the "set non-edges to infinity" step out of the _floyd_warshall helper function, as it operates on a densified input matrix.
The PR fixing this should make sure to add a test case exercising the issue.

I can work on this and add a test case.


add ws timezone and app info support

Codecov Report
Attention: Patch coverage is 71.01911% with 91 lines in your changes missing coverage. Please review.

Project coverage is 70.57%. Comparing base (243bb35) to head (3d29c0f).

Files with missing lines
Patch %
Lines

...in/java/com/taosdata/jdbc/utils/DateTimeUtils.java
58.13%
31 Missing and 5 partials :warning:

...main/java/com/taosdata/jdbc/ws/BlockResultSet.java
43.33%
13 Missing and 4 partials :warning:

...ain/java/com/taosdata/jdbc/rs/ConnectionParam.java
70.00%
7 Missing and 5 partials :warning:

.../java/com/taosdata/jdbc/common/SerializeBlock.java
38.46%
6 Missing and 2 partials :warning:

...java/com/taosdata/jdbc/TSDBResultSetBlockData.java
33.33%
3 Missing and 1 partial :warning:

...in/java/com/taosdata/jdbc/rs/RestfulResultSet.java
0.00%
3 Missing :warning:

...va/com/taosdata/jdbc/ws/TSWSPreparedStatement.java
88.46%
2 Missing and 1 partial :warning:

...va/com/taosdata/jdbc/utils/DataTypeConverUtil.java
88.23%
1 Missing and 1 partial :warning:

src/main/java/com/taosdata/jdbc/utils/Utils.java
50.00%
2 Missing :warning:

...in/java/com/taosdata/jdbc/rs/RestfulStatement.java
50.00%
1 Missing :warning:

... and 3 more

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #208      +/-   ##
============================================
+ Coverage     70.56%   70.57%   +0.01%
- Complexity     2962     3010      +48
============================================
  Files           151      152       +1
  Lines          9755     9932     +177
  Branches       1182     1203      +21
============================================
+ Hits           6884     7010     +126
- Misses         2056     2096      +40
- Partials        815      826      +11

Flag
Coverage Δ

unittests
70.57% <71.01%> (+0.01%)
:arrow_up:

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

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


Book section url not exported

I have a book section that contains an url field. The url field shows up fine in zotero but is missing in the better bibtex exported bib file. What can I do?
better bibtex exports the "urldate" just fine but misses to export the url field.

Is this for BibTeX or BibLaTeX? BibTeX doesn't have url support, so if you want them regardless, you can enable it in the "Export" tab of the BBT preferences.

Great, Thanks! I am actually using BibLaTeX. So I changed the export from BibTeX to BibLaTeX and everything works just fine.


drawing issue with line and rgba colors

If i assign a rgba color to a line with setColor i can see some drawing issues in curves.

is that solvable?
Could we may be get an opacity property for ````VectorElement``` that would solve that issue?
Thanks

This is a known issue and happens with half-transparent overlapping lines. We do not plan to fix this due to major performance impact and some none-trivial techical complexity. The workaround is to carefully choose another color for the line that has alpha=1.0.

@mtehver what about an opacity property for all VectorElement? That would solve the issue

@farfromrefug Actually that was the option I described (basically adding an 'opacity' property of Layer objects), it is expensive performance-wise (separate drawing surface, full screen blending of the surface) and proper implementation would require lots of change. I am keeping this open now, but as I said earlier, we do not have this on the roadmap.

@mtehver sorry i thought you were talking about the drawing of every single VectorElement. Is that difficult too?
I understand your point about layer and surfaces. It would be a great addition, especially for people trying to migrate from google maps, but i understand it might be technically chalenging.
And i can live without it!
Thanks

There is now remedy for this: you should draw lines without any transparency, but then set Opacity for whole Layer as per #184 , now available since SDK 4.1.3


feat: reformat outputs to always show if the secret is known

Codecov Report

Merging #505 (adfde3a) into main (c379922) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #505   +/-   ##
=======================================
  Coverage   93.84%   93.85%
=======================================
  Files          78       78
  Lines        3609     3629   +20
=======================================
+ Hits         3387     3406   +19
- Misses        222      223    +1

Flag
Coverage Δ

unittests
93.85% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

ggshield/cmd/secret/scan/archive.py
100.00% <ø> (ø)

ggshield/cmd/secret/scan/docker.py
100.00% <ø> (ø)

ggshield/cmd/secret/scan/dockerarchive.py
100.00% <ø> (ø)

ggshield/cmd/secret/scan/docset.py
67.74% <ø> (ø)

ggshield/cmd/secret/scan/precommit.py
100.00% <ø> (ø)

ggshield/cmd/secret/scan/prepush.py
97.67% <ø> (ø)

ggshield/cmd/secret/scan/prereceive.py
95.57% <ø> (ø)

ggshield/cmd/secret/scan/pypi.py
95.12% <ø> (ø)

ggshield/cmd/secret/scan/range.py
100.00% <ø> (ø)

...ield/cmd/secret/scan/secret_scan_common_options.py
100.00% <ø> (ø)

... and 7 more

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Windows: Fail fs ops on running Hyper-V containers gracefully

Signed-off-by: John Howard (VM<EMAIL_ADDRESS>@thaJeztah Fixes https://github.com/docker/docker/issues/31606
Note - can't do a CI test for this as CI doesn't support Hyper-V containers on the Azure hosts. Manual verification with the daemon set to run with default isolation of Hyper-V.
PS E:\go\src\github.com\docker\docker> docker run -d --rm microsoft/nanoserver ping -t <IP_ADDRESS>
6fdd49a48088cd269a30dfc1873cbc5da3c625bbf2c6d04bc170fd59d94b823d
PS E:\go\src\github.com\docker\docker> docker cp 6fd:c:\dockerfile e:\dockerfile
Error response from daemon: filesystem operations against a running Hyper-V container are not supported
PS E:\go\src\github.com\docker\docker>

Thanks! Much appreciated 👍
/cc @johnstep ptal

ugh hitting https://github.com/docker/docker/issues/31724

LGTM


Please add a license file and a screen shot

I'd like to add this to awesome-zsh-plugins, but I prefer to add themes with screen shots in the readme so that users can see what they look like without having to install them.
Also, please add a license file, some people won't use any code that doesn't have an open source license. If you don't have one in mind, https://choosealicense.com is a good resource to help you select an appropriate one.
Finally I noticed the readme mentions shrink-path and I think you need to change that to omszt.
Thanks!

Sorry to reply so late. I have added the screen shot and the license file.


Error in team (team list?) in DET-CMP

Detroit Edison division shows team #365 as unnamed from Springfield VA (and their team page lists them as from the GW Community School). Team #365 is (and always has been, I think) team MOE from Delaware. Their team page correctly shows them competing in the DE Championship, etc. So it looks like somehow this team's name, city, state, etc. are in error. I suspect there may be other similar types of errors in team info. ???

BTW, the firstinspires team search correctly shows them as team MOE from Delaware, so it looks like something got messed up somehow on the TOA team info side.

As another data point, team 965 is from Springfield, VA from the GW Community school. It looks like the data from 365 and 965 got comingled somehow?

Yet another data point: All of the team names for 3 digit teams are missing. Maybe an issue with pulling info for 3 digit teams? (252, 310, 365, 701)

This is fixed now. Something to do with how we interpreted FIRST's data a few months ago. I just wrote and  ran our retriever software to update all the teams. We now are up-to-date with all of the data for Teams from TIMS.


fix: strict prerelease channels

This catches if an invalid channel version is provided.

:tada: This PR is included in version 1.7.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v1.7.3
v1.7.3

Your semantic-release bot :package::rocket:


Random crash for OpenGL context on old iOS devices was reintroduced in 1.22

Steps to Reproduce
We are using official Flutter engine and recently we upgraded from v1.12.13+hotfix.8 to 1.22.6.
We found a crash on iOS9 & armv7 device related to OpenGL context.
Step to reproduce:

iOS9 & armv7 device.
Wrap FlutterView in another view controller.

- (void)viewDidLoad {
    [super viewDidLoad];
    [DLTimeTracker beginTimeTrackingForService:@"flutter" label:@"duration"];
    // Do any additional setup after loading the view.
    [DLMonitorMemonry beginMonitorMemonryForService:@"flutter_performance" metric:[@"mem_" stringByAppendingString:self.initialRoute]];
    [self setDeviceOrientationWithRouteName:self.initialRoute];
    [self addChildViewController:self.flutterVC];
    [self.flutterVC didMoveToParentViewController:self];
    self.flutterVC.view.frame = self.view.bounds;
    [self.view addSubview:self.flutterVC.view];
}

self.flutterVC.view will trigger to create iOS GL stuffs and finally crash.
1585_70494727097_1625050432_7d8ca7f6d99111ebb15800163e32753e_symbolicated.txt

Analysis:

To easily trace IOSSwitchableGLContext in release build, I build a local engine with a callback function registered to FlutterEngine so that I can break at this function in main image.

To trace the deallocation of EAGLContext, I replaced 'dealloc' function and add a breakpoint.

In IOSSwitchableGLContext::SetCurrent, current_context is logged.

Continue to run, and it stopped at my replaced 'dealloc', you can see that this EAGLContext is freed and is now a zombie.

Continue to run. In IOSSwitchableGLContext::RemoveCurrent, zombie context instance is being used and crashed.

This issue is brought in by this commit. https://github.com/flutter/engine/commit/1482d9b05e3cb0b0ebd36555e5e30bc8f2734770#diff-c2bad27fb0b75488076d84fda2ce4287303b5602225b5febb3259a2915790459
BTW, in this commit, I believe a previous fix also related to zombie GLContext is missing. The PR is also introduced by me.
https://github.com/flutter/engine/pull/17366/files

@cyanglaz

Hi @wqyfavor
Can you please provide a minimal complete reproducible code sample in a repository
Thank you

Close for now because we have workaround to avoid this crash. And armv7/ios9 is to be deprecated.


Chart Type Update not working

Expected Behavior

I'm trying to make a chart that can switch between multiple views. In this situation, it's changing the type of chart from 'line' to 'horizontalBar' or 'radar'. I've created a simple pen script that would showcase the issue I'm currently having.
I expected after updating the config variable ( that contains all the labels, data and options ) to help render a new Chart with the different type.
Current Behavior

What i've noticed is that I can set a variable
let var = { type: "horizontalBar", data: [ data goes here ] };
and I could use that to generate a new functional Chart. The problem is, the title, positioning of the legend and everything else gets reset to defaults and the title is missing. I'd like to keep all of those settings intact, but if I'd use a similar object as initially, the chart no longer displays any values.
Possible Solution

Steps to Reproduce

On https://codepen.io/derains/pen/jOVywBv I've created a sample of the code that would showcase the problem.

If you click the "Horizontal Update" - this is the one that is expected to work as intended but it renders an empty Chart.
If you click on "Radar Update" - this will work but the INITIAL display of the chart title and legend gets messed up.

Context

Environment

Chart.js version: CDN ( 2.8.0 ) -<EMAIL_ADDRESS>Browser name and version: Google Chrome Version 88.0.4324.150 (Official Build) (64-bit)
Link to your project: https://codepen.io/derains/pen/jOVywBv

If you make your bar at the same way as your radar it works, so instead of setting only the type give it a new object. It is kind of buggy since hover for some reason makes it go back to its old state
https://codepen.io/leelenaleee/pen/vYygZPW

The solution you've provided still has the issue that I'm talking about. The legend and title get reset if I do it as you've mentioned. So it's not a reliable workaround. I would want to work around it if it would provide a way to update the title of the chart along with the legend position.

@LeeLenaleee , after further review, I'd also want to mention that your solution is identical to what I had in place for the Radar part, which as I've mentioned in the post on this issue is not working as expected.

Updated the pen, the options didn't get passed along. This should be what you want, might want to clean it up a bit.

Awesome! That works exactly how I'd expected, could you please help clarify why do you need to set the object using Object.assign() rather than just passing the same object?

I believe it's due to how the options are merged together. This is part of the reason why #8374 is under development

I have a Eureka moment here. The actual cause is not how the options are merged together. The problem seems to be that if you switch from type: 'line' to type: 'horizontalBar', the algorithm somehow doesn't adapt properly. This is further proven by 2 things:

This code:
{scales: {yAxes: config.options.scales.xAxes, xAxes: config.options.scales.yAxes}
Shows an inversion of the axes, without which it seems that the chart crashes.
You don't need the Object.assign() to be able to fix this, you can simply duplicate/store initial values in a variable then inverse them on the time of the script calling and it would solve the problem.

My estimation would be that it has to do with how the code is rendering the values, because something is causing it to either make the x/y have a problem OR the horizontalBar aglorithm is by default set to be with values at the x axis and the labels on the Y axis, which is causing an issue to adapt ( most likely ).
However, after testing something else, it seems that changing the values of the options.scales ( regardless if it's x OR y axis ) would make the render happen. Any ideas?

If you need me to update my codepen to understand the above statement, let me know and I'll do so.

I have further investigated this. It seems that if you would do the following on each new object, it works correctly. This leads me to think that these two items don't get reset properly on the .destroy() method.
this.config.options.scales.yAxes = []; this.config.options.scales.xAxes = [];

No need to destroy/re-create, at least in v3: https://codepen.io/kurkle/pen/XWNaPYw
Would probably make sense to have options objects for each chart types and switch the whole options along the type.

Closing since this looks nicely resolved in v3


Topic transactions

Codecov Report
Attention: Patch coverage is 9.60452% with 320 lines in your changes are missing coverage. Please review.

Project coverage is 41.00%. Comparing base (d33eeab) to head (f41954f).

Files
Patch %
Lines

...src/main/java/tech/ydb/table/impl/BaseSession.java
2.85%
68 Missing :warning:

...main/java/tech/ydb/topic/read/impl/ReaderImpl.java
0.00%
60 Missing :warning:

...c/read/impl/TransactionMessageAccumulatorImpl.java
0.00%
42 Missing :warning:

...c/main/java/tech/ydb/table/impl/TxControlToPb.java
0.00%
34 Missing :warning:

.../java/tech/ydb/topic/settings/ReceiveSettings.java
0.00%
18 Missing :warning:

...src/main/java/tech/ydb/query/impl/SessionImpl.java
68.08%
8 Missing and 7 partials :warning:

.../java/tech/ydb/topic/read/impl/SyncReaderImpl.java
0.00%
12 Missing :warning:

...ain/java/tech/ydb/topic/settings/SendSettings.java
0.00%
11 Missing :warning:

.../java/tech/ydb/topic/write/impl/MessageSender.java
0.00%
8 Missing :warning:

...a/tech/ydb/table/transaction/TableTransaction.java
0.00%
6 Missing :warning:

... and 16 more

Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #243      +/-   ##
=============================================
- Coverage      41.99%   41.00%   -0.99%
+ Complexity      1409     1408       -1
=============================================
  Files            282      289       +7
  Lines          10912    11201     +289
  Branches        1073     1101      +28
=============================================
+ Hits            4582     4593      +11
- Misses          5970     6246     +276
- Partials         360      362       +2

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


WIP: First working version for MiniHub

The sample MiniHub document has been created under path /content/documents/lks/minihubs/working-with-the-public which reference to 2 guidance page named Introduction and Copyright.
This MiniHub pages can be accessed from the local URLs http://localhost:8080/site/minihub/working-with-the-public/introduction and http://localhost:8080/site/minihub/working-with-the-public/copyright

@smanivasag My first working version which used our discussed approach using hst:sitemap

@smanivasag I think we may need to support direct access of minihub page http://localhost:8080/site/minihub/test as well (which would essentially render http://localhost:8080/site/minihub/test/introduction) in order to ensure no missing page hierarchy for SEO optimization. Agree with this. I think I should implement this.

@thojava - Sure, thanks 👍
@thojava @duncanheron @MattBrooksManifesto - Do we have a separate ticket to add canonical tags to pages ? If not, could we add one as we definitely need one to accommodate canonical tags for minihub and minihub's first guidance pages ?
Fyi - Looking at www.nhs.uk website, it seems like they use direct minihub page URL as the minihub's first guidance page URL. E.g. https://www.nhs.uk/conditions/weight-loss-surgery/.
Thanks!

@thojava - Thanks for the latest commits, looks good 👍

@thojava @duncanheron @MattBrooksManifesto - Now that we have added support for both http://localhost:8080/site/minihub/working-with-the-public and http://localhost:8080/site/minihub/working-with-the-public/introduction and that they are aliases, I think it would be good to add a canonical tag to http://localhost:8080/site/minihub/working-with-the-public/introduction indicating that http://localhost:8080/site/minihub/working-with-the-public is the canonical.
On page http://localhost:8080/site/minihub/working-with-the-public/introduction, we can add the following tag:
<link rel="canonical" href="http://localhost:8080/site/minihub/working-with-the-public">

Alternatively, we can also remove the support for http://localhost:8080/site/minihub/working-with-the-public/introduction fom the implementation (which removes the necessity for canonical tag unless we would like to support self-referential canonical tags for the entire website which of course could be handled part of a separate ticket).
Thoughts ?

@smanivasag Thank you for the latest review. Some how I missed things. But I have resolved them

@smanivasag Thank you for the latest review. Some how I missed things. But I have resolved them

No problem @thojava, thanks for resolving them 👍

@duncanheron @MattBrooksManifesto I will go ahead and merge this PR. If there are any enhancement needed for the minihub we can work on new ticket.


Function param with generic type are not recognized correctly

TypeScript Version:  2.7.0-dev.201xxxxx
2.8.3

Search Terms:
Code
class Protocol<T>{}

interface ReqTest {
    abcde: string;
}

const PtlTest = new Protocol<ReqTest>();

function request<T>(ptl: Protocol<T>, req: T) { }

request(PtlTest, {})

Expected behavior:
Compile error.
Because request(PtlTest, {}), param 2 {} not match ReqTest
Actual behavior:
Compile success
Playground Link:
Related Issues:

You need an explicit type argument here, because {} is a valid inference. PtlTest validly a Protocol<{}> (T is never even used, so everything but null or undefined is aProtocol<T> for any T), and {} of course is a valid {}.

@andy-ms shouldn't T be infered from PtlDemo?
Actually it works well in previous TypeScript version.

@andy-ms shouldn't T be infered from PtlDemo?
Actually it works well in previous TypeScript version.

I don't see a compile error in any of<EMAIL_ADDRESS><EMAIL_ADDRESS>typescript@next. Protocol is an empty type, so almost anything is assignable to it.

function test<T>(value1: T, value2: T) {
    return value2
}
test({ a: 1 }, {});

@andy-ms  Just see this.
It would compile error at<EMAIL_ADDRESS>
error TS2345: Argument of type '{}' is not assignable to parameter of type '{ a: number; }'.
Property 'a' is missing in type '{}'.
But compile succ at<EMAIL_ADDRESS>
And it's strange that it works well if it is this:
function test<T>(value1: T, value2: T) {
    return value2
}
test(<{ a: number }>{ a: 1 }, {});

I'll tentatively mark this as a bug due to the inconsistency in:
declare function equal<T>(a: T, b: T): boolean;
equal("", 0); // Error
equal({ a: 1 }, { b: 2 }); // No error

See more about the rationals here in https://github.com/Microsoft/TypeScript/wiki/What's-new-in-TypeScript#improved-type-inference-for-object-literals


feat: implement CairoLayout

Should close #228.

@lana-shanghai As this is CairoLayout and it is possible, in unit tests, to check each parameter in a hardcoded way, I did it. This is obviously a lot of lines of code but this way we are 100% sure and we will always see any changes. Don't hesitate to tell me if it's too many tests I can refactor, but I said to myself that I might as well test everything.

@lana-shanghai FYI I just removed the unnecessary layouts leaving only the following layouts:

Plain
Small
All Cairo
Dynamic (I added it in addition to those that we mentioned yesterday because it is a configurable layout which could be useful very soon in my opinion)


Add LEAP hand MJCF description from Menagerie.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

8 of 8 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 97.374%

Totals

Change from base Build<PHONE_NUMBER>0:
0.02%

Covered Lines:
927

Relevant Lines:
952

💛  - Coveralls

Thanks! :+1:


Feature request: harden-below-nxdomain with NSEC3 opt-out

(Hi! I hope it's appropriate to file feature requests here.)
The documentation for harden-below-nxdomain explains:

          Very  old  software might return nxdomain for empty nonterminals
          (that usually happen for reverse IP address lookups),  and  thus
          may  be  incompatible  with  this.   To  try  to avoid this only
          DNSSEC-secure nxdomains are used, because the old software  does
          not  have DNSSEC.  ...  The nxdomain must be secure,
          this means nsec3 with optout is insufficient.

I'd like to request that harden-below-nxdomain also apply to NSEC3 opt-out responses. The critical thing that indicates RFC 8020 compliance is whether the zone is secure or not. Even if the response lacks the AD bit because it's using NSEC3 opt-out, the authoritative DNS server is still implementing DNSSEC and ought to be handling ENTs correctly.
To quote myself making an equivalent feature request in PowerDNS/pdns#9901:
It would complicate the code, but from a user perspective, I think it would be nice if harden-below-nxdomain did also cover NSEC3 opt-out. This is a total guess, but I'd think queries for expired or misspelled domains in NSEC3 opt-out TLDs would be a medium-sized fraction of NXDOMAINs.
But I understand if it's not worth it. :)

There's a solid argument that harden-below-nxdomain shouldn't apply to insecure responses (like NSEC3 opt-out responses) to reduce the damage caused by forged NXDOMAINs. But that's not the argument currently being made in the man page.

The difference is that for NSEC3 optout, underneath the optout is is nearly impossible to figure out where the zone really is. You may say that it is possible, with debug information, but from the results from the upstream wire not so. We can tell it is optout, but then if it is still 'in zone' or not, is not that easy, certainly not for co-hosted zones where there is an NSEC3 opt-out insecure delegation from one zone to the other.
Your argument that the server may still be newer, and thus it would work fine for unsigned zones as well, is true for a whole bunch of servers. But critically, not all of them, and specifically, for a number of zones hosted under load balancing efforts, like from a cdn, those are from different sourced servers. And those have an issue detecting zones underneath the nxdomain, because they may not know about it.
So if we enable this for nsec3-optout, and then in .com, this happens to a load balanced domain in a CDN, I think it would disappear?

Hmm. Can you trust one label underneath a secure zone?
Hypothetical example: com is a secure TLD. example.com doesn't exist.
Putting aside the issue of foo.example.com or foo.bar.example.com, can you rely on an NSEC3 opt-out NXDOMAIN for example.com?
When I filed this issue, I would have said "yes", but now I'm not entirely sure.
Hypothetical scenario:
com is a secure zone.
example.com is an insecure zone cut seemingly hosted on the same nameservers.
Verisign actually uses dnsdist to direct queries for example.com/DS to their good nameservers, and other example.com queries to a horrible load balancer.
example.com/DS gets an NSEC3 opt-out NODATA response.
example.com/A gets NXDOMAIN.
www.example.com/A returns real A records.
That case seems possible (but maybe tricky) to distinguish.

Not sure, without the secure on the NXDOMAIN itself, how to know what type of server it is from.
If we just allow optout ranges for NSEC3, that would be similar to allowing use for unsigned zones. If anything under the nsec3 is allowed. But one label only, not sure, if that would work. And that could depend on individual cases. Is the case you cite something that happens, you think? I would mean you could not really use that NXDOMAIN to harden-below from.


Batching commands triggered by {Normal,Insert}Idle

It seems that we are getting more and more of requests which are expected to be hooked into {Normal,Insert}Idle (buffer change, semantic highlighting, inlay hints, auto-hover...). I wonder if we can add some kind of batching support, so Kakoune sends only a single request which is then transformed by kak-lsp daemon into multiple requests to language server and stitched back into a single response to the editor.

We should also be able to set higher idle intervals for all/some of these requests. Thats probably best done on the kak-lsp side.

@ul would we want to stitch them back into a single response for the editor?
Some LS features might be more computationally expensive than others.
If we stitch them back together, we have to wait for the slowest operation.

It seems I was too fast in PR comment assuming we want stitching responses in this use case. I don't see any significant benefits in doing it. At the same time the drawback of waiting for the slowest response is quite huge.


Grafana - Cannot read properties of undefined (reading 'digest')

What happened: hi, we are trying to view the metrics of the opensearch version on grafana using the opensearch datasource, but the following error is returned once the dashboard is created: Cannot read properties of undefined (reading 'digest')
What you expected to happen: we expect the data to appear
How to reproduce it (as minimally and precisely as possible): have grafana version 10.1.2, opensearch version 2.9 (Service software version: OpenSearch_2_9_R20230926), OpenSearch plugin version 2.13.0 and try to make a dashboard with opensearch datasource
Anything else we need to know?:
Environment:

Grafana version: 10.1.2
OpenSearch version:  2.9
Plugin version: 2.13.0

Hi @skycannavod ! I haven’t been able to reproduce the issue yet, but I have a few questions:

What response are you getting from Opensearch for the query? (Please redact any sensitive information, but is it returning an error, what’s the structure of the response, and what headers is it getting?)
What auth options are you using for your OpenSearch datasource?

grafana gives me the error: "Cannot read properties of undefined (reading 'digest')", if I try to execute the query on "Query Inspector" it remains in a loop
as regards the auth we use the SigV4 auth AWS SDK Default, once the auth and the index pattern have been set, it takes everything correctly, but when I try to view the data on the dashboard it gives me the error explained above

Hi @skycannavod ! I took another look at this, and the crypto interface that we're using to generate the SigV4 signatures requires that it's run in a secure context. Are you running Grafana in a secure context?

Hi @skycannavod I have set up the following locally according to your notes:

grafana version 10.1.2
opensearch version 2.9 (Service software version: OpenSearch_2_9_R20230926)
OpenSearch plugin version 2.13.0
SigV4 auth AWS SDK Default
our grafana is on http, opensearch in https (I am building and running grafana on localhost, opensearch hosted on AWS)

Finally, the fact that you have this "wrong" ElasticSearch 7.10.2 version sounds like this issue so maybe you have compatibility mode on. I also created a datasource with compatibility enabled which gives me a similar version in the OpenSearch plugin. I went to the Dashboard as in your gif and was unable to reproduce your issue.
Here's what I have set up, can you let me know if it's the same as your setup? Also let me know if any of the above differs from your setup.

Hi, yes the configuration is the same, with compatibility mode on

Thanks for the info.
I spoke to @sarahzinger about this, and looping back to our grafana is on http, opensearch in https, this may be an issue as @iwysiu mentioned above. In my case using localhost is considered secure, but if you are elsewhere this may not be sufficient. You might need to either run Grafana locally or run it somewhere with https.
It may be that our refactor which is still in progress will resolve this issue for you because it will no longer depend on your browser's context security. This work is done for some queries and not for others. As a test, would you be able to run a "Raw Data" query and tell us if you still observe the error?

Once this refactor is done, you should be able to see data with your current setup.
Right now you need to either run Grafana locally or run it somewhere with https.

Hi @skycannavod can you confirm that this is fixed in the latest version of the plugin? Thanks!

Hi @idastambuk, i confirm that it works
Thanks!


SPE-14359 bug fix patch

@TylerHendrickson I still need to investigate a couple test errors, but this is ready for initial review

@TylerHendrickson tests now run with python run-tests.py

@lindseybrockman Looks good!  Peer reviewed.


clustermesh-apiserver: fix key name for delete during k8s->kvstore sync

The handler which synchronizes CiliumNodes from k8s to the KV store
includes the cluster name in the update operation but not the delete
operation. This might lead to nodes not being correctly deleted from the
KV store when they disappear.
Follows commit 0ea6cba801d4 ("operator: fix key name for delete during
k8s->kvstore sync") which did the same for the operator.

/test

Reviews are in, all tests passed except for test-1.16-4.9 which failed on known flake #20288. Marking as ready to merge.


UI: Blank volume details page after volume deletion from drop down menu

Longhorn UI version: e077811
Steps to reproduce:

Create a volume.
Go to volume details page.
From upper right drop down menu click Delete, After successfully deletion volume, volume details get removed and page stay blank.

Suggestion:  we may redirect the user back to volume list page.

Longhorn UI version: 9f1e1cf
UI now redirect to volume list page after deletion from drop down menu in volume details page.


Website crashes on opening.

Screenshot

Reproduce

Open Website
Click login via GitHub button

Now the website crashes all the time while opening.
I have tried to disable all track/ad blockers, but that didn't help.
Error log
react-dom.production.min.js:4483 TypeError: Cannot read property 'issues' of null
    at Se (Goals.js:28)
    at ia (react-dom.production.min.js:3550)
    at Pa (react-dom.production.min.js:4050)
    at el (react-dom.production.min.js:6548)
    at wl (react-dom.production.min.js:5531)
    at bl (react-dom.production.min.js:5520)
    at cl (react-dom.production.min.js:5252)
    at react-dom.production.min.js:2775
    at scheduler.production.min.js:266
    at Qo (react-dom.production.min.js:2740)
ec @ react-dom.production.min.js:4483
(anonymous) @ react-dom.production.min.js:4928
mi @ react-dom.production.min.js:2998
vi @ react-dom.production.min.js:2986
Sl @ react-dom.production.min.js:6222
(anonymous) @ scheduler.production.min.js:266
Qo @ react-dom.production.min.js:2740
kl @ react-dom.production.min.js:5985
cl @ react-dom.production.min.js:5267
(anonymous) @ react-dom.production.min.js:2775
(anonymous) @ scheduler.production.min.js:266
Qo @ react-dom.production.min.js:2740
$o @ react-dom.production.min.js:2770
Wo @ react-dom.production.min.js:2760
nl @ react-dom.production.min.js:5041
ma @ react-dom.production.min.js:3775
(anonymous) @ Goals.js:18
Promise.then (async)
(anonymous) @ Goals.js:16
rc @ react-dom.production.min.js:4553
zl @ react-dom.production.min.js:6351
(anonymous) @ scheduler.production.min.js:266
Qo @ react-dom.production.min.js:2740
Tl @ react-dom.production.min.js:6332
(anonymous) @ react-dom.production.min.js:6321
D @ scheduler.production.min.js:203
(anonymous) @ scheduler.production.min.js:94
Goals.js:28 Uncaught (in promise) TypeError: Cannot read property 'issues' of null
    at Se (Goals.js:28)
    at ia (react-dom.production.min.js:3550)
    at Pa (react-dom.production.min.js:4050)
    at el (react-dom.production.min.js:6548)
    at wl (react-dom.production.min.js:5531)
    at bl (react-dom.production.min.js:5520)
    at cl (react-dom.production.min.js:5252)
    at react-dom.production.min.js:2775
    at scheduler.production.min.js:266
    at Qo (react-dom.production.min.js:2740)

Thanks so much for opening this. Sorry I missed this until now.
I will look into this tonight.

I'll investigate to see if I come up with a fix


Update Graphql top-level url.

Coverage remained the same at 88.234% when pulling e0fc3e767bc5381274f302086a208c5279c8e0d4 on akatsoulas:update-graphql-url into ba950071c6c10c2741387dbb9a99a46cff03da00 on mozilla:master.


pull request for e2e test at 01/20/2017 00:45:16

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Improve merge() algorithm

This is revival of my previous https://github.com/dlang/phobos/pull/3315 merged with existing algorithm implementation.
Improvements:

Bidirectional support if all inputs are bidirectional
Propagates infiniteness if any of the inputs is infinite.

Discussion:
Should non-sortedness of any input be considered an error? If so, in debug mode, we could assert that all inputs are sorted according according less. This has same time-complexity as the algorithm itself. A cheaper alternative is to do lazy assert of sortedness of output. However, this doesn't separate the merge algorithm from the out contract checking.
Other notes:

Checking for isSortedRange in my previous implementation has been removed because we currently cannot compare all the different formats for the comparison alias (pred) fed to SortedRange.
CommonElementType is just a suggestion from my side, as it's used twice both in the template restriction and inside the definition. I can, of course, expand away it's definition, if you think otherwise.

Update:
I just noticed that Merge is used in setops. I guess this removes the opportunity for making this a Voldermort type, right?

You may assert that the ranges are sorted opportunistically; an eager call to isSorted seems a bit much.

So no debug assertions of sortedness of output either, then?

Only thing left now is potential checking for sortedness in frontIndex.

Ok, REFs look good to me.

Added debug-only efficient checking for sortedness inside Merge.pop{Front,Back}.

Current coverage is 89.39% (diff: 93.40%)

Merging #4888 into master will increase coverage by <.01%

@@             master      #4888   diff @@
==========================================
  Files           124        124
  Lines         76659      76704    +45
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          68527      68570    +43
- Misses         8132       8134     +2
  Partials          0          0

Powered by Codecov. Last update 027839f...e1dfb51

@andralex ping

As I mentioned via email I'm on the road this week.

Ok. Thanks, @andralex
Can anyone else approve this?

So I guess I should remove the ref again then?

Yes, we shouldn't link to deprecated symbols. Sorry.

Restore.

ping, @andralex

On 11/21/2016 10:53 PM, Per Nordlöw wrote:

These |foreach|es are static ones. I'm pretty sure that the break
associates with |switch| here. If it we're I don't see how the
unittests could pass.

You're mistaken. The break applies to the foreach. The behavior is a bit
weird, though: The break doesn't end the static foreach loop. It jumps
behind the foreach body at run-time, still inside the switch. If there's
code there (e.g. a default case) that gets executed. There may be other
subtle differences.
To break the switch instead, use a label:
sw: switch (foo)
{
     foreach (x; alias_seq)
     {
         case x: /* ... */ break sw;
     }
}

Ok, @aG0aep6G I'll fix.

@aG0aep6G Changed the two switch breaks to be labeled.

And cleaned up the order of the imports and private defs at the top of Merge.

@aG0aep6G, what do you think about the updated limiting of the sortedness logic?

Sorry, I'm not following the actual objective of this PR. Just skimming the messages.
By the way, I think you've edited in that question later on. It's not in the notification email from GItHub. I wouldn't have noticed it if I hadn't had this page open in the browser.
Also, you seem to update the PR by amending. In my experience, that makes it much harder to review the changes. I'd suggest to squash once at the end instead of amending. But Andrei does the actual review here, so that's his call.

Removed isSorted calls in unittest.

Ok, I'll change to not amending and then squashing at the end.

Rebased on master. One thing has changed: improved sortedness checking by breaking out CT-logic in canCheckSortedness.

ci/circleci — Your tests failed on CircleCI fails. Don't know why.

Ready to pull?

Good engineering, thanks.


Publishable Dokka dokumentation

This adds support for generating documentation for the public consumable API.
The dokka artifact is not immediately consumable from the IDE, but I added the missing source jars, so now it should be available for all targets. This should be sufficient to make the dokka-documentation available in Quick Documentation etc.
The task of writing meaningful documentation is considered to be part of the individual issues for the major API parts (#77,  #83, etc.)
Includes

[x] Generate Dokka documentation for library and runtime-api
[x] Generate distributable containing the documentation

@cmelchior Somehow missed that you put in some final remarks. Will address them in next PR (#107) instead

@cmelchior Somehow missed that you put in some final remarks. Will address them in next PR (#107) instead


strange chars on decompression

hello,
i'm developing a mobile app, but sometimes i have problem using decompressFromUtf16, because it works but gives strange chars on result. This problem seems to happen only on Iphone
This is what i do:

from Cordova app i compress (compressToUtf16) a json based string and i send it to a .net web service.
It's a simple string with userid, password, name, etc
the web service takes the string, decompress it (decompressFromUtf16) and sometimes it works and sometimes not, displaying half string ok and half not, with uncomprensible chars.

I'm not able to understand if the problem is on javascript compression or .NET decompression
thanks for help!

i found that the problem is in the javascript compressToUtf16, you can see the strange chars inside the compressed string ("PLU", "IND"):

Can you give me a string that you cannot decompress?

On another note, the best way to determine if the problem is in the Js or C# side it to try to decompress the string in the browser before sending it to see if it works. If it works it means the JS code is able to decompress the compressed version.
From there there could be two issues:

could be in the transport layer. Make sure your string is the same on the server side than it was in the client side. For the transport I recommend compressToEncodedURIComponent which only produces uri-safe characters. So make sure the compressed string in the browser is the exact same as he one on the server.

There could be a bug in the C# decompression routine.

Hello.
I have a very similar problem. I am not sure if the problem is when compressing or decompressing. Here are the original string I am trying to compress/decompress, and the resulting string after decompression, so you can compare and find out what the problem is.

Thank you very much in advance.

@rreghin i just tested your original string  and the decompressed text was exactly the same.
i used the following code:
var string = 'YOUR_STRING_HERE';
console.log(string + "\n" + string.length);
var compressed = LZString.compress(string);
var stringFinal = LZString.decompress(compressed);
console.log(string + "\n" + stringFinal.length);

@zyphersworld did you try using compressToUTF16, or any other than compress?

@rreghin I wrote a small page to test this issue: http://palmdrive.net/lzstring/
It works on every browser I have tested so far. Can you tell me if that fails on your end? Just have a look at the source code, it is dead simple.

Ok, so I was testing the wrong stuff. I fixed the page and now the test is successful in all browsers I've tested so far (linux Chrome+Firefox and my iPhone's). Can you tell me if you can get it to fail? And if so on what browser/platform ?

@fsimon21 ANy update on this issue ?

sorry for my late answer, no i haven't tried yet because i had to finish another project.
I will try with the test page you have provided, thanks!

@pieroxy I forgot to mention I'm NOT running on a browser. I'm using Java JDK 1.6.0 Mozilla's Rhino JavaScript script engine.
This is an extract from the Oracle's site about it:
"Sun's implementation of JDK 6 is co-bundled with the Mozilla Rhino based JavaScript script engine. This is based on Mozilla Rhino version 1.6R2. ... " (http://docs.oracle.com/javase/6/docs/technotes/guides/scripting/programmer_guide/#jsengine)
Maybe the compression function would work 100% on newer JDKs, but I cannot update to any other version. I'm stuck to this very version JDK/JRE 1.6.0.
Could this information be of any help to you to solve the problem?

Errrr, yes, it is a great deal different. I'll be testing this and will let you know.

@pieroxy, thank you for all the attention you are giving to this problem!!
I worked the problem around, changing the conflicting character with another one. I know, it´s ugly, but I needed the compression working right away.
That's what I did...
When compressing: result = LZString.compress(data.replace(/-/g,'¬'));
When decompressing: result = LZString.decompress(data).replace(/¬/g,'-');
The conflicting character seems to be the dash "-".

Any update here?  Still running into the same issue running on Oracle's NetSuite servers using SuiteScript modules.  Loading the module in a browser works perfectly but when it runs on Oracle's server, several of the hyphens in strings of numbers get alternately corrupted into 0's and removed entirely.

Facing some issue while decompressing the string:
ጂ邬ǁÀ鳐䍨ࣚŰ㎁᦭Ⱉ肁곳 뀯您₀ﶀीᘌͥ쯮緯각ᐴ䣱挤井㙌拾ᐮ酋ବ㒅革㴇㉸ ៏鰥狝ꌀ 瀃悀➀匀一耜퀁䖐ŕⰀ밀딂Ą鉁 Ŝƌ墒<䕹腱議耀

@niteshwashivale Can you provide the original uncompressed string ?

@pieroxy, thank you for all the attention you are giving to this problem!!
I worked the problem around, changing the conflicting character with another one. I know, it´s ugly, but I needed the compression working right away.
That's what I did...
When compressing: result = LZString.compress(data.replace(/-/g,'¬'));
When decompressing: result = LZString.decompress(data).replace(/¬/g,'-');
The conflicting character seems to be the dash "-".

I just re-read your message. As per the documentation, DO NOT USE THE compress METHOD. It is buggy.
So if you cannot decompress, it's expected.
You should use compresstoUtf16 instead.