Avoid Sidecar Cluster Config Generation for UpstreamClusters when CredentialName is set

The solution in https://github.com/istio/istio/pull/25517 was slightly incomplete. As it doesn't avoid TLS generation error later in the function. Right now if 1000 DR are deployed with credentials, we'll see 1000 Errors in istiod logs regarding absence of tls.privatekey and tls.clientcert

@nschhina: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
e9b207781a03508318b4cd7e5de60902794915be
link
/test release-notes_istio

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

/retest

I will actually add an explicit test for this case. But yeah tests pass because the test cases I added when refactoring TLS generation to it's own function explicitly set proxy type to be Router whenever CredentialName was set

@nschhina: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
386c16407ae8139ee56b1aaca3cd5ec256e0f900
link
/test lint_istio

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

In response to a cherrypick label: new pull request created: #25902


Use views in concretization

To concretize, we need the runtime shapes, and to get them, we actually apply the lenses: https://github.com/TuringLang/AbstractPPL.jl/blob/9b64dd89dcde2dc0d8845cd01b4cc305ea83086e/src/varname.jl#L428
It would be good if this were only views. Related: https://github.com/TuringLang/DynamicPPL.jl/issues/341

It would be nice if something like Base.maybeview was implemented for lenses

Yea, I thought about this yesterday after writing. Maybe we can implement some utils locally and then ask the Setfield people whether they want them (although they explicitely exclude new stuff for Setfield, IIRC...)

Fixed by https://github.com/TuringLang/AbstractPPL.jl/pull/43?

No, it's not fixed, the implementation of get uses normal indexing (see linked issue).


Unexpected difference in width and margin percentages when using context overrides

Hi there.
Here are two examples, one where the split gutter style is set via sgs-change (the first link below), and another where the split gutter style is set with a context override (the second link below):
sgs-change: http://sassmeister.com/gist/0cee713988a358bd4a97
context override: http://sassmeister.com/gist/e260628e665fd15571ac
The only difference between the two are in lines 22 and 40 of the scss source. Shouldn't these two produce the exact same result? Is this a bug or am I doing something wrong?
Thanks!

The second one you're showing me is using a different version of Singularity than the first. Can you reproduce this using the same versions of Singularity in each?
As for why they're different, gutter-styles acts the same as grid and gutter, so simply overriding the setting doesn't work, you need to use @include add-gutter-style


I've created an example here: https://github.com/hugelgupf/gocoverdir/blob/main/main_test.go
The gist is: I'd like to be able to use GOCOVERDIR with go test -cover.
What did you see happen?
go test -cover seems to create a temporary GOCOVERDIR, but does not expose that collected coverage literally anywhere.
Clone that repo, and run the following commands:
$ go test -v
=== RUN   TestXxx
    main_test.go:18: gocoverdir:
    main_test.go:22: out: warning: GOCOVERDIR not set, no coverage data emitted
        hello world
--- PASS: TestXxx (0.28s)
PASS
ok      github.com/hugelgupf/gocoverdir 0.307s

ok, that seems expected.
$ go test -coverprofile=cover.out -v
=== RUN   TestXxx
    main_test.go:18: gocoverdir: /tmp/go-build4220322233/b001/gocoverdir
    main_test.go:22: out: hello world
--- PASS: TestXxx (0.35s)
PASS
coverage: 0.0% of statements
ok      github.com/hugelgupf/gocoverdir 0.378s

$ go tool cover -func=cover.out
github.com/hugelgupf/gocoverdir/main.go:7:      main            0.0%
total:                                          (statements)    0.0%

blowing a raspberry here. Why does go test create a GOCOVERDIR but not give me the results?
$ mkdir cover
$ GOCOVERDIR=cover go test -coverprofile=cover.out -v
=== RUN   TestXxx
    main_test.go:18: gocoverdir: /tmp/go-build4220322233/b001/gocoverdir
    main_test.go:22: out: hello world
--- PASS: TestXxx (0.35s)
PASS
coverage: 0.0% of statements
ok      github.com/hugelgupf/gocoverdir 0.378s

$ go tool cover -func=cover.out
github.com/hugelgupf/gocoverdir/main.go:7:      main            0.0%
total:                                          (statements)    0.0%

$ ls -l cover
total 0
$ go tool covdata func -i=cover
warning: no applicable files found in input directories

If I supply my own GOCOVERDIR, why is it not available?
$ GOCOVERDIR=cover go test -v
=== RUN   TestXxx
    main_test.go:18: gocoverdir: cover
    main_test.go:22: out: hello world
--- PASS: TestXxx (0.29s)
PASS
ok      github.com/hugelgupf/gocoverdir 0.315s

$ go tool covdata func -i=cover
github.com/hugelgupf/gocoverdir/main.go:7:      main            100.0%
total                                           (statements)    100.0%

the only combo that works!
What did you expect to see?
The use case of GOCOVERDIR=x go test -coverprofile is that I'd like to be able to have (a) mixed tests, and (b) be able to have one go test -cover ./... at the root of the project. This makes that hard, as a test that wants GOCOVERDIR can't be run with -cover.
In order of priority, I'd request:

Document this behavior somewhere, because I don't think either the blog post nor go cmd docs document any of this.
If you supply your own GOCOVERDIR, don't make go test set one.
If no GOCOVERDIR is set, I'd like go test -cover to set it, collect the coverage, and expose it through the old text format with a conversion.
GOCOVERDIR=./relative-path go test ./... is likely a bit weird, because it'll expect a relative-path directory in each package directory that it is running in. May be worth printing a warning and pointing the user to the go test -coverprofile=foo behavior auto-collecting GOCOVERDIR coverage in that case, as in point 3.

cc @thanm

Thanks for the report. This issue is somewhat similar in nature to the existing #60182.
When you say "I'd like to be able to use GOCOVERDIR with go test -cover" do you mean that you want all of the coverage data files generated during the test to be copied over into the directory that GOCOVERDIR points to when the test completes?
Or are you more interested in a use case like the one in #60182 where the object is to ensure when a Go test builds (with -cover) a binary and runs it, that the resulting profiles get included in the final coverage number reported by the top level test?
I agree that things are not in an ideal state, and the documentation could be better.

It's even more confusing with the -test.gocoverdir test flag, that you have you supply via -args -test.gocoverdir=foo.
I second all the above. The linked issue is also relevant but is distinct from the set of problems here.
IMO go test should directly respect -coverdir which would write new-style coverage data.


Gnosis Chain Support

Currently we do not properly support Gnosis Chain as our uniswap integration does not have a subgraph for this chain: https://github.com/polywrap/AutoTx/blob/main/autotx/utils/ethereum/uniswap/swap.py
Let's use this thread to track individual tasks needed to be done to properly support Gnosis Chain.

that's because uniswap is not deployed in gnosis chain, to fix this, i'd propose we follow path proposed in https://github.com/polywrap/AutoTx/issues/222

Sounds great, agreed. Also it's worth noting that even simple use-cases like "send 1 ETH to vitalik.eth" do not work.


Reboot Device

Hello,
Does anybody know how to trigger locally a reboot of the device?
Any suggestions are greatly appreciated!
THanks, Joachim

I guess you could try investigating the Appliance.Config.Wifi message. By issuing the wifi credentials to the device you could see if this forces a reboot.
Here a brief description of the message but you could also inspect this Meross utilities


formula: add missing extend/os require

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

This allows Formula#valid_platform? to work correctly on MacOS. It was left out of #15072 by accident. This meant MacOS only formulae were left out of search results.
How it should work
MacOS
/u/l/Homebrew (search-by-platform|✔) $ brew search xmount
==> Casks
mounty
/u/l/Homebrew (search-by-platform|✚1) $ brew search chisel
==> Formulae
chisel

Linux
/u/l/Homebrew (search-by-platform|✚1) $ brew search xmount
==> Formulae
xmount
/u/l/Homebrew (search-by-platform|✚1) $ brew search chisel
Error: No formulae or casks found for "chisel".

Thanks again @apainintheneck!


the future of qs

This module has become overly complex and difficult/annoying to maintain. I have no desire to continue maintaining it in its current state. As such, I have two thoughts that I'd like to get some feedback on.

Rewrite the module

By this I mean a total rewrite, likely removing the majority of features, and following the JSON web forms spec. Some tweaks to the spec will be made for security purposes, but aside from those it will be completely compliant. This would be a major version bump, and would remove many features.

Hand the module over to someone else

This would mean moving this module out of the hapijs organization entirely, and giving it to someone else. Perhaps @dougwilson has some interest? If not, I'm open to volunteers.
If I take the second route, the JSON web form compliant parser will be written under a totally new name and the new maintainer can choose what they want to support in this module.
I'm opening this issue to gather feedback, so please voice your opinions. I'll be making a final decision on this by the end of next week.
/cc @hueniverse @dougwilson

So if something like pillarjs/jshttp were to take over the module, it would be re-written to be JSON web form spec matching anyway, so if there is intention to keep the module as-is with it's strange quirks and stuff, it would probably be best if some other organization/person wanted to take over :)

I am dropping it as a built-in option in hapi. You will still be able to BYO query parser. I never used this module so don't care about how it evolves.
I would recommend giving this to someone else to deal with and coming up with a new name for a new module. If no one steps forward, we'll just EOL it.

I'd like to help, but frankly I don't have any experience maintaining anything past (very) small-scale projects.
If no one else volunteers and you're fine with me taking over, let me know. If not, just shut it down and move on.

I use this module in a few projects, but would be in favor of transferring it or EOL'ing it when a simplified replacement is ready. Happy to help with that if needed.

Not that anyone asked, but I've switched to https://www.npmjs.com/package/query-string for its light weight (I use it client side too) and that I don't ever need nesting. Do we need a new simplified replacement? Maybe just calling qs "done" and leaving it for those who want to keep using it is an option?

With the release of 1.0 of multer (an multipart/post-data parsing middleware for express) we switched to be W3C JSON forms compatible. The logic is contained in a separate module called append-field which is an implementation of the algorithm described in the spec, with tests to see that it works as well.
If you decide to rewrite the module, it would be very easy to implement it with append-field since it does all of the complicated logic. You just need to feed it the keys and values.

