Requests dependency 'urllib3' must be version >= 1.21.1, < 1.22!

Description
Outline the issue here:
victor:ASQ Victor$ az login -u *********** -p **********
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/Users/Victor/lib/azure-cli/lib/python2.7/site-packages/azure/cli/__main__.py", line 23, in <module>
    exit_code = azure.cli.main.main(args)
  File "/Users/Victor/lib/azure-cli/lib/python2.7/site-packages/azure/cli/main.py", line 52, in main
    error_code = handle_exception(ex)
  File "/Users/Victor/lib/azure-cli/lib/python2.7/site-packages/azure/cli/core/util.py", line 34, in handle_exception
    from msrestazure.azure_exceptions import CloudError
  File "/Users/Victor/lib/azure-cli/lib/python2.7/site-packages/msrestazure/__init__.py", line 28, in <module>
    from .azure_configuration import AzureConfiguration
  File "/Users/Victor/lib/azure-cli/lib/python2.7/site-packages/msrestazure/azure_configuration.py", line 33, in <module>
    from msrest import Configuration
  File "/Users/Victor/lib/azure-cli/lib/python2.7/site-packages/msrest/__init__.py", line 27, in <module>
    from .configuration import Configuration
  File "/Users/Victor/lib/azure-cli/lib/python2.7/site-packages/msrest/configuration.py", line 36, in <module>
    import requests
  File "/Users/Victor/lib/azure-cli/lib/python2.7/site-packages/requests/__init__.py", line 61, in <module>
    raise RuntimeError('Requests dependency \'urllib3\' must be version >= 1.21.1, < 1.22!')
RuntimeError: Requests dependency 'urllib3' must be version >= 1.21.1, < 1.22!

Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here: curl -L https://aka.ms/InstallAzureCli | bash
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here: 2.0.6
OS Version: What OS and version are you using?
Answer here: macOS Sierra 10.12.5
Shell Type: What shell are you using? (e.g. bash, cmd.exe, Bash on Windows)
Answer here:
bash

I have been gotten this same issue.  I am still finding workaround.

@Xurma2 I have been found resolve steps:

jump into /azure-cli/lib/python2.7
pip install urllib3
If it still doesn't work. The reason is that the python you are running is not under azure-cli folder. It might be located at: /usr/local/lib/python2.7
In this situation, you have to copy urllib3 manually, like this:
cd /usr/local/lib/python2.7/site-packages
ls (to confirm urllib3 already exists. If not,please run "pip install urllib3")
cp -r urllib3 /azure-cli/lib/python2.7/site-packages/

When done above, you will run az login correctly.

Getting the same issue when trying to login:
broda@standartubuntu:/usr/local/lib/python2.7/site-packages$ az login Traceback (most recent call last):   File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main     "__main__", fname, loader, pkg_name)   File "/usr/lib/python2.7/runpy.py", line 72, in _run_code     exec code in run_globals   File "/home/broda/lib/azure-cli/lib/python2.7/site-packages/azure/cli/__main__.py", line 23, in <module>     exit_code = azure.cli.main.main(args)   File "/home/broda/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/main.py", line 52, in main     error_code = handle_exception(ex)   File "/home/broda/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/core/util.py", line 34, in handle_exception     from msrestazure.azure_exceptions import CloudError   File "/home/broda/lib/azure-cli/local/lib/python2.7/site-packages/msrestazure/__init__.py", line 28, in <module>     from .azure_configuration import AzureConfiguration   File "/home/broda/lib/azure-cli/local/lib/python2.7/site-packages/msrestazure/azure_configuration.py", line 33, in <module>     from msrest import Configuration   File "/home/broda/lib/azure-cli/local/lib/python2.7/site-packages/msrest/__init__.py", line 27, in <module>     from .configuration import Configuration   File "/home/broda/lib/azure-cli/local/lib/python2.7/site-packages/msrest/configuration.py", line 36, in <module>     import requests   File "/home/broda/lib/azure-cli/local/lib/python2.7/site-packages/requests/__init__.py", line 61, in <module>     raise RuntimeError('Requests dependency \'urllib3\' must be version >= 1.21.1, < 1.22!') RuntimeError: Requests dependency 'urllib3' must be version >= 1.21.1, < 1.22! broda@standartubuntu:/usr/local/lib/python2.7/site-packages$ pip show urllib3 Name: urllib3 Version: 1.21.1 Summary: HTTP library with thread-safe connection pooling, file post, and more. Home-page: https://urllib3.readthedocs.io/ Author: Andrey Petrov Author-email<EMAIL_ADDRESS>License: MIT Location: /usr/local/lib/python2.7/dist-packages Requires: broda@standartubuntu:/usr/local/lib/python2.7/site-packages$

Had the same issue. Upgrading the package resolved it: pip install --upgrade urllib3
Looks like it is pinned to 1.16 in a few places.
root@add5eb5ef660:/gopath/src/github.com/Azure/acs-engine# pip show urllib3
---
Metadata-Version: 2.0
Name: urllib3
Version: 1.16
Summary: HTTP library with thread-safe connection pooling, file post, and more.
Home-page: https://urllib3.readthedocs.io/
Author: Andrey Petrov
Author-email<EMAIL_ADDRESS>Installer: pip
License: MIT
Location: /usr/local/lib/python2.7/dist-packages
Requires:

I had the same issue and the workaround of copying urllib3 to the azure directory works.
thanks @micli

+1 had the same issue today while trying to follow a docs.microsoft.com article.
Manually copying my installation of urllib3 fixed it. Thanks @micli !

My installation log clearly shows the installer removing the "urllib3" 1.21.1 (that was already installed), then installing "urllib3" 1.16.

As devonbarrett suggested above, "urllib3" 1.16 is likely pinned somewhere.
I fixed this by manually upgrading "urllib3" after the install, using the command:

sudo -H pip install --upgrade urllib3
This reinstalls urllib3 1.21.1 and everything works great after this

Note, I've encounter this issue on five-out-of-five Ubuntu 14.04 Instances, so this is not a niche or uncommon problem.
Specifics version:

azure-cli (2.0.6)
Python (Linux) 2.7.6
Ubuntu 14.04
Bash 4.3.11(1)

Before installing the Azure CLI - the following were performed:

install all OS updates, OS packages (needed), and Python dependancies
PIP upgraded to 9.0.1

ISSUE ILLUSTRATION IN CONSOLE OUTPUT
Here's an excerpt from the console output of my install.  The uninstall (of the correct urllib3) occurs right after the install of azure-datalake-store:
Running setup.py install for azure-datalake-store ... done
Found existing installation: urllib3 1.21.1
  Uninstalling urllib3-1.21.1:
    Successfully uninstalled urllib3-1.21.1

Soon after, in the final portion of the install "urllib3" 1.16 is installed.  It's the very last item listed in the output below (\n's added to make it readable and not one long line):
Successfully installed azure-batch-2.0.1 azure-cli-2.0.6 azure-cli-acr-2.0.4 azure-cli-acs-2.0.6
azure-cli-appservice-0.1.6 azure-cli-batch-2.0.4 azure-cli-cdn-0.0.2 azure-cli-cloud-2.0.2
azure-cli-cognitiveservices-0.1.2 azure-cli-command-modules-nspkg-2.0.0
azure-cli-component-2.0.4 azure-cli-configure-2.0.6 azure-cli-core-2.0.6
azure-cli-cosmosdb-0.1.6 azure-cli-dla-0.0.6 azure-cli-dls-0.0.6 azure-cli-feedback-2.0.2
azure-cli-find-0.2.2 azure-cli-interactive-0.3.2 azure-cli-iot-0.1.5 azure-cli-keyvault-2.0.4
azure-cli-lab-0.0.4 azure-cli-monitor-0.0.4 azure-cli-network-2.0.6 azure-cli-nspkg-3.0.0
azure-cli-profile-2.0.4 azure-cli-rdbms-0.0.1 azure-cli-redis-0.2.3 azure-cli-resource-2.0.6
azure-cli-role-2.0.4 azure-cli-sf-1.0.1 azure-cli-sql-2.0.3 azure-cli-storage-2.0.6
azure-cli-vm-2.0.6 azure-common-1.1.6 azure-datalake-store-0.0.8 azure-graphrbac-0.30.0rc6
azure-keyvault-0.2.0 azure-mgmt-authorization-0.30.0rc6 azure-mgmt-batch-3.0.1
azure-mgmt-cdn-0.30.2 azure-mgmt-cognitiveservices-1.0.0 azure-mgmt-compute-1.0.0rc1
azure-mgmt-containerregistry-0.2.1 azure-mgmt-datalake-analytics-0.1.4
azure-mgmt-datalake-nspkg-2.0.0 azure-mgmt-datalake-store-0.1.4
azure-mgmt-devtestlabs-2.0.0 azure-mgmt-dns-1.0.1 azure-mgmt-documentdb-0.1.3
azure-mgmt-iothub-0.2.2 azure-mgmt-keyvault-0.31.0 azure-mgmt-monitor-0.2.1
azure-mgmt-network-1.0.0rc3 azure-mgmt-nspkg-2.0.0 azure-mgmt-rdbms-0.1.0
azure-mgmt-redis-1.0.0 azure-mgmt-resource-1.1.0rc1 azure-mgmt-sql-0.4.0
azure-mgmt-storage-1.0.0rc1 azure-mgmt-trafficmanager-0.30.0 azure-mgmt-web-0.32.0
azure-monitor-0.3.0 azure-multiapi-storage-0.1.0 azure-nspkg-2.0.0 azure-servicefabric-5.6.130
urllib3-1.16

I hope this helps narrow it down.  For the record I love azure and the cross-platform/cross-language support.  However, I haven't had any difficulties installing the aws CLI, libs (boto, botocore) or even gcloud CLI and libs.  This has raised more that a few eyebrows amongst my customers.

Fixed, and we will release at some time this afternon

Thank you & your team!


uwsgi 2.0.23 builds failed

Comment:
Hi all, we were trying to install uwsgi 2.0.23 from conda to make be compatible with Python 3.12 and realized the builds for https://github.com/conda-forge/uwsgi-feedstock/pull/73 (which was supposed to publish uwsgi 2.0.23 to conda) were failing.
I took a quick look at the build failure and saw RuntimeError: Failed to download or patch source. Please see build log for info, while it looked like the source download succeeded:
Source cache directory is: /home/conda/feedstock_root/build_artifacts/src_cache
Downloading source to cache: uwsgi-2.0.23_0cafda0c16.tar.gz
Downloading https://pypi.io/packages/source/u/uWSGI/uwsgi-2.0.23.tar.gz
Success

Could someone take a moment to check and see what's happening? I really appreciate the help!

