get our decline rate under control

From: Braintree

Hi Chad,

I hope this e-mail finds you well. We just received notification that our risk department flagged your Gratipay account as a result of a large amount of transactions being declined in your Braintree Gateway.

Of the 386 transactions recently processed, 222 have been declined. This is a huge concern for our Risk team and sponsoring bank, and our goal is to work with you to decrease the amount of declines on your merchant account so that you can resume normal processing.

In order to complete their review, the risk department is requesting:

An explanation for the large amount of declines.
The last three months’ bank statements showing beginning and ending balances to ensure that you are supportive of the increased risk exposure brought about by a large decline ratio.
Any processing statements you may have from your most recent merchant processing relationship
Itemized invoices complete with customer contact information and any applicable shipping documents for transactions [], [], and []

Funds are currently not on hold at this time. If I can be of any assistance whatsoever, please don’t hesitate to email me or call me directly at [].

Thank you in advance,
https://gratipay.freshdesk.com/helpdesk/tickets/2295

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/20695881-get-our-decline-rate-under-control?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

Hi, []!

Quick note to say that we've received your email, thanks for bringing the matter to our attention, and we're sorry for the trouble. :-(

The primary explanation for the large number of declines on our account is that our business model is based on weekly recurring payments, and we've simply accumulated a large number of expired or otherwise deactivated cards in the three years we've been operating, without a mechanism in place to stop attempting to charge failed cards. The easy solution for us to drastically reduce our decline rate will be to stop attempting to charge cards after one or two failures.

A secondary explanation for why the ratio is even higher than normal for us right now is that we recently started rebuilding our customer base from scratch (see Gratipocalypse and Gratipay 2.0 for background on that). I suspect that the customers for whom we have resumed processing under our new terms of service include some of our oldest customers (including Gratipay itself, which is funded on Gratipay), and, in virtue of their age, they carry a higher-than-normal decline ratio, which is skewing the overall figure.

Here's a ticket from a couple years ago when we first turned our attention to our decline rate. I've made a new ticket to address the current issue as you've raised it.

I'll follow up on Monday regarding the other three items you've requested.

From: Braintree

Thank you very much for the explanation. We will look forward to receiving the documentation and assisting you further!

To: Braintree

Please find attached and below the remainder of the initial information you've requested. Could we schedule a call to discuss? I'd love to be able to put this info in context for you, and answer any further questions. I assume you're in Chicago, US/Central. Do any of these times work for you?

Wednesday, June 10 at 10:00 am (US/Central)
Thursday, June 11 at 10:00 am
Thursday, June 11 at 2:00 pm
Friday, June 12 at 10:00 am

2. The last three months’ bank statements showing beginning and ending balances to ensure that you are supportive of the increased risk exposure brought about by a large decline ratio.

We are a marketplace, so we keep separate operations and escrow/holding accounts. I presume you're primarily interested in the escrow. Our escrow is currently spread between five accounts. Our main escrow account is the one to which our Braintree account currently settles. It's a local credit union, and, unfortunately, I can't seem to find downloadable statements on their website. For now, I've attached a history export for this account. Let me know if you need actual statements and I will contact our credit union. Let me know, too, if you'd like to see statements for our other escrow accounts, or our operating accounts.

3. Any processing statements you may have from your most recent merchant processing relationship

We were with Balanced Payments. Their printable account statements are not the most intuitive, so for now I've attached a CSV export of our account history there. I can work on providing more detailed information if necessary.

4. Itemized invoices complete with customer contact information and any applicable shipping documents for transactions [], [], and []

Here's what I'm able to say about these three transactions:

transaction: []
customer: https://gratipay.com/~[]/
contact info: [social network link]
weekly payments:
  $[] https://gratipay.com/~[]/

transaction: []
customer: https://gratipay.com/~[]/
contact info: [social network link]
weekly payments:
  $[] https://gratipay.com/~[]/

transaction: []
customer: https://gratipay.com/~[]/
contact info: [social network link]
weekly payments:
  $[] https://gratipay.com/~[]/

From: Braintree Risk

Thank you so much for following up and elaborating on your business.
Unfortunately, our sponsoring bank does prefer copies of statements for documentation purposes. Would you please be able to reach out to your credit union for the statements? Additionally, would you please be able to reach out to Balanced Payments for a copy of your previous processing statement?
Regarding invoices and customer information, are you able to provide us a copy of what the customers would receive as confirmation or what you keep internally for receipts? Do you collect customer phone numbers or emails when you accept payment? If so, would we please be able to have this information with the copy of the invoices?
We apologize for any frustration that may go into this additional work. We are looking forward to resolving this with you. If you have any additional questions, please let me know!

Adding to Balanced Shutdown because this is related to onboarding with Braintree.

When we migrated from Balanced, we didn't bring over failure information. Consequently, we have cards in exchange_routes that we think worked last time, when in fact they failed last time.

network
error = ''
error > ''
total

balanced-cc
3,541
125
3,666

braintree-cc
3,109
909
4,018

It looks like the address in Braintree is the card_id from Balanced, which means we should be able to update the db to reflect the most recent info we have about each card.

Of the 386 transactions recently processed, 222 have been declined.

Do Braintree's numbers line up with our logs?

157 was clean, so the logs should be clean and greppable.

$ grep Holding gratipay-157.log | wc -l
     267
$ grep Holding gratipay-156.log | wc -l
     284
$

Hmmm ... where does the 386 come from?

I called Braintree Risk and spoke with our rep there. I found in the Braintree dashboard that doing an advanced search for all transactions does show 386, though filtering by "All Unsuccesful" yields 221 results instead of 222. She said she usually looks at a "Reporting" interface that I'm not seeing, so she's going to see about the availability of that as well as the off-by-one discrepancy.

Now to line up what we're finding in Braintree with what we've got in the logs ...

!m @rohitpaulk
Let's pick up with numbers over on #3540.

Actually, let's keep it here, I think.

156 was the only other week on Braintree, and it looks like it has a clean log as well.

267 + 284 = 551
551 != 386

At least failures line up:
$ cat gratipay-15{6,7}.log | grep "Holding.*" | wc -l
     551
$ cat gratipay-15{6,7}.log | grep "Holding.*failed" | wc -l
     221
$

$ cat gratipay-15{6,7}.log | grep "Captured.*cents" | wc -l
     164
$

221 + 164 = 385 Off by one.

So it looks like holds that neither fail nor settle don't register on the Braintree dashboard or reports.

Okay! Our numbers line up.

Let's close this after checking the today's numbers (#3540).

Actually, we should keep this open until we're off the risk radar. :-(

Declines are way down in #3540

From: Braintree

Thank you for your call earlier today.

I was able to find the answers to the questions we discussed. The reporting tool where we quoted the “222 unsuccessful transactions” are taken from the “Transaction Summary” tab. The additional 1 transaction included in this report, which differs from the “Transactions Advanced Search”, is the 1 voided transaction, which would not be preselected for “Unsuccessful Transactions” under the “Transactions Advanced Search” report.

Hopefully this helps! Thank you for your patience and we look forward to assisting you further.
To: Braintree

Thanks for following up, I see the Transaction Summary now.
Our decline rate did end up being much lower today: 8%, compared to 59% and 56% for the previous two weeks. What's the acceptable range for our decline rate? What should we be aiming for?

Unfortunately, our sponsoring bank does prefer copies of statements for documentation purposes.

Okay, I intend to track down the New Alliance and Balanced statements tomorrow.

Do you collect customer phone numbers or emails when you accept payment?

No, we don't collect phone numbers when we accept payment. We do collect email addresses, but they're optional for buyers.

[A]re you able to provide us a copy of what the customers would receive as confirmation or what you keep internally for receipts?

I've attached an example of the email notifications (success and failure) we send when we charge someone and they have a verified email on file. Internally for receipts we use the user's so-called "history page." They have access to it as well. I've included an example screenshot for the user on the [] transaction. For successful charges, we do generate a receipt that is linked from the history page as well as the "Receipt" button/link in the success email notification. I've attached an example of that as well.

I will follow up with statements when I have them.

To: Braintree

I stopped in at the credit union and asked for statements. Attached is what they were able to provide. Is this sufficient?

I will follow up when I have something from Balanced ...

Thank you for providing the statements. We’ll provide these to our sponsoring bank to verify that they are sufficient. We’ll reach out to you Monday to follow up on your case. As for the decline rate, it should be as low as possible, ideally 0%. However, given our previous discussions about your business model, I’ll follow up with our sponsoring bank to verify an optimal range.

Looking forward to assisting you further.

To: Braintree

Thanks, []. In addition to guidance on the target decline rate, could we also ask for guidance on the number of times to retry a failing card in a recurring payment scenario such as ours?

I've attached a ZIP file with weekly statements from Balanced going back through March (I only included statements that have credit card charges on them). With this, I believe I've provided all of the information you've requested. If I'm missing something or there is anything else that would be helpful, please let me know!

From: Braintree

Thank you for following up with us about this review.

After speaking with our sponsoring bank, we suggest a decline rate around 15% or ideally lower. However, we understand that situations may arise where it would increase in the future. Let us know if you encounter any of these situations.

As for retrying declined cards, it does depend on the decline reason. Youll be able to view the decline reasons on the transaction page under Processor Response Text in the Transaction Information section. If the decline reason is one such as insufficient funds, or a generic decline with a reason code of 2046, we suggest trying the card again the next day. If it would decline after that, we suggest reaching out the cardholder directly to resolve the issue. However, if for another reason, such as Expired Card or Invalid credit card number, we suggest not retrying these declines and reaching out to cardholder directly to resolve it. You can see the decline reasons on the transaction page under Feel free to hover over the question mark for more detailed information about the decline.

Additionally, our sponsoring bank suggested collecting and saving cardholder information, such as phone numbers, in the future for cases where documentation may be necessary. These situations may occur if you have to dispute a chargeback and provide evidence for your supporting case.

We are closing the review at this time. Please let us know if you have any additional questions or plan to possibly have a spike in decline.

Thank you again for all of your help,

Yesssssssss. :snake:

Retrying cards reticketed as https://github.com/gratipay/gratipay.com/issues/3558.

Reviewing decline rate during payday documented in https://github.com/gratipay/inside.gratipay.com/commit/9a5a5a9f64d985ddb85b24510df730e41255d61a.

Collecting email noted on https://github.com/gratipay/gratipay.com/issues/3506.

That's good news! :D

I've added a review of our decline rate as step 24 of our weekly payday batch process. It calls or emailing<EMAIL_ADDRESS>if the rate is 15% or greater for a given batch.

Now that we're not retrying failed cards at all, I've made a ticket about retrying failed cards in some circumstances as you suggest. Once we implement that it will probably increase our decline rate a bit. We'll keep an eye on it as mentioned.

We don't do a lot with phone numbers, but we do have a ticket open about requiring emails before charging users, so I've added your sponsoring bank's suggestion about collecting and saving cardholder information to that ticket.

Thanks for working with us on this, []. Let us know if anything else crops up! :-)

From: Braintree

Thank you very much for the additional detail. This is great to have on file and we’re glad to hear that you’re taking proactive steps! That’s wonderful news. Please let us know if you have any additional questions. Don’t hesitate to reach out to us in the future.


In OSM if exactly_one=True expect to have in query "limit=1" to limit http transport

I have the code below
>>> print(request.POST)
<QueryDict: {'city': ['Chisinau'], 'street': ['str. Alecu Russo'], 'language': ['ro'], 'country': ['Moldova'], 'csrfmiddlewaretoken': ['RMScTXnPa40Lu6fxdLcPgqrTti7BHONn'], 'limit': ['1']}>
>>> location = Nominatim().geocode(request.POST, exactly_one=True)

I'm expecting to have something
                                                                                ^^^^^^
 Nominatim.geocode: https://nominatim.openstreetmap.org/search?country=Moldova&city=Chisinau&street=str.+Igor+Vieru&format=json&limit=1
                                                                                ^^^^^^

But I got something like this
Nominatim.geocode: https://nominatim.openstreetmap.org/search?city=Chisinau&country=Moldova&street=str.+Alecu+Russo&format=json

no limit!
Is this a missed feature or it should be alternative approach to minimize http transport ?

limit kwarg is just not implemented in Nominatim geocoder yet. This is certainly a valid feature request. Photon geocoder implements this, so it can be taken as an inspiration. PR is welcome.


Makes the RenderImages and RenderWindow resize properly

It appears that CluwneLib.CurrentClippingViewport doesn't get updated.  May need to fix that later, or remove it.

God bless you @YotaXP.


Convert the main and development compose files to be v2

Signed-off-by: Ying Li<EMAIL_ADDRESS>

Oh :|  I guess circleci's docker engine is not 1.10.0 yet


make install error

Hello, does it work for now?  I will make sure it builds on stable release. Thanks for reporting

@tami5 apologies for posting on this closed issue, but I'm seeing issues as well with running make install. Primarily, xcodebuild can't be found. As far as I can tell this is your library you wrote for this and is only available on GitHub right?
➜ make install                                                           22:01:51
    Updating crates.io index
error: no matching package named `xcodebuild` found
location searched: registry `crates-io`

Am I supposed to do something to allow it to be fetched during the make call?

@tami5 apologies for posting on this closed issue, but I'm seeing issues as well with running make install. Primarily, xcodebuild can't be found. As far as I can tell this is your library you wrote for this and is only available on GitHub right?
➜ make install                                                           22:01:51
    Updating crates.io index
error: no matching package named `xcodebuild` found
location searched: registry `crates-io`

Am I supposed to do something to allow it to be fetched during the make call?

Oh @akinsho, no worries, my apologies I the version is wrong, it needs to be 0.1.8 instead of 0.1.7 that's why it's isn't found in crates.io. pushing a fix rn
Thanks a lot @akinsho

I'm still hitting this issue unfortunately on the latest commit

🤔  that's odd https://crates.io/crates/xcodebuild/versions is published.
try cargo update.

I think maybe another dependency might need to be updated as well simctl?
➜ cargo update                                                            1:07:18
    Updating crates.io index
    Updating git repository `https://github.com/tami5/simctl`
error: no matching package named `xcodebuild` found
location searched: registry `crates-io`
required by package `xbase v0.1.0 (~/.local/share/nvim/site/pack/packer/opt/xbase)`

@akinsho  🤔 simctl is directly cloning from my fork of simctl as specified in cargo.toml and it doesn't depend on xcodebuild at all.
That must be some weird issue with crates.io, could you try changing line 92 in Cargo.toml from
xcodebuild            = { version  = "0.1.8", optional = true, features = ["tracing"] }

to
xcodebuild            = { git = "https://github.com/tami5/xcodebuild", optional = true, features = ["tracing"] }

➜ make install                                                                                8:58:31
    Updating git repository `https://github.com/tami5/xcodebuild`
error: failed to get `xcodebuild` as a dependency of package `xbase v0.1.0 (/Users/akin/.local/share/nvim/site/pack/packer/opt/xbase)`

Caused by:
  failed to load source for dependency `xcodebuild`

Caused by:
  Unable to update https://github.com/tami5/xcodebuild

Caused by:
  failed to parse manifest at `/Users/akin/.cargo/git/checkouts/xcodebuild-4c1e65814fc5ca94/d83797e/Cargo.toml`

Caused by:
  namespaced features with the `dep:` prefix are only allowed on the nightly channel and requires the `-Z namespaced-features` flag on the command-line
make: *** [install] Error 101

Is what I get running that command

Uh it requires nightly release of rust, but that's unintended requirement. I will make sure it builds on stable release. Thanks for reporting

Yeah, I have make it success. Thanks very much.
But now, I got a new problem, when I execute Watch, Run or  Build.  vim.loop.cwd() always got nil.
E5108: Error executing lua .../dein/repos/github.com/tami5/xbase/lua/xbase/pickers.lua:57: No project info found
stack traceback:
        [C]: in function 'error'
        .../dein/repos/github.com/tami5/xbase/lua/xbase/pickers.lua:57: in function 'get_selections'
        .../dein/repos/github.com/tami5/xbase/lua/xbase/pickers.lua:154: in function 'watch'
        .../dein/repos/github.com/tami5/xbase/lua/xbase/pickers.lua:215: in function 'run_replace_or_original'
        ...im-telescope/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'key_func'
        ...nvim-telescope/telescope.nvim/lua/telescope/mappings.lua:242: in function 'execute_keymap'
        [string ":lua"]:1: in main chunk
Press ENTER or type command to continue

xbase-daemon.log
2022-06-11T10:07:39.476883Z  INFO xbase::daemon::requests::drop: Drop(29469: Project_ToScanner)
2022-06-11T10:07:39.476924Z  INFO xbase::store::projects: [Projects] remove("Project_ToScanner")
2022-06-11T10:07:39.476938Z  INFO xbase::store::watcher: [Watcher] remove("Project_ToScanner")
2022-06-11T10:07:39.476957Z DEBUG xbase::store::clients: [Clients] remove(29469)
2022-06-11T10:07:39.476972Z  INFO xbase::state: Syncing state to all nvim instance
2022-06-11T10:07:42.609978Z  INFO xbase::store::clients: [Clients] add(31167)
2022-06-11T10:07:42.639041Z  INFO xbase::store::projects: [Projects] add(Project_ToScanner)
2022-06-11T10:07:42.654594Z  INFO xbase::store::watcher: [Watcher] add("Project_ToScanner")
2022-06-11T10:07:43.063669Z ERROR xcodebuild::parser::manual: Skipping: 2022-06-11
2022-06-11T10:07:43.618145Z ERROR xcodebuild::parser::manual: Skipping: Resolved
2022-06-11T10:07:43.767677Z ERROR xcodebuild::parser::manual: Skipping: **
2022-06-11T10:07:44.493449Z ERROR xcodebuild::runner: Fail to parse step Fail to parse compilation step `Trying to overwrite set path with -import-objc-header`
2022-06-11T10:07:48.792637Z ERROR xcodebuild::parser::manual: Skipping: /Users/xpeng/Project/ToScanner/ToScanner/Main/MainViewController.swift:9:8:
2022-06-11T10:07:48.793279Z ERROR xcodebuild::runner: Fail to parse step Unexpected EOF while trying to parse `CompileSwift`, key: `CompileSwift`
2022-06-11T10:07:48.793641Z ERROR xcodebuild::runner: Fail to parse step Unexpected EOF while trying to parse `CompileSwift`, key: `CompileSwift`
2022-06-11T10:07:48.793991Z ERROR xcodebuild::runner: Fail to parse step Unexpected EOF while trying to parse `CompileSwift`, key: `CompileSwift`
2022-06-11T10:07:48.794457Z ERROR xcodebuild::runner: Fail to parse step Unexpected EOF while trying to parse `CompileSwift`, key: `CompileSwift`
2022-06-11T10:07:48.794867Z ERROR xcodebuild::runner: Fail to parse step Unexpected EOF while trying to parse `CompileSwift`, key: `CompileSwift`
2022-06-11T10:07:48.795060Z ERROR xcodebuild::runner: Fail to parse step Unexpected EOF while trying to parse `CompileSwift`, key: `CompileSwift`
2022-06-11T10:07:48.795237Z ERROR xcodebuild::runner: Fail to parse step Unexpected EOF while trying to parse `CompileSwift`, key: `CompileSwift`
2022-06-11T10:07:48.795423Z ERROR xcodebuild::runner: Fail to parse step Unexpected EOF while trying to parse `CompileSwift`, key: `CompileSwift`

Uh it requires nightly release of rust, but that's unintended requirement. I will make sure it builds on stable release. Thanks for reporting

Yeah, I have make it success. Thanks very much.
But now, I got a new problem, when I execute Watch, Run or  Build.  vim.loop.cwd() always got nil.

E5108: Error executing lua .../dein/repos/github.com/tami5/xbase/lua/xbase/pickers.lua:57: No project info found

stack traceback:

        [C]: in function 'error'

        .../dein/repos/github.com/tami5/xbase/lua/xbase/pickers.lua:57: in function 'get_selections'

        .../dein/repos/github.com/tami5/xbase/lua/xbase/pickers.lua:154: in function 'watch'

        .../dein/repos/github.com/tami5/xbase/lua/xbase/pickers.lua:215: in function 'run_replace_or_original'

        ...im-telescope/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'key_func'

        ...nvim-telescope/telescope.nvim/lua/telescope/mappings.lua:242: in function 'execute_keymap'

        [string ":lua"]:1: in main m

I see two separate issues here. first one, no project info, I have experienced it in the past, it doesn't happen often right? not sure why it happens so I need to track it.
The second issue, could you do
xcodebuild clean build > build.log

in the project directory. I need to do tests on the parsing the build log, maybe apple changed something .
Thanks

➜ make install                                                                                8:58:31

    Updating git repository `https://github.com/tami5/xcodebuild`

error: failed to get `xcodebuild` as a dependency of package `xbase v0.1.0 (/Users/akin/.local/share/nvim/site/pack/packer/opt/xbase)`

Caused by:

  failed to load source for dependency `xcodebuild`

Caused by:

  Unable to update https://github.com/tami5/xcodebuild

Caused by:

  failed to parse manifest at `/Users/akin/.cargo/git/checkouts/xcodebuild-4c1e65814fc5ca94/d83797e/Cargo.toml`

Caused by:

  namespaced features with the `dep:` prefix are only allowed on the nightly channel and requires the `-Z namespaced-features` flag on the command-line

make: *** [install] Error 101

Is what I get after editing the cargo.toml

😕 more nightly related issues. Let me remove dep: syntax from xcodebuild toml file

@TMTBO for the issue with log parsing, it turns out that my refactor of nightly stuff and replacement introduced the issue. it should work now, with recent commit.
@akinsho I believe it should work now, could you please retry.

@tami5 that worked 🎉, one thing to note though is that it changed my Cargo.lock
diff --git a/Cargo.lock b/Cargo.lock
index 75a34b8..6c1cee4 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -1672,7 +1672,7 @@ checksum = "c811ca4a8c853ef420abd8592ba53ddbbac90410fab6903b3e79972a631f7680"

 [[package]]
 name = "xbase"
-version = "0.1.0"
+version = "0.1.2"
 dependencies = [
  "anyhow",
  "async-stream",
@@ -1710,9 +1710,9 @@ dependencies = [

 [[package]]
 name = "xcodebuild"
-version = "0.1.9"
+version = "0.1.10"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "747a42b03340a5b1c2b8c30ca7a5e9b1258c7428ed9ce9b1f233ea37a4c6588a"
+checksum = "3c97aaa241577daa45fbd798b716ce123174f29c61544d172d9e3250f394e6cb"
 dependencies = [
  "anyhow",
  "async-stream",

which will make updating after running make install fail in the future

that worked 🎉

🌟🌟🌟

one thing to note though is that it changed my Cargo.lock which will make updating after running make install fail in the future

🤔 how so, isn't Cargo.lock needs to change when Cargo.toml changes, make install I believe shouldn't update Cargo.lock as it feeds from it, could you clarify

@tami5 I think the issue is that the Cargo.lock in the project hasn't had changes committed to it to represent the state that it should be in. I think you need to run make install on your end and commit the changes so that when I run make install on my side, it doesn't need to change the lock file. At least based on how lock files work in other languages.

uhhh @akinsho I see what I mean. with each change to Cargo.toml I need to make sure I push Cargo.lock, (I just checked my end and sure enough Cargo.lock is left uncommitted). Thanks @akinsho I will make sure for future release to commit Cargo.lock changes

👍🏿 that's fixed now

The second issue, could you do
xcodebuild clean build > build.log

in the project directory. I need to do tests on the parsing the build log, maybe apple changed something .

build.log
Yeah, I got the build.log now. Maybe it help.
Or should I create a new issue for this?

The second issue, could you do
xcodebuild clean build > build.log

in the project directory. I need to do tests on the parsing the build log, maybe apple changed something .

build.log
Yeah, I got the build.log now. Maybe it help.
And I got the error in xbase-daemon.log

Yah I will look into the build log in a bit. I do plan to do some progress on xcodebuild crate today.

Or should I create a new issue for this?

Maybe if you experience more of no project info found, because this issue I've seen sometimes happen to me. But for logs parsing issues please open in  xcodebuild

ERROR xcodebuild::runner: Fail to parse step Fail to parse compilation step Trying to overwrite set path with -import-objc-header

The build logs as noted in the readme is controlled by xcodebuild and it still under progress, this error is basically a debug message saying, hay parser, you didn't do anything with this line.

build.log

@TMTBO  it turned out that the reason the build failed is due to Linker Duplicate Symbols Location, I'm switching to regex match on xcodebuild and I was able to extract that FlexLayoutYogaKit.o and FlexLayoutYoga.o.
does this build okay with Xcode? maybe I'm missing some arguments with xcodebuild, idk
btw, the build logs was helpful in refining my matchers, thanks

build.log

@TMTBO it turned out that the reason the build failed is due to Linker Duplicate Symbols Location, I'm switching to regex match on xcodebuild and I was able to extract that FlexLayoutYogaKit.o and FlexLayoutYoga.o.
does this build okay with Xcode? maybe I'm missing some arguments with xcodebuild, idk
btw, the build logs was helpful in refining my matchers, thanks

Yes, the FlexLayout is dumplicated. One for SPM, and another for Pod. And this work with Xcode.


React to autoFit's input change

Right now, toggeling the autoFit flag will resize and reposition the graph only on window resize. It would be brilliant, if on toggeling the autoFit (input changes to true), the graph would fit once, without the need for the window resize.

Good improvement! Thanks a lot 👍


docs: fix punctuation in step1_lexing.md

Someone must've hit " by accident, which is right next to : on the ANSI keyboard layout.

Thanks @srfsh 👍


Conan

Brief Issue Summary
I recently started to use conan for handling third parties in my project and for some weird reasons, now CMake Tools is like disconnected from IntelliSense.
I just imported cmake-conan helper script that allows CMake configure to automatically run conan install with support for multiple configurations. I've checked under Visual Studio Community and all seem to work fine. However when going into VSCode no more any definitions is identified, all imported names in sub modules are errors.
The project still builds fine under Windows, Mac and even Linux can be checked under Travis.
Expected:
I expected VSCode C++ IntelliSense to correctly resolve symbols with conan enabled.
Apparent Behavior:
IntelliSense for VSCode is completely unable to find any symbols at all not just the symbols of the conan libs. VSCode C++ IntelliSense is like useless on the project now...
Developer Tools Log
I get these errors when opening any file in the project now:
Command: global --encode-path " " -f "c:\Users\Yuri Edward\BlockProject3D\Framework\Compression\include\Framework\Compression\ZInflater.hpp" running...
console.ts:137 [Extension Host] Error: Error: Command failed: global --encode-path " " -f "c:\Users\Yuri Edward\BlockProject3D\Framework\Compression\include\Framework\Compression\ZInflater.hpp"'global' is not recognized as an internal or external command,
operable program or batch file.
global --encode-path " " -f "c:\Users\Yuri Edward\BlockProject3D\Framework\Compression\include\Framework\Compression\ZInflater.hpp" (exited with error code 1)
EDIT: I just captured the entire log of a fresh window reload and sent it to pastebin: https://pastebin.com/9RwNmSbQ
Platform and Versions

Operating System: Windows
CMake Version: cmake version 3.16.0-rc3
VSCode Version: 1.42.1
CMake Tools Extension Version: 1.3.1
Compiler/Toolchain: Visual Studio Community 2019 Release - amd64
Build type: Debug
Conan version: Conan version 1.24.1

Other Notes/Information
I'm not entirely sure at which moment in the process of enabling conan it started to fail...

Wait after a complete re-build from VSCode (delete build folder) let VSCode recreate the build folder, the conan symbols are now correctly resolved but all other symbols are broken. Also weirdly enough I can neither reproduce the bug nor hide it/revert to before it occured.
I will keep trying to find the trigger to the bug...

The 1.4 release will have a fix to prevent Ninja from being chosen despite a setting preferring something else.
So, from the later edits of the second comment of this issue, are you unblocked now? Any other problems?
If your project is still not working as expected, it would help us if you make a standalone project that would manifest this issue, so that we use our time debugging the extension, investigating the root cause, finding a workaround and implementing a fix instead of ramping up on the Conan technology and creating a repro from scratch.

I am using the Arduino toolset and having the same problem, a spam of :
console.ts:137 [Extension Host] Error: Error: Command failed: global --encode-path " " -f "c:\Users\silent\Documents\Projects\airsoft_fcu\airsoft_fcu.ino"'global' is not recognized as an internal or external command,
operable program or batch file.
 `global --encode-path " " -f "c:\Users\silent\Documents\Projects\airsoft_fcu\airsoft_fcu.ino"` (exited with error code 1)

And
console.ts:137 [Extension Host] Error: Error: Command failed: global -u'global' is not recognized as an internal or external command,
operable program or batch file.
 `global -u` (exited with error code 1)

Eventually it will give up completely and I need to restart VSCode:
[Extension Host] undefined
abstractExtensionService.ts:155 Extension host terminated unexpectedly. Code:<PHONE_NUMBER>  Signal:  null
_onExtensionHostCrashed @ abstractExtensionService.ts:155
notificationsAlerts.ts:40 Extension host terminated unexpectedly.

I havent touched this project in a few months and now this started happening.

This issue might be related to #602: it seems that if you get the error of 602 your project IntelliSense support is permanently broken and you fall back under this issue where absolutely no symbols and no macros not even headers can be found due to incorrect or non existant c_cpp_properties.json.
My guess is for some reasons CMake Tools can no longer generate c_cpp_properties after the 602 error.
Any idea why?

Intellisense for packages installed with conan in CMake based project doesn't work at all in vscode. E.g. the path can be C:\.conan\4d8898\1\include on windows. In both VS2019 and CLion it works out of the box.

The following settings won't work either (even if the path is specified explicitly):
"includePath": [
    "${default}",
    "~/.conan/data/**",
    "C:/Users/xxx/.conan/data/**",
    "C:/Users/xxx/.conan/data/catch2/2.13.3/_/_/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/include",
    "${workspaceFolder}/**"
],

Is the includePath overwritten by configurationProvider? The error message says:
#include errors detected based on information provided by the configurationProvider setting. Squiggles are disabled for this translation unit (<file_path>).C/C++(1696)

Yes, that's right. If C_Cpp.default.configurationProvider is set to ms-vscode.cmake-tools, then the CppTools extension will ignore c_cpp_properties.json (and compile_commands.json as well).


replacing formatargspec() per inspect.signature()

The formatargspec() function is deprecated since Python 3.5, and removed in Python-3.11
Description of Changes

I replace The formatargspec() function, deprecated since Python 3.5; per inspect.signature()
[x] should resolve https://github.com/spyder-ide/spyder/issues/20296
[x] I did a simple check posted in the issue itself

I asked ChatGPT, as my first try to see what it can do, on a code I don't understand
Issue(s) Resolved
to solve https://github.com/spyder-ide/spyder/issues/20296
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

checking on lambda:

Thanks @stonebig for your help with this! This needs to be solved in Spyder-kernels, and it has already solved in spyder-ide/spyder-kernels#435.

oups ! ok


README.md doesn't reflect the correct path to angular-cropme

<script src="components/angular-cropme/cropme.js"></script>
<link rel="stylesheet" href="components/angular-cropme/cropme.css">

Those two links should point to cropme instead of angular-cropme.
Are you maintaining this repo?

Yes, I will update this, thank you

Thanks for the feedback :smile:


Fixed incorrect example query results

Fixes https://github.com/cockroachdb/docs/issues/12026.

This change is


refactor(website): hero animation with GSAP

Refactored the taikoGeom image animation with the GSAP  library to make it smoother, clearer and better maintainable.

there are some merge conflicts @wolfderechter let me know if you need any help

The animation is not as smooth as it used to be.

have you considered the react-spring library maintained by poimandres? wonder if it is any smoother. poimandres maintains some good stuff. maybe a fun thing to look as well: https://github.com/pmndrs/react-spring

@dantaik what about it is less smooth? did you try this URL: https://website-4nidotxwy-taikoxyz.vercel.app/

True, it's not as smooth as the current website deployment. I'm using a MacBook Air.

I wanted to try using a framework made for animations to simplify the code but it seems like my own approach works better overal. I'll close this PR


[Feature Request]: Add "serverUrl" param to GitLab's OAuth provider config

Description
Besides "gitlab.com", GitLab can also be installed on-premise.
So, it would be nice to add the possibility to provide the url of a self-managed GitLab instance as parameter in the provider configuration here.
type Config = {
  clientId: string;
  // ...
  serverUrl?: string; // "https://my-gitlab.mysite.com"
};

The parameter could be optional and, if not provided, the OAuth provider should just keep using the default "gitlab.com".
I can open a PR, if needed.
Thanks

Yeah sure, I'm open to the change. Can you create a PR? Make sure to update the docs as well.


Get sync tests working on Ubuntu and Windows CI

Is your feature request related to a problem? Please describe.
In #1179, we disabled sync tests on Linux and Windows CI, because it was unreliable.
We want to test Zebra sync across multiple platforms, to catch platform-specific bugs.
In particular, if we want to distribute Zebra using Linux docker containers, we need to test sync on Linux.
Our CD runs on Debian Buster, so we'll only catch Linux-specific sync errors after we merge to main.
Describe the solution you'd like
Work out how to get reliable network on our Linux and Windows CI runners.
Describe alternatives you've considered
If the tests fail, sleep for a while, then repeat the test, hoping the network will be better. This isn't a great alternative, because failures are correlated, and retries can mask genuine bugs.
Additional context
We also added a 2000-block sync test in #1185.

We don't think this fix is possible - it depends on CI network configuration, which is not under our control.


path/name and message maximum size update

queues and topics have a path of 260 (was omitted)
rule is not listed in entities
namespace, subs, rules, or eventhubs names have a limit of 50 char only. Queues and topics can be up to 260 since name could be the whole path.
Max message size to reflect the Premium tier (it's ok to mention that you can do up to a meg!)
Update to the max message size note to be tier agnostic

Hi @SeanFeldman, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

@sethmanheim - Could you review this proposed update from @SeanFeldman in the public repo?

@SeanFeldman Hi Sean, and thank you for your contribution! Can you show me where you obtained this information? If the engineering team confirmed these limits, that's great -- otherwise I will need to verify with them. Many thanks again.

@sethmanheim here :grinning:
I'm compiling doco for our product which is based on ASB, so all those items are based on the work I've done and things have seen (the things I have seen... :laughing:). If you'd like a quick confirmation, ServiceBus explorer could do the trick as well.

@djrosanova could you please verify the change?

@SeanFeldman Wow, great info and thorough research you've done -- thanks again! @tysonn OK to merge.

@sethmanheim thank you.
Glad to be of help.


Change images from heapster:canary to gcr.io/heapster:latest

The heapster examples and container makefiles all reference dockerhub's heapster:canary.  However, the releases are using gcr.io.  This was confusing for us when we copied some example YAML and tried to pin to a version (the last version pushed to dockerhub is 0.19).
Is there a reason why the "latest" image is pushed to dockerhub and the releases are to gcr.io?
If not, I'd be happy to put together a PR that switches them over to use gcr.io.
I image there are some containers that haven't been pushed to gcr.io yet (such as heapster_influxdb, kubernetes/heapster_riemann), but at least the main heapster image should be moved over.
For reference
https://github.com/kubernetes/heapster/search?p=1&q=canary&utf8=✓

@r2d4 Why not push to both and let the users choose which one they will use. BTW, i agree that we should be consistent. :)


Animations of a group or a view

Is it possible to apply animations to a group as a whole, and/or apply an animation block to a group such that each view inside will use it?
Or all views should just be treated regularly?

Can you give me an example of what you mean?

take the vertical social bar as an example
if the icons are grouped vertically, I might want to animate the entire thing as a whole to slide from the right. Or I would animate each one to slide on its own.
Is Facade just better syntax for autolayout or does it affect in any way on how the view behaves if its in a group?

In a nutshell, the UIView+Facade category is just a collection on convenience methods that set the frames of views based on their superview and/or sibling views. Take the vertical grouping in the upper-right corner for a simple example:
- (void)groupVertically:(NSArray *)subviews inUpperRightWithRightPadding:(CGFloat)right topPadding:(CGFloat)top spacing:(CGFloat)spacing width:(CGFloat)width height:(CGFloat)height {
    CGFloat yOrigin = top;

    for (UIView *view in subviews) {
        view.frame = CGRectMake(CGRectGetWidth(self.frame) - width - right, yOrigin, width, height);

        yOrigin += height + spacing;
    }
}

That's all it does under the covers, there's no actual auto-layout anywhere in the code. I would imagine you'd be able to simply wrap any Facade API's in a UIView animateWithDuration:animations: call, and all of the frames should animate properly.

I just tried that out and it seemed to work fine... Here's the original setup code:
    [_redView anchorInCenterFillingWidthAndHeightWithLeftAndRightPadding:10 topAndBottomPadding:10];
    [_redView groupVertically:@[_orangeView, _yellowView, _greenView] inUpperRightWithRightPadding:7 topPadding:7 spacing:7 width:50 height:50];

When I clicked the button here's the method I called:
- (void)testAnimate {
    [UIView animateWithDuration:2 animations:^{
        [_redView groupVertically:@[_orangeView, _yellowView, _greenView] inUpperLeftWithLeftPadding:7 topPadding:7 spacing:7 width:50 height:50];
    }];
}

...and here was the result:
https://dl.dropboxusercontent.com/u/142773735/animate-example.mov

ok, great, thanks!


entc/gen/template: Wrap error returned by field validator

Changes proposed in this pull request:

Use verb %w on field validation call to fmt.Errorf, so that the error type can be checked using errors.Is() function.

fixes: #508

Thanks for submitting this PR @petersondmg.
Please run go generate ./entc/gen/... and then go generate ./....

♻

Thanks @petersondmg


Revert "fix: Respect student_view_multi_device key for HTML xBlocks in the Block API"

Reverts edx/edx-mobile-marketplace-ios#79

@shafqat-muneer I'm merging this so that I can start working on release 6.0.3


UIA3Automation.FromPoint gets a wrong element in Windows 11 Notepad

In Window 11 there's a problem with new Notepad. This is the current status of FlaUInspect

But, if you try to get Minimize button with FromPoint using coordinates ({X=1643,Y=165,Width=47,Height=30}), FlaUI returns the entire Notepad window.
The same behavior can be reproduced by using "Hover Mode" in FlaUInspect trying to select Minimize button.
This problem could be applied also on "Save Dialog"

There's no way to get any of those element using FromPoint

[Inspect.exe](Using https://learn.microsoft.com/en-us/windows/win32/winauto/inspect-objects) from Microsoft I got same behavior for Minimize, Maximize and Close. So I'm guessing it's a problem for UIAutomation itself

I don't have Win11 to test but some notes/guesses:
@peerpalo the titlebar for windows and those buttons are an exceptionally special case.  There are a whole host of api's and window messages exclusively around the titlebar.  Apps can provide their own titlebar (and specify the titlebar region at the same time), apps can forgo  the titlebar completely and try to emulate the entire thing but this is more uncommon.    The most obvious way to determine what titelbar item you are over is to try the WM_NCHITTEST message:
https://learn.microsoft.com/en-us/windows/win32/inputdev/wm-nchittest
https://superuser.com/questions/1512761/wm-nchittest-and-detecting-the-title-bar-of-a-window
UWP/WinUI3 itself (what the new notepad I assume uses) has some interesting UI issues, the new notepad using the tabs in the titlebar the official 'titlebar' control area may be scoped only to the very right most rectangle.  https://learn.microsoft.com/en-us/windows/apps/develop/title-bar?tabs=wasdk  .
Still if you send the WM_NCHITTEST it should still return the right item (HTREDUCE/HTMAXBUTTON/HTCLOSE)


Remove Ely sigil cost from corp BG

🚀 Feature Proposal
Currently, the corp BG requires you to sink an Arcane, spectral AND Elysian sigil as a repeatable cost, if you were to switch backgrounds. The 2 cheaper sigils alone make up a more than reasonable price, but the elysian is one that is often the players last item - and one they'd rather keep. Not to mention, it is worth over 1b gp making the cost for this bg MUCH higher than other backgrounds. Simply put, Elysian cost is a bit too much.

there is one that costs more.


Help with your instructions?

I did download the zip and restored it in api.ai but how or where to deploy the project?

You can pick where you want to host the fulfillment logic: https://developers.google.com/actions/tools/fulfillment-hosting


jquery problem after running set up procedure

Followed the exact set up but there is a problem with jquery??? I have latest npm. Why am I getting such a simple error with jquery? ...
Uncaught Error: Bootstrap's JavaScript requires jQuery
on both vector-bundle.js and bootstrap.min.js

I installed latest node and npm, and then from the tutorial page on the aurelia site:-
To get Bootstrap setup, we begin by installing the library itself with NPM. Execute the following on the command line to do this:
npm install bootstrap --save
Next, because Bootstrap uses jQuery, we want to install jQuery as well, like this:
npm install<EMAIL_ADDRESS>--save
then restarted the app as packages were updated and ran it again ... FIXED!


Where is the prepare_dataset.py?

Firstly, this is an excellent work.
I found that in the README, prepare_dataset.py is under the estimator folder, but actually it is under the loader folder.
By the way, could the author provide an End-to-End case for study and evaluation. Since it is a bit of hard to reproduce the result through several separate readme.

Also, could the author provides the root_dir folder as well as the files under this folder, thanks very much.


Automatically switch between tenant admin and regular site

Currently, commands that require connection to the tenant admin site (contoso-admin.sharepoint.com), check if the user connected to the tenant admin site using the spo connect site. If not, they display an error telling the user to explicitly connect to the tenant admin site before running the command. We could improve this experience by automatically getting the access token for an access token to the tenant admin site without bothering the user. Doing this is possible, because AAD issues multi-resource refresh tokens that can be used for obtaining access tokens to different resources in the same AAD.
Any opinions @estruyf, @andrewconnell, @wictorwilen?

Automatically switching is great, but it might have an impact. What if the user accidentally adds a command which is tenant scoped in his script? At this moment the user will get an error to log in to the tenant admin sites, which makes him aware he is going to use a higher scoped command.
When it is all done automatically, the user will not notice it, and if he accidentally took the wrong command, it can mess things up. Of course, this can also happen when the user is already logged in.

That could potentially be an issue. If we look at the non-interactive mode, we could require users to include an extra spo connect <tenant-admin-url> command in their script, which would make it more explicit. When we can store credentials, we could silently switch without halting the script flow.
@erwinvanhunen how is context switching handled in scripts built using PnP PowerShell when some commands apply to tenant and some to a site? Do you require users to call Connect-PnPOnline each time they want to work on a different site or do they switch automatically based on the provided URL/called cmdlet?

Would an additional flag be an option? That way, the user has to explicitly tell the CLI that it wants to use that automatic flow.

You mean something like spo connect <random-site> --autoswitch?

Let's see what others think

In the case of the PnP Cmdlets the cmdlets only autoswitch to tenant level if the cmdlet is based upon the PnPAdminCmdlet baseclass. Those are all called -PnPTenant
There is one cmdlet that does an under the hood switching to another side: Remove-PnPApp. It switches context to the Tenant App Catalog

@erwinvanhunen but when they do switch there is no confirmation or explicit flag the user has to set to make it happen seamlessly?

At this moment we support automatically switching from tenant admin to regular sites but not the other way around. We use this approach in a few commands and seems to be working. Let's keep using it for now and revise in the future if necessary.


Adding support for sleep time, ignored pools, and spare agents

Please forgive my overbearing editor for the whitespace changes. :|
I found some extra features of acs-engine-autoscaler that may be helpful, so I modified the chart's command field to include them. @ritazh can you do a once over on this? :)

@k8s-bot ok to test

Awesome work @bnookala! Thanks for adding these! Just left one comment for review.

@bnookala accidentally merged this before @ritazh had a chance to review—very sorry!

Hit me up if you need a follow-up PR to fix anything. Will make sure you both sign off before merging.


Update descriptions of Pdf neuron close to tritocerebrum, EH neuroscretory neuron and adult CCAP neuron

FBrf0238423 has a lot of new detail on the following neurons (summarised table 2)
Pdf neuron close to tritocerebrum - pharate & adult stages
EH neurosecretory neuron - post larval stages - note current definition of this term implies larval specific, should be changed
adult CCAP neuron - pharate adult stage

new subclasses of CCAP neurons and new class for adult EH neuron as well as description edits


Maybe a little bug - Boot Priority cannot be negative

Hi Henrik,
My tests are continuing with maybe the most difficult image: The A3000 Dual Boot for KS 1.4 one, which requires 4 partitions (two DOS1 and two PFS3)
I am having some other issue with the format that goes a little mad with either two FS in the RDB or too many partitions to format (but it is too early to raise an issue here, need more testing over the weekend with a fresh mind)
For now, I believe I have spoiled a little tiny bug.
The boot-priority in the real amiga can be a negative number (IIRC the range is from -127 to 127), but here it returns some bogus number when set to -1
I could have simply used the boot-priorities from 0 to 4 (5 is normally the floppy) to make it work, but for the sake of completeness of Hst-Imager I wanted to report to you in case you would feel to fix that :-)

This is the full list of commands:
hst.imager.exe blank WB3000.hdf 32GB
hst.imager.exe rdb init WB3000.hdf --size 99%
hst.imager.exe rdb fs add WB3000.hdf pfs3aio PFS3
hst.imager.exe rdb fs add WB3000.hdf FFS_v401 DOS1
hst.imager.exe rdb part add WB3000.hdf WB_1.3 DOS1 1GB --bootable --max-transfer 130560 --buffers 80 --boot-priority -1
hst.imager.exe rdb part add WB3000.hdf WB_2.x DOS1 15MB --bootable --max-transfer 130560 --buffers 80 --boot-priority 1
hst.imager.exe rdb part add WB3000.hdf DH0 PFS3 500MB --max-transfer 130560 --buffers 80
hst.imager.exe rdb part add WB3000.hdf DH1 PFS3 * --max-transfer 130560 --buffers 80
hst.imager.exe rdb part format WB3000.hdf 1 WB13
hst.imager.exe rdb part format WB3000.hdf 2 WB2x
hst.imager.exe rdb part format WB3000.hdf 3 Workbench
hst.imager.exe rdb part format WB3000.hdf 4 Data
hst.imager.exe rdb info WB3000.hdf

Fixed boot priority with negative values in v0.9.255.


Add: Liran Tal

Looks like it is rendering well:

Thanks @MichalPorag
Do I need to shoot out another PR to have this deployed on the website?


Remove yarn test-without-relay

Once Relay compiler will work without watchman. 1.2.0 was supposed to fix it, still...
Then we can remove generated from the repo and build them in yarn build.

Removed


Make it easier to change oauth user identity

Currently with --auth=oauth you need to rm -rf .config/gcpdiag to force a relogin. Provide an option to force a login.

@schweikert
As per release 0.58, we can close this enhancement since authentication using OAuth (--auth-oauth) has been deprecated


Series.to_bag

A small feature request: I frequently run into cases where I want to do something to a single column of a distributed dataframe using the Bag abstractions, something like, e.g.:
freqs = ddf['text'].map(str.split).flatten().frequencies()
An easy way to create a Bag from a Series would make this more straightfoward.

Never mind!

Never mind!

Never mind!

Never mind!


Custom config archivist entries don't seem to override default config.

npm install mage --create --typescript --prefix my-gameserver

config/custom.yaml

archivist:
    vaults:
        # Our vaults, augment this with as many as you need.
        # You should take care to name them appropriately given the context of how they will be used.
        volatileVault:
            type: memory
        userVault:
            type: file
            config:
                path: ./filevault/userVault
        tableVault:
            type: file
            config:
                path: ./filevault/tableVault

    # When doing "list" operations, we will attempt each mentioned vault until successful
    listOrder:
        - userVault
        - tableVault

    # When doing "get" operations, we will attempt each mentioned vault until successful
    readOrder:
        - volatileVault
        - userVault
        - tableVault

    # When doing "add/set/touch/del" operations, we will write to each mentioned vault in the given order
    writeOrder:
        - client
        - volatileVault
        - userVault
        - tableVault

add to file

lib/archivist/index.ts

exports.table = <mage.archivist.ITopic> {
  index: ['tableId'],
  vaults: {
    tableVault: {
    }
  }
};

Then run npm run archivist-create;
m-18602 - 16:26:19.071     <debug> [MAGE archivist] Vault "tableVault" is not in list/read/write-order, skipping.
m-18602 - 16:26:19.075 <emergency> [MAGE archivist] None of the mentioned vaults for topic "table" is available.
  data: {
    "file": "configuration.js",
    "line": 620,
    "offset": 10,
    "type": "Error",
    "stack": [
      "node_modules/mage/lib/archivist/configuration.js:620:10",
      "Array.forEach (native)",
      "registerTopics (node_modules/mage/lib/archivist/configuration.js:596:9)",
      "Object.exports.setup (node_modules/mage/lib/archivist/configuration.js:675:3)",
      "EventEmitter.exports.setup (node_modules/mage/lib/archivist/index.js:20:16)",
      "setupArchivist (node_modules/mage/lib/tasks/serve.js:23:23)",
      "node_modules/async/lib/async.js:610:21",
      "node_modules/async/lib/async.js:249:17",
      "iterate (node_modules/async/lib/async.js:149:13)",
      "node_modules/async/lib/async.js:160:25"
    ]
  }

Copying the config from the custom.yaml to default.yaml fixes this.
I expect config in the custom config to override or add to the default config.

I am assuming this is the problem.  My actual problem gave the error.
m-18602 - 16:26:19.071     <debug> [MAGE archivist] Vault "tableVault" is not in list/read/write-order, skipping.
m-18602 - 16:26:19.075 <emergency> [MAGE archivist] None of the mentioned vaults for topic "table" is available.
  data: {
    "file": "configuration.js",
    "line": 620,
    "offset": 10,
    "type": "Error",
    "stack": [
      "node_modules/mage/lib/archivist/configuration.js:620:10",
      "Array.forEach (native)",
      "registerTopics (node_modules/mage/lib/archivist/configuration.js:596:9)",
      "Object.exports.setup (node_modules/mage/lib/archivist/configuration.js:675:3)",
      "EventEmitter.exports.setup (node_modules/mage/lib/archivist/index.js:20:16)",
      "setupArchivist (node_modules/mage/lib/tasks/serve.js:23:23)",
      "node_modules/async/lib/async.js:610:21",
      "node_modules/async/lib/async.js:249:17",
      "iterate (node_modules/async/lib/async.js:149:13)",
      "node_modules/async/lib/async.js:160:25"
    ]
  }

And copying my config from custom.yaml to default.yaml fixed it.

I do not think 'custom.yaml' will work unless it is part of your NODE_ENV, as documented: https://mage.github.io/mage/#location

I don't have 'default' in my NODE_ENV.  Therefore...

The documentation might not make this clear enough, but it is loaded by default:
The files are located in your game’s config folder. Here you will find a default.yaml configuration file,
that you can use to collect all configuration that every single environment should use (that is,
configuration that is not unique to a single developer’s environment).

Perhaps the custom file should not be supplied on install?  It kind of implies that I can edit / use it.

OMG, you are right. For some reason, this file is in our template despite the fact this feature is no longer (has never been?) supported by MAGE.
cc @ronkorving

Another issue is that this file is not in the .gitignore, so this means devs who would want to use it would end up committing over each other some custom config anyway.
I'll see what I can do do fix that, and make sure it is also reflected in the documentation.

It was never a mage pattern, so much as that a few projects adopted this as "the way to do things from now on". But now that a lot of developers share the same configurations (thanks to containers, etc), I don't see a need to even mention custom.yaml/json.

@ronkorving I already have added support for it in #115 since it was relatively straightforward. Let me know if you believe this is misguided.

I think it's overkill, but if any users can tell me differently from experience (ie: having a custom.yaml around made sense for them in recent times), I'm cool with making it official 👍
If nobody ends up using it, I think it's added complexity for no gain and we should remove it.

The code to add this isn't all that complex and the documentation has been updated to explain more clearly the loading order.
https://github.com/mage/mage/pull/115/files#diff-5122ec46a4f7ff0f1909a379117ff89bR404
Since it has been a feature that has been used in the past (but where you had to tweak your own NODE_ENV to make it work - which is sort of annoying, especially if you switch to non-MAGE project not supporting the comma-separated env list), I'd then to go with keeping this feature. The only good reason would be if it over-complexified the loading order in a way that confused developers. I think.

When I mentioned complexity, I was more talking about cognitive than code (every concept a user needs to understand extra should be considered carefully). But your comment and our conversation in person yesterday convinced me there's enough usability to warrant keeping this 👍


Get-TestsFromBCContainer: -companyName

Hi @freddydk,
I examined your example Run-Tests.ps1
While Run-TestsInBCContainer offers -companyName as a parameter,
Get-TestsFromBCContainer does not.
Since I usally run my tests in a test company this would be a nice feature ;-)
Or do I miss something?

fixed in <IP_ADDRESS>


Watson Conversation loses state, resets to root node rather than waiting for user input.

Hello,
I downloaded and updated the .env file with both my Watson Conversation Service (WCS) creds and my Slack API.
When I started the simple-bot example it seems to start fine. It receives input from Slack, and it queries WCS and posts a response back to Slack.
However, I am seeing an issue in one scenario.
I have a node that asks user for some clarification, the node is set to wait for user response and then continue.
When I try this on the WCS dialog chat, it works as expected. Context is kept, and the system picks up form the user input and continues on.
But when I try this from Slack, the user input doesn't seem to start from the node where it is expecting user input, but from the root node.
Another thing I am noticing which might be related to this issue is that the middleware seems to be pinging WCS. (See console log below)
`>npm start

<EMAIL_ADDRESS>start /Users/robertloredo/Documents/projects/botkit-demo/examples/simple-bot
node simple-bot-slack.js

info: ** No persistent storage method specified! Data may be lost when process shuts down.
info: ** Setting up custom handlers for processing Slack messages
info: ** API CALL: https://slack.com/api/rtm.start
Client server listening on port 6002
notice: ** BOT ID: cqs-bot ...attempting to connect to RTM!
notice: RTM websocket opened
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Warning: using temporary storage. Data will be lost when process restarts.
info: Slack message received!
I `
As you can see this is repeating as when I post a slack message it works fine, but after the process is completed, it goes back to looping this message.
I'm wondering if this is also pinging WCS, if so, then this might be why it is resetting back to root node?
Any help is appreciated.
Thank you!
Robert

@robert-loredo I notice you're using an old version. Can you install the latest 1.1.0 and then try running the example included with it?
We're now advocating the use of the interpret function as opposed to the receive function(which was pinging Conversation for all events and not just heard events).
Let me know if that fixes it.

Yup! Updating to the latest v1.1 fixed the issue! Thanks!
One thing I also learned by linking with Slackbot.
I originally had my WCS on a chat screen. So when client first opened the window, it would send the conversation_start.
When integrating with Slack, it seems best to not have the conversation_start since the client would just ask it a question right away, (unless of course they send an initial greeting before asking question). Otherwise it would respond with the conversation_start and not the answer to the question. :)
Thanks again aarora91 ! Happy Holidays!


[21.05] kubernetes: 1.21.1 -> 1.21.6

Motivation for this change
Updating the k8s version for NixOS 21.05.
Fixes https://github.com/NixOS/nixpkgs/issues/142980
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This upgrade broke the coredns pods for me.  I think I finally ran into the "Pod stuck in ContainerCreating: Unit ...slice already exists #102676" kubernetes bug.  According to the nixos tests for kuberetes, I should've hit it sooner, but it never affected my live cluster for some reason.  Also, the bug manifested in a more silent way for me.
Workaround
The workaround we made in the nixos tests earlier this year worked in my live cluster:
                kubelet.extraOpts = ''\
                  --cgroups-per-qos=false \
                  --enforce-node-allocatable="" \
                '';

Symptoms (upgrading 1.21.1 -> 1.21.2):
Upgrading to 1.21.2 as an intermediate step just causes exactly the error described in the Kubernetes thread.
Symptoms (upgrading 1.21.1 -> 1.21.6):
Upgrading to 1.21.6 (i.e. this PR) caused a weirder failure: the coredns pods would get SIGTERM'd after a while.  Sometimes they would run for a few seconds and sometimes they would run for a few minutes.  They'd serve some requests according to the logs, and then they'd report [INFO] SIGTERM: Shutting down servers then terminating coredns with no further details.  I couldn't find any indication in any of the systemd logs as to what exactly was killing the pods.
None of my other pods seemed affected, and I speculate that the coredns pods were the only ones setting CPU limits in their config, but I haven't checked this.
Anyway, the workaround worked for me.  I'm keeping my fingers crossed that this will be fixed in 1.22.


scikit.ml site is down

I believe the site is down again. I haven't been able to access it for at least a week.


Text in Square brackets is supressed

When I have text in my blog post that contains things like:
[JavaScript :{ jsname:"AngularJS"}]
They are simply deleted from the output. Maybe the token replace engine is to rigid ?

I digged a little bit more into this. In fact it is the token replace engine that strips out the square bracket statements from the content. In my opinion this is clearly a bug because tokens in the CONTENT should never be replaced. For the TEMPLATES it's ok (and mandatory for the functionality), but the content should always be left as it is!
Unfortunately I'm not able to change this by myself (too much is going on in this module, and its vb what is hard to read for me), so I leave this to you Peter

So, this did not leave me any peace and I fiddled it out:
PostsController_CRUD.vb line 29:
Public Shared Function GetPost(contentItemId As Int32, moduleId As Int32, locale As String) As PostInfo
    Dim retVal As PostInfo  = CBO.FillObject(Of PostInfo)(DataProvider.Instance().GetPost(contentItemId, moduleId, locale))
    retVal.Content = retVal.Content.Replace("[","%SQUAREBRACKETSOPEN%").Replace("]","%SQUAREBRACKETSCLOSE%")
    Return retVal
  End Function

ViewTemplate.vb line 52:
Protected Overrides Sub Render(ByVal writer As System.Web.UI.HtmlTextWriter)
   If Template IsNot Nothing Then
    Dim contents as String = Template.ReplaceContents
    contents = contents.Replace("%SQUAREBRACKETSOPEN%","[").Replace("%SQUAREBRACKETSCLOSE%","]")
    writer.Write(contents)
   End If
  End Sub

does the job. Maybe only a hack and there are better places to implement this but it works for me.

There is another issue regarding double curly braces (often used in Angular stuff). See GenericTokenReplace.vb line 73ff:
Public Shadows Function ReplaceTokens(strSourceText As String) As String
   strSourceText = strSourceText.Replace("\[", "^{{").Replace("\]", "}}^")
   Return MyBase.ReplaceTokens(strSourceText).Replace("^{{", "[").Replace("}}^", "]")
  End Function

  Public Shadows Function ReplaceTokens(strSourceText As String, ParamArray additionalParameters As String()) As String
   strSourceText = strSourceText.Replace("\[", "^{{").Replace("\]", "}}^")
   PropertySource("custom") = New CustomParameters(additionalParameters)
   Return MyBase.ReplaceTokens(strSourceText).Replace("^{{", "[").Replace("}}^", "]")
  End Function


RFC - When is the help tool expected to be publicly available?

I spoke with Tobias about this promising tool at Tech'ed in Berlin and I'd like to know in which quarter this is going to be released.

Hi Pieter,
while this tool is being implemented using UI5, it is not part of UI5, at least for the time being.
Regards
Andreas


Hongliboston patch 1

another pull request after a couple of commits

merged using Squash & merge.

reopen the pull request and use "Rebase and merge"


Fix linting to get CI to pass

The following directories need to be updated to pass pylint and enabled in CI:

[x] utils - #460 + #465
[x] changhc - #431 + #460 + #463
[ ] claims hosp - #433 + #462
[x] googlesymptoms #455
[x] nchs mortality #454
[ ] quidel - need to figure out branching?
[x] usafacts - #429

some of these will be fixed by the smoothing refactor, since those modules are throwing errors

pydocstyle open directories:

[ ]  utils
[ ] goog symptoms
[ ] jhu
[ ] nchs mortality
[ ] quidel
[ ] quidel covidtest


neo-cli preview3.0 Unable to start

dotnet version : 3.1.401
  neo> System.Collections.Generic.KeyNotFoundException: The given key 'LevelDBStore' was not present in the dictionary.
   at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
   at Neo.NeoSystem..ctor(String storageEngine)
   at Neo.CLI.MainService.Start(String[] args)
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__139_1(Object state)
   at System.Threading.QueueUserWorkItemCallbackDefaultContext.Execute()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
   at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()

Reading package lists... Done
Building dependency tree
Reading state information... Done
Note, selecting 'libunwind-dev' instead of 'libunwind8-dev'
libleveldb-dev is already the newest version (1.20-2).
libunwind-dev is already the newest version (1.2.1-8).
libsqlite3-dev is already the newest version (3.22.0-1ubuntu0.4).
sqlite3 is already the newest version (3.22.0-1ubuntu0.4).
0 upgraded, 0 newly installed, 0 to remove and 7 not upgraded.

You should add a libleveldb.dll into root directory;

You should have a LevelDBStore plugin in Plugin folder

Yes I have

neo-cli v <IP_ADDRESS>  Run normally on this machine

Nearly forgot，My System is Ubuntu 18.04

Nearly forgot，My System is Ubuntu 18.04

did you try to re-install the LevelDBStore plugin?

Nearly forgot，My System is Ubuntu 18.04

did you try to re-install the LevelDBStore plugin?

Yes，I've deleted and downloaded it several times

Are you using the data folder from a different version?

Are you using the data folder from a different version?
Maybe my system is also running a different version of neo-cli?  but neo-cli v2.11 does not set the data folder

Are you using the data folder from a different version?

Maybe my system is also running a different version of neo-cli? but neo-cli v2.11 does not set the data folder

Are you using the data folder from a different version?

I chose RocksDBStore and it won't work


GEOMESA-1123 Adding max features for bin queries

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>Note: stacks on top of #845

looks good

dc5e471cacb93c3c66a26ad0cede7a6cb8a556dd


Fix unused import & fix build for eslint packages

When this is merged, it should fix this #82

Looks mostly good to me


Issue Starting Drivers

I've tried multiple times and nothing seems to work. Any suggestions.

Did you put these drivers to binpath which you specified? And secondly you can use Dbgview form sysinternals suite to check error messages, be sure to have checked these options

Closed due to no response

Oh shit I complete forgot I had this open. I do have an AMD CPU if that's the reason it wasn't working.


Stash Maven in bins to fix #88

This fixes issues #88 by stashing Maven in /home/vagrant/chef-bcpc/bins for later use.

Tested this on my test VM. Maven tar is only downloaded once and stored in bins. Also confirmed that hannibal is built


Update scalafmt-core to 3.7.1

Updates org.scalameta:scalafmt-core from 3.7.0 to 3.7.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report
Base: 87.20% // Head: 87.20% // No change to project coverage :thumbsup:

Coverage data is based on head (5b1ba1a) compared to base (6ac7fb0).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #725   +/-   ##
=======================================
  Coverage   87.20%   87.20%
=======================================
  Files         141      141
  Lines        1524     1524
  Branches      226      226
=======================================
  Hits         1329     1329
  Misses        195      195

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Too many opened tcp connections on low load

Recently I tried to do some stress testing stuff with my application (works with RR). I used artillery (https://artillery.io) to do this. When I firstly run the tests for 60 seconds with 10 requests/sec everything was fine, but many tcp connections were opened by RR process. On a longer distance (3 minutes) errors start to happen.
2020/10/15 11:43:45 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 10ms
2020/10/15 11:43:45 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 20ms
2020/10/15 11:43:45 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 40ms
2020/10/15 11:43:45 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 80ms
2020/10/15 11:43:45 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 160ms
2020/10/15 11:43:46 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 320ms
2020/10/15 11:43:46 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 640ms
2020/10/15 11:43:47 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:48 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:49 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:50 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:51 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:52 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:53 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:54 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:55 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:56 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:57 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s
2020/10/15 11:43:58 http: Accept error: accept tcp [::]:80: accept4: too many open files; retrying in 1s

lsof -a -p $(pidof rr) shows 30 worker pipes, some other connections and loads of tcp connections like ubuntu2004.localdomain:http->_gateway:58032. I've found only a temporary solution - increase opened files limit, but I think this is not the root of the problem. Maybe these connections are not closed immediately after the response and I don't know how to fix that.

@evonicgu I may guess, that this is expected behavior because such load testing frameworks generate a huge number of TCP connections. Each open TCP connection is a file-descriptor (aka open file, active TCP socket). Also, in the case of a TCP4 listener, we use a few optimizations like SO_REUSEPORT, TCP_DEFER_ACCEPT, and TCP_FASTOPEN if acceptable. I may guess, that this tool leaves a connection open until the testing will be finished.
But, thanks for the report, I will double-check this.

Sorry, I forgot to give some more information. RR is running on Ubuntu VM, and the testing framework makes requests not from that VM, so they are separated, and tcp connection overuse is detected on VM only. Maybe, this is an issue with net/http package you are using.

We are using the standard net/tcp package. I am not very familiar with artillery framework, but can you please try wrk with, let's say 10 number of connections for the same period of time? wrk -t1 -c10 -d360s http://<IP_ADDRESS>:8080/index.html

I've tried wrk  making requests from VM, but there are still too many connections errors sometimes. lsof -a -p $(pirof rr) shows that there are some connections opened. The connection amount doesn't increase, however I don't think, that it is a good result, because when I set -c option to 1000 there are nearly 50 opened tcp connections.

Also, the performance is bad, despite the only thing the worker was doing is making a response and responding.

I've tried wrk making requests from VM, but there are still too many connections errors sometimes. lsof -a -p $(pirof rr) shows that there are some connections opened. The connection amount doesn't increase, however I don't think, that it is a good result, because when I set -c option to 1000 there are nearly 50 opened tcp connections.

Could you please provide a PHP script which you are trying to bench (if possible) and your TCP connections limit? I tried to bench with -t12 -c1000 -d60s on default worker (from docs) and everything went fine.

Also, the performance is bad, despite the only thing the worker was doing is making a response and responding.

There is no bad or good performance. You have to measure and show the stat 😄

The script is pretty simple:
<?php

declare(strict_types=1);

... (import)

require __DIR__ . '/vendor/autoload.php';

$worker = new Worker(new StreamRelay(STDIN, STDOUT));
$psr7 = new PSR7Client($worker, new ServerRequestFactory(), new StreamFactory());

while ($req = $psr7->acceptRequest()) {
    try {
        $response = new Response();
        $response->getBody()->write('index');
        $response->withHeader('Connection', 'Close');
        $psr7->respond($response);
    } catch (Throwable $e) {
        $psr7->getWorker()->error((string) $e);
    }.
}

It's very strange, that your benchmark went fine.

Let me give you benchmark results with your configuration (-t12 -c1000 -d60s).
./wrk -t12 -c1000 -d60s http://<IP_ADDRESS>:80/
Running 1m test @ http://<IP_ADDRESS>:80/
  12 threads and 1000 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency   219.82ms   81.72ms   1.03s    76.69%
    Req/Sec   364.12    196.83     1.38k    64.22%
  253575 requests in 1.00m, 34.82MB read
Requests/sec:   4219.26
Transfer/sec:    593.33KB

@evonicgu Do you use Ubuntu? Send me please your OS details.

Yep, I use Ubuntu20.04 installed on a VM powered by Vagrant + Virtualbox. Php version is 8.0 RC1. RR version is 1.8.2.

@evonicgu Few notes: PHP 8.0RC1 is not supported. RR should be 1.8.3. Also, can you please share your ulimit stats with max number of TCP connections?

Just now I'm going to switch my PHP version to 7.4 and update RR. Here is ulimit -a output.
ulimit -a
core file size          (blocks, -c) 0
data seg size           (kbytes, -d) unlimited
scheduling priority             (-e) 0
file size               (blocks, -f) unlimited
pending signals                 (-i) 7607
max locked memory       (kbytes, -l) 65536
max memory size         (kbytes, -m) unlimited
open files                      (-n) 1024
pipe size            (512 bytes, -p) 8
POSIX message queues     (bytes, -q) 819200
real-time priority              (-r) 0
stack size              (kbytes, -s) 8192
cpu time               (seconds, -t) unlimited
max user processes              (-u) 7607
virtual memory          (kbytes, -v) unlimited
file locks                      (-x) unlimited

Tested that benchmark with PHP7.4 and RR 1.8.3. Nothing changed, there are still loads of opened TCP connections.

open files (-n) 1024 I will try to test with -c 100 and open files limit 1024.

 I think they should be closed just after the response is sent.

I think they should be closed just after the response is sent.

That depends. You are making a request to the application. The application listens on a passive TCP socket. When OS handles the incoming TCP connection it will copy the passive socket, and it becomes an active TCP socket (open file in term of Linux). So, you can have only 1 passive socket (for the port, interface) and in your case (or default case) up to 1024 active sockets (in a very ideal world). The active socket not necessary will be closed right after request. It depends on the headers you send, on the application (early I mentioned our options for the TCP)

Tested on Ubuntu 20.04 with the worker in docs.
❯ wrk -t12 -c1000 -d60s http://<IP_ADDRESS>:8080/index.html
Running 1m test @ http://<IP_ADDRESS>:8080/index.html
  12 threads and 1000 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    33.19ms   11.63ms 148.13ms   85.54%
    Req/Sec     2.49k   284.46     3.47k    66.88%
  1783130 requests in 1.00m, 256.78MB read
Requests/sec:  29682.55
Transfer/sec:      4.27MB

❯ ulimit -n
1024

@evonicgu You can ask for help with your PHP worker from our PHP programmers and community in our Discord channel.

So, to have all tcp connections closed immediately after response I have to set Connection header to Close value?

That seems to be very strange, because my test with that worker was awful. Does the hardware affect the results?

That seems to be very strange, because my test with that worker was awful. Does the hardware affect the results?

For sure )

So, can you give me some info about your VM (RAM, CPU cores)?

My VM has 2GB RAM and 2 CPU cores.

By the way, what is in your .rr.yaml?

My VM has 2GB RAM and 2 CPU cores.

8 CPU cores, 64 Gb of RAM.
Here is my .rr.yaml
http:
  address: <IP_ADDRESS>:8080

  http2:
    enabled: true
    h2c: true
    maxConcurrentStreams: 128

  workers:
    command:  "php psr-worker.php"
    relay:    "pipes"
    user: ""
    pool:
      numWorkers: 10
      maxJobs:  0
      allocateTimeout: 60
      destroyTimeout:  60

That's why you have so amazing results)

That's why you have so amazing results)

We are not comparing our hardware here. It's obvious, that VM with limited resources will be slower than non virtualized host OS with better hardware.
To summarize: I could not reproduce your issue nor on VM nor on host OS. Feel free to open a new issue with steps to reproduce.

try to increase open files (-n) to much larger
every tcp connection entering TIME_WAIT takes fd

same error, because I started RR from root
Run from user and error has disappeared for me)


Update README.md

Description
Added a link for Zendesk form

Related Issue
Added a link for Zendesk form

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Lets try "For issues, bugs, questions, or feedback please file a ticket by submitting a form at https://psd-services.zendesk.com/hc/en-us/requests/new.

Done.


Changed optinal -> optional

@VesaJuvonen: Hope this is better than PR #99 ;-)

Hi @Laskewitz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Hi Daniel,
This one indeed is better than changing the generated documentation :-). We'll get it merged.


