Critical logs appear when configuring the remote Wasm module

I wrote two wasm EnvoyFilter and applied them to samples/httpbin.
When I use local wasm file, they run normally and meet my expectations.
But now I want to use remote wasm file, so I have made the following modifications:
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: httpbin-wasm
  namespace: default
spec:
  workloadSelector:
    labels:
      app: httpbin
  configPatches:
  - applyTo: BOOTSTRAP
    patch:
      operation: MERGE
      value:
        bootstrap_extensions:
        - name: envoy.bootstrap.wasm
          typed_config:
            "@type": type.googleapis.com/envoy.extensions.wasm.v3.WasmService
            singleton: true
            config:
              name: my_plugin
              configuration:
                "@type": type.googleapis.com/google.protobuf.StringValue
                value: "main"
              vm_config:
                vm_id: main_thread
                runtime: "envoy.wasm.runtime.v8"
                code:
                  remote:
                    http_uri:
                      uri: https://raw.githubusercontent.com/HZHgagaga/wasm/main/main_thread.wasm
                      cluster: remote_wasm
                      timeout: 3s
                    sha256: "6940d386f9d9360a3d4aaa10511c1efa179b9e7341e5f2385acf819195b6da9d"
  - applyTo: HTTP_FILTER
    match:
        context: SIDECAR_INBOUND
        listener:
          portNumber: 80
          filterChain:
            filter:
              name: "envoy.filters.network.http_connection_manager"
              subFilter:
                name: "envoy.filters.http.router"
    patch:
      operation: INSERT_FIRST
      value:
        name: envoy.filters.http.wasm
        typed_config:
          "@type": "type.googleapis.com/envoy.extensions.filters.http.wasm.v3.Wasm"
          config:
            vm_config:
              vm_id: work_thread
              runtime: envoy.wasm.runtime.v8
              code:
                remote:
                  http_uri:
                    uri: https://raw.githubusercontent.com/HZHgagaga/wasm/main/work_thread.wasm
                    cluster: remote_wasm
                    timeout: 3s
                  sha256: "975be293190520c97ef63a32e152f720b2b799178dc9135064baddd3bdffa7be"
            configuration:
              "@type": "type.googleapis.com/google.protobuf.StringValue"
              value: "work"
  - applyTo: CLUSTER
    match:
      context: SIDECAR_OUTBOUND
    patch:
      operation: ADD
      value:
        name: consul
        type: STATIC
        connect_timeout: 1s
        lb_policy: ROUND_ROBIN
        load_assignment:
          cluster_name: consul_service
          endpoints:
          - lb_endpoints:
            - endpoint:
                address:
                  socket_address:
                    address: <IP_ADDRESS>
                    port_value: 8500
  - applyTo: CLUSTER
    match:
      context: SIDECAR_OUTBOUND
    patch:
      operation: ADD
      value:
        name: remote_wasm
        type: STRICT_DNS
        connect_timeout: 1s
        dns_refresh_rate: 5s
        dns_lookup_family: V4_ONLY
        lb_policy: ROUND_ROBIN
        load_assignment:
          cluster_name: remote_wasm_service
          endpoints:
          - lb_endpoints:
            - endpoint:
                address:
                  socket_address:
                    address: raw.githubusercontent.com
                    port_value: 443
        transport_socket:
          name: envoy.transport_sockets.tls
          typed_config:
            "@type": type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.UpstreamTlsContext
            sni: "raw.githubusercontent.com"

httpbin:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: httpbin
---
apiVersion: v1
kind: Service
metadata:
  name: httpbin
  labels:
    app: httpbin
    service: httpbin
spec:
  ports:
  - name: http
    port: 8000
    targetPort: 80
  selector:
    app: httpbin
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: httpbin
spec:
  replicas: 1
  selector:
    matchLabels:
      app: httpbin
      version: v1
  template:
    metadata:
      labels:
        app: httpbin
        version: v1
    spec:
      serviceAccountName: httpbin
      containers:
      - image: docker.io/kennethreitz/httpbin
        imagePullPolicy: IfNotPresent
        name: httpbin
        ports:
        - containerPort: 80
      - name: istio-proxy
        image: auto
        env:
          - name: "BOOTSTRAP_XDS_AGENT"
            value: "true"

istio version:
client version: 1.12.5
control plane version: 1.12.5
data plane version: 1.12.5 (10 proxies)

But when I used the above configuration, the following logs were output:
$ kubectl logs httpbin-6d84d49667-gqdrd -c istio-proxy
2023-05-10T03:16:17.382429Z     info    FLAG: --concurrency="2"
2023-05-10T03:16:17.382459Z     info    FLAG: --domain="default.svc.cluster.local"
2023-05-10T03:16:17.382464Z     info    FLAG: --help="false"
2023-05-10T03:16:17.382467Z     info    FLAG: --log_as_json="false"
2023-05-10T03:16:17.382469Z     info    FLAG: --log_caller=""
2023-05-10T03:16:17.382472Z     info    FLAG: --log_output_level="default:info"
2023-05-10T03:16:17.382474Z     info    FLAG: --log_rotate=""
2023-05-10T03:16:17.382477Z     info    FLAG: --log_rotate_max_age="30"
2023-05-10T03:16:17.382479Z     info    FLAG: --log_rotate_max_backups="1000"
2023-05-10T03:16:17.382483Z     info    FLAG: --log_rotate_max_size="104857600"
2023-05-10T03:16:17.382486Z     info    FLAG: --log_stacktrace_level="default:none"
2023-05-10T03:16:17.382492Z     info    FLAG: --log_target="[stdout]"
2023-05-10T03:16:17.382495Z     info    FLAG: --meshConfig="./etc/istio/config/mesh"
2023-05-10T03:16:17.382497Z     info    FLAG: --outlierLogPath=""
2023-05-10T03:16:17.382500Z     info    FLAG: --proxyComponentLogLevel="misc:error"
2023-05-10T03:16:17.382503Z     info    FLAG: --proxyLogLevel="warning"
2023-05-10T03:16:17.382506Z     info    FLAG: --serviceCluster="istio-proxy"
2023-05-10T03:16:17.382509Z     info    FLAG: --stsPort="0"
2023-05-10T03:16:17.382511Z     info    FLAG: --templateFile=""
2023-05-10T03:16:17.382515Z     info    FLAG: --tokenManagerPlugin="GoogleTokenExchange"
2023-05-10T03:16:17.382518Z     info    FLAG: --vklog="0"
2023-05-10T03:16:17.382521Z     info    Version 1.12.5-6332f0901f96ca97cf114d57b466d4bcd055b08c-Clean
2023-05-10T03:16:17.382646Z     info    Proxy role      ips=[<IP_ADDRESS>] type=sidecar id=httpbin-6d84d49667-gqdrd.default domain=default.svc.cluster.local
2023-05-10T03:16:17.382721Z     info    Apply proxy config from env {}

2023-05-10T03:16:17.383780Z     info    Effective config: binaryPath: /usr/local/bin/envoy
concurrency: 2
configPath: ./etc/istio/proxy
controlPlaneAuthPolicy: MUTUAL_TLS
discoveryAddress: istiod.istio-system.svc:15012
drainDuration: 45s
parentShutdownDuration: 60s
proxyAdminPort: 15000
serviceCluster: istio-proxy
statNameLength: 189
statusPort: 15020
terminationDrainDuration: 5s
tracing:
  zipkin:
    address: zipkin.istio-system:9411

2023-05-10T03:16:17.383802Z     info    JWT policy is first-party-jwt
2023-05-10T03:16:17.429151Z     info    CA Endpoint istiod.istio-system.svc:15012, provider Citadel
2023-05-10T03:16:17.429186Z     info    Opening status port 15020
2023-05-10T03:16:17.429222Z     info    Using CA istiod.istio-system.svc:15012 cert with certs: var/run/secrets/istio/root-cert.pem
2023-05-10T03:16:17.429308Z     info    citadelclient   Citadel client using custom root cert: istiod.istio-system.svc:15012
2023-05-10T03:16:17.444004Z     info    ads     All caches have been synced up in 64.740906ms, marking server ready
2023-05-10T03:16:17.444277Z     info    sds     SDS server for workload certificates started, listening on "etc/istio/proxy/SDS"
2023-05-10T03:16:17.444300Z     info    xdsproxy        Initializing with upstream address "istiod.istio-system.svc:15012" and cluster "Kubernetes"
2023-05-10T03:16:17.444312Z     info    sds     Starting SDS grpc server
2023-05-10T03:16:17.444630Z     info    Pilot SAN: [istiod.istio-system.svc]
2023-05-10T03:16:17.444775Z     info    starting Http service at <IP_ADDRESS>:15004
2023-05-10T03:16:17.445868Z     info    Pilot SAN: [istiod.istio-system.svc]
2023-05-10T03:16:17.453437Z     info    xdsproxy        connected to upstream XDS server: istiod.istio-system.svc:15012
2023-05-10T03:16:17.468435Z     info    received server-side bootstrap in 20.855447ms
2023-05-10T03:16:17.468465Z     info    Starting proxy agent
2023-05-10T03:16:17.468474Z     info    Epoch 0 starting
2023-05-10T03:16:17.468492Z     info    Envoy command: [-c etc/istio/proxy/envoy-rev0.json --restart-epoch 0 --drain-time-s 45 --drain-strategy immediate --parent-shutdown-time-s 60 --local-address-ip-version v4 --file-flush-interval-msec 1000 --disable-hot-restart --log-format %Y-%m-%dT%T.%fZ        %l      envoy %n        %v -l warning --component-log-level misc:error --concurrency 2]
2023-05-10T03:16:17.554161Z     info    xdsproxy        connected to upstream XDS server: istiod.istio-system.svc:15012
2023-05-10T03:16:17.572491Z     info    ads     ADS: new connection for node:httpbin-6d84d49667-gqdrd.default-1
2023-05-10T03:16:17.586056Z     info    ads     ADS: new connection for node:httpbin-6d84d49667-gqdrd.default-2
2023-05-10T03:16:17.594363Z     info    cache   generated new workload certificate      latency=149.971297ms ttl=23h59m59.405650193s
2023-05-10T03:16:17.594392Z     info    cache   Root cert has changed, start rotating root cert
2023-05-10T03:16:17.594403Z     info    ads     XDS: Incremental Pushing:0 ConnectedEndpoints:2 Version:
2023-05-10T03:16:17.594454Z     info    cache   returned workload trust anchor from cache       ttl=23h59m59.405548354s
2023-05-10T03:16:17.594462Z     info    cache   returned workload certificate from cache        ttl=23h59m59.405539782s
2023-05-10T03:16:17.594583Z     info    cache   returned workload trust anchor from cache       ttl=23h59m59.405424962s
2023-05-10T03:16:17.594612Z     info    ads     SDS: PUSH request for node:httpbin-6d84d49667-gqdrd.default resources:1 size:4.0kB resource:default
2023-05-10T03:16:17.594755Z     info    ads     SDS: PUSH request for node:httpbin-6d84d49667-gqdrd.default resources:1 size:1.1kB resource:ROOTCA
2023-05-10T03:16:17.594811Z     info    cache   returned workload trust anchor from cache       ttl=23h59m59.405191662s
2023-05-10T03:16:17.594865Z     info    ads     SDS: PUSH for node:httpbin-6d84d49667-gqdrd.default resources:1 size:1.1kB resource:ROOTCA
2023-05-10T03:16:17.651669Z     warning envoy wasm      createWasm: failed to load (in progress) from https://raw.githubusercontent.com/HZHgagaga/wasm/main/work_thread.wasm
2023-05-10T03:16:17.651722Z     critical        envoy wasm      Plugin configured to fail closed failed to load
2023-05-10T03:16:17.651726Z     warning envoy wasm      createWasm: failed to load (cached) from https://raw.githubusercontent.com/HZHgagaga/wasm/main/work_thread.wasm
2023-05-10T03:16:17.651728Z     critical        envoy wasm      Plugin configured to fail closed failed to load
2023-05-10T03:16:18.142180Z     critical        envoy wasm      Plugin configured to fail closed failed to load
2023-05-10T03:16:18.142209Z     critical        envoy wasm      Plugin configured to fail closed failed to load
2023-05-10T03:16:18.142232Z     critical        envoy wasm      Plugin configured to fail closed failed to load
2023-05-10T03:16:18.142245Z     critical        envoy wasm      Plugin configured to fail closed failed to load
2023-05-10T03:16:19.405671Z     info    Readiness succeeded in 2.025342004s
2023-05-10T03:16:19.406048Z     info    Envoy proxy is ready

