Error when running dev-latest.jar

dev-latest.jar is failing to run in Cypress tests (example here). This error is being thrown on startup:
##[error]Exception in thread "main" java.util.ServiceConfigurationError: javax.imageio.spi.ImageWriterSpi: Provider com.sun.media.imageioimpl.plugins.jpeg.CLibJPEGImageWriterSpi could not be instantiated
	at java.base/java.util.ServiceLoader.fail(ServiceLoader.java:581)
	at java.base/java.util.ServiceLoader$ProviderImpl.newInstance(ServiceLoader.java:803)
	at java.base/java.util.ServiceLoader$ProviderImpl.get(ServiceLoader.java:721)
	at java.base/java.util.ServiceLoader$3.next(ServiceLoader.java:1394)
	at java.desktop/javax.imageio.spi.IIORegistry.registerApplicationClasspathSpis(IIORegistry.java:188)
	at java.desktop/javax.imageio.spi.IIORegistry.<init>(IIORegistry.java:113)
	at java.desktop/javax.imageio.spi.IIORegistry.getDefaultInstance(IIORegistry.java:134)
	at java.desktop/javax.imageio.ImageIO.<clinit>(ImageIO.java:66)
	at com.conveyal.analysis.BackendMain.startServer(BackendMain.java:52)
	at com.conveyal.analysis.BackendMain.main(BackendMain.java:43)
Caused by: java.lang.IllegalArgumentException: vendorName == null!
	at java.desktop/javax.imageio.spi.IIOServiceProvider.<init>(IIOServiceProvider.java:76)
	at java.desktop/javax.imageio.spi.ImageReaderWriterSpi.<init>(ImageReaderWriterSpi.java:233)
	at java.desktop/javax.imageio.spi.ImageWriterSpi.<init>(ImageWriterSpi.java:213)
	at com.sun.media.imageioimpl.plugins.jpeg.CLibJPEGImageWriterSpi.<init>(CLibJPEGImageWriterSpi.java:84)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:490)
	at java.base/java.util.ServiceLoader$ProviderImpl.newInstance(ServiceLoader.java:779)

Is this due to the new build system?

Yes, JAI is super touchy about metadata included in the JAR and it must not be setting this right. I should be able to fix it.

I have grabbed the JAR built with the patch v5.10.0-7-g0121a55 and it seems to start cleanly. It's not clear to me though why I need to manually inject this metadata into the JAR. The stacktrace shows it reading the vendor name from com.sun.media.imageioimpl.common.PackageUtil in which this is a hard-coded literal field. Seems like the implementation I see in the IDE was getting replaced somehow when packaging the JAR.

I'm wondering if in our decade-long use of Geotools we've failed to follow the migration to JAI-EXT. Our dependencies seem to contain a JAI implementation with a hard-coded vendor name, which we're not using at runtime. We might have to do some setup to register this implementation instead of the long-abandoned Oracle one. Some links to dig into:
https://docs.geotools.org/stable/userguide/tutorial/raster/jaiext.html#jaiext
https://github.com/geosolutions-it/jai-ext
https://github.com/geotools/geotools/wiki/Replace-JAI-Background-Discussion
https://docs.geotools.org/latest/javadocs/org/geotools/image/jai/package-summary.html
The last one says services may not be registered in some circumstances, which may include Shaded / Shadow JARs.

As a stopgap though, injecting these vendor name strings into the JAR metadata might keep it running without the potential disruption of swapping out our image reading/writing implementation. If this version v5.10.0-7-g0121a55 works as a worker or backend then it might be a viable solution. Others, please feel free to test, and I will look into it more tomorrow.

Using v5.10.0-7-g0121a55, worker exits with:
OpenJDK 64-Bit Server VM Corretto-8.265.01.1 (build 25.265-b01, mixed mode)
Error: A JNI error has occurred, please check your installation and try again
Exception in thread "main" java.lang.UnsupportedClassVersionError: com/conveyal/r5/R5Main has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0
        at java.lang.ClassLoader.defineClass1(Native Method)
        at java.lang.ClassLoader.defineClass(ClassLoader.java:756)
        at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142)
        at java.net.URLClassLoader.defineClass(URLClassLoader.java:468)
        at java.net.URLClassLoader.access$100(URLClassLoader.java:74)
        at java.net.URLClassLoader$1.run(URLClassLoader.java:369)
        at java.net.URLClassLoader$1.run(URLClassLoader.java:363)
        at java.security.AccessController.doPrivileged(Native Method)
        at java.net.URLClassLoader.findClass(URLClassLoader.java:362)
        at java.lang.ClassLoader.loadClass(ClassLoader.java:418)
        at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:352)
        at java.lang.ClassLoader.loadClass(ClassLoader.java:351)
        at sun.launcher.LauncherHelper.checkAndLoadMain(LauncherHelper.java:495)

Before uploading the JARs in the Github Action should we run the jars to verify they can be run?


Direct Convert 3D Blu-ray to 3D F/HSBS, T&B

Is it possible ?

It might be possible with customization, I don't have experience with it, maybe ask at doom9 or videohelp.


Is ParseConfig supported?

And if so how do you set it?

Parse Config has experimental support right now, which you can enable by setting the PARSE_EXPERIMENTAL_CONFIG_ENABLED environment variable. However there are no docs yet, and won't be until support leaves experimental mode.


Be consistent with the Op wording

When developing https://github.com/frees-io/freestyle/pull/353, i needed to pattern match on the cases of the ADT generated by a @free algebra. In those cases, the OP suffix is added to the name of the method, capitalized.
Also, on @module annotated traits, a  Op type is used to refer to the operation.
As of my understanding both refer to operation, so it would be great to use the same term for both, either OP or Op. It's not a biggie, but I needed to investigate a bit further asking scalac to expand the macro for me to figure out that in ADT cases, it's OP.

Makes sense, we should take care of this before 0.4.0. Thanks for reporting it :)


Label click for input element (accessibility)

I've noticed that the input form element label is not clickable. This means that its not possible to click on the label to focus on the input element. Is this something that was omitted or something I missed?
I know the current workaround is to provide my own htmlFor element like this but I would prefer that the Form.Input would work like this out of the box.
<Form.Field>
  <label htmlFor="password">Password</label>
  <input
    id="password"
    name="password"
    type="password"
  />
</Form.Field>

This seems like a logical thing to do.  Though, I think it only makes sense if you pass an id as I don't think it is a good idea to guess at which ids to add to the inputs.  Thoughts?

@levithomason if id is not provided it is possible to use uniqueId from lodash to ensure that the element always has a unique id field. This may have a very small performance hit.

You could probably set ID to something like 'form-field-${props.name}' if no ID is provided. Requiring passing an ID is a little redundant especially since within a form each field's name should be unique.