feat(Interaction): alter velocity of thrown object based on distance

Replace PR #673
Currently when throwing grabbed object, the velocity is based on center
of steam controller. Change to increase velocity based on distance
object is attached to controller.
Example: Ball at end of stick will move at greater speed,
compared to where stick is being held.
Based on article at http://www.normalvr.com/blog/throwing-throw-down/

Thanks much!
I also have code that calculates on the RigidBody AttachPoint.  And another set that reproduces the articles implementation if anyone is interested.
The one in the branch is the simplest solution though.


Add bundlesize to Travis build

The fat code shall not pass! :D
The bundlesize repo and docs: https://github.com/siddharthkp/bundlesize
There's a possibility to have a lovely GitHub status notification: https://github.com/siddharthkp/bundlesize#2-build-status

In order to have that, BUNDLESIZE_GITHUB_TOKEN env variable should be added to Travis config.

This looks like a great addition, thanks for the suggestion!
I'll probably properly integrate this tomorrow.


More detailed description for miniaturization setup

It would be really nice if the manual in the shrinking device had a detailed description of how to setup the field projectors. All you need to do is say how far apart the blocks need to be. The arrows never seem to stay on long enough to help out and it gets frustrating.

You are absolutely right, the current in game documentation about this is just horrible. Sorry!
Improvements on this are planned for the next release.