I did some investigation and realized the issue was not in "download" but in "patch". There's a patch file in uwsgi-feedstock which tried to make uwsgi work with python 3.12. I looked into the uwsgi (not uwsgi-feedstock) source,  the content of the patch was already part of the uwsgi 2.0.23 source code, as a result, the patch failed.
I removed this field and the issue was gone when building locally (I still got a build error but I believe that's specific to my local Xcode build chain since the linux aarch 64 build (docker based) succeeded.

Done in #73. Thanks for investigating this.


feature request: support self-signed SSL certs in rancher-compose

relates to #1853 but for rancher-compose.
As it is now, all I get is:
ERRO[0000] Failed to open project default: Get https://rancher..../: x509: certificate signed by unknown authority

@kaos I'll need to talk to @ibuildthecloud if this would even be possible/make sense.

It shouldn't have to be more than being able to point at a ca bundle to use, or something like that..
Skickat från min iPhone

2 okt. 2015 kl. 20:51 skrev Denise<EMAIL_ADDRESS>@kaos I'll need to talk to @ibuildthecloud if this would even be possible/make sense.
—
Reply to this email directly or view it on GitHub.

The CLI for 2.0 has an option for this, and interactive prompt to take the cert from the server if desired.

is this issue been resolved? I am getting same error
root@K8S-MASTER 0.3.1]# rancher-compose -p es up
ERRO[0000] Failed to open project es: Get https://xxxxxxxxxxx/v3: x509: certificate signed by unknown authority
FATA[0000] Failed to read project: Get https://xxxxxxxxx/v3: x509: certificate signed by unknown authority
[root@K8S-MASTER 0.3.1]#

Hello, I got the same one (with the same compose template, from elastic). export DOCKER_TLS_VERIFY=0 has not effect.

you need to start rancher with custom certificates, i was able to resolve that issue. just run this SH Script
#!/bin/bash
if [[ $DEBUG == "true" ]]; then
set -x
fi
export FQDN="rancher.net"
export CA_SUBJECT="Rancher CPI CA"
export CA_EXPIRE="1825" # CA expires in 5 years
export SSL_EXPIRE="365" # Certificate expires in 1 year
export SSL_SUBJECT="${FQDN}"
export SSL_DNS="${FQDN}" # Additional SANs (comma separated) can be added
export SILENT="true"
docker run -v /opt/certs:/certs
-e CA_SUBJECT
-e CA_EXPIRE
-e SSL_EXPIRE
-e SSL_SUBJECT
-e SSL_DNS
-e SSL_IP
-e SILENT
superseb/omgwtfssl
docker run -d --restart=unless-stopped
-p 80:80 -p 443:443
-v /opt/rancher:/var/lib/rancher
-v /opt/certs/cert.pem:/etc/rancher/ssl/cert.pem
-v /opt/certs/key.pem:/etc/rancher/ssl/key.pem
-v /opt/certs/ca.pem:/etc/rancher/ssl/cacerts.pem \
If you have proxy then enable these options.
-e HTTP_PROXY="http://10.xx.xx.xxx:8080" \
-e HTTPS_PROXY="http://10.xx.xx.xxx:8080" \
-e NO_PROXY="localhost,<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>/24,<IP_ADDRESS>/24,rancher.net" \
rancher/rancher:latest
hope this helps let me know if you still have issues :)


list of discriminated unions

hi,
I'm trying to parse a list of discriminated unions but it doesn't seem to be implemented.
Is there a workaround ? or maybe I'm not doing it properly ?

class FloatData(BaseXmlModel):
    '''group data'''
    id: str = attr(tag='name')
    type: Literal['float'] = attr()
    data: Floats

class StringData(BaseXmlModel):
    '''group data'''
    id: str = attr(tag='name')
    type: Literal['string'] = attr()
    data: str

DataTypes = Annotated[Union[FloatData, StringData], Field(discriminator='type')]

class Doc(BaseXmlModel):
    data_list: list[DataTypes] = element(tag='data', default=[])

@A-Debord Hi,
That's a bug, I will fix it in the next release.

Fixed in version 2.12.1

Okay, thanks for the answer !
Le lun. 26 août 2024, 18:55, Dmitry Pershin @.***> a
écrit :

@A-Debord https://github.com/A-Debord Hi,
That's a bug, I will fix it in the next release.
—
Reply to this email directly, view it on GitHub
https://github.com/dapper91/pydantic-xml/issues/206#issuecomment-2310649556,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGHJGJYR3R7M3DXL7Y4FZG3ZTNMZXAVCNFSM6AAAAABND4ASX6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJQGY2DSNJVGY
.

Good to see this is now fixed.
Thanks!


Use WebCodecs for a faster encoding!

The README says to create an issue for suggestions on how to make it faster :)
https://web.dev/webcodecs/ offers a way to directly use the hardware encoders for the various codecs that Chrome supports. It's in origin trial right now, so feedback is appreciated!
https://github.com/WICG/web-codecs
https://bugs.chromium.org/p/chromium/issues/entry?components=Blink>Media>WebCodecs

Thanks Dale – I'd be really interested in using WebCodecs (I have a webcodecs branch here testing vp9) but I felt blocked by two problems:

I haven't found a simple way of encoding H264 frames, it seems to be mostly focused on codecs like vp9?
WebCodecs doesn't help with building an MP4 container – I might be able to use my WASM build for this, but how would I get NAL data from WebCodecs API?

H264 encoding should be working on stable, though I recommend developing against canary/dev channel since the API is in flux.
Here's a web test which shows how H264 encoding can be done. A key aspect with H264 is the second parameter on the output callback which delivers you the decoder configuration. For H264 the description field contains the AVCC payload data that's needed for an MP4.
Since you're bundling ffmpeg, muxing shouldn't be much of an issue using their internal muxing API. The outputs are already in AVCC format, so you just need to provide the AVCC extradata when configuring the muxer.
We do want to allow users to choose AnnexB or AVCC output. This bug tracks WebCodecs providing an option to choose AnnexB or AVCC. With AnnexB, ffmpeg's normal raw h264 demuxer will accept the bitstream as is. So simply ffmpeg -i .h264 -vcodec copy output.mp4 will mux without any custom code.

Thanks! I'm actually not using ffmpeg here, but small open source C libraries, minimp4 (for turning h264 frames into mp4 container) and minih264 (for encoding YUV to h264).
I don't know much about AnnexB vs AVCC, but to use the minimp4 library I have to send it chunks of NAL data to mux into a mp4. I guess that NAL data is just the same as the AnnexB data?

The second parameter of the output callback is a VideoDecoderConfig object. The description field on that object should have the NAL chunks you need to hand to minimp4. The payload is already in AVC format so it should be ready to insert into an MP4.

Awesome, glad it's working! Sorry it's a bit complicated to get going.

We're working on making it easier to get AnnexB right now: https://chromium-review.googlesource.com/c/chromium/src/+/2587596 hopefully that'll be available on canary later this week.
The encoders are tuned for RTC so it may not always yield better quality at the moment. Did you try setting the "acceleration" prefer to require or deny to see if there were any differences? https://source.chromium.org/chromium/chromium/src/+/master:third_party/blink/renderer/modules/webcodecs/video_encoder_config.idl;l=16

What platform are you testing on? We'll want to track your issues in the Chromium tracker so they don't get forgotten. @chcunningham

Alright thanks! I think this might have actually just been my own code not setting the size correctly, so it was sending lower quality frames, although I'm still noticing a few issues here and there and overall the image appears more faded/muted in color. I'll try to log them in the tracker as I see them.

Probably color space information isn't correctly passed to the encoder somewhere (could be on our side). If you have a repro case with a comparison shot we can dig further.


how to pin to taskbar?

normally, you right click the running app on taskbar and pin it. Seems there's no pin for sysmontask.

I guess what you are looking for is "add to favourites".

Yes Sir, in Mint they call it pin.. I just tried in Ubuntu and add to favorites doesn't show on right click.
was just looking to replace bland gnome-system-monitor. Impressed python can handle it.  In Windows I use Process Hacker, now that's a task manager :)
Thanks!

On which version of Ubuntu you have tried it on?
I am using PopOS and I can add it my favourites.
On mint I will verify the issue and will see about it.

In VMware that would be 21.10.

I just tried in PopOS and it's same problem. I think it's because i'm installing from source, have not tried 'sudo apt install sysmontask' at all.

confirmed, it does work fine in Kubuntu even when installing from source.


Dropped fields on events

Hi,
I see some of the fields are dropped from the following events:
on_client_connected
https://github.com/vishr/emq-web-hook/blob/master/src/emq_web_hook.erl#L52

username
ip_address

on_client_disconnected
https://github.com/vishr/emq-web-hook/blob/master/src/emq_web_hook.erl#L65

username

These properties are essential for us to identify the clients in a multi tenant environment. Any reason for dropping them?
PS: Another change I saw is renaming event to action. Action sounds more like what you need to do for an event e.g.  on_client_disconnect, IMHO changing it back to event is more appropriate. If this change is for consistency, please ignore this.
@emqplus @turtleDeng @s4kibs4mi

@turtleDeng Do we have any explanation for wontfix?

With a little bit of research, I figured out that during auth, I have the ip address and username in the payload. For disconnect, I can simply treat client id as unique.
Reference: https://github.com/emqtt/emq-auth-http/blob/master/etc/emq_auth_http.conf
Closing.


(feat)timePicker: add option to switch to the minutes selector after …

… the hour has been picked
(feat)datePicker: add option to customize the date format

Change toolbar color ( use md-primary )

Change day color ( use md-primary )

Remove the check on the input being a date type ( because date type input are ugly for now )

Improve timePicker toolbar look

Add live folder with ready to go html for coding purposes ( can also be used for the codepen )

BREAKING CHANGE: $mdpDatePicker & $mdpTimePicker now have a show method in order to show the picker
change code from : $mdpTimePicker(args).then()

to

$mdpTimePicker.show(args).then()

This is quite a lot of changes and you might not agree with everything. I'll just leave this here and if you want you can use it :)

:+1:
I agree here. After hour pick, it should switch to minutes, as same as the Material Design Specs.
Also agree with input. Sometimes I just want to use an input text with some filter(moment). This probably would fix different format for dates as well. In Australia, we use dd/mm/yyyy instead of mm/dd/yyyy
However, @corentin-gautier. It would be nice, have a pull request for each change. So the owner @alenaksu would choose one by one what he wants.

@leocaseiro will do if he asks me too :)

@alenaksu any thoughts?

@corentin-gautier Thanks for your contribution!
@leocaseiro
I'm honest, I didn't have time to take a look at the pull request, These days I'm very busy at work.
Please be patient :)

@alenaksu thank you for this plugin ;)

@alenaksu I'm really sorry if I push you. Wasn't my intention.


Velocity v1 does not work inside Polymer v1 elements

Your system information

VelocityJS version: v1
Browser: Chrome
Operating System: Any

Checklist

Is this an issue with code?: Yes
Is this an issue with documentation?: No
Have you reproduced this in other browsers?: Yes
Have you checked for updates?: Yes
Have you checked for similar open issues?: Yes

Please remember that this is an issue tracker, support requests should be posted on StackOverflow - see CONTRIBUTING.md

Please describe your issue in as much detail as possible:
Velocity v1 does not work inside Polymer v1 elements
Steps for reproducing this issue (code):
Just use it
How to fix it:
Just set syntacticSugar variable in code to false

Polymer is currently in version 3, so does it happen in either V2 or V3 too?
Can you provide a JSFiddle example as "Just use it" is a completely useless comment.

...And just noticed that you're mentioning Velocity V1, that is completely unsupported now - check against V2 beta (though it should be identical as that also allows CoffeeScript style calling).

The only Polymer with the support of Chrome 49 (Last XP version) is v1. That's why YouTube still use version 1.

About jsfiddle:
https://stackoverflow.com/questions/34074772/can-i-use-polymer-on-jsfiddle (check example)
It looks like there is no way to make working Polymer jsfiddle example

...That literally tells you how to do it.


Explore - What Metrics are available and are meaningful to a user to measure popularity