Just a side note: Calling it "W3C JSON forms" at this point might be misleading. To quote the W3C website:

Beware. This specification is no longer in active maintenance and the HTML Working Group does not intend to maintain it further.

Beware. This specification is no longer in active maintenance and the HTML Working Group does not intend to maintain it further.

Oh, wow. I think the W3C has got fed up with the endless bikeshadding just like the current qs maintainers have.

Given the lack of interest I would recommend we EOL this module. Someone can always fork it and publish something else.

Agreed, I think I'm going to EOL it and move on. I'll leave this issue open and put a notice in the README.
@hueniverse when do you plan on removing it from hapi?

Probably this week. We have a template for EOL projects in hapi. See https://github.com/hapijs/flod. Also we turn off wiki and issues when we do that.

How would someone request ownership of the module if issues are closed ? I think it's also good to leave them for historical reasons.

I don't mind leaving the issues but I am no longer going to pay any attention to this module. Who is going to keep maintaining this and deal with new issues?
If someone wants to take over this later, I am sure they can figure out who was the last maintainer from npm and contact via email.

I'd be happy to take it over, if solely to retain compatibility across browsers and older node versions.

@ljharb it's all yours

Thanks!

@nlf would you mind also running npm owner add ljharb qs?

done

<3 thanks!


Incorrect quote language in multilingual assembly

When you build multilingual documentation ([ru, en]) with the --static-content flag, Russians get into the English documentation Notes

When you build only the English version with the --lang=en flag, there is no such problem

The notes plugin supports the lang option. You can find the source code for this option here: https://github.com/diplodoc-platform/transform/blob/master/src/transform/plugins/notes/index.ts#L49.
However, somewhere in the process from the command line to the transform to the notes plugin, this option was not passed, and instead, the default value was used.


Make logos consistent

Description
Currently, "company logos" are not the daisy-collection logo in the footers folder
Benefits
Consistency
Implementation suggestions
We can replace them with the daisy-collection logo

You are right! Can you make a pull request fixing this issue?

Hey @StrawHatHacker, I don't know if you know, but it is possible to work on multiple issues in the same fork (I just learned this, I'm still discovering ways to contribute to a repository). Just create a branch in your fork for each pull request you want to make, and make pull requests through these branches instead of master. This way you would have a branch in your fork for each pull request you want to make
If you already knew this, never mind, I'm still a noob on the subject 😅

I will add this to the CONTRIBUTING guide

Noted, I'll makenew pull requests


SPV files are not built

When I reconfigure CMAKE, I see in logs that it tries to build spv files:
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/colored_triangle.frag
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/colored_triangle.vert
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/colored_triangle_mesh.vert
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/default_lit.frag
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/funky_triangle.frag
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/gradient.comp
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/gradient_color.comp
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/mesh.frag
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/mesh.vert
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/sky.comp
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/tex_image.frag
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/textured_lit.frag
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/tri_mesh.vert
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/tri_mesh_descriptors.vert
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/tri_mesh_pushconstants.vert
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/tri_mesh_ssbo.vert
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/triangle.frag
[cmake] -- BUILDING SHADER
[cmake] -- C:/Users/***/Documents/GitHub/vulkan-guide/shaders/triangle.vert

However, no spv file is added to shaders folder. Does anyone else have same problem? I am working on Windows 11 and trying all-chapters or all-chapters-1.3-wip branches. Vulkan SDK is installed as rest of c++ code is compiled normally. Any help would be appreciated.

I have no idea why CMake does not compile shaders on Windows, so, in a mean time, I installed this extension for VSCode https://marketplace.visualstudio.com/items?itemName=majicDave.vscode-shaderc and now I can compile shaders with it.

The cmake script doesnt build the shaders directly, it only searches for them to add to build. You need to compile the Shaders target in visual studio or your favourite IDE


How to use this library

Hello, I followed the steps to install the library, but when I run the code I get an error
ModuleNotFoundError: No module named 'azailib'

I double test the installation, it works well, what do you see if you run
pip show azailib ?

made some updates and solve potential installation issues


Soundcloud Bridge returns error 401

Soundcloud Bridge returns error 401
Bridge returned error 401! (18226)
Soundcloud Bridge - ID was unable to receive or process the remote website's content!
Error message: The requested resource cannot be found! Please make sure your input parameters are correct! cUrl error: (0) PHP error:
Query string: action=display&bridge=Soundcloud&u=ID&format=Mrss
Version: dev.2019-09-12]

Same here. Probably related to #1061 which was an API key that didn't work anymore. Also see https://github.com/RSS-Bridge/rss-bridge/issues/1061.
Looks like we need a new one? Last time @Roliga fixed this. Mayb you could also describe how to acquire that API key? :)
Cheers
Thomas

Eup key expired again. Current one seems to be bkcJLoXNaiFlsLaKBQXOxO5FhW0NJVnu. You can manually update CLIENT_ID in bridges/SoundcloudBridge.php for now.
To find that key manually you just visit any page on soundcloud.com, open the "Inspector"/"Developer tools" in your browser and go to the network tab. Refresh the page and look for a request to api-v2.soundcloud.com, and in that request URL you should see something that says client_id=XXX, that's the API key!
Like @TwizzyDizzy said it'd be a good idea to automate this. I'll look into that ASAP and see if it's possible in some convenient way! ^^


Schedule for Today has no room number

removed in 1962964
Due to updating studentvue.js can't figure out how to yarn add properly without errors

fixed in 7fb32644ffd4bb2417a4a8eb478ee3145fd8a465


Fix eosio.token

Enforce max-supply
Enforce same symbol (symbole_code+precision) when transfer/issuing
Fix ABI
Allow precision greater than 15 decimals
Add tests

#2291

Hope this pr be merged asap.
I want to test it.

root@b8cac977051a:~# cleos push action eosio.token issue '[ "user", "100.0000 EOS", "memo" ]' -p eosio
Error 3030000: transaction validation exception
Ensure that your transaction satisfy the contract's constraint!
Error Details:
condition: assertion failed: unable to find key
I tested this PR. And got errors above. @elmato

@Akagi201 can you post the create command you are using?

@elmato cleos push action eosio.token create '[ "eosio", "1000000000.0000 EOS", 0, 0, 0]' -p eosio.token

@Akagi201
Check that the token is created before executing the issue action.

@elmato Sure, I already created the token.
I may check it twice.


Support gas refunds

Currently when a user estimates the amount of gas required for their transaction, if they underestimate, their transaction is not executed but they still pay the gas, if they overestimate they pay more than what was necessary. Striking this balance is difficult but doesn't need to be. We can simply refund any remaining fee back to the user so they can safely overestimate and only pay for what they use.
For the implementation this would mean defining a post antehandler which is run after the transaction

Rough steps:

Test the existing behavior: https://github.com/celestiaorg/celestia-app/pull/2885
Draft a prototype of fee refunds based on gas consumption
Write a CIP
Polish prototype and get it ready to merge

During a discussion w/ @evan-forbes it became clear that "gas refunds" may not be the best way to describe this line of work to users. It's not clear why user's would receive a "gas refund". Also current name collides with the Cosmos SDK method RefundGas which actually isn't needed for this.
Since all of the tx fee is deducted from the fee payer via the ante handler, we're exploring adding a posthandler that refunds the fee payer a portion of the tx fee that is based on the gas consumption of the tx. As a result, tx fees experienced by users are gas consumption-based rather than always the entire fee.
As a result, we could describe the entire fee system after this change as: gas consumption based tx fees. The portion we're adding to the posthandler could be described as gas consumption based tx fee refunds.

if they underestimate, their transaction is not executed but they still pay the gas

Hmm I can't repro this behavior. See this pastebin

We put this line of work on pause b/c the mitigation strategies here were deemed too complicated.


Add files via upload

Please NOTE your REFERENCES for any Path you declaring to be the TRUTH!

The Straight Path can be only One! As you are in your "own internal world".
The one and only Straight Path need to be proved to be the truth!
My Truth is Submission to one and only God. I have searched alot, and this is the Only reasonable, heart touching, heart satisfiying, and overall peace and humbly path. It's basis is coming from Adam until  Muhammad (peace and blessings be upon them all), prophets and messengers who were sent to deliver the message.
Straight Path throught this life, with the goal to reach the True Infinite Paradise Life.


Deprecate util.JSONBuilder

I admit it, it was silly of me to try to reinvent the wheel.  We should be using something like Play's JsonObject to create JSON.

closed by commit a139f90870eff762dd060e95c7f7523e0e9987cb


JSON-RPC server support for multiple networks

What was wrong?
Closes #728
How was it fixed?

Add a new rpc_server and builder module in the rpc crate. Those include builder structs to initialize  ethportal-api server APIs.
Use the new rpc crate API to launch the JSON-RPC server based on a selected combination of portal networks and transports.

Note:

HTTP endpoints are tested locally with portal-hive.  ethportal-peertest IPC tests for the beacon network will be added in a separate PR.
State network endpoints will be included in the server launch when we add the API trait for the state network in ethportal-api.

To-Do

[ ] Add entry to the release notes (may forgo for trivial changes)
[ ] Clean up commit history

Well, this looks pretty great. However, I will say that this is a fairly large pr which makes it tough to review thoroughly. I'd have liked to see it split into smaller chunks and at least extract the ws stuff into a later pr (though, maybe that wasn't possible).

I know it is a large PR but it is hard to extract it into smaller PRs because the building blocks are interconnected. If you insist I can try harder but I'm not sure if we will gain much.

I'm curious as to why we add support for ws here? Sure, it's a pretty common transport to support, but imo it falls in the category of "if we don't have a use case for it, there's no reason for the extra complexity atm". Is there a use case that requires ws? In general, this project is becoming increasingly complex, and the fewer moving pieces to keep track of, the better. I can imagine scenarios where ws support might cause unexpected behavior in trin. Now, maybe this is a good thing (it exposes bugs in the core code) or maybe it's a bit of a distraction until this library reaches a more mature stage.

As discussed in the sync call, we agreed that WS would be a useful thing and I don't think it adds too much complexity. For the record, we support WS since the first transition to jsonrpsee server because it comes built-in with the HTTP server but with this PR is possible to turn it off or launch it only for a specific network.