Partial for #3

I've made all refactoring I wanted.
Breaking change I've mentioned it is extracting ISection interface from Section class.
I've made it because needed to reimplement Section extension methods in PartialProxySection. Now it need only to implement ISection interface, all extension methods we'll be added.
Fixed ViewData to be more strongly typed, though some reflection is used for this.
PS Sorry for the mess of PRs.

I've found several bugs in my implementation.

Looks like binding from partial views doesn't work at all if PartialFor used. For now I'm sure that calling EditorFor for is not sufficient to work.
BeginSection in child partial works wrong if use Form.PartialFor, Looks like it writes to another TextWriter.

Fixing the HtmlHelper in ProxyForm looks like fixes issue number 2.
Though can't write test for all this issues.

#104


Sort record fields by keys after parsing type and inferring expression

I was trying out Fika for the first time and seeing this error when running the included router example:

23:00:25.921 [debug] Setting typecheck result of public function: router.routes() as {:error, "Expected type: List({method: String, path: String, handler: Fn(->String)}), got: List({handler: Fn(->String), path: String, method: String})"}

To resolve it, I modified the compiler so that fields in a Record type definition or record expression are being sorted by the key. Not sure if this is the correct fix (maybe ordering of fields is significant and Fika may want to change the internal Record representation to something different than a map?) - if this is wrong, feel free to close without merging :)

