JS Import Updates on File Rename Expand Node_Module Paths

VSCode Version: 1.25.1
OS Version: MacOS 10.13.6

Steps to Reproduce:
This may be an issue with a specific plugin, but I'm going to start by reporting it here.

Have a JS file that has some imports from libraries in node_modules
Rename that JS file
Other files that import the module from the renamed JS file will also expand the imports from these 3rd party libraries from node_modules, so that they are now relative imports from my machine's code location (ie, Yarn cache) instead of the library import that it should remain as.

For example, if the affected file had this import:
import React from "react";

It would then get incorrectly changed to:
import React from "../../../../../../Library/Caches/typescript/2.9/node_modules/@types/react";

Note that the following setting needs to be set in my workspace preferences (equivalent to "Always Yes" in a popup that appears had the setting not been set):
"javascript.updateImportsOnFileMove.enabled": "always"

Does this issue occur when all extensions are disabled?: Yes

I got the same issue, many times

@mjbvz I'll give it a shot on Insiders, but I won't be able to until later today. Do you know the EYA on the fix releasing to the stable VSCode version? Also, I saw that you marked this ticket as a duplicate. Can you please link the duplicated ticket? Thank you.

This is so frustrating,

@mjbvz okay, I have confirmed that this is not broke in the Insiders build.
I am still curious about what caused you to mark this as a duplicate ticket and thus close it. I would have expected you to link another issue here when marking it as a duplicate. Can you please link to that duplicate ticket? I want to track the status of the fix (even if it has already been fixed).


Don't write keywords with a None value to the files

Until now, I thought that if a keywords is Optional and has a None value, that HYDROLIB-core would not write these keywords to the mdu file. However, recently we have added a lot of new research keywords that are all optional and have a default value of None. It turns out that if the user uses one research keywords, then all new research keywords will be written the mdu file. This is unwanted behaviour.
We then checked the behaviour for non-research keywords and we found that there are still a lot of regular keywords that do not have a default value. That problem will be tackled in #648.
In this issue we will change that HYDROLIB-core will no longer write keywords with a None value to the MDU file. The exception is for FileModels as they can't have a default value.

To avoid that non-research keywords are not printed to the mdu, I looked up the default values in the FM kernel code for the keywords for which HYDROLIB-core sets None as a default.
Numerics > Comments > qhrelax: "Relaxation on Q-h open boundaries" (keyword not present in *.f90 files, deprecated?)
Numerics > fixedweirtopfrictcoef: -999 (src/dflowfm_data/m_flowparameters.f90)
Physics > itcap: 0 (src/dflowfm_data/m_flowparameters.f90)
Time > autotimestep: 1 (difficult to find in code, seems to be ja_timestep_auto in src/dflowfm_data/m_flowtimes.f90)
Time > startdatetime: ' ' (src/dflowfm_data/m_flowtimes.f90)
Time > stopdatetime: ' ' (src/dflowfm_data/m_flowtimes.f90)
Restart > restartdatetime: ' ' (src/dflowfm_data/m_flowtimes.f90)
ExternalForcing > rainfall: 0 (guessed, too much search results)
ExternalForcing > qext: 0 (guessed, no search results)
ExternalForcing > evaporation: 0 (guessed, too much search results)
ExternalForcing > windext: None (wind should come via the ext file, this keyword should be deprecated one day I think) >>??
Trachytopes > trtmxr: 8 (md_mxrtrach in src/dflowfm_data/unstruc_model.f90)
Geometry > Comments uniformwidth1d: "Uniform width for channel profiles not specified by profloc" (src/dflowfm_data/unstruc_model.f90)
Geometry > uniformwidth1d: 2 (wu1DUNI in src/dflowfm_data/m_flowgeom.f90)
Geometry > dztop: -999 (src/dflowfm_data/m_flow.f90)
Geometry > floorlevtoplay: -999 (src/dflowfm_data/m_flow.f90)
Geometry > dztopuniabovez: -999 (src/dflowfm_data/m_flow.f90)
Groundwater > Comments > groundwater: "0=No (horizontal) groundwater flow, 1=With groundwater flow"
Groundwater > Comments > hinterceptionlayer: "Intercept this amount of rain (m)"
Groundwater > hinterceptionlayer: None? (no default value set)
Groundwater > bgrwuni: -999 (src/dflowfm_data/m_grw.f90)
Groundwater > sgrwini: -999 (src/dflowfm_data/m_grw.f90)
Processes > Comments > dtmassbalance: "waq mass balance output time step" >> deprecated keyword according to src/dflowfm_data/fm_deprecated_keywords.f90

filepaths (indirectly in src/dflowfm_data/unstruc_model.f90)
should None not be replaced with this, just like for flowgeomfile and many others?
default_factory=lambda: DiskOnlyFileModel(None)
ExternalForcing > extforcefile (md_extfile)
ExternalForcing > extforcefilenew (md_extfile_new)
Trachytopes > trtdef:
Trachytopes > trtl:
Output > outputdir:
Output > waqoutputdir:
Output > obsfile:
Output > crsfile:
Geometry > bathymetryfile:
Geometry > inifieldfile:
Geometry > thindamfile:
Geometry > fixedweirfile:
Geometry > pillarfile:
Geometry > vertplizfile:
Geometry > frictfile:
Geometry > crossdeffile:
Geometry > crosslocfile:
Geometry > storagenodefile:
Geometry > partitionfile:
Geometry > gridenclosurefile:
Geometry > roofsfile:
Geometry > gulliesfile:
Particles > particlesfile:

After the implementation in #663, there are still some errors generated by hydrolib core:

two keywords are in the FM code, but not yet in hydrolib-core. These were incorrectly marked as deprecated AFAIK
several keywords are deprecated, it would be nice to mention this separately, follow-up issue created in https://github.com/Deltares/HYDROLIB-core/issues/668

Missing keywords

jadelvappos (int, 0) (from src/dflowfm_data/m_physcoef.f90)
umodlin (float, 1.0) (from src/dflowfm_data/m_physcoef.f90)


Providing an asyncio interface

Since newspaper3k is now a python3 library, it'd be nice to also support an asyncio-compatible interface for users that want to integrate its use into existing async applications.

Any plans of adding this?

Any update on this?


[WIP] Add kubernetes Ingress backend

This PR adds Kubernetes Ingress backend.
This implementation is working, but there is a major issue, the outrageous dependencies needed by the official k8s client: https://github.com/kubernetes/kubernetes/issues/11606
Here are the dependencies imported by the client when compiling:
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/api/unversioned
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/third_party/forked/reflect
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/conversion
github.com/emilevauge/traefik/vendor/github.com/gambol99/go-marathon
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/conversion/queryparams
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/errors
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/sets
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/types
github.com/emilevauge/traefik/vendor/github.com/spf13/pflag
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/runtime
github.com/emilevauge/traefik/vendor/speter.net/go/exp/math/dec/inf
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/auth/user
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/fields
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/api/resource
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/validation
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/labels
github.com/emilevauge/traefik/vendor/github.com/ghodss/yaml
github.com/emilevauge/traefik/vendor/github.com/juju/ratelimit
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/yaml
github.com/emilevauge/traefik/vendor/github.com/docker/docker/pkg/mount
github.com/emilevauge/traefik/vendor/github.com/docker/go-units
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/api/meta
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/runtime/serializer/recognizer
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/runtime/serializer/versioning
github.com/emilevauge/traefik/vendor/github.com/opencontainers/runc/libcontainer/configs
github.com/emilevauge/traefik/vendor/github.com/opencontainers/runc/libcontainer/system
github.com/emilevauge/traefik/vendor/github.com/opencontainers/runc/libcontainer/cgroups
github.com/emilevauge/traefik/vendor/github.com/pborman/uuid
github.com/emilevauge/traefik/vendor/github.com/opencontainers/runc/libcontainer/cgroups/fs
github.com/emilevauge/traefik/vendor/github.com/beorn7/perks/quantile
github.com/emilevauge/traefik/vendor/github.com/golang/protobuf/proto
github.com/emilevauge/traefik/vendor/github.com/prometheus/common/internal/bitbucket.org/ww/goautoneg
github.com/emilevauge/traefik/vendor/github.com/prometheus/common/model
github.com/emilevauge/traefik/vendor/github.com/prometheus/procfs
github.com/emilevauge/traefik/vendor/golang.org/x/crypto/curve25519
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/intstr
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/net
github.com/emilevauge/traefik/vendor/golang.org/x/crypto/ssh
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/runtime
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/rand
github.com/emilevauge/traefik/vendor/github.com/emicklei/go-restful/log
github.com/emilevauge/traefik/vendor/github.com/emicklei/go-restful
github.com/emilevauge/traefik/vendor/github.com/prometheus/client_model/go
github.com/emilevauge/traefik/vendor/github.com/matttproud/golang_protobuf_extensions/pbutil
github.com/emilevauge/traefik/vendor/github.com/prometheus/common/expfmt
github.com/emilevauge/traefik/vendor/github.com/prometheus/client_golang/prometheus
github.com/emilevauge/traefik/vendor/github.com/emicklei/go-restful/swagger
github.com/emilevauge/traefik/vendor/github.com/google/cadvisor/info/v1
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/validation/field
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/api/errors
github.com/emilevauge/traefik/vendor/github.com/docker/docker/pkg/parsers
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/parsers
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apimachinery
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/api/util
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/capabilities
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/client/transport
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apimachinery/registered
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/client/metrics
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/util/wait
github.com/emilevauge/traefik/vendor/github.com/blang/semver
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/watch
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/version
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/watch/json
github.com/emilevauge/traefik/vendor/github.com/mailgun/manners
github.com/emilevauge/traefik/vendor/github.com/mailgun/oxy/forward
github.com/emilevauge/traefik/vendor/github.com/mitchellh/mapstructure
github.com/emilevauge/traefik/vendor/github.com/spf13/cobra
github.com/emilevauge/traefik/vendor/github.com/hashicorp/hcl/hcl/strconv
github.com/emilevauge/traefik/vendor/github.com/hashicorp/hcl/hcl/token
github.com/emilevauge/traefik/vendor/github.com/kr/text
github.com/emilevauge/traefik/vendor/github.com/hashicorp/hcl/hcl/ast
github.com/emilevauge/traefik/vendor/github.com/hashicorp/hcl/hcl/scanner
github.com/emilevauge/traefik/vendor/github.com/hashicorp/hcl/json/token
github.com/emilevauge/traefik/vendor/github.com/kr/pretty
github.com/emilevauge/traefik/vendor/github.com/hashicorp/hcl/hcl/parser
github.com/emilevauge/traefik/vendor/github.com/hashicorp/hcl/json/scanner
github.com/emilevauge/traefik/vendor/github.com/hashicorp/hcl/json/parser
github.com/emilevauge/traefik/vendor/github.com/magiconair/properties
github.com/emilevauge/traefik/vendor/github.com/hashicorp/hcl
github.com/emilevauge/traefik/vendor/github.com/spf13/jwalterweatherman
github.com/emilevauge/traefik/vendor/github.com/spf13/cast
github.com/emilevauge/traefik/vendor/github.com/thoas/stats
github.com/emilevauge/traefik/vendor/github.com/unrolled/render
github.com/emilevauge/traefik/vendor/github.com/spf13/viper
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/runtime/serializer/json
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/runtime/serializer
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/api
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/extensions
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/api/v1
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/api/validation
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/authorization
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/authorization/v1beta1
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/componentconfig
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/authorization/install
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/kubelet/qos
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/metrics
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/componentconfig/v1alpha1
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/componentconfig/install
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/api/install
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/metrics/v1alpha1
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/extensions/v1beta1
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/metrics/install
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/apis/extensions/install
github.com/emilevauge/traefik/vendor/k8s.io/kubernetes/pkg/client/unversioned

