process the sigterm signal

when the pxc cluster is in recovery or the use the debug image to start up. The sh is run as pid 1, and the singterm will not be caught. As a result, the pod will in teminating state until the terminationGracePeriodSeconds. So, add trap "exit" SIGTERM to deal with it

Test name
Status

storage
passed

haproxy
passed

upgrade-haproxy
passed

big-data
passed

scaling
passed

recreate
passed

scheduled-backup
passed

cross-site
passed

scaling-proxysql
passed

self-healing
passed

self-healing-chaos
failed

init-deploy
passed

restore-to-encrypted-cluster
passed

security-context
passed

upgrade-proxysql
passed

self-healing-advanced
passed

self-healing-advanced-chaos
failed

demand-backup
passed

smart-update
passed

limits
passed

operator-self-healing
passed

operator-self-healing-chaos
failed

monitoring-2-0
passed

pitr
failed

affinity
passed

upgrade-consistency
passed

one-pod
passed

auto-tuning
passed

demand-backup-encrypted-with-tls
passed

proxysql-sidecar-res-limits
passed

users
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

proxy-protocol
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1189/commits/42608cb86d43bcc0e04f2e1c1648f6018d67cb46
image: perconalab/percona-xtradb-cluster-operator:PR-1189-42608cb8

Test name
Status

big-data
passed

recreate
passed

storage
passed

upgrade-haproxy
passed

haproxy
passed

restore-to-encrypted-cluster
passed

self-healing
passed

scheduled-backup
passed

init-deploy
passed

cross-site
passed

scaling
passed

demand-backup
passed

self-healing-chaos
failed

scaling-proxysql
passed

limits
passed

pitr
passed

security-context
passed

monitoring-2-0
passed

affinity
passed

one-pod
passed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

users
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

proxy-protocol
passed

upgrade-proxysql
passed

smart-update
passed

upgrade-consistency
passed

self-healing-advanced
passed

self-healing-advanced-chaos
failed

operator-self-healing
passed

operator-self-healing-chaos
failed

demand-backup-encrypted-with-tls
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1189/commits/42608cb86d43bcc0e04f2e1c1648f6018d67cb46
image: perconalab/percona-xtradb-cluster-operator:PR-1189-42608cb8

Test name
Status

upgrade-haproxy
passed

scaling
passed

storage
passed

recreate
passed

cross-site
passed

haproxy
passed

big-data
passed

scheduled-backup
failed

scaling-proxysql
passed

self-healing
passed

self-healing-chaos
failed

security-context
passed

restore-to-encrypted-cluster
passed

upgrade-proxysql
passed

init-deploy
passed

self-healing-advanced
passed

self-healing-advanced-chaos
failed

demand-backup
failed

smart-update
passed

operator-self-healing
passed

operator-self-healing-chaos
failed

limits
passed

monitoring-2-0
passed

affinity
passed

one-pod
passed

upgrade-consistency
failed

auto-tuning
failed

pitr
failed

proxysql-sidecar-res-limits
passed

users
passed

demand-backup-encrypted-with-tls
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

proxy-protocol
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1189/commits/507bf5fdfb042702af766a2042dcdfe540a56ab8
image: perconalab/percona-xtradb-cluster-operator:PR-1189-507bf5fd

@nmarukovich

Test name
Status

big-data
passed

recreate
passed

scheduled-backup
passed

haproxy
failed

storage
passed

cross-site
passed

upgrade-haproxy
passed

scaling
passed

self-healing
passed

self-healing-chaos
passed

scaling-proxysql
passed

restore-to-encrypted-cluster
passed

security-context
passed

upgrade-proxysql
passed

init-deploy
passed

self-healing-advanced
passed

self-healing-advanced-chaos
passed

demand-backup
passed

limits
passed

smart-update
passed

operator-self-healing
passed

operator-self-healing-chaos
passed

monitoring-2-0
passed

affinity
passed

one-pod
passed

auto-tuning
passed

upgrade-consistency
passed

proxysql-sidecar-res-limits
passed

pitr
failed

users
failed

demand-backup-encrypted-with-tls
passed

tls-issue-self
passed

tls-issue-cert-manager
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

proxy-protocol
passed

We run 36 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1189/commits/c8bf291d8edd1e7a15fe3fae643cc2fcd727cff6
image: perconalab/percona-xtradb-cluster-operator:PR-1189-c8bf291d

Test name
Status

users-5-7
passed

haproxy-8-0
passed

scaling-8-0
passed

big-data-8-0
passed

upgrade-haproxy-8-0
passed

cross-site-8-0
passed

scheduled-backup-8-0
passed

recreate-8-0
passed

storage-8-0
passed

self-healing-8-0
passed

self-healing-chaos-8-0
passed

init-deploy-8-0
passed

scaling-proxysql-8-0
passed

restore-to-encrypted-cluster-8-0
passed

security-context-8-0
passed

one-pod-5-7
passed

upgrade-proxysql-8-0
passed

self-healing-advanced-8-0
passed

self-healing-advanced-chaos-8-0
passed

limits-8-0
passed

scheduled-backup-5-7
passed

monitoring-2-0-8-0
passed

demand-backup-8-0
passed

smart-update-8-0
passed

operator-self-healing-8-0
passed

operator-self-healing-chaos-8-0
passed

affinity-8-0
passed

one-pod-8-0
passed

init-deploy-5-7
passed

auto-tuning-8-0
passed

haproxy-5-7
passed

pitr-8-0
failed

proxysql-sidecar-res-limits-8-0
passed

users-8-0
passed

upgrade-consistency-8-0
passed

tls-issue-self-8-0
passed

tls-issue-cert-manager-8-0
failed

We run 37 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1189/commits/154dc53bfb77f3a814ad8a6c2d780ede0cc9f0a0
image: perconalab/percona-xtradb-cluster-operator:PR-1189-154dc53b

Test name
Status

recreate-8-0
passed

scheduled-backup-8-0
passed

upgrade-haproxy-8-0
passed

restore-to-encrypted-cluster-8-0
passed

users-5-7
passed

haproxy-8-0
passed

demand-backup-8-0
passed

one-pod-5-7
passed

init-deploy-8-0
passed

pitr-8-0
passed

scheduled-backup-5-7
passed

limits-8-0
passed

storage-8-0
passed

init-deploy-5-7
passed

monitoring-2-0-8-0
passed

cross-site-8-0
passed

big-data-8-0
passed

self-healing-8-0
passed

haproxy-5-7
passed

affinity-8-0
passed

self-healing-chaos-8-0
passed

one-pod-8-0
passed

self-healing-advanced-8-0
passed

auto-tuning-8-0
passed

self-healing-advanced-chaos-8-0
passed

proxysql-sidecar-res-limits-8-0
passed

operator-self-healing-8-0
passed

users-8-0
passed

operator-self-healing-chaos-8-0
passed

scaling-8-0
passed

tls-issue-self-8-0
passed

scaling-proxysql-8-0
passed

tls-issue-cert-manager-8-0
passed

security-context-8-0
passed

upgrade-proxysql-8-0
passed

tls-issue-cert-manager-ref-8-0
passed

smart-update-8-0
passed

upgrade-consistency-8-0
passed

validation-hook-8-0
passed

proxy-protocol-8-0
passed

demand-backup-encrypted-with-tls-8-0
passed

We run 41 out of 36

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/1189/commits/154dc53bfb77f3a814ad8a6c2d780ede0cc9f0a0
image: perconalab/percona-xtradb-cluster-operator:PR-1189-154dc53b


H6159, H6008, H6003 BLE Support

Thank you, I also have H6159, I tried other similar models but they are not compatible, please add support

I've got two H6008s but am unsure how to provide the correct information.

I ended up chopping the cable and replacing the controller with a Zigbee one, had enough !

If anyone has any information on how to extract or construct these JSON files, I would love to take a shot at adding the H6008. I have several of them and would love to remove them from the cloud and control them locally.

@treysullivent I have already done some of the reverse engineering work for h6159 and h6008 please check here to see how the Bluetooth packets are done https://github.com/Fefedu973/Universal-Govee-Bluetooth-control

I own both H6006 and H6008. I'm reading the JSON files and I only see scenes and special effects, nothing bluetooth protocol related. I do wonder if adding the empty file with no scenes would add preliminary support for these. I am otherwise going to return the H6008 and just use the 6006.

I was able to look at the Universal Govee Bluetooth Control code and some Bluetooth captures of my lights. The color control in that project is correct when the light is in RGB mode.
However, in the Govee application, when you are choosing white colors, you get to pick a color temperature between the 2700K and 6500K range*. When you do that, there are 5 additional bytes that are sent along with an initial RGB value, and these values are neither sent by the Universal Govee Bluetooth code nor this integration.
I think this is why choosing warm white/cold white with the H6006 can result in a lower brightness than if using the Govee app - the additional 5 bytes are dropped, and these seem to be the bytes that tell the (presumably CW/WW) LEDs in the bulb to do something.
Here's an example of me playing with the lights, in case somebody smarter than me can figure out what the meaning of the 5 bytes are.
In my testing, the btatt.value starts with 0x33 0x05 0x0d as documented, and the light sends a brightness packet right before send the color one. Following the 0x0d bytes, here's the rest of the packets ([...] indicates N zero bytes):
['red', 'green', 'blue', UKN1, UKN2, Red2,Gren2,Blue2, [...], 'checksum'],
[ 0xFF,    0xAE,   0x54, 0x0A, 0x8C, 0xFF, 0xAE, 0x54, [...], 0xBD], # 0xFFAE54 # WW (10, 140)
[ 0xFF,    0xF9,   0xFB, 0x19, 0x64, 0xFF, 0xF9, 0xFB, [...], 0x46], # 0xFFF9FB # CW (25, 100)
[ 0x88,    0xFF,   0xC4, 0x00, 0x00, 0x00, 0x00, 0x00, [...], 0x88], # 0x88FFC4 # Green
-------------------------------------------------------------------
At this point I'm playing with illumination color but don't know which command is which.
-------------------------------------------------------------------
[ 0xFF,    0xFF,   0xFF, 0x19, 0x64, 0xFF, 0xF9, 0xFB, [...], 0x44], # ??
[ 0xFF,    0xFF,   0xFF, 0x0A, 0x8C, 0xFF, 0xAE, 0x54, [...], 0x47], # ??
[ 0xFF,    0xFF,   0xFF, 0x17, 0x70, 0xFF, 0xF4, 0xED, [...], 0x45], # ??
[ 0xFF,    0xFF,   0xFF, 0x0B, 0x54, 0xFF, 0xB6, 0x62, [...], 0xB0], # ??

Above, I have named them UKN1, UKN2, Red2, Gren2 and Blue2. I suspect the two unknown values have to do with warm white and cold white control, and maybe the last bytes are the RGB light settings but in color temp mode.
*Range according to the Govee application

