Missing Service Tag AzurePortal

AzurePortal Service Tag is missing from documentation, but appears in Azure Portal as an available Service Tag for NSG.

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

ID: 08217244-82bb-13b5-46f7-454801d8f865
Version Independent ID: 4a8b0a32-bf57-7544-c60d-6b7f250c4477
Content: Azure service tags overview
Content Source: articles/virtual-network/service-tags-overview.md
Service: virtual-network
GitHub Login: @asudbring
Microsoft Alias: allensu

@adammontlake
Thanks for your feedback! We will investigate and update as appropriate.

@adammontlake
Thank you for reaching out.
AzurePortal tag is currently not supported by NSG i.e. although you can list it in the nsg rule but it will not have the desired effect. If you wish to block access to Azure Portal from your VM you can do it via Azure Firewall. Azure Firewall offers FQDN filtering functionality as shown here. You can also take a look at Web Categories of Azure Firewall.
Hope this helps! Please tag me in your response if you have any additional questions. Thank you!


Enable network inspection for HttpClient

Is it possible to enable network inspection for HttpClient? thx.

Yes, however we provide no helpers to facilitate this.  There is a lower
level "raw" API for hooking this up yourself called NetworkEventReporter.
See stetho-okhttp/urlconnection source for clues as to how you'd wire this
up manually.
HttpClient does also have an interceptor type system similar to okhttp, but
in my experience it is much less complete and is harder to properly
generalize.  That's why we didn't pursue a stetho-httpclient helper.

@jasta thanks for your info and quick reply.

jasta, which part is harder to generalize? it seems friendlier than url connection. i'll write it if you guys would merge it into stetho.
btw, when i was at fb i was working on this very tool, but 3 years earlier :) never finished but glad you guys did. looks great.

thanks