Unfortunately, this leads to a binary that weights 43M (compared to 18M initially)!
-rwxr-xr-x 1 emile emile  43M févr.  9 19:50 traefik*

You are welcome to propose alternatives ;) I will not merge this one until a decent size is back.

[x] initial implementation
[ ] clean code
[ ] find alternative to k8s official API
[ ] create an example k8s resources file

Fixes #11

You're not alone: https://github.com/kubernetes/kubernetes/issues/11606

Maybe provide it with a build tag? Than you can build and release a slim version without k8s and a bloated version including k8s.

@tboerger I'm :+1: on that :angel:

Yeah, I was thinking about that too. This could be a temporary solution waiting for a lighter client.

Yeah, I was thinking about that too. This could be a temporary solution waiting for a lighter client.
Hide all checks

Sure, it can always be removed if the client shrinks sizewise.

A few days ago Kubernetes 1.2 was released.
Is it possible that the newest Kubernetes client has less dependencies?
Yesterday i tried to understand your code and found the following line.
What's the use of this "Limiter"?
rateLimiter := util.NewTokenBucketRateLimiter(0.1, 1)
By the way, the method "NewTokenBucketRateLimiter()" is no longer part of the "k8s.io/pkg/util/" package.
https://godoc.org/github.com/kubernetes/kubernetes/pkg/util

Is it possible that the newest Kubernetes client has less dependencies?

Not really :'(
-rwxrwxr-x 1 emile emile  49M mars  31 13:57 traefik*

I had this exact same issue when adding Kubernetes discovery to Prometheus. What I ended up doing is copying over the types I needed (a tiny subset of the whole Kubernetes API) & just using a standard http.Client with chunking for watches on service endpoints. This seems like it should work perfectly well here too.

what about a server that runs parallel to traefik, which syncs kubernetes-ingress via the traefik-file (just a go-template)? the go-template could be provided via a kubenetes-configmap, so one can define or update the templates by using kubectl {create,apply} traefik.yml.
that way traefik would no need any k8s-dependencies (less binary-size AND loc) and traefik would not need to be updated everytime k8s changes ingress-behavior :D

something like this:
# k8s-ingress-controller-traefik.yml

apiVersion: v1
kind: ReplicationController
metadata:
  name: nginx-ingress-controller
  labels:
    k8s-app: ingress-controller-traefik
spec:
  replicas: 1
  selector:
    k8s-app: ingress-controller-traefik
  template:
    metadata:
      labels:
        k8s-app: ingress-controller-traefik
    spec:
      containers:

      # traefik is running as load-balancer

      - image: traefik
        ports:
        - containerPort: 80
          hostPort: 80
        - containerPort: 443
          hostPort: 4444
        command: /traefik -c /var/traefik-file/traefik.yml
        volumeMounts:
        - name: data
          mountPath: /var/traefik-file

      # kube2traefik will sync k8s-ingress-resources to the file:
      # /var/traefik-file/traefik.yml
      # also kube2traefik subscribes to k8s-api for a configmap
      # with the name "ingress-controller-traefik-configmap"

      - image: kube2traefik
        volumeMounts:
        - name: data
          mountPath: /var/traefik-file

      volumes:
      - name: data
        emptyDir: {}

---

apiVersion: v1
kind: ConfigMap
metadata:
  name: ingress-controller-traefik-configmap
data:
  template: "<golang-template for traefik-config-file>"

Yeah sounds good. I'd love to see native kubernetes support, but I see some other issues that will come up like config providers not being able to pass certificates for SNI.

Romulus already has some kind of traefik support. https://github.com/albertrdixon/romulus
But it would be really nice to have batteries included.

With regard to shrinking binaries, have you checked out these options? https://blog.filippo.io/shrink-your-go-binaries-with-this-one-weird-trick/

Hi all, I finally decided to copy the API ;)
This branch is now working with Kubernetes backend with a descent binary size:
-rwxrwxr-x 1 emile emile  16M avril 19 16:40 traefik*

I'm thinking of merging this one as is, but: for now, it doesn't support Ingress TLS: as a workaround for the certs, we could use secrets and mount the secrets as volumes in traefik container.
WDYT?
(PR description updated)

I like the idea of using secrets for the certs. No need to implement something new or have the user learn anything other than what is already documented with kubernetes.

Rather than mounting the secrets, which is only done on pod startup, I think you'll need to get the secret via the Kubernetes API, especially when ingresses are dynamically updated.

@jimmidyson totally agree :)
But, I'm not sure that the majority uses Kubernetes secrets to store their SSL certs.
Besides, you could use Let's Encrypt with Traefik to dynamically get your certs.
My proposal is to merge this one, as a first iteration.
I will add the ingress secrets support later.
Seems reasonable?

@emilevauge SGTM

@emilevauge Absolutely.

@emilevauge +1
I currently have Romulusd configured as an ingress controller and then use Let's Encrypt along with Traefik for SSL and it's totally great

i played a little bit the the example.
i found it would not work when:
a) namespaces get added or
b) the port changes (i used gcr.io/google_containers/echoserver:1.3 on port 8080)
i am looking forward to see traefik read its config directly from kubernetes.
great job :D

@damoon: thanks! Just pushed a fix that add namespace management and change rule PathPrefix -> PathPrefixStrip

I just tested the implementation in worked right away, good work!
I wonder if it's possible to route directly to the pods instead of using the local kube-proxy as another layer in between. But if I understand correctly, that's how ingress is supposed to be used right now.
Is it already possible to register https letsencrypt support out of the ingress yaml?
Pretty nice so far!

As an example of k8s secrets for SSL this might help you out.  They have a similar setup, but it's k8s only.
https://github.com/deis/router#ssl

@sstarcher: thanks, but that has nothing to do with traefik at all, or am I wrong? Actually I don't like to store TLS certs as secrets since pods need to restarted to get access to updated certs. I'd like to have traefik handle that.

Few comments / questios, but otherwise looks good :+1:

@emilevauge for the k8s example can you add daemonset too. I think this kind more appropriate for this app
  apiVersion: extensions/v1beta1
  kind: DaemonSet
  name: traefik-ingress-controller
  labels:
    k8s-app: traefik-ingress-lb
  spec:
    template:
      metadata:
        labels:
          k8s-app: traefik-ingress-lb
      spec:
        nodeSelector:
          app: traefik-ingress-lb
      terminationGracePeriodSeconds: 60
      containers:
      - image: containous/traefik
        name: traefik-ingress-lb
        imagePullPolicy: Always
        ports:
        - containerPort: 80
          hostPort: 80
        - containerPort: 443
          hostPort: 443
        args:
        - --kubernetes
        - --logLevel=DEBUG

@ekozan: Why not, but your example is not working. Official examples of ingress controllers are deployed as ReplicationController.
I'll keep the replication controller one for now ;)

not only ReplicationController: https://github.com/kubernetes/contrib/blob/master/ingress/controllers/nginx/examples/daemonset/as-daemonset.yaml
hum ok forget it
I'll make a pull request after yours with a working example and readme

The only reason why DaemonSets should not be used is because currently there's no way to update or rolling-update them. Ideally as DaemonSets mature and allow updating, they would be the best choice as every host should really be running Traefik with dns round robin in front. But +1 to official example being an RC for now

Unless you are targeting K8S 1.1 I would recommend using a Replica Set over a Replica Controller in K8S 1.2

LGTM 🐻


Start with development help: Installing my build on minikube/kind

Bug Description
I follow the instruction Using the Code Base.
In the step Building the code the code is built into some output (e.g. ..istio/out).
The the step that follows is Run the code:
go run ./istioctl/cmd/istioctl install

and the command is preceded by the description saying that it uses the build from the previuos step. But it's not true - it's a regular installation with the docker image set to, e.g.  gcr.io/istio-testing no my build.
So, the questions are:

