[video_player_web] iOS Shows Video Controls (even when disabled in source)

When an iOS device is in "Low Power Mode", the Video Player in Safari will show a big Play button that interferes with the normal lifecycle and the user supplied styling of the video_player_web Widget:

Normal
Low power

Note that the controls property is explicitly disabled in the plugin source.
Can this button be removed or styled out?
Steps to Reproduce

Run flutter create bug.
Add a dependency on video_player: ^2.1.10
Update main.dart as follows:

main.dart

import 'package:flutter/material.dart';
import 'package:video_player/video_player.dart';

void main() {
  runApp(
    MaterialApp(
      home: _App(),
    ),
  );
}

class _App extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Video player example'),
      ),
      body: _BumbleBeeRemoteVideo(),
    );
  }
}

class _BumbleBeeRemoteVideo extends StatefulWidget {
  @override
  _BumbleBeeRemoteVideoState createState() => _BumbleBeeRemoteVideoState();
}

class _BumbleBeeRemoteVideoState extends State<_BumbleBeeRemoteVideo> {
  late VideoPlayerController _controller;

  bool _initialized = false;

  @override
  void initState() {
    super.initState();
    _controller = VideoPlayerController.network(
      'https://flutter.github.io/assets-for-api-docs/assets/videos/bee.mp4',
    );

    _controller.addListener(() {
      if (_controller.value.isInitialized) {
        setState(() {
          _initialized = true;
        });
      }
    });
    _controller.setLooping(true);
    _controller.initialize();
  }

  @override
  void dispose() {
    _controller.dispose();
    super.dispose();
  }

  void onPressed() {
    if (_controller.value.isPlaying) {
      _controller.pause();
    } else {
      _controller.play();
    }
  }

  @override
  Widget build(BuildContext context) {
    return Center(
      child: SingleChildScrollView(
        child: Column(
          children: [
            TextButton.icon(
              onPressed: _initialized ? this.onPressed : null,
              icon: Icon(Icons.play_arrow_rounded),
              label: Text('Play/Pause'),
            ),
            AspectRatio(
              aspectRatio: 16 / 9,
              child: VideoPlayer(_controller),
            ),
          ],
        ),
      ),
    );
  }
}

Run your flutter web app so you can access it in a browser. Or click here.
Set your device in "Low Power Mode" ("yellow battery")
Visit the site (you will see the Play button)
Unset "Low Power Mode" in your device
Refresh the page (you will NOT see the Play button).

Expected results:
I would like to NOT see the Play button in Low Power Mode.
Actual results:
Well, the Play button shows up in Low Power Mode and can interfere with other UIs built on top of the video player widget.

Logs

No error output.

No issues found! (ran in 1.3s)

[✓] Flutter (Channel master, 2.4.0-1.0.pre.104, on Debian GNU/Linux rodete 5.10.40-1rodete1-amd64, locale
    en_US.UTF-8)
    • Flutter version 2.4.0-1.0.pre.104 at /usr/local/google/home/dit/github/flutter
    • Upstream repository git@github.com:ditman/flutter.git
    • Framework revision de36e51155 (13 days ago), 2021-07-08 03:21:02 +0530
    • Engine revision a1468f9b24
    • Dart version 2.14.0 (build 2.14.0-285.0.dev)

This is most likely a Safari bug/feature, and not an issue limited to Flutter Web, more info/possible web workarounds on Stack Overflow: Hide iPhone HTML5 video play button.
Originally reported internally by @gdcolella (Thanks, Greg!)

Webkit bug: https://bugs.webkit.org/show_bug.cgi?id=219889


mvn: absolute raml input files should be allowed

Currently only the classpath is scanned for the input file. It should be possible to provide absolute paths.

That would be good. I have managed to specify a RAML file in src/raml by using <raml>../../src/raml/api.raml</raml> configuration option (after looking at the code), but it's more like a workaround.


net: LookupAddr does not set IsNotFound with cgo resolver

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

$ go version
go version go1.19.4 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="$HOME/.cache/go-build"
GOENV="$HOME/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="$HOME/dev/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/go/go1.19.4"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/go/go1.19.4/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.19.4"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/run/user/$UID/go-build1957738364=/tmp/go-build -gno-record-gcc-switches"

What did you do?
$ cat > x.go <<EOF
package main
import (
        "fmt"
        "net"
)
func main() {
        _, err := net.LookupAddr("<IP_ADDRESS>")
        fmt.Printf("%v\n", err.(*net.DNSError).IsNotFound)
}
EOF

$ GODEBUG=netdns=go go run x.go
true

$ GODEBUG=netdns=cgo go run x.go
false

What did you expect to see?
Both go and cgo resolver correctly report IsNotFound.
What did you see instead?
Only go resolver does so.
There's already a pull request for this but it wasn't merged: https://github.com/golang/go/pull/45959

I should be fixed when CL 459955 gets merged

I think this CL is unrelated because it changes cgoLookupIPCNAME and not cgoLookupAddrPTR which is used by LookupAddr.

@rudis  you are right, my mistake, deleted that comment. I will make a new CL for this.

cc @neild @ianlancetaylor

@rudis you are right, my mistake, deleted that comment. I will make a new CL for this.

Thanks for the quick response. CL looks good to me.


xrefs for O:0051722 protein C-terminal methylesterase activity

id: GO:0051722
name: protein C-terminal methylesterase activity
namespace: molecular_function
def: "Catalysis of the reaction: C-terminal protein amino acid methyl ester + H2O = protein amino acid + methanol." [PMID:10318862, PMID:8650216]
synonym: "protein phosphatase methylesterase activity" NARROW []
is_a: GO:0051723 ! protein methylesterase activity
As per the narrow synonym "protein phosphatase methylesterase activity", EC:<IP_ADDRESS> (protein phosphatase methylesterase-1 ) = RHEA:48548 could be added to this term as narrowMatch xrefs. That reaction is:
[phosphatase 2A protein]-C-terminal L-leucine methyl ester + H2O <=> [phosphatase 2A protein]-C-terminal L-leucine + H(+) + methanol

also add MetaCyc:RXN-12322 as narrowMatch


Gh 543 loading rdfxml where rdfparse type literal

GitHub issue resolved: #543
Briefly describe the changes proposed in this PR:

We have added the XML namespace to the outer tags as described in the issue.
PR Author Checklist (see the contributor guidelines for more details):

[ ] my pull request is self-contained
[ ] I've added tests for the changes I made
[x] I've applied code formatting (you can use mvn process-resources to format from the command line)
[x] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change
[x] every commit has been signed off

Note: we merge all feature pull requests using squash and merge. See RDF4J git merge strategy for more details.

We have made the changes in the code to add namespaces only to the outer elements. However, we are trying to figure out how to add a test for the same. All existing tests have been specified by the W3C, as you mentioned. Please suggest how and at what location the test can be added.
Our output adds the xml namespace to the outer tags as expected. I'm attaching a screenshot for the same.

There a problem with this pull request: it looks as if you've also included changes from some of the other issues that you've been working on (for example, I see several commits related to #1481, #1270, and #1105). This makes your PR not self-contained, and makes it really hard to judge if you've actually solved the particular problem you're working on: the CI build currently fails on a test that (as far as I can tell) has nothing to do with the RDF/XML parser, but is probably caused by a problem with one the other issues. Also: even if the build didn't fail, we wouldn't be able to merge your change until all the other included stuff was also approved.
Can you try and clean this up? Doing this will require a bit of advanced usage of git. One way to do this is check out the master branch, make sure it's up to date, then create a completely new branch for issue GH-543, and cherry-pick the individual commits from this branch onto that new branch (but obviously only the commits that are about issue GH-543). Once you've done that, put that new branch up in a new pull request, and we can close this one and continue there.

The output screenshot looks good btw! As for where to add a test case: there is a unit test class RDFXMLWriterTest, I think that's a logical place to add a test case to.


eclipses.plugins: Replace SHA-1 hashes

Description of changes
Replace eclipses.plugins.ivyde{,rv}.src{Feature, Plugin}.sha1 with a hash
attribute, containing a SHA-256 hash in SRI format.
cc @r3dl3g
Things done

Built on platform(s)

[x] x86_64-linux

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

maintainer seems inactive so merged

Sorry had no time until now but looks good to me


Doc typo in core-semantics.adoc

Merged: https://github.com/apache/incubator-groovy/commit/2e1e733d54ef32a59ab86be0fb706cee314c800a
Thanks!


Unable to Use GPU with llama-cpp-python on Jetson Orin

I'm trying to install the llama-cpp-python package to run code on NVIDIA Jetson AGX Orin (CUDA version: 12.2) using the GPU, but it's running on the CPU instead.
I attempted the following commands to enable CUDA support:
CMAKE_ARGS="-DGGML_CUDA=on" FORCE_CMAKE=1 pip install llama-cpp-python --force-reinstall --upgrade --no-cache-dir
CMAKE_ARGS="-DGGML_CUDA=on" FORCE_CMAKE=1 pip install --no-cache-dir llama-cpp-python --extra-index-url https://abetlen.github.io/llama-cpp-python/whl/cu122
However, I am still unable to get it to use the GPU.

I fixed the issue by the running this command:
CUDACXX=/usr/local/cuda-12.2/bin/nvcc CMAKE_ARGS="-DGGML_CUDA=on" FORCE_CMAKE=1 pip install llama-cpp-python --force-reinstall --upgrade --no-cache-dir --verbose

very thank you


Expose local port as env value

Noticed the request in #131 and realized it's like... one line. Closes #131.

Thanks!

Ah drat, CI is broken. Sorry do you know how to fix this 😅

lol no. It's been a minute, but I'll take a look.

Looks like it's using an outdated job ... definition... thing? Man, I totally get why devops is a whole job these days...


ES6 support

Enjoy!

Could you resolve these conflicts and I'll merge everything?

Hold on, will close this and create a fresh one

Scandalous!


Make the ghosts client authoritative

Description
I assume this will prevent getting affected by lag when moving around with ghost, helps with adminning and observing in general.

this seems reasonable, ghosts can teleport around anyway


ScheduledTask: TaskPath should be a key property

Details of the scenario you tried and the problem that is occurring
We're in the process of moving scheduled tasks to a different path and we'd like to keep the names.
However, creating two ScheduledTask resources (one to create a new scheduled task, the other one to remove old ones in the root) causes a conflict, since only TaskName property is the key property.
Verbose logs showing the problem
Suggested solution to the issue
Mark TaskPath property as a key property.
The DSC configuration that is used to reproduce the issue (as detailed as possible)
ScheduledTask "new_task" {
    TaskName = "task_name"
    TaskPath = "\NewPath"
    Ensure = "Present"
    ...
}

ScheduledTask "old_task" {
    TaskName = "task_name"
    TaskPath = "\"
    Ensure = "Absent"
    ...
}

Happy to submit a patch.
The operating system the target node is running

Version and build of PowerShell the target node is running

Version of the DSC module that was used ('dev' if using current dev branch)
<IP_ADDRESS>

Oh, I see the problem. If I mark a property as a Key, the corresponding parameter has to be Mandatory, which means it has to be specified when creating a resource instance.

Hi @bozho - I think this is worth considering. It would result in a breaking change but may be something the community would be ok with to gain the functionality you describe.
It also would allow jobs with the same name to be created in different folders.
Let's open this up for community discussion and also bring it up at the next community call..

Ok, sounds like a plan.
An easy win here would be to allow TaskName to be given as path\to\task\MyTask, which would then be parsed for the path and task name, as task names cannot contain \.

So the options would be:

Leave as is.
Remove TaskPath parameter and combine with TaskName.
Remove default from TaskPath and make a key.

Options 2 and 3 would both be breaking change.
I tend to like Option 3 if we're going to be making a breaking change. @johlju ?

I like No. 3, too.

I actually like option 2. It allows using the same task name in different folders and for those that don’t care of path can just specify the name of the task and the resource defaults to root (for TaskPath). Going with option 2 then a consumer must always specify path even if they don’t care because the want it to be in the root path. I feel that adds unnecessary configuration for something that should be optional.
Going with option 2 the path is still optional, but still can be used to control where the task should end up.
With option 2 we could keep the TaskPath parameter as is. If specified it will override any path in the TaskName parameter. That would keep existing configurations to still work, meaning it would not be a breaking change going with option 2.

Updated previous comment. Commenting from my phone is not the best experience :)