The id cannot change over time, so let's explore the tmp id based on the name.
The only issue I see here is if you have two form fields on a page with the same name, say, due to two forms. I could easily see this happening in an interface where there is a loop involved (we're building two of those ATM).
The other potential solution there is to base the unique id on the entire props object.  This increases the chances of the id being unique but is still not guaranteed.  We have a generator that does this for unique child keys in #645.
I am still a little reluctant to implement any of these automated solutions as there are cases where they fail silently with no way to warn the user.  Looking for more ideas here, perhaps a form prop where you can pass a unique name that can be used?

I'm going to close this one for housekeeping.  Feel free to reopen if there is a clear path to resolution.

We are building an accessible site and also need the id on the input to match to the label for attribute.  It would be nice if the id was passed to the input itself instead of the div containing it.

I am running into this same issue..  We can get around it by putting the inputs nested inside the label, but doesn't seem to work for drop downs.  An explicit ID can help (and eslint react-a11y complains about the nested approach w/o an id).

Alrighty folks, looks like there is enough support here to pass the Form.Input's id to the label's htmlFor prop.  PR would be very much appreciated.

Not only is the label not clickable by mouse, but the input isn't accessible for screen reader users because the input and label aren't paired with an id and for attribute. (The label could also wrap the input to achieve the same behavior.)
There is also no visible keyboard focus style, so these controls aren't usable from the keyboard only. These are both really big issues for accessibility that I'd hope to see fixed at the framework level.

@marcysutton

Not only is the label not clickable by mouse, but the input isn't accessible for screen reader users because the input and label aren't paired with an id and for attribute.

The Form.Input is a convenience API. You are free to add ids and any other prop you like to your forms. The full control is up to you:
http://react.semantic-ui.com/collections/form#form-example-form

The label could also wrap the input to achieve the same behavior.

This repo only includes React components. We are not in control of the CSS so we must use valid Semantic UI markup.

There is also no visible keyboard focus style, so these controls aren't usable for someone who can't use a mouse.

All SUI form controls are completely keyboard accessible and include focus state styles:
http://react.semantic-ui.com/elements/input#input-example-focus

These are both really big issues for accessibility that I'd hope to see fixed at the framework level.

We also hope to see every issue fixed and feature implemented.  As mentioned, PRs are very much appreciated. We also love to collaborate and assist with PRs as soon as they are opened. This is the best way to affect change in library.
You can refer to our CONTRIBUTING.MD for great instructions on how to contribute.

Hi guys,
I would like to fix this bug as my first contribution here.  I'm thinking of adding a new prop "ID" to Form.Input Shorthand to use as ID in input and as htmlFor in Label.
Does anyone object or have better idea?

Sounds great!  This seems to be inline with the agreed solution above:

Alrighty folks, looks like there is enough support here to pass the Form.Input's id to the label's htmlFor prop. PR would be very much appreciated.


Ignored auth-user-pass option

I'm bit struggling with auth-user-pass option. Even tough I follow instructions by adding new line to my vpn.conf
auth-user-pass vpn.auth

and store it beside vpn.conf I still get the following error:
ovpn  | --- Running with the following variables ---
ovpn  | VPN configuration file: vpn.conf
ovpn  | Use default resolv.conf: on
ovpn  | Allowing subnets: none
ovpn  | Kill switch: iptables
ovpn  | Using OpenVPN log level: 3
ovpn  | ---
ovpn  |
ovpn  | info: original configuration file: vpn/vpn.conf
ovpn  | info: modified configuration file: vpn/openvpn.xihEvEdw.conf
ovpn  | info: kill switch is using iptables
ovpn  | 2023-01-17 12:32:37 DEPRECATED OPTION: --cipher set to 'AES-256-CBC' but missing in --data-ciphers (AES-256-GCM:AES-128-GCM). Future OpenVPN version will ignore --cipher for cipher negotiations. Add 'AES-256-CBC' to --data-ciphers or change --cipher 'AES-256-CBC' to --data-ciphers-fallback 'AES-256-CBC' to silence this warning.
ovpn  | 2023-01-17 12:32:37 WARNING: file 'vpn.auth' is group or others accessible
ovpn  | 2023-01-17 12:32:37 OpenVPN 2.5.6 x86_64-alpine-linux-musl [SSL (OpenSSL)] [LZO] [LZ4] [EPOLL] [MH/PKTINFO] [AEAD] built on Apr 17 2022
ovpn  | 2023-01-17 12:32:37 library versions: OpenSSL 1.1.1o  3 May 2022, LZO 2.10
ovpn  | 2023-01-17 12:32:37 NOTE: the current --script-security setting may allow this configuration to call user-defined scripts
ovpn  | 2023-01-17 12:32:37 neither stdin nor stderr are a tty device and you have neither a controlling tty nor systemd - can't ask for 'Enter Private Key Password:'.  If you used --daemon, you need to use --askpass to make passphrase-protected keys work, and you can not use --auth-nocache.
ovpn  | 2023-01-17 12:32:37 Exiting due to fatal error

It is very similar to #34 but they resolved with this option, I couldn't. It seems like it is ignoring it because when I change it to nonexisting file I get:
Options error: --auth-user-pass fails with 'vpn.auth2': No such file or directory (errno=2)

Here's my OpenVPN configuration file:
client
dev tun
proto udp
remote <my-address> 1194
resolv-retry infinite
nobind
persist-key
persist-tun
remote-cert-tls server
#auth SHA512
auth-user-pass vpn.auth
cipher AES-256-CBC
ignore-unknown-option block-outside-dns
#block-outside-dns
verb 11
<ca>
-----BEGIN CERTIFICATE-----
*****
-----END CERTIFICATE-----
</ca>
<cert>
-----BEGIN CERTIFICATE-----
*****
-----END CERTIFICATE-----
</cert>
<key>
-----BEGIN ENCRYPTED PRIVATE KEY-----
*****
-----END ENCRYPTED PRIVATE KEY-----
</key>
<tls-crypt>
-----BEGIN OpenVPN Static key V1-----
*****
-----END OpenVPN Static key V1-----
</tls-crypt>

And docker-compose file:
version: '3'

services:
  ovpn:
    image: ghcr.io/wfg/openvpn-client
    container_name: ovpn
    cap_add:
      - NET_ADMIN
    devices:
      - /dev/net/tun
    environment:
      VPN_CONFIG_FILE: "vpn.conf"
    volumes:
      - ./openvpn-data:/data/vpn
    restart: unless-stopped

I even tried to use secrets with VPN_AUTH_SECRET option but the same error pops up.
Could it be because of some missing/wrong option in configuration file?

Resolved this.
This is completely my mistake. The private key was protected using passphrase instead and so there was no need for auth-user-pass.
Replacing auth-user-pass with askpass instead with only the passphrase on the first line inside vpn.auth resolved it.


sometimes msgbacklog size is not correct when restart

Describe the bug
we restart a broker, then  one topic in the broker msgbacklog is wrong
Expected behavior
Screenshots
If applicable, add screenshots to help explain your problem.

"cursors" : {
    "C_tort_url_patrol_formal" : {
      "markDeletePosition" : "13438589:33134",
      "readPosition" : "13487114:46889",
      "waitingReadOp" : false,
      "pendingReadOps" : 0,
      "messagesConsumedCounter" : -9139902,
      "cursorLedger" : 13685682,
      "cursorLedgerLastEntry" : 12009,
      "individuallyDeletedMessages" : "[(13438589:33135..13438589:49999],(13439440:-1..13439440:13991],(13439440:13992..13439440:14059],(13439440:14060..13439440:14093],(13439440:14094..13439440:14233],(13439440:14234..13439440:49999],(13440145:-1..13440145:18170],(13440145:18171..13440145:18221],(13440145:18222..13440145:18340],(13440145:18341..13440145:19341],(13440145:19342..13440145:19346],(13440145:19349..13440145:19351],(13440145:19352..13440145:19353],(13440145:19354..13440145:19355],(13440145:19357..13440145:19360],(13440145:19368..13440145:19370],(13440145:19373..13440145:19377],(13440145:19378..13440145:19379],(13440145:19381..13440145:19387],(13440145:19388..13440145:19389],(13440145:19393..13440145:19396],(13440145:19397..13440145:19398],(13440145:19407..13440145:19409],(13440145:19410..13440145:19412],(13440145:19

as we can see messagesConsumedCounter is negative number.
then we unload the topic , expect this will recover,  but  messagesConsumedCounter maybe more small

Desktop (please complete the following information):

OS: [e.g. iOS]

Additional context
Add any other context about the problem here.

@leizhiyuan  Hi, I can help with this issue.
Another question:

Have you noticed that messages are lost in this case?

@leizhiyuan
This counters are used to compute the numberOfEntries and numberOfEntriesInBacklog values, without having to look at the list of ledgers in the managed ledger. They are initialized to (-backlog) at opening, and will be incremented each time a message is read or deleted.

Hi, @leizhiyuan
I can help with this issue.
Another question:

Have you noticed that messages are lost in this case?

It seems not
but when I restart broker, the backlog will increase. I restarted twice, the backlog increase to 4* (original number) .
It will decrease with time fly (maybe consumers consumed)


Spinner freeze on mobile

Image gif Spinner freeze on mobile.
May be it would be interesting to do a css3 spinner?
http://projects.lukehaas.me/css-loaders/

Yes, I know issue with gif animation on mobile, most likely it's disabled for powersafe reason. In turn, it was a reason to include work with https://github.com/daneden/animate.css into lazyloadxt's docs and demo.


Question about Boxes used in RCNN

Hi, do you use boxes from RPN (so usually is (1000, 4)), or boxes after faster-RCNN box regression (also (1000, 4)), or the final boxes after NMS?

I use boxes after faster-RCNN box regression before NMS.

Minor question: so in Fig.4 of your paper, there shouldn't be an arrow from RPN proposals pointing to RCNN training? What does that arrow mean?

The main reason is to make the figure looks clean without overlapping arrows @weiyc
It should come from the Faster R-CNN outputs as stated in Section 4.2.

Thanks. Have you tried using RPN box instead? If you weren't using class-agnostic regressor head then the regression output depends on the Faster RCNN classification output, which I think may negatively affect the Decoupled RCNN performance.

The bounding box regression in Faster R-CNN is class agnostic. That is, all classes shared the same box for each proposal.


Update existing PRs for updates instead of opening new ones

Description
Currently, Packit opens a new PR for any new update it sees. For example, for fastavro Packit submitted PR #11 for updating to 1.9.0 and some time later PR #13 for updating to 1.9.1.
However, since both PRs are based on rawhide, automatic merging will fail for one or the other. The reason for fastavro not being updated yet, is a missing dependency introduced by upstream in 1.9.0. This will take some time and effort to solve.
Instead of submitting a new PR, Packit should amend the existing PR, if it is still open. That is update the title and add another commit with the changes for the latest release.
Benefit
The obvious benefit is a clear path forward for merging into rawhide, skipping intermediate versions. Updating an existing PR will also preserve the comments from that PR, possibly containing information why the update is still pending. Last but not least, additional changes may have been pushed to Packit's update branch by the package maintainers. Preserving these changes, the next update can take those into account.
Importance
I'd say this is medium importance. The benefits speak for themselves. With packages that see new releases frequently, this becomes more important.
Workaround

[ ] There is an existing workaround that can be used until this feature is implemented.

Participation

[ ] I am willing to submit a pull request for this issue. (Packit team is happy to help!)

Blocked waiting for https://pagure.io/fedora-infrastructure/issue/11791

Blocked waiting for https://pagure.io/pagure/pull-request/5457


Remove skip markers for balanced since LR criterion is fixed

After this https://github.com/ott-jax/ott/pull/547#event-13382808561 we can finally expect the tests to pass again. So the skip markers are removed.
Closes:

https://github.com/theislab/moscot/issues/678
https://github.com/theislab/moscot/issues/703

@selmanozleyen I have done that also here just FYI https://github.com/theislab/moscot/pull/697/files

Ah ok then I am closing this


Fixed issues with color_key alias

Coverage decreased (-0.2%) to 80.666% when pulling 0f43b6049211bb3be9d83f1aa84e5235b9a0f800 on color_key_cleanup into 1e24d058eb78af53fd284618b419f7a8ce799dd3 on master.


Can't login. 404 on the URL provided

Just tried to use this today.
Can't login with user/pass and the link to: https://plex.tv/link:5J9R
is 404.
They change something? Would LOVE to have my music on a native app.

To be clear, there are two login options:

user/password using your plex.tv account and the login button in the app
PIN code - go to https://plex.tv/link , login to your account there and then enter the 4-character PIN displayed on the screen

It seems like you may have appended the pin to the URL. Let me know if it works as described above.

 Not sure why my user/pass didn't work, but link was fine once I stopped trying to add the pin to the URL :)


Add Version: Buttercup.Buttercup version 2.10.0

- [X] Have you signed the Contributor License Agreement?

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

#24144 :D

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

cough 🙄
If only wingetbot would merge faster i would see these when searching in the repo :c


Made changes in README.md

purpose of the project
Updated README.md

@amruta-07 am closing this PR as I can see there are no changes needed in this file. The changes that you have made is incorrect.


Ignore abandoned and merged changes for attentionset

Attention set based notifications would still come in for abandoned and
merged changes due to the status portion of the query being left out.
This is annoying as I was often in the attentionset for changes abandoned
and it was difficult to figure out why gerrit-monitor continued to think I
should be notified for these for the past few months.

Thank you. Sorry that it took such a long time. I completely missed the notification about this pull request.


windows hostprocess: k8s services in CNI only reachable after a pod is scheduled

Expected Behavior
After calico-node-startup, felix and kube-proxy have set up everything, the CNI is fully ready without needing a pod to be running. A service within the CNI can be reached also from the host.
We don't actually need to reach services from the host, but we noticed the CNI is not fully setup without a non-hostprocess pod to be scheduled.
This issue is closely related to #6829.
Current Behavior
When booting up a new windows node, calico startup, felix and kube-proxy do the startup sequence and setup the networking. If no pod is scheduled afterwards, the networking is only half-way set-up though.
See Slack for discussion.
Possible explanation by @fasaxc:
Calico creates HNS resources only on demand. Calico supports multiple IP pools but Windows only supports one CIDR per HNS Network.
Possible Solution
Adjust startup sequence if possible to set up the necessary bits earlier.
Steps to Reproduce (for bugs)
See #6829 for steps to reproduce. The fix from our fork needs to be applied to calico-node-startup to ensure it can actually get to the point where we can observe this behavior.
Context
Same as in #6829.
Your Environment

Calico version v3.23.0
Orchestrator version (e.g. kubernetes, mesos, rkt): k8s 1.24.6 with kubeadm
Operating System and version: windows

A possible workaround is using a noop pod with a DaemonSet. Example is below. Ensure your windows node are booting up with the following set:
label: node.example.com/ready-taint=true
taint:
key: "node.helio.exchange/ready"
value: "false"
effect: "NoSchedule"

```yaml
---
kind: ConfigMap
apiVersion: v1
metadata:
  name: calico-tainter-script
  namespace: kube-system
  labels:
    app: calico-tainter
data:
  script.ps1: |
    echo "Sleeping for 30s before untainting $env:NODE_NAME"
    Start-Sleep 30
    echo "Done sleeping, untainting node"
    C:\kubectl.exe taint nodes $env:NODE_NAME node.example.com/ready-
    echo "Taint removed"
    echo "Sleeping for 10min until evicting pod"
    Start-Sleep 600
    echo "Removing label so DS pod gets evicted"
    C:\kubectl.exe label nodes $env:NODE_NAME node.example.com/ready-taint-
    echo "Label removed"
---
apiVersion: v1
kind: ServiceAccount
metadata:
  name: calico-tainter
  namespace: kube-system
---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  name: calico-tainter
rules:
  - apiGroups:
      - ""
    resources:
      - nodes
    verbs:
      - get
      - patch
      - update
---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: calico-tainter
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: calico-tainter
subjects:
  - kind: ServiceAccount
    name: calico-tainter
    namespace: kube-system
---
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: calico-tainter
  labels:
    app: calico-tainter
  namespace: kube-system
spec:
  selector:
    matchLabels:
      app: calico-tainter
  template:
    metadata:
      labels:
        app: calico-tainter
    spec:
      serviceAccountName: calico-tainter
      tolerations:
        - operator: Exists
          effect: NoSchedule
        - effect: NoExecute
          operator: Exists
      nodeSelector:
        kubernetes.io/os: "windows"
        node.example.org/ready-taint: "true"
      containers:
        - name: main
          image: windows-image-with-kubectl:v0.0.1
          imagePullPolicy: IfNotPresent
          args:
            - /tainter/script.ps1
          env:
            - name: NODE_NAME
              valueFrom:
                fieldRef:
                  fieldPath: spec.nodeName
          volumeMounts:
            - name: calico-tainter-script
              mountPath: /tainter
      volumes:
        - name: calico-tainter-script
          configMap:
            name: calico-tainter-script

Dockerfile for the kubectl image:
FROM mcr.microsoft.com/windows/servercore:ltsc2019

SHELL ["powershell", "-Command", "$ErrorActionPreference = 'Stop'; $ProgressPreference = 'SilentlyContinue';"]

WORKDIR C:/

RUN curl -o kubectl.exe "https://dl.k8s.io/release/v1.24.0/bin/windows/amd64/kubectl.exe"

ENTRYPOINT ["powershell"]

pinging @song-jiang @coutinhop

Thanks @mweibel for the details. Are you sure you are using Calico v3.23.0, I'm pretty sure an IPAM block will be allocated for the node and HNS network should be created without any pod running on the node.  code here


[question] acl: protect a route

can someone help me to use this https://github.com/mikemclin/angular-acl#protect-a-route to protect ui-routes?

Hi @andresrivas1506 , its already implemented by default, on protected routes you simply add
data { auth: true } example: [routes.config.js#L36] ( https://github.com/silverbux/laravel-angular-admin/blob/master/angular/config/routes.config.js#L36 )
and to show / hide elements use vm.can() nav-sidebar.component.html#L28 which is referenced on its controller nav-sidebar.component.js#L6

Hello @silverbux thanks for your time. Yes, its already implemented, but only using data { auth: true } and showing/hiding things in template. If you go, for example, to url http://myapp/#/user-lists from url bar, you can see the html.
I want to protect the route using permissions:
.when('/content', {
      resolve : {
        'acl' : ['$q', 'AclService', function($q, AclService){
          if(AclService.can('view_content')){
            // Has proper permissions
            return true;
          } else {
            // Does not have permission
            return $q.reject('Unauthorized');
          }
        }]
      }
    });

Sorry for my bad english

I can solve with this:

In routes.run.js#L48

$rootScope.$on('$stateChangeError', function (event, toState, toParams, fromState, fromParams, rejection) {
    if (rejection === 'Unauthorized') {
      $state.go('app.landing');
    }
  });

my protected route

.state('app.someorute', {
      url: '/someroute',
      resolve : {
        'acl' : ['$q', 'AclService', function($q, AclService){
          if(AclService.can('some_permission')){
            // Has proper permissions
            return true;
          } else {
            // Does not have permission
            return $q.reject('Unauthorized');
          }
        }]
      },
      data: {
        auth: true
      },
      views: {
        'main@app': {
          templateUrl: getView('some')
        }
      }
    })

Thank you @silverbux

Coolness, or try this approach as well.
data: {
   auth: true,
   access: 'manage.user'
}

then  you could add a function on your /angular/run/routes.run.js and call AclService from there.

very nice that approach, thanks


Ubuntu 22.04 Repository Instructions Missing

This page doesn't include documentation for Ubuntu 22.04. Please update it.

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

ID: 25943cc5-9927-5ca7-b9db-4c1d398726c5
Version Independent ID: ae5bf48b-26cf-b5c8-22c8-68fa9a70258f
Content: Linux Software Repository for Microsoft Products
Content Source: WindowsServerDocs/administration/Linux-Package-Repository-for-Microsoft-Software.md
Product: windows-server
Technology: administration
GitHub Login: @victorcheng7
Microsoft Alias: wscontent

This should also include Enterprise Linux 9.


ISSUE-28 - Fix lesson-1

looks very good. Can you also change example.com to something like example.lan, or something from Appendix G, RFC 6762?

@dbecvarik thanks for the feedback! Sure, I can use example.lan instead of example.com.

thanks, LGTM


Postgres connector for Semantic Memory

Motivation and Context

Postgres supports cosine similarity search by installing the pgvector extension.
Description

Add a new Semantic Memory Connector leveraging Postgres with pgvector extension. Update the syntax example to demonstrate how it is used.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

Close #945

@microsoft-github-policy-service agree

Hi, @adrianwyatt,
Can you review these new commits and let me know if there is anything else that can be done?
Thank you!

Hi @lemillermicrosoft. It looks like the formatting style of the .cs file was changed by #1030. Can you retrigger these actions?
Thanks.


[Event Request] Table 38 "Purchase Header", field 53 "Applies-to Doc. No.", OnValidate trigger; OnBeforeValidateAppliesToDocNo

Hello, we need additional validation in field 53 of Purchase Header table, thank you in advance.
field(53; "Applies-to Doc. No."; Code[20])
        {
            trigger OnValidate()
            var
                // - 22.11.21 NDA ND#0303347
                // moved to the subscriber OnValidateDocumetType in oodeunit "Purchase Header MGB"
                // VendDepLedgEntryL: Record "Vend. Deposit Ledger Entry";
                // + 22.11.21 NDA ND#0303347
                IsHandeld: Boolean;
            begin
                if "Applies-to Doc. No." <> '' then
                    TestField("Bal. Account No.", '');
// - new
                IsHandeld := false;
                OnBeforeValidateAppliesToDocNo(Rec, IsHandeld);
                if IsHandeld then
                    exit;
// + new
                if ("Applies-to Doc. No." <> xRec."Applies-to Doc. No.") and (xRec."Applies-to Doc. No." <> '') and
                      ("Applies-to Doc. No." <> '')
                then begin
                    SetAmountToApply("Applies-to Doc. No.", "Buy-from Vendor No.");
                    SetAmountToApply(xRec."Applies-to Doc. No.", "Buy-from Vendor No.");
                end else
                    if ("Applies-to Doc. No." <> xRec."Applies-to Doc. No.") and (xRec."Applies-to Doc. No." = '') then
                        SetAmountToApply("Applies-to Doc. No.", "Buy-from Vendor No.")
                    else
                        if ("Applies-to Doc. No." <> xRec."Applies-to Doc. No.") and ("Applies-to Doc. No." = '') then
                            SetAmountToApply(xRec."Applies-to Doc. No.", "Buy-from Vendor No.");
            end;
        }

    [IntegrationEvent(false, false)]
    local procedure OnBeforeValidateAppliesToDocNo(var PurchHeader: Record "Purchase Header"; var IsHandeld: Boolean)
    begin
    end;

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


update to gosu 1.17

See https://github.com/tianon/gosu/releases/tag/1.17.
Similar to previously-merged move to 1.16 at https://github.com/docker-library/postgres/pull/1040.

Thank you!


How to cancel the checked "retain ip"

Rancher versions:
rancher/server: 1.6.14
rancher/agent: v1.2.9
Infrastructure Stack versions:
healthcheck: v0.3.3
ipsec:
network-services:
scheduler:
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
Docker version 17.12.0-ce, build c97c6d6
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
CentOS Linux release 7.4.1708 (Core)
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher, internal DB
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Results:
Can not cancel retain ip on or off of container.

It's not supported. You can clone the service and unselect the retain ip. Then you can delete the old service.


AVRO-3458: Added tests for GenericRecord

Jira

[ ] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-3458

Tests

[ ] My PR does not need testing for this extremely good reason: It is just adding tests

Commits

[ ] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

GenericTests.cs already has many GenerricRecord related tests. Why are these tests in a seperate GenericRecordTests.cs file?
Could these tests utilize the mkRecord and other helper code implemenetd in GenericTests.cs?
schema definition strings are preferred to be defined as const/readonly/static fileds or use test case to specify them

GenericTests.cs already has many GenerricRecord related tests. Why are these tests in a seperate GenericRecordTests.cs file?
Could these tests utilize the mkRecord and other helper code implemenetd in GenericTests.cs?
schema definition strings are preferred to be defined as const/readonly/static fileds or use test case to specify them

These are unit tests specific to just GenericRecord.
It could, but the tests don't require something that complex
Moved the base schema to a const.

GenericTests.cs already has many GenerricRecord related tests. Why are these tests in a seperate GenericRecordTests.cs file?
Could these tests utilize the mkRecord and other helper code implemenetd in GenericTests.cs?
schema definition strings are preferred to be defined as const/readonly/static fileds or use test case to specify them

These are unit tests specific to just GenericRecord.
It could, but the tests don't require something that complex
Moved the base schema to a const.

I usually wait for the checks to go green before approval. Can you fix the failed checks plz?

Thank you, @KyleSchoonover !


Fix wrong bias shape of ConvTranspose

Previously, the bias shape of ConvTranpose was wrong since, unlike Conv, it should using the 1st dimension of weight shape and not the 0th. See described in https://github.com/microsoft/onnxscript/issues/1299
In other words,
    if bias is None:
        weight_dim_0 = op.Shape(weight, start=0, end=1)
        bias_shape = op.Expand(weight_dim_0, op.Constant(value_ints=[1]))
        zero = op.CastLike(0.0, input)
        bias = op.Expand(zero, bias_shape)

should be changed to something like:
weight_dim_0 = op.Shape(weight, start=1, end=2) if transposed else op.Shape(weight, start=0, end=1)

However, I think it's more efficient to just eliminate bias altogether if it's not provided instead of filling it with zeros, since the ONNX spec allows bias to be absent.

Amazing, thanks!


📃 Write a blog post about detecting sensitive data using Akto

💭 Introduction
Akto is an open source API security product.
Your task is to write a blog post about testing for detecting sensitive datausing Akto.
🎯 Requirements

Your article has to be publicly available.
Your article must tag Akto in any way (hashtag, embedded, link...).
Your article should be at least 1000 words long.
Your article should look nice. 👀 Use titles, subtitles, screenshots, images, gifs, or even memes.
The blog has to be factually correct. Incorrect submission will be rejected.
You have to deep dive into the product by signup and using it.

✅ Task summary:

[ ] Drop a comment on this issue indicating that you’re working on it.
[ ] Write a blog post with title  How I detected APIs leaking sensitive data using Akto?
[ ] Publish an article on your favorite platform or website. (Medium, Dev.to, Hashnode...)
[ ] Submit a pull request here.
[ ] Share your work on social media and tag https://github.com/akto-api-security/akto (Reddit / HackerNews / Twitter / Facebook / Linkedin)

🙋🏼♂ Questions:
If you have questions, need any help, or just want to hang out, make sure to join us on our Discord server.

@Ankita28g I would like to work on this issue please assign it to me.

@Ankita28g Me and @Roshani9731 would like to work together please assign us.

So awesome to see you interest 🥳. Assigning to you! Happy hackfesting🎉

Testing for sensitive data sounds familiar to me and it would be really great to advocate about the same. I would like to work on this issue, could you please assign me.

Assigning it to you! Happy hackfesting 😃

Assigned to you @Sa24miksha
Happy hacktoberfest 😀😃

@Ankita28g, thank you for assigning me this work. It was enjoyable to explore and write about Akto. Here is my blog that fulfills all the requirements:
https://medium.com/@samikshayadaav/how-i-detected-apis-leaking-sensitive-data-using-akto-b8e700bcac01
also the social media link : https://x.com/Samiksh2431/status/1712083399199052261?s=20

@Ankita28g @ankush-jain-akto please review it .

@Sa24miksha I reviewed and left comment in Discord and LinkedIn message. Once it's corrected, you can resubmit

@Ankita28g, can you assign this issue to me

I've assigned it to you, @Vishalk91-4 . Happy hacking! Feel free to join our Discord if you need assistance.

@Ankita28g  can you please assign this issue to me? I would like to work on it.

I've assigned it to you, @MacroAndMicro  . Happy hacking! Feel free to join our Discord if you need assistance.


Documentation

As we create more files to put documentation on, we can put them in comments here. Also comment when we've finished documentation for a file.

sign-up.php

create.php


Difference in the outputs of FloatTensor and CudaTensor for torch.mm

I am seeing difference in the outputs of matrix multiplication (torch.mm), in FloatTensor and CudaTensor. (Cuda version: 7.5, GPU: Titan X, Ubuntu 14.04 LTS)
Here is the test code along with output:
https://gist.github.com/sourabhd/f422969c00f0ea4310e8365bc2e254d6

i dont see a problem here. it's somewhat expected....

Can you elaborate ? The difference gets significant as the dimensions increase. Is there any alternative/workaround for accurate calculations ?

both are being accurate to float precision. the order of accumulations is different on CPU and GPU, that's all (and hence the difference).


[bug]: colors broken after latest update

After the most recent pull request merge #5 , colors are not displayed correctly anymore as demonstrated here:
before update:

after update:

This happens without changing anything in the config, just updating the plugin.

Thanks, I've reverted the change. I'll need to test more the PR, sorry for the inconvenience...

works great now with the latest merge, thanks.


click event for gui controls: onPointerClickObservable

In reference to this thread on the forums:
http://www.html5gamedevs.com/topic/35775-babylon-gui-missing-button-event-click
gui controls only had pointer up and pointer down events, but no event that only fires on valid "clicks".
(click: cursor down while the cursor is over the control, cursor up while the mouse is still over the control)
Adding this to any playground the click event can be tested:
(The button toggles color and text on click)
var gui = BABYLON.GUI.AdvancedDynamicTexture.CreateFullscreenUI("myUI");
var button = BABYLON.GUI.Button.CreateSimpleButton("button", "Clicks: 0");
button.top = "0px";
button.left = "0px";
button.width = "150px";
button.height = "50px";
button.cornerRadius = 20;
button.thickness = 4;
button.children[0].color = "#DFF9FB";
button.children[0].fontSize = 24;
button.color = "#FF7979";
button.background = "#EB4D4B";

var clicks = 0;
button.onPointerClickObservable.add(function () {
	clicks++;
	if (clicks % 2 == 0) {
	  button.background = "#EB4D4B";
	} else {
	  button.background = "#007900";
	}
	button.children[0].text = "Clicks: " + clicks;
});

gui.addControl(button);

Explanation:
When the Up event is fired there is now a check if the cursor is still over the control.
If it is, the Click event is fired too.
The private _onPointerUp has a new boolean argument "notifyClick" that decides if the "click" event should propagate to the parent control or not.
(This is set by the return value from the onPointerUpObservable.)

Excellent
Can you also update gui doc to add reference to this new observable?

I'll update the documentation tomorrow.


Display Issue with Multiline Deprecation Reason

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Currently, when a deprecation reason is displayed in GraphiQL, only the first paragraph of the deprecation message is visible, while the rest of the information (second paragraph and the bulleted list) is missing.
for example with the following node in schema:
{
  "name": "sampleField",
  "description": "Sample description for the field.",
  "args": [],
  "type": {
    "kind": "SCALAR",
    "name": "String",
    "ofType": null
  },
  "isDeprecated": true,
  "deprecationReason": "The 'sampleField' field is deprecated.\n\nThis field has been deprecated due to sample reasons.\n\n- Marked as deprecated on 2023-07-14.\n- Will be removed on 2024-07-14."
}

the following html tags are created:

Deprecated

The 'sampleField' field is deprecated.

This field has been deprecated due to sample reasons.

Marked as deprecated on 2023-07-14.

Will be removed on 2024-07-14.

but it's displayed as shown in the following image:

Expected Behavior
I expected the entire deprecation message, including all paragraphs and the bulleted list, to be displayed in GraphiQL without any missing information.
Steps To Reproduce
No response
Environment

GraphiQL Version: 2.4.1
OS: mac os
Browser: Chrome Version 114.0.5735.198 (Official Build) (arm64)
react Version: 17.0.2
graphql Version 16.4.0

Anything else?
No response

It is because of this CSS which is designed to make these sections expandable, but this behavior was overlooked for deprecated fields!

@zahraaalizadeh thank you for reporting this issue!
this canary version will fix it for you;
<EMAIL_ADDRESS>
we should release the bugfix by next week :) have a lovely weekend!


Improve README

Hello ٩(｡・ω・｡)
I'm here for my promise https://github.com/dooboolab/react-native-iap/pull/543#issuecomment-504322432 to @hyochan.
I do a complete review of README.md and try to improve the experience both on GitHub and TextEditor, also fixed some wrong code.

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


Fix bug where config isnt updated on reload.

While testing #205, I found that mv reload doesn't update the config change. Looks like its due to calling the wrong load method, so just a simple fix for it.

What does reloadConfig do then?

Should have explains more about it. But basically bukkit's JavaPlugin abstract class provides a default way of loading config.yml. The methods are getConfig(), reloadConfig() and saveConfig() However, MVNP doesn't use these methods to store/load config file, MVNP has its own field MVNPConfiguration and loadConfig() method to do so. So in short the reloadConfig() does nothing in the context of MVNP.

I guess we can merge this now, but for MV 5 I think we should rename the method to reloadConfig as that makes more sense to me.


[v2] Add "real" material navigation drawer

@nolimits4web
Could you please add a "real" material navigation drawer example?

We need this!

What do you mean by "real"?

@nolimits4web
With this app like design

Closing as not the issue, and it is more about customization


[Synapse] Add spark job releated commands based on track2 sdk

Description

[Data Plane] -- Add synapse spark job releated commands based on track2 sdk
See details: #14722
Testing Guide
az synapse spark job -h
az synapse spark session -h
az synapse spark statement -h
commands
History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

Synapse


Error with packer-related setup.sh script

Hi DO curators, first of all, thanks a lot for this resource!
When I follow the packer README and I run packer build . I get the following output:
digitalocean.supabase: output will be in this color.

==> digitalocean.supabase: Creating temporary RSA SSH key for instance...
==> digitalocean.supabase: Importing SSH public key...
==> digitalocean.supabase: Creating droplet...
==> digitalocean.supabase: Waiting for droplet to become active...
==> digitalocean.supabase: Using SSH communicator to connect: <IP_ADDRESS>
==> digitalocean.supabase: Waiting for SSH to become available...
==> digitalocean.supabase: Connected to SSH!
==> digitalocean.supabase: Uploading ./supabase => /root
==> digitalocean.supabase: Provisioning with shell script: ./scripts/setup.sh
    digitalocean.supabase: Hit:1 http://archive.ubuntu.com/ubuntu kinetic InRelease
    digitalocean.supabase: Get:2 http://archive.ubuntu.com/ubuntu kinetic-updates InRelease [118 kB]
    digitalocean.supabase: Get:3 http://archive.ubuntu.com/ubuntu kinetic-backports InRelease [99.9 kB]
    digitalocean.supabase: Get:4 http://archive.ubuntu.com/ubuntu kinetic-updates/main amd64 Packages [322 kB]
    digitalocean.supabase: Get:5 http://archive.ubuntu.com/ubuntu kinetic-updates/main Translation-en [99.3 kB]
    digitalocean.supabase: Get:6 http://archive.ubuntu.com/ubuntu kinetic-updates/main amd64 c-n-f Metadata [9444 B]
    digitalocean.supabase: Get:7 http://archive.ubuntu.com/ubuntu kinetic-updates/restricted amd64 Packages [161 kB]
    digitalocean.supabase: Get:8 http://archive.ubuntu.com/ubuntu kinetic-updates/restricted Translation-en [25.2 kB]
    digitalocean.supabase: Get:9 http://archive.ubuntu.com/ubuntu kinetic-updates/restricted amd64 c-n-f Metadata [552 B]
    digitalocean.supabase: Get:10 http://archive.ubuntu.com/ubuntu kinetic-updates/universe amd64 Packages [283 kB]
    digitalocean.supabase: Get:11 http://archive.ubuntu.com/ubuntu kinetic-updates/universe Translation-en [110 kB]
    digitalocean.supabase: Get:12 http://archive.ubuntu.com/ubuntu kinetic-updates/universe amd64 c-n-f Metadata [9700 B]
    digitalocean.supabase: Get:13 http://archive.ubuntu.com/ubuntu kinetic-updates/multiverse amd64 Packages [6620 B]
    digitalocean.supabase: Get:14 http://archive.ubuntu.com/ubuntu kinetic-updates/multiverse Translation-en [2056 B]
    digitalocean.supabase: Get:15 http://archive.ubuntu.com/ubuntu kinetic-updates/multiverse amd64 c-n-f Metadata [348 B]
    digitalocean.supabase: Get:16 http://archive.ubuntu.com/ubuntu kinetic-backports/universe amd64 Packages [4236 B]
    digitalocean.supabase: Get:17 http://archive.ubuntu.com/ubuntu kinetic-backports/universe Translation-en [1516 B]
    digitalocean.supabase: Get:18 http://archive.ubuntu.com/ubuntu kinetic-backports/universe amd64 c-n-f Metadata [196 B]
    digitalocean.supabase: Get:19 http://security.ubuntu.com/ubuntu kinetic-security InRelease [109 kB]
    digitalocean.supabase: Get:20 http://security.ubuntu.com/ubuntu kinetic-security/main amd64 Packages [209 kB]
    digitalocean.supabase: Get:21 http://security.ubuntu.com/ubuntu kinetic-security/main Translation-en [68.0 kB]
    digitalocean.supabase: Get:22 http://security.ubuntu.com/ubuntu kinetic-security/main amd64 c-n-f Metadata [6980 B]
    digitalocean.supabase: Get:23 http://security.ubuntu.com/ubuntu kinetic-security/restricted amd64 Packages [161 kB]
    digitalocean.supabase: Get:24 http://security.ubuntu.com/ubuntu kinetic-security/restricted Translation-en [25.0 kB]
    digitalocean.supabase: Get:25 http://security.ubuntu.com/ubuntu kinetic-security/restricted amd64 c-n-f Metadata [552 B]
    digitalocean.supabase: Get:26 http://security.ubuntu.com/ubuntu kinetic-security/universe amd64 Packages [176 kB]
    digitalocean.supabase: Get:27 http://security.ubuntu.com/ubuntu kinetic-security/universe Translation-en [76.9 kB]
    digitalocean.supabase: Get:28 http://security.ubuntu.com/ubuntu kinetic-security/universe amd64 c-n-f Metadata [7568 B]
    digitalocean.supabase: Get:29 http://security.ubuntu.com/ubuntu kinetic-security/multiverse amd64 Packages [5984 B]
    digitalocean.supabase: Get:30 http://security.ubuntu.com/ubuntu kinetic-security/multiverse Translation-en [1672 B]
    digitalocean.supabase: Get:31 http://security.ubuntu.com/ubuntu kinetic-security/multiverse amd64 c-n-f Metadata [320 B]
    digitalocean.supabase: Fetched 2102 kB in 5s (407 kB/s)
    digitalocean.supabase: Reading package lists...
==> digitalocean.supabase: E: Could not get lock /var/lib/dpkg/lock-frontend. It is held by process 1579 (apt-get)
==> digitalocean.supabase: E: Unable to acquire the dpkg frontend lock (/var/lib/dpkg/lock-frontend), is another process using it?
    digitalocean.supabase: Get:1 https://download.docker.com/linux/ubuntu kinetic InRelease [48.9 kB]
    digitalocean.supabase: Hit:2 http://security.ubuntu.com/ubuntu kinetic-security InRelease
    digitalocean.supabase: Get:3 http://mirrors.digitalocean.com/ubuntu kinetic InRelease [267 kB]
    digitalocean.supabase: Hit:4 http://mirrors.digitalocean.com/ubuntu kinetic-updates InRelease
    digitalocean.supabase: Hit:5 http://mirrors.digitalocean.com/ubuntu kinetic-backports InRelease
    digitalocean.supabase: Get:6 https://download.docker.com/linux/ubuntu kinetic/stable amd64 Packages [13.2 kB]
    digitalocean.supabase: Fetched 329 kB in 7s (48.1 kB/s)
    digitalocean.supabase: Reading package lists...
==> digitalocean.supabase: E: Could not get lock /var/lib/dpkg/lock-frontend. It is held by process 1579 (apt-get)
==> digitalocean.supabase: E: Unable to acquire the dpkg frontend lock (/var/lib/dpkg/lock-frontend), is another process using it?
==> digitalocean.supabase: /tmp/script_1752.sh: line 25: docker: command not found
==> digitalocean.supabase: Provisioning step had errors: Running the cleanup provisioner, if present...
==> digitalocean.supabase: Destroying droplet...
==> digitalocean.supabase: Deleting temporary ssh key...
Build 'digitalocean.supabase' errored after 1 minute 39 seconds: Script exited with non-zero exit status: 127. Allowed exit codes are: [0]

==> Wait completed after 1 minute 39 seconds

==> Some builds didn't complete successfully and had errors:
--> digitalocean.supabase: Script exited with non-zero exit status: 127. Allowed exit codes are: [0]

==> Builds finished but no artifacts were created.

Is there something I am missing or is there maybe a version of docker or the droplet's Ubuntu which is not fixed to the specific version you have been using?
Cheers!

Okay, it seems like I managed to fix the issue with the following:

Updated Script:
#!/bin/bash
# A Bash script to install Docker and Docker Compose on Ubuntu

# Install Docker

# Update existing packages:
sudo apt-get -y update

sudo apt-get -y install \
    ca-certificates \
    curl \
    gnupg

# Add the GPG key for the official Docker repository to the system:
sudo mkdir -m 0755 -p /etc/apt/keyrings
curl -fsSL https://download.docker.com/linux/ubuntu/gpg | sudo gpg --dearmor -o /etc/apt/keyrings/docker.gpg
# Add the Docker repository to APT sources:
echo \
  "deb [arch=$(dpkg --print-architecture) signed-by=/etc/apt/keyrings/docker.gpg] https://download.docker.com/linux/ubuntu \
  "$(. /etc/os-release && echo "$VERSION_CODENAME")" stable" | \
  sudo tee /etc/apt/sources.list.d/docker.list > /dev/null

# Install Docker:
sudo apt-get -y update
sudo apt-get -y install docker-ce docker-ce-cli containerd.io

# Install Docker Compose:
sudo curl -sSL https://github.com/docker/compose/releases/latest/download/docker-compose-$(uname -s)-$(uname -m) -o /usr/local/bin/docker-compose
sudo chmod +x /usr/local/bin/docker-compose

# Test your installation:
docker-compose versionr

I sometimes get the following errors:
==> digitalocean.supabase: E: Could not get lock /var/lib/dpkg/lock-frontend. It is held by process 1569 (apt-get)
==> digitalocean.supabase: E: Unable to acquire the dpkg frontend lock (/var/lib/dpkg/lock-frontend), is another process using it?

But it terminates with code 0.

Hi @RicoFio, the script is following the official docker documentation (granted we probably could remove the docker-buildx-plugin part as we don't use this).
Heads up that your last line in your script has an extra r at the end to it (docker-compose versionr should be docker-compose version)
The error you're getting is due to the fact that there is another process (apt-get) using the file /var/lib/dpkg/lock-frontend. But this should be released once the process finishes what it needs to do. Unsure as to why you get this as an error as the process should realise the file (I also see it as information, not an error).

Hi @RicoFio, the script is following the official docker documentation (granted we probably could remove the docker-buildx-plugin part as we don't use this). The error you're getting is due to the fact that there is another process (apt-get) using the file /var/lib/dpkg/lock-frontend. But this should be released once the process finishes what it needs to do.
I've seen the error before but I usually rerun packer a second time and it works (haven't fully figured out why this happens, might be I need to increase the droplet size being used by packer, but for now this is an acceptable hack)
Heads up that your last line in your script has an extra r at the end to it (docker-compose versionr should be docker-compose version). Also this is the old way of installing docker compose (more info here).


Adding integration tests for Teams Extensions

Primary set of changes
Adding unit tests to ensure that newer version of Microsoft.Bot.Builder and it's children packages does not break Microsoft.Bot.Builder.Teams package.
It is intentionally failing right now because there was a breaking change from 4.2.0 to 4.2.2 which broke the released Microsoft.Bot.Builder.Teams package.
New integration tests in a separate solution since they need Release build to run (coz they need signed assemblies).
Additional tests
Adding build/Common.props to control the package versions. This allows developers to get a package with higher version than before everytime they compile on local machine.
@cleemullins , @tomlm This PR requires onboarding the new solution to build and running it's tests in CI pipeline to ensure the back compat is not broken.

Thanks @cleemullins for setting the API compat as part of build!! Closing this.


Avoid adding newlines to output by default

When newlines are hardcoded in the library, all user's of this library are forced into that ui-design choice without a way to modify that behaviour. This way, user's can opt to add newlines as they wish ... in their own tools.
For example, in the example below ... I would like not two have two empty lines after "Failed to validate". And even if I did, I'd prefer to have to add them myself for flexibility.
Before:
$ cli-tool-that-does-something
What is your token uuid?
Enter a value: afdsf
Failed to validate input string: afdsf

Enter a value: 2343242

After:
$ cli-tool-that-does-something
What is your token uuid?
Enter a value: afdsf
Failed to validate input string: afdsf
Enter a value: 2343242

I took the liberty to remove other newlines.

@tcnksm any chance you could provide some feedback on this? The newlines are definitely odd and would be welcome if they were removed.


Add Power Automate

Language name
Power Automate  is a visual scripting language, but you can share your flows in a text format.
URL of example repository
I don't think there's an official sample repo, but i found this. But this repo has .zip files that are used to export/import the flow on the web portal, desktop version only supports ctrl+c ctrl+v'ing the flows.
URL of syntax highlighting grammar
I don't really know if there are any syntax highlighters, but i guess this works?
Most popular extensions
From what i know, they don't have their own extension, but the flow can be distributed in any format, most popular are .json and .txt though.
Detected language
.txt and .json don't usually get detected automatically.

I assume your intent is to have repositories categorised as "Power Automate", rather than interpret well-formed JSON files as some format other than JSON (assuming these files aren't concealed inside a ZIP archive, where Linguist isn't going to notice or scan them).
Remember, Linguist's responsibility is to identify file formats and programming languages on a file-by-file basis. Classifying repositories by "project type" (i.e., a Power Automate project) is well beyond Linguist's scope, and is something that topics are better suited to.

Alr, thank you!


Questions about the number of lanes

In opensource datasets, there are some limits about the maximum number of lanes.(Tusimple is 5, CULane is 4...)
When I train the lanedetection algorithm on my datasets, I have preprocessed my data annotation format as same as Tusimple format, but my maximum number of lanes is larger than 5 (maybe 8 or 9). What will happen if I directly use your codebase to train on my dataset without any modification？
If I need to modify some codes to fit my number of lanes, can you tell me where should I modify?
Thanks a lot :)

Args related to TuSimple max lane numbers are in config.yaml, from L121-L126. Some methods has other parameters that may be influenced, for instance,
LSTR restricts a max prediction num 7: https://github.com/voldemortX/pytorch-auto-drive/blob/4c7e1b9eec592f0bd0637133cd34e9969319df18/torchvision_models/lane_detection/lstr.py#L32
And the bottleneck in segmentation lane existence branch (in this case 5) should not fall too far from MAX_LANE:  https://github.com/voldemortX/pytorch-auto-drive/blob/4c7e1b9eec592f0bd0637133cd34e9969319df18/torchvision_models/common_models.py#L277

That's about all I can recall that depends on max lane numbers. One thing is worth noting though. TuSimple has 5 lanes, but segmentation methods must see it as 6 (left and right).

Thank you very much for your reply.
Actually, I want to train LSTR on my dataset. As mentioned above, I have preprocessed my data annotation format as same as Tusimple json file format. But I found there are two more annotation files in your codebase, segmentation labels, list6_train.txt.
In your extra two annotation files, it seems like have annotation about which lane from left to right. But I do not have this annotation, I only know there are how many lanes in one images. If I just give them the indexes in range(0, max num of lanes in this img) from left to right, and then generate segmentation label map, every thing will be run normally?
Further more, what's the difference between annotation 0 1 1 1 1 0 0 and 0 0 1 1 1 1 0 in LSTR?

I don't think you need segmentation labels for LSTR?

非常感谢您的回复。
小鸟，我想在我的数据集上训练 LSTR。上面，我已经将我的注释格式与 Tusimple 文件格式进行了数据格式。但是我发现您的代码库中还有两个注释文件，翻译标签，list6_train.txt。
在你额外的两个注释文件中，似乎有关于从左到右哪条车道的注释。但是我没有这个注释，我只知道一张图片有多少条车道。如果我只是给他们从左到右范围里面的索引（0，这个img中的最大数据），然后生成翻译标签图，一切正常运行吗？
另外，LSTR 中的注释 0 1 1 1 1 0 0 和 0 0 1 1 1 1 0 之间有什么区别？

Hello, can i ask how do you label your tusimple dataset? What tools do you use? How to generate the JSON file required by tusimple because it requires isometric annotation.  thank you for your reply !

@August-en Hi! Was your LSTR training problem resolved? In fact the current PytorchAutoDrive provides a config-based framework and it should be much easier to integrate a new dataset.
Btw, could you give some insight on dataset labeling for @dreamlover0 , thank you very much.

非常感谢您的回复。
小鸟，我想在我的数据集上训练 LSTR。上面，我已经将我的注释格式与 Tusimple 文件格式进行了数据格式。但是我发现您的代码库中还有两个注释文件，翻译标签，list6_train.txt。
在你额外的两个注释文件中，似乎有关于从左到右哪条车道的注释。但是我没有这个注释，我只知道一张图片有多少条车道。如果我只是给他们从左到右范围里面的索引（0，这个img中的最大数据），然后生成翻译标签图，一切正常运行吗？
另外，LSTR 中的注释 0 1 1 1 1 0 0 和 0 0 1 1 1 1 0 之间有什么区别？

Hello, can i ask how do you label your tusimple dataset? What tools do you use? How to generate the JSON file required by tusimple because it requires isometric annotation. thank you for your reply !

@August-en Hi! Was your LSTR training problem resolved? In fact the current PytorchAutoDrive provides a config-based framework and it should be much easier to integrate a new dataset. Btw, could you give some insight on dataset labeling for @dreamlover0 , thank you very much.

Thanks for your reply. Finally, I trained the LSTR algorithm with the codes in the paper. But I'm following your latest paper BezierLaneNet.

@August-en Hi! Was your LSTR training problem resolved? In fact the current PytorchAutoDrive provides a config-based framework and it should be much easier to integrate a new dataset. Btw, could you give some insight on dataset labeling for @dreamlover0 , thank you very much.

Thanks for your reply. Finally, I trained the LSTR algorithm with the codes in the paper. But I'm following your latest paper BezierLaneNet.

非常感谢您的回复。
小鸟，我想在我的数据集上训练 LSTR。上面，我已经将我的注释格式与 Tusimple 文件格式进行了数据格式。但是我发现您的代码库中还有两个注释文件，翻译标签，list6_train.txt。
在你额外的两个注释文件中，似乎有关于从左到右哪条车道的注释。但是我没有这个注释，我只知道一张图片有多少条车道。如果我只是给他们从左到右范围里面的索引（0，这个img中的最大数据），然后生成翻译标签图，一切正常运行吗？
另外，LSTR 中的注释 0 1 1 1 1 0 0 和 0 0 1 1 1 1 0 之间有什么区别？

Hello, can i ask how do you label your tusimple dataset? What tools do you use? How to generate the JSON file required by tusimple because it requires isometric annotation. thank you for your reply !

Sorry for the late reply. I annoate the lane line as polygon, which is same as the method of instance segmentation. Then I fit each polygon to a curve line using least square fitting method. Finally, I'm sampling the points on the line with equally y-axis distance.

@August-en Thank you for sharing your labeling strategies!

非常感谢您的回复。
小鸟，我想在我的数据集上训练 LSTR。上面，我已经将我的注释格式与 Tusimple 文件格式进行了数据格式。但是我发现您的代码库中还有两个注释文件，翻译标签，list6_train.txt。
在你额外的两个注释文件中，似乎有关于从左到右哪条车道的注释。但是我没有这个注释，我只知道一张图片有多少条车道。如果我只是给他们从左到右范围里面的索引（0，这个img中的最大数据），然后生成翻译标签图，一切正常运行吗？
另外，LSTR 中的注释 0 1 1 1 1 0 0 和 0 0 1 1 1 1 0 之间有什么区别？

Hello, can i ask how do you label your tusimple dataset? What tools do you use? How to generate the JSON file required by tusimple because it requires isometric annotation. thank you for your reply !

Sorry for the late reply. I annoate the lane line as polygon, which is same as the method of instance segmentation. Then I fit each polygon to a curve line using least square fitting method. Finally, I'm sampling the points on the line with equally y-axis distance.

@dreamlover0 Here is the answer.

Hi @voldemortX, I am currently using your bezierlanenet model to try and train on my own dataset. I am trying to use initially Tusimple Data format and I have a two part question for you, namely:

Could you please explain how the numbering of Lane-ids in the 'list6_train.txt' e.g. is done? For my dataset, supposedly there are max. 5 lanes(6 lane markings) and in the current frame, I see only 3 lanes and my Ego vehicle is on the leftmost lane, how should I number the Ids?

Should it be like: 1 1 1 1 0 0 (3 lanes, ego vehicle on left most lane) and 0 0 1 1 1 1 (3 lanes, ego on the right most lane) ?
This Tusimple format seems to me to a bit unclear in comparison to the Culane format, where it is distinct that the Ids are simply lane markings from left to right.

The Culane *.txt format for the above lane markings is easier to understand and use in training. But I am unclear about the license for the Culane config and dataset format.
Can I use the Culane dataset format to train your bezierlanenet model on a new dataset for commercial purposes?
And also, do I need semantic labels, besides bezier labels for training If I use Culane dataset format and config files?

@aspandey2021 For your first question. It is common practice to number your ego lanes as 1 2, left right immediate as 3 4, and so on. If you are using a dataset purely within 4 lanes, 1234 from left to right may also work. This thing is flexible, as long as your network can learn the distinctions.
For your second question, the CULane dataset has been out there for almost 5 years, it should be okay to use it commercially, if you are unsure you can check the CULane official repo and raise an issue there.
As for training BézierLaneNet, everything in this repo and related to the paper are considered free open-source techniques, and can be used for commercial purposes.
If you are training BézierLaneNet, you don't need the semantic classification of lanes into different classes. However, you can find in this repo that all our txts are labeled with lane existence for consistency. You can simply append all 0 or all 1 for your own dataset, or you can not read them if you implemented a custom dataset Class for your custom dataset. Although BézierLaneNet does require a binary segmentation on CULane as auxiliary supervision. You could just classify all lane areas to the same class in the segmentation label. Without this auxillary loss, the performance can be notably worse on CULane, while on easier datasets such as TuSimple/LLAMAS, you can remove this auxiliary loss without hurting performance much.

Thank you so much for your prompt reply @voldemortX!
So regarding my plan to use a custom dataset for training and regarding your answers to my questions, I would like to summarize a few points:

I understood about the numbering strategy for the lane Ids as you mentioned that it is flexible. Thank you for that.

As you mentioned that one doesn't need semantic classification for lane-existence in the txt file for Bezierlanenet, however I got an error when I didn't use these values of 1 or 0 in the txts (File not found error). So I think I would have to use these values anyhow (or is there a way in the code to completely ignore the list6.txt for Training?).
My concern is to label and recognise the Highway exits and ramps also. And my dataset will have variable number of lanes (going upto 7-8 sometimes). So do you think that using a variable number of '1's in txt file,for each line present in a ground truth frame, would be a good strategy to train the model?

Regarding your point that Bezierlanenet requires binary auxillary segmentation only for Culane, I tried without it for Tusimple (without segGT6 folder in ROOT) and it gave me an error (that masks could not be loaded). So I think this is also needed for a new custom dataset ?(please correct me if I am wrong)

Finally for Inference, I tried an input image in 2 formats : format 1 (Tusimple with 1280x720) and format 2 (1920x1440). The predicted lane lines match quite good for Tusimple resolution but for the higher resolution, they appear very small in the image(due to the higher resolution I guess). Do you think this will improve when I train the model on format 2, because I want that the model is able to predict on different incoming resolutions in the end?
Or is there a way to control that through the config file ?

I am sorry If I asked too much, thank you for your patience in advance :)!

@aspandey2021 Thanks for your interest in our works.

As you mentioned that one doesn't need semantic classification for lane-existence in the txt file for Bezierlanenet, however I got an error when I didn't use these values of 1 or 0 in the txts (File not found error). So I think I would have to use these values anyhow (or is there a way in the code to completely ignore the list6.txt for Training?).

The dataset loading scheme is unfortunately coded-in. You will need to adjust that yourself. The easiest way without touching too much code will be changing the train.txt. For instance we split the image file name by space, you can add a space to every line or mod the code:
https://github.com/voldemortX/pytorch-auto-drive/blob/9c62c0a721d11c8ea6bf312ecf1c7b238a54dcda/utils/datasets/lane_as_bezier.py#L101
Frankly speaking, for non-segmentation methods, you only need the filenames in train.txt, the 0 0 0 1 1 1 can be removed.

My concern is to label and recognize the Highway exits and ramps also. And my dataset will have variable number of lanes (going upto 7-8 sometimes). So do you think that using a variable number of '1's in txt file,for each line present in a ground truth frame, would be a good strategy to train the model?

If you are using BézierLaneNet, you only have to adjust max_lane in configs. It treats all lanes as the same class. For improving performance, you might want to add an classification of lane type, but that will require some efforts & coding.

Regarding your point that Bezierlanenet requires binary auxillary segmentation only for Culane, I tried without it for Tusimple (without segGT6 folder in ROOT) and it gave me an error (that masks could not be loaded). So I think this is also needed for a new custom dataset ?(please correct me if I am wrong)

Yes you will need a new Dataset Class coded for this, which does not load seg labels. A new loss Class will also be needed that do not compute the seg loss. And you might want to also remove the seg branch in model config to save some training time. Although a simpler hack will be providing all 0 seg maps, and set the seg loss weight to 0.

Finally for Inference, I tried an input image in 2 formats : format 1 (Tusimple with 1280x720) and format 2 (1920x1440). The predicted lane lines match quite good for Tusimple resolution but for the higher resolution, they appear very small in the image(due to the higher resolution I guess). Do you think this will improve when I train the model on format 2, because I want that the model is able to predict on different incoming resolutions in the end?
Or is there a way to control that through the config file ?

The TuSimple models in this repo are all trained in 640x360, and will inference at 640x360, then rescale the results to your desired resolution. There are multi-scale augmentation in training for BézierLaneNet, so you don't really need to worry about this:
https://github.com/voldemortX/pytorch-auto-drive/blob/9c62c0a721d11c8ea6bf312ecf1c7b238a54dcda/configs/lane_detection/common/datasets/train_level1b_360.py#L8
The reason for a smaller-looking result probably is in your config the desired resolution is set as 1280x720 here:
https://github.com/voldemortX/pytorch-auto-drive/blob/9c62c0a721d11c8ea6bf312ecf1c7b238a54dcda/configs/lane_detection/bezierlanenet/resnet18_tusimple_aug1b.py#L51


Structured Questions in Determination Messages

The Problem
Round leads have to manage a lot of communications with applicants and reviewers, and it can be hard to track which questions have been answered and resolved, and which questions are still open.
A Solution
Having a structured way to indicate, reference, and open/close questions in a "request for more information" would go towards streamlining internal discussions.
One UI possibility is the following.

And as responses from applicants come in, it should be possible for round leads to mark a question as (e.g.)

resolved/closed
unresolved/open
waiting for response [default state]

Requirements

Round leads have to be able to add many questions (at least up to 10)
Questions should be numbered for future reference

Open questions

Is it more like a GitHub issue (open/closed) or more like a Google Doc comment thread (resolved or unresolved)?
Are questions anchored to the text in the application (think: editor flow, peer reviewing, Hypothes.is-style annotation) or are there general questions?
Should their status be internal-only or also visible to the applicant?
How to display the change history?

Priority

Low priority (annoying, would be nice to not see)

Affected roles

Staff
Reviewers

Ideal deadline
New review and determination forms ship on July 1, but this doesn't need to be part of that release.

This is a rough idea and I'm curious to hear if it could be useful for reviewers.

i like that a lot! thanks for putting this together.

Related to #711

Closing because related to https://github.com/HyphaApp/hypha/issues/2809


Prefetch Count Default

For Prefetch Count, the default value is zero, but what does a value of zero mean?

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

ID: 1752a37b-b40a-5613-3416-fd604b281dbc
Version Independent ID: f088004f-f80e-ba5c-e08e-c76ac395c809
Content: Azure Service Bus output bindings for Azure Functions
Content Source: articles/azure-functions/functions-bindings-service-bus-output.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Hello @swettstein,  Thank you for your feedback! We will review and update as appropriate.

Hello @swettstein, the value zero means that prefetch is turned off by default. You can find more information here regarding why prefetch is turned off by default.

Perfect. Thank you!


fix(next): support app/favicon.ico

#448  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hi-ogawa and the rest of your teammates on  Graphite


⚠ Bofrost.be has degraded performance

In e4749b5, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1655 ms

Resolved: Bofrost.be performance has improved in 3f60363 after 13 minutes.


Add page type and attributes permissions handling

WIP add new configuration menu

Add missing permissions

Changed filtering to one based on arrays of permissions

Add hasAnyPermissions and hasAllPermissions functions

Simplify getConfigMenuItemsPermissions function

Simplify routing logic

I want to merge this change because...

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://master.staging.saleor.cloud/graphql/

Task linked: SALEOR-3164 [Permissions] Page Type & Attribute - user cannot see configuration menu.


Intermittent errors from code coverage mapping (Transformation error, An error occurred while trying to read the map file, ENAMETOOLONG)

Expected Behavior
Produce coverage report
Observed Behavior
I was banging my head against a wall trying to get nyc to work with my project then suddenly it was working flawlessly, and I couldn't figure out why. After some messing around I realized that if I modified a file I'd get the error (shown below) for that one file.
I eventually figured out that if I open package.json and modified the nyc config and set "sourceMap": false and ran the tests they'd run fine (no errors). Then if I set it back to "sourceMap": true it would produce coverage as expected.
I've also deleted the coverage and .nyc_output directories and it doesn't seem to cause the problem to come back. They are re-created as expected when I run npm test
Error Output
Transformation error for /src/controllers/root/root.controller.ts ; return original code [removed long base 64 string, presumably source maps]

An error occurred while trying to read the map file at /controllers/root/data:application/json;charset=utf-8;base64,[removed long base 64 string, presumably source maps]

Error: ENAMETOOLONG: name too long, open '/controllers/root/data:application/json;charset=utf-8;base64,[removed long base 64 string, presumably source maps]

Code (or Repository) that Reproduces Issue
The project is a TypeScript Node server using Jasmine for tests.
npm script (package.json)
"test": "nyc jasmine --config=jasmine.json",

nyc config (package.json)
  "nyc": {
    "extension": [
      ".ts"
    ],
    "exclude": [
      "**/*.spec.ts",
      "**/*.mock.ts",
      "src/test/**/*.*"
    ],
    "reporter": [
      "html"
    ],
    "require": [
      "ts-node/register",
      "source-map-support/register"
    ],
    "sourceMap": true,
    "all": false
  },

jasmine.json
{
  "spec_dir": "./src",
  "spec_files": [
    "**/*.spec.ts"
  ],
  "helpers": [
    "test/test-requirements.ts"
  ],
  "stopSpecOnExpectationFailure": true,
  "random": false
}

Forensic Information
Operating System: OSX 10.11.6
Environment Information:
Node 9.2.0
nyc 12.0.1
(full details)

Update...
I'd like to find an answer to my problem, but I've realized the intermittent nature of it seems to be a bug in the caching in nyc. If I go into the nyc config and set "cache": false the error happens consistently.
Additionally, after looking at the coverage output it doesn't appear to be correctly aligned to the source TypeScript.
My guess is that when toggling source map on and off it's producing coverage without source map and storing that in cache, but not clearing cache when there is an nyc config changes (when I turn source map back on).

@itslenny whenever the issue is intermittent like this, it tends to be the order that the coverage files are loaded and merged from .nyc_output/. TypeScript seems to love bumping into issues related to this.
For starters give the 12x release a try, which swaps out Babel 6 for Babel 7.

@bcoe Thanks for the response.
I was already on 12.0.1
Also, I figured out that if I set cache:false in the nyc config it stops being intermittent and is just a consistent error.

I'm seeing the exact same issue. I'm using jasmine with TypeScript and a very similar setup. Any additional insight into the issue?

I figured it out for my project. The issue is that there was a double encoding of source maps. Source maps were being generated as external, then they were inlined and finally read as if they were external. I fixed the issue by using the inlineSourceMap: true flag in my tsconfig.json. I then changed the nyc config to:
  "nyc": {
    "extension": [
      ".ts"
    ],
    "exclude": [
      "**/*.spec.ts"
    ],
    "reporter": [
      "html"
    ],
    "sourceMap": true,
    "cache": true,
    "require": [
      "ts-node/register"
    ]
  },```

I did test with `cache: false`.

Please see https://github.com/Microsoft/TypeScript/issues/24993, I'm interested in overhauling our TypeScript support and would love your feedback and requirements.

@niedzielski What are your tsconfig.json and nyc settings?

@devonhumes we're using ES6 with Babel, not TypeScript. Here's the patch (with sourceMap disabled): https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Popups/+/440904/7/.nycrc.json.

@devonhumes It's working for you? What version(s) are you running?
Here's what I've tried. Let me know if you have any other ideas...
 I tried adding "inlineSourceMap": true, but it didn't change anything. Still getting the "name too long" errors with the base64 sourcemaps in the path.
I tried every combination of "inlineSourceMap": true and "sourceMap": true (true/false) and never saw a difference.
I also tried including "source-map-support/register" in nyc config (require) which I saw in some examples, but it doesn't seem to make a difference with/without it.
The only way it works at all is if I set sourceMap: false in nyc config, but then the line numbers are completely wrong in the resulting coverage output.
full nyc config
  "nyc": {
    "extension": [
      ".ts"
    ],
    "exclude": [
      "**/*.spec.ts",
      "**/*.mock.ts",
      "src/test/**/*.*"
    ],
    "reporter": [
      "html"
    ],
    "require": [
      "ts-node/register",
      "source-map-support/register"
    ],
    "sourceMap": true,
    "cache": false,
    "all": false
  },

Figured it out. I had "mapRoot": "" in my tsconfig which is not valid with "inlineSourceMap": true. nyc seems to swallow that compiler error and continues to output the "name too long" error.
I found the problem by just running tsc directly which told me that I'm not allowed to use mapRoot with inlineSourceMap and refuses to build. Once I removed mapRoot nyc started producing coverage reports as expected.

@itslenny Would you mind sharing your final config please? I'm struggling with the same problems!


Documentation conflicting terms

On this page relating to the new keybase teams feature: https://keybase.io/docs/teams/design
The terms implied admin and implicit admin seem to be used interchangeably, causing unneeded confusion.
Personally, I prefer the term implicit admin.

OK, should be consistent now as implicit. Thanks!!!


Add in-memory log caching and a logs dialog

Don't really like these being in a dialog... but I couldn't come up with another way to add them to the drawer while keeping the light theme.

lgtm :shipit:


react vendor files

react is not part of the vendor files, it is bundled.
on the other hand, react is listed as a bower component in the gulpfile, and bower isn't even a part of this repository (I prefer npm myself).

If you write a pull request, I'm happy to merge it. I won't fix this myself as I don't use browserify anymore. Switched to webpack.

I noticed similar problems when attempting to use this project. I intended to send PR's, but I decided to go the webpack route as well. I ended up creating https://github.com/badsyntax/react-seed which is inspired by this project.

Nice, I have a similar project that I do maintain: https://github.com/tcoopman/boilerplate-webpack-react


Preprocessing ascii script check is unsuported for python 3.6

Issues
I am using Python 3.6 to run DeepFace version 0.0.85 (which supported Python >= 3.5.5 from PyPi). However, I got an error as above:
My code was like
from deepface import DeepFace
import deepface
import sys

print(deepface.__version__)
print(sys.version)
print(DeepFace.analyze('image1.png'))

Trace
0.0.85
3.6.9 (default, Mar 10 2023, 16:46:00)
[GCC 8.4.0]
Traceback (most recent call last):
  File "/home/ubuntu/.../index.py", line 4, in <module>
    print(DeepFace.analyze('image1.png'))
  File "/usr/local/lib/python3.6/dist-packages/deepface/DeepFace.py", line 222, in analyze
    silent=silent,
  File "/usr/local/lib/python3.6/dist-packages/deepface/modules/demography.py", line 126, in analyze
    expand_percentage=expand_percentage,
  File "/usr/local/lib/python3.6/dist-packages/deepface/modules/detection.py", line 79, in extract_faces
    img, img_name = preprocessing.load_image(img_path)
  File "/usr/local/lib/python3.6/dist-packages/deepface/modules/preprocessing.py", line 47, in load_image
    if img.isascii() is False:
AttributeError: 'str' object has no attribute 'isascii'

After a quick research, I know that the str.isascii() is only supported for Python >= 3.7, which part is missing on Python 3.6.
Environment
Running on

Ubuntu 18.04 (arch64)
Python 3.6.9
DeepFace 0.0.85

Solution

Re-implement an check ascii on string instead of using a built-in function (or change the required Python >= 3.7).

Right, we should enforce min python 3.7.

Ps: A little upset since I am trying to use the library on Jetson Nano (which only supported Python 3.6) and is not operating as expected. Hopefully, the older version can alter this issue.

If you need to use the library on jetson, and it has to use 3.6, then we should find a workaround.

Just updated setup.py in this commit - https://github.com/serengil/deepface/commit/c09e94ecaa5d3a5ed97182ed9daaad98020a1ac6


New indexes CORD-19 04/17 data drop

@edwinzhng  @rodrigonogueira4
Minor concern is that for paragraph index, size went up from 3.4G to 4.7G. The only relevant commit I can see that would impact this is: https://github.com/castorini/anserini/commit/cc3b0a3bd2eb83d8f4802fe2e57e37187586a5be where we us pmc json is available.
Thoughts?

BTW, the full-text index (which we don't use anyway) is down from 2.2G to 3.3G; this is easily explained by https://github.com/castorini/anserini/commit/1a030d83ba6d5482c2eb2c9f8880c3345bcdd629 - we now properly parse out contents(), whereas before we just glommed on the entire JSON.

@edwinzhng @rodrigonogueira4
Minor concern is that for paragraph index, size went up from 3.4G to 4.7G. The only relevant commit I can see that would impact this is: cc3b0a3 where we us pmc json is available.
Thoughts?

Yeah, that seems right, but @edwinzhng might note something else...

Okay, I've taken a detailed look at this issue - it's basically this article: https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4896250/
Run this and you'll see ;)
$ grep 'ij3ncdb6' metadata.csv

Basically, the "abstract" is really really long:
$ grep 'ij3ncdb6' metadata.csv | wc
       1   26358  180437

So what happens is that the "abstract" gets appended to every paragraph... which results in an explosion of documents. This doesn't happen as much with the pdf_json because the JSON is shorter.
Compare:
$ ls -l comm_use_subset/pmc_json/PMC4896250.xml.json
-rw-r--r--  1 jimmylin  staff  3932453 Apr 10 16:08 comm_use_subset/pmc_json/PMC4896250.xml.json
$ ls -l comm_use_subset/pdf_json/a5293bb4f17ad25a72133cdd9eee8748dd6a4b8d.json
-rw-r--r--  1 jimmylin  staff  1628393 Apr 10 16:06 comm_use_subset/pdf_json/a5293bb4f17ad25a72133cdd9eee8748dd6a4b8d.json

Okay, I'm satisfied that this isn't a bug now...

I'll wait for @edwinzhng for a quick sanity check before merging.


[Snyk] Upgrade @opentelemetry/exporter-metrics-otlp-proto from 0.49.1 to 0.50.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @opentelemetry/exporter-metrics-otlp-proto from 0.49.1 to 0.50.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2024-04-03.

Release notes

Package name: @opentelemetry/exporter-metrics-otlp-proto

0.50.0 - 2024-04-030.50.0
💥 Breaking Change

fix(exporter-*-otlp-grpc)!: lazy load gRPC to improve compatibility with @ opentelemetry/instrumenation-grpc #4432 @ pichlermarc

Fixes a bug where requiring the gRPC exporter before enabling the instrumentation from @ opentelemetry/instrumentation-grpc would lead to missing telemetry
Breaking changes, removes several functions and properties that were used internally and were not intended for end-users

getServiceClientType()

this returned a static enum value that would denote the export type (SPAN, METRICS, LOGS)

getServiceProtoPath()

this returned a static enum value that would correspond to the gRPC service path

metadata

was used internally to access metadata, but as a side effect allowed end-users to modify metadata on runtime.

serviceClient

was used internally to keep track of the service client used by the exporter, as a side effect it allowed end-users to modify the gRPC service client that was used

compression

was used internally to keep track of the compression to use but was unintentionally exposed to the users. It allowed to read and write the value, writing, however, would have no effect.

feat(api-events)!: removed domain from the Events API #4569 @ martinkuba
fix(api-events)!: renamed EventEmitter to EventLogger in the Events API #4569 @ martinkuba
feat(api-logs)!: changed LogRecord body data type to AnyValue and AnyValueMap types #4575 @ martinkuba

🚀 (Enhancement)

feat(instrumentation-xhr): optionally ignore network events #4571 @ MustafaHaddara
refactor(instrumentation-http): use exported strings for semconv #4573 @ JamieDanielson
perf(instrumentation-http): remove obvious temp allocations #4576 @ Samuron
feat(sdk-node): add HostDetector as default resource detector #4566 @ maryliag
feat(api-events): added data field to the Event interface #4575 @ martinkuba

🐛 (Bug Fix)

fix(exporter--otlp-): use parseHeaders() to ensure header-values are not 'undefined' #4540

Fixes a bug where passing undefined as a header value would crash the end-user app after the export timeout elapsed.

fix(sdk-logs): ensure default resource attributes are used as fallbacks when a resource is passed to LoggerProvider.

📚 (Refine Doc)

docs(instrumentation-http): document semantic conventions and attributes in use. #4587 @ JamieDanielson

0.49.1 - 2024-02-290.49.1
🐛 (Bug Fix)

fix(instrumentation): don't add @ opentelemetry/api-logs as a peerDependency #4515 @ pichlermarc

    from @opentelemetry/exporter-metrics-otlp-proto GitHub release notes

Commit messages

Package name: @opentelemetry/exporter-metrics-otlp-proto

5231aa2 chore: prepare release 1.23.0/0.50.0 (#4601)
928796d feat(sdk-metrics): added synchronous gauge to SDK (#4565)
d66e1d7 Add `data` field to the Event interface (#4575)
f3aedb7 chore(deps): bump express from 4.17.3 to 4.19.2 (#4591)
b418d36 perf(instrumentation-http): remove obvious temp allocations (#4576)
9a5688e chore(deps): update dependency chromedriver to v123 (#4577)
1a0fb4f docs(instr-http): document semantic conventions and attributes in use (#4587)
900b7d8 feat(opentelemetry-instrumentation-xhr): optionally ignore network events (#4571)
f6a075b Rename EventEmitter to EventLogger in the Events API (#4568)
97af8e6 feat(sdk-node): add `HostDetector` as default resource detector (#4566)
fec6286 Remove domain from the Events API (#4569)
5fb65b4 perf(export): do not allocate arrays if resource has no pending async attributes (#4550)
f9a0d3e refactor(instr-http): use exported strings for semconv (#4573)
6547440 fix(sdk-logs): ensure default resource attributes are used as fallbacks when a resource is passed to LoggerProvider (#4564)
5489797 chore(changelog): move entry that accidentally ended up in a previous version (#4559)
aabd1a9 SBOM workflow using "npm sbom" (#4521)
3a426e8 fix(exporters): use parseHeaders() to ensure header-values are not 'undefined' (#4540)
1b4999f docs: spelling corrections (#4530)
7fa2c84 chore(deps-dev): bump ip from 1.1.8 to 1.1.9 (#4495)
63d74cd chore(renovate): remove import-in-the-middle from all-patch group (#4513)
5a033e5 fix(exporter-*-otlp-grpc)!: lazy load gRPC (#4432)
75bd723 fix: ConsoleMetricExporter Should Not Export Shallowly (#4522)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

https://github.com/DTS-STN/secure-client-hub/pull/631


"Connect to running Jupyter server" launches the wrong ipython kernels when I'm using a virtualenv.

Environment data

VS Code version: 1.36.1
Extension version (available under the Extensions sidebar): 2019.6.24221
OS and version: Ubuntu 19.04
Python version (& distribution if applicable, e.g. Anaconda): Ubuntu 19.04-packages 3.7
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pipenv

Expected behaviour
sys.path in the ipython repl should be {my virtualenv}/bin/python
Actual behaviour
sys.path in the ipython repl is /usr/bin/python3 and I can't import any libraries that are installed in my virtualenv.
Steps to reproduce:
Setup

Create a pipenv environment: pipenv --three should do this I think.
Install jupyter and ipython in the environment.
Run pipenv shell (this is a wrapper around venv/activate.sh that launches a subshell)

Intermission / show other stuff is working properly

Run jupyter notebook. Launch a new notebook. I can choose between [my virtualenv] and python3 as kernels.
Launch vs code in your project directory. You can do this in the virtualenv activated or outside; it makes no difference.
Make sure vs code picks up your venv's python interpreter in the status bar.
With the default setting of "launch a local jupyter server", verify that running something in the ipython repl uses your venv's python. (this works great, nice job guys :) )

the actual problem

Change the vs code jupyter connection setting to "connect with URI". Paste the connection uri from the jupyter notebook server you started in 4. Start a new ipython repl.
Observe that ipython is using your system python and is oblivious to your virtualenv.

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
User belongs to experiment group 'ShowPlayIcon - start'
Starting Jedi Python language engine.
##########Linting Output - mypy##########
src/lp8_features_jarrad.py:219: error: invalid syntax

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
-1565003180007.log
Thanks for the great extension guys. :)

I believe this is a problem with jupyter. The remote jupyter server doesn't have the environment variables for the virtual environment.
What happens if you run jupyter notebook from outside of the virtual env (from the command line) and then try to select the virtual env's kernel? Does that work in normal jupyter?

This will be solved when we complete #3123


fix(connlib): prevent routing loops on windows

In connlib, traffic is sent through sockets via one of three ways:

Direct p2p traffic between clients and gateways: For these, we always explicitly set the source IP (and thus interface).
UDP traffic to the relays: For these, we let the OS pick an appropriate source interface.
WebSocket traffic over TCP to the portal: For this too, we let the OS pick the source interface.

For (2) and (3), it is possible to run into routing loops, depending on the routes that we have configured on the TUN device.
In Linux, we can prevent routing loops by marking a socket 0 and repeating the mark when we add routes 1. Packets sent via a marked socket won't be routed by a rule that contains this mark. On Android, we can do something similar by "protecting" a socket via a syscall on the Java side 2.
On Windows, routing works slightly different. There, the source interface is determined based on a computed metric 3 4. To prevent routing loops on Windows, we thus need to find the "next best" interface after our TUN interface. We can achieve this with a combination of several syscalls:

List all interfaces on the machine
Ask Windows for the best route on each interface, except our TUN interface.
Sort by Windows' routing metric and pick the lowest one (lower is better).

Thanks to the abstraction of SocketFactory that we already previously introduced, Integrating this into connlib isn't too difficult:

For TCP sockets, we simply resolve the best route after creating the socket and then bind it to that local interface. That way, all packets will always going via that interface, regardless of which routes are present on our TUN interface.
UDP is connection-less so we need to decide per-packet, which interface to use. "Pick the best interface for me" is modelled in connlib via the DatagramOut::src field being None.

To ensure those packets don't cause a routing loop, we introduce a "source IP resolver" for our UdpSocket. This function gets called every time we need to send a packet without a source IP.
For improved performance, we cache these results. The Windows client uses this source IP resolver to use the above devised strategy to find a suitable source IP.
In case the source IP resolution fails, we don't send the packet. This is important, otherwise, the kernel might choose our TUN interface again and trigger a routing loop.

The last remark to make here is that this also works for connection roaming. The TCP socket gets thrown away when we reconnect to the portal. Thus, the new socket will pick the new best interface as it is re-created. The UDP sockets also get thrown away as part of roaming. That clears the above cache which is what we want: Upon roaming, the best interface for a given destination IP will likely have changed.
Fixes: #5955.

On Windows, routing works slightly different. There, the source interface is determined based on a computed metric. (TODO: Reference source blog post here. To prevent routing loops on Windows, we thus need to find the "next best" interface after our TUN interface. We can achieve this with a combination of several syscalls:

@conectado Can you link the blog post here that explains the metric stuff?

One thing I'm noting here to test is what happens if I enable the weak host model.
This is not enabled by default but it might lead to problems, since packets can be routed to different adapters even if the source IP doesn't match!
AFAIK it's not recommended to use it but at least we need to document it if it breaks Firezone.

@thomaseizinger feel free to make a round of review, want to do a last round of manual testing which I will do on Monday but it's basically at the point where I want to merge

Thank you @thomaseizinger ! Just tested this manually, it's ready to merge! Let's goo