The following logs make me very uneasy:
2023-05-10T03:16:17.651669Z     warning envoy wasm      createWasm: failed to load (in progress) from https://raw.githubusercontent.com/HZHgagaga/wasm/main/work_thread.wasm
2023-05-10T03:16:17.651722Z     critical        envoy wasm      Plugin configured to fail closed failed to load

It looks like there was an error loading the EnvoyFilter, but I was able to respond to httpbin's request normally.
When I tested my wasm using Envoy alone, there were no errors:
bootstrap_extensions:
- name: envoy.bootstrap.wasm
  typed_config:
    "@type": type.googleapis.com/envoy.extensions.wasm.v3.WasmService
    singleton: true
    config:
      configuration:
        "@type": type.googleapis.com/google.protobuf.StringValue
        value: "main"
      vm_config:
        vm_id: "main_thread"
        runtime: "envoy.wasm.runtime.v8"
        code:
          remote:
            http_uri:
              uri: https://raw.githubusercontent.com/HZHgagaga/wasm/main/main_thread.wasm
              cluster: remote_wasm
              timeout: 10s
            sha256: "6940d386f9d9360a3d4aaa10511c1efa179b9e7341e5f2385acf819195b6da9d"

static_resources:
  listeners:
    - name: http
      address:
        socket_address:
          address: <IP_ADDRESS>
          port_value: 18000
      filter_chains:
        - filters:
            - name: envoy.http_connection_manager
              typed_config:
                "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
                stat_prefix: ingress_http
                codec_type: auto
                route_config:
                  name: local_route
                  virtual_hosts:
                    - name: local_service
                      domains:
                        - "*"
                      routes:
                        - match:
                            prefix: "/"
                          direct_response:
                            status: 200
                            body:
                              inline_string: "example body\n"
                http_filters:
                  - name: envoy.filters.http.wasm
                    typed_config:
                      "@type": type.googleapis.com/envoy.extensions.filters.http.wasm.v3.Wasm
                      config:
                        configuration:
                          "@type": type.googleapis.com/google.protobuf.StringValue
                          value: "work"
                        vm_config:
                          vm_id: "work_thread"
                          runtime: "envoy.wasm.runtime.v8"
                          code:
                            remote:
                              http_uri:
                                uri: https://raw.githubusercontent.com/HZHgagaga/wasm/main/work_thread.wasm
                                cluster: remote_wasm
                                timeout: 0s
                              sha256: "975be293190520c97ef63a32e152f720b2b799178dc9135064baddd3bdffa7be"
                  - name: envoy.filters.http.router
                    typed_config:
                      "@type": type.googleapis.com/envoy.extensions.filters.http.router.v3.Router

  clusters:
    - name: consul
      type: static
      connect_timeout: 1s
      lb_policy: ROUND_ROBIN
      load_assignment:
        cluster_name: consul_service
        endpoints:
          - lb_endpoints:
              - endpoint:
                  address:
                    socket_address:
                      address: <IP_ADDRESS>
                      port_value: 8500
    - name: remote_wasm
      type: strict_dns
      connect_timeout: 1s
      dns_refresh_rate: 5s
      dns_lookup_family: v4_only
      lb_policy: ROUND_ROBIN
      load_assignment:
        cluster_name: remote_wasm
        endpoints:
          - lb_endpoints:
              - endpoint:
                  address:
                    socket_address:
                      address: raw.githubusercontent.com
                      port_value: 443
      transport_socket:
        name: envoy.transport_sockets.tls
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.UpstreamTlsContext
          sni: "raw.githubusercontent.com"

Although there are no exceptions in the individual Envoy, but I want to use remote wasm normally in istio. How can I modify my configuration?
Please help me, I would be very grateful.

Yes, but I did pull wasm file, It's working properly.
I don't understand why wasm ran normally but reported critical logs

+1


Dismiss hides the notice but does not always remove it.

I'm using growl to display ajax update notices. So I growl before the ajax request then dismiss the notice inside the ajax request. The dismiss/remove does not actually work due to the callback parameter being required.
  var args = {
    title: 'Updating',
    message: 'Adding new item...',
    style: 'system',
    duration: 0,
    location: 'br',
    size: 'large'
  };
  var $growl = $.growl(args);
  $.get(url, data, function(response)
  {
    // Minimum display time of 2.6 seconds.
    setTimeout(function() {$growl.dismiss();}, 2600);
  }, 'json').error(ajaxErrorHandler);

I was able to fix this by calling remove instead of dismiss and then editing remove to look like:
Growl.prototype.remove = function(callback) {
  this.$growl().remove();
  /** My new code **/
  if (callback == undefined)
  {
    return;
  }
  /** end new code **/
  return callback();
};

Their must be a better way to do this but the return callback always fails causing the animation to fade out the notice but not clear the div tags causing the invisible divs to prevent sections of the page from being selected.

@danielson317 I made the callback parameter optional for dismiss. If you install the latest version it'll be included. Closing but feel free to re-open if it doesn't fix your issue.


Split Commands

Currently the shell command is only executed in non-AM containers.  We should:

Allow commands to be executed on the AM
Allow for a different command to be executed on the AM compared to other containers

Skein (https://github.com/jcrist/skein) allows running containers with different specifications, which allows different services to be managed by the same application master. The recommended way is to use skein instead of knit, closing.


nixos/cage: add environment config

Description of changes
cage has a bunch of configuration options via environment, so add a simple extraEnvironment variable that maps to the systemd unit.
Also, set WLR_LIBINPUT_NO_DEVICES to 1 by default. This allows launching without any input devices, which I believe makes sense for a kiosk system.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] 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/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Why not just set systemd.services.cage-tty1.environment directly?  It doesn't feel like this extra option adds very much…

That is what I currently do. However, I don't think that's ideal:
Cage is specifically documented as receiving config via environment (more so than via arguments actually), consequently I believe having that config in the module is reasonable for completeness. There are already some modules that also do this, so it's not without precedence.
Relatedly, I wanted to use the module to run a device in kiosk mode, which didn't work without WLR_LIBINPUT_NO_DEVICES. So I thought it would be a nice bonus to make that aspect easily discoverable in the module, without adding a specific option for it or making it an opinionated default in a way that's annoying to change.
All of that said, I don't  feel particularly strongly about this change. If nobody agrees it's a good idea, I'll just stick to the systemd. route :)

Relatedly, I wanted to use the module to run a device in kiosk mode, which didn't work without WLR_LIBINPUT_NO_DEVICES. So I thought it would be a nice bonus to make that aspect easily discoverable in the module, without adding a specific option for it or making it an opinionated default in a way that's annoying to change.

I think a specific option would be better — it's still not very discoverable if it's just in the default value for environment (that's actually pretty surprising).

There are already some modules that also do this, so it's not without precedence.

I suppose it's okay, but if we do it it should be called "environment", as attrsets are always merged in NixOS modules, so the "extra" is meaningless.  (And I think it should default to {}).

Works for me. Changed it to {} and moved the flag into the example, following the tradition of having "configs you might wanna use" there :)

LGTM but I'll give others a chance to comment.

Cage 0.15 was released, when I searched for related pr, I found that this pr was forgotten to be processed

change is still good from my side :ok_hand:


Allow using custom time format