The two "mystery bytes" are simply the color temp in hex facepalm.
It seems the RGB value is also sent along, I am not sure if this drives the LEDs or if it's complementary.
Cold White = (hex) 1964 fff9fb
Warm White = (hex) 0a8c ffae54

I was able to look at the Universal Govee Bluetooth Control code and some Bluetooth captures of my lights. The color control in that project is correct when the light is in RGB mode.
However, in the Govee application, when you are choosing white colors, you get to pick a color temperature between the 2700K and 6500K range*. When you do that, there are 5 additional bytes that are sent along with an initial RGB value, and these values are neither sent by the Universal Govee Bluetooth code nor this integration.
...

@gparent  This is a great start! Especially now that you've identified some piece of data that is not being sent by those integrations that seem to have trouble controlling the H6008.
Based on a somewhat low understanding of the different configurations of LEDs in the bulb, it sounds like the code is setup for a RGB bulb when instead we should have it set up as an RGBCW light. Now the question is, what is the simplest way to send along those extra bytes using this existing communication code.
I am new to any kind of BT or BLE sniffing, but I can't imagine it is so different than any kind of network communication. I don't have a spare Android, but I do have some ESP32 devices that might be able to help me.

I was able to look at the Universal Govee Bluetooth Control code and some Bluetooth captures of my lights. The color control in that project is correct when the light is in RGB mode.
However, in the Govee application, when you are choosing white colors, you get to pick a color temperature between the 2700K and 6500K range*. When you do that, there are 5 additional bytes that are sent along with an initial RGB value, and these values are neither sent by the Universal Govee Bluetooth code nor this integration.
...

@gparent This is a great start! Especially now that you've identified some piece of data that is not being sent by those integrations that seem to have trouble controlling the H6008.
Based on a somewhat low understanding of the different configurations of LEDs in the bulb, it sounds like the code is setup for a RGB bulb when instead we should have it set up as an RGBCW light. Now the question is, what is the simplest way to send along those extra bytes using this existing communication code.
I am new to any kind of BT or BLE sniffing, but I can't imagine it is so different than any kind of network communication. I don't have a spare Android, but I do have some ESP32 devices that might be able to help me.

You're in for a treat today.
https://github.com/Beshelmek/govee_ble_lights/compare/master...gparent:govee_ble_lights:master
This branch enables color temp control for H6006 and H6008. It probably has lingering bugs, but it's a start.

@gparent Are you just converting temp to RGB? Typically I think these lights actually have different LEDs inside for RBG and Cool/Warm, so even if this looks really good, it may not be fully utilizing the bulb's features.
It really seems like just sending dummy bytes for RBG and then send the proper values for CW would be the solution, specifically based on what you discovered with the BT communication.

What I'm sending matches what the application sends for H6006/H6008. It seems to utilize the different bytes unlike this repo's master branch.
I'll update the code further, but it will break making my own personal branch and dropping support for everything I don't personally own. So I moved my original attempt in the H6006-stock branch.

@gparent I just installed this in Home Assistant using HACS, and it is not exposing any entities for each device. Is there some alternate way you are controlling the bulbs?

No, however I am using an ESP32 BLE proxy and I have not touched the API implementation, only BLE.

master...gparent:govee_ble_lights:H6006-stock
This branch enables color temp control for H6006 and H6008. It probably has lingering bugs, but it's a start.
I need to check the licensing for that color temp conversion code and see if it can be replaced by an HA function instead.

Hey THis is fantastic. I'm having trouble installing the branch with the 6008 option. Any thoughts on the process of doing so?
Also thank you for all the work you've out into this project.


Bed Respawn doesnt save & all armor doesnt take damage 1.20.2

Setting a respawn point works up until after you die, having to reset respawn point after every death
and all armor doesn't take any durability damage, this affects both SP and MP

This should now be fixed in Origins 1.12.0


Add the ability to differentiate tests with their class name in Zebrunner

Basically, our problem is that whenever we have an abstract test class (for example, if for 10 different test classes we need to test X, we create an abstract test class that tests X and our 10 classes will inherit from it), it is shown as the same test in Zebrunner.
What this means is that it will log each and every test that has the same name as the same row in Zebrunner, which is not really the desired behavior. Zebrunner needs to have the ability to know that if it's a different class that tests this method, it should not be considered as exactly the same test.

closing as won't fix. that's might be an enhancement for reporting.
generally, there are a lot of instruments to customize naming on test level: https://zebrunner.com/documentation/reporting/testng


Consider to upgrade Docker-Bind to latest version (0.17)

ISSUE TYPE

Feature request

SUMMARY

While working on Auto DNS feature to try to solve issue #604, I've found that :

the main cytopia/bind image used in docker-compose.yml still use an outdated version 0.15  ; see https://github.com/cytopia/devilbox/blob/v1.7.2/docker-compose.yml#L37

I've also noticed that feature that came with version 0.17, has no env variable defined in env-example file
See https://github.com/cytopia/docker-bind#optional-environmental-variables

ALLOW_QUERY
ALLOW_RECURSION

Goal

Raise cytopia/bind either to 0.17 or latest tag available on docker hub => https://hub.docker.com/r/cytopia/bind/tags

Defines two new env variables in env-example under section 10. Bind Docker Settings

BIND_ALLOW_QUERY
BIND_ALLOW_RECURSION
And modify the bind service in docker-compose.yml => https://github.com/cytopia/devilbox/blob/v1.7.2/docker-compose.yml#L36
With version 0.15 I got this output with docker-compose logs bind command
bind_1   | [INFO] Debug level: 2
bind_1   | [INFO] BIND logging: disabled explicitly
bind_1   | [INFO] Using default DNS TTL time: 3600 sec
bind_1   | [INFO] Using default DNS Refresh time: 1200 sec
bind_1   | [INFO] Using default DNS Retry time: 180 sec
bind_1   | [INFO] Using default DNS Expiry time: 1209600 sec
bind_1   | [INFO] Using default DNS Max Cache time: 10800 sec
bind_1   | [INFO] Adding wildcard DNS: *.local -> <IP_ADDRESS>
bind_1   |
bind_1   | zone local/IN: loaded serial<PHONE_NUMBER>
bind_1   | OK
bind_1   | [INFO] Not adding any extra hosts
bind_1   | [INFO] DNSSEC Validation: no
bind_1   | [INFO] Adding custom DNS forwarder: <IP_ADDRESS>,<IP_ADDRESS>
bind_1   | [INFO] Starting BIND 9.11.3

And with new version 0.17, I got this output
bind_1   | [INFO] Debug level: 2
bind_1   | [INFO] BIND logging: disabled explicitly
bind_1   | [INFO] Using default DNS TTL time: 3600 sec
bind_1   | [INFO] Using default DNS Refresh time: 1200 sec
bind_1   | [INFO] Using default DNS Retry time: 180 sec
bind_1   | [INFO] Using default DNS Expiry time: 1209600 sec
bind_1   | [INFO] Using default DNS Max Cache time: 10800 sec
bind_1   | [INFO] Adding wildcard DNS: *.local -> <IP_ADDRESS>
bind_1   |
bind_1   | zone local/IN: loaded serial<PHONE_NUMBER>
bind_1   | OK
bind_1   | [INFO] Not adding any extra hosts
bind_1   | [INFO] $ALLOW_QUERY not set.
bind_1   | [INFO] DNS query rules will not be set
bind_1   | [INFO] $ALLOW_RECURSION not set.
bind_1   | [INFO] DNS recursion rules will not be set
bind_1   | [INFO] DNSSEC Validation: no
bind_1   | [INFO] Adding custom DNS forwarder: <IP_ADDRESS>,<IP_ADDRESS>
bind_1   | [INFO] Starting BIND 9.11.5

Friendly ping @cytopia

@llaville how does the new Bind versoin resolve #604?

@cytopia It does not solve #604.
For WSL 2 and AutoDNS I've solved part of problem, but I got CORS issues when I visit vhosts intranet devilbox page. I'll keep continue to search but my answer will be publish on 604 bug report

My docker-compose.overwrite.yml contains at least
version: '2.3'

services:

  bind:
    image: cytopia/bind:0.17
    environment:
      - ALLOW_QUERY=${BIND_ALLOW_QUERY:-any}
      - ALLOW_RECURSION=${BIND_ALLOW_RECURSION:-localnets,localhost}

FYI: Currently working on a separation of the bind container and autodns (so that bind would not be required at all, but only if you want to have auto-dns). It's also related to this PR: https://github.com/cytopia/devilbox/pull/411
Will keep you posted.

Thanks a lot !


Error initializing fasterrcnn_resnet50_fpn with num_classes and pretrained

🐛 Bug
In torchvision.models.detection.fasterrcnn_resent50_fpn there are two parameters, num_classes and pretrained.
If I pass just one of them, I am able to initialize the model
model = torchvision.models.detection.fasterrcnn_resnet50_fpn(num_classes=6)

or
model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained = True)

^this obv won't work for me since I have just 5 classes and not 91.
hence, when I pass both, I get this error
code:
model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained = True, num_classes=6)

error:
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
/var/folders/5q/5dh5_83x359fqcb949hhtj8m0000gp/T/ipykernel_11119/3787706091.py in <module>
----> 1 model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained = True, num_classes=6)
      2 # num_classes = 6 # should be initialized as target_col.nunique + 1
      3 # in_features = model.roi_heads.box_predictor.cls_score.in_features
      4 # model.roi_heads.box_predictor = FastRCNNPredictor(in_features, num_classes)

    370         state_dict = load_state_dict_from_url(model_urls['fasterrcnn_resnet50_fpn_coco'],
    371                                               progress=progress)
--> 372         model.load_state_dict(state_dict)
    373         overwrite_eps(model, 0.0)
    374     return model

   1404
   1405         if len(error_msgs) > 0:
-> 1406             raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
   1407                                self.__class__.__name__, "\n\t".join(error_msgs)))
   1408         return _IncompatibleKeys(missing_keys, unexpected_keys)

RuntimeError: Error(s) in loading state_dict for FasterRCNN:
	size mismatch for roi_heads.box_predictor.cls_score.weight: copying a param with shape torch.Size([91, 1024]) from checkpoint, the shape in current model is torch.Size([6, 1024]).
	size mismatch for roi_heads.box_predictor.cls_score.bias: copying a param with shape torch.Size([91]) from checkpoint, the shape in current model is torch.Size([6]).
	size mismatch for roi_heads.box_predictor.bbox_pred.weight: copying a param with shape torch.Size([364, 1024]) from checkpoint, the shape in current model is torch.Size([24, 1024]).
	size mismatch for roi_heads.box_predictor.bbox_pred.bias: copying a param with shape torch.Size([364]) from checkpoint, the shape in current model is torch.Size([24]).

To Reproduce
Steps to reproduce the behavior:
import torchvision

model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained = True, num_classes=6)

Expected behavior
It should not throw an error.
Environment

PyTorch / torchvision Version (e.g., 1.0 / 0.4.0):

