[spring-cloud-gateway-mvc] GET requests fail on HTTP/2 upgrade

Boot Version: 3.2.0-M3
Gateway Version: 4.1.0-RC1
We recently started with a new application and decided to use spring-cloud-gateway-mvc as the basis. When writing tests "around" our application with WireMock standing in for the target service we noticed that very simple GET requests were not processed successfully but failed with this error message:
java.io.EOFException: EOF reached while reading
	at java.net.http/jdk.internal.net.http.Http2Connection$Http2TubeSubscriber.onComplete(Http2Connection.java:1587) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$ReadSubscription.signalCompletion(SocketTube.java:648) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$InternalReadSubscription.read(SocketTube.java:853) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$SocketFlowTask.run(SocketTube.java:181) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.common.SequentialScheduler$SchedulableTask.run(SequentialScheduler.java:207) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.common.SequentialScheduler.runOrSchedule(SequentialScheduler.java:280) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.common.SequentialScheduler.runOrSchedule(SequentialScheduler.java:233) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$InternalReadSubscription.signalReadable(SocketTube.java:782) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$ReadEvent.signalEvent(SocketTube.java:965) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.SocketTube$SocketFlowEvent.handle(SocketTube.java:253) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.HttpClientImpl$SelectorManager.handleEvent(HttpClientImpl.java:1467) ~[java.net.http:na]
	at java.net.http/jdk.internal.net.http.HttpClientImpl$SelectorManager.lambda$run$3(HttpClientImpl.java:1412) ~[java.net.http:na]
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) ~[na:na]
	at java.net.http/jdk.internal.net.http.HttpClientImpl$SelectorManager.run(HttpClientImpl.java:1412) ~[java.net.http:na]

This only happens when WireMock is called through the gateway, attempts to reproduce the error with several HTTP client implementations calling WireMock directly were unsuccessful. Putting NGINX between the gateway and WireMock prevented the HTTP/2 upgrade and the request worked as expected. After some digging we found out that the JDK HTTP client used by Spring will send the GET request as stream with a Transfer-encoding: chunked header. This happens because the content-length is reported as -1 to the HTTP client as the request body that is passed to the bodyPublisher methond in HttpRequest.BodyPublisher is empty but not null.
I'm aware of the fact that the mentioned classes are residing in the spring-web part of the spring-framework project, but since it is no  longer possible at this point to check for an empty body this will probably have to be fixed at an earlier stage.
What seems to trigger this error is that the server (Jetty in this case) initiates the upgrade and waits for the client to send the preface. Instead of doing so the client appears to send an empty chunk to indicate that it is done which then causes the server to send an ìnvalid_preface error and terminate the connection.
Sample
I added a small project that can be used to reproduce the issue. It just defines one route that passes everything to WireMock and contains tests that include a call to WireMock through the gateway (fails) and directly using the JDK client (succeeds). I would expect both tests to work once a fix has been applied.
http2-upgrade-demo.zip

Setting the following as a workaround switches the http client and all the tests pass
spring.cloud.gateway.mvc.http-client.type=autodetect

The wiremock test dependency brings in org.apache.httpcomponents.client5:httpclient5 which is what us used in the successful tests.


redirect all participants to their return URL after estimation has been confirmed

solved with a nice result page and a button


Question: Can I pause a job while it's running in a try/catch

My job is accessing a rate limited service.  When the limit is exceeded the code throws an exception which can be caught in the worker.  The error contains the time when the rate limit will reset - I want to restart the job at that time.
Is this the correct way to handle this:
 _queue.process('cachejob', 1, function(job, ctx, done) {
    let buildCache = new BuildCache();

    buildCache.processJob(job)
      .then(() => {
        done();
      }).catch((err) => {
        debugger;
        if (err
            && err.status
            && err.status === 403) {
          ctx.pause(err.ratelimitReset * 1000, (err)=> {
            if (!err) {
              ctx.resume();
            } else {
              done(err);
            }
          });
        } else {
          done(err);
        }
      });
  });

pause's first input is the timeout of the process of trying to pause the worker, not the interval that worker will be paused. After the pause callback is called. you can delay until rateLimitReset and then resume.
You are currently resuming right after pause callback is called.