Hey @lklots, have you done this for us? (:

nope :(


Revert "Output warning in LSP for untyped code"

Reverts sorbet/sorbet#6797

The original PR was not ready.


fixes scrolling animation bug

...the bug I introduced in 3698ede .

Good catch. Thanks for fixing this. :+1:


change log level to debug

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[X] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

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

Test name
Commit
Details
Required
Rerun command

release-notes_istio
623fcab3d3deee8411bc5e6ac7c3943be9bcbef4
link
true
/test release-notes_istio

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

/test release-notes_istio

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

Test name
Commit
Details
Required
Rerun command

integ-pilot-k8s-tests_istio
623fcab3d3deee8411bc5e6ac7c3943be9bcbef4
link
true
/test integ-pilot-k8s-tests_istio

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

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

Test name
Commit
Details
Required
Rerun command

integ-pilot-k8s-tests_istio
623fcab3d3deee8411bc5e6ac7c3943be9bcbef4
link
true
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
623fcab3d3deee8411bc5e6ac7c3943be9bcbef4
link
true
/test integ-security-k8s-tests_istio

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

/retest

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

Test name
Commit
Details
Required
Rerun command

integ-security-k8s-tests_istio
623fcab3d3deee8411bc5e6ac7c3943be9bcbef4
link
true
/test integ-security-k8s-tests_istio

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

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

Test name
Commit
Details
Required
Rerun command

integ-security-k8s-tests_istio
623fcab3d3deee8411bc5e6ac7c3943be9bcbef4
link
true
/test integ-security-k8s-tests_istio

integ-security-multicluster-tests_istio
623fcab3d3deee8411bc5e6ac7c3943be9bcbef4
link
true
/test integ-security-multicluster-tests_istio

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

@howardjohn can you PTAL when you get chance?


[AA-1075] Formatting and remove spurious comment

Thanks for this. I'd like to have a Tools team member perform and merge AA-1074 ahead of this, partly to ensure all post-2.0.0 builds get an appropriate semver number and partly to have them exercise what they learned during the recent presentation on the modernized Kotlin builds. They can then merge this as the first thing to go in after AA-1074.


Implement scitype for new temporal scalar types

https://github.com/alan-turing-institute/ScientificTypes.jl/pull/98

Codecov Report

Merging #21 into dev will decrease coverage by 3.34%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##              dev      #21      +/-   ##
==========================================
- Coverage   99.51%   96.17%   -3.35%
==========================================
  Files           7        7
  Lines         208      209       +1
==========================================
- Hits          207      201       -6
- Misses          1        8       +7

Impacted Files
Coverage Δ

src/MLJScientificTypes.jl
100.00% <ø> (ø)

src/convention/scitype.jl
81.48% <37.50%> (-18.52%)
:arrow_down:

src/schema.jl
96.96% <0.00%> (-3.04%)
:arrow_down:

src/autotype.jl
97.91% <0.00%> (-2.09%)
:arrow_down:

src/coerce.jl
97.61% <0.00%> (-0.06%)
:arrow_down:

src/convention/coerce.jl
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Request for diagrams on networking containers

File: engine/tutorials/networkingcontainers.md
This really needs some diagrams to illustrate how the networks inter-relate in the host. As it is, the page presumes an underlying comprehension of the terminology being used.

You can use graphics from https://success.docker.com/Architecture/Docker_Reference_Architecture%3A_Designing_Scalable%2C_Portable_Docker_Container_Networks :)

Hey! I want to work on this. Do I have to modify the diagram from the above link?

You can reuse them!

@Rubab-Syed Any progress? All you should need to do is just slot in the existing images!

@mstanleyjones this issue should be close with the merge of #2988

Points to @Rubab-Syed on this one. (Thanks @jmaitrehenry for the heads up to close)


End-to-End Encryption

[Enter feedback here]
The page referenced below uses the term end-to-end encryption, but what it describes is not end-to-end encryption in any way. The application gateway does not support TLS passthrough, which is a necessary feature that allows TLS traffic to flow through the gateway unmolested. Rather, this page describes a way to ensure that traffic is always encrypted in transit, which is a useful feature.
By calling this page "End to End Encryption", and referencing that phrase throughout the page, you are misleading engineers and wasting their time. I've just had a junior engineer try to stop the process by which I raise an exception to my client's standard architecture because he believes your misleading article solves my problem (it doesn't).
Please rewrite this page asap to represent what it truly offers, which is ensuring all traffic in transit is encrypted.

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

ID: 8b9fd744-85fa-0a72-4805-b92decdf05c5
Version Independent ID: 948878b1-6224-e4c5-e65a-3009c4feda74
Content: Configure end-to-end TLS encryption using the portal - Azure Application Gateway
Content Source: articles/application-gateway/end-to-end-ssl-portal.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@greg-lindsay
Could you Please review this and update as appropriate.

@BryanDollery
I'm going to assign this to the document author so they can take a look at it accordingly

Thanks for the comment. I will address this with the product team. This is internal work item #240846 #please-close


Credit Card Fraud Detection

💥 Proposal (GSSOC 23)
A machine learning algorithm to recognize fraudulent credit card transactions so that the customers of credit card companies are not charged for items that they did not purchase.
Main challenges involved in credit card fraud detection are:

Enormous Data is processed every day and the model build must be fast enough to respond to the scam in time.
Imbalanced Data i.e most of the transactions (99.8%) are not fraudulent which makes it really hard for detecting the fraudulent ones
Data availability as the data is mostly private.
Misclassified Data can be another major issue, as not every fraudulent transaction is caught and reported.
Adaptive techniques used against the model by the scammers.

I would like to work on this issue but can you elaborate more on what I need to do.

@sujanrupu - you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best!  Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.

@sujanrupu Please specify the ml model which you are going to train

@theyashwanthsai  may I start working on the issue?

@sujanrupu you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best!  Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.


Fix loading orkl sites on HTTPs domains.

I was unable to load the site I made, https://rangermauve.hashbase.io/ , wasn't loading posts when it wasn't served from dat://. I think I tracked the problem down to this line and it seems to have fixed the problem.

Hey, I don't think this fixes the problem, because if the new Archive() fails (which happens when you pass an https:// url to it) the content will be loaded from the /content.json file. Is the content.json missing or invalid?

kodedninja: I think the content.json was loaded. Could it be related to public: false?

You know what, that was probably it! When I changed the HTTPS thing it was loading the DatArchive and detecting that I had ownership of it, and thus displaying the post.
I totally missed the part about drafts 😂 Sorry!

Haha, no problem @RangerMauve! I'm glad you've found that out.

Really great project, by the way! It's exactly what I want from a blogging platform. 😄

Happy you like it! Feel free to PR if you found something ;)


Add internationalization support

Add internationalization (i18n) support to the project, so that we can translate user interface texts. Once the i18n package is configured, determine how to follow-up with user interface internationalization, including the following

menu texts
form texts

@brylie is issue #99 related to duplicate to this issue?
If related, I can reference it here.
If duplicate, I can close down the issue.

Jep, it is a duplicate. Sorry, I didn't search first.


a spelling mistake

Mistake here: http://cssreference.io/#flex-flow

https://developer.mozilla.org/en-US/docs/Web/CSS/flex-flow
The flex-flow CSS property is a shorthand property for flex-direction and flex-wrap individual

So this should be "Shorthand property for flex-direction and flex-wrap. "

Thanks!


Newsletter Timeline: Drop Date 07/20/2016

06/29/2016  - Wednesday - Deadline for content submission
07/05/2016 -Tuesday - Select topics and outline first draft
07/05/2016 - Tuesday  - Send initial draft to editors on medium, technical review
07/13/2016 - Wednesday - Feedback due from editors. Make graphic request to design team
07/15/2016 - Friday - Graphic Due
07/19/2016 - Tuesday - Run Test: Send letter, Final Review
07/20/2016  - Wednesday  - Publish
@ellisonbg @fperez @Carreau @willingc

Side question:
https://medium.com/@projectJupyter
Does not list any of the archived newsletter. Which IIRC was the point of doing that on Medium IIUC. Am I missing something.
There still a draft "DO NOT EDIT THIS VERSION" can it be deleted ?

We are archiving the newsletters in the 'Jupyter Newsletter' Medium account, which is separate from the 'Project Jupyter' account (and very confusing)
I'll delete the DO NOT EDIT version. Thanks for flagging that for me!

Ah... Yes, confusing.
we definitively need a link to that.
The template could probably have a "See past newsletter and subscribe to
new one at https://newsletter.jupyter.org/"
M
On Wed, Jun 22, 2016 at 10:02 AM, Katie White<EMAIL_ADDRESS>wrote:

We are archiving the newsletters in the 'Jupyter Newsletter' Medium
account, which is separate from the 'Project Jupyter' account (and very
confusing)
I'll delete the DO NOT EDIT version. Thanks for flagging that for me!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jupyter/newsletter/issues/39#issuecomment-227809950,
or mute the thread
https://github.com/notifications/unsubscribe/AAUez78P4FBxcLj9RF9gZ3JeT0P8PgV2ks5qOWq0gaJpZM4I7_Ay
.

FYI all, PDFs of the newsletters are being archived in this repo as well https://github.com/jupyter/newsletter/tree/master/archives

From standup today: @lbustelo, @parente to comment here with a bit about the enhancement proposals for dashboards and declarative widgets.

@katiewhite360 - note the content Gino & Peter from the post immediately above should be included in the next newsletter scheduled to drop on 7/20 since they are looking for feedback on the work they're doing - lmk if you have any questions since this was discussed in a call today. Thanks!

The jupyter-declarativewidget extension enables the user to create interactive areas of the Notebook that connect to functions and data on the kernel. It works on top of Python, Scala and R.
In conjunction with the jupyter-dashboard extension and related sub-projects, it is possible to turn Notebooks into full-featured dashboards and applications.

The jupyter-dashboards extension lets users arrange notebook outputs and widgets in grid- and report-like layouts. It is the first of three capabilities in the Jupyter Incubator Dashboards effort and a nice compliment to the jupyter-declarativewidgets.
The dashboard project developers recently proposed making the extension a top-level Jupyter project. Give it a try and provide your feedback on the incorporation proposal.

Thanks @parente and @lbustelo !
Could I get a header and an image for both of these, ideally by Friday?

Though if an image doesn't enhance the content or one doesn't exist that's fine too...

@katiewhite360 would the image in the readme in the dashboards extension repo linked above do? It's showcasing both the widgets and the dashboard layout.

That's perfect. A header?
Thanks!

Just a reminder for headers - could even be anything 5-10 words. @parente @lbustelo

@katiewhite360 Ah, sorry. I thought you meant header image.
How about one of these? You can choose or wordsmith.

A new way to consume notebooks with dashboards and declarative widgets
Cross language widgets and dashboard layouts for Jupyter notebooks
Incubating Jupyter declarative widgets and dashboard projects ready to graduate

Dashboard and declarative widgets projects graduating from Jupyter incubator
@katiewhite360 @parente What about this one?


add cluster finalizer

What type of PR is this?

[ ] bug
[x] feature
[ ] enhancement

What problem(s) does this PR solve?
Issue(s) number:
Description:
How do you solve it?
Special notes for your reviewer, ex. impact of this fix, design document, etc:

`
apiVersion: apps.nebula-graph.io/v1alpha1
kind: NebulaCluster
metadata:
creationTimestamp: "2023-12-12T14:14:26Z"
finalizers:

apps.nebula-graph.io/cluster-cleanup
`

`
$ time kubectl delete nc nebula
nebulacluster.apps.nebula-graph.io "nebula" deleted
real    0m3.210s
user    0m0.054s
sys     0m0.030s
`


LUCI installation issue on ubuntu 16.04

hi,
I'm trying to install LUCI application on ubuntu 16.04 by following this tutorial. install LUCI on Ubuntu.
when I run "make runuhttpd" I'm getting this error:
""""""""
root@toor-VirtualBox:/home/Openwrt/luci# svn info svn://svn.openwrt.org/openwrt/branches/backfire/package/uhttpd/src
svn: E170013: Unable to connect to a repository at URL 'svn://svn.openwrt.org/openwrt/branches/backfire/package/uhttpd/src'
svn: E670002: Unknown hostname 'svn.openwrt.org'
""""""""

According to this error it means server is not alive.
is there any way to figure it out.
Urgent request :)
Thanks in advance.

There is no svn server any more.
That external Chinese guide is likely outdated.
I close this issue, as installing LuCI separately from OpenWrt is not intended.


Ansible run command not found

Description
Understand the ansible was installed via pipx but if we directly invoke ansible in the DevOps pipeline task, it will fail with message "ansible command not found".
The only way to invoke the ansible is by put export PATH="$PATH:/opt/pipx_bin" ahead of the command.
Is this me or everyone must use ansible with self-hosted agent in this way? Thanks.
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
N.A.
Is it regression?
no
Expected behavior
N.A.
Actual behavior
N.A.
Repro steps
N.A.

Do we need to add python3 -m pipx ensurepath to ensure the pipx PATH is correctly setup on the Ubuntu OS?

Hi @xianlin - Thank you for bringing this issue to our attention. We will look into this issue and will update you shortly on this issue after investigating.

Hi @xianlin - The issue you're encountering, where the ansible command is not found, is due to the pipx installation directory not being included in the system PATH by default. When you install Ansible via pipx, it installs executables in a location (/opt/pipx_bin) that is not included in the PATH unless explicitly added.
You can try using below steps:-
A. Add the pipx Bin Directory to the PATH:
You can add the pipx bin directory to the PATH environment variable in your pipeline configuration. This ensures the ansible command is accessible.
export PATH="$PATH:/opt/pipx_bin"
ansible <your-ansible-command>

B. By Using pipx ensurepath to Automatically Set Up PATH:
You can run python3 -m pipx ensurepath to ensure that the pipx bin directory is added to the PATH environment variable. This command modifies your shell configuration to include the pipx bin directory in the PATH.
python3 -m pipx ensurepath
source ~/.bashrc  # or ~/.zshrc depending on your shell

So, Once this is done, the ansible command should be available without needing to manually export the PATH each time.
C. Set Up PATH in Your DevOps Pipeline Configuration:
If you are using a CI/CD pipeline, you can add a step to modify the PATH environment variable at the start of your pipeline script.
steps:
- script: |
    export PATH="$PATH:/opt/pipx_bin"
    ansible <your-ansible-command>
  displayName: 'Run Ansible Command'

By using above steps, it should get resolved.....thanks

@Prabhatkumar59 , thank you but why we cannot make the PATH work for Ansible in the first place? I saw there is a export PATH="$PATH:/opt/pipx_bin" in the install-pipx-packages.sh script and that is why the Test script invoked in the same installation script succeeded.
Can we add the above PATH to the final image by using the /scripts/helpers/etc-environment.sh file? wouldn't it be better? thanks.

Hi @xianlin- Thanks for your response.
You can modify the /scripts/helpers/etc-environment.sh file to include the necessary PATH modification. This way, the PATH is set up correctly for all sessions, including new shell sessions.
Here’s how to modify the /scripts/helpers/etc-environment.sh file to include the pipx binary path:
A. Steps to Modify /scripts/helpers/etc-environment.sh
Open the /scripts/helpers/etc-environment.sh File:
Edit the /scripts/helpers/etc-environment.sh file to include the pipx binary path.
B. Add the PATH Modification:
Add the following lines to ensure the pipx binary directory is included in the PATH:
# Ensure pipx bin path is added to PATH
PIPX_BIN_PATH="/opt/pipx_bin"
if [ -d "$PIPX_BIN_PATH" ]; then
  echo "Adding pipx bin path to /etc/environment"
  # Append to the PATH variable in /etc/environment
  echo "PATH=\"\$PATH:$PIPX_BIN_PATH\"" >> /etc/environment
  export PATH="$PATH:$PIPX_BIN_PATH"
else
  echo "pipx bin path does not exist: $PIPX_BIN_PATH"
fi

C. Build Custom Image: Use the modified script during the image build process to ensure the PATH is correctly set.
D. Verify in CI/CD Pipeline: Check the PATH and ansible installation in the CI/CD pipeline to confirm everything is set up correctly.
By using above steps, you can ensure that the pipx binary path is correctly added to the PATH environment variable, making ansible accessible by default in all sessions.

Hi @xianlin - As I did not get any update from you. so, I am considering your issue is resolved and we are closing this issue. Please feel free to reach out to us in case of any query..thanks


Fail

My command: python3 youtube-playlist-downloader.py
Error I got:
Traceback (most recent call last):
File "/home/humanbeing/Public/YTDownloader/youtube-audio-downloader/youtube-playlist-downloader.py", line 12, in
with open("api_key.txt", "r") as f:
FileNotFoundError: [Errno 2] No such file or directory: 'api_key.txt'

Hi, I just updated the readme to be significantly more complete, which you might find helpful. Your problem is due to the fact that you need to create a api_key.txt file yourself, and put your youtube api key inside of it. More details available in the readme, in the "setup" section. Hope this helps!

The API seems very in-depth and useful. I'll take it for a spin soon and let you know what I think. Thank you so much!

Whooo! It works!


Can no longer access DLRS

I tried to access our DLRS today and have run into a problem.
First, I hit this error message:

I follow the instructions and hit this error message:

Having spent the past 90 minutes digging around, it appears pretty common and seems to be linked to the remote site settings. To be on the safe side, I decided delete the existing remote site setting related to this and then to recreate a new one. As that had been listed as a solution previously. Unfortunately that hasn't worked.

We haven't changed our domain recently but we did renew our certificate. I don't know if that's causing the trouble?
Any help would be much appreciated.

I'm going to mark this as closed but if you're still having a problem I'd recommend reaching out to the Trailblazer Community Group for support.


doc: fix link to sample in replica clusters page

Fixes #476
Signed-off-by: Gabriele Bartolini<EMAIL_ADDRESS>

/ok-to-merge


Allow generic requirements that augment Self in Swift 3 compatibility mode

PR #5857 started rejecting generic requirements that adding
constraints directly to 'Self', which means the requirements would be
unsatisfiable by some models.
It turns out that, with assertions disabled, these protocols would be
accepted and could be used, so downgrade the error to a 'deprecated'
warning in Swift 3 compatibility mode.

@swift-ci please smoke test and merge

Wait, how are you supposed to express it?

@jrose-apple , SE-0142

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

Oh, of course. I missed that this was on the extension.


Druid doesn't cancel the query request explicitly

Affected Version
Druid Version :0.22.1
Description
In my usecase I am using Druid data on dasboard with UI cancel option, so that user can cancel the request explicitly and run the next query.
I was testing the way mentioned on the Druid doc  https://druid.apache.org/docs/latest/querying/sql-api.html#cancel-a-query
Test 1
Long running POST request:
curl --request POST 'https://router:8888/druid/v2/sql' \ --header 'Content-Type: application/json' \ --data-raw '{"query" : "SELECT COUNT(*) from MyDataSource", "context" : {"sqlQueryId" : "myQuery01"}}'
Cancel Request:
curl --request DELETE 'https://router:8888/druid/v2/sql/myQuery01'
I was expecting as soon I run the Cancel curl request then long running request should be interupted. But nothing happening.
Cancel request always return 404 http status, I was expecting 202 as per the doc.
Where as following endpoint always return 202 even the query is not running or queryid is not present in druid system as well, which is wrong too.
curl --request DELETE 'https://router:8888/druid/v2/myQuery01'
(Removed /sql from the URL)
Test 2
To be more precise I have added timeout 60 seconds also in the query context and run cancel/delete request many times before 60 seconds.
Long running POST request:
curl --request POST 'https://router:8888/druid/v2/sql' \ --header 'Content-Type: application/json' \ --data-raw '{"query" : "SELECT COUNT(*) from MyDataSource", "context" : {"sqlQueryId" : "myQuery01", "timeout": 60000}}}'
Finally long running request query timeout happend and I got:
{
"error": "Query timeout",
"errorMessage": "Query [cacb33d5-f575-44c0-8672-de364e827382] timed out!",
"errorClass": "org.apache.druid.query.QueryTimeoutException",
"host": "[<IP_ADDRESS>:8083](http://<IP_ADDRESS>:8083/)"
}

Still cancel request always returning 404.

@nisart007 : Thanks a lot for the investigation and reporting of the issue! Cancellation for SQL queries was added a part of 0.23.0 release (ref : https://github.com/apache/druid/issues/12510#0.23.0-new-features-sql-cancel-api-for-sql-queries).
Could you please try with latest Druid version 24.0 and see if that works for you?

@rohangarg Thanks for your help.
Yes, you are right!! It is working fine version in 24.0 for SQL Query and native Query.
Let me conclude Cancel API testing:
Native query
Long running POST request:
curl --request POST 'https://router:8888/druid/v2/sql' \ --header 'Content-Type: application/json' \   --data-raw '{"queryType":"timeseries","dataSource":{"type":"table","name":"myDataSource"},"intervals":{"type":"intervals","intervals":["2022-09-01T00:00:00.000Z/146140482-04-24T15:36:27.903Z"]},"granularity":{"type":"all"},"aggregations":[{"type":"longSum","name":"a0","fieldName":"sum_Views"}],"context":{"sqlOuterLimit":1001,"queryId":"nativeQueryId"}}'
Cancel Request:
curl --request DELETE 'https://router:8888/druid/v2/nativeQueryId'
Cancel request returns 202 Accepted and it was successfully cancelled.
SQL query (Support Druid version >=0.23.0 )
Long running POST request:
curl --request POST 'https://router:8888/druid/v2/sql' \ --header 'Content-Type: application/json' \ --data-raw '{"query" : "SELECT COUNT(*) from MyDataSource", "context" : {"sqlQueryId" : "sqlQueryId"}}'
Cancel Request:
curl --request DELETE 'https://router:8888/druid/v2/sql/sqlQueryId'
Cancel request returns 202 Accepted and it was successfully cancelled.
Both Test Long Running POST response after cancel:
{
"error": "Query cancelled",
"errorMessage": "Future.cancel() was called.",
"errorClass": "java.util.concurrent.CancellationException",
"host": "<IP_ADDRESS>:8083"
}

This is not an issue hence closing it.

SQL query cancel was added in the Druid version 0.23.0. Teste in version 24.0 for SQL Query and native Query.


grafana: 6.1.6 -> 6.2.0

Motivation for this change
Latest release with many bug fixes and quite a few improvements / new features.
Things done

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

[x] NixOS (x86_64-linux & aarch64-linux)
[ ] macOS
[ ] other Linux distributions

[x] 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 nix-review --run "nix-review wip"
[x] 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
[x] Fits CONTRIBUTING.md.

Even though this release includes a small breaking change (the prefix/suffix options were removed for the gauge panel) I'm in favor of having this backported to stable because of the bug fixes.
/cc @lheckemann @samueldr

@GrahamcOfBorg test grafana

https://github.com/grafana/grafana/blob/master/CHANGELOG.md
The most notable breaking changes are:

Gauge Panel: The suffix / prefix options have been removed from the new Gauge Panel (introduced in v6.0). #16870.
Plugins: Data source plugins that process hidden queries need to add a "hiddenQueries: true" attribute in plugin.json. #17124


Added comments feature

Thanks for this PR. We need to discuss how this feature could be added, while we focus on bringing MERN to the next few versions we'll take a look at it.


Various existential l-value fixes

This combines #9637 and #9852.

@swift-ci Please test.

@slavapestov Does this seem at all reasonable?

LGTM.


Lite app declares errors when it is asked to do something it doesn't do

I tried sending POLYX to another account
I tried registering a ticker - I'm seeing an unexpected call Index error in the wallet.
It should just tell us that that is not available in the lite app - try the xl app for this

maybe this is something our wallet can handle, but maybe it needs a more informative error to start with?

Above is specific to sending POLYX with a memo
sending without a memo works

Transfer with memo wasn't supported. It was included in the latest release and also the error message is more intuitive now 😄


GPC-NONE: Upgrade to Gradle 8.1.1

Draft because https://github.com/springdoc/springdoc-openapi-gradle-plugin/issues/107 - looking at a couple of workarounds


Callrequest fee validation

#4645

ok found the problem. was unwrapping a none value

Looks like will have to give some value in blob_versioned_hashes in tests . As None reverts

I modified the test expecting a particular error . Let me what what you think


IPSL-CM6A-LR-INCA: missing activity RFMIP

@durack1: This fortnight, there is a new issue of an unregistered activity for a source_id:
Activity RFMIP is missing in the registration of source_id=IPSL-CM6A-LR-INCA.
Thanks Martina

@glipsl it seems the above registration has also not been completed. Can you confirm that RFMIP should be added to the activity participation for IPSL-CM6A-LR-INCA it appears it is currently only registered for AerChemMIP

@glevava it seems the above registration has also not been completed. Can you confirm that RFMIP should be added to the activity participation for IPSL-CM6A-LR-INCA it appears it is currently only registered for AerChemMIP

Hi @durack1 yes, that's a mistake from us. RFMIP should be added to the activity participation of IPSL-CM6A-LR-INCA as some RFMIP experiments have been published already (ex. piClim-aer). That's for fixing this and sorry again : )

@glevava no problem, that change is now in place. Please take a peek at CMIP6_source_id.html and let me know if any further tweaks are required.
Apologies about the delay with this, the CA fires are miles from home, so we evacuated earlier this week, now catching back up with the backlog


Dev2

:bug: fixed typed views
:white_check_mark: added test for views.ViewHandler.get_available_mimetypes function

bypassing code review since @raspitim isn't here

Im Sorry :(


Implement library API for aggregate query

Resolves #1060.

@piotrszul What is the reason for locating the pathling.query.* classes in the fhirpath module? Are they more general than the library API?

I was able to load data from an existing Delta warehouse using code like this:
from delta import configure_spark_with_delta_pip
from pathling import PathlingContext, DataSources, Expression as exp, find_jar
from pyspark.sql import SparkSession

builder = (
    SparkSession.builder
    .config("spark.jars", find_jar())
    .config("spark.executor.userClassPathFirst", "true")
    .config("spark.sql.extensions", "io.delta.sql.DeltaSparkSessionExtension")
    .config("spark.sql.catalog.spark_catalog", "org.apache.spark.sql.delta.catalog.DeltaCatalog")
)

spark = configure_spark_with_delta_pip(builder).getOrCreate()

pc = PathlingContext.create(spark)

mimic_warehouse = DataSources(pc).from_warehouse(
    "/Users/gri306/Library/CloudStorage/OneDrive-CSIRO/Data/mimic-fhir", database_name="warehouse")

query = mimic_warehouse.aggregate("Observation",
                                  aggregations=[exp("count()", "Number of observations")],
                                  groupings=[exp("code.coding.system", "Code system")])

query.show(truncate=False)

@piotrszul What is the reason for locating the pathling.query.* classes in the fhirpath module? Are they more general than the library API?

Yes, they are more general. They are also being used by query executors in fhir-server.  Maybe the package name could be changed to something like 'data' or 'datasource'

I was able to load data from an existing Delta warehouse using code like this:
from delta import configure_spark_with_delta_pip
from pathling import PathlingContext, DataSources, Expression as exp, find_jar
from pyspark.sql import SparkSession

                                  groupings=[exp("code.coding.system", "Code system")])

query.show(truncate=False)

Yes, it does run locally (I am guessing it was locally) without problems on the delta data imported with fhir server.
Another way to run this locally (get the PathilngContext) would be:
pc = PathlingContext.create(enable_delta=True)

mimic_warehouse = DataSources(pc).from_warehouse(
    "/Users/gri306/Library/CloudStorage/OneDrive-CSIRO/Data/mimic-fhir", database_name="warehouse")
...

I have been experiencing some strange issues however, when I was trying to do the same on DataBricks (with a copy of the delta encoder files that worked locally saved to dbfs).

Why does DatabaseComponent implement Cacheable, rather than Database?

Or maybe I should change my question to: what drove the need to split Database into two classes?

Or maybe I should change my question to: what drove the need to split Database into two classes?

Decoupling from Spring and other aspects that seem to be specific to the Fhir server needs (such as caching) so that the basic functionality of a delta warehohouse can be implemented in 'fhirpath' and then used in library-api/python api.

Another thought: we currently constrain the types of values that can be returned in an aggregate or extract query, based upon what is possible or desirable to return within a FHIR response.
Is this still appropriate in the context of running this from the library? Maybe we could relax these requirements within this context?


Add Travis build

Should include travis badge, release on tag, and publish to hackage.
May need to consult other projects on how to do this.

From this article it may be wise to turn on --ghc-options=-Werror when building inside Travis.

Travis config exists but it doesn't look like Haddock docs are making it up to Hackage...? Also haven't incorporated --ghc-options=-Werror.
Keeping this open until those issues are resolved.

It looks like Haddock gets built automatically, no need to do anything else.


[core] Change axisCount to be an array

Currently there is 1 axisCount variable for all gamepads and it is rewritten each time a gamepad is initialized (or polled for WASM).
This change switches axisCount to an array, makes GetGamepadAxisCount use the passed in gamepad index, and updates the logic that sets the gamepad axisCount in each platform to reference an array element.
There is also some updates to the core_input_gamepad_info example included with this to display more useful information.

And there are tons of commits from me merging in upstream changes, etc.  The actual changes this PR represents are relatively small and simple.

@michaelfiber Nice! Thanks for the update! :D


Scope to other subscriptions

Based on https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/deploy-to-subscription?tabs=azure-cli#scope-to-other-subscription how can we express scope to other subscription within bicep, namely:
{
    "$schema": "https://schema.management.azure.com/schemas/2018-05-01/subscriptionDeploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "resources": [
        {
            "type": "Microsoft.Resources/deployments",
            "apiVersion": "2020-06-01",
            "name": "nestedDeployment",
            "subscriptionId": "00000000-0000-0000-0000-000000000000",
            "location": "westus",
            "properties": {
                "mode": "Incremental",
                "template": {
                    subscription-resources
                }
            }
        }
    ],
    "outputs": {}
}

