[dask] use random ports

This is intended to solve the problem that @jameslamb defined in #3768 by running socket.bind('', 0) with client.run, which finds a random port for each worker. I've included a small test to check that the found ports are indeed different for a LocalCluster. Happy to receive any feedback.

Thank you for your feedback, @jameslamb! I don't think there will be a problem by using a different type of cluster, since client.run executes the function in every worker, so the found ports will be open in that (possibly remote) worker, I will try this out tomorrow using coiled. I understand your point about loosing control of the range of ports found, however in a remote setting the client only communicates to the scheduler and I believe the ports chosen to run each worker are defined in the same way as I proposed (I haven't been able to confirm this), so I think the scheduler should be able to access the ports found for lightgbm without a problem. I'm by no means an expert in this matter and could be wrong, so I'm happy to get your thoughts on this and discuss it further.

Thank you for your feedback, @jameslamb! I don't think there will be a problem by using a different type of cluster, since client.run executes the function in every worker, so the found ports will be open in that (possibly remote) worker, I will try this out tomorrow using coiled. I understand your point about loosing control of the range of ports found, however in a remote setting the client only communicates to the scheduler and I believe the ports chosen to run each worker are defined in the same way as I proposed (I haven't been able to confirm this), so I think the scheduler should be able to access the ports found for lightgbm without a problem. I'm by no means an expert in this matter and could be wrong, so I'm happy to get your thoughts on this and discuss it further.

