v20.07.01 ontap-san unable to write

Describe the bug
I had to install a Cluster from scratch today while deploying it with trident v20.07.1 i always (4 times) ended up with volumes (created from ontap-san driver) which were not writeable from within the pod. As soon as i deleted the Cluster and reinstalled with v20.07.0 it worked again, as i use a playbook to install my cluster i can guarantee that there hasnt been any change inbetween those two versions.
Environment
Dev

Trident version: 20.07.01
Trident installation flags used: operator install with autosupport disabled
Container runtime: CRIO
Kubernetes version: v1.18.3+b0068a8
Kubernetes orchestrator: OpenShift 4.5.11
Kubernetes enabled feature gates: Default
OS: Rhel CoreOS Nodes on 4.5.11
NetApp backend types: ontap-san

To Reproduce
Create Cluster from Scratch with v20.07.1, create a pvc with backend type ontap-san and try to write to the disk
Expected behavior
Write should be possible
Additional context
Please note that this only occured on ontap-san my other pvc's with ontap-nas were working as expected. I also noted that writing on the Host directly as root to the disk was working, only when inside the pod (which doesnt run as root) it didnt work.
If you need more Infos let me know i will install another Cluster next week so i could gather Logs for you :)

Hi @Numblesix,
We test the ontap_san storage driver in a fresh OCP 4.5 cluster in our CI. Obtaining logs using tridentctl logs -a for the issue you are experiencing would be very helpful. We'll also want to open a NetApp Support case. Let me know if you need any assistance.

Hello,
I have the same problem, with v20.07.1 the pod cannot write with random uid:
(openshift 4.5.11 and 4.5.13)
only root is allowed to write to volume
$ls -al /iscsi-20-07-1/
total 20
drwxr-xr-x. 3 root root  4096 Oct  6 10:45 .
drwxr-xr-x. 1 root root   117 Oct  6 10:50 ..
drwx------. 2 root root 16384 Oct  6 10:45 lost+found

after a downgrade to v20.07 and create a new volume:
ls -al /iscsi-20-07/
total 20
drwxrwsr-x. 3 root<PHONE_NUMBER>  4096 Oct  6 10:50 .
drwxr-xr-x. 1 root root         117 Oct  6 10:50 ..
drwxrws---. 2 root<PHONE_NUMBER> 16384 Oct  6 10:50 lost+found

@sitzm that was exactly my issue could you share logs via tridentctl logs -a

@sitzm and @Numblesix we haven't been able to reproduce this issue in our OCP CI tests. We suspect that you may be running into an issue with the new version of the CSI provisioner sidecar based on your configuration. Please provide examples of the Storage Class, Backend config (redact personal information), PVC, and Pod definitions.

@gnarl here is the configuration:
backend:
{
    "version": 1,
    "storageDriverName": "ontap-san",
    "backendName": "OpenShift RF",
    "managementLIF": "10.x.x.135",
    "dataLIF": "10.x.x.128",
    "svm": "svm1234",
    "igroupName": "eGov_client4",
    "username": "username",
    "password": "password",
    "storagePrefix": "nonprod"
}

Storageclass
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: trident-iscsi
parameters:
  backendType: ontap-san
provisioner: netapp.io/trident
reclaimPolicy: Delete
volumeBindingMode: Immediate

pvc
kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  annotations:
    pv.kubernetes.io/bind-completed: 'yes'
    pv.kubernetes.io/bound-by-controller: 'yes'
    volume.beta.kubernetes.io/storage-provisioner: csi.trident.netapp.io
  selfLink: /api/v1/namespaces/test-sim/persistentvolumeclaims/test-iscsi-4-5-13
  resourceVersion: '25896884'
  name: test-iscsi-4-5-13
  uid: d8ef5807-8779-46b3-8740-9864017acb87
  creationTimestamp: '2020-10-06T10:32:59Z'
  namespace: test-sim
  finalizers:
    - kubernetes.io/pvc-protection
spec:
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 2Gi
  volumeName: pvc-d8ef5807-8779-46b3-8740-9864017acb87
  storageClassName: trident-iscsi
  volumeMode: Filesystem
status:
  phase: Bound
  accessModes:
    - ReadWriteOnce
  capacity:
    storage: 2Gi

pod
kind: Pod
apiVersion: v1
metadata:
  generateName: nginx-7bf97cbfb6-
  annotations:

    openshift.io/generated-by: OpenShiftWebConsole
    openshift.io/scc: restricted
  selfLink: /api/v1/namespaces/test-sim/pods/nginx-7bf97cbfb6-kxhw2
  resourceVersion: '26854815'
  name: nginx-7bf97cbfb6-kxhw2
  uid: 8403ff59-8f83-454b-a9ce-98a0c63dc1cd
  namespace: test-sim
  ownerReferences:
    - apiVersion: apps/v1
      kind: ReplicaSet
      name: nginx-7bf97cbfb6
      uid: c3a9aa63-011f-4cfb-8f97-e2cb23f7c019
      controller: true
      blockOwnerDeletion: true
  labels:
    app: nginx
    deploymentconfig: nginx
    pod-template-hash: 7bf97cbfb6
spec:
  nodeSelector:
    node-role.kubernetes.io/worker: ''
  restartPolicy: Always
  serviceAccountName: default
  imagePullSecrets:
    - name: default-dockercfg-f2fss
  priority: 0
  schedulerName: default-scheduler
  enableServiceLinks: true
  terminationGracePeriodSeconds: 30
  nodeName: node1
  securityContext:
    seLinuxOptions:
      level: 's0:c30,c20'
    fsGroup:<PHONE_NUMBER>
  containers:
    - resources: {}
      terminationMessagePath: /dev/termination-log
      name: nginx
      securityContext:
        capabilities:
          drop:
            - KILL
            - MKNOD
            - SETGID
            - SETUID
        runAsUser:<PHONE_NUMBER>
      ports:
        - containerPort: 8080
          protocol: TCP
        - containerPort: 8443
          protocol: TCP
      imagePullPolicy: IfNotPresent
      volumeMounts:
        - name: test-iscsi-4-5-13
          mountPath: /test
        - name: default-token-f6lp5
          readOnly: true
          mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      terminationMessagePolicy: File
      image: >-
        bitnami/nginx@sha256:ff9da47b2c33fb253550f1ad50350c1e65edc9c18770435ff752a3f017fb7ab6
  serviceAccount: default
  volumes:
    - name: test-iscsi-4-5-13
      persistentVolumeClaim:
        claimName: test-iscsi-4-5-13
    - name: default-token-f6lp5
      secret:
        secretName: default-token-f6lp5
        defaultMode: 420
  dnsPolicy: ClusterFirst

I can also confirm this issue with 20.07.1 and ontap-nas.  Specifically, I noticed that post the PVC mount  that the securityContext: fsgroup was not being applied as usual. kubelet logs would normally register a step for "Setting volume ownership" after completion of the WaitForAttach and MountDevice steps.
I also verified with a test pod that the pvc  only had the default 0755, root:root permissions where in 20.07.0 I have the set ownership step logged and 2775 root:mygroup  on the volume as defined by the securityContext.  Downgrading to 20.07.0, removing the underlying volume (must do this) and recreating fixed the issue and perms were correct again.

We got the same problem with Trident v20.07.1 and Kubernetes v1.18.6.
It seems that the issue was due to the absence of fsType=ext4 in PersisnteVolume created by v20.07.1. When we set fsType=ext4 parameter explicitly in StorageClass, we do not see this issue.
I think updating external-provisioner to v2.0.1 in https://github.com/NetApp/trident/commit/b33021d64da7588d65b7f38cda7cd01873850e14 introduced this issue. I guess default-fstype argument should have been added with this update.

https://github.com/kubernetes-csi/external-provisioner/blob/release-2.0/CHANGELOG/CHANGELOG-2.0.md#changelog-since-v160
The fstype on provisioned PVs no longer defaults to "ext4". A defaultFStype arg is added to the provisioner. Admins can also specify this fstype via storage class parameter. If fstype is set in storage class parameter, it will be used. The sidecar arg is only checked if fstype is not set in the SC param. (#400, @humblec)

I can also share some stuff now :)
StorageClass
allowVolumeExpansion: true
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: netapp-csi-block
mountOptions:
- discard
parameters:
  backendType: ontap-san
provisioner: csi.trident.netapp.io
reclaimPolicy: Delete
volumeBindingMode: Immediate

Backend
{
    "version": 1,
    "storageDriverName": "ontap-san",
    "backendName": "ontap_san",
    "managementLIF": "<IP_ADDRESS>",
    "svm": "netapp-server",
    "username": "trident",
    "password": "adsfasdfasdf",
    "storagePrefix": "osr2_iscsi",
    "igroupName": "osr2_iqn",
    "useCHAP": true,
    "chapInitiatorSecret": "adsfadsf",
    "chapTargetInitiatorSecret": "adsfasdf",
    "chapTargetUsername": "adfadf",
    "chapUsername": "asdfadsf",
    "defaults": {
        "spaceReserve": "volume",
        "spaceAllocation": "false",
        "snapshotPolicy": "none",
    }
}

PVC
kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: pvc-mysql-block
spec:
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 5Gi
  storageClassName: netapp-csi-block

Pod
apiVersion: v1
kind: Pod
metadata:
  annotations:
    k8s.v1.cni.cncf.io/network-status: |-
      [{
          "name": "openshift-sdn",
          "interface": "eth0",
          "ips": [
              "<IP_ADDRESS>"
          ],
          "default": true,
          "dns": {}
      }]
    k8s.v1.cni.cncf.io/networks-status: |-
      [{
          "name": "openshift-sdn",
          "interface": "eth0",
          "ips": [
              "<IP_ADDRESS>"
          ],
          "default": true,
          "dns": {}
      }]
    openshift.io/scc: restricted
  generateName: wordpress-mysql-77cb4466fc-
  labels:
    app: wordpress
    pod-template-hash: 77cb4466fc
    tier: mysql