I need to use custom time format in `conf/app.ini' like
[time]
FORMAT = 2006-01-02 15:04:05

so that Gitea will display '2017-01-30 08:41:49'
check this answer for more constants to format date  http://stackoverflow.com/a/20234207/2570425
PS: First GO commit

LGTM

LGTM


🧑💻: Adding the static data of Books

:red_circle: Title :  Static Data of Books
:red_circle: Enhancement Aim : Add static data
:red_circle: Changes :
Adding the static data in the json format which will be received in future backend api
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Siddharth
What is your participant role?
GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Avdhesh-Varshney  I am willing to work on this issue

@Sid-80 no, static addition of books pdfs or other things will increase the size of repo which is not required. So, wait for it, backend is under processing.

@Avdhesh-Varshney sure!!


Increase the word limit in the event description and tell the user what that limit is.

The current the word limit is 255 (the default limit) and we want it increased to 350. We should also tell the user what the word limit is somewhere.

 Increase the word limit in the event description and tell the user what that limit is. #747

I have updated the column to be a TEXT field in production. Change still needs to be made to the model


personnalize account icon

The account icon use the first letter of an account. It could be a nice enhancment if we could personnalize the profil icon with colours, emoji or letter of our choice.

Hi! Yes, that could be a great improvement. If you've time to add that feature you're welcome to do a PR :)

I'm not sure that I'm able to push some kind of code, but I could write a better description of the functionnality.


feat: switch styling to tailwind

Switch styling to tailwind via twrnc. As this package (and actually all implementations I could find) do not support theming via different  Material Design light / dark color palettes out-of-the-box implemented custom provider & hook that take care of this.

👍🏻


Rename NullColumnAccess as NullColumnAccessor

@davies
I think NullColumnAccessor follows same convention for other accessors

Merged build triggered.

Merged build started.

Test build #43402 has started for   PR 9028 at commit f0c8773.

LGTM

Do I need to create JIRA ?

Either is fine to me.

Its always better to create a JIRA incase this breaks the build or something and we have to revert.  That way we can track remerging it.

Test build #43402 has finished for   PR 9028 at commit f0c8773.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test PASSed.


tflite_runtime problem on Arm64 linux with multiprocessing

System information

Have I written custom code (as opposed to using a stock example script
provided in TensorFlow): no
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux (Arm64)
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue
happens on a mobile device: n/a
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.4.1
Python version: 3.6 (PC), 3.7 (embedded Linux)
Bazel version (if compiling from source): n/a
GCC/Compiler version (if compiling from source): n/a
CUDA/cuDNN version: 9
GPU model and memory: Titan xp
Exact command to reproduce:

Describe the problem
I trained a neural network using quantization-aware training from tensorflow_model_optimization and converted it to a quantized tflite model. I have a python script that runs on a device with Linux (Arm64, 4 cores) and a NPU. This script uses the tflite_runtime module to make inference on the NPU with the NNAPI delegate:

INFO: Created TensorFlow Lite delegate for NNAPI.
Applied NNAPI delegate.

This script uses multiprocessing to perform other operations. The inference is made on the main process while other operations are executed like this:
p = Process(target=my_function, args=(my_args,))
p.start()

The problem is that when I start a new process, the inference stops working: the neural network keeps returning the same output, no matter which input I provide. Even when the child process ends, the problem remains. This does not happen if I use threads instead of process.
Source code / logs
Quantization (on the training PC):
import tensorflow as tf
import tensorflow_model_optimization as tfmot
with tfmot.quantization.keras.quantize_scope():
    model = tf.keras.models.load_model(path)

converter = tf.lite.TFLiteConverter.from_keras_model(model)
converter.optimizations = [tf.lite.Optimize.DEFAULT]
quantized_tflite_model = converter.convert()
open("converted_model.tflite", "wb").write(quantized_tflite_model)

Interpreter initialization (on the embedded linux):
import tflite_runtime.interpreter as tflite
interpreter = tflite.Interpreter(model_path=my_path)
interpreter.allocate_tensors()
input_details = interpreter.get_input_details()
output_details = interpreter.get_output_details()
input_shape = input_details[0]['shape']

Inference:
interpreter.set_tensor(input_details[0]['index'], my_input)
interpreter.invoke()
output = interpreter.get_tensor(output_details[0]['index'])

Pseudo-code of the script:
initialize_itnerpreter()
while True:
    acquire_input()
    perform_inference()
    if condition:
        p = Process(target=my_function, args=(my_args,))
        p.start()

Hi @FSet89 !
I think you are not using multi-threading in interpreter correctly during inference. You have to pass the number of threads in interpreter too.
import multiprocessing
tf.lite.Interpreter(modelfile, num_threads=multiprocessing.cpu_count()). // modelfile means tflite file

Could you also test above syntax with 2.8/2.9 version and Python 3.7/3.8 (Python 3.7 is minimum version now)
Thank you!


[3.x] Fix output port type mismatch for some VisualScript nodes

Same as #51147 but on 3.4
(#51122 without mistake of wrong source branch)
This PR fix the regression and other nodes.
Before the fix:

After the fix:

For the nodes that still have 'any' output, I honestly don't know how to fix theses since it's dynamic. Plus it's not really a mismatch since it's dynamic.

You forgot a semicolon:
modules\visual_script\visual_script_builtin_funcs.cpp(587): error C2143: syntax error: missing ';' before '}'

Thanks! And congrats for your first merged Godot contribution :tada:

Cherry-picked for 3.5.1.


setItem function show placeholder

As what I mentioned in the title.
https://jsfiddle.net/d8nr7sog/

fixed in https://github.com/lekoala/bootstrap5-tags/releases/tag/1.6.13


Disable LIMIT and Alias when retrieving data from connector

About You:
Name: Anthony Louis
Company: Simbiose Ventures
Your question:
We created a general connector using the Tableau Connector SDK to connect with the multiple connections we have, some of them are MySQL databases and others are Oracle Databases.
To load the data from the tables in the sources, Tableau sends this query:
SELECT "alias"."column1" AS "column1", "alias"."column2" AS "column2" .... FROM "database"."table" AS "alias" LIMIT 1000

The problem is that LIMIT is only recognized by Mysql sources and the " to quote alias are supported only by the Oracle sources.
Is there some option that I can add in the connector files to disable the alias for columns and tables and the LIMITwhen the Tableau is retrieving the data from the sources?
I have already searched in the capabilities reference but I did not found any option related to this.

Internally tracked by item 1259981

@jainam1995 did you mean that is not possible to do what I want in the actual version of the connector SDK and the question is being discussed and executed internally by the Tableau team?

@anthonylouisbsb  I don't think  it's a good idea to have one connector to connect to different kind of databases. The connetor capabilities and dialect are used to tune the connector for a specific database.
How ever, I will try to answer your specific questions.

You can set capability CAP_QUERY_TOP_N to no to disable Tableau generating query using LIMIT clause or TOP N.
We need use alias to support join.  If your question is how to disable quoting the identifiers, please refer to
this issue.

@anthonylouisbsb I am closing this issue.  Please file a new issue or reopen if you have further questions.


display block not working while using materializecss

Description
While I'm using materialize.css i can't dislay page in block in console /or in container

shows if i also write in it
as you can see in screenshot 298 there is in container type page but in screenshot 297 there is full screen page
i want to convert in container
Repro Steps
Layout the steps to reproduce your issue.
Screenshots / Codepen
Add supplemental screenshots or code examples. Look for a codepen template in our Contributing Guidelines.

I don't seem to understand your issue. Why do you want use container class if your page is fine without it, just use a main wrapper for whole page and apply your custom css to it.
Using container class should not be a problem  just override container class a tweak some properties and you will be fine wiht it.
Container class can have some down points so use it carefully.
:smile: :smile_cat: :smile_cat:

I'm not sure I entirely understand either, it is not really possible for us to break display: block; Please ask support questions on Stack Overflow instead. Or comment here if you think I have closed this in error


unixPB: Install jdk21 instead of 11 from dockerhost.yml

Keep up to date :-)
Checklist

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

/thaw


[i18nIgnore] Prepare to remove legacy.astroFlavoredMarkdown in config reference

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)

Description

Removes links to the AFMD config entry in the /en/ migration guide.
Mark the migration guide as i18nReady: false — we’ll change this once the v1 -> v2 guide replaces this
Removes links to the AFMD config entry in the /ru/ Markdown Content guide.
Uninstalls a direct dependency MAINTAINERS.md said was only needed for AFMD and updated that note

Does not change Spanish, Portuguese, French, and Chinese pages as these should not cause any broken links. I think it’s safe to leave these as is until they get a proper refresh.

Looks great! The only other thing I am tempted to do is to delete all the other language's migrate.mdx files at this point? Not opposed to keeping them if people feel strongly, but I'd nix them now.

I think my preference would be to remove the translated migration guides in the same PR that adds the new English migration guide. What do you think?

We can discuss then. If we’re now thinking of “migration == coming to Astro from elsewhere” and talking more about “upgrading Astro versions”, we might want to reflect that in the URL/filename too and add a redirect for links to the old location.


v4.0.0

This pull request upgrades the library to version 4 which comes with the following changes.

All event handlers are now longer interfaces. They are async events now.
The internal memory management is improve a lot.
The internal message queueing in the server is build from scratch.
The repository now contains working samples so that the Wiki can be cleaned up.
Support for more MQTTv5 features.
Remove of several interfaces and extensions to keep the public API small and clean.

 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.:x: chkr1011 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


lib64-nvidia did not contain ['libcudart.so', 'libcudart.so.11.0', 'libcudart.so.12.0']

Welcome to bitsandbytes. For bug reports, please run
python -m bitsandbytes
and submit this information together with your error trace to: https://github.com/TimDettmers/bitsandbytes/issues
bin /usr/local/lib/python3.10/dist-packages/bitsandbytes/libbitsandbytes_cuda118.so
/usr/local/lib/python3.10/dist-packages/bitsandbytes/cuda_setup/main.py:149: UserWarning: /usr/lib64-nvidia did not contain ['libcudart.so', 'libcudart.so.11.0', 'libcudart.so.12.0'] as expected! Searching further paths...
warn(msg)
/usr/local/lib/python3.10/dist-packages/bitsandbytes/cuda_setup/main.py:149: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/sys/fs/cgroup/memory.events /var/colab/cgroup/jupyter-children/memory.events')}
warn(msg)
/usr/local/lib/python3.10/dist-packages/bitsandbytes/cuda_setup/main.py:149: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('//<IP_ADDRESS>'), PosixPath('8013'), PosixPath('http')}
warn(msg)
/usr/local/lib/python3.10/dist-packages/bitsandbytes/cuda_setup/main.py:149: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('//colab.research.google.com/tun/m/cc48301118ce562b961b3c22d803539adc1e0c19/gpu-t4-s-30kei7iq2aw6l --tunnel_background_save_delay=10s --tunnel_periodic_background_save_frequency=30m0s --enable_output_coalescing=true --output_coalescing_required=true'), PosixPath('--logtostderr --listen_host=<IP_ADDRESS> --target_host=<IP_ADDRESS> --tunnel_background_save_url=https')}
warn(msg)
/usr/local/lib/python3.10/dist-packages/bitsandbytes/cuda_setup/main.py:149: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/env/python')}
warn(msg)
/usr/local/lib/python3.10/dist-packages/bitsandbytes/cuda_setup/main.py:149: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('module'), PosixPath('//ipykernel.pylab.backend_inline')}
warn(msg)
CUDA_SETUP: WARNING! libcudart.so not found in any environmental path. Searching in backup paths...
/usr/local/lib/python3.10/dist-packages/bitsandbytes/cuda_setup/main.py:149: UserWarning: Found duplicate ['libcudart.so', 'libcudart.so.11.0', 'libcudart.so.12.0'] files: {PosixPath('/usr/local/cuda/lib64/libcudart.so.11.0'), PosixPath('/usr/local/cuda/lib64/libcudart.so')}.. We'll flip a coin and try one of these, in order to fail forward.
Either way, this might cause trouble in the future:
If you get CUDA error: invalid device function errors, the above might be the cause and the solution is to make sure only one ['libcudart.so', 'libcudart.so.11.0', 'libcudart.so.12.0'] in the paths that we search based on your env.
warn(msg)
CUDA SETUP: CUDA runtime path found: /usr/local/cuda/lib64/libcudart.so.11.0
CUDA SETUP: Highest compute capability among GPUs detected: 7.5
CUDA SETUP: Detected CUDA version 118
CUDA SETUP: Loading binary /usr/local/lib/python3.10/dist-packages/bitsandbytes/libbitsandbytes_cuda118.so...
Loading model tensors: 100%|##########| 517/517 [01:51<00:00,  4.64it/s]INIT       | Starting   | LUA bridge
INIT       | OK         | LUA bridge
INIT       | Starting   | LUA Scripts
INIT       | OK         | LUA Scripts
Setting Seed
MESSAGE    | KoboldAI has finished loading and is available at the following link for UI 1: https://potentially-principal-christ-competitive.trycloudflare.com/
MESSAGE    | KoboldAI has finished loading and is available at the following link for UI 2: https://potentially-principal-christ-competitive.trycloudflare.com/new_ui
MESSAGE    | KoboldAI has finished loading and is available at the following link for KoboldAI Lite: https://potentially-principal-christ-competitive.trycloudflare.com/lite
MESSAGE    | KoboldAI has finished loading and is available at the following link for the API: https://potentially-principal-christ-competitive.trycloudflare.com/api

Bitsandbytes was not supported windows before, but my method can support windows.（yuhuang）
1  open folder J:\StableDiffusion\sdwebui，Click the address bar of the folder and enter CMD
or WIN+R, CMD  。enter，cd /d J:\StableDiffusion\sdwebui
2  J:\StableDiffusion\sdwebui\py310\python.exe -m pip uninstall bitsandbytes
3 J:\StableDiffusion\sdwebui\py310\python.exe -m pip uninstall bitsandbytes-windows
4 J:\StableDiffusion\sdwebui\py310\python.exe -m pip install https://github.com/jllllll/bitsandbytes-windows-webui/releases/download/wheels/bitsandbytes-0.41.1-py3-none-win_amd64.whl
Replace your SD venv directory file（python.exe Folder） here（J:\StableDiffusion\sdwebui\py310）


fix(markerCompilation): use specified scope to listen to $includeContentLoaded

It was an error to rely on popupContent to verify origin of event emission. Use scope instead. The event is only fired on the compilation target scope.
This will fix potential layout issues.

@tombatossals will this fix make it to the release?

Hi @davidovich, everything perfect, thanks for all your fantastic work.
I'm going to make a release of version 0.7.11 with all the latest changes.


1.0.0

Coverage increased (+3.2%) to 87.162% when pulling dfa45f8031a5b2e8b7a5a74303662ad132bb9cd2 on 1.0.0 into b5dc37afadfc86d34688d7071a8e429254426ab4 on master.


abort() when trying to revalidate expired output resource

I am using custom integration of PSOL and recently all instances started to crash with the same message:
[0605/094417:FATAL:output_resource.cc(223)] Output resources shouldn't be loaded via LoadAsync, but rather through FetchResource

Backtrace:
[Switching to Thread 0x7ffea2ffd700 (LWP 3018)]
base::debug::(anonymous namespace)::DebugBreak () at third_party/chromium/src/base/debug/debugger_posix.cc:246
246     }
(gdb) bt
#0  base::debug::(anonymous namespace)::DebugBreak () at third_party/chromium/src/base/debug/debugger_posix.cc:246
#1  0x0000000000b4a83d in base::debug::BreakDebugger () at third_party/chromium/src/base/debug/debugger_posix.cc:259
#2  0x000000000071cf7e in logging::LogMessage::~LogMessage (this=0x7ffea2ffbe20, __in_chrg=<optimized out>) at third_party/chromium/src/base/logging.cc:657
#3  0x00000000007a90c7 in net_instaweb::OutputResource::LoadAndCallback (this=0x7ffe94015868, not_cacheable_policy=net_instaweb::Resource::kReportFailureIfNotCacheable, request_context=...,
    callback=0x7ffe94018498) at net/instaweb/rewriter/output_resource.cc:223
#4  0x00000000007add85 in net_instaweb::Resource::LoadAsync (this=0x7ffe94015868, not_cacheable_policy=net_instaweb::Resource::kReportFailureIfNotCacheable, request_context=..., callback=0x7ffe94018498)
    at net/instaweb/rewriter/resource.cc:166
#5  0x0000000000bfd692 in net_instaweb::RewriteContext::OutputCacheRevalidate (this=0x7ffe9c038f68, to_revalidate=std::vector of length 1, capacity 1 = {...}) at net/instaweb/rewriter/rewrite_context.cc:1934
#6  0x0000000000bfbe96 in net_instaweb::RewriteContext::OutputCacheDone (this=0x7ffe9c038f68, cache_result=0x7ffe94016da8) at net/instaweb/rewriter/rewrite_context.cc:1674
#7  0x0000000000c10a9d in net_instaweb::MemberFunction1<net_instaweb::RewriteContext, net_instaweb::RewriteContext::CacheLookupResult*>::Run (this=0x7ffe94016f38) at ./pagespeed/kernel/base/function.h:171
#8  0x000000000089428f in net_instaweb::Function::CallRun (this=0x7ffe94016f38) at pagespeed/kernel/base/function.cc:51
#9  0x00000000006dde28 in net_instaweb::QueuedWorkerPool::Run (this=0x169b538, sequence=0x17884b8, worker=0x7ffe9c038528) at pagespeed/kernel/thread/queued_worker_pool.cc:157
#10 0x00000000006e5388 in net_instaweb::MemberFunction2<net_instaweb::QueuedWorkerPool, net_instaweb::QueuedWorkerPool::Sequence*, net_instaweb::QueuedWorker*>::Run (this=0x17afe08)
    at ./pagespeed/kernel/base/function.h:202
#11 0x000000000089428f in net_instaweb::Function::CallRun (this=0x17afe08) at pagespeed/kernel/base/function.cc:51
#12 0x00000000006e9850 in net_instaweb::Worker::WorkThread::Run (this=0x7ffe9c03a2f8) at pagespeed/kernel/thread/worker.cc:84
#13 0x00000000006ecbab in net_instaweb::PthreadThreadImpl::InvokeRun (self_ptr=0x7ffe9c03a3c8) at pagespeed/kernel/thread/pthread_thread_system.cc:122
#14 0x00007ffff735c374 in start_thread (arg=0x7ffea2ffd700) at pthread_create.c:333
#15 0x00007ffff49d760f in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:105

I've found metadata key it was using:
#5  0x0000000000bfd692 in net_instaweb::RewriteContext::OutputCacheRevalidate (this=0x7ffe9c038f68, to_revalidate=std::vector of length 1, capacity 1 = {...}) at net/instaweb/rewriter/rewrite_context.cc:1934
1934            new ResourceRevalidateCallback(this, resource, input_info));
(gdb) p partition_key_
$1 = "rname/ce_JfZ4rtPZUfry0kIJ6v3S/http://XXX/images/xpage1_img1.jpg.pagespeed.ic.965L1HZqE1.jpg@@_"

I've decoded InputInfo from that cache key and it was recently expired after being year in cache.
I'm using latest stable version of psol (<IP_ADDRESS>) but believe that last dev version still has the issue. It's also custom integration but I believe it doesn't matter. Well perhaps only except of long-living cache which helped to trigger the issue.

So far no luck hitting this code path with vanilla mod_pagespeed.
It looks like somehow ResourcePtr resource = slots_[input_info->index()]->resource(); returns an OutputResource, which I think should not happen: I think that should only resturn InputResources.
So I wonder: how did the OutputResource end up there?

I've tried to modify pagespeed to force return expired for every metadata cache lookup:
diff --git a/net/instaweb/rewriter/rewrite_context.cc b/net/instaweb/rewriter/rewrite_context.cc
index 484a14499..d90f9aa72 100644
--- a/net/instaweb/rewriter/rewrite_context.cc
+++ b/net/instaweb/rewriter/rewrite_context.cc
@@ -396,15 +396,15 @@ class RewriteContext::OutputCacheCallback : public CacheInterface::Callback {
         if (!input_info.has_expiration_time_ms()) {
           return false;
         }
-        int64 ttl_ms = input_info.expiration_time_ms() - now_ms;
-        if (ttl_ms > 0) {
-          return true;
-        } else if (
-            !rewrite_context_->has_parent() &&
-            ttl_ms + options->metadata_cache_staleness_threshold_ms() > 0) {
-          *stale_rewrite = true;
-          return true;
-        }
+        // int64 ttl_ms = input_info.expiration_time_ms() - now_ms;
+        // if (ttl_ms > 0) {
+        //   return true;
+        // } else if (
+        //     !rewrite_context_->has_parent() &&
+        //     ttl_ms + options->metadata_cache_staleness_threshold_ms() > 0) {
+        //   *stale_rewrite = true;
+        //   return true;
+        // }

This way both my server and pagespeed_automatic_test abort() with the same backtrace. Change itself is silly one but seems to do the job at least.
% ./pagespeed_automatic_test --gtest_filter=CacheHtmlFlowTest.TestCacheHtmlFlushSubresources                                                                                [src/tags/<IP_ADDRESS>]
Running main() from gtest_main.cc
Note: Google Test filter = CacheHtmlFlowTest.TestCacheHtmlFlushSubresources
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from CacheHtmlFlowTest
[ RUN      ] CacheHtmlFlowTest.TestCacheHtmlFlushSubresources
[0605/174716:FATAL:output_resource.cc(223)] Output resources shouldn't be loaded via LoadAsync, but rather through FetchResource
./pagespeed_automatic_test() [0x11c1ecc]
./pagespeed_automatic_test() [0x11c32c1]
./pagespeed_automatic_test() [0x1313f8a]
./pagespeed_automatic_test() [0x13169f0]
./pagespeed_automatic_test() [0x100718d]
./pagespeed_automatic_test() [0x1005a5d]
./pagespeed_automatic_test() [0x101b9a3]
./pagespeed_automatic_test() [0x136deb6]
./pagespeed_automatic_test() [0x13bcafa]
./pagespeed_automatic_test() [0x13c2df8]
./pagespeed_automatic_test() [0x136deb6]
./pagespeed_automatic_test() [0x13c9462]
./pagespeed_automatic_test() [0xefddae]
/lib64/libpthread.so.0(+0x7374) [0x7f583083e374]
/lib64/libc.so.6(clone+0x5f) [0x7f583058960f]

zsh: abort (core dumped)  ./pagespeed_automatic_test

This happens with both vanilla and patched pagespeed for me. I do have some modifications for PSOL and obviously there is integration step but changes do not seem to be related. I didn't touch rewriting at all, only cache/system stuff.
Can you reproduce above steps to confirm the issue on your side?
What happens in that test case I think is that cache extend filter requires already rewritten resource as input.  It seems that OutputResource is being assigned to slot in RewriteContext::Propagate. After that RewriteContext::RunSuccessors starts new rewrite with OutputResource in cache. In test case above pagespeed first runs image compression filter and then cache extends it.
RewriteContext::FetchInputs has comment saying about chained rewrites and requiring pagespeed resources as inputs. FetchInputs is called when metadata cache is missing at all for some input and it checks if URL of input is pagespeed URL, avoiding LoadAsync for such inputs and instead using FetchResource.
It seems that RewriteContext::OutputCacheRevalidate should do similar: if input is already rewritten then use FetchResource, otherwise fallback to LoadAsync.

This triggers the issue with vanilla pagespeed without modification above.
TEST_F(CacheHtmlFlowTest, TestCacheHtmlRevalidateInputs) {
  GoogleString text;
  RequestHeaders request_headers;
  ResponseHeaders response_headers;
  FetchFromProxy("http://test.com/flush_subresources.html"
                 "?PageSpeedFilters=+extend_cache_css,-inline_css", true,
                 request_headers, &text, &response_headers, NULL, false);
  SetTimeMs(MockTimer::kApr_5_2010_ms + 2 * Timer::kYearMs);
  FetchFromProxy("http://test.com/flush_subresources.html"
                 "?PageSpeedFilters=+extend_cache_css,-inline_css", true,
                 request_headers, &text, &response_headers, NULL, false);
}

% ./pagespeed_automatic_test --gtest_filter=CacheHtmlFlowTest.TestCacheHtmlRevalidateInputs                                                                                [src/tags/<IP_ADDRESS>]
Running main() from gtest_main.cc
Note: Google Test filter = CacheHtmlFlowTest.TestCacheHtmlRevalidateInputs
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from CacheHtmlFlowTest
[ RUN      ] CacheHtmlFlowTest.TestCacheHtmlRevalidateInputs
[0605/191504:FATAL:output_resource.cc(223)] Output resources shouldn't be loaded via LoadAsync, but rather through FetchResource
./pagespeed_automatic_test() [0x11c22da]
./pagespeed_automatic_test() [0x11c36cf]
./pagespeed_automatic_test() [0x1314398]
./pagespeed_automatic_test() [0x1316dfe]
./pagespeed_automatic_test() [0x10074e7]
./pagespeed_automatic_test() [0x1005db7]
./pagespeed_automatic_test() [0x101bdb1]
./pagespeed_automatic_test() [0x136e2c4]
./pagespeed_automatic_test() [0x13bcefa]
./pagespeed_automatic_test() [0x13c31f8]
./pagespeed_automatic_test() [0x136e2c4]
./pagespeed_automatic_test() [0x13c9862]
./pagespeed_automatic_test() [0xefe108]
/lib64/libpthread.so.0(+0x7374) [0x7fb43e66d374]
/lib64/libc.so.6(clone+0x5f) [0x7fb43e3b860f]

zsh: abort (core dumped)  ./pagespeed_automatic_test

I should point out that CacheHtmlFlow was never part of mod_pagespeed or
ngx_pagespeed; it was in support of a feature in PageSpeed Service (
https://developers.google.com/speed/pagespeed/service/CachePartialHtml).
The test you modified is no longer part of the source trunk on github.
This might be not be related to where you decided to put the repro-test,
though.
Can you explain more about your PSOL integration?  As you may be aware
there are a few entry-points in the system to support various server
architectures (threaded, async, multi-process) and network setups (proxy,
origin-modifying).  I suspect it's the edge of the system that you've
diverged from the paths that have a ton of mileage.
On Mon, Jun 5, 2017 at 8:16 AM, cellscape<EMAIL_ADDRESS>wrote:

This triggers the issue with vanilla pagespeed without modification above.
TEST_F(CacheHtmlFlowTest, TestCacheHtmlRevalidateInputs) {
GoogleString text;
RequestHeaders request_headers;
ResponseHeaders response_headers;
FetchFromProxy("http://test.com/flush_subresources.html"
"?PageSpeedFilters=+extend_cache_css,-inline_css", true,
request_headers, &text, &response_headers, NULL, false);
SetTimeMs(MockTimer::kApr_5_2010_ms + 2 * Timer::kYearMs);
FetchFromProxy("http://test.com/flush_subresources.html"
"?PageSpeedFilters=+extend_cache_css,-inline_css", true,
request_headers, &text, &response_headers, NULL, false);
}
% ./pagespeed_automatic_test --gtest_filter=CacheHtmlFlowTest.TestCacheHtmlRevalidateInputs                                                                                [src/tags/<IP_ADDRESS>]
Running main() from gtest_main.cc
Note: Google Test filter = CacheHtmlFlowTest.TestCacheHtmlRevalidateInputs
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from CacheHtmlFlowTest
[ RUN      ] CacheHtmlFlowTest.TestCacheHtmlRevalidateInputs
[0605/191504:FATAL:output_resource.cc(223)] Output resources shouldn't be loaded via LoadAsync, but rather through FetchResource
./pagespeed_automatic_test() [0x11c22da]
./pagespeed_automatic_test() [0x11c36cf]
./pagespeed_automatic_test() [0x1314398]
./pagespeed_automatic_test() [0x1316dfe]
./pagespeed_automatic_test() [0x10074e7]
./pagespeed_automatic_test() [0x1005db7]
./pagespeed_automatic_test() [0x101bdb1]
./pagespeed_automatic_test() [0x136e2c4]
./pagespeed_automatic_test() [0x13bcefa]
./pagespeed_automatic_test() [0x13c31f8]
./pagespeed_automatic_test() [0x136e2c4]
./pagespeed_automatic_test() [0x13c9862]
./pagespeed_automatic_test() [0xefe108]
/lib64/libpthread.so.0(+0x7374) [0x7fb43e66d374]
/lib64/libc.so.6(clone+0x5f) [0x7fb43e3b860f]
zsh: abort (core dumped)  ./pagespeed_automatic_test
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/1553#issuecomment-306173710,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB2kPaZmRXuroeUHvOjpv23nFiT6EH3Sks5sA_GggaJpZM4NvmuS
.

CacheHtmlFlow was just the first place where I observed the crash in tests with modification above. I just modified it to be able to reproduce quickly. It's in no way specific and I believe the issue is general one.
Basically this test does rewrite (with image compression to cache extender filter chain), expires stored metadata for InputInfo and then aborts when trying to revalidate output of image compression. I might be wrong though, don't fully understand rewriting process.
As for integration, I already described it some time ago on mailing list: https://groups.google.com/forum/#!msg/mod-pagespeed-discuss/_LJJ0UmQ8SU/sw5cvqedjmMJ. There probably were some changes but overall architecture is the same. As I said I don't think integration is to blame in that case though, we don't touch any rewriting or filters.

This reproduces on master:
oschaaf@ubuntu:~/code/google/mps-abo/mod_pagespeed⟫ git diff
diff --git a/pagespeed/automatic/proxy_interface_test.cc b/pagespeed/automatic/proxy_interface_test.cc
index bc900b84a..89f7d32f8 100644
--- a/pagespeed/automatic/proxy_interface_test.cc
+++ b/pagespeed/automatic/proxy_interface_test.cc
@@ -381,6 +381,37 @@ TEST_F(ProxyInterfaceTest, LoggingInfo) {
   EXPECT_TRUE(logging_info()->is_request_disabled());
 }

+TEST_F(ProxyInterfaceTest, TestAbort) {
+  GoogleString text;
+  RequestHeaders request_headers;
+  ResponseHeaders response_headers;
+  response_headers.Add(HttpAttributes::kContentType, kContentTypeHtml.mime_type());
+  response_headers.SetStatusAndReason(HttpStatus::kOK);
+  GoogleString url = "http://test.com/flush_subresources.html"
+    "?PageSpeedFilters=+extend_cache_css,-inline_css";
+  mock_url_fetcher_.SetResponse("http://test.com/flush_subresources.html", response_headers,
+                                "<link rel='stylesheet' type='text/css' href='test.css'>");
+
+  response_headers.Replace(HttpAttributes::kContentType, kContentTypeCss.mime_type());
+  mock_url_fetcher_.SetResponse("http://test.com/test.css", response_headers,
+                                kCssContent);
+
+  FetchFromProxy(url,
+                 request_headers,
+                 true,  /* expect_success */
+                 &text,
+                 &response_headers,
+                 false  /* proxy_fetch_property_callback_collector_created */);
+  SetTimeMs(MockTimer::kApr_5_2010_ms + 2 * Timer::kYearMs);
+
+  FetchFromProxy(url,
+                 request_headers,
+                 true,  /* expect_success */
+                 &text,
+                 &response_headers,
+                 false  /* proxy_fetch_property_callback_collector_created */);
+}
+
 TEST_F(ProxyInterfaceTest, SkipPropertyCacheLookupIfOptionsNotEnabled) {
   GoogleString url = "http://www.example.com/";
   GoogleString text;

So what's weird here is that normally you shouldn't get an OutputResource
in re-validate array, since it would have a 1 year TTL, so won't need
revalidation.
(Though I guess technically that could happen if a year passed..)
Hmm. You could probably trigger something like this via cache flushes,
though, looking at the code.
Probably OutputResource should use an ALWAYS_VALID InputInfo.Type or
something.
On Mon, Jun 5, 2017 at 8:48 AM, cellscape<EMAIL_ADDRESS>wrote:

CacheHtmlFlow was just the first place where I observed the crash in tests
with modification above. I just modified it to be able to reproduce
quickly. It's in no way specific and I believe the issue is general one.
Basically this test does rewrite (with image compression to cache extender
filter chain), expires stored metadata for InputInfo and then aborts when
trying to revalidate output of image compression. I might be wrong though,
don't fully understand rewriting process.
As for integration, I already described it some time ago on mailing list:
https://groups.google.com/forum/#!msg/mod-pagespeed-discuss/_LJJ0UmQ8SU/
sw5cvqedjmMJ. There probably were some changes but overall architecture
is the same. As I said I don't think integration is to blame in that case
though, we don't touch any rewriting or filters.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/1553#issuecomment-306179111,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADl1RAt4vCDVcRLNkSoJbZVBIyuXckRaks5sA_kEgaJpZM4NvmuS
.

Oh, you are advancing time by 2 years.
On Mon, Jun 5, 2017 at 9:07 AM, Maksim Orlovich<EMAIL_ADDRESS>wrote:

So what's weird here is that normally you shouldn't get an OutputResource
in re-validate array, since it would have a 1 year TTL, so won't need
revalidation.
(Though I guess technically that could happen if a year passed..)
Hmm. You could probably trigger something like this via cache flushes,
though, looking at the code.
Probably OutputResource should use an ALWAYS_VALID InputInfo.Type or
something.
On Mon, Jun 5, 2017 at 8:48 AM, cellscape<EMAIL_ADDRESS>wrote:

CacheHtmlFlow was just the first place where I observed the crash in
tests with modification above. I just modified it to be able to reproduce
quickly. It's in no way specific and I believe the issue is general one.
Basically this test does rewrite (with image compression to cache
extender filter chain), expires stored metadata for InputInfo and then
aborts when trying to revalidate output of image compression. I might be
wrong though, don't fully understand rewriting process.
As for integration, I already described it some time ago on mailing list:
https://groups.google.com/forum/#!msg/mod-pagespeed-discuss/
_LJJ0UmQ8SU/sw5cvqedjmMJ. There probably were some changes but overall
architecture is the same. As I said I don't think integration is to blame
in that case though, we don't touch any rewriting or filters.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/1553#issuecomment-306179111,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADl1RAt4vCDVcRLNkSoJbZVBIyuXckRaks5sA_kEgaJpZM4NvmuS
.

I want to note that in release mode there is no abort, and with the simple test that excites this the results seem to look good

It would probably do a useless rewrite but otherwise behave, yeah.
On Mon, Jun 5, 2017 at 9:14 AM, Otto van der Schaaf <
<EMAIL_ADDRESS>wrote:

I want to note that in release mode there is no abort, and with the simple
test that excites this the results seem to look good
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/1553#issuecomment-306184265,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADl1RLWw8uBvp1g1LHikqM1OGEbrFhAiks5sA_85gaJpZM4NvmuS
.

Looks like it's matter of having NDEBUG defined? This explains why nobody triggered that with apache or nginx then.

We do run all of our system-tests with debug compiles so we get these extra
checks.  But evidently didn't hit that case until your new unit tests.
On Mon, Jun 5, 2017 at 9:25 AM, cellscape<EMAIL_ADDRESS>wrote:

Looks like it's matter of having NDEBUG defined? This explains why nobody
triggered that with apache or nginx then.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/1553#issuecomment-306186585,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB2kPU-bydW1qY_LhS69lXCKPqQBnh95ks5sBAHOgaJpZM4NvmuS
.

It sounds like in the short term what we could do is to remove the VLOG(DFATAL) with a warning for those running debug builds, and add a TODO for modifying OutputResource  to indicate it is always valid as @morlovich suggested above? It would also be nice to include the new unit test.
@cellscape Would you like to create a PR for this? :)

Is CLA still required? If yes then I pass otherwise will prepare one tomorrow.

@cellscape a CLA is required.. so I'll create the PR then

Closing, as the fix for this was merged


added promise to loadRemote() method

Hi,
because I needed a loadRemote promise, I forked your repo and added the functionality.
So it's possible now to work with the loaded data in controllers, after the file is loaded.
gettextCatalog.loadRemote('locales/en_US/foobar.json').then(function(data) { //data optionally returns the local object
    console.log(gettextCatalog.getString('lorem-ipsum'));
});

It would be nice if you take a look at the PR and add the code to your master, so that I can upgrade to later angular-gettext version without merging.
Thanks ;)

You do realise that $http returns a promise?

Why do you need to wrap it in $q, doesn't $http already return a promise?
On Monday, March 2, 2015, Mischa Brammer<EMAIL_ADDRESS>wrote:

Hi,
because I needed a loadRemote promise, I forked your repo and added the
functionality.
So it's possible now to work with the loaded data in controllers, after
the file is loaded.
gettextCatalog.loadRemote('locales/en_US/foobar.json').then(function(data) { //data optionally returns the local object
console.log(gettextCatalog.getString('lorem-ipsum'));
});
It would be nice if you take a look at the PR and add the code to your
master, so that I can upgrade to later angular-gettext version without
merging.
Thanks ;)
You can view, comment on, or merge this pull request online at:
https://github.com/rubenv/angular-gettext/pull/163
Commit Summary

added promise to loadRemote() method

File Changes

M dist/angular-gettext.js
https://github.com/rubenv/angular-gettext/pull/163/files#diff-0 (21)
M dist/angular-gettext.min.js
https://github.com/rubenv/angular-gettext/pull/163/files#diff-1 (2)
M src/catalog.js
https://github.com/rubenv/angular-gettext/pull/163/files#diff-2 (21)

Patch Links:

https://github.com/rubenv/angular-gettext/pull/163.patch
https://github.com/rubenv/angular-gettext/pull/163.diff

—
Reply to this email directly or view it on GitHub
https://github.com/rubenv/angular-gettext/pull/163.

Damn!  Didn't know that :D Okay, ignore my PR ;)


Fix highlight mode for inlay hints

Inlay hints weren't applying the underlying highlight from the cursorline. Setting hl_mode="combine" for the extmark fixes this

Thanks for the PR!


added registered users table and modal for delete button

wooo great work with the table and modal :sparkles:
Unfortunately, these changes currently overwrite the registered users table, which we still need to be able to access. Instead of editing app/templates/admin/registered_users.html, let's create a new template app/templates/room-request/manage.html with all of this HTML code. Then, in app/room_request/views.py, we can add a view at the route / that renders this newly created template:
# app/room_request/views.py
...
@room_request.route('/', methods=['GET'])
def manage():
    # Get all room requests from the database
    # so we can render them in the table
    room_requests = RoomRequest.query.all()
    # Render the new template and pass in the room_requests
    return render_template('room_request/manage.html', room_requests=room_requests)

The last line would also allow us to access room_requests in the template! This means that we can loop over the room requests using {% for r in room_requests %} in the HTML code and render individual fields with {{ r.patient_name }} or {{ r.email }}, etc.

@caroljli I made some updates to the code and am planning to merge it in now! :blush: Feel free to take a look at some of the revisions and let me know if you have any questions! I basically just went through the RoomRequest model and made sure that the new table you added is getting populated correctly!


Unphysical extrapolation of area if decreasing at top of profile

The area of a profile is extrapolated linearly above the given profile data. This is usually not a problem when the area increases with the level, but in the edge case where the area decreases with the level at the top of the given profile data, then the area is extrapolated until it is negative from some level onward. This eventually yields a square root domain error.
If we do not want to support decreasing area at the top of the profile data we should add a validation for this, otherwise we need to define how this profile should be extrapolated properly.

I think it's fine to disallow a decreasing area at the top. If people want a constant area extrapolation they can just add a higher level with the same area.


[BUG] ui is not automatically refreshed.

Describe the bug
UI is not auto refreshed after user's action, say, creating an image or creating an vm in v0.3.0.
Expected behavior
UI is automatically refreshed after user's action, say, creating an image or creating an vm.
Environment:

Harvester ISO version: v0.3.0
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): qemu/kvm

Close,  Feel free to open it if you have any questions.


Add support to handle relative sdk.framework path

A few things:

Can you please remove the strange "history" files that you committed?

Similarly, can you remove these extra unnecessary lines:

import { Console } from "console";
// let orange = vscode.window.createOutputChannel("Orange");?

vscode.workspace.rootPath is deprecated, and it should not be used. Especially since vscode-as3mxml is an extension that supports multi-root workspaces. A correct solution is probably as follows:

If vscode.workspace.workspaceFile is not undefined, the path should be relative to its parent folder.
If vscode.workspace.workspaceFile is undefined, and vscode.workspace.workspaceFolders is not undefined, then it should be relative to vscode.workspace.workspaceFolders[0].
Otherwise, the relative path cannot be resolved, so it should be considered invalid.

Thanks @joshtynjala ,
I fixed those things.
If vscode.workspace.workspaceFile is not undefined, the path should be relative to its parent folder.
Ig the vscode.workspace.workspaceFile is not undefined, then it shouldn't be the relative path to the parent folder, but the parent parent.... because the parent is the .vscode folder, and i think, we need to the parent of the .vscode

Thank's @joshtynjala, it's done!

Ig the vscode.workspace.workspaceFile is not undefined, then it shouldn't be the relative path to the parent folder, but the parent parent.... because the parent is the .vscode folder, and i think, we need to the parent of the .vscode

I see that you deleted this comment, but I wanted to make sure this was addressed, just in case there's still confusion.
The .code-workspace file is not required to be saved inside the .vscode/ folder. In fact, I believe that it would be a very uncommon situation because a multi-root workspace often has multiple .vscode/ folders, so which one would take precedence? Ideally, none of them would be considered more important. I can say for certain that I never place my ,code-workspace file in a .vscode/ folder, or even necessarily the parent of .vscode/. With that in mind, the parent of .vscode/ would definitely break things if I were to try to use relative paths from the .code-workspace file.
I stand by my initial suggestion of the folder containing the .code-workspace file is where the relative paths should resolve from. I don't have a chance to look right now. Is that what you ended up implementing?

agree with these statements.
yes i did this
if(vscode.workspace.workspaceFile!= undefined){
      sdkPath = path.join(path.dirname(vscode.workspace.workspaceFile.path),sdkPath);
    }

Okay, I'll review this when I have a chance, and get it merged. Thank you!

Me too!

Looks good to me. Merged. Thanks again!

Hi @joshtynjala!
unfortunately i'm tested this pull request just on macos, and not on windows. On windows, it's not working, because the [Uri].path give back a path with a wrong delimiter.
I will create another pull request.


Streams are not generic

TypeScript Version: 3.0.0-dev.20180626

Search Terms: TransformStream Streams ReadableStream WritableStream
WHATWG streams can be used for more than just ArrayBufferViews. For example, they can be used to build observables that carry events.
lib.dom.d.ts is not only missing TransformStream, but is also not implemented with generics.  @types/whatwg-streams provides a correct implementation, but it can’t be used as it conflicts with lib.dom.d.ts, and I have found no way to fix that on my end,
lib.dom.d.ts should definitely be updated, but we should also talk about the fact that there doesn’t seem to be a way to overwrite whatever the standard lib provides.

PRs welcomed. You can find more information about contributing lib.d.ts fixes at https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes.

Cool! Will open a PR for the streams.
Are there any plans/ideas to allow developers to override types specified by the stdlib?

Declarations in the lib file are not any special. So not sure what you mean by "override". but you can always use --noLib and have your own lib file.

So in my scenario above I found the types for ReadableStream and WritableStream from lib insufficient. I found ones that suited my needs in DefinitelyTyped, but couldn’t use them as tsc complains about conflicting definitions.
You are right that I could use --noLib, copy lib.dom.d.ts and monkeypatch my types — but doesn’t that seem like a really big hammer for a rather small problem?
I am obviously not familiar with the TypeScript internals, so my idea should be highly scrutinized. I was envisioning something along the lines of:
/// <reference undeclare="ReadableStream" />
/// <reference types="@types/whatwg-streams" />

(Also, I realize I am completely hijacking this issue. Should I open a new one?)

You are right that I could use --noLib, copy lib.dom.d.ts and monkeypatch my types — but doesn’t that seem like a really big hammer for a rather small problem?

As i noted earlier declarations in library files are not different from any other declarations. so the issue really is about granularity. we think of the dom today as a single group of declarations, you either take them all or not. it is possible that int he future we split the lib.dom.d.ts into multiple files,  one for each dom proposal/area. in that sense you can opt-in the parts of the dom you need.
having said that, from experience, these declarations are better be updated in the default library. most ppl do not want to go mix and match their declarations and manage them independently. So in short i think the best route here is to update lib.dom.d.ts rather than going to DT for such changes.


No output from subjack

Im not getting any output not on my terminal and its not even creating the output file.
Here is the command I used:
subjack -w merged.txt -t 20 -timeout 30 -o results_subjack.txt -ssl -c ~/go/src/github.com/haccer/subjack/fingerprints.json
Here is a snippet of my merged.txt file:
v.tiktok.com
vm.tiktok.com
vt.tiktok.com
www.tiktok.com
I installed suback via go get and then moved it into /usr/local/bin
I also tried other subdomains and subdomain lists that I know contain the toplevel domain of github and therefore should return something.

If you add the -v flag you will get output and it will write to the output file! Very weird behaviour. I'm gonna dig into this


Relaxed math testing uses Full Profile ULP requirements for Embedded Profile

The embedded profile ULP requirements for fast relaxed math are not considered.

Related, but distinct ticket: https://github.com/KhronosGroup/OpenCL-CTS/issues/703
I mention it because  I suspect that solving both problems at once is probably as easier as solving either alone.

I'd like to start looking at this.
My initial proposal is to keep it simple, and add something along the lines of float relaxed_error_embedded to the Func struct in FunctionList.h.
The current logic for changing the required ULP for the relaxed builtins would be slightly modified to check whether the device is EMBEDDED_PROFILE and if so, use relaxed_error_embedded instead.
If that is acceptable, I'll create a PR.
Feedback is most welcome.

Given there isn't separate ULP requirements defined for either double or half precision embedded relaxed math, this approach makes sense to me @Jeremy-Kemp

Thanks for the feedback.
If there are no objections, please could someone move this issue to "In Progress" in the 3.0 project and I'll get started.

@Jeremy-Kemp Now assigned to you and marked as "In Progress".


[BEAM-8605] Function display_graph() in example do not exist

R: @qinyeli
I have changed just comment.

R: @youngoli

Run Python PreCommit


On mobile, crashing when gathering metadata.

Mobile crashes while gathering metadata from: https://www.novelhall.com/rangers-of-the-three-kingdoms-9217/
Note: Does not crash on Windows 10/Desktop variant.

Seems fixed.


[Machine Learning] Building Ensemble Classifiers in Azure Machine Learning

Topic Suggestion
[Machine Learning] Building Ensemble Classifiers in Azure Machine Learning
Proposal Submission
Proposed article introduction
Ensemble classifiers in Azure Machine Learning is the most recent contribution to the Azure Machine Learning article series. Azure Machine Learning has two-class classification techniques available.
Every data modeler will wonder how to choose the best algorithms from the many options accessible. In order to determine the optimal classification technique, we can use accuracy, precision, recall, F1 measure, and ROC curve. This means that we will compare the assessment parameters to select a single technique.
We'll examine at how to perform predictions using several classification strategies in ensemble Classifiers so that we may produce better models with higher accuracy or avoid overfitting. This is the same as a patient who refers numerous specialists to diagnose a sickness rather than relying on just one.
Key takeaways

Building the standard Classifier using Azure Machine Learning
Configuring Ensemble Classifiers in Azure Machine Learning
What are Ensemble Classifiers in Azure Machine Learning

Article quality
How is your article unique? Tell us what makes your approach different from similar articles that have published on the same topic?
Is yours more in-depth? Does it cover additional topics? Do you provide handy tips or anecdotal advice?
References
Please list links to any published content/research that you intend to use to support/guide this article.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Good afternoon and thank you for submitting your topic to the EngEd program.
After some careful consideration it struck us that this topic may be a bit over saturated throughout other blog sites and official documentations. We typically refrain from publishing content that is covered widely on the net or other blogs. We're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.
But in order to approve the topic it has to serve value to the larger developer community at large. Please feel free to suggest an alternate topic to explore. 🚀
An option and a great way to write this as an in-depth article and make it more add value to the greater developer community at large would be to walk the reader through the USE of methods and functions by building a unique, different, useful project.
If you believe your article is unique enough - please present your case to explain how it will be different from what is currently out there?
https://www.google.com/search?q=Ensemble+Classifiers+in+Azure+Machine+Learning&rlz=1C1CHBF_enUS891US891&sourceid=chrome&ie=UTF-8


open-falcon over wan

Hi,
Thanks for the excellent work! However, I am just wondering how to set up the open-falcon over WAN link with http, https or ssh port only.
My company has machine rooms all around the earth and we have very restricted firewall rules to secure our network. Is it possibile use tunnel or proxy for all comunications, such as http, https or ssh?
I have gone through your docunmentation, the gateway component seems the most closed solution. But, I am not quite understand how to configure it for that purpose.
Initially, what I assume the architecture is like that:
remote area     ----------    central monitoring
agent   -------http,https,ssh--> gateway ---normal---> central open-falcon server
Therefore, I am a bit shocked when I notice that 4 services, http, rpc, socket and transfer, need to be configured in gateway config.
Please explain how to set up open-falcon over WAN link or refer me to an appropriated document.
Thanks and regards,

check this: https://github.com/open-falcon/falcon-plus/blob/master/modules/gateway/README.md


update deprecated call

In Rails 5, an application using espinita shows the warning below:
DEPRECATION WARNING: before_filter is deprecated and will be removed in Rails 5.1.
Use before_action instead.

It would be nice if this PR checks Rails.version , if it's >4.0.0 then use before_action , else before_filter.

Any update on this?

nope, this PR will not work on Rails versions < 5 , so we can't merge this PR as it is right now. When I have some time I could fix this,  but @dtengeri if you need the feature you can always fork and submit a PR.
cheers!


Advise for ~/.bashrc

Can we comment everything except
"export BASH_IT="/home/chengca/.bash.d/bash-it"
"export BASH_IT_THEME='bobby"
"source $BASH_IT/bash_it.sh"
In ~/.bashrc after install?
Because I think we should let the user decide which function to use except those necessary?

Just like oh-my-zsh.

And, can I random the theme?
Every time open the shell you got different theme?

Regarding the ~/.bashrc: Sure, you can remove anything you like from the file once it has been generated by the install script - only keep the three lines above, as they are required for Bash-it to work.
Other than that, feel free to change the file as you like. Some of the options are in there for historical reasons (since nobody cared to remove them) - I find they don't hurt.

Regarding the random theme: There's a random theme in #301, but the PR was never merged since it contained lots of other changes that shouldn't have been in the PR.
Feel free to take just the random theme from this PR and provide it as a new clean PR...

Ok, I see.
Thanks for your comment.


Cron job for expired accounts

Once accounts (like for tutorials) have an expiration (see issue #158 ), add a cron job to run nightly, Sunday through Thursday, and notify admins of accounts that have expired (should be deleted).
Notification email can go to idp_approval_email

The email should list the expired usernames, one per line. It can also include a link to the shib-idp site, and mention removing these accounts from projects / disabling the account on the portal.

Emails idp_approval_email the list of expired accounts (username_requested).
Run with --printonly to print to STDOT the usernames instead of emailing.
Uses /etc/geni-ar/settings.php to find the email address and DB connection info.


Revert "[Runtime][IRGen] Trap C++ exceptions on throw, not catch."

This reverts commits 76f2389fd91c3f1292e5992c88dec7970a0ef3cd, 25e9ca02b0862206e8454eb68e7e4f3fadc87051, 25e7f90af05382565e89610f62ad3f4b5a6afa63, and 143a473aa41215811f800f6656e459781de33dee from PR #71056.

@swift-ci Please smoke test

Sounds like we don't need this.


Requires the creation of an empty constructor, but it is!

PlayerData class:
@Getter
@Setter
@NTable(name = "player_data")
public class PlayerData extends NEntity<UUID> {

    @JsonProperty("coins")
    @Indexed
    private int coins;

    public PlayerData() { }

    public PlayerData(UUID uuid, int coins) {
        super(uuid);

        this.coins = coins;
    }
}

Hello, something is probably wrong between java versions.
What is the java version your spigot is running in ? and with what java version you exported your .jar plugin ?
Maybe your plugin java version doesn't match your jvm version.
As for NDatabase, it has been compiled in java 1.8, so it should work in java 1.8 and higher (tested until java 18)

I previously did an issue and since that time the version of java has not changed.
That is, previously the plugin worked, on version 0.1.5 NDatabase.
But I made a screenshot of the java version again:

No, NDatabase is located as a separate plugin on the server.
PlayerCounter.zip

I just tried to use your jar with a similar configuration as yours (except I'm using oracle java 17 and not a linux build) and it works fine. I suggest you to try with a fresh server with only NDatabase and your plugin to check if you have the same issue.
The way the constructor is called is the same as in 0.1.6 so it's quite surprising, does the issue is gone if you remove the @indexed and you downgrade to 0.1.6 ?

Oh ok, so as I said earlier maybe your second plugin contains the src of Ndatabase so you had a conflict.
As for the ">>" error, I realised that some PaperSpigot version was using an old SQLite driver but 3.38> is required to use JSON operators in SQLite.
I'm going to figure out the best approach for this limitation..
In the meantime you can try to change the SQLite driver yourself (usually all libs are installed into the "/libraries" folder, so the sqlite driver should be in libraries\org\xerial\sqlite-jdbc)

Hello, FYI I found a way to index the database without using 3.38 > GENERATED COLUMNS
You can re-test your playerCounter plugin.
https://github.com/NivixX/NDatabase/releases/tag/1.1.4
Also, I would advice to use other database type for more high scale servers

Hello, try this one
https://github.com/NivixX/NDatabase/releases/tag/1.1.6
I will write an adapter in a near future because the driver version used in spigot are not consistent

Hi, it works now, thanks!


is VECTOR_SEARCH_INDEX_NAME customizable?

mongodb-chatbot-server uses VECTOR_SEARCH_INDEX_NAME from .env variables.
but mongodb-rag-ingest does not.
I wanted to setup 2 environment: one with text-embedding-ada-002 and one with text-embedding-3-large, so I could test the difference in performance. I was wondering if I could use VECTOR_SEARCH_INDEX_NAME for that purpose (basically having 2 "embedded_content" collections, each with its search index).

Hi Eric,
If I understand what you're trying to do, you want to test the difference in performance of two different embedding models. Here's how you could do that:

Create two separate ingest.config.ts files, each one with a different embedder function and embeddedContentStore:

// ingest.config-ada-02.ts - using ada-02

// imports and set up

export default {
  embedder: async () => {
    // Use dynamic import because `@azure/openai` is a ESM package
    // and this file is a CommonJS module.
    const { OpenAIClient, OpenAIKeyCredential } = await import("@azure/openai");
    return makeOpenAiEmbedder({
      openAiClient: new OpenAIClient(new OpenAIKeyCredential(OPENAI_API_KEY)),
      deployment: "text-embedding-ada-002", // or whatever deployment name is
      backoffOptions: {
        numOfAttempts: 25,
        startingDelay: 1000,
      },
    });
  },
  embeddedContentStore: () =>
    makeMongoDbEmbeddedContentStore({
      connectionUri: MONGODB_CONNECTION_URI,
      databaseName: "ada-content", // or whatever database name is
    }),
  // other config the same
} satisfies Config;

// ingest.config-ada-02.ts - using text-embedding-03-large

// imports and set up same

export default {
  embedder: async () => {
    // Use dynamic import because `@azure/openai` is a ESM package
    // and this file is a CommonJS module.
    const { OpenAIClient, OpenAIKeyCredential } = await import("@azure/openai");
    return makeOpenAiEmbedder({
      openAiClient: new OpenAIClient(new OpenAIKeyCredential(OPENAI_API_KEY)),
      deployment: "text-embedding-03-large", // or whatever deployment name is
      backoffOptions: {
        numOfAttempts: 25,
        startingDelay: 1000,
      },
    });
  },
  embeddedContentStore: () =>
    makeMongoDbEmbeddedContentStore({
      connectionUri: MONGODB_CONNECTION_URI,
      databaseName: "text-embedding-03-large-content", // or whatever database name is
    }),
  // other config the same
} satisfies Config;

You'll need to set up the Atlas Vector Search indexes for each of these embedded content collections - https://mongodb.github.io/chatbot/mongodb#3-create-atlas-vector-search-index-required-for-rag.

You can give the indexes different names here, say "ada-02" and "text-embedding-03-large".

In your server config, use the same 2 embedders for different models and embedded content stores. Make sure to use the right Atlas Vector Search Index in addition for the relevant EmbeddedContentStore. You could either set up two server instances to run side by side or test in sequence.

// server config using ada-02

// rest the same

const embeddedContentStore = makeMongoDbEmbeddedContentStore({
  connectionUri: MONGODB_CONNECTION_URI,
  databaseName: "ada-content",
});

const embedder = makeOpenAiEmbedder({
  openAiClient,
  deployment: "text-embedding-ada-02",
  backoffOptions: {
    numOfAttempts: 3,
    maxDelay: 5000,
  },
});

// rest the same

// server config using text-embedding-03-large

// rest the same

const embeddedContentStore = makeMongoDbEmbeddedContentStore({
  connectionUri: MONGODB_CONNECTION_URI,
  databaseName: "text-embedding-03-large-content",
});

const embedder = makeOpenAiEmbedder({
  openAiClient,
  deployment: "text-embedding-03-large",
  backoffOptions: {
    numOfAttempts: 3,
    maxDelay: 5000,
  },
});

// rest the same

to make testing multiple embedding models side-by-side easier in the future, we're planning to support generating multiple embeddings from different models using the same ingest config file.
however, there's no immediate plan to implement that. so to test multiple models now, i recommend using the above approach.
i'll leave this issue open for now in case you have any additional questions.
thanks for raising all these issue. we really appreciate your interest in the project.

I ended up modifying the 'makeApp' function, to a custom function like
`
export interface LLLConfig
extends Omit<AppConfig, 'maxRequestTimeoutMs' | 'serveStaticSite'> {
apiPrefix: string
}
export interface ServerConfig
extends Omit<AppConfig, 'conversationsRouterConfig'> {}
export const makeAppAPI = async ({
app,
serverConfig,
llmConfigs,
}: {
app: Express
serverConfig: ServerConfig
llmConfigs: LLLConfig[]
}): Promise => {
const {
maxRequestTimeoutMs = DEFAULT_MAX_REQUEST_TIMEOUT_MS,
corsOptions,
serveStaticSite,
} = serverConfig
logger.info('Server has the following configuration:')
logger.info(
stringifyFunctions(
cloneDeep(llmConfigs) as unknown as Record<string, unknown>
)
)
app.use(makeHandleTimeoutMiddleware(maxRequestTimeoutMs))
app.set('trust proxy', true)
app.use(cors(corsOptions))
app.use(express.json())
app.use(reqHandler)
if (serveStaticSite) {
app.use('/', express.static(path.join(__dirname, '..', 'static')))
}
app.get('/health', (_req, res) => {
const data = {
uptime: process.uptime(),
message: 'Ok',
date: new Date(),
}
res.status(200).send(data)

})
app.use(errorHandler)
llmConfigs.forEach((llmConfig) => {
app.use(
${llmConfig.apiPrefix}/conversations,
makeConversationsRouter(llmConfig.conversationsRouterConfig)
)
})
app.all('*', (req, res, _next) => {
return sendErrorResponse({
reqId: getRequestId(req),
res,
httpStatus: 404,
errorMessage: 'Not Found',
})
})
return app
}
`
and an example of a LLMConfig:
  const llmConfig3: LLLConfig = {     conversationsRouterConfig: {       llm: llm3,       conversations,       generateUserPrompt,       middleware: [someMiddleware],     },     apiPrefix: '/api/gpt35',   }

nice! the implementation makes sense to me.


Feature request: Directly link to the last folder

In order to go quickly throw empty sub-folder and directly to the one containing files. It's could be interesting to link to folder containing file when it's a sub-folder of folder containing only one folder. To explain more easely look at this schema showing a folder structure.
/other.file
/etc -> /app -> /sub-folder -> - file.1
- file.2
For example in the root of the project clicking on the etc folder could get us directly to the sub-folder since there is nothing to see in the folder between.
I know that gitbucket does it like that :

The folder custom only contain a folder agirar01 so when we click on it we go directly to custom/agirar01.
I hope it' clear enough to understand.  :smirk:

Thanks your feedback!
But this is duplicated to https://github.com/gogits/gogs/issues/1149


(PUP-7791) add debug message when using latest packages

Before this commit, there's no indication of the current package
version when package resource is configured with latest, even
when running puppet agent with the debug flag,
After this commit a debug message with current version will be shown
if no upgrade upgrade is available.
Debug: Package[rsync](provider=yum): Yum didn't find updates, current version(3.1.2-10.el7) is the latest.

CLA signed by all contributors.


Projects missing

I added two projects for deployment in my Sanity Studio instance and overnight they both disappeared. No new build was triggered and the "Deploy" tab in the Studio was still there. I was just presented with an empty page and the option to add a new project. Is there additional configuration necessary to make sure projects are persisted for longer lengths of time?

@nring deploy entries are just documents unde the hood, if they are missing then they were deleted from your dataset (either through the API, CLI, or the Deploy interface).
There's nothing in the core plugin that deletes entries automatically.


3d pooling problem

Hi,all. I compiled the latest mxnet with cudnn 5.0.  3d conv is ok but 3d pooling is not. Below is an example code.
https://gist.github.com/WesleyZhang1991/dad733bb1ee8584dc4246672e9783164
import mxnet as mx
import numpy as np

import sys
from collections import namedtuple

batch_size = 2
crop_height = 112
crop_width = 112
clip_lenth = 16
chnum = 3

def Conv(data, num_filter):
    conv = mx.sym.Convolution(data, num_filter=num_filter, kernel=(3,3,3), pad=(1,1,1), stride=(1,1,1))
    relu = mx.sym.Activation(conv, act_type="relu")
    return relu

data_bm = mx.symbol.Variable('data', shape=(batch_size,chnum,clip_lenth,crop_height,crop_width))
conv1 = Conv(data_bm, 64)
pool1 = mx.sym.Pooling(conv1, kernel=(1,2,2), stride=(1,2,2), pool_type="max")
mod = mx.mod.Module(symbol=pool1,
                    context=mx.gpu())
provide_data = [('data', (batch_size,chnum,clip_lenth,crop_height,crop_width))]
provide_data = [(name, (batch_size,) + shape[1:]) for name, shape in provide_data]

mod.bind(data_shapes=provide_data, for_training=False, inputs_need_grad=False)
mod.init_params(initializer=mx.init.Xavier(magnitude=2.))
Batch = namedtuple('Batch', ['data'])

for i in range(10):
    data = np.random.rand(batch_size,chnum,clip_lenth,crop_height,crop_width)
    mod.forward(Batch([mx.nd.array(data)]))
    feature = mod.get_outputs()[0].asnumpy()
    print feature.max()

The right output from 3d pooling should be no larger than 1.0.However, I often get results -3.40282e+38 which seems to me that cudnn has not return calculated output. Any advice?

Can you try this after pulling the latest code from the repo?
Create the pool sym as this:
pool1 = mx.sym.Pooling(conv1, kernel=(1,2,2), stride=(1,2,2), pool_type="max", cudnn_off=True)

This will turn off cudnn 3D pooling and use MXNet pooling operator instead. Check whether the results are expected and we can go from there.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Remove year from copyright verbiage

Goodbye 2020 👋
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@abhay-krishna: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

helm-chart-tooling-presubmit
d093a93f7961d67803457903014ddb8cfc16f359
link
/test helm-chart-tooling-presubmit

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: abhay-krishna, jaxesn
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [abhay-krishna,jaxesn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Better collapse functionality for Structure

Now dragging into a collapsed structure won't expand it, and dragging
right of a collapsed structure creates a new node.

https://vimeo.com/167344323 Demo of the new process. @jpstroop @joycebcat ?

That looks better to me. So a structure has to be collapsed in order to create a sibling? I think that makes sense.

@jpstroop It doesn't -have- to, you can make one the same way you used to - drag too far and then drag left. You just -can- do it this way now.

@tpendragon I am unable to replicate what I see in the video.  Is it in hydra-dev now?  Everytime I try to create a new section as shown, the collapsed section pops open and either nothing new is created or they end up nested under the first section.  I find that I have sometimes actually changed the order of the images while I'm trying to create a section where I want to place it.

No, it won't be on hydra dev until this PR is merged. Sorry for the confusion!


Create LICENSE

This pull request is part of an initiative to make the licensing information about OBO Foundry ontologies more actionable.
While this ontology contains licensing information in its metadata, GitHub is not aware of the license unless you create a LICENSE file with standardized text. This PR creates a LICENSE file with the standard text for the CC BY 4.0 license.
More information about this initiative and further discussion is taking place here: https://github.com/OBOFoundry/OBOFoundry.github.io/issues/1840.

@pfabry thanks! I made a copy/paste error when I pinged Marek. Sorry about that.

No problem! 6 months for merging such a PR is pretty terrible on my part, but now that my Github alarms are configured correctly I should be more reactive...


create index.json from file structure

An example call to the function is presented here:
plex util generate index --app="diffdock" --directory="home/ubuntu/dir"

when app=="diffdock":

a .plex file is read that contains a JSON that specifies the file endings that are related to inputs for app. The entry for diffdock would look like this:

{"app": "diffdock",
   "inputs": {"protein_path": ".pdb"},
                   {"ligand": ".sdf"},
   "outputs": {"pdb", "sdf"}}

a temporary directory tmp is created
all files matching the file endings listed for app are listed
all listed files are copied to tmp in a flat directory structure
a jsonlines object is created that lists the files in tmp according to the JSON that is stored in .plex

{"protein_path", "ligand"}
{"protein1.pdb", "ligand1.sdf}
{"protein1.pdb", "ligand2.sdf}

missing pieces as of 23-02-04:

[ ] index.csv generation from moved files
[ ] instruction.json generation (will require upgrade to app-config
[ ] additional tests
[ ] refactoring of main for integration into plex

plex = ProtocoL EXchange

instead of an index.csv, I decided to create a dictionary that specifies the nature of each file. From there, a .csv file can be created.

output is creating job directory locally and index file locally
then we can create two CIDS.


Show types in Intellisense during object initialization

During object initialization, there is no hint for what type a property has which makes working with unfamilliar classes difficult. There should be an indication of the type of a property.
How it is now:

Proposed solution:

VSCode version: 1.41.0
C# Extension: 1.21.8
Dotnet Information
.NET Core SDK (reflecting any global.json):
 Version:   3.1.100
 Commit:    cd82f021f4
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.18362
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.100\
Host (useful for support):
Version: 3.1.0
Commit:  65f04fb6db
.NET Core SDKs installed:
2.1.701 [C:\Program Files\dotnet\sdk]
2.2.103 [C:\Program Files\dotnet\sdk]
2.2.301 [C:\Program Files\dotnet\sdk]
3.0.101 [C:\Program Files\dotnet\sdk]
3.1.100 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.14 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.14 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.0.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.14 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.0.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.0.1 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

this looks like a regression to me - it used to work. Will check.


Would like to use this for a school project

alexjohntomy.github.io/dataConn/
Using MIDI pad as input device for numbers quiz

actually nvm, i was able to use another proxy, since i only need this for a few days. thank you for providing this.


fix: Standardize precedence when options and htmlProps conflict

This PR standardizes the precedence when options and htmlProps have conflicts. This is only an issue when using props hooks instead of components.
Does this PR introduce a breaking change?
No

@diegohaz this should probably be mentioned in the docs right? In the composition page maybe?

@tom-sherman Sure! Could you open a follow up PR?