Description: What Metrics are available and are meaningful to a user to measure popularity of a resource
All users have said that they would find know the popularity of a resource relative to the other resources would give them value and help them choose if they want to use a particular resource or to choose between two similar resources.
Users have mentioned Github stars and Docker Image pulls as two metrics they felt would have value.
Acceptance Criteria:
A list of which Metrics are the most meaningful.
Ideally something showing that Metric for each resource so we can get a sanity check from the Concourse team.

Concourse resources

https://github.com/concourse/git-resource
https://github.com/concourse/semver-resource
https://github.com/concourse/pool-resource

Community resources

https://github.com/telia-oss/github-pr-resource
https://github.com/frodenas/gcs-resource
https://github.com/carlo-colombo/telegram-resource
https://github.com/seadowg/hockey-resource

@carlo-colombo @monika-hrda This is great! Thank you both for this. I will incorporate the above into the next prototype.
Thanks!
Olwyn

@carlo-colombo @monika-hrda thanks for that. That's really useful. My thinking is we add the GitHub stars. It has come up in several interviews and it sounds like it isn't too expensive to add.
I like the idea of the downloads as well but I think that requires a slightly longer conversation.
Any thoughts objections?
I will accept and close the story.


FUNCTION HEX [REQUEST]

FUNCTION HEX DECODE/ENCODE
As there is now a possibility to use sockets, it would be interesting to put a function like this.

There is already one in the UTILITY block under the Conversion group I believe

There's no function for that yet.

como criar funçoes


Tutorial with option #1 doesn't run

When using option #1 (pre-generated API, openapi-sample project) to run the tutorial, I get the attached error. This happens when I click on "Transaction" item in the side menu.

I had the wrong path inside the Program arguments field when I started the Braid server. Instead of using ~/ I should use /home/my-user/:
localhost:10004 user1 test 10200 3 "/home/my-user/bootcamp-openapi/build/nodes/PartyA/cordapps"


Python 3.5 referenced

https://github.com/dbgrigsby/Postr/blob/84a4a56cba2a4955a760619e718d0583a6437f4d/Makefile#L10
This should be 3.6

fixed


Add custom per-block data fields

Contains
Allow mods to register additional 4, 8 or 16 bit data fields to be stored for each block along with the block IDs, biome data &ct. If data-fields of the same size are registered for disjoint sets of blocks, they may be made aliases of the same field, to save space.
This has a similar intent to #617, but there are several design changes:

Fields are registered in the java code, rather than module.txt.
The size may vary (all of the bit-sizes for which TeraArray implementations exist are available).
Every class for accessing block data has methods [get/set]ExtraData with an extra ID parameter to identify which extra data field, rather than having only one field accessible per worldProvider.

How to test
As this is a framework rather than user-facing thing, the easiest way to test that it works would be to load FlowingLiquids, which (on the customDataStorage branch), uses a custom data field to store the liquid flow data. There are non-trivial interactions with networking stuff, too, so if you want to generally test things, doing it on multi-player as well could be worthwhile.
Outstanding before merging
There's nothing I know of that needs to be changed before merging, but there are several things to be improved relating to this:

If there's leftover space in a large (8 or 16 bit) field, that may as well be combined with smaller fields like currently happens with fields of the same size.
If the built-in liquid data is relevant at all, it certainly at least deserves to be demoted to an extra data field, so it might as well be removed.
When 8 and 4 bit fields are serialised, they are padded to 16 bits for convenience. This could be avoided, with further effort.

Uh oh, something went wrong with the build. Need to check on that

Hooray Jenkins reported success with all tests good!

I like the idea behind this, but what about larger numbers?  32- and 64-bit fields can also be super useful, like for battery charge or nuclear power output.

@LadySerenaKitty Extra sizes of data field shouldn't be too hard to add, it's just that there aren't currently TeraArray implementations of those sizes. The cases you mentioned might be better dealt with through block entities, although it's plausible that some worlds might have enough batteries to make a sparse data array more efficient for that.

@4Denthusiast There are two behaviors I've seen in Minecrack with energy storage:

MASSIVE monolithic battery bank, usually used to power entire bases spanning several chunks
Lots of small battery banks (or even just singles) scattered everywhere (people's 1-chunk bases/houses in multikitty Minecrack)
Both of the above, mixed in same world (open cities of NPCs along with player-created towns)

MASSIVE monolithic battery bank, usually used to power entire bases spanning several chunks

This case is probably better handled with the BlockNetwork type approach: handle the connector points (nodes in a network) but otherwise treat the object as a whole, powered by a single entity tracking involved locations and no block data past the base block id.
In any case I'm glad the system can be extended to support larger data types anyway if needed - even if I'm not sure there'd be any use cases for it not better handled with alternatives 👍

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!

Although this has no directly visible effects, here are some cool pictures of the FlowingLiquids module, which it enables:
The column of bubbles as an ocean drains down a railgun hole:

The dent in the ocean's surface caused by another, similar hole:

Actually, I've noticed some weird errors in FlowingLiquids to do with saving and loading which are quite possibly bugs in this. I should check on that before this gets merged.

Tested this out after some fairly minimal fixes to modules in a mega-workspace - works really well! I hadn't even realized you included liquid flow pushing players so you can get nicely sucked into a maelstrom slowly filling up a newly railgunned chasm :-)
I did have to also merge in #3477 and then hit an odd issue I thought might relate to #3473 so I merged that to, although to no effect. Both of those test out as well though so I'm happy to merge everything in together, just need a couple things checked on. Surprisingly in headless multiplayer a crash occurs when a client instance joins:
23:42:41.578 [main] ERROR o.terasology.engine.TerasologyEngine - Uncaught exception, attempting clean game shutdown
java.lang.NullPointerException: null
	at org.terasology.flowingliquids.rendering.primitives.BlockMeshGeneratorDebugLiquid.generateChunkMesh(BlockMeshGeneratorDebugLiquid.java:85)
	at org.terasology.rendering.primitives.ChunkTessellator.generateMesh(ChunkTessellator.java:58)
	at org.terasology.rendering.world.RenderableWorldImpl.pregenerateChunks(RenderableWorldImpl.java:155)
	at org.terasology.rendering.world.WorldRendererImpl.pregenerateChunks(WorldRendererImpl.java:577)
	at org.terasology.engine.modes.loadProcesses.PrepareWorld.step(PrepareWorld.java:44)
	at org.terasology.engine.modes.StateLoading.update(StateLoading.java:243)
	at org.terasology.engine.TerasologyEngine.tick(TerasologyEngine.java:458)
	at org.terasology.engine.TerasologyEngine.mainLoop(TerasologyEngine.java:421)
	at org.terasology.engine.TerasologyEngine.run(TerasologyEngine.java:397)
	at org.terasology.engine.Terasology.main(Terasology.java:154)
23:42:41.578 [main] INFO  o.terasology.engine.TerasologyEngine - Shutting down Terasology...

BlockMeshGeneratorDebugLiquid is also changed in #3473 so I thought maybe that change was needed too but no difference. Other than then also having the API breakage from there affect FlowingLiquids :-)
In BlockMeshGeneratorLiquid in there you've got a BlockMeshPart basePart = block.getLoweredLiquidMesh(side); - is that fair to fix with block.getLowLiquidMesh(side); ? Game seemingly ran with that tweak applied

#3473 and the FlowingLiquids module both replace BlockMeshGeneratorSingleShape, so if FlowingLiquids is active, #3473 isn't necessary (and breaks some API that the module uses). I'll have to work out how to get FlowingLiquids working with that PR, which may involve some slightly non-trivial changes, but it can just be left out for now (none of this is meant to depend on it).
If the error you gave happened on the server, it would seem that it's trying to do some of the rendering stuff there despite the fact that it doesn't need to and hasn't loaded the resources it needs for that. The standard BlockMeshGeneratorSingleShape would avoid this (if this is actually the problem) by just ignoring that mesh, but that doesn't seem like the right solution. If it's on the client, I have no idea what would cause it to happen specifically in that situation. I've tried placing and viewing debugLiquid blocks both on the client and server of a non-headless server, and it seemed to work fine.

The reason this happens is that the server doesn't use debug-liquid itself, so it never registers it for a block ID. The client isn't able to register blocks itself, so it fails when it tries to assign debugLiquid its special renderer.

Merged with a workaround in FlowingLiquids for now, documented with #3491 to improve later :-)
Awesome work @4Denthusiast thank you so much!
I also submitted https://github.com/Terasology/TutorialAssetSystem/issues/5 for some tutorial work on this (although sadly the tutorial code itself needs fixing right now ...)


allow admins to delete events.

there's people, in particular beginners, who would submit world-spanning changesets, containing possibly just two edits, like one in New Zealand, and one in Alaska (or one in California, and one in South America ---- much more common).
it would be nice if we could tag the event as irrelevant, so that it will be filtered out in the overview: there's no need to welcome them, and there's no need for them to reply.

In most regions, they use the note for such use case (SEO spam, ...).

well, yes, sure, I can write a note, but it will still be in the way, distract the reader, alter statistics.

for July 2022 we have 3 such false positives in Panama, out of 9 matches. I welcomed 2 people, and I wish to make clear that it's still 4 to consider, not 7.

Since #808 a new mapper will appear in every region it is detected. This should fix the issue.


chore: fix a broken link "FrameworkHandle"

closes #26479

https://deploy-preview-26480--kubernetes-io-master-staging.netlify.app/docs/concepts/scheduling-eviction/scheduling-framework/#permit

/assign @kbarnard10


[Nima] Set running flag to false in shutdownhook

Currently shutdownhook doesn't set flag running to false.
Setting to false is useful because we can test that WebServer.isRunning()  in user custom hook

@wojtask9 Not sure I follow the use case you have in mind. Could you include a unit test? Also shutdown hooks are executed in an undefined order by the JDK.

@spericas
Not particular use case.
As you said shoutdown hooks are executed in an undefined order so there is no guarantee that this be useful but at least shows truth about current webserver status.
Also this align with method stopIt() where such flag is correctly assigned
https://github.com/helidon-io/helidon/blob/main/nima/webserver/webserver/src/main/java/io/helidon/nima/webserver/LoomServer.java#L204


⚠ Mobitopup has degraded performance

In 0c1b9a8, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3822 ms

Resolved: Mobitopup performance has improved in 27ea977 after 8 minutes.


ParseCloud.CallFunctionAsync doesn't always return

We are noticing sometimes that ParseCloud.CallFunctionAsync doesn't always return when internet is bogged down or reception is low on the device. The cloud call works most of the time but it does just sit there forever on occasions. One the app is restarted then it start functioning again.
The function we are calling is just returning static data from some tables so the amount of data and time to retrieve data is always the same so it shouldn't be triggering an internal time out. But even if it does, it should eventually through an exception, return blank or time out rather than just sit there indefinitely.
Is this is a bug with the Parse Unity SDK?

Seems to me like this is a bug in Unity's network stack, as we don't do anything specific related to network connectivity in the SDK.
If you can provide some example code that shows us that this issue is in our SDK and not Unity's, I'd be glad to help. Otherwise, there's unfortunately not much we can do.

It's a concern that for some users the app will remain on a loading screen indefinitely.  We will attempt to implement our own timeout checks for calls we find it occurs on. We plan on reattempting the call after 10 seconds if the return function is not reached in that time frame. Time will tell if it resolves the issue as it is difficult to reproduce. However, there are many locations this may occur on in the future. Is there any such timeout logic implemented in the Parse Unity API's server cloud call CallFunctionAsync?

So far the time out check we added seems to have resolved the issue. It would be great if this was implemented as part of the SDK so the I don't have to add this time out logic for each CallFunctionAsync call that we make in the code. If just basically reattempts the call if more than 10 seconds pass without the callback function being triggered.

@santacrs I'm not sure how we can do that, as each of our REST requests that we make are intended to have side-effects (and cannot be easily cancelled/reverted), re-sending the request could cause a lot of unforeseen issues.
I would strongly suggest filing a bug with the Unity folks instead, as it seems there's a fundamental flaw with their underlying web request library.

I can't speak to the underlying cause.
I can see an automatic reattempt being an issue if it's modifying data but if it's retrieving data as we are it either returns the data or it doesn't. A safer option is just have it force the callback with an exception/error state if the timeout occurs and then the developer can handle the scenario as needed.


HPC Pack 2016 + Azure ARM documentation is lacking

Hi team,
Can we include directly the following release note and how-to on the Azure documentation:
https://social.microsoft.com/Forums/en-US/f609f8cb-8838-49b8-9501-d23904333079/microsoft-hpc-pack-2016-update-1-now-available?forum=windowshpcitpros
The overall documentation on how to integrate HPC Pack 2016 with the Azure Resource manager is confusing and mixes scenarios only supported in the Classic deployment in the current online doc
We should have a documentation directly in Azure (and indexed) for the following topics:

Deploy a cluster on Azure (ok as of today)
Configuring Azure with HPC Pack 2016 (only available in PDF http://download.microsoft.com/download/B/D/B/BDB8782A-FAAF-457D-AF3D-0B157FEEDF4C/Burst to Azure IaaS nodes in HPC Pack.pdf)
configuring HPC Pack 2016 with Azure Batch (only available in PDF : http://download.microsoft.com/download/B/D/B/BDB8782A-FAAF-457D-AF3D-0B157FEEDF4C/Burst to Azure IaaS nodes in HPC Pack.pdf)
Docker support in HPC Pack

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

ID: f4bfc22c-1197-0b37-5f75-5803dbcfaa32
Version Independent ID: c5c8c5d6-2786-7746-7635-9d22003570a8
Content: Windows HPC Pack cluster options in Azure
Content Source: articles/virtual-machines/windows/hpcpack-cluster-options.md
Service: virtual-machines-windows
GitHub Login: @dlepow
Microsoft Alias: danlep

@nyuen Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@nyuen - Nicolas, thanks for the helpful feedback and suggestions. We already have work items in our doc queue to publish latest HPC Pack 2016 update 1 docs and to retire or refactor the HPC Pack classic deployment content in the Azure docset so that we highlight ARM deployments. In the short term, you have a good suggestion to modify the  "HPC Pack cluster options in Azure" article to highlight HPC Pack 2016 Update 1 options, even though they're mostly available now in preview download form. I'll draft an update for that.
#in-progress

Fixed in https://github.com/MicrosoftDocs/azure-docs-pr/pull/41108
Additional work in open work items
#please-close


Application crashes when running SenTestingKit Tests with RestKit

When using SenTestingKit (and in my case ARC) the application crashes as soon as Restkit is setup inside the Application Delegate in didFinishLaunchingWithOptions during the test phase. The crash occurs inside NSManagedObject (ActiveRecord):
+ (NSManagedObjectContext*)managedObjectContext {
    NSAssert([RKObjectManager sharedManager], @"[RKObjectManager sharedManager] cannot be nil");
    NSAssert([RKObjectManager sharedManager].objectStore, @"[RKObjectManager sharedManager].objectStore cannot be nil");
    return [[[RKObjectManager sharedManager] objectStore] managedObjectContext];
}

*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: '[RKObjectManager sharedManager] cannot be nil'

The application crashes although the RKObjectManager shared instance is set explicitly using [RKObjectManager setSharedManager:objectManager];. When the debugger halts on line 31 of NSManagedObject (ActiveRecord), the static, shared instance of RKObjectManager is NOT nil but the assert causes the Application to crash anyway.
Maybe it's related to ARC, but it also crashes when i set a strong reference to the RKObjectManager in the Application Delegate.

It occurs when linking the RestKit library as with the test bundle. But when I don't link it with the test bundle the unit tests fail when running in CLI mode (run within jenkins). Thats because RestKit.lib is not loaded the NSURL+RestKit Category is not loaded and therefore it results in: -[NSURL queryParameters]: unrecognized selector sent to instance 0x804f440
So this is not really a bug inside RestKit, but rather a linking issue.

Does use of all_load linker flag fix it?

Same problem here, running OCUnit tests with Jenkins. Tests run fine on my machine, but crash on the CI server.
Any update?

I have just brought up a Jenkins environment over the last couple of days. I'll add RK to the system one evening this week and update the issue.

Ok, thanks.


(SP: 2) Check profile edit Confirms

Confirmations should be absent when switching between the Profile, professional info & notifications tabs.
If the user exits the editing page, a confirmation should appear

Epic #2016

not relevant anymore


Sidebar logo and title resizing issue with long titles

Description
The Sidebar's Logo and Title have an inconsistent appearance based on the length of the title. Specifically:

When the title is too long, the icon appears very small and is not resized adequately to match the title
When the title is shorter, the logo and title appear well-proportioned

Expected Behavior
The logo should resize proportionally to maintain a consistent and visually appealing appearance regardless of the title length.
Environment
Browser: Chrome, OS: Windows, Screen res: 1920x1080
Example
Title Too Long:

Title shorter:

@popovayoana But the logo size on both pictures is the same and I'm pretty sure it stays the same no matter the title. If you mean the logo should take the full height of the title, e.g. if the title spans two lines, then I don't think it's a good idea and I don't think it's even possible to achieve what you want.
Why? When you increase the height, you also increase the width, meaning the title will have even less space and will now take e.g. 4 lines... To illustrate:
Original:

Logo size increased:

What we can do is make the title font a bit smaller - I think it's way to large anyway.

FIXED
@popovayoana Let me know if you still thinks it doesn't look good


Forbid adding well-known gardener finalizers on Shoot creation

How to categorize this PR?

/area robustness
/kind enhancement
What this PR does / why we need it:
This PR adds validation to forbid users from creating Shoots with well-known gardener finalizers added.
Which issue(s) this PR fixes:
Fixes #8208
Special notes for your reviewer:
/cc @dguendisch
Release note:

NONE

Look good. Just has one question.
This implementation will anyway allow adding the gardener.cloud/reference-protection right? Only while deletion it will check if the gardener finalizer is present or not.
https://github.com/gardener/gardener/blob/89b18393fb860cf1b5432fec0a13e69b2cd7b723/pkg/controllermanager/controller/shoot/reference/reconciler.go#L80-L88
Adding the gardener.cloud/reference-protection finalizer is the Shoot reference reconciler's task and we should not allow users to add the finalizer.
WDYT?

Looks good. Just has one question. This implementation will anyway allow adding the gardener.cloud/reference-protection right? Only while deletion it will check if the gardener finalizer is present or not.
https://github.com/gardener/gardener/blob/89b18393fb860cf1b5432fec0a13e69b2cd7b723/pkg/controllermanager/controller/shoot/reference/reconciler.go#L80-L88
Adding the gardener.cloud/reference-protection finalizer is the Shoot reference reconciler's task and we should not allow users to add the finalizer. WDYT?

If there's no referenced object, the controller removes the finalizer on creation as well, even if it's not added by it. See
https://github.com/gardener/gardener/blob/89b18393fb860cf1b5432fec0a13e69b2cd7b723/pkg/controllermanager/controller/shoot/reference/reconciler.go#L115-L120

Sure, but wouldn't it be more clean to simply disallow gardener.cloud/reference-protection as well (instead of relying on the controller's current behaviour) as it is not intended to be set by a user creating a shoot? Could also help to "educate" the user that he did some copy&pasta :).

Sure, but wouldn't it be more clean to simply disallow gardener.cloud/reference-protection as well (instead of relying on the controller's current behaviour) as it is not intended to be set by a user creating a shoot? Could also help to "educate" the user that he did some copy&pasta :).

Okay, I chose not to do it because now the constant is in the gardener-controller-manager controller package and using it would pull in some unwanted dependencies. Will now move it to a common place and use it.

/cc @timuthy


Implement Chunk Extension for IAsyncEnumerable

Maybe we should open a PR/issue in System.Linq.Async since this method exists in the BCl for IEnumerable

Open Issue @ https://github.com/dotnet/runtime/issues/27449

That's the issue for IEnumerable, right? I think we should open an issue for System.Linq.Async in the dotnet/reactive repository.

I pasted the wrong link: https://github.com/dotnet/reactive/issues/1573

Merged in funcky3


Set a custom build description.

Summary:
This is a test to display custom information on the build page.

Won't do.


Slider thumb at 0 zooms incorrectly on click in Safari

Hovering over the slider thumb should zoom it up as per the spec. That effect is almost working, but the counter inside the thumb ends up offset diagonally toward the bottom right, and not quite large enough:

This issue was observed in Safari Version 8.0.7 (10600.7.12) on OS X Yosemite Version 10.10.4 (14E46). It doesn't affect Chrome on the same machine.

Thanks for the bug report, @kitgrose!
It took me a while to reproduce, because it only seems to happen on non-retina screens, but I can see it now when clicking on the thumb.


Organize PostgreSQL extension functions/types in database navigator

When PostgreSQL database has an extension all functions and types from that extensions are shown together with user functions/types in navigator. This is not a problem for extensions with few functions/types but for postgis extension navigator becomes useless in finding my own functions because they are all grouped and sorted together and postgis has 700+ functions.

In navigator under connection node there is an Extensions folder that lists all enabled extensions.
Would it be possible to show extension functions and types here as child nodes?

If it is possible and somebody does not like this organization then maybe add an toggle option for connection to show extension functions/types under extensions or keep it like it is now.

I personally like to just hide extension-installed functions most of the time so I've written this query to generate a string to filter them out.
Docs on how to configure (global) filters.


Add bokeh tab

Hi, in the backend section, I thought we should also show bokeh since people might not read the document in order.

I have a PR with that too. Same for data backend. There should be a Pandas example.
Also because the home page is not great for communicating like the video you are creating, so it's important everything works also if things are not shown too to bottom.

I'll merge this one and then rebase yours @MarcSkovMadsen.


No Option for Regional CPU core Increase

I have run out of Regional CPU cores but not 1 specific Core type. There is no Option to only increase the regional core count.

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

ID: fe04a05a-f753-ee1f-5171-fd2a81a030f9
Version Independent ID: 1507a7ac-11cc-81a7-e932-9e8ef9644e2b
Content: Azure Resource Manager vCPU quota increase requests
Content Source: articles/azure-supportability/resource-manager-core-quotas-request.md
Service: microsoft-docs
GitHub Login: @ganganarayanan
Microsoft Alias: gangan

@steynr Thank you for your feedback! We will investigate and get back to you with the relevant info.

@steynr you are correct. When asking for more cores they will apply to an entire region

From there you can request a core increase for specific series.
As a side note, you can select multiple series so you can pick the ones you use the most. You can also request that the engineer working on your ticket increase the regional cores as a whole.
You can find a list of all the limits we have set in Azure
https://docs.microsoft.com/en-us/azure/azure-subscription-service-limits
For anything else we simply suggest you open a ticket to the subscription and billing team. Even if the option is not specifically listed the engineers know how to handle additional requests so you can just add that information.
I would suggest you leave your feedback to support the option to request regional cores as a whole on
UserVoice that way the engineering team can see it and consider adding that functionality.
I will close this for now as there is nothing to be added to this specific doc.
CC @ganganarayanan as she owns this experience in the portal.