Hmm... keeping TaskPath as suggested above could potentially break in a certain scenario. When using a TaskName with different paths but the same TaskName, and TaskPath is set to the same path. In that scenario it will create two task with the same name in the same path which would fail.
Maybe if we opt to keep TaskPath it could override the path, but write out a deprecation warning. Then we can remove that deprecation when we have more breaking changes.
Still like option 2 better (with or without keeping TaskPath)


Multiple Nodes

Versions

PGSync version: 1.1.32
Postgres version: 13
Elasticsearch version: 7.9.3
Redis version: (latest docker version)
Python version: 3.8

Problem Description
I've looked at #32 but I'm still confused about having multiple nodes in a given database/index combo. For example I would assume the following would grab the book, magazine, and tape tables and sync them all in the elasticsearch library_index index.
  {
    "database": "db01",
    "index": "library_index",
    "nodes": [
      {
        "table": "book",
        "columns": ["id", "name"]
      },
      {
        "table": "magazine",
        "columns": ["id", "name"]
      },
      {
        "table": "tape",
        "columns": ["id", "name"]
      }
    ]
  }

But running this only syncs the book table to library_index; magazine and tape are not synced.

Good question. Nodes were originally intended to be top-level collections.
But you are right and this can/should be flattened into a singular attribute - node as they can't stand on their own.
I will do this in the next major version as a breaking change.
[
    {
        "database": "mydb",
        "index": "mydb-index",
        "node": {
            "table": "table1",
            "schema": "public",
            "columns": []
        }
    },
    {
        "database": "mydb1",
        "index": "mydb-index2",
         "node": {
            "table": "table1",
            "schema": "public",
            "columns": []
        }
    }
]

Do you have a suggestion on how to sync multiple tables into a single index? I can see it being difficult to do with pgsync as you'd have to make sure the columns being synced have to match labels. Could this be done with a plugin of sorts?

The typical use case is to group unrelated tables into separate indices.
You can slo group them into the same index with a relationship.
Having them in a separate index would be the easiest thing

That makes sense, thank you.


Gateway into gRPC - I thought I had it right

Describe the bug
Update: If you run into problems using gRPC and Istio, I can tell you that I have been using a week and some days without success - see below description.
I really look forward to see Istio solving this, if it is a bug, or document this if - and probably is - just me who is unable to figure this out :-)
Until this great software and all its telemetries and other cool stuff is working for you, you can use Ambassador.
Ambassador gets the job done for “north/south” traffic - you can use this if you need to move forward in you development while waiting: https://www.getambassador.io/user-guide/grpc

I am trying to call a gRPC service - can you help. The following is the setup:
Gateway:
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: my-gateway
spec:
  selector:
    istio: ingressgateway # use istio default controller
  servers:
  - port:
      number: 3000
      name: grpc-my
      protocol: grpc # also tried TCP, HTTP/2, HTTP
    hosts:
    - "my-service-grpc.default.svc.cluster.local"

Service:
apiVersion: v1
kind: Service
metadata:
  name: my-service-grpc
  labels:
    app: my-service-grpc
spec:
  selector:
    app: my-deployment-grpc
  ports:
  - port: 3000
    name: grpc-my
    protocol: grpc # Also tried TCP, HTTP/2, HTTP

Deployment:
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: my-deployment
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: my-deployment-grpc
        version: v1
    spec:
      containers:
      - name: my
        image: my/image:v1.0
        ports:
        - containerPort: 3000
          name: grpc-my
          protocol: TCP

VirtualService:
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: my-virtualservice
spec:
  hosts:
  - "my-service-grpc.default.svc.cluster.local"
  gateways:
  - my-gateway
  http:
  - match:
    - uri:
        prefix: /helloworld.Greeter/
    route:
    - destination:
        host: my-service-grpc

Istio helm value.yaml file
   ingress:...
    - port: 6000
      name: grpc

   gateways:...
    - port: 15031
      targetPort: 15031
      name: http2-grafana
# Added for gRPC
    - port: 6000
      targetPort: 3000
      name: grpc-app
    secretVolumes:
   ...

Then:
grpc_cli ls localhost:3000 helloworld.Greeter -l
I get the following error:
D0814 16:49:43.653073000<PHONE_NUMBER>48256 ev_posix.cc:142]              Using polling engine: poll
D0814 16:49:43.693399000<PHONE_NUMBER>48256 dns_resolver.cc:331]          Using native dns resolver
I0814 16:49:43.740829000<PHONE_NUMBER>48256 subchannel.cc:646]            Connect failed: {"created":"@1534258183.740786000","description":"Failed to connect to remote host: OS Error","errno":61,"file":"src/core/lib/iomgr/tcp_client_posix.cc","file_line":201,"os_error":"Connection refused","syscall":"connect","target_address":"ipv6:[::1]:3000"}
I0814 16:49:43.741021000<PHONE_NUMBER>48256 subchannel.cc:646]            Connect failed: {"created":"@1534258183.740998000","description":"Failed to connect to remote host: OS Error","errno":61,"file":"src/core/lib/iomgr/tcp_client_posix.cc","file_line":201,"os_error":"Connection refused","syscall":"connect","target_address":"ipv4:<IP_ADDRESS>:3000"}
I0814 16:49:43.741037000<PHONE_NUMBER>48256 subchannel.cc:470]            Subchannel 0x7fa04cd02150: Retry in 1000 milliseconds
Received an error when querying services endpoint.
I0814 16:49:43.741683000<PHONE_NUMBER>48256 proto_reflection_descriptor_database.cc:51] ServerReflectionInfo rpc failed. Error code: 14, details: Connect Failed

Note: I have all this working with Ambasador, getambassador.io/config:
Note: istioctl get gateway istio-autogenerated-k8s-ingress -n istio-system -o yaml
Only list port 80?
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  creationTimestamp: null
  name: istio-autogenerated-k8s-ingress
  namespace: istio-system
  resourceVersion: "813085"
spec:
  selector:
    istio: ingress
  servers:
  - hosts:
    - '*'
    port:
      name: http
      number: 80
      protocol: HTTP2

Expected behavior
Steps to reproduce the bug
Version
istio 1.0
kubernetes 1.10
Is Istio Auth enabled or not?
No
Environment
Cluster state
If you're running on Kubernetes, consider following the
instructions
to generate "istio-dump.tar.gz", then attach it here by dragging and dropping
the file onto this issue.

Similar issues reported elsewhere:
https://github.com/istio/istio/issues/7843
https://github.com/istio/istio/issues/7980

Hi @rshriram
Thanks, I now tried opening a new port in helm values.yaml
   ...
    - port: 15031
      targetPort: 15031
      name: http2-grafana
# Added for gRPC
    - port: 3000
      targetPort: 3000
      name: grpc-app
    secretVolumes:
   ...

Should that be enough?

just add
      name: grpc
      nodePort: 31399
      port: 9090
      targetPort: 9090

in kind: Service name: istio-ingressgateway
and then access the gRPC-server  by  ingress service ip : 9090 is ok for me

Thanks @begoat
I am giving up on Istio - thanks for you help

hello, @JCzz
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: backend-gateway
spec:
  selector:
    istio: ingressgateway # use Istio default gateway implementation
  servers:
  - port:
      number: 9090
      name: grpc-gateway
      protocol: GRPC
    hosts:
    - "*"
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: backend
spec:
  hosts:
  - "*"
  gateways:
  - backend-gateway
  http:
  - match:
    - uri:
        prefix: /
    route:
    - destination:
        host: backend
        port:
          number: 50051

apiVersion: v1
kind: Service
metadata:
  name: istio-ingressgateway
  namespace: istio-system
  annotations:
    service.beta.kubernetes.io/azure-load-balancer-internal: "true"
  labels:
    chart: gateways-1.0.0
    release: RELEASE-NAME
    heritage: Tiller
    app: istio-ingressgateway
    istio: ingressgateway
