Provide a get API on the dialect object to retrieve registered dialects so they can be extended

It would be great to be able to retrieve registered dialects in order to extend them. For e.g.
let MySqlContext = xql.dialect.get("mysql");
class CustomMySqlContext extends MySqlContext {
 ...
}

Commit https://github.com/exjs/xql/commit/454f7bd3b3731eb752422a44e85071e925a6aa99 adds the possiblity to introspect SQL dialects through xql.dialect.registry object. Thanks for this suggestion!


number/boolean values not working

re #9560
{
    "command": "echo",
    "isShellCommand": true,
    "args": ["Hello World", "${workspaceRoot}", "${config.editor.lineNumber}"],
    "showOutput": "always",
    "version": "0.1.0"
}

have a task.json like above
the value of editor.lineNumber isn't interpreted, the same for boolean values

I'd expect that I can use primitive values like boolean and number in addition to string. Maybe even arrays.

@jrieken, I believe this can be resolved. Will submit a new PR for this tomorrow once I have tested the code thoroughly and updated the unit tests.
Thanks for identifying this.

@DonJayamanne if you plan to work on an item in this iteration than assign it to the july milestone. If the code is unit tested and the changeset is not huge feel free to push to master without a pr. Though if you prefer a code reveiw feel free to submit a pr and assign to me :)

@isidorn, it is a small change and will push directly to master tomorrow. Cheers.

@jrieken, please re-test #9560 with the new insider build when the new build gets released.
In the future, do I just close these issues (such as this) or wait for you to close it?
Thanks.

@DonJayamanne we just close the issue once we pushed the fix. @jrieken does not need to re-test this now, during our verification process (usually thursday) bug creators verify that the fix is good. More about verification can be found here
https://github.com/Microsoft/vscode/wiki/Issue-Tracking#verification


The Kubernetes nginx CSI example test is failed

Is this a BUG REPORT or FEATURE REQUEST?:
Maybe a bug or not
What happened:
I follow the two guides to install Kubernetes & OpenSDS successfully and test them.
Everything is OK, but nginx CSI example test is failed.
https://github.com/opensds/opensds/wiki/OpenSDS-Integration-with-Kubernetes-CSI
https://github.com/opensds/opensds/wiki/OpenSDS-Cluster-Installation-through-Ansible
What you expected to happen:
The kubectl create -f nginx.yaml was passed,
but the kubectl get pod nginx got a nginx pod in Pending status.
I expected the nginx was in Running status.
How to reproduce it (as minimally and precisely as possible):
(1) Install Kubernetes & OpenSDS on one Virtualbox VM
(2) The used csi-configmap-opensdsplugin.yaml during OpenSDS deployment
kind: ConfigMap
apiVersion: v1
metadata:
  name: csi-configmap-opensdsplugin
data:
  opensdsendpoint: http://<IP_ADDRESS>:50040
  opensdsauthstrategy: noauth
  opensdsstoragetype: block
  osauthurl: http://<IP_ADDRESS>/identity
  osusername: admin
  ospassword: opensds@123
  ostenantname: admin
  osprojectname: admin
  osuserdomainid: default
  passwordencrypter: aes
  enableEncrypted: F
  osuserdomainid: default

(3) After finishing OpenSDS deployment, run kubectl get pod to show
the four pods: csi-attacher-opensdsplugin, csi-nodeplugin-opensdsplugin,
csi-provisioner-opensdsplugin and csi-snapshotter-opensdsplugin were Running normally.
(4) Run kubectl create -f nginx.yaml and kubectl get pod nginx
Anything else we need to know?:
The kubectl get pvc showed csi-pvc-opensdsplugin was in Pending status and
kubectl describe pvc csi-pvc-opensdsplugin showed a failed message:
"failed to provision volume with StorageClass "csi-sc-opensdsplugin": rpc error: code =
InvalidArgument desc = get profile abc failed"
Detailed message snapshot:
https://1drv.ms/u/s!AtJnzpDx-p62uDRM8QWCFBBL00-E
Environment:

NBP version: latest version
OS (e.g. from /etc/os-release): ubuntu 16.04.6 LTS
Kernel (e.g. uname -a): 4.15.0-45-generic
Install tools: ansible
Docker version: 18.06.1-ce
Kubernetes version: v1.14.0

Hi Kevin, thanks for telling us the issue. From message you provided, maybe on problem is that csi plugin was installed correctly. I assume you have changed the host_ip (such as: <IP_ADDRESS>) in installer configuration. please update configmap like the bellowing:
kind: ConfigMap
apiVersion: v1
metadata:
  name: csi-configmap-opensdsplugin
data:
  opensdsendpoint: http://<IP_ADDRESS>:50040
  opensdsauthstrategy: keystone
  opensdsstoragetype: block
  osauthurl: http://<IP_ADDRESS>/identity
  osusername: admin
  ospassword: opensds@123
  ostenantname: admin
  osprojectname: admin
  osuserdomainid: default
  passwordencrypter: aes
  enableEncrypted: F
  osuserdomainid: default

Then re-install  csi plugin by using kubectl delete/create -f *

Thanks Erik,
But the HOST_IP has same IP configuration as configmap file.
The export HOST_IP=<IP_ADDRESS> was configed before running opensds-ansible playbook deployment.
After opensds deployment, the <IP_ADDRESS> was also applied to {your_real_host_ip} in opensds CLI tool configuration and all osdsctl command tests are OK.

OK, maybe I know the issue. As a container, CSI plugin cannot reach to opensds hotpot service if we use <IP_ADDRESS>.  So we should change the HOST_IP.

The HOST_IP has been changed from <IP_ADDRESS> to <IP_ADDRESS>,
but the issue still exists. I highlight the installation and test details as follows.
Kubernetes installation and test:
 ENABLE_DAEMON=true ALLOW_PRIVILEGED=true
 FEATURE_GATES=VolumeSnapshotDataSource=true
 RUNTIME_CONFIG="storage.k8s.io/v1alpha1=true" LOG_LEVEL=5 hack/local-up-cluster.sh -O

 kubectl get nodes

NAME
STATUS
ROLES
AGE
VERSION

<IP_ADDRESS>
Ready
none
51m
v1.14.0

OpenSDS installation and osdsctl test:
group_vars/common.yml
host_ip: <IP_ADDRESS>

group_vars/sushi.yml:
nbp_plugin_type: csi

export HOST_IP=<IP_ADDRESS>

osdsctl profile create '{"name": "default", "description": "default policy", "storageType": "block"}'
osdsctl volume create 1 --name=test-001
osdsctl volume list

Id
Name
Description
GroupId
Size
Status
ProfileId

13311bbe-a46c-4403-acfd-0589f449cd21
test-001

1
available
d10ec339-3357-43ff-8626-4ccdb854af3d

OpenSDS CSI Plugin test:
cat csi-configmap-opensdsplugin.yaml
kind: ConfigMap
apiVersion: v1
metadata:
  name: csi-configmap-opensdsplugin
data:
  opensdsendpoint: http://<IP_ADDRESS>:50040
  opensdsauthstrategy: keystone
  opensdsstoragetype: block
  osauthurl: http://<IP_ADDRESS>/identity
  osusername: admin
  ospassword: opensds@123
  ostenantname: admin
  osprojectname: admin
  osuserdomainid: default
  passwordencrypter: aes
  enableEncrypted: F
  osuserdomainid: default

kubectl create -f nginx.yaml
kubectl get pods

NAME
READY
STATUS
RESTARTS
AGE

csi-attacher-opensdsplugin-0
3/3
Running
0
34m

csi-nodeplugin-opensdsplugin-gd9j2
2/2
Running
0
34m

csi-provisioner-opensdsplugin-0
2/2
Running
0
34m

csi-snapshotter-opensdsplugin-0
2/2
Running
0
34m

nginx
0/1
Pending
0
27m

kubectl get pvc

NAME
STATUS
VOLUME
CAPACITY
ACCESS MODES
STORAGECLASS
AGE

csi-pvc-opensdsplugin
Pending

csi-sc-opensdsplugin
30m

kubectl describe pvc csi-pvc-opensdsplugin
Name:          csi-pvc-opensdsplugin
Namespace:     default
StorageClass:  csi-sc-opensdsplugin
Status:        Pending
Volume:
Labels:
Annotations:   volume.beta.kubernetes.io/storage-provisioner: csi-opensdsplugin
Finalizers:    [kubernetes.io/pvc-protection]
Capacity:
Access Modes:
VolumeMode:    Filesystem
Events:

Type
Reason
Age
From
Message

Normal
ExternalProvisioning
4m46s (x121 over 34m)
persistentvolume-controller
waiting for a volume to be created, either by external provisioner "csi-opensdsplugin" or manually created by system administrator

Normal
Provisioning
73s (x15 over 34m)
csi-opensdsplugin_csi-provisioner-opensdsplugin-0_93c03670-ade6-11e9-a4aa-0242ac110003
External provisioner is provisioning volume for claim "default/csi-pvc-opensdsplugin"

Warning
ProvisioningFailed
73s (x15 over 34m)
csi-opensdsplugin_csi-provisioner-opensdsplugin-0_93c03670-ade6-11e9-a4aa-0242ac110003
failed to provision volume with StorageClass "csi-sc-opensdsplugin": rpc error: code = InvalidArgument desc = get profile abc failed

Thanks Kevin, it's very detailed. Profile need to be updated in nginx.yml, I'm sorry the wiki didn't mention it.
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: csi-sc-opensdsplugin
provisioner: csi-opensdsplugin
parameters:
  attachMode: rw
  profile: d10ec339-3357-43ff-8626-4ccdb854af3d

BTW, I updated the CSI WIKI:
https://github.com/opensds/opensds/wiki/OpenSDS-Integration-with-Kubernetes-CSI
Some of us are working on container storage, you can join opensds.slack.com to get quick feedback. :)

Thank for your invitation, I have joined your channels.
The updated profile has released the pended persistent volume claim, but the nginx pod still stuck at a ContainerCreating status.
kubectl get nodes

NAME
STATUS
ROLES
AGE
VERSION

<IP_ADDRESS>
Ready
none
29h
v1.14.0

kubectl create -f nginx.yaml
kubectl get pvc

NAME
STATUS
VOLUME
CAPACITY
ACCESS MODES
STORAGECLASS
AGE

csi-pvc-opensdsplugin
Bound
pvc-5c00e92f-addb-11e9-aae2-0800275197c7
1Gi
RWX
csi-sc-opensdsplugin
21s

kubectl get pod

NAME
READY
STATUS
RESTARTS
AGE

csi-attacher-opensdsplugin-0
3/3
Running
1
8m51s

csi-nodeplugin-opensdsplugin-n7nlp
2/2
Running
0
8m50s

csi-provisioner-opensdsplugin-0
2/2
Running
0
8m50s

csi-snapshotter-opensdsplugin-0
2/2
Running
0
8m50s

nginx
0/1
ContainerCreating
0
31s