@ganganarayanan @MicahMcKittrick-MSFT I need Total Regional vCPUs increased and created a SR following comments in the docs thread that suggested increasing a specific SKU (which cannot be left blank) would also increase Total Regional vCPUs. Unfortunately SR granted the SKU increase instantly and closed the SR with no way to communicate back to an engineer.
I then tried an airs ticket, but the priority is not selectable.
Could not find a way to re-open or get a response from a closed SR so just made another one and did not choose Compute - but instead chose other.
Side note - the preview experience would not allow saving a description with a SR pasted in - said it recognized a credit card number.

@kfasick Can you please provide the SR number that you created your initial case to increase specific SKUs and the total regional limit did not increase? I would like to talk to the quota team internally and ask them to look into it.


Improve csv sniffer accuracy

As seen in #98, if the lines are broken, passing many lines to the CSV sniffer can actually decrease the detection accuracy =>  by reducing the number of lines passed, the header weights more => the detection is more accurate.

Coverage remained the same at 82.919% when pulling 47aed1b4d78170f2643fb416641438189a21dcfd on hotfix/improve_sniffer_accuracy into b3693836aface5df7415ba3abc0bfd2bd5fc2e11 on master.


Added getMonthLabel to ContributionGraph

This is in response to issue #250 to allow for a custom function (getMonthLabel) to be passed as a property to the ContributionGraph component that renders a custom label for each month; if the function is not provided, the "old" MONTH_LABELS constant from the project is used by default

Thank you for your contribution! This is now available in v5.0.0

Thank you too for the quick review, happy to help!


design: Improve design of Subscribe/Unsubscribe buttons in message view.

Before:

After:

Fixes #5196

So I have a few thoughts visually:

This is mostly a different bug, but I think we probably only want subscribe/unsubscribe buttons at the bottom, not every time it changes.  But I think this means it's not super important to have a pill shape around the separators.
Given that, I think stylistically, the "subscribed to stream Verona" notices probably make the most sense to be more like the date separators -- i.e. something that looks more like an hrule.  And maybe the text should be "You subscribed to stream Verona"?

Not sure what's up with the duplicate subscribe/unsubscribe sections. Maybe it's worth creating a separate issue for it, since it's been happening pretty consistently on my end (with or without my changes)

Made changes accordingly.

Yeah, we should re: duplicate subscribe/unsubscribe sections.  Can you open one?

That design looks good, except that I'd probably not do the capitalized style for the notice?
I'm willing to take this one for the 1.6 release.

New UI:

This looks great! The only thing is that this seems to introduce another button style. Try using the .new-style .button.sea-green combo.

I just tried switching the buttons, and it worked great.  Merged as 11e68606b4fba1520bc0a99d97b361c14dfa9cea, thanks @synicalsyntax!  I also did a half dozen commits fixing #5237.


Focus on one type of POI

My use case is for bicycle parking, but it would probably help on other POI types as well.
The idea would be to allow users to focus on only one type of POI, using some kind of filters / settings.
This would allow for much more efficient mapping in a "completionist" approach when working on making sure that some areas aren't missing even 1 POI of a specific type (could be used for bicycle parking, benchs, fire hoses, shops...
I'm a believer that some people who are not OSM-informed might be highly motivated to work on one type of data in the map and organize their communities to micro map whole areas for their special interest, provided they have the tools to do so.

Nice idea, and would probably be doable after #119. Closing this in favor of that.
In general, I'd say the editor is for mapping things directly around you, so other object types should not overwhelm a mapper if they want to focus on a single one.


ATS 10.0.0, regex_revalidate plugin does not work

hello,
i am using regex_revalidate plugin to invalidate cache (revalidation, refetch)
so, i set like these
plugin.config is below
xdebug.so --enable=x-remap,x-cache
regex_revalidate.so -c regex_revalidate.config

regex_revalidate.config is below to ### refetch
https://test1111\.kinx\.net/.*<PHONE_NUMBER> MISS
https://test1111\.kinx\.net/dog/*\.jpg<PHONE_NUMBER> MISS

but does not work refetch at all
even, revalidate does not work
https://test1111\.kinx\.net/.*<PHONE_NUMBER>
1725440551 is   Wed, 04 Sep 2024 9:02:31 GMT
i checked event log and
result of curl -v -H "Content-Type: image/jpeg" https://test1111.kinx.net/dog/dog2.jpg
* Using HTTP2, server supports multiplexing
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* Using Stream ID: 1 (easy handle 0x55f6f746feb0)
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
> GET /dog/dog2.jpg HTTP/2
> Host: test1111.kinx.net
> user-agent: curl/7.81.0
> accept: */*
> content-type: image/jpeg
>
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* old SSL session ID is stale, removing
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
< HTTP/2 200
< x-amz-id-2: lZMgkp573RoUxPSzDhBpJSrah7zC/GnGy1NNA/Oblt5tcFH23QVKfQoqMP1OYhaHi2X0qlEGx08=
< x-amz-request-id: NJZPNZDX80J5JJWG
< date: Wed, 04 Sep 2024 06:41:38 GMT
< last-modified: Mon, 29 Jul 2024 08:09:26 GMT
< etag: "a4f413872f433d450711d5a5ae30bc67"
< x-amz-server-side-encryption: AES256
< accept-ranges: bytes
< content-type: image/jpeg
< server: ATS/10.0.0
< content-length: 9111
< age: 64830
<
Warning: Binary output can mess up your terminal. Use "--output -" to tell
Warning: curl to output it to your terminal anyway, or consider "--output
Warning: <FILE>" to save to a file.
* Failure writing output to destination
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* stopped the pause stream!
* Connection #0 to host test1111.kinx.net left intact

age is 64830, that says regex_revalidate does not work
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
and i did upper situation again
plugin.config
xdebug.so --enable=x-remap,x-cache
regex_revalidate.so -c regex_revalidate.config -l rr_log -f rr_state

regex_revalidate.config
https://test1111\.kinx\.net/.*<PHONE_NUMBER> MISS
https://test1111\.kinx\.net/dog/*\.jpg<PHONE_NUMBER> MISS

https://test1111\.kinx\.net/dog/*\.jpg<PHONE_NUMBER>

rr_log
root@ats-dev2:/opt/ts/var/log/trafficserver# tail -f rr_log.log
20240905.10h33m31s Current config:
20240905.10h33m31s https://test1111\.kinx\.net/.* epoch:<PHONE_NUMBER> expiry:<PHONE_NUMBER> result: MISS
20240905.10h33m31s https://test1111\.kinx\.net/dog/*\.jpg epoch:<PHONE_NUMBER> expiry:<PHONE_NUMBER> result: MISS
20240905.10h34m41s Current config:
20240905.10h34m41s EMPTY
20240905.10h36m41s Current config:
20240905.10h36m41s https://test1111\.kinx\.net/.* epoch:<PHONE_NUMBER> expiry:<PHONE_NUMBER> result: MISS
20240905.10h36m41s https://test1111\.kinx\.net/dog/*\.jpg epoch:<PHONE_NUMBER> expiry:<PHONE_NUMBER> result: MISS
20240905.10h37m41s Current config:
20240905.10h37m41s EMPTY
20240905.10h40m41s Current config:
20240905.10h40m41s https://test1111\.kinx\.net/dog/*\.jpg epoch:<PHONE_NUMBER> expiry:<PHONE_NUMBER> result: STALE
20240905.10h42m41s Current config:
20240905.10h42m41s EMPTY

maybe regex revalidate plugin works, but not to do refetch or revalidate at all
give me any information please~
help me please!
thanks a lot in advance
@bneradt @calavera @janfrode @biilmann @jpeach @sekimura @cmcfarlen @bryancall @mlibbey

hello
i am sorry for upside issue.
regex revalidate works well setting with origin server url  NOT CLIENT URL (on remap.config, ex) map
it works
https://jinyong-test-3.s3.ap-northeast-2.amazonaws.com/dog/*<PHONE_NUMBER> MISS
but new issue
regex revalidate plugin doc says, we support PCRE Style Regex Rule
but, these are does not work
https://jinyong-test-3.s3.ap-northeast-2.amazonaws.com/dog/*.jpg<PHONE_NUMBER> MISS
https://jinyong-test-3.s3.ap-northeast-2.amazonaws.com/dog/*\.jpg<PHONE_NUMBER> MISS

please help, give me some information !!
thanks a lot in advance
@bneradt @calavera @janfrode @biilmann @jpeach @sekimura @cmcfarlen @bryancall @mlibbey

I haven't worked with it too much, but I'm guessing you want .*.jpg rather than *.jpg. As it is, dog/*.jpg will match "dog" followed by 0 or more /, followed by some character, followed by "jpg". Thus:
https://jinyong-test-3.s3.ap-northeast-2.amazonaws.com/dog/.*\.jpg<PHONE_NUMBER> MISS

Can you try that and see whether it works?

hello @bneradt
you comment is absolutely right
PCRE Regex . is any so .*.jpg is right.
/* had worked well, so i thought /*\.jpg would work well on regex revalidate plugin
https://jinyong-test-3.s3.ap-northeast-2.amazonaws.com/dog/.*\.jpg<PHONE_NUMBER> MISS
root@ats-dev2:/opt/ts/etc/trafficserver# curl -v -H "X-Debug: X-Cache" http://localhost/dog/dog1.jpg
*   Trying <IP_ADDRESS>:80...
* Connected to localhost (<IP_ADDRESS>) port 80 (#0)
> GET /dog/dog1.jpg HTTP/1.1
> Host: localhost
> User-Agent: curl/7.81.0
> Accept: */*
> X-Debug: X-Cache
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< x-amz-id-2: TzuD3HsHdmY2BrVo4jbAeu0DRBy7FfjbGx4tXcBj2/28UKu+p6HaDv3I9nElFRymKQk2xd9zEu2/3o8UGEOPPg==
< x-amz-request-id: H5J2MQ2YX2GCYBHQ
< Date: Mon, 09 Sep 2024 00:36:50 GMT
< Last-Modified: Thu, 05 Sep 2024 05:26:20 GMT
< ETag: "a82df4e00d6474b411fe18b26ece5971"
< x-amz-server-side-encryption: AES256
< Accept-Ranges: bytes
< Content-Type: image/jpeg
< Server: ATS/10.0.0
< Content-Length: 64396
< Age: 0
< Connection: keep-alive
< X-Cache: miss
<
Warning: Binary output can mess up your terminal. Use "--output -" to tell
Warning: curl to output it to your terminal anyway, or consider "--output
Warning: <FILE>" to save to a file.
* Failure writing output to destination
* Closing connection 0

thanks for your scrupulous help @bneradt !!!!!!!
have a nice day


Websocket 303 in Safari