PDF link feature

Linking in PDF is added from Chauvet 2.6.19 (internal page links and web links).
Support same feature.

Version v0.3.0 now supports internal links and web links.


Correct moduleIds schema description

What kind of change does this PR introduce?
Fixes the docs typo introduced in #7686.
Did you add tests for your changes?
N/A
Does this PR introduce a breaking change?
No
What needs to be documented once your changes are merged?
Nothing

Thanks


Test with gccrs

gccrs is GNU's Rust compiler.

Yes, but it's not usable yet: https://github.com/Rust-GCC/Reporting


[fix] Key release events no longer wrongly swallowed by swhkd

In some cases, key release events for non-modifier keys are swallowed by swhkd.
Given a simple modifier+key keybinding (ctrl+p for instance): when first pressing a key (p in this example) without a modifier, a key press event is sent to the virtual device to be consumed by other libraries and applications. Pressing the modifier key afterwards triggers the hotkey as expected. Releasing only the key at this point does not send a key release event to the virtual output, as the event_in_hotkeys boolean is true (control is still being pressed and control + p is mapped to a hotkey after all). To any library afterwards, the key is still being pressed as no release event has been received.
To fix this issue, this changes the event_in_hotkey boolean to check for each hotkey if both the event and the hotkey are of the same type. This means that swhkd no longer swallows key press events if a matching hotkey triggers on key release only and it no longer swallows key release events if a matching hotkey triggers on key presses.

@ajanon feel free to merge this, patch looks good!

I'm not sure if I'll be merging this one as-is.
This has not completely solved #166 and has brought other issues.


question

How to disable netherex ore gen?

You need to edit the biome json configs. For 1.12, they are located in your world save config folder (if "usePerWorldBiomeConfigs" is true) and/or your global config folder (if  "useGlobalBiomeConfigs" is true). For 1.14/1.15, you need to create a datapack with the ore gen config sections removed.
In 1.12, the sections you need to remove are located under the "traits" section and look similar to this:
{
	"generationAttempts": 16,
	"generationStage": "ore",
	"generationProbability": 1.0,
	"minimumGenerationHeight": 10,
	"veinSize": 14,
	"blockToSpawn": {
		"block": "netherex:quartz_ore"
	},
	"randomizeGenerationAttempts": false,
	"blockToReplace": {
		"block": "netherex:icy_netherrack"
	},
	"trait": "libraryex:ore",
	"maximumGenerationHeight": 108
}

In 1.14/1.15, the sections you need to remove are located under the "features" section and look similar to this:
{
        "feature": {
            "name": "minecraft:ore",
            "config": {
                "state": {
                    "Name": "netherex:quartz_ore"
                },
                "target": "netherrack_tag",
                "size": 14
            }
        },
        "decorator": {
            "name": "minecraft:count_range",
            "config": {
                "count": 8,
                "bottom_offset": 10,
                "top_offset": 30,
                "maximum": 128
            }
        },
        "stage": "underground_ores"
}

To create your own datapacks you can look at the 1.14 datapacks here: https://github.com/LogicTechCorp/NetherEx/tree/7efd60638856a79ca7baa351365ad55a1587615e/src/main/resources/datapacks and the 1.15 datapacks here: https://github.com/LogicTechCorp/NetherEx/tree/4b7b2faf73f098341b063741f2e30773b9a459c9/src/main/resources/datapacks.

Thanks. Where can I find the "useGlobalBiomeConfigs" parameter on 1.12.2

@LogicTechCorp Thanks :)


can we calculate text'phoneme duration time form StochasticDurationPredictor or DurationPredictor ?

amazing work! excuse me, how to extract text'phoneme duration time form StochasticDurationPredictor or DurationPredictor ? I want to extract the delay time of the phoneme corresponding to each piece of text。

You may use the attn output from the forward method as the phoneme-audio alignment result

You may use the attn output from the forward method as the phoneme-audio alignment result

thanks for replying,i will experience in inference

You may use the attn output from the forward method as the phoneme-audio alignment result

hello,i print the attn output in inferncing the model ,but I don’t know the correspondence between phoneme duration time and attn output of text，thank you for your reply


Changing mail icon

Changing the mail icon so that it'll go along with the other icons.

please assign this under hacktober


doc/mon_livre.pdf is not PDF (seems HTML)

Seems the generator outputed some HTML as doc/mon_livre.pdf

forgot a push :D


[FLINK-36355][runtime] Remove deprecated KeydStream#asQueryableState

What is the purpose of the change
Remove deprecated KeydStream#asQueryableState
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

According to offline discussion with @Zakelly, this component will temporarily not be removed until a new query state solution is proposed in Flink 2.x.


[camera] Resolve example null exception

Description
I did some automation tools, and find camera example may have null exception.
Related Issues
════════ Exception caught by widgets library ═══════════════════════════════════════════════════════
The following NoSuchMethodError was thrown building CameraExampleHome(dirty, state: _CameraExampleHomeState#5438f):
The getter 'isEmpty' was called on null.
Receiver: null
Tried calling: isEmpty

The relevant error-causing widget was:
  CameraExampleHome
When the exception was thrown, this was the stack:
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:53:5)
#1      _CameraExampleHomeState._cameraTogglesRowWidget (package:camera_example/main.dart:249:17)
#2      _CameraExampleHomeState.build (package:camera_example/main.dart:111:17)
#3      StatefulElement.build (package:flutter/src/widgets/framework.dart:4334:27)
#4      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:4223:15)
...
════════════════════════════════════════════════════════════════════════════════════════════════════

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

hi @includecmath this is now merged. I'm curious to know which automation tools you have been using?

@ened It's my homemade integration tool while learning Flutter. Are you interested? I want to share it in English, and just a chance to communicate with foreigners.😄

@ened My other Pull requests were also discovered using that integration tools. Can you help me review them?


fsmonitor: Don't watch for WRITE events (do not merge)

Switch away from github.com/fsnotify/fsnotify package to k8s.io/utils/inotify.
There is an unfortunate behaviour in the fsnotify package in that it watches for all events
on a path, and doesn't give the user the option of only watching for specific events.
This means that when watching, for example on /dev/net, any writes to /dev/net/tun will cause
a WRITE event to be sent to LXD. As you can image, for high traffic interfaces this can cause
many events and causes LXD to use a lot of CPU. This is equally the case for other devices
that can be written to, such as block devices.
The issue is identified in the upstream package (https://github.com/fsnotify/fsnotify/issues/7)
but there doesn't seem to be any consensus on how to move forward so the issue has languished.
LXD however doesn't need cross-platform support that fsnotify package offers and so can use a
lightweight wrapper around the inotify feature (which is what we use with fsnotify anyway).
There used to be an inotify wrapper at golang.org/x/exp/inotify which has subsequently moved
to gopkg.in/fsnotify.v0. Unfortunately that doesn't support go mod and so isn't usable by
LXD. Its also not been updated since 2015.
There is however a fork of golang.org/x/exp/inotify at https://github.com/kubernetes/utils/tree/master/inotify
which has seen some minor maintenance this year. This package also has go mod support.
If preferred we can easily copy this package into LXD instead.
This PR also makes the following changes:

Avoids the need to make a stat syscall to detect if event is for a directory by using the
InIsdir flag in the event Mask.
Avoids calling every registered handler for every new directory and instead performs a path
prefix match.
Improves comments and readability.
Cleans the path coming from the event to ensure comparison with registered handlers works.
Reduces repetition in event action logic.
Adds device exists detection to unixCommon handler, otherwise we were getting an error logged if a CREATE event occurred in the same directory as was being watched but for a different file.
Renames fsnotify driver to inotify.
Adds Name() function to driver.
Logs at Info level the driver used during start up.
Switches forkdns to use inotify as well.

Fixes https://github.com/lxc/lxd/issues/11151

Documentation preview available at: https://linuxcontainers.org/lxd/docs/pr.11152/


no mirrorQL and https://mirror-api.com/graphql

It seems that there is no mirrorQL and https://mirror-api.com/graphql

updated!


Example usage

Most GH actions have a ready to copy paste example in their README. We should also have an example.
According to https://docs.github.com/en/actions/creating-actions/creating-a-javascript-action#creating-a-readme the README should have:

A detailed description of what the action does.
Required input and output arguments.
Optional input and output arguments.
Secrets the action uses.
Environment variables the action uses.
An example of how to use your action in a workflow.

Once #71 PR is merged there already is test-action job in .github/workflows/testing.yml which does this, but it uses . instead of tortellini-tools/action@v1 which makes it only work on this repo and not some random repo.

We could add a workflow to test the main branch using tortellini-tools/action@main

Fixed in #92


Proposal: Save state on exit

It would be great if package optionally save state (as a .gob or .json files for example) on os interrupt signals. And restore it when run again. You can set path to save state files as a third parameter in .Start(). If it set, saving will be active.

@alehano I think I understand what you're asking. I guess one case would be, if cron job is in mid process, then there's an interruption that kills the app, ability to either pick up where things were left off or restart from the last incomplete cron job. Right?
One solution may be to take a snapshot of the cron job before processing it, in which the snapshot will expire if the job is executed successfully. This way if interruption happens, and there's an existing outstanding job (snapshot), execute it first before starting a new job.
The more I think about this, the more i feel it's on a case by case basis where each development team would want to do it in their own way. On the app level, it's simple to implement a custom process to handle these type of cases. One the package level, though, I'm not sold it's necessary.

Yeah, may be you right.

:+1:


Update license-your-installation.md

Proposed changes

Unreleased project version (optional)

Related issues (optional)

I think "Settings" is in bold because that's the label text in the UI, so this one stays as is. Thanks though!

@johndmulhausen Yes, "Settings" should be in bold, but "page" should not. So, I moved it out of the bold characters :)


Link to Attestation

Azure PCI DSS Attestation of Compliance (AoC) link doesn't work.

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

ID: 38b95bd5-5663-90ab-f902-bab7887e4489
Version Independent ID: 28d8c81c-5e29-b300-5b23-137e9692940e
Content: Payment Card Industry (PCI) Data Security Standard (DSS) - Microsoft Compliance
Content Source: microsoft-365/compliance/offering-PCI-DSS.md
Product: microsoft-365-enterprise
GitHub Login: @robmazz
Microsoft Alias: robmazz