kubectl describe pod nginx
    Name: nginx
    Namespace: default
    Priority: 0
    PriorityClassName: <none>
    Node: <IP_ADDRESS>/<IP_ADDRESS>
    Start Time: Wed, 24 Jul 2019 14:22:11 +0800
    Labels: <none>
    Annotations: <none>
    Status: Pending
    IP:
    Containers:
      nginx:
        Container ID:
        Image: nginx
        Image ID:
        Port: 80/TCP
        Host Port: 0/TCP
        State: Waiting
          Reason: ContainerCreating
        Ready: False
        Restart Count: 0
        Environment: <none>
        Mounts:
          /var/lib/www/html from csi-data-opensdsplugin (rw)
          /var/run/secrets/kubernetes.io/serviceaccount from default-token-z7vs2 (ro)
    Conditions:
      Type Status
      Initialized True
      Ready False
      ContainersReady False
      PodScheduled True
    Volumes:
      csi-data-opensdsplugin:
        Type: PersistentVolumeClaim (a reference to a PersistentVolumeClaim in the same namespace)
        ClaimName: csi-pvc-opensdsplugin
        ReadOnly: false
      default-token-z7vs2:
        Type: Secret (a volume populated by a Secret)
        SecretName: default-token-z7vs2
        Optional: false
    QoS Class: BestEffort
    Node-Selectors: <none>
    Tolerations: node.kubernetes.io/not-ready:NoExecute for 300s
                 node.kubernetes.io/unreachable:NoExecute for 300s
    Events:

Type
Reason
Age
From
Message

Warning
FailedScheduling
46s (x3 over 48s)
default-scheduler
pod has unbound immediate PersistentVolumeClaims

Normal
Scheduled
43s
default-scheduler
Successfully assigned default/nginx to <IP_ADDRESS>

Normal
SuccessfulAttachVolume
43s
attachdetach-controller
AttachVolume.Attach succeeded for volume "pvc-5c00e92f-addb-11e9-aae2-0800275197c7"

Warning
FailedMount
13s (x6 over 34s)
kubelet, <IP_ADDRESS>
MountVolume.MountDevice failed for volume "pvc-5c00e92f-addb-11e9-aae2-0800275197c7" : rpc error: code = FailedPrecondition desc = failed to find device: Please stop the iscsi process outside the container first: exit status 1

Can you please check if the /sbin/iscsid exists? Generally if it doesn't exist you can install open-iscsi manually.

Erik,
The /sbin/iscsid was already installed, I resolved the issue by killing the iscsid processes.

My system woks fine, but I will check the mount function to find the reason, thanks Kevin.

@wisererik @kevinjantw If the example test is working fine, Can we close this issue?

The /sbin/iscsid was already installed, I resolved the issue by killing iscsid processes.
Now nginx pod can execute normally into Running status.

Hello~ I got similar issue in my system. Can you know me how to resolve the issue? Did you kill process of iscsi?

@himanshuvar please have a look at it again, thanks~

I changed configMap from domain name to Cluster-IP in kubernetes service.
it starts to work. but I failed to apply "nginx.yaml"
please check syslog. let me know if you need to more information.
# This YAML file contains configmap objects,
# which are necessary to run csi opensds plugin.
kind: ConfigMap
apiVersion: v1
metadata:
name: csi-configmap-opensdsplugin
data:
opensdsendpoint: http://apiserver.opensds.svc.cluster.local:50040
opensdsauthstrategy: keystone
opensdsstoragetype: block
osauthurl: http://authchecker.opensds.svc.cluster.local/identity
osusername: admin
ospassword: opensds@123
ostenantname: admin
osprojectname: admin
osuserdomainid: default
passwordencrypter: aes
enableEncrypted: F
osuserdomainid: default
root@opensds:~/nbp/csi/server/deploy/kubernetes# kubectl get all
NAME                                     READY   STATUS    RESTARTS   AGE
pod/csi-attacher-opensdsplugin-0         3/3     Running   0          6m22s
pod/csi-nodeplugin-opensdsplugin-9sztb   2/2     Running   0          6m21s
pod/csi-provisioner-opensdsplugin-0      2/2     Running   0          6m20s
pod/csi-snapshotter-opensdsplugin-0      2/2     Running   0          6m20s
root@opensds:~# cat nginx.yaml
This YAML file contains nginx & csi opensds driver objects,
which are necessary to run nginx with csi opensds driver.
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
name: csi-sc-opensdsplugin
provisioner: csi-opensdsplugin
parameters:
attachMode: rw
profile: 477132ef-d9b6-4a2f-a0a0-a0468be08d90
allowedTopologies:

matchLabelExpressions:

key: topology.csi-opensdsplugin/zone
values:

default'

/var/log/syslog
Sep  6 20:48:39 localhost kubelet[1109]: E0906 20:48:39.722420    1109 nestedpendingoperations.go:270] Operation for ""kubernetes.io/csi/csi-opensdsplugin^47d2555a-ba5c-4f47-a22c-d8f0d2be0189"" failed. No retries permitted until 2019-09-06 20:49:43.722403621 +0900 KST m=+24114.113890425 (durationBeforeRetry 1m4s). Error: "MountVolume.MountDevice failed for volume "pvc-2398729a-6daf-4835-a215-f69d18e5c37f" (UniqueName: "kubernetes.io/csi/csi-opensdsplugin^47d2555a-ba5c-4f47-a22c-d8f0d2be0189") pod "nginx" (UID: "41d344d7-ab0b-4359-b983-cb1c690575b1") : rpc error: code = FailedPrecondition desc = failed to find device: exit status 21"
Sep  6 20:48:52 localhost kubelet[1109]: I0906 20:48:52.848147    1109 reconciler.go:177] operationExecutor.UnmountVolume started for volume "default-token-t4wqc" (UniqueName: "kubernetes.io/secret/41d344d7-ab0b-4359-b983-cb1c690575b1-default-token-t4wqc") pod "41d344d7-ab0b-4359-b983-cb1c690575b1" (UID: "41d344d7-ab0b-4359-b983-cb1c690575b1")
Sep  6 20:48:52 localhost kubelet[1109]: I0906 20:48:52.862561    1109 operation_generator.go:860] UnmountVolume.TearDown succeeded for volume "kubernetes.io/secret/41d344d7-ab0b-4359-b983-cb1c690575b1-default-token-t4wqc" (OuterVolumeSpecName: "default-token-t4wqc") pod "41d344d7-ab0b-4359-b983-cb1c690575b1" (UID: "41d344d7-ab0b-4359-b983-cb1c690575b1"). InnerVolumeSpecName "default-token-t4wqc". PluginName "kubernetes.io/secret", VolumeGidValue ""
Sep  6 20:48:52 localhost kubelet[1109]: I0906 20:48:52.948748    1109 reconciler.go:297] Volume detached for volume "default-token-t4wqc" (UniqueName: "kubernetes.io/secret/41d344d7-ab0b-4359-b983-cb1c690575b1-default-token-t4wqc") on node "opensds" DevicePath ""
Sep  6 20:49:02 localhost kubelet[1109]: I0906 20:49:02.873967    1109 reconciler.go:297] Volume detached for volume "pvc-2398729a-6daf-4835-a215-f69d18e5c37f" (UniqueName: "kubernetes.io/csi/csi-opensdsplugin^47d2555a-ba5c-4f47-a22c-d8f0d2be0189") on node "opensds" DevicePath ""

@jaemin-joo Could you please try Kubernetes cluster v1.14.0
Here is the link for OpenSDS integration with Kubernetes CSI.
https://github.com/opensds/opensds/wiki/OpenSDS-Integration-with-Kubernetes-CSI
Pods are deployed successfully for me, except the example test issue reported by @kevinjantw . The application is stuck in ContainerCreating state

@himanshuvar OK. I will try to test in v1.14.0.
Can I use dns in the configMap file like below? (e.g. http://apiserver.opensds.svc.cluster.local:50040)
When I try to use dns, csi-nodeplugin pod has failed. and when I use Cluster-IP of apiserver & authchecker, it works.
Another pod(csi-attacher, csi-provisioner, csi-snapshotter) works whatever DNS and ClusterIP.
kind: ConfigMap
apiVersion: v1
metadata:
name: csi-configmap-opensdsplugin
data:
opensdsendpoint: http://apiserver.opensds.svc.cluster.local:50040
opensdsauthstrategy: keystone
opensdsstoragetype: block
osauthurl: http://authchecker.opensds.svc.cluster.local/identity
osusername: admin
ospassword: opensds@123
ostenantname: admin
osprojectname: admin
osuserdomainid: default
passwordencrypter: aes
enableEncrypted: F
osuserdomainid: default

When I checked configuration file. I found the issue is related to IP. It works when I change tgtBindIp to host-IP  in /etc/opensds/driver/lvm.yaml
root@opensds-srv01:~# vi /etc/opensds/driver/lvm.yaml

tgtBindIp: <IP_ADDRESS> # This section!!
tgtConfDir: /etc/tgt/conf.d
pool:
  lb001:
    storageType: block
    availabilityZone: default
    extras:
      dataStorage:
        provisioningPolicy: Thin
        isSpaceEfficient: false
      ioConnectivity:
        accessProtocol: iscsi
        maxIOPS: 7000000
        maxBWS: 600
      advanced:
        diskType: SSD
        latency: 5ms

But dock is possible to move to other node if node is down. so It doesn't solve the issue.

Can I ask you why node iscsid process is checked more?
In my case, I attached iscsi volume to node which dock and csiplugin is placed.
so I can find some of errors to use nginx example for CSI.
root@opensds-srv01:~# kubectl logs -f pod/csi-nodeplugin-opensdsplugin-4dk5j -c opensds

I0925 17:25:35.358873       1 logs.go:40] 2019/09/25 17:25:35 helper.go:236: connmap info:  map[discard:false targetDiscovered:true targetIQN:[iqn.2017-10.io.opensds:cdda11b8-e04f-4f49-8c16-0f88d761859e] targetLun:1 targetPortal:[<IP_ADDRESS>:3260]]
I0925 17:25:35.358885       1 logs.go:40] 2019/09/25 17:25:35 helper.go:237: conn info is:  &{    true [iqn.2017-10.io.opensds:cdda11b8-e04f-4f49-8c16-0f88d761859e] [<IP_ADDRESS>:3260]  1 false}
I0925 17:25:35.358892       1 logs.go:40] 2019/09/25 17:25:35 common.go:27: Command: /bin/bash -c pgrep -f /sbin/iscsid:
I0925 17:25:35.360933       1 logs.go:40] 2019/09/25 17:25:35 common.go:27: Command: /bin/bash -c /sbin/iscsid:
E0925 17:25:35.362435       1 volume.go:569] failed to find device: Please stop the iscsi process outside the container first: exit status 1
I0925 17:25:35.362444       1 node.go:79] end to node stage volume


[DOCS]: Create an auto semantic version tutorial