I understand your point about loosing control of the range of ports found, however in a remote setting the client only communicates to the scheduler and I believe the ports chosen to run each worker are defined in the same way as I proposed (I haven't been able to confirm this), so I think the scheduler should be able to access the ports found for lightgbm without a problem

Ah! Let me add more information on this.
By default Dask does find ports for that worker-to-scheduler communication randomly, but it exposes options to control that more tightly: https://docs.dask.org/en/latest/setup/cli.html#handling-ports.
The ports for LightGBM aren't related to Dask directly. LightGBM has its own distributed learning framework written in C++ (https://github.com/microsoft/LightGBM/tree/master/src/network). This Dask package just wraps that framework. In the Dask package here, we set up one long-running Dask task per worker, and that task is running a LightGBM worker. The LightGBM workers talk to each other directly over TCP sockets (by default), and Dask has no knowledge of that communication.

I will try this out tomorrow using coiled

Thanks very much! If it's easier, I have a testing setup that uses dask-cloudprovider to run a cluster on AWS Fargate: https://github.com/jameslamb/lightgbm-dask-testing.
When I was working on #3766, I found that there were several fixes I tried which worked well on LocalCluster that didn't hold up in a multi-machine setting.

I understand your point about loosing control of the range of ports found, however in a remote setting the client only communicates to the scheduler and I believe the ports chosen to run each worker are defined in the same way as I proposed (I haven't been able to confirm this), so I think the scheduler should be able to access the ports found for lightgbm without a problem

Ah! Let me add more information on this.
By default Dask does find ports for that worker-to-scheduler communication randomly, but it exposes options to control that more tightly: https://docs.dask.org/en/latest/setup/cli.html#handling-ports.
The ports for LightGBM aren't related to Dask directly. LightGBM has its own distributed learning framework written in C++ (https://github.com/microsoft/LightGBM/tree/master/src/network). This Dask package just wraps that framework. In the Dask package here, we set up one long-running Dask task per worker, and that task is running a LightGBM worker. The LightGBM workers talk to each other directly over TCP sockets (by default), and Dask has no knowledge of that communication.

I will try this out tomorrow using coiled

Thanks very much! If it's easier, I have a testing setup that uses dask-cloudprovider to run a cluster on AWS Fargate: https://github.com/jameslamb/lightgbm-dask-testing.
When I was working on #3766, I found that there were several fixes I tried which worked well on LocalCluster that didn't hold up in a multi-machine setting.

Hi @jameslamb. I've successfully ran the test_training_does_not_fail_on_port_conflicts test on a coiled cluster with 4 machines and 2 cpus each, however I'm not sure how to share that with you. I have a software environment called jmoralez/lightgbm-test there if you want to try it out (however you'll still need the local version which is the part I'm not sure about). I had to set sample_size=10_000 because I was getting the Connecting to rank x failed error. Do you have any suggestions for sharing the tests or how to proceed?

Hi @jameslamb. I've successfully ran the test_training_does_not_fail_on_port_conflicts test on a coiled cluster with 4 machines and 2 cpus each, however I'm not sure how to share that with you. I have a software environment called jmoralez/lightgbm-test there if you want to try it out (however you'll still need the local version which is the part I'm not sure about). I had to set sample_size=10_000 because I was getting the Connecting to rank x failed error. Do you have any suggestions for sharing the tests or how to proceed?

Hi James. I've set up the version of my branch here. I think this random port assignment could be a fallback when specifying local_listen_port=0 and otherwise use the current implementation, what do you think?
As a side note I've noticed that with few samples I get Connecting to rank x failed like crazy. For example, by running the notebook as is I got stuck at the second fit and had to do a client.restart, however with sample_size=10_000 it runs the 5 fits in 4 seconds. I believe this is related to #3797 and I experienced this a couple of months ago as well as the TCP fail with dask-lightgbm and made me drop it. I'll dig deeper into this.

Hi James. I've set up the version of my branch here. I think this random port assignment could be a fallback when specifying local_listen_port=0 and otherwise use the current implementation, what do you think?
As a side note I've noticed that with few samples I get Connecting to rank x failed like crazy. For example, by running the notebook as is I got stuck at the second fit and had to do a client.restart, however with sample_size=10_000 it runs the 5 fits in 4 seconds. I believe this is related to #3797 and I experienced this a couple of months ago as well as the TCP fail with dask-lightgbm and made me drop it. I'll dig deeper into this.

As a side note I've noticed that with few samples I get Connecting to rank x failed like crazy. For example, by running the notebook as is I got stuck at the second fit and had to do a client.restart, however with sample_size=10_000 it runs the 5 fits in 4 seconds

Can you please open a separate issue for this so we can discuss there? I have some theories.

I've set up the version of my branch here. I think this random port assignment could be a fallback when specifying local_listen_port=0 and otherwise use the current implementation, what do you think?

really cool, thank you! I clicked into the notebook and ran it. I really really appreciate the effort you've put into this PR and into setting up this nice test harness in Coiled. I can see that the 5 consecutive runs worked!

I even tried changing the test and running  50 consecutive trainings back to back. No problems :).

I can see that this cluster's workers are on different IPs, so I think this is a great test that this PR will work in other multi-machine settings.

I like the idea of possibly using both this implementation and the current one, but I'd rather switch the order. I think it makes sense to have your proposed (faster) approach be the default behavior, and for the current behavior to only be used if people opt in to it because they're working in an environment with networking constraints like firewall rules that only permit specific port ranges. I think users will be ok with paying a small performance penalty to comply with their security setup, and that anyone NOT constrained in that way should get the faster method by default.
However, I don't think we should support setting local_listen_port to 0 or any other special value. I want the result of model.get_params() to return valid LightGBM parameters that you could take and use in other non-Dask LightGBM APIs, and 0 is not a valid value for local_listen_port.https://lightgbm.readthedocs.io/en/latest/Parameters.html#local_listen_port.
Ok sorry, that was a lot of information. Given all that, here's my proposal

if local_listen_port is missing from parameters, use your approach from this PR
if local_listen_port is set to 12400 (LightGBM's default), use your approach from this PR
if local_listen_port is set to a value other than 12400, use the existing approach where we specifically search 1000 ports from local_listen_port to local_listen_port + 999.

What do you think?

As a side note I've noticed that with few samples I get Connecting to rank x failed like crazy. For example, by running the notebook as is I got stuck at the second fit and had to do a client.restart, however with sample_size=10_000 it runs the 5 fits in 4 seconds

Can you please open a separate issue for this so we can discuss there? I have some theories.

I've set up the version of my branch here. I think this random port assignment could be a fallback when specifying local_listen_port=0 and otherwise use the current implementation, what do you think?

really cool, thank you! I clicked into the notebook and ran it. I really really appreciate the effort you've put into this PR and into setting up this nice test harness in Coiled. I can see that the 5 consecutive runs worked!

I even tried changing the test and running  50 consecutive trainings back to back. No problems :).

I can see that this cluster's workers are on different IPs, so I think this is a great test that this PR will work in other multi-machine settings.

I like the idea of possibly using both this implementation and the current one, but I'd rather switch the order. I think it makes sense to have your proposed (faster) approach be the default behavior, and for the current behavior to only be used if people opt in to it because they're working in an environment with networking constraints like firewall rules that only permit specific port ranges. I think users will be ok with paying a small performance penalty to comply with their security setup, and that anyone NOT constrained in that way should get the faster method by default.
However, I don't think we should support setting local_listen_port to 0 or any other special value. I want the result of model.get_params() to return valid LightGBM parameters that you could take and use in other non-Dask LightGBM APIs, and 0 is not a valid value for local_listen_port.https://lightgbm.readthedocs.io/en/latest/Parameters.html#local_listen_port.
Ok sorry, that was a lot of information. Given all that, here's my proposal

if local_listen_port is missing from parameters, use your approach from this PR
if local_listen_port is set to 12400 (LightGBM's default), use your approach from this PR
if local_listen_port is set to a value other than 12400, use the existing approach where we specifically search 1000 ports from local_listen_port to local_listen_port + 999.

What do you think?

Hi James. That sounds good, I'll work on that and update this PR. Thank you!

Hi James. That sounds good, I'll work on that and update this PR. Thank you!

Hi James. Since local_listen_port is set to 12400 in case it is None then if its 12400 I search for a random open port and otherwise use the current implementation. Looking forward for your feedback.

Thank you for your feedback, James. I've included your comments.

Sorry for the inconvenience, but can you update this PR and #3897 to master? I just merged a PR that changes the Dask interface to move client from .fit() and .predict() into the estimator constructors (https://github.com/microsoft/LightGBM/pull/3883).

Thanks @StrikerRUS . @jmoralez , don't make any changes yet. I need to go re-read the code in network again to be sure I understand the implications of the choice we make here. Will report back soon.

I haven't forgotten about this PR. I did a lot of research this weekend, reading through LightGBM's network/ source code to understand exactly how the params local_listen_port and machine_list are used. I'll have a recommendation to share soon.

Ok @jmoralez we just merged https://github.com/microsoft/LightGBM/pull/3994, so this PR should now be unblocked.
Could you please update this to the latest master and then replace my use _find_ports_for_workers() with your implementation here?
https://github.com/microsoft/LightGBM/blob/1f73f55938a6bfc84eb18b0f04288a5f039eef8a/python-package/lightgbm/dask.py#L418

Thank you, James! Should I remove _find_ports_for_workers, _find_open_port and modify test_find_open_port_works?

Thank you, James! Should I remove _find_ports_for_workers, _find_open_port and modify test_find_open_port_works?

yes please

Thanks a lot for all your help with this, James!


Format code with Black

This will help us to avoid unproductive style discussions and no-op line diffs due to formatting issues.
https://black.readthedocs.io/en/stable/

Switching to ruff would provide the exact same functionality as black but also provide the linting that flake8 does.
https://docs.astral.sh/ruff/faq/


[Bug] [function] NullPointerException occurs when creating udf function

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When I use dinky1.0 version to create a udf task, a null pointer error occurs regardless of java/scala/python.

What you expected to happen
Tasks can be created normally
How to reproduce
dinky1.0 creates udf task
Anything else
No response
Version
1.0.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

fixed, #3225

Do I need to pull the dev package or download the release version?

Do I need to pull the dev package or download the release version?

If you need to use the latest one now, you need to use the dev branch for packaging. Please refer to the Compile and Deployment chapter on the official website.
If you are not in a hurry, please wait for the 1.0.1 realease version

okky, thanks


[refactor] posix.mak: Enforce whitespace before opening parenthesis for version …

https://github.com/dlang/phobos/pull/6717/files#diff-228b518f4545d060f900578830ade5da
The rest is just mechanical.
This should be reflected in the style guide also.
https://dlang.org/dstyle.html#phobos_whitespace

Churn aside, enforcing this is questionable because version( appears more times in phobos than version (. What is the basis for making the minority use the required convention?

What is the basis for making the minority use the required convention?

version () outranks version() 3067 to 2316 across dmd, druntime and phobos. Ignoring dmd, it still has a small majority.
I'm adding new ports, and I'm seeing:
version( A )
  // ...
else version (B)
  // ...
else version ( C )
  // ...
else version(D)
  // ...
else version (E)
  // ...
else version( F )
  // ...

Also, covering this in the style guide. https://github.com/dlang/dlang.org/pull/2472


Adding Microsoft.Batch management api version 2017-09-01

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1923

Codecov Report

Merging #1598 into master will increase coverage by 1.39%.
The diff coverage is 91.01%.

@@            Coverage Diff             @@
##           master    #1598      +/-   ##
==========================================
+ Coverage   54.69%   56.08%   +1.39%
==========================================
  Files        3680     3889     +209
  Lines       90960    94195    +3235
==========================================
+ Hits        49750    52831    +3081
- Misses      41210    41364     +154

Impacted Files
Coverage Δ

...ure-mgmt-batch/azure/mgmt/batch/models/resource.py
100% <ø> (ø)
:arrow_up:

.../models/batch_account_regenerate_key_parameters.py
100% <ø> (ø)
:arrow_up:

...gmt/batch/models/check_name_availability_result.py
100% <ø> (ø)
:arrow_up:

...-mgmt-batch/azure/mgmt/batch/models/application.py
100% <ø> (ø)
:arrow_up:

...mt/batch/models/batch_account_update_parameters.py
100% <ø> (ø)
:arrow_up:

...gmt-batch/azure/mgmt/batch/models/batch_account.py
100% <ø> (ø)
:arrow_up:

...re-mgmt-batch/azure/mgmt/batch/models/operation.py
100% <ø> (ø)
:arrow_up:

...mt/batch/models/batch_account_create_parameters.py
100% <ø> (ø)
:arrow_up:

...tch/azure/mgmt/batch/models/application_package.py
100% <ø> (ø)
:arrow_up:

...k-testutils/devtools_testutils/storage_testcase.py
65.78% <ø> (+1.9%)
:arrow_up:

... and 291 more

Continue to review full report at Codecov.

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

@lmazuel - looks good to go?


Geany: Support for GTK2 and GTK3

Motivation for this change
Support both GTK2 as well as GTK3 for Geany.
Things done
The change keeps the current defaults for both used GTK as well as VTE.

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

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

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

@jtojnar  GTK3? My thought was as long as there is Xfce 4.12 around GTK2 support should be kept and I decided against switching the default for now. I'm totally fine on defaulting to GTK3 but at least option for GTK2 should be kept for some time IMHO

This PR doesn't add support for anything, it just renames parameters and changes code formatting. It seems like you can already do everything you want in current nixpkgs: geany.override { gtk2 = gtk3; }

@frlan are you able to address the above comments and continue with this PR?

Yes, I will.


feat(breadcrumb): add shadow parts

Related Ticket(s)
Jira
Description
we should add CSS Shadow Parts to v2 components to allow light customizations to component internals.
All non-slot elements in the shadow DOM should be given a unique "part" name allowing CSS to target and override component default styles.
This PR targets the breadcrumbs component
Changelog
New
Adding shadow parts to the breadcrumbs component

@m4olivei Thanks for the review!
The requested changes are addressed on my latest commit


Default FW password is not working

Hello Dennis,
Default password (Aviatrix#1234) is not working when deploying firewalls without firenet_password argument

Should be solved in 1.0.2


Profile pictures

When I create a new user, I don't have a profile picture at all, can you fix that?

should be fixed now

Is it ok that some pictures are .jpeg and others are .jpg? Are they still being found just fine?

Yes, it is


MudToolBar: Optional Wrapping and Appbar compatibility

#6808 broke the primary use of --mud-appbar-height. This fixes this as discussed after that PR was merged

Description

This PR adds the Wrapping Bool Parameter to both MudAppbar and MudToolBar. This is so as to not make the default change from non-wrapping to wrapping, since the original PR was merged in a Hotfix Release.
This PR changes _toolbar.scss, so that the pixel values do not have to be duplicated throughout the file. They are now assigned to --mud-internal-toolbar-height instead of height or min-height. --mud-internal-toolbar-height is instead assigned as height only once and height is unset if the ToolBar is wrapping. In that case --mud-internal-toolbar-height is the min-height.
This PR makes it so that the minimum of --mud-internal-toolbar-height and --mud-appbar-height is chosen if the wrapping toolbar is actually an AppBar.

How Has This Been Tested?

Imma be honest: I didn't. Please do so before merging, though.
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

@TimWeidner Not sure about the name Wrapping, I'd say IsWrapping or Wrap or WrapContent
@bdebaere Note: you'll need to set the new parameter for the new wrapping behavior you introduced. Can you test this against your app please?

Furthermore, would it be feasible to create bUnit tests to ensure that future changes do not introduce breaking changes? While we already have numerous tests and good coverage, it's possible that certain scenarios may not be fully covered.

I prefer WrapContent myself. And yes, please add unit tests at least for the case of MudAppbar applying the wrapping class to its child MudToolbar. 😄

Hi @TimWeidner,
I hope you're doing well. I wanted to check if you still plan to continue working on this pull request. Please let me know if you need any assistance or if there's anything I can do to help move it forward. Thank you!

I hope you're doing well. I wanted to check if you still plan to continue working on this pull request. Please let me know if you need any assistance or if there's anything I can do to help move it forward. Thank you!

Thanks for checking up! No, it's fine, I just had to wait until I was back in the office to not just change each file individually over the github web interface. Should be fine now.

@TimWeidner #6929 is a semver-compliant hotfix for you, reverting the WrapContent default value to false in MudAppBar: https://github.com/MudBlazor/MudBlazor/releases/tag/v6.3.1


Do the I/O ports work on anything other than QTS?

Hi there, first of all, thank you for making this!
I myself have a QNAP TS-453D, with TrueNAS SCALE installed on it. I would like to make something like this, but for TrueNAS instead (since that's what I prefer, rather than QTS).
Looking at SIOPoller.cpp, there is kRegPort, how did you find out about this? (is this mapped in the BIOS? I've never done any of this, so I'd love to know.) And, is this valid, even though there's a different OS on it, or is this value purely for QTS? And is this the same for all of the ports, such as the portStatusLed and 0x91?

Hi,
You're welcome, thanks for your interest. The values were found by trial and error since I could not be bothered to try reverse-engineer gigabyte-sized OS images and megabyte-sized HAL binaries.
I believe it might be possible to make sense of some long text config files that describe these I/O ports distributed with the upgrade images, but last time I tried using this, it confused me more than anything else.
Anyway, this is already 2+y old and I forgot the specifics, but you can have a look at the Gist where I documented my research, which led me to write this here project. You'll find the config file I'm talking about. By extracting the official image for your specific model you might be able to find the right values.
Best of luck!

Thank you so much for providing this! I'll take a look at it. :)


After installing the update EXP broken

Run EXP - Click to install an update - The EXP is still running, already around 2h.
Or we stop processes or continue from the point where it's stopped.
Do not know what to do now.
https://user-images.githubusercontent.com/98249521/181160934-2031b394-40af-4f8c-82f8-5bd140588901.mov

The cause is probably https://github.com/iterative/dvc/issues/8031.
I recommend downgrading.

Good. Just wanted to mention this. I know I am not a typical user, but sill :) Bay the way it is still running. 14h :)

It won't ever finish you need to run dvc exp gc -f --all-tags and delete .dvc/tmp/exps, the data is corrupted and the extension isn't actually running anything.

Sorry, issue was probably https://github.com/iterative/dvc/issues/8045. I linked the wrong one.

@maxagin can you update the DVC version (install the development one?) It should be fixed there. Otherwise, advice is yes - cleanup .dvc/tmp/exps and lock files (.dvc/tmp/*lock*).


BDSA-2024-0720 Netty Vulnerable to Denial-of-Service (DoS) via Uncontrolled Memory Allocation in 'HttpPostRequestDecoder' Component

Hello team,
Thank you for the work on this project.
Our company runs daily scans using BlackDuck, SonarQube, OWASP etc...
And it found:
BDSA-2024-0720
High
CVSS3 Base : 7.5
CVSS3 Impact : 3.6
CVSS3 Exploitabity : 3.9
Netty Vulnerable to Denial-of-Service (DoS) via Uncontrolled Memory Allocation in 'HttpPostRequestDecoder' Component
Description
Netty is vulnerable to denial-of-service (DoS) due to insufficient restrictions on the amount of memory that is allocated in the HttpPostRequestDecoder component. An attacker could exploit this by sending maliciously crafted data in order to cause an out-of-memory (OOM) error and a denial-of-service (DoS). Note: The vendor has mentioned that any Netty based HTTP server that uses the HttpPostRequestDecoder to decode a form is impacted.
Technical Description
This vulnerability exists within the HttpPostRequestDecoder functionality in multiple files in the codec-http/src/main/java/io/netty/handler/codec/http/multipart/ directory. There are insufficient limits on the number of fields a form can have when it is decoded. An attacker could send a crafted chunked post with a number of small fields which will result in the accumulation of data in the bodyListHttpData list. This results in the decoder cumulating bytes in the undecodedChunk buffer which does not have any limits, thus, leading to a denial-of-service (DoS) condition via an out-of-memory (OOM) error. It has been mitigated by introducing limits on both bodyListHttpData and the undecodedChunk buffer. maxFields defines the maximum number of fields a form can have and maxBufferedBytes defines the maximum amount of bytes a field can cumulate. The decoder controller will handle if a limit has been reached and will throw a decoder exception in a safe manner.
Vulnerability Source
BDSA
EXPLOIT | https://gist.github.com/vietj/f558b8ea81ec6505f1e9a6ca283c9ae3
We do use HttpPostRequestDecoder, could you please help?

Just upgrade to the latest release which has the fix.


Reduplicative ations with same hash shows in action list

https://iotexscan.io/address/io15ed8a4lyqgglm7z9wdq40kjksc0j5z0erw5v3x

ref https://github.com/iotexproject/iotex-core/issues/1386

Closed in favor of https://github.com/iotexproject/iotex-core/issues/1386


Registration Page

A registration page for users to create an account.

Basic registration page with validation of the fields is completed.

Adding validations to the registration fields.

Validations added in the registration form with the form throwing hints when violated.


[nodejs][09.message routing] Sample missing documentation file

Sample information

Sample type: \samples\
Sample language: nodejs
Sample name: 09.message-routing

Describe the bug
The sample's README in the install instructions lacks the deployment documentation that is linked.
For this reason the sample can't be correctly deployed neither states clearly what services makes use of.
[bug]

Closed due to Microsoft#712


Play Json Support

This is my initial attempt to get Play JSON support into Finch. Currently all the tests in PlayJsonSpec fail  with similar errors (Cannot find implicit values for decoder/encoders for the case classes defined in JsonCodecProviderProperties).  The purpose of opening this PR is to get some feedback on what might be missing/why these tests fail. I understand the error message but I can't figure out comparing to other JSON libs whats different and how they are properly allowing Finch to create appropriate implicits.

Okay, I know what the problem is. Sorry, it's been a while a had a look at the JSON implementation and I totally forgot that we use the same case class for testing - ExampleCaseClass . So you should remove Bar.scala and provide Writes[ExampleCaseClass] and Reads[ExampleCaseClass] in the PlayJsonSpec.scala.

Opened a new one at #538.


chatgpt-clone-api-1 exited with code 1

First off. This is an amazing project. Thanks for putting this together.
I downloaded the latest update (this morning) and I get the following error when deploying with Docker Compose:
chatgpt-clone-api-1     | node:events:490
chatgpt-clone-api-1     |       throw er; // Unhandled 'error' event
chatgpt-clone-api-1     |       ^
chatgpt-clone-api-1     |
chatgpt-clone-api-1     | Error: getaddrinfo EAI_AGAIN "localhost"
chatgpt-clone-api-1     |     at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:107:26)
chatgpt-clone-api-1     | Emitted 'error' event on Server instance at:
chatgpt-clone-api-1     |     at GetAddrInfoReqWrap.doListen [as callback] (node:net:1934:12)
chatgpt-clone-api-1     |     at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:107:17) {
chatgpt-clone-api-1     |   errno: -3001,
chatgpt-clone-api-1     |   code: 'EAI_AGAIN',
chatgpt-clone-api-1     |   syscall: 'getaddrinfo',
chatgpt-clone-api-1     |   hostname: '"localhost"'
chatgpt-clone-api-1     | }
chatgpt-clone-api-1     |
chatgpt-clone-api-1     | Node.js v19.7.0

I have also tried filling in - HOST="" in the Docker Compose file with the proper hostname and it is still not working.
Any ideas?

I can reach the client UI, and this is the error I see there when it tries to contact the API.

I know the issue and working on this!

I'm glad you've been enjoying this project! I rolled back to latest release since I had trouble debugging how a recent PR affected docker. Will investigate further but for now is confirmed working with docker.


docs(guild): add missing features

Summary
This PR documents the guild features ANIMATED_BANNER, APPLICATION_COMMAND_PERMISSIONS_V2, INVITE_DISABLED and ROLE_ICONS.
Checklist

[ ] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.
[ ] I have run task pyright and fixed the relevant issues.

[ ] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[x] This PR is not a code change (e.g. documentation, README, ...)

But their functions being documented are incorrect now.


Writing multiline text in & changing the height of code-terminal.

Now, Code Terminal component has set fixed value, in my opinion. We can see it in the image which is a screenshot from a smartphone.

It has more free space. We should get rid of it. Do we need something additional like turnOffFixedValue?
Also is it possible to write multiline code in this terminal component or not? If no, please implement it.

@bdrtsky Any updates? In theses days, On what the tams is working? If it is not a secret, can you tell?

@otabekoff sorry for late answer. Next week Elements should open public, and Terminal is tokenized, so you can set height yourself.
As for multiline code, it's already possible. Just pass an array of strings instead

Also, I will use it in my project. I am closing this as it is live and working at docus.dev as an instance.


Add command attribute to show current context

Currently kubens display current namespace in the color formatted way in the namespace list, which is difficult to grep and find the current namespace.
the separate attribute to display current namespace can be helpful in finding it and then it can be easily utilized for tmux status or zsh prompt.

I signed it!

Thanks, but same PR arrived many more times. You should use kubectl config view current-context. #14, #28.
You're not supposed to use kubectx/kubens with scripting. They're meant to be interactive convenience programs that just wrap kubectl.


Hugo Docker Image

When will Hugo Docker Image be published in Docker Hub?

There are plenty of them on Docker Hub. I modified one to suit my needs: https://hub.docker.com/r/dettmering/hugo-build/
However, it could be optimized in terms of image size.


[Code Addition Request]: SONAR Rock vs Mine Prediction with Machine learning

Have you completed your first issue?

[X] I have completed my first issue

Guidelines

[X] I have read the guidelines
[ ] I have the link to my latest merged PR

Latest Merged PR Link
N/A
Project Description
This project is a machine learning solution for classifying objects detected by a SONAR signal as either a rock or a mine. SONAR (Sound Navigation and Ranging) is often used in underwater detection systems, and it can be challenging to accurately identify whether the detected object is a harmless rock or a dangerous mine. This project aims to build a machine learning model to predict the classification of the object based on the characteristics of the SONAR signal.
Project Workflow:
Data Collection: Load the SONAR dataset.
Data Exploration and Preprocessing:
Explore the data structure and summary statistics.
Handle missing or inconsistent data (if applicable).
Encode categorical labels ('R' and 'M') into numerical form.
Feature Selection: Assess feature importance or correlation with the target label.
Model Selection: Choose algorithms like Logistic Regression, K-Nearest Neighbors (KNN), Decision Trees, Random Forest, or Support Vector Machines (SVM).
Model Training: Train selected models on the training dataset.
Model Evaluation: Evaluate the models using metrics like accuracy, precision, recall, and confusion matrix.
Hyperparameter Tuning: Optimize the model using techniques like GridSearchCV or RandomizedSearchCV.
Full Name
Shrishti Singh
Participant Role
Contributor in GSSOC -extd and Hacktoberfest

@UTSAVS26  Please assign me this issue
I would be more than happy to contribute in this project
Thank you!

@UTSAVS26 Please assign me this issue
I would be more than happy to contribute in this project
Thank you!

You have already worked on this one.


[stable/mongodb] Add loadBalancerIP setting

Signed-off-by: James Robson<EMAIL_ADDRESS>What this PR does / why we need it:
Allows you to set loadBalancerIP when using the LoadBalancer service type.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/ok-to-test
/lgtm


Code for the drawing of the characters is a mess

Each new line of code is harder to write than the last, it's a real pain. We need to rewrite all of it.

All cleaned up and absolutely beautiful!


Convert ByDay Graph to Stacked Bar

should either be solid colors of project or stacked bar if all projects are selected

As a more advanced user, I want to see a full breakdown by project so that I can fully analyze my productivity


Error fix, to support Crosswalk-Lite.

Lite only has Canary release.
Remove the 'lzma -d' decompress part which is no use.

@r0b5t4   Please have a look. Thanks.

@r0b5t4   I don't know why "Some checks were not successful"   Is it caused by the Proxy bug?

I don't know either. There shouldn't be any proxy when running in the appveyor environment. Do the tests pass locally in a Windows machine? Or could it be that something changed in the directory structure now that canary releases are generated automatically?

Please note that starting with 19.48.498.0 the Windows canary is called "crosswalk64-.zip" to convey that it's for 64-bit architectures.
Changing the name back, or using a suffix like we do for Android is easy, @darktears just needs to agree with it.

This is now tracked in https://crosswalk-project.org/jira/browse/XWALK-6567

XWALK-6567 is fixed, restarted the CI tests

All good, merging.


Merge 2.1 rc1 branch into release/2.1

Required manually merging conflicts to ResponseTests.cs

RequestDrainingFor100ContinueDoesNotBlockResponse [FAIL]
14:25:29   Failed   RequestDrainingFor100ContinueDoesNotBlockResponse
14:25:29    Assert.Equal() Failure
14:25:29             ↓ (pos 0)
14:25:29   Expected:
14:25:29   Actual:   HTTP/1.1 400 Bad Request\r\nConnection: clo···

Ideas @halter73 @Tratcher ?


Inline variants

Hi,
You just released inline variants for Styled-System v.5.1.0
Can we also do this with rebass? Are there any plans to do this or do we need to always define our variants in the theme file?
Thanks!

Hi, Rebass works a bit differently from Styled System from an API standpoint. The variant feature in both libraries works mostly the same, but the Rebass variants must be defined in the theme. You can extend a Rebass component with @emotion/styled and use the variant utility from Styled System, but I haven't tried this and it might or might not work like expected

That's what we expected. We are going with this approach:
add a variants.ts file to your components where you handle all the component variants and type them. Include every variants.ts file in your theme.variants.

@DiemasMichiels Can you post a gist of that approach?


Redesigned "Rate Us" Page #1265

Here are the changes that i has made in "Rate Us " Page.
Before

After

@vaibhavrajharsh Please resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Support for other grids that RectilinearZGrids in FieldSet.from_data()

It is currently not easy to create Fields on other Grids than RectilinearZGrids using FieldSet.from_data() or Field.__init__(). The only way is to first create the Grid, and only then call Field.__init__. It is not possible by providing lon, lat etc arrays.

@delandmeterp, is this now fixed in #496?

Fixed in #498


Update interface toward Multi-Agent framework

[x] Implementing simple wrapper class for each client
[x] Abstract method to generate multi-modal output
[x] Implementing Agent class
[x] Abstract LLMClient and LLMModel
[x] Support OpenAI API for new interface
[x] Support Gemini API for new interface
[x] Support Anthropic API for new interface
[ ] Fix database modules
[x] Cleanup legacy modules
[ ] Update example notebooks
[x] Update README.md
[ ] Linting
[ ] Revup major version

Remaining plans will be in other PR.


Add support to let PLATFORM add/customize it's own VPD variables

It would be nice if we can let each PLATFORM customize their own VPD variables. Right now Makefile.inc adds internal_versions and firmware_version to VPD variables, for each platform it may want to add more such as fsp_log_enable, coreboot_log_level .,etc.
$(VPD_TOOL) -f "$<" -O -i RO_VPD -s internal_versions="cat $(FINAL_CONFIG_OUT)"
[ -z "$(VERSION)" ] || $(VPD_TOOL) -f "$<" -i RO_VPD -s firmware_version=$(VERSION)
Maybe running a script provided by the PLATFORM for this VPD customization or by other approaches.

@jonzhang-fb it's not a priority but should be needed if people are going to use osf-builder.


Forgot Page

Fixes #38

Great work, couple of suggestion and we are good to merge.

Thank you very much.
Is this better?

Thank you very much. Is this better?  @iZooGooD

That should be fine, also add my-2 to the header text "Reset your password".

@iZooGooD
I have pushed the latest changes


chore: remove pymongo, add bson

Doesn't seem to be used anywhere in the code

meh, bson is a sub module of pymongo.. closing.


Ninja And Cities

💥 Proposal
Ninja decided to find the distance between the neighbouring cities and then store them for future use. He took data from the map and developed an input format. He is given an integer ‘N’ denoting the number of cities and then he has an array of size ‘N - 1’ that stores a pair of numbers at each index. Let the pair be ‘I1’ and ‘I2’, which will denote a bidirectional edge between the two cities ‘I1’ and ‘I2’.
A subtree is a subset of cities, where each city can be reached from every other city of the subset. The path between each pair passes only though the cities present in the subset. Two subtrees are taken differently if there are one or more cities in one subtree not present in the other.
Now, you need to create an array of ‘N - 1’ elements where the ‘ith’ element is the number of subtrees in which the maximum distance between any two cities is equal to ‘i’.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


npm version 0.1.6 contains syntax error

Installing via npm has a syntax error on line 218 of lib/ftpsync.js (seems to be fixed in the master branch of repo).

Fixed, version 0.1.7 is published.


Missing class definition

The sample code under "Auto-property initializers" seems to be missing the declaration of class Student.

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

ID: 36fef60d-04e9-dc86-2e3f-134c10ea37bb
Version Independent ID: 09b5aa7a-1b43-8bf9-9786-f10f457488dd
Content: What's New in C# 6 - C# Guide
Content Source: docs/csharp/whats-new/csharp-6.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@BillWagner I would like to resolve this issue, but what should these examples look like?
I.e., should the very first code example be a complete class file with all properties defined and set via constructor parameters?  The other two examples, Grades and YearInSchool, making modifications to the original class, and then having the fully modified class available at the end as the "new way"?

Thanks for asking @CalvinAllen
Fixing this will take 2 PRs: Our samples are in the dotnet/samples repository.
There is working sample code for the "What's new in C# 6" article in this folder: https://github.com/dotnet/samples/tree/master/snippets/csharp/new-in-6  The new or updated samples should be part of that project.
Instructions on including samples are in our Contributor Guide - Code section. Let me know if you have any questions.

Thanks, Bill, Ill take a look and get started on shortly.

@BillWagner Is it possible to have nested tags?  In the old code, I want to flag the entire class for inclusion, but there are already tags defined within it for the properties and constructor - or, should I create a new class (in the same file, same properties/etc) but without the extra tags?

@CalvinAllen Yes, nested tags are supported.  That is the preferred way to accomplish what you want.

@BillWagner Great, thanks!


POI utterance duration

I recently requested the voxceleb1 data and got access to it. But I couldn't find the POI start and end time values in any of the .txt files provided in the dataset. Any idea whether the dataset format changed and or I am missing anything here to get my research forward?

I have the same problem. Have you solved it?


Distinguish between C 'long double' and C '__float128'.

The AArch64 platform defines C 'long double' as 128-bit floats.
A C '__float128' data type is not supported. This situation results
in wrong C++ mangling: 'g' is used instead of 'e'.
This PR fixes the situation with a new flag Target.realislongdouble.

Auto-merge toggled on

You must be doing something wrong in ldc.

Actually why do we even pretend to support __float128?

@ibuclaw I don't know why there is mangling support for __float128. Given the fact that the whole std.math stuff falls back to C functions which do not support __float128 there is no real need support mangling of this type.

I am unsure what is wrong in fixing a mangling error...

I am unsure what is wrong in fixing a mangling error...

I should have some constraint and not make comments within 60 seconds of consciousness. ;-)
Nothing wrong, but I'd rather it be considered and fixed properly.  Which is why I posed the question.

Maybe the proper solution is to remove support completely. Just leave a comment that there is g used for __float128 but it is not used elsewhere. The GCC page lists the targets which have support for __float128: https://gcc.gnu.org/onlinedocs/gcc/Floating-Types.html It is not a very long list...

Even then it sounds like it should go in the specialist types list, something that could be handled by Target::cppMangle, which you will need for va_list too.


Python: adds JSON.ARRINSERT command

Issue link
This Pull Request is linked to issue (URL): #645
Checklist
Before submitting the PR make sure the following are checked:

[ ] This Pull Request is related to one issue.
[ ] Commit message has a detailed description of what changed and why.
[ ] Tests are added or updated.
[ ] CHANGELOG.md and documentation files are updated.
[ ] Destination branch is correct - main or release
[ ] Commits will be squashed upon merging.

lgtm

thanks!

ci is red - rebase?


🐛 Bug: Text selection around the game should be disabled

Pressing/tapping and dragging triggers text selection

Thanks for the spot @ayoayco! Agreed, you shouldn't be able to do that.
Modifying src/index.css should be the right place to add new styles. But, pending some infrastructural work in EightBittr, you'll have to manually copy over src/index.css to lib/index.css whenever you want it to be updated locally.

Sounds good! I'll try to setup the project and create a PR when I get time soon.

Hi @JoshuaKGoldberg, I set up the project according to the README, and encountered some errors. I'm running http-server on the dist folder here. Maybe you have encountered this before? Don't know if this is the right to ask 😅

@ayoayco this is as good a place to ask as any - and sorry this isn't documented! You want to open lib/index.html directly. No need to run an http-server or anything.
https://github.com/FullScreenShenanigans/EightBittr/issues/336 tracks the lack of docs around getting started. This is very good user research to see what's not described - thank you for posting! 😅

Looks like the text selection happens on the buttons area that shows up on mobile. I was committing a fix on my machine then noticed some temporary files generated by vim is not ignored. I have a small PR to ignore vim temporary files: #30


Image add using admission-controller doesn't work

I'm currently running anchore-engine using the public helm chart and aws ECR repository added using IAM auth.
anchore-cli image add does works, eg:
anchore-cli image add<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com/my-app-name:b076ab4
I can also see in the catalog logs:
[service:catalog] 2019-08-21 10:19:27+0000 [-] [Thread-1088017] [anchore_engine.services.catalog/handle_repo_watcher()] [DEBUG] FIRING: handle_repo_watcher
[service:catalog] 2019-08-21 10:19:27+0000 [-] [Thread-1088017] [anchore_engine.services.catalog.catalog_impl/refresh_registry_creds()] [DEBUG] checking registry for up-to-date: admin :<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com : awsecr
[service:catalog] 2019-08-21 10:19:27+0000 [-] [Thread-1088017] [anchore_engine.services.catalog.catalog_impl/refresh_registry_creds()] [DEBUG] registry up-to-date: admin :<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com : awsecr
[service:catalog] 2019-08-21 10:19:27+0000 [-] [Thread-1088017] [anchore_engine.services.catalog/handle_repo_watcher()] [DEBUG] FIRING DONE: handle_repo_watcher
[service:catalog] 2019-08-21 10:19:27+0000 [-] [Thread-11] [anchore_engine.clients.services.simplequeue/run_target_with_lease()] [DEBUG] Target thread returned

When I started testing the admission-controller the calls to add images fails and I can see in the catalog logs:
[service:catalog] 2019-08-21 10:21:04+0000 [_GenericHTTPChannelProtocol,156582,<IP_ADDRESS>] [PoolThread-twisted.internet.reactor-2] [anchore_engine.clients.skopeo_wrapper/get_image_manifest_skopeo()] [WARN] CMD failed - exception: Error encountered in skopeo operation. cmd=/bin/sh -c skopeo  inspect --raw --tls-verify=false  docker://1234567890.dkr.ecr.eu-west-1.amazonaws.com/my-app-name:b076ab4, rc=1, stdout=None, stderr=b'time="2019-08-21T10:21:04Z" level=fatal msg="Error reading manifest b076ab4 in<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com/my-app-name: unauthorized: authentication required" \n', error_code=REGISTRY_PERMISSION_DENIED
[service:catalog] 2019-08-21 10:21:04+0000 [-] Traceback (most recent call last):
[service:catalog] 2019-08-21 10:21:04+0000 [-]   File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/anchore_engine/clients/skopeo_wrapper.py", line 261, in get_image_manifest_skopeo
[service:catalog] 2019-08-21 10:21:04+0000 [-]     raise err
[service:catalog] 2019-08-21 10:21:04+0000 [-]   File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/anchore_engine/clients/skopeo_wrapper.py", line 243, in get_image_manifest_skopeo
[service:catalog] 2019-08-21 10:21:04+0000 [-]     rawmanifest = get_image_manifest_skopeo_raw(pullstring, user=user, pw=pw, verify=verify)
[service:catalog] 2019-08-21 10:21:04+0000 [-]   File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/anchore_engine/clients/skopeo_wrapper.py", line 226, in get_image_manifest_skopeo_raw
[service:catalog] 2019-08-21 10:21:04+0000 [-]     raise err
[service:catalog] 2019-08-21 10:21:04+0000 [-]   File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/anchore_engine/clients/skopeo_wrapper.py", line 224, in get_image_manifest_skopeo_raw
[service:catalog] 2019-08-21 10:21:04+0000 [-]     raise err
[service:catalog] 2019-08-21 10:21:04+0000 [-]   File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/anchore_engine/clients/skopeo_wrapper.py", line 219, in get_image_manifest_skopeo_raw
[service:catalog] 2019-08-21 10:21:04+0000 [-]     raise err
[service:catalog] 2019-08-21 10:21:04+0000 [-]   File "/opt/rh/rh-python36/root/usr/lib/python3.6/site-packages/anchore_engine/clients/skopeo_wrapper.py", line 214, in get_image_manifest_skopeo_raw
[service:catalog] 2019-08-21 10:21:04+0000 [-]     raise SkopeoError(cmd=cmd, rc=rc, out=sout, err=serr)
[service:catalog] 2019-08-21 10:21:04+0000 [-] anchore_engine.clients.skopeo_wrapper.SkopeoError: Error encountered in skopeo operation. cmd=/bin/sh -c skopeo  inspect --raw --tls-verify=false  docker://1234567890.dkr.ecr.eu-west-1.amazonaws.com/my-app-name:b076ab4, rc=1, stdout=None, stderr=b'time="2019-08-21T10:21:04Z" level=fatal msg="Error reading manifest b076ab4 in<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com/my-app-name: unauthorized: authentication required" \n', error_code=REGISTRY_PERMISSION_DENIED
[service:catalog] 2019-08-21 10:21:04+0000 [_GenericHTTPChannelProtocol,156582,<IP_ADDRESS>] [PoolThread-twisted.internet.reactor-2] [anchore_engine.clients.docker_registry/get_image_manifest()] [ERROR] could not fetch manifest/digest: Error encountered in skopeo operation. cmd=/bin/sh -c skopeo  inspect --raw --tls-verify=false  docker://1234567890.dkr.ecr.eu-west-1.amazonaws.com/my-app-name:b076ab4, rc=1, stdout=None, stderr=b'time="2019-08-21T10:21:04Z" level=fatal msg="Error reading manifest b076ab4 in<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com/my-app-name: unauthorized: authentication required" \n', error_code=REGISTRY_PERMISSION_DENIED

Is there any additional config to fix the auth for skopeo or is this an issue with the controller itself ?

I think I found the issue: it is needed to add the ECR registry within admission-controller account
anchore-cli --as-account admission-controller registry add<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com awsauto awsauto --registry-type=awsecr
even though the registry added as admin was showing when I ran anchore-cli --as-account admission-controller registry list it was needed to add it again within admission controller account

Hi @brennoo, from that log snippet it looks like your anchore engine installation is not configured with credentials to be able to pull the images for analysis. You need to add credentials via the anchore-cli registry add command to enable that access and things should work.

Oh, looks like you just beat me to it. We've fixed some issues in the CLI that should make the registry listing show the correct listing when you are admin using the --as-account flag (https://github.com/anchore/anchore-cli/issues/33)


Training on all classes?

Hello, have you tried training on all CityScapes classes and not just the 5? If so, what score did you achieve at the end?
Thanks!

sorry，i have not tried this yet.


feat(checkbox): Add accessor properties / foundation methods to API

Add checked, indeterminate, and disabled accessor properties /
methods to component / foundation (respectively) mirroring the native
HTMLCheckboxElement API.
Light cleanup and refactoring.

Resolves #4681
[Delivers #128594215]

LGTM, thanks @traviskaufman! Love how much simpler the API now is :)

False negatives on build (Sauce Timeouts). Merging.


Can homebridge query the state of non-homebridge, official HomeKit devices?

I have a Schlage Sense lock, which does not have any sort of API for querying lock state.  It does not support HomeKit triggers at all, like when somebody locks or unlocks it.  Can homebridge, or any open source software, periodically poll the lock state?  I am looking to integrate the Schlage Sense into home-assistant.io.  Maybe I am looking in the wrong place, but I thought I'd ask.

Homebridge itself cannot connect to other HomeKit accessories. Only iOS device can do that.

Thanks


Adds Christopher Prohm as a Contributor

Expanding our list of contributes! Thanks Christopher!
[Short description explaining the high-level reason for the pull request]
Additions

Removals

Changes

Testing

Screenshots
If applicable
Notes

Todos

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the TODO link to standards
[x] Passes all existing automated tests
[x] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future todos are captured in comments
[x] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[x] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Python

[ ] python 3.6
[ ] python 3.7

@elijahbenizzy Thanks. I wasn't quite as active, as I hoped, since all kinds of other (non-tabular) projects got in the way, and @skrawcz  drove most of the implementation. If you think my contributions warrant me being listed, I would be happy about it, though.

@elijahbenizzy Thanks. I wasn't quite as active, as I hoped, since all kinds of other (non-tabular) projects got in the way, and @skrawcz drove most of the implementation. If you think my contributions warrant me being listed, I would be happy about it, though.

Yeah! You pushed for it and got us started with the design. Thanks!

@elijahbenizzy Thanks. I wasn't quite as active, as I hoped, since all kinds of other (non-tabular) projects got in the way, and @skrawcz drove most of the implementation. If you think my contributions warrant me being listed, I would be happy about it, though.

@chmp Hamilton now supports non-tabular data too FYI (we're working on adding better documentation for that)! Thanks for getting the ball rolling!


Autorest Regen Preview to 3.0.0-alpha.20230424.6 by Dapeng Zhang from refs/pull/3259/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3259

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Authorization


Report console messages and errors to driver

We need to report changes to the console in order to show those messages when replaying.

Let's worry about errors first, these will be handled differently from other messages.


it

hello
it's can widget to other widget like HERO?
thank you

Try to wrap your Sidekick into Hero widget like that:
Hero(
   tag: 'heroTag',
   child: Sidekick(
    ...
    )
)


Screenshot in Windows Service

Description
Taking a screenshot of a page only works while debugging inside Visual Studio, if I build my windows service I get stuck on this method.
Complete minimal example reproducing the issue
public async System.Threading.Tasks.Task Async(string url, string outPath)
{
    await new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision);
    var browser = await Puppeteer.LaunchAsync(new LaunchOptions
    {
        Headless = true
    });
    var page = await browser.NewPageAsync();
    await page.GoToAsync(url);
    await page.SetViewportAsync(new ViewPortOptions
    {
        Width = 1366,
        Height = 768
    });
    await page.ScreenshotAsync(outPath);//The process hangs here
    await page.CloseAsync();
    await browser.CloseAsync();
}

Expected behavior:
I can take the screenshot
Actual behavior:
The process hangs in the ScreenshotAsync method
Versions

Puppeteer-Sharp v 1.13.0, I know 1.14.0 is out but I can't update because I get this when trying to update:
"Severity	Code	Description	Project	File	Line	Suppression State
Error		Failed to add reference. The package 'Mono.Posix.NETStandard' tried to add a framework reference to 'Mono.Posix' which was not found in the GAC. This is possibly a bug in the package. Please contact the package owners for assistance.
Error HRESULT E_FAIL has been returned from a call to a COM component."
Targeting .NET Framework 4.7.2

Could you try adding ConfigureAwait(false) on all your awaited tasks?

Done that but it's still not working, tried calling with a simple google url. I find no differences from before. I can see the Chromium process being created in my task manager, as I could before, but the task is still hanging

How do you know that it hangs there if you are not being able to debug it? (honest question)

I actually put a log to file after every await to see where it got stuck
`
await new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision).ConfigureAwait(false);
Log.Debug("Fetched browser with configured await");
        var browser = await Puppeteer.LaunchAsync(new LaunchOptions
        {
            Headless = true
        }).ConfigureAwait(false);
        Log.Debug("Launched browser");
        var page = await browser.NewPageAsync().ConfigureAwait(false);
        Log.Debug("Page created");
        await page.GoToAsync(url).ConfigureAwait(false);
        Log.Debug("Url resolved");`

@AzeExMachina Can you connect a LoggerFactory on LaunchAsync so we can have a full dump of all Puppeteer`s messages?

Ok, I was able to add the trace to the launcher, these are the last lines I can see:
https://pastebin.com/jWBBCGDm

@AzeExMachina I would try this:
Open the Services control panel. For example: Start > Control Panel > Administrative Tools > Services.
Select the {your service} service.
Right-click the service name, and select Properties. The Service Properties window is displayed.
Select the Log On tab.
Select Local System account and then select Allow service to interact with desktop.
Click OK.
Click Restart.

It might be that chrome still need a window(that's not shown) to actually take the screenshot.

@AzeExMachina I was able to reproduce #1022. Maybe they are related.

@scabana Thanks for your suggestion, however that didn't work, I've tried a couple of times but it still got stuck in the same place.
@kblok Thanks, hope so!
I've seen that this issue only happens when inside the service, I've tried building a console application and that actually works, sadly that's not an acceptable solution for my case

Maybe related to this https://github.com/GoogleChrome/puppeteer/issues/4290 ?

Yes, definitely related. I've tried downgrading to puppeteer-sharp 1.11.0 and it actually works there. I don''t know if the issue is related to puppetteer running headless, because it actually works if you run it as a fake console application, outputting to Windows Application and not console. However, for now I'm keeping puppeteer to 1.11 until they are able to solve this.
Thanks @kblok for the heads up!

@AzeExMachina I downgrade to 1.11 but it still doesn't work. Do you deploy webapp with InProcess profile?

its about how to break session0 in windowsService


Prepare the changelog for 0.1.3 release

Summary
Prepare the changelog for the 0.1.3 release.
Metadata

Question
Answer

Did you use a meaningful pull request title?
yes

Did you apply meaningful labels to the pull request?
yes

Did you perform a self review first?
yes

Documentation reflects changes?
no

CHANGELOG reflects changes?
yes

Unit/Functional tests reflect changes?
n/a

Did you perform browser testing?
no

Risk level
Low

Relevant links
RIG-6

LGTM! /approved


Ability to specify order for configs and environments

Is your feature request related to a problem? Please describe.
Currently it's not possible to define an order for configcat_config and configcat_enenvironment. They are linked with the product, but sometimes they end up in random order and require manual re-ordering in configcat console.
Describe the solution you'd like
Provide order attribute to the configcat_config & configcat_environment resources.

Hello @opendba ,
Thanks for reaching out. Our Public Management API currently does not support ordering the resources and the Terraform provider uses our Public Management API.
We just received another request regarding supporting the ordering in our Public Management API in the past few days from a customer. We added both of these to our backlog and we'll inform you if we made any progress with it.
By the way, if the order is not provided somehow for the resource we have a fallback to alphabetical ordering, and maybe that's why you see it as random. As a short-term workaround I can suggest you to prepend a number order before your resource names. I totally understand that it is not ideal, but it can work till we develop these features.
Cheers,

Hello @opendba ,
I just release v3.0.0 with supporting the order in the configcat_product, configcat_config, configcat_environment and the configcat_setting resources. It is a breaking change, you'll have to define order for these resources.
I hope you'll like it.
If you are happy with our service, please help us grow by leaving a review. https://www.trustradius.com/welcome/configcat
It really makes a difference for us.

Hello @opendba ,
I'm closing this issue. Let us know if you have any questions. Have a nice day and happy feature flagging!
If you are happy with our service, please help us grow by leaving a review. https://www.trustradius.com/welcome/configcat
It really makes a difference for us.


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit 4d7ce3f:
:clock10: Pending: waiting for processors (30 builds ahead of you)
:warning: Docs Build is busy, currently there are 30 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 4d7ce3f:
:clock10: Full build: the average full build time is 5 min(s) 47 sec(s), this is based on the last 30 build(s) for this repository.


Legend only works if chart wrapped properly

When using the legend=true option, the ability of the legend to display is dependent upon the parent element of the chart.
    if (this.get('legend')) {
      var legend = chart.generateLegend();
      this.$().parent().append(legend);
    }

When the parent was the html body, the legend didn't show up.  When I specifically wrapped the chart in a div, it showed up.
There are two ways I see of fixing this

Document the requirement
Have the legend put inside the chart component

Thanks for writing a great addon, and I hope this helps make it even better!

@jeffreybiles thanks for reporting this!
Option 2 can't be done because the component is a <canvas> element so we can't put html in there. So we either document this better or change the component into a div with a canvas in it.

Awesome, thanks!
On Jun 6, 2015 7:57 PM, "Ahmed Omran"<EMAIL_ADDRESS>wrote:

Closed #21 https://github.com/aomra015/ember-cli-chart/issues/21 via #25
https://github.com/aomra015/ember-cli-chart/pull/25.
—
Reply to this email directly or view it on GitHub
https://github.com/aomra015/ember-cli-chart/issues/21#event-324468198.


Is it possible to use multiple settings file?

We can have a scenario where there are certain integration test cases written which can run on some specific configuration mentioned in one setting file and some other integration test cases written which run on another settings file.
Is it possible to run all of them by providing both the settings file and have a criteria which setting file should be used to run any particular integration test case?

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

ID: 16bd8dd1-7650-8c18-2553-38a8bc54e892
Version Independent ID: f9fa517a-7adc-e327-d12e-256295a1ed52
Content: Visual Studio Test task - Azure Pipelines
Content Source: docs/pipelines/tasks/test/vstest.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @PBoraMSFT
Microsoft Alias: pbora

@DhirajKhot Thank you for the question, assigning this to the author for review of the current documentation.
If this feature is not yet supported, please use the Azure DevOps community to solicit additional feedback from the product team.
For additional support options, see:

Azure DevOps Support
Azure DevOps on StackOverflow

@DhirajKhot - take a look at the multi config setting for driving the VSTest task. You can use the config variable in different ways - to drive settings in the same file, use that to drive file path names, etc. to achieve your scenario.
https://devblogs.microsoft.com/devops/vstest-task-dons-a-new-avatar-testing-with-unified-agents-and-phases/


WalletId not an instance of GHC.Enum.Enum

Area
[] Plutus Foundation Related to the GHC plugin, Haskell-to-Plutus compiler, on-chain code
[x] Plutus Application Framework Related to the Plutus application backend (PAB), emulator, Plutus libraries
[] Marlowe Related to Marlowe
[] Other Any other topic (Playgrounds, etc.)
Summary
Using Wallet in List Comprehension produces the following error:
    • No instance for (GHC.Enum.Enum WalletId)
        arising from the arithmetic sequence ‘1 .. 3’
    • In the expression: [1 .. 3]
      In a stmt of a list comprehension: w <- [1 .. 3]
      In the first argument of ‘Map.fromList’, namely
        ‘[(Wallet w,
           case w of
             1 -> v
             _ -> v') |
            w <- [1 .. 3]]’
   |
49 |                                                                 _ -> v') | w <- [1 .. 3]]) def def

Steps to reproduce
    emCfg :: EmulatorConfig
    emCfg = EmulatorConfig (Left $ Map.fromList [(Wallet w, case w of
                                                                1 -> v
                                                                _ -> v') | w <- [1 .. 3]]) def def

Expected behavior
Expected it to work like it used to.
System info (please complete the following information):

OS: Ubuntu (WSL)
Version 20.04
Plutus version or commit hash: f4aa043660eb8f137796f6e2d6684cf8d0418c1e

Since recent update WalletId contains either private key for mock wallets and pub key hash for actual wallets. Having Enum for wallet id would not make sense, as it doesn't work with integers anymoore.
You can use knownWallets :: [Wallet] from Wallet.Emulator.Wallet module instead.

Yep, @kelizarov is right (thanks for the answer). WalletId was changed in 58c8b4bde7177a18cd6761ba328bc105e5c82a47.
If you want to enumerate wallets you should use knownWallets :: [Wallet] or knownWallet :: Integer -> Wallet.


[FEATURE] Support heterogeneous shape

Like of_size(short,char,fixed<N>, int> etc

Also
shape<some_axis[short(n)]> x;

shoudl have proper base_type as short

OK this is getting in shape along with #99 but it makes the dynamic oeprator[] almost impossible to implement.
Cna we ditch it ? @psiha ?

OK problem solved. Dynamic operator[] requries homogenous typing inside the shape/stride


[Event Request] Table 36 Sales Header - handled pattern for UpdateBillToCust

Dear BaseApp Team,
In our vertical we need an alternate handling of contacts on sales headers, to be able to send invoices to different recipients. To do that, we need the Handled pattern for the function UpdateBillToCust
Requested code in Table 36 "Sales Header":
local procedure UpdateBillToCust(ContactNo: Code[20])
var
ContBusinessRelation: Record "Contact Business Relation";
Cust: Record Customer;
Cont: Record Contact;
CustTemplate: Record "Customer Template";
SearchContact: Record Contact;
ContactBusinessRelationFound: Boolean;
IsHandled: Boolean;
begin
//>>L365
IsHandled := false;
OnBeforeUpdateBillToCustL365(Rec,ContactNo,IsHandled);
if IsHandled then
exit;
//<<L365
    if not Cont.Get(ContactNo) then begin
        "Bill-to Contact" := '';
        exit;
    end;

...
[IntegrationEvent(false, false)]
local procedure OnBeforeUpdateBillToCustL365(var SalesHeader: Record "Sales Header"; ContactNo: Code[20]; var Handled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


MJO xcdat conversion

This PR will resolve #1084.
Draft PR made to track changes. PR is not ready to merge until it is explicitly marked as ready.

Progress note:
The new code was able to run the demo notebook without noticeable error, but with differences in result metrics.
A rigor test (run the code for all CMIP5/6 models/realizations) conducted.

Calendar-related error for non-standard calendar using models is found.
Some differences in the results are found.

Code revision:

To avoid date comparison error from non-standard calendar using models, more simple and straightforward date comparison is applied. It is confirmed the error from the non-standard calendar models is resolved.
Common grid is changed from uniform grid to Gaussian grid to make the xCDAT code more consistent to the CDAT/CDMS version.

Next step:

Another rigor test (run the code for all CMIP5/6 models/realizations) is launched. Its result will be analyzed once the test run is completed.
Custom season capability when crossing calendar year is not been fully tested, and there to be potential issue (maybe using discontinued months, which could affect to NDJFMA season)

Usage of more consistent grid (uniform --> Gaussian) did not dramatically changed results (Compare below with above).

CDAT vs xCDAT results seem to be acceptable (with high correlation and p<0.05), but more investigation for a few outliers would be helpful because their diagnostics fields are looking like identical (see second below figure).

Progress note: after https://github.com/PCMDI/pcmdi_metrics/pull/1091/commits/f45530781b579664c3e98c3df9171213b71e9233, result of demo notebook 5 became much more consistent than before.
3rd round rigorous test is in progress.

The above fix completed resolved the inconsistency! The 3rd round of he rigorous test results below.

East West Power Ratio:

East Power:

West Power:

@acordonez I believe this PR is ready. Can you please try install code in this PR and run the demo 5 notebook to see if that works okay to you as well?

@lee1043 Sorry for taking a few days to get to this! I ran the notebook and found no issues there. I remember you said it was expected to see some slight differences in the metrics? E.g. an east power of 0.016568144258499968 versus 0.016600374830362644?

@lee1043 Sorry for taking a few days to get to this! I ran the notebook and found no issues there. I remember you said it was expected to see some slight differences in the metrics? E.g. an east power of 0.016568144258499968 versus 0.016600374830362644?

@acordonez thank you for reviewing. Yes, I consider that the difference you found is subtle enough to be neglected.


npm package should also deploy base sass

Describe the bug
The npm package should also deploy base sass so variables and helpers can be used in prototype kit

 npm package should also deploy base sass


Add dbus & pgi to python requirements

These are used by the python bindings; add them to requirements.in.
Fixes #4087

@andy31415 @saurabhst ?


LogicalType is nil when is Decimal

Using a parquet file with the following schema:
message schema {
  optional int32 int32_decimal (DECIMAL(9, 0));
  optional int32 int32_decimal_s2 (DECIMAL(9, 2));
  optional int64 int64_decimal (DECIMAL(12, 0));
  optional int64 int64_decimal_s2 (DECIMAL(12, 2));
  optional fixed_len_byte_array(16) decimal_s0 (DECIMAL(38, 0));
  optional fixed_len_byte_array(16) decimal_s2 (DECIMAL(38, 2));
}

When the following code that prints the PhysicalType and LogicalType for each Field is run:
parquetFile, err := parquet.OpenFile(reader, metadata.Size)
if err != nil {
	return err
}

for _, field := range parquetFile.Schema().Fields() {
	dataType := field.Type().PhysicalType().String()

	if field.Type().LogicalType() != nil {
		dataType += ":" + field.Type().LogicalType().String()
	}

	fmt.Println(dataType)
}

it is expected to get:
INT32:DECIMAL(9,0)
INT32:DECIMAL(9,2)
INT64:DECIMAL(12,0)
INT64:DECIMAL(12,2)
FIXED_LEN_BYTE_ARRAY:DECIMAL(38,0)
FIXED_LEN_BYTE_ARRAY:DECIMAL(38,2)

But I got:
INT32
INT32
INT64
INT64
FIXED_LEN_BYTE_ARRAY
FIXED_LEN_BYTE_ARRAY

Hello @senekis, thanks for reporting and providing a file that exhibits the issue! We will look into the reason why this is happening 👍

Actually, it appears that I do not have permissions to read the file you linked. Would you be able to upload it to the Github issue?

Hello @senekis
Just a friendly ping in case you have a few spare cycles to provide a copy of the file.

Hi @achille-roussel, sorry for the late response, I 'm on vacation..
I changed the permission to the file, you should be able to see it now.

@senekis Thanks for reporting this bug! Here's the schema I see in the file you posted:
  optional int32 int32_decimal (DECIMAL(9,0));
  optional int32 int32_decimal_s2 (DECIMAL(9,2));
  optional int64 int64_decimal (DECIMAL(12,0));
  optional int64 int64_decimal_s2 (DECIMAL(12,2));
  optional fixed_len_byte_array(16) decimal_s0 (DECIMAL(38,0));
  optional fixed_len_byte_array(16) decimal_s2 (DECIMAL(38,2));
  required int64 timestamp_millis (TIMESTAMP(MILLIS,false));
}

and here's what I see running your code after merging #406:
INT32:DECIMAL(0,9)
INT32:DECIMAL(2,9)
INT64:DECIMAL(0,12)
INT64:DECIMAL(2,12)
FIXED_LEN_BYTE_ARRAY:DECIMAL(0,38)
FIXED_LEN_BYTE_ARRAY:DECIMAL(2,38)
INT64:TIMESTAMP(isAdjustedToUTC=false,unit=MILLIS)

Sadly, it seems that parquet-go's string representation of DECIMAL (scale, precision) is opposite of parquet-cli (precision, scale) which is what I'm using to print the schema above. Regardless, this should be fixed.

Sadly, it seems that parquet-go's string representation of DECIMAL (scale, precision) is opposite of parquet-cli (precision, scale) which is what I'm using to print the schema above. Regardless, this should be fixed.

@lmarburger I haven't found any formal specification of the string representation of parquet schemas, but this seems worth changing to align with parquet-cli?
The thrift fields are ordered with scale first and precision second, but if the convention is the opposite for human-readable formats then let's align?

I have no problem either way. It was strange looking through the parquet-cli code and seeing all references as (scale, precision) and then printed in reverse order. Changed in #407.


Test and fix the profiling way

We currently disable the profiling way in Settings/User.hs, partly to speed up default builds. However, it may actually work correctly. We need to test it and consider enabling it by default.

[x] Test and fix the profiling way
[x] Consider enabling it by default or as part of the profiling flavour
[x] Add a command line flag to enable/disable profiling

Just a heads up: mk/flavours/prof.mk is for profiling GHC itself (and hardly anyone ever does, that's why it so slow, ha ha ha). To profile GHC, you need to build the libraries the p way first. Some build flavours skip this by setting BUILD_PROF_LIBS=NO.
GHC releases don't ship with a profiled GHC. They do include a copy of the libraries built the p way (excluding the ghc library), so users can profile their code.

@thomie Good to know, thanks! I didn't realise libraries and GHC were treated differently w.r.t profiling.

Library profiling seems to work in my tests, so I am closing this issue.
Building a profiled GHC (GhcProfiled = YES) is another matter, which I think deserves a separate issue.


有时不能控制 unexpected data length

-------> a.toggle_display()
2023-08-11:02:06:24,776 ERROR    [security.py:105] unexpected data length
2023-08-11:02:06:24,777 ERROR    [lan.py:127] Authentication failed for <IP_ADDRESS>:38288 -> <IP_ADDRESS>:6444 retries: 5 time: 0.0
2023-08-11:02:06:24,777 WARNING  [base.py:87] Got Null from <IP_ADDRESS>:6444 Version: 3 Count: 0 Spend time: 1.04
2023-08-11:02:06:24,799 ERROR    [security.py:105] unexpected data length
2023-08-11:02:06:24,799 ERROR    [lan.py:127] Authentication failed for <IP_ADDRESS>:38298 -> <IP_ADDRESS>:6444 retries: 6 time: 0.02
2023-08-11:02:06:24,799 WARNING  [base.py:87] Got Null from <IP_ADDRESS>:6444 Version: 3 Count: 0 Spend time: 0.02

有时隔两个小时可以，有时不行  ，显示屏背光不亮

Can you enable debug logging?
e.g.
logging.basicConfig(level=logging.DEBUG)

2023-08-12:22:43:07,60 WARNING  [appliance.py:165] Device is not capable of operational mode 2.
2023-08-12:22:43:07,61 WARNING  [appliance.py:169] Device is not capable of swing mode 0.
2023-08-12:22:43:07,62 ERROR    [lan.py:70] Send <IP_ADDRESS>:48614 -> <IP_ADDRESS>:6444 retries: 0 time: 11802.12 Error: [Errno 104] Connection reset by peer
2023-08-12:22:43:07,152 INFO     [lan.py:122] Got TCP key for <IP_ADDRESS>:45740 -> <IP_ADDRESS>:6444 retries: 0 time: 0.06 tcp_key: 6a3145393b555d49b82b32aa049f04ed30707d67cbecd9be756156166712cf43

In [5]: U.set_log_level(10)
Out[5]: 10

In [6]: a.power_state=True;a.target_tem
   ...: perature=17;
   ...: a.display_on
   ...: a.apply()
2023-08-12:22:43:32,633 WARNING  [appliance.py:165] Device is not capable of operational mode 2.
2023-08-12:22:43:32,633 WARNING  [appliance.py:169] Device is not capable of swing mode 0.
2023-08-12:22:43:32,635 INFO     [appliance.py:155] Ignored special response. <IP_ADDRESS>:6444 a100005370000000000000000070ff000000030002
2023-08-12:22:43:32,635 INFO     [appliance.py:155] Ignored special response. <IP_ADDRESS>:6444 a1000053700000000000000000717a000000350002
2023-08-12:22:43:32,636 INFO     [appliance.py:155] Ignored special response. <IP_ADDRESS>:6444 a00b40667f7f00000000000000000000000000000000002000000000000002
2023-08-12:22:43:32,636 INFO     [appliance.py:155] Ignored special response. <IP_ADDRESS>:6444 a00b40667f7f00000000000000000000000000000000002000000000000002

In [7]: a.power_state=True;a.target_temperature=30;
   ...: a.apply()
2023-08-12:23:58:46,247 WARNING  [appliance.py:165] Device is not capable of operational mode 2.
2023-08-12:23:58:46,248 WARNING  [appliance.py:169] Device is not capable of swing mode 0.
2023-08-12:23:58:46,249 ERROR    [lan.py:70] Send <IP_ADDRESS>:45740 -> <IP_ADDRESS>:6444 retries: 0 time: 4539.15 Error: [Errno 104] Connection reset by peer
2023-08-12:23:58:48,254 ERROR    [lan.py:42] Connect Error: <IP_ADDRESS>:6444 timed out
2023-08-12:23:58:48,254 ERROR    [lan.py:60] Sokcet is None: <IP_ADDRESS>:6444
2023-08-12:23:58:48,254 ERROR    [security.py:105] unexpected data length
2023-08-12:23:58:48,255 ERROR    [lan.py:127] Authentication failed for <IP_ADDRESS>:45740 -> <IP_ADDRESS>:6444 retries: 1 time: 4541.16
2023-08-12:23:58:48,255 WARNING  [base.py:87] Got Null from <IP_ADDRESS>:6444 Version: 3 Count: 0 Spend time: 2.01

In [8]: a.power_state=True;a.target_temperature=30;
   ...: a.apply()
2023-08-12:23:59:02,999 WARNING  [appliance.py:165] Device is not capable of operational mode 2.
2023-08-12:23:59:02,999 WARNING  [appliance.py:169] Device is not capable of swing mode 0.
2023-08-12:23:59:03,74 INFO     [lan.py:122] Got TCP key for <IP_ADDRESS>:47042 -> <IP_ADDRESS>:6444 retries: 0 time: 0.06 tcp_key: af904dc8d8b4d9bd1b59b35834cd031da0eb78949227df4cfcced1e58a55c9b0

NO ,  sometimes can't control anything

I don't recognize this as a valid command: In [5]: U.set_log_level(10)
Enable debug logging with logging.basicConfig(level=logging.DEBUG)

@mill1000  the two commands are equal.  U.set_log_level just a wrap.

But there are no DEBUG lines in your logs above.


(Suggestion) Mention virtualenv in "Getting Started"

I recommend adding a section on virtualenv (+ virtualenvwrapper?) in the Getting Started page. Given how quickly things are still changing with DroneKit-Python (which is good!) I think anyone writing code that they want to deploy or upgrade would benefit from virtualenv. It'd be helpful both in allowing you to install an older version of dronekit if your app/script relies on that, and in safely testing new versions. I'd be willing to give it a shot in a few days, but I wanted to mention it now so anyone can chime in with any additional suggestions.
Thanks!

Hi @atomictom
Great idea ... but:
I see virtualenv as a deployment solution not as a getting started topic. I do agree that it is a useful model for defining a common environment (and is what is used in Solo). So we can link from gettting started but this should not be fully documented there.
I'm not an expert so would really appreciate if you could write this (as a standalone topic - part of "Deploying a script"). Can this have test and example code that i can run on Windows/Linux/Mac?
Thanks!


[Support] How to accept both event and my parameter?

Hello @KoRiGaN, I would like to have some help on this:
I have a set of markers like this:
<l-marker
    v-for="(site, index) in sites"
    :key="site.site_name + index"
    :lat-lng="site.coordinates"
    :icon="siteIcon"
    @moveend="updateCoordinates(site)"
/>

I can access the event object if call only @moveend="updateCoordinates", but in my function, I need to access both, event and site object.
methods: {
    updateCoordinates(event, site) {
        // Do something with both event and site.
    }
}

If I use both @moveend="updateCoordinates(event, site)", Vue will say event does not declare. If I use only site, I can not access event.
Do you have any suggestion to work around this?
Thank you in advance.

Hi @sangdth,
Please use $event as explained in VueJS doc.
Micka

I was using @click event to work around, but the $event is cooler. Thank you sooo much. I close this.


Add reference_array option for train

When enhancing training data, the reference_array is not defined. This PR adds an option to fix the reference_array to one array.

Jenkins run was successful. I merge now.


Add SSL support to JedisCluster

Creating new pull request for these changes to reopen #1550.

This would be very nice to have on a release. Tested it with Elasticache and works well.

Can this PR be closed? Assuming we have https://github.com/xetorthio/jedis/pull/1896?

@gkorland We shouldn't close this PR without merging #1896 . This would allow @joroda to do further works here if he wants to.


Add access control information

Add info on how we manage access to accounts.

One broken link, but otherwise, all good

LInk fixed


HTML title attribute for column header should not be added when custom template is being used

I'm submitting a ...
[ ] bug report => search github for a similar issue or PR before submitting
[X] feature request
[ ] support request => Please do not submit support request here

Current behavior
When you use ng2 template tag for column header, TITLE attribute is being added automatically.
Expected behavior
When you use ng2 template tag for column header, TITLE attribute shouldn't be added (or be configurable at least).
Reproduction of the problem
N-A.
What is the motivation / use case for changing the behavior?
This may confuse user, especially if any tooltip/popover plugin is being used within custom template.
Would be nice to have TITLE attribute in column headers configurable.
Please tell us about your environment:
N-A

Table version: <IP_ADDRESS>

Angular version: 4.0

Browser:
All

Language:
ES6/7

Good call, can you PR?

@litzebauer is your commit above ready for PR or is it WIP?

@eddy-geek It's ready for PR


thomas 10/21

Move billing content to the main repo

Hi @genlin, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

genlin is a MSFT employee; I will be working on this pull request on Wednesday

Moved to private repo.


Ubuntu hack

This PR and associated process will create a kubernetes cluster on AWS using Ubuntu via kraken.  This is a research project and should be treated as such.
The process works as follows:

run hack/dockerdev to get into a good kraken-lib environment
run bin\up.sh --generate <your config file>
edit generated config file to only have etcd, master and clusterNodes nodePools
run KRAKEN_EXTRA_VARS="distro=ubuntu" bin/up.sh --config <your config file> --tags pre_provider
edit bin\ubuntu-aws.py to  to point to the cloud config files generated by the previous call to kraken lib on your machine
run ubuntu-aws.py
replace the existing cloud-config tarballs:

cd to location of cloud-config files
rm *.gz
gzip < etcd.cloud-config.yaml > etcd.cloud-config.yaml.gz
gzip < master.cloud-config.yaml > master.cloud-config.yaml.gz
gzip < clusterNodes.cloud-config.yaml > clusterNodes.cloud-config.yaml.gz

run KRAKEN_EXTRA_VARS="distro=ubuntu" bin/up.sh --config /Users/pat/.kraken/ubuntu-aws.yaml --tags provider_only to create the physical infrastructure
get the CNAME for the master elb for this cluster
run KRAKEN_EXTRA_VARS="distro=ubuntu" bin/up.sh --config /Users/pat/.kraken/ubuntu-aws.yaml --tags post_provider -k <master elb> to configure the cluster

There is a bit of a race condition so sometimes you need to manually restart the kubelet on a worker node.  ssh to the node and run systemctl restart kubelet

closing this, kraken-lib is dead and this was a fun but ultimately futile code path.  bye!


Allow me to RE-introduce the DevTools plugin! 🧰

Hey, I just made a Pull Request!
Allow me to RE-introduce the DevTools plugin - another handy tool to add to your Backstage 🧰 ! This will close #9737 and hopefully open a world of new features to help Backstage Adopters who run the Ops side of things 😉
Note: I've moved this to my work account so that I can more easily get this merged in; the previous Draft PR is #14642
The plugin offers the following features:
Lists info about your current Backstage instance - OS, NodeJS version, Backstage version and packages:

Lists the config being used by your current Backstage instance:

There's also an optional tab where you can list connectivity with External Dependencies:

The plugin has also been designed so you can customize the tabs - remove them, add them, and/or create your own.
Support for the permissions framework is included in an open way so that you can control this as best fits your needs. There is detailed guidance in the plugin's README.md on this topic.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@adamdmharvey @sennyeya @regicsolutions @efenner-cambia @webark just a heads up that I moved PR #1642 to my work account and moved it to ready so that I can more easily get it merged in. Thanks a ton for your patience, just a bit longer as I work though getting this reviewed now 🚀

@freben do I need to do anything special with the DCO? @ahhhndre is me so not sure if this needs anything special.

This looks great! Is it possible to configure any external dependency through the config?
Looking forward to test it 😁

Thanks for the initial review @benjdlambert, ready for another look when you have time 👍


GH action to publish extension to VSCode marketplace

Maybe use this: https://github.com/marketplace/actions/publish-vs-code-extension?

took some trial and error but fixed by the following:
https://github.com/lwkchan/make-a-type/pull/24
https://github.com/lwkchan/make-a-type/pull/25
https://github.com/lwkchan/make-a-type/pull/26


Details on devices without user affinity

Please add details on how to use this on iOS devices that are enrolled without user affinity. Seems like the app does nothing unless a user logs in. Users can also logout, which stops protection, so seems rather pointless.

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

ID: f9d7718e-f707-9e4d-8583-4a6385b9b40c
Version Independent ID: e5abdc8e-1b61-ca7c-d73c-b4d9d3a6de48
Content: App-based deployment for Microsoft Defender for Endpoint on iOS
Content Source: microsoft-365/security/defender-endpoint/ios-install.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@mjcaparas Please help us in resolving this issue. Thanks

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


74 activity tab content

Issue #74

Description

Created the function that makes up all correctly positioned components of the Activity Tab with all the data getting API values.
Chart of Commits per Week and Pull Request Interaction, Progress Bars and Check Model of Recent Realese Note.
Screenshots:

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Codecov Report

Merging #81 into devel will increase coverage by 0.18%.
The diff coverage is 6.25%.

@@           Coverage Diff            @@
##           devel     #81      +/-   ##
========================================
+ Coverage   4.08%   4.26%   +0.18%
========================================
  Files         19      20       +1
  Lines        294     305      +11
  Branches      13      13
========================================
+ Hits          12      13       +1
- Misses       269     279      +10
  Partials      13      13

Impacted Files
Coverage Δ

app/scripts/contentscript.js
0% <0%> (ø)
:arrow_up:

app/scripts/activityPage.js
100% <100%> (ø)

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 543af02...3c0d942. Read the comment docs.


Adding ArgoCD notifications controller metrics

What does this PR do?

Adding Notification Controller Metrics: https://argo-cd.readthedocs.io/en/stable/operator-manual/notifications/monitoring/
Motivation

They were not currently exposed via this integration
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

👋 @maxknee Thanks for the contribution! I took a look and left some small tidbits. Since we're adding a new endpoint to the instance that the e2e uses, there's going to be failing test because the new metrics won't be asserted. I think running ddev env test argocd -ne should flag them for you. If you could update the e2e test with the new notification-controller metrics, I think we wrap this up. If not no worries, I can take this over. Thanks in advance and thanks again for the contribution!

@maxknee left 2 more nits. But other than that, it looks great! Thanks again for the contribution.


allow to hide job via GUI

Also realized that the delete of columns in the sample info file wasn't asking for confirmation from user.

Codecov Report

Merging #2593 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2593   +/-   ##
=======================================
  Coverage   94.42%   94.42%
=======================================
  Files         160      160
  Lines       19489    19489
=======================================
  Hits        18403    18403
  Misses       1086     1086

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 434f782...577e31d. Read the comment docs.

Forgot to add screenshot!


Feature Request: Ability to have certain panels be open by default when opening a new bin

I think that a lot of developers use the awesome JSBin like I do, which is when I need to test something out I immediately open up the website and start typing.  But I do this often enough that it becomes tedious to have to select the tabs(?) that I want open each time - for me this is generally JS (ES6/Babel) and Console.
An example implementation of this feature would be that after logging in you can maybe 'star' a few tabs right on the tabs themselves (using a filled/unfilled star button on the tab) which would then cause them to be the default ones whenever you load a new JSBin.
Thanks again for JSBin it is an incredible time-saver.

Whoops, duplicate of #2614


Bump version: NirSoft.DNSLookupView version from 1.12 to 1.15

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Resolve #201814

[!WARNING]
This is an attempt squashing addition and removal into a single pull request.

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.9 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Manual validation


allow react version to 18.2.0 for dependency

Added support for react version 18.2.0

It would be great to have support for React 18; much needed. Author, please merge the ticket as soon as possible.


Bluetooth: Controller: Fix CTE_REQ disable lock if there is no CTE_RSP

First item:
There is an error in the condition that checks if new CTE_REQ should
be started after end of connection event. The condition verifies if
counter req_expire is set to zero. Then new CTE_REQ is started
irrespectively to CTE_REQ being disabled.
req_interval is used to store information if the CTE_REQ is:

periodic, then value doesn't equal zero,
single shot or disabled, then value equals zero.

The condition should verify if the req_interval is not zero and
req_expire is not zero. The second part of the if condition is
required to avoid starting next CTE_REQ until last one has been
completed.
Second item:
The periodic CTE_REQ disable command, requested by Host, may be locked
until connection is dropped due to missing CTE_RSP from peer device.
That is caused by implementation of CTE_REQ disable and CTE_REQ
control procedure handling.
The procedure is marked as active when CTE request was send to peer
device. It is marked as inactive after completion of the procedure.
That caused locking of CTE disable on a semaphore.
The BT 5.3 Core Spec, Vol 4, Part E, section 7.8.85 says the HCI_LE_-
Connection_CTE_Request_Enable should be considered active on a conne-
ction from when Host successfully issues the command with Enable=0x1
until a command is issued with Enable=0x0 or single LLCP CTE request
has finished (CTE_Request_Interval=0x0). Also there is a clarification
from BT SIG that the command with Enable=0x0 does not affect any
initiated LLCP CTE request. That means Controller is allowed to finish
already started procedure and it is not allowed to start new LLCP CTE
request procedure after completion of the command with Enable=0x0.
Taking that into account, there is no need to synchronize ULL and LLL
in regard of disable the LLCP CTE request while the procedure is
pending. Controller is free to complete the procedure or terminate it.
The change removes all code related with cte_req.is_active, disable
callback and waiting of ULL for LLL to finish the LLCP CTE request.
The ULL will complete the HCI_LE_Connection_Request_Enable with
Enable=0x0 immediately. In case the procedure is disabled in before
the response arrives, then further processing of the response is
dropped and the procedure context released.
The context is not released by the code responsible for disable
handling, to have single place where it is done.

Updated handling of disable procedure after receive clarification in erratum: https://www.bluetooth.org/errata/errata_view.cfm?errata_id=18686.

Updated due to conflict with upstream code.

Fixed failing test in CI.