spec:
  containers:
  - env:
    - name: MYSQL_ROOT_PASSWORD
      valueFrom:
        secretKeyRef:
          key: password
          name: mysql-pass
    image: mysql:5.6
    imagePullPolicy: IfNotPresent
    livenessProbe:
      failureThreshold: 3
      periodSeconds: 10
      successThreshold: 1
      tcpSocket:
        port: 3306
      timeoutSeconds: 1
    name: mysql
    ports:
    - containerPort: 3306
      name: mysql
      protocol: TCP
    resources: {}
    securityContext:
      capabilities:
        drop:
        - KILL
        - MKNOD
        - SETGID
        - SETUID
      runAsUser:<PHONE_NUMBER>
    terminationMessagePath: /dev/termination-log
    terminationMessagePolicy: File
    volumeMounts:
    - mountPath: /var/lib/mysql
      name: mysql-persistent-storage
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: default-token-9v4rc
      readOnly: true
  dnsPolicy: ClusterFirst
  enableServiceLinks: true
  imagePullSecrets:
  - name: default-dockercfg-h8xcc
  nodeName: node.domain.de
  nodeSelector:
    node-role.kubernetes.io/app: ""
  priority: 0
  restartPolicy: Always
  schedulerName: default-scheduler
  securityContext:
    fsGroup:<PHONE_NUMBER>
    seLinuxOptions:
      level: s0:c26,c5
  serviceAccount: default
  serviceAccountName: default
  terminationGracePeriodSeconds: 30
  tolerations:
  - effect: NoExecute
    key: node.kubernetes.io/not-ready
    operator: Exists
    tolerationSeconds: 300
  - effect: NoExecute
    key: node.kubernetes.io/unreachable
    operator: Exists
    tolerationSeconds: 300
  volumes:
  - name: mysql-persistent-storage
    persistentVolumeClaim:
      claimName: pvc-mysql-block
  - name: default-token-9v4rc
    secret:
      defaultMode: 420
      secretName: default-token-9v4rc
status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: "2020-10-09T07:46:12Z"
    status: "True"
    type: Initialized
  - lastProbeTime: null
    lastTransitionTime: "2020-10-09T07:46:12Z"
    message: 'containers with unready status: [mysql]'
    reason: ContainersNotReady
    status: "False"
    type: Ready
  - lastProbeTime: null
    lastTransitionTime: "2020-10-09T07:46:12Z"
    message: 'containers with unready status: [mysql]'
    reason: ContainersNotReady
    status: "False"
    type: ContainersReady
  - lastProbeTime: null
    lastTransitionTime: "2020-10-09T07:46:12Z"
    status: "True"
    type: PodScheduled
  containerStatuses:
  - image: mysql:5.6
    imageID: ""
    lastState: {}
    name: mysql
    ready: false
    restartCount: 0
    started: false
    state:
      waiting:
        message: secret "mysql-pass" not found
        reason: CreateContainerConfigError
  hostIP: <IP_ADDRESS>
  phase: Pending
  podIP: <IP_ADDRESS>
  podIPs:
  - ip: <IP_ADDRESS>
  qosClass: BestEffort
  startTime: "2020-10-09T07:46:12Z"

Thanks for all of the feedback! It does look like an issue with the CSI external-provisioner sidecar v2.0.1. We are investigating how to fix or workaround the issue.

It looks like the problem is here.
Since the CSI external-provisioner no longer enforces a default fsType (and is something that Trident should take care of from now on), there are two ways to fix it in the meantime:

Provide a fsType in your StorageClass.
Generate Trident's custom YAMLs and edit the trident-deployment.yaml file to include an additional arg for the csi-provisioner like this:

      - name: csi-provisioner
        image: quay.io/k8scsi/csi-provisioner:v2.0.1
        args:
        - "--v=2"
        - "--timeout=600s"
        - "--csi-address=$(ADDRESS)"
        - "--default-fstype=ext4"

Option (1) is the easiest way to fix it for now.

@gnarl I believe that the bug are occurred by missed fsType. As you may know, the parameter in StorageClass is immutable value. So we need to delete and re-create it. I think the bug is critical for customer. When we use Trident Operator, we can't chose 2nd workaround in https://github.com/NetApp/trident/issues/454#issuecomment-706301546 .  Customer has already ask it to us. I hope that NetApp provide Trident v20.07.2.

This bug exists outside of Trident and additional information can be found in this NetApp KB article.
The fix for this issue is to use a StorageClass that specifies the fsType to be used with SAN volumes. Since StorageClasses cannot be patched, existing StorageClasses must be deleted and recreated as is, along with parameter.fsType defined.
This will not impact any existing volume already created and does not carry any dependency with pre-provisioned volumes associated with the StorageClass.