spec:
  type: LoadBalancer
  selector:
    app: istio-ingressgateway
    istio: ingressgateway
  ports:
    -
      name: http2
      nodePort: 31380
      port: 80
      targetPort: 80
    -
      name: https
      nodePort: 31390
      port: 443
    -
      name: grpc
      nodePort: 31399
      port: 9090
      targetPort: 9090

I run it on azure platform

@JCzz I just saw you exposed your service via "prefix: /" in VirtualService, how can you expose other gRPC services?

Hi @xiajidexue
I never got it to work. I have asked around a lot, incl. gitter grpc, Ambassador, Istio - no one seems to know. Let me know if you get any closer?

@JCzz I got it working! And also added TLS! I added the port 30051 to the istio-ingressgateway but you can use 31400 without any changes (I don't know if other service uses this port, but it is enabled by default). My service is called gateway and exposes an http server and the gRPC service (if there is too much confusion I'll refactor the examples below).
For my TLS certificates I've used Let's Encrypt.
The gateway:
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: gateway-gateway
  namespace: development
spec:
  selector:
    istio: ingressgateway # use istio default controller
  servers:
  - port:
      number: 30051
      name: grpc
      protocol: https
    tls:
      mode: SIMPLE
      serverCertificate: /etc/istio/ingressgateway-certs/tls.crt
      privateKey: /etc/istio/ingressgateway-certs/tls.key
    hosts:
    - "development.redacted.co"
  - port:
      number: 80
      name: http
      protocol: HTTP
    hosts:
    - "development.redacted.co"

VirtualService:
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: gateway
  namespace: development
spec:
  hosts:
  - "development.redacted.co"
  gateways:
  - gateway-gateway
  http:
  - match:
    - uri:
        prefix: /api/
    rewrite:
      uri: /
    route:
    - destination:
        host: gateway
        port:
          number: 5000
  - match:
    - port: 30051
    route:
    - destination:
        host: gateway
        port:
          number: 50051

Service:
kind: Service
apiVersion: v1
metadata:
  name: gateway
  namespace: development
  labels:
    app: gateway
spec:
  selector:
    app: gateway
  ports:
    - port: 50051
      name: grpc
    - port: 5000
      name: http

Hi @facundomedica
Thank you so much for sharing, I will get back and verify this when I deploy. For now I am developing on just port 80.
Very cool!

@facundomedica  I am unable to access grpc application (listen 3000) using loadbalncer and can be accessed using Nodeport. Although if I setup any http application using http config it works without a sweat. Please help.
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: my-gateway
spec:
  selector:
    istio: ingressgateway # use istio default controller
  servers:
  - port:
      number: 80
      name: grpc-test
      protocol: GRPC
    hosts:
    - "*"

---

apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: frontend
spec:
  hosts:
  - "*"
  gateways:
  - my-gateway
  http:
  - route:
    - destination:
        host: frontend
        port:
          number: 3000
          name: grpc-test

---

apiVersion: v1
kind: Service
metadata:
  name: frontend
  labels:
    app: frontend
spec:
  selector:
    app: frontend
  ports:
  - port: 3000
    name: grpc-test

---

apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: frontend-prod
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: frontend
        version: prod
    spec:
      containers:
      - name: frontend
        image: ****:grpc
        imagePullPolicy: Always
        ports:
        - containerPort: 3000
        env:
        - name: SERVICE_NAME
          value: "frontend-prod"
      imagePullSecrets:
        - name: gitlab-registry

@facundomedica when you say add port 30051 to the istio-ingressgateway does it mean you have edited kubectl edit svc istio-ingressgateway -n istio-system and added something like
  - name: grpc
    nodePort: 31124       #Node port is automatically picked
    port: 30051
    protocol: TCP
    targetPort: 30051

I am following the same way. and here is my gateway and VirtualService
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: grpc-gateway
spec:
  selector:
    istio: ingressgateway # use Istio default gateway implementation
  servers:
  - port:
      number: 30051
      name: grpc
      protocol: GRPC
    hosts:
    - "*"
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: grpc
spec:
  hosts:
  - "*"
  gateways:
  - grpc-gateway
  http:
  - match:
    - port: 30051
    route:
    - destination:
        port:
          number: 50051
        host: grpchelloworld
---
apiVersion: v1
kind: Service
metadata:
  labels:
    app: grpchelloworld
  name: grpchelloworld
  namespace: default
spec:
  clusterIP: <IP_ADDRESS>
  ports:
  - name: http
    port: 50051
    protocol: TCP
    targetPort: 50051
  selector:
    app: grpchelloworld
  sessionAffinity: None
  type: ClusterIP
status:
  loadBalancer: {}
---

Let me know what I am doing wrong.
NOTE: I am doing all this on IBM blue cloud and able to access the app using nodeport settings.

Thank you guys for the help. I just successfully configured gRPC load balancing with Istio on GKE. The two lessons I learned are

Use local DNS(/etc/hosts) for your self-defined host name, especially when you don't have a public domain.
Use port 31400 instead of 80 for Gateway if you are using the default istio-ingressgateway.

Thanks for all the help on this -- from what I can tell this looks resolved. If there are more issues feel free to open a new bug, as any comments here will probably get lost

@JCzz I got it working! And also added TLS! I added the port 30051 to the istio-ingressgateway but you can use 31400 without any changes (I don't know if other service uses this port, but it is enabled by default). My service is called gateway and exposes an http server and the gRPC service (if there is too much confusion I'll refactor the examples below).
For my TLS certificates I've used Let's Encrypt.
The gateway:
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: gateway-gateway
  namespace: development
spec:
  selector:
    istio: ingressgateway # use istio default controller
  servers:
  - port:
      number: 30051
      name: grpc
      protocol: https
    tls:
      mode: SIMPLE
      serverCertificate: /etc/istio/ingressgateway-certs/tls.crt
      privateKey: /etc/istio/ingressgateway-certs/tls.key
    hosts:
    - "development.redacted.co"
  - port:
      number: 80
      name: http
      protocol: HTTP
    hosts:
    - "development.redacted.co"

VirtualService:
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: gateway
  namespace: development
spec:
  hosts:
  - "development.redacted.co"
  gateways:
  - gateway-gateway
  http:
  - match:
    - uri:
        prefix: /api/
    rewrite:
      uri: /
    route:
    - destination:
        host: gateway
        port:
          number: 5000
  - match:
    - port: 30051
    route:
    - destination:
        host: gateway
        port:
          number: 50051

Service:
kind: Service
apiVersion: v1
metadata:
  name: gateway
  namespace: development
  labels:
    app: gateway
spec:
  selector:
    app: gateway
  ports:
    - port: 50051
      name: grpc
    - port: 5000
      name: http

Is that all you need for grpc tls

@sandeeplamb In your case, trying out without tls work? I mean setting your Gateway resource to listen without tls.

Nevermind. I figured it out. nginx.ingress.kubernetes.io/upstream-vhost annotation is required in case Nginx is fronting Istio gateway in case of GRPC

Nevermind. I figured it out. nginx.ingress.kubernetes.io/upstream-vhost annotation is required in case Nginx is fronting Istio gateway in case of GRPC

@sands6 Yeah for me I am using Istio, so it works with plane GRPC mode without TLS and also with GRPC and TLS. No problem at all.
But I have seen you already have a solution now

@sandeeplamb I want to learn from your experience of using Istio-gateway at the edge. Are you running Istio gateway in host-network mode or LB fronting istio gateway? How do manage multiple vhost endpoints point to gateway with different protocols http/https/grpc as we have to use different port for grpc?

@sandeeplamb I want to learn from your experience of using Istio-gateway at the edge. Are you running Istio gateway in host-network mode or LB fronting istio gateway? How do manage multiple vhost endpoints point to gateway with different protocols http/https/grpc as we have to use different port for grpc? in your example above, port 30051

In our case, we have AWS-NLB in front as LB which is created using terraform.
istio-ingressgateway service created as NodePort and all ports in service have corresponding NodePort with them.
  ports:     -       name: status-port       port: 15020       targetPort: 15020       nodePort: 31952     -       name: http2       port: 80       targetPort: 80       nodePort: 30314     -       name: https       port: 443       targetPort: 443       nodePort: 31454     -        name: grpc       port: 31400       nodePort: 31400
All these nodeports are listening at NLB with Targets are EKS worker nodes.
We have just 4 ports open at NLB 80,443,15020(status), 31400(we use as grpc).
For more grpc ports, you can add more ports in istio-ingressgateway service.
Create gateway with same host together and separate virtual hosts/destination-rules

hello, @JCzz
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: backend-gateway
spec:
  selector:
    istio: ingressgateway # use Istio default gateway implementation
  servers:
  - port:
      number: 9090
      name: grpc-gateway
      protocol: GRPC
    hosts:
    - "*"
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: backend
spec:
  hosts:
  - "*"
  gateways:
  - backend-gateway
  http:
  - match:
    - uri:
        prefix: /
    route:
    - destination:
        host: backend
        port:
          number: 50051

apiVersion: v1
kind: Service
metadata:
  name: istio-ingressgateway
  namespace: istio-system
  annotations:
    service.beta.kubernetes.io/azure-load-balancer-internal: "true"
  labels:
    chart: gateways-1.0.0
    release: RELEASE-NAME
    heritage: Tiller
    app: istio-ingressgateway
    istio: ingressgateway
spec:
  type: LoadBalancer
  selector:
    app: istio-ingressgateway
    istio: ingressgateway
  ports:
    -
      name: http2
      nodePort: 31380
      port: 80
      targetPort: 80
    -
      name: https
      nodePort: 31390
      port: 443
    -
      name: grpc
      nodePort: 31399
      port: 9090
      targetPort: 9090

I run it on azure platform

@JCzz I just saw you exposed your service via "prefix: /" in VirtualService, how can you expose other gRPC services?

Someone got this working?
I have several GRPC services that need to be exposed via Istio Ingress Gateway, how do I route to them? I tried changing the prefix and no luck, the istio config in the ingress gateway always set the prefix of the service in "/"