Thanks @michallepicki , I was just about to start looking into this issue when I saw your PR, thank you so much for working on this!

Thanks @michallepicki , I was just about to start looking into this issue when I saw your PR, thank you so much for working on this!

@emilsoman No problem! Just to confirm, the plan is for record fields to be valid in any order user provides, and to keep the underlying erlang map representation, right?

@emilsoman No problem! Just to confirm, the plan is for record fields to be valid in any order user provides, and to keep the underlying erlang map representation, right?

@michallepicki correct! Order shouldn't matter for Fika records and behind the scenes, it's going to be erlang maps.

@michallepicki correct! Order shouldn't matter for Fika records and behind the scenes, it's going to be erlang maps.


Optional geometry for maps without them

Some Google My Maps does'nt have a lat/lng and this script fail
EXAMPLE https://www.google.com/maps/d/u/0/kml?forcekml=1&mid=11wDBB-obMHU2jgOSRYceC6Ix8aNr1qhb

Consulting the KML documentation, the Geometry element in Placemark is always optional, so this pattern should always be permitted. I'll look into a general solution later today.

Okay, so further investigation:

'fail' in this case doesn't mean that it produces an error or crashes. It doesn't include those placemarks in the output.
The fix will be to include features with null geometries instead of ignoring those features entirely.