Describe alternatives you've considered
Would you please share your clear and concise description of any alternative/similar documentation you've considered.
Additional context
Would you please share any other context or screenshots about the documentation request here.

The Semantic Version Tutorial is now in the document framework.
Content needs creating.


Add automated testing infrastructure and configuration

Comprehensive testing setup across the project:

Configure Vitest for unit testing
Set up @testing-library/react for component testing
Add coverage reporting and thresholds
Create test documentation
Add test scripts to package.json
Configure .gitignore for test artifacts

The testing infrastructure includes:

Unit tests for services and utilities
Component tests with React Testing Library
Coverage requirements (80% threshold)
CI integration with coverage checks

@AlexioDanje did you note there is a new security warning with the dependency js-suites - the Synk check failed. To solve you need to: "Upgrade jsuites to version 5.0.1 or higher."
See:
https://app.snyk.io/org/decode/project/c0d35626-380d-4551-87ac-7bbf6a072d8c/pr-check/c1a33835-f2d7-4b11-b228-af8b2cefd175


Palomar: Add alt text to ES Doc Text field

Adds alt text for images to the ES doc's text field.
For example:
Given
Text: i should buy a boat
Image 1 Alt: cat thinking i should buy a boat
Image 2 Alt: cat at the insurance office getting boat insurance

Then
ES Doc Text: i should buy a boat cat thinking i should buy a boat cat at the insurance office getting boat insurance

I can't seem to reproduce the test failure locally. Is it a related failure?

It passed in this CI build FWIW https://github.com/jcsalterego/indigo/actions/runs/5662477475/job/15342470966

Yeah, thats a known flaky test. I need to figure out how to deal with that better...

rerunning the tests to try and get a clean merge

Would love to know if there's anything I can do to help this get merged!

I assume this is on hold for a larger mapping change and task to reindex all posts?

I assume this is on hold for a larger mapping change and task to reindex all posts?

I think this would be good to get merged and online, even if it's only for a portion of the data (no backfill), and pending some larger changes in the schema.

This feature should be working in prod as of the past couple weeks, as part of a broader refactor of how we index posts. It took forever!
I'm going to close this PR; if the feature isn't working the way you expect, please re-open an issue. Sorry to leave you hanging on this helpful PR for so long, and thanks again for the contribution.

@bnewbold The big search refactor looks great and was worth the wait. Thanks!


Expose more Plot internals

Exposes more Plot internals, for observablehq/plot#170.
In addition to the comment by @mbostock to expose all of options.js,
this exposes other internals that I used when implementing
a custom mark.

We probably want to be more selective, since everything we expose must be documented and (as much as possible) have a "definitive" API. Can you share a concrete example of a custom mark?

We probably want to be more selective, since everything we expose must be documented and (as much as possible) have a "definitive" API. Can you share a concrete example of a custom mark?

I think I found a better way to accomplish what I needed with the custom mark. Sounds like this PR is missing the mark for #170, so I'll close it.


Cannot update NPC home

It's seems that home of an NPC cannot be set via .x home <p> (neither with a GM command nor via a script).
Steps to reproduce:

create an npc, check that initial home is -1,-1
try to set home via .x home 4545,1475 (or whatever)
home is not updated and still is -1,-1

I noticed that home can be initialized by simply setting .x homedist <somehting>. Then the home is atomatically updated to the current NPC location. But still, I haven't find a way of updating this home value later. .x home <new p> just does not have effect.

Here works properly

The correct command is .xhome (without blank space) not .x home btw

Hm.. doesn't work for me, let me double check with the default script pack and get back to you.

Sorry, the problem was on my side. @drk84 thanks for checking on that.


Static or const ignored?

Hi,
Does static or const variables are ignored?
I asked ChatGPT, but ... 🥴
Ah, I see. Tapper is a transpiler, which is a tool that converts code written in one programming language into another language.

In this case, it seems that Tapper is excluding the constant string value "LOL" assigned to the field "Test" due to its inappropriate language. As a general rule, it's always a good idea to use clear, concise, and professional language in your code to avoid issues with code generation and deployment.

Regards

Yes, static fields (including const fields) are ignored.

Oh okey, how I can include it?

I will not support transpiling static fields. Most serializers ignore static fields, so there is no benefit to transpile.

No it should be useful for parameters or api error codes, like this:
[TranspilationSource]
public class DefaultApiErrorCodes
{
    public const string UserIdMissing = "UserIdMissing";
}

Even if we were to introduce that feature in the Tapper, the attribute should be created separately from the TranspilationSource. I am also skeptical that bringing embedded constants from C# into TypeScript is good.
It would be better to share types for configuration between C# and TypeScript and set values from a shared configuration file such as JSON.

My use case is to handle api error codes but with different types, but sharing the same origin.


Remove venv from codebase

Purpose

Fixes issue with debugging Azure Functions in VS Code, by removing all references to venvs

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Rebuild devcontainer in VS Code
Create code/backend/batch/local.settings.json with storage connection string value for AzureWebJobsStorage
Run "Attach to Python Functions" within "Run and Debug"

What to Check

Functions start locally

My question here is "why" does Functions need a venv? Will Functions work if we remove that setting?

The description for azureFunctions.pythonVenv says:

The name of the Python virtual environment used for your project. A virtual environment is required to debug and deploy Python functions.

I could try removing all references to .venv and see if that works, what do you think?

I know it's a pain - but are you able to try? We are not using a venv for the other python parts of this project and I'm inclined to not introduce it if we can help it

I know it's a pain - but are you able to try? We are not using a venv for the other python parts of this project and I'm inclined to not introduce it if we can help it

Tested locally without a venv in a new container and seemed to work fine


Update getting_started.md

Add jaeger-compact and jaeger-binary ports to all-in-one docker command

Coverage remained the same at 100.0% when pulling bf0bfc842f554c64eaabc143f8f247321c8c946c on black-adder-patch-1 into 7a965e984fac3e7beb4684913a48c1affd76024b on master.

Bah, I already did in #177 - I also opened the sampler port.


Proposal: Linting for missing spaces between tags

If your React markup looks like this:

  Here is a
  link
  to Google

The whitespace will be collapsed and it will look like

Here is alinkto Google

Note that the whitespace is collapsed. The recommended solution from React is to write your code as

  Here is a
  {' '}
  link
  {' '}
  to Google

which will render correctly.
It would be great if there was a lint rule that caught this.

Does this already exist?
Is this something desirable? I am happy to look into a PR

I don't think a rule could catch this, because there's no guarantee that the user wants the space there - and in fact, they might not want it. That's why the best practice is to be explicit.

In my experience, it is very rare that the developer does not want the space. It is almost always indicative of an error that manifests as incorrectly rendered copy
In the case where the developer does not want the space, I would consider it reasonable to require that they be directly adjacent, as in

  Here is alinkto Google

Or when the line is long

  Here is a{/*
  */}link{/*
  */}to Google

In general, as written originally the developer's intent is not clear. It appears as though they want some whitespace between the tags, but they may not
One could imagine a rule that forces you to be explicit always - that is, including {' '} when you want a space and a line continuation or

It might not be safely autofixable at least; but sure, it'd be interesting to see the test cases in a PR.

Thanks - I added some illustrative test cases in https://github.com/yannickcr/eslint-plugin-react/issues/1515


VS Code interpreter / REPL