torch                             1.9.0
torchvision                       0.10.0

OS (e.g., Linux):

MacOS

How you installed PyTorch / torchvision (conda, pip, source):

pip

Build command you used (if compiling from source):

None

Python version:

python3 --version
>Python 3.8.5

CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

I think that the pretrained model is pretrained for a specific number of classes, which is why we cannot set both. But it would be nice to have a better error message.

yes or you can change the final if statement to if pretrained and num_classes = 91: only then load the state dict

All pre-trained models in TorchVision have a fixed number of classes that depends on the dataset that we used to train the model. If you want to define your own number of classes, you could load the pretrained model, do model surgery to overwrite those layers that make the class prediction and then fine-tune the model on your own data.
I'm closing the ticket as I believe addresses the question but feel free to reopen if you have more concerns.

I had kept it open so as to not forget about the fact that we should raise a better error message, as mentioned above. It's fine to close though, I'll follow up on that soon(ish)

It's definitely worth addressing it on the future but note it might be tricky to cover all corner-cases/parameters and return the right error message for each.
There are models that offer quite generic configuration and in theory someone can pass incompatible config for a specific pre-trained model. This is likely to be addressed more naturally when a proper versioning mechanism is introduced. There are ideas to link specific model configs to specific weights/versions but this is still under investigation.

I agree it might be tricky to fully validate the kwargs, but for the non-kwargs argument it should be fairly straightforward, even if a bit verbose


Add ObfuscatePipe to tibco module

Hi @alisihab, did you also add this PR to the master already? Usually we change master first and after approval, we create a patch for versions < master.

I was asked to share the file only


Remove websockets package

The installation of the 'websockets' package is no longer required since the websockets package is part of the uvicorn[standard] installation.
In fact, I had some problems when I installed this package manually. Specifically, a SecurityError 'line too long' was thrown during the websocket handshake when there were too many cookies on the page. Removing the installation of the package fixed this.
That being said, it is a little confusing to install a package in a tutorial and not use it. I think without this it is clearer and easier.

Thanks for the interest! But this would only apply if people installed uvicorn[standard], if they didn't they would need to ensure they have it. :thinking:
If you had problems installing it, it would be better to create a new discussion with your use case so we can check it.
For now, I'll pass on this one, but thanks for the effort!

Yeah, but uvicorn[standard] is a dependecy of fastapi[standard], so if fastapi is installed (which you assume in the tutorial), uvicorn and thereby websockets are installed. So I don't see the need for the explicit install, indeed I find it more confusing than helpful.


7.1.x TS-5107: Changes to autolinking in Docutils 0.13

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1396/ for details.

Why are there 4 commits on this? Can you squash those?

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1290/ for details.

Also, please remember to add Milestone, Label's etc. on each Issue / PR, otherwise @bryancall  gets all pissy. :)

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1398/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1292/ for details.

Also, please remember to add Milestone, Label's etc. on each Issue / PR, otherwise @bryancall gets all pissy. :)

Will do! :-)

Why are there 4 commits on this? Can you squash those? Or do we actually need these reverts as well?

Make that 5 ... I just added #1377 to this PR because this is broken without it. Whoops.
I think we need the reverts, this doesn't apply without them, and I think it makes most sense to maintain the parallel between the commits we're backporting and master -- so apply them as individual commits. That said, I'm happy to do it differently! I'm happy to squash any combination of them.

I saw what happened, the reverts makes sense now.


Feature Request: Add XmlWriter .NET Wrapper

We have performance critical code that is currently using XmlWriter by C# code injected into C/Side meta data and using reflection to add refferences. We would like to translate this to a more sane model, but the XDocument (that the current wrappers use?) are not performant as in CPU and memory as the XmlWriter.
XDocument  ( 5000 times   1k) : 15.561 sec
XML Writer ( 5000 times   1k) : 8.795 sec
XDocument  (    5 times 200k) : 8.046 sec mem 755 mb
XML Writer (    5 times 200k) : 2.269 sec mem 67 mb

Numbers shows that XmlWriter is over 3 times as fast when using larger object and consumes less than 10 times the memory. These numbers increase the larger the document. Main difference is that XmlWriter writes directly/forward only (no DOM) to a stream.
Thank you

We are about to write a large project that emits large chunks of XML data - any status on this?

We also have a lot of performance issues due to XML Writer not being available. Please, can someone look into this?
Also, the Xml Writer code in C/AL seems to match with the Xml Writer in C#. Expecting AL will have the exact same functions as C#, it will be a lot easier to maintain C/AL and AL at the same time.

hi @angela2389 and co. - please vote here: https://experience.dynamics.com/ideas/idea/?ideaid=f07b7746-6c80-e911-80e7-0003ff68bdb5 - thanx

Hi! This issue is beyond the scope of the AL language and development tools. Please go ahead and post this to our Ideas forum at https://aka.ms/BusinessCentralideas, or vote up the idea if its already there. We're constantly monitoring top Ideas and will consider them for a future release.


Continuous animation of Polygon2D points no longer works. (Animation Player)

Godot version
v4.2 beta2 [f8818f85e]
System information
Windows 11 - v4.2.beta2.official - Compatibility
Issue description
In version 4.2 dev3, I could animate the points of a Polygon 2D in the animation player using the continuous update mode. This no longer works the way it used to. The polygon will remain on the initial frame, no matter how many keyframes are added. I've noticed that when setting the update mode to discrete, the points will still animate but return to the initial shape if no keyframes are present at the current spot in the timeline.
Steps to reproduce

Create a Polygon2D and an Animation player.
Animate two keyframes with different Polygon2D points.
Set the animation update mode to continuous.
See Issue.

Minimal reproduction project
polygon_2d_animation_player_issue.zip

Can confirm issue. Also discovered this when animating a polygon, but the same thing happens when trying to animate any PackedVector2Array.
In version 4.1.1 the animation was played in the continuous mode, but only with linear interpolation. Cubic was broken completely:  it made polygon invisible and removed array data.

May be fixed by https://github.com/godotengine/godot-proposals/issues/8085.


Bug: Defining schema shape causes a TS error when it shouldn't

Description
When defining the schema shape using a type (const Schema: z.ZodType<MyType> = ...) Typescript is raises an error when using a schema and the pipe method.
I believe it to be a bug as the correct type is inferred from the schema.
Example
Zod Version 3.23.8
import { z } from "zod";

export const StringListSchema = z.string().transform((input) => input?.split(","));

export type InputType = {
  strings?: string[];
};

const Schema: z.ZodType<InputType> = z.object({
  strings: StringListSchema.pipe(z.array(z.string())).optional(),
});
// Type Error: Type 'string | undefined' is not assignable to type 'string[] | undefined'.

export type OutputType = z.infer<typeof Schema>;
// Infered Type: type OutputType = { strings?: string[] | undefined; }

console.log(Schema.parse({ strings: "one,two" }));
// Output: { strings: [ 'one', 'two' ] }

My apologies, I had not noticed that the inferred input type was in fact the issue, and the bug was due to my misunderstanding.
If anybody comes across this it is because the input component  of z.ZodType is inferred from the output component of the generic.


Add support for Authenticator plugins

We have an internal authenticator implementing the Authenticator interface. That authenticator will integrate with various internal libraries and currently uses various information from the HttpServletRequest. There is a PasswordAuthenticator but the interface only passes in a user and password, whereas we need other information from the HttpServletRequest. Specifically we are using:
-headers (AUTHORIZATION and X_FORWARDED_FOR)
-remote address
We want to move this internal authenticator out of the Presto code into its own plugin, so we want to see if we can create a mechanism to load Authenticators into Presto. What would be an appropriate interface for this?

Do you need the ability to send back a WWW-Authenticate header on failure?

yes

Duplicate of #2117


Move "show results folder" button

From #42 (@bertfrees )
The "Show results folder" button is spatially separated from the "Results" box with the links to all the individual results inside. That is not super intuitive I think. It's good that the "Show results folder" gets a prominent place, I would maybe move the "Results" box closer to it, or even integrate them with each other somehow.

I disagree, I think it's intuitive how it is but this change may happen anyway as a result of #56 and #57

Fixed by b0301a43b72c4317f87e55b0f1958a70f2b623f9


moved replicaset_test.go to use libvmi

What this PR does
Before this PR:
the should replace a VMI immediately when a virt-launcher pod gets deleted in
replicaset_test.go had a little bit of code not using libvmi
After this PR:
the should replace a VMI immediately when a virt-launcher pod gets deleted in
replicaset_test.go uses only libvmi.

partially address #12059
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/approve
/test all

Thank you for your PR. Please update the PR description not using the word Fixes, otherwise that gh issue will be closed by this. Suggestion:
Partially address: https://github.com/kubevirt/kubevirt/issues/12059

/lgtm

/test pull-kubevirt-e2e-k8s-1.28-sig-compute

@KyriosGN0 The PR has lgtm and approve, the kubevirt-bot will retest this automatically to get it merged :)


Allow running code before main

I found I needed support for running code at startup recently when I found that the .data region in one project grew large enough that initializing it was taking long enough that the default hardware watchdog was triggering a reset. Because of this usecase r0::run_init_array is placed before r0::zero_bss and r0::init_data in case either takes too long.
The only downside from the user's perspective of the current implementation is the requirement on users to add #![feature(used)] to their crate if they want to use the macro.
From the perspective of cortex-m-rt this does add another unstable feature in use_extern_macros (tracking issue). It appears that this feature will stay unstable until macros 2.0 lands. Users do not need to enable use_extern_macros and can use the macro either fromr0 or through cortex-m-rt via #[macro_use].

I have updated the pull request with a method of passing the Peripherals from cortex-m from init_array to main.
The main idea of this method passes the peripherals through a static. In addition this static is of a typeInitArrayPeripherals (I know, creative naming. Please bikeshed) which mirrors Peripherals, but each peripheral is contained in an Option. This allows peripherals to optionally not be passed from init_array. In addition, while the InitArrayPeripherals struct can be passed back in init_array it is not allowed to be passed back in main. Trying to do so simply drops the struct and all peripherals inside. A alternative to dropping the stuct would be to return it back on failure, but I do not see this as a valid usecase as it is clear where InitArrayPeripherals should be given back and where it should not.
This does not solve passing other structures, such as peripherals from board crates, from init_array. A similar method would have to be done for each crate.

Hello, I know you are a very busy man, but I was wondering if I could get some feedback on the current solution.

@yodaldevoid sorry for the long radio silence here.
I'm afraid that your proposed solution is not general enough. One of the use cases for this feature is configuring the clock to operate at higher speed before initializing the RAM with the goal of making the boot / pre-main process faster. Your solution doesn't solve that problem because configuring the clock requires access to device specific peripherals. I also think that the extensive use of Option and unwrap is unfortunate.
I'm currently leaning towards not solving the peripherals access problem in this crate, at least not right now. Instead we simply add the mechanism (e.g. .pre_init) to run custom code before zero_bss / init_data and leave it up to other crates to iterate and develop safe mechanisms to share peripherals between pre_init functions and main.
One such mechanism could be a macro that uses the unsafe Peripherals::steal under the hood. See below (jus a sketch off the top of my head, untested and likely contains errors):
/* user code */
entry!(stm32f103xx, before_main, main);

