Sitemap: sitemap generation may not respect standards for ,  (Incomplete fix for #2937)

Still needs some work to handle #.## and strings in sitemap.priority correctly.
I'm a novice in Go, so it may not respect standards fully and it's unfortunately incomplete at the moment.
I created tests and did my best to complete it, but I've had difficulties to handle #.## and strings correctly for sitemap.priority.
I think it should be a good start for someone to pick from and finish. I did as best as I could.
Tests should be complete.
(*) I didn't handle "loc" yet and I would probably suggest splitting handling that one in another issue and another fix as it doesn't seem at the same place in the code.

This line:

priority := cast.ToFloat64(value)

Uses this: https://github.com/spf13/cast/blob/master/cast.go
I've found this to possibly handle strings:
http://stackoverflow.com/questions/36391664/go-convert-string-to-float32
I've also found this to possibly handle #.##:
http://stackoverflow.com/questions/24852317/how-to-count-decimal-places-of-float
I'm unsure what the best way according to projects standards to handle these two validations would be.

I think these 3 lines and their related lines may also need to be reviewed:

viper.SetDefault("sitemap", Sitemap{Priority: -1, Filename: "sitemap.xml"})
(see: https://github.com/spf13/hugo/blob/master/hugolib/config.go)

Keywords:    []string{}, Sitemap: Sitemap{Priority: -1},
(see: https://github.com/spf13/hugo/blob/master/hugolib/page.go)

if page.Sitemap.Priority == -1 {
(see: https://github.com/spf13/hugo/blob/master/hugolib/site_render.go)

My last commits should make this complete. I think I may only need to squash my commits.
Please review it.


Crashes after upload

As soon as I download a file, this server crashes. The file uploads just fine, and the program keeps running, so I can upload/download more files, but when I get a "Connection was reset" error in my browser, it's easy to assume the upload failed.
Here's the terminal output with python3:
server_version: simple_http_server/0.3.1, python_version: Python/3.8.10
sys encoding: utf-8
Serving http on: <IP_ADDRESS>, port: 8000 ... (http://<IP_ADDRESS>:8000/)
True File '/dev/shm/simple_http_server/test_file' upload success! by:  ('<IP_ADDRESS>', 55798)
----------------------------------------
Exception happened during processing of request from ('<IP_ADDRESS>', 55798)
Traceback (most recent call last):
  File "/usr/lib/python3.8/socketserver.py", line 316, in _handle_request_noblock
    self.process_request(request, client_address)
  File "/usr/lib/python3.8/socketserver.py", line 347, in process_request
    self.finish_request(request, client_address)
  File "/usr/lib/python3.8/socketserver.py", line 360, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/usr/lib/python3.8/socketserver.py", line 747, in __init__
    self.handle()
  File "/usr/lib/python3.8/http/server.py", line 427, in handle
    self.handle_one_request()
  File "/usr/lib/python3.8/http/server.py", line 415, in handle_one_request
    method()
  File "simple_http_server.py", line 82, in do_POST
    f.write(b"
back" % self.headers['referer'].encode('utf-8'))
AttributeError: 'NoneType' object has no attribute 'encode'
----------------------------------------
^CYou chose to stop me.

And Here's the python2 output:
server_version: simple_http_server/0.3.1, python_version: Python/2.7.18
sys encoding: utf-8
Serving http on: <IP_ADDRESS>, port: 8000 ... (http://<IP_ADDRESS>:8000/)
(True, u"File '/dev/shm/test_file' upload success!", 'by: ', ('<IP_ADDRESS>', 55800))
----------------------------------------
Exception happened during processing of request from ('<IP_ADDRESS>', 55800)
Traceback (most recent call last):
  File "/usr/lib/python2.7/SocketServer.py", line 293, in _handle_request_noblock
    self.process_request(request, client_address)
  File "/usr/lib/python2.7/SocketServer.py", line 321, in process_request
    self.finish_request(request, client_address)
  File "/usr/lib/python2.7/SocketServer.py", line 334, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/usr/lib/python2.7/SocketServer.py", line 655, in __init__
    self.handle()
  File "/usr/lib/python2.7/BaseHTTPServer.py", line 340, in handle
    self.handle_one_request()
  File "/usr/lib/python2.7/BaseHTTPServer.py", line 328, in handle_one_request
    method()
  File "simple_http_server.py", line 82, in do_POST
    f.write(b"
back" % self.headers['referer'].encode('utf-8'))
  File "/usr/lib/python2.7/rfc822.py", line 393, in __getitem__
    return self.dict[name.lower()]
KeyError: 'referer'
----------------------------------------
^CYou choose to stop me.

I think I figured it out. My browser is set to never give a referal header (because privacy). So, this server was trying to convert an empty string into binary, which doesn't work.
To work around this issue, I changed line 82 (in simple-http-server.py) to:
        f.write("
Back").encode('utf-8')

And that seems to have worked! When you have a hyperlink that points to ., that's a quick and easy way to reload the current page, without needing any POST data. It's basically a relative path to "wherever you already are". The above code also ignores your referal header, so it's not an issue anymore.
Tested with python versions 2.7.18 and 3.8.10

add PR

I will make a PR. But first, I'd like to point out that my above code doesn't seem to work on Python3. I guess *.encode commands don't work well with string that start with an unquoted b. Here are two pieces of code that seem to work better:
Option 1:
    f.write(b"
Back")

Option 2:
    f.write(("
Back").encode('utf-8'))

I have no idea which is better. If someone who knows more about Python than I do could weigh in, that'd be great. Maybe it makes no difference?

Since no one has weighed in, I've decided to make a PR with the shorter one. That seems to work well enough. If you choose to accept the PR, feel free to close this report.


Does transformer layer supported??

i am trying to pass transformer model, but encountering issues while passing model.
simplified_model = simplify(model, dummy_input,fuse_bn=False)
Am getting  AssertionError
Does transformer layer is supported at the time??

HI, could you please provide a minimal reproducible example? Namely:

How do you define the model.
How do you pruned the model.
How do you apply simplify.

And if possible the whole error you encounter.

I would be happy to create an example which would help to improve library

Please refer this example.:
I am using gelectra model, you can get it from here
https://huggingface.co/deepset/gelectra-base/tree/main
Steps: first you load the model with
tokenizer = AutoTokenizer.from_pretrained("./gelectra")
model = AutoModelForSequenceClassification.from_pretrained("./gelectra")

then you can call pytorch pruning tool:
for name, module in model.named_modules():

    # prune 20% of connections in all 2D-conv layers
    if isinstance(module, t.nn.Embedding):
        prune.l1_unstructured(module, name='weight', amount=0.2)
    # prune 40% of connections in all linear layers
    elif isinstance(module, t.nn.Linear):
        prune.l1_unstructured(module, name='weight', amount=0.1)

removing origanl weights in model
for name, module in model.named_modules():

    if isinstance(module, t.nn.Embedding):
        prune.remove(module, 'weight')
        #prune.l1_unstructured(module, name='weight', amount=0.2)
    # prune 40% of connections in all linear layers
    elif isinstance(module, t.nn.Linear):
        prune.remove(module, 'weight')

which results in zeroing some weights inside the model. Now comes the simplify, when you try to load model it fails
if you have any issue, i can assist you

Ok, thank you. We will get back to you asap.

@OriAlpha I'm not really familiar with NLP models, could you please give me an example of the dumm_input you are using?

This is where it gets confusing, while passing data. Usually you can pass inputs as below:
input_ids = tokenizer("Studies have been shown that owning a dog is good for you", return_tensors="pt").input_ids  # Batch size 1
decoder_input_ids = tokenizer("Studies show that", return_tensors="pt").input_ids  # Batch size 1

out = model(input_ids)

simplified_model = simplify(model, input_ids)

This may gives more ideas
https://github.com/huggingface/transformers/issues/16234

@OriAlpha Ok, so for what I could find there are a couple of problems with this model:

https://github.com/EIDOSlab/simplify/blob/c7db97fbf3a30ede351d2b400e73d5825ab83134/simplify/utils.py#L105 it seems that building the model graph doesn't work. Do you know if it is possible to torchscript this model? https://pytorch.org/tutorials/beginner/Intro_to_TorchScript_tutorial.html
even if we ignore the previous step (you would have to supply the pinned_out list), currently we identify the neurons to remove using forward hooks with specifically coded inputs https://github.com/EIDOSlab/simplify/blob/70f4dc7439c4357fe8a5d5866af60a8e42900fa8/simplify/remove.py#L157-L158 but this NLP model requires either int or long which cannot be nan as far as I understand.

I currently wouldn't know how to solve these issues but you are welcome to propose a PR in the meantime.

Thanks for looking into this.
While testing i had also came across this issue, you could try to pass.
x = torch.LongTensor(1, 512).random_(0, 2^53)
model(x)

I will look into this torchscript


Setup traffic generator/benchmark testing software on CNF system

TBD

Provisioned additional system for investigation of testing software


x/net/http2: clientTrace has no field or method WroteHeaderField

What version of Go are you using (go version)?
go version go1.11beta1 linux/amd64
Does this issue reproduce with the latest release?
No, because it is related only to Go 1.11 (which beta) (https://github.com/golang/net/blob/master/http2/go111.go).
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/travis/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/travis/gopath"
GOPROXY=""
GORACE=""
GOROOT="/home/travis/.gimme/versions/go1.11beta1.linux.amd64"
GOTMPDIR=""
GOTOOLDIR="/home/travis/.gimme/versions/go1.11beta1.linux.amd64/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build968009971=/tmp/go-build -gno-record-gcc-switches"
VGOMODROOT=""

https://travis-ci.org/kamilsk/click/jobs/405409835
What did you do?
Import the latest golang.org/x/net and run tests on go1.11beta1.
What did you expect to see?
Tests passed.
What did you see instead?
vendor/golang.org/x/net/http2/go111.go:12:30: trace.WroteHeaderField undefined (type *clientTrace has no field or method WroteHeaderField)
vendor/golang.org/x/net/http2/go111.go:16:26: trace.WroteHeaderField undefined (type *clientTrace has no field or method WroteHeaderField)
vendor/golang.org/x/net/http2/go111.go:17:8: trace.WroteHeaderField undefined (type *clientTrace has no field or method WroteHeaderField)

As I see https://github.com/golang/go/blob/a12c1f26e4cc602dae62ec065a237172a5b8f926/src/net/http/httptrace/trace.go#L80-L159 ClientTrace doesn't have WroteHeaderField method.

https://github.com/golang/go/issues/19761 shows the history of the related changes. 1.11beta was released on June 26. The trace api change was merged on June 27. So you’ll need a release later than beta1 or use tip.

sorry, I see it on master https://github.com/golang/go/blame/master/src/net/http/httptrace/trace.go#L148. my fault

Rolling back to 10.3 worked for me.


OIDC: Add support for dynamic user mapping

Enhancement
The HTTP API provides support for static user mapping via the configuration file. This was fine as a PoC, but now there's a need to allow admins to change the user mapping at runtime.
To facilitate this, the HTTP API needs to provide a plugin mechanism for changing how the user mappings are generated.

@MartinFlores751 - Please close if complete. Thanks


[dask] Work around segfault in prediction.

https://github.com/dmlc/xgboost/issues/7111

make data copy for base_margin.
Specify the workers.

Codecov Report

Merging #7112 (dd0e489) into master (3457968) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #7112   +/-   ##
=======================================
  Coverage   81.59%   81.60%
=======================================
  Files          13       13
  Lines        3901     3903    +2
=======================================
+ Hits         3183     3185    +2
  Misses        718      718

Impacted Files
Coverage Δ

python-package/xgboost/dask.py
82.31% <100.00%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3457968...dd0e489. Read the comment docs.


nested group support

Hi,
Now that we have nested group in gitlab since v9 it would be very interesting to implement it in gitlab-mirrors.
Would be super useful, as we could mirror several git repositories of a unique project to a unique sub group in a base "gitlab-mirrors" group (wow!)
Cheers

I was not aware of nested group.  Thanks for filing the issue.


Service example does not follow Configuration Best Practices

The “Configuration Best Practices” at https://kubernetes.io/docs/concepts/configuration/overview/#services suggest creating a Service object before creating a Deployment/ReplicaSet/Pod.
One example at https://kubernetes.io/docs/concepts/services-networking/service/#defining-a-service utilizes ports in Services by name, where the name is defined in the pod’s definition.  In that example the definition of the Pod precedes the definition of the Service.  It is unclear, whether the port-by-name concept will work if the Service is defined before the Pod.

Adjust the example at https://kubernetes.io/docs/concepts/services-networking/service/#defining-a-service to follow “Configuration Best Practices” by defining the Service after the Pod.

It's OK to create these in either order. On the page https://kubernetes.io/docs/concepts/services-networking/service/ let's make it clear that Kubernetes does the right thing, regardless of whichever order you create the Service and its backing Pods.
/language en
/priority backlog
/triage accepted

For me it is not clear how the description shall be updated.  There is a discrepancy between the example and the “Best Configuration Practices” at https://kubernetes.io/docs/concepts/configuration/overview/#services.

When I triaged this as accepted, that was based on an assumption that my proposed fix would be the approach we take.
/remove-triage accepted
I want to make it a bit more obvious that there's possibly a difference of opinion here. I don't agree with the change that you propose @dilyanpalauzov: I worry that it might make people believe that the order matters more than it actually does.
Folks are welcome to open PRs with proposed fixes. In the PR description, please highlight this issue and the open question about the best fix.


JS files are served without gzip since 1.22.7

Is there an existing issue for this?

[X] I have searched the existing issues

Output of ddev debug test
Expand `ddev debug test` diagnostic information
-2:nginx-test username_redacted$ ddev debug test
Running bash [-c /var/folders/1q/17qbltx92nbgvsq6y8sn2k_w0000gn/T/test_ddev.sh]
======= Existing project config =========
These config files were loaded for project nginx-test: [/Users/username_redacted/work/nginx-test/.ddev/config.yaml]
name: nginx-test
type: php
php_version: 8.1
webserver_type: nginx-fpm
webimage: ddev/ddev-webserver:v1.22.7
additional_hostnames: []
additional_fqdns: []
database: {mariadb 10.4}
omit_containers: [db]
project_tld: ddev.site
use_dns_when_possible: true
composer_version: 2
nodejs_version: 18
default_container_timeout: 120
======= Creating dummy project named  tryddevproject-16467 in ../tryddevproject-16467 =========
OS Information: Darwin MachineName_Redacted.local 23.3.0 Darwin Kernel Version 23.3.0: Wed Dec 20 21:30:44 PST 2023; root:xnu-10002.81.5~7/RELEASE_ARM64_T6000 arm64
ProductName:		macOS
ProductVersion:		14.3.1
BuildVersion:		23D60
User information: uid=501(username_redacted) gid=20(staff) groups=20(staff),12(everyone),61(localaccounts),79(_appserverusr),80(admin),81(_appserveradm),98(_lpadmin),701(com.apple.sharepoint.group.1),33(_appstore),100(_lpoperator),204(_developer),250(_analyticsusers),395(com.apple.access_ftp),398(com.apple.access_screensharing),399(com.apple.access_ssh),400(com.apple.access_remote_ae)
DDEV version:  ITEM             VALUE
 DDEV version     v1.22.7
 architecture     arm64
 db               ddev/ddev-dbserver-mariadb-10.4:v1.22.7
 ddev-ssh-agent   ddev/ddev-ssh-agent:v1.22.7
 docker           25.0.3
 docker-compose   v2.24.5
 docker-platform  orbstack
 mutagen          0.17.2
 os               darwin
 router           ddev/ddev-traefik-router:v1.22.7
 web              ddev/ddev-webserver:v1.22.7
PROXY settings: HTTP_PROXY='' HTTPS_PROXY='' http_proxy='' NO_PROXY=''
======= DDEV global info =========
Global configuration:
instrumentation-opt-in=false
omit-containers=[]
performance-mode=none
router-bind-all-interfaces=false
internet-detection-timeout-ms=3000
disable-http2=false
use-letsencrypt=false
letsencrypt-email=
table-style=bright
simple-formatting=false
use-hardened-images=false
fail-on-hook-fail=false
required-docker-compose-version=v2.24.5
use-docker-compose-from-path=false
project-tld=ddev.site
xdebug-ide-location=
no-bind-mounts=false
router=traefik
wsl2-no-windows-hosts-mgt=false
router-http-port=80
router-https-port=443
mailpit-http-port=8025
mailpit-https-port=8026
traefik-monitor-port=10999

======= DOCKER info =========
docker location: lrwxr-xr-x  1 root  wheel  53 Sep 20 22:57 /usr/local/bin/docker -> /Applications/OrbStack.app/Contents/MacOS/xbin/docker
Docker Desktop Version: Docker Desktop for Mac 4.26.1 build 131620
docker version:
Client:
 Version:           25.0.3
 API version:       1.44
 Go version:        go1.21.6
 Git commit:        4debf41
 Built:             Tue Feb  6 21:13:26 2024
 OS/Arch:           darwin/arm64
 Context:           default

Server: Docker Engine - Community
 Engine:
  Version:          25.0.3
  API version:      1.44 (minimum version 1.24)
  Go version:       go1.21.6
  Git commit:       f417435
  Built:            Tue Feb  6 21:14:35 2024
  OS/Arch:          linux/arm64
  Experimental:     false
 containerd:
  Version:          v1.7.13
  GitCommit:        7c3aca7a610df76212171d200ca3811ff6096eb8
 runc:
  Version:          1.1.12
  GitCommit:        51d5e94601ceffbbd85688df1c928ecccbfa4685
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0
DOCKER_DEFAULT_PLATFORM=notset
======= Mutagen Info =========
Mutagen is installed in ddev, version=0.17.2
--------------------------------------------------------------------------------
No synchronization sessions found
--------------------------------------------------------------------------------
======= Docker Info =========
Docker platform: orbstack
Using Docker context: default (unix:///Users/username_redacted/.orbstack/run/docker.sock)
docker-compose: v2.24.5
Using DOCKER_HOST=unix:///Users/username_redacted/.orbstack/run/docker.sock
Docker version: 25.0.3
Able to run simple container that mounts a volume.
Able to use internet inside container.
Docker disk space:
Filesystem                Size      Used Available Use% Mounted on
overlay                 271.8G     58.0G    213.9G  21% /

Stopped Mutagen daemon
The ddev-ssh-agent container has been removed. When you start it again you will have to use 'ddev auth ssh' to provide key authentication again.
Network ddev_default removed
Existing docker containers:
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
Network ddev_default created
Starting tryddevproject-16467...
Network ddev-tryddevproject-16467_default created
 Container ddev-ssh-agent  Created
 Container ddev-ssh-agent  Started
ssh-agent container is running: If you want to add authentication to the ssh-agent container, run 'ddev auth ssh' to enable your keys.
Building project images...
Project images built in 0s.
 Container ddev-tryddevproject-16467-web  Created
 Container ddev-tryddevproject-16467-db  Created
 Container ddev-tryddevproject-16467-db  Started
 Container ddev-tryddevproject-16467-web  Started
Waiting for web/db containers to become ready: [web db]
Starting ddev-router if necessary...
 Container ddev-router  Created
 Container ddev-router  Started
Waiting for additional project containers to become ready...
All project containers are now ready.
Successfully started tryddevproject-16467
Project can be reached at https://tryddevproject-16467.ddev.site https://<IP_ADDRESS>:33196
======== Curl of site from inside container:
HTTP/1.1 200 OK
Server: nginx
Date: Wed, 06 Mar 2024 13:50:40 GMT
Content-Type: text/html; charset=UTF-8
Connection: keep-alive
Vary: Accept-Encoding

======== curl -I of http://tryddevproject-16467.ddev.site from outside:
HTTP/1.1 200 OK
Content-Type: text/html; charset=UTF-8
Date: Wed, 06 Mar 2024 13:50:40 GMT
Server: nginx
Vary: Accept-Encoding

======== full curl of http://tryddevproject-16467.ddev.site from outside:
Success accessing database... db via TCP/IP
ddev is working. You will want to delete this project with 'ddev delete -Oy tryddevproject-16467'
======== Project ownership on host:
drwxr-xr-x  4 username_redacted  staff  128 Mar  6 14:50 ../tryddevproject-16467
======== Project ownership in container:
drwxr-xr-x 4 username_redacted dialout 128 Mar  6 13:50 /var/www/html
======== In-container filesystem:
Filesystem     Type     1K-blocks      Used Available Use% Mounted on
mac            virtiofs 971350180 732850808 238499372  76% /var/www/html
======== curl again of tryddevproject-16467 from host:
Success accessing database... db via TCP/IP
ddev is working. You will want to delete this project with 'ddev delete -Oy tryddevproject-16467'
Thanks for running the diagnostic. It was successful.
Please provide the output of this script in a new gist at gist.github.com
Running ddev launch in 5 seconds

Please run cleanup after debugging with 'ddev debug testcleanup'
MachineName_Redacted:nginx-test username_redacted$

Expected Behavior
JavaScript files should be served with a Content-Encoding: gzip response header.
Actual Behavior
Since v. 1.22.7, .js files are no longer served with a Content-Encoding header (i.e. files are not gzipped). Tested w/ both Nginx and Apache.
I believe https://github.com/ddev/ddev/pull/5661 (commit https://github.com/ddev/ddev/commit/967695097d3bacd06b597a3b76edf7680cfffcd5) is the reason, as the MIME type for the js extension was changed from application/javascript to text/javascript, but text/javascript hasn't been added to gzip_types in the Nginx conf, or in mod_deflate for Apache.
Side note: as the js extension is already mapped to application/javascript in Nginx' default mime.types, the following warning has started to appear when Nginx boots:
nginx: [warn] duplicate extension "js", content type: "text/javascript", previous content type: "application/javascript" in /etc/nginx/nginx.conf:40
Steps To Reproduce

Bootstrap a project using mkdir nginx-test && cd nginx-test && ddev config --auto --omit-containers db
Add a JavaScript file in the project that is more than 256 bytes
Confirm that the Content-Encoding: gzip header is missing:
ddev exec curl -I --compressed https://nginx-test.ddev.site/test.js:

HTTP/2 200
accept-ranges: bytes
content-type: text/javascript
date: Wed, 06 Mar 2024 13:44:39 GMT
etag: "65d79342-3058c"
last-modified: Thu, 22 Feb 2024 18:32:34 GMT
server: nginx
content-length: 198028

Anything else?
No response

Thanks for the good research on this. Would you be willing to do a PR to fix it?

@rfay Sure, I can do that :)

Put up a PR. Note that the PR does not include a fix for the duplicate extension Nginx warning mentioned above, as I'm not sure how that could be solved without DDEV shipping its own mime.types file. I guess it's also a separate problem and should probably have its own issue :)

Would love to have you solve that one too :) Thanks for paying attention! We can ship our own mime.types, thought we were actually.

We can ship our own mime.types, thought we were actually.

In that case, I guess the types declaration in nginx.conf should probably be removed, and its contents moved to mime.types?
I can take a stab at it if you like me to, but running out of time for today unfortunately :)

Would love to have you do this. Having community experts who are impacted by these things makes it all come out so much better. Thank you!

Can it be that in version 1.23.4 we still have this problem?

@KarelJanVanHaute if you think you still have this problem, please open a new issue and give a good demonstration of it.  Make sure you mention whether you're using Apache or Nginx.


[SUPPORT] I am trying to run hudi as mentioned in the getting started guide first time and I am facing this issue dyld: lazy symbol binding failed: Symbol not found: ____chkstk_darwin. I am using MacOS Sierra 10.12.6 . Can anyone help me with this issue ! Thanks in advance

Tips before filing an issue

Have you gone through our FAQs?

Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
A clear and concise description of the problem.
To Reproduce
Steps to reproduce the behavior:

Expected behavior
A clear and concise description of what you expected to happen.
Environment Description

Hudi version :

Spark version :

Hive version :

Hadoop version :

Storage (HDFS/S3/GCS..) :

Running on Docker? (yes/no) :

Additional context
Add any other context about the problem here.
Stacktrace
Add the stacktrace of the error.

Issue was with spark 3.0 and resolved after downgrading to 2.4.5

cc @bhasudha @bvaradar  but we have tested quickstart on spark 3, right?


cpu.share is not properly set for containers in kubernetes

I were trying to use limitRange to set default cpu request for each container to prevent overcommit a node badly before we have more intelligent scheduler, and identified the issue.
Through kubernetes I scheduled a container asking for 400m, (409 cpu.share) but kubernetes always assigns 1024 cpu.share to that cgroup.
# cat /sys/fs/cgroup/cpu/docker/63b0724fdfa08d44401bf7965689bd4105f099beb751b31d1bf26e2aeb71fadc/cpu.shares
1024

Initially I thought kubelet doesn't pass cpu limit properly to docker, but docker inspect proves I were wrong:
# docker inspect 63b0724fdfa08d44401bf7965689bd4105f099beb751b31d1bf26e2aeb71fadc
[{
    "AppArmorProfile": "",
    "Args": [
        "-g",
        "daemon off;"
    ],
    "Config": {
        "AttachStderr": false,
        "AttachStdin": false,
        "AttachStdout": false,
        "Cmd": [
            "nginx",
            "-g",
            "daemon off;"
        ],
        "CpuShares": 409,
        "Cpuset": "",
        "Domainname": "",
        "Entrypoint": null,
        "Env": [
...

Then I thought it might be capped at docker level, but I tested it through docker cli, and proves I were wrong again:
# docker run -c 200 busybox sh
[{
    "AppArmorProfile": "",
    "Args": [],
    "Config": {
        "AttachStderr": true,
        "AttachStdin": true,
        "AttachStdout": true,
        "Cmd": [
            "sh"
        ],
        "CpuShares": 200,
...

# cat /sys/fs/cgroup/cpu/docker/efc87fa9a7c0087c60e116d0832e11d7f74a3c3c4ad51bd0348d7ec28a871037/cpu.shares
200

It might be docker/libcontainer issue and triggered by the way how we are using docker itself. Filed the issue here for investigation. @vmarmol  is going to look into it.

Did you see a problem with memory as well if set?  I swore I had verified this on the node a month or so ago, but maybe it was memory and not CPU.
/sub
Sent from my iPhone

On May 15, 2015, at 7:41 PM, Dawn Chen<EMAIL_ADDRESS>wrote:
I were trying to use limitRange to set default cpu request for each container to prevent overcommit a node badly before we have more intelligent scheduler, and identified the issue.
Through kubernetes I scheduled a container asking for 400m, (409 cpu.share) but kubernetes always assigns 1024 cpu.share to that cgroup.
cat /sys/fs/cgroup/cpu/docker/63b0724fdfa08d44401bf7965689bd4105f099beb751b31d1bf26e2aeb71fadc/cpu.shares
1024
Initially I thought kubelet doesn't pass cpu limit properly to docker, but docker inspect proves I were wrong:
docker inspect 63b0724fdfa08d44401bf7965689bd4105f099beb751b31d1bf26e2aeb71fadc
[{
"AppArmorProfile": "",
"Args": [
"-g",
"daemon off;"
],
"Config": {
"AttachStderr": false,
"AttachStdin": false,
"AttachStdout": false,
"Cmd": [
"nginx",
"-g",
"daemon off;"
],
"CpuShares": 409,
"Cpuset": "",
"Domainname": "",
"Entrypoint": null,
"Env": [
...
Then I thought it might be capped at docker level, but I tested it through docker cli, and proves I were wrong again:
docker run -c 200 busybox sh
[{
"AppArmorProfile": "",
"Args": [],
"Config": {
"AttachStderr": true,
"AttachStdin": true,
"AttachStdout": true,
"Cmd": [
"sh"
],
"CpuShares": 200,
...
cat /sys/fs/cgroup/cpu/docker/efc87fa9a7c0087c60e116d0832e11d7f74a3c3c4ad51bd0348d7ec28a871037/cpu.shares
200
It might be docker/libcontainer issue and triggered by the way how we are using docker itself. Filed the issue here for investigation. @vmarmol is going to look into it.
—
Reply to this email directly or view it on GitHub.

Do we have an integration test that can verify   This integration with docker and kubelet?
Sent from my iPhone

On May 15, 2015, at 7:41 PM, Dawn Chen<EMAIL_ADDRESS>wrote:
I were trying to use limitRange to set default cpu request for each container to prevent overcommit a node badly before we have more intelligent scheduler, and identified the issue.
Through kubernetes I scheduled a container asking for 400m, (409 cpu.share) but kubernetes always assigns 1024 cpu.share to that cgroup.
cat /sys/fs/cgroup/cpu/docker/63b0724fdfa08d44401bf7965689bd4105f099beb751b31d1bf26e2aeb71fadc/cpu.shares
1024
Initially I thought kubelet doesn't pass cpu limit properly to docker, but docker inspect proves I were wrong:
docker inspect 63b0724fdfa08d44401bf7965689bd4105f099beb751b31d1bf26e2aeb71fadc
[{
"AppArmorProfile": "",
"Args": [
"-g",
"daemon off;"
],
"Config": {
"AttachStderr": false,
"AttachStdin": false,
"AttachStdout": false,
"Cmd": [
"nginx",
"-g",
"daemon off;"
],
"CpuShares": 409,
"Cpuset": "",
"Domainname": "",
"Entrypoint": null,
"Env": [
...
Then I thought it might be capped at docker level, but I tested it through docker cli, and proves I were wrong again:
docker run -c 200 busybox sh
[{
"AppArmorProfile": "",
"Args": [],
"Config": {
"AttachStderr": true,
"AttachStdin": true,
"AttachStdout": true,
"Cmd": [
"sh"
],
"CpuShares": 200,
...
cat /sys/fs/cgroup/cpu/docker/efc87fa9a7c0087c60e116d0832e11d7f74a3c3c4ad51bd0348d7ec28a871037/cpu.shares
200
It might be docker/libcontainer issue and triggered by the way how we are using docker itself. Filed the issue here for investigation. @vmarmol is going to look into it.
—
Reply to this email directly or view it on GitHub.

From my testing this applies to both CPU and memory. Docker accepts the limit (and reports it), but it doesn't ask libcontainer to set the limit.
@derekwaynecarr we do not have a test today, we need one :)

I swore I checked memory a while back, and it works too. This is my first time checking cpu though. I teared down my cluster just now. Will bring up one and reported it back soon.
We do have integration tests but running a fake docker daemon. We also have a bunch of e2e tests to make sure pods on node running as expected, but no resource related validation. I am filing an issue to add e2e test for this.

Memory is not set either, all resource limits are dropped.

filed #8365

Just found the issue, it is in go-dockerclient and how we use it. This has actually never worked I think :(
We set the limits in the Config here which is what is provided by the Docker client we use.
But the resource fields actually don't exist in the official Docker API. There is verbiage that states that it should be in HostConfig which we also set but not for the resource values. However, the Docker client doesn't have these fields in HostConfig.
What is even more confusing is that Docker inspect will happily show the config we provide with the unknown fields, but since they are not in HostConfig they don't take effect.
Will file and fix the issue in go-dockerclient and then change our use to fill in HostConfig.

The plot thickens, looks like it was a breaking change in the Docker remote API. The last version had the field on the request: https://docs.docker.com/reference/api/docker_remote_api_v1.17/#create-a-container


fix: ensure CreateButton respects disabled prop in UI packages

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://refine.dev/docs/guides-concepts/contributing/#commit-convention

Bugs / Features

[x] Related issue(s) linked
[x] Tests for the changes have been added
[ ] Docs have been added / updated
[x] Changesets have been added https://refine.dev/docs/guides-concepts/contributing/#creating-a-changeset

What is the current behavior?
Create Button onClick still works when disabled prop is applied
What is the new behavior?
Create Button respects disabled prop in UI packages.
fixes #6513
Notes for reviewers

Hello @OmkarBansod02, thanks for the improving Refine!
the PR needs a changeset to release packages. https://refine.dev/docs/guides-concepts/contributing/#creating-a-changeset

Hello @OmkarBansod02, Thanks for improving Refine,we will release this PR in the first week of December!


[ollama] Add better screenshots

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


[Common]: Running tests pops up a dialog asking to confirm removing certs

Failed   Microsoft.AspNetCore.Certificates.Generation.Tests.CertificateManagerTests.EnsureCreateHttpsCertificate_DoesNotCreateACertificate_WhenThereIsAnExistingHttpsCertificates
RUNDOTNET : error Message:
   Internal.Cryptography.CryptoThrowHelper+WindowsCryptographicException : The operation was canceled by the user
  Stack Trace:
     at Internal.Cryptography.Pal.StorePal.Remove(ICertificatePal certificate)
     at System.Security.Cryptography.X509Certificates.X509Store.Remove(X509Certificate2 certificate)
     at Microsoft.AspNetCore.Certificates.Generation.CertificateManager.RemoveCertificateFromTrustedRoots(X509Certificate2 certificate) in D:\work\Common\shared\Microsoft.AspNetCore.Certificates.Generation.Sources\CertificateManager.cs:line 506
     at Microsoft.AspNetCore.Certificates.Generation.CertificateManager.RemoveCertificate(X509Certificate2 certificate, RemoveLocations locations) in D:\work\Common\shared\Microsoft.AspNetCore.Certificates.Generation.Sources\CertificateManager.cs:line 467
     at Microsoft.AspNetCore.Certificates.Generation.CertificateManager.RemoveAllCertificates(CertificatePurpose purpose, StoreName storeName, StoreLocation storeLocation, String subject) in D:\work\Common\shared\Microsoft.AspNetCore.Certificates.Generation.Sources\CertificateManager.cs:line 451
     at Microsoft.AspNetCore.Certificates.Generation.Tests.CertificateManagerTests.EnsureCreateHttpsCertificate_DoesNotCreateACertificate_WhenThereIsAnExistingHttpsCertificates() in D:\work\Common\test\Microsoft.Extensions.Internal.Test\CertificateManagerTests.cs:line 140
  [xUnit.net 00:00:17.2386222]     Microsoft.AspNetCore.Certificates.Generation.Tests.CertificateManagerTests.EnsureAspNetCoreHttpsDevelopmentCertificate_CanRemoveCertificates [SKIP]
  [xUnit.net 00:00:17.2400855]     Microsoft.AspNetCore.Certificates.Generation.Tests.CertificateManagerTests.EnsureAspNetCoreHttpsDevelopmentCertificate_ReturnsCorrectResult_WhenUserCancelsTrustStepOnWindows [SKIP]

cc @javiercn

@pranavkm What platform?
Its probably due to a trusted cert. There's nothing really we can do other than skipping the tests, but we can look into it in the future.

This was on my Windows dev box.

Ok, I'll take a look next week.

This is probably something specific to your machine. Doesn't repro on the CI and we don't have more info, so closing it for now until is reproed again and we can capture more information.


"Source" title is missing from the dropdown

ah, I was confused for a second.
It's only hidden on small screens with small menu items

As usual, this is caused by interactions with rustdoc.css :facepalm:

I think we can just remove class="title" and be ok? I'm not sure what that class actually does.

Fixed in https://github.com/rust-lang/docs.rs/pull/1437.


Rating text backgroundcolor

Summary
Please include a summary of changes

Type of change
Select all that apply (add an x inside brackets)

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

This component will be restyled with scss soon, so the work will be redone.


User federationLink

Could you add the federationLink property to the user representation?

Done, it's in release 1.0.12


Do I need to include my vendor libs in webpack or just in package.json?

Hello guys, I came across a misunderstanding in my current development, I setup a project to use the webpack features, however when I try to use a component that implements some JS libraries, the component crashes, I then went to the package.json and realized my packages were there but not in the webpack.config.vendor.js, I was wondering, if I must include the packages also there? and is there any cli or command that does it?

With the current setup, you need to put them in there, yes.
There are other ways of creating vendor bundles, such as adding a specific entry file (vendor.ts lets say), and having all your import polyfills/vendor things there. But that's a different setup all together. Just a fyi!

I'd say it's optional whether you put them in the vendor package list or not:

If you do, then when you rebuild the vendor bundle (webpack --config webpack.config.vendor.js), the package will go into that bundle, and will no longer be included in your own app code's bundle
If you don't, then when you rebuild your app code bundle (webpack), the third-party library will be included in that bundle

Generally if it really is third-party code, it makes sense to have it in the vendor bundle rather than your app code bundle, as it reduces build times and deployed code churn.

I setup a project to use the webpack features, however when I try to use a component that implements some JS libraries, the component crashes

If it crashes, that suggests there is some other problem besides which bundle the code is being built into.
Closing as I think the original question is answered, but please let us know if not.

@SteveSandersonMS What do you think about removing array with vendor libraries in webpack.config.vendor.js and replacing it with distinguish dependencies and devdependencies. In such case dependencies would be all vendor libraries and devDependencies would be all other libraries that are required only to build application.

@rosieks I suspect a lot of people would not like that, because dependencies and devDependencies already have specific meanings - it's not the same distinction as "vendor vs non-vendor".


Error with latest angular2 rc app running with systemjs

Here is the error:
2016-08-15T04:45:41.731Z getting http://localhost:3001/nearby
Error: eval@[native code]
invoke@http://localhost:3001/node_modules/zone.js/dist/zone.js:323:34
run@http://localhost:3001/node_modules/zone.js/dist/zone.js:216:50
http://localhost:3001/node_modules/zone.js/dist/zone.js:571:61
invokeTask@http://localhost:3001/node_modules/zone.js/dist/zone.js:356:43
runTask@http://localhost:3001/node_modules/zone.js/dist/zone.js:256:58
drainMicroTaskQueue@http://localhost:3001/node_modules/zone.js/dist/zone.js:474:43
g@http://localhost:3001/node_modules/core-js/client/shim.min.js:8:19360
http://localhost:3001/node_modules/core-js/client/shim.min.js:8:19482
k@http://localhost:3001/node_modules/core-js/client/shim.min.js:8:14322
Evaluating http://localhost:3001/app
Error loading http://localhost:3001/app
http://localhost:3001/app is not supposed to be evaluated. System.js has this mapping that will change it to http://localhost:3001/app/app.js . See https://angular.io/docs/ts/latest/quickstart.html for the sample systemjs.config.js
The app works in browser on its own. But this service cannot load it.

It's probably a problem with PhantomJS or Angular2 I've filed a report at
https://github.com/ariya/phantomjs/issues/14475 and
https://github.com/angular/angular/issues/10804

Thanks! When testing with your site of https://www.craigsmenu.com, here are the requests + and responses - we're seeing:
2016-08-15T14:49:41.957Z getting https://www.craigsmenu.com/nearby
2016-08-15T14:49:47.027Z + 1 https://www.craigsmenu.com/nearby
2016-08-15T14:49:47.416Z - 0 https://www.craigsmenu.com/nearby
2016-08-15T14:49:47.419Z + 1 https://www.craigsmenu.com/node_modules/ng2-material/ng2-material.css
2016-08-15T14:49:47.420Z + 2 https://www.craigsmenu.com/node_modules/core-js/client/shim.min.js
2016-08-15T14:49:47.421Z + 3 https://www.craigsmenu.com/node_modules/zone.js/dist/zone.js
2016-08-15T14:49:47.422Z + 4 https://www.craigsmenu.com/node_modules/reflect-metadata/Reflect.js
2016-08-15T14:49:47.422Z + 5 https://www.craigsmenu.com/dist/bundle.min.js
2016-08-15T14:49:47.727Z - 4 https://www.craigsmenu.com/node_modules/ng2-material/ng2-material.css
2016-08-15T14:49:47.744Z - 3 https://www.craigsmenu.com/node_modules/reflect-metadata/Reflect.js
2016-08-15T14:49:47.826Z - 2 https://www.craigsmenu.com/node_modules/zone.js/dist/zone.js
2016-08-15T14:49:47.827Z - 1 https://www.craigsmenu.com/node_modules/core-js/client/shim.min.js
2016-08-15T14:49:48.038Z - 0 https://www.craigsmenu.com/dist/bundle.min.js
2016-08-15T14:49:48.192Z + 1 https://www.craigsmenu.com/app/app.template.html
2016-08-15T14:49:48.193Z + 2 https://www.craigsmenu.com/app/list/recent/recently-uploaded.template.html
2016-08-15T14:49:48.195Z + 3 https://www.craigsmenu.com/app/list/nearby/nearby.template.html
2016-08-15T14:49:48.195Z + 4 https://www.craigsmenu.com/app/list/restaurant-card.template.html
2016-08-15T14:49:48.196Z + 5 https://www.craigsmenu.com/app/details/restaurant-details.template.html
2016-08-15T14:49:48.197Z + 6 https://www.craigsmenu.com/app/upload/upload.template.html
2016-08-15T14:49:48.327Z - 5 https://www.craigsmenu.com/app/app.template.html
2016-08-15T14:49:48.334Z - 4 https://www.craigsmenu.com/app/list/nearby/nearby.template.html
2016-08-15T14:49:48.335Z - 3 https://www.craigsmenu.com/app/details/restaurant-details.template.html
2016-08-15T14:49:48.337Z - 2 https://www.craigsmenu.com/app/list/recent/recently-uploaded.template.html
2016-08-15T14:49:48.338Z - 1 https://www.craigsmenu.com/app/list/restaurant-card.template.html
2016-08-15T14:49:48.340Z - 0 https://www.craigsmenu.com/app/upload/upload.template.html
2016-08-15T14:49:48.341Z + 1 https://www.craigsmenu.com/app/upload/upload.css
2016-08-15T14:49:48.431Z - 0 https://www.craigsmenu.com/app/upload/upload.css
Angular 2 is running in the development mode. Call enableProdMode() to enable the production mode.
Sorry, your browser does not support geolocation services. Showing Seattle downtown.
2016-08-15T14:49:49.158Z got 200 in 7201ms for https://www.craigsmenu.com/nearby

I'm not seeing an error. I'm also not seeing app.js being loaded with I visit your site in my browser. Is there a difference between your site and what you were running on localhost?

@thoop Sorry about that. What the site has and what I have locally are different. Production site bundled all the js and does not use systemjs to do dynamic load.
I'm going to provide you a site that runs like the local by tonight.

@thoop I have made dev.craigsmenu.com serve the js files using systemjs . Please have a look
https://dev.craigsmenu.com/recently-uploaded

Sorry for the delay here! Is dev.craigsmenu.com still serving js with systemjs? I'm not seeing that error.

No. I changed it because I need to work on it. I'll build a separate one
and leave it untouched
On Friday, August 26, 2016, Todd Hooper<EMAIL_ADDRESS>wrote:

Sorry for the delay here! Is dev.craigsmenu.com still serving js with
systemjs? I'm not seeing that error.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prerender/prerender/issues/339#issuecomment-242825927,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBefoPqRFUJqeyAGieJy-aUAVqCaokkks5qjzrZgaJpZM4JkHJ-
.

Thanks. Sorry for the delay.

Hello. I have made
http://systemjs.craigsmenu.com/
to use systemjs and note that this site is very slow since it is async loading dozens of big js files.

Thanks for doing that. Could this be an issue related to https://github.com/jspm/registry/issues/358 ?
Are you using angular.bootstrap with systemjs? Are you making sure to wait until after systemjs has loaded the config files and everything? Seems like there shouldn't be any issue with PhantomJS setting those configs through systemjs just like Chrome does unless it's a timing issue.
If you are using angular.boostrap, can you try to have it wait a little longer before bootstrapping to see if that does anything differently?

@thoop I don't think it is about the timing of JavaScript load. If we have problem like that, won't it show this error for all browsers? The situation is that Chrome, Safari, and other browsers works fine, just PhantomJS

I filed a ticket to PhantomJS
https://github.com/ariya/phantomjs/issues/14515

Well PhantomJS isn't as lenient when it comes to timing. If you were able to reproduce this with vanilla PhantomJS, then let's see what they say on that issue.


Fix null reference on document load

When the script is included in the <head>, it fails attempting to reference document.body
which is not yet available at the moment.
Related: https://github.com/intljusticemission/react-big-calendar/issues/283

Has this been fixed to better work with Rails and the Ract_On_Rails gem or is it still pending?

It will be fixed if this PR gets merged.

Any chance of a merge on this?

I'm not super comfortable with using document.head as the element here. It seems to me the real issue is loading the script in the HEAD, why are ya'll doing that?

@jquense Since we need a node just for the feature detection, in this regard <head> should be as good as <body>. Probably, there's a different way to check for available methods, I was striving for a minimal diff.
Loading script in the <head> is a default in Ruby on Rails, for example.

can you change that default? it's a bad default... sorry I'm not trying to be difficult, this may be fine for this one method there are lots of other methods here that do feature detection and using the head element wouldn't work for some of them.

One of the reasons that comes to mind, why they did it this way is to support a feature called "turbolinks" — when you navigate between pages, it replaces document body via AJAX call, keeping the <head> loaded, to save time on the js/css loading and parsing. I don't think there's any chance of changing that.
Do you have an idea how we can rewrite this without <body>?
One alternative is to do the feature detection in runtime(by exporting a wrapper function), which would certainly hurt performance, but not sure how much.

Changing the way Rails works doesn't seem ideal. Is there any way this code can be deferred until after the body element exists?
I'm not sure if this'll work...
function matches (node, selector) {
  var fn = null;
  if (canUseDOM) {
    if (fn === null) {
      let body = document.body

      if (body) {
        let nativeMatch = body.matches
                     || body.matchesSelector
                     || body.webkitMatchesSelector
                     || body.mozMatchesSelector
                     || body.msMatchesSelector;

        fn = nativeMatch
        ? (node, selector) => nativeMatch.call(node, selector)
        : ie8MatchesSelector;
      }
    }
    fn.call(node, selector)
  }
}

Is this still being worked on or has this fix for Rails been included? If so what is the current version number to install?

I have the same issue building a Meteor Android app using Cordova. Since I'm including react-big-calendar I see the same exception as @RKushnir. This seems to be the only package I'm including which is causing this issue. Is there any chance of this being merged?


Fix lazy document parsing and error reporting

Motivation
We identified two issues after the server refactor in main:

Error reporting was failing, because the response was not in the expected structure
Trying to open an untitled Ruby file was failing because we were trying to run lazy document parsing event on text synchronization notifications (which we didn't do before the refactor)

This PR fixes both.
Implementation
We never want to parse documents on text synchronization notifications, so I excluded them by checking against textDocument/did, which all of the notifications use.
For the error, the issue was simply that the fields have to be nested inside the error key.
Manual Tests

Start the LSP on this branch
Open a new untitled file
Switch the language mode to Ruby
Verify it does not fail because of lazy parsing

Note: I actually identified a bug in RuboCop while trying this out and you will see an IndexError when opening the empty file. RuboCop is trying to access some index in the source buffer which fails because the string is empty.

Looks like the RuboCop error is surfaced through #1459. I have openend https://github.com/rubocop/rubocop/pull/12802


Linux machine steps refers to an .exe file

[Enter feedback here]
Why do we have steps 11 to 15 in "Install on Linux machine" section.
It refers to an .exe file..
Can you pls double check?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b8860a2d-9dcc-481e-a650-441b9609454f
Version Independent ID: 2f160eea-82f2-1731-a59c-9ade1ca409a4
Content: Prepare source machines to install the Mobility Service through push installation for disaster recovery of VMware VMs and physical servers to Azure
Content Source: articles/site-recovery/vmware-azure-install-mobility-service.md
Service: site-recovery
GitHub Login: @Rajeswari-Mamilla
Microsoft Alias: ramamill

@kumarallamraju Thanks for pointing this out. I have assigned the issue to the doc author for updating the document accordingly.

@Rajeswari-Mamilla Can you please look into this and update the document as applicable.

Thanks for your feedback. The steps explain that you have to add the Linux username in the CSPSconfigtool on the configuration server.

My original question was why we have references to .exe in the Linux install.
THis page https://docs.microsoft.com/en-us/azure/site-recovery/vmware-physical-mobility-service-overview (Locate installer files) shows
there's an install for Linux too. Why are we referencing  a windows installer. May be I'm missing something..  If you think my understanding is wrong, feel free to close this issue

The installer files for the Mobility service on Linux are listed here - https://docs.microsoft.com/en-us/azure/site-recovery/vmware-physical-mobility-service-overview#locate-installer-files. There's no exe for Linux listed here.
The reference to the exe file in the Linux section on this page - https://docs.microsoft.com/en-us/azure/site-recovery/vmware-azure-install-mobility-service - refers to adding the Linux account on the configuration server, using the CSPSConfigtool.
Let me know if this is unclear or if you have more questions. Thanks!

THanks @rayne-wiselman

Welcdome. I'll sharpen up the text a bit to make this all a bit clearer.


[Bug Report] Dialog close event not trigger when bind visible without .sync

Element UI version
1.4.1
OS/Browsers version
Chrome 60
Vue version
2.4.2
Reproduction Link
https://jsfiddle.net/mrku058q/
Steps to reproduce

bind :visible="dialogVisible" without .sync
click the close button at the top right-hand corner,  the close event not trigger.

What is Expected?
Whether or not there is a modifier, the close event must be triggered by close button.
What is actually happening?
with .sync modifier, the event can be triggered.
without this modifier, can't be triggered

In this case, please use visible-change: https://jsfiddle.net/mrku058q/1/ .
This is a down side when we try to be compatible with the original usage v-model="dialogVisible". Hopefully this will be improved in the next major version.

@Leopoldthecoder Thanks for you reply

And .... visible-change, will be triggered by unvisible -> visible and visible -> unvisible, can be equal with @close

visible-change is a temporary event and will be removed in the next major version.

🙄。。。

Because of this it is actually impossible (or isn't it?) to use the dialog component inside another component (children of the main app). With visible.sync it throws
Avoid mutating a prop directly since the value will be overwritten whenever the parent component re-renders. Instead, use a data or computed property based on the prop's value. Prop being mutated: "visible"
https://jsfiddle.net/6f7jvws0/
With visible-change it would work, but if that is going to be deprecated, I'm hesitant to implement it.

@haffla We're deprecating visible-change only because the close event will work in this case.


mavlink : fix pointer for main_mode and sub_mode

Hotfix for master. Not sure how it got through CI successfully.

fmu-v2 flash overflowed.


Default Add-AppxPackage Installer can provide an invalid winget source list

Hey Team, I'm not sure how far reaching this issue may be, but I've seen this as a problem with 2 specific Windows 11 22H2 AVD SKU's within Azure recently after you first deploy a VM and try to use the following command:
Add-AppxPackage -RegisterByFamilyName -MainPackage Microsoft.DesktopAppInstaller_8wekyb3d8bbwe
Image Plans Affected in Azure:

win11-22h2-avd (Multi-Session: Did not have AppInstaller by default)
win11-22h2-ent (Single-Session: Did have AppInstaller by default)

Once AppInstaller is verified to be installed from the Base O/S, when you do a winget source list, you see this on the list:
https://winget.azureedge.net/cache
When attempting to install an app via winget with that source, it will fail.  It should be https://cdn.winget.microsoft.com/cache.   Workaround is either remove that source/add the correct one or I've had to download and install winget directly from GitHub and that seemed to work:
# Downloads and installs Winget
$URI = "https://aka.ms/getwinget"
$Installer = "$env:SystemDrive\packages\Microsoft.DesktopInstaller.msixbundle"
Invoke-WebRequest -Uri $URI -OutFile $Installer
Add-AppxPackage $Installer

I don't know if this is a code/bug defect in certain images or Windows 11 22H2 itself.  Thought it would be worth sharing.

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

ID: 98654069-5528-4f98-7218-a803bee0a532
Version Independent ID: b49c32ea-2efa-b090-7123-87c31d10279b
Content: Use the winget tool to install and manage applications
Content Source: hub/package-manager/winget/index.md
Product: dev-environment
Technology: package-manager
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

We're working on CDN migration:

https://github.com/microsoft/winget-cli/issues/3652

If App Installer (delivery vehicle for WinGet) is updated, the newer client versions use the https://cdn.winget.microsoft.com/ URL.
One way to do this is to run winget install "App Installer" -s msstore --force after the initial registration. The "msstore" source URL isn't affected by the CDN migration work in progress.

Hey @brmcmill - Could you let me know if @denelon's suggestion to update the App Installer / WinGet version addresses the issue you're encountering when using WinGet with a Windows VM in Azure? This sounds like something that should be included in the docs at least until the issue is resolved.

Hey @denelon / @mattwojo  -- thank you for the info, I was not aware of the CDN changes!  That method will work too.  Some customers disable the MS Store for various reasons, so it may not always be an option.

I tested the store cmdline out, but it did not update the source list.  Did not find it.


Record Events when InferenceService goes in and out of readiness

What this PR does / why we need it:
Adds logic to record events when the InferenceService moves from
non-Ready --> Ready
and
Ready --> non-Ready
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

//TODO

/ok-to-test

/retest

@ifilonenko Cool! mostly lgtm, just a few comments.

/retest

/retest

/retest

@ifilonenko Awesome work! This is good to merge, can you help make an issue to link this PR and describe more how it can be used, so other people can reference.

Thanks! Done, ready for merge

/lgtm
/approve


Proposal for init method

Hello,
I'm suggesting about an init method which initialize a struct type:
package my_initialized_type

type AType struct {
    ExportedField string

    privateField int
}

func (v *AType) init() {
    if v.ExportedField == "value" {
        v.privateField = 1234
    } else {
        v.privateField = 4321
    }
}

func (v *AType) Get() int { return v.privateField }

The init method will be called when an instance if created. I think that sometimes it's useful non-exported field can be initialized without another user of the package have to call a method explicitly.
Like this:
package main

import "fmt"
import "my_initialized_type"

func main() {
    val := &my_initialized_type.AType{ExportedField:"value"}
    fmt.Println("Value=", val.Get())
}

I know it can be resolved with a function used as a constructor like this:
package my_initialized_type

func NewValue(exportedField) *AType {
    res := &AType{ExportedField:exportedField}
    if res.ExportedField == "value" {
        res.privateField = 1234
    } else {
        res.privateField = 4321
    }

    return res
}

But it's more fun to create an instance by struct initialization than by a function, plus the lib user can do like he want, if he want initialize empty with new(my_initialized_type.AType), it's simpler than my_initialized_type.NewValue("").
The init method is a true constructor.

Having an "init" method for a type is already quite simply (https://play.golang.org/p/6x52RIJ_rd)
Furthermore, this would allow the "init" method to explicitly override values given for struct initialization which is completely non obvious.


Revert naming test-proxy

Save ourselves the downstream pain of:

[x] not aligning with the namespace (which plays hell with the .net usage of this thing)
[x] needing to worry about name squatters (generic test-proxy being the name :|)
[x] fixing the numerous other places I have probably broken stuff by the rename

Just...just save myself the pain.
spoiler alert
I did not save myself anything unfortunately. I have discovered all of these issues with the latest eng/common update that involves this. Reverting, updating that eng/common one last time, and hopefully being done with this.

Real talk though I think this was an ill-advised change to make the standalone executable a bit nicer to use. At the end of the day I don't think that anyone will really care about the difference between test-proxy and azure.sdk.tools.testproxy when it's the same binary.

This is going to require another update of azure-sdk-build-tools to fix, due to the fact that the method of detecting the binary is flawed.
It's my code so I can't be sad about it.

This is going to come through the same avenue as a previous build that required azure-sdk-build-tools update. Will kludge these changes into the automatically submitted sdk-build-tools reference update. Closing.

This PR is now reflected in the sdk-build-tools update PR.


Fix c.getCanonicalName to be JDK 11 and JLS compliant

Fixes #567
On JDK 8, given
package p1
object O1 {
  object O2 {
    object O3 {
    }
  }
}

p1.O1.O2.O3.getClass.getCanonicalName fails with java.lang.InternalError: Malformed class name. #374 adds a workaround for this by connecting enclosing class's canonical name + member simple name with $.
Now that https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8057919 is fixed, on JDK 11 p1.O1.O2.O3.getClass.getCanonicalName succeeds with p1.O1.O2$.O3$. This makes sense because Java Language Specification 6.7 says:

A member class or member interface M declared in another class or interface C has a canonical name if and only if C has a canonical name.
In that case, the canonical name of M consists of the canonical name of C, followed by ".", followed by the simple name of M.

This fixes the implementation to follow JLS by changing $ to ..
Ref #540

To validate this pull request against other sbt modules, please visit this link.

on JDK 11 p1.O1.O2.O3.getClass.getCanonicalName succeeds with p1.O1.O2$.O3$

Why does this make sense?  I feel like either O2 shouldn't have a dollar sign or O1 should.

@dwijnand I think it's because Scala encodes it that way:
$ ls target/scala-2.12/classes/p1/
O1$.class        O1$O2$.class     O1$O2$O3$.class  O1.class

O2$ can hang off of O1 class as public static member probably, but there's no facility for that for  others.

Just out of interest, this is how it how I assumed it would look like:
07:19:34 $ m O1.java
package p1;

public class O1 {
  public static class O2 {
    public static enum O3 {
      INSTANCE
    }
  }
}

07:19:38 $ mkdir target

07:19:44 $ javac O1.java -d target

07:19:47 $ JENV_VERSION=11 sc -cp target
Welcome to Scala 2.12.8 (OpenJDK 64-Bit Server VM, Java 11).
Type in expressions for evaluation. Or try :help.

scala> p1.O1.O2.O3.INSTANCE.getClass.getCanonicalName
res0: String = p1.O1.O2.O3

Yeah... the encoding for top-level object is tweaked if there's no top-level. We have some tests for it in the codebase IIRC.


test: replaced 3rd apty api in API_On_Load spec

replacing 3rd party API with TED api
/ok-to-test<EMAIL_ADDRESS>

[!TIP]
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/11030794737
Commit: ee0ac5c8e34d0bf5c863fb9ef0091678ccec692d
Cypress dashboard.
Tags: @tag.JS
Spec:
Wed, 25 Sep 2024 10:41:35 UTC

/ci-test-limit-count run_count=25

closing this as its redundant


Documentation of SML with SMC as child in Submodel Template (SMT) specifications

There is the constraint AASd-120 in the metamodel specification stating that "idShort of submodel elements being a direct child of a SubmodelElementList shall not be specified".

However in some SMT specifications like IDTA 02036-1-0 Product Change Notifications (Draft) an idShort is awarded and a separate class table is used to document the elements of the SMC. See the picture below.

My question is whether contraint AASd-120 is effective for SMT and whether the existing SMT specifications with idShort are valid or not.

We need a different kind of table for SubmodelElementList, e.g.

With the constraint AASd-120 in the metamodel specification stating that "idShort of submodel elements being a direct child of a SubmodelElementList shall not be specified" there is in the example above no idShort available for the SubmodelElementCollection.
In the screenshot above there is a class name "Marking" defined in the specification document. This class name is not modeled in the AASX.
The SubmodelElementCollection with class name "Marking" would need a separate class table for its elements where no idShort would be available for (if constraint AASd-120 applies).
Question: Should the line idShort be replaced by "Class name" in this case for the standard table for SMC?

The SMC table for Marking would be part of the SMT specification and is used by the SML. But you are right, it would not be part of the aasx file itself. There we only have the semanticId for referencing.
I would still use the idShort Marking for the SMC table as well. It is some kind of SMT dropin that could also be reused at other places different to the SML Markings.
What could be done today with V3.0: create a ConceptDescription for the semanticIdListElement and define Marking as the idShort of  the ConceptDescription.

But perhaps we should relax constraint AASd-120  for SMT.

I will close this issue, please feel free to reopen it if the steps taken are not considered to be sufficient. Thank you, Birgit


Sub site area enablement

The check of the hierarchy consistency does not work anymore. Please check my comments

Thanks again for the review. I have implemented your suggestions, added the properties to the JSON schemes and also added the tests for the missing properties.

@OliveGerste : I cannot create a site area anymore even with no parent site area (with the latest changes).
Please double check.

@OliveGerste : I cannot create a site area anymore even with no parent site area (with the latest changes). Please double check.

You are right, there is a problem if no parent is set.
Reason: the front end is sending empty strings for the parent ID if it is not set. These empty strings are used to search for the parent site area (which is not existing of course).
Would you consider this as a front end bug? Or should I handle empty strings in the backend?

@OliveGerste : I added a few comments.
This is still DRAFT? Are you ok to make it "Ready for review"?
Thanks
Claude

@LucasBrazi06  Thanks a lot for the review. I implemented most of the changes, but need some additional input on some things:

I am not completely sure about the renaming:

https://github.com/sap-labs-france/ev-server/pull/3062#discussion_r777597094 In fact I am building multiple trees and it returns an array of root site areas. So if there are 10 site areas in the input list, this could result in an array of 5 different root site areas (trees)

Consistency in Site Area Tree: https://github.com/sap-labs-france/ev-server/pull/3062#discussion_r783796825

I think the last site area of a chain can only be ignored by excluding all charging stations on this site area from smart charging (Or using not supported ones), otherwise this will always exclude the whole power of the site area, which will result in almost no power left for other site areas even if no one is charging there.

Circular dependency check in every tree build https://github.com/sap-labs-france/ev-server/pull/3062#discussion_r777577819:

We can check the constraint in every build, but I am not sure how to do the Error handling, because I need to specify the error type when catching it in UtilsService.  I have not found any reference in the server where this is done. I see two options:

Throwing a Backend Error and save the type in the message property --> check the type with switch case in Utilsservice
Introducing a new property like errorCode to the BackendError Type --> check the type with switch case in Utilsservice

Sorry for the long explanation. We can have a meeting if you think it is more effective.

Sub site area enablement #3062 (comment) In fact I am building multiple trees and it returns an array of root site areas. So if there are 10 site areas in the input list, this could result in an array of 5 different root site areas (trees)

Yes, this is the goal of the method: build a tree of Site Areas.
Next you need a new one to find the node (Site Area) you want in the tree.
And finally you check the Smart Charging consistency with a third one :-).
One method -> One task: so I can reuse the build of the site area tree outside the Smart Charging context for instance.

Consistency in Site Area Tree: Sub site area enablement #3062 (comment)

Okay, I understand, let's keep it simple then. No changes in Smart Charging constraints.

We can check the constraint in every build, but I am not sure how to do the Error handling, because I need to specify the error type when catching it in UtilsService.  I have not found any reference in the server where this is done.

You can throw a backend error when building the tree and in UtilsService you provide a specific try...catch around this method and return a specific error code to the UI?

Yes, this is the goal of the method: build a tree of Site Areas. Next you need a new one to find the node (Site Area) you want in the tree. And finally you check the Smart Charging consistency with a third one :-). One method -> One task: so I can reuse the build of the site area tree outside the Smart Charging context for instance.

Building all the site area trees of one site refers to Utils.buildSiteAreaTrees() (We need this method to check consistency of all independent trees in one site)
This one should be renamed to Utils.buildSiteAreaTree(), but in fact it is not returning only one tree, but an array of trees
Getting a specific tree refers to Utils.buildSiteAreaTree() Which first builds the trees and only returns the tree (rootSiteArea and all children) which contains the requested site area. I agree with renaming this to findRootElementInSiteAreaTree()
So according your comment building the tree and checking consistency should be completely separated? So that we have separate methods for this?

You can throw a backend error when building the tree and in UtilsService you provide a specific try...catch around this method and return a specific error code to the UI?

Yes, my question was, which property of BackEndError I should use for saving the errorcode.
When throwing the error in Utils, I could use e.g. the message property to save the error code.
After this I catch in UtilsService and determine the type by a switch case on the error message?

Building all the site area trees of one site refers to Utils.buildSiteAreaTrees() (We need this method to check consistency of all independent trees in one site)
This one should be renamed to Utils.buildSiteAreaTree(), but in fact it is not returning only one tree, but an array of trees

Ok to build Trees and name the method Utils.buildSiteAreaTrees() -> Build only an array Tree

Getting a specific tree refers to Utils.buildSiteAreaTree(siteAreas) Which first builds the trees and only returns the tree (rootSiteArea and all children) which contains the requested site area. I agree with renaming this to findRootElementInSiteAreaTree()

I would rather name this method like findSiteAreaInTree(siteAreas);

So according your comment building the tree and checking consistency should be completely separated? So that we have separate methods for this?

And a third one to check consistency like checkSiteAreaTree(siteArea).
So you have 3 methods with exactly one purpose each.
Separation of concerns they call it :-)

Yes, my question was, which property of BackEndError I should use for saving the errorcode.
When throwing the error in Utils, I could use e.g. the message property to save the error code.
After this I catch in UtilsService and determine the type by a switch case on the error message?

I don't think we need a new property like 'errorCode' from the Backend exception.
We didn't need that concept so far but maybe I'm mistaken.
We can return a generic error in the UI like 'Inconsistencies occurred in the tree of Site Areas, check the logs for more details' or something more user friendly :-).
So, throwing a BackendError with a specific message to the Service layer which will log it and return always the same specific HTTP error code to the UI to display the message above would be sufficient I think.
And having to handle all the different error codes + translation will probably be a pain!
Anyway, to solve any issues with the Tree, the user will have to cancel his current change to go ahead.
What do you think?

Ok to build Trees and name the method Utils.buildSiteAreaTrees() -> Build only an array Tree
I would rather name this method like findSiteAreaInTree(siteAreas);
And a third one to check consistency like checkSiteAreaTree(siteArea). So you have 3 methods with exactly one purpose each. Separation of concerns they call it :-)

Okay so :three: methods it will be :soon:

I don't think we need a new property like 'errorCode' from the Backend exception. We didn't need that concept so far but maybe I'm mistaken. We can return a generic error in the UI like 'Inconsistencies occurred in the tree of Site Areas, check the logs for more details' or something more user friendly :-). So, throwing a BackendError with a specific message to the Service layer which will log it and return always the same specific HTTP error code to the UI to display the message above would be sufficient I think. And having to handle all the different error codes + translation will probably be a pain! Anyway, to solve any issues with the Tree, the user will have to cancel his current change to go ahead. What do you think?

All right. I am fine with this. The request for specific error messages in the front end came from a review from Claude I think.
So I will just summarize these two codes:
SITE_AREA_HIERARCHY_INCONSISTENCY_ERROR = 541,
SITE_AREA_HIERARCHY_CIRCULAR_STRUCTURE_ERROR = 542,
to one code and send it to the front end displaying a generic error message.

Closed, created a new branch


Get rid of Timidity++

Timidity++ is not maintained since 2004: https://timidity.sourceforge.net/#news
2 ways:

Allow custom synthesizer
Migrate to FluidSynth (still maintained)
or more: https://www.linuxlinks.com/synthesizers/

https://github.com/nwhitehead/pyfluidsynth
https://github.com/bzamecnik/midi2audio


Add keybindings for selecting ⬆⬇⬅➡ & adding selection above/below

This pull request adds the following bindings:

shift + ↑ = Select up
shift + ↓ = Select down
shift + ← = Select left
shift + → = Select right
control + shift + ↑ = Add selection above
control + shift + ↓ = Add selection below

Demo

Looks really nice ✨

Nice!


Add support for xwayland command

Sorry! I forgot to mention this in the old issue, honestly I'm not too familiar with vim syntax files otherwise I would submit a pr.
The commands are xwayland enable , xwayland disable and xwayland force

I haven't used those commands in my config. Could you post an example?

You can find more info about it on the sway wiki, the xwayland website, and on the sway manpage under commands

Implemented in 4c959b7.

Sorry for the late response, I can confirm that it works!


Listen and process incoming sig messages

This builds on top of #2 to implement the server for the client (the stakeholder) to have us watch for specific vaults. The listener loop is kept simple. A functional test suite is added to start testing paths not really accessible through unit tests.

Ack  up to 227f9a380ba8138a7a9cd6511e0087c280679980 for now.
Gonna take some time to go through the test framework, will try to finish review tomorrow.

How about adding a little README for the txbuilder tool?

As noted in the commit, it is a hack not meant to be used by anyone so i'd rather not add README

Ok, sorry for the force push noise. Was checking CI runs.
Added a few CI jobs, and rebased on master.

You need to update to 22.0 https://bitcoincore.org/en/releases/22.0/
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
Le mardi 28 septembre 2021 à 5:42 PM, JSwambo @.***> a écrit :

@JSwambo commented on this pull request.

In tests/test_framework/bitcoind.py:

       else:

           wait_for(lambda: len(self.rpc.getrawmempool()) >= wait_for_mempool)

   old_blockcount = self.rpc.getblockcount()

   addr = self.rpc.getnewaddress()

   self.rpc.generatetoaddress(numblocks, addr)

   wait_for(lambda: self.rpc.getblockcount() == old_blockcount + numblocks)

def create_utxo(self, address, amount):

   """Send {amount} coins to this {address}, return the outpoint of the

   created UTXO."""

   txid = self.rpc.sendtoaddress(address, amount)

   vout = 0

   while True:

       txoinfo = self.rpc.gettxout(txid, vout)

       if txoinfo["scriptPubKey"]["address"] == address:

"address" gave a KeyError. I changed to
if
txoinfo
[
"scriptPubKey"
][
"addresses"
][
0
]
==
address
:
to fix it (as per the docs)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Not introduced in this PR but re-reading the code this is wrong:
pub fn db_revoc_confirmed(
    db_path: &path::Path,
    vault_id: i64,
    height: i32,
) -> Result<(), DatabaseError> {
    db_exec(&db_path, |db_tx| {
        db_tx.execute(
            "UPDATE vaults SET revoc_height = (?1) WHERE id = (?2)",
            params![height, vault_id],
        )?;

        Ok(())
    })
}

It should set should_cancel to 0. Or maybe cleaner: have db_canceling_vaults rule out rows with a revoc_height set.


Do Not Merge!!! Test Phrases.

Do Not Merge!!! Test Phrases.

PR Checklist

[ ] The PR title follows our guidelines: https://github.com/NativeScript/NativeScript/blob/master/CONTRIBUTING.md#commit-messages.
[ ] There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it.
[ ] You have signed the CLA.
[ ] All existing tests are passing: https://github.com/NativeScript/ios-runtime/blob/master/CONTRIBUTING.md#contribute-to-the-code-base
[ ] Tests for the changes are included.

What is the current behavior?

What is the new behavior?

Fixes/Implements/Closes #[Issue Number].

test --ios-runtime-tests --performance --debug

test --performance-only

test --debug

test --debug

test --debug

test --debug

test --debug

test --debug

test --debug


MOUSE GETTING STUCK IF I HARD SWIPE

I'M HAVING AN ISSUE WITH THIS APP, WHENEVER I START IT MY MOUISE IS GETTING STUCK WHEN I HARD SWIPE OR FLICK IN VALORANT, I just found the settigns on youtube
Windows Sensitivity : 4
Valorant Sens : 0.195
DPI : 800
Polling Rate : 500 or 1000 (Both get stuck)
Raw Accel Settings :
Sensitivity : 0.8625
Rotation : 0
Linear
Acceleration : 0.08625
Sensitivity Cap : 2
Weight : 2
Offset : 0
IDK how to fix it, I installed MarkC mouse reg settings before using RAW accel
PLS HELP

from what i've seen, this behavior usually stems from a dirty mouse sensor. try cleaning it along with your pad
interference could also be an issue if you're using a wireless mouse
and if you're only getting in-game stutter/drops, try playing with the app closed, your settings will still apply after closing

I'M HAVING AN ISSUE WITH THIS APP, WHENEVER I START IT MY MOUISE IS GETTING STUCK WHEN I HARD SWIPE OR FLICK IN VALORANT,

Calm down on the caps man, not the end of the world. If it's actually raw accel causing issues as well, it would be more likely a driver issue than the GUI application.

I just found the settigns on youtube

https://www.youtube.com/watch?v=h51iDaFvp8o&ab_channel=tristan
A quick search with you putting Valorant into your issue and quickly had this popup.
From looking at your settings, they are almost a 1:1 of someone else's settings. There are nothing wrong with copy someone's settings if it feels good to you. I would look into mouse sensitivity as you may find your own settings that you like, which may or may not be used.
I have my own issues with videos like this. As they get into the subject of muscle memory and aiming, but that's a topic which you can research on your own time, and come to your own conclusion.
Because it was easy to find where you got your settings, I can guess you are using both legacy cap and gain systems. Next time, please specify that though :)

Polling Rate: 500 or 1000 (Both get stuck)

You should try staying on polling rate 1000, just because it will have less latency. Only should really be dropping down to 500 polling if you are on wireless and saving battery power, and doing something that doesn't require low latency.
The response time from 500 and 1000 isn't much though, only being a millisecond difference. It's just more recommended if possible.

IDK how to fix it, I installed MarkC mouse reg settings before using RAW accel
PLS HELP

MarkC mouse reg settings? http://www.esreality.com/?a=post&id=1846538?
Valorant is a recent game that released stable in 2020, it shouldn't need the mouse fix, nor should it apply it.
@a1xd gave some pretty good responses on what could be the issue. Interference with wireless mice can be usually solved by moving the receiver closer, easily solvable with a USB extender.
It may also be the LOD (lift-off distance) of your mouse, and you will either need to look for firmware or replacing the sensor on it. If that's too advanced, I would just look into buying another mouse. (You may not like going through the trouble of taking apart your mouse, or have a spare sensor glass part on hand.)
(Delux m800, aka a Chinese wireless knockoff of the Razer Viper Mini, not even that bad to be honest, had issues with hard swipes. The company I believe has released beta software to fix this issue, and it was leaked in the deep areas of Reddit.)
If your mouse's LOD is customizable (glorious, razer, etc), look into messing with that. It's more likely your mouse than raw accel, as no one I've seen has this type of issue with it.
Worst case, you just don't use mouse acceleration. Realize many good players have appeared on the scene without the need of mouse acceleration, there even being the time where a lot of people saw it as bad. Actually, the majority of players play without mouse acceleration, as the stigma of it ruining muscle memory is deeply ingrained in the gaming community. So is muscle memory, but personal opinion.

This one of a similar set of issues for which there is no cause in our software, and @a1xd and @KaJe5380 gave comprehensive responses as to other possible causes. Closing.


Added TorchServe support to documentation

Skipped the following:
Your pull request needs to reference a filed issue. Please fill in the template that is populated for the pull request. Only pull requests addressing small typos can have no issues associated with them.
As this feels like something that falls within the category of small text changes.

Codecov Report

Merging #537 (00ce5c7) into main (2e78108) will decrease coverage by 5.92%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #537      +/-   ##
==========================================
- Coverage   90.62%   84.70%   -5.93%
==========================================
  Files          78       78
  Lines        4510     4510
  Branches      839      839
==========================================
- Hits         4087     3820     -267
- Misses        237      498     +261
- Partials      186      192       +6

Flag
Coverage Δ

unittests
84.70% <ø> (-5.77%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ummingbird/ml/containers/sklearn/tvm_containers.py
17.24% <0.00%> (-75.87%)
:arrow_down:

...mmingbird/ml/operator_converters/sparkml/linear.py
42.85% <0.00%> (-57.15%)
:arrow_down:

...bird/ml/operator_converters/sparkml/discretizer.py
52.94% <0.00%> (-35.30%)
:arrow_down:

.../ml/operator_converters/_linear_implementations.py
75.55% <0.00%> (-22.23%)
:arrow_down:

...ird/ml/operator_converters/onnx/one_hot_encoder.py
62.50% <0.00%> (-18.75%)
:arrow_down:

hummingbird/ml/_topology.py
67.79% <0.00%> (-17.38%)
:arrow_down:

hummingbird/ml/convert.py
65.40% <0.00%> (-16.59%)
:arrow_down:

hummingbird/ml/containers/_input_containers.py
87.50% <0.00%> (-12.50%)
:arrow_down:

...ml/operator_converters/sparkml/vector_assembler.py
90.00% <0.00%> (-10.00%)
:arrow_down:

...perator_converters/_discretizer_implementations.py
90.32% <0.00%> (-9.68%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2e78108...00ce5c7. Read the comment docs.


Install Test: Copy Files on Data folder doesn't work due to incredibly long filename

CopyData.py failed when copying the R:\OPSServerInstall\Server\Staging\Data folder due to a long file name:

The name was so long Explorer wouldn't let me copy and paste the filename, so here is a screenshot:

I deleted the folder on the disk and re-ran the script. I'll manually copy the files back if possible. Not sure if anyone else will see this issue; It will be necessary to keep the file paths as short as possible.

@kerryrobinson I believe this issue might go away when we address this issue. https://github.com/Esri/ops-server-config/issues/1243

@ACueva OK thanks.  Yes it is the same thing, so we can ignore the folder.

Removing the files from the release build included on the USB drive.  In addition, different copy tool is used for install (RoboCopy)

@ACueva I removed the contents of the folder FoundationOverlays\Elevation\LasCache\LasCache from the current release build location


Homebridge Not Accessible Off Network

I know there has been discussion about accessing HomeKit outside of a home network, but my problem is only with Homebridge.  My other HomeKit accessories work outside of my network, but my Homebridge accessories are unreachable.  Any tips on troubleshooting this without completely resetting my HomeKit configuration?

Have you got an Apple TV in your network? There are some accessories, which are accessible from anywhere, but Homebridge requires an Apple TV bound to your Apple-ID.

Yes, I have an Apple TV 4 on the network logged into the same iCloud
address as my phone and mac.  Other HomeKit compatible accessories do work
outside of the network (Phillips Hue and iHome plug) but Homebridge is not.
On Tue, May 3, 2016 at 12:07 PM, DerEisvogel<EMAIL_ADDRESS>wrote:

Have you got an Apple TV in your network? There are some accessories,
which are accessible from anywhere, but Homebridge requires an Apple TV
bound to your Apple-ID.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/nfarina/homebridge/issues/624#issuecomment-216597576

Are you using Eve? When pressing on the little error triangle in Eve it gives you an error message.

In the Eve app the error triangle says "The accessory cannot be reached at
the moment. Make sure Wi-Fi and Bluetooth are enabled and try again.  You
may also try moving closer to the accessory."
I did log out and back in on the ATV a few days ago, but will try again
when I am home.
On Tue, May 3, 2016 at 1:43 PM, DerEisvogel<EMAIL_ADDRESS>wrote:

Are you using Eve? When pressing on the little error triangle in Eve it
gives you an error message.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/nfarina/homebridge/issues/624#issuecomment-216625984

I logged out of iCloud on the Apple TV then restarted the Apple TV and
logged back in.  I also rebooted my modem and router.  All HomeKit and
Homebridge accessories seem to be working off network now.  Thanks for the
help.
On Tue, May 3, 2016 at 1:49 PM, Kevin Stillson<EMAIL_ADDRESS>wrote:

In the Eve app the error triangle says "The accessory cannot be reached at
the moment. Make sure Wi-Fi and Bluetooth are enabled and try again.  You
may also try moving closer to the accessory."
I did log out and back in on the ATV a few days ago, but will try again
when I am home.
On Tue, May 3, 2016 at 1:43 PM, DerEisvogel<EMAIL_ADDRESS>wrote:

Are you using Eve? When pressing on the little error triangle in Eve it
gives you an error message.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/nfarina/homebridge/issues/624#issuecomment-216625984

today i have same problem. i have 2 raspberry pi and 2 homebridge, one work from outside and other not. the reason was one is connect to my living room Apple TV and other in my bedroom Apple TV . this morning i disconnect my bedroom Apple TV and one homebridge stop working from outside.

Take long time to homebridge to connect to my living room appletv but now is working!

Same problem here! Any news on this?

I thought this used to work, but after a reinstall of the Pi it does not work. At home everything is fine and the Apple TV is always on.


Human-readable description of the decoding steps

We need to explain the decoding steps on the site. In #1, I have already implemented a pseudo code rendering of them:

But we also want to describe them as continuous text.

Here's my go at what the description for the steps from the parent comment might look like:

Parse (the request body) as JSON.
Get the property at JSONPath batch_app_id in (the return value of line 1). Store that in (the result for the body at batch_app_id).
Parse (the property at JSONPath batch in the return value of line 2) as JSON.
Get the property relative_url for every element in (the return value of line 3).
Parse (the return value of line 4) as a query string. Store that in (the result for the body).

The values in parentheses are "blanks". The sentence needs to work with an arbitrary "identifier" put there.
There's three categories of identifiers that we can encounter, and here's how I would put those into text:

properties from the request (i.e. the input): the request body/the property at JSONPath a.b.c in the request path/the request query string
variables (temporary values only used in the decoding process but not returned at the end): the return value of line 2/the property at JSONPath a.b.c in the return value of line 3
the result object, grouped by context: the result for the request body/the result for the request path at a.b.c

I'm pretty happy with this concept, I think that should work for just about anything and it reads okay.
I don't even want to think about how we would translate that, though…

Oh, actually, I think we might need to abandon the idea of replacing temporary variable names with "the return value of line n". I'm not sure whether that's possible since we allow writing to arbitrarily nested paths within variables. Consider (a bit contrived, but just to demonstrate what I'm thinking):
b = base64Decode(body)
b.a.c = something(b.a.c)
b = parseJson(b)
b.a.c = parseQueryString(b.a.c)
res.body.b.a = something(b.a)

Generating a description for b.a.c in line 4 would already be annoying, but still doable. We'd have to walk backwards from that line and find the first instance where any prefix of b.a.c is written to. So, it would have to be "the property at JSONPath a.c in the return value of line 3", since that overrides the b.a.c = ... from line 2.
But for b.a in line 5, it's really not clear what the description should even be. It's a combination of the return values of lines 3 and 4…
While I'm quite sad about that, I think we need to stick with "the variable something", at least for now.

OK, the very first (and basic) implementation is done in 9e0c572d079c259f0596fe1f36f8dfde732a6081:

Currently, the identifiers are just pasted in as-is.

And now we also have identifier descriptions:

One slight improvement:

can be a little less verbose and robot-y:

Mostly fixed with #1 but translations are still pending.


Explicitly fail when we are asked to add a duplicate ip to the ipset

A user is (somehow) able to produce a scenario where, without CNI restarting, a pod IP goes missing from the host node ipset, causing an active pod to begin failing host node healthchecks.
Since we destructively mutate the ipset contents in exactly 3 spots:

Flush on istio-cni startup
Flush/destroy on istio-cni shutdown
Removing pod IPs on K8S pod unlabel/delete events

and no restart happens, that leaves (3) as the only way this could manifest, barring outside interference with the contents of the ipset.
The only scenario where I can imagine this happening is if

two pods reuse the same IP relatively close to each other on the same node.
K8S pod add and pod remove events for those two pods both overlap, and come in out-of-order - we add the new pod with the reused IP, and then somehow later remove the old pod with the original use of the IP, leading to the set being out of sync.

This PR changes the addPodIPToIpset logic, so that if we try to add a pod IP that is already there, we log/return an error - previously, we would silently overwrite the existing entry.
This will at least help us see if the above is what is happening - as otherwise we should never legitimately have an overwrite.
(We might have missed this in testing months back because we were using ip:port maps, and the odds of two pods using the same IP AND the same ports is even tinier)

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
8a720e05d57ae3f71926ad060a27cca551909b75
link
true
/test release-notes

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

@bleggett: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
8a720e05d57ae3f71926ad060a27cca551909b75
link
true
/test release-notes

unit-tests-arm64_istio
8a720e05d57ae3f71926ad060a27cca551909b75
link
true
/test unit-tests-arm64

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

Can the node level interception be deprecated now

Can the node level interception be deprecated now

What do you mean, exactly?
We don't do node level interception at all, but we still do need to do node level SNAT for kubelet healthchecks.

OK, NVM, I thought is was intercetion on node

we still do need to do node level SNAT for kubelet healthchecks.

Why still need this, now if ambient behaves like sidecar interception, we can eliminate host ipset completely

we still do need to do node level SNAT for kubelet healthchecks.

Why still need this, now if ambient behaves like sidecar interception, we can eliminate host ipset completely

Even for sidecars we must special-case host kubelet probes - with sidecars we just do it with webhook rewrites for the pod manifest.
Ambient doesn't have mutating webhooks, so we do it this way instead.
Sidecar mutates the pod manifest to redirect the health probes, ambient mutates the source IP to redirect the health probes. It's tricky.

Yes, we know the src addrss of kubelete probe, so can do that better than sidecar via bypass this kind of traffic

Yes, we know the src addrss of kubelete probe, so can do that better than sidecar via bypass this kind of traffic

We don't know the source address of the kubelet probe from the context of the pod in many cases actually.
Also, not all packets coming from the kubelet IP originate from the kubelet, the IP is not a reliable check.
Also it is actually difficult to reliably determine the host kubelet IP. We don't bother to derive the pod-facing kubelet IP with ambient, we just use iptables to tell us if the packet originated locally.

In response to a cherrypick label: #51081 failed to apply on top of branch "release-1.22":
Applying: Explicitly fail when we are asked to add a duplicate ip to the ipset
Using index info to reconstruct a base tree...
M	cni/pkg/nodeagent/net.go
Falling back to patching base and 3-way merge...
Auto-merging cni/pkg/nodeagent/net.go
CONFLICT (content): Merge conflict in cni/pkg/nodeagent/net.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Explicitly fail when we are asked to add a duplicate ip to the ipset
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #51131