@Lorenbain - Thank you for your feedback. I have checked and confirmed that the link is working fine and pointing to https://servicetrust.microsoft.com/ViewPage/MSComplianceGuide?command=Download&downloadType=Document&downloadId=425af30f-1236-41bc-b45c-98a52ee84c28&docTab=4ce99610-c9c0-11e7-8c2c-f908a777fa4d_PCI_DSS which is the correct resource.
From there, please click on Azure PCI DSS 3.2.1 AOC Package.zip to download the documents.

Thanks
Imran
@mypil FYI Please.

@Lorenbain - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by @joinimran  so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


[move-prover][paper] Fixing reference to Move book

Reference fix
Motivation
paper
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
NA

:broken_heart: Test Failed - ci-test

/land


Adding note that teams need repository access

Why:
It's not so obvious why a team does not show up in the possible list of reviewers.  Updating this note to also address teams, along with a link to how to fix it would have helped me
Closes N/A
What's being changed:
NOTE regarding PR reviewers was extended to include teams as well as individual authors
Check off the following:

[ ] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@deeno35
Thanks so much for opening a PR! I'll get this triaged for review :zap:

Hi @deeno35, this note is about who can request a review, not who can be requested. This information about who can be requested is just above:

Owners or collaborators can assign a pull request review to any person that has been explicitly granted read access to a user-owned repository. Organization members can assign a pull request review to any person or team with read access to a repository.

I believe that addresses your confusion so I'm going to close this PR, but let me know if I misunderstood!

this note is about who can request a review, not who can be requested
Yeah that's accurate and absolutely np if you don't find it belongs here, but I would suggest that this is a hole in the documentation overall (here or elsewhere).  Something that answers the question - How do I get my team to show up as a possible reviewer in the request reviewer area.  It is currently not a simple answer to find via search nor is it intuitive (oddly it is not intuitive, but once the solution is seen it is very obvious)

but I would suggest that this is a hole in the documentation overall

As noted in my previous comment, the explanation is elsewhere in that same article, just above the callout you edited in this PR. It states:

Organization members can assign a pull request review to any person or team with read access to a repository.

Every sentence we add to an article increases the cognitive load of reading the article, which is especially impactful for our readers with cognitive disabilities or for whom English is a second language. We therefore avoid adding duplicative content that is already found elsewhere in the article. Thanks again for your interest in our docs!


Change Serial Number prefix to SPKR

Issue

Enhancement
Following the change of accessory icon from TV to Speaker, the Serial Number should now be prefixed with SPKR instead of TV
homebridge-zp v1.2.1

OK, I found config option tvPreFix
But it doesn't work, even after a Homebridge restart

the Serial Number should now be prefixed with SPKR instead of TV

Why?

OK, I found config option tvPreFix

It's tvIdPrefix

But it doesn't work, even after a Homebridge restart

Of course not.  You cannot change the serial number of an accessory.  You'd need to remove the accessory from HomeKit, remove the accessory from the Homebridge cache (or unexpose the accessory, restart Homebridge, re-expose the accessory, restart Homebridge), and re-pair the accessory.  And what for?
The only reason I introduced this setting is so I can have a production and test installation of Homebridge ZP on the same IP network, each exposing the external accessories.  If multiple HAP servers expose the same accessory (as defined by the UUID derived from the serial number), Bonjour would choke.