I've implemented the Websockets like described in Endpoints section of the Silhouette docs, however when using it I always get "unexpected response code: 303" in Safari. In Chrome everything works fine. Any ideas what this can be caused by?
When I debug my application I see that when using Safari the app always jump into the left hand Left(r) line instead of Right(MessageSocketActor.props(account) _).
(Before I use SecureSocial and I didn't have any issue with Websockets.)
EDIT: I'm using the CookieAuthenticator if that's relevant.
EDIT2: Tried SessionAuthenticator too, but doesn't help.

Mhh.., maybe Safari cannot handle the cookies for the Websocket!?
Does SecureSocial can handle Websockets?

Yes it does and auth worked fine there.

Could you please debug the request to see if the cookie is located in it?

Just verified it, it’s there.

Am 22.07.2015 um 18:52 schrieb Christian Kaps<EMAIL_ADDRESS>Could you please debug the request to see if the cookie is located in it?
—
Reply to this email directly or view it on GitHub https://github.com/mohiva/play-silhouette/issues/393#issuecomment-123789556.

It would be nice if you could find the root cause. I do not have a Safari to reproduce this issue.
I would set some breakpoints in the retrieve method of the authenticator to see if the authenticator could be extracted from cookie. Do you have enabled fingerprinting for the authenticator?

I'm happy if I can help you. However I spend a lot of time finding the issue myself, but I didn't yet - guess I've to less knowledge about what's going on in your library.
About retrieve: just debugged it, extraction works - the Failure(error) branch of the match statement is never visited.

Do you have set the logging level to debug? You have not answered my question regarding the fingerprinting. Where does it step out? On line 206, on line 207 or on line 209?

Right now fingerprinting and SessionAuthenticator is used (since I hoped this would, however there is no change).
During the establishment of the websocket via Safari the following lines are written to the log:
[info] c.m.p.s.i.a.SessionAuthenticatorService - [Silhouette][session-authenticator] Fingerprint Some(E418F2D4251E3F3DEC7C502F899B5DA0FCDFF498) doesn't match authenticator: SessionAuthenticator(LoginInfo(credentials,me@michaelhohl.net),2015-07-22T21:18:31.130+02:00,2015-08-21T19:12:38.536+02:00,Some(432000 seconds),Some(C626E55359030FB4CEB2D1D29455D9982CDEB703))
[debug] a.c.ConnectionController - [Silhouette] Unauthenticated user trying to access '/api/connection/stream'
[info] c.m.p.s.i.a.SessionAuthenticatorService - [Silhouette][session-authenticator] Fingerprint Some(E418F2D4251E3F3DEC7C502F899B5DA0FCDFF498) doesn't match authenticator: SessionAuthenticator(LoginInfo(credentials,me@michaelhohl.net),2015-07-22T21:18:31.130+02:00,2015-08-21T19:12:38.536+02:00,Some(432000 seconds),Some(C626E55359030FB4CEB2D1D29455D9982CDEB703))
[debug] a.c.ChannelController - [Silhouette] Unauthenticated user trying to access '/api/channel/stream'

It seems that Safari sends different headers for Websocket and default HTTP requests and therefore the fingerprint doesn't match. So either you define a new FingerprintGenerator that can handle the difference or you disable fingerprinting. You should debug the headers currently implemented in the DefaultFingerprintGenerator to see which headers are different.

You are right they differ. Safari doesn't send ACCEPT_LANGUAGE and ACCEPT_CHARSET on websocket request.

And yes, disabling fingerprinting fixed my problem. Thank you for your help and for developing that great library. I previously used SecureSocial, but the project seemed to be dead and manually fixing bugs wasn't fun too, since code wasn't structured that well. I guess the transition to Silhouette was a good decision.

Nice to hear that!

Just for information I also had the issue for Microsoft Edge. I solved by creating a FingerprintGenerator that doesn't take into account ACCEPT_LANGUAGE and ACCEPT_CHARSET.


add package: musicplayerplus

closes #670

Fair warning to anyone who might be interested in using this program:
It has a ton of dependencies. And it installs even more upon first run.


Remove use of goog.array.isEmpty

A small step to remove our dependency on the Closure Library (see #4128)

LGTM

Thanks for the review


custom class instance returned by asyncdata

Version
nuxt-edge
Reproduction link
https://github.com/syffs/nuxt-asyncdata-class-instance-issue
Steps to reproduce
see the repro link: is there some workaround to make this work?
ssr works well, but client rendering fails (as methods are undefined)
all the hooks beforeCreate, created, mounted in the world are unfortunately unhelpful here, because computed property gets somehow called before
What is expected ?
being able to pass a class instance somehow
What is actually happening?
SSR JSON serialization ignores all the methods, but I'm unable to reinstate the class instance on client side with some hook.
Additional comments?
it is not really a bug, but cmty questions are ignored.

This bug report is available on Nuxt.js community (#c6906)

The problem hasn't been solved?

@koteezy AFAIK, no
Even though, it's not ideal, I simply avoided the issue by passing data only and externalizing methods

Hi @syffs
Well, I found a workaround by I don't recommend it.
<template>

    {{ formattedValue }}

</template>

But this is custom to every class you have, either you use data for instances or you create them in the created hook, for instance:
<template>

    {{ formattedValue }}

</template>


Share to a FB page

Is it possible to share to page that a FB profile admis?
When share dialog opens it does not shows a dropdown where you can switch page,group,...
Do we need to enable something, somewhere...any thoughts, anyone?
Thank you!

I would ask this on stackoverflow, clearyl not an active enough community for good answers, sorry. But if you learn something please report back for future people!


Viper not supporting Boolean variables from environment variable

Viper not supporting Boolean variables from environment variable.
The possible solution currently we use internally is to pass strings...

I tested with etcd.embedded (SKYDIVE_ETCD_EMBEDDED=true), it worked. What variable are you testing with ?

Closing it. Feel free to reopen it if the problem still occurs


Translate Event type API from SAM template.yaml

I'm trying to convert a Lambda Function from a SAM CLI template to Terraform:
Resources:
  MessageTransformerFunction:
    Type: AWS::Serverless::Function # More info about Function Resource: https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#awsserverlessfunction
    Properties:
      PackageType: Image
      CodeUri: MessageTransformerFunction
     [...]
      Events:
        MessageTransformer:
          Type: Api # More info about API Event Source: https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#api
          Properties:
            Path: /transform
            Method: get

How would I convert the Events section? I believe it's related to the aws_lambda_event_source_mapping resource but I do not see any examples that are similar to my use case.

This turned out to not be relevant to my problem which was related to setting runtime, main class in a Java Lambda uploaded by Image instead of the default Zip.


Incorrect path to webMIDIAPIShim in tests/index.html. Incorrect handling of midiOutput

I'm very new at this, so please bear with me and let me know if I'm doing anything incorrectly.
I noticed that, in the tests/index.html, the path to the SHIM is incorrect.
Also the assignment of the output port using midiAccess.outputs is incorrectly handled - the latest SHIM and the native Web MIDI API in Chrome handle .outputs as a "maplike Interface", not a function (in fact Chrome throws an error in ln 188 in the original file saying that outputs is not a function).
I've forked the project and made a few changes to this file. I'll do a pull request. Again, let me know if this is not what I'm supposed to do :)

I did a pull request fixing this issue in test/index.html. Closing...


Topic error passed to cb

I noticed when performing callback-style tests with Vows and Zombie that errors that occurred midway through the test sequence (e.g. clicking a nonexistent button) weren't being caught in my assertions. Instead, the value argument passed to my Vows assertions were the Error objects being thrown by Zombie.
It turns out that Vows swallows topic Errors and passes them down as values; this behavior is confusing for callback-style tests, as the error argument comes through as null but the value argument is the error.
The change I'm introducing adjusts that behavior to make sure that callback-style tests receive the Error in the expected argument position. I've added a test as well.

This has been already fixed in 6f84e3b535239e8b36adfc656510b99b37d78859 (test in 239df6022e4ba6951fed5dce9fb3c3eb0a81ec67).

The change you mentioned unfortunately creates a weird issue for async-style tests: the error argument is null, while the topic argument actually contains the error. Please see my test at 5bbe7a1.


ModelCheckpoint tries to remove already removed checkpoint in DDP mode

🐛 Bug
When training in DDP mode with ModelCheckpoint callback, the train process fails, when ModelCheckpoint callback tries to remove previous checkpoint. I assume that it was already deleted by another process.
To Reproduce
Steps to reproduce the behavior:
Run training with "ddp" backend and ModelCheckpoint callback with save_top_k={some_number}
  File "/home/myuser/miniconda3/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 19, in _wrap
    fn(i, *args)
  File "/home/myuser/miniconda3/lib/python3.7/site-packages/pytorch_lightning/trainer/distrib_data_parallel.py", line 342, in ddp_train
    self.run_pretrain_routine(model)
  File "/home/myuser/miniconda3/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 830, in run_pretrain_routine
    self.train()
  File "/home/myuser/miniconda3/lib/python3.7/site-packages/pytorch_lightning/trainer/training_loop.py", line 343, in train
    self.run_training_epoch()
  File "/home/myuser/miniconda3/lib/python3.7/site-packages/pytorch_lightning/trainer/training_loop.py", line 452, in run_training_epoch
    self.call_checkpoint_callback()
  File "/home/myuser/miniconda3/lib/python3.7/site-packages/pytorch_lightning/trainer/training_loop.py", line 737, in call_checkpoint_callback
    self.checkpoint_callback.on_validation_end(self, self.get_model())
  File "/home/myuser/miniconda3/lib/python3.7/site-packages/pytorch_lightning/callbacks/model_checkpoint.py", line 204, in on_validation_end
    self._do_check_save(filepath, current, epoch)
  File "/home/myuser/miniconda3/lib/python3.7/site-packages/pytorch_lightning/callbacks/model_checkpoint.py", line 221, in _do_check_save
    self._del_model(delpath)
  File "/home/myuser/miniconda3/lib/python3.7/site-packages/pytorch_lightning/callbacks/model_checkpoint.py", line 121, in _del_model
    os.remove(filepath)
FileNotFoundError: [Errno 2] No such file or directory: {PREVIOUS_CHECKPOINT_NAME}

Expected behavior
I expect that ModelCheckpoint callbacks from different DDP processes will not concurrent with each other in saving/deleteng files.
I fixed by rewriting _del_model method of ModelCheckpoint callback:
class DDPModelCheckpoint(ModelCheckpoint):
    def _del_model(self, filepath):
        try:
            os.remove(filepath)
        except Exception:
            pass

Environment

PyTorch Version: 1.4
OS: Ubuntu 18.04
How you installed PyTorch - conda
Python version: 3.7
CUDA/cuDNN version: 10.2
GPU models and configuration: 2x2080Ti
pytorch-lightning version: 0.7.1

Your suggestion to pass on an Exception is not the best, at least you should make it the specific error, i.e., FileNotFoundError. But in this case, I suggest to do simply
if os.path.isfile(filepath):
    os.remove(filepath)

Is there also an issue with saving? Does it save/overwrite the file in multiple processes?

I encountered that one too. From my perspective, model updates should be happening within main worker only (master worker). However, I guess each workers lightning created is trying to delete their own checkpoints. However, slave worker never created one (and it shouldn't be). I solved the problem in a similar way with @belskikh 's workaround but it did not feel right and downgraded to 0.6.0.

@awaelchli of course it is not the best (it may be the worse, actually) solution
It is just a fast workaround, waiting for solid fix
I agree with logic, when only one ModelCheckpoint callback should save/delete weights, because all weights are the same on all nodes at the end of the training step.
It can be done somehow like this:
....
    def _del_model(self, filepath):
        if self.main_worker_rank == dist.get_rank():
             # do delete

And the same for the saving code.

@williamFalcon should Lightning do checkpoints only on rank 0? It could be a problem if writing to a shared filesystem between nodes. AFAIK the loggers already do that by only logging in process 0.

I do not think, that it should do it only on specific rank, I think user should have ability to specify rank (node), where checkpoints will be saved

Your suggestion to pass on an Exception is not the best, at least you should make it the specific error, i.e., FileNotFoundError. But in this case, I suggest to do simply
if os.path.isfile(filepath):
    os.remove(filepath)

this does not work in async, I have observed many times that the if pass for both but then when you try really delete it, it is missing for one of them...

should checkpointing be done in only one process then, like loggers?

yup

well, ModelCeckpoint doesn't have a rank...


Test image background

Test applying styles to images inline
Fixes #5072

@joaofnfernandes , the 10th commit should do the trick. Please confirm: /machine/overview.md


Use TouchableNativeFeedback for Button on Android

See: react-native-button. This would allow getting material ripple effect on Android.

Closing this issue due to inactivity


Support Templater templates

As for now the only way to use Templater template would be to use Folder templates in Templater.
The downside is - there is no Journal related into in frontmatter at time when Templater plugin processes template content.

Released in 1.3.0


feat(notification service): implement notification service using gmail application passwords

Pull Request

Description
I implemented a notification service for users using passwords for gmail applications to rule out problems with APIs. This PR replaces the Mailgun API implementation due to problems.

Type of Change

[X] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality
to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Task linked: CU-8689mr891 Order confirmation


disconnect error

Hello Takatoshi Kondo,
I have been using the mqtt_cpp library and I it works very well and I am very happy with using it. There is however one thing I noticed. In asio, if you call 'close' on a socket, the read or write operation callback will be called with a error code 'boost::asio::error::operation_aborted'.
In your code, if I call disconnect on the mqtt client connection, the disconnect handler will be called. However I do not know in this callback if this was a willful or unwillful disconnect. If an error/remote disconnect occurs, for example, I want to start a reconnect operation. If no error occurs (shutdown is called from within the code), I just want to cancel the connection and shutdown the application cleanly.
Is it possible to add an error code to the disconnect operation ? I use a separate boolean now, but the error code would be more clean and in line with asio.
Also it seems the error handler is not called when a disconnect occurs ?

Hello @kleunen ,

In your code, if I call disconnect on the mqtt client connection, the disconnect handler will be called.

Are you talking about client side? If it is, close_handler will be called. You can set the close_handler using set_close_handler.
mqtt_cpp's disconnect() sends MQTT disconnect packet but not close the socket from the client side.
https://github.com/redboltz/mqtt_cpp/blob/master/include/mqtt/endpoint.hpp#L771
When the server receives the MQTT disconnect packet, the server closes the session of the client form the server side.
When the client detects disconnect from the server, check the reason. See https://github.com/redboltz/mqtt_cpp/blob/master/include/mqtt/endpoint.hpp#L2337
Then call close_handler or error_handler.
That's disconnect sequence.
mqtt_cpp client can distinguish error or normal close. However, mqtt_cpp client can't distinguish the normal close is caused by MQTT disconnect packet sent by the client, or other reason. The server might close correctly if something error in the server happens. It could happen before and after MQTT disconnect packet sending.
You can store disconnect_requested information as follows:
    // c is client

    bool disconnect_requested = false;

    // ...
    c->set_close_handler(
        [&]
        (){
            std::cout << "closed." << std::endl;
            if (!disconnect_requested) c->connect();
        });
    c->set_error_handler(
        [&]
        (boost::system::error_code const& ec){
            std::cout << "error: " << ec.message() << std::endl;
            if (!disconnect_requested) c->connect();
        });

{  // somewhere. might be in some handler.
    c->disconnect();
    disconnect_requested = true;
}

Again, mqtt_cpp client can't know the actual reason of normal close, so mqtt_cpp shouldn't assume it.

Hello @redboltz,
indeed I am referring to the client and indeed the code sample you give is what I am doing now. And I understand now that the client sends a disconnect request to the server, and the server will close the socket on a normal disconnect operation.
However I do feel that the mqtt_client should track whether a disconnect was requested. I think in your handle_close_or_error, when EOF occurs, it should be an error if this was done without the client requesting a disconnect, or it should be a close when the client did request a shutdown.
https://github.com/redboltz/mqtt_cpp/blob/master/include/mqtt/endpoint.hpp#L2337
I suggest the following close handling:
https://github.com/redboltz/mqtt_cpp/blob/master/include/mqtt/endpoint.hpp#L771
void disconnect() {
        if (connected_ && mqtt_connected_) {
            close_requested = true;
            send_disconnect();
        }
    }

Line
bool handle_close_or_error(boost::system::error_code const& ec) {
       // if (!ec) return false; (A successful close should also be handled ?)
        if (connected_) {
            connected_ = false;
            mqtt_connected_ = false;
            shutdown_from_server(*socket_);
        }
        if (ec == as::error::eof ||
            ec == as::error::connection_reset
#if defined(MQTT_USE_WS)
            ||
            ec == boost::beast::websocket::error::closed
#endif // defined(MQTT_USE_WS)
#if !defined(MQTT_NO_TLS)
            ||
#if defined(SSL_R_SHORT_READ)
            ERR_GET_REASON(ec.value()) == SSL_R_SHORT_READ
#else  // defined(SSL_R_SHORT_READ)
            ERR_GET_REASON(ec.value()) == boost::asio::ssl::error::stream_truncated
#endif // defined(SSL_R_SHORT_READ)
#endif // defined(MQTT_NO_TLS)
        ) {
            if(!close_requested)
               handle_error(ec);
            else
              handle_close();
            return true;
        }
        handle_error(ec);
        return true;
    }

I would say, this follows the asio convention more closely (every async operation results in the matching callback to be called with an error or success). disconnect -> close called (always success), an intermittent close is an error.

@kleunen , your approach seems good :) I will implement it.
There is a related issue. It's about force_disconnect().
https://github.com/redboltz/mqtt_cpp/blob/master/include/mqtt/client.hpp#L327
https://github.com/redboltz/mqtt_cpp/blob/master/include/mqtt/endpoint.hpp#L780
When the client calls force_disconnect(), the client closes the socket. Then handle_close_or_error(ec) is called with 'ec =  Bad file descriptor`. I'm not 100% sure ec's value. It might have various value depends on environment. e.g) WebSocket, TLS combination, Linux/Windows...
Anyway, what is the excepted behavior when the client calls force_disconnect(). I think that just calling error_handler is ok.

Indeed I agree, if you can force_disconnect, you get an unclean shutdown, correct ? The server does not get informed of the close, so from mqtt perspective, this would be indeed an error and calling error_handler is ok.

One more question: is error_handler always called on a socket error ? Or is it also called for protocol errors or when a subscribe fails or something like this ? Because if the error_handler is also used for other errors, then it would be good to be able to detect if the error is recoverable, or the client needs to disconnect and reconnect.

Indeed I agree, if you can force_disconnect, you get an unclean shutdown, correct ?

Yes.

One more question: is error_handler always called on a socket error ?

Yes.

Or is it also called for protocol errors or when a subscribe fails or something like this ?

No.
Each request after connection established has own handler. For example, suback_handler is corresponding to subscribe , connack_handler is corresponding to connect.

ok, good. Then indeed, my proposed handler seems like a good approach :)

I'm implementing it.
Some test cases report errors.
When MQTT broker returns connect_return_code except accepted,
https://github.com/redboltz/mqtt_cpp/blob/master/include/mqtt/connect_return_code.hpp#L16
first, connack_handler is called with connect_return_code.
Then the broker close the socket from the broker(server) side.
So error_handler is called. Before the fix, close_handler is called.
We have 3 choices after connack_handler is called.

call close_handler (current master behavior)
call error_handler (this fix behavior)
no handler is called

I think 3 is not good. Because MQTT protocol error and socket error (or close) are different. So the both of them are reported.
If the user want to call connect() again only in error_handler, 2 is good.
It's a breaking change.
Do you think 2 is good ?

Yes, I agree option 2 is good here. Indeed, the user then in all cases has to perform reconnecting in only one location: the error_handler

@redboltz I tested this, it works correctly. I get an error now, if the connection was broken unexpectedly, and I do a reconnect from the error handler. If I request a disconnect, on shutdown, the connection is shutdown cleanly and I get a closed handler call. This is very convenient.

Hi,
I just stumbled across this while searching for a reason why my client seems to sometimes have issues with reconnecting to the broker (with older 1.x verisons). I think I understand the new handling you propose in this situation, but may I request that this is somewhere more prominently documented with i.e. an example? I think this is a very common use-case and right now does not seem to be obvious how to handle this case (or at least I did not notice it).
Other from that: thank you for investing so much work into this. I really appreciate it!


Safari 18 drops JPEG 2000 support

https://webkit.org/blog/15443/news-from-wwdc24-webkit-in-safari-18-beta/

Images
WebKit for Safari 18 beta removes support for the JPEG2000 image format. Safari was the only browser to ever provide support.
If you've been serving JPEG2000 files using best practices, then your site is using the picture element to offer multiple file format options to every browser. Safari 18 beta will simply no longer choose JPEG2000, and instead use a file compressed in JPEG XL, AVIF, WebP, HEIC, JPG/JPEG, PNG, or Gif --- choosing the file that's best for each user. Only one image will be downloaded when you use <picture>, and the browser does all the heavy lifting.
We have noticed that some Content Deliver Networks (CDN) use User Agent sniffing to provide one file to each UA, offering only JPEG2000 images to Safari --- especially on iPhone and iPad. If you expect this might be happening with your site, we recommend testing in Safari 18 beta on both macOS Sequoia and iOS or iPadOS 18. If you see problems, contact your SaaS provider or change your image delivery settings to ensure your website provides fallback images using industry best practices.
If you notice a broken site, please file an issue at webcompat.com.

RIP JPEG 2000 I guess!
Thanks!


Split packages don't have issues enabled

@TomasVotruba Some of the split packages such as https://github.com/symplify/package-builder or https://github.com/symplify/phpstan-extensions don't have GitHub issues enabled. That's likely an oversight now that the monorepo is discontinued, isn't it? Or do you prefer to still track issues here instead of the individual repos?

Good catch :+1:
I've just fixed that

Thank you!


The gloves flying after searching container

Mod Version: 11.6
Game Version: 2.02
Launcher: Steam
Additional mods installed: No
Perspective: Client
Description
I've got into the truck and searched the glove box. I've found the gloves but I don't need it so I clicked RMB and the gloves must return into the glove box, but they hung in the air.
See [19:18:31.752] in the log.
How to reproduce

Being client on public server
On region Pleasant Valley
Take a rifle in your hands
Go into the truck at Point of Disagreement
Search the glove box
When some loot from the glove box appears. click RMB
This loot will hung in the air

Expected behavior
Loot must return into the glove box.
Logs
Host side log
See public server's logs.
Client side log
MelonLoader_23-05-20_18-33-31.131.log
Screenshots

This happends because in Sky Co-op if you don't take gear after searching container, it suppose to be dropped, to avoid extra call of processing container content,  but in that case it did not worked well because dropping usual not a thing in vanila. To need something with this.


[WIP] Manual: use nodes for Alpha clipping

Update the manual for #2111.

Closing, as we created a single PR with all alpha clipping stuff: #2155


Implemented files for fetchAudio

Now you can send Files to Crunker (or combine it with file paths).
Code in example is bad practice because it uses 'private' method decodeAudioData.
https://github.com/jackedgson/crunker/blob/9b8afbb1db87c540b099c13d56c78adf95326c72/examples/client/index.html#L45

@Alynva Merging this commit now, would you like to update the example so you send a file directly without having to invoke the private methods?

Thanks for this improvement @ToTheHit


Checkpoint: Stand in Line. Giving, RangeError: Maximum call stack size exceeded in Chrome

When I added my solution it was this error. Then after some research, I tried in Firefox and it worked.
So basically this is stack issue from chrome so I thought of reporting as other campers might get the same issue.

@ShivamSarawagi
Can you please send here code of your submitted solution on which you got this error?

@dmytroyarmak
Please find the code below:
function queue(arr, item) {
arr.push(item);
return arr.shift();  // Change this line
}

@ShivamSarawagi
Here is I tried your code and everithing works fine on Chrome.
http://www.freecodecamp.com/challenges/checkpoint-stand-in-line#?solution=%2F%2F Setup
var myArr %3D [1%2C2%2C3%2C4%2C5]%3B

function queue(arr%2C item) {
  arr.push(item)%3B
  return arr.shift()%3B %2F%2F Change this line
}

%2F%2F Display Code
console.log("Before%3A " %2B JSON.stringify(myArr))%3B%0Aconsole.log(queue(myArr%2C%206))%3B%20%2F%2F%20Modify%20this%20line%20to%20test%0Aconsole.log(%22After%3A%20%22%20%2B%20JSON.stringify(myArr))%3B%0A
Can you try too again?
This kind of error is possible if there is recursion in your code, but you don't have it.

Maximum call stack error is solved by simply refreshing your page.
Your code is stored locally, so you should not lose your progress if you refresh the page.

Confirmed OriBon - I had the same issue, refreshed the page, issue gone

Duplicate of https://github.com/FreeCodeCamp/FreeCodeCamp/issues/5715

I just had the same issue. Reloading the browser fixes it though.

+1


Run pre/post install scripts during bootstrap

Born from https://github.com/lerna/lerna/pull/358.
Need to merge non-squash to preserve authorship.
CC @seansfkelley
This is an awesome improvement.  Lays the groundwork for other easy topo-ordered operations.

I'd like to get this merged in before getting too far into #14, since they're likely to conflict.
This looks great to me.  @rygine?

Thanks @seansfkelley!

This landed in<EMAIL_ADDRESS>


Unselect / Skip select

Check for existing issues

[X] Completed

Describe the feature
Add commands to unselect/skip next/previous selection.
If applicable, add mockups / screenshots to help present your vision of the feature
Unselect

Skip select

Hey @liebkne. You can use ⌘+U to undo a selection. ⌘+⇧+U to redo a selection and ⌘+K D to skip the current selection. Hopefully that helps! 🙂

@hovsater Thanks for the information about undo. How about skipping?

@liebkne like I said, use ⌘+K D to skip a selection. That is, Press ⌘+K, release and press D.

@hovsater 😅 Sorry, I should read it more carefully.

No worries. 🙂

Press and hold ⌘ and then press K followed by D.

I think that should be called ⌘-k ⌘-d, ⌘-k d could be a little confusing since the default keymap looks like
      "cmd-k cmd-d": [
        "editor::SelectNext",
        {
          "replace_newest": true
        }
      ],

Both works. You can either press ⌘+K followed by ⌘+D or just hold ⌘ and then press K followed by D.

Hi, I mean the way of writing up the key sequences, not the way how to press it.

and ⌘+K D to skip the current selection.

looks like press a ⌘+K, release both then a D.

Right. I actually think we do write them up as you expect. E.g., ⌘+K M means press and release ⌘+K followed by M. Sorry for the confusion. 🙂


RNSScreenStackHeaderConfig was not found in UIManager

Environment
react-native -v: 5.0.1
npm ls react-native-macos<EMAIL_ADDRESS>node -v: v18.8.0
npm -v: 8.18.0
yarn --version: 1.22.18
xcodebuild -version: Xcode 13.3
Build version 13E113

Steps to reproduce the bug

Initialize a macOS project following the steps listed in the react-native-macos website until the new app screen is built and run
Copy and paste the navigation sample code from official react-native doc page https://reactnative.dev/docs/navigation
follow the steps to install npm install @react-navigation/native @react-navigation/native-stack npm install react-native-screens react-native-safe-area-context
do pod install on macos/ folder
build and run the app, error reported when run. ERROR  Invariant Violation: requireNativeComponent: "RNSScreenStackHeaderConfig" was not found in the UIManager.

Expected Behavior
There should not be any error.
Actual Behavior
ERROR  Invariant Violation: requireNativeComponent: "RNSScreenStackHeaderConfig" was not found in the UIManager.
Reproducible Demo
No response
Additional context
No response

i also have same issue, i was able to rewrite the bottom navigation tab library to not use the RNSScreenStackHeaderConfig but not for the stack navigator

any updates about this !

React Native Navigation doesn't support macOS, as it has some native dependencies. To use with a React Native macOS project, you would have to be careful that you use the Javascript only parts of it, if possible.

I have same issue, how to resolve it?

I think you'd have to open an issue for macOS support in react native navigation repo.

@qd-wl & @yaozhang-this , Please Use @react-navigation/stack, & not @react-navigation/native-stack. Like @Saadnajmi said, libraries with js parts work fine.
I faced the same error today & solved it via using @react-navigation/stack instead of @react-navigation/native-stack

@kaiyes Your right. I have tried many options but nothing works. But replacing import for stack navigator works well.
-import { createNativeStackNavigator } from '@react-navigation/native-stack';
+import { createStackNavigator } from '@react-navigation/stack'


Issue #323 dynamic property assignment

This Pull Request will address the deprecation of dynamically adding properties to objects, as outlined in this issue. From the Author's suggestion, I have chosen to break the API and add the setTemplate() method into the Interface.
This change will pass all tests and has updated documentation. This will break older versions of the library that rely on the $template property to be public (or non-existent). I am open to looking to a better way to compose the necessity of having the Template class injected into the Func class that further injects into the ExtensionInterface concrete classes.

Hey, thanks for this submission, but I'm going to close it in favor of https://github.com/thephpleague/plates/pull/329


Update Legacy Survey dataset

This PR supersedes #115
Adds Mask and Catalog Information
It adds masks and catalog information to the legacy survey dataset.
Update has already been uploaded to HG dataset hub. Although the mask object image appears only black on the hub, it actually contains the information as expected. One can check with the following that we correctly obtain the images below:
import datasets
import numpy as np
import matplotlib.pyplot as plt

dataset = datasets.load_dataset("MultimodalUniverse/legacysurvey", split="train")
plt.imshow(np.array(dataset[6]["model_image"]))
plt.show()
plt.imshow(np.array(dataset[6]["object_mask"]))
plt.show()

Data Generation Fix
On rusty the data generation has been fixed by:

Checking problematic sweep catalogs;
Downloading corrupted catalogs.

It is not excluded that some images were missing, which may have resulted in corrupted entries in the dataset.
We ran the sanity_check.py script successfully. This doesn't mean however that data are perfectly safe.
Data are available on rusty at the following path /mnt/ceph/users/polymathic/AstroPile_v1/legacysurvey_update/
Other Script Approach
In #115 and the related branch, we experimented with dask. We managed to make it work. However, the parallelism was poor because of locks on concurrent writings on HDF5 files. Consequently, we decided to keep with the current version of the script.
Closes #110

In the meantime, I've written this script to identify and remove files that were corrupted in the previous version of the data:
https://gist.github.com/EiffL/d4ab8e7964572b7453bef81b1d863207
This was necessary for the cross matching script. Overall it removed like 40 files out of 1400.
Interestingly, some files had corrupted values, but did not raise an inflate error, so would go silent in the dataset loading script. We could run the same logic on the new export for a minimal check that everything is ok.

I have also renamed model_image to blobmodel and updated the files on rusty consequently. The rational here is that the pipeline calls this image a blobmodel, it's better to keep the original name.

I think we are good for this PR. Should we fix the tests and merge? Regarding the test I don't understand why they are failing.

I've pushed a few commits doing the following:

fix the magnitude selection bug that preserved negative z fluxes
changed the mask code to generate ellipcies instead of disks
added shape parameters to the catalogs
added rgb image as a another "modality" in addition to blob model
changed the compression filter to lzf which the h5py documentation says: Low to moderate compression, very fast.

Here is an example of the updated mask and the 2 rgb images:

and with the shape information in the catalogs, we can build "bounding boxes" offline:

I'm currently regenerating the data using 20 nodes, so should be pretty fast

Really nice !
At some point I was trying to get the ellipsis instead of the disk for the objects, but I couldn't find a good way to set the ellipsis aspect ratio. I saw that you set it to 2 (c.f. below). Do you think it will fit most of the cases (i.e. there will not be images where the factor 2 is either too small or too big)?
diff --git a/scripts/legacysurvey/build_parent_sample.py b/scripts/legacysurvey/build_parent_sample.py
index b205fcc..555b0ce 100644
--- a/scripts/legacysurvey/build_parent_sample.py
+++ b/scripts/legacysurvey/build_parent_sample.py
@@ -103,24 +102,22 @@ class CatalogSelector:
         centers = np.stack([j, i]).T
         radii = self.catalog["SHAPE_R"].value / ARCSEC_PER_PIXEL
         object_types = self.catalog["TYPE"].value.astype(str)
-        for c, r, t in zip(centers, radii, object_types):
-            # Enforce a minimum size of the disk mask
-            r = max(2, r)
-            rr, cc = skimage.draw.disk(c, r, shape=self.cutout.shape)
+        e1 = self.catalog["SHAPE_E1"].value
+        e2 = self.catalog["SHAPE_E2"].value
+        angle = 0.5 * np.arctan2(e2, e1)
+        q = (1 - np.sqrt(e1**2 + e2**2)) / (1 + np.sqrt(e1**2 + e2**2))
+        height = 2 * radii
+        width = 2 * radii * q
+        for c, h, w, a, t in zip(centers, height, width, angle, object_types):
+            rr, cc = skimage.draw.ellipse(c[0], c[1], w, h, shape=self.cutout.shape,
+                                          rotation=a)

I think it should be fine for most objects. We only keep in the catalog objects that are marked as "extended" meaning they should be larger than point sources. The factor 2 is just the conversion to radius to diameter essentially.
I think it all looks good, no? ready to merge?


feat: make forwarder storing non-blocking

This change is

Codecov Report

Merging #2664 (caa63dd) into master (294ab28) will decrease coverage by 0.02%.
The diff coverage is 26.08%.

:exclamation: Current head caa63dd differs from pull request most recent head ec32ed7. Consider uploading reports for the commit ec32ed7 to get more accurate results

@@            Coverage Diff             @@
##           master    #2664      +/-   ##
==========================================
- Coverage   63.57%   63.54%   -0.03%
==========================================
  Files         236      236
  Lines       26520    26520
==========================================
- Hits        16859    16852       -7
- Misses       8142     8150       +8
+ Partials     1519     1518       -1

Impacted Files
Coverage Δ

pkg/pushsync/pushsync.go
66.16% <26.08%> (-0.22%)
:arrow_down:

pkg/localstore/subscription_push.go
73.68% <0.00%> (-5.27%)
:arrow_down:

pkg/shed/schema.go
75.58% <0.00%> (ø)

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 294ab28...ec32ed7. Read the comment docs.