@nelsonyaccuzzi by changing the prefix did you mean doing something like this:
apiVersion: networking.istio.io/v1beta1
kind: Gateway
metadata:
  name: my-gateway
  namespace: istio-system
spec:
  selector:
    istio: ingressgateway
  servers:
  - hosts:
    - my.domain.com
    port:
      name: http
      number: 80
      protocol: HTTP
    tls:
      httpsRedirect: true
  - hosts:
    - my.domain.com
    port:
      name: https
      number: 443
      protocol: HTTPS
    tls:
      credentialName: my-domain-cert
      mode: SIMPLE
      privateKey: sds
      serverCertificate: sds
---
apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
  name: vs
  namespace: istio-system
spec:
  gateways:
  - istio-system/my-gateway
  hosts:
  - my.domain.com
  http:
  - match:
    - uri:
        prefix: /helloworld.Greeter/
    route:
    - destination:
        host: hello.default.svc.cluster.local
        port:
          number: 3000
  - match:
    - uri:
        prefix: /hotel.Rates/
    route:
    - destination:
        host: hotel.default.svc.cluster.local
        port:
          number: 3000

This works for me (using istio 1.5.0).

I've made no changes to the istio-ingressgateway Service.
The prefixes are the FQN of the gRPC packages
TLS is working, if testing with grcp_cli be sure to do: grpc_cli --channel_creds_type=ssl ...

Hope it helped.

Hi everyone , I got the grpc server running without auth .


feat(storage): add support for signing URLs using token

Issue [7627]

@Emar-Kar i know it's tricky to add system test for this feature so for that use google-cloud-iam to generate access token and then pass to the function, but for that i need to install library and edit nox file, not sure about we can edit nox or not, so for that i'll ask chris may be.

@HemangChothani thanks for the detailed explanation. A good point to ask Chris first.

@Emar-Kar  and @mf2199  can you PTAL , for the system test @chris wilcox   ask to add iam dependency to nox file as it required.

@Emar-Kar  yes you are right for that you need to install google-cloud-iam and also need to update noxfiel.py , just got the conformation that we have permission to update nox file so going to commit that.


add createProgram docs

Refers to a little bit to #15693. I try to learn the compiler API and have a "hard" time to find good documentation and resources. I noticed I can't use code completion to see the documentation for certain APIs and I thought it would be a good idea to add them.
I took documentation from https://github.com/Microsoft/TypeScript/wiki/Architectural-Overview,  https://github.com/Microsoft/TypeScript/wiki/Using-the-Language-Service-API and https://github.com/Microsoft/TypeScript/wiki/Using-the-Compiler-API to write this comment.
Are you interested in pull requests like that?

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Updated :)

thanks @donaldpipowitch!

Thank you :)


Error Message in English

When creating a new directory without name. The error message is in English but should be in German.
EDIT: my machine location is English-US. It seems on German machines it's in German.

dependent on browser locale
working as intended


[site-demo] wrong color hex code for material theme

DemoColor block doesn't retrieve the right color hex code for material theme

Resolved in v0.24.0


Don't load columns from tables in other schemas

By default Rails puts its tables in the public schema. But sometimes other schemas are used, for instance for history, reporting, multiple tenants, etc. It is common for these other schemas to have tables with the same name as the table from the public schema (see links above). Currently when pg_audit_log queries information_schema.columns, it gets all columns from all schemas, which causes errors when it tries to record the changes that happened. This change makes pg_audit_log look only in the public schema, so it doesn't break if other schemas are used.

Is it possible to specify the schema in the database.yml? If so this
should use the value specified in the connection config instead of a
hard-coded default.
On Mon, Jun 6, 2016 at 19:42 Paul A Jungwirth<EMAIL_ADDRESS>wrote:

By default Rails puts its tables in the public schema. But sometimes
other schemas are used, for instance for history
https://www.youtube.com/watch?v=TRgni5q0YM8, reporting, multiple tenants
https://github.com/influitive/apartment, etc. It is common for these
other schemas to have tables with the same name as the table from the
public schema (see links above). Currently when pg_audit_log queries
information_schema.columns, it gets all columns from all schemas, which
causes errors when it tries to record the changes that happened. This
change makes pg_audit_log look only in the public schema, so it doesn't
break if other schemas are used.
You can view, comment on, or merge this pull request online at:
https://github.com/Casecommons/pg_audit_log/pull/15
Commit Summary

Don't load columns from tables in other schemas

File Changes

M lib/pg_audit_log/function.rb
https://github.com/Casecommons/pg_audit_log/pull/15/files#diff-0 (2)

Patch Links:

https://github.com/Casecommons/pg_audit_log/pull/15.patch
https://github.com/Casecommons/pg_audit_log/pull/15.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Casecommons/pg_audit_log/pull/15, or mute the thread
https://github.com/notifications/unsubscribe/AAJWV7pLUcy7tbZzpTiA1lpGOo-gAsrWks5qJLBwgaJpZM4IvbB4
.

The best you can do from database.yml is schema_search_path, which seems not quite right. I changed the PR so that the old behavior is unchanged by default, but you can restrict pg_audit_log to just one schema by passing a param to the install function or setting an envvar run you run the rake task.


Move VSCode extention to its own repository

No bandwidth to maintain the extension in another repo. Close as not planned.


Add Qt 5.0-5.6 PPAs (supercedes #212)

This adds Qt PPAs to allow for Qt-based projects to move off of legacy architectures.
This commit supercedes #212 as it had test errors.
bin/travis-add-source ubuntu.json qt502 ppa:beineri/opt-qt502-trusty '' -i
bin/travis-add-source ubuntu.json qt511 ppa:beineri/opt-qt511-trusty '' -i
bin/travis-add-source ubuntu.json qt521 ppa:beineri/opt-qt521-trusty '' -i
bin/travis-add-source ubuntu.json qt532 ppa:beineri/opt-qt532-trusty '' -i
bin/travis-add-source ubuntu.json qt542 ppa:beineri/opt-qt542-trusty '' -i
bin/travis-add-source ubuntu.json qt551 ppa:beineri/opt-qt551-trusty '' -i
bin/travis-add-source ubuntu.json qt56 ppa:beineri/opt-qt56-trusty '' -i

Closes: #27, #69, #110, #143, #212

@Nava2 Thanks!

@BanzaiMan, can you please review this PR? Qt5 is may be a huge thing, but it benefits a lot of GitHub users who want to use Travis for testing/building their cross-platform apps on Linux.

It would be awesome to have this merged.
@BanzaiMan is there anything that can be done to bring this ahead? Not having recent Qt5 versions available complicates things considerably for us.

I've rebased this branch against master and added qt56.
bin/travis-add-source ubuntu.json qt56 ppa:beineri/opt-qt56-trusty '' -i

Booster.com will be happy to have this merged as well. We are using Trusty Beta now, but it will be great to use container that will speed up booting time. Thanks a lot.

What's the status of this? Any chance of getting this merged in soon?

I'd be very happy to see this land, as well.  Is there anything I can do to help make this PR easier to accept?

It has been a while since the first request of adding a recent Qt5 version. Do we know how if this issue will ever be solved (before Qt6)?

News on this?

I've updated this PR to only have LTS versions and the most recent 5.10 release. @BanzaiMan this has been stagnating for almost two years -- it'd be great to hear a response on it.

Note: asking for qt5-default from apt kinda works: https://travis-ci.org/fenollp/sofa/builds/346694149 but only kinda.


xq: 0.2.45 -> 0.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/MiSawa/xq/releases.
meta.description for xq is: Pure rust implementation of jq
meta.homepage for xq is: https://github.com/MiSawa/xq
Updates performed

Rust version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.0 with grep in /nix/store/7q3vp69cdsfh79bwxaa86d7cv89w7vm0-xq-0.3.0
found 0.3.0 in filename of file in /nix/store/7q3vp69cdsfh79bwxaa86d7cv89w7vm0-xq-0.3.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
xq

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/93803edeeeeb204a3a8c0800848c307e830bb514#xq

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

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

1 package built:

xq

Maintainer pings
cc @MatthewCroughan for testing.

Result of nixpkgs-review pr 283294 run on x86_64-linux 1

1 package built:

xq


[Question] Kernel dies while creating Parameter server for HugeCTR Inference

I followed this hugectr inference notebook to create an inference code with DLRM config file. Now, every time I am trying to create the parameter server object, kernel dies without any exception.
parameter_server= CreateParameterServer(['./config_files/dlrm_inference.json'], ['DLRM'], False)
I am attaching the dlrm_inference.json file for reference
{
    "inference": {
    "max_batchsize": 1024,
    "dense_model_file": "./hugeCTR_saved_model_DLRM/_dense_18000.model",
    "sparse_model_file": "./hugeCTR_saved_model_DLRM/0_sparse_18000.model"
  },

    "layers": [
          {
          "name": "data",
          "type": "Data",
          "slot_size_array": [49865, 54893, 337691, 9523, 164],
          "slot_size_array_orig": [49865, 54893, 337691, 9523, 164],
          "source": "./hugeCTR/filelist.txt",
          "eval_source":"./hugeCTR/valid_filelist.txt",
          "check": "None",
          "cache_eval_data": true,
          "label": {
                  "top": "label",
                  "label_dim": 1
          },
          "dense": {
                  "top": "dense",
                  "dense_dim": 2
          },
          "sparse": [
                  {
              "top": "data1",
              "type": "LocalizedSlot",
              "max_feature_num_per_sample": 5,
              "max_nnz": 1,
              "slot_num": 5
                  }
          ]
        },

        {
          "name": "sparse_embedding1",
          "type": "LocalizedSlotSparseEmbeddingHash",
          "bottom": "data1",
          "top": "sparse_embedding1",
          "sparse_embedding_hparam": {
            "slot_size_array": [49865, 54893, 337691, 9523, 164],
            "embedding_vec_size": 64,
            "combiner": 0
          }
        },

        {
          "name": "fc1",
          "type": "FusedInnerProduct",
          "bottom": "dense",
          "top": "fc1",
          "fc_param": {
            "num_output": 64
          }
        },

        {
          "name": "fc2",
          "type": "FusedInnerProduct",
          "bottom": "fc1",
          "top": "fc2",
          "fc_param": {
            "num_output": 128
          }
        },

        {
          "name": "fc3",
          "type": "FusedInnerProduct",
          "bottom": "fc2",
          "top": "fc3",
          "fc_param": {
            "num_output": 64
          }
        },

        {
          "name": "interaction1",
          "type": "Interaction",
          "bottom": ["fc3", "sparse_embedding1"],
          "top": "interaction1"
        },

        {
          "name": "fc4",
          "type": "FusedInnerProduct",
          "bottom": "interaction1",
          "top": "fc4",
           "fc_param": {
            "num_output": 1024
          }
        },

        {
          "name": "fc5",
          "type": "FusedInnerProduct",
          "bottom": "fc4",
          "top": "fc5",
          "fc_param": {
            "num_output": 1024
          }
        },

        {
          "name": "fc6",
          "type": "FusedInnerProduct",
          "bottom": "fc5",
          "top": "fc6",
          "fc_param": {
            "num_output": 512
          }
        },

        {
          "name": "fc7",
          "type": "FusedInnerProduct",
          "bottom": "fc6",
          "top": "fc7",
          "fc_param": {
            "num_output": 256
          }
        },

        {
          "name": "fc8",
          "type": "InnerProduct",
          "bottom": "fc7",
          "top": "fc8",
          "fc_param": {
            "num_output": 1
          }
        },

        {
          "name": "sigmoid",
          "type": "Sigmoid",
          "bottom": "fc8",
          "top": "sigmoid"
        }
    ]
}

From your dlrm_inference.json config file, it seems that you are training DLRN with your own data set. The DLRM trained in the sample notebook uses the normal format with embedding key format as I32, so if your model is trained in parquet format or enable i64_input_key =true , then format embedding key is i64, you should enable I64 parameter server with
parameter_server= CreateParameterServer(['./config_files/dlrm_inference.json'], ['DLRM'], True)

So, I ran it with I64 parameter server as True. I have trained the DRLM model with Norm dataset format following what was shown in the movie lens notebook.
I got the following error:
*** The MPI_Comm_rank() function was called before MPI_INIT was invoked. *** This is disallowed by the MPI standard. *** Your MPI job will now abort. [51ddcc4e8547:00305] Local abort before MPI_INIT completed completed successfully, but am not able to aggregate error messages, and not able to guarantee that all other processes were killed!

HugeCTR Inference does not enable MPI. Could you please try to use Inference docker from NGC. nvcr.io/nvidia/merlin/merlin-inference:0.5

is the MPI error reported during the training or inference ?

It was reported during Inference. I imported MPI and that issue was solved.

Is this one nvcr.io/nvidia/merlin/merlin-training:0.4 ?

Yes

Thanks for your feedback. We have updated the latest HugeCTR Python API code and instruction in each sample. You could  use nvcr.io/nvidia/merlin/merlin-training:0.5 and  activate the merlin conda environment by running
source activate merlin
So I can understand issue has been solved？

I tried with the updated nvcr.io/nvidia/merlin/merlin-training:0.5 version. Again, I tried using with both supported argument types for the predict method-  hugectr.inference.InferenceSession, dense_feature: List[float], embeddingcolumns: List[int], row_ptrs: List[int], i64_input_key: bool) and (self: hugectr.inference.InferenceSession, dense_feature: List[float], embeddingcolumns: List[int], row_ptrs: List[int]) -> List[float].
I guess it keeps on printing the predicted values. How to solve this? I am attching the python script i am using for inference. Sorry for late follow-up.
[22d10h49m18s][HUGECTR][INFO]: default_emb_vec_value is not specified using default: 0.000000
[22d10h49m19s][HUGECTR][INFO]: Global seed is 986885782
[22d10h49m21s][HUGECTR][INFO]: Peer-to-peer access cannot be fully enabled.
[22d10h49m21s][HUGECTR][INFO]: algorithm_search is not specified using default: 1
[22d10h49m21s][HUGECTR][INFO]: cuda_graph is not specified using default: 1
[22d10h49m21s][HUGECTR][INFO]: start create embedding for inference
[22d10h49m21s][HUGECTR][INFO]: sparse_input name data1
[22d10h49m21s][HUGECTR][INFO]: create embedding for inference success
Traceback (most recent call last):
  File "dlrm_inference.py", line 30, in <module>
    dlrm_inference(config_file, model_name, data_path)
  File "dlrm_inference.py", line 18, in dlrm_inference
    output = inference_session.predict(dense_features, embedding_columns, row_ptrs, False)