In the Readme: (https://github.com/ebaauw/homebridge-zp) it states:
tvPreFix | TV | Prefix for serial number of TV accessories, to enable multiple instances of Homebridge ZP on the same network.
which is why I used tvPrefix.
I changed to tvIdPrefix and can now change the prefix of the serial number.
I guess the Readme needs updating.
And as to why do I want to do this? It's called "completeness".

I guess the Readme needs updating.

Indeed.  Unfortunately, I need to maintain this at three places: in the code, in config.schema.json (for the Homebridge UI Settings form), and in the README.  Missed the last one.


checkov: 3.2.234 -> 3.2.235

Diff: https://github.com/bridgecrewio/checkov/compare/refs/tags/3.2.234...3.2.235
Changelog: https://github.com/bridgecrewio/checkov/releases/tag/3.2.235
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 336477 run on x86_64-linux 1

2 packages built:

checkov
checkov.dist


there should be a debug mode for python-wechaty

python-wechaty is at a early stage which need more detail running detail information for debugging.  We have occured some problems that we can't reproduction error from the issue and we can't find more detail information to fix. So, we should enable debug mode for python-wechaty by default which can output all of the debug-level info into log file, so the user can easily put log file to the issues to check the error.
When python-wechaty is at a stable stage, we can change the debug mode to false by default. Should we do it ? @huan

I think one of the solution would be following the TS style: set WECHATY_LOG to enable the log output, and use redirect to save log messages to file?

@huan I think we should save log message to file at this early stage. There are some reasons:

debug information is too detaile d

I think this solution is ok for me.
Please go ahead with it.


Performance leak

We've been chasing a performance bug for the past week. While running a few hours the app cpu usage slowly creeps up to 100%. We've managed to isolate the culprit to our signalR client trying to connect/reconnect to an unreachable server (5 seconds between retries). The time between reconnects is constant, yet there seems to be some sort of performance leak (this might be a .Net core team issue).
This example is enough to exhibit the problem:
class Program {
    static HubConnection connection;
    static void Main(string[] args) {
        connection = new HubConnectionBuilder()
                        .WithUrl("http://<IP_ADDRESS>:5000/badhub") // connect to an offline server
                        .Build();
        Task.Run(ConnectAsync);
        System.Threading.Thread.Sleep(Int32.MaxValue);
    }

    public static async Task ConnectAsync() {
        while (true) {
            try {
                Console.WriteLine(DateTime.Now + " " + Thread.CurrentThread.ManagedThreadId + " Connect");
                await connection.StartAsync();
                break;
            } catch {
                await Task.Delay(0); // Set delay low to invoke leak faster
            }
        }
    }
}

We're running 2.1 preview 2 SCD'ed to linux-arm on raspian linux.
A linux perf snapshot reveals most processor time is spent at PROCRemoveThread

We're seeing the same behaviour on Darwin, but it takes a few days instead of hours to creep up. It effectively renders the SignalR C# client useless for long running processes.

@myrup and you're sure the problem isn't in your application? We actually haven't done any long running tests ourselves but we're getting that up and running soon. We'll see if we can reproduce the issue. It also might have fixed itself in the latest if you're willing to grab a ci build (look at the readme).
Does PROCRemoveThread method ring a bell @jkotas @stephentoub ?

PROCRemoveThread is called when thread exits. It walks the linked list of all threads in the process. If you are spending a lot of time in it, it means that there are likely many thousands threads in the process (ie the process is leaking threads).
Should be able to confirm this theory by attaching the debugger to the process, and checking how many threads are there are and what they are doing.

@davidfowl we’ve created the minimal sample to show it’s not our app. @jkotas The ThreadId is included in stdout to confirm that just a few threads are being cycled. Cat /proc/pid/status when process is visiting crazytown shows only 14 threads.

@davidfowl we’ve created the minimal sample to show it’s not our app.

Can you share that sample application with us?

You're not fully consuming nor disposing of the http responses, this creating a very fast leak of open sockets.

In the naive example I added and in the SignalR code, the client.SendAsync call is throwing the System.Net.Http.HttpRequestException with a "Connection refused"  and not returning an HttpResponseMessage to be inspected and disposed.  I'll look into the HttpClient code in CoreFX to see where my own misunderstanding is.

@stephentoub I can take this over to CoreFX, but as I'm looking through HttpClient on master, https://github.com/dotnet/corefx/blob/master/src/System.Net.Http/src/System/Net/Http/HttpClient.cs#L506, FinishSendAsyncUnbuffered isn't disposing of the response when it's throwing, whereas FinishSendAsyncBuffered does dispose.
My level of exposure to CoreFX code is pretty limited, so I'm not sure if there's a compelling reason for not disposing of the response in FinishSendAsyncUnbuffered without spending more time to dig in, but in the scenario I described above and your note about the response not being cleaned up, I think I can see where sockets are being left laying around when HttpClient is throwing.

FinishSendAsyncUnbuffered isn't disposing of the response when it's throwing

I'm not following; what's not disposing of what?  If await sendTask.ConfigureAwait(false) throws, there's nothing to be disposed of.  If it returns null, there's nothing to be disposed of.  If it returns non-null, we're returning it and wouldn't want to dispose it.  What am I missing?  Thanks.

Compare FinishSendAsyncBuffered's catch block (https://github.com/dotnet/corefx/blob/master/src/System.Net.Http/src/System/Net/Http/HttpClient.cs#L479-L496) to FinishSendAsyncUnbuffered's catch block (https://github.com/dotnet/corefx/blob/master/src/System.Net.Http/src/System/Net/Http/HttpClient.cs#L511-L523).
If an exception is thrown in FinishSendAsyncBuffered, the response is disposed.  The same is not true in FinishSendAsyncUnbuffered.
When the connection is refused during the course of SignalR trying to connect, the exception is being thrown from within FinishSendAsyncUnbuffered, which, unless I'm missing something, leads me to the conclusion that the response, if it's not null, isn't being disposed, as you suggested it should be.
SignalR exception stack:
System.Net.Http.HttpRequestException: Connection refused ---> System.Net.Sockets.SocketException: Connection refused
   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.CreateConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.WaitForCreatedConnectionAsync(ValueTask`1 creationTask)
   at System.Threading.Tasks.ValueTask`1.get_Result()
   at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
   at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Http.Connections.Client.Internal.LoggingHttpMessageHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken) in /Users/josh/projects/signalr/src/Microsoft.AspNetCore.Http.Connections.Client/Internal/LoggingHttpMessageHandler.cs:line 31
   at System.Net.Http.HttpClient.FinishSendAsyncUnbuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at Microsoft.AspNetCore.Http.Connections.Client.HttpConnection.Negotiate(Uri url, HttpClient httpClient, ILogger logger) in /Users/josh/projects/signalr/src/Microsoft.AspNetCore.Http.Connections.Client/HttpConnection.cs:line 302
   at Microsoft.AspNetCore.Http.Connections.Client.HttpConnection.GetNegotiationResponse() in /Users/josh/projects/signalr/src/Microsoft.AspNetCore.Http.Connections.Client/HttpConnection.cs:line 438
   at Microsoft.AspNetCore.Http.Connections.Client.HttpConnection.SelectAndStartTransport(TransferFormat transferFormat) in /Users/josh/projects/signalr/src/Microsoft.AspNetCore.Http.Connections.Client/HttpConnection.cs:line 215
   at Microsoft.AspNetCore.Http.Connections.Client.HttpConnection.StartAsyncCore(TransferFormat transferFormat) in /Users/josh/projects/signalr/src/Microsoft.AspNetCore.Http.Connections.Client/HttpConnection.cs:line 142
   at System.Threading.Tasks.ForceAsyncAwaiter.GetResult() in /Users/josh/projects/signalr/src/Common/ForceAsyncAwaiter.cs:line 39
   at Microsoft.AspNetCore.Http.Connections.Client.HttpConnection.StartAsync(TransferFormat transferFormat) in /Users/josh/projects/signalr/src/Microsoft.AspNetCore.Http.Connections.Client/HttpConnection.cs:line 116
   at Microsoft.AspNetCore.SignalR.Client.HubConnection.StartAsyncCore() in /Users/josh/projects/signalr/src/Microsoft.AspNetCore.SignalR.Client.Core/HubConnection.cs:line 130
   at System.Threading.Tasks.ForceAsyncAwaiter.GetResult() in /Users/josh/projects/signalr/src/Common/ForceAsyncAwaiter.cs:line 39
   at Microsoft.AspNetCore.SignalR.Client.HubConnection.StartAsync() in /Users/josh/projects/signalr/src/Microsoft.AspNetCore.SignalR.Client.Core/HubConnection.cs:line 64

Should HttpClient be trying to dispose of the response in FinishSendAsyncUnbuffered like it does in FinishSendAsyncBuffered, or am I completely missing some other important piece of logic?
Having said all that, interestingly, I'm still running into the same issue when issuing a SendAsync with HttpCompletionOption.ResponseHeadersRead or without, meaning, I still get "too many open files" exceptions no matter if SendAsync is finishing with a call to FinishSendAsyncBuffered or FinishSendAsyncUnbuffered, but maybe that's a result of the tight while loop.

Should HttpClient be trying to dispose of the response in FinishSendAsyncUnbuffered like it does in FinishSendAsyncBuffered, or am I completely missing some other important piece of logic?

The difference is that other stuff happens that can throw an exception in FinishSendAsyncBuffered after successfully getting the HttpResponseMessage.  That's not the case for FinishSendAsyncUnbuffered; if it successfully gets an HttpResponseMessage, it'll be returned and won't throw an exception (unless something catastrophic happens of course).

@davidfowl The sample code is included in the first post. Happy this is being looked into!

Are you only able to repro this leak when (a) you're using an IP address as the host name and (b) the connection fails?  I think there may be a leak (until a finalizer runs) in that situation in Socket itself; will investigate further.
cc: @geoffkizer

I'm currently running my naive HttpClient test I included in https://github.com/aspnet/SignalR/issues/2124#issuecomment-383618523 with http://localhost/badhub/ instead of http://<IP_ADDRESS>:5000/badhub/.  I don't have any http servers running locally at the moment.
I'm still getting the connection refused exception, but I'm not leaking file descriptors like crazy anymore.  I'm monitoring the process as it continues the tight loop, and it appears to be stable.  Looks like you're on to something, @stephentoub.

Thanks, Jodacola.  Could you also try using "<IP_ADDRESS>" but adding GC.Collect(); GC.WaitForPendingFinalizers(); into your loop?  Just for additional confirmation... I expect/hope you'll see the issue similarly go away.

I opened https://github.com/dotnet/corefx/issues/29285 for the socket issue.  Assuming that's the only problem here, this SignalR issue can probably be closed.

@stephentoub does daily build 2.2.100-preview1-008633 include your dotnet/corefx#29285  fix?
I've changed the sample app to, instead of an ip, target a domain which refuses the connection. Testing SCD'ed to linux-arm on raspian linux (have not tried other systems) leaks are still occurring (possibly different issue?).

does daily build 2.2.100-preview1-008633 include your dotnet/corefx#29286 fix?

No

@stephentoub OK. Assuming it's included tomorrow I will test and give feedback :)

It won't be; it's not in a release branch yet. If it's important to you, you can build the System.Net.Http library yourself from corefx's master branch and use that.

ps My fix addresses the socket leak, and I'm confident it does that. That's a very different manifestation than originally reported, so it's likely is not the same issue.

@myrup it could also be this https://github.com/aspnet/SignalR/commit/bd46d757ba008eac6196e0ecd8c6c9894ada831c. Try the latest SignalR builds from the dev branch (currently versioned at 1.1.0-*) https://dotnet.myget.org/feed/aspnetcore-dev/package/nuget/Microsoft.AspNetCore.SignalR.Client.

@davidfowl Initial tests shows the heavy leak is gone with 1.1.0-preview1-34041! I'll leave it running and have a party tomorrow if nothing has spilled ;)

Ready to close this as fixed by #2137 (and not a bad side effect @Jodacola / @stephentoub uncovered two other issues on the way). 🎉

uncovered two other issues on the way

Two?  I'm aware of one issue in sockets, which we haven't yet fixed but did work around in SocketsHttpHandler.  Just making sure I'm not missing something that needs to be addressed. :)

@stephentoub Ah ok, I mistook the socket workaround as a separate issue from the real socket issue. Regardless, Christmas has come early and signalR team has the honours of doing the closing of this matter ;)


After install docker run hello-world is unsuccessful

I followed the instructions for the installation below, but when I got to the docker run hello-world I got the following error:
user@computer: ~ $ docker run hello-world
Unable to find image 'hello-world:latest' locally
latest: Pulling from library/hello-world
c04b14da8d14: Already exists
Digest: sha256:0256e8a36e2070f7bf2d0b0763dbabdd67798512411de4cdcf9431a1feb60fd9
Status: Downloaded newer image for hello-world:latest
docker: Error response from daemon: Device is Busy.
See 'docker run --help'.

I checked the logs and this is what I see:
user@computer: ~ $ sudo tail /var/log/messages | grep docker
Nov  3 09:03:28 computer dockerd: time="2016-11-03T09:03:28.410929179-06:00" level=error msg="devmapper: Error unmounting device 2c4d95bcc88a425c6c2045f82b522fb2923fb941f8bb8cb9e083a38bfe4aba4f-init: Device is Busy"
Nov  3 09:03:28 computer dockerd: time="2016-11-03T09:03:28.411111247-06:00" level=error msg="Handler for POST /v1.24/containers/create returned error: Device is Busy"

Is there something that I haven't setup correctly?  One thing I'm dealing with is the installation of docker behind an authenticated proxy... this error doesn't appear to be a proxy issue.  I have set up the http-proxy.conf file as suggested in the instructions as well.
I'm have tried uninstalling then reinstalling docker along with rebooting the machine.  Does anyone have an idea of what the issue could be?
Thanks in advance.
URL: https://docs.docker.com/engine/installation/linux/rhel.md

Lots of people reported similar problems on RHEL 6. Docker on RHEL 6 is not supported. You can check out this Github issue for some info and workarounds to the problem you might be running into. If you're not on RHEL 6 and still having this issue, and those workarounds don't work for you, please check out https://forums.docker.com/ or participate in the Github issue above. Thanks!


Understanding and exploit application stage id change

the app model has been extended to include extra staging info
need to understand if anything needs to change client side (redeploy/restage?) or if we can add any extra helpful info for the user
see https://github.com/epinio/epinio/pull/1214

Updates have been made to support current functionality


Error Utilizing Local Files to Populate Navigator

I have set the navigator to use the Cyber Threat Intelligence Repository expressed in STIX 2.0. What I have done is cloned the github repo to my local instance of the navigator and did the following:

Put the files in “src/assets” in the navigator code. This will tell the server hosting the navigator to host the data as well.
Change “enterprise_attack_url” (and mobile and pre-attack depending on what you're trying to do) in “src/assets/config.json” to the path to the file (probably something like “/assets/enterprise-attack.json”
Also in that file, change “taxii_server -> enabled” to false

However, after stopping and starting/compiling ng serve --port 80 --host <IP_ADDRESS> the mitre technique matrix isn't showing.

Hey @Naroko - it sounds most likely that the Navigator is still not able to properly access the files. I'd recommend checking the paths that you're adding for enterprise_attack_url etc.
If that doesn't solve the problem, check out what errors you might be receiving in the browser console and feel free to post them here to see if we can provide further assistance.
Thanks!

@Naroko - did you end up having any luck with this?

closing due to lack of activity.


Any component that imports utils gives an error

So I'm having several issues with components that import the anything inside ./utils.
Running into trouble for the radioBtn and Dropdown. If I replace the import with the actual thing that is imported everything works like a charm.
Any idea what might cause this? I've just started a project using the latest vue-cli.

Can you try with v1.0.11 (just released)

@amanpatel worked like a charm. Cheers!


Good SDK 3.1.0 integration with Xcode 8.3.2

I have installed xcode v8.3.2 and trying to integrate good sdk v3.1.0 but i am unable to do so.When I went through release notes I found that this latest version will not support to Xcode 8.3 and later versions.But one of my colleagues is able to use the latest version of good sdk with Xcode 8.3.1 without any issue.How can we resolve this?

Hi @manishbajo2008 ,
Please post this question in the community forum here: https://community.blackberry.com/community/gdn/resources/developer_community
Thanks
Brent


temporalio/admin-tools:1.24.0 missing from dockerhub

Expected Behavior
temporalio/admin-tools:1.24.0 should be present in dockerhub
Actual Behavior
it isn't
Steps to Reproduce the Problem

docker pull temporalio/admin-tools:1.24.0

Expected Behavior
temporalio/admin-tools:1.24.0 should be present in Docker Hub.
Actual Behavior
It isn't.
Steps to Reproduce the Problem

docker pull temporalio/admin-tools:1.24.0

Troubleshooting
This issue is likely due to a missing Docker image in Docker Hub. It is important to verify the image tag's existence and the correct image name.
Verify Image Tag

Go to Temporal Admin Tools on Docker Hub.
Check the "Tags" section for the availability of 1.24.0.

Correct Image Name

Ensure that the image name is temporalio/admin-tools and not temporalio/temporal.
Refer to the Temporal CLI documentation for the correct image name and tag.

Image Build Process

Check the Temporal GitHub repository for potential updates or changes related to image builds.
Verify the Dockerfile used to build the admin-tools image.

Check for Updates

Update your Docker CLI to the latest version to ensure compatibility.
Refer to the Temporal Release Notes for recent updates and potential image changes.

Resolution

If the image tag is missing, contact the Temporal team via the Support Forum or Ask an Expert.
If the image name is incorrect, adjust your docker pull command accordingly.
If the issue persists, check the Temporal repository for potential build errors or updates.

Additional Notes

Docker Hub images can be pulled from Temporal Docker Hub
Temporal Admin Tools are used for managing and monitoring Temporal Workflows and Activities.

By following the troubleshooting steps and verifying the image details, you should be able to resolve the issue and pull the temporalio/admin-tools:1.24.0 image.

Thanks for reporting this.
The temporalio/admin-tools image is not yet available on Dockerhub for version 1.24.0.
You can use a workaround by specifying the correct Docker image tag for the current release of admin-tools.
To check the available tags, run this command:
docker pull temporalio/admin-tools:latest

This will pull the latest version of the admin-tools image and make it available for use.

Thanks for reporting this issue!
Docker images with temporalio/admin-tools tags are built on demand and pushed to Dockerhub. This means that they won't be available in Dockerhub unless you have requested the image to be built, for example by using docker pull temporalio/admin-tools:1.24.0.

Thanks for reporting this issue.
The temporalio/admin-tools:1.24.0 image is not yet available on DockerHub.
You can use the latest available version of the temporalio/admin-tools image by replacing 1.24.0 with the latest available version in the command.
You can check the available versions on DockerHub: https://hub.docker.com/r/temporalio/admin-tools/tags.
Please let us know if you have any other questions.


docs(transposable): 📖 simplify transposeBy documentation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
956

Relevant Lines:
956

💛  - Coveralls


REG_None settings have encoding errors for the value

This hasn't come up for a valid key yet but some troubleshooting should be done to see what these should look like in the resources and make that happen.

I can't find a way to reproduce this and left basically no details in the original issue. So I'm closing this until it comes back again.


Release planner - Remove current onboarding task work item

We need to decide whether we need to retain onboarding questionnaire work item and delete and alter it accordingly.

Not sure what this issue is about. @praveenkuttappan  could you clarify?

ping @praveenkuttappan


add https checking for links

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Check the HTML for the usage of HTTPS.

Good call, thanks @SMillerDev!


Develop and Run SAP Fiori Application With SAP Business Application Studio

Tutorials: https://developers.sap.com/tutorials/abap-environment-deploy-cf-production.html
Issue:
Step 2: Create dev space:
1.Select trial > Subscriptions > SAP Business Application Studio and click Go to Application.
It should be changed to:  Select trial > Instances and Subscriptions > SAP Business Application Studio and click Go to Application.
Screenshot needs to be updated.
Expected result:

Actual Result:

Labels :
Landscape Labels: trail1, trail2
SCPTest-fabap : Develop a Fiori App Using ABAP
Label: SCPTest-2102A
Priority label : 'Low'
Thanks and regards,
Farozan

Hi @FarozanFarman ,
thank you for your e-mail. We will address this issue as soon as possible.
Best regards,
Merve

Hi @MichaelCzcz ,
I created following pull request for this issue: #6552.
Can you merge?
Thank you.
Best regards,
Merve


Bluetooth: controller: split: Fix PHY update tx pause

Data transmission was paused when PHY update request control
PDU was enqueued in ULL. If there was pending data PDU in
ULL that was not enqueued towards LLL, this caused
transmission to stall.
Move the tx pause due to PHY update request/response being
enqueued to pre_tx_ack callback, this way all pending PDUs
in ULL is enqueued to LLL.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

From: Morten Priess (MTPR<EMAIL_ADDRESS>Sent: 06 February 2020 16:20
To: Chettimada, Vinayak Kariappa<EMAIL_ADDRESS>Cc: Thomas Ebert Hansen (THOH<EMAIL_ADDRESS>Subject: LL_FEATURE_RSP race condition for master role
Hi Vinayak,
We have been analyzing an issue where the LLCP state machine locks during feature exchange between master and slave.
The scenario is as follows:

Master sends LL_FEATURE_REQ
Slave sends LL_SLAVE_FEATURE_REQ
Slave sends LL_FEATURE_RSP
Master puts LL_FEATURE_RSP in queue and resets req flag (DONE)
Master host requests PHY update via HCI
Master does not process LL_FEATURE_RSP towards LLL because PHY update sets pause flag
Master does not continue PHY update procedure (for unknown reason)
For testing we added a call to ull_conn_tx_lll_enqueue() in event_phy_req_prep() - before setting the llcp_phy.pause_tx flag, thereby flusing the pending procedure.
This makes things work as expected.

We are in no way certain that this is what should be done - but it makes good sense to flush to LLL before pausing for PHY or ENC procedures.
What do you think?
Thanks in advance.
B.R.
Morten

@kruithofa please review

This PR does in fact solve the issue we have seen during testing. I have run our relevant regression tests and simulations, and everything looks good. Started an EBQ run, which will complete over night - but I don't expect any issues there either. Thanks

Unfortunately EBQ run fails the following two test cases:
LL/CON/SLA/BV-52-C.Initiating PHY Update Procedure - Packet Time Restrictions
LL/CON/SLA/BV-53-C.Responding to PHY Update Procedure - Packet Time Restrictions
I will attach the .btt and .thd files, and investigate.

LL_CON_SLA_BV_52_53.zip

This change (commit 260c40c) appears to be what causes LL/CON/SLA/BV-52-C and LL/CON/SLA/BV-53-C to fail. I was suspicious and ran those two test cases without this, and they pass. Note that the original tests still pass.

I will re-run all tests without this change, and report back.


Add parameters table1_name,table2_name in data_compare method.

Current implementation of data_compare assigns default values of 'table1' and 'table2' to the dataframes.

However, it would be good if the user specify name of the dataframe, to be able to distinguish the dataframes better.
The signature can be as follows:
distribution_compare_pretty(_df1, _df2, _df1_name='table1',_df2_name='table2', col, figsize=None, date_flag=False)

I would be happy to make the changes, if you agree.

Yes, please help create a PR and everyone can use the same function.
2020年4月12日(日) 午後10:40 Anilkumar Panda<EMAIL_ADDRESS>
I would be happy to make the changes, if you agree.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/SauceCat/pydqc/issues/23#issuecomment-612626053, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADXNPFFKLIUJCT6SX6WQTBLRMHHFXANCNFSM4MGN6LSQ
.


Need a proper Header for home page as well as common pages

Issue type

Bug

Description
Header is missing in Home page as well as all common pages
Steps to reproduce

Add proper header

Screenshots or videos
If possible, please provide screenshots or videos of the issue.

@AnupK1234 Please assign this issue to me

@AnupK1234 Can I start working on this issue ?

Hey @kumar007ambi ,
Kindly elaborate what enhancement you would like to make.

If we visit any wepage ,if we go to contact or about page you can see the header remain same just page chnages and the url of that page chnages i.e.if suppose home then the url will like some www.website/home, after going to contact the url look like www.wesite/contact. But the header remain same in the all pages.

@kumar007ambi So are you going to show which page is currently active in the header by doing some CSS changes ?

@AnupK1234 No we have to show header to all pages.

@AnupK1234 No we have to show header to all pages.
@kumar007ambi ,
Sure! You can start working on this issue. Update your codebase by pulling the latest changes made.
Kindly read the Contributing Guidelines carefully.

Thank You

@kumar007ambi
Are you doing working on this issue ?

@kumar007ambi
Are you done ?

@AnupK1234 Actually I am kinda busy with some personal work. So you can assign this issue to others.

@AnupK1234, if no one is handling this issue, could you please assign it to me? I'll take care of modifying the icon, making necessary changes to the navbar (font, color, alignment), and fixing the overlapping issue. Thanks!

@Nandini80
Sure you can start working on this.

Is it ok now?


Crash, when module is not found

Tried importing module Linalg, had no modules numpy, scipy installed, so software crashed.

I'm no sure what you mean, did you install scipy and numpy?

I'm no sure what you mean, did you install scipy and numpy?
In order to import a module you must have all required libraries installed. If that's not the case, in the current version the editor will terminate. I agree that this isn't convenient at all, sorry, so that's something I will try to change in the future.

If you try to import package, and have no module installed, you get an error and program crashes without saving. Bascially I tried to import one of the packages Linalg, but it needs extra modules to work such as numpy, scipy, thus if they are not on pc, you get an error and program crashes.

yeah, that's what I meant.  Because of those things, you should save your projects quite frequently. I'll try to change the behaviour in this case for a future release.

yeah, that's what I meant.  Because of those things, you should save your projects quite frequently. I'll try to change the behaviour in this case for a future release.

Sure, no rush, just reported an issue. Maybe consider about doing something like autosave automatically for backups etc...

fixed in 2.4.2


Magisk Module

Hi,
Can u make this for magisk? I prefer magisk over xposed because I still can play POGO while using Magisk. :)

This please.

any progress?


Cupy fusion bug

(not sure if this is supposed to go in github.com/cupy/cupy yet or if this is still the right place)
This code works:
@cupy.fuse()
def sigsbwd(gy, y1, y2):
    for y in (y1, y2):
        gy = gy * y * (1.0 - y)
    return gy
sigsbwd(gy, *ys) # ys is a list of 2 ndarrays

but this doesn't (the code generator makes two v1 variables, one of which is a &float and the other a float, and nvcc throws an error):
@cupy.fuse()
def sigsbwd(gy, *ys):
    for y in ys:
        gy = gy * y * (1.0 - y)
    return gy
sigsbwd(gy, *ys) # ys is a list of 2 ndarrays

I thought these were syntactically equivalent at the Python level, but I guess fusion is doing some fairly low-level Python manipulation and is tripped up by the varargs.

Thank you for reporting the issue!
I made a PR for this issue: #2598


8105 - Explicitly enable EXT_float_blend to silence WebGL warnings

Fixes #8105
You can test this PR by:

Opening http://localhost:8080/Apps/Sandcastle/index.html?src=Billboards.html in Firefox and observing Error: WebGL warning: drawElementsInstanced: Using format enabled by implicitly enabled extension: EXT_float_blend. For maximal portability enable it explicitly. Context.js:321:37 is printed to the console.
Building with this PR and verifying this message is no longer logged.

I'm using Firefox 72.0.1 on Windows 10.0.18363.418, Couldn't reproduce warning message in Chrome 79.0.3945.117

Thank you so much for the pull request @Samulus! I noticed this is your first pull request and I wanted to say welcome to the Cesium community!
The Pull Request Guidelines is a handy reference for making sure your PR gets accepted quickly, so make sure to skim that.

:x: Missing CONTRIBUTORS.md entry.

Please add yourself to the contributors file!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

@Samulus Works perfectly. Just one small fix needed for CHANGES.md. You can put [skip ci] in the commit message or description to avoid starting another CI build.

Confirmed that this fix works in Firefox 72.0.1 and Ubuntu 18.04.2.


Context menu keyboard shortcut rendered strangely

This is cmd+shift+1. If you hover the "replace" icon on the row, it's rendered as I would expect with 3 symbols. Technically this isn't wrong but...

IMO it's how macOS context menu works as what we pass toaccelerator is shift+cmd+1
https://github.com/microsoft/vscode/blob/730f47da21d26733734b9d3d1d361638379d9404/src/vs/workbench/services/contextmenu/electron-browser/contextmenuService.ts#L161-L162


Port must be an integer and other small improvements in Uri

@weierophinney can we keep this changes to v2.0?

can we keep this changes to v2.0?

I'd rather not.
Since the spec itself doesn't have explicit scalar type hints defined, what the constructors and various with*() methods accept is of less importance than what the accessors return. If we can massage the incoming data to the required type without data loss, we should.
If/when we do an update to the PSR-7 specification to add scalar type hints, we can revisit.

:+1:

Thanks, @snapshotpl!


Update html-minifier to v4.0.0

Coverage remained the same at 79.951% when pulling fb175cf7a9d45bea0d560912d27696e9e8405018 on ratson:update-html-minifier into 729447561d0fca865d2bb59424f2afbb0be005b0 on aui:master.


icon extractor its a optional only package

Coverage remained the same at 65.751% when pulling 913cd2918c246549c4563551a84b055db4239028 on fix-windows-only-package into c0afc0cdbf00519892c6fdb55608fda00f66ea40 on v3-feature-branch.


[Dependency Scanning] When re-using a CompilerInvocation in batch scanning, update it with the entry-specific arguments.

Otherwise, the arguments from the parent invocation of the batch scanner may overrule the arguments the batch entry expects.
Resolves rdar://74812312

@swift-ci please smoke test


Merge GO:0001034 RNA polymerase III transcription factor activity, sequence-specific DNA binding  into GO:0000995 RNA polymerase III general transcription initiation factor activity.

GO:0001034 RNA polymerase III transcription factor activity, sequence-specific DNA binding
has 3 annotations to terms that correspond to the new definition of GO:0000995 RNA polymerase III general transcription initiation factor activity.
@krchristie @ValWood @RLovering


Use variable when printing asset name

Variable asset_name should be used when printing, instead of predefined string.
https://github.com/mu373/paypaycard-fetcher/blob/88515a89ddda152a8f8fa06fcd29817b19103940/app/moneyforward.py#L85

Fixed in #3


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and publish to npm yourself or setup this action to publish automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.
Releases
<EMAIL_ADDRESS>Minor Changes

abbe9ad: - Currently if a user enters an amex card and only enters 3 digits in the CVC it is correctly marked as invalid, however, if the user then switches to a Visa card, the CVC is still marked as invalid even though a 3 digit CVC should be considered valid.

If a user enters an amex card and a 4 digit CVC of '1234' and then switches to a Visa card the CVC mask is updated and it is truncated to '123', however, the underlying value for the CVC will still be '1234' when it should b changed to '123'.

<EMAIL_ADDRESS>Patch Changes

Updated dependencies [abbe9ad]

<EMAIL_ADDRESS>

@SocketSecurity ignore<EMAIL_ADDRESS>dependency of expo in react-native-example app


Updated: atom-nightly 1.43.0-nightly7

Newer version of atom-nightly (1.43.0-nightly9) is now available in master. a90dec0fb9bcdc1eb818ae4ff92425f0ba943203


Fix typos

PR https://github.com/yaml/libyaml/pull/214 contains many of these.  Perhaps merge that first.

@perlpunk I've rebased it; workflow waiting approval again. thx

ping @perlpunk


ionic emulate android bug

https://github.com/driftyco/ionic-app-scripts
Please ensure that you are on the latest version of the CLI.
npm view ionic@latest version
2.1.18
Short description of the problem:
When running ionic emulate android - besides the fact that it only works on android 7,0.0, and fails to open on android 6 or 7.1 for some reason, it shows this screen, and when approving the app crashes:

What behavior are you expecting?
App not to crash
Steps to reproduce:
I have no idea if this is something individual or not
Post the output of ionic info below please

ordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.4
Ionic CLI Version: 2.1.18
Ionic App Lib Version: 2.1.9
Ionic App Scripts Version: 0.0.47
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.1
Xcode version: Not installed

Its because of loading times.
Added to config:


Generate Commit Message with Copilot gives incorrect commit message

Type: Bug
1 - Make some changes in a GitHub project.
2 - Hit the 'Generate Commit Message with Copilot' button
The commit message has nothing to do with the changes I made.

For example, I just made some basic changes to my app and this is the generated commit message:
"Update user authentication logic to use bcrypt hashing algorithm"
I've tried restarting the app to no avail. Every commit message is like this.
 I opened a new window and the other project is getting normal commit messages.

This just started to happen to me as well in a few projects.

@albinhubsch yes, I get that message a lot as well. Something to do with 'login' or cryptography.

I'm experiencing a similar issue. It frequently says some update was made regarding "login" or "authentication". I'm using the Insider version. I also have many repos in a workspace, not sure if that's related. The issue persists even after committing all changes in other repos.

I'm experiencing a similar issue. It frequently says some update was made regarding "login" or "authentication". I'm using the Insider version. I also have many repos in a workspace, not sure if that's related. The issue persists even after committing all changes in other repos.

I've seen this so many times on my own. It might be something worth working on

One eternity later

Since this issue has been filed we have updated both the model as well as the context users to generate the commit message. With the latest release of Copilot Chat we have also added support for specifying custom instructions when generating the commit message - https://code.visualstudio.com/updates/v1_96#_custom-instructions-for-commit-message-generation. I will go ahead and close this issue but please feel free to reopen it in case you are still hitting this issue. Thanks!


Delete row for Matrix field does not work in Firefox

Describe the bug
The delete row button in a Matrix field is behaving weirdly in Firefox, doing an api call and refreshing the view with no change
To Reproduce
Steps to reproduce the behavior:
Just try to delete a row after adding content in Advanced Forms element provided as an example
Expected behavior
Well, just delete the row :)
Desktop (please complete the following information):

OS: macOS 10.15.5
Browser 1: Firefox 77
Browser 2: Firefox Developer 78
Note: seems to work as it should in webkit-based browsers

This should be fixed in https://github.com/orchidsoftware/platform/releases/tag/7.11.2

It seems to work perfectly indeed :)

Fine. Then I close this issue.
Feel free to open it again.


In the current Version of Davinci Beta8, there is a problem that if only one refresh mode is timed refresh, all components will refresh.

In the current Version of Davinci Beta8, there is a problem that if only one refresh mode is timed refresh, all components will refresh.
A video was recorded to illustrate this.
illustrateVideo.zip

Doesn't appear in beta.9 release, please upgrade.

Beta9 works. Thanks


not working without anti ban

i just want the bot to do "owo bot" and "owo hunt." Turning off the anti ban makes it so that the bot doesn't work at all. is this intentional?

no error at all, it legit just does a few commands like
"owo daily"
"owo level"
then
'owo b"
"owo h"
after that i get no error or anything it just simply doesn't work

send me a screenshot

https://ibb.co/6n5Cc4S

fixed https://github.com/Mid0aria/owofarmbot/commit/854d3639196e0f4fc1fcb03c7491147105ea9c60


Additional endpoints to ITN graphql server

Can we add the following endpoints (from existing default graphql server) to ITN  GraphQL server?
• query { account }
• query { bestChain }
• query { block }
• query { transactionStatus }
• query { snarkPool }
• query { pooledUserCommands }
• query { pooledZkappCommands }
• mutation { sendPayment }
• mutation { sendZkapp }

Do we want the same sequence check and authorization for these as we have for the other ITN endpoints?


GIT_EDITOR=edit does not wait when running "git commit"

Setting the GIT_EDITOR to edit gives some problems, when doing e.g. git commit in a connected terminal. The commit message is opened in the kak client, but the command does not wait, till the message is written and saved.
Is there a way, to use the "connect"-edit command in such situations? Or is the only solution to unset the GIT_EDITOR variable?

You can do send git commit, but let me think more about a proper solution.

Ok I may have a solution.
How about adding a -wait switch to the edit shell command.  When specified, add a hook in the client to detach on buffer save or delete.

When I think about it, there is no need, to open the GIT_EDITOR in the same client. For me the problem is solved, if I just do export GIT_EDITOR="kak -c $KAK_SESSION", as the connected terminal has to wait for the the command to be finished anyway. So it is more convenient if I don't have to switch to the other terminal window. Or are there other use cases, where it is good to open the GIT_EDITOR in the same client?

It was requested by @vbauerster in #5.

I don't mind removing it at all, if it causing such troubles.

This tig example shows, that setting GIT_EDITOR=edit can be really useful. It might be good, if there is a convenient way, how the user can set the environment variables in a connected terminal. Maybe a "connectrc" file in %val{config}, that is sourced just after the .env files in the connect-terminal and connect-shell command.

I prefer to work file-less with an option, such as connect_environment.


fix(typo): fix metrics wording in agent

What does this PR do?
Fix metrics wording in various parts of the Agent.
Motivation
Fixing a typo.
Describe how to test/QA your changes
QA done by unit test.

/merge


refactoring: metric -> inference

We have a bunch of methods in our inference classes that do nothing but call a function in the metric directory. e.g.,
https://github.com/YosefLab/scVI/blob/e1c210135351e434aa619e72cdcf3dab04d35c3d/scvi/inference/variational_inference.py#L71-L75
It'd be clearer to just have the code the implements the method in the method -- better for reading the code not to have to jump to a different file in a different directory.
I'd be glad to get rid of the metric directory entirely, if possible.

In https://github.com/YosefLab/scVI/tree/delete_metrics, I have move most methods as you said and deleted the metrics folder.
I just move log_likelihood.py into models rather that variational inference, since it is used by the models. Is that ok ?

Great! Yes, putting log likelihood there makes sense


Global shortcut, media tracking not working in windows

Electron version: Latest
Operating system: Windows 10

Haven't tested the others functions. but: MediaNextTrack, MediaPreviousTrack, MediaStop and MediaPlayPause - do not work, my keyboard is Microsoft Sculpt if that helps

The media keys can only be registered when they are not taken by other applications, in your case I believe Windows itself has taken all of them.

So the solution is: can't be done?

I removed the keyboard software and it worked. Now i wonder how i am going to get the media tracking on my bluetooth headset to work.
But now i understand that it has nothing to do with Electron :+1:


added a demo of my app

:boom: Breaking Changes

:rocket: Enhancements

:memo: Documentation

added a demo of my app. Using vx was a blast. Specifically used stack, heatmap, and tooltip for data visualization. Please let me know what you think. Any feedback would be greatly appreciated. You all are awesome.

:bug: Bug Fix

:house: Internal

Awesome! Nice work!

Thanks! Just finished up a bootcamp... never really coded before 4 months ago.
Had a great time using VX. Any issues I can help out with? Haha, I got a lot of free time on my hands.


Sample code ImportError: cannot import name 'messages'

After installing the library, I am trying to run the sample code.
From the repository root folder:
$ examples/print_hierarchy.py
Traceback (most recent call last):
  File "examples/print_hierarchy.py", line 11, in <module>
    from systemrdl import RDLCompiler, RDLListener, RDLWalker, RDLCompileError
  File "/root/my/systemrdl-compiler/examples/../systemrdl/__init__.py", line 2, in <module>
    from .compiler import RDLCompiler
  File "/root/my/systemrdl-compiler/examples/../systemrdl/compiler.py", line 5, in <module>
    from . import messages
  File "/root/my/systemrdl-compiler/examples/../systemrdl/messages.py", line 12, in <module>
    from .preprocessor.preprocessor import PreprocessedInputStream
  File "/root/my/systemrdl-compiler/examples/../systemrdl/preprocessor/preprocessor.py", line 11, in <module>
    from .. import messages
ImportError: cannot import name 'messages'

Is this a configuration issue or a bug in the sample code?
Commit id: a1303675eee8d309db78c2e4d18ff96da1051d60

Local tests as well as seem to work.
I suspect this is also related to running using Python 3.4
If possible, see if you can use Python 3.5 or newer.
In the meantime, I'll setup up a virtualenv and see if I can work through getting 3.4 supported as it is still pretty common.

That's a good point!  I'll see if I can test this with a more recent version of Python.

Fixed. See details in #16


Typo "Azure key vault"→"Azure Key Vault"

https://learn.microsoft.com/en-us/azure/data-factory/store-credentials-in-key-vault
https://github.com/MicrosoftDocs/azure-docs/edit/main/articles/data-factory/store-credentials-in-key-vault.md
#PingMSFTDocs

@hyoshioka0128 - thanks for submission, but the branding doesn't apply here.


Update document with 'record' type

[Enter feedback here]
The "Types and Variables" section of this document needs to be updated to include the C# 9.0 'record' type.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 14618620-83fc-fd68-93a9-03626c76d680
Version Independent ID: ea7fbbaf-fd41-75cb-28fb-a2f36c8d3967
Content: A Tour of C# - C# Guide
Content Source: docs/csharp/tour-of-csharp/index.md
Product: dotnet-csharp
Technology: csharp-get-started
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for noting this @nick1941
I've added to the project for .NET 6 / C# 10. We'll make sure to get it updated by then.


Can you do a pause in Karabiner?

For example I want to wait for 0.5 seconds before issuing next command, something like this :
               <autogen>
                 __KeyToKey__
                 KeyCode::O,
                 KeyCode::ESCAPE,
                 KeyCode::WAIT_0.5S,
                 KeyCode::O
               </autogen>

try KeyCode::VK_WAIT_100MS, change 100MS to your desire time.

another way is to use applescript refer to doc
https://pqrs.org/osx/karabiner/xml.html.en
search for Using AppleScript via <vkopenurldef>

Thank you a lot @dunkarooftop.
This worked. :)