// NOTE unsafe because accessing `static` variables in this context is forbidden
// (a lint instead of unsafe would be ideal)
unsafe fn before_main(c: &mut cortex_m::Peripherals, d: &mut stm32f103xx::Peripherals) {
    // ..
}

fn main(c: cortex_m::Peripherals, d: stm32f103xx::Peripherals) -> ! {
    // ..
}
/* end of user code */

/* expansion of `entry!` */
#[link_section = ".pre_init"]
pub static PRE_INIT: unsafe extern "C" fn() = pre_init;

unsafe extern "C" fn pre_init() {
    before_main(&mut cortex_m::Peripherals::steal(), &mut stm32f103xx::Peripherals::steal());
}

#[export_name = "main"]
pub unsafe extern "C" fn __impl_main() -> ! {
    main(cortex_m::Peripherals::steal(), stm32f103xx::Peripherals::steal())
}
/* end of expansion of `entry!` */

/* cortex-m-rt */
#[no_mangle]
pub unsafe extern "C" fn Reset() -> ! {
    for f in PRE_INIT_ARRAY {
        f();
    }

    r0::zero_bss(..);
    r0::init_data(..);

    extern "C" {
        fn main() -> !;
    }

    main()
}

That solution sounds fine by me. I understand not wanting to add hacked together solutions to this crate and as I think I mentioned before I had had reservations about this solution anyway. As it sounds like you are leaning towards the other (simpler) solution, I will close this and wait with baited breath.


Unstyled Circle appears at bottom when searching

Duplicates

[X] I have searched the existing issues

Related page
https://mui.com/
Kind of an issue
Other
Issue description
While searching and when my internet is slow, the page scrolls me down to part as shown in the page:-

Context 🔦
What brought me to this page? Search

I have experienced this issue, from time to time. I agree that something is off with the way the style are loaded. Related to #16502.


[shared_preferences] Removed deprecated AsyncTask API

AsyncTask was deprecated in API level 30.
In this PR I have used Executor.
fixes flutter/flutter#71595
fixes flutter/flutter#74039
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
No breaking changes
Pre-launch Checklist

[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I signed the CLA.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@matthew-carroll Can review this please?

@matthew-carroll Can review this please?

@blasten This should be a quick review, do you want to take a look?

@cyanglaz @blasten null-safe version of this plugin is already merged, so should I make the version changes as 2.0.0-nullsafety.1 ?

@blasten is it ready to merge?

Hey @cyanglaz @blasten I changed the package version to 2.0.0 following this article on medium.
Please let me know if you want change version back to 2.0.0-nullsafety.2

@blasten @cyanglaz anyone of you can please review this?

I've resolved the conflicts I created with the latest NNBD migration; it should auto-land once the tests have had a chance to re-run.


Selection, cursor, backspace, full deletion broken for image/icon entities

Hello,
I have added two types of custom entities. One is a component of a font-icon, using the css pseudo selector ::before, and the second is a simple image.
When adding any one of them to the editor: selection, cursor, backspace, and select all & delete, are not working properly. The cursor jumps to weird locations, I can't use backspace to remove the entity, can't add text after the entity, and many other problems. Just see for yourself: Fiddle
Env:

Draft.js version 0.7.0 (Didn't try any other versions)
Windows 7
Google Chrome Version 51.0.2704.106 m (64-bit)

For you I'd try to use the AtomicBlockUtils is very much simple to add entities and stuff. If you don't want a block you can check de draft-js-plugins they have some stuff with pretty similar stuff.
Now the issue: draft-js has some problems when you force the focus from outside its container, breaking entirely the selection and leaving your component unusable. Maybe its related:
#538
#485
And a few more.

@gabriel-miranda thanks for the response. Using AtomicBlockUtils inserts a block on a separate line (I guess this can be hacked by using css), we need it inline. I have looked at the draft-js-plugins while developing my code (especially since it is hard to understand draft.js from the docs), but couldn't find a good solution to our problems.

Even inserting it as a Atomic Block doesn't make selecting, deleting or replacing work properly.

Thanks for reporting this, and for the demo. We would love to see fixes for this. For some reason it has not been an issue with our internal use of Draft.js, which includes some similar use cases. I wonder if there is a work-around and will look to see if we have something I can recommend.

Did anybody found solution to this problem. I am able to add entities to the draftjs but my backspace is broken. On hitting backspace entire entity is not removed and cursor delete any earlier emoji entity and not the current added one.


Add Hacker News

@premii's Hacker News mobile app
Site: https://hn.premii.com/
Github: https://github.com/premii/hn

I haven't investigated this thoroughly, but the web app install banner isn't triggered on this site... Any idea why?

AFAICT, it doesn't use a service worker, so won't show up an install banner. Doesn't work offline too.
I would vote a 'no' on this one in the current form.

I agree. @zcdunn happy to include it once these issues are fixed :) Closing PR for now.


New task registration failed

I have defined a new task, and the entire data collection for 'Defining New MBO Tasks' ran successfully. However, when I used design_bench.make to set up the new task, it failed to download the data.
The error message displayed is: 'ValueError: No registered task with the name: NEW-TASK-v0'.
I'm wondering why the data collection ran smoothly, but the new task seems to fail during registration with design_bench. Could you help me fix this? Also, could you please explain how to verify if the task was successfully registered?

Hi @ksr157
All of the tasks are actually registered in design_bench/__init__.py, I suppose you could use the register() function in design_bench.registration but I feel like it would be easier to just instantiate the task manually. I'm not sure if there's functionally a difference between using design_bench.make or not in terms of whether it's just syntactic sugar or if it's needed to fulfil a specific purpose (I've never implemented a new task ever in this lib*).
(* n.b: take everything I say with a grain of salt, I'm just a maintainer of the lib, not the originating author, so I'm ccing @brandontrabucco here)


Flutter dosen't work on Meizu 7 Pro plus

While running the flutter app on my Meizu 7 Pro plus, got the error msg:
Launching lib\main.dart on PRO 7 Plus in debug mode...
Initializing gradle...
Resolving dependencies...
Running Gradle task 'assembleDebug'...
Built build\app\outputs\apk\debug\app-debug.apk.
Installing build\app\outputs\apk\app.apk...
Error: ADB exited with exit code 1
adb: failed to install C:\Users\Yoav\AndroidStudioProjects\event_chat\build\app\outputs\apk\app.apk: Failure [INSTALL_FAILED_USER_RESTRICTED]
Error launching application on PRO 7 Plus.
is there a way to fix that?
Thanks,
Yoav.

Resolved:
Disabled all performance optimizations in the debug setting, much like:
https://stackoverflow.com/questions/47239251/install-failed-user-restricted-android-studio-using-redmi-4-device


Fix error in OrderBook entity caused by class loading order bug in Grape

When trying to access the API endpoint for order books http://peatio:8000/api/v2/order_book?market=usdbtc I get the following error:
undefined method `represent' for #<Class:0x00007fc2cd8c8330>

Tracked it down to this Github issue: https://github.com/ruby-grape/grape-entity/issues/80
Fixing the class loading order by requiring the Order entity seems to fix the bug.

It is due to the same constant name at app/models/order.rb

which branch are you working on @kriskelly?

@mod This is very easy to port on 1.5+

@mod I was working on master


Bump digitalmarketplace-frameworks to 18.3.0

Pick up the new employment status question added in https://github.com/alphagov/digitalmarketplace-frameworks/pull/685

https://trello.com/c/l9lnd1GD/14-2-add-new-ir35-question-to-the-create-an-opportunity-journey

Will need to be released AFTER https://github.com/alphagov/digitalmarketplace-api/pull/1133


Unexpected response status code [OK] for [CreateOrderAsync]

I use Digicert ACME API, I want to test win-acme in few scenarios. I am trying to renew a certificate that's valid for 10 months, and I get this:
> wacs.exe --renew --baseuri "https://acme.digicert.com/v2/acme/directory/nOTaReaLUrI-ItsjUst-anExample" --for
ce --verbose
 [VERB] Verbose mode logging enabled
 [VERB] Looking for settings.json in D:\Work\win-acme
 [DBUG] Config folder: C:\ProgramData\win-acme\acme.digicert.comv2acmedirectorynOTaReaLUrI-ItsjUst-anExample
 [DBUG] Log path: C:\ProgramData\win-acme\acme.digicert.comv2acmedirectorynOTaReaLUrI-ItsjUst-anExample\Log
 [DBUG] Cache path: C:\ProgramData\win-acme\acme.digicert.comv2acmedirectorynOTaReaLUrI-ItsjUst-anExample\Certificates
 [VERB] Arguments: --renew --baseuri https://acme.digicert.com/v2/acme/directory/nOTaReaLUrI-ItsjUst-anExample --force --verbose
 [DBUG] Renewal period: 90 days

 [INFO] A simple Windows ACMEv2 client (WACS)
 [INFO] Software version <IP_ADDRESS>6 (RELEASE, PLUGGABLE)
 [INFO] ACME server https://acme.digicert.com/v2/acme/directory/nOTaReaLUrI-ItsjUst-anExample
 [VERB] SecurityProtocol setting: SystemDefault
 [DBUG] Connection OK!
 [INFO] IIS version 8.5
 [INFO] Running with administrator credentials
 [INFO] Scheduled task looks healthy
 [INFO] Please report issues at https://github.com/win-acme/win-acme
 [VERB] Test for international support: Φ¬₧Φ¿Ç ╤Å╨╖╤ï╨║ ┘ä╪║╪⌐
 [VERB] Checking renewals
 [VERB] Sending e-mails False
 [DBUG] Scanning IIS site bindings for hosts
 [VERB] 1 named bindings found in IIS
 [VERB] No site filter applied
 [VERB] No host filter applied
 [VERB] 1 matching binding found
 [VERB] Adding <IP_ADDRESS> as DNS server
 [VERB] Adding <IP_ADDRESS> as DNS server
 [VERB] Adding <IP_ADDRESS> as DNS server
 [INFO] Force renewing certificate for [IIS] (any site), (any host)
 [DBUG] Refreshing order...
 [VERB] Loading ACME account signer...
 [DBUG] Loading signer from C:\ProgramData\win-acme\acme.digicert.comv2acmedirectorynOTaReaLUrI-ItsjUst-anExample\Signer_v2
 [VERB] Constructing ACME protocol client...
 [DBUG] Send GET request to https://acme.digicert.com/v2/acme/directory/directory
 [VERB] Request completed with status BadRequest
 [DBUG] Send GET request to https://acme.digicert.com/v2/acme/directory/nOTaReaLUrI-ItsjUst-anExample
 [VERB] Request completed with status OK
 [DBUG] Send HEAD request to https://acme.digicert.com/v2/acme/new-nonce/nOTaReaLUrI-ItsjUst-anExample
 [VERB] Request completed with status NoContent
 [DBUG] Loading account information from C:\ProgramData\win-acme\acme.digicert.comv2acmedirectorynOTaReaLUrI-ItsjUst-anExample\Registration_v2
 [DBUG] Send GET request to https://acme.digicert.com/v2/acme/order/eXampleOrder_T0KeN/7-ak8Qu18wb23gymoe0-K_9RjRDdlRe2EENa7Vh3hHk
 [VERB] Request completed with status OK
 [DBUG] Cached order has status invalid, discarding
 [VERB] Creating order for hosts: ["test.digicert-acme.example.com"]
 [DBUG] Send POST request to https://acme.digicert.com/v2/acme/new-order/nOTaReaLUrI-ItsjUst-anExample
 [VERB] Request completed with status OK
 [EROR] (AcmeProtocolException): Unexpected response status code [OK] for [CreateOrderAsync]
 [DBUG] Exception details: AcmeProtocolException {ProblemType=Unknown, ProblemTypeRaw=null, ProblemDetail=null, ProblemStatus=-1, TargetSite=Void MoveNext(), St
ackTrace="   at ACMESharp.Protocol.AcmeProtocolClient.SendAcmeAsync(Uri uri, HttpMethod method, Object message, HttpStatusCode[] expectedStatuses, Boolean skipN
once, Boolean skipSigning, Boolean includePublicKey, CancellationToken cancel, String opName)\r\n   at ACMESharp.Protocol.AcmeProtocolClient.CreateOrderAsync(IE
numerable`1 dnsIdentifiers, Nullable`1 notBefore, Nullable`1 notAfter, CancellationToken cancel)\r\n   at PKISharp.WACS.Clients.Acme.AcmeClient.Retry[T](Func`1
executor, Int32 attempt)\r\n   at PKISharp.WACS.Clients.Acme.AcmeClient.Retry[T](Func`1 executor, Int32 attempt)\r\n   at PKISharp.WACS.Clients.Acme.AcmeClient.
CreateOrder(IEnumerable`1 identifiers)\r\n   at PKISharp.WACS.Clients.Acme.OrderManager.CreateOrder(IEnumerable`1 identifiers, String cacheKey)\r\n   at PKIShar
p.WACS.Clients.Acme.OrderManager.GetOrCreate(Renewal renewal, Target target, RunLevel runLevel)\r\n   at PKISharp.WACS.RenewalExecutor.Execute(Renewal renewal,
RunLevel runLevel)\r\n   at PKISharp.WACS.RenewalManager.ProcessRenewal(Renewal renewal, RunLevel runLevel)", Message="Unexpected response status code [OK] for
[CreateOrderAsync]", Data=[], InnerException=null, HelpLink=null, Source="ACMESharp", HResult=-2146233088}
 [EROR] Renewal for [IIS] (any site), (any host) failed, will retry on next run
 [VERB] Exiting with status code -2146233088

Is this a bug, or something on my side (or Digicert's)?

It's complicated to lay blame. The ACME RFC states - in at least two seperate sections - that the status code for the HTTP response to a newOrder request is 201, but it stops short of saying that it MUST be 201. The ACMESharpCore library that win-acme uses only accepts 201, because Let's Encrypt and others use 201. Digicert uses 200. So it technically not wrong what they're doing, but it's definitely a bit out of line.
Should be fixed in this build though: https://ci.appveyor.com/project/WouterTinus/win-acme-s8t9q/builds/33394405/artifacts

Thanks, one step closer :-)


[sozo] add a call command

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I want to test some view function using sozo while I need to switch to starkli.
Describe the solution you'd like
A clear and concise description of what you want to happen.
add a sozo subcommand to call the function
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I'm on it 👍


Add logging middleware

Closes #20

@jakenichols2719 requesting re-review. Log entries now include a timestamp and the request's origin (or an appropriate flag value.) If users wish to read their logs in a Go program or bring their own tool, ParseLog is provided for processing logs or providing documentation respectively.


Update tile.mdx

Adding text as part of an online demo. Including information that tiles and tables are often options for the user.

Changes

Testing

Docs

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


Update and Fix Wiki

Description
Fix and update wiki links and content
Changes

Fix some error related to issue #459

Related Issues
Fixes #459 #wiki
Checklist

[x] Pre-commit hook is installed to do formatting checks before committing code...etc
[ ] Tests included
[x] Build pass
[x] Documentation
[x] Code format and docstrings

Thanks :)