CA1024 - Rule no longer report on interface implementation

Hi there,
With the release of fxcop 3.X the analyzer will no longer report on interface implementation as there is no way to fix them. I am not sure whether or not you want to update the doc to mention this part.

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

ID: b128d86f-bbc8-2446-f67c-d58696605c27
Version Independent ID: 300d4a05-887f-3cd9-0161-eeb42b8c91ac
Content: CA1024: Use properties where appropriate - Visual Studio
Content Source: docs/code-quality/ca1024.md
Product: visual-studio-windows
Technology: vs-ide-code-analysis
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

Yes, I will plan to update the doc. Thx!

Looks like interfaces are not currently mentioned in the doc, so it seems OK to me not to add any new information for this.


TypeError when multi-thread inference using tensor_parallel

When inference model with multiple thread, it will throw TypeError
  File "/data/miniconda3/envs/python39/lib/python3.9/concurrent/futures/_base.py", line 446, in result
The attention mask and the pad token id were not set. As a consequence, you may observe unexpected behavior. Please pass your input's `attention_mask` to obtain reliable results.
    return self.__get_result()
  File "/data/miniconda3/envs/python39/lib/python3.9/concurrent/futures/_base.py", line 391, in __get_result
    raise self._exception
  File "/data/miniconda3/envs/python39/lib/python3.9/concurrent/futures/thread.py", line 58, in run