@gnarl
As @tksm has already commented (https://github.com/NetApp/trident/issues/454#issuecomment-705926951 ) , it is necessary to specify the defaultFStype from external provisioner v2.0.1 .
https://github.com/kubernetes-csi/external-provisioner/blob/release-2.0/CHANGELOG/CHANGELOG-2.0.md#changelog-since-v160
The fstype on provisioned PVs no longer defaults to "ext4". A defaultFStype arg is added to the provisioner.
Admins can also specify this fstype via storage class parameter. If fstype is set in storage class parameter, it will be used.
The sidecar arg is only checked if fstype is not set in the SC param. (#400, @humblec)

Unfortunately, NetApp/Trident v20.07.1 doesn't set the defautlFStype.
I think that NetApp/Trident should set it.
So, I have few questions.
Is the permanent workaround that sets fsType parameter in a StorageClass ?
Or,
the temporary workaround until the bug is fixed in NetApp/Trident .
Please let me know about the policy of NetApp/Trident for the bug.
Our support policy for the next Trident version depends on whether fsType in StorageClass is required.

Hello @ysakashita
Setting the fsType on a StorageClass is a permanent fix. The reason Trident does not set defaultFStype is because it also needs to create NFS volumes. For NFS volumes, the fsType needs to be blank. The fsType that is provided in the StorageClass will override the default value (which is blank)

@balaramesh Got it thanks

Closing this issue as information regarding how to set the fsType is now included in the Trident documentation.

Closing this issue as information regarding how to set the fsType is now included in the Trident documentation.


Add setter methods for Sentry client

ref: #69

Coverage decreased (-4.4%) to 78.283% when pulling 97be22407627312bc509258f5402bda9bb148a61 on issue/69 into fbeefd0fc915181cdfc145b1edc059c15429a4d0 on master.


zero_copy branch example code

I just wanted to ask if there is an example code snippet, on how to use this branch, because I heard I need to use special functions. I got told by @Luxonis-Erik that opening an issue might give me an answer.

Thanks.  We'll circle back.  I don't recall the context of this.  Will ask.

Hi @0y8w1x,
if you checkout the branch, all output queues (the return value of the device.getOutputQueue(...)) will automatically have the zero-copy functionality.
I'm not sure how tested it is and if there are any known issues with the implementation, so CC: @themarpe  regarding this.

@0y8w1x
Main thing to note there is that the data isn't available through regular getData means but through packet->data instead. Also, getFrame & getCvFrame and other message helpers aren't available.
This branch is experimental / feature testing branch, and it might or might not be upstreamed one day - it also isn't inteded for regular consumption, but for those working at the very edge of compute / latency / overhead boundaries.

thanks @themarpe and @moratom for clarifying. I guess I need to go through the commits, to create a working example then 👍


feat: remove node 14 and update gh actions

Coverage: 100.0%. Remained the same when pulling f5573ebf200ff995b6ba17c2694544df00bc60e1 on lholmquist:update-gh-actions-to-latest into 2d1060d1096f7bc6b34ca94d62d33ecf5708a606 on nodeshift:main.


Use old cache when internet not available

If the cache can't be refreshed then it would be good to be able to use the file currently in cache.

i have fixed the documentation

I'm confused here - I don't see how the code uses old cache even if it's present but not valid.
It's seems to me that is it's not valid then an attempt is made to get the data again, which may fail and then return null

sorry please check the example again i fix it

Thanks, however while the syntax of the code is now good. It still throws an exception on this line
var domainParser = new DomainParser(webTldRuleProvider); if the internet is not available.
Looking at the code, the above line ALWAYS attempts to validate the cache and load it if invalid.
So the additional code IsCacheValid will always return true.

can you check again i have update the example again and push a new nuget package

I have downloaded the latest nuget package and got the update, the exception is no longer thrown which is good. However it is still not possible to load the existing data or old data from the cache when the internet is not available.
When the internet is not available - DomainParse has only 1 rule in it.

If your internet not available increase the cache time or write a own implementation

The ICacheProvider allow you to implement your own cache provider.

OK, that won't work, I see more is required ...


Search box no longer in Source view in Aardvark as of 2.6

There used to be a search box in the "Source" tab of the document detail view triggered if you press Ctrl+F (probably Cmd+F on OSX). As of 2.6 that seems to be gone. I'm suspecting we updated the ACE editor at some point and missed a flag or accidentally removed the key binding (I recall an earlier issue where we removed some key bindings).
This wouldn't be much of a problem but the browser's "Find in page" feature can only search the parts of the source code that is presently visible, rendering it pretty much useless in any scenario where you'd want to use search.

fixed (updated jsoneditor lib): bb963fe8c0e57bfa092c6910896de7d4ef0e6407

This doesn't seem to have been fixed in 2.6.9.

Some minor issues persist even after the update:

after clicking into the search box, the cursor is not visible. It is unclear to the user that the cursor is actually in the search box.
typing a non-existing attribute name into the search box will clear all output. So far, so good. However, when removing the attribute name from the search box (so the search box is empty), the document attributes don't show up again. It is then necessary to type some valid attribute name to make them re-appear.

Just to be clear: I'm talking about search in source mode, not in tree mode.

I was talking about the one in tree mode. There isn't (yet?) a search box in source mode in current 2.7.

Exactly. There was a search box in 2.5 and it disappeared in 2.6. Because the editor doesn't actually render the entire source tree into the DOM, the browser's "find" command doesn't work for documents longer than ~20 lines.

The search box at the source view is back again. You have to enable it using CTRL+F or CMD+F (Mac). Hitting that shortcuts again opens a replace dialog.

@jsteemann

after clicking into the search box, the cursor is not visible. It is unclear to the user that the cursor is actually in the search box.

cannot reproduce this issue. tested on os x chrome,firefox and safari.

typing a non-existing attribute name into the search box will clear all output. So far, so good. However, when removing the attribute name from the search box (so the search box is empty), the document attributes don't show up again. It is then necessary to type some valid attribute name to make them re-appear. This sucks.

added event to check if box is empty or not. when value is empty, all attributes will expand.

@hkernbach has this been fixed in 2.7.0?

seems to be a bug in firefox (seen in ubuntu). can't fix


Try Dockstore on a degraded network

Bug Report
Encountered by @briandoconnor  in Orlando, on a very slow connection, it looks like search might not load properly and times out.

Simulate a slow connection or try a variety of mobile devices on different connections.

Notice the length of time the page takes to load and the size (in MB) of data the page pulls.  On a degraded/slow connection this is very noticeable.  Seems strange the returns would be MB in size!?

@agduncan94 @garyluu
Part of it actually makes sense, we haven't implemented pagination properly yet.
The stranger issue is that it's making that call four times.
We can lower the default limit of tools+workflows from 200 to 50, which will help a bit. Thanks for the heads-up.

My local one has 3 small 2 large requests.  Staging looks like it has 4 large, 2 small requests.

From discussion:
https://www.elastic.co/guide/en/elasticsearch/reference/current/search-request-source-filtering.html should cut down on what's being returned enough actually without changing limits

We could implement pagination with elastic search as it pretty straightforward. Datatables pagination is a bit weird anyways. This also will help with permalinks referencing a specific page in the search.

Ideas: search looks more usable on a mobile connection at least at home and downloads about as much as the main landing page.
Take a look and see what else breaks if you browse to dockstore with a weak connection. Either simulate a slow connection using Linux utilities or just use the laptop in the elevator lobby or the OICR bridge as examples.

Can use https://developers.google.com/web/tools/chrome-devtools/network-performance/reference#throttling to simulate slow speeds.
Looks like Dockstore needs at least "Fast 3G" to be sane


Bulk Export Status Report

Regarding one more scenario for bulk export
A scenario while testing in Inferno
Request URL:
https://***:9443/fhir-server/api/v4/$bulkdata-status?job=HRheZqFY_KT5TXhdXFuY7g
Response Body:
{
"transactionTime": "2021-08-18T10:21:53.82Z",
"request": "https://***:9443/fhir-server/api/v4/Group/17b58c31108-77f6763e-008b-4010-b428-c42ee061aa57/$export",
"requiresAccessToken": false,
"output": [{
"type": "AllergyIntolerance",
"url": "_jVMAj-5zOenYBhcTN5pa5b6DvR2rsx2Cpv4l9wuXOY/AllergyIntolerance_1.ndjson",
"count": 10
},
{
"type": "CarePlan",
"url": "_jVMAj-5zOenYBhcTN5pa5b6DvR2rsx2Cpv4l9wuXOY/CarePlan_1.ndjson",
"count": 2
}
]
}
This test case is a pass, except that it needs few extra things in the response as mentioned below:
"requiresAccessToken": false. This parameter in the response is false, we need to return this in response as true. We have done all the required changes but are not able to get this. Seems that it is fixed value.
Could you please let us know how to achieve this?
https://chat.fhir.org/#narrow/stream/212434-ibm/topic/releases/near/250499231

Marked this as an enhancement.

Adding an entry to the storageProviderfhirServer/bulkdata/storageProviders/" + provider + "/requiresAccessToken

I confirmed that the "requiresAccessToken" in the output now takes it value from this new config property and that the default is still false.


make selection algoirithm model independent

One mesh can have its faces selected and then displayed to the player. The mesh that is the target of the selection must be labeled "selectable". This is sufficient for now, but may need to be expanded to groups of meshes in the future.

Is expanded to work with groups of meshes as of 5/22


Add Keploy

website:- https://github.com/keploy/keploy
Keploy is an open-source tool for developers that captures network calls, analyzes code paths, and generates API tests. It automates API test scenario creation, filters redundant tests using code coverage, and converts outgoing calls into stubs.

I'll open a newer one with more updated details!


fix: use output when looking up relational related type index

Description of changes
This allows the implicitly added 'id' field to be properly used.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

lgtm pending ci

Codecov Report

Merging #8657 (d61bddd) into master (70547ad) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #8657   +/-   ##
=======================================
  Coverage   56.88%   56.88%
=======================================
  Files         756      756
  Lines       42238    42239    +1
  Branches     8649     8649
=======================================
+ Hits        24027    24028    +1
  Misses      17390    17390
  Partials      821      821

Impacted Files
Coverage Δ

...mplify-graphql-relational-transformer/src/utils.ts
100.00% <100.00%> (ø)

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 70547ad...d61bddd. Read the comment docs.


Add support for local OPTIONS CORS headers

Looks like I couldn't push directly to the #437 for some reason so I opened a separate PR.
closes #437
fixes #436
cc @kyleknap @jamesls

Codecov Report

Merging #444 into master will increase coverage by 0.22%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #444      +/-   ##
==========================================
+ Coverage    93.6%   93.83%   +0.22%
==========================================
  Files          18       18
  Lines        2863     3097     +234
  Branches      375      426      +51
==========================================
+ Hits         2680     2906     +226
- Misses        132      141       +9
+ Partials       51       50       -1

Impacted Files
Coverage Δ

chalice/local.py
97.61% <100%> (+0.14%)
:arrow_up:

chalice/constants.py
100% <0%> (ø)
:arrow_up:

chalice/deploy/deployer.py
95.02% <0%> (+1.29%)
:arrow_up:

chalice/cli/__init__.py
83.69% <0%> (+1.82%)
:arrow_up:

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 69f18d7...d55a125. Read the comment docs.

Great commenting. I wasn't sure if you folks would like the change hence my quick-n-dirty fix. Thanks for reviewing!


[CONTRIB] TVM download utility based on urllib2/urlib.request

Moving nnvm/testing/download.py to python/tvm/contrib/download.py, and cleaning it up. Tested with Python 2 & 3 for correctness and exception handling.

Fixed the progress bar completion derivation


Make responsive

Description
##Issue no #558
Made the menu items cards on the home page more responsive. I have attached videos showing changes before and after updating.
Related Issues
None

Closes #

Type of PR

[] ()

Screenshots / videos (if applicable)
Responsiveness before updating
https://github.com/user-attachments/assets/5c3e9025-bb08-47ae-9e30-14b3be386471
Responsiveness after updating
https://github.com/user-attachments/assets/13b38dad-e8e9-4d0b-ae2f-22b4d13d5011
Checklist

[X] I have gone through the contributing guide
[X] I have updated my branch and synced it with project main branch before making this PR
[X] I have performed a self-review of my code
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional context:
I have made changes in style.css in .menu-items by adding flex-wrap and justifying content to the center. Also, I have changed the padding to 20px and height to 400px to make the cards be arranged systematically.

@Amrut00
send a preview screen recording of ur changes from the below deploy link
https://deploy-preview-562--rococo-mooncake-0d89f7.netlify.app/

Here's the video.
https://github.com/user-attachments/assets/d0b549ff-5417-4aed-a7be-c711efa475ad

Here's the video.
Recording.2024-10-15.200323.mp4

only that section u made responsive

Yup, the other sections were already responsive.

I can work with other sections too if any?


DropColumnIfExists

drop has a dropIfExists. But why does dropColumn not have a dropColumnIfExists function?
I feel that the method should exist as when I tried some migrations for pluggable components which has their own migrations, I faced an issue in this scenario:

Plugin B modifies migration of Plugin A to add column user_id .
Plugin A is reset (rollback and upgrade), now when Plugin B is reset it throws an error:

[PDOException]
SQLSTATE[42000]: Syntax error or access violation: 1091 Can't DROP 'user_id'; check that column/key exists

I ended up changing
public function down()
{
    if (Schema::hasColumn(‘users', 'user_id'))
    {
        $table->dropColumn(['user_id']);
    }
}

to:
public function down()
{
    if (Schema::hasColumn(‘users', 'user_id'))
    {
        Schema::table('users', function($table) {
            $table->dropColumn(['user_id']);
        });
    }
}

I feel it would be simpler to just have a dropColumnIfExists. Makes it consistent with the drop - dropIfExists pair as well.

Here the reason why i gave a thumbs down reaction. Because dropColumnIfExists doesn't have a direct SQL translation. Take the example of the commands you listed:
drop (DROP TABLE foo;)
dropIfExists (DROP TABLE IF EXISTS foo;)
dropColumn (ALTER TABLE foo DROP COLUMN bar;)
So by this pattern dropColumnIfExists should return ALTER TABLE foo DROP COLUMN bar; which is invalid SQL.
Here is how this code block works:
    Schema::table('users', function($table) {
        $table->dropColumn(['user_id']);
    });

First Schema/Builder.php is loaded.  This deals with the database itself.
For callback Schema/Blueprint.php is loaded. This deals with the table.

What you are suggesting is setting a $builder property inside a Blueprint.php file to access Builder.php methods, which means every time you call any methods through Schema you will always set the $builder property. $builder will always remain redundant & of no use unless you specifically call dropColumnIfExists in Blueprint.php.
I can't think of any reason to have a dropColumnIfExists

I agree above with @srmklive that the method doesn't make sense just because it does not translate to SQL. I do however think there is something to the issue that should be addressed.
Perhaps for now though the real issue is a package assuming it can safely modify an existing table. For the sake of non-collision, either the developer of a project needs to rollback migrations in the order they migrated them, or the developer has to reject any package that tries to modify an existing table. Instead, a package should create a 1-1 relationship with an existing table and a new table generated with a migration in the package.

If Eloquent is just a translation library might as well use DROP TABLE foo;, DROP TABLE IF EXISTS foo;, etc. There would be no point for the Eloquent ORM system. I see the whole point of this is to bring in schema building and migration scripts to make it simpler for developers.
My issue was just pointing out the flaw in the system that pluggable components break down because of this (do see the Plugin A, B sample scenario I provided). Like @miscbits suggested, perhaps this becomes the developer’s headache now since Eloquent system fails to cover this flaw and collisions do occur.

@SaifurRahmanMohsin kindly look at my last reply as why i think i don't see any compelling reasons for inclusion of such method in the framework. However if you believe that it is a issue, instead of discussing it back & forth, why don't you simply generate a PR for inclusion into the framework.

Closing this since it's not a bug report, but please feel free to open a PR in case you're still willing to propose such functionality.


Unique name for output file

Great plugin, thanks!
It's not critical, but can you add the option for setting the name of output file?
This can be useful for users like me who have a lot of notes in the root folder and who constantly forget what the output file is called :)

You can just recall the command. The output file will be opened automatically. With the newest release even in an new pane. But if you want to I can add this option too.

The output file will be opened automatically

Hmm, in my case it works on a mac, but does not work on win - the output file was only created there, not opened

Hmm, if you open the console and call the command on win, is there any error? Which version is it?

I reloaded Obsidian via ctrl+R, and now it's opening :)
Now I happy and don't want anymore else 😊
Thanks!

Great to hear. I dont know why it didn't work, but if it works after reload I think everything is fine.


Direct local .aar file dependencies are not supported when building an AAR.

Your Environment

Plugin version: 0.08
Platform: Android
OS version: Microsoft Windows [Version 10.0.19041.508]
Flutter info (flutter doctor):
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 1.22.0, on Microsoft Windows [Version 10.0.19041.508], locale en-US)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
[√] Android Studio (version 4.0)
[√] Connected device (1 available)
• No issues found!

Plugin config:

  flutter_zoom_plugin:
    git:
      url: git://github.com/decodedhealth/flutter_zoom_plugin.git
      # Android
      ref: 0.0.8
      # iOS
      #ref: master

Expected Behavior
Application runs
Actual Behavior
Launching lib\main.dart on Android SDK built for x86 in debug mode...
Running Gradle task 'assembleDebug'...
Note: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_secure_storage-3.3.4\android\src\main\java\com\it_nomads\fluttersecurestorage\ciphers\RSACipher18Implementation.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\in_app_purchase-0.3.4+9\android\src\main\java\io\flutter\plugins\inapppurchase\MethodCallHandlerImpl.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\webview_flutter-1.0.0\android\src\main\java\io\flutter\plugins\webviewflutter\FlutterWebView.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':flutter_zoom_plugin:bundleDebugAar'.
> Direct local .aar file dependencies are not supported when building an AAR. The resulting AAR would be broken because the classes and Android resources from any local .aar file dependencies would not be packaged in the resulting AAR. Previous versions of the Android Gradle Plugin produce broken AARs in this case too (despite not throwing this error). The following direct local .aar file dependencies of the :flutter_zoom_plugin project caused this error: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\commonlib\commonlib.aar, D:\Multiverse_Projects\Flutter\flutter\.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\mobilertc\mobilertc.aar

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 2m 45s
Exception: Gradle task assembleDebug failed with exit code 1

Steps to Reproduce

add the dependency in pubspec (0.0.8)
flutter run

Any updates?

@praharshbhatt I am also facing same issue ? Did you fixed the issue ? If yes, please let me know how did you fixed.
Execution failed for task ':flutter_zoom_plugin:bundleDebugAar'.

Direct local .aar file dependencies are not supported when building an AAR. The resulting AAR would be broken because the classes and Android resources from any local .aar file dependencies would not be packaged in the resulting AAR. Previous versions of the Android Gradle Plugin produce broken AARs in this case too (despite not throwing this error). The following direct local .aar file dependencies of the :flutter_zoom_plugin project caused this error: C:\Softwares\flutter.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\commonlib\commonlib.aar, C:\Softwares\flutter.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\mobilertc\mobilertc.aar

@Mallikarjun7657 Can you retry with the following configuration:
flutter_zoom_plugin:
git:
url: git://github.com/decodedhealth/flutter_zoom_plugin.git
# Android
ref: 0.0.8
  # iOS
  ref: master

flutter clean
flutter pub cache repair
flutter run

Your Environment

Plugin version: 0.08
Platform: Android
OS version: Microsoft Windows [Version 10.0.19041.508]
Flutter info (flutter doctor):
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 1.22.0, on Microsoft Windows [Version 10.0.19041.508], locale en-US)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
[√] Android Studio (version 4.0)
[√] Connected device (1 available)
• No issues found!

Plugin config:

  flutter_zoom_plugin:
    git:
      url: git://github.com/decodedhealth/flutter_zoom_plugin.git
      # Android
      ref: 0.0.8
      # iOS
      #ref: master

Expected Behavior
Application runs
Actual Behavior
Launching lib\main.dart on Android SDK built for x86 in debug mode...
Running Gradle task 'assembleDebug'...
Note: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_secure_storage-3.3.4\android\src\main\java\com\it_nomads\fluttersecurestorage\ciphers\RSACipher18Implementation.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\in_app_purchase-0.3.4+9\android\src\main\java\io\flutter\plugins\inapppurchase\MethodCallHandlerImpl.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\webview_flutter-1.0.0\android\src\main\java\io\flutter\plugins\webviewflutter\FlutterWebView.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':flutter_zoom_plugin:bundleDebugAar'.
> Direct local .aar file dependencies are not supported when building an AAR. The resulting AAR would be broken because the classes and Android resources from any local .aar file dependencies would not be packaged in the resulting AAR. Previous versions of the Android Gradle Plugin produce broken AARs in this case too (despite not throwing this error). The following direct local .aar file dependencies of the :flutter_zoom_plugin project caused this error: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\commonlib\commonlib.aar, D:\Multiverse_Projects\Flutter\flutter\.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\mobilertc\mobilertc.aar

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 2m 45s
Exception: Gradle task assembleDebug failed with exit code 1

Steps to Reproduce

add the dependency in pubspec (0.0.8)
flutter run

I am facing the same issue,
even tried importing the AAR module and then in gradle file. still no luck !

Your Environment

Plugin version: 0.08
Platform: Android
OS version: Microsoft Windows [Version 10.0.19041.508]
Flutter info (flutter doctor):
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 1.22.0, on Microsoft Windows [Version 10.0.19041.508], locale en-US)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
[√] Android Studio (version 4.0)
[√] Connected device (1 available)
• No issues found!

Plugin config:

  flutter_zoom_plugin:
    git:
      url: git://github.com/decodedhealth/flutter_zoom_plugin.git
      # Android
      ref: 0.0.8
      # iOS
      #ref: master

Expected Behavior
Application runs
Actual Behavior
Launching lib\main.dart on Android SDK built for x86 in debug mode...
Running Gradle task 'assembleDebug'...
Note: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\flutter_secure_storage-3.3.4\android\src\main\java\com\it_nomads\fluttersecurestorage\ciphers\RSACipher18Implementation.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\in_app_purchase-0.3.4+9\android\src\main\java\io\flutter\plugins\inapppurchase\MethodCallHandlerImpl.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\webview_flutter-1.0.0\android\src\main\java\io\flutter\plugins\webviewflutter\FlutterWebView.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':flutter_zoom_plugin:bundleDebugAar'.
> Direct local .aar file dependencies are not supported when building an AAR. The resulting AAR would be broken because the classes and Android resources from any local .aar file dependencies would not be packaged in the resulting AAR. Previous versions of the Android Gradle Plugin produce broken AARs in this case too (despite not throwing this error). The following direct local .aar file dependencies of the :flutter_zoom_plugin project caused this error: D:\Multiverse_Projects\Flutter\flutter\.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\commonlib\commonlib.aar, D:\Multiverse_Projects\Flutter\flutter\.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\mobilertc\mobilertc.aar

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 2m 45s
Exception: Gradle task assembleDebug failed with exit code 1

Steps to Reproduce

add the dependency in pubspec (0.0.8)
flutter run

I am facing the same issue,
even tried importing the AAR module and then in gradle file. still no luck !

Found a solution & it worked with Debug-RELEASE  APK/Bundle
if you have a flutter project demo
then in android module of demo
--> Open build.gradle (app level) insert following code
   **Sample**
  implementation fileTree(include: '*.aar', dir: '<path to zoom-plugin-libs>') //path relative to  folder(aar files are stored) present in zoom-plugin dir

    **Example**
   implementation fileTree(include: '*.aar', dir: '../../flutter_zoom_plugin/android/libs/commonlib/')
   implementation fileTree(include: '*.aar', dir: '../../flutter_zoom_plugin/android/libs/mobilertc/')

--> Open gradle.properties insert following code
   android.enableDexingArtifactTransform=false

One last change is a suggestion to flutter-zoom-plugin, We changed a line in build.gradle  of android module of this plugin :
   compileOnly files('libs/commonlib.aar') //path relative to libs folder present in zoom-plugin dir
   compileOnly files('libs/mobilertc.aar')    //path relative to libs folder present in zoom-plugin dir

Don't forget to add the proguard-rules for zoom libs/classes (if you skip it, it may not work in release APK/Bundle)
Preview for file proguard-rules.pro
  -keep class us.zoom.** { *; }
  -keep class us.zipow.** { *; }
  -keep class com.zipow.** { *; }
  -keep class org.webrtc.** { *; }
  -dontwarn us.zoom.**
  -dontwarn com.zipow.**

Found a solution & it worked with Debug-RELEASE  APK/Bundle
if you have a flutter project demo
then in android module of demo
--> Open build.gradle (app level) insert following code
   **Sample**
  implementation fileTree(include: '*.aar', dir: '<path to zoom-plugin-libs>') //path relative to  folder(aar files are stored) present in zoom-plugin dir

    **Example**
   implementation fileTree(include: '*.aar', dir: '../../flutter_zoom_plugin/android/libs/commonlib/')
   implementation fileTree(include: '*.aar', dir: '../../flutter_zoom_plugin/android/libs/mobilertc/')

--> Open gradle.properties insert following code
   android.enableDexingArtifactTransform=false

One last change is a suggestion to flutter-zoom-plugin, We changed a line in build.gradle  of android module of this plugin :
   compileOnly files('libs/commonlib.aar') //path relative to libs folder present in zoom-plugin dir
   compileOnly files('libs/mobilertc.aar')    //path relative to libs folder present in zoom-plugin dir

Don't forget to add the proguard-rules for zoom libs/classes (if you skip it, it may not work in release APK/Bundle)
Preview for file proguard-rules.pro
  -keep class us.zoom.** { *; }
  -keep class us.zipow.** { *; }
  -keep class com.zipow.** { *; }
  -keep class org.webrtc.** { *; }
  -dontwarn us.zoom.**
  -dontwarn com.zipow.**

Found a solution & it worked with Debug-RELEASE APK/Bundle
if you have a flutter project demo
then in android module of demo
--> Open build.gradle (app level) insert following code
   **Sample**
  implementation fileTree(include: '*.aar', dir: '<path to zoom-plugin-libs>') //path relative to  folder(aar files are stored) present in zoom-plugin dir

    **Example**
   implementation fileTree(include: '*.aar', dir: '../../flutter_zoom_plugin/android/libs/commonlib/')
   implementation fileTree(include: '*.aar', dir: '../../flutter_zoom_plugin/android/libs/mobilertc/')

--> Open gradle.properties insert following code
   android.enableDexingArtifactTransform=false

One last change is a suggestion to flutter-zoom-plugin, We changed a line in build.gradle of android module of this plugin :
   compileOnly files('libs/commonlib.aar') //path relative to libs folder present in zoom-plugin dir
   compileOnly files('libs/mobilertc.aar')    //path relative to libs folder present in zoom-plugin dir

Don't forget to add the proguard-rules for zoom libs/classes (if you skip it, it may not work in release APK/Bundle)
Preview for file proguard-rules.pro
  -keep class us.zoom.** { *; }
  -keep class us.zipow.** { *; }
  -keep class com.zipow.** { *; }
  -keep class org.webrtc.** { *; }
  -dontwarn us.zoom.**
  -dontwarn com.zipow.**

@amaurya08 Where i need to add these lines?
compileOnly files('libs/commonlib.aar') //path relative to libs folder present in zoom-plugin dir
compileOnly files('libs/mobilertc.aar')    //path relative to libs folder present in zoom-plugin dir

Found a solution & it worked with Debug-RELEASE APK/Bundle
if you have a flutter project demo
then in android module of demo
--> Open build.gradle (app level) insert following code
   **Sample**
  implementation fileTree(include: '*.aar', dir: '<path to zoom-plugin-libs>') //path relative to  folder(aar files are stored) present in zoom-plugin dir

    **Example**
   implementation fileTree(include: '*.aar', dir: '../../flutter_zoom_plugin/android/libs/commonlib/')
   implementation fileTree(include: '*.aar', dir: '../../flutter_zoom_plugin/android/libs/mobilertc/')

--> Open gradle.properties insert following code
   android.enableDexingArtifactTransform=false

One last change is a suggestion to flutter-zoom-plugin, We changed a line in build.gradle of android module of this plugin :
   compileOnly files('libs/commonlib.aar') //path relative to libs folder present in zoom-plugin dir
   compileOnly files('libs/mobilertc.aar')    //path relative to libs folder present in zoom-plugin dir

Don't forget to add the proguard-rules for zoom libs/classes (if you skip it, it may not work in release APK/Bundle)
Preview for file proguard-rules.pro
  -keep class us.zoom.** { *; }
  -keep class us.zipow.** { *; }
  -keep class com.zipow.** { *; }
  -keep class org.webrtc.** { *; }
  -dontwarn us.zoom.**
  -dontwarn com.zipow.**

@amaurya08 Where i need to add these lines?
compileOnly files('libs/commonlib.aar') //path relative to libs folder present in zoom-plugin dir
compileOnly files('libs/mobilertc.aar')    //path relative to libs folder present in zoom-plugin dir

@amaurya08 Where i need to add these lines?
compileOnly files('libs/commonlib.aar') //path relative to libs folder present in zoom-plugin dir
compileOnly files('libs/mobilertc.aar') //path relative to libs folder present in zoom-plugin dir

Just for the sake of make my project working i added the lib into proj root folder. Here is the screenshot for the same and for the libs folder

@amaurya08 Where i need to add these lines?
compileOnly files('libs/commonlib.aar') //path relative to libs folder present in zoom-plugin dir
compileOnly files('libs/mobilertc.aar') //path relative to libs folder present in zoom-plugin dir

Just for the sake of make my project working i added the lib into proj root folder. Here is the screenshot for the same and for the libs folder

I'm facing a similar issue, but Google states:

Note: You can't use the compileOnly configuration with AAR dependencies.

Source: https://developer.android.com/studio/build/dependencies

I'm facing a similar issue, but Google states:

Note: You can't use the compileOnly configuration with AAR dependencies.

Source: https://developer.android.com/studio/build/dependencies

add compileOnly in plugin's android module project level (build.gradle) file. I am using follwoing gradle configuration
`group 'com.decodedhealth.flutter_zoom_plugin'
version '1.0-SNAPSHOT'
buildscript {
repositories {
google()
jcenter()
}
dependencies {
    classpath 'com.android.tools.build:gradle:3.4.0'
}

}
rootProject.allprojects {
repositories {
google()
jcenter()
}
}
apply plugin: 'com.android.library'
android {
compileSdkVersion 29
defaultConfig {
    minSdkVersion 21
    targetSdkVersion 29
    testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
    vectorDrawables.useSupportLibrary = true
}
lintOptions {
    disable 'InvalidPackage'
}

}
dependencies {
implementation 'androidx.appcompat:appcompat:1.0.2'
implementation 'androidx.constraintlayout:constraintlayout:1.1.3'
implementation 'com.google.android.material:material:1.0.0'
compileOnly files('libs/commonlib.aar')
compileOnly files('libs/mobilertc.aar')
}
`
Here is my flutter application's project level gradle
buildscript {
ext.kotlin_version = '1.3.50'
repositories {
google()
jcenter()
}
dependencies {
    classpath 'com.android.tools.build:gradle:3.4.3'
    classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
    classpath 'com.google.gms:google-services:4.3.3'
}

}
allprojects {
repositories {
google()
jcenter()
}
}
rootProject.buildDir = '../build'
subprojects {
project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
project.evaluationDependsOn(':app')
}
task clean(type: Delete) {
delete rootProject.buildDir
}
App level gradle
flutter {
source '../..'
}
dependencies {
implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
implementation 'com.android.support:multidex:1.0.3'
implementation fileTree(include: '*.aar', dir: '../../flutter_zoom_plugin-master/android/libs/')
}

It is not working. I'm having the same issue.
Execution failed for task ':flutter_zoom_plugin:bundleDebugAar'.

Direct local .aar file dependencies are not supported when building an AAR. The resulting AAR would be broken because the classes and Android resources from any local .aar file dependencies would not be packaged in the resulting AAR. Previous versions of the Android Gradle Plugin produce broken AARs in this case too (despite not throwing this error). The following direct local .aar file dependencies of the :flutter_zoom_plugin project caused this error: D:\Chandan\src\flutter_windows_2.0.1-stable\flutter.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\commonlib\commonlib.aar, D:\Chandan\src\flutter_windows_2.0.1-stable\flutter.pub-cache\git\flutter_zoom_plugin-88d1fa2685f28d67410d46e094461a0a030d3192\android\libs\mobilertc\mobilertc.aar

any progress or solutions to this "direct local .aar file" problem?

See solution posted to https://github.com/flutter/flutter/issues/28195

Execution failed for task ':flutter_zoom_plugin:bundleReleaseAar'.

Direct local .aar file dependencies are not supported when building an AAR. The resulting AAR would be broken because the classes and Android resources from any local .aar file dependencies would not be packaged in the resulting AAR. Previous versions of the Android Gradle Plugin produce broken AARs in this case too (despite not throwing this error). The following direct local .aar file dependencies of the :flutter_zoom_plugin project caused this error: /home/jarves/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_zoom_plugin-0.0.7/android/libs/commonlib/commonlib.aar, /home/jarves/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_zoom_plugin-0.0.7/android/libs/mobilertc/mobilertc.aar

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 2m 29s


Getting errors one field at a time, instead of a full array of them

Hi, I'm wondering if I'm doing something wrong here, or if Indicative is supposed to send errors one field at a time. My code is:
var schema = {
            title: 'required|min:10|max:100',
            description: 'required|min:30|max:300',
            seller: {
                name: 'required|min:3|max:30',
                email: 'required|email',
            }
 }
var messages = {
            required: 'Por favor ingresá {{ field }}',
            email: 'Por favor ingresá un email válido',
            number: 'Por favor ingresá sólo números en {{ field }}'
}

 Indicative.validate(schema, item, errorMessages).then(function (validation_passed) {
        itemModel.insert(item, function() {
            res.json({
                result: item
            });
        });
    }).catch(function (err) {
        console.log(err);
        res.json({
            error: err
     });
 });

So if the form is empty on Submit, I'm getting the error message just for title, and then when I complete that field and re-submit, then I get an error for the empty description, and so on. Is there any way to get them all at once?
Thank you!
PS: Sorry if my code is ugly, I'm a very newbie programmer :)

Use validateAll method instead of validate

Also as a quick tip you can use dot notation inside your scehma.
var schema = {
            title: 'required|min:10|max:100',
            description: 'required|min:30|max:300',
             'seller.name': 'required|min:3|max:30',
             'seller.email': 'required|email',
 }

This will remove unnecessary curly braces

Thank you!!
One more thing, is it possible to refer to the min and max values on custom error messages as with {{ field }} ?
Thanks again!

Yes checkout http://indicative.adonisjs.com/docs/node-validation-messages#templating

Sorry, forgot to check if that was already in the docs. Thanks so much!
On Aug 21, 2015 12:09 AM, "Harminder Virk"<EMAIL_ADDRESS>wrote:

Yes checkout
http://indicative.adonisjs.com/docs/node-validation-messages#templating
—
Reply to this email directly or view it on GitHub
https://github.com/Adonis-Js/indicative/issues/13#issuecomment-133262213
.


Using ShorterMaps seems to mess with line numbers on errors

When I use the ~M sigil, I find that the line numbers I see on errors thrown from Elixir are way off (eg. line 83 instead of line 3).
Is there a way to solve this issue? I'm on Elixir 1.5.1 and Erlang/OTP 20

Can you give me a bit more to help reproduce the error?
Minimally, copy and paste over your stacktrace, and if you can provide a gist or a project for me to clone that demonstrates the issue succinctly, that would be really helpful.

Created a minimal repo for reproduction here: https://github.com/hanrelan/shorter_maps_repro
When I clone this repo and run mix I get the following:
warning: variable "something" is unused
  lib/shorter_maps_repro.ex:82

Ok, I've found the problem and pushed a fix in v2.2.2 to hex.  Give it a whirl and make sure that it fixes the problem in your production code.
Thanks for taking the time to file an issue!

Woot thanks! Will reopen this issue if I see it again


Migrate doc from Google Docs to Markdown

As stated in the prompt.
I also updated the design of the documentation so that we will be using a modal design.

@tangyuantyy do you mind having a look at this and merge this PR?

I think we may need to fix the space issue in Markdown styling.


Toolchain config alternative to --per_file_copt

Description of the problem / feature request:
Provide a way to express --per_file_copt in the toolchain configuration, as an alternative to putting it in .bazelrc.
Feature requests: what underlying problem are you trying to solve with this feature?
--per_file_copt can be used to control warnings from third-party dependencies. For example, --per_file_copt=external/.*@-w will silence all such warnings (which is often appropriate, since projects often don't have control over their dependencies' third-party code).
Have you found anything relevant by searching the web?
Yes. This feature was originally requested here: https://groups.google.com/g/bazel-discuss/c/0zk1csyp4uU
It came up in https://bazelbuild.slack.com/archives/CGA9QFQ8H/p1698699534921939, too.
Any other information, logs, or outputs that you want to share?
This seems like an important piece of UX for configuring toolchains with strict warnings.

I bumped into this again. It really is a recurring problem.
Fundamentally, in a multirepo world where you don't own all the code you're pulling into your build, you want to be able to configure warnings on a per-repo basis. It's weird that this ends up in .bazelrc instead of in the toolchain configuration, as this makes the two hard to keep in sync. In practice you're reduced to disabling all warnings for external code, which is a hammer I'd rather not use.

Silencing external warnings through --per-file-copt in .bazelrc is particularly inconvenient in projects that build the same targets with more than one toolchain (e.g., arm and clang). We apply different warnings for different toolchains, and so need different warning silencing instructions.

@fmeum FYI in case you have comments on this (beyond the high-level suggestion in the Bazel Slack that maybe this should be handled by an extension of https://github.com/bazelbuild/bazel/issues/12009 instead; I'm not quite sure how such an extension would work?).

Hey y'all,
I think I'm fairly close to a solution proposal for this, but I wanted to make sure that what I have in mind actually meets the needs of the users.
In particular, it seems fairly straightforward to add the possibility to expand on regex matches. Example code:
expand_if_regex_matches = variable_with_value(
    name = "source_file",
    value = "external.*",
)

In my intended scenario, the above code would be essentially equivalent to the example code in the first message of the thread in the first comment on this issue. In addition to being much easier to implement, this seems like a much more generic solution that could satisfy use cases (ie, checking variables other than source_file).
Do y'all have any use cases that this would not satisfy, or any other reason you feel this solution is not ideal? I could probably implement what is described in that thread, but it would take much longer and, as far as I can tell, it would only satisfy a small subset of the use cases the regex solution does. Also open to suggestions for other solutions.
Thanks!

@tpudlik Can you give me more detail about your last message?
I took a look at the issue you linked, and I'm not sure how the functionality described there doesn't meet the needs described in this one? I see that users were posting saying that it didn't work for them, but I don't see the bug being reopened or anything like that. I do see how this solution is more generic, but I don't understand how the initial intention is different.
Also I really think someone should comment on that issue and help those users out who are trying to use the feature but can't.

IIUC https://github.com/bazelbuild/bazel/issues/12009 specifically handles the case of warnings coming from included headers from external repositories. But it won't help you if your project has deps on (and attempts to build) cc_library targets defined in external repositories.

Got it, thanks for the explanation.

Back to the regex solution, it was brought to my attention that regex can be a performance concern. Are we worried about that in this case? If so, are there any ways that we can mitigate those concerns?

In my opinion, this feature would be a layering violation: Toolchains should be reusable across projects and should thus not encode assumptions about the source file path layout of a given project. Furthermore, evaluating regexes for every compilation action could at least in theory become a performance concern.
Instead, how about the following:

Add a disable_all_warnings or external_project feature to your toolchain that disables all warnings in a way appropriate for your toolchain's compiler. This is of course possible today.
Request that feature for all external repositories. Today, this requires patching a REPO.bazel with repo(features = ["disable_all_warnings"]) into each repo, but this could be improved. For example, we could add an extension to rules_cc that can be used to configure features for all repos created by given modules from the MODULE.bazel file. This allows for far more expressive, regular starlark syntax instead of flags, e.g.:

cc.feature_config(feature = "disable_all_warnings", all_modules = True)
cc.feature_config(feature = "-disable_all_warnings", modules = ["my_root_module"])

The particular syntax is completely made up and not well thought out, but I'm sure we can get something more ergonomic and flexible this way compared to a flag or toolchain config.
Cc @Wyverald

Thank you for the feedback! But would this proposed solution only work if all your (transitive) dependencies have migrated to bzlmod? That's not practical, since so much of the ecosystem is still on WORKSPACE.

We could build arbitrary logic into rules_cc to handle WORKSPACE repos, so yes, we could cover this.
Maybe @meteorcloudy also has some more insights into this interaction of C++ feature config and external deps.

@fmeum @Wyverald I'm also not sure how do we inject REPO.bazel file based on information collected by cc.feature_config. I guess we'll have to change Bazel itself?

@meteorcloudy That sounds tricky indeed. But since features are essentially a C++/Objc-only mechanism, we could just have the relevant rules read the result of the module extension and add the features in their rule impls.

So, just had a chat with @tpudlik, @comius, @Wyverald, @fmeum, and a few others...
We came to the conclusion that what we're after here can be achieved today by doing the following:
You can essentially add a feature disable_warnings that is enabled by default, then add the following to your REPO.bazel
repo(features = ["-disable_warnings"]

We discussed that this may not meet higher granularity needs, we determined that, since the needs motivating this specific bug are met, we're going to close it for now.

@t-rad679 One downside of the REPO.bazel approach is that it disables warnings for downstream users as well as for third-party dependencies. Downstream users would also have to copy the REPO.bazel in order to enable warnings.

Another case where REPO.bazel is unsuitable is if you're pulling in the repository as a submodule and adding it as a local_repository in order to make iterating on it for developers easier. In that case if you don't have a custom fork + branch that you're rebasing as you update, you can't change the REPO.bazel there


Codeception yii console application init added.

Not sure if this should work this way or not, but without it codeception throwed me "class not found" exceptions, when I tried to use classes from my components folder $helper = new \app\components\XmlHelper(); something like that in my test function.

In which type/suite did you get errors (unit, functional, acceptance)?

Unit, one test file with one test method.

@akaSStalkALEX Could you double check your unit tests.
I've just had a similar problem, but extending from yii\codeception\TestCase with the correct $appConfig setting should take care about application initialization.
I've added some tests for login/logout with the user module, which requires correct initialization of several application components. See also https://github.com/phundament/app/blob/master/tests/codeception/unit/prod/models/UserTest.php
I've also tried your PR, which also makes the tests pass, because an application gets initialized.
But it would be cleaner, if the TestCase would do the job.
CC: @Quexer69 Unit test setup.

And they pass when you've done the correct user setup, see https://travis-ci.org/phundament/app/jobs/113484331#L1181

I changed my test as you proposed. It works. But thing is test that I used was generated by console command codecept generate:test unit Example.  It looks like this
class ExampleTest extends \Codeception\TestCase\Test
{
    /**
     * @var \UnitTester
     */
    protected $tester;

    protected function _before()
    {
    }

    protected function _after()
    {
    }

    // tests
    public function testMe()
    {

    }
}

Codecept creates this file in unit folder and it doesn't include $appConfig by default.

@akaSStalkALEX About the generated tests ... can you specify a template for the tests in Codeception?
Alternatively we could provide a custom gii command.
A phpdoc variable defnition like
<?php
/* @var $this Codeception\ClassXYZ */
?>

should fix the code-completion.

By the way: Which codecept version are you using? There were some changes from 2.0 to 2.1.
Do you use Phundament with Docker or directly on your machine?

Unfortunately we're unsing Phundament app without Docker due to our local architecture.
Codeception's verison is 2.1.5 installed through composer.


Extra condition owner assignment

Hi,
I added similar condition owner assignment to CueSetConditionChecker, AnswerSetShowConditionChecker, and AnswerSetSelectConditionChecker.

I've actually already implemented this fix. Thank you though!

Sorry, I misread you! I'm Sorry, I misread you! I'm not sure if owner is needed for conditioncheckers or just conditions, but I suppose it can't hurt.not sure if owner is needed for conditioncheckers or just conditions, but I suppose it can't hurt.

No worries. I did some tests with CompanionInParty with CueSetConditionChecker and it had the same problem.


VSS vector search extension and bundling option

I really love the bundling idea .
VSS https://github.com/asg017/sqlite-vss couild be also a good idea for ai powered app vector search is a critical feature

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

💰 requested or PRs are welcome

Duplicate of #79


Support for filter

Added support for filters, e.g. to hide Ens packages use 'Ens*

Examples are 'Ens*,'HS*
* means 0 or more characters
_ means any one character
' at the beginning means NOT pattern

I'm hoping the CI changes will fix the CI failure in my #54 PR once merged. Ideally they would have been on their own PR, but I can live with them being part of this one.


fix(cactus-example-discounted-asset-trade): enable tsconfig strict flag and fix all the warnings

Fixed all warnings when strict flag is enabled

Closes: #2145
Signed-off-by: Tomasz Awramski<EMAIL_ADDRESS>

@izuru please review

@rwAT17 Thanks for contribution and I'm very sorry to wait you for a while! I'm asking @izuru0 to review this PR again. He will review it again next Monday, so please wait for a minute?


XMLRFC v3: Test and improve Asciidoc v3 converter against a wide range of Asciidoc features

Addressed through unit testing.


Update SnomedConcept.java

Removed redunant parenthesis at lines 149,169

Hey @eranhash,
Thank you for your contribution.
Are you going to add the fix to the third minor code smell here? Or in a new PR?
The next release is about to be released today or next Monday, so these fixes can be included as well if needed. :)
Cheers,
Mark

Hi @cmark
removing redundant parenthesis  in two lines counts as 2 and moving trailing comment is the third, so im done.
sorry that wasn't clear.
thank you for allowing me to make my first steps in the open source community in your project.
Cheers
Eran

Hi @eranhash,
Cool, we will merge this PR then.
Thank you and have a nice day,
Mark


a true issue

Challenge create-a-form-element has an issue.
User Agent is: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.81 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

CatPhotoApp

Click here to view more cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

  <form <input type="text" action="/submit-cat-photo"> </form>
.

@stewartburns
Thanks for reporting this. But your code is incorrect.
GitHub Issues are for reporting bugs on the website only.
Please use the Help chat room for getting challenge related help.
Happy Coding.

I believe it is buggy.
Here is my code:

and it is not letting me pass this lesson.
I am assuming this is still an issue?

''

''


bash scripts/eval.sh not runs

Hello,  I'm trying to run this.
In file eval.sh

config_path=/$PROJECT_ROOT_PATH/config/linemod/"$seq"_fw0.5.yml
It seems to need cat_fw0.5.yml file in that directory. But i can't find "cat_fw0.5.yml" in the directory
Is the problem caused by skipping the "4. Data Preparation" section?
someone can tell me what the problem is!
thanks!

Hi, you could refer to this link for solutions.
https://github.com/DecaYale/RNNPose/issues/5
I will modify the readme file later.


Support for Azure CosmosDB for PostgreSQL

Hi Team,
In this document, Azure CosmosDB for PostgreSQL is not listed as data source but is it currently supported?
I wonder this point because ADF integration with Azure CosmosDB for PostgreSQL is mentioned in the below doc but in the data factory side doc, it's not clearly mentioned.
https://learn.microsoft.com/en-us/azure/cosmos-db/postgresql/howto-ingest-azure-data-factory#steps-to-use-data-factory
Thank you very much!

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

ID: ae2389f5-6e57-bd3e-2228-83479983ca0b
Version Independent ID: af4d7832-5f25-59cc-e9fc-74f32ec8dbc0
Content: Connector overview - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/connector-overview.md
Service: data-factory
Sub-service: data-movement
GitHub Login: @jianleishen
Microsoft Alias: jianleishen

Hi @jianleishen
Could you please review add comments on this, update as appropriate.

@yutaro113
I've delegated this to content author @jianleishen, who will review it and offer their insightful opinions.


WEB-612863: Update typography under branding guidelines to include DIN Alternate Regular

Fixes: https://helpdesk.vatsim.uk/scp/tickets.php?id=30777

Approved by Marketing - Ready to be approved and merged.

:tada: This PR is included in version 4.14.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Unable to load /bundle.js from initrd.

This is what I got:
Unable to load /bundle.js from initrd.
Kernel error: abort() requested

Did you remember to require('runtimejs')?

@SpaceboyRoss01 Seeing as how you're asking about other things that are OS functions and this is a startup problem, is it safe to assume this is resolved and can be closed?


updated to so there is no extra comma on top selects

Nice find. We'll run the MS2 tests and make sure nothing is broken.


Wrong font color

When I set the style, the letters are not visible, they are black, should be white.

Please submit a patch, because I stopped using Zulip.


Deteting the first list causes a persistent error banner in the WebUI

The only interaction available with a list, aside from adding people to it is deleting it. Deleting the first list, created automatically when registering an account, creates a persistent error banner with the text:
id 'https://$DOMAIN/api/collection/$LIST_ID' not found in stream 'user:$USERNAME:lists:person'
Whenever lists are referenced in any way by the WebUI.

This seems to be a bug in the stream API.


added into

I just added .try_into().unwrap()  to allow the casting

Seems like there are no changes here?

Thanks, the changes do seem to be applied in master already


Better introduction in readme

I would really appreciate a nice short and to the point introduction on what aesop is, what can it do, why and when I might want to use it.
I had to chew through setting up custom list and some simp lemmas before to even get to the first feature of aesop.
Also the name "aesop" which to me is just a random collection of letters does not help. An explanation of that name would help.
Improving readme would be highly appreciated! I have a feeling aesop is something I might be interested in. Right now I'm just not motivated enough to keep reading through the rather technical readme.

Would like to echo this. To be more specific, the question I had after looking at the README was essentially "in terms of use cases and capabilities, where does this fit on the spectrum of aggressive use of simp (least work) and writing a custom tactic to solve something (most work)".
Also this looks really cool, thanks for sharing it with the community.

Apologies for the very long delay. I've now added an intro which describes Aesop's major features and use cases. Does that help? If not, please feel free to reopen this issue.

Much better now, thank you!


Script to convert text to image

A python script to convert the given text to image.
The text may contain special characters such as emojis as well

hey @DhanushNehru , I would like to work on this issue !!!

is someone still working on this issue and if not can you assign this to me?

@kishanshinde please confirm
FYI @nishantattrey07

@DhanushNehru does someone already did it?
If no one, I would be happy to work on this.


Review cardano-node documentation with regard to p2p

We need to make sure that the documentation is up-to date and includes / explains all the p2p options.

[ ] https://github.com/input-output-hk/cardano-node/blob/p2p-master/doc/getting-started/understanding-config-files.md


atk does not build

install-x64-windows-rel-out.log
install-x64-windows-static-rel-out.log
config-x64-windows-static-dbg-out.log
config-x64-windows-static-rel-out.log

I'm not able to reproduce this issue. Could you confirm you're using the latest vcpkg and all packages are up to date?
>vcpkg version
>vcpkg update

Building package atkmm[core]:x86-windows...
-- Using cached D:/work/vcpkg/downloads/atkmm-2.24.2.tar.xz
-- Applying patch D:/work/vcpkg/ports/atkmm/fix_properties.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Applying patch D:/work/vcpkg/ports/atkmm/fix_charset.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Building D:/work/vcpkg/buildtrees/atkmm/src/atkmm-2.24.2/MSVC_Net2013/atkmm.sln for Release
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:56 (message):
Command failed: msbuild;D:/work/vcpkg/buildtrees/atkmm/src/atkmm-2.24.2/MSVC_Net2013/atkmm.sln;/p:Configuration=Release;/t:atkmm;/p:Platform=Win32;/p:PlatformToolset=v141;/p:VCPkgLocalAppDataDisabled=true;/p:UseIntelMKL=No;/p:WindowsTargetPlatformVersion=10.0.17134.0;/m;/p:ForceImportBeforeCppTargets=D:/work/vcpkg/scripts/buildsystems/msbuild/vcpkg.targets;/p:VcpkgTriplet=x86-windows
Working Directory: D:/work/vcpkg/buildtrees/atkmm/x86-windows-rel
See logs for more information:
D:\work\vcpkg\buildtrees\atkmm\build-x86-windows-rel-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_build_msbuild.cmake:119 (vcpkg_execute_required_process)
ports/atkmm/portfile.cmake:27 (vcpkg_build_msbuild)
scripts/ports.cmake:71 (include)
Error: Building package atkmm:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: atkmm:x86-windows
Vcpkg version: 0.0.113-nohash
Uploading patch-x86-windows-0-err.log…
Uploading patch-x86-windows-1-err.log…
build-x86-windows-rel-out.log

@barcharcraz & @DengHaijian
I can install this port successfully with the latest VCPKG and VS2017 15.9.7.


Remove rpcq cloning from the build once it is in QuickLisp

@karalekas @rigettizach This seems to be fixed, yes?

Yes -- we do something a bit different now, we build quilc directly from the latest stable version of the rpcq image. Closing.


Where is the DirectoryDialog

I use compose develop a build tools ,need to select a Directory,so where is the DirectoryDialog,how can i use it.

See https://github.com/JetBrains/compose-jb/blob/a7b837257940492e045fe8b81b82cb96b82698a1/examples/notepad/src/main/kotlin/util/Dialogs.kt#L17 for example.

@zhanweisun
do you have any other questions here?

Thanks!no questions!

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


quests?

Perhaps the game could use some quests that would give you subgoals and rewards (rox/xp)?

Now that I think of it, the game has a few "puzzles" that act as quests, so for this game I think I'll be leaving any official system out of it.


Add the missing logic to the Processing Lambda Skeleton Code

Description:
Fill out the blanks and complete the logic for the Processing Lambda Function.
Repo to function: https://github.com/HERMES-SOC/swsoc-processing-lambda
Deliverable:
Completed Processing Lambda Function

Is this the right place for this issue?


Remove nth node from end of list solution added here. Thank you.

Pull Request Template
What have you Changed(must)
Remove nth node from end of list solution added here.
Write here what you have changed in cosebase
Issue no.(must) - #275
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me if not it will marked as spam.
[x] My file is in proper folder
[x] I am following clean code and Documentation.
[x] I have added title and what program will do first in the file
[x] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

You have to add the Issue Number of the issue that you were assigned too. We're closing this PR.


What are the plans on WebApiCompatShim?

I am about to start using WebApiCompatShim on a project but I wanted to make sure what the plans are for this. Is it the plan to continue supporting this and ship as part of stable 1.0.0 release?

Yes! It's a supported feature in ASP.NET 5. Having said that, we strongly recommend porting Web API apps to ASP.NET MVC 6 because that's where the future innovation will be.


Somalian Civil War

I do think this civil war should be expanded upon as it lasted for over twenty years  and it is still ongoing.However a lot of things have happened in Somalia since then.I would like to get  permission to create a commit depicting this event as the current event for the Somalian Civil War only increases militancy and the conscience of it's population which doesen't represent the way how Somalia almost fell into complete anarchy in the early 1990s divided between 10 factions.
Edit
Anyways about the Somali Civil War I have completed the first stage of it which will the Somali rebellion against Siad Barre in the 1980s,I haven't worked on it for a couple of days.

Please do :+1:

What is the status of this issue?

I apologize for the exceptionally long hiatus on this issue, I had managed to create an entire event chain for the Somali rebellion and early 1990s part of the civil war, and also created several countries like Galmudug to serve as autonomous regions since Somalia adopted a federal Constitution in 2012 giving its states a very large amount of autonomy. The Ogaden War will be the next part I will be working on however I didn't managed to start it yet.

While I've previously this discussed this at the forums, I wouldn't still rule out the possibility of adding in new releasable countries in Ethiopia since in the civil war there were many armed separatist groups such as the Afar Liberation Front and the Ogaden National Liberation Front present throughout Ethiopia. Even the Tigrayan Liberation Front considered separatism as a last resort. Also I would consider the option of  a breakup of Ethiopia in alternate civil war in which Ethiopia lost the Ogaden War to be quite possible as the Ethiopian military had suffered very heavy losses especially in equipment during the Ogaden War. Had the Ethiopians lost the Ogaden War, it would have been very likely that Ethiopia could have collapsed into anarchy similarly to Somalia however unlike Somalia, Ethiopia could collapse because of its diverse ethnic groups rather than the tribal society of Somalia.
Sources
https://www.google.ie/url?sa=t&source=web&rct=j&url=https://www.rand.org/content/dam/rand/pubs/reports/2006/R3347.pdf&ved=0ahUKEwip4unfu5jSAhXpL8AKHdNHAm8QFghHMAk&usg=AFQjCNGyNKNikDO1NR5aIqDxgXROQwJXJA&sig2=HewYksop14QJxcnkgR9-aQ
http://www.bbc.com/news/world-africa-16620783
http://www.cfr.org/ethiopia/ogaden-national-liberation-front-onlf/p13208

Sure, Oromia exists in the mod and one could add more.

Probably Al Shabab belongs as part of this issue. No need for separate issue #1064

For my original plan for the Somalian Civil War event chain, I was going to use tags I've created for the warlords in the early 90s phase of the civil war, however I've changed my mind on that instead using tags for the autonomous states I've created.

As a part of this event chain I'm planning on creating a modifier for Somali piracy during the civil war however I'm not sure what it effects will be. Do you have any suggestions on it.


Bugfix/attention mask and implementation

Issue: https://github.com/OpenGVLab/OmniQuant/issues/46
I had to change attn implementation initialization. Unexpectedly (at least for me :sweat_smile:), it turned out that it is not possible to specify attention in the model's config.json file. One can only set it as an argument when creating an object (attn_implementation is taken from "kwargs", not "config_dict", see: https://github.com/huggingface/transformers/blob/v4.36.1/src/transformers/configuration_utils.py#L772). So, I add attention to args and give it to config object when it is created.
I hope that this change is OK (new argument to parser + modified AutoConfig.from_pretrained call).

Good job! Thanks for your time again.


Fixes ordering key and improves rejection

Q
A

Bug fix?
yes

New feature?
yes

BC breaks?
yes

Deprecations?
no

Tickets
~

License
MIT

Currently, even if the code takes into account the "Ordering Key", it's not really sent to Google PUB / SUB because the setOrderingKey method in MessageBuilder class returns a new instance of Message and does not modify the current message.
Next, In case of rejection of a message, we want to notify Google PUB / SUB right away and not 10 seconds later, so I have filled the reject method.
So this PR:

Fixes the problems on "Ordering Key".
Adds the reject feature. There is therefore a BC break, we now wait less than 1 second (versus minimum 10 secodes before) for informing PUB / SUB that the message has failed. I was inspired by this documentation: https://googleapis.dev/python/pubsub/latest/subscriber/index.html
Adds the ".editorconfig" file for consistent coding styles between different editors
Adds the ".gitattributes" file to exclude some files and folders when compressing
Updates the ".gitignore" file. The ".idea" file is specific to PHPStorm, the exclusion must be in a gitignore specific to the environment and not to the project.
Adds PHP Coding Standards Fixer tool to follow standards. Command to execute: vendor/bin/php-cs-fixer fix
Adds PHPStan for focuses on finding errors in the code without actually running it. Command to execute: vendor/bin/phpstan analyse Tests Transport --level=5
Adds PHPUnit for unit testing. I respected the "boy scout rule" by adding some tests. Command to execute: vendor/bin/phpunit

Later:

It would be necessary to execute in a CI the 3 commands described above.
Increase the level of PHPStan.

Ping @marforon


Seemed Can't install

Python 3.11.4,
after poetry install,
• Installing tensorflow (2.17.0): Failed

  RuntimeError

  Unable to find installation candidates for tensorflow (2.17.0)

  at ~/.pyenv/versions/3.11.4/lib/python3.11/site-packages/poetry/installation/chooser.py:76 in choose_for
       72│
       73│             links.append(link)
       74│
       75│         if not links:
    →  76│             raise RuntimeError(f"Unable to find installation candidates for {package}")
       77│
       78│         # Get the best link
       79│         chosen = max(links, key=lambda link: self._sort_key(package, link))
       80│

  • Installing torch (2.5.0): Failed

  RuntimeError

  Unable to find installation candidates for torch (2.5.0)

  at ~/.pyenv/versions/3.11.4/lib/python3.11/site-packages/poetry/installation/chooser.py:76 in choose_for
       72│
       73│             links.append(link)
       74│
       75│         if not links:
    →  76│             raise RuntimeError(f"Unable to find installation candidates for {package}")
       77│
       78│         # Get the best link
       79│         chosen = max(links, key=lambda link: self._sort_key(package, link))
       80│

Hi, on my side, the commands work well on 2 of my PCs.
I will double-check soon.

aaaj) mingchen agent-as-a-judge % poetry install

Installing dependencies from lock file

Package operations: 130 installs, 1 update, 0 removals

  - Downgrading charset-normalizer (3.4.0 -> 3.3.2)
  - Installing mdurl (0.1.2)
  - Installing typing-extensions (4.12.2)
  - Installing annotated-types (0.7.0)
  - Installing attrs (24.2.0)
  - Installing catalogue (2.0.10)
  - Installing fsspec (2024.9.0)
  - Installing h11 (0.14.0)
  - Installing markdown-it-py (3.0.0)
  - Installing markupsafe (3.0.1)
  - Installing numpy (1.26.4)
  - Installing pydantic-core (2.23.4)
  - Installing pygments (2.18.0)
  - Installing pyyaml (6.0.2)
  - Installing rpds-py (0.20.0)
  - Installing sniffio (1.3.1)
  - Installing tqdm (4.66.5)
  - Installing absl-py (2.1.0)
  - Installing anyio (4.6.0)
  - Installing click (8.1.7)
  - Installing cymem (2.0.8)
  - Installing frozenlist (1.4.1)
  - Installing grpcio (1.66.2)
  - Installing h5py (3.12.1)
  - Installing httpcore (1.0.6)
  - Installing huggingface-hub (0.25.1)
  - Installing marisa-trie (1.2.0)
  - Installing markdown (3.7)
  - Installing ml-dtypes (0.4.1)
  - Installing mpmath (1.3.0)
  - Installing multidict (6.1.0)
  - Installing murmurhash (1.0.10)
  - Installing namex (0.0.8)
  - Installing optree (0.13.0)
  - Installing propcache (0.2.0)
  - Installing protobuf (4.25.5)
  - Installing pydantic (2.9.2)
  - Installing referencing (0.35.1)
  - Installing rich (13.9.2)
  - Installing six (1.16.0)
  - Installing srsly (2.4.8)
  - Installing tensorboard-data-server (0.7.2)
  - Installing werkzeug (3.0.4)
  - Installing wrapt (1.16.0)
  - Installing aiohappyeyeballs (2.4.3)
  - Installing aiosignal (1.3.1)
  - Installing astunparse (1.6.3)
  - Installing blis (0.7.11)
  - Installing cloudpathlib (0.19.0)
  - Installing confection (0.1.5)
  - Installing distro (1.9.0)
  - Installing flatbuffers (24.3.25)
  - Installing gast (0.6.0)
  - Installing google-pasta (0.2.0)
  - Installing httpx (0.27.2)
  - Installing iniconfig (2.0.0)
  - Installing jinja2 (3.1.4)
  - Installing jiter (0.6.1)
  - Installing joblib (1.4.2)
  - Installing jsonschema-specifications (2024.10.1)
  - Installing keras (3.6.0)
  - Installing language-data (1.2.0)
  - Installing libclang (18.1.1)
  - Installing networkx (3.3)
  - Installing opt-einsum (3.4.0)
  - Installing pluggy (1.5.0)
  - Installing preshed (3.0.9)
  - Installing regex (2024.9.11)
  - Installing safetensors (0.4.5)
  - Installing scipy (1.14.1)
  - Installing smart-open (7.0.5)
  - Installing sympy (1.13.3)
  - Installing tensorboard (2.17.1)
  - Installing tensorflow-io-gcs-filesystem (0.37.1)
  - Installing termcolor (2.5.0)
  - Installing threadpoolctl (3.5.0)
  - Installing tokenizers (0.20.0)
  - Installing tree-sitter (0.21.3)
  - Installing typer (0.12.5)
  - Installing wasabi (1.1.3)
  - Installing yarl (1.14.0)
  - Installing aiohttp (3.10.9)
  - Installing contourpy (1.3.0)
  - Installing coverage (7.6.3)
  - Installing cycler (0.12.1)
  - Installing et-xmlfile (1.1.0)
  - Installing fonttools (4.54.1)
  - Installing jsonschema (4.23.0)
  - Installing kiwisolver (1.4.7)
  - Installing langcodes (3.4.1)
  - Installing lxml (5.3.0)
  - Installing openai (1.51.2)
  - Installing pathspec (0.12.1)
  - Installing pillow (10.4.0)
  - Installing pyparsing (3.1.4)
  - Installing pytest (8.3.3)
  - Installing python-dateutil (2.9.0.post0)
  - Installing python-dotenv (1.0.1)
  - Installing pytz (2024.2)
  - Installing scikit-learn (1.5.2): Installing...
  - Installing scikit-learn (1.5.2)
  - Installing soupsieve (2.6)
  - Installing spacy-legacy (3.0.12)
  - Installing spacy-loggers (1.0.5)
  - Installing tensorflow (2.17.0)
  - Installing thinc (8.2.4)
  - Installing tiktoken (0.8.0)
  - Installing torch (2.4.1)
  - Installing transformers (4.45.2)
  - Installing tree-sitter-languages (1.8.0)
  - Installing tzdata (2024.2)
  - Installing weasel (0.4.1)
  - Installing xlsxwriter (3.2.0)
  - Installing beautifulsoup4 (4.12.3)
  - Installing docx (0.2.4)
  - Installing grep-ast (0.3.3)
  - Installing litellm (1.48.19)
  - Installing logging (<IP_ADDRESS>)
  - Installing matplotlib (3.9.2)
  - Installing opencv-python (<IP_ADDRESS>)
  - Installing openpyxl (3.1.5)
  - Installing pandas (2.2.3)
  - Installing pylatexenc (2.10)
  - Installing pypdf2 (3.0.1)
  - Installing pytest-cov (5.0.0)
  - Installing python-docx (1.1.2)
  - Installing python-pptx (1.0.2)
  - Installing rank-bm25 (0.2.2)
  - Installing sentence-transformers (3.1.1)
  - Installing spacy (3.7.5)
  - Installing tenacity (9.0.0)
  - Installing tf-keras (2.17.0)

Installing the current project: agent-as-a-judge (0.1.5)

Hi @Obs01ete @Wenyi-AI-Wang, Could you help me check on your side?

@femto thanks for checking out our code. What is your OS and hardware?
Seems that your platform does not have pre-compiled tensorflow==2.5. You can try installing tensorflow separately with
pip install tensorflow-gpu

and then running poetry install.
Alternatively, you can delete poetry.lock and re-run poetry install. Please leave feedback here on what worked for you. Thanks!

I don't think I have gpu, on mac 14.6.1, Python 3.11.4
Collecting tensorflow-gpu
  Downloading tensorflow-gpu-2.12.0.tar.gz (2.6 kB)
  Preparing metadata (setup.py) ... done
WARNING: Ignoring version 2.12.0 of tensorflow-gpu since it has invalid metadata:
Requested tensorflow-gpu from https://files.pythonhosted.org/packages/8a/45/fa31ced1db38f9424f262dfbf35747fe5378b5c808cecb373c8cb8e515d3/tensorflow-gpu-2.12.0.tar.gz has invalid metadata: Expected end or semicolon (after name and no valid version specifier)
    python_version>"3.7"
                  ^
Please use pip<24.1 if you need to use this version.
ERROR: Could not find a version that satisfies the requirement tensorflow-gpu (from versions: 2.12.0)
ERROR: No matching distribution found for tensorflow-gpu

Try using lower versions:
pip install tensorflow==2.13.0
pip install torch==2.0.1

downgrade the tensorflow and torch worked,
maybe
poetry remove sentence-transformers
and poetry add sentence-transformers torch=2.0.1
to make sentence-transformers to depend on torch=2.0.1,
possibly delete poetry.lock and adding it back.
now it works


feat(rust): add flag to control whether a node redirects the logs to a file

A couple of problems were found on the node create command: the first is a bug about how we handle node logs files; the second is an inconsistency about the expected behaviour when passing the foreground flag or not.
Writing to logs
When creating a fresh foreground node with logging enabled (i.e. passing the -v flag), it won't persist the logs into files because the logger setup is done before the node state is created:
https://github.com/build-trust/ockam/blob/d4d921fc5a503a7e38c9a4869662da2abf005717/implementations/rust/ockam/ockam_command/src/lib.rs#L428-L440
If the node is stopped (e.g. via ctrl+c, or EOF), and the node is recreated again using ockam node create ..., it will start writing the logs into the files beacuse the node state is now accessible when setting up the logger. So, the first time a foreground node is created it won't write into files; subsequent node create calls will do.
There are two problems here:

We don't have a way to create a fresh foreground node that write the logs into files (you have to recreate it to do so)
We don't have a way to create a foreground node that doesn't write the logs into files

Proposed solutions:

A little tricky but doable: we just need a way to partially initialize the state (create the main node directory at least), so we can query the logs file path before actually creating the node state
Add a new argument to control whether we want to write logs into files or not (--disable-file-logging)

Inconsistency of foreground/background nodes when recreating nodes
There was an inconsistency when creating a background node, where it would return an error when running the command for a stopped node. As explained before, this can be done with foreground nodes and it's a nice feature, since it doesn't force the user to learn/use the ockam node start command.
# The following is expected to work, recreating the node after it was stopped
ockam node create foreground -vv -f
ockam node stop foreground
ockam node create foreground -vv -f

# The following was returning an "Already exists" error
ockam node create background
ockam node stop background
ockam node create background

The second case has now the same behaviour as the first one.

⚠  I'm leaving as a draft after today's release.


Question

Hello I was wondering how you normally go about testing your applications seeing as there does not seem to be a Paper Trading option for TD's api. Thanks

Very carefully 😂.
I'm hoping they add paper trading to the API in the future


Create github workflow to automatically generate rule docs in PR

Following up from: https://github.com/Azure/PSRule.Rules.Azure/pull/1259#issuecomment-1040997298, It would be good to add a workflow to automatically build rule docs in a PR.

Isn't this a dup of #1242?

@BernieWhite Ah didn't see that issue, will close this as dupe. Thanks for pointing that out.

Closing as dupe of #1242


NDArray: Complex numbers

Add support for complex numbers, vectors, matrices, tensors.

https://observablehq.com/@mateogianolio/complex-numbers-in-javascript


Unknown encoder 'libfdk_aac'

Why isn't --enable-libfdk-aac enabled here?

libfdk-aac is nonfree and thus cannot be in redistributable builds.
If you want libfdk, you got to produce a nonfree build locally yourself.

@BtbN
In redistributable builds, which audio encoder can I use?

ffmpeg -encoders | grep -i aac


Scrape release metrics from repositories

Fixed prometheus HTTP handler, as the current build is broken by this bug.

Add downloads metric for releases.

# TYPE github_repo_release_downloads gauge
github_repo_release_downloads{name="release-1.0.0",repo="github-exporter",user="infinityworks"} 3500

@Rucknar Im'a gonna bother you once more 😄