So I changed to this but the jobs just keep running and I never see the 'resuming'.
          console.log('*********         pausing', pause);  ///  prints 8000
          ctx.pause(100, (err)=> {
            if (!err) {
              setTimeout( () => {
                console.log('*********    resuming');     //never see this
                ctx.resume();
              }, pause );
            } else {
              done(err);
            }

My bad - the job queue does stop processing.


MINORFIX: Fixing Owner Name in License file.

When the project is created, using the template, the license text is starting with:
**ADotNet - .NET Library
Copyright (c) 2021 Hassan Habib All rights reserved.**

Hi @mabroukmahdhi, we can close this one.  When you install ADotNet, that Licence file gets added to the root of that project automatically.  This is by design.

Okay, I close this issue.


Incorrect Version Number Printed In Logs

(While submitting an issue briefly describe the problem you are facing or a new feature you would want to see added)
What happens?
The incorrect version number is printed in the logs when running the container image. The below log line was captured from the v0.1.5 container image.
2022/04/18 21:57:35 blackbox-helloworld-responder, version v0.1.13 (branch: HEAD, revision: 2ed20806ee9b01fe8c16a157b2c4d50e5f107188), build date: 20220418-18:49:21, go version: go1.18.1

What were you expecting to happen?
The correct version number is printed in the logs
Steps to reproduce:

Download container image
Run it
Look at the version number printer by the first log line

Any errors, stacktrace, logs?
See above.
Additional comments:
N/A

This was fixed in #67


Updates to README.md

Updates for readability and clarity
I had trouble following some of the directions as they were written, so my edits are to try to make them more clear for the next people who try to use them.

Thank you for this plug-in!

Hello.  Thank you for your contribution!

Please correct the numbering in the README. Here's screenshot off your pull:

Old numbering (everywhere) was 1,2,3... now its (also everywhere) 1,1,1...

In the sectionInstallation in Caché:

CACHELIB namespace should be CACHELIB database
%CACHELIB database should be CACHELIB database
'%CACHELIB' should be CACHELIB

Hi - you're actually not using markdown appropriately by numbering things 1, 2, 3.  Look at the preview and you'll see that it looks correct when you use 1, 1, 1. See documentation here.
I made the CACHELIB changes.
Thanks,
Michael

Didn't knew about numbering by 1,1,1. Still 1,2,3 produces the same result but looks nicer in my opinion. @adaptun?

Also CACHELIB namespace should be CACHELIB database. There's no CACHELIB namespace.


Use of internal package not allowed

OS: Ubuntu 22.04
Go: Golang 1.18.1

I follow official instruction to build microservice.
However, as I execute go run user.go -f etc/user.yaml in bash, it raises the error:
package command-line-arguments
        imports go-zero-demo/mall/user/rpc/internal/server
        imports go-zero-demo/mall/user/rpc/internal/svc
        internal/svc/servicecontext.go:4:2: use of internal package go-zero-demo/mall/order/api/internal/config not allowed
package command-line-arguments
        imports go-zero-demo/mall/user/rpc/internal/server
        imports go-zero-demo/mall/user/rpc/internal/logic
        internal/logic/getuserlogic.go:7:2: use of internal package go-zero-demo/mall/order/api/internal/svc not allowed
package command-line-arguments
        imports go-zero-demo/mall/user/rpc/internal/server
        imports go-zero-demo/mall/user/rpc/internal/logic
        internal/logic/getuserlogic.go:8:2: use of internal package go-zero-demo/mall/order/api/internal/types not allowed

The problem possibly  causes by the generated-directory name internal.
Is there any way to fix it without renaming the default directory name?

go env -w GO111MODULE="on"
and DO NOT modify golang default env variables

Do you mean goctl changed go env?

It's my own mistake. I changed my golang default env variables before.

I see. Thanks for your reply!


Redundant tapping features functionality

Why there are two functions with very similar use cases? get_tapping_features and tapping_features? Do we really need them both?

I just named them that ways, so that if need arises, we can incorporate models into tapping like we did for accelerometer and gyroscope. If not, then we can possibly get rid of one of those

I would remove the tapping_features.R and move all of its content to get_tapping_features.R. If needed we can add this later. For now, we keep this repo slim and less-redundant.


Facebook captcha challenge when trying to generate RSS feed for Facebook. Help?

Says it cannot fetch requested page. Wants me to fill in a captcha but doesn't show with what. How to fix?

I have the same issue. Problem is the image for the captcha is broken, so I can't solve the captcha!

Have you tried using the other Facebook bridge (FB2Bridge)?

Wants me to fill in a captcha but doesn't show with what. How to fix?

Well, this might be hard to fix as it requires the capture challenge (which I've never seen). Can you create a gist for an example page including the capture challenge (the HTML document)?
The bridge is likely broken somewhere along these lines: https://github.com/RSS-Bridge/rss-bridge/blob/704a87ad978488e9db52b03ba4fddd96d30d1918/bridges/FacebookBridge.php#L387-L414
Maybe the capture challenge isn't an image but something else entirely (like the "I'm not a robot" button)...

I wrote this part of the bridge for use with this script to automate captcha solving.
However since then my server is not blacklisted anymore so I can't get a sample captcha page.
An easy way for you to get the page would be inserting the following line:
file_put_contents('facebook-captcha.html', $html->outertext);

... just before captcha related code at line 386. Then you can download facebook-captcha.html using your FTP client and investigate changes in the captcha.

I've the same issue on my server (captcha challenge without any image). It worked fine, but as my server does a lot of requests everyday, I think Facebook blacklisted it, so that's why I receive a captch now...
But even if you can show the captcha, how is it supposed to work when the RSS is accessed by applications? no one can solve the captcha in this case...

Please provide a sample captcha page as described in my comment above.

here is what I get on my server: https://pastebin.com/tuLhVgwF

Thank you, here is the HTML part that's interesting:

  <form method="post" action="https://www.facebook.com/FFW.Loederburg" onsubmit="" id="u_0_g">
    <input type="hidden" name="lsd" value="AVqvnF4q" autocomplete="off" />

Security Check

      <input type="hidden" autocomplete="off" id="captcha_persist_data" name="captcha_persist_data" value="AZl_EHzZUF3fYjYrXhbPue-Q_7WamYv7os2i8XMZRlbtqlqiqahfI-7z5s9GTOOx60msJ5Qi2ZBtll-gXu2nND66o72i9P0ItW3DcUly1TK1-cmjyKKJ2nLNBJU5A_Qd6NxwkpWO2we90J4mEwtQZdPTCEI3rWuxUB-FNNRtLxH6vRuIycSiV00JKaBxSw1u0Y2-_c8GKs6mtBxKyo1EYjBTh6qakN54_qhK5DOQv17hReF54bFgQ8O1OWTl90-IMHu5bkIoqw8YevLIaMtsQZb6s9AdZClzPV0CpSi_byhRbGOTBZ4E-1eFFxVZ-L_lHVkCwjVHVFcSi3LUxofdAhndtNJSNPFEwB2xdyXqe6MGAxFMwVVXsdgcQ6a-K552qEHmZSr7lnAVqCRMq1rFVG9hMgGea7AGVFyGh0JdM_F40Y8LAENNLGTn7YAFFYGUdLNHb0zQ0zfZhC6OAbizxkx9" />

        <input name="captcha_response" id="captcha_response" type="hidden" />

        Why am I seeing this?

Security Check

          This is a standard security test that we use to prevent spammers from creating fake accounts and spamming users.

    <button value="1" class="_42ft mts _4jy0 _4jy3 _4jy1 selected _51sy" name="captcha_submit" id="captcha_submit" type="submit">Submit</button>
  </form>

It seems like they are switching from their own (flawed) Captcha mechanism to the (much stronger, Google-provided) ReCaptcha mecanism. I don't think we will be able to bypass that anymore. Maybe proxy it properly through the bridge, but that's not interesting if we cannot automate solving :thinking:

ok, thanks for your feedback
and do you know when Facebook displays this captcha? because right now it is not displayed anymore.
Is it when my server does too many requests? (I've a lot of users using facebook rss feeds)

ok, thanks!

I suppose this issue is closed.


Yes
Performance Enchancers
Sodium
Searched for known issues?
Searched closed issues

You need to install indium https://modrinth.com/mod/indium

oh thanks, i didnt expect a fast reply, and also i didnt realize to use that mod


Updated link to install instructions

Summary
Updated link to install instructions
Details and comments
With release of Qiskit 0.25.0 some changes were made to documentation. This pull request changes README.md file to reflect the new location for installation instructions.

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

@jaygambetta can you look at this? you approved similar pull request: #1228


Bug fixes and Improvements for Identity verification providers feature

Describe the issue:
The identity verification providers feature is disabled in IS 7.0 vanilla pack. It can be enabled via following TOML configuration.
[console.identity_verification_providers]
enabled = true

Once the feature is enabled, the following issues were noticed;

[BE] Even after enabling the configuration for IDVP, the page is not displayed in the console. This issue arises because the tokens do not return IDVP-related scopes, even for administrators who have already been granted permission.
[FE] After setting up OnFido in IS, when trying to configure it in the console, the identity verification provider's name does not appear when typed.

Following UX gaps were identified specific to the OnFido connector;

Make the Webhook Token field optional at the OnFido setup. Webhook token can not be obtained from OnFido dashboard before creating the IdVP.
Provide a place to obtain the URL to generate the webhook token.

How to reproduce:

Refer to the description.

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 7.0.0
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Improvement-related PRs

UI: https://github.com/wso2/identity-apps/pull/6949
Framework: Enable IdVP feature - https://github.com/wso2/carbon-identity-framework/pull/5994
Framework: Add feature scopes - https://github.com/wso2/carbon-identity-framework/pull/5989


Mode field profile interpolation + overlap integrals

Improvements to simulation/modes/ Mode objects:

Mode.grid_interp() method to generate a callable that can interpolate the field
In overlap.py, innerprod_trapz  function that can compute the transverse field integral <1|2> of two modes. Uses trapezoid rule on some user-provided grid.
Added test in overlap.py

TODO
Since the interpolated field profiles are callables, could extend to more efficient integration schemes

Thank you Simon!


unexpected [, expecting semicolon or newline or }

When I use the official example
package main

import (
	"context"
	"fmt"
	openai "github.com/sashabaranov/go-openai"
)

func main() {
	client := openai.NewClient("sk-xxxxxxxxxxxxxxxxxxxxxxxxx")
	resp, err := client.CreateChatCompletion(
		context.Background(),
		openai.ChatCompletionRequest{
			Model: openai.GPT3Dot5Turbo,
			Messages: []openai.ChatCompletionMessage{
				{
					Role:    openai.ChatMessageRoleUser,
					Content: "Hello!",
				},
			},
		},
	)

	if err != nil {
		fmt.Printf("ChatCompletion error: %v\n", err)
		return
	}

	fmt.Println(resp.Choices[0].Message.Content)
}

Reported the following error
# github.com/sashabaranov/go-openai
/root/go/pkg/mod/github.com/sashabaranov/go-openai@v1.5.7/chat_stream.go:29:15: syntax error: unexpected [, expecting semicolon or newline or }
/root/go/pkg/mod/github.com/sashabaranov/go-openai@v1.5.7/chat_stream.go:58:60: syntax error: unexpected {, expecting comma or }
/root/go/pkg/mod/github.com/sashabaranov/go-openai@v1.5.7/chat_stream.go:60:49: syntax error: unexpected ) at end of statement
/root/go/pkg/mod/github.com/sashabaranov/go-openai@v1.5.7/chat_stream.go:63:42: syntax error: unexpected comma after top level declaration

I need your help

Did you fix it? me too

I think its go version. Could you please check your version?

@venvis856 Perhaps your locally installed version of Go is less than 1.18. go-openai uses generics internally, so Go 1.18 or higher is required.

Closing this due to lack of comments for an extended period.


Feature/zoe cta

Description

link to: https://joinzoe.com/about-zoe
On what platform have you tested the change?

[x] Android - Emulator
[ ] Android - Physical device
[x] iOS - Emulator
[ ] iOS - Physical device

Screenshots
Please attach screenshots showing the change if relevant
Checklist

[ ] I have updated mockServer
[ ] I've added analytics as relevent
[ ] I have run npm test
[ ] I have run npm run test:i18n

Out of scope and potential follow-up
Is there any related changes that you plan to do in a follow-up PR or voluntarily excluded from the scope?

Killed by legal :(


dash-problem

Hi.
As I see "-" isn't a valid character in connection string for cluster.
For example I have cluster with 3 nodes: "zk1.test-dm.al:2181,zk2.test-d.al:2181,zk3.test-d.al:2181". But when I try to use it "OK" button doesn't become active.
It can be fixed in kafkalytic too, please.

hi, can't reproduce it in zoolytic but can reproduce in kafkalytic, will look into it today. Meanwhile, it works for single server with dash character,  you can use it as workaround.

@adubinchak i uploaded version 0.5.3 into alpha channel, can you please check if it works for you? it will be available for download tomorrow

thank you! it works now
will it be available for kafkalytic too?

hi
please check kafkalytic 0.9.7 in alpha channel.
Regards
Danila
On Fri, Jul 10, 2020 at 12:23 PM adubinchak<EMAIL_ADDRESS>wrote:

thank you! it works now
will it be available for kafkalytic too?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ermadan/zoolytic/issues/4#issuecomment-656763244, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEBJ3QUBR6F5UDL3T2MUGW3R2456VANCNFSM4OIDD3PA
.

--
Regards,
Danila Ermakov


two new exercises f5 workshop - irules, and bigip_config save running configuration

SUMMARY
f5workshop additional two new exercises
ISSUE TYPE

Feature Idea

COMPONENT NAME
f5workshop
ADDITIONAL INFORMATION
n/a

@payalsin

https://github.com/network-automation/linklight/pull/248


fix resource leak in InputStreamDownload.java (input stream is never …

…closed)

thanks @siddjain! this sounds ok to me
may you think in a way to ensure this behaviour in a test case?

Unfortunately I don't think I have time to write a test case at least for now. Perhaps create a work item for the test case?
Date: Mon, 23 May 2016 06:38:31 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [caelum/vraptor4] fix resource leak in InputStreamDownload.java (input stream is never … (#1069)
thanks @siddjain! this sounds ok to me
may you think in a way to ensure this behaviour in a test case?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Hey @siddjain, Can I help you writing a test case? I like to aprove that pull request. :)

Sure

From: Rogerio Angeliski<EMAIL_ADDRESS>Sent: Tuesday, December 06, 2016 2:37 PM
To: caelum/vraptor4
Cc: siddjain; Mention
Subject: Re: [caelum/vraptor4] fix resource leak in InputStreamDownload.java (input stream is never … (#1069)
Hey @siddjainhttps://github.com/siddjain, Can I help you writing a test case? I like to aprove that pull request. :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/caelum/vraptor4/pull/1069#issuecomment-265295712, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAuSMeZnIEm_cKUBT1M9Po5t7zCf64nEks5rFeOugaJpZM4Ijqti.

@siddjain I will open a new PR with the test case and that modification. Ok?

Ok
Get Outlook for Androidhttps://aka.ms/ghei36
On Wed, Dec 7, 2016 at 12:59 AM -0800, "Rogerio Angeliski" <notifications<EMAIL_ADDRESS>wrote:
@siddjainhttps://github.com/siddjain I will open a new PR with the test case and that modification. Ok?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/caelum/vraptor4/pull/1069#issuecomment-265392164, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAuSMU_a01gZwMF4KQDuSz7u0U56EWfmks5rFnVWgaJpZM4Ijqti.

@siddjain Can I close that issue?

Sure thanks
Get Outlook for Androidhttps://aka.ms/ghei36
On Thu, Dec 8, 2016 at 6:23 AM -0800, "Rogerio Angeliski" <notifications<EMAIL_ADDRESS>wrote:
@siddjainhttps://github.com/siddjain Can I close that issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/caelum/vraptor4/pull/1069#issuecomment-265751152, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAuSMdfdA8Fi3ZhuT-hSaeW3VxUVsqSJks5rGBK3gaJpZM4Ijqti.


Supply public_updated_at for coming soon

If we want to make public_updated_at mandatory for everything, we need this
change.

:+1:


Editor support

Proposal:
Editor support for .bolt files (e.g. WebStorm plugin, VS Code plugin)
Reason:
It's great to write rules with bolt but still it's very easy to misspell something, miss a bracket and so on. Yes, that is checked by the compiler on building but it would be nice, to have some editor intellisense (for types, functions, ...) and errors within the IDE without building every expression.
This is maybe not the correct place for that request but I didn't know where to put it and I didn't find any issue according to that. Please let me know if there is some editor support already.

This is a totally valid request. Do you know if there is a way to write a plug-in once (maybe specify the Grammer) and get plugins for multiple editors?
/cc @ryanpbrewster

I'd like to make an attempt. If anyone else would like to work on this with me please let me know.

I used something unofficial from the firebase team for Sublime text awhile ago (@samstern authored it afaik). Just seeing a vscode plugin here, might be a nice starting fork https://github.com/smkamranqadri/vscode-bolt-language

That extension is not supported nested path yet 😢
https://github.com/smkamranqadri/vscode-bolt-language/issues/3

In case anyone is interested I ended up using vscode plugin https://marketplace.visualstudio.com/items?itemName=smkamranqadri.vscode-bolt-language

Just release new version, will appreciate feedback.
https://marketplace.visualstudio.com/items?itemName=smkamranqadri.vscode-bolt-language


SGEMM does not calculate output when input is all 0?

v19.08
arm64-v8a
Working on using NEGEMM, while trying to figure out why the output was not correct I realized while monitoring input/output pairs that it seems NEGEMM::run() only does anything when the input is nonzero? That means that whenever the input is zero it just kept outputting the last output with nonzero input. This is obviously not the sort of behaviour one wants.
Is there a flag or something of that sort which would change the behaviour to always calculate the output?

Hi @RuABraun
Would you please share the source code to reproduce the problem?

Sorry I don't have the time and have found another library that suits my needs better.
Feel free to close if you want.


feat(java): browse objects/synonyms/rules

🧭 What and Why
🎟 JIRA Ticket: APIC-616
Implement browse for object/synonyms/rules.
They are meant to be used with for loop like:
Iterator<Actor> browse = client.browseObjects(indexName, new BrowseParamsObject(), Actor.class);
for(Actor actor : browse) {
  System.out.println(actor.name);
}

Changes included:

Not sure if the doc is outdated or the query was wrong but _highlightResult only contains "synonym" and never the expected object, so we decided to remove it from the spec.
Move the helpers to it's own mustache file

🧪 Test
Playground with the snippet above

adding this here for future reference of motivations:

https://github.com/algolia/algoliasearch-client-python/pull/545
https://github.com/algolia/algoliasearch-client-javascript/blob/master/packages/client-search/src/methods/index/browseRules.ts#L32
https://github.com/algolia/algoliasearch-client-javascript/blob/master/packages/client-search/src/methods/index/browseSynonyms.ts#L32


RPC Client remote connection

Hello there
  #localhost works
  var client = hbase({
    zookeeperHosts: ['localhost:2181'],
    zookeeperRoot: '/hbase',
    rootRegionZKPath: '/meta-region-server',
    rpcTimeout: 30000,
    pingTimeout: 30000,
    callTimeout: 5000
  });

#remotely, it doesn't work
  var client = hbase({
    zookeeperHosts: ['<IP_ADDRESS>:2181'],
    zookeeperRoot: '/hbase',  #is this path correct?
    rootRegionZKPath: '/meta-region-server',
    rpcTimeout: 30000,
    pingTimeout: 30000,
    callTimeout: 5000
  });

Error message when connecting remotely..
Failed to connect to zookeeper. zkHosts: [<IP_ADDRESS>:2181] zkRoot: '/hbase'

I am a windows user running a vagrant machnine with hbase.
But when I run the same app on my windows remotely, I get the above mentioned error message.
But on my server I can see that my client-ip is listed correctly..

Do you have any idea, what could be the problem?

Hi, your zookeeperRoot is correct. Please post DEBUG=* with simple script. Are you sure that your vagrant ip (<IP_ADDRESS>) is reachable from your windows machine?

Here is the result of debug..

I guess so, because if i type a false vagrant-ip, then my ip (<IP_ADDRESS>) is removed from the clients list of zookeeper. But I am not quite sure.

Your habase reports that the region server is running on localhost.. that's the reason why it won't connect from your windows machine.  That might be the problem.

Yes, that's right. By the way, I use this vagrant settings (https://github.com/zsiciarz/vagrant-hbase)
So isn't it possible to connect from windows?

No idea with the pseudo-distributed mode.. but you can start zookeeper in your vagrant and then run hbase in fully distributed mode. That should work.

Ok. I give it a try. Thank you very much for your help.


Always create a new environment block before we spawn a process

This commit ensures that we always furnish a new process with the
cleanest, most up-to-date environment variables we can. There is a minor
cost here in that WT will no longer pass environment variables that it
itself inherited to its child processes.
This could be considered a reasonable sacrifice. It will also remove
somebody else's TERM, TERM_PROGRAM and TERM_PROGRAM_VERSION from the
environment, which could be considered a win.
I validated  that GetCurrentProcessToken returns a token we're
technically able to use with this API; it is roughly equivalent to
OpenProcessToken(GetCurrentProcess) in that it returns the current
active access token (which is what CreateEnvironmentBlock wants.)
There's been discussion about doing a 3-way merge between WT's
environment and the new one. This will be complicated and I'd like to
scream test the 0-way merge first ;P
Related to #1125 (but it does not close it or resolve any of the other
issues it calls out.)
Fixes #7239
Fixes #7204 ("App Paths" value creeping into wt's environment)

I'm cool with this as-is, but I think I want to hear the optional<void*> discussion first

So, that's a fair question and one for which I don't really have a good answer. std::optional<void*> was driven by my innate aversion to a raw void*. I would have passed an EnvironmentBlockPtr as @zadjii-msft suggested, but since that's a typedef over wil::unique_any it enforces ownership semantics at the function signature. It's generally incorrect for a function to require particular ownership semantics of its parameters.
I'm fine just dropping it to void*, but I somewhat wish that we had a better type to express that it's an environment block. I'd much rather not write more boilerplate just to do so, so I'm not interested in introducing another manual reference counting struct type. 🤷

There, I switched it to a std::optional<gsl::not_null<void*>>.

I don't see any gsl::not_null in the diff….

Well, I decided that an optional-not-null-void* was the same as a void* so I cut out the middle party 😉

This PR broke my baulk, I implemented a package manager baulk, baulk provides baulktermainl and baulk-dock After initializing specific environment variables, start Windows Terminal, and then perform the corresponding operations in the shell.
But this PR removed this possibility, which made me have to think of other ways to solve this problem.

You know, I'm not surprised that we broke somebody ... but I am surprised that we broke somebody so instantly. Thanks for letting us know.
It looks like you fixed it on your end, so we're not going to index too much on getting a fix in for 1.3... but we'll keep use cases like that in mind.

Also, thanks for testing top-of-tree builds so much :smile:

https://github.com/microsoft/terminal/issues/7399
It turns out that the Volatile Environment isn't part of the new block. How silly.

@DHowett Hello. Has this issue regressed? I'm on WT version 1.11.2921.0 and I'm having the same problem albeit using the "setx /M" command to update the system wide env variable. Is this different?

@kpentaris, these changes have been repeatedly reverted from stable releases of Windows Terminal, e.g. in https://github.com/microsoft/terminal/commit/616a71dd23e8fff717919c20c382edeaf59f433a for Windows Terminal 1.11.2921.0. If you want this feature, use Windows Terminal Preview for now. I guess it will continue this way until the Windows PowerShell (x86) incompatibility https://github.com/microsoft/terminal/issues/7418 is fixed somehow.

@kpentaris I believe we never ended up shipping this to stable, and it's only in the Preview builds of the Terminal

D'oh! It's actually mentioned in the latest commit... Very well, hopefully at some point this is fixed.


Wrong Labels in Last Graph in "The basics of double/debiased machine learning"

Description of the issue or change proposal
The labels in the legend of the last graph on the page are in the wrong order.

I tried running the code to generate the graph on my system but I failed to reproduce it. The graph that I got after running the code was the right one.

I even tried to compile the code on my system to generate the HTML files but it still generated the right graph.
Below is my system info that I got after running the following code:
import platform; print(platform.platform())
import sys; print("Python", sys.version)
import doubleml; print("DoubleML", doubleml.__version__)
import sklearn; print("Scikit-Learn", sklearn.__version__)
import seaborn; print("Seaborn", seaborn.__version__)

Linux-5.11.0-41-generic-x86_64-with-glibc2.10
Python 3.8.8 (default, Apr 13 2021, 19:58:26)
[GCC 7.3.0]
DoubleML 0.4.1
Scikit-Learn 0.24.1
Seaborn 0.11.1

Comments, context or references
No response

So the problem is, that you locally get the correct labels / label order but on the deployed page the labels are wrong?

Yes, precisely.

Explicitly setting the labels fixes the issue, see #69.

The issue was successfully fixed, see the dev version https://docs.doubleml.org/dev/guide/basics.html. I will trigger a deploy to stable...


Filtering group_concat field descriptors does not work

the resulting sql is not correct and therefore not working ;)
i'm going to provide more information tomorrow

Tomorrow has passed ;-)

the listbuilder is creating a query where it tries to apply a condition to the group_concated value.
actually it should then apply the condition to the original entity.
e.g..: tags
...
WHERE GROUP_CONCAT(tags.name) LIKE :tagname

should just be
...
WHERE tags.name LIKE :tagname

i guess

@wachterjohannes fixed?

is fixed by introducing a getWhere in the FieldDescriptor


Speed up localization synching

We noticed the same crashes when updating entries that have been fixed in the latest 0.17.2 release (https://github.com/contentful/contentful-persistence.swift/commit/bb43c169798a5088b1fa1eec4b5eaa8e3ab9d0cb).
I have compared our 2 fixes and found out that our solution has a better performance when there are multiple locales. Did a few time measurements on the different environments we have. They each have 2-4 localizations.
I can attach some measured times if you are interested.
This commit optimizes the locales loop from O(L^2) to O(L) runtime by using a dictionary lookup instead of a linear array search. This especially increases performance when there are many localizations.

Hello @sjaindl! Thanks for catching that and creating this PR. I'll create separate PR that will consist of your addition and with one more change for creating/updating assets. I noticed there might be issue when you have localized assets and then you may end up with the same invalid behaviour like we had in 0.17.1.
Many thanks!

@sjaindl 0.17.3 is released now. Many thanks for your input!


Build failed: verifies that "hot exit" works for dirty files (without delay)

Build: https://dev.azure.com/monacotools/a6d41577-0fa3-498e-af22-257312ff0545/_build/results?buildId=246670
Changes: https://github.com/Microsoft/vscode/compare/86baeb9...1b358ac

ca194d96-476c-4150-b5de-ab1a72546714.dmp.zip


Dimmer only closes first time with modal

I have a modal, that opens ok, closes ok, then when reopened and closed again, the dimmer stays. This is using "hide" from a button click, if I click the dimmer itself it seems to work.
When opening the second time there is a bit of flashing.
my code, html incomplete as it is inserted via ajax, see lower down

var EmptyFormModel = (function () {
    function EmptyFormModelx(url, data, title, width, height) {
        this.url = url;
        this.data = data;
        this.title = title;
        this.width = width;
        this.height = height;
        this.eModal = $("#empty-form-modal");
        $("#empty-form-modal .header").html(this.title);
    }
    EmptyFormModelx.prototype.open = function () {
        var p = $.get(this.url, this.data, function (data) {
            $("#empty-form-modal #content-box").html(data);
        }, "html");
        p.done(function () {
            $("#empty-form-modal #save-button").off("click.emptyModalSave");
            $("#empty-form-modal #save-button").on("click.emptyModalSave", function () {
                var f = $("#empty-form-modal").find("form");
                $.validator.unobtrusive.parse(f);
                if (f.valid()) {
                    $.post(f.attr("action"), f.serialize(), function (data) {
                        $("#empty-form-modal #content-box").html(data);
                    }, "html");
                }
            });
            $("#empty-form-modal #close-button").off("click.emptyModalClose");
            $("#empty-form-modal #close-button").on("click.emptyModalClose", function () {
                $("#empty-form-modal").modal("hide");
                this.eModal = null;
            });
            $("#empty-form-modal").modal("show");
        });
        return p;
    };
    return EmptyFormModelx;
})();

html returned by ajax

<EMAIL_ADDRESS>

            <img class="img-responsive"<EMAIL_ADDRESS><EMAIL_ADDRESS>/>

Select Size

            @For Each size As EF.Size In Model.Sizes
<EMAIL_ADDRESS>@Html.RadioButton("Size", size.SizeId, False)
                @

            Next

Please select size

Close

Add to cart @ThatsFontAwesome.GetFont

(Function(f) f.CartPlus).Size(Function(f) f.Large2x).Render

Thanks

Please create a jsfiddle that reproduces your issue and a list of reproducible steps.

Jack the jsfiddle worked, so I started a process of elimination till I got down to just the bare semantic-ui  code and still had problem. I then replaced the refs to semantic css and js to those on the semantic-ui.com site and hay presto it worked.
The files I had I got out of the from the site download zip dist folder (min) and added them to my mvc.net application
I have now used bower on Visual Studio 2015 to get the files and they work.
Thanks
From: Jack Lukic<EMAIL_ADDRESS>Sent: Wednesday, 18 March 2015 12:38 AM
To: Semantic-Org/Semantic-UI
Cc: Alan Mosley
Subject: Re: [Semantic-UI] Dimmer only closes first time with modal (#1970)
Please create a jsfiddle that reproduces your issue and a list of reproducible steps.
—
Reply to this email directly or view it on GitHubhttps://github.com/Semantic-Org/Semantic-UI/issues/1970#issuecomment-82459324.

Glad it worked out! Thanks for the nice update


[code health] Improve gRPC C++ by using a more regular syntax

It's almost Java now. You mainly swap to using . instead of :: as the namespace separator in places like grpc::experimental::Interceptor.


grafana: add screenshot of dashboard

Signed-off-by: Alexander Trost<EMAIL_ADDRESS>

@galexrt looks good


Internal Server Error

https://xxxxx:2083/cpsess5536639656/3rdparty/letsencrypt-cpanel-ui/index.live.pl
Internal Server Error
500
No response from subprocess ( (cpanel)):
cpsrvd/<IP_ADDRESS> Server at xxxxx

I get the same error :(

@Tntdruid @weltonwe Could you please send the cPanel error_log file to<EMAIL_ADDRESS>

Log send

Seems to be a cPanel installation pbm. I could see lots of errors regarding perl and others.
        Cpanel::AdvConfig::apache::modules::_get_options_support() called at /usr/local/cpanel/Cpanel/GlobalCache/Build/cpanel.pm line 30
        Cpanel::GlobalCache::Build::cpanel::build() called at (eval 1) line 3
        eval '
                require Cpanel::GlobalCache::Build::cpanel;
                Cpanel::GlobalCache::Build::cpanel::build();
            ' called at bin/build_global_cache.pl line 27
[2016-11-15 01:54:02 +0000] warn [Cpanel::AdvConfig::apache::modules] Unable to locate executable Apache binary, /usr/local/apache/bin/httpd
[2016-11-15 01:54:02 +0000] warn [Cpanel::AdvConfig::apache::modules] Unable to locate executable Apache binary, /usr/local/apache/bin/httpd
[2016-11-15 01:54:02 +0000] warn [build_global_cache] Unable to locate executable Apache binary, /usr/local/apache/bin/httpd at /usr/local/cpanel/Cpanel/AdvConfig/apache/modules.pm line 272.

me too!
Also "Installing Protocol::ACME" takes for ever! Any solution?


Improve iam and security group naming convention

I'm submitting a...

[ ] bug report

[x] feature request

[ ] support request
[ ] kudos, thank you, warm fuzzy

What is the current behavior?
At the moment security groups and IAM roles are not immediately clear  on what infrastructure they belong too.  As a example if I name the cluster 'test' the cluster iam role is named something like
test201810021044448743111111112
I can understand the prefix is to prevent collisions but in my opinion it would be better to prefix it with something like eks-${var.cluster_name}-role- which would up up with something like
eks-test-role-201810021044448743111111112
Are you able to fix this problem and submit a PR? Link here if you have already.
I have not submitted a PR yet until I can get an idea of everyones opinion but will be more then happy too.
Kind Regards
Simon

I agree.
I also created a similar issue here: https://github.com/terraform-aws-modules/terraform-aws-eks/issues/88
I don't think the "prevent collisions" idea holds up, as what are the chances of you having a role of eks-cluster-name elsewhere?

Hi Max
I agree totally.  I am more than happy to submit a pull request without any prefixing.
Many thanks

I am more than happy to submit a pull request without any prefixing

Awesome 👍

Please note that changing the IAM role names will cause all existing clusters to be recreated.
A change of this magnitude should wait for a major version number change and have some form of migration path for existing clusters. Something similar to #137 for the EKS cluster resource.

I don't think the "prevent collisions" idea holds up, as what are the chances of you having a role of eks-cluster-name elsewhere?

Justification for the use of name_prefix was given here: https://github.com/terraform-aws-modules/terraform-aws-eks/issues/88#issuecomment-412424297
Currently you can use the same terraform config in multiple regions as IAM is global.

Please note that changing the IAM role names will cause all existing clusters to be recreated.

Yeah this is really annoying. You can't change the EKS cluster role without recreating it.

Currently you can use the same terraform config in multiple regions as IAM is global.

This is true. The IAM role names will be unique but the cluster names could be identical.
Any more opinions on this? Personally, I could go either way.

I could put in a variable, something like 'use_new_naming_convention' and if this is set to true create the roles and security groups with a new format.  This will allow new users to have the new naming and not break any existing clusters.
This will be pretty easy to do, but may cause some confusion.
Simon

I could put in a variable, something like use_new_naming_convention and if this is set to true create the roles and security groups with a new format. This will allow new users to have the new naming and not break any existing clusters.

Yeah would work but then we would need the IAM resources duplicated, once for old and once for new. Kinda messy.
I'm on the fence on this one 🤔

A quick way to maybe implement this could be something like this?
resource "aws_iam_role" "role" {
  name_prefix = "${var.use_new_naming_convention == 1 ? "eks-${aws_eks_cluster.this.name}-role-" : "${aws_eks_cluster.this.name}" }"
}

A quick way to maybe implement this could be something like this?

This is slightly better but will still result in names like xxxxx20180919094539278700000002. But yes, just having the hyphen and keyword there would be better I guess.

Please refer to https://github.com/terraform-aws-modules/terraform-aws-eks/issues/88


Corrected File, ContentFile, and ImageFile signatures in docs.

@adamchainz Thanks :+1: I fixed also signature of ImageFile.

Ah, well spotted 👍


[stable/airflow] add labels/annotations

What this PR does / why we need it:
Add support for custom labels and annotations on the deployments and statefulset
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

Hello. Can you rebase?

@gsemet ok, done

I am approving it. Just a question for worker: does that « annotation » duplicate « podannotation »?

/lgtm

thanks. no, the new "annotations" are set on the statefulset itself, and the "podannotations" are set on the pod templates. So at the end on different objects.

Yes that what I thought :) thanks !


Experiment/find r output files on win

Codecov Report

Merging #7 (4ddc0ac) into master (4de992c) will increase coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
+ Coverage   82.40%   82.60%   +0.20%
==========================================
  Files         244      255      +11
  Lines       56209    59674    +3465
==========================================
+ Hits        46318    49295    +2977
- Misses       9891    10379     +488

Impacted Files
Coverage Δ

rust/arrow/src/array/transform/fixed_binary.rs
0.00% <0.00%> (-84.22%)
:arrow_down:

rust/arrow/src/array/transform/null.rs
0.00% <0.00%> (-66.67%)
:arrow_down:

rust/arrow/src/array/transform/structure.rs
44.44% <0.00%> (-38.89%)
:arrow_down:

rust/arrow/src/array/transform/primitive.rs
71.42% <0.00%> (-28.58%)
:arrow_down:

rust/arrow/src/array/transform/mod.rs
70.23% <0.00%> (-22.96%)
:arrow_down:

rust/arrow/src/datatypes/native.rs
76.59% <0.00%> (-12.88%)
:arrow_down:

rust/arrow/src/array/transform/variable_size.rs
87.87% <0.00%> (-12.13%)
:arrow_down:

rust/arrow/src/array/array_binary.rs
86.05% <0.00%> (-5.70%)
:arrow_down:

rust/parquet/src/schema/parser.rs
86.27% <0.00%> (-3.93%)
:arrow_down:

rust/datafusion/src/physical_plan/group_scalar.rs
61.84% <0.00%> (-3.44%)
:arrow_down:

... and 114 more

Continue to review full report at Codecov.

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


Deprecation in pq.Quantity of the copy argument

Spotted today on CIs in MNE-Python: https://github.com/mne-tools/mne-python/actions/runs/10577451925/job/29305474492?pr=12815#step:17:5232
 neo/io/proxyobjects.py:296: in load
    anasig = AnalogSignal(
neo/core/analogsignal.py:202: in __new__
    obj = pq.Quantity(signal, units=units, dtype=dtype, copy=copy).view(cls)
quantities/quantity.py:120: in __new__
    warnings.warn(("The 'copy' argument in Quantity is deprecated and will be removed in the future. "
E   quantities.QuantitiesDeprecationWarning: The 'copy' argument in Quantity is deprecated and will be removed in the future. The argument has no effect since quantities-0.16.0 (to aid numpy-2.0 support).

@zm711 can you handle this?

Sure looks like a pretty quick fix. I'll assign myself. I'm busy today, but will work on this week!


Support for color type of input field

@KovaCro my thinking was that we would extend jp-input to allow for color as a type not create a new component. If you think this is a better approach we can leave it as a component, but let's add all of the validation options and functionalities a jp-input would have or let's fold it into jp-input.


Site design/ Site script AddSPView how to enable mobile view

Category

[x] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

Expected or Desired Behavior
I am creating a new list with a SharePoint Site Design / Site Script and create one or several custom views. I expect an (sub-)action to enable the view(s) as mobile view.
Observed Behavior
I can't find a way to activate the view as mobile view. All created views are disabled for mobile...
(https://github.com/SharePoint/sp-dev-docs/blob/master/docs/declarative-customization/site-design-json-schema.md)
Am I missing something or is there no way to add the view as mobile at the moment?
Thanks for your help!

That setting is currently not unfortunately supported by the Site Design / Site Scripts capability.


[BUG] SubscriptionResourceGetMonitorMetricsOptions does not accept more than one metrics

Library name and version
Azure.ResourceManager.Monitor.Models
Describe the bug
The SubscriptionResourceGetMonitorMetricsOptions class (used by GetMonitorMetrics(options)' method of the 'SubscriptionResource class) has a property:
        /// <summary> The names of the metrics (comma separated) to retrieve. </summary>
        public string Metricnames { get; set; }

As it seems it should get the demanded metrics based on a string like:
"Network In,Network Out" or "Network In, Network Out"
However, when I set it as:
options.Metricnames = "Network In, Network Out";

or
options.Metricnames = string.Join(", ", new[] {"Network In", "Network Out"});

I get the error:
{"error":{"additionalInfo":[{"type":"string","info":"TraceId={d696c57f-e7bf-4027-8597-7e3d334b92bb}"},{"type":"string","info":"ExceptionType=Microsoft.Online.Metrics.MetricsMP.Utilities.RPRequestFormatException"}],"code":"BadReq uest","message":"Failed to find metric configuration for provider: Microsoft.Compute, resource Type: virtualMachines, metric: Network In, Network Out, Valid metrics: Percentage CPU,Network In,Network Out,Disk Read Bytes,Disk Wri te Bytes,Disk Read Operations/Sec,Disk Write Operations/Sec,CPU Credits Remaining,CPU Credits Consumed,Data Disk Read Bytes/sec,Data Disk Write Bytes/sec,Data Disk Read Operations/Sec,Data Disk Write Operations/Sec,Data Disk Que ue Depth,Data Disk Bandwidth Consumed Percentage,Data Disk IOPS Consumed Percentage,Data Disk Target Bandwidth,Data Disk Target IOPS,Data Disk Max Burst Bandwidth,Data Disk Max Burst IOPS,Data Disk Used Burst BPS Credits Percent age,Data Disk Used Burst IO Credits Percentage,OS Disk Read Bytes/sec,OS Disk Write Bytes/sec,OS Disk Read Operations/Sec,OS Disk Write Operations/Sec,OS Disk Queue Depth,OS Disk Bandwidth Consumed Percentage,OS Disk IOPS Consum ed Percentage,OS Disk Target Bandwidth,OS Disk Target IOPS,OS Disk Max Burst Bandwidth,OS Disk Max Burst IOPS,OS Disk Used Burst BPS Credits Percentage,OS Disk Used Burst IO Credits Percentage,Premium Data Disk Cache Read Hit,Pr emium Data Disk Cache Read Miss,Premium OS Disk Cache Read Hit,Premium OS Disk Cache Read Miss,VM Cached Bandwidth Consumed Percentage,VM Cached IOPS Consumed Percentage,VM Uncached Bandwidth Consumed Percentage,VM Uncached IOPS  Consumed Percentage,Available Memory Bytes,VmAvailabilityMetric,VM Remote Used Burst IO Credits Percentage,VM Remote Used Burst BPS Credits Percentage,VM Local Used Burst IO Credits Percentage,VM Local Used Burst BPS Credits Pe rcentage"}}
When I use just one metric, like:
options.Metricnames = "Network In";

or
options.Metricnames = "Network Out";

everything works as it should (it gets the required metrics).
Retrieving multiple metrics with a corresponding number of requests works as well.
I assume there is no option of retrieving multiple metrics in one request.
Expected behavior
According to the documentation it should be possible to retrieve multiple metrics from Azure with a single request, by usign a single string with comma separated names of metrics, like:
"Percentage CPU,Network In,Network Out,Disk Read Bytes"
Actual behavior
It gives an error:
`{"error":{"additionalInfo":[{"type":"string","info":"TraceId={d696c57f-e7bf-4027-8597-7e3d334b92bb}"},{"type":"string","info":"ExceptionType=Microsoft.Online.Metrics.MetricsMP.Utilities.RPRequestFormatException"}],"code":"BadReq
uest","message":"Failed to find metric configuration for provider: Microsoft.Compute, resource Type: virtualMachines, metric: Network In, Network Out, Valid metrics: Percentage CPU,Network In,Network Out,Disk Read Bytes,Disk Wri
te Bytes,Disk Read Operations/Sec,Disk Write Operations/Sec,CPU Credits Remaining,CPU Credits Consumed,Data Disk Read Bytes/sec,Data Disk Write Bytes/sec,Data Disk Read Operations/Sec,Data Disk Write Operations/Sec,Data Disk Que
ue Depth,Data Disk Bandwidth Consumed Percentage,Data Disk IOPS Consumed Percentage,Data Disk Target Bandwidth,Data Disk Target IOPS,Data Disk Max Burst Bandwidth,Data Disk Max Burst IOPS,Data Disk Used Burst BPS Credits Percent
age,Data Disk Used Burst IO Credits Percentage,OS Disk Read Bytes/sec,OS Disk Write Bytes/sec,OS Disk Read Operations/Sec,OS Disk Write Operations/Sec,OS Disk Queue Depth,OS Disk Bandwidth Consumed Percentage,OS Disk IOPS Consum
ed Percentage,OS Disk Target Bandwidth,OS Disk Target IOPS,OS Disk Max Burst Bandwidth,OS Disk Max Burst IOPS,OS Disk Used Burst BPS Credits Percentage,OS Disk Used Burst IO Credits Percentage,Premium Data Disk Cache Read Hit,Pr
emium Data Disk Cache Read Miss,Premium OS Disk Cache Read Hit,Premium OS Disk Cache Read Miss,VM Cached Bandwidth Consumed Percentage,VM Cached IOPS Consumed Percentage,VM Uncached Bandwidth Consumed Percentage,VM Uncached IOPS
Consumed Percentage,Available Memory Bytes,VmAvailabilityMetric,VM Remote Used Burst IO Credits Percentage,VM Remote Used Burst BPS Credits Percentage,VM Local Used Burst IO Credits Percentage,VM Local Used Burst BPS Credits Pe
rcentage"}}
Reproduction Steps
async Task CollectMetricsForVirtualMachines(string subscriptionId, string location)
{
        var startDate = DateTime.UtcNow.AddDays(-10);
        var stopDate = DateTime.UtcNow;
        var options = new SubscriptionResourceGetMonitorMetricsOptions(location);
        options.Metricnames = "Percentage CPU,Network In,Network Out,Disk Read Bytes";
        options.Timespan = $"{startDate:o}/{stopDate:o}";
        options.Metricnamespace = "Microsoft.Compute/virtualMachines";

        var resourceIdentifier = SubscriptionResource.CreateResourceIdentifier(subscriptionId);
        SubscriptionResource subscription = await client.GetSubscriptionResource(resourceIdentifier).GetAsync();

        var metrics = subscription.GetMonitorMetrics(options).ToList();
}

Environment
.NET SDK:
Version:   8.0.100-preview.4.23260.5
Commit:    2268e7b15c
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19045
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\8.0.100-preview.4.23260.5\
.NET workloads installed:
There are no installed workloads to display.
Host:
Version:      8.0.0-preview.4.23259.5
Architecture: x64
Commit:       84a3d0e37e
.NET SDKs installed:
6.0.410 [C:\Program Files\dotnet\sdk]
7.0.200 [C:\Program Files\dotnet\sdk]
7.0.203 [C:\Program Files\dotnet\sdk]
7.0.302 [C:\Program Files\dotnet\sdk]
8.0.100-preview.4.23260.5 [C:\Program Files\dotnet\sdk]

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Sure, thanks

Hi @af-af-af,
I used your provided reproduction code, and it worked well in both Azure.ResourceManager.Monitor 1.1.0 and 1.2.0.

Could you please provide additional details to help me reproduce the issue you're facing?

Hi @dvbb, I double checked, now I could not reproduce it as well. I guess we can close the bug.


Divide by 0 bug

Agents position is set to NaN/ NaN when attempting to go to a node they are already at.

Fixed, added mistakenly missed bugfix


No close/minimize buttons in VS Code Insiders

I just updated VS Code Insiders and the close/minimize buttons have disappeared.

Version: 1.74.0-insider (system setup)
Commit: 86f0f9bfa268fe21af7efae37d8f0da1dba8137f
Date: 2022-11-03T05:36:55.498Z
OS: Windows_NT x64 10.0.19044

Same: #165395

/duplicate - see https://github.com/microsoft/vscode/issues/165337#issuecomment-1302040524 for workaround.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


fix(files): Ignore included:false pattern

Do not include a file, when include=false.
Give include=false precedence over include=true.
Closes #1530

@dignifiedquire There's an integration test that fails, because it assumes some other semantic than changed precedence. Should the test be changed or should the semantics?

@budde377 right, so we have to think about this carefully, because in the 0.12 and before the semantics where that the most specific pattern would match, rather than false > true. So ideally I thought we could revert to that, what do you think?

@dignifiedquire. Lets do this the cool way! Inspired by the CSS specificity calculations, we should define a specificity-measure. I propose the following:
Minimatch supports:

Globstar
Wildcard
Extended glob matching
Brace expansion

Given a path, p, we can define a tuple s(p) = (ng, nw, ne, nb) where ng, nw, ne, and nb are the number of globstars, wildcards, extended glob match patterns, and brace expansion patterns, respectively.
With any two paths, p and q, where s(p)=(p1, p2, p3, p4) and s(q)=(q1, q2, q3, q4) we can define a order of the corresponding tuples
p1 < p2 ⇔ ∃ m > 0: ∀ i > m: pi = qi ∧ pm < qm
i.e. an lexicographical order.
Given two paths with different tuples, we can define the one with the smallest tuple (wrt. the order above) as the most specific. If the tuples are equal we can choose either one.

Or we could just let the number of files generated from a given pattern decide how specific it is. The more files, the less specific.

@budde377 :100: for the cool order you suggested above. We just need to make sure to document and test this well, so people understand what's happening :)

@dignifiedquire Yes. I'll start on it this afternoon.

After some failed attempts, I'm pretty sure that I have a solution. I'll have it implemented and tested primo next week.

Thanks @budde377 for the great work, looking forward to see it :)

@dignifiedquire I've now implemented an order. Please let me know if it needs more documentation etc.

Nice work, will review more closely tomorrow

Thanks @budde377 looking good, going to merge this and hope for the best :)


new test: check that an object uploaded with multipar have only 1 version (with aws s3api list-object-versions)

dupe of #17


stats layout

stats should/will include:
employee/employees assigned to day,
assigned/unassigned jobs,
hours assigned,
assigned employee(s) present,

how to present stats about tasks and employee details.

need to decide on functionality of the fields and button(s). if we want the fields to be dropdowns, the pictures function will be hover. if we want expanded info on click we will need a second button, the first existing button being "new", the second being "edit".

for now the stats are: emp names, tasks, and who is assigned to what.


N-bit binary numbers having more 1s than 0s using recursion in c++

🚀 Feature
(A clear and concise description of what the feature is.)
Have you read the Contributing Guidelines on Pull Requests?
Yes
##Question in brief
Given a positive integer n,
print all n-bit binary numbers having more 1’s than 0’s for any prefix of the number.

Please assign this to me


Pressing j or k in nvim after resizing tmux pane undos the resize

Issue description
After resizing the tmux pane fully (with Z option) I'm often going to a pane with neovim. In this pane I directly start navigating with j and k. However, this results in undoing the resizing of the pane. It seems to me like the resizing should only be undone with another resize command.
Required information
Please provide the following information:

tmux version: tmux 3.3a
Platform: Darwin i386
$TERM inside and outside of tmux: screen-256color

Tmux conf
set -g default-terminal "screen-255color"

set -g prefix C-a
unbind C-b
bind-key C-a send-prefix

# Split horizontally and open new pane in same path
unbind %
bind | split-window -h -c "#{pane_current_path}"

# Split vertically and open new pane in same path
unbind '"'
bind - split-window -v -c "#{pane_current_path}"

# Open new pane in same path
bind c new-window -c "#{pane_current_path}"

unbind r
bind r source-file ~/.tmux.conf

bind -r j resize-pane -D 5
bind -r k resize-pane -U 5
bind -r l resize-pane -R 5
bind -r h resize-pane -L 5

bind -r m resize-pane -Z

set -g mouse on
setw -g mouse on

set-window-option -g mode-keys vi
set-option -sg escape-time 10
set-option -g focus-events on
set-option -sa terminal-overrides ',XXX:RGB'

bind-key -T copy-mode-vi 'v' send -X begin-selection # start selecting text with "v"
bind-key -T copy-mode-vi 'y' send -X copy-selection # copy text with "y"

unbind -T copy-mode-vi MouseDragEnd1Pane # don't exit copy mode after dragging with mouse

# tpm plugin
set -g @plugin 'tmux-plugins/tpm'

# list of tmux plugins
set -g @plugin 'christoomey/vim-tmux-navigator' # for navigating panes and vim/nvim with Ctrl-hjkl
set -g @plugin 'jimeh/tmux-themepack' # to configure tmux theme
set -g @plugin 'tmux-plugins/tmux-resurrect' # persist tmux sessions after computer restart
set -g @plugin 'tmux-plugins/tmux-continuum' # automatically saves sessions for you every 15 minutes

set -g @themepack 'powerline/default/cyan' # use this theme for tmux

set -g @resurrect-capture-pane-contents 'on' # allow tmux-ressurect to capture pane contents
set -g @continuum-restore 'on' # enable tmux-continuum functionality

# Initialize TMUX plugin manager (keep this line at the very bottom of tmux.conf)
run '~/.tmux/plugins/tpm/tpm'

I believe the behavior you’re describing is standard tmux behavior and not something that this plugin can change.
To be more specific, if you zoom a tmux pane and then use the navigation keys (C-hjkl) then tmux will automatically unhook the pane.
Note: if you zoom a tmux pane that contains a vim session, you should be able navigate between vim windows within that tmux pane without unzooming, but if you navigate past the edge of the vim windows then tmux will unzoom. There is no logic to “wrap” or “trap” within the zoomed session.
hopefully that clarifies things, but I think given the above notes that this is working as intended and so I’m going to close this now.


Error: [node-persist][readFile] /Users/username/.homebridge/homebridge-lg-thinq/persist/devices/device-id does not look like a valid storage file!

Noticed this repeating error message.  A reboot of the server, the plugins and clearing of cache didn't resolve.   Happens every 5-10 minutes starting today.
On 2.0.0-beta 1
Error: [node-persist][readFile] /Users/*username*/.homebridge/homebridge-lg-thinq/persist/devices/*device-id* does not look like a valid storage file!
    at /usr/local/lib/node_modules/homebridge-lg-thinq/node_modules/node-persist/src/local-storage.js:339:89
    at FSReqCallback.readFileAfterClose [as oncomplete] (node:internal/fs/read/context:68:3)
[11/15/2024, 4:34:29 PM] [homebridge-lg-thinq] Child bridge process ended
[11/15/2024, 4:34:29 PM] [homebridge-lg-thinq] Process Ended. Code: 1, Signal: null
[11/15/2024, 4:34:36 PM] [homebridge-lg-thinq] Restarting Process...
[11/15/2024, 4:34:36 PM] [homebridge-lg-thinq] Launched child bridge with PID 3991
[11/15/2024, 4:34:36 PM] Registering platform 'homebridge-lg-thinq.LGThinQ'
[11/15/2024, 4:34:36 PM] [homebridge-lg-thinq] Loaded homebridge-lg-thinq v2.0.0-beta.1 child bridge successfully
[11/15/2024, 4:34:36 PM] Loaded 1 cached accessories from cachedAccessories.*id*.
[11/15/2024, 4:34:36 PM] [homebridge-lg-thinq] Loading accessory from Homebridge cache: Air Conditioner

Quick update, I found this plug-in has a node.js version dependency (18).  There are several plugins with this dependency.  This instance was not obvious in the log data.  It caused a child bridge to restart frequently and created some odd errors in the debug (-NodeJS:DataStream*,HAP-NodeJS:*)
I suspect this was related.  If there is a timeline on node.js compatibility, would be great to know when.


Readme updates for Static web apps

Can the readme change the wording in the fourth bullet so that static apps refer to static web apps - which has a different extension? Then add another bullet point for Storage else where? Or could I submit a pr?

Closed via https://github.com/microsoft/vscode-node-azure-pack/pull/46


subprocess.CalledProcessError: Command returned non-zero exit status -11

subprocess.CalledProcessError: Command returned non-zero exit status -11

did you deal with it?I meet a same problem with you. If you know how to do it please tell me when you see it,thank you very much


Add support for Playwright

Add support for Playwright

Check out https://nextjs.org/docs/pages/building-your-application/optimizing/testing as Lee mentioned


Ex Nihilo Sieve Recipes in Ex Astris

hey,
with version

ModTweaker-2-0.9.1
Ex-Astris-MC1.7.10-1.16-36
Ex-Nihilo-1.38-44

Custom Ex Nihilo recipes no longer work with Ex Astris.
For example adding this recipe:
mods.exnihilo.Sieve.addRecipe(<GalacticraftCore:tile.moonBlock:3>, <minecraft:blaze_rod>, 80);
It is possible to sieve it with Ex Nihilo sieves but not with the auto sieve from Ex Astris

I changed the way the handlers apply the recipes in the New ModTweaker versions. I'll check if we can fix this.

That would be great, thanks.

I'll guess it is because Ex-Astris caches which items are allowed or not allowed to be sieved. Have you tried reloading minecraft or did you change the recipe in-game via /mt reload?

I didn't change the recipes and also didn't reload it.

@Zixxl any news to this?

Ex nihilo caches their recipes, nothing we can do about it.
On Tuesday, August 4, 2015, SirWilli<EMAIL_ADDRESS>wrote:

@Zixxl https://github.com/Zixxl any news to this?
—
Reply to this email directly or view it on GitHub
https://github.com/jaredlll08/ModTweaker2/issues/211#issuecomment-127417851
.

What do you mean?

Sorry, I meant that ex astris caches them, basicly, when the game loads,
they copy the list that has all the recipes, but we only edit the list
later.
On Tuesday, August 4, 2015, SirWilli<EMAIL_ADDRESS>wrote:

What do you mean?
—
Reply to this email directly or view it on GitHub
https://github.com/jaredlll08/ModTweaker2/issues/211#issuecomment-127504560
.

Okay, created an issue there: https://github.com/Ex-Astris-Rebirth/Ex-Astris-Rebirth/issues/59

Are there any work arrounds for this or is this just something we need to wait for the Ex Astris dev to fix?

Wait for Ex nihilo dev to fix.

Ex Astris does not cache sieve recipes, and has not done for a while. They are pulled directly from SieveRegistry.siftables in Ex Nihilo.

Looks like that this feature get broken between ModTweaker-2-0.8.3 and ModTweaker-2-0.9.0.
Version ModTweaker-2-0.8.3 still works well with all other plugins.
Probably this can be fixed or workaround on ModTweaker side? This is so annoying that you cannot use this plugins together.

Btw, Hammer recipes working good in AutoHammer from Ex Astrisю
Also I have tested other version of Ex Astris: Ex-Astris-Rebirth https://github.com/Ex-Astris-Rebirth/Ex-Astris-Rebirth and it has exactly the same issue.

Also having this issue with
Ex-Astris-MC1.7.10-1.16-36
MineTweaker3-1.7.10-3.0.10B
ModTweaker2-0.9.5

Closed due MC1.7.10 end of life.


bug: cacheTime for hooks doesn't defaults to zero as it's mention in the docs

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
1.0.5
Current Behavior
As I am passing the args to the hooks useContractRead and useContractReads and I don't need the information to be cached (should be the latest information), even if I am not passing the cacheTime, there is still some persistent information in the queryCache in the localStorage, only when I pass a cacheTime: 0, the information is not persistent anymore.
I don't if it's a bug in the hooks, or if it's intended, but is not clear if this defaults to zero.
Expected Behavior
Hooks useContractRead and useContractReads and others should have cacheTime to zero default and not persist information until is clearly intended in the cacheTime.
Steps To Reproduce
No response
Link to Minimal Reproducible Example (StackBlitz, CodeSandbox, GitHub repo etc.)
No response
Anything else?
No response

The intended behavior is the persist the data according to the cacheTime set on the global query client – this is defaulted to 24 hours. You can change this by passing in a custom queryClient to createConfig with your custom cacheTime.

Man, I spent so long trying to debug this. I'm gonna come back and add this to the docs.


Improve UX for Resuming Old Dat

Make it clear when you are resuming a previously created Dat (share or download).

also would be cool: dat resume dl/

i think a person shouldn't have to know whether or not it was a dat already, because you're still just sharing folder contents and getting the link.

You can just do dat <folder> now and it'll resume for you


Publish 02/09/2022, 10:30 AM

Docs Build status updates of commit 626f02a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Set-SPOTenant.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


feat: expose PHP version

Add a new function to retrieve the PHP version and display the used version in the logs of the Caddy module.

We should use ldflags to do that. If we manage to switch to GoReleaser (#82), we could use its native feature: https://goreleaser.com/cookbooks/using-main.version


vcpkg, msbuild and manifest on ARM64 fails with wrong triplet

Describe the bug
Using vcpkg with msbuild and manifest file on ARM64 builds fails with an error about invalid ARM64-windows triplet
Environment
Windows 11 on ARM64
latest VS2022 preview
Workaround
Add
<VcpkgPlatformTarget Condition="'$(Platform)' == 'ARM64'">arm64</VcpkgPlatformTarget>
to line 58 of vcpkg.targets.

I guess this solved by https://github.com/microsoft/vcpkg/pull/42182.


Please consider supporting names as well as IP address

Im using the DebugViewRemoteClient because I have two computers on my development site.  Its very handy to have DebugView++ running on the other computer showing all messages and still being able to run my test subject in fullscreen on my main computer

I somehow missed this request and now suddenly noticed it. I'm sorry about that. Where did you suggest to add also hostnames exactly ?

OMG :D
This was a long time ago and I havent been using DebugView++ in  quite a while.  Not sure I remember this anymore, but lets make a guess
Can it bee that adding a remote client demands knowledge of the IP address?
If so, I probably asked you to redesign the settings to also support host names.  You know, normally a client computer is getting the IP from a DHCP server... looking up the IP address using DNS or something like that is quite usefull in those situations..


Add documentation for "global" commands

Make a webpage for the bot?

We should put it on the club website

Done: https://longhornriichi.com/ronhorn/


How to let enter key for next cell in edit mode?

It's need to use the mouse to click  to go to next cell in edit mode, can I just use the enter key for next cell (like in excel) ?
Thanks!!

In v3, both enter and tab will go to the cell below or next respectively.

I am using V3 but I mean when I editing the cell it can't use the enter to go to next one :(
if I just click the cell with mouse not in edit mode, then it can go to next by enter...

any updates?
thanks!

I have tested that and when press enter I can jump to the cell below. If I press tab I go to the cell in the right as expected.

oh~~~sorry, I tried in your demo also is working, hum...maybe there are some other issue in my project, because I uses a lot of other plugins in my project :(
ok, I will check it myself, thanks a lot!

ok, I found that's why I have that problem, if I set to use the custom column, then will be cacuse my issue, and I also try your demo here, it also can't use the enter or tab key
another difference in the custom column, if you edit it and it will just append the content and not replace it.

I really want to solve this problem, thanks a lot!


Separate bucket calculation logic from scanner

Separate bucket calculation logic from scanner. Feel free to let me know if you have any request for amendment/alteration.
Fixes #196

Pull Request Test Coverage Report for Build 1420

15 of 15 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 82.257%

Totals

Change from base Build 1417:
-0.4%

Covered Lines:
602

Relevant Lines:
686

💛  - Coveralls

Pull Request Test Coverage Report for Build 1420

15 of 15 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 82.257%

Totals

Change from base Build 1417:
-0.4%

Covered Lines:
602

Relevant Lines:
686

💛  - Coveralls

Pull Request Test Coverage Report for Build 1420

15 of 15 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 82.257%

Totals

Change from base Build 1417:
-0.4%

Covered Lines:
602

Relevant Lines:
686

💛  - Coveralls

Looks good.
Would be even better if you, create an interface for BucketCalc with getBuckets(): Promise<IBuckets> and use IBucketCalc. Also since only getBuckets(): Promise<IBuckets> is used outside the class, rest should be set to private (that will require some tests rewrites)

Ok, I have made the requested changes and rebased.


COLLECTIONS-870: Added ExtendedIterator and tests

Implement COLLECTIONS-870
Adds an ExtendedIterator as defined in COLLECTIONS-870 as well as in the discussion on the mailing list.
mailing list discussion: https://lists.apache.org/thread/rpcsoz3jwz7x9sp3cgzf315ls7pcd90w

I responded to this on github but it does not seem to have come through
here.
On Tue 22 Oct 2024, 12:20 Gary Gregory, @.***> wrote:

@.**** commented on this pull request.
In
src/main/java/org/apache/commons/collections4/iterators/ExtendedIterator.java
https://github.com/apache/commons-collections/pull/564#discussion_r1810533509
:

/**

* Create an ExtendedIterator returning the elements of it.

* If it is itself an ExtendedIterator, return that;

* otherwise wrap it.

* @param it The iterator to wrap.

* @return An Extended iterator wrapping<EMAIL_ADDRESS>it}

*/

public static  ExtendedIterator create(final Iterator it) {

   return it instanceof ExtendedIterator<?>

           ? (ExtendedIterator<T>) it

           : new ExtendedIterator<>(it, false);

}

/** the base iterator that we wrap */
private final Iterator<? extends T> base;

Hi @Claudenw https://github.com/Claudenw
You wrote:
If we do this then why not expose setFunction() and something else for
andThen().
Which is why I asked about setFunction(). Maybe I'm the one missing
something? 🤔
—
Reply to this email directly, view it on GitHub
https://github.com/apache/commons-collections/pull/564#discussion_r1810533509,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASTVHWDUPBUIJH4RO33A73Z4YYGHAVCNFSM6AAAAABQIKXUCCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGOBUHA4DOMZRHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


chore(main): release 1.0.1

:robot: I have created a release beep boop
1.0.1 (2024-07-30)
Bug Fixes

release with also generate major version (523c6b5)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.1
:sunflower:


Update govuk elements and assets symlinks

@robmckinnon What do you think? good to go? We could really do with some tests for this gem.
linked to #4

Thanks for the pull request. As we discussed, the plan is to not commit any asset files directly into this repository.
I've created a separate pull request to upgrade to govuk_elements v1.1.2 keeping the symlink approach to pulling in asset files: https://github.com/ministryofjustice/govuk_elements_rails/pull/6


ADM-628:[docs] update known issues about readme and add tech spike result of impact regarding status and column name change

Summary
...
Before
Description
Screenshots
If applicable, add screenshots to help explain behavior of your code.
After
Description
Screenshots
If applicable, add screenshots to help explain behavior of your code.
Note
Null

Hey, @mikeyangyun add your file path here docs/src/i18n/en/nav.ts to display your page

Already add the file path.


fix tree heights file

@gonzalezeb and @mcgregorian1,  our tree height file has a problem -- sources/ researchers and their respective DOIs do not match. This should be easy to fix, but I'm not sure which columns to trust. Perhaps an older version of the file has this correct?

I will look into this today

@teixeirak I've gone through Atticus' paper again and there is no mention of ground-truthed measurements, nor the methodology he described via email about using the clinometer. As this isn't part of the paper, the raw data file for the measurements we have in this heights.csv must be from somewhere on a local drive. Do you have an idea where?
Otherwise, I've updated his doi and removed the wrong sources for him, but I kept yours and Thompson's the same. I also updated the doc with the methods used and put some notes.

Thanks! I sent you Atticus’ original file by email.


Can't paste into jupyter widget text cell

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
I can't paste into widgets.Text. I have a value in my clipboard but I can't copy or paste from widgets

VS Code Version
1.86
Jupyter Extension Version
v2024.1.0
Jupyter logs
Visual Studio Code (1.86.0, undefined, desktop)
Jupyter Extension Version: 2024.1.0.
Python Extension Version: 2024.0.1.
Pylance Extension Version: 2023.12.1.
Platform: darwin (arm64).
Workspace folder ~/dev/atomicfi/snoopy, Home = /Users/devon
10:43:12.693 [info] Start refreshing Kernel Picker<PHONE_NUMBER>693)
10:43:12.700 [info] Using Pylance
10:43:12.733 [info] Start refreshing Interpreter Kernel Picker
10:43:13.066 [info] Starting Kernel startUsingPythonInterpreter, .jvsc74a57bd0a42c715be713525d15fd29f5663eeb051c39a3e7bcf602f9bd56dec31009c098.~/miniconda3/envs/snoopy/python.~/miniconda3/envs/snoopy/python.-m#ipykernel_launcher  (Python Path: ~/miniconda3/envs/snoopy/bin/python, Conda, snoopy, 3.11.0) for 'Untitled-1.ipynb' (disableUI=true)
10:43:15.813 [info] Process Execution: ~/miniconda3/envs/snoopy/bin/python -c "import ipykernel; print(ipykernel.__version__); print("5dc3a68c-e34e-4080-9c3e-2a532b2ccb4d"); print(ipykernel.__file__)"
10:43:15.816 [info] Process Execution: ~/miniconda3/envs/snoopy/bin/python -m ipykernel_launcher --f=~/Library/Jupyter/runtime/kernel-v2-9014JEULwNNorgxa.json
    > cwd: ~/dev/atomicfi/snoopy
10:43:15.817 [info] Kernel process 9187.
10:43:15.917 [info] Process Execution: ~/miniconda3/envs/snoopy/bin/python -m pip list
10:43:16.669 [warn] StdErr from Kernel Process This version of python seems to be incorrectly compiled
(internal generated filenames are not absolute).
This may make the debugger miss breakpoints.
Related bug: http://bugs.python.org/issue1666807
10:43:17.546 [info] Handle Execution of Cells 4 for Untitled-1.ipynb
10:43:18.451 [info] End refreshing Kernel Picker<PHONE_NUMBER>693)
10:43:18.524 [info] Kernel acknowledged execution of cell 4 @<PHONE_NUMBER>524
10:43:18.534 [info] End cell 4 execution after 0.01s, completed @<PHONE_NUMBER>534, started @<PHONE_NUMBER>524
10:43:19.613 [info] Process Execution: ~/miniconda3/envs/snoopy/bin/python ~/.vscode/extensions/ms-toolsai.jupyter-2024.1.0-darwin-arm64/pythonFiles/printJupyterDataDir.py
10:43:19.694 [warn] Got a non-existent Jupyter Data Dir file://~/.local/share/jupyter
10:43:32.999 [info] Restart requested Untitled-1.ipynb
10:43:33.000 [info] Dispose Kernel process 9187.
10:43:33.013 [info] Process Execution: ~/miniconda3/envs/snoopy/bin/python -c "import ipykernel; print(ipykernel.__version__); print("5dc3a68c-e34e-4080-9c3e-2a532b2ccb4d"); print(ipykernel.__file__)"
10:43:33.021 [info] Process Execution: ~/miniconda3/envs/snoopy/bin/python -m ipykernel_launcher --f=~/Library/Jupyter/runtime/kernel-v2-9014QhnUuQM0iLIy.json
    > cwd: ~/dev/atomicfi/snoopy
10:43:33.021 [info] Kernel process 9410.
10:43:33.285 [warn] StdErr from Kernel Process This version of python seems to be incorrectly compiled
(internal generated filenames are not absolute).
This may make the debugger miss breakpoints.
Related bug: http://bugs.python.org/issue1666807
10:43:33.970 [info] Restarted a32747d0-7803-4275-a2f5-dcc7ed57e9cc
10:43:34.724 [info] Handle Execution of Cells 4 for Untitled-1.ipynb
10:43:34.743 [info] Kernel acknowledged execution of cell 4 @<PHONE_NUMBER>743
10:43:34.745 [info] End cell 4 execution after 0.002s, completed @<PHONE_NUMBER>745, started @<PHONE_NUMBER>743

Coding Language and Runtime Version
python 3.11.0
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
miniconda 23.1.0 mac arm
Running Jupyter locally or remotely?
Local

@amunger re-assigning as this is related to a core feature that applies to HTML outputs.

I'm not able to repro on Windows.
Can you paste into a field in the html renderer? e.g.
from IPython.display import HTML
HTML('<input type="text" placeholder="Enter your name" /><')

Also, can you try running Developer: Toggle Keyboard Shortcuts Troubleshooting and see if something is trying to act on that keyboard action?

Developer: Toggle Keyboard Shortcuts Troubleshooting does not show anything
HTML('<input type="text" placeholder="Enter your name" /><')```
paste does work in this input.

When I type cmd+v to the ipywidgets text, this is the output from Keyboard Shortcuts Troubleshooting
```2024-02-08 09:55:38.107 [info] [KeybindingService]: + Storing single modifier for possible chord meta.
2024-02-08 09:55:38.409 [info] [KeybindingService]: + Clearing single modifier due to 300ms elapsed.```

When I type cmd+v in the html input this is the output
```2024-02-08 09:56:35.084 [info] [KeybindingService]: / Received  keydown event - modifiers: [meta], code: MetaRight, keyCode: 93, key: Meta
2024-02-08 09:56:35.086 [info] [KeybindingService]: | Converted keydown event - modifiers: [meta], code: MetaRight, keyCode: 57 ('Meta')
2024-02-08 09:56:35.089 [info] [KeybindingService]: \ Keyboard event cannot be dispatched in keydown phase.
2024-02-08 09:56:35.327 [info] [KeybindingService]: / Received  keydown event - modifiers: [meta], code: KeyV, keyCode: 86, key: v
2024-02-08 09:56:35.327 [info] [KeybindingService]: | Converted keydown event - modifiers: [meta], code: KeyV, keyCode: 52 ('V')
2024-02-08 09:56:35.327 [info] [KeybindingService]: | Resolving meta+[KeyV]
2024-02-08 09:56:35.328 [info] [KeybindingService]: \ From 4 keybinding entries, matched editor.action.clipboardPasteAction, when: no when condition, source: built-in.
2024-02-08 09:56:35.328 [info] [KeybindingService]: + Invoking command editor.action.clipboardPasteAction.
2024-02-08 09:56:35.496 [info] [KeybindingService]: + Storing single modifier for possible chord meta.
2024-02-08 09:56:35.798 [info] [KeybindingService]: + Clearing single modifier due to 300ms elapsed.```

ok, that's exactly what I see in the logs as well.
Feels like a long shot, but can you open the developer tools and watch for any errors in the console when you try to paste into either of those fields: Developer: Toggle Developer Tools

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


The --device option fails on command line in v53.0

Is this a bug report?
Yes
Have you read the Contributing Guidelines?
Yes
Environment
Environment:
OS: macOS High Sierra 10.13.3
Node: 8.9.1
Yarn: Not Found
npm: 5.5.1
Watchman: Not Found
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: 3.0 AI-171.4443003
Packages: (wanted => installed)
react: 16.2.0 => 16.2.0
react-native: 0.53.0 => 0.53.0
Steps to Reproduce

Initiate a new React Native project using react-native init projectName
Run the command react-native run-ios --device deviceName

Expected Behavior
The app should have launched on my physical device. Additionally, if I already have an instance of Metro Bundler running using react-native start, it should just hook into that one and use that.
Actual Behavior
Metro Bundler opens and complains that Metro Bundler can't listen on port undefined. It then prompts the user to hit enter because the process has terminated. I've run into other random errors too. I've included one of those here as well.
Metro Bundler error 1
Metro Bundler error 2
Additionally, there is a lot of output in my main terminal about the Xcode build failing. I can include the full output in a comment below if requested.
Build error 1
Build error 2
Even if I try running an instance of Metro Bundler using react-native start before running react-native run-ios --device deviceName, it just starts a new instance in a new terminal window.
Reproducible Demo
You can reproduce this with a brand new project created using react-native init projectName.

I have the same issue

Same issue too

This has already been fixed for me. Update to the latest react-native.

@Gulci Which version fixed the error? I have the same error with "react-native": "0.54.0". Running from Xcode works fine. The error appears only when I'm trying to run from the terminal with react-native run-ios --device="Mihai"

@mlazari
Lol wow. Want me to open a new issue? Yeah I was on 0.53.3.

@Gulci
Haha if you want yes :)
I have the same problem on the 0.54.0

I'll open the exact same issue. ;)

I have the same issue v0.55.0-rc . I'm on
macOS High Sierra 10.13.3
yarn 1.3.2
Problem does not exists on v0.52
So I'm guessing this issue introduced in v0.53

@tintorres the fix for this issue is on 0.55-stable branch so technically, there should be no errors on latest RC. Please double check.
To everyone else in this thread, this has been fixed in 0.55.0-rc.0 and will soon be available in the stable channel.


Intermittent FAIL_INVALID failures on eth_call

Description
The relay is occasionally experience errors on consensus node calls with the error
transaction @ failed precheck with status FAIL_INVALID
Steps to reproduce
Make the occasional eth_call
More info to come
Additional context
No response
Hedera network
other
Version
v0.22.0
Operating system
None

No occurrences on previewnet, testnet, or mainnet lately.


Unable to find long double symbols on aarch64-apple-darwin

I am working on https://github.com/rust-lang/rust/pull/114607 and unfortunately can't build on my system due to linker errors:
  = note: ld: warning: ignoring duplicate libraries: '-lSystem'
          ld: Undefined symbols:
            ___addtf3, referenced from:
                std::num::test_num::h83194c034ce7125a in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.12.rcgu.o
            ___divtf3, referenced from:
                std::num::test_num::h83194c034ce7125a in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.12.rcgu.o
            ___multf3, referenced from:
                std::num::test_num::h83194c034ce7125a in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.12.rcgu.o
            ___subtf3, referenced from:
                std::num::test_num::h83194c034ce7125a in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.12.rcgu.o
            ___trunctfdf2, referenced from:
                _$LT$$RF$T$u20$as$u20$core..fmt..Debug$GT$::fmt::h8dd8a3b2b4bc54ac in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.06.rcgu.o
                core::fmt::float::_$LT$impl$u20$core..fmt..Debug$u20$for$u20$f128$GT$::fmt::h0686fe3ef7164d45 in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.10.rcgu.o
            ___unordtf2, referenced from:
                std::num::test_num::h83194c034ce7125a in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.12.rcgu.o
                std::num::test_num::h83194c034ce7125a in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.12.rcgu.o
                std::num::test_num::h83194c034ce7125a in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.12.rcgu.o
                std::num::test_num::h83194c034ce7125a in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.12.rcgu.o
                std::num::test_num::h83194c034ce7125a in std-a7f7c4c812353510.std.3e4ab7aa3f61e492-cgu.12.rcgu.o
          clang: error: linker command failed with exit code 1 (use -v to see invocation)
---
          ld: Undefined symbols:
            ___extenddftf2, referenced from:
                __RNvNtCsgSq1suy33ib_12clippy_utils6consts19lit_to_mir_constant in libclippy_utils-52261686d0193f09.rlib[10](clippy_utils-52261686d0193f09.clippy_utils.c495e003dc0e053d-cgu.07.rcgu.o)
            ___trunctfdf2, referenced from:
                __RINvXs1_NtCsgSq1suy33ib_12clippy_utils6constsNtB6_8ConstantNtNtCsdSdPP26wXhD_4core4hash4Hash4hashNtCsgzWzvnNrs6k_10rustc_hash8FxHasherEB8_ in libclippy_utils-52261686d0193f09.rlib[10](clippy_utils-52261686d0193f09.clippy_utils.c495e003dc0e053d-cgu.07.rcgu.o)

It seems like a fix similar to https://github.com/rust-lang/libc/pull/1034 may work, I can submit a PR if this seems correct. @Amanieu

If we are supporting these types directly in Rust then we should add these intrinsics to compiler-builtins. For now you can just change the compiler-builtins build script to include the versions from compiler-rt, but eventually we will want to have these fully implemented in Rust.

@Amanieu are you able to move this issue to compiler-builtins?
It seems like they should be getting included already, so I'm not sure what is going on https://github.com/rust-lang/compiler-builtins/blob/47e64bfa0a387c811e4c409959a4d2668f0b1c37/build.rs#L506C20-L512. This is on aarch64-apple-darwin, I don't have any reason to think +nofp is set


english short words infer error

run inference code as shown below:
ref_path="female01.wav"
cosyvoice = CosyVoice("pretrained_models/CosyVoice-300M")
# zero_shot usage
prompt_speech_16k = load_wav(ref_path, 16000)
output = cosyvoice.inference_cross_lingual(
    "hello world",
    prompt_speech_16k,
)

encounter error:
Traceback (most recent call last):
  File "my_local_infer.py", line 18, in <module>
    output = cosyvoice.inference_cross_lingual(
  File "/data/CosyVoice/cosyvoice/cli/cosyvoice.py", line 75, in inference_cross_lingual
    return {'tts_speech': torch.concat(tts_speeches, dim=1)}
RuntimeError: torch.cat(): expected a non-empty list of Tensors

please add punctation at sentence end, for example 'hello world.'


[Merge] implement is_valid_gas_limit

Description
implement gasLimit validation as per latest specs:
https://github.com/ethereum/consensus-specs/blob/v1.1.0-beta.5/specs/merge/beacon-chain.md#is_valid_gas_limit

I believe this is fixed already with #4418


Create Major Release

Summary of PR changes
Allow the creation of a major release adjacent to the normal major + minor + patch release creation.
This allows downstream workflows to reference the major version instead of the specific minor + patch.  Doing so will reduce dependabot alerts and unnecessary PRs to get the latest major version.
This conforms to GitHub Action versioning recommendations.
https://github.com/actions/toolkit/blob/main/docs/action-versioning.md#recommendations
PR Requirements

[x] For major, minor, or breaking changes, at least one of the commit messages contains the appropriate +semver: keywords.

See the Incrementing the Version section of the repository's README.md for more details.

[x] The action code does not contain sensitive information.

NOTE: If the repo's workflow could not automatically update the README.md, it should be updated manually with the next version.  For javascript actions, if the repo's workflow could not automatically recompile the action it should also be updated manually as part of the PR.

If this approach is not desired, a separate GitHub action will be produced instead.

From @mike-schenk  comment, closing this PR and opening #26 .  This action's responsibility needs to remain small and specific to generating the version - not creating releases.
I will create a new action that will create the releases.  After that PR is created, it is suggested the creation of releases in this action be removed as a breaking change.


New way to configure stacks to use spot instances when possible, but fallback to OnDemand if necessary

In #710 (released in 5.1.0) we made a few changes to how spot prices and instance types are configured:

deprecated the SpotPrice param
added the OnDemandPercentage param, which sets the instances that should be OnDemand
allow InstanceType to specify up to 4 types (instead of 1)

The intention of was to get "always use a minimum of x% OnDemand, and for the rest use Spot when you can, but fallback to OnDemand if spot isn't available".
It was hard to test though, as we can't predict when Spot instances will be unavailable.
Since shipping 5.1.0 and seeing OnDemandPercentage in use, it seems what we actually have is "always use a minimum of x% OnDemand, and for the rest use Spot when you can, and when Spot isn't available don't fallback to OnDemand - just keep trying to launch Spot until it works".
Some price sensitive users of the stack might prefer the current behaviour, but I suspect many users would prefer to use Spot when available and OnDemand if they must. Engineers sitting around waiting for a build to complete might be more expensive than paying for a few OnDemand instances.
Would we need a new stack parameter for this?

We had an issue crop up a couple of times recently where our region just didn't have spot instances available, so our jobs hung out for 10+ minutes waiting. This wasn't a case of getting outbid, there just weren't instance types at our desired size available. Given that spot is usually perfect for us (none of our jobs take longer than 5-6 minutes), being able to fall back to on demand if spot isn't there would be really useful.

@geoffharcourt I’ve started looking at this use case, could you share some details on your current auto scaling configuration?
I’m looking for details on the instance types and and MaxSize of your auto scale group, what split of OnDemand vs Spot you have chosen, how long you usually wait for your jobs to start, and how long you’d be prepared to wait before incurring the additional cost of an on-demand instance. Thanks!

Hi @keithduncan, thanks for looking into this!
Our setup was:
100% Spot
0-150 instances
m5.large
Normally during the work day our wait is negligible for jobs to run, but if there are no agents and we're under the instance cap it can take a minute or so for instances to get going. This issue where instances are simply not available (the ASG event message was not that we had been outbid, but rather that there were no instances of the type available at all).
We've only seen the scenario where this happens three times, but it's all been in the past 3 weeks. In every occurrence we responded by adding a second instance type, which was not ideal because meant that when the scarcity had subsided the ASG was almost always picking the non-m5.large instance type which was either slower or more expensive.

We had an issue crop up a couple of times recently where our region just didn't have spot instances available, so our jobs hung out for 10+ minutes waiting. This wasn't a case of getting outbid, there just weren't instance types at our desired size available. Given that spot is usually perfect for us (none of our jobs take longer than 5-6 minutes), being able to fall back to on demand if spot isn't there would be really useful.

I'd like to also add a +1 to this request. We have a couple different ASGs that were initially set up with 100% spot requests (across 3 AZ's), most with a min of 0 and a max of 20 or 200. Some of them have a min of 1, and some of them have 1 agent per box and some have more like 12. The instances also include c5a.large, c5a.2xlarge, and c5a.4xlarge.

We had an issue crop up a couple of times recently where our region just didn't have spot instances available, so our jobs hung out for 10+ minutes waiting. This wasn't a case of getting outbid, there just weren't instance types at our desired size available. Given that spot is usually perfect for us (none of our jobs take longer than 5-6 minutes), being able to fall back to on demand if spot isn't there would be really useful.

I'd like to also add a +1 to this request. We have a couple different ASGs that were initially set up with 100% spot requests in one region across AZs, but due to some occasional lack of availability of spot instances where we had to manually adjust the ASG's allocation percentage, we switched to 100% on-demand to be safest.

After upgrading past 5.1 we've had several long waits for spot capacity. https://github.com/buildkite/elastic-ci-stack-for-aws/issues/700 might help somewhat, but e.g. 'detect that spot instances have not launched after a minute and switch to ondemand for X hours' would solve it more reliably.

On 5.9.0 I am seeing annoying cases of builds waiting for agents for unbounded times as the auto-scale simply gives up when spot agents are not available:
aws console shows repeated errors that are not causing the auto-scale logic to fallback to OnDemand:
Launching a new EC2 instance. Status Reason: Could not launch Spot Instances. UnfulfillableCapacity - Unable to fulfill capacity due to your request configuration. Please adjust your request and try again. Launching EC2 instance failed.

Is this the intended behavior? If so, low values of OnDemandPercentage are effectively unusable...


Postgres password exposed in docker-compose file.

Database connection credentials are exposed in the docker-compose file. Please use environment variables.
https://docs.docker.com/compose/environment-variables/

Hi, I'm using the env file locally. I included the credentials so that it will be easy for others to run the application at once. I'll upload  .env.example. Thanks!


Make 'ajaxcart__footer' visible at all times in bottom right of Ajax Cart?

I'm having a hard time getting my head around how I can adapt the CSS in order to make the product rows of the ajax cart scrollable whilst keeping the Checkout button fixed in the bottom right of the cart so thats its visible at all times.
I know that the scroll is currently applied to '.drawer'. I thought maybe I could wrap a div around the items in the ajax-cart-template.liquid snippet and make them scrollable but I just seem to be breaking it.
I've applied the following to '.ajaxcart__footer' which is working as it should but obviously because its inside '.drawer' its just being fixed inside a scrollable div.
.ajaxcart__footer {
 position: fixed;
 bottom: 0;
 right: 0;
}

If this is a simple problem I would appreciate any help you can give.

Take a look at the styles on http://carson-dev-shop-2.myshopify.com/. It does exactly what you're looking for. It's a bit too opinionated to be in Timber by default but definitely doable with some adjustments.

Thanks so much, this should help a lot, been stuck on this for a little while!


fix(go/adbc/driver/flightsql): propagate headers in GetObjects

Fixes #1995.

I'm going to expand the test here to get more coverage