Thanks :)


fix(versions): update Activiti/activiti-cloud-runtime-bundle-service versions into develop

UpdateBot pushed maven dependency: org.activiti.dependencies:activiti-dependencies to: 7.1.42

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.42 org.activiti.api:activiti-api-dependencies 7.1.31 org.activiti.core.common:activiti-core-common-dependencies 7.1.32 org.activiti.build:activiti-parent 7.1.14

Solved here: https://github.com/Activiti/activiti-cloud-runtime-bundle-service/pull/337


[WIP] DataFrame.count

This patch implements DataFrame.count based on Pandas' syntax. There are a few things that I don't yet know what we should do:

Should count return a ks.Series, or pd.Series?

What do we do with Inf values?

Where can I copy test cases from?

Resolves #184

Ok that makes sense. I will add this to design principles too.
Where should I take the test cases from?
On Fri, Apr 26, 2019 at 12:42 AM Timothy Hunter<EMAIL_ADDRESS>wrote:

@thunterdb commented on this pull request.
In databricks/koalas/frame.py
https://github.com/databricks/koalas/pull/188#discussion_r278838388:

@@ -497,9 +497,7 @@ def count(self):

     Returns
     -------

   Series or DataFrame

       For each column/row the number of non-NA/null entries.

       If `level` is specified returns a `DataFrame`.

   pandas.Series

We should return a pandas series in my opinion. Here is the reasoning:

the series is guaranteed to be 'small'
it is the output of an aggregation. if it was spark, we would return
a Row object
koalas series are supposed to be for 'big' data and have different
performance expectations when doing a point lookup. It should be instant
and not trigger a spark job.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/databricks/koalas/pull/188#discussion_r278838388, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACO6PB764Y2XVFIG2S4YOLPSKW6RANCNFSM4HITADRA
.

Here are a couple:
https://github.com/pandas-dev/pandas/blob/master/pandas/tests/frame/test_analytics.py#L1322
https://github.com/pandas-dev/pandas/blob/master/pandas/tests/series/test_analytics.py#L434
I could not find tests in dask, but I did not look long.

PTAL

Going to merge this so I can submit another PR. @thunterdb if you still think we should remove that reflection, we can work on it later.


Add .editorconfig with tab indent_style

This should define clearly what the preferred indent style is for the project, and make it easier for (most) editors to adjust automatically.

Is indent_size = 4 optional? As the indent_style is set to tab there is no need to forcefully overwrite the user's setting for that. 🙂

@tzellman Thanks!


MF-1506 - Move Groups back to domain services

Signed-off-by: dusanb94<EMAIL_ADDRESS>What does this do?
This pull request makes domain services (Users and Things) use the internal Groups package for entity groups manipulation.
Which issue(s) does this PR fix/relate to?
This pull request resolves #1512, resolves #1513, and closes #1506, closes #649.
List any changes that modify/break current functionality
Groups are moved to domain services, which changes internal architecture when it comes to entity groups, also moving the corresponding API to the respective services.
Have you included tests for your changes?
Yes.
Did you document any new/modified functionality?
Yes.
Notes
N/A

Codecov Report

Merging #1517 (d7af657) into policies (4f7d583) will increase coverage by 0.02%.
The diff coverage is 35.29%.

@@             Coverage Diff              @@
##           policies    #1517      +/-   ##
============================================
+ Coverage     64.84%   64.86%   +0.02%
============================================
  Files           136      136
  Lines         11330    11326       -4
============================================
  Hits           7347     7347
+ Misses         3388     3384       -4
  Partials        595      595

Impacted Files
Coverage Δ

auth/api/http/groups/endpoint.go
0.00% <0.00%> (ø)

auth/api/http/groups/requests.go
0.00% <0.00%> (ø)

auth/api/http/groups/transport.go
0.00% <0.00%> (ø)

internal/groups/api/requests.go
7.40% <0.00%> (ø)

pkg/sdk/go/groups.go
0.00% <0.00%> (ø)

pkg/sdk/go/sdk.go
100.00% <ø> (ø)

things/api/things/http/transport.go
85.30% <0.00%> (ø)

users/postgres/users.go
53.88% <ø> (ø)

users/service.go
63.97% <ø> (ø)

internal/groups/api/transport.go
45.37% <28.57%> (+0.41%)
:arrow_up:

... and 5 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 4f7d583...d7af657. Read the comment docs.

We changed the approach to access control and we'll soon integrate a new solution with corresponding docs. Closing due to irrelevance.


S113 only considers timeout= probable, unpacking is not recognised.

Rule S113 does not consider anything except a hard-coded request= to be "probable". This can be frustrating when one has a lot of configuration which is shared between many requests calls.
from typing import TypedDict
import requests

requests.post('test', **{"timeout": 1000})

class Options(TypedDict):
    timeout: int

options: Options = {"timeout": 1000}

requests.post('test', **options)

ruff --select S113 --isolated test.py

test.py:4:1: S113 Probable use of requests call without timeout
test.py:11:1: S113 Probable use of requests call without timeout
Found 2 errors.

Version: ruff 0.1.15

I can't share the actual code, but basically something like this.
class RequestOptions(TypedDict):
    headers: dict[str, str]
    timeout: int

class MyClass:
    def __init__(self) -> None:
        """Initialises settings."""
        self.options: RequestOptions = {
          'headers': {'Authorization': f'Bearer {env.TIMECHIMP_TOKEN}'},
          'timeout': 20000
        }

    def get(self, endpoint: str) -> requests.Response:
        # More function parameters and request customisation here, irrelevant.
        return requests.get(endpoint, **self.options)

    # A lot more similar functions here, that all use self.options.

This may not be what you need/want but I would say that what I am seen most people do is create a _internal_request or such method in the MyClass and then call that instead. this way you have all the requests going through 1 method with just different params. basically mimicing what requests does internally with get just proxying to requests('get', ...)


enhancement: ske-operator helm chart should wait for ske-operator to be ready to create ske deployment

Context
We don't have any logic to wait for ske-operator deployment become ready before create kratix deployment, which could lead to failure such as:
"syntasso" has been added to your repositories
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "syntasso" chart repository
Update Complete. ⎈Happy Helming!⎈
Error: INSTALLATION FAILED: failed to create resource: Internal error occurred: failed calling webhook "vkratix.kb.io": failed to call webhook: Post "https://ske-operator-webhook-service.kratix-platform-system.svc:443/validate-platform-syntasso-io-v1alpha1-kratix?timeout=10s": no endpoints available for service "ske-operator-webhook-service"