Is this instruction still valid?
How to install istio locally into my e.g. minikube from either my local build (I suppose it's a challanging task;)) or my docker hub?

Version
n/a

Additional Information
No response

You can set the --hub and --tag flags to where you pushed the images


DAT-68229  Highlight row on mount

Hi @fadiDL @makc-dataloop @guyDataloop
Please review the code.

DAT-68229  Highlight row on mount

Thank you for your contribution to the repo. Before submitting this PR, please make sure:

[ ] Your code builds clean without any errors or warnings
[ ] You are using approved terminology
[ ] You have added unit tests
[ ] You have updated documentation
[ ] You have tested your changes

Please provide a description of the changes proposed in the pull request and reference any related issues in the repository.
Please indicate if this PR contains:

[ ] A bug fix
[ ] A feature request
[ ] Breaking changes
[ ] An enhancement

@fadiDL @guyDataloop context: <dl-tree-table> component used by label picker has its own selection mechanism that presents checkboxes, so back in DAT-67499 PR https://github.com/dataloop-ai/components/pull/901 I toggled <tr> class instead, however it only worked on click. here Yogesh does the same but via data attribute.


Fix stablelm

Stabilityai has uploaded transformers implementation. https://huggingface.co/stabilityai/stablelm-3b-4e1t/discussions/10
from vllm import LLM, SamplingParams

# Sample prompts.
prompts = [
    "Hello, my name is"
]
# Create a sampling params object.
sampling_params = SamplingParams(temperature=0.0, max_tokens=64)

# Create an LLM.
llm = LLM(model="stabilityai/stablelm-3b-4e1t", dtype="half", tensor_parallel_size=1, enforce_eager=True, trust_remote_code=True)
# Generate texts from the prompts. The output is a list of RequestOutput objects
# that contain the prompt, generated text, and other information.
outputs = llm.generate(prompts, sampling_params)
# Print the outputs.
for output in outputs:
    prompt = output.prompt
    generated_text = output.outputs[0].text
    print(f"Prompt: {prompt!r}, Generated text: {generated_text!r}")

Ouput:
Prompt: 'Hello, my name is', Generated text: " and I'm writing you today to learn more about the 2018 Ford F-150 XLT SuperCrew 5.5' Box 4WD listed for $39,995.00. I live in the area and I would like to hear back from you soon and learn more about this vehicle. Please call me at at"

@esmeetu Should we replace the old version with this? Do we have a reason to keep the old one?

AFIAK the old version is needed for older huggingface version?

@simon-mo We already updated our transformers version to >=4.38.0 so it should be fine.

@WoosukKwon There only have small configuration changes on model and I think we can keep the backwards compatibility for a while.

@esmeetu Could you explain more about the backward compatibility? Do you mean the situation where the user uses an older version of transformers?

@esmeetu Could you explain more about the backward compatibility? Do you mean the situation where the user uses an older version of transformers?
No, this will provide a more kindly user experience for those (including me) who just use a local offline model which downloads repo before update. Otherwise when upgrading vLLM, it will throw an error and let them confused since they don't know that update information and think this is must an issue about vLLM upgrade.🤯

@esmeetu Got it. Thanks for the explanation!


Scalable annotation rendering

This is a (potentially ignorant) question(s) and a feature/enhancement request(s). There are scalability concerns in the current system, which I think will need to be addressed as we move from displaying nucleus annotations inside relatively small regions of interest (about 1/20th of tissue area) to whole-slide nucleus annotations. Put in other words, the volume of annotation items (polygons) we need to render is about 20x anything we have so far, so we need to make sure the performance is very smooth with current annotation volume given it can only go worse as we massively scale up the number of annotations. To make the formulation of this request more well-defined, I've recorded my browser (Chrome), including the debug panel, as I walk through a slide with a moderate number of annotations. The video of the recording can be accessed in this link:
** https://youtu.be/YbPD7OSjQoQ **
00:13 - I click on the 'eye' icon to show all annotation polygons in the slide. It takes up to 00:30 (about 20 seconds) to load all polygons. Not sure if this is good enough or can be improved. The more important issue, however, is the loading behavior. A server request is made for each annotation document (group of polygons) and as soon as the entire document is loaded, it gets rendered. This is related to issue #625 . The fact that a huge annotation document needs to be loaded (i.e. our 'unit' is the annotation document not the individual polygon) means a rather large file is loaded before any of the, say, 1000+ polygons is displayed. While all these documents are loading, the page is almost always unresponsive (naturally). The following can be done to address this:
1- As we address issue #625 , we may decide to make the individual polygon (element) as our unit of interest and render each polygon after it is received. Advantage: makes more sense. Disadvantage: not sure if making 30,000 server requests (one per polygon) makes sense or is efficient.
2- Alternatively, we may decide to bundle together polygons (say, 1000 elements at a time) automatically when they are requested. This makes sure there is a maximum number of polygons per "bundle" and ensure the wait time before something renders on the screen is reasonable.
3- Regardless of how we handle this, a 'loading' bar is needed to show loading progress like:

[=== >                 ] annotation 5 of 10,000

Not only is this good from a user experience perspective, it also prevents the user from trying to zoom in on their area of interest before all annotations have completely loaded (hence preventing crashing or unresponsiveness). This was suggested in issue #592 .
00:31 - 1:43 - I inspect a few of the server requests made to get annotation documents. Nothing out of the ordinary. Of notice, however, is that this is the only time these requests appear. In other words, as I pan to different area of the slide afterwards we do not re-fetch any annotations from the server anymore but use the ones already fetched at the start. That's not necessarily a bad thing, as server requests take time and impact performance. BUT, this implies that one of two things is happening:
1- Either the annotations are rendered once (like a canvas) and that's it. --> Scalable??
2- Or, they are stored locally in the browser cache or some local database o the client side and get fetched a few at a time as I pan different areas of the slide. --> Scalable
Which of these two scenarios is happening? From all I could find, it seemed scenario no. 1 is happening which may not be very scalable (I could be wrong about this though).
1:44 - 3:22 - I pan various areas of the slide. There are a bunch of server requests for tiles (images) but none for annotations.
3:23 - 4:20 - I check local database and browser cache. The only thing that is stored, it seems, are the annotation styles. This seems to confirm scenario no. 1 where the annotations are only rendered once like a canvas.
Sorry for really long question/request, but I anticipate (and @cooperlab seemed to agree) that these issues will only get magnified as we scale up annotation efforts.

Depending on the size of an annotation, either of scenario 1 and 2 are used.
Currently, when you view an annotation, we make a request for that annotation.  If you click the 'view all', we make one request for each annotation, all basically at the same time.  As the annotations are returned, we render what has come back.  Each of these initial queries has a "detail limit" (crudely speaking, this the maximum number of vertices that will be returned -- so polygons are more details than rectangles are more details than points).  If the annotation has less than the detail limit, we have loaded all of it and then display it all without any need to update.  Assuming you have webgl, the system should be tolerably responsive with all of these loaded.
If an annotation exceeds the detail limit, we ask for only the elements within the window that is viewed plus some surrounding area, getting the largest annotation elements by preference if we can't load them all.  Then, when you pan or zoom beyond some near location, we requery to get the elements in that area.  This behavior allows scaling to an annotation that has millions of polygons, since we'll never have the millions in browser memory all at once.
There are several pinch points where things take time: it takes time to make the server query to get the annotations, it takes time to send that information across the network, it takes time to convert the received information into a form we can render, and it takes time to render the annotations.
If we requested less data initially, the responses are faster, the rendering is faster, but the rendering is less complete.
Currently, we transmit data in json; if we used something like a protobuf, we could halve the network traffic volume.  I have an internal hacky branch to try this out -- the network delay is reduced, but my hack makes the server take longer to produce the result (which it wouldn't need to do).
When there are too many annotations to render all at once, we could send, for instance, just the centroids of the polygons and render them as dots until you zoom in.  On large nuclei sets, the time to send these centroids is still too slow to feel peformant, so this wouldn't be sufficient unto itself.
We could use web workers for the data reception, which would keep things feeling more performant, allowing us to fetch chunks of data and assemble it in the background, then push it to the foreground.  This actually uses more memory (since the worker gets a partial response that then has to be combined with the collected response), so this may reduce the maximum data we want to send to a browser.  However, if done well it could make things feel more responsive.
We could precompute some decimated results for large annotation sets, so that we fetch those subsets initially (simplified polygons at low resolution, for instance, or some sort of heatmap).  This requires more database bookkeeping and more complexity in the rendering client, but could feel more responsive.  This would be added if we used something like PostGIS to store the annotations rather than Mongo, since we could then have an optimized database module do the decimation.
Our default detail limit was picked arbitrarily and is not optimal (and may not be balanced between different element types such as polygons versus rectangles).

I should add, there are other places that can be changed to possibly speed things up as well.  Currently each annotation is sent to webgl in a separate buffer.  This is fine for a few annotations with lots of elements, but sub-optimal for many annotations with a medium number of elements.  We reformat the annotations between their transmission format and a display format; this could certainly be improved.

@manthey Thank you for such a detailed and nice response. I agree that these would all be nice to improve performance, and my opinion is to incorporate as many of these as possible since we’ll definitely scale up the number of annotations that need rendering.

Showing centroids is a great idea, especially for polygons that are too small to show at a magnification anyway. We could set a limit that links the size of polygon to the magnification at which it is rendered as a boundary.
The current distinction between an annotation with many elements and many annotations with a few elements will eventually be revised anyways as we address the annotation styles CS documents issue.
Regardless of how we handle this, the user should be blocked from sending any requests when they click the eye icon to show all annotations. Loading bar should be sufficient for this.
Definitely agree that the detail limit would need to be set more rationally, perhaps as a function of type of elements.
Related to the annotation document va style issue, it maybe more sensible to make a document for each annotations within a geographic area (eg groups of tiles), regardless of style. The idea being that if there are millions of elements, this gets just one query as you pan to a certain area. Current organization as I understand it would be less efficient for annotations that have thousands of elements geographically scattered in different areas of the slide.
Usinng protobuff or other efficiency enhancements would definitely help.
Thanks!

As we now send centroids on large annotation collections and fixed an issue when saving changes in quick succession, I'm closing this issue.  Any remaining details should made into new issues.


organize files

I'm creating a python program to organize files in their folders
Can you give this issue to me, and tag it with the Hacktoberfest tag?
@theadeyemiolayinka theadeyemiolayinka

Go ahead, @Jacintomuambumba


fix: typing annotation None to Generator for pytest setup functions (TDE-285)

Change Description:
Use Generator type for return type annotation of pytest setup() functions.

Looks good, but why not put the duplicate setup functions in conftest.py?

Good idea, we haven't think about it. I've made a new commit.


Issue with k8s.io/docs/setup/independent/install-kubeadm/

This is a documentation issue

[ ] Feature Request

Problem:
Poorly worded install instructions.
Proposed Solution:
improve it please.
Page to Update:
https://kubernetes.io/docs/setup/independent/install-kubeadm/
I find that this section,
"Configure cgroup driver used by kubelet on Master Node"
is very poorly explained, can it be expanded and improved to make it clear what to do please, thanks

NAK.
i would say that the cgroup driver part is sufficient.
all the details important to the end user are presented - i.e. the kubelet / docker drivers should match.
this is a UX complication, but at the same time we are not going to expand on things like what a cgroup driver really is.
/close


Insert images from anywhere

Instead of having to save images in the Hugo images folder, it would be great to save them anywhere to insert into Bear and then have Bhugo copy the image into the Hugo directory.
Probably will require an additional query to determine image location.

Hi Zach!  Wanted to thank you for this library - it's made writing my blog way more enjoyable.  Seems like you aren't really updating the library anymore, but if you had plans to - this feature would be at the top of my list for sure!

Also happy to try to take a crack at a PR if you had some context or ideas on the best way to do this.

Hey @michaellee1! Thanks, I'm glad you are finding this library useful.
If you want to take a crack at implementing this, I'm happy to help review. It's been awhile since I've looked at this, but I think to implement this, I think we'd need to do a couple of things. The text that Bear saves for an image looks like this:
[image:7BD34BA7-1D41-4634-B42B-0C6D20B88E33-34561-0000B3447A4CA4D0/img.jpg]

Where that ID I believe is some internal Bear ID for the image, and img.jpg is the name of the image. I think what you would need to do is in the function that parses image lines (parseImages) you would need to query the Bear DB (it's a SQLite DB, it's what I'm polling for changes to notes) and retrieve the directory of the image. Then you would need to read and copy that file from wherever it's saved to the Hugo image directory with original name of the image. There is already an environment variable for specifying the Hugo image directory in the config.
I think that should be enough to get started, happy to work through it more on a PR if you would like to take a crack at it!


chore: Add provenance field to My Collection artworks

The type of this PR is: Enhancement

This PR resolves CX-602
Description
I paired with @pepopowitz on this.

Created a generalized TextArea based on the one in InquiryModal
Added an input for Provenance to My Collection artwork additional details

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

There has been a lot of churn in these files, and I'm having some trouble rebasing. I'm going to close this and make a new branch and PR.


Drive API v3: File.iconLink returns null

Some metadata returns a value, for example, File.name correctly returns the name of the File on Google Drive. However, File.iconLink returns null. Is this a known issue, and is there a workaround?

we just forward on the data we get. Unless you have a clear notion that this is a bug in our implementation, I'm going to close.
I'd start here https://developers.google.com/drive/api/v3/about-sdk if you have further questions


chore(graphql): Update ts-morph to v19.0.0

package.json in the root folder depends on ts-morph but the one in query-graphql package depends on the older version. This causes dependency resolution conflict with the latest @nestjs/graphql (v12.0.8)

Thanks!


list-registers error

When running the command "list-registers", there is an error:
sort: Wrong type argument: number-or-marker-p, :sanityinc/split-window
Could you look into it?

After reclone the repo, there is no such issue any more.


Add WatchAddress endpoint to RPC

Thanks @roeierez I've pushed commits using NotifyReceived but please hold off reviewing until I test it a bit.


Update org query to support 2GP

In 2GP, 1 DevHubOrg could have many packages.
This change fixes the org query so that fetch org only pulls subscriber org for a registered package and not all the packages that exist in devhub org(for example - packages that are not being managed by package manager instance).

@michaelhoefer Awesome! Could you please merge it to master if you think it is ready? I am not authorized to merge it.

Merging into intermediate branch first, to remove one change that isn't necessary.


Service Connection instead of PAT?

It's awesome to see there are cmlets out there to automate this. Is it possible to use a service connection instead of a PAT token for automating the process using PowerShell cmdlet? If so would we just pass the service principal password/key as the token? We really don't want to have all of our sync jobs tied to a personal account.

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

ID: 83c90e64-b615-711f-a53d-fc76606e2ecd
Version Independent ID: 2d164036-6886-4440-50f7-369f99f41cea
Content: Source Control integration in Azure Automation
Content Source: articles/automation/source-control-integration.md
Service: automation
GitHub Login: @georgewallace
Microsoft Alias: gwallace

@jzuchora87 Thanks for the Comment. We are actively investigating and will get back to you soon.

@jzuchora87 Thanks for reaching out.  At this time the cmdlets only can use a PAT.

We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


man page completion

I saw you had this as a todo and managed to make it work, so I thought I'd share (there's probably a cleaner way of doing this, I just mashed your code and an example from the fzf wiki until it worked).
man -k . | fzf --preview "echo {} | awk '{print $1}' | xargs -r man" | awk '{print $1}' | xargs -r man

I've used {} instead of {q} in the preview to show what's under the cursor rather that what's typed. This is more useful to me personally.
I came across your post on HN this morning, had never seen fzf's preview option before - thanks for posting it

I have a similar mans function but I'm not sure what I have does the same thing.
I'm not at a computer right now to test but here's mine for comparison.
https://github.com/rothgar/dotfiles/blob/master/zdotdir/.zshrc#L330
FWIW I found this in someone else's dotfiles but I don't remember where

@rothgar thank you, I will check it out later and get back to you as soon as I can


sasquatch: 4.3 -> 4.4

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119996 at ccd4be6e run on aarch64-linux 1

5 packages built successfully:

python38Packages.binwalk
python38Packages.binwalk-full
python39Packages.binwalk
python39Packages.binwalk-full
sasquatch

Hey @SuperSandro2000,
Question about the lz4 ? null becoming lz4 (and the related assert... removal):

Why? What's the point of including lz4 if lz4Support is False?
Was that superfluous before? Why?

Thanks :slightly_smiling_face:

lz4 ? null is unnecessary because, if it's unused, it won't be included in the output / closure. Nix is lazy, so any unused variables (packages) won't be evaluated / built / etc.

@Pamplemousse Do you perchance have any test cases for this? I want to make sure that I didn't break unsquashing LZMA / etc when I went the easy route of just incrementing the #defines (https://github.com/devttys0/sasquatch/issues/33#issuecomment-821858800).

@cole-h Unfortunately no. I haven't used this since I packaged it, and it was a "one-off" need...


Site flashes gray in Chrome when toggling between tabs after PPA tab is inactive for a certain period of time

After some investigation, this issue seems related to this
Sadly, it was just reported three days ago and is undergoing triage. My guess is that the JS draw is getting called on tabbing over, which shouldn't be happening, but it's causing a gray frame while everything is being redrawn. (The Chrome profiler actually locks-ups when trying to observe what might be going on. Firefox is showing a pretty long JS spike during loading.)
I'd defer to @thatbudakguy as to whether we can effectively do anything about that but this is probably going to be more rather than less difficult to fix. However, reporting my poking for the benefit of others.

thanks @bwhicks for looking into this. @mnaydan could you keep an eye out next time this happens to see if it also affects your other (non-ppa) open tabs? I think ultimately it's probably something occurring at the level of the rendering engine and so we can't address it, but it would be nice to confirm that we're seeing the bug @bwhicks identified.
going to close this as wontfix since it's likely not actionable.

@bwhicks @thatbudakguy interesting... I have been keeping an eye on it since I noticed it in December and weirdly it only happens with the PPA tab for me. But I understand why it's a wontfix for us.


Displaying wrong location

Hello team,
I am using esri.Geocoding.geocode to create marker using address.
some of address display wrong location on the map, why?
other thing
I copy these code from online for getting user's current location. it works fine for chrome but not for firefox. do i use right codes?
    //Load user's current location
    mymap.locate({ setView: true, maxZoom: 15 });

    mymap.on('locationfound', onLocationFound);
    function onLocationFound(e) {
        console.log(e);
        // e.heading will contain the user's heading (in degrees) if it's available, and if not it will be NaN. This would allow you to point a marker in the same direction the user is pointed.
        L.marker(e.latlng).addTo(mymap).bindPopup('you are here').openPopup();;
    }

some of address display wrong location on the map, why?

"The accuracy of a match returned by a search depends on the amount and quality of information submitted." - (reference)

I copy these code from online for getting user's current location. it works fine for chrome but not for firefox.

sorry, but thats not a problem we can help you with here.

sorry, but thats not a problem we can help you with here. yes please help

sorry i don't get you

that was just a cheeky way of saying that we cannot help with generic questions regarding HTML5 geolocation in Leaflet like the one you asked because they have nothing to do with the code for this plugin.
https://github.com/Esri/esri-leaflet/blob/master/CONTRIBUTING.md#before-filing-an-issue

thank you, any recommendations of plugin?


Update ReadTheDocs/Coveralls badges

ReadTheDocs badge now links to "stable" build, instead of "latest"
Attempt to fix Coveralls badge stuck at old values

Pull Request Test Coverage Report for Build 704

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.725%

Totals

Change from base Build 698:
0.0%

Covered Lines:
4541

Relevant Lines:
4845

💛  - Coveralls


Adding jsl schema

Coverage decreased (-0.9%) to 73.991% when pulling 69d50d5121d37b205d9f954c179b9e7def93ef37 on feat-schema into 39c0a5550c1e7251d6d0e342113582c830dff6a2 on master.

Coverage decreased (-0.9%) to 73.991% when pulling 6ebb9feb4d49eeb764744e628b8ae8a4ce4d70fa on feat-schema into 39c0a5550c1e7251d6d0e342113582c830dff6a2 on master.


win11 无法拖拽文件到窗口，智能通过点击添加按钮添加

EN: Unable to drag files into the sharing window on Windows 11 as you can see in the video @yvbbrjdr
https://user-images.githubusercontent.com/51252864/185775380-e0f68814-6177-4b22-9c01-dbc9f9eb8257.mp4


[BUG] System.DllNotFoundException: libSkiaSharp when using SkiaSharp.Blazor.Views in Blazor WASM in .NET 9 RC2

Description
I created a new WASM project using SkiaSharp.Blazor.Views to utilize SKPath, but it throws an exception, preventing access to the SkiaSharp object.
My project references the following package:
      <PackageReference Include="SkiaSharp.Views.Blazor" Version="2.88.8" />

.NET 8:
https://github.com/user-attachments/assets/8f04ba38-4760-41a5-baf4-68349638dc58
.NET 8 Sample:
SkiaSharpNET8.zip
.NET 9:
https://github.com/user-attachments/assets/190564b7-c46a-4710-8219-d894141041cb
.NET 9 Sample:
SkiaSharpNET9.zip
This issue only occurs in .NET 9; it worked perfectly in .NET 8. Could you please investigate this issue and provide suggestions?
Code
Code Snippet:
Home.razor
@page "/"
@using SkiaSharp;

SkiaSharp NET 9

@code {
    SKPath DrawingPath = new SKPath();
}

.NET9 Sample:
SkiaSharpNET9.zip
Expected Behavior
Ensure the sample runs without exceptions, similar to how it worked in .NET 8.
Actual Behavior
It throws a System.DllNotFoundException: libSkiaSharp exception.

Version of SkiaSharp
2.88.8 (Current)
Last Known Good Version of SkiaSharp
2.88.6 (Deprecated)
IDE / Editor
Visual Studio (Windows)
Platform / Operating System
Windows
Platform / Operating System Version
Windows 11
Devices
No response
Relevant Screenshots

Relevant Log Output
crit: Microsoft.AspNetCore.Components.WebAssembly.Rendering.WebAssemblyRenderer[100]
      Unhandled exception rendering component: libSkiaSharp
System.DllNotFoundException: libSkiaSharp
   at SkiaSharp.SKPath..ctor()
   at SkiaSharpNET9.Pages.Home..ctor() in D:\Net9\SkiaSharpNET9\SkiaSharpNET9\Pages\Home.razor:line 6
   at System.Object.InvokeStub_Home..ctor(Object , Object , Object , Object , Object )
   at System.Reflection.ConstructorInvoker.InvokeImpl(Object arg1, Object arg2, Object arg3, Object arg4)
   at System.Reflection.ConstructorInvoker.Invoke()
   at Microsoft.Extensions.DependencyInjection.ActivatorUtilities.<>c__DisplayClass17_0.<CreateFactoryReflection>b__0(IServiceProvider serviceProvider, Object[] arguments)
   at Microsoft.AspNetCore.Components.DefaultComponentActivator.CreateInstance(Type componentType)
   at Microsoft.AspNetCore.Components.ComponentFactory.InstantiateComponent(IServiceProvider serviceProvider, Type componentType, IComponentRenderMode callerSpecifiedRenderMode, Nullable`1 parentComponentId)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.InstantiateChildComponentOnFrame(RenderTreeFrame[] frames, Int32 frameIndex, Int32 parentComponentId)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.InitializeNewComponentFrame(DiffContext& diffContext, Int32 frameIndex)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.InitializeNewSubtree(DiffContext& diffContext, Int32 frameIndex)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.InsertNewFrame(DiffContext& diffContext, Int32 newFrameIndex)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.AppendDiffEntriesForRange(DiffContext& diffContext, Int32 oldStartIndex, Int32 oldEndIndexExcl, Int32 newStartIndex, Int32 newEndIndexExcl)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.ComputeDiff(Renderer renderer, RenderBatchBuilder batchBuilder, Int32 componentId, ArrayRange`1 oldTree, ArrayRange`1 newTree)
   at Microsoft.AspNetCore.Components.Rendering.ComponentState.RenderIntoBatch(RenderBatchBuilder batchBuilder, RenderFragment renderFragment, Exception& renderFragmentException)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.RenderInExistingBatch(RenderQueueEntry renderQueueEntry)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.ProcessRenderQueue()
crit: Microsoft.AspNetCore.Components.WebAssembly.Rendering.WebAssemblyRenderer[100]
      Unhandled exception rendering component: Error: No element is currently associated with component 6
Error: No element is currently associated with component 6

Code of Conduct

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

Hi Team,
We have updated to the latest preview of 3.0.0-preview.5.3 in the early access NuGet feed and added the following codes in the .csproj file, which works fine with SkiaSharp access in Blazor WASM:
<PropertyGroup>
    <WasmNativeStrip>true</WasmNativeStrip>
    <WasmBuildNative>true</WasmBuildNative>
</PropertyGroup>

<ItemGroup>
    <NativeFileReference Include="$(SkiaSharpStaticLibraryPath)\3.1.56\st\*.a" />
</ItemGroup>

I have attached a sample for your reference. When can we expect the standard release for these changes on nuget.org? Given that .NET 9 is scheduled for release on November 12, is it possible to have this release on or before that date?
Sample:
SkiaSharpNET9.zip
Thank you.
Best regards,
Gokulprasath


Port Request: freeopcua

https://github.com/FreeOpcUa/freeopcua
Open Source C++ OPC-UA Server and Client Library

@sirexeclp , thank you for posting this issue.
The PR in #5590 has been merged to the master. So I close it.


ImportError: cannot import name 'DependencyWarning'

I'm getting this error when running 'import telepot' on Python 3.5.1 on Arch Linux. It was working not too long ago but now it throws the following:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.5/site-packages/telepot/__init__.py", line 5, in <module>
    import requests
  File "/usr/lib/python3.5/site-packages/requests/__init__.py", line 61, in <module>
    from .packages.urllib3.exceptions import DependencyWarning
ImportError: cannot import name 'DependencyWarning'

I'm running the latest version of Telepot (installed via pip)

Update: This was an issue in requests on Arch. Solved by reinstalling requests.

@NickGeek, but how to do this with pip?

I managed to fix this issue just by upgrading urllib3:
sudo pip install --upgrade urllib3

On Arch I always face problems to reconcile distro's packages with pip's packages.

@alexandre-mbm I did it with the AUR package.

thanks for your replies, fixed an issue regarding layman thanks to you

Solution for me:
$ sudo pip uninstall requests
$ sudo pip install requests
$ sudo pip uninstall docopt
$ sudo pip install docopt

me too.  thanks!


Check parameter consistensy in signal.iirdesign

Ref. https://github.com/scipy/scipy/issues/11567

Reference issue

Ref. #11567
What does this implement/fix?

This aims at raising ValueError from the scipy.signal.iirdesign function rather than from anywhere else (as it is now, in some cases) to provide uniform behaviour and clear explanation of an error. Docs are also updated by a single sentence
Additional information

Everytime I look at this function docstring I always get confused. Probably we should clean this up a bit without the terminology clashes bandstop --> stopband, bandpass --> passband etc. The examples would really benefit from one line explanations.

@ilayn if You take a look at this (which imho can be used as source) than everything makes sense. https://dspguru.com/dsp/reference/filter-terminology/
That is, bandstop != stopband. "Bandstop" is used as an alias for "Bandstop filter" as I guess, and stopband is a range of frequencies. We could probably add these words to documentation, or this reference I've pasted before. What do You think?

Yes I am quite familiar with the terminology but still this can be much clearer without the need to have all these jargon. In the end, what is being done here is locating where the shape starts and ends. The docs should reflect this simple operation with equal simple language. I'll try to have a stab at it today.

@ilayn  I've looked at build issues and for me they seem unrelated with my changes. Could I do something more in here?

@AtsushiSakai  maybe You would review proposed changes? Would be very grateful if so.
@ilayn why not open another issue for documentation being unclear? IMHO proposed changes do solve the problem raised in issue

Looks like a straightforward sanity check, so I'll go ahead and merge. Thanks for iterating on the wording @erheron and reviewing @AtsushiSakai

If the related issue is now fixed, feel free to close it with the 1.6.0 milestone

Sorry for the late reply folks. Thanks for your efforts I got caught up with daily work


Field "struct Groups" not loading on Rcol file

I tried opening the file "GreatSword.rcol.20" and it gives this error: "ERROR Line 5916: Template passed end of file at variable 'CombinedKeys". The fields where you can modify the motion values of attacks seems to load fine, but the "struct Groups" mentioned on the title doesn't want to load. Sorry if this is hard to understand, english is not my first language.

Alright I fixed it. There are two new unknown ints in the rcol.20 file header.

OMG Thank you! Now time for me to go back to modding.


[ADD ME TO THE ORGANIZATION]

Name:  Gaurab Kundu
Location: Kolkata, West Bengal, India
Interest: Data Science, AI, ML and Development
Discord username(optional):- Gaurab01
Why do you want to join?

I want to join because
I would like to contribute and share new ideas with like minded people.
You can also join our Discord community here
Feel free to check out other cool repositories of the Open Source Community here.

@GaurabKundu1 Thanks for your interest. Invitations have been sent to you. Kindly accept it and make your membership public.


Convert clang-tidy scripting to python.

Codecov Report
Base: 28.70% // Head: 28.70% // No change to project coverage :thumbsup:

Coverage data is based on head (21ccf80) compared to base (889abd6).
 Consider uploading reports for the commit 8f469c6 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           stable     #977   +/-   ##
=======================================
  Coverage   28.70%   28.70%
=======================================
  Files          72       72
  Lines        5852     5852
  Branches      306      306
=======================================
  Hits         1680     1680
  Misses       4109     4109
  Partials       63       63

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Not work in -sdk iphoneos

hello, I hava a trouble  in this case, what should i do?
infer  -- xcodebuild -workspace xxx.xcworkspace -scheme xxx  -configuration Debug -sdk iphoneos
Capturing using xcpretty...
Starting translating 0 files
.
Starting analysis...
legend:
"F" analyzing a file
"." analyzing a procedure
Found 0 source files in /Users/sangfor/Documents/xxx/infer-out
No issues found

Maybe you need to clean the build first? If there is nothing to compile, Infer won't analyse anything.

yeah, I forget it , thanks


Macro lifetimes aren't a thing (yet)

The return value of one of my named! parsers has a lifetime parameter. I need to make the function take a lifetime parameter. Does nom currently support this?
My situation:
pub enum Expression<'a> {
  String(&'a str)
  // other stuff...
}

named!(pub string_parser<Expression<'a> >,
  // other stuff...
  );

Cargo gives:

src/expressions.rs:15:22: 15:24 error: use of undeclared lifetime name `'a` [E0261]
src/expressions.rs:15 named!(pub string_parser<&'a str>,
                                                ^~
src/expressions.rs:15:1: 19:5 note: in this expansion of named! (defined in <nom macros>)
src/expressions.rs:15:22: 15:24 help: run `rustc --explain E0261` to see a detailed explanation

The expanded macro definition:

pub fn string_parser(i: &[u8]) -> ::nom::IResult<&[u8], Expression<'a>, u32> {

'a is equal to the lifetime of the data being parsed, if that helps. Rust was able to infer that when it wasn't public, but as public it can't make that assumption.

Turns out this is an issue in rust: rust-lang/rust#27832

Just a few days ago, this issue was fixed in the RFC rust-lang/rfcs#1590. The PR against rustc is coming soon. So it'll be in nightly soonish. At that point, I'll probably do a personal fork of nom to add the nightly feature, unless there's a way for nom to handle nightly.
Closing because this isn't an issue anymore.

you could at least leave me some time to answer before closing ;)
named! is just a helper, you can always write your own function definition. I probably wouldn't add something that only appears in nightly, or at least I would hide it behind a feature flag.


GoMetalinter using bazel

Hi,
I am trying to use gometalinter using bazel build system with reference the example code shared in this repo. Layout of codebase

WORKSPACE
BUILD
 --> uses third party golang github packages
WORKSPACE
load("@com_github_atlassian_bazel_tools//gometalinter:deps.bzl", "gometalinter_dependencies")
gometalinter_dependencies()
BUILD
load("@com_github_atlassian_bazel_tools//gometalinter:def.bzl", "gometalinter")
gometalinter(
name = "gometalinter",
config = "//:gometalinter.json",
paths = [
"/...",
],
prefix = "<repo_path>",
)
On executing "bazel run //:gometalinter", bazel is having issues importing third party github repos internally used by code in dir1 folder.
Can you please let me know any solution for this?
The same is easily achievable using deps for go_binary/cc_binary, etc
Thanks
Kailash

Hey, can you share the exact full WORKSPACE and BUILD files and also the exact full error message that bazel is giving you? I cannot help with just the information you have provided, sorry.

Double checking: which version of bazel are you running?

Hi,
I am using 0.16.0 version of bazel. Please find below output of version command.
bazel version
Build label: 0.16.0
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Tue Jul 31 17:01:24 2018<PHONE_NUMBER>)
Thanks
Kailash

Can you paste the contents of those files rather than an image? (so I can attempt to reproduce the issue)

BUILD.txt
WORKSPACE.txt
Hi,
I have uploaded the file in text format. Steps to reproduce:
go get -u github.com/stretchr/testify
cd $GOPATH/src/github.com/stretchr/testify
copy BUILD and WORKSPACE to root of testify repo.
execute gometalinter rule using bazel
Thanks
Kailash

Is there a typo in the attached files?
bazel run //:gometalinter
ERROR: error loading package '': Encountered error while reading extension file 'go/def.bzl': no such package '@io_bazel_rules_go//go': Error downloading [https://github.com/bazelbuild/rules_go/releases/download/0.15.1/rules_go-0.15.1.tar.gz] to /home/amckague/.cache/bazel/_bazel_amckague/d1309ac1e2719cdf777c0d6e936fc92b/external/io_bazel_rules_go/rules_go-0.15.1.tar.gz: Checksum was 5f3b0304cdf0c505ec9e5b3c4fc4a87b5ca21b13d8ecc780c97df3d1809b9ce6 but wanted 5756a4ad75b3703eb68249d50e23f5d64eaf1593e886b9aa931aa6e938c4e301
ERROR: error loading package '': Encountered error while reading extension file 'go/def.bzl': no such package '@io_bazel_rules_go//go': Error downloading [https://github.com/bazelbuild/rules_go/releases/download/0.15.1/rules_go-0.15.1.tar.gz] to /home/amckague/.cache/bazel/_bazel_amckague/d1309ac1e2719cdf777c0d6e936fc92b/external/io_bazel_rules_go/rules_go-0.15.1.tar.gz: Checksum was 5f3b0304cdf0c505ec9e5b3c4fc4a87b5ca21b13d8ecc780c97df3d1809b9ce6 but wanted 5756a4ad75b3703eb68249d50e23f5d64eaf1593e886b9aa931aa6e938c4e301
INFO: Elapsed time: 5.107s
INFO: 0 processes.

Removing references to the this lib and it's rules doesn't help the above error.

Another issue that I noticed is that go_rules_dependencies() and go_register_toolchains() are invoked twice. The second pair of calls can be removed.

Hi,
The same WORKSPACE file is working fine for me.
In your case, line 4 in WORKSPACE file having sha256 can be replaced by 5f3b0304cdf0c505ec9e5b3c4fc4a87b5ca21b13d8ecc780c97df3d1809b9ce6.
This will ensure checksum validation completes.
I agree second pair is redundant and can be removed.
Also, please try bazel run command keeping vendor directory and then without any vendor directory in the git repo.
Thanks
Kailash

Does bazel clean --expunge change the error for you?
The attached WORKSPACE file below works for me. metalinter runs and reports various linting issues (including linting issues in /vendor). I suggest adding config = "//:.gometalinter.json", to the gometalinter configuration in BUILD.bazel and use it to configure linting to ignore the vendor folder altogether.
WORKSPACE.txt

Also see https://github.com/atlassian/smith/blob/master/.gometalinter.json and https://github.com/atlassian/smith/blob/master/BUILD.bazel#L68-L79 for inspiration =)


[Dev] Improve merge script when milestone is already assigned

Describe the enhancement requested
When an issue already has a milestone assigned the merge script ignores this and just shows the normal default milestone (e.g. next version). We should create more visibility there to avoid overriding existing milestones.
Component(s)
Developer Tools

cc @raulcd

Duplicate of #36656


Add error handling for payu checkout with non-writable laboratory

This PR:

In payu checkout - initialise laboratory directories before generating any metadata. This will fail with PermissionError if directories are non-writable, and prints some information on how to recover
Fix a small error with changing archive symlinks when a previous directory has been deleted - added test
Fix a uncaught warning when running payu checkout pytests

I was running with the idea of checking out a git branch - running checks on that branch and then if they failed, reverting the checkout back to the previous branch (and delete any new branches that were created). Though I then finally realised that it would make it difficult for the user to apply fixes on the branch/tag/commit they were starting from, e.g. fix project/shortpath/laboratory in config.yaml. It would require adding command-line args to payu clone/checkout that will require modifications to config.yaml.
There is the pre-existing --laboratory command-line flag to payu clone/checkout. It doesn't modify the config.yaml so will either need to be passed to subsequent payu commands or shortpath/laboratory/project options will need to modified manually in config.yaml.
I initialised laboratory directories (using mkdir_p) rather just checking for write access using os.access, so it wouldn't fail when a laboratory base directory does not exist but it can be created by a user. I added laboratory specific error messaging to Laboratory class method, and also call laboratory.initialize() in Experiment() as when running payu setup/sweep/run it would be good to know first thing if the laboratory directory is even writable.
Closes #524

Hello @jo-basevi! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file test/test_branch.py:

Line 533:1: W293 blank line contains whitespace

coverage: 58.517% (+0.2%) from 58.29%
when pulling 522d86a2b39cb4ccdc2134f9ccae39aa435941b8 on ACCESS-NRI:524-Checkout-project-error-bug
into 1186c46e47866468f4e56ed74a578d83ffcee0e0 on payu-org:master.


Added support for FUNCTION DUMP, FUNCTION FLUSH, FUNCTION RESTORE commands

Closes #954
Closes #955
Closes #960

coverage: 77.637% (+0.04%) from 77.6% when pulling a6ddc1393a33612107c7b6dcbbaac7d88be0b770 on vladvildanov:vv-functions-dump-flush-restore-commands into 90da582efc88d5168fa126f1c3527733d81690eb on predis:v2.x.


Basic template app crashed on URI malformed error

Reporting a bug?
Hi, I encountered the below issue while testing the basic template with URL http://localhost:3000/?query=%%uri_malformed%%
An internal error occured!

URIError: URI malformed
    at decodeURIComponent (<anonymous>)
    at RoutesInner (http://localhost:3000/main.js:68271:17)
    at renderWithHooks (http://localhost:3000/main.js:35496:22)
    at mountIndeterminateComponent (http://localhost:3000/main.js:38108:17)
    at beginWork (http://localhost:3000/main.js:39188:20)
    at HTMLUnknownElement.callCallback (http://localhost:3000/main.js:20888:18)
    at Object.invokeGuardedCallbackDev (http://localhost:3000/main.js:20937:20)
    at invokeGuardedCallback (http://localhost:3000/main.js:20990:35)
    at beginWork$1 (http://localhost:3000/main.js:43754:11)
    at performUnitOfWork (http://localhost:3000/main.js:42718:16)

Environment

  System:
    OS: Linux 4.15 Ubuntu 18.04.4 LTS (Bionic Beaver)
    CPU: (8) x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
    Memory: 6.19 GB / 15.11 GB
    Container: Yes
    Shell: 4.4.20 - /bin/bash
  Binaries:
    Node: 14.3.0 - /usr/bin/node
    npm: 6.14.5 - /usr/bin/npm
  Browsers:
    Chrome: 83.0.4103.61
    Firefox: 77.0.1
  npmPackages:
    react: ^16.9.0 => 16.13.1
    react-dom: ^16.9.0 => 16.13.1
    react-static: ^7.2.0 => 7.4.0
    react-static-plugin-reach-router: ^7.2.0 => 7.4.0
    react-static-plugin-sitemap: ^7.2.0 => 7.4.0
    react-static-plugin-source-filesystem: ^7.2.0 => 7.4.0

Steps to Reproduce the problem

Run react-static create basic
Open chrome browser and enter http://localhost:3000/?query=%%uri_malformed%%

Expected Behavior
Would not crash and redirect to the root/default route?

Personally, I don't think we should fix this, because malformed URIs will result in malformed / unpredicatable application state, BUT, we'll take a PR that captures the issue.
You only need to try/catch in RoutesInner.

try/catch in RoutesInner sounds good enough. However, i don't think it can be done via configuration or plugins. I couldn't find an example in the docs

Yeh, we would need to change the core code (packages/react-static)


CTriggerManager and superblock changes

Implemented new CTriggerManager superblock handling
Fixed several JSON parsing issues
Added CSuperblock initializer list to avoid uninitialized data
Additional debugging output

Notes:

Temporarily set required votes to 1 for testing
CSuperblock::GetBlockStart just returns the event_block_height from sentinel

Code seems to be working at least through calls to GetBestSuperblock after voting up the superblock.

Perfect thanks


remote development git change tracking

VSCode Version: 1.35 insiders (23.5.19)
OS Version: Windows 10 1903
Remote Server: Ubuntu Server 18.04

Steps to Reproduce:

cloning a git repo on a remote development ssh server
changing git tracked files
git side bar doesn't show change files
clicking the refresh button we see the changed files

expectation:
vs code should show the tracked changes without manual refreshing

https://github.com/microsoft/vscode-remote-release/issues/314


weston-conf: use fbdev-backend for qemuarm{64}

This should go to the poky upstream in the future.

thanks


Intermittent No alive nodes found in your cluster error

Versions

scout-elastic-driver:  3.10.0
laravel/framwork: 5.7.28
php: 7.2
elasticsearch: 6.7.0

Description

I am trying to setup docker for an already existing project. However, when I attempt a scout import command, "No alive nodes found in your cluster" is being thrown from the StaticNoPingConnectionPool class. However, this does not occur for every model.
Excluding php artisan scout:import "App\Models\ParentMontessoriCard", every import will fail.
The strange thing is, if I got to http://localhost:9200/_cluster/health, it looks fine:
{
  "cluster_name": "docker-cluster",
  "status": "green",
  "timed_out": false,
  "number_of_nodes": 1,
  "number_of_data_nodes": 1,
  "active_primary_shards": 0,
  "active_shards": 0,
  "relocating_shards": 0,
  "initializing_shards": 0,
  "unassigned_shards": 0,
  "delayed_unassigned_shards": 0,
  "number_of_pending_tasks": 0,
  "number_of_in_flight_fetch": 0,
  "task_max_waiting_in_queue_millis": 0,
  "active_shards_percent_as_number": 100.0
}

Steps to reproduce

Using my dockerfile hereMy docker-compose.yml file is here. Once the containers are running, I'll run the scout import command for each of the models, like docker-compose exec php php artisan scout:import "App\Models\Blog".

Any errors logged on your elastic search container?

@iget-esoares Do you mean the logging from the daemon or specific logs within the container?

Found the problem. I didn't realise we had two drivers installed and it wasn't this one that was actually being used.
To conclude, my issue is nothing to do with this library.
@iget-master Thanks for your quick response, though.

Thanks @JayyWalker

How are you found that ?
I meet same question :((

@NominationP that's usually an error on your elasticsearch cluster. Check the logs on your elasticsearch for any errors. Also check the elasticsearch host and port.


How to Join Tables from Different Databases in SQL Server

Model: ModJobsEntity in DBConnection
Model: HRDynamicCategoryEntity in DBConnectionHR (link server)

u can try use sql server linkserver link db2 to db1.
and set HRxxxEntity tableName as full link name

u can try use sql server linkserver link db2 to db1. and set HRxxxEntity tableName as full link name

Is it like this?

perhaps, you can try it yourself, i donot have env to test that.
debug print sqlvalue or aop to print generate sql.

//多表跨库
var list1 = db.Queryable<Order>().AS("xx.dbo.order") // AS("")
        .LeftJoin<OrderItem>((o,i)=> o.Id == i.OrderId).AS<OrderItem>("yy.dbo.OrderItem") //AS<T>
        .LeftJoin<Custom>((o, i, c) => c.Id == o.CustomId) .AS<Custom>("zz.dbo.Custom")//AS<T>
        .Select((o,i,c)=>new classA() { oid=o.Id , iname=i.Name })
        .ToList();

The function you said will be developed, at present, can only be implemented in the way I mentioned above

The function you said will be developed, at present, can only be implemented in the way I mentioned above

//多表跨库
var list1 = db.Queryable<Order>().AS("xx.dbo.order") // AS("")
        .LeftJoin<OrderItem>((o,i)=> o.Id == i.OrderId).AS<OrderItem>("yy.dbo.OrderItem") //AS<T>
        .LeftJoin<Custom>((o, i, c) => c.Id == o.CustomId) .AS<Custom>("zz.dbo.Custom")//AS<T>
        .Select((o,i,c)=>new classA() { oid=o.Id , iname=i.Name })
        .ToList();

Great ^^

perhaps, you can try it yourself, i donot have env to test that. debug print sqlvalue or aop to print generate sql.

Yes, Thanks u ^^
Is running 🥉


Videos not playing on 0.84.2

Hi there
Since the last FW update, the video player is no longer working.

Software starts
Video selection shows videos correctly
Video does not play, just shows a blank screen

Hello. I have updated source code in the main repo today to make the app work on new firmware version

Thank you for the update, can confirm it is working again


Highlight tests that didn't pass all checks

In the test output (under the Test name heading), it would be nice to highlight in bold or some other way, the tests that didn't give full score... (not urgent)

In the redesigned frontend this has been implemented, as shown in the image below:

Tests that give full score are highlighted in green, while failed tests are highlighted in red.

Excellent. Closing as already fixed.


drivers: wireless: Fix mtu info for gs2200m

Summary

I noticed that mtu info with ifconfig shows incorrect size.
This commit fixes this issue.

Impact

None

Testing

Tested with spresense:wifi_smp

@masayuki2009 Seems that you submit wrong place ? This repo is forked from https://github.com/apache/nuttx for my local development.

@no1wudi
Sorry for the mistake.


more examples

Codecov Report

Merging #2 into master will increase coverage by 0.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   95.42%   95.61%   +0.19%
==========================================
  Files           8        8
  Lines         525      525
==========================================
+ Hits          501      502       +1
+ Misses         24       23       -1

Impacted Files
Coverage Δ

lib/src/text.dart
100.00% <0.00%> (+3.84%)
:arrow_up:


Thoughts on how to implement radio buttons

I'm struggling to work my head around using bindInput with radio buttons
Any thoughts?

can you add an example in codesandbox.io and describe the issue and how to replicate it?

This approach seems to work for me: https://codesandbox.io/s/ol3x26jqq5
import React from 'react';
import { render } from 'react-dom';
import { connectForm } from 'form-container';

const styles = {
  fontFamily: 'sans-serif',
  textAlign: 'center',
};

const RadioGroup = ({
  name,
  value,
  onChange,
}) => (

    <input type="radio" name={name} value="male" checked={value === "male"} onChange={onChange}/> Male
    <input type="radio" name={name} value="female" checked={value === "female"} onChange={onChange} /> Female
    <input type="radio" name={name} value="other" checked={value === "other"} onChange={onChange} /> Other

)

const App = ({ formMethods: { bindInput }, form }) => {
  console.log(form.model);
  return (

      <RadioGroup
        {...bindInput('gender')}
      />

  );
}

const ConnectedApp = connectForm()(App);

render(<ConnectedApp />, document.getElementById('root'));

What issue are you seeing @jonathanalbangoodman ?

Have spoken with @jonathanalbangoodman and he's happy with the approach.

he can star the repo then ;)


License

I think it's better to use the AGPLv3 than the ApacheV2.
Check here for more details

No i really want to to distribute the code under the apache v2 license


Solution File

Hello,
Hope you're doing well.
I just wanted to add if you could add a solution.zip file to the repository as well as release.
Regards

Zip file is there, however, it is unusable.  Corrupt file, perhaps.

Hey Guys I checked the zip it's working perfectly when I clone the repo and import the zip in the power app(Canvas).
path of the zip is "Solutions/bin/Debug/Solutions(zip)".

However, I will share how to create a zip file on your own. may be in a couple of days Thx.

The solution will not import.  I will wait for your instructions.  Thanks.

see the article https://nebulaaitsolutions.com/blog/web-resource-content-size-is-too-big-in-pcf-control for importing the large size custom control.

Imported.


Correctly implemented numeric sort for ArrayColumns and DictColumns

As suggested in #580 this replaces the lexicographical sorting implemented on ArrayColumns and DictColumns with more useful general sorting as is already in place on the DFColumns and NdColumns interfaces. In the case of ArrayColumns we can simply create a record array view into the actual array, which we can apply the sorting to. For DictColumns we need to actually construct a record array but this shouldn't be any more expensive than the existing implementation, which already creates an intermediate array to compute the sort order on.
This should be ready to merge once the tests pass.

I've made a comment but now I'm wondering what was wrong with the originally intended (lexicographical) sorting behavior...

In Lancet, I use lexical sorting without record arrays. The default sort in Python is stable so cycles of sorting do the job. Here is the relevant code in Lancet.
Not saying it is an efficient approach but it is simple, works for heterogenous types and doesn't use record arrays.

Okay, so the problem wasn't lexicographical sorting but the fact that it only works on homogeneous arrays. I think the record array approach is still the way to go but I'm happy to make the recarray method into a function that specifically computes the lexsorted indices.

Ok, that sounds reasonable. I would make it a utility function and maybe call it lexsorted_indices?
After that, I'm happy to merge once the tests pass.


Gatsby page always redirects to landing

Description
Whenever I send a link to someone, the Amplify app redirects to the landing page. I'm not sure how to handle the redirects (this is my first Amplify app). Here is what I have so far.

How should I change this so that I can send someone https://www.tylernorlund.com/resume and when they click it, it goes to the resume page?
Repro
Repository: https://github.com/tnorlund/Blog
Amplify link: https://www.tylernorlund.com/

It looks like you were able to figure this out since the link is now working as far as I can tell.
If you are still experiencing issues please re-open this ticket and provide your appId and region (appId is part of your app ARN in the General tab under App Settings

@tnorlund Do you have a solution to this available? I am currently having the same issue.

@zoosissoos I ended up committing my code again, and it fixed whatever it was. Make sure that your pages are uniquely identified in your src/pages and that the slug is declared in Gatsby-node.js.

Ok thanks. Would it be possible to copy/paste your redirects config in the amplify console?
Whenever I click a link within my deployed gatsby site, it will go to the correct page, but if I enter within the browser bar it goes to the root index.
[
    {
        "source": "/<*>",
        "target": "/index.html",
        "status": "404-200",
        "condition": null
    },
    {
        "source": "</^[^.]+$|\\.(?!(css|gif|ico|jpg|js|png|txt|svg|woff|ttf|map|json)$)([^.]+$)/>",
        "target": "/index.html",
        "status": "200",
        "condition": null
    }
]

I'm not using redirects.


Sorting Now Deletes Lines (ST2)

Possibly due to a recent update (?) but it seems that currently sorting by any method will remove all lines.

Hello,
I need more information.
What OS are you using?
Which version is installed (Package Control: List Package, and find SortBy)
Thanks!

I think I just found the issue, you rare probably using a version not designed for ST2 (> 1.2.0).
If it's the case, I made a mistake when setting the version on Package Control. :(

Hey @Doi9t,
I am currently using v1.4.1 so that would make sense. I hadn't changed anything locally that would affect the package and the I saw that there were some recent commits + Sublime packages auto updating. Thank you for taking a look!

I will fix the issue in Package Control, this can take more than one day to take effect; I highly suggest that you uninstall the plugin for the moment, and I will comment here when the change is applied (so you can reinstall).
You can also manually install the 1.2.0 if you want (https://github.com/Doi9t/SortBy/archive/refs/tags/st2-1.2.0.zip), if you need to use the plugin.
With the help of the documentation: https://github.com/Doi9t/SortBy#manual-installation

Will do, thanks @Doi9t!

Hello,
FYI, the PR is merged on Package Control; this issue should be fixed.
You should be able to reinstall and have the 1.2.0 as soon Package Control is deployed / updated correctly.


React-IOS app CpasS SDK

React-IOS app CpasS SDK
Module added -

Login

SMS

Please review again.


Error Importing Snowflake Account

Provider Version
0.66.2 (latest)
Terraform Version
1.39.0
Terragrunt Version
0.45.3
Describe the bug
We believe we’re running into a bug when importing Snowflake Accounts using the resource snowflake_account. We’re currently using the latest version (0.66.2).
Whenever, we perform an import,  we are running into a permission issues.  Form our end, we can confirm it’s a TF provider issue for the following reasons:

We used the same query the provider is running when doing an import, which is SHOW ORGANIZATION ACCOUNTS LIKE ''. On the Snowflake console, the query runs without any issues. It only fails when we use TF for the import.
We are able to create new accounts using the resource snowflake_account with the exact same permissions.
We are using the same permissions when creating as we are when importing.

Whenever we perform an import, this is the error message we get:
Error reading account: 003001 (42501): SQL access control error: │ Insufficient privileges to operate on 'SYSTEM'

It almost seems like the import is needing more elevated permissions compared to creating new accounts, or the import query is incorrect. Your help to resolving this issue would be greatly appreciated.
Expected behavior
We expect the import to work correctly when using the ORGADMIN role.
Code samples and commands
terraform import 'snowflake_account.snowflake_account["<ACCOUNT NAME>"]' <ACCOUNT LOCATOR ID>
Additional context

1 on the above-mentioed issue

The bug seems to be resolved when we upgraded to the latest release that just came out 20 hours ago (version 0.67.0)


Question about RGBDiff

Hi, I have a question about RGBDiff in your code. What does that mean? Is that RGB+Flow?
Thanks a lot!

RGB Diff means the difference of RBG values between two consecutive frames. You can get it by simply subtracting the RGB values of frame t and frame t+1.


[Feature] Upgrade target K8s cluster version to 1.20

Is your feature request related to a problem? Please describe.
Since the majority community users are using K8s v1.20+ now, we should upgrade our target K8s  cluster version to 1.20+
Describe the solution you'd like

upgrade our CI settings and test cases if related
upgrade our definition template
upgrade our dependency lib

Describe alternatives you've considered

Additional context

Kubernetes Version skew policy  https://kubernetes.io/releases/version-skew-policy/


README: A CSS required for keyboard navigation

Mention that geosuggest-item--active is necessary to make keyboard navigation in the suggestion list "work" visually

That's great!


Allow specifying terragrunt-iam-role in terraform.tfvars

Would it be possible to add in the ability to specify the terragrunt-iam-role in the terraform.tfvars as well as through the env var and the parameter?

That seems reasonable enough. Are you up for a quick PR to add it?

Sure thing


add unregister to client

so that we can continue to use this nice library in IT tests for unregister: https://jira.corp.adobe.com/browse/NUI-447
also so our asset compute client is more complete

@tmathern, it is confusing that there are 2 implicit methods: initialize and register.  Also, it is fragile that the state of registration is determined by a variable registered, would could easily be changed by the client on accident.
const assetComputeClient = new AssetComputeClient(integration, options);
assetComputeClient.registered = true; // trick implementation to think register is true
// process renditions
await assetComputeClient.process(...); /

Maybe, I should make assetComputeClient.registered  => assetComputeClient._registered  since registered is too close in name to register

@jdelbick Agreed, the change is nice :). But nothing is ever truly private in JS, so... Nothing prevents anyone to access any other variable. And this class is stateful now. So we'll have to prevent moving to "wrong state" (actually, we could turn this into a state machine).


refactor: remove mount logic in harvester

Solution:
LH introduces VolumeAttachment CRD after v1.5. It can automatically attach/detach volume when there is a request from LH API / controller / CSI. We can just leverage LH mechanism and remove our implementation.
Related Issue:
https://github.com/harvester/harvester/issues/4907
Test plan:
Setup

Create a 3-node harvester from this branch.
Setup backup-target.

Case 1: snapshot can work on a stopped VM.

Create a VM.
After the VM is ready, stop the VM.
Check VM volumes are detached.
Take a snapshot on the VM. The snapshot can be ready.

Case 2: restore a snapshot from detached volumes can work.

Follow Case 1.
Make sure VM volumes are detached.
Restore the snapshot to a new VM. The new VM can be ready.
Restore the snapshot to replace the old VM. The old VM can be ready.

Case 3: backup can work on a stopped VM.

Create a VM.
After the VM is ready, stop the VM.
Check VM volumes are detached.
Take a backup on the VM. The backup can be ready.

Case 4: race condition doesn't break VMBackup.

Create a VM.
After the VM is ready, stop the VM.
Check VM volumes are detached.
Take multiple backup on the VM in a short time. All backup can be ready.

Case 5: restore a backup from detached volumes can work.

Follow Case 3.
Make sure VM volumes are detached.
Restore the backup to a new VM. The new VM can be ready.
Restore the backup to replace the old VM. The old VM can be ready.

@mergify backport v1.3

@mergify backport v1.2


Stop a running acquisition in sequencing mode (acq.abort() not functional)

An Aqcuisition in sequencing mode cannot be stopped. This would be very helpful for time series or longer acquisitions.
It is not possible to use acq.abort() in sequencing mode and the events cannot be thrown.
The work aorund at the moment is to restart uManager and pycromanager. This is very inconvenient, especially because during restart the hardware is set to default values, which might damage the sample.

I just tried it, but it still freezes microManager

attempt at fixing this in https://github.com/micro-manager/AcqEngJ/pull/77
Now going to write some testing code to ensure it works

This now works on the demo configuration and passes tests for a timelapse and a zstack. Let me know if you run into any issues


[Feature Request] Set as now playing in Android

If the media is set as now playing in android device, the poster of movie will be on Android Lock Screen and can be directly paused/played from the Lock screen.

Enable notifications in settings. Will get you half way there.

Yes half the way, but not the whole…


rescuetime: <IP_ADDRESS> -> <IP_ADDRESS>

Motivation for this change
I am trying to upgrade NixOS but I can't because rescuetime is broken. See https://github.com/NixOS/nixpkgs/issues/87226.
$ sudo nixos-rebuild switch --upgrade

The shas were broken for <IP_ADDRESS> because the urls where pointing directly at the current version of rescuetime. e.g.
https://www.rescuetime.com/installers/rescuetime_current_amd64.deb
So, I've upgraded to the latest version, fixed the urls, and changed the sha256s.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] ~macOS~
[ ] other Linux distributions

[ ] ~Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)~
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/). NOTE: only able to test amd64 platform.
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] ~Ensured that relevant documentation is up to date~
[ ] Fits CONTRIBUTING.md.

rescuetime was already updated in #93770. But this seems to be a newer version, perhaps you could rebase this PR on master?

@danieldk I sure can. I did intentionally want to 20.03 as it's broken and I can't upgrade. I'll rebase it on master, then how can we ensure that it is cherry-picked into nixos-20.03?


prevent meta property og:image from triggering on a crawl

for my dotenvx.com/docs site i use urlbox.io to generate screenshots when the pages are sent over social media. this is under the meta attribute:
<meta property="og:image" content="">

the problem is when i crawl my sitemap it triggers tons of api requests to urlbox (costing me money) because llmstxt makes a request like a normal user rather than a bot or spider or crawler.
@devin-ai-integration can you fix llmstext here so it doesn't do that - it doesn't treat the request like a real user. instead it somehow treats the request in a similar way to google crawling your site?

@devin-ai-integration can you work on this ^


Android shows just black space

Hello,
we use this plugin to show ads. With ios it works, but on android there are problems.
If we set in the config isTesting = true, the testbanner is shown. But if we change to isTesting = false, there is just a black space, instead of the banner
With isTesting = false

With isTesting = true

Our config:
window.plugins.AdMob.setOptions( {
publisherId: admobid.banner,
interstitialAdId: admobid.interstitial,
adSize: window.plugins.AdMob.AD_SIZE.SMART_BANNER,
bannerAtTop: false,
overlap: false,
offsetTopBar: false,
isTesting: true,
autoShow: false
});
call the banner:
window.plugins.AdMob.createBannerView();

This can happen if the ad ID is still new. Admob doesnt serve ads sometimes for 24 hours. Please let us know if it still happens after that time.

No this doesn`t work.
If I add this event listener document.addEventListener('onFailedToReceiveAd', function(data){});
there is this internal error.

That description isn`t helpfull.

If the test ads work, then this is a problem from the admob end. Admob does not 'trust' your app for some reason.
Can you tell us the app id?

If it's on a real device, is the apk signed?

This happens after install the app on a real device with "phonegap run android".
What is the role of the AppId?

Is the app signed?
The appid might be already used by someone else. If you tell me I can check.

This helps, thank you. You have to build a signed app. It doens`t work with "phonegap run android".
It would be nice, if there is a error message, with this information.

You're welcome, glad to hear it.


How to use SHAP for large dataset?

Hi @slundberg
If I have to use SHAP for large data( say 1 million and above records). How can I use it as its taking a lot of time to compute for small data with 30000 records.
I came across shap.sample(data,100) which takes a sample of data. Doesn't this account for variance and bias in data?
Please help me with this.

It depends on which explainer you are using. For TreeExplainer you could switch to tree_path_dependent or even the approximate mode to get more speed. For any explainer you can parallelize by running on different sets of samples. If you just want a global view of 1 million records then I would use shap.sample(X, 10000) and explain those samples for all the plots you might need.

@slundberg
Hi, Thanks for the quick reply.
When we select 10000 records out of a million records wont that cause bias in data. what if i wanted a global interpretation of all million records without sampling and without taking more time.

^ I think @vaishkiva asked a really important question. Looking forward to your answer @slundberg

SHAP estimation is embarrassingly parallel, so moving TreeExplainer to GPU can be the answer you are looking for.
In fact it has been achieved recently by good folks at RAPIDS, who report some very good results (an order of magnitude speed-ups can be observed even for moderately sized 10k datasets, so unlike for the boosted trees algos themselves, in case of SHAP values, GPUs do not require Higgs-scale 10M datasets to outperform CPUs). See their paper at arXiv. And to see that these speed-ups (and a reasonable GPU memory footprint) hold for big data sets as well, see NVIDIA's recent blog post, where SHAP values estimation by a GPU-enabled version of shap_values() was speeded up 22 times on 11M rows of data.
Given that RAPIDS projects are incubated by NVIDIA, they take great care about portability, so their GPU implementation (see https://github.com/rapidsai/gputreeshap) has a form of an open sourced C++ library (a single CUDA header file).
The usage example they provide here demonstrating how to call their main GPUTreeShap function keeps the entire simplified modeling pipeline in the GPU. The question is how they used XGBoost - did they include XGBoost CUDA code or invoked their CUDA library from python using bridges like pycuda? Hopefully, we shall get their reply soon :)

It turns out the work on integrating the above CUDA version is already ongoing in slundberg/shap#1571!:)

This work is also integrated into xgboost 1.3 https://xgboost.readthedocs.io/en/latest/gpu/index.html#gpu-accelerated-shap-values.
1.3 is in release candidate phase at the time of writing and can be obtained via
python -m pip install xgboost==1.3.0rc1

@vaishkiva @ms8909 : great news guys, because thanks to @RAMitchell CUDA work there is now a GPU implementation at least for tree-based algos (shap.GPUTreeExplainer()). Have a try yourselves, you may be pleasantly surprised by the performance improvements: here's a reproducible performance comparison test for LightGBM models using our CUDA 10.1 py38 container: https://github.com/slundberg/shap/issues/1650#issuecomment-748482915.
When we moved the estimation of SHAP values on a 100k sample (times a few hundred columns) from the CPU (a single thread of Intel Xeon Platinum) to the GPU (a single Tesla V100), wall clock time shortened from an hour to... less than a minute!:) GPU has an enormous advantage here, because unlike LightGBM itself, the standard shap.TreeExplainer implementation was only single-threaded...