None of these work currently:
targetScope = 'subscription'
module my_rg './resourcegroup.template.bicep' = {
    name: 'rg'
    scope: resourceGroup('12345678-1234-1234-1234-123456789012', 'resGrp')

Unsupported scope for module deployment in a "subscription" target scope. Omit this property to inherit the current scope, or specify a valid scope. Permissible scopes include current subscription: subscription(), named resource group in same subscription: resourceGroup(), or tenant: tenant().

module my_rg './resourcegroup.template.bicep' = {
    name: 'rg'
    scope: subscription('12345678-1234-1234-1234-123456789012')

Unsupported scope for module deployment in a "resourceGroup" target scope. Omit this property to inherit the current scope, or specify a valid scope. Permissible scopes include current resource group: resourceGroup(), named resource group in same subscription: resourceGroup(), named resource group in a different subscription: resourceGroup(, ), or tenant: tenant().

@anthony-c-martin - according to the resource scopes spec, it looks like the above examples should work. Have we not getting to these uses of the subscription() (and managementGroup()) function yet?

The example from the ARM template docs above does not pass ARM template VSCode plugin validation:

Template validation failed: The template resource 'nestedDeployment' at line '5' and column '9' is invalid. 'SubscriptionId' property is not supported for nested deployments inside a deployment at subscription scope. Please see https://aka.ms/arm-template/#resources for usage details.

It seems to me like your trying to do a deployment to subscription A and within that deployment do a subdeployment to subscription B. Is this even possible in ARM?

Coincidentally, we have just relaxed the cross-scope deployment limitations. You should now be able to deploy from one scope to any other scope as long as you have the right permissions. So subscription A -> subscription B should now be possible in ARM templates. We will light this up in Bicep before we release v0.3

@alex-frankel a side question - where do you post updates on ARM templates functionalities such as mentioned relaxation of cross-scope deployments? https://azure.microsoft.com/en-us/updates/ or is there another place? Sometimes is hard to follow what's now available in each part of Azure and such things might be very useful in daily developer/devops work.

Good question - tbh, this is something we need to get better at. Azure Updates is probably the right channel, we just need to get more systematic about posting there. I'll take a note to discuss this internally. There are some other things that we need to a similar process around, so we will try to address these together.

Btw @stan-sz, I didn't realize in ARM today you also cannot go from sub-scoped deployment -> rg in a different sub. You can go from rg in sub A -> different rg in sub B, and the resourceGroup() function works accordingly. So this code works today:
targetScope = 'resourceGroup'

module mod './module.bicep' = {
  name: 'test'
  scope: resourceGroup('<SUB-GUID-GOES-HERE>', '<RG-NAME-GOES-HERE')
}


Update Users List

Per feedback from @fabiangmpf
https://github.com/fabiangmpf/substrate-adoption

Question:
Should I also include the "Tooling and Infrastructure" people?
https://github.com/fabiangmpf/substrate-adoption#Tooling-and-Infrastructure

Should I also include the "Tooling and Infrastructure" people?

Yeh, I'd say we include the tooling & infrastructure stuff.


cmd/pprof: unescaped labels cause dot output to be malformed

What version of Go are you using (go version)?

$ go version
1.18.3

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/snaipe/.local/var/cache/go-build"
GOENV="/home/snaipe/.local/etc/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/snaipe/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/snaipe/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18.3"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build4178454973=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Label values don't seem to be escaped in the dot output, causing the result to be malformed. For example, given this test program: https://go.dev/play/p/duT2NS2iD7I
Running the program prints the profile base64-encoded to be demonstrable on the go playground. Using go tool pprof on the decoded profile shows the following (emphasis mine):
File: play
Type: cpu
Time: Nov 11, 2009 at 12:00am (CET)
Duration: 200ms, Total samples = 350ms (175.00%)
Entering interactive mode (type "help" for commands, "o" for options)
(pprof) tags
 unescaped: Total 350.0ms
            350.0ms (  100%): some "string" with quotes

(pprof) svg
Error: <stdin>: syntax error in line 5 near '"'
failed to execute dot. Is Graphviz installed? Error: exit status 1
(pprof) dot
digraph "play" {
node [style=filled fillcolor="#f8f8f8"]
subgraph cluster_L { "File: play" [shape=box fontsize=16 label="File: play\lType: cpu\lTime: Nov 11, 2009 at 12:00am (CET)\lDuration: 200ms, Total samples = 350ms (175.00%)\lShowing nodes accounting for 350ms, 100% of 350ms total\l\lSee https://git.io/JfYMW for how to read the graph\l" tooltip="play"] }
N1 [label="sha256\nblock\n270ms (77.14%)" id="node1" fontsize=24 shape=box tooltip="crypto/sha256.block (270ms)" color="#b20d00" fillcolor="#edd7d5"]
N1_0 [label = "unescaped:some "string" with quotes" id="N1_0" fontsize=8 shape=box3d tooltip="270ms"]

<snip>

As you can see, the label = "unescaped:some "string" with quotes" does not parse because "string" terminates the label string. The label should have been label = "unescaped:some \"string\" with quotes" instead.
I confirmed that the correction works by running sed 's/"string"/\"string\"/g' on the generated profile.dot.
See attached profile.zip for the profile file used in this session, as well as expected dot and expected resulting svg.

cc @golang/runtime

Thanks for the report.
This certainly needs to be fixed, but I think it should be fixed upstream in https://github.com/google/pprof (which we pull back into go tool pprof). pprof labels are just arbitrary strings, I don't think it makes sense to modify them with "escaping" on the Go side (plus how do we know what form of escaping is correct for all downstream usage?).
In https://github.com/google/pprof/blob/c488b8fa1db3fa467bf30beb5a1d6f4f10bb1b87/internal/graph/dotgraph.go#L250, I see escaping (escapeForDot) on a variety of values, but not on the labels. This bug could affect dot/svg generation in pprof from profiles for sources other than Go as well.
@aalexand @evanj

Sounds good, please open an issue on https://github.com/google/pprof. We pull the latest pprof every release, so a fix will be included for go tool pprof in a subsequent Go release.

My PR https://github.com/google/pprof/pull/689 should fix this bug. It is waiting on review.

I've just hit the same issue, I was wondering if we could get any progress on this, and whether some help is needed.

The https://github.com/google/pprof/pull/689 that @evanj mentioned got stalled for the lack of attention, I think it was my turn. I'll comment there, but not sure if Evan is still willing to finish that work after this long of a context switch.

I left some comments on that PR.


Update outdated Git tag in example project READMEs

Fixed

Update READMEs of example projects to reference v0.8.3 instead of outdated versions. This text should have been updated back when 0.8.3 was released, but it seems it slipped through.

Sorry for the second rubber-stamping, guys. This should be another very small and uncontroversial change, so I appreciate the quick review, @antigravityla.


Update dockerizing.md

Proposed changes
The sentence in which the change is made missed the word "to".
Unreleased project version (optional)

Related issues (optinoal)

Just added "to" in the sentence which may have been missed earlier.

Thanks @athend !


Update documentation

Update documentation and Sphinx to latest version.

Added a PR (#29) to fix this. I will reference this is issue in it as well.


[Support] vpn question or support idk

[yes] I have read the documentation
[yes] I have read and understood the contributors guide.
[yes] The issue I am reporting can be replicated.
[yes] The issue I am reporting can be is directly related to the pivpn installer script.

i wanted to know that i cant connect to my local pcs that are not on the vpn is there a way to let me see them and use them while i have the vpn on without the computer not running the vpn
wiregurd
raspberry pi 4gb

Sorry . dont understand your question. you are connecting to a vpn and want to see the computers in the internal network of the server without having to connect them to the vpn?

Yes sorry about what I said I had no time typeing it

are you there?

?

A VPN can do what you want but pivpn does not do it.

i thought you can do it locally not the server

but on my ipad it lets me

If you feel confident then read the Openvpn howto

But WireGuard is safer right?


Firefox Error: Argument 2 of URL.createObjectURL can't be converted to a dictionary.

Error in Firefox (50.0):

Temporary fix discussed on the Blob PR here https://github.com/jmaister/excellentexport/pull/48#issuecomment-255954805

The Firefox error persists in the 2.0.0 version (downloaded today) for Firefox 49.0.1 when using the CSV export.
A temporary fix involves replacing "exporttype" on line 105 with
{ type : exporttype}

Same fix will work for XLS

I had the same problem today, with the bower download of version 2.0.0.
The fix mentionned by @gspringtech in @craignicol comment is working.
What about integrating that in the trunk?
Thanks


lines change after save

before save  ;
after save
<
div > < /div>;
lines change and spaces added in code any solution please help in vscode im facing this problem

WP29 RTBF 2014-07-31
https://drive.google.com/file/d/0B8syaai6SSfiT0EwRUFyOENqR3M/view?usp=drivesdk


Detect Oracle Maxymiser

Example site - https://www.matalan.co.uk/
Vendor site - https://www.oracle.com/uk/cx/marketing/personalization-testing/
Categories to be used - Personalisation and A/B Testing
Detection - look for https://service.maxymiser.net/api/eu/new.matalan.co.uk/1192b7/mmapi.js
JS object - I also see object called 'maxy' on this site but not 100% sure

Hey there , added


[Feature] Main features

Describe the solution you'd like
The main features of this app is basically a UI to quickly create a github Pr search. The main configuration options are:

base-filters
repos
saved searches
current URL.

These should be split into 4 mains parts of the UI.

top-bar = actual URL, its ugly/long so it can be isolated at the top.
repos = list of entry-fields for github-repos. Validation + drag-drop re-ordering

this should take up most of the UI in the middle, under the bar.

base-filters = list of common filters that can be added/removed.

these can be shown to the left, and take up only some of the UI.

custom text = shown on the bottom of the main-repos fields. These should be just a plain-text field.

May need to add a new component/part for "base-url", or "enterprise" options to support a prefix for a given github.
Also need a settings/memory/saved component for other filters.

drag and drop:
https://www.npmjs.com/package/react-dnd


[Hardware] [Intel] Enable Multiprocessing and tensor parallel in CPU backend and update documentation

This PR enabled vLLM multiprocessing in CPU backend for improving async LLM engine performance and supporting TP.
The main changes include:

Use utilities from vllm.executor.multiproc_worker_utils to manage workers in CPUExecutor.
Enable tensor parallel for the CPU bcakend.
Provide a new env VLLM_CPU_OMP_THREADS_BIND to simplify the OpenMP thread affinity setting, especially for multiple TP workers.
Transparently separate api_server process and async LLM engine process to avoid the GIL affect.
Update related documents.
Enable a E2E online serving test in CI.

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

/ready

Hi @WoosukKwon, I think this PR is ready, would you please help to review it if you have time? Thanks!

@bigPYJ1151 Hi, do you have any plan support pipeline parallel in CPU backend?

@lhtin Not yet. Looks like PP has no significant benefit for CPU device.

@bigPYJ1151 Sorry for the delay. I've been busy with some other tasks. I will review the PR in ~48 hours. Does this sound ok to you?

@WoosukKwon No problem, just review it when you are available, thanks! :)

Hi @WoosukKwon , thanks for your comments! I have updated them, please check, thanks!

@bigPYJ1151 The PR looks good to me overall. Can you fix the failed CI test? Thanks!

Hi @WoosukKwon, thanks for your feedback! The CPU CI is fixed. The CUDA CI should pass in general but some cases may fail due to random connection errors.

Hi @bigPYJ1151 , the CPU target fails to build on my machine.
Could you please take a look at this https://github.com/vllm-project/vllm/pull/6931 ?
Thanks.


timed out waiting for dataset to mount

I am getting this error message when trying to run the example in rhel7.2:
docker run -v apples:/data --volume-driver flocker busybox sh -c "echo hello > /data/file.txt"
docker: Error response from daemon: Timed out waiting for dataset to mount..
I looked at journalctl to look at the logs,  and nothing seemed to stand out.   No error messages.
What should I be looking for?   please advice.  thanks.

Hey @beth2016
You need to check the journalctl logs for all the Flocker processes and the docker process.
Somewhere, there'll be an error message.

Docker will be connecting to the flocker-docker-plugin
flocker-docker-plugin will be connecting to flocker-control
flocker-docker-plugin will be polling the Flocker cluster state from flocker-control
flocker-dataset-agent on the server where you ran docker run ...  should have received a new Flocker cluster configuration and should be attempting create, attach, format and mount a new volume.

See https://flocker-docs.clusterhq.com/en/latest/administering/debugging.html
-RichardW.

Yeah, i did yesterday.   there is no error message.
I followed that same link yesterday ( i see no err messages)

I'm pretty sure there'll be an "error" / "exception" / "failure" message somewhere in the logs.
Or, perhaps the Docker daemon hasn't been able to connect to the flocker-docker-plugin.
Is that process running?
Perhaps the flocker-docker-plugin can't connect to flocker-control.
Is flocker-control running and listening on a TCP port?
Perhaps the flocker-dataset-agent isn't running or isn't connected to the flocker-control service.
Is there  a flocker-dataset-agent process running on the server?
Can you see an established connection to port 4454 on the server where flocker-control is running?
Check the list of Flocker volumes reported by the Flocker REST API using curl or using flockerctl.
Try creating a volume through the REST API or using flockerctl.
If you're working on AWS I'd also look at the list of EBS block devices and or check the cloudtrail logs to see what AWS API operations have been performed.
There are lots of moving parts and you have to check that all the components are running and connected.

I am running into the same issue:
Have looked through the logs and cannot find anything.
Using Ubuntu 16.04  with one control node and two workdr nodes - followed  the instructions for  manually installing flocker with docker swarm
flockerctl list shows the volume as detached as was able to create another volume using  flockerctl
d031b929-0add-4c9f-a600-cea2a9491b46   75.00G   name=apples                detached       e9870d25 (192.x.x.x)


Mono: Image.Load() doesn't recognize res:// outside of editor

Godot version:
3.0.6
OS/device including version:
Windows 10
Issue description:
Image.Load won't accepted res:// path's in exported projects, but works fine in editor. I was told image.load() wasn't meant for res:// paths (which doesn't make logical sense), so I'm assuming this method is broken or doesn't explicitly state its correct usage in the method name.
Steps to reproduce:

Create an asset
Try loading it using