This happens rarely but it's not a nice experience. Perhaps we can leverage chart hooks to improve this: https://helm.sh/docs/topics/charts_hooks/

Could you use something like a HealthCheck - or something else?
From my experience - when using helm charts with Hooks, they don't work well when using them Kratix.


Wsl/Service/CreateInstance/ERROR_FILE_NOT_FOUND

Discussed in https://github.com/microsoft/WSL/discussions/8729

Originally posted by frankli-dev August 10, 2022
Version
Microsoft Windows [Version 10.0.22622.450]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
Ubuntu
Distro Version
Ubuntu
Other Software
None
Repro Steps

Install wsl default distro
Uninstall wsl
Reinstall wsl
Restart OS and open Ubuntu WSL
I get the following error

...
Wsl/Service/CreateInstance/ERROR_FILE_NOT_FOUND

Expected Behavior
I want to see running Ubuntu WSL
Actual Behavior
Wsl/Service/CreateInstance/ERROR_FILE_NOT_FOUND

Diagnostic Logs
None

Hello all,
The command wsl --unregister (version)
Will it remove all the old data ?
In my case, I reinstalled the opensusu tumbleweed software from Microsoft store and I got the same error.
Its a 3 month project data and we don't want to loose it.
We need to mount the old files safely and the opensusu software work fine.
Is there any way to take backup before following unregister command ?
Thanks,
Muthu Raman

@Muram1602 - yes, you can use --export to back up an existing distro, --unregister removes it.
/logs


Generate colmap data

I try to generate data like yours. But cannot. I only got 400-500 3D points in total instead of more than 2000. No matter by GUI or Command-Line mode. And it seems that you did not provide how to get the npy file from output.

10 images will get me more than 2000 depth points. I strongly need an explanation on how to get 2000 depth points by two images.

I even used the original resolution of two images. (40323024, yours is 1008756). The colmap output 900 3D points which is impossible to get 2000 depth information. I think you should clarify this problem as soon as possible. Or I think it's necessary to report this to the CVPR conference.

Hi, sorry for the confusion. We run COLMAP on images of the original resolution and downscale the cameras and depths when post-processing the data here and here.
Here is an example COLMAP run to get over 2000 keypoints on two views of the fern scene.
colmap feature_extractor --database_path database.db --image_path images  --SiftExtraction.max_image_size 4032 --SiftExtraction.max_num_features 32768 --SiftExtraction.estimate_affine_shape 1 --SiftExtraction.domain_size_pooling 1

colmap exhaustive_matcher --database_path database.db --SiftMatching.guided_matching 1 --SiftMatching.max_num_matches 65536

mkdir sparse

colmap mapper --database database.db --image_path images --output_path sparse --Mapper.ba_local_max_num_iterations 40 --Mapper.ba_local_max_refinements 3 --Mapper.ba_global_max_num_i
terations 100

Here is link to the COLMAP result I got with 2604 keypoints: https://drive.google.com/file/d/1YqozhVM63u9aN1r2ibW5Gqlm-i3iZad2/view?usp=sharing
Actually, I used to be able to get such many keypoints by using the default extreme quality setting of COLMAP, e.g. colmap automatic_reconstructor --image_path images --workspace_path ./ --sparse 1 --dense 0 --quality extreme. But colmap seems to have updated the settings. So now I have to specify the arguments for each stage. Actually, most of the settings are still the same as extreme quality here except that I increase the max number of features and image size. This setting might still not perfectly match what's in the dataset, but you can always further increase the max number of features and matches to get more keypoints.
Thanks for pointing this out. I'll update the document accordingly.

Yes, thanks for the clarification. The imgs2poses.py file indeed indicates the wrong command which will make people doubt the experiment result. I will try the auto-one with "--quality extreme", and your step-by-step command.

I've tried. Indeed the Colmap changed the default setting and your command could help to generate more than 2000 depth points. Thanks


Add ip masquerading option

Enable masquerading in GKE and the relative configuration options for the IP masquerade agent.
Reference:

Docs: https://cloud.google.com/kubernetes-engine/docs/how-to/ip-masquerade-agent
What is actually created in the GKE module: https://github.com/terraform-google-modules/terraform-google-kubernetes-engine/blob/master/modules/beta-private-cluster/masq.tf

Ciao @nikever!!
I've got mixing fillings with this PR:
Let me explain:

This is something that could be implemented in all the cloud providers.

EKS will require more work for sure as it uses a "rare" CNI by default, but it is possible.

As it is just "kubernetes stuff" (a daemon set + configmap) I don't like to include it in the installer.

It will open a big big big door that I don't consider is necessary, deploying Kubernetes stuff from terraform as our stack to deploy kubernetes things is kustomize + kubectl

Can you deploy it using the definition available here: https://raw.githubusercontent.com/kubernetes-sigs/ip-masq-agent/master/ip-masq-agent.yaml, then deploying the required configmap? (using kustomize + kubectl, as usual)?

We agree on holding this PR until doing a test deploy ip-masq daemon set manually (without terraform).

Hi @angelbarrera92,
I agree with you approach. We can leave ip-masq outside of Terraform.
I've tested it on GKE using the daemon set and the configmap and it works.
FYI, there is an updated yaml for the ip-masq-agent on gcp that you can find here:
https://cloud.google.com/kubernetes-engine/docs/how-to/ip-masquerade-agent#create_manual

Nice @nikever, maybe we can do something in the ´fury-kubernetes-networking` module/katalog. What do you think?

Sure. I'm closing this PR.


Question on how is translation working?

Just found this library and it sounds very promising before I give it a try I have some question I couldn't find answers on the website:
How is translation of  java to c# working?
If transpiled what java packages from jdk are supported or are they also transpiled on the fly?

Hi.   Thanks for the questions.  Good feedback--I'm updating the site to make this clearer.   But the answers are in brief are:
Yes, it's a transpiler.
As for the JDK packages, the Java -> C# source translator / transpiler itself only translates a few core Java classes, like String and StringBuilder, that have overlap into the language proper.  They are translated to matching classes in the target language (Java String -> C# String).   For the the rest of the commonly used parts of the JDK, the expectation is that you use the JSimple libraries, which also include commonly needed 3rd party libraries (like OAuth, logging, unit testing, etc.).   That is, change references to java.util.HashMap to jsimple.util.HashMap instead.   The JSimple version is pretty much the same as the standard Java version (based off Apache Harmony) and in some cases has a few enhancements thrown in (plus some rarely used functionality stripped out).  That approach helps ensure that the code works consistently in Java and the target language.   JSimple itself is transpiled to target languages, to make it available there.
Holler with any other questions / problems / feedback.  Thanks.

Briliant

How about adding jvm bytecode translation with a decompiler like https://github.com/JetBrains/intellij-community/tree/master/plugins/java-decompiler/engine

@ckulenk can you write up a separate github issue for that.   I will mark it as an enhancement request.


Migrate GCC tenants

Mandatory reviewers for this PR:

Tamar Komornik
Yaniv Kravitz
Gadi Palatchi

Please do not merge before their approval.

#sign-off, did we full test it ?

#sign-off

@TomJanetscheck - please merge this PR.


Add dependency check for current child samples

Which area/kind this issue is related to?

/area ci
/area registry
Issue Description

In order to ensure our current child samples remain compatible with changes to their parent stacks, a dependency check on these samples will be necessary to prevent breaking changes from being introduced to the parents. This dependency check should be run on all PRs made to the community registry repository.
We need to:

Set up a dependency tree that shows the current child samples for each parent stack
Triggers the testing of child samples as part of the PR tests of the parent stacks

The auto-generated dependency tree and inclusion of additional child stacks/samples are planned in the upcoming #1060 changes.
Target Odo version

N/A
Target Date: TBA

Targeting this issue for next sprint to focus on higher prior items.

For the parent stack dependency tree we can use a parents yaml file which could be built from a script that runs nightly.
Proposed parents file:
parents:
  stack1_name:
    children:
      - name: child1_name
      - name: child2_name
        version: 1.0.2
  stack2_name:
    version: 1.1.0
    children:
      - name: child3_name

How quick is the script to build the parent.yaml? If the generation of the script is quick, can we just generate it on the fly instead of storing it as part of the repo?

How quick is the script to build the parent.yaml? If the generation of the script is quick, can we just generate it on the fly instead of storing it as part of the repo? The issue with running nightly is what if the PR introduces a new dependency? Then the PR build may not be able to trigger the dependent child test run.

@elsony I'll need to look into building the file using multiple versions but yeah this makes for sense on the fly over nightly due to it building fast enough and getting the recent sample changes in real-time. I could still build this yaml in the check script then passing it into the Go test suite to be worked on.

Have created test suite for validating child samples, continuing to work out errors and fit multi versions for both stacks and samples. Targeting next sprint to accomplish this.

Additional test suite has been scrapped for reuse of existing test suite for stack validation, some stack testing cannot be reused and is omitted from these additions.

Ready for review: https://github.com/devfile/registry/pull/195

Investigating a problem with the validate devfile schema test suite: https://github.com/devfile/registry/pull/195#discussion_r1280895720

Investigating a problem with the validate devfile schema test suite: devfile/registry#195 (comment)

Continuing in Sprint 245 due to vacation leave.

Moved item to 245 as requested

Change in priority this sprint requires my focus on other issues at this time.

Repriorizing this issue to be planned for upcoming sprint due to issues caused by recent changes to parent stacks, increasing priority to critical.

Failures of cache_samples.sh helper script (https://github.com/devfile/api/issues/1553) is blocking this issue.

PR ready for review: https://github.com/devfile/registry/pull/397


suggestion: default to ./node_modules/.bin/nodemon in launch.json

Testing #27463
The generated launch.json sets the runtimeExecutable to nodemon.  Press F5 to debug and it fails because it cannot find nodemon.
i suggest to always refer to the local ./node_modules/.bin/nodemon instead as that has a higher likelihood of being present once npm install has been run.
            "runtimeExecutable": "./node_modules/.bin/nodemon",

dupe of #27774


Deb. package installation problem

When I try to install the son-cli tools from the *.deb package on a Ubuntu Trusty machine, I get:
Unpacking son-python3-setuptools (24.0.3) ...
dpkg: error processing archive /var/cache/apt/archives/son-python3-setuptools_24.0.3_all.deb (--unpack):
 trying to overwrite '/usr/bin/easy_install', which is also in package python-setuptools 3.3-1ubuntu2
Selecting previously unselected package son-python3-six.
Preparing to unpack .../son-python3-six_1.10.0_all.deb ...
Unpacking son-python3-six (1.10.0) ...
Selecting previously unselected package son-python3-cryptography.
Preparing to unpack .../son-python3-cryptography_1.4_amd64.deb ...
Unpacking son-python3-cryptography (1.4) ...
Selecting previously unselected package son-python3-decorator.
Preparing to unpack .../son-python3-decorator_4.0.10_all.deb ...
Unpacking son-python3-decorator (4.0.10) ...
Selecting previously unselected package son-python3-jsonschema.
Preparing to unpack .../son-python3-jsonschema_2.5.1_all.deb ...
Unpacking son-python3-jsonschema (2.5.1) ...
Selecting previously unselected package son-python3-paramiko.
Preparing to unpack .../son-python3-paramiko_2.0.1_all.deb ...
Unpacking son-python3-paramiko (2.0.1) ...
Selecting previously unselected package son-python3-pyyaml.
Preparing to unpack .../son-python3-pyyaml_3.11_all.deb ...
Unpacking son-python3-pyyaml (3.11) ...
Selecting previously unselected package son-python3-pyaml.
Preparing to unpack .../son-python3-pyaml_15.8.2_all.deb ...
Unpacking son-python3-pyaml (15.8.2) ...
Selecting previously unselected package son-python3-requests.
Preparing to unpack .../son-python3-requests_2.10.0_all.deb ...
Unpacking son-python3-requests (2.10.0) ...
Selecting previously unselected package son-python3-validators.
Preparing to unpack .../son-python3-validators_0.10.3_all.deb ...
Unpacking son-python3-validators (0.10.3) ...
Selecting previously unselected package sonata-cli.
Preparing to unpack .../sonata-cli_0.9_all.deb ...
Unpacking sonata-cli (0.9) ...
Errors were encountered while processing:
 /var/cache/apt/archives/son-python3-setuptools_24.0.3_all.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)

There seem to be a problem if someone has already installed `setuptools on his machine. Which might happen quite often.