Setting `pad_token_id` to `eos_token_id`:0 for open-end generation.
Setting `pad_token_id` to `eos_token_id`:0 for open-end generation.
    result = self.fn(*self.args, **self.kwargs)
  File "/data/workspace/code/generate-demo/inference/starcoder_multithread.py", line 174, in gen_streamer
    outputs = model.generate(input_ids=input_ids, streamer=streamer, max_new_tokens=256, top_p=0.8, early_stopping=True, temperature=0.8, repetition_penalty=1.0)
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/transformers/generation/utils.py", line 1437, in generate
    return self.greedy_search(
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/transformers/generation/utils.py", line 2248, in greedy_search
    outputs = self(
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/tensor_parallel/pretrained_model.py", line 82, in forward
    return self.wrapped_model(*args, **kwargs)
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/tensor_parallel/tensor_parallel.py", line 129, in forward
    return parallel_apply(self.module_shards, inputs, kwargs_tup, self.devices)[self.output_device_index]
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/torch/nn/parallel/parallel_apply.py", line 89, in parallel_apply
    output.reraise()
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/torch/_utils.py", line 543, in reraise
start generate: 10
    raise exception
TypeError: Caught TypeError in replica 0 on device 0.
Original Traceback (most recent call last):
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/torch/nn/parallel/parallel_apply.py", line 64, in _worker
    output = module(*input, **kwargs)
  File "/data/miniconda3/envs/python39/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
 ...........
TypeError: '>' not supported between instances of 'NoneType' and 'int'

Hi! Mutli-thread inference isn't supported at this point. See #86


@Builder will prevent Intellij to detect unused fields

@Builder
public class Test {
private static final String notUsed = null;
}
is there a work around for this?

Unfortunately, at the moment this is not possible
I will close this issue as a duplicate, there is already more general issue #131


The cocoapod does not compile in my app.

Install via podfile use_frameworks! & pod 'CheckoutKit'
Build
26 errors such as:

/.../CheckoutKit/CheckoutKit/CardValidator.swift:75:50: Cannot invoke 'count' with an argument list of type '(String)'

Hi Rob,
What version of Swift/Xcode are you using?


🧹 Cleaning Up Third-Party Routes

⚠ CHANGES REQUIRED FOR TESTING :warning:

[x] Moved to causeId based suggestions search (NO LONGER causeName, see client/src/utils/constants to update Survey.js hard-coded causes ⚠ https://github.com/CharityFinder/CharityFinder/blob/005bb49e21548dc54b1a948eaf5805060bb9125b/client/src/pages/Survey.jsx#L97-L101
[x] Diversified suggestions response
[x] Closes #39
[x] Fixes No Results API Return Bug mentioned https://github.com/CharityFinder/CharityFinder/pull/45#pullrequestreview-644668165

lgtm! suggestions don't work atm but everything else seems to be fine


body ScrollBar disappear on clicking Text Field or TablePagination row page

hii i've an issue where i noticed in every clickable mui component so when i click for example on Text Field or Table Pagination row page the body scroll disappear any solution for it i know that in Select Component for example we use disableScrollLock but this option not working with others components

https://codesandbox.io/s/modest-hooks-s0k53z?file=/src/TableData.js:55-65
please check this demo test so when you click on row per page notice that the body scroll is disappearing

scroll bug.webm

same effect with Select and Textfield

@landerover yes sure but unfortunately i did not receive any fixing solution

@landerover thank you it works

Here is the workaround i had for this https://github.com/mui/material-ui/issues/17353#issuecomment-1401753708


taoup: 1.1.19 -> 1.21

Description of changes
https://github.com/globalcitizen/taoup/compare/v1.1.19...v1.21
(Apparently the release numbering has changed - will check with upstream.)

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build taoup

Just saw #209957 - will test that and close this one.