TypeError: predict(): incompatible function arguments. The following argument types are supported:
    1. (self: hugectr.inference.InferenceSession, dense_feature: List[float], embeddingcolumns: List[int], row_ptrs: List[int]) -> List[float]
    2. (self: hugectr.inference.InferenceSession, dense_feature: List[float], embeddingcolumns: List[int], row_ptrs: List[int], i64_input_key: bool) -> List[float]

Invoked with: <hugectr.inference.InferenceSession object at 0x7f638b3c48f0>, [0.001652892561983471, 0.7450980392156863, 0.0, 0.08888888888888889, 0.0, 0.0, 0.9583333333333331, 0.7450980392156863, 0.4791666666666666, 0.0007182188173330142, 0.0, 0.001652892561983471, 0.006222382541257103, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.001652892561983471, 0.6431372549019607, 0.04166666666666666, 0.11851851851851855, 0.0, 0.0, 0.9166666666666666, 0.6431372549019607, 0.4791666666666666, 0.0007182188173330142, 0.0, 0.001652892561983471, 0.006222382541257103, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.001652892561983471, 0.5764705882352941 ............(continued)

python script for inference:
import sys
from mpi4py import MPI
from sklearn.metrics import roc_auc_score
from hugectr.inference import CreateParameterServer, CreateEmbeddingCache, InferenceSession

def dlrm_inference(config_file, model_name, data_path):
      # read data from file
      data_file = open(data_path)
      labels = [int(item) for item in data_file.readline().split(' ')]
      dense_features = [float(item) for item in data_file.readline().split(' ')]
      embedding_columns = [int(item) for item in data_file.readline().split(' ')]
      row_ptrs = [int(item) for item in data_file.readline().split(' ')]
      # create parameter server, embedding cache and inference session
      parameter_server = CreateParameterServer([config_file], [model_name], False)
      embedding_cache = CreateEmbeddingCache(parameter_server, 0, True, 0.2, config_file, model_name, False)
      inference_session = InferenceSession(config_file, 0, embedding_cache)
      # make prediction and calculate accuracy
      output = inference_session.predict(dense_features, embedding_columns, row_ptrs, False)
      accuracy = calculate_accuracy(labels, output)
      print("[HUGECTR][INFO] prediction number samples: {}, accuracy: {}".format(len(labels), accuracy))

def calculate_accuracy(labels, output):
      score= roc_auc_score(labels, output)
      return score

if __name__ == "__main__":
      config_file = sys.argv[1]
      model_name = sys.argv[2]
      data_path = sys.argv[3]
      dlrm_inference(config_file, model_name, data_path)

I think this is because you did not use the complete criteo data(13 dense+26 categorical features) to train the dlrm model if you follow the movie-lens-example.ipynb instructions.
hugectr_inference.ipynb uses the completed criteo data set to train the dcn model, and it is very important to use criteo2predict.py script to convert the preprocessed data to CSR format (for inference), you need to convert the your customized input data to csr format for inference , please refer to CSR for details.

You can also refer to the Prepare the inference session part in ecommerce-example.ipynb, especially Converting data to CSR format step

I am working on a different dataset not the criteo one. I did follow the criteo2predict.py script before running the inference. Will go through the CSR documentation and try to resolve the problem i am facing. Thanks for the help!


Custom table column widths in markdown

@lena-larionova
Let me know, if such solution is acceptable.
Unfortunately, I could not find a way how to extend markdown parser, so it would be able to parse such specific tags to html element styles.
Another way how to do it (which is already available) is to add


Dedupe entries when generating schema tables

Something like this should only generate one table entry for EmailAddress:
               'emailDomains', (select array_agg(split_part(value, '@', 2))
                    from "EmailAddress" EA where "personId"="Person".id),
               'emailAddresses', (select array_agg(value) from "EmailAddress" EA where "personId"="Person".id),

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (752171f) 95.06% compared to head (f179a80) 95.12%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
+ Coverage   95.06%   95.12%   +0.06%
==========================================
  Files           8        8
  Lines         243      246       +3
==========================================
+ Hits          231      234       +3
  Misses         12       12

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


Hash HelmRepository index contents instead of relying on 'generated' timestamp

Ran in to an interesting issue earlier where a Helm repo was updating the contents of their index.yaml but not updating the generated timestamp. My reading of the source is that this causes the source controller to ignore the changes.
Clearly, forgetting to update this value is a bug on the part of the repository maintainer. However it strikes me that we could avoid this relatively easy to make mistake by hashing the contents of the index.yaml and updating if the hash changes.
Thoughts? Happy to submit a PR.

For example Chart Releaser does not update the generated timestamp field, #90.

For example Chart Releaser does not update the generated timestamp field, #90.

I'm in favour of this change, we do use checksums for the Bucket source type, we could do it for the HelmRepository artifact as well.

I'm in favour of this change, we do use checksums for the Bucket source type, we could do it for the HelmRepository artifact as well.

@Niksko a PR for this would be greatly appreciated!  I find that I need to manually reconcile most of my HelmRepositories due to this behavior.

@Niksko a PR for this would be greatly appreciated!  I find that I need to manually reconcile most of my HelmRepositories due to this behavior.


Add pypa/gh-action-pypi-publish

https://github.com/pypa/gh-action-pypi-publish

@webknjaz your changes have been merged, thanks for your contribution 👍


Failed to Beepons. Script exited

Hi
I am running the script from windows cmd. The script is exited after some time. Do I need to manually run the script again? What should I do about this?

Is this a default behavior?

I am not sure why there's a word 'beepons' in it. Did you happen to modify the code before running?

I did not change anything. I executed the exe same issue was happening there.


I've And Idea To Reduce Big Size Of Produced Files

Feature request type
Performance improvement
Component name
No response
Is your feature request related to a problem?
Hello ,
I've Built an e commerce website (www.elmoul.com) fully based on mudblazor
But I'm facing issue in loading time due to mudblazor package is very large about
1 Mb regardless the css files  some issues in my software architecture.
Describe the solution you'd like
If it possible to make the package spitted into files and loaded by demand
for example I need to load component like mudcard, button , text in landing page
so I need them to load in first load, once the website loaded I can load other
components also on demand if possible
Have you seen this feature anywhere else?
Like Angular Lazy Components Lazy Loading
Describe alternatives you've considered
No response
Pull Request

[X] I would like to do a Pull Request

Code of Conduct

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

I think this is duplication of this issue #2424. As I understand, the biggest problem right now are the built-in Icons that increase the package size.


Added support for VS2015

I've removed the dependency on the .NET Framework since it's irrelevant for shipping snippets.
This pull requests fixes bug #14

Thanks!


Spotify API Error (unknown): received an empty response

same issue here. did you find any fix?

Also getting this error output before each song my Zotify downloads. I'm running Windows 11, and it seems to still download the songs, it just takes longer than on my Arch Linux pc Zotify installation.

I am getting the same error on my mac

+1

+1

same here


Move shutdown/reboot handling to manager

Closes #22821
We should probably add something to boardd to put the panda back into elm/no output mode when it's shut down cleanly. Think that should go in a separate PR.

I think we want to keep it short. What about "Disengage to Reboot"?
A side note: we should have a timeout on the settings when onroad. All the dialogs already handle this, so it should be as easy as adding a QTimer like we do for turning off the display.


Create Upload screenshoot

https://github.com/GSSoC24/Postman-Challenge/pull/4285

@aslams2020 did it. Please could it be considered. Thank you

Thank yo so much. Omg the last minutes were so challenging. Really appreciate the fast support with my issue.


fix for azure open ai not returning content sometimes for the when stopping.

When making hundreds of async calls to azure openai, sometimes the response is missing content. {"finish_reason":"stop", ...}
This happens in  _create_chat_result(). From here _convert_dict_to_message() is called and _dict["content"] is accessed.
This leads to very long chain crashing. This error is not caught by any retry mechanism.

hmmm do we know why the response is missing content?

hmmm do we know why the response is missing content?

not really. It happens rarely.

believe this has been fixed


Accordion

Can we auto-focus the Accordian body once open.
I seen there is API exposed for the same, but unable to achieve as none of the standard scrolling tricks responds.
Can you please suggest the idea for the same.
Thanks,

What do you mean by auto-focus the Accordian body?

Once I click on the Accordion header, It opens the same and there is hook along with this.
My concern is whenever the accordion opens the body gets slides downwards, which is expected; but most of the user's didn't get there is change happened If it open within off-canves (last element on the list); It opens and manually we have to scroll little up.
I tried to use many plugins.tricks (get the offset from top), and forcefully pushes the body/container upwards. Bad luck .. unable to achieve the same.

Shouldn't be hard:
$$('.accordion-item').on('opened', function(){
    var pageContent = $$(this).parents('.page-content');
    var newScroll = $$(this).offset().top - 44 + pageContent[0].scrollTop;
    $$(this).parents('.page-content').scrollTop(newScroll, 400);
});

And please, use Forum for questions


=act Mark TaskHolder a final class.

This class is visible from Java and should be final.

I don't see why a protected class should be final. I don't see the point in this.

wo, yes.


Promise.resolve the prepare function

- route.prepare(routeData).then(function() {
+ Promise.resolve(route.prepare(routeData)).then(function() {

This lets you support sync and async prepare methods.
Depends on #13. Competes with https://github.com/Betterment/backbone.blazer/pull/12#discussion_r25262538

done in: https://github.com/Betterment/backbone.blazer/commit/db9a6a0fcc7cc3a528648021e1871a15fc290736
will replace with Promise.resolve once we do #13


audio: expose UnplayedBufferSize

This would be useful especially to implement a real-time audio streaming. With UnplayedBufferSize, the application can adjust the amount of the bytes to send to Ebiten's Audio player (io.Reader).

Or perhaps enabling to adjust the buffer size in Oto (0.25[s] currently) might be enough. Let's see...


panic: opengl: mapPixelBuffer failed

I get this error when I run my code, which runs fine (the graphics work) until I put on screen the images of my main user interface, at which it crashes and I get:
goroutine 19 [running]:
github.com/hajimehoshi/ebiten/internal/graphicsdriver/opengl.(*pboState).mapPBO(0x784f58, 0x1577be0)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/graphicsdriver/opengl/pbo_desktop.go:46 +0x138
github.com/hajimehoshi/ebiten/internal/graphicsdriver/opengl.(*Image).ReplacePixels(0x1577be0, 0x17e60f8, 0x2, 0x2)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/graphicsdriver/opengl/image.go:115 +0x58
github.com/hajimehoshi/ebiten/internal/graphicscommand.(*replacePixelsCommand).Exec(0x15ec130, 0x3c, 0x0, 0x0)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/graphicscommand/command.go:447 +0x44
github.com/hajimehoshi/ebiten/internal/graphicscommand.(*commandQueue).Flush(0x7853b0)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/graphicscommand/command.go:265 +0x388
github.com/hajimehoshi/ebiten/internal/graphicscommand.FlushCommands(...)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/graphicscommand/command.go:298
github.com/hajimehoshi/ebiten/internal/restorable.ResolveStaleImages()
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/restorable/images.go:55 +0x20
github.com/hajimehoshi/ebiten/internal/shareable.EndFrame(0x140c0c0, 0x14733c0)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/shareable/image.go:546 +0x20
github.com/hajimehoshi/ebiten/internal/buffered.EndFrame(...)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/buffered/image.go:43
github.com/hajimehoshi/ebiten.(*uiContext).Update(0x1490750, 0x17fccb0, 0x0, 0x0)
	/home/###/go/src/github.com/hajimehoshi/ebiten/uicontext.go:129 +0x28c
github.com/hajimehoshi/ebiten/internal/uidriver/glfw.(*UserInterface).update(0x5b8948, 0x3d4e28, 0x1490750, 0x0, 0x0)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/uidriver/glfw/ui.go:846 +0x1a8
github.com/hajimehoshi/ebiten/internal/uidriver/glfw.(*UserInterface).loop(0x5b8948, 0x3d4e28, 0x1490750, 0x0, 0x0)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/uidriver/glfw/ui.go:888 +0xac
github.com/hajimehoshi/ebiten/internal/uidriver/glfw.(*UserInterface).run(0x5b8948, 0x500, 0x400, 0x0, 0x3ff00000, 0x35face, 0x12, 0x3d4e28, 0x1490750, 0x5bd7f8, ...)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/uidriver/glfw/ui.go:746 +0x140
github.com/hajimehoshi/ebiten/internal/uidriver/glfw.(*UserInterface).Run.func1(0x14707c8, 0x148f980, 0x5b8948, 0x500, 0x400, 0x0, 0x3ff00000, 0x35face, 0x12, 0x3d4e28, ...)
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/uidriver/glfw/ui.go:610 +0xa4
created by github.com/hajimehoshi/ebiten/internal/uidriver/glfw.(*UserInterface).Run
	/home/###/go/src/github.com/hajimehoshi/ebiten/internal/uidriver/glfw/ui.go:607 +0x150

It seems like it's an OpenGL issue, but I can't find documentation for this online, and the error isn't very verbose.

It would be nice if I could see which part of my code causes this issue, or even to get an error returned instead of a panic.

Update: After running go get github.com/hajimehoshi/ebiten/... (after backing up the current version as I use it in a production environment), my issue was resolved. I noticed that the file that raised that panic no longer exists in the stable version. Thankyou!


Error trying to parse csv input with PSAT CLI v0.10

Describe the bug
Sitemap and URL input on the command line produce a PSAT report, but passing in a csv file results in an error
To Reproduce
Steps to reproduce the behavior:

Download and install PSAT CLI v0.10
Run npm run cli -- -c ./path/to/any/csv
See the error: "Error: ./path/to/csv is not valid numeric value."

Expected behavior
Aggregated PSAT report is generated
Screenshots
In this screenshot I show the csv file does exist in my directory structure using the ls command. I also gave the CLI a numeric value just to see what happens, but then I get an error that path must be a string (expectedly). I also run a basic URL and the sitemap URL just to show the other inputs are good

d
Desktop (please complete the following information):

OS: MacOS Sonoma 14.5
Browser : Chrome
Version : 127

Additional context
Add any other context about the problem here.

Hello @Dbrown910
Thank you for doing a thorough test and reporting back to us with the issues you encountered. As we checked your description and screenshot, we found that you are using old CLI options. -c
The version 0.10.0 introduced some changes to the CLI options. The -c option is now replaced by the -f option.
With the -f option, you can specify the file path to CSV or sitemap. E.g., npm run cli -- -f /path/to/any.csv or npm run cli -- -f /path/to/any.xml
Please consider using -h or --help to learn about new options while running PSAT CLI, or please check our updated wiki page.

Ahh okay! Thanks for clarifying @milindmore22 !


cmd/scollect: Do not reuse haproxy stats connections.

@kylebrandt @mjibson

See also: https://github.com/golang/go/issues/8946


latest aiohttp support for 5.x branch

Issue #37 was fixed for elasticsearch-py-async versions 6.x and above.
Version 6.0.0 depends on elasticsearch 6.0.0, with has compatability with ElasticSearch 6.x
In real world, Elasticsearch 5.x is widely installed, which means elasticsearch-py-async 5.x is needed to talk to ES 5.x cluster.
Using 5.x causes the same issue as #37 with latest aiohttp.
Is there a way to get the Timeout to async_timeout change implemented for 5.x branch?
In essence, similar to how https://github.com/elastic/elasticsearch-py package is available for different version of ElasticSearch, similarly, the async version of the client should be available.
I am willing to contribute.

The 5.x branch is no longer supported so I'm closing this issue.


Update links in Introduction

Update link to official demo config and sharec package.

Feel free to reopen the PR, when you'll be ready to finish the work :)


Create CNAME

Create a custom domain for my portfolio website

Accepted the addition of the CNAME file to create a custom domain name for my portfolio website


Sync and Downloads updates

This is a big change removing references to the old sync jobs and make available offline and replacing it with download
@ebr11 @LukePulverenti @softworkz Please review the changes and let me know of any corrections / changes I need to do
I have kept md filenames as they are - only changed titles
Not sure if we should add the desktop app as one allowing offline downloads - at the moment we only mention ios and android
But offline for laptops is a valid use case

[!NOTE]
To make changes, simply make additional commits and push to the same branch.
(in case you didn't know how GitHub expects this to work :wink )

@softworkz  I have made the changes and pushed up to my repo. For some reason ended up as 2 new fixup commits instead of just one
As you have gathered I am an expert on github - so I am not sure how to add these 2 fixup commits to this draft PR
The 2 new commits are
https://github.com/EmbySupport/Emby.Docs/commit/0c539f7cab84b3da8c4d81b3e957752df7fb42a8
https://github.com/EmbySupport/Emby.Docs/commit/a127e2f6c23c9650c86fc0c075b27a904fbe7e07
Let me know if you are happy with the changes and if you are, let me know what I need to click to make it proper PR for nerging - but it must include all 3 commits

Oh - sorry - all 3 commits are linked to this draft PR

OK once this moves out of draft status I will merge. Thanks.

Converted to a regular PR.
@LukePulverenti - you can hit the button now.


npm package

A npm package would be cool!
If you need help we can work on it during a pomodoro™® studio

Released on npm.

waaaaa


add outDir property as dist

Thanks for contribution but I can't merge this. It's redeclaring the same thing and in a wrong place.

Didn't see that, sorry. But have a look at the other pr.


Model undefined not found?

Sorry I  put the model to the right directory, but it still isnt found.
E:\ComfyUI_windows_portable\ComfyUI\models\llm\mistral-7b-exl2-b5
The model is not renamed .  Its called output.safetensors
What am i missing ?
THX in Advance....
Metamountain

Do you mean you're not seeing the model listed in the loader node? Do you have a config.json and other files in the mistral-7b-exl2-b5 folder, next to output.safetensors? If you're not using git to clone the model repo you should download all the files in it manually and place them in the same folder.

No, All done correctly i  had hoped....

That seems correct except there shouldn't be anything in the gpu_split field if you're not using that. Make sure your nodes are up to date, then remove and add the loader node again.

Thx for you immidiate help. Unfortunately still not working, even after reinstalling from scratch...

That's quite strange, looking at the screenshot above I'm pretty sure your nodes were just outdated. I assume there are no errors in the console? If you're using the manager to install nodes try cloning the repo manually instead. Also did you try opening the example workflow from the readme?

Sure !

Yuuhuu!!!!!!!!!!  You save me. Had to delete the custom_node and install it manually. there was another llama node i also deleted...
Thank you very much !


Fix failure in SearchCancellationIT.testMSearchChildReqCancellationWi…

…thHybridTimeout
In some cases as one shared with issue #1099, the maxConcurrentSearchRequests was chosen as 0 which
will compute the final value during execution of the request based on processor counts. When this
computed value is less than number of search request in msearch request, it will execute all the
requests in multiple iterations causing the failure since test will only wait for one such
iteration. Hence setting the maxConcurrentSearchRequests explicitly to number of search requests
being added in the test to ensure correct behavior
Signed-off-by: Sorabh Hamirwasia<EMAIL_ADDRESS>Description
Port of https://github.com/opensearch-project/OpenSearch/pull/1103 to 1.1 branch
Issues Resolved
Test failure
Check List

[N/A] New functionality includes testing.

[Y] All tests pass

[N/A ] New functionality has been documented.

[N/A] New functionality has javadoc added

[Y] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check


[TfL Elizabeth Line] Add Barking Riverside

Hi David,
First of all, I would like to commend you for making such an outstanding software, I really enjoy pranking my class.
I would like to suggest a new station to be added to the TfL Elizabeth Line system. Barking Riverside recently got introduced to the London Overground.
I often play a Train Simulator called Train Sim World 4 and I get the announcements using your website to improve my experience. It features the London Overground from Gospel Oak - Barking Riverside. That’s why I am suggesting this.
I hope you are doing alright!
Best regards,
Transport5040

TfL does not provide a file for Barking Riverside (BGV), so it is not on the website. You can search for 'BGV' in the audio files for the Elizabeth Line yourself, but it is not there.


Is there any way I can collapse the child options?

Hello, just wondering what the best way to collapse all the child options on init, I have too many options , it will be nice if I can hide the child options and  only show the parent options,  the user will click the eahc parent option and it will drop down the child options.
thanks!

This functionality isn't built in, but you could probably manage it with css and some additional js
This assumes you have your parents with isLabel set to true.
e.g.
js
document.querySelectorAll(".dropdown-header").forEach(header => {
  header.addEventListener("click", event => { var li=event.target.nextSibling; while(li && li.classList && !li.classList.contains("dropdown-header")) { li.style.display=li.style.display==="none"?"block":"none"; li = li.nextSibling; }})
})

css
li.dropdown-header {
    display: block;
    cursor: pointer;
}
.dropdown-item {
    display: none;
}

You could also look at the change event (ngModelChange)="onChange($event)" and update your model to dynamically insert and remove the children when a parent is clicked, but this seems more trouble than it's worth.

Hello, thanks for the help, I am very new to angular.
I am just wondering if I can apply a bootstrap collapse class to the options, like use ngIf to  find the parents and add the bootstrap collapse class?
thanks

You can't use bootstrap collapse because the parent items in the dropdown are not container elements for the children.


Make it possible to define generic visitors

Is your feature request related to a problem? Please describe.
There is a high possibility that visitors with the same "shape" will be reused a lot. This can partially be solved by #68 , but I think it'd still be nice to be able to generate generic visitors.
Describe the solution you'd like
There should be a possibility to reference to the defined generic parameters. The problem is that attributes can't reference the generic parameters declared inside the classes:
[SyntaxTreeVisitor(typeof(Node), ReturnType = typeof(TValue))] // error CS0416: 'TValue': an attribute argument cannot use type parameters
public abstract partial class MyVisitorBase<TValue> { /* ... */ }

Describe alternatives you've considered
A very simple solution could be to introduce special marker types to reference the first, second, third, ... (8 should be plenty) generic parameters, like so:
[SyntaxTreeVisitor(typeof(Node), ReturnType = typeof(GenericParam.P1))] // P1 will reference TValue
public abstract partial class MyVisitorBase<TValue> { /* ... */ }

Another simple solution is referencing them by name in a separate property on the attribute. The advantage here is that nameof works from the outside:
[SyntaxTreeVisitor(typeof(Node), GenericReturnType = nameof(TValue))]
public abstract partial class MyVisitorBase<TValue> { /* ... */ }

This latter case would only complicate in the fact that ReturnType and GenericReturnType have to be orthogonally specified.

First, I think this is a good idea, and something I've thought too.
Regarding the alternatives, I don't really like the first alternative because it is limited to an arbitrary number (it's very likely 8 will be enough for most cases, but why go for a limit when you can avoid?) and also because it doesn't use the generic type parameter name. Using position (P1, P2...) could result in hard to debug errors if the order of the generic type arguments changes later, and it also makes reading a bit more obscure.
On the other hand, the second alternative completely removes the arbitrary number limit and directly references the generic type argument by name. Even if the use of nameof is required, it is not completely counterintuitive as many other arguments already require the user to know typeof, and the typecheck of CSC will also disallow people from wrongly passing a type instead of a string. Furthermore, there exist lints in Microsoft analyzers (I've seen them before) that would suggest the user to use nameof if they try to write the name of a type verbatim as a string literal.


Backport support for Graviton 4 to upstream rnaseq

This PR brings the support for Graviton 4

Closing in favour of https://github.com/nf-core/rnaseq/pull/1414


composer validate warnings

When using composer validate against the composer.json of WordPress Bedrock,
some warnings are emitted:
See https://getcomposer.org/doc/04-schema.md for details on the schema
require.johnpbloch/wordpress : exact version constraints (4.7.2) should be avoided if the package follows semantic versioning
require.roots/wp-password-bcrypt : exact version constraints (1.0.0) should be avoided if the package follows semantic versioning

There's already a PR open about this: https://github.com/roots/bedrock/pull/290


Sign the assemblies that we publish

We have had a request from @gregsdennis to sign the assemblies, so that we can support upstream users who have signed assemblies, without producing a warning.

Could you or @gregsdennis clarify whether this is a request to:

make these strongly-named assemblies with a corresponding signature (i.e., nothing certificate based, just the old-school strong name signature that we're now all encouraged not to rely on)
embed an authenticode code signature in the actual DLLs themselves
generate a signature for the NuGet package

Or Some combination of the above?
I'm not clear on what exactly is blocking upstream users, so I'm not sure exactly what will be required to support them properly.

The first one.  I've been signing my packages forever because it was requested of me long ago. And the system still complains when you reference an unsigned package from a signed one.
I am curious why it's discouraged now, though.

https://learn.microsoft.com/en-us/dotnet/standard/assembly/strong-named#why-strong-name-your-assemblies
Specifically
For .NET Core and .NET 5+, strong-named assemblies do not provide material benefits. The runtime never validates the strong-name signature, nor does it use the strong-name for assembly binding.

If you are an open-source developer and you want the identity benefits of a strong-named assembly for better compatibility with .NET Framework, consider checking in the private key associated with an assembly to your source control system.

Personally, I suspect that most users who are operating in a strong-named environment are unaware that there are no benefits in .NET Core and that there are no security benefits in .NET Framework.
Forcing people to acknowledge and disable the strong-name warnings strikes me as a good way to educate people about the fact that they probably don't want to strong-name sign their assemblies in this way.

Fair enough, but the signing checks do still exist in Framework, and we're publishing Standard-compatible code to be used in Framework.
Moreover, there are users (specifically in Framework) who require signed packages through company policies.  We can try to educate them, but until they change those policies, they won't use our packages if they're not signed.
As it stands, I'm still signing my packages. I have committed the signing file, though.


resize the image

the photo from 360 camera generally provide the over sized image for quick loading from the web.
provide a option to reduce the image size to 1024px(to be determined)

https://github.com/lovell/sharp

choose 2048 instead


remove max_coins field

@gregLibert the goal of this Pr is to remove max coins and be able to deploy on buildnet.
To summarise:

After this I'll release a new npm version so the lastest will be compatible with buildnet
then we will add again support for max coins so the dev version will be compatible with the last tetstnet


DNR Redirects Fail

Any DNR redirect I set up simply fails to go anywhere. See screen recordings below.
https://github.com/user-attachments/assets/c3b3204e-ce3e-4f70-b8fd-7ca51e0c682e
https://github.com/user-attachments/assets/520a2e21-47a1-4079-a57d-f75e587beedc

Hi @karimroshan, I believe that your issue is discussed in #48. Would you please take a look? (Sorry, I forgot to add the DNR label to that issue.)
TL;DR: This should be a bug in Safari, which Redirect Web can't control. As of now, I recommend using the Original type rather than DNR. Sorry for the inconvenience.

Hello. Agreed on that one. I will use original for search redirect. Problem is, it also doesn't work for normal links. See screen recording below.
With Original:
https://github.com/user-attachments/assets/7116f9cd-6f34-4ded-b1f3-f0fdb100ac8b
With DNR:
https://github.com/user-attachments/assets/3223d202-9f43-42ff-ada5-194a359660a4
(it works on a typed link, but NOT a clicked one--loads forever)
Thanks,
Roshan

Oh, sorry, I haven’t checked your second video yet.
I see. It does look strange. I’ll check it soon.
By the way, it might be worth trying to re-enable the extension from Settings > Extensions or restarting your Mac.
Additionally, I'd like to note that the DNR type is still an experimental feature. It has multiple issues, as mentioned in the documentation, primarily due to Safari’s implementation.

Yeah, no worries on this one, just wanted to let you know.

Thanks, I've successfully replicated the issue on my end on both iPhone and macOS.
I have submitted feedback to Apple as FB16160560, including a minimal reproducible example as follows:
example.zip
The following code sets a DNR rule for redirecting from Reddit to example.com:

```js
// background.js
(async () => {
    await browser.declarativeNetRequest.updateSessionRules({
        removeRuleIds: [1],
        addRules: [
            {
                id: 1,
                priority: 1,
                action: {
                    type: "redirect",
                    redirect: {
                        url: "https://example.com/",
                    },
                },
                condition: {
                    regexFilter: "https://www.reddit.com/.*",
                    resourceTypes: ["main_frame"],
                },
            },
        ],
    })
    console.log("Background script loaded")
})()
```

Using this rule, I expected a link from Reddit to redirect to example.com. However, in practice, Safari does not transition to example.com.

Please refer to the attached videos (`demo-macos_.mov` and `demo-ios_.mov`) demonstrating the issue on each platform.

Additionally, I have attached a minimal reproducible example to further illustrate the problem. I hope it assists in your investigation.

### Reproduction Steps

1. Download and extract the attached `example.zip`.
2. Build and run the project using Xcode, then enable the extension in Safari.
3. Search for "Reddit" on Google.
4. Click on a link from the search results.

Then:

- Expected Behavior: Following the redirection rule set by the extension, Safari navigates to example.com.
- Actual Behavior:
  - On macOS: The address bar displays infinite loading, but Safari does not navigate to example.com.
  - On iOS: Clicking the link has no effect; the browser remains idle.

### Environment

- macOS 15.2 (24C101)
- Safari
    - Technology Preview 210
    - 18.2
- Xcode 16.2 (16C5032a)

### Context of the Issue

One of the users of my Safari extension, Redirect Web for Safari, reported this issue. I have been able to replicate it as well. You can find the details here:
https://github.com/mshibanami/redirect-web/issues/64


Snap: Project directory always changes on app start producing erros

Description
Hi. I'm using project directory that resides on separate mounted storage and DBeaver on every start reports: Resource does not exists. DBeaver can't load project directory and all scripts because directory with prefix path /run/user/1000/doc always changes on app start. I need to manually remove project and reload it again setting correct path.
Parts of path that marked with bold are variable and can be different:
/run/user/1000/doc/5544c4ec/
/run/user/1000/doc/6d79da0d
DBeaver Version
Community 22.2.1
Operating System
Ubuntu 22.10
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Seems this is xdg portal issue.
Workaround for bypassing portal and using direct access: sudo snap connect dbeaver-ce:removable-media


Add includeHiddenTests parameter to setTestFilter(toMatch:) APIs.

Motivation:
When constructing a Configuration instance, the public setTestFilter(toMatch:) API can be used to specify a particular set of test IDs that should be executed. Today, the implementation includes tests that are annotated with the internal .hidden trait. This PR changes the implementation to instead exclude these hidden tests.
Modifications:
I added an internal variant of the setTestFilter(toMatch:) API that has an additional parameter, includeHiddenTests. If false is passed as an argument to this parameter, a test annotated with the .hidden trait will not be included, even if its ID is present in the set of IDs (or if it's a hidden descendant of a non-hidden suite whose ID is present in the set).
Result:
I added a unit test to verify this new functionality.

@swift-ci please test


fix error with pyserial and ignore some errors

allow write command to serial console with read() function then doing a read.
fix read_page size (16) always different then expect (2048)
ignore some read errors to prevent some parsing errors.

tested in Ubuntu 16.04.7 (Python 3.6 + pySerial v3.4) with cmd:
$ python3 bcm_cfedump.py -D /dev/ttyUSB0 -O nand.bin -t 0.01 nand

I did not test the nand_bulk command.

Hi @Depau  please review this PR. thanks

Hi @danitool , @Noltari  please help to review this PR, thanks.

Hi,
Alright, I see. Though feel free to make improvements, don't be afraid to
make "too many changes" if they're actually meaningful.
Like for example here i really had a hard time figuring out what was the
difference between the two ifs, so if you think it helps feel free to also
change the rest of the code as you see fit to make it more readable and
compatible with recent python versions.
For what it matters to me this code should be compatible with the latest
stable release of python all the way to whatever version of python the
latest Ubuntu release is using. All older versions may work on a best
effort basis.
Anyway, thanks for the contribution and sorry for the very late/lazy
reviews, I quickly wrote this script cause I needed it at the time but i
barely remember any details.
On Tue, Dec 15, 2020, 03:07 hpo14<EMAIL_ADDRESS>wrote:

@hpo14 commented on this pull request.
In bcm_cfedump.py
https://github.com/Depau/bcm-cfedump/pull/3#discussion_r542989451:

@@ -311,6 +328,9 @@ def init(self, serial: serial.Serial, block_size: int = BLOCK_SIZE, page_siz
self.ser = serial

 def _read(self, *a, **kw) -> bytes:

   if not type(*a) is int:

       self.ser.write(*a, **kw)

With pySerial v3.4 Spec, the read function only accept "Read Length" as
parameter.
https://pyserial.readthedocs.io/en/latest/pyserial_api.html#serial.Serial.read
For example: you code on Line 328 0r 331
self._read(b"\r\n")
combine a write action then follow a read action
I dont want making too much change to the original, so i wrote
    if not type(*a) is int:
        self.ser.write(*a, **kw)

to identiy the "read" is a pure read or a combine write+read function call.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Depau/bcm-cfedump/pull/3#discussion_r542989451, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIZCWNBMEFZ4DC4AWJPPEDSU3AG5ANCNFSM4T5CB3CQ
.


Integrate site license statement

As an author, I want to add a license statement to my site.
Would you accept a pull request that integrates the hexo-global-license statement?

no, no i would not

no, no i would not


Add support for optional package name

In case a package has a different name than the library, the path might never be found.
Case in point odc (in pyodc project):

Library name: odccore
Library filename: libodccore.{so,dylib}
Supported environment variables: ODC_DIR, odc_DIR

By providing an optional package name as a second parameter to find(), we can check expected environment variables. Additionally, support for both upper and lowercase variable names has been added and search extended to two architectures (lib and lib64).
Test suite based on pytest and pytest-mock was also added, since the module lacked any tests.
The README was suitably extended to document the new behaviour.q

 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.


did:pkh support

did:pkh meets the guidelines for did method selection we've outlined. Would be great to add support:
https://github.com/w3c-ccg/did-pkh/blob/main/did-pkh-method-draft.md

Anyone implementing or assessing verite with did:pkh use-cases or architectures in mind can reach out to me on discord, i'm #1488 there.

Consider using did-kit libs

All good mate.

Use this: https://www.spruceid.dev/didkit/didkit-packages/javascript


improve PHP 8.2 support

This builds upon #796 and removes the custom logic implemented there in process(), by fixing the commands string as suggested in https://wiki.php.net/rfc/deprecate_partially_supported_callables#backward_incompatible_changes

Just out of curiosity, why not keep it the way @driesvints did it?

Just out of curiosity, why not keep it the way @driesvints did it?

Not really important but just because that contains some extra logic that can be avoided, specifically the call to strpos, explode that kind of always runs.
In construct we build strings with 'static::<method>' just so that we will replace them with [$this, '<method>'] before execution. I mean, we could build them directly like that.
Also, as those are static methods I believe it would have been a bit more accurate to have them as [static::class, '<method>'].

I actually didn't change that commands array because I thought it could be a breaking change if the contents were changed. But if the authors here are okay with it then go for it 👍

I actually didn't change that commands array because I thought it could be a breaking change if the contents were changed. But if the authors here are okay with it then go for it 👍

Thanks for explaining.
However, KeyPrefixProcessor::$commands is private and there are no getters so I can't see how that would be a breaking change.