                var image = new Image();
                image.Load($"res://pathtoasset");

Create an image texture

                var imageTexture = new ImageTexture();
                imageTexture.CreateFromImage(image, (int) Texture.FlagsEnum.Mipmaps);

Add it to the scene
Run scene to make sure it outputs
Export project and watch the image not print to the screen because it can't find the .png path

Minimal reproduction project:
Will create one after ludum dare 42

@exts shouldn't you be using an absolute path, not a resource path?

@FeralBytes yes, the above is a placeholder. the original code was something along the lines with:
var img = new Image();
img.Load($"{assetPath}/{food.Value}");

where $"{assetPath}/{food.Value}" looked something like: res://Assets/Art/Food/Meats/Chicken.png

@exts just to make sure we are on the same page. I am saying the path should not be res://Assets/Art/Food/Meats/Chicken.png but rather it should be an absolute path like /home/user/.godot/myGame/Downloaded/Assets/Art/Food/meats/Chicken.png. The path could be to any wheres but since you are using things outside of the executable then your path should not be res:// If the path is interal to the executable then the path should work fine, but you will have to handle 2 conditions when running in the editor use the standard res://Assets/Art/Food/Meats/Chicken.png but once exported to executable if you path is dynamically generated realize that the code will be only seeing res://Assets/Art/Food/Meats/Chicken.png.import.

that makes absolutely no sense lol why would you use absolute paths with an api library that targets multiple os's?
If I pass "res://" to any load method it should grab my local resources from the project root. Else the method needs to be renamed to something else

Your part about that making no sense is you mis understanding what I meant. as I state if you read the entire post, if the path is a resource within your project then res:// is fine but outside of your project you will need to use absolute paths. It seemed to me from what you wrote that you were actually trying to load an external file.
I think you need to read Data Paths.
Yes exactly it will grab your local resources from the project root. So lets do this as a test you try and load an asset using the path you provided res://Assets/Art/Food/Meats/Chicken.png, not dynamic and tell me if it works.
For reference the method is Image.load()

Your part about that making no sense is you mis understanding what I meant. as I state if you read the entire post, if the path is a resource within your project then res:// is fine but outside of your project you will need to use absolute paths. It seemed to me from what you wrote that you were actually trying to load an external file.

Not the case. I was only loading resources from within the project assets that were exported which have .stex files. Using Image.Load() does not load those assets, only can I load those assets using GD.Load()
Here you can see the code here https://github.com/exts/godotsharp-shortserve/blob/master/App/Food/FoodLoader.cs#L81 this is the updated code.
The original method was:
private static void CreateTextures(string type, Godot.Dictionary<string, string> foods, ICollection<FoodItem> foodItemTexturess, string assetPath)
{
    foreach(var food in foods)
    {
        var image = new Image();
        image.Load($"{assetPath}/{food.Value}");

        var imageTexture = new ImageTexture();
        imageTexture.CreateFromImage(image, (int) Texture.FlagsEnum.Mipmaps);

        foodItemTexturess.Add(new FoodItem(food.Key, type, imageTexture));
    }
}

It seemed to me from what you wrote that you were actually trying to load an external file.
You can try compiling that code and tell me why it shouldn't work with the original method because I'm not looking for an external file.

To clarify a bit, Image.Load() seems to expect a path to a physical file: it works when the file is present while you're running the project from the editor, but once you're into exported virtual filesystem .pck territory, Image.Load doesn't get redirected to the imported .stex files (the .png doesn't exist in the .pck at that point).
In contrast to that, GD.Load() explicitly tries to load a Resource and as such probably respects the virtual filesystem (ie it loads the .stex when you tell it to load the corresponding .png).
Absolute or relative paths are irrelevant to the problem.

Use GD.Load and import the file as an image. Should probably unbind that
and other functions, as it confuses users.
On Mon, Aug 13, 2018, 18:32 PetePete1984<EMAIL_ADDRESS>wrote:

To clarify a bit, Image.Load() seems to expect a path to a physical file:
it works when the file is present while you're running the project from the
editor, but once you're into exported virtual filesystem .pck territory,
Image.Load doesn't get redirected to the imported .stex files (the .png
doesn't exist in the .pck at that point).
In contrast to that, GD.Load() explicitly tries to load a Resource and as
such probably respects the virtual filesystem (ie it loads the .stex when
you tell it to load the corresponding .png).
Absolute or relative paths are irrelevant to the problem.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/20972#issuecomment-412672151,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2zzlEcLrqfJ6cks1rCyNSdR-4Nxrks5uQfB-gaJpZM4V6j6p
.

@reduz yeah that was the solution to fix my problem. Almost didn't make the ldjam when I ran into the issue 😄. Definitely confused me when I found out.

Update: Seems to also be the case for ImageTexture.load()

Please don't remove Image.load, or you will piss off everyone loading non-resource images dynamically in their games :/

or better yet just have proper support for res:// and user:// in exported games because currently it doesn't seem to work @Zylann

@exts does this work?
var f = File.new()
f.open("res://something.x", File.READ)

@Zylann yes that definitely works https://github.com/exts/godotsharp-shooter/blob/master/Application/Core/Formations.cs#L90
I've done it in a tutorial game a while back. Can test it via https://g4mr.itch.io/godotsharp-shooter

Please don't remove Image.load, or you will piss off everyone loading non-resource images dynamically in their games and tools :/

@Zylann yep, I really depend on this as it's one of the core features currently used in my game. I extract contours from images and create collision shapes from them dynamically.
As I understand, loading a texture by path will return imported one (StreamTexture). What if it can be made that if the loader doesn't find imported texture by path, it should fallback to loading other compatible resource types? In this case it would be Image.
var img = load("my_image.png") # returns `StreamTexture` if found, else it's an `Image`

Can anyone still reproduce this bug in Godot 3.2.1 or any later release (e.g. 3.2.2-rc1)?
If yes, please ensure that an up-to-date Minimal Reproduction Project (MRP) is included in this report (a MRP is a zipped Godot project with the minimal elements necessary to reliably trigger the bug). You can upload ZIP files in an issue comment with a drag and drop.

@KoBeWi let me try reproducing it today.

@KoBeWi it seems like this is only a limitation when you try to image.load in exported games, which doesn't work, and should likely just be documented there.
The "opposite" use case is here though: #39396 (the warning should be moved to documentation IMO).

Tagging as documentation then.

Looks like this is intended then.

I wouldn't say this is intended, but that's just a limitation that you have to be aware of. You could optionally try to include all image files while exporting, I would be actually interested if this works:

But instead, if you need to load an image specifically, you just need to reimport the texture as an Image resource:

If you want to distribute some images along with exported project you can also try globalizing the res:// path before doing so, so that the engine won't look into the virtual filesystem, but instead search for OS filesystem.
static func image_load(filepath):
    var image = Image.new()
    image.load(ProjectSettings.globalize_path(filepath))
    return image


[spill](regression test) check spill temporary files are deleted after all cases are done

Proposed changes
Issue Number: close #xxx

run buildall

run p0


Ouathlib raises exception for insecure transport on HTTPS

I managed to set up a Quart instance with hypercorn on DigitalOcean's PaaS (App Platform) but for whatever reason, oauthlib refuses any and all callbacks due to "Insecure transport", which is interesting as DO provides SSL and HTTPS out of the box. Do I need additional configuration for hypercorn, this plugin, or anything else?
Thanks for your time.

Solution: Quart does not seem to support reverse-proxies properly, and fools oauthlib into thinking that the request came through http, triggering the error. So if you're like me, and use Quart with a reverse-proxy, enable insecure transport.

getting the same error here, website is proxied with Cloudflare..

Solution: Quart does not seem to support reverse-proxies properly, and fools oauthlib into thinking that the request came through http, triggering the error. So if you're like me, and use Quart with a reverse-proxy, enable insecure transport.

How can it possibly trust headers in the request promising the request was secure?

i actually fixed this just by changing my redirect_uri inside the discord developer portal from https over to http.


All cryptocurrency

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[x] The relevant Azure service or technology.
[x] A link to the published documentation article that you have feedback about.
[x] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[x] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[x] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/d365community/forum/79b1327d-d925-ec11-b6e6-000d3a4f06a4.

@NesredinJemal Can you please share the Azure document link and elaborate what the feedback is?

@NesredinJemal Can you please share the Azure document link and elaborate what the feedback is?

As we didn't hear back from you so closing this issue for now. Let me know if you have any further questions with regards to this topic so we can continue our discussion here. Thanks.


I am getting this message: HMAC signature cannot be verified

I have a client in perl which sends request to a kong proxy with HMAC header.
Request has send like this:
Authorization: hmac username= "f79f1ab2-9520-4e7b-948e-ee7e8bd9a060", algorithm="hmac-sha1", headers="X-Date", signature="t2BOexU7x9c8oLqmZd4Q/mO4KoU="
X-Date: Fri, 10 Jun 2016 06:17:31 GMT
String to Sign created as :
date: Fri, 10 Jun 2016 06:17:31 GMT
Also as per your previous suggestion I tested request by setting the clock skew to very high like 86400. But still I getting the same error.
Please help me to find the resolution.

I would need more details to be able to help you.
What error you getting?
How you calculating signature?
here is another ticket related to HMAC, and users are able to use it successfully.
https://github.com/Mashape/kong/issues/599

Thanks Shashi for quick reply.
We are trying to integrate kong with another tool where we store all consumers information. So in our case username is the unique id.
Url: consumers/AdminTest/hmac-auth
Response:
{
"data": [
{
"secret": "PnMFnPd4BmW9TvnX",
"id": "9703b9f5-7cac-4cab-89ea-2491bcb95225",
"username": "f79f1ab2-9520-4e7b-948e-ee7e8bd9a060",
"created_at":<PHONE_NUMBER>000,
"consumer_id": "eb031583-fdbd-42ab-a440-2dd5ac4593ba"
}
],
"total": 1
}
So when I am sending the HMAC request with headers:
Authorization: "hmac username= "f79f1ab2-9520-4e7b-948e-ee7e8bd9a060", algorithm="hmac-sha1", headers="X-Date", signature="+5ZUljHINesjnNYUzGtct9KVEho=""
X-Date="Fri, 10 Jun 2016 18:14:49 GMT"
I am getting error 403 Fordidden
{
"message": "HMAC signature cannot be verified"
}
The script which is creating the header is in perl
sub createHMAC {
$stringToSign = "";
$username = $[0];
$secret = $[1];
$date = createDateString();
$stringToSign ="date: " . $date ;
$signaturestr = sha1_base64(chomp($stringToSign), chomp($secret));
   # Fix padding of Base64 signature
while (length($signaturestr) % 4) {
	$signaturestr .= '=';
}
$authHeader = "hmac username= ". "\"$username\", " . "algorithm=" . "\"hmac-sha1\", " .      "headers=\"X-Date\", " . "signature=" . "\"$signaturestr\"";

my %headers= ('Authorization' => $authHeader,'X-Date' => $date);
    return %headers;

}
Below all the details:
Username : f79f1ab2-9520-4e7b-948e-ee7e8bd9a060
Secret : PnMFnPd4BmW9TvnX
Date and Time : Fri, 10 Jun 2016 18:14:49 GMT
String to sign: date: Fri, 10 Jun 2016 18:14:49 GMT
Signature : +5ZUljHINesjnNYUzGtct9KVEho=
Also the question is the "String to sign" will be date: Fri, 10 Jun 2016 18:14:49 GMT or only Fri, 10 Jun 2016 18:14:49 GMT (excluding "date:")?

$date = createDateString(); $stringToSign ="date: " . $date ; $signaturestr = sha1_base64(chomp($stringToSign), chomp($secret));
why you creating new date? You supposed to use the x-date.

I am seding this request using Postman. I am not sure how to populate X-Date value in perl. I dont see and library in perl which supports that.
Any otherway I can pass date as a stringToSign?

So instead of headers="X-Date" , Can I use headers="Date" and in request header I can add Date instead of X-Date.
Eq
Date: "Fri, 10 Jun 2016 18:14:49 GMT"

Sorry actually date seems to be right,
$stringToSign ="date: " . $date ;  should be
$stringToSign ="x-date: " . $date ;

I changed date to x-date but still getting same error.

I am not perl expert and its difficult to debug like this. You can refer our tests for working example.
https://github.com/Mashape/kong/blob/master/spec/plugins/hmac-auth/access_spec.lua
also what  sha1_base64 funtion does?

Please debug your code to check correct signature has been generated. for following signature string
x-date: Fri, 10 Jun 2016 18:14:49 GMT base64 of hashed signature would be Vxz0P8ROwo4ak8uVvDJzUOurKSc=

The access_spec.lua along with the Cavage spec (https://tools.ietf.org/html/draft-cavage-http-signatures-01) is a great resource when trying to put together the proper hmac signature.
For what its worth, here is some node.js JS code that works:
`
var crypto = require("crypto");
exports.headers = function(apikey, username, secret, host, path, verb) {
var date = new Date().toUTCString();
var requestLine = verb + ' ' + path + ' HTTP/1.1';
var stringToSign = 'date: ' + date.trim() + '\n'
+ 'host: ' + host.trim() + '\n'
+ requestLine;
var encodedSignature = crypto.createHmac("sha1", secret).update(stringToSign).digest("base64");
var hmacAuth = 'hmac username="' + username + '",algorithm="hmac-sha1",headers="date host request-line",signature="' + encodedSignature + '"';
var headers = {
'apikey': apikey,
'date': date,
'Authorization': hmacAuth
}
return headers;
};
`

I am getting same issue while authenticating an API from my java application.
Date curDate = new Date();
SimpleDateFormat format = new SimpleDateFormat("E, dd MMM yyyy HH:mm:ss z");
format.setTimeZone(TimeZone.getTimeZone("GMT"));
String dateToStr = format.format(curDate);

String authorizeString = org.apache.commons.codec.binary.Base64.encodeBase64String(HmacUtils.hmacSha1("secret_123", "date: "+dateToStr));

Header oauthHeader = null;
Header dateHeader = null;
   authHeader = new BasicHeader("Authorization", "username=\"abc\", algorithm=\"hmac-sha1\", headers=\"date \", signature=\""+authorizeString +"\"" );
        dateHeader = new BasicHeader("date", dateToStr);

String serviceURL = "serviceURI";
HttpClient httpClient = HttpClientBuilder.create().build();
HttpGet httpGet = new HttpGet(serviceURL);
httpGet.addHeader(authHeader );
httpGet.addHeader(dateHeader);

HttpResponse response = httpClient.execute(httpGet);
int statusCode = response.getStatusLine().getStatusCode();

String to sign:
date: Thu, 06 Oct 2016 14:51:50 GMT
secret: "secret_123"
Auth String: //IZHcJjfS3nq448u4VQtSVrxos=
It returns 403:
{
"message": "HMAC signature cannot be verified"
}
Any help in this regard would is appreciated.

authHeader seems to be set wrong, its missing hmac in the beginning. Also make sure client and kong server clock in sync or debug with very high clock skew.

Thanks @shashiranjan84  appreciate your quick response.


Drop fs-extra as dependency

Current NodeJS versions have support for fs.promises. In this case, fs-extra is not necessary. See also #638 for drop support for NodeJS 8.

Anyone have an idea why they don't pass the Node 10 tests?

I agree with this change in principle, but I can't merge this if the tests don't pass. Looking at the failed cases it looks like there might be an issue with directory cleanup. I'm currently unable to check out the branch to try locally because it looks like it was deleted. Maybe we can revisit this in the future as part of another PR. Thank you for your effort @ad-m

@leontastic you can always pull old pull request via:
git clone git@github.com:jamhall/s3rver.git
cd s3rver
git fetch origin "pull/639/head:pr-639"

@ad-m Neat! I gave her a shot and the tests are still failing on Node 10. Not sure if we still want to support Node 10 but it's probably a decision for the next major release. This is a nice-to-have but it's not that urgent.
I would reopen this branch but I can't seem to since it was deleted. Maybe re-submit a PR to track progress on making the Node 10 tests pass? It's up to you if you'd like to take the initiative. Thanks again @ad-m

Currently, I do not have the resources to continue the work. I deleted my fork, so GitHub show branch as "unknown repository".
If you want - go ahead and take the code.

Btw, the issue here is simply that the recursive flag wasn't added to fs.rmdir(Sync) until Node 12:

If we want to keep Node 10 support, we could bring in a more specialized library like rimraf. My vote is currently just to drop it v10 for the next major.


The _pywrap_tensorflow_internal.so files generated by the two compilations are different，why？

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from source :
TensorFlow version:1.15.2
Python version:3.7
Installed using pip:
Bazel version 0.25.0:
GCC/Compiler version :7.3.0

@zxin-what
Please provide details about what platform you are using (operating system, architecture).Please,provide the exact sequence of commands / steps that you executed before running into the problem.Please, share error log as well.Thanks!


[5.3] Use relative path instead of absolute

Currently, if the symbolic links maps to /home/vagrant/project/storage/app/public, homestead will work but finder won't know where it is and the link will be only valid from inside the vm.
If we make it relative, it works from inside the vm and outside on hosts file manager.
The real issue that led me to make this PR is because rental servers often fakes your paths in a chroot jail or something I don't understand too much.
This way when we SSH into the rental server and link it to something like /user/project/storage/app/public, it will be fine for our SSH connection but I think what happens it that apache/nginx's user reads from the real path /home/users/user/project/storage/app/public and not from the fake resulting in broken urls.
I feel bad by removing the helper but I hope you consider merging it giving the benefits!

path.storage can be overwritten, and it is used in storage_path as you can see here. This PR would break functionality for anyone who has changed their storage path by overwriting path.storage.


cpplint.py conflict with Code Style Guide in repository

I found a question about this. the "repository" option can detect .git directory, and the default path is from second level directory. If I have follow files:
 Example
     - src/
         - hello.h
         - hello.cpp
      - .git/

and in hello.h file,
# ifndef EXAMPLE_SRC_HELLO_H_
# define EXAMPLE_SRC_HELLO_H_
...
# endif  // EXAMPLE_SRC_HELLO_H_

cpplint will waring as: header guard has wrong style.
this can fix by:
python cpplint.py --repository=`cd ..;pwd` src/hello.h

But, when I configure CPPLINT.cfg file, set repository, I found it's a invalid option.
and set
root=`cd ..;pwd`

is not working.
So, can you fix it let root support cd ..;pwd or add repository to CPPLINT.cfg.
THX!

I just ran into this
I have a git repo @ ~/ for dot files, and it is detected, I would imagine checking if the CD has a .git repo it should be used first


WARN: Link element does not have a linkpath to a file.

Thanks to your great plugin.
In the console of Windown 10 (with latest releases of Obsidian and your plugin) I see this:

What can I do to narrow the cause?

generally, you can ignore the warning messages. these are just debugging messages, i've forgotten to turn them off or have some sort of toggle.
the errors are not coming from this plugin.


upgrade imgui to v1.81

Upgrades DearImGui to version 1.81

Which triplets are supported/not supported? Have you updated the CI baseline?
No changes to existing triplets

Does your PR follow the maintainer guide?
I believe so...

Awesome, thanks for the PR!

fyi: the feature 'docking-experimental' is not updated, resulting in build failures. (#16276 )


Added Kazer Network

General:

[x] The pull request title is descriptive. (ex. Added Lunar Network, not Updated servers.json)
[x] The pull request does not contain any unrelated commits.  (ex. commits from previous pull requests)

Mapping Additions or Updates:

[x] No changes were made to the file's formatting.
[x] There are no syntax errors.
[x] There is no pre-existing mapping matching my id.
[x] My field values match your requirements.
You can view our patterns here: servers.schema.json, or take a look below and complete the field checklist:

[x] id: a lowercase string (ex. myserver)
[x] name: a string (ex. MyServerPvP)
[x] addresses: an array with lowercase strings (ex. ["my.server", "your-server.com"])

You do not need to specify subdomains, Lunar Client services automatically detect them.

[x] primaryAddress: an address (ex. my.server)
[x] minecraftVersions: an array with Minecraft versions as strings (ex. ["1.18", "1.19"])
[x] primaryMinecraftVersion: a Minecraft version as a string (ex. 1.18)

Logo Additions or Updates:

[x] My image is a png file.
[x] I uploaded my image to the logos directory.
[x] My image has a transparent background and is square (1:1 aspect ratio).
[x] My image is 512 pixels in width and height.
[x] My image's file name matches its mapping id.

Ok, i upload it, please check it again! Thanks!
------------------ 原始邮件 ------------------
发件人:                                                                                "LunarClient/ServerMappings"                                                                                @.>;
发送时间: 2022年7月31日(星期天) 晚上10:21
@.>;
<EMAIL_ADDRESS>主题: Re: [LunarClient/ServerMappings] Added Kazer Network (PR #433)
@itsJordan requested changes on this pull request.
Could you please include a higher resolution version of this logo? (still 512x512)
If the logo is supposed to be pixilated, could you please up-scale this image using a more appropriate method to avoid the blurry appearance.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

oh, i forgot it, sorry! can you check it again?
------------------ 原始邮件 ------------------
发件人:                                                                                "LunarClient/ServerMappings"                                                                                @.>;
发送时间: 2022年7月31日(星期天) 晚上10:34
@.>;
<EMAIL_ADDRESS>主题: Re: [LunarClient/ServerMappings] Added Kazer Network (PR #433)
@GrowlyX requested changes on this pull request.
Your logo does not match our sizing requirements of 512x512 pixels.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

i just upload failed, now i succssfully upload it, can you check it again?
------------------ 原始邮件 ------------------
发件人:                                                                                "LunarClient/ServerMappings"                                                                                @.>;
发送时间: 2022年7月31日(星期天) 晚上10:34
@.>;
<EMAIL_ADDRESS>主题: Re: [LunarClient/ServerMappings] Added Kazer Network (PR #433)
@GrowlyX requested changes on this pull request.
Your logo does not match our sizing requirements of 512x512 pixels.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

i upload this, can you check it again?
------------------ 原始邮件 ------------------
发件人:                                                                                "LunarClient/ServerMappings"                                                                                @.>;
发送时间: 2022年7月31日(星期天) 晚上10:21
@.>;
<EMAIL_ADDRESS>主题: Re: [LunarClient/ServerMappings] Added Kazer Network (PR #433)
@itsJordan requested changes on this pull request.
Could you please include a higher resolution version of this logo? (still 512x512)
If the logo is supposed to be pixilated, could you please up-scale this image using a more appropriate method to avoid the blurry appearance.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

hey, can you check it again? Thanks!
------------------ 原始邮件 ------------------
发件人:                                                                                "LunarClient/ServerMappings"                                                                                @.>;
发送时间: 2022年7月31日(星期天) 晚上10:34
@.>;
<EMAIL_ADDRESS>主题: Re: [LunarClient/ServerMappings] Added Kazer Network (PR #433)
@GrowlyX requested changes on this pull request.
Your logo does not match our sizing requirements of 512x512 pixels.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

hello? can you check it again?
------------------ 原始邮件 ------------------
发件人:                                                                                "LunarClient/ServerMappings"                                                                                @.>;
发送时间: 2022年7月31日(星期天) 晚上10:21
@.>;
<EMAIL_ADDRESS>主题: Re: [LunarClient/ServerMappings] Added Kazer Network (PR #433)
@itsJordan requested changes on this pull request.
Could you please include a higher resolution version of this logo? (still 512x512)
If the logo is supposed to be pixilated, could you please up-scale this image using a more appropriate method to avoid the blurry appearance.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

hello? can you check it again?
…
------------------ 原始邮件 ------------------ 发件人: "LunarClient/ServerMappings" @.>; 发送时间: 2022年7月31日(星期天) 晚上10:21 @.><EMAIL_ADDRESS>主题: Re: [LunarClient/ServerMappings] Added Kazer Network (PR #433) @itsJordan requested changes on this pull request. Could you please include a higher resolution version of this logo? (still 512x512) If the logo is supposed to be pixilated, could you please up-scale this image using a more appropriate method to avoid the blurry appearance. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

Logo is still not being detected as a valid PNG file - Could you please re-export your logo in the PNG file format and remove the background from the image.


Reduce API calls for wireless access point testing

https://github.com/ctomkow/capt/blob/612a8df0ea2bceb7de1efca69023d0dc11d32eb4/capt.py#L269
The testing is currently limited to one access point due to higher number of API calls required.

See issue about VoIP phones. Same issue.
https://github.com/ctomkow/capt/issues/41


Settings Export

Description of the new feature / enhancement
A button or option to export all PowerToys settings as a single file.
Scenario when this would be used?
Two main uses:
First, I have had to completely re-do my settings multiple times after updates, which is super annoying and time-consuming since I already did it once before.
Second, I have employees and it'd be great to have a default PowerToys settings file that all users could start from.
Supporting information
No response

/dup #586

you could copy over a known to their user directory


Server with TCP Transport handling multiple connections

Hi, I see that in tcp_transport.cpp, if the connection is closed by the client, TCPTransport::close() brings down the server thread by setting m_runServer = false;
Please correct me if I'm wrong but it seems to me that this means that for a subsequent client connection, the server and transport need to be reinitialized again. This also looks like it would incur the overhead of having to create a new thread. I was just wondering if you could comment on why this approach was taken?

Hi @mentaal,
i am sorry i was not working on this transport. But as i see it you can reuse only "open" and/or "configure" functions again. You don't need new server or thread. This way you can decide what happen with server when client will close connection.

Hi @Hadatko,
No problem. I was just curious. Indeed, TCPTransport::open looks like it is required to be rerun but note that it does call m_serverThread.start(this); which creates a new thread under the hood. This is because when ::close() is called, m_runServer is set to false, which breaks out of serverThread's while loop which closes the server's socket the the server thread completes its execution.
I am thinking about refactoring this code a bit for my project to allow the server to continue waiting for new connections without having to reinitialize the server. I am also a bit skeptical about having accept() and read() run concurrently without providing more infrastructure for concurrent client support in general. (Though I am no expert in network programming - in fact I have very little experience in this domain so take this with a grain of salt!) For example, what should happen if a new connection is accepted whilst read is still blocking, waiting for more data on the old socket?
This stackoverflow comment is worth reading on that thought: https://stackoverflow.com/questions/3589723/can-a-socket-be-closed-from-another-thread-when-a-send-recv-on-the-same-socket#27790293
I have more thinking to do on the topic. Any feedback you have on it is most welcome.

Of course, that is no problem at all. I could be a while with it as I have to develop it first.

Here's a link to a branch implementing a tcp server without threading. Before adding it as a pull request, tell me what you think: https://github.com/mentaal/erpc/tree/tcp_no_threading

Hi @mentaal ,
although i like your idea, do you think it is good idea to keep two solutions of one transport layer?


Include "fromnow" as a format.

This is very handy when getting a format string from a remote source.

Are you using fromNow in a format string along with other tokens? Like YYYY-MM-DD fromNow?

Yes, sometimes, though that's not the only use case.  It also makes user provided formats much simpler to deal with.
Instead of
format === 'fromnow' ? m.fromNow() : m.format(format);

I can simply do
m.format(format);


rebuild py311 for snowflake channel

build for win-64:
`TEST END: C:\Users\48667\miniconda3\conda-bld\win-64\snowflake-ml-python-1.0.8-py311h2bbff1b_1.tar.bz2
Renaming work directory 'C:\Users\48667\miniconda3\conda-bld\snowflake-ml-python_1695381269855\work' to 'C:\Users\48667\miniconda3\conda-bld\snowflake-ml-python_1695381269855\work_moved_snowflake-ml-python-1.0.8-py311h2bbff1b_1_win-64_main_build_loop'
INFO:conda_build.utils:Renaming work directory 'C:\Users\48667\miniconda3\conda-bld\snowflake-ml-python_1695381269855\work' to 'C:\Users\48667\miniconda3\conda-bld\snowflake-ml-python_1695381269855\work_moved_snowflake-ml-python-1.0.8-py311h2bbff1b_1_win-64_main_build_loop'
shutil.move(work)=C:\Users\48667\miniconda3\conda-bld\snowflake-ml-python_1695381269855\work, dest=C:\Users\48667\miniconda3\conda-bld\snowflake-ml-python_1695381269855\work_moved_snowflake-ml-python-1.0.8-py311h2bbff1b_1_win-64_main_build_loop)
INFO:conda_build.utils:shutil.move(work)=C:\Users\48667\miniconda3\conda-bld\snowflake-ml-python_1695381269855\work, dest=C:\Users\48667\miniconda3\conda-bld\snowflake-ml-python_1695381269855\work_moved_snowflake-ml-python-1.0.8-py311h2bbff1b_1_win-64_main_build_loop)
WARNING conda.gateways.disk.delete:unlink_or_rename_to_trash(188): Could not remove or rename C:\Users\48667\miniconda3\conda-bld\snowflake-ml-python_1695381269855_build_env\Library\lib\modules.  Please remove this file manually (you may need to reboot to free file handles)
Automatic uploading is disabled
If you want to upload package(s) to anaconda.org later, type:
To have conda build upload to anaconda.org automatically, use
$ conda config --set anaconda_upload yes
anaconda upload ^
C:\Users\48667\miniconda3\conda-bld\win-64\snowflake-ml-python-1.0.8-py311h2bbff1b_1.tar.bz2
anaconda_upload is not set.  Not uploading wheels: []
INFO :: The inputs making up the hashes for the built packages are as follows:
{
"snowflake-ml-python-1.0.8-py311h2bbff1b_1": {
"recipe": {
"c_compiler": "vs2017",
"channel_targets": "defaults",
"target_platform": "win-64"
}
}
}
####################################################################################
Resource usage summary:
Total time: 0:13:10.5
CPU usage: sys=0:00:47.0, user=0:02:00.3
Maximum memory usage observed: 875.2M
Total disk usage observed (not including envs): 13.1G
####################################################################################
Source and build intermediates have been left in C:\Users\48667\miniconda3\conda-bld.
There are currently 9 accumulated.
To remove them, you can run the conda build purge command
(base) PS C:\Users\48667\anaconda\aggregate>`


Query: Apply conditional collapse to comparisons only for truly comparison Expressions

Issue:
We have optimization to convert a ? b : c where a, b, c are all comparison expression to a && b || c && d.
For this case, while b seemed like comparison expression but it was in such pattern due to being nullable bool. (b == true)
A true comparison expression is of type bool since it gives true/false answer. If it is not then its value type expression.
In user case, since we tried incorrect optimization, we ended up with mismatched types in AndAlso expression.
Fix is to improve logic of IsComparisonExpression to only identify boolean expression
Resolves #9468

@maumar - Ping. Tiny 😄


Ensure values from secrets has higher precedence than from configmap

In case both values are present in ConfigMap and Secret, one from Secret is being used. This is already documented in the ConfigMap example files, however logic in the deploymentconfig was vice-versa.
Describe the behavior changes introduced in this PR
Linked Issues?
resolves # <-- Use this if merging should auto-close an issue
related to # <-- Use this if it shouldn't
Testing Instructions
Please include any additional commands or pointers in addition to our standard PR testing process.
@redhat-cop/mdt

/LGTM