Hi,
As opposed to Visual Studio, VS code is lightweight and loads in seconds. I thought why not make it even faster, by enabling it to act as an interactive code interpreter.
In my vision, ideally, after launching vs code, clicking Ctrl+N for creating a new temp file at an undisclosed location, should allow me rename the file to indicate its type (let's say *.cpp or *.cs).
Once file type is known to the editor I can benefit from intellisense, which would already be a lot to me without even being able to run. This would allow me to include headers and ctrl+click types or definitions etc. to look them up in the header files (cpp), to see how they were defined, or clicking on types to see their structure, much like C# interactive window in Visual Studio does, but with current renamable code file as source.
Anyway, last step would be being able to run a single command that launches the appropriate interpreter/compiler and runs the code printing the output with a single click according to the file type.
I just spent 40 minutes installing and uninstalling extension after extension that claims to do what I'd like, no one which worked as desired, each one with its quirks and imperfections.
Wouldn't it be time for this basic editor functionality to fast-interpret code to finally become a recognized part of VS Code already?
Thank you!

Use Ctrl+K,M to change the untitled file's language, and then install the code runner extension, and press Ctrl+Alt+N to run the code

That worked great.
The Ctrl+K,M is what I was looking for!
Anyway I wonder if I should close this issue or keep it open for the sake of the second part.

The Ctrl+K,M only sets the file-type, but seems it doesn't do the job thoroughly as renaming. When Ctrl+Clicking types or headers doesn't open them as with a file named *.cpp.

In my vision, ideally, after launching vs code, clicking Ctrl+N for creating a new temp file at an undisclosed location, should allow me rename the file to indicate its type (let's say *.cpp or *.cs).

You can write an extension to do this.


AR 1.0 Interface Support

Reminder issue to add backwards AR 1.0 compatibility.
(Depending on demand this may or may not be implemented.)
If we implement, we would "only" add it for the following resolvers:

FileResolver
CachedResolver

To be discussed how to best add this without "fracturing" the code base with ifelses'.
(We could add on build patching of code or keep it on a branch.)

Closing as AR 1.0 is legacy now.


Setup ci

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ab34b57). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #8   +/-   ##
=========================================
  Coverage          ?   76.08%
=========================================
  Files             ?       19
  Lines             ?      995
  Branches          ?       53
=========================================
  Hits              ?      757
  Misses            ?      223
  Partials          ?       15

Continue to review full report at Codecov.

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


Support for multiple accounts

Hey!
Please, add the support to have multiple accounts enabled - many people, including me, have multiple FB accounts for some different reasons.
It would be good to be able to chat with people from all the accounts. Pidgin, e.g. was able to do this.
Thanks.

Well, thanks for the reply... I guess... :D

Sorry, I don't have plans to support this soon. I'll accept PRs.


when i don't select a item in autoCompleteTextField (android)

The Screen don't Open More
Another exception was thrown: setState() called after dispose(): AutoCompleteTextFieldState<banks>#f9733(lifecycle state: defunct, not mounted
Another exception was thrown: NoSuchMethodError: The method 'clear' was called on null
I/flutter ( 1454): When the exception was thrown, this was the stack:
I/flutter ( 1454): #0      Object.noSuchMethod  (dart:core-patch/object_patch.dart:50:5)
I/flutter ( 1454): #1      new AutoCompleteTextFieldState.<anonymous closure>
package:autocomplete_textfield/autocomplete_textfield.dart:182
I/flutter ( 1454): #2      ChangeNotifier.notifyListeners
package:flutter/…/foundation/change_notifier.dart:206
I/flutter ( 1454): #3      FocusNode._notify
package:flutter/…/widgets/focus_manager.dart:103
I/flutter ( 1454): #4      FocusManager._update
package:flutter/…/widgets/focus_manager.dart:474
I/flutter ( 1454): (elided 10 frames from package dart:async)
I/flutter ( 1454):
I/flutter ( 1454): The FocusNode sending notification was:
I/flutter ( 1454):   FocusNode#eab7a(FOCUSED)
I/flutter ( 1454): ════════════════════════════════════════════════════════════════════════════════════════════════════

Not really sure what you mean here...

What I think @breitembach is trying to say is that "losing focus" event is not triggering. He is selecting the autocomplete widget that then displays the suggestions but if a suggestion isn't selected and tries to leave the page then this results in an exception.
The autocomplete tries to call setState() but the page is now disposed.


Pre-compute style

The Style class should pre-compute its value instead of making the lambda do it every time it gets applied.

f8b82dcdb9f70f4ca3d578b1ecf6d6792ce3a245 Implemented optimization


[MMC]

Is your enhancement request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Can I help you? You must write something!


Can`t build FLIGHT.hdbcds

Hello, when i build file FLIGHT.hdbcds, console show error

Where did I make a mistake?

Impossible to say without seeing the source of your FLIGHT.hdbcds. I suspect that you are missing some lines or you have not created the synonyms to SFLIGHT correctly.

Did everything exactly according to the instructions, but the core_db is not build.
Аttach my code below, thanks.
openSAPHANA6.zip

First question - do you have the SFLIGHT schema installed on your system? It doesn't come in HANA by default and does have to be installed.
Second if you do a build with just the synonyms not the FLIGHT.hdbcds, does it work? Can you see the synonyms in the database explorer?
Third, what HANA release are you running on?  You seem to have an older version of the FLIGHT.hdbcds syntax.  It is updated for newer releases - see latest here: https://github.com/SAP/com.sap.openSAP.hana5.example/blob/hana2_sps03/core_db/src/data/FLIGHT.hdbcds

Yes, SFLIGHT schema installed

Yes, i can see the synonyms in the database explorer

SAP Web IDE for HANA 2
SPS 02 Patch 1 ( Build 4.2.18 )

I dont' understand how you can see the synonyms based upon the source files you sent me.  The sources you sent me have the synonyms folder outside of the src file and therefore aren't getting created. So either the sources you sent me aren't actually what you are building or these screen shots are from a different project than the sources you sent me. In the sources you sent me, there are also other problems that are keeping it from building. For example you are using an INSERT privilege on the hdbgrants and your admin.hdbrole has the trigger privilege (which isn't valid).  However after I correct for these things and move your synonyms folder back into src, then it builds correctly. There are no errors in the FLIGHT.hdbcds.

openSAPHANA6 (2).zip


Refactor to use StreamReader and StreamWriter directly

[This includes the changes in the other PR, can rebase or whatever you like, but seems easier to review this independently.]
Maybe the 40K file should be generated, given it's encrypted it won't compress.

i am on vacation right now but will look into this afterwards.


Not works with the updated Candy Machine V2 program.

The Metaplex made some changes to the Candy Machine V2 program on May 27th, the UI of this project may not suit the candy machine created after May 27th.
Clicking the mint button will not get a right NFT, but an ‘Unkown Token’ without metadata.
https://twitter.com/metaplex/status/1530187094211149824

Just downloaded this and am seeing this too. Any initial thoughts on a fix?
Thanks :)

Issue should have been fixed. You can watch duplicate issue https://github.com/Fulgurus/candy-machine-v2-responsive-ui/issues/13 for more details ;). Thank you.


Spike: how do we modify the transcoder so we can use separate processes for Collection and Exhibition videos?

Currently the transcoder is geared towards collection videos only. We want a separate pipeline for Exhibition videos, and flexible pipelines in general.
I think we should define pipelines as sets of functions:

file-generator - a function that generates file paths for input files to process
processes - an ordered dictionary of lists of functions, each taking as input the output of the function before it.
finally - final tasks to do with the output of all the processes once they have all completed.

Greg can you close this since it sounds like you had a conversation with the conservators and this is no longer necessary.

Reopening as it is still unresolved @hvanness
For both collection and exhibition videos we want to ensure there is an access copy and web copy uploaded to S3.
For collection videos, we transcode the master to make the access copy. But for exhibition videos, we usually want to upload the original, so long as it is below a given bitrate, say 20Mbps.
To close this ticket, let's look at some collection pres masters and see what bitrates we get. If they are consistently above 20Mbps, let's set that as the ceiling. If the master is below that ceiling, we upload it directly. If it is above 20Mbps, we transcode.
However, if collection pres masters are often 20Mbps or less, then we may need a different approach, perhaps an alternative transcoder with alternative settings and watch folders.


Add @OptionField and @ParametersField composite annotations for groovy script

Similar to groovy.cli.OptionField in Groovy 2.5:
package picocli.groovy

import groovy.transform.AnnotationCollector
import groovy.transform.Field
import picocli.CommandLine.Option

@Option
@Field
@AnnotationCollector
@interface OptionField { }

This is possible but it would increase the jar size by 2K for the OptionField class and 2K for the ParametersField class. However, the improvement for the client code is minimal...
I don't think this is a good trade-off, so marking this as wontfix.


Which is the best way, to have same module multiple times in an html page.

I started working with T3js framework in order to keep my code structured and easy scalable.
Loved the ideas that the creators had and the simplicity behind them.
I want to find out the right way to include the same module twice (or more times) in the same page without the ids and the data-modules to get messed up.
Take a look at the T3js documentation and examples and specifically at the TODOS example (http://t3js.org/examples/todo/). What I want to do is something like that -> http://prntscr.com/c30r0y, but at the same time make sure that the ids are not messed up, and the instantiations of the module are independent.

All you need to do is have data-module="module-name" on the element. You do not need to specify an id as that will be auto-populated for you with a unique name

Could you please explain this with an example ?
some-module.js
Box.Application.addModule('some-module', function(context) {
    'use strict';
    var moduleEl,
        moduleId;

    return {
        init: function () {
            moduleEl = context.getElement();
            moduleId = moduleEl.getAttribute('id');
            $("button").click(function () {
                console.log('Only one module has been clicked!
One way for sure is using  context.getConfig() but is that the right way or there is something even better that I could do in this case ?

Yep, each module is self contained and handles the DOM inside of itself.


prepare for segwit merge

allowing high S value prevents bitwasp from working.
also, version 2 of transaction is already in action  in testnet:
https://sandbox.smartbit.com.au/tx/5029542bfc2d8feb4ca9ee33e78ae52f6fabe302ee0755771eb19dcd72110ff4

That's happened before :-) Bitcoin is a moving target sometimes! This PR looks fine.
Have you looked at bitwasp/bitcoin-php? It has full SegWit support already. Just worry when people are still using this!

Guessing you mean Bitwasp the marketplace maybe? r


Add CM rule winner size with schematron manually optimized (TEDEFO-3052)

The first commit adds the schematron generated from the DB (with the rules re-activated as-is).
The second commit moves the rules to the "manually optimized" version.
So if you wish you can check that the manual version is equivalent by looking at the commits individually.

Thanks for the suggestions @pdonohoe !
The changes for the variables do not seem to work for me: the rule then does not detect failures.
In any case, I don't expect they would bring any significant performance improvement, and I indeed prefer to keep things consistent.
The change for the rule does bring any noticeable performance improvement, and I prefer to keep the test close to the EFX expression.


REQUIRE cannot compare pointer types

Description
In file included from Catch.RwUtility.cpp:5:0:
./catch/single_include/catch.hpp: In instantiation of ‘T& Catch::Internal::opCast(const T&) [with T = void(void*)]’:
./catch/single_include/catch.hpp:1313:32:   required from ‘static bool Catch::Internal::Evaluator<T1, T2, (Catch::Internal::Operator)0u>::evaluate(const T1&, const T2&) [with T1 = void(void*); T2 = void (*)(void*)]’
./catch/single_include/catch.hpp:1358:47:   required from ‘bool Catch::Internal::compare(const T1&, const T2&) [with Catch::Internal::Operator Op = (Catch::Internal::Operator)0u; T1 = void(void*); T2 = void (*)(void*)]’
./catch/single_include/catch.hpp:1888:50:   required from ‘void Catch::BinaryExpression<LhsT, Op, RhsT>::endExpression() const [with LhsT = void (&)(void*); Catch::Internal::Operator Op = (Catch::Internal::Operator)0u; RhsT = void (* const&)(void*)]’
Catch.RwUtility.cpp:79:5:   required from here
./catch/single_include/catch.hpp:1298:59: error: invalid use of const_cast with type ‘void (&)(void*)’, which is a pointer or reference to a function type
     inline T& opCast(T const& t) { return const_cast<T&>(t); }

Steps to reproduce
TEST_CASE("utility::invoke_blockable_async_callback - `nullptr` supplied to `uponComplete` argument is replaced by internal callback", "[utility::invoke_blockable_async_callback]") {
    TestClass test;
    utility::invoke_blockable_async_callback(&test, &TestClass::testMethod, nullptr, nullptr);
    REQUIRE(utility::synchronizing_callback == test.testMethod_uponComplete_arg);
}

typedef void (*signal_t)(void * context_);

namespace utility {
...
inline static
void
synchronizing_callback (
    void * context_
) {
    /*
     * `std::promise` is honored on a separate thread
     * to ensure underlying implementation is able to
     * invoke the callback on the originating thread
     */
    (void)std::thread(
        [&](void) {
            reinterpret_cast<std::promise<void> *>(context_)->set_value();
        }
    ).detach();
}

template <class T>
inline static
void
invoke_blockable_async_callback (
    T * object_,
    int(T::*method_)(signal_t uponComplete_, void * context_),
    signal_t uponComplete_,
    void * context_
) {
    int error;
    bool block = false;
    std::promise<void> callback_signal;
    std::future<void> callback_gate(callback_signal.get_future());
/*
    if ( !uponComplete_ ) {
        block = true;
        uponComplete_ = synchronous_callback;
        context_ = reinterpret_cast<void *>(&callback_signal);
    }

    if ( 0 != (error = */(object_->*method_)(uponComplete_, context_);/*) ) {
        errno = error;
        ::perror("ERROR: Underlying implementation encountered error!");
    } else if ( block ) {
        switch (callback_gate.wait_for(std::chrono::milliseconds(REMOTE_DEVICE_TIMEOUT_MS))) {
          case std::future_status::ready:
            break;
          case std::future_status::timeout:
            errno = ETIME;
            ::perror("ERROR: Block on future timed out!");
            break;
          case std::future_status::deferred:
          default:
            errno = EWOULDBLOCK;
            ::perror("ERROR: Block on future returned unhandled result!");
            break;
        }
    }
*/}
...
}  // utility

Extra information
It appears to be having difficultly comparing the pointer address of the free function to

Catch version: v1.9.4
Operating System: Ubuntu 5.4.0-6ubuntu1~16.04.4
Compiler+version: gcc version 5.4.0 20160609

If I give Catch a hint by casting the free function to a signal_t, then it can compile and make the comparison.
REQUIRE(static_cast<signal_t>(utility::synchronizing_callback) == test.testMethod_uponComplete_arg);

For some reason, lhs's type is not deduced as a pointer to function, but rather the function itself (it deduces to void (void*) and not to void (*) (void*)).
An easy fix for now is to manually take address of the function: REQUIRE(&utility::synchronizing_callback == test.testMethod_uponComplete_arg);

The recent rewrite of evaluate layer seems to have fixed this, at least under MSVC (modulo warnings).
Can you check if 1.10 works for you as well?

Awesome! I'll give it a try as soon as I can get to it, it may be a week or so.

I tested it in the meantime, it was not in fact fixed on other compilers. Since fixing this would require a breaking change, it can only happen in Catch 2.


different number of targets after each vmagent restart in the OpenShift

Hello
We are testing vmagent as the substitution for OpenShift platform Prometheus running agent in the openshift-monitoring namespace.  All targets for vmagent providing vmagent-operator , as we expect. But we have some uncertainty, indeed... Sometimes, started vmagent pod feels 125 targets ( it's looks pretty well , like full cluster monitoring coverage by.) But after restarting the vmagent number of targets unpredictable changing. From one start to the other it would be or 110, or maybe 45. What could be the reason for this behavior?
Thank you  very much!

Hello, which version of vmagent do you use? probably, its related to this issue:
https://github.com/VictoriaMetrics/VictoriaMetrics/issues/1240
When some service wasn't properly update for corresponding endpoints.

Hello,
vmagent coming from victoriametrics-operator.v0.14.1

Is vmagent from victoriametrics-operator v0.14.1 affected by https://github.com/VictoriaMetrics/VictoriaMetrics/issues/1240 ?

Yep, its v1.58.0 by default. Can you try version of vmagent from master branch? Does it solve an issue?
You can build it with
git clone git@github.com:VictoriaMetrics/VictoriaMetrics.git
make package-vmagent

It will produce docker image.

related issue was fixed at vmagent.


hover effect

Description
The header button don't have hover effect.
Expected behavior
hover effect.

done.


Stub out OAuth server flow

This is a duplicate of work in https://github.com/HeroicKatora/oxide-auth.  We can use it to support an oauth server flow.
This is conflated with client request right now, but we will split these up soon

it's getting real tho


Make the templated message in sign and verify const

Without this then Agora fails when Schnorr sign and verify are used with const(Transaction) for example.

We probably could use a test with indirections. A type such as struct SomeData { ubyte[] data; } should trigger the bug.


Recover source and license options in dataset metadata modal

Previously, besides description and tags, the parameters source and license were available:

cc @viddo @saleiva

We should add attributions field too #4826

source should support .md links as well as attributions.

@ethervoid @saleiva Looking at the old code/modals, the source and licence is only visible if the user account type is "internal", does this still hold true or is that changing due to new data-library feature?
Also, from a more UX perspective, attributions I presume is what is displayed at the bottom of a map(?) instead of the default, but what about source and license? I don't think it's very clear what/where this fields are intended for.

Yes, we should  open to everybody.
Source and license will be shown on the public pages. The attributions should be on the attributions on the map, yeah.

Yes, we should open to everybody.

Do you mean that these fields should always be editable then?

Yes, sure. Is the user who defines these

The attributions should be on the attributions on the map, yeah.

Loving this. Right now people hack the legend to add more attributions to the map. :revolving_hearts:


ADBDEV-6047: Add OpenJDK 11 to Ubuntu image

Add OpenJDK 11 to Ubuntu image
We need Java for PXF, ADBC etc, so this patch adds OpenJDK 11 to Ubuntu image,
as it did for CentOS 7 image.

Allure report https://allure.adsw.io/launch/79396

Was JAVA_HOME not necessary in the end? Or will this still be left in PXF dockerfile?

Was JAVA_HOME not necessary in the end? Or will this still be left in PXF dockerfile?

@silent-observer , can you explain, what happens without JAVA_HOME?

Was JAVA_HOME not necessary in the end? Or will this still be left in PXF dockerfile?

We decided move it to PXF Dockerfile

Was JAVA_HOME not necessary in the end? Or will this still be left in PXF dockerfile?

@silent-observer , can you explain, what happens without JAVA_HOME?

@andr-sokolov, checked, seems like Gradle can still find Java without an issue. Maybe it's not necessary even in PXF?

Was JAVA_HOME not necessary in the end? Or will this still be left in PXF dockerfile?

@silent-observer , can you explain, what happens without JAVA_HOME?

@andr-sokolov, checked, seems like Gradle can still find Java without an issue. Maybe it's not necessary even in PXF?

It can't find Java without JAVA_HOME env var

Allure report https://allure.adsw.io/launch/79440

Failed job Regression tests with ORCA on x86_64: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1866607

Failed job Regression tests with Postgres on x86_64: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1866606

Allure report https://allure.adsw.io/launch/79468

Failed job Regression tests with Postgres on x86_64: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1867730

Failed job Regression tests with ORCA on x86_64: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1868727

bender build

Failed job Deploy multiarch Dockerimages: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1869443

Failed job Deploy multiarch Dockerimages: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1869764

Failed job Deploy multiarch Dockerimages: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1869766

Failed job Deploy multiarch Dockerimages: https://gitlab.adsw.io/arenadata/github_mirroring/gpdb/-/jobs/1869795

bender build

Allure report https://allure.adsw.io/launch/79509


Replace cpuid based CPU detection with Win registry-based detection in aja_getcputype()

The aja_getcputype() function has been updated to retrieve the CPU brand string using the Windows registry instead of relying on the __cpuid instruction.
This approach improves compatibility across different architectures (especially for Windows on ARM systems) that do not support the __cpuid instruction, while still maintaining the ability to retrieve CPU type.
The changes were tested on both x64 and ARM-based Windows machines, and the build was successful in both environments.

@thirumalai-qcom Please change the implementation to use the existing aja::read_registry_string method. Examples can be seen in the same ajabase\system\windows\infoimpl.cpp file that your changes are in, such as in the aja_getosversion function.
Another AJA software engineer also requested that the aja_getcputype function should return a valid CPU type string or an empty string if unable to do so.

@paulh-aja I've refactored the aja_getcputype function to use the aja::read_registry_string method as suggested.

Another AJA software engineer also requested that the aja_getcputype function should return a valid CPU type string or an empty string if unable to do so.

This method already handles returning an empty string if the CPU information cannot be retrieved.

Thank you for your contribution, @thirumalai-qcom!


Help with reviving plugin

Devs should recreate a data over sound algo. It was created here but it failed due to lack of support. Check it out here: https://github.com/Zfinix/dataoversound

This isn't something that the Flutter team would build, but as always community member interested in building new plugins are encouraged to work together to do so.


postsrsd: 1.10 -> 1.11

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/roehling/postsrsd/releases.
meta.description for postsrsd is: "Postfix Sender Rewriting Scheme daemon"
meta.homepage for postsrsd is: "https://github.com/roehling/postsrsd"
meta.changelog for postsrsd is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.11 with grep in /nix/store/7yr3xnwaljab2j0xrcfw263mrjlfq7zc-postsrsd-1.11

directory tree listing: https://gist.github.com/ecad42907c6a25ba6df4c05c4312f64a

du listing: https://gist.github.com/ffe96f656b1f5a03478d0fc5108fb625

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
postsrsd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7yr3xnwaljab2j0xrcfw263mrjlfq7zc-postsrsd-1.11 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A postsrsd https://github.com/r-ryantm/nixpkgs/archive/6c81f568877c7cac80d8b8b168950c42d8fc604b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7yr3xnwaljab2j0xrcfw263mrjlfq7zc-postsrsd-1.11
ls -la /nix/store/7yr3xnwaljab2j0xrcfw263mrjlfq7zc-postsrsd-1.11/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

postsrsd

Maintainer pings
cc @abbradar for testing.

Result of nixpkgs-review pr 117235 at 6c81f568 run on aarch64-linux 1

1 package built successfully:

postsrsd

1 suggestion:

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/servers/mail/postsrsd/default.nix:25:5:
   |
25 |     license = licenses.gpl2;
   |     ^


slight bug in Certificate creation wiki

The wiki says currently:
Click several times on “Continue” until you get to the “Specify a Location For The Certificate” screen, then set “Keychain to System”.
I am on the latest OSX; and it does not say " “Keychain to System”. On my OSX it has a combo box with two choices. One say "System".

All worked fine, i restarted , and then did the last step
CERT=dlv-cert make install
You might want to mention this must be done in the dir that was go getted


Refactor text method to remove duplication

Why do you create a new method? for me, your generate() should be inside text().

Habbit 😄, will change!

@fzaninotto updated.

Thanks!


AV: Allow customized AVClients and VOICE_MODES

An AVClient implentation may change the VOICE_MODES that are
implemented. To do so, AVSettings.VOICE_MODES may be overridden with
different or removed values.
Additionally, an AVClient may implement thier own custom properties or
methods. In this case, game.webrtc.client needs to recognize that
these may be used.
Implement both of these changes using recommentations from:
https://github.com/Microsoft/TypeScript/wiki/Breaking-Changes#strict-object-literal-assignment-checking

To be honest, I don't think this is the right approach. In particular, you are saying you want to access custom public properties of AVClient via game.webrtc.client. But foundry documents this as AVClient so no properties aside from those that AVClient provides should be available.
What is it your are actually trying to do? Explaining that might help us find a proper solution that is coherent with the rest of the types and allows you to do what you want to do.

When a module creates a new AVClient (I.e., Jitsi), the type of game.webrtc.client is the class defined by the module. These clients must provide the core methods and properties, but may also expose their own additional items.
My custom AVClient provides an additional method that I also need to call from a hook. Since the hook doesn't have context, it calls the  method via game.webrtc.client.customMethod.
If there's a better way to do this, great. I'm new to typescript and this was the only way I could find to allow for this use case without just telling typescript to ignore the error.

To be honest, I don't think this is the right approach. In particular, you are saying you want to access custom public properties of AVClient via game.webrtc.client. But foundry documents this as AVClient so no properties aside from those that AVClient provides should be available.
What is it your are actually trying to do? Explaining that might help us find a proper solution that is coherent with the rest of the types and allows you to do what you want to do.

When a module creates a new AVClient (I.e., Jitsi), the type of game.webrtc.client is the class defined by the module. These clients must provide the core methods and properties, but may also expose their own additional items.
My custom AVClient provides an additional method that I also need to call from a hook. Since the hook doesn't have context, it calls the method via game.webrtc.client.customMethod.
If there's a better way to do this, great. I'm new to typescript and this was the only way I could find to allow for this use case without just telling typescript to ignore the error.

I think the way to make this work in a way that is consistent with the types, we need to make the configured AVClient type configurable, similar to how document types are configurable.
The only reasonable alternative seems to be to just do the type assertion in that one place or check if game.webrtc.client is actually of the type you expect, i.e.
if(game.webrtc.client instanceof JitsiRTCClint) {
  game.webrtc.client.customMethod();
}

But I think it's reasonable to make the class configurable.
As a side note: AVClient hasn't been updated yet for 0.8.x, so there might be some incorrect things in there. Also we do not have the SimplePeerAVClient yet.

I think I found a good solution: We'll simply make the type of VOICE_MODES configurable, too.

Closing in favor of #1160


Update to 3.8 native SDKs

Update the Dolby.io React Native SDK to use the 3.8 native SDKs.
Tasks:

[ ] Update the Android part of Dolby.io React Native SDK
[ ] Update the iOS part of Dolby.io React Native SDK

Update to Android SDK 3.8.2.

Hi!
I was wondering if there is an estimate for adding support for the 3.9.0 SDK?
Due to https://github.com/DolbyIO/comms-sdk-react-native/issues/61 I am unable to release my app without being able to connect to a conference using real-time-streaming.

We are currently planning the 3.9.0 SDK release, but we have not finalised the date yet.


[SPARK-6583][SQL] Support aggregated function in order by

Spark support the below syntax now(order by alias):
SELECT a, sum(b) as sum_b
FROM orderByData
GROUP BY a
ORDER BY sum_b

This patch is used to support the syntax below:
SELECT a
FROM orderByData
GROUP BY a
ORDER BY sum(b)

SELECT sum(b)
FROM orderByData
GROUP BY a
ORDER BY sum(b)

SELECT a, sum(b)
FROM orderByData
GROUP BY a
ORDER BY sum(b)

My idea is change
SELECT a
FROM orderByData
GROUP BY a
ORDER BY sum(b)

to
SELECT a, sum(b) as sum_b
FROM orderByData
GROUP BY a
ORDER BY sum_b

and then project them away after the sort.

Can one of the admins verify this patch?

ok to test

Test build #30094 has started for   PR 5290 at commit bea1370.

Test build #30094 has finished for   PR 5290 at commit bea1370.

This patch passes all tests.

This patch merges cleanly.

This patch adds the following public classes (experimental):

case class TaskCommitDenied(jobID: Int, partitionID: Int, attemptID: Int) extends TaskFailedReason

class ExecutorSource(threadPool: ThreadPoolExecutor, executorId: String) extends Source

class MLPairRDDFunctions[K: ClassTag, V: ClassTag](self: RDD[(K, V)]) extends Serializable

class NaiveBayesModel(Saveable, Loader):

class SqlParser extends AbstractSparkSQLParser with DataTypeParser

case class CombineSum(child: Expression) extends AggregateExpression

case class CombineSumFunction(expr: Expression, base: AggregateExpression)

protected[sql] class DataTypeException(message: String) extends Exception(message)

  class CheckpointWriteHandler(

This patch adds the following new dependencies:

activation-1.1.jar
aopalliance-1.0.jar
avro-1.7.7.jar
breeze-macros_2.10-0.11.2.jar
breeze_2.10-0.11.2.jar
commons-cli-1.2.jar
commons-compress-1.4.1.jar
commons-io-2.1.jar
commons-lang-2.5.jar
gmbal-api-only-3.0.0-b023.jar
grizzly-framework-2.1.2.jar
grizzly-http-2.1.2.jar
grizzly-http-server-2.1.2.jar
grizzly-http-servlet-2.1.2.jar
grizzly-rcm-2.1.2.jar
guice-3.0.jar
hadoop-annotations-2.2.0.jar
hadoop-auth-2.2.0.jar
hadoop-client-2.2.0.jar
hadoop-common-2.2.0.jar
hadoop-hdfs-2.2.0.jar
hadoop-mapreduce-client-app-2.2.0.jar
hadoop-mapreduce-client-common-2.2.0.jar
hadoop-mapreduce-client-core-2.2.0.jar
hadoop-mapreduce-client-jobclient-2.2.0.jar
hadoop-mapreduce-client-shuffle-2.2.0.jar
hadoop-yarn-api-2.2.0.jar
hadoop-yarn-client-2.2.0.jar
hadoop-yarn-common-2.2.0.jar
hadoop-yarn-server-common-2.2.0.jar
jackson-jaxrs-1.8.8.jar
jackson-xc-1.8.8.jar
javax.inject-1.jar
javax.servlet-3.1.jar
javax.servlet-api-3.0.1.jar
jaxb-api-2.2.2.jar
jaxb-impl-2.2.3-1.jar
jersey-client-1.9.jar
jersey-core-1.9.jar
jersey-grizzly2-1.9.jar
jersey-guice-1.9.jar
jersey-json-1.9.jar
jersey-server-1.9.jar
jersey-test-framework-core-1.9.jar
jersey-test-framework-grizzly2-1.9.jar
jettison-1.1.jar
jetty-util-6.1.26.jar
management-api-3.0.0-b012.jar
protobuf-java-2.4.1.jar
spark-bagel_2.10-1.4.0-SNAPSHOT.jar
spark-catalyst_2.10-1.4.0-SNAPSHOT.jar
spark-core_2.10-1.4.0-SNAPSHOT.jar
spark-graphx_2.10-1.4.0-SNAPSHOT.jar
spark-launcher_2.10-1.4.0-SNAPSHOT.jar
spark-mllib_2.10-1.4.0-SNAPSHOT.jar
spark-network-common_2.10-1.4.0-SNAPSHOT.jar
spark-network-shuffle_2.10-1.4.0-SNAPSHOT.jar
spark-repl_2.10-1.4.0-SNAPSHOT.jar
spark-sql_2.10-1.4.0-SNAPSHOT.jar
spark-streaming_2.10-1.4.0-SNAPSHOT.jar
stax-api-1.0.1.jar
xz-1.0.jar

This patch removes the following dependencies:

breeze-macros_2.10-0.11.1.jar
breeze_2.10-0.11.1.jar
commons-el-1.0.jar
commons-io-2.4.jar
commons-lang-2.4.jar
hadoop-client-1.0.4.jar
hadoop-core-1.0.4.jar
hsqldb-<IP_ADDRESS>.jar
spark-bagel_2.10-1.3.0-SNAPSHOT.jar
spark-catalyst_2.10-1.3.0-SNAPSHOT.jar
spark-core_2.10-1.3.0-SNAPSHOT.jar
spark-graphx_2.10-1.3.0-SNAPSHOT.jar
spark-launcher_2.10-1.3.0-SNAPSHOT.jar
spark-mllib_2.10-1.3.0-SNAPSHOT.jar
spark-network-common_2.10-1.3.0-SNAPSHOT.jar
spark-network-shuffle_2.10-1.3.0-SNAPSHOT.jar
spark-repl_2.10-1.3.0-SNAPSHOT.jar
spark-sql_2.10-1.3.0-SNAPSHOT.jar
spark-streaming_2.10-1.3.0-SNAPSHOT.jar

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30094/
Test PASSed.

Test build #30151 has started for   PR 5290 at commit 36c8126.

Test build #30151 has finished for   PR 5290 at commit 36c8126.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30151/
Test PASSed.

@marmbrus Any more comment on this?

Test build #30672 has started for   PR 5290 at commit f119849.

Test build #30672 has finished for   PR 5290 at commit f119849.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/30672/
Test PASSed.

@cloud-fan do you mind reviewing this?

@cloud-fan Can you review it for me?

I think we are close.  What about this: https://github.com/watermen/spark/pull/1/files


[RULE]  Zonal-deployed Azure Firewalls should consider using an Azure NAT Gateway for outbound access

Existing rule
No response
Suggested rule
Azure Firewall can be deployed with up to 250 public IP addresses, each providing 2,496 SNAT ports. This setup offers a maximum of 1,248,000 SNAT ports.
Managing a large number of public IP addresses comes with challenges, particularly regarding downstream IP address filtering requirements. When Azure Firewall is associated with multiple public IP addresses, these filtering requirements must be applied to all associated addresses.
Even when using Public IP address prefixes, associating 250 public IP addresses requires managing 16 public IP address prefixes on the downstream side.
A more efficient solution for scaling and dynamically allocating outbound SNAT ports is to use an Azure NAT Gateway:

High Capacity: Each public IP address on a NAT Gateway provides 64,512 SNAT ports, and up to 16 public IP addresses can be associated, resulting in up to 1,032,192 SNAT ports.
Dynamic Allocation: SNAT ports are dynamically allocated at the subnet level, making all provided SNAT ports available on demand for outbound connectivity.

This configuration simplifies management for downstream systems, as it requires handling only up to 16 public IP addresses.
Pillar
Reliability
Additional context

RE:05 Redundancy
Scale SNAT ports with Azure NAT Gateway
Plan for inbound and outbound internet connectivity
Azure deployment reference - Virtual Network
Azure deployment reference - Subnet

Hi, @BernieWhite. Let me know what you think about this one regarding our discussion.


tleap ToDo

[ ] if pbc_type is None there is very little filtering of template lines. We probably need to at least get rid of quit lines. Also, if output_prefix is not None we should probably get rid of saveamberparm in template_lines

Okay. Should we have the ability to write a default salvation input file, if there is no template and no template list of strings? In other words, should we expect everyone who uses paprika know how to use tleap?

Seems like a good idea, but how would it work? You have to have a force field (leaprc) plus solute frcmods and mol2s at minimum.  Those are always different.   I think we have to expect that anyone who uses paprika.tleap will know how to use tleap.


Create an annotation from a segmented mask

My actions before raising this issue

[ ] Read/searched the docs
[ ] Searched past issues

Question
Hello.
I ran into a problem when using "Upload annotations" for the CVAT 1.1 annotation.
For my project, I need to load an annotation with rotated bounding boxes created from a segmented masks for markup. I use the function OpenCV.minAreaRect and get 4 coordinates. After that, I calculate the angle on the sides of the box. But no matter what coordinates and angles of the bounding box I insert, it is displayed incorrectly in the task. It seems that when calculating a rotated bounding box, CVAT recalculates its coordinates for drawing, but I could not find information on how it does this.

Hello,
First you need to calculate the rotation angle. It must be a number of degrees in range [0; 360].
Secondly rotate your box to -angle, you will get coordinates of not-rotated bounding box.
Finally put x1,y1, x2,y2 of the not-rotated box to CVAT and angle separately.
In the UI CVAT rotates two points (x1,y1) (x2,y2) around the bounding box center.
I hope it will help, I will close the issue for now.
Do not hesitate to reopen if you still have questions, thank you.


[improve] One optimization point

Search before asking

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

Description

newApp.setArgs(StringUtils.isNotEmpty(appParam.getArgs()) ? appParam.getArgs() : oldApp.getArgs());
This modification may be better
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

fixed


Swap Stormpath for Auth0

Despite the branch name, this ended up being the long running branch for all the work to transition from Stormpath to Auth0.
Some highlights:

Authentication was happening in the express app side with Stormpath, but now with Auth0 it happens in the Angular side primarily.
The express app is passed an access token to get user data for the front end.
We had to set up a non-interactive client configuration for the express app to search user data on Auth0 in response to file processing messages.
Protected API calls check a JWT that is included in the headers.
Configuration has to be generated for the Angular app from the environment, leading us to rely more on grunt now.
We had to jump many versions of node to be compatible with the jwt auth libraries. Everything still works as far as we can tell.

@tie-rack Yeah, the duplication in notifications/sender.js and data-testing/sender.js is a known issue, and I just put in a card to refactor that when we get the chance. It'll involve some bigger structural changes (notifications is too feed-processing specific, so we want to change that, for example) in metis as well as changes in batch-address-test-tool to s/groupName/fipsCode. Here's the card: https://www.pivotaltracker.com/story/show/149821997


Fixed issue where HTTP headers set in a ClientOptions object passed to a client builder would not be set on a client instantiated by said builder.

Would it be useful to have utility methods in azure-core that help with this?

Probably. I'm not sure what the shape of those APIs would look like but setting headers included in CLientOptions is definitely something most of our Track 2 SDKs do.


Fix the wrong sortN condition

The isNormal is not supported in GraphQL-java, I changed MetricsCondition before, but forgot this one.
Now, it is working,
{
  sortMetrics(condition : {
    name: "test_long_metrics",
    topN: 10,
    scope: Service,
    order: DES
  },
  duration: {step:MINUTE, start: "2020-05-14 1030", end: "2020-05-14 1040"}
  ) {
    name,
    id,
    value,
    refId
  }
}

Codecov Report

Merging #4784 into master will decrease coverage by 51.12%.
The diff coverage is n/a.

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


Bug: WireMock config Live reload glitches on Windows

The WireMock config live reload starts working only after the first class reload. It looks like the extension properly registers the reload monitors only after the first Code reload. Or maybe it is a Quarkus glitch.
Reproduction steps

Run https://github.com/wiremock/wiremock-quarkus-devservice-demo on Windows /Powershell
Run quarkus dev
Access REST API for Hacker News
Edit the configuration resource
Reload the page

Error: The configuration does not change until the first classes reload
Versions

Quarkus Version and BOM: 3.5.3
WireMock Extension: 1.0.0

Hi @oleg-nenashev, thanks for reporting this. To be honest, it works as designed: WireMock config files aren't watched yet. That's why your config changes are just recognised when the live reload has been triggered by something else. However, I agree it would make totally sense to watch for such config changes as well. I'll provide a PR as soon as I have some time.
Does WireMock only support JSON config files or also YAML files? I've just checked the docu quickly, but couldn't find anything else than JSON. I'm just asking because I would need to know what type of files need to be watched.

Hi. YAML support is coming soon. There is a staged PR that we hope to merge soon https://github.com/wiremock/wiremock/issues/2411

I've implemented the requested feature with https://github.com/quarkiverse/quarkus-wiremock/pull/85. Would be nice if you could review it! Thanks!


Pretrained model url link broken?

This link pretrained weight parameters seems to be broken.

Sorry @oalieno, it was accidentally deleted. Just fixed.

Thanks for the update.


Radar chart

Extended to support a radar plot. Introduces a new drawing type for the series too: Area.

Going to merge. There is probably more that we can do, but I think this is a good start.


Why do I see H1 next to my heading & how do I turn that off?

Why do I see H1 next to my heading & how do I turn that off?

Style Settings (plugin) -> Blue Topaz Theme -> 2.2.3 Headering -> disable "Clutter Free Headings"

Thank you.
Regards,
Gloria (Peeps Graphics http://www.peepsgraphics.com/)
https://www.instagram.com/peepsgrfx
https://www.facebook.com/peepsgraphics
https://www.etsy.com/shop/peepsgraphics
https://www.redbubble.com/people/peepsgraphics/shop
On Sat, Apr 9, 2022 at 8:51 AM web8823 @.***> wrote:

Style Settings (plugin) -> Blue Topaz Theme -> 2.2.3 Headering -> disable
"Clutter Free Headings"
—
Reply to this email directly, view it on GitHub
https://github.com/whyt-byte/Blue-Topaz_Obsidian-css/issues/264#issuecomment-1094072727,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHABECMLOUMW4TCU3VQTYDVEGRPNANCNFSM5S5ZBBOQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


test: use -log-path for stderr access in ModuleInterface tests

The stderr handles seem to behave differently under Python 3.  Use the
-log-path option to capture the standard error output and validate the
output.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @tachoknight @drexin @CodaFi

@swift-ci please test

⛵


Fix #3579: Unexpected end of row when setup diesel

We use PRAGMA table_info for older versions of sqlite. This returns fewer columns that PRAGMA table_xinfo which we use in newer versions of sqlite, making the two code paths incompatbile with the Queryable implementations. We instead now use sql_query with an implementation of QueryableByName to return default values for the case of PRAGMA table_info.
Note this is a WIP implementation. It's my first contribution and dive into the diesel codebase so there is a good chance this is not the right way to do it.
Questions

Should I usederive instead of a manual implementation of QueryableByName?

Tests

Run cargo test --features sqlite --no-default-features successfully.
Run the shell script provided in #3579 .

NOT TESTED!
I have not tested the case of an older sqlite version as described in #3579 as I currently do
not have a setup for this. I will refer to the author of the issue to test this branch for me.

Okay I just found an issue in the implementation:
The following yields different results between xinfo and info:
sqlite> CREATE TABLE generated (
   ...>     id integer primary key,
   ...>     generated integer generated always as (id * 3)
   ...> );
sqlite> PRAGMA table_info(generated);
0|id|INTEGER|0||1
sqlite> PRAGMA table_xinfo(generated);
0|id|INTEGER|0||1|0
1|generated|INTEGER|0||0|2

This is the diesel_cli/tests/print_schema/print_schema_generated_columns_generated_always testcase.

Okay looking more into this. PRAGMA table_info won't return information on generated or hidden columns. So i think we have 3 choices:

Don't support sqlite < 3.26 without support for PRAGMA table_xinfo
Support 3.26 with PRAGMA table_info but don't support generated or hidden columns.
Find an alternative way to generate the schema. One approach (and the only one I found thus far), is to parse the CREATE TABLE statement that one can obtain via SELECT sql FROM sqlite_master WHERE type='table' AND name='<TABLE_NAME>' to generate the schema.

@weiznich I'll defer to you what is the right approach. My personal opinion is that 2. is the worst solution, since it will generate incorrect schemas. 1. would be my preferred appraoch, given that 3.26 is over 5 years old at this point, but it depends on the guarantees you want to give users of diesel. If we go with 3. I can give it a shot, depending on time available on my end. Just let me know waht you think is best.

Thanks for opening this PR and working on the underlying issue.
We want to go with the second variant (using PRAGMA table_info() on old versions). Its no problem that we don't get information about generated columns on such old sqlite versions, because generated columns itself are only supported since version 3.31.
The manual impl of QueryableByName is required there to set some of the fields to None.

@weiznich Okay cool. In this case i think the PR is actually fine as is, i just need a way to fix the unit tests for the case of PRAGMA table_info. Let me see that i can figure it out.

I think its fine for the unit test to assume that it is run with an sqlite version that supports generated columns. That means it will always use the table_xinfo branch. After all there is a difference between: You can use diesel with a really old sqlite version and you canbrun the tests against that old version and all of them pass.

Good catch. In fact this made me realised that getting the primary key information also relied upon pragma_info_table being Queryable.
Now the pull requests consists of 3 steps (I prefer smaller, insolated chunks):

Implement QueryableByName for ColumnInformation.
Implement QueryableByName for PrimaryKeyInformation and remove FullTableInfo as it is no longer needed.
Remove table! definition of pragma_info_table as it is now unused.

Hope this is what you hoped for :). Thanks for the patience


Addressable object should be extended

So far, the design of Addressable is more HTTP friendly.  We encountered several issues about the insufficient fields of Addressable on different protocols.  For example:

There is no field for Modbus RTU, so the current Modbus Device Service (Java) borrow the address and path field.
https://wiki.edgexfoundry.org/display/FA/Modbus+-+Adding+a+device+to+EdgeX
There is no field to put key and cert for the connection with higher security requirement.

We had better to add some fields for the common security related requirement, such as key, cert, or vault.  Also, we could add a generic object field for flexible extension of other protocols.

This is coming from my work for AWS export distro support. AWS (as with Google Core IoT, sometime MQTT) requires cert/key pair for connection but we do not have such properties in Addressable today. Current workaround is to define a export distro wide of cert/key in configuration.toml. The downside is obviously that we can only have one set of cert/key within an instance of export distro. I ever try to reuse current properties username and password for certificate and private key, only found that Google IoT core make user both cert/key and username/password, so we can reuse username/password. This probably true for reusing other properties since you will never know it will cause such conflict in the future or not. So here comes the issue that we should probably extend addressable object to suit those requirements.
@jduranf @feclare @drasko your thought ?

Cloud - refactor on Addressable is something we have always meant to come back to.  I don't think this happens in Delhi (impact too wide and deep).  Could happen for Edinburgh (a target we should set), but also will require more conversation in the Core WG - so as we don't have to make additional changes to Addressable in the future (to address other needs, Clouds, protocols, etc.)  Leave in @tsconn23 to cover in WG agenda as needed and put in roadmap after discussed.

@jpwhitemn Yes, we understand it won't happen in Delhi.  We just filed this issue for record and my team member is writing a proposal for this.  We could discuss in WG and Edinburgh.

hi @jpwhitemn @tsconn23 I have prepared a write-up here https://docs.google.com/document/d/1LawSpfUZ52FiVOkq0zISF9wRpcU-cxWpLDGgd1V-D2U/edit?usp=sharing for the proposal to extend Addressable in the future. Feel free to comment and let me know when do you want to discuss it in WS call.

Duplicated with #831. Please reopen if I'm wrong.


Tag container images by build

Currently, we tag all images as 'web','db' ... these get pushed to registry as 'web:latest','db:latest'. This happens even if the rest of the build fails. As a result, we have the potential for collision and version drift: one test might pick up the containers built by another. Also, a job deploying to prod might grab a version that's actually no good.
We should tag the images with build-specific tags and push those specific images to the test environment.
(Possibly: at the end of the build—on success—re-tag as "latest")

Issue was opened while in development, closing for now while project is in maintenance mode.


Failing tests on Windows

There are failing tests on Windows. My system is Node 4.2.4 with Windows 10:
BYK@ ~▶Documents▶Projects▶shelljs⛕ master$ npm test

><EMAIL_ADDRESS>test C:\Users\BYK\Documents\Projects\shelljs
> node scripts/run-tests

'.' is not recognized as an internal or external command,
operable program or batch file.
*** JSHINT FAILED! (return code != 0)

Running test: cat.js
Running test: cd.js
Running test: chmod.js

assert.js:89
  throw new assert.AssertionError({
  ^
AssertionError: 438 == 493
    at Object.<anonymous> (C:\Users\BYK\Documents\Projects\shelljs\test\chmod.js:23:8)
    at Module._compile (module.js:435:26)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Function.Module.runMain (module.js:467:10)
    at startup (node.js:136:18)
    at node.js:963:3
*** TEST FAILED! (missing exit code "123")

Running test: common.js
Running test: config.js
Running test: cp.js

assert.js:89
  throw new assert.AssertionError({
  ^
AssertionError: 0 == 1
    at Object.<anonymous> (C:\Users\BYK\Documents\Projects\shelljs\test\cp.js:145:8)
    at Module._compile (module.js:435:26)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Function.Module.runMain (module.js:467:10)
    at startup (node.js:136:18)
    at node.js:963:3
*** TEST FAILED! (missing exit code "123")

Running test: dirs.js
Running test: echo.js
Running test: env.js
Running test: exec.js
Running test: find.js
Running test: grep.js
Running test: ln.js

assert.js:89
  throw new assert.AssertionError({
  ^
AssertionError: false == true
    at Object.<anonymous> (C:\Users\BYK\Documents\Projects\shelljs\test\ln.js:51:1)
    at Module._compile (module.js:435:26)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Function.Module.runMain (module.js:467:10)
    at startup (node.js:136:18)
    at node.js:963:3
*** TEST FAILED! (missing exit code "123")

Running test: ls.js
Running test: make.js
Running test: mkdir.js
Running test: mv.js
Running test: popd.js
Running test: pushd.js

assert.js:89
  throw new assert.AssertionError({
  ^
AssertionError: 'resources\\pushd' == 'resources/pushd'
    at Object.<anonymous> (C:\Users\BYK\Documents\Projects\shelljs\test\pushd.js:193:8)
    at Module._compile (module.js:435:26)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Function.Module.runMain (module.js:467:10)
    at startup (node.js:136:18)
    at node.js:963:3
*** TEST FAILED! (missing exit code "123")

Running test: pwd.js
Running test: rm.js
Running test: sed.js
Running test: tempdir.js
Running test: test.js

assert.js:89
  throw new assert.AssertionError({
  ^
AssertionError: false == true
    at Object.<anonymous> (C:\Users\BYK\Documents\Projects\shelljs\test\test.js:76:8)
    at Module._compile (module.js:435:26)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Function.Module.runMain (module.js:467:10)
    at startup (node.js:136:18)
    at node.js:963:3
*** TEST FAILED! (missing exit code "123")

Running test: to.js
Running test: toEnd.js
Running test: touch.js
Running test: which.js

*******************************************************
WARNING: Some tests did not pass!
*******************************************************
npm ERR! Test failed.  See above for more details.

Well, this looks like a dupe of #294.

This is still a good issue to keep open. This will help us diagnose the actual errors with the tests.
#295 is a partial fix for #294. It looks like your PR #297 will help with the individual test cases.
The only other thing is to see if we can fix the issues with exec() spawning in the home directory instead of current directory for cmd.exe users (not sure if this is really even fixable).

The only other thing is to see if we can fix the issues with exec() spawning in the home directory instead of current directory for cmd.exe users (not sure if this is really even fixable).

Can you reference the issue from here too?

@BYK The issue with exec() is related to #250. When running a shelljs script from cmd.exe, I've seen that any exec() calls will run in the home directory.
require('shelljs/global');
echo(pwd()); // This is ShellJS's pwd command, which **will be** correct on Windows
exec('cd'); // Windows 'cd' is the equivalent of Unix's pwd command

I'm observing that, given I'm in a directory other than my home directory, then this will not output the same string twice (as I would expect it to).
I'm running this on Windows 7 64bit, using cmd.exe as my shell. If you see different behavior, with or without a different configuration, let me know.

@nfischer I can't reproduce this on my Windows 10, Node 4 installation. Tried both from cdm.exe and the shell I'm using (xonsh). Both output the directory I started node from which is the correct working directory. I simply run the following:
$ node
> require('shelljs/global')
{}
> exec('cd')
'the working directory I'm in followed by \r\n'

Interesting. I'm running through node v5 on my Windows machine. Perhaps it's Windows 7 specific, or node v5 specific (unlikely), or perhaps it's specific to my machine.
Personally, I don't care too much about running exec() on my personal windows machine, but I imagine that if I have the issue, others might as well. I've seen some articles online that mention that child_process.execSync calls will spawn in the home directory, so it appears to not be only me (but I can't find the posts right now). I've heard that running as admin can help the issue, but haven't tested this.
Either way, if anyone else can confirm either behavior on their windows system, that would be appreciated.

Fixed in #297


No Audio when download video using --format 'mp4[height<=720]' option

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Why there is no audio in the video when i download. I found https://github.com/yt-dlp/yt-dlp/issues/10189 I do not think this is a duplicate because I just said video height<=720 and did not tell anything about audio.
Is there any other option i need to use to ask to download audio as well? How can I be more specific and say download video with height height<=720 and that has audio.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
$ yt-dlp -vU --format 'mp4[height<=720]' --output "%(title)s.%(ext)s" "https://www.youtube.com/watch?v=7FChPvEOUHQ"
[debug] Command-line config: ['-vU', '--format', 'mp4[height<=720]', '--output', '%(title)s.%(ext)s', 'https://www.youtube.com/watch?v=7FChPvEOUHQ']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [c6387abc1] (zip)
[debug] Python 3.11.2 (CPython x86_64 64bit) - Linux-6.1.0-25-amd64-x86_64-with-glibc2.36 (OpenSSL 3.0.14 4 Jun 2024, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.6-0 (setts), ffprobe 5.1.6-0, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.46.0, pyxattr-0.8.1, requests-2.28.1, sqlite3-3.40.1, urllib3-1.26.12, websockets-10.4
[debug] Proxy map: {}
[debug] Request Handlers: urllib
[debug] Loaded 1838 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[youtube] Extracting URL: https://www.youtube.com/watch?v=7FChPvEOUHQ
[youtube] 7FChPvEOUHQ: Downloading webpage
[youtube] 7FChPvEOUHQ: Downloading ios player API JSON
[youtube] 7FChPvEOUHQ: Downloading web creator player API JSON
[youtube] 7FChPvEOUHQ: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[info] 7FChPvEOUHQ: Downloading 1 format(s): 609
[debug] Invoking hlsnative downloader on "https://manifest.googlevideo.com/api/manifest/hls_playlist/expire/1728339320/ei/GAkEZ8LZIJuHjuMP9NafwQo/ip/<IP_ADDRESS>/id/ec50a13ef10e5074/itag/609/source/youtube/requiressl/yes/ratebypass/yes/pfa/1/wft/1/sgovp/clen%3D12390688%3Bdur%3D505.713%3Bgir%3Dyes%3Bitag%3D247%3Blmt%3D1653137061502232/rqh/1/hls_chunk_host/rr1---sn-pjjoxu-q5j6.googlevideo.com/xpc/EgVo2aDSNQ%3D%3D/mh/VA/mm/31,29/mn/sn-pjjoxu-q5j6,sn-h557sn6d/ms/au,rdu/mv/m/mvi/1/pcm2cms/yes/pl/24/vprv/1/playlist_type/DVR/dover/13/txp/5319224/mt/1728317161/fvip/1/short_key/1/keepalive/yes/fexp/51300761/sparams/expire,ei,ip,id,itag,source,requiressl,ratebypass,pfa,wft,sgovp,rqh,xpc,vprv,playlist_type/sig/AJfQdSswRQIgb-K7JVgEHifoYliSbGRemBC3S3ubHoMMqll5symC5FkCIQCYGQFF5U7rVDdf9tyv0zFa0xI-L5dsp1Lut_9xK5d-Ng%3D%3D/lsparams/hls_chunk_host,mh,mm,mn,ms,mv,mvi,pcm2cms,pl/lsig/ACJ0pHgwRAIgW1QwYvTE1zKqXvpx5TW0sjda-bKUgbqNk3VT4c8Fw8cCIFNSDQPra773MPxjgR4yznrkqJ-jiJ4rcLNNaHItb5gK/playlist/index.m3u8"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 95
[download] Destination: 6 Minute core workout for beginners.mp4
[download] 100% of   11.88MiB in 00:01:35 at 127.45KiB/s
[debug] ffprobe command line: ffprobe -hide_banner -show_format -show_streams -print_format json 'file:6 Minute core workout for beginners.mp4'

Use -S "res:720" instead. If you want a mp4 for some reason do not pass -f mp4 since it will choose only one format (video+audio if available, only video otherwise), but use -S res:720,vcodec:h264,acodec:aac

Duplicate of #11185


added .query_raw(), .index(), .mget(), fixed missing import, always POST on query/query_raw

Thanks, I am running some tests and it is failing in the requests module. It looks like auth  = (None, None) evaluates to true for some reason which then gets passed on to requests that craps out when it sees that. I don't think this has been changed recently, so I am not sure what is going on.

I am running into an issue with unit tests passing. I've created a new dev branch, lets use that branch for active development. I will merge back to master once everything is passing.

Can you post your test log ?

Mine is:
 ('N/A', "HTTPConnectionPool(host='localhost', port=963): Max retries exceeded with url: /solr/SolrClient_unittest/schema/fields?wt=json&indent=false (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x7f7056795b70>: Failed to establish a new connection: [Errno 111] Connection refused',))", ConnectionError(MaxRetryError("HTTPConnectionPool(host='localhost', port=963): Max retries exceeded with url: /solr/SolrClient_unittest/schema/fields?wt=json&indent=false (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x7f7056795b70>: Failed to establish a new connection: [Errno 111] Connection refused',))",),))

For some reason solr is refusing the connection. Maybe solr has a connection limit or soemthing..

I have the other issues figured out, will commit them a bit later.
This connection refused error is most likely because Solr 6.3 is not in Vagrant config, so it doesn't get installed. There are a few files that need to be edited to get the new version to work and have to make a sample config (schema, solrconfig). There are some notes in that repo and in SolrClient test readme on, you can try to take a stab at it, otherwise I can do it tonight. I want to remove some of the older versions that I am not working on from the list as well and trim the fat on that VM.

I will leave that to you since you have experience on it.
I'm working on the smart routing that I mentioned on the other issue. Hopefully after that we'll have a new release ?


50% failure rate to gather info with  gdrive info  | grep Modified

While the info command seems to have a 100% success rate, however when using grep to return just one line of the results it fails to give any result about 50% of the time.  Weird, eh.
The help for grive info doesn't make sense for the info subcommand, or I'm expecting different detail from it.

It's most likely caused by API limit issue that currently everyone experiencing. Check #426 for workaround for API limits, until we introduce an official fix

Can you confirm whether it's related to API limit issue?

Closing, reopen if you have followup.