The root of this problem was linked to the versions of setuptools from the Ubuntu 14.04 repositories, which is version 3.3.x. However, since paramiko depends on cryptography which depends on setuptools >=11.3, this was a problem. For now the workaround was to include a recent version of setuptools in ubuntu 14.04 repository. For the ubuntu 16.04 repository, for now we are relying on default ubuntu repository setuptools, which is version 20.x..
It was either this workaround or remove the support for Ubuntu 14.04 LTS.
@mpeuster can you test now if it's working? thanks.

This problem was solved. It can now be closed.


Asset compilation fails when updating to 6.5.3 (Mini CSS Extract Plugin initialization error)

Hi, I think it is this change:
https://github.com/shakacode/shakapacker/blob/0750276b2d686252cb3fb1dce8a3e67ba3a782a3/package/environments/base.js#L76
CI Failure
https://github.com/diowa/ruby3-rails7-bootstrap-heroku/pull/270
Error
$ ./bin/webpacker
[webpack-cli] Failed to load '~/dev/ruby3-rails7-bootstrap-heroku/config/webpack/webpack.config.js' config
[webpack-cli] Invalid options object. Mini CSS Extract Plugin has been initialized using an options object that does not match the API schema.
 - options.chunkFilename should be one of these:
   non-empty string | function
   -> This option determines the name of non-entry chunk files.
   -> Read more at https://github.com/webpack-contrib/mini-css-extract-plugin#chunkfilename
   Details:
    * options.chunkFilename should be a non-empty string.
    * options.chunkFilename should be an instance of function.

How to replicate
$ rails _7.0.4_ new minicss-shakapacker --skip-active-record --skip-javascript
$ cd minicss-shakapacker/
$ echo "gem 'shakapacker', '6.5.3'" >> Gemfile
$ bundle exec rails webpacker:install
$ yarn add css-loader style-loader mini-css-extract-plugin css-minimizer-webpack-plugin
$ bin/webpacker

My #202 try to fix it, hopefully.

Closed via #202 and released in 6.5.4


Add create_transaction method

Adds the ability to create new transactions.
Resolves #11
Tested locally with positive (credit) and negative (debit) numbers, with and without decimals, and with a decimal more than 2 places (logic in method rounds it to 2). Also tested with and without notes.

Tested with id result.
In [1]: from monarchmoney import MonarchMoney
   ...:
   ...: import asyncio
   ...:
   ...: import os
   ...:
   ...: import json
   ...: import logging
   ...:
   ...: #logging.basicConfig(level=logging.DEBUG)
   ...:
   ...: mm = MonarchMoney(session_file = ".mm/mm_session.pickle")
   ...: asyncio.run(mm.login())
   ...:
   ...: result = asyncio.run(mm.create_transaction(
   ...:         "2023-11-07",
   ...:         "160542457753312071",
   ...:         15.326,
   ...:         "My Test Merchant 2",
   ...:         "160539862437534145"
   ...:         ))
Using saved session found at .mm/mm_session.pickle

In [2]: result
Out[2]:
{'createTransaction': {'errors': None,
  'transaction': {'id': '160643997525678328'},
  '__typename': 'CreateTransactionMutation'}}

...

In [8]: list(filter(lambda txn: txn['id'] == "160643997525678328", asyncio.run(m
   ...: m.get_transactions())['allTransactions']['results']))