How to install/ overwrite previous version in MACOS

Hi,
I am not familiar with the programing.
A basic question here.
After I downloaded the newest release of BASIL, how do I overwrite the previous version in existing FSL 5.0.10 in MACos?
Thanks

Hi
I need to use BASIL for my PhD Thesis Analysis
I downloaded the Source Code from https://github.com/ibme-qubic/oxford_asl/releases
I wanted to know how do I install the newest release of BASIL  in Mac OS or Ubuntu.?
Thanks

Hi, apologies for the lack of response, I have not been getting GitHub notifications and a number of issues have been raised here without me realising.
Updated versions of oxford_asl and basil are available in FSL 6.0.1 and later - if you still have this issue please try upgrading FSL and remove any previous releases of BASIL.

Closing historical issue


chrome-devtools "Extension server error: Object not found: "

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Please describe your issue:
The below warnings appears every time I start an electron instance with chome-devtools enabled. This is a similar issue to #664. It has no apparent effect on the behaviour of the app.

Console output when you run electron-forge with the environment variable DEBUG=electron-forge:*. (Instructions on how to do so here). Please include the stack trace if one exists.
The following is output approx 2 seconds after the electron window opens and each line appears in approx 1 second intervals.
[7496:0214/213940.214:ERROR:CONSOLE(7830)] "Extension server error: Object not found: <top>", source: chrome-devtools://devtools/bundled/shell.js (7830)
[7496:0214/213941.213:ERROR:CONSOLE(7830)] "Extension server error: Object not found: <top>", source: chrome-devtools://devtools/bundled/shell.js (7830)
[7496:0214/213942.213:ERROR:CONSOLE(7830)] "Extension server error: Object not found: <top>", source: chrome-devtools://devtools/bundled/shell.js (7830)
[7496:0214/213943.214:ERROR:CONSOLE(7830)] "Extension server error: Object not found: <top>", source: chrome-devtools://devtools/bundled/shell.js (7830)
[7496:0214/213944.214:ERROR:CONSOLE(7830)] "Extension server error: Object not found: <top>", source: chrome-devtools://devtools/bundled/shell.js (7830)
[7496:0214/213945.213:ERROR:CONSOLE(7830)] "Extension server error: Object not found: <top>", source: chrome-devtools://devtools/bundled/shell.js (7830)
[7496:0214/213946.214:ERROR:CONSOLE(7830)] "Extension server error: Object not found: <top>", source: chrome-devtools://devtools/bundled/shell.js (7830)

What command line arguments are you passing?
electron-forge start

What does your config.forge data in package.json look like?
    "forge": {
      "make_targets": {
        "win32": [
          "squirrel"
        ],
        "darwin": [
          "zip"
        ],
        "linux": [
          "deb",
          "rpm"
        ]
      },
      "electronPackagerConfig": {
        "packageManager": "npm"
      },
      "electronWinstallerConfig": {
        "name": "forge_react"
      },
      "electronInstallerDebian": {},
      "electronInstallerRedhat": {},
      "github_repository": {
        "owner": "",
        "name": ""
      },
      "windowsStoreConfig": {
        "packageName": "",
        "name": "forgereact"
      }
    }

Please provide either a failing minimal testcase (with a link to the code) or detailed steps to
reproduce your problem. Using electron-forge init is a good starting point, if that is not the
source of your problem.
Create a new electron-forge app using the 'react' template. Then start an instance with no modifications to code.
electron-forge init my-new-project --template=react
cd my-new-project
electron-forge start

Does this happen with the default template?

I don't think this is a bug in Electron Forge. I think this is a weird behavior in either electron-devtools-installer. I tried the React template and the React tab in devtools seems to load correctly.

I just tired with the default template and the warning appears again, repeated twice this time. I will open an issue in the electron-devtools-installer repo. I may request this issue to be reopened however if they tell me is an electron-forge problem.

I'm pretty sure it's not. I searched GitHub and found this error in repos that do not use Electron Forge.

After electron-forge init myapp --template=react I was having the same problem. s a boob, having found little to help online, I threw the following hail Mary and it worked.
npm install npm-check -g
npm-check -u
using the last command above I interactively update all outdate packages to their latest versions and this error is gone now. In my case the updated packages were:
[npm-check] Update complete!
[npm-check]<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>[npm-check] You should re-run your tests to make sure everything works with the updates.
I hope this is helpful.

It worked @aplahitko


Export FileExpired for use by consumers

FileExpired is an elegant solution https://github.com/dask/s3fs/pull/428.
My problem is that it's not exported from the module so I can't catch it. Instead I'm catching its parent IOError. Am I missing something about how to import it? Was it only intended for internal use? Apologies if it's obvious to everyone but me.

Hello @jeffplourde
Sorry for the late answer.
You can use it from the s3fs.utils namespace like that:
from s3fs.utils import FileExpired

Thank you @anetbnd. I suspect my virtual environment was in a questionable state when importing failed for me previously. My apologies for the non-issue issue. Thanks for your patience.