Out[8]:
[{'id': '160643997525678328',
  'amount': 15.33,
  'pending': False,
  'date': '2023-11-07',
  'hideFromReports': False,
  'plaidName': None,
  'notes': '',
  'isRecurring': False,
  'reviewStatus': None,
  'needsReview': False,
  'attachments': [],
  'isSplitTransaction': False,
  'category': {'id': '160539862437534145',
   'name': 'Fun Money',
   'icon': '😜',
   '__typename': 'Category'},
  'merchant': {'name': 'My Test Merchant 2',
   'id': '160615828759320958',
   'transactionsCount': 2,
   '__typename': 'Merchant'},
...

Please also make sure to configure black in your Python environment to handle indentation and formatting.
@CalvinChanCan recently enabled it against all PRs and it'll prevent any code not meeting its standard from merging cleanly.

Ran black, and made indentation consistent across the class based on the convo above.


[Has anyone ever use teaser to marge more than 2 pointcloud together?]

Have you read the documentation?

[x] Yes
[] No - then this issue will be closed.

Post your theoretical questions / usage questions here.
I'm attempting to merge six point clouds using Teaser ++. Initially, I successfully stitched two of them together. My plan was to stitch pairs to create three merged point clouds, and then continue merging these results step-by-step until all were combined. Unfortunately, this method isn't producing the expected outcomes. Has anyone tried a similar approach with multiple point clouds? If so, how did it work out for you?

Could you provide some snapshots or something for the unexpected outcomes please?

Hello, I was able to get it to work finally, I used a for loop and merge two at a time

@Monsurat-Onabajo Great to hear!


Update protos

@michael-berlin

Going to merge this as it's breaking all Travis builds.

LGTM
The python files should have changed as well? But you didn't include them?

Nope, that's all that changed...

The Python proto files changed on my machine because I didn't have the latest protobuf + gRPC version installed :-)
I had to delete dist/protobuf and dist/grpc and then re-run bootstrap.sh :-)


Error with install.bat

@generalloki
Thanks a lot to solve very quickly issue #56
New Errors with 7c88d4cfe18e2bf3aa655bf252169600e27f683a version :

@neoxeo , Thanks! I have updated master with the fix.

@generalloki
New error with install.bat on branch AlternetUI-MasterWithWebBrowser with 0bce38540953f43fc834780d99e64ce4f32c748c version

@neoxeo , I executed Install.bat just now and it works fine on my pc. Could you please rerun it on the latest version of master or MasterWithWebBrowser (doesn't matter)?

@generalloki
I Have find the problem. I have add this parameter in AlternetUI.csproj "True"
to generate a nuget package of AlternetUI when it build solution, to use this nuget pacage as local.
If I switch it to False, problem is not present. I don't understand why.

@neoxeo , I build nugets like is described in STEP 6 of readme.md, so I have no idea what is wrong with your approach.


Hotfix: explicit tests for string scaling sets

Since these are sourced from JSON, this is how they'll usually be input
(percentages especially).
Failing test case on Python2 threw:
TypeError: bad operand type for abs(): 'unicode'
Fixing said test case with small refactor: cast earlier.

+1


add user as author to match entity

this is so we can identify ownership of the matches and identify the interactive player

oh no, tests failing...


Cross cluster querying

Closes #1952

What changed?
This PR aims to answer the question: "How can we efficiently query resources across all clusters with users' credentials?"
That was done using couple strategies:

We periodically fetch all clusters and save the values in memory
We periodically fetch all namespaces from all clusters and save the values in memory
When a user requests a list of resources, we loop through all clusters and ask their kube-api which namespaces that particular user has access and save those values in memory with an arbitrary TTL. Next time a request of that given user comes through we use the cached namespaces for the queries.

The entity responsible for all that coordination is the ClientsFactory. It is a long running process that periodically queries the clusters, namespaces and when returning the list of clients to be used, checks the namespaces the user has permissions before returning.
In possession of the individual impersonated kube clients and the namespaces the user have access to, the Clusters Client  is responsible to actually do the querying and make sure all calls run in parallels to maximize efficiency.
Pagination support was added but with caveats given the nature of Kube API and the parallelization of calls. The Limit param usually specified by the Client caller to limit the number of total items returned in query will have a different meaning here.
Since we have to dispatch the call to a N number of clusters and namespaces, its hard to coordinate all those calls to return at max that number, given it would require kube-api to provide functionally it doesn't support yet if ever will. As a trade off, I decide to use that limit as the params for all calls, which means will endup with a total number o itens of Limit * Num. Clusters * Num. Namespaces.
Given the goal of pagination is to reduce the pressure on Kube API of calling a huge number of resources at once and the desire to minimize the load time the page will have, I think that choice accomplishes that.

How did you validate the change?

Manual and Automated tests

Release notes

Documentation Changes

The requests made by the UI seem to return faster. Currently in the main branch, the responses from sources and flux runtime tabs are consistent every time. They take around ~11seconds for the Sources tab and ~10seconds in Flux Runtime tab. But with this PR the first response in the Sources tab takes around ~6seconds and the subsequent requests take around ~2.5 seconds. On the Flux Runtime tab the first request took 4.5seconds and the subsequent request took ~1.5 seconds. So pretty awesome results. Great work.


Add NoAuthIPAddr option for go swiftclient

Previously swiftclient go binding written in ProxyFS repository is pinning the no auth proxy server ip address as <IP_ADDRESS> but that hard coded configuration gives us constraints like 1. no-auth proxy sever cannot be listening any internal ip address for any no-auth fuse system. 2. proxyfsd cannot be configured to look up existing proxy-server for testing. And also the deployment model that proxyfsd should be located swift proxy would give more operational consideration because backend swift requires Ring distribution for all swift proxy nodes in the cluster.
This new option called NoAuthIPAddr is for setting the IP address from proxyfsd to no-auth swift proxy server enables us to separate the deployment operation between proxyfsd and backend swift and make the flexible configuration for us in my opinion.

In swiftclient/config.go, the change made immediately mandates that the config file specifies the SwiftClient.NoAuthIPAddr. This is definitely my preference i.e. that there be no defaults in the code. That said, the practice of providing a temporary default until such time that scripts/tools building the ProxyFS configs catch up might be nice.
If you don't mind, I'd like to make that temporary default present and merge the result. Ok?

Temporary default sounds ok to me to migrate until the option is available as required.

Ok - great! I'll pull this I to 1.6 we hope to tag this Friday. Thanks for the PR.

Brought over PR into swiftstack/ProxyFS repo as PR https://github.com/swiftstack/ProxyFS/pull/155


Publish API documentation

Checklist

[X] I've looked through the issues and pull requests for similar request
[ ] This feature could be solved with a custom docker image (optional)

Describe your request
Can we get static API docs published for cross as a library?
Describe why this would be a good inclusion for cross
So that it's easier to enjoy cross capabilities via downstream Rust libraries and tools.
For example, safely looping over targets the Cross.toml file without having to assume a particular or permanent schema.

The library api is available, but contain a warning.

https://docs.rs/cross/latest/cross/


Code cleanup

Codecov Report

Merging #26 into master will increase coverage by 1.05%.
The diff coverage is 66.66%.

@@            Coverage Diff            @@
##           master     #26      +/-   ##
=========================================
+ Coverage   65.14%   66.2%   +1.05%
=========================================
  Files           7       7
  Lines         439     435       -4
=========================================
+ Hits          286     288       +2
+ Misses        153     147       -6

Impacted Files
Coverage Δ

aioelasticsearch/compat.py
44.44% <ø> (-1.71%)
:arrow_down:

aioelasticsearch/transport.py
59.41% <0%> (-0.24%)
:arrow_down:

aioelasticsearch/connection.py
79.31% <100%> (+2.11%)
:arrow_up:

aioelasticsearch/helpers.py
100% <0%> (+1.29%)
:arrow_up:

aioelasticsearch/__init__.py
95.23% <0%> (+9.52%)
: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 b2f5a80...b430afc. Read the comment docs.


[zh] localize blog: 2023-03-10-image-registry-change.md

I will localize this blog in days.
To avoid possible conflicts with other contributors, this issue is opened.
content/zh-cn/blog/_posts/2023-03-10-image-registry-change.md

/assign
/language zh
/area blog

/triage accepted
thanks.


All posts from Medium and Sitepoint

Updated json file to include all posts
@chris-pearce @bnhovde

LGTM


Nit: Wrong Credit

Sorry to bother, just wanted to let you know the credit is wrong ( I did the one on the left):

Im sorry! Fixing right now!

Im sorry! Fixing right now!

@LucasPilla thank you!


[suggestion]add -output to command usage description

Interesting project!
When I first try, I followed the usage, I omitted the -output flag, I think the output pic will appear in the input dir, but I didn't find it
shiming@pro ➜  notpro git:(master) notpro -input /Users/shiming/Desktop/2017-12-23_18-02-13.jpg
2019/03/19 11:09:13 Width: 500, Height: 499

Then I checked the main.go and found there's -output option, I think it's better to add -output option to usage :
notpro -input head.jpeg -output notpro.jpg

newhead.png is added to project root dir, which is not prompted to the user


The feed image in the bottom is overlapping the content in the mobile view

Issue Type:
[x] Bug Report
[  ]  Feature Request
[  ] Documentaion
Describe the bug

The feed image which in in fixed position in bottom is overlapping the other content of site in mobile view

Possible solution

Describe the solution you thought of.
I can remove that image for the mobile view only or
can make the size of image small
or can add some feature like that image will be only visible on hovering on the number icon (190)in the bottom (only in mobile view)

Screenshots

If applicable, add screenshots to help explain your problem.

Yes @gnikita432  I am working on this

Ohh Okay I am sorry for opening a duplicate issue.
Closing this issue.

Ohh Okay I am sorry for opening a duplicate issue.
Closing this issue.

Is ok you can find a new one thanks for contribution


Page Not Found on "Action" category on Algolia

Free Text

It looks like that Algoria still holds indices of /tfaction/docs/actions which no longer exist?
❌ https://suzuki-shunsuke.github.io/tfaction/docs/actions/list-targets/#outputs
✅ https://suzuki-shunsuke.github.io/tfaction/docs/config/github-token/#required-permissions-of-repositorys-github_token
✅ https://suzuki-shunsuke.github.io/tfaction/docs/feature/linter/#enable-or-disable-linters

We deleted the document.

https://github.com/suzuki-shunsuke/tfaction-docs/pull/185

I'm not sure why Algoria still holds indices.
I'll take a look.

Oh, Crawler was paused...

I ran the crawling manually just now.
The crawling succeeded.
It looks good.


Be resilient to new lockfile formats

Currently we're not handling updates in lockfile versions or formats very gracefully. Sure, we catch the exception that is thrown, but it's not obvious that someone is using a new version of a tool. See #503 or #476

I think a solution to this would be to add telemetry to each detector that has a lockfile version. We'd just need to send a record with the lockfile version, and the detector type.
Then, we could configure Application Insights to generate an alert when it detects a new version.
We would need a query sort of like this one to find the maximum lockfile version per detector. If we add the telemetry, we can then fine-tune this query.
let timeRange = ago(6h);
let lookBack = ago(14d);
let latestLockfileVersions = customEvents
| where timestamp > lookBack
| where name == "LockfileVersion"
| extend Props = parse_json(customDimensions)
| extend Detector = Props['Detector'], LockfileVersion = toint(Props['LockfileVersion'])
| summarize max(LockfileVersion) by Detector;
customEvents
| where timestamp > timeRange
| where name == "LockfileVersion"
| extend Props = parse_json(customDimensions)
| extend Detector = Props['Detector'], LockfileVersion = toint(Props['LockfileVersion'])
| join kind=inner latestLockfileVersions on Detector
| where LockfileVersion > max_LockfileVersion
| project timestamp, Detector, LockfileVersion

Closed. Monitors created.


Web Authentication Broker Twitter OAuth 2 Authentication

I am testing WebAuthenticationBroker sample app. I cannot understand what should I type in "Callback URL" in Twitter Scenario. I have created a developer account in Twitter and got my Consumer Key and Consumer Secret.  But I cannot full fill the Callback URL . What should I type ? What is my redirect URL ? How can I get it ? Please teach me.

Same here. Twitter api is extremely confusing. And web broker sample seems use oauth 1 authenticate. And everytime I call for 1.1 home_timeline api, it just shows invalid authenticate.

I've asked someone from the WebAuthenticationBroker team to investigate. Thanks for your patience.

Before using the WebAuthenticationBroker sample (twitter scenario ) you must register the application in https://apps.twitter.com/ .   for eg
name - WAB_test
desc   -  sample WAB test
website url -  http://<IP_ADDRESS>  (this is just a place holder for testing . you can use your applications url if   you want .  )
callback uri -  http://placeholder.com/   (this must be a valid url . Twitter needs to send a oauth_callback value when we try to request a token . you don't really have to create one if you are just using OAuth for authentication . You can even use https://google.com  if you want )

Can I really use  https://google.com for redirect_url  for OAuth 2.0 Authentication ? I don't  have any website of my own.

According to the twitter api if we are just using our test sample to get a success token , we can use a placeholder like  http://placeholder.com/ , the api returns a response like http://placeholder.com?oauth_token="our token"...
this token can now be extracted from the response string

ok. thanks.


Using "variables" in AET suit

Describe the solution you'd like
In some cases user could use few same steps in aet's suite i.e
<open/>
<wait-for-image-completion css="#logo"/>
<click css="#logo"/>

The idea is instead above can use just:
<var1>
Additional context
In my project I used ENTITY for huge javascript command
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE variables [
	<!ENTITY var1 "Fuction.setValue(.......]
	]>

And after I can use
			<executejavascript cmd="&var1;"/>
To call this value every time is neccessary. Will be great when I can use one markup then sequence of 5 every time.

While the idea of re-using parts of the suite is very noble, I don't think that implementing variables feature into XML is a good idea.
You can try some kind of XML templating, so that the final XML file is combined by some tool before sending it to AET.
You can find example in the AET integration tests module. We make use of two maven plugins there: maven-resources-plugin and xml-maven-plugin.
I'm sure there are plenty of tools in many languages that will help you combining the final XML and user variables.

Ok, thank you for the answer. I'll try then those solution


Explicitly define inputs, outputs and latch conditions for state machines

State machines should explicitly define inputs, outputs and latch conditions.
State machines are usually queried by a plookup with selector. The input (committed / witness data) is introduced in some row, then a computation occurs spanning multiple rows until the result has been computed. On the row that holds the result, the boolean latch polynomial is set to true and this is the row that is queried.
There is a slight complication due to the fact that input and output need to be on the same row. The computation could be either before the row or after the row (just verifying the result that is introduced as a witness). We have to see what is practical in the end, especially concerning the witness generation.
If latch condition and inputs/outputs are defined, we can "query" the state machine just using something like
MachineName(input1, input2, ..., output1, output2, ..)

We could simplify the latch poly by just specifying the length of a single computation in the state machine, this making it implicit.

I think this is now completed, @chriseth ?


senate membership updates for the 115th Congress

This is a draft of senate changes for the 115th Congress.
Bioguide IDs for the new senators have not yet been assigned so that's missing. Otherwise it should be good to merge on Jan 3.
I ran:

a new script scripts/election_results_senate_2016.py
scripts/sweep.py (renamed from sweep_memberships and now also sweeps the social media file)

see #375 and h/t to @mrumsky who started the senate elections google doc that I started from

Approved, but then I noticed tests were failing for what look like legit reasons.

Yeah there's a datetime.now() call in the new validation script that won't let it pass until Jan 3 actually comes around, and the missing bioguide ID.


Fix missing import

Thank you. Fore was planning to delete the unnecessary instances below, so a fix was made to remove the unnecessary parts.
The current necessary modifications are as follows:

Parsing the response text and storing it in a dict
Implementing the content from the Bard API, such as replit, as is (not too difficult)
Organizing the logic for the async client by referring to core.py

Thank you for your contribution, but please understand that we cannot merge it as the modifications required are slightly different. Thank you, and have a happy weekend.

😉👍


703 bug outcomes

Coverage decreased (-0.002%) to 76.092% when pulling 9b2778a1977cbfdd06452eb295a28ba04bec6fe0 on 703-bug-outcomes into 07b4dc14323716fe4cecb7826a8ba2ddbabdf82d on master.

Coverage decreased (-0.002%) to 76.092% when pulling 54866d6e02d90bcb6038794b98e27f10ee613a3d on 703-bug-outcomes into 07b4dc14323716fe4cecb7826a8ba2ddbabdf82d on master.

Coverage decreased (-0.002%) to 76.092% when pulling 54866d6e02d90bcb6038794b98e27f10ee613a3d on 703-bug-outcomes into 07b4dc14323716fe4cecb7826a8ba2ddbabdf82d on master.