feat(atomic): reduce bundle size by adding nanocss

https://coveord.atlassian.net/browse/KIT-3442
Adding this the dist folder go from 47.7 to 46.6 mb.

I saw many people were using nanocss + purgecss together. I tried it and it broke the dev sample. 🤷 For now nanocss only is good enough.

I also tried to fix the big css chunk in every component js file. I made this MRE and it seem to be a limitation of postcss, tailwind, stencil. Ill continue digging.
https://github.com/alexprudhomme/stencil-tailwind-postcss

Leaving traces for future possible implementations of nanocss in stencil.
Issues with this button rendering as a regular html  with only default style.
https://github.com/coveo/ui-kit/blob/45e156a66f15fa613b193119345b82bc6f25305b/packages/atomic/src/components/common/smart-snippets/atomic-smart-snippet-collapse-wrapper/atomic-smart-snippet-collapse-wrapper.tsx#L62


The "Create a channel registration" section is outdated.

The "Create a channel registration" section no longer matches Azure. The instruction says "For Location, select West US." The Azure page instead offers "Data residency" and choice "Global" or "Regional".

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

ID: 4a0e7068-9f6c-5b21-f12f-f44d27c17acd
Version Independent ID: 49c118b3-3e55-b321-26be-f4f668745f72
Content: Tutorial: Voice-enable your bot - Speech service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/tutorial-voice-enable-your-bot-speech-sdk.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @eric-urban
Microsoft Alias: eur

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

@BruceHaley
Thank you for bringing this to our attention. I've assigned this issue to the author who will investigate and update as appropriate.

@BruceHaley - Thanks for flagging this! We have an internal work item to refresh this article top down. I'll close this to avoid duplicates.

#please-close


rename noEmitOnErrors to emitOnErrors

What kind of change does this PR introduce?
refactor

Did you add tests for your changes?
updated snapshot.

Does this PR introduce a breaking change?
Maybe yes.

What needs to be documented once your changes are merged?
Optimization.noEmitOnErrors is changed to Optimization.emitOnErrors

We should not change it on webpack side, maybe we should add more meta information for cli helper

This was suggested by @sokra,
Also, with --optimization-no-emit-on-errors CLI add --no-optimization-no-emit-on-errors (maybe can confuse sometime?),
for --optimization-emit-on-errors -> --no-optimization-emit-on-errors

maybe we should add more meta information for cli helper

Information like?

Yes, I suggessed that. Negative flags are a bad practice and we best should avoid them. They are confusing to read (noSomething: false or disableSomething: false).
But this should be changed with deprecation. I also did a PR: #11193 (Didn't know that you were working on that sorry)

(Didn't know that you were working on that sorry)

No worries, I will close this one.


How to check pricing tier of existing service bus namespace

Hi. I have a service bus namespace created a long time ago. How can i check/change which pricing tier it is running in.

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

ID: 56e4bcb2-e585-af85-8abd-0bc37b2b7992
Version Independent ID: 1ce8abec-50fc-cd5b-2853-7ba2dbd01149
Content: How to create a Service Bus namespace in the Azure portal
Content Source: articles/service-bus-messaging/service-bus-create-namespace-portal.md
Service: service-bus-messaging
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@Rachitgupta1994 Thanks for the feedback! We are currently investigating and will update you shortly.

@Rachitgupta1994 There should be a 'Scale' blade in the portal for your Service Bus.
Please let us know if you are unable to see this tab.

I am able to see it. But it doesn't show options to change from say premium to basic

Looks like in order to do premium, you have to create a new namespace with it premium. You cannot "Scale" to premium from an existing Basic/Standard tier.


182054571 variable dialog in drawing tool

Adds a variable tile toolbar button to the drawing tile when specified by the unit.
Adds a mock dialog modal when the user clicks on the variable toolbar button. Dialog box is modeled after the text tool tile variable dialog modal.

@scytacki Broken cypress tests were caused by too many comments in the logged in teachers. Once the comments were deleted, cypress tests passed again.

Looks good, just waiting on the merge of master into this branch to remove the merge conflicts.

@eireland I thought I saw some cypress tests of this somewhere. Was I just imagining that?


Add Disk Manager

Add local disk manager to manage all the LMCache processes' disk read and write. Centralize the KV Cache dict and evictor.
BEFORE SUBMITTING, PLEASE READ THE CHECKLIST BELOW AND FILL IN THE DESCRIPTION ABOVE

 PR Checklist (Click to Expand)
Thank you for your contribution to LMCache! Before submitting the pull request, please ensure the PR meets the following criteria. This helps us maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Please try to classify PRs for easy understanding of the type of changes. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Core] for changes in the core LMCache logic (e.g., LMCacheEngine, Backend etc.)
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

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

Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
 Please include sufficient tests to ensure the change is stay correct and robust. This includes both unit tests and integration tests.

What to Expect for the Reviews
We aim to address all PRs in a timely manner. If no one reviews your PR within 5 days, please @-mention one of KuntaiDu, ApostaC or YaoJiayi.

Consider two scenarios:

A V100 instance and an A100 instance share a common KV Cache in float16 and bfloat16 formats. Should this KV Cache be shared between them?
Suppose a machine has two disk directories available for KV Cache storage, such as /local/local_disk and /dataheart/local_disk. Who should determine where the KV Cache is stored—the global manager or the backend within the LLM engine itself?

To address these challenges, I propose a global manager that oversees the storage and retrieval of KV Cache. This manager would determine:

Which machine, disk directory, and data type (dtype) should be used for KV Cache storage and retrieval.
Which KV Cache should be evicted when needed.

The global manager should function as an address manager, implemented in two distinct roles:

Disk Path Manager: Oversees the organization and management of storage paths for KV Cache on local or shared disks.
Distributed Router Manager: Handles routing and coordination for KV Cache storage across distributed storage servers using URLs.

Core Functions of the Address Manager:

contains(key): Checks if a specific key exists and returns the associated address (as a string). If the key does not exist, it returns an empty string.
write_end(key): Signals the address manager that the specified key is ready to be read.
read_ready(key): Retrieves the address (as a string) for the specified key. Returns an empty string if the key does not exist or is not ready for reading.
remove(key): Evicts the specified key globally.


Use latest style of react and react-bootstrap to suppress console errors

I confirm (by marking "x" in the [ ] below):

[X] This is not a feature idea. Feature ideas go here: http://www.mattermost.org/feature-ideas/
[X] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/
[X] This is a product defect issue that can be reproduced using the steps listed below.
[X] I have read CONTRIBUTING.md.

Summary
App is using old style of react and react-bootstrap, causing console errors

Operating System: Windows 7
Mattermost Desktop App v3.4
Mattermost Server v3.4

Not urgent to fix, but creating an issue for us to track

Closed by #371, thanks @yuya-oc! :)


SCSS partials not properly updating the files that import them

Which @angular/* package(s) are the source of the bug?
Don't known / other
Is this a regression?
Yes
Description
After updating to angular 17, when making changes to SCSS partials used in components, the changes do not take effect unless you go into the file that's importing them and trigger a save.
You can easily see this by going to the link for the reproduction, changing the color to something else, like "purple", and then saving. The color will not be changed until going into the component stylesheet (test-component.component.scss) where the variable is imported and used, and triggering a save there instead.
Please provide a link to a minimal reproduction of the bug
https://stackblitz.com/edit/stackblitz-starters-y1ihag?file=src%2Ftest-component%2F_test-import.scss
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in (run ng version)
Angular: 17.0.3
... animations, cli, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Anything else?
No response

I am not able to replicate this outside of stackblitz. If you are replicating it locally, can you please provide the output of ng version.

Ok, no problem. I also created a brand new app and replicated the issue outside of stackblitz (https://github.com/brianmtreese/scss-partials-issue).
Angular CLI: 17.0.1
Node: 18.18.0
Package Manager: npm 10.2.1
OS: win32 x64
Angular: 17.0.3
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
Package                         Version
@angular-devkit/architect       0.1700.1
@angular-devkit/build-angular   17.0.1
@angular-devkit/core            17.0.1
@angular-devkit/schematics      17.0.1
@angular/cli                    17.0.1
@schematics/angular             17.0.1
rxjs                            7.8.1
typescript                      5.2.2
zone.js                         0.14.2

Is this still an issue for you with<EMAIL_ADDRESS>

Nope, it appears to be fixed now. Thanks!

Nope, it appears to be fixed now. Thanks!

Good to hear and thank you for confirming.


Wrong characters in etc."xyz".source causes system crash / unbootable system

Describe the bug
Incorrect input to the source attribute for environment.etc files can cause a system crash and make a system unbootable.
To Reproduce
Steps to reproduce the behavior:

create an etc file with a source path that includes a newline (and possibly other whitespace is dangerous here as well) via:

    environment.etc."example.conf".source = (
        pkgs.writeText "example.conf" "my example" + "\n" + "more config");

nixos-rebuild
watch your system go down in flames

Expected behavior
This should trigger an error during the evaluation. I think "sources" in etc.nix must consist of proper paths (the types.path type is quite naive) and in this case for the etc derivation (there's also a "Use toXML" not there) we also trust that this is actually reasonably well quoted for bash word splitting.
We could fix this immediate issue by prohibiting newlines in paths, but I haven't been able to check for the whole chain of bash quoting here and we might need to prohibit other characters, too. I'm not sure whether all of this needs fixing in types.path or whether shell quoting at the appropriate places in the derivation would be needed as well.
Screenshots
If applicable, add screenshots to help explain your problem.
Here's what a dry-activate looks like:

Additional context
The original that made me find this was:
    environment.etc."ceph/ceph.conf".source = (
        pkgs.writeText "ceph.conf"
        cfg.config + "\n"
        + cfg.client.config);

I thought the operator precedence was + before parameter binding, but it turns out I was wrong. So this example really is:
    environment.etc."ceph/ceph.conf".source = (
        (pkgs.writeText "ceph.conf"
        cfg.config) + "\n"
        + cfg.client.config);

But I expected it to be (and this then works properly):
    environment.etc."ceph/ceph.conf".source = (
        pkgs.writeText "ceph.conf"
        (cfg.config + "\n"
        + cfg.client.config));

This results in the "etc" derivation record the content of the client config as source path, which looks quite funny:
("sources","/nix/store/cp12kkzly00bmykrlry2ihc0py02iz9y-nix-phps /nix/store/8vzivf5zby0dacmm9s3rj7jrdvf96rw3-etc-bashrc /nix/store/g28l15mbdbig59n102zd0ardsfisiw32-binfmt_nixos.conf
/nix/store/j9p8xsfbzfi3fjzjq3npa0fp2grxanw7-etc-ceph.client.admin.keyring /nix/store/fmvadclsh0svrmyl7jmnldvkjak8flx6-ceph.conf\n[mon]\nadmin socket = /run/ceph/$cluster-$name.asok\n\nmon
addr = <IP_ADDRESS>:6789,<IP_ADDRESS>:6789,<IP_ADDRESS>:6789\nmon data =
/srv/ceph/mon/$cluster-$id\nmon osd allow primary affinity = true\n
mon pg warn max per osd = 3000\n\nosd pool default size = 3\n
osd pool default min size = 2\nosd pool default pg num = 64\n
osd pool default pgp num = 64\n\n[mon.cartman02]\nhost = cartman02\n
mon addr = <IP_ADDRESS>:6789\npublic addr = <IP_ADDRESS>:6789\ncluster addr = <IP_ADDRESS>:6789\n /nix/store/2r74xqz5xj1lv642j7cr7a7qx6jq92s6-crontab
 /nix/store/zhn865282k7m51g8sa85y6bs2qs530fs-dbus-1 /nix/store/gzs0sknys6a2li8iw8xb

And that then results in the etc hierarchy not being properly built and systemd missing the units.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Looks like the problem is https://github.com/NixOS/nixpkgs/blob/ef9be9288ba44656f5b9b9f0f64ff50dcfd3f1c2/nixos/modules/system/etc/etc.nix#L19-L24, which just passes these variables as a list of strings to a derivation, which then uses toString to turn the list into an string to be a variable value in bash. toString notably concatenates entries with a " ". Here's the bash that receives these, after having went through toString:
https://github.com/NixOS/nixpkgs/blob/ef9be9288ba44656f5b9b9f0f64ff50dcfd3f1c2/nixos/modules/system/etc/make-etc.sh#L5-L13
And reading this into a bash array will split elements with both \n and
So, with two etc paths, one sourced at /hello\nthere and another at /foo

Nix passes sources = [ "/hello\nthere" "/foo" ]
The derivation turns this into "/hello\nthere /foo" via toString
Bash reads this into an array via sources_=$($sources), ending up with sources_=(/hello there /foo)
This is three elements, not two!

So, finally, the for loop in bash assumes that there's a one-to-one mapping for each of the sources_, targets_, etc. arrays. But this extra entry that was introduced messes this up, causing all etc entries from a certain point on to be mismatched with their sources!
Check this mess out:
$ cat configuration.nix
{
  fileSystems."/".device = "/o";
  boot.loader.grub.device = "nodev";

  environment.etc."hello".source = "/hello\nthere";
}
$ nix-build '<nixpkgs/nixos>' --arg configuration ./configuration.nix -A system
/nix/store/wzd40nwjhi0yps66vkbjvcgcnxvcaz38-nixos-system-nixos-21.11pre-git
$ cat result/etc/os-release
passwd:    files systemd
group:     files systemd
shadow:    files

hosts:     files mymachines dns myhostname
networks:  files

ethers:    files
services:  files
protocols: files
rpc:       files

/etc/os-release contains the contents that should normally be in /etc/nsswitch.conf!

Thanks for digging into this - and yeah, that completely fits my expectation of what's happening ... /o\

Thanks for looking into this! Personally I'd prefer the second approach and it looks solid to me. I'd be happy to contribute a test case (which should be simple enough by just proving that the system is able to boot through the first activation properly).
The scalability issues shouldn't be much of an issue. If they do become an issue then I guess the most likely hotspot would be the escapeShellArgs usage.


Find better name for the project.

Ideas:

stellar_prefilter, local_prefilter

straight-forward but kind of long

FILS (pre-FIlter for Local sequence Similarity search)

similar to how they named DREAM-Yara, Lambda etc

valik

this one means "filter/selection" in Estonian but should still be easy to spell/pronounce for anyone

I like the idea of the Estonian one, but I also like the really obvious ones.
stellar_prefilter might be too specific, as the method itself is more general for local alignments. On the other hand we could also optimize the use case for stellar if we name it as such.
Maybe tähe-valik xD


graph parameters do not match URL for some events

Create a graph
Copy the URL from the browser URL entry, see below
Paste the URL into an new browser URL entry
"dial" and "incoming-dial" is added to the events being graphed
URL in the browser URL entry is unchanged
Strangely this doesn't reproduce if the only event chosen is "wildcard-line-play-four". It does reproduce if the only event chosen is "outgoing-dialtone" or "futel-conf-dial", and the events in my URL also have "dial" substrings.
 I think the pipe delimited event list isn't being split, and is then being passed into setter thing as a single string, so the substrings "dial" and "incoming-dial" are substring matching instead of checking list membership.
I think I can fix this pretty simply.

Confirmed fix. You might have to force a full page refresh so that the js bundle is fetched, but it fixes this. :beers:


Listen statechange on waiting as well

Prior to filing a PR, please:
Fixes #2199
R: @jeffposnick @philipwalton
Description of what's changed/fixed.
Changes as described in https://github.com/GoogleChrome/workbox/issues/2199#issuecomment-526790836
Writing tests with my scenario.

Coverage remained the same at 79.314% when pulling f4b6bb6e298f070b40a0bc54ab94d5a18aed2915 on azizhk:statechange_waiting into ae80e3958c79cb93f4c21bca63cd6e81f7878536 on GoogleChrome:master.

Hi @philipwalton,
I don't think this solution works. I'm still missing something. I applied the patch to my bug reproduction repo, but it did not fix there.
Now not getting any events getting triggered.

So I realized that our problem would also occur for installing service workers as well.
I wrote some tests but I am having some problems getting them working on all browsers.

Hmmm, regarding triggering an installing event with wasInstallingBeforeRegiser:true, I'm not sure how much that makes sense. I agree it can happen in the case in your example in #2199, but in order for it to happen you have to delay calling register(), and in general I'm not sure if that's a pattern we want to promote or support with workbox-window.
Can you elaborate as to why you want to do this?
@jeffposnick what are your thoughts here? Are you aware of reasons (for or against) delaying the register() call? If we choose to support this use case then we'd have to make a lot of changes to the Workbox class, because at the moment it doesn't add any lifecycle listeners until register() is called, which means updates triggered by the browser's soft update checks would be missed.

Well I don't necessarily need to trigger installing. I just added it to keep parity with waiting.
But I would like to add statechange listener on the installingSW.
This is because the user/developer can call register at any point. (After requestIdleCallBack, etc.)
And this can cause race conditions with browser's soft update.
I'm ok with not adding any lifecycle listeners until register() is called. It's just that documentation needs to mention that one can't expect all events to be triggered in sequence.
They can expect workbox to trigger "waiting" & "activated" events (as that covers majority use case)
But for "installing" & "installed" they should check registration.installing as well as listen for these events.

Also I've add statechange listeners on both waiting & installing SWs if both exist.
I think we should add them on all this._ownSWs that you are adding in #2211 diff

Hey @azizhk after discussing with @jeffposnick I've filed #2216 to track support for issues and API changes surrounding conditional registration.
Can we keep this PR focused on the original problem where service workers that were waiting before register do not receive any additional lifecycle events?
Then we can discuss in #2216 how best to update workbox-window to handle conditional/deferred registration.

Sure, I've reverted the code. But am not sure why tests are failing on "Safari Stable"
I'll also isolate my other failing test in a different PR.

Hi there,
Workbox is moving to a new engineering team within Google. As part of this move, we're declaring a partial bug bankruptcy to allow the new team to start fresh. We realize this isn't optimal, but realistically, this is the only way we see it working. For transparency, here're the criteria we applied:

For PRs, we closed everything that doesn't target the current v7 branch.
For Issues, we closed everything that was created before 2024.

Thanks, and we hope for your understanding!
The Workbox team


How do you delete a specific interactive tool?

I have multiple trendlines and Fibonacci up and I just want to delete the specific one I've selected.  Currently the example charts only delete the last trendline in the array.
Can anybody help me out please?
Thank you very much in advance.

if you have 5 trendlines, and wanted to remove the 2nd, then you would have to modify the array without the 2nd trendline. The examples do not show advanced usecases, only simple ones, but I was hoping it would give an idea of how to achieve a more complex requrement


[Python] Add fixit.

Fixit allows the creation of custom lint rules, just like flake8 or pylint. In addition, fixit allows developers to apply code transformations, similar to 2to3.
flake8/pyflakes and pylint use the ast or astroid modules to analyze the source code. The resulting abstract syntax tree is missing information, such as whether a string was surrounded by single or double quotes, or the number of whitespaces between two tokens. Fixit is based on libCST, which uses a concrete syntax tree as its base representation and retains all formatting.
Therefore, Fixit is not just a framework for lint rules, but a framework for code transformations. This can be used to enforce coding conventions or to ascertain certain properties of the source code like HTTP request timeouts.
Although Fixit is relatively new, the underlying libCST library is quite mature and does most of the heavy lifitng.

Thanks for the PR @seifertm. Thanks also for adding resources. 👍


Vote and Answer counts in "Related StackOverflow Questions" sidebar out-dated

The vote and answer counts of the Related StackOverflow Questions (sidebar in docs pages) are out-dated.
So for example the Undoing a git rebase question linked here shows
160 votes / 7 Answers
but in fact has 690 votes and 7 answers today.
Instead of being hardcoded here you should use the StackExchange API to get the current vote and answer counts (or at least update them regularly with a cronjob).
This of course is not a real issue, rather a feature request.

I think this sidebar is gone entirely since b6a61b9dc9523041ccc8ab9ecd1d73e9454132a1.


fix(boolean ops): fix && ops

Thanks for the awesome extensions! We just started using your library and came across a minor typo. This PR will fix it.

Oh wow, nice catch! Thanks @ngochai94! I'll release a fix asap.

@ngochai94 0.12.2 is released and should be on maven central. Thanks again!

The release was bumpy this time (sonatype returned a weird error at the end of the release process) – but it looks like it was successful after all. Please let me know if something's not right.

thanks @yurique !


Error messages in form_validation_lang.php not contained in a namespace

For example, it should be:
$lang['form_validation_min_length']="The %s field must be at least %s characters";

Rather than:
$lang['min_length']="The %s field must be at least %s characters";

I see your point, maybe possible in CI3.


[Question] How can I set,that I have the voice changer as a different input?

I wanna set for example,that discord uses the voice changed input,but obs the normal microphone.

This is possible by changing the input microphone in the application specific settings, or by using pavucontrol under the "Recording" tab. Lyrebird will be named as "Lyrebird" or in some cases "Null output".


[Device Support Request] Develco Keypad KEPZB-110

Hello, I was hoping to get support added for a Develco Keypad KEPZB-110. This keypad also features RFID support, proximity and light sensor as well as tamper switch. As far as I know it is standard Zigbee 3.0 (although I could be wrong).
So far in Home Assistant 2021.5 I can add it, and it indeed shows up as an alarm, although ias_zone doesn't work, either does battery sensor or on_off switch (not sure what that is?):

The alarm component works and the state etc works properly, you can set the alarm via the keypad or via HA and all functions correctly, except I have not found a way to change the default code from 1234.
Device signature:
{
  "node_descriptor": "NodeDescriptor(byte1=2, byte2=64, mac_capability_flags=128, manufacturer_code=4117, maximum_buffer_size=82, maximum_incoming_transfer_size=1500, server_mask=11264, maximum_outgoing_transfer_size=1500, descriptor_capability_field=0, *allocate_address=True, *complex_descriptor_available=False, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=True, *is_full_function_device=False, *is_mains_powered=False, *is_receiver_on_when_idle=False, *is_router=False, *is_security_capable=False, *is_valid=True, *logical_type=<LogicalType.EndDevice: 2>, *user_descriptor_available=False)",
  "endpoints": {
    "1": {
      "profile_id": 49353,
      "device_type": "0x0001",
      "in_clusters": [
        "0x0005",
        "0x0006"
      ],
      "out_clusters": []
    },
    "44": {
      "profile_id": 260,
      "device_type": "0x0401",
      "in_clusters": [
        "0x0000",
        "0x0001",
        "0x0003",
        "0x0020",
        "0x0500"
      ],
      "out_clusters": [
        "0x0003",
        "0x000a",
        "0x0019",
        "0x0501"
      ]
    }
  },
  "manufacturer": "Develco Products A/S",
  "model": "KEPZB-110",
  "class": "zigpy.device.Device"
}

Additional context
I also captured some events from zha_events listener of arming and disarming in case that was helpful:
{
    "event_type": "zha_event",
    "data": {
        "device_ieee": "00:15:bc:00:43:00:05:52",
        "unique_id": "00:15:bc:00:43:00:05:52:44:0x0501",
        "device_id": "75f001581c1c89353997bdf30c9c956e",
        "endpoint_id": 44,
        "cluster_id": 1281,
        "command": "arm",
        "args": {
            "arm_mode": 0,
            "arm_mode_description": "Disarm",
            "code": "1234",
            "zone_id": 0
        }
    },
    "origin": "LOCAL",
    "time_fired": "2021-05-28T16:15:46.496075+00:00",
    "context": {
        "id": "3d4100541614489dc4faa9a259eb1b40",
        "parent_id": null,
        "user_id": null
    }
}

{
    "event_type": "zha_event",
    "data": {
        "device_ieee": "00:15:bc:00:43:00:05:52",
        "unique_id": "00:15:bc:00:43:00:05:52:44:0x0501",
        "device_id": "75f001581c1c89353997bdf30c9c956e",
        "endpoint_id": 44,
        "cluster_id": 1281,
        "command": "arm",
        "args": {
            "arm_mode": 3,
            "arm_mode_description": "Arm_All_Zones",
            "code": "1905",
            "zone_id": 0
        }
    },
    "origin": "LOCAL",
    "time_fired": "2021-05-28T16:15:32.102086+00:00",
    "context": {
        "id": "27e8fba46f028ee0c28bdb50fa07e26a",
        "parent_id": null,
        "user_id": null
    }
}

If I can provide anything else please let me know. There is also a technical manual here which has the Zigbee commands and cluster info.

Just to add, I figured out how to change the code, didn't realise/notice there was a new option in the ZHA configuration menu to change it since the last release, good job!
Do you think it's possible to get the other stuff working?

Hi there, I have the same device and after a night of breaking my head trying to find the basic instructions, I am in the same state as you are.
I can see the events coming in, can lock and unlock the alarm, but all the other stuff is not working.
I might be willing to make the custom code for this device myself, but I've never done that and wouldn't know where to start testing etc.

@SanderLam : On the Integrations page (Devices & Services), locate the ZHA integration and press 'Configure'. There is an option here called 'Alarm Control Panel Options'. I assume this is the place @EverythingSmartHome meant in his reply.
I have the same Keypad in my cabin, and will try it out the next time I'm there.


Migrate 7650 7800 7795 7893 7944 8064 8075

What problem does this PR solve?
Migration status_server related PR.
The migration work is not smooth, I manually resolved many conflicts, so I need the original author’s review.
close #8077 #8078
Check List
Tests

Unit test
Integration test

Side effects
No
Release note

Move config of encryption to under security config, i.e. [encryption] becomes [security.encryption] in tikv config.
Fix "Invalid cross-device link" error when trying to rewrite config file
Fix wrong toml format cause by rewrite config to empty file
Update the dependency of the status server to resolve a memory safety problem.
Add --advertise-status-addr flag to specify the status address to advertise.
Allow wildcard address for advertise-status-addr to keep compatible with tidb-operator.
Skip CN (common name) check when accessing metrics endpoint when TLS is enabled.
Fix the issue that TiKV cannot connect to PD when TLS is enabled.

/run-all-tests

@NingLin-P Seems that DCO bot is complaining

/run-all-tests

@NingLin-P Seems that DCO bot is complaining

It is an old issue of the DCO bot, you can use git reset d72ba17 --soft uncommit my pr and commit the change again with git commit -s -m "xxx".

Please fix the DCO issue

Please fix the DCO issue

Ok, I'm about to do this

/run-all-tests

/test

/run-all-tests

/rebuild

/run-all-tests

/rebuild

/test

/rebuild

/merge

/run-all-tests

/run-all-tests

/merge

@hunterlxt Please fix the conflict

/run-all-tests

/merge

/run-all-tests

/merge

/test


Add coingecko id and serum market id

Hi, team!
We added coingecko id and serum market id of RATIO token.
This is the PR.
https://github.com/solana-labs/token-list/pull/25335
Best regards.

Hi, team!
Please help us! We love solana and you!
We are in hurry mode. Thanks.

Hi, Team!
Hi, @aeyakovenko, I admire you.
Would you check this issue?
Thanks to you, We can have the fastest blockchain and can launch a MVP version of our platform.
Then, We need this issue to be resolved.
Thanks in advance.


3.0 preview 6 release

Is https://github.com/dotnet/source-build/commit/f00f8fef779a3a788866cc82eb2b6fee319e4d25 good to go? Could it be tagged as preview 6?

Sorry about that, yes, it's ready.  Tagged as v3.0.100-preview6-012264-SDK.

Thanks for tagging it!


BeIN Sports (Qatar)

Issues with this epg no longer working.
It's still accessible via Google search but not being directed to by specific apps.

The guide has been removed from the list because there is no current program on the site: https://www.beinsports.com/en/tv-guide

It's here?
https://www.bein.com/en/tv-guide/?c=gb

Still waiting for this. It's been almost a month without a fix.


[Android] Fix the lint warnings for Media classes.

Adapt to:
https://codereview.chromium.org/1506013009
https://codereview.chromium.org/1588503004
Some interfaces for media were added from API level 16, we need to check
it at runtime otherwise app will be crashed when it runs on ICS.
Add the checks for API level, if API level is less than JELLY_BEAN,
return default value.
BUG=XWALK-6494

What is the impact of essentially always returning false or empty lists on ICS? It looks like we're basically making some functionality not work as expected.

@rakuco The methods being modified this time are like canDecode or getSupportedCode, but not something must expect clear result. Even the device is above 4.0, it also could return false. And each of these methods give default return value too.

@wuhengzhi Based on your changes, I can see that if the device is 4.0, the method almost do nothing. For these situations, it's better to check the version and return default value at first to reduce the code change, also reduce the possibility of further conflict.

@lincsoon Yes, thanks.

@rakuco @lincsoon PTAL, thanks.

LGTM

@rakuco The methods being modified this time are like canDecode or getSupportedCode, but not something must expect clear result. Even the device is above 4.0, it also could return false. And each of these methods give default return value too.

My question was more along these lines: did the code that uses these new methods receive a different, working value before M49? In other words, does this new code result in things that used to work before not working anymore?

@rakuco All of the checks just for the API level is less than Jelly Bean.
Before M49, if app runs on Android system which version is less than 4.1, it will be crashed without these checks.

In other words, does this new code result in things that used to work before not working anymore?

So these checks won't impact the result which API level is equal or greater than JB.

@rakuco All of the checks just for the API level is less than Jelly Bean.

I know, I'm only talking about the ICS case which is the reason we need to change this code in the first place.

Before M49, if app runs on Android system which version is less than 4.1, it will be crashed without these checks.

What I've been saying from the beginning: this code as-is does not exist before M49. There were no lint issues in the media code before M49. So what I'm asking is if, compared to the previous version of the code that did not use API not present in ICS, this patch reduces functionality on ICS.


[ALLUXIO-2754] Make FileBlockInfoTest final

https://alluxio.atlassian.net/browse/ALLUXIO-2754
Make FileBlockInfoTest final

alluxio-bot, check this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15545/
Test PASSed.

@JayceSYH This does not look like the described change, could double check your commit?

Closing as this is addressed by #5518.


Add csinodeinfos rbac to attacher

The csi-attacher pod complains that the service-account it uses doesn't have access to csinodeinfos.
Before merging need to check whether it validates on k8s versions before CSINodeInfo was introduced (Alpha in 1.13)

Thanks. I've added the same permission for the operator so that it can grant those same permissions to the attacher.

Before merging need to check whether it validates on k8s versions before CSINodeInfo was introduced (Alpha in 1.13)

It did work fine on k8s 1.11 as per the CI.


Added CodeIgniter 4

Congrats! I'm glad you've made this after all :clap:

Well, v3 won't be but v4 most definitely! Thanks for spearheading this.

Great to hear that :+1:
To bad it's not the same repository as other projects, would save me lot of time and work.
I'll get to the PHP 7.1 issue in couple of days


Update cats-core to 1.4.0

Updates

org.typelevel:cats-core
org.typelevel:cats-laws
org.typelevel:cats-testkit

from 1.1.0 to 1.4.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Codecov Report

Merging #109 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #109   +/-   ##
=======================================
  Coverage   91.64%   91.64%
=======================================
  Files          63       63
  Lines         993      993
  Branches       53       53
=======================================
  Hits          910      910
  Misses         83       83

Continue to review full report at Codecov.

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


Is this ever getting merged?

Thanks for your contributions!
We're currently fighting with null OAS types and was wondering if there're any plans of ever opening a PR with in the cebe/php-openapi repository? I see the README statement about OAS 3.1 taking forever, but that seems to be in now. There does however seem to be some fixes in this fork that would be good to have in cebe's repository.

That depends on @cebe, @DEVizzent set this up while Carsten can't/won't/(whatever reason) keep cebe/php-openapi up to date with the questions and PR's coming in to continue development here. And when the time comes to contribute back. It should also be a drop in replacement until then.

Thanks for the fast response. I've left another comment on the other side and I guess I'll see where this is in a few weeks :shrug:


How to know when all playlist videos as been downloaded?

When downloading playlist files, the line 'Downloading video x of x' is useful.
But it would be even more useful to have a line like that at the end of the output:
'All playlist videos downloaded'
I am using youtube-dl in a php app. I am parsing result lines to get progress, ETA, bandwidth, filename, etc..
But there is no information at all that i can use to know when the playlist download as ended.
Is there something to know when all playlist videos as been downloaded?

I don't see why you need that. You just check one by one and in the last one x/x you can see when it ends

no, because when the last x/x is displayed, the video has not yet been downloaded, or converted.

I see your point, and it is a good idea. Anyway I checked what I did and it is to wait for the process to end (it is in java anyway), you can also read the percentage after last x/x or even look for the word "error" for some cases.

@jaimehrubiks thanks for your answer

waiting for the process to end is not relevant if youtube-dl is fired via an api: we just launch the download command and don't wait for a response. the response is the parsing of the command log (stored somewhere) each x sec. this is it that can indicate progress, download finish, conversion, etc...

read the percentage after last x/x is not relevant because there could be a converting process after the last download. if we want to do something on all the playlist files (in my case, zip the whole playlist), we have to wait for the end of the last converting process. To know the last download/convert as ended, we have to use very ugly, long, performance-killing regex that parse not the just the last line, but the whole file (last line and up to 'Downloading video x of x').

when youtube-dl is embeded in an external app (i've done it several times), it is better for performance to only parse the last line of the cmd output. We can use --exec echo 'blablabla' to output something that indicates the end of the download/conversion process when asking for a single video, but --exec is fired for each videos in a playlist, so this workaround is ineffective for playlists.

What about creating a new post processing option like that:
--exec-playlist CMD : Execute a command after all playlist's videos has been downloaded.
Or even easier, output a short message when the last video has been downloaded/converted: 'Process finished'.
It could be added to everything, not just playlists...
I don't know the python language, so i cannot propose a pull request :(

@hedii I see now. I wish someone adds this.
Anyway, again, I will try to help you :P
Just this:
youtube-dl https://www.youtube.com/playlist?list=PLAYLISTID end
and the last line is...
ERROR: u'end' is not a valid URL. Set --default-search "ytsearch" (or run  youtube-dl "ytsearch:end" ) to search YouTube
So you check for: "ERROR: u'end'...." and you got it!

@jaimehrubiks jajaja, dirty but it works. thanks

You may need a more precise workaround for it:
if result_type == 'playlist':
    self.to_screen('[download] Process finished')

As these codes are also executed when a multi_video is finished.


Refactor compute resources compare functions

Changes
This commit creates a new package for comparison of compute resources
and refactors limitrange-related tests to use these comparison helpers.
This refactoring will make upcoming changes to compute resource code
easier to write. No functional changes.
/kind cleanup
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

n/a Has Docs included if any changes are user facing
n/a Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
n/a Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
n/a Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

FYI @austinzhao-go

/retest

/approve


New algorithm: CCM*-with-no-tag

CCM* is a cipher mode based on a block cipher which has an optional authentication tag. CCM* with authentication is identical to CCM which is already present in the specification as an AEAD algorithm. This commit adds an algorithm declaration for CCM* without authentication as a cipher algorithm.
This will be in a future edition of the PSA crypto specification (private link).

Moving to Q4 as this is part of a Q4 epic


Add UIKit tests

Bond looks very useful.
I was looking at the UIKit tests and thought I could add some to cover the operators.
I split the tests to match the split up of the UIKit extensions code.
Also I added an iOS app as a test host so you can send control events in the tests.
Hope you find them useful.

Thank you Anthony, this is useful. Merging.


Why are are there no examples of how to do some things in C#

Hey,
why are there no examples of how to do some things in C#
For example I'm stuck with the following example

Why is CreatePhysicsBody returning IntPtr? I don't even know what that is.

Sry if I'm a bit harsh, but I'm just mentally broken right now, because I switched between c and C# 5 times last week.
I know that you are doing great work, but I would be so greatful if you could help me with that.
ǎ̷̢̢̛͈̯̟͇͇̘̞̝̲͎̦̰̦̘͙͖̤̻̩̯̻̪̗̔̀̊͐͗̀͂́̀̄̄͆͌͛̏̿̑́̇̈́̚͘͜͝͠

Sry I just saw that you have an extra repository for that fuc*** me
Sry for everything,
ḩ̴͈͕̱͔̠͓̣̯͎̺̻̖͍̖͕͍̼̯̹̮͆̊͂͛è̶̮̬͕͉͔̤̙̬̼̭̦͈̮̼̍̌̐ļ̸̧̨̛̛̼̫̫̱͔̻̮͚̦̬̹̫͇͍̭͙͕̣̖̤͇̰̫̼̪̥̉̒̾̒̏̈̈́̐͒̎͋͒͂̉̌̓̇̿͌͌́̀̉̿̆͌̽̽͆̐͜͜͜͝͝p̸̛̬͎͙̣̦͚̱͋̐̆̎̀̀͋͗͝͝


Bar stack horizontal is breaking, saying that zScale is marked as required in 'BarStackHorizontal but its value is undefined'.

@okanji the zScale prop is a function that returns the color for each segment in the bar stack based on the key. Here's an example of it working on the latest version of @vx/shape: https://vx-demo.now.sh/barstackhorizontal using an ordinal scale to map keys to colors.
But you could also do zScale={key => 'blue'} if you wanted each segment to be blue.
here's the source: https://github.com/hshoff/vx/blob/8b3a6fe0c6997caef243009ea47121313f520672/packages/vx-shape/src/shapes/BarStackHorizontal.js#L65
note: looks like you're on an older version of @vx/shape as the zScale prop was renamed color about 6 months ago.

Thank you for your reply, this sorted me out and it works now. Much appreciated.


Encode rank to make key string in order.

I understand that rank is not sorted. But I'm afraid that if we change to big endian, all users's data is changed. And I believe there are many interfaces is affected, you can see check by the test cases by running ctest.
What is your reason to change this encode?

https://discuss.nebula-graph.com.cn/t/topic/2774/9
for this discuss, I want implement paging functionality to use "go from "1" over follow where follow._dst > "100000" and follow.rank > 3 |  limit 100000". but rank is not in ordered.
I have run unit testing, only one test case "rocks_engine_config_test" is failed which is not related to this change.
65/69 Test #65: storage_http_ingest_test .........   Passed    1.24 sec
Start 66: scan_edge_test
66/69 Test #66: scan_edge_test ...................   Passed    2.24 sec
Start 67: scan_vertex_test
67/69 Test #67: scan_vertex_test .................   Passed    2.15 sec
Start 68: kv_test
68/69 Test #68: kv_test ..........................   Passed    1.99 sec
Start 69: kv_client_test
69/69 Test #69: kv_client_test ...................   Passed    6.81 sec
99% tests passed, 1 tests failed out of 69
Label Time Summary:
codec             =   0.03 secproc (4 tests)
kvstore           =  95.98 secproc (7 tests)
kvstore/raftex    = 267.75 secproc (9 tests)
kvstore/wal       =   9.41 secproc (4 tests)
meta              = 185.12 secproc (16 tests)
storage           = 174.91 secproc (26 tests)
utils             =   0.02 sec*proc (3 tests)
Total Test time (real) = 733.25 sec
The following tests FAILED:
25 - rocks_engine_config_test (Failed)

For keys in V2.0 have already been changed, and upgrade is need from V1.0. Why not further to make edge key in order?

ctest fail seems not because of this change.

For keys in V2.0 have already been changed, and upgrade is need from V1.0. Why not further to make edge key in order?

Yeah, it is the right time if we need to change key format (If we really need to do it). Let's talk in the forum.


Display additional patient info when searching patients to add as Related Person

🚀 Feature Proposal
When searching for a match by name in the Add Related Person dialog, it should display additional identifying information (such as zip code) alongside the name of each listed match.
Motivation
When searching for a match in existing patients, it only searches by name, and doesn't display any other info to the user before they confirm the match as a related person.
This seems likely to cause issues if say, the user is searching for a person with a fairly common name, and they are returned as a match when in fact the user is looking for someone completely different who is not yet on the system.  Similarly, when presented with three matching names and no way to tell them apart, the user either has to guess, or exit the dialog and navigate away from the current page to look them up individually.

Example
The search bar in the Add Related Person dialog should also display part of the address, for example, for each match so that the user can at least distinguish between multiple matches with the same name, and can tell straightaway if the match is someone in a different city when it's a person from the same household that they're looking for.
If Dr Smith is searching for Jane Doe to add as John Doe's partner, they can select the correct Jane Doe with the additional matching info.
Suggested solution is to display Name, Age, Sex, then the first line of the address (truncated to a set max length) and zip code:

Jane Doe - 32 - F - 10a High Street - AA12345

Can I work on this feature?

@JDarke did you just report the feature or were you also planning to develop it?

I'm happy for you to take this one, @nischayv .
Any redesign of the layout of the dialog box could be affected by #2106 at some point, as we might be moving the dialog box fields onto the main page...but that's not an issue if you're just going to change the text displayed in the search dropdown, as that functionality will be carried over unchanged.

Do we still want to retain the patient code in the text? If we choose to retain that we should ideally skip the address otherwise the text will become too long (we would have to truncate most of the address and it wouldn't make sense to have it at that point). What does the patient code signify? If it is some sort of unique identifier I think it makes sense to retain it. Also, if we do choose to display the address, do we arbitrarily display the first one the user entered while creating a new patient? Since it doesn't look like we have any concept of primary address, secondary address, etc and if the user entered it in the incorrect order, the address may not necessarily serve as a unique way to identify an individual as the person adding the related person may not be aware of the other address.
Also, if the sex of the person is Other, as mentioned in the example would O be sufficient to understand what the letter signifies? Might make sense to display the full text for sex - Male, Female, Other.
@JDarke @jackcmeyer thoughts?

@nischayv Hmmm.... maybe a better option would be to display date of birth. It is, possible, but unlikely that someone with the same name and DOB would exist.

Even though unlikely, in the off chance that it does happen, it would be impossible to pick the correct patient with just the name and DOB. If the patient code is a unique identifier, I believe it would be reasonable to have the DOB and code. Something like this - John Doe - 1990-12-10 (P-WYLx1mc1_).

@nischayv let's do that.


Improve brace matching

Brace matching in ATOM is like this:

In e.g. Php Storm, brace matching prioritizes parens based on their direction - like this:

Why this is better: because, regardless of whether the cursor is over an opening or closing brace, it's matched in such a way that you're typing outside the matching parens - consistently.
The way it works in ATOM currently, if I start typing in the first example, I'm typing on the outside of the matched parens - but if I start typing in the second example, I'm typing inside the matched parens.
Having to cursor right before you start typing after a closing brace, and then having the "wrong" parens highlight when you do, is really confusing.

This issue was moved to atom/bracket-matcher#116


fresh: command not find

I used go get -v -u github.com/pilu/fresh in ubuntu, but when I use fresh, I get the error about command not find.
And I can't see the fresh in go/bin, but I can see the fresh in go/pkg.
Could anyone give me some help to fix this issue?

using go install does not make the fresh command work for me. What I ended up having to do was use the full go run command: go run github.com/pilu/fresh
I'm using Windows 11, with Go1.18.4

using go install does not make the fresh command work for me. What I ended up having to do was use the full go run command: go run github.com/pilu/fresh
I'm using Windows 11, with Go1.18.4

Ok, thank you, I will try it.

using go install does not make the fresh command work for me. What I ended up having to do was use the full go run command: go run github.com/pilu/fresh
I'm using Windows 11, with Go1.18.4

It does work!!!!! Thank you!!!!! 🥳🥳🥳🥳🥳🥳🥳🥳


Add some hover effect on Navbar links update dark/light mode icon not visible when dark mode is turn on.

What's the style idea?
We need to improve  Navbar links by adding some minimal hover effect on it. When dark mode on not visible to user.
Docs section can be modified by providing each steps in a beginner friendly way by adding each commands images step wise.
Add screenshots
Add screenshots
Code of Conduct

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

Hey, I'm a GSSoC '23 contributor, and would like to work on this

@priyankarpal can I have it assigned?

@Geeta259 I've reproduced the issue and will be able to solve this. Can I be assigned this?

What's the style idea?
We need to improve Navbar links by adding some minimal hover effect on it. When dark mode on not visible to user. Docs section can be modified by providing each steps in a beginner friendly way by adding each commands images step wise.
Add screenshots
Add screenshots
Code of Conduct

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

duplicate of https://github.com/priyankarpal/ProjectsHut/issues/333
& https://github.com/priyankarpal/ProjectsHut/issues/373


Add info to the article

As such, the size of a Linked List is not fixed. It can be altered, unlike in an array where the size remains constant once initialized.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@varchasva-tiwari I believe the fixed array memory footprint is only specific to some programming languages. If you can show that this is a more general phenomenon we may be to add this to the article. For now I'm closing this PR.


How to create a python sdk for mxnet-operator

Platform informations:
kubernetes: v1.14.4
Kubernetes-python-client: 10.0.1
OS: Centos 7.6
I want to create a kubeflow job such TFJob and MXJob via python ,  the former has the corresponding python sdk, but the later
has not.
I can create the mxnet-job with kubectl create -f mxjob_dist_gpu_v1.yaml,
I tried to archieve this goal by calling the api client.CustomApi().create_namespaced_custom_object(), but it complained the
following errors:
kubernetes.client.rest.ApiException: (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Content-Type': 'text/plain; charset=utf-8', 'X-Content-Type-Options': 'nosniff', 'Date': 'Tue, 07 Jul 2020 09:35:26 GMT', 'Content-Length': '19'})

@jinchihe , is there anything else I need to do to implement such a python sdk for mxnet-operator?
Can you give me some instructions or advices? Thanks

Does TFJob pythobn sdk work when you create TFJob?

Yes, the python sdks both works for TFJob and PyTorchJob. This really confused me!

Thus your question is that Kubernetes SDK does not support MXNet?

I am not sure about that but I doubt it

You can illustrate your problem here in Chinese, I can translate for you. I am not sure if we are on the same page.

It may be my problem. I have set JOB_PLURAL to mxnetjobs for MXJob.
I will try it again.

@gaocegege  I am sorry,It's all my fault. I have fix the problem via change the value of JOB_PRLURAL.


KAFKA-8399: bring back internal.leave.group.on.close config for KStream

As title states. We plan to merge this to both trunk and 2.3 if it could fix the stream system tests globally.
Reference implementation: https://github.com/apache/kafka/pull/6673
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Retest this please

Can we run the system tests (at least streams_upgrade_test#test_version_probing) to see if the test is fixed with this PR and to verify that we can close https://github.com/apache/kafka/pull/6779

Yep, it's here:https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2613/

@abbccdda Made a pass, looks good overall. Could you also add back the unit test checking that upon shutting down Streams does not send leave group request as well, so that we can capture this earlier in unit tests than system tests?

Second try passed too: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2616/

retest this please

Merged #6779 into trunk

Cherry-picked to 2.3 as well.


Add an optional argument to the space function to allow the user to specify the number of divisions.

If the divisions function returns 0, the space function will throw an error when trying to divide by 0.

Finished with implementation with #35


Admin DataGrid: Keep search terms after page reload

When searching in a DataGrid in the admin panel, the search term appears as a "post-it" beneath the search bar.

But as soon as a filter is added, e.g. a channel is selected, the search term disappears, although it remains as a URL parameter. The result is filtered correctly, but the user is confused, because the search term is not shown.
This PR fixes this issue: The search term will remain with all applied filters beneath the search bar. The channel and the customer group will appear there as well now.
That means, the gray terms are directly corresponding to the URL parameters.

EDIT: I just added encoding of the search terms, to let us be able to search for weird coupon codes as "AB#CD" or "All20%Off".

Hi @AnnikaWolff
Please correct this

Hi @vaishaliwebkul
Yes, the search term is converted from a url encoded string to a url decoded string for the sake of better readability.
But I do not think, that is a problem, do you?
For instance, I have a simple product called "Simple Product". I search for it with the terms "Simple Product", having a space between the two words. In the URL, the search terms are encoded to "Simple%20Product", that is not nice to read. So coming out of the URL, the terms are decoded to "Simple+Product", which is easier to read.
The search result is always the same: If looking for "Simple Product", "Simple%20Product" or "Simple+Product", I always find my simple product.
Concerning the failed check, I already notified @jitendra-webkul about the faulty migration causing those errors.

@AnnikaWolff

My concern is for search key changed when locale added;
for example When I search for 'simple product' the search result be shown 'simple product', as and when I added the locale key 'en' then previous search key will be decoded to 'simple+product', I think it's a bug, we should keep the search key same before and after adding locale key.

Another Bug I just found while filtering list on the basis of status->True/Active (converted to False/Inactive)

Hi @vaishaliwebkul
Alright, I will check that.

Dear @vaishaliwebkul
1st: I removed the "+" symbol that appeared when searching for multiple terms and adding a locale filter.
2nd: I fixed the bug of the boolean filter always showing the "false / inactive" value.
And 3rd: I added a label to the locale filter.

Please feel free to test again.

Hi @AnnikaWolff
Thanks for resolving the above issue :)
One more update, In the cart rule page, filtered channels & customer groups tag with their ID, can't we show them by their name likewise in the product list.

Hi @vaishaliwebkul
This is much more effort because I have to refactor all DataGrids and their views at once.
So stay tuned, it may take a while.

Hi  @vaishaliwebkul
Done :)
Filtered channels, locales and customer groups are now shown by their names instead of code or id.
And all my changes have another advantage, I suppose: It is much easier now to add those filters to other DataGrids, if needed.


⚠ Hartman Foundation has degraded performance

In 3a2415e, Hartman Foundation (https://gordonhartman.com/) experienced degraded performance:

HTTP code: 200
Response time: 2417 ms

Resolved: Hartman Foundation performance has improved in 77652e9.


Gradle dependencies for android integration?

Android/kotlin/java/gradle noob here, so apologies in advance: but I'm trying to follow the instructions for android integration and I'm getting unresolved reference and overrides nothing errors. I've tried adding a dependencies block to my app/build.gradle
dependencies {
    implementation "org.flame-engine:gamepads-android:0.1.4"
}

but gradle can't find it.
Am I missing something? Is there an example with android integration that I can look at somewhere?
Thanks in advance.

This example should run on android:
https://github.com/flame-engine/gamepads/tree/main/packages/gamepads/example

@spydon again I apologise for my android noobishness, but that doesn't have an android folder, let alone a MainActivity.kt. I've tried adding support through the flutter CLI and using the default MainActivity.kt and the replacement from the README.md but have had no luck.
Could you provide build steps? I'm coming from a flutter for web background, and so maybe I'm just missing a flutter pub get equivalent command...

@jackd you must run flutter create --platform =android . first to get the platform directory.

@spydon yep, got that far. My question is about the changes to MainActivity.kt that are documented in the readme.

@markvideon maybe you can assist? I've never actually run it on android yet. 😅

To clone the gamepads repo, navigate to the example project, and locally create an android project to build & run it I ran this command:
git clone https://github.com/flame-engine/gamepads.git && cd gamepads/packages/gamepads/example/ && flutter create . --platforms=android && flutter pub add gamepads

If I were to build and run at this point, it would fail because changes to MainActivity.kt are required. I use the excerpt from the project README, copying everything below the first line (as the stock MainActivity.kt already contains a line defining the name of the package. For the gamepads example project my adjusted MainActivity.kt file looks like this:

package com.example.example

import android.hardware.input.InputManager
import android.os.Handler
import android.view.InputDevice
import android.view.KeyEvent
import android.view.MotionEvent
import io.flutter.embedding.android.FlutterActivity
import org.flame_engine.gamepads_android.GamepadsCompatibleActivity

class MainActivity: FlutterActivity(), GamepadsCompatibleActivity {
    var keyListener: ((KeyEvent) -> Boolean)? = null
    var motionListener: ((MotionEvent) -> Boolean)? = null

    override fun dispatchGenericMotionEvent(motionEvent: MotionEvent): Boolean {
        return motionListener?.invoke(motionEvent) ?: false
    }

    override fun dispatchKeyEvent(keyEvent: KeyEvent): Boolean {
        return keyListener?.invoke(keyEvent) ?: false
    }

    override fun registerInputDeviceListener(
        listener: InputManager.InputDeviceListener, handler: Handler?) {
        val inputManager = getSystemService(INPUT_SERVICE) as InputManager
        inputManager.registerInputDeviceListener(listener, null)
    }

    override fun registerKeyEventHandler(handler: (KeyEvent) -> Boolean) {
        keyListener = handler
    }

    override fun registerMotionEventHandler(handler: (MotionEvent) -> Boolean) {
        motionListener = handler
    }
}

To clone the gamepads repo, navigate to the example project, and locally create an android project to build & run it I ran this command:
git clone https://github.com/flame-engine/gamepads.git && cd gamepads/packages/gamepads/example/ && flutter create . --platforms=android && flutter pub add gamepads

If I were to build and run at this point, it would fail because changes to MainActivity.kt are required. I use the excerpt from the project README, copying everything below the first line (as the stock MainActivity.kt already contains a line defining the name of the package. For the gamepads example project my adjusted MainActivity.kt file looks like this:

package com.example.example

import android.hardware.input.InputManager
import android.os.Handler
import android.view.InputDevice
import android.view.KeyEvent
import android.view.MotionEvent
import io.flutter.embedding.android.FlutterActivity
import org.flame_engine.gamepads_android.GamepadsCompatibleActivity

class MainActivity: FlutterActivity(), GamepadsCompatibleActivity {
    var keyListener: ((KeyEvent) -> Boolean)? = null
    var motionListener: ((MotionEvent) -> Boolean)? = null

    override fun dispatchGenericMotionEvent(motionEvent: MotionEvent): Boolean {
        return motionListener?.invoke(motionEvent) ?: false
    }

    override fun dispatchKeyEvent(keyEvent: KeyEvent): Boolean {
        return keyListener?.invoke(keyEvent) ?: false
    }

    override fun registerInputDeviceListener(
        listener: InputManager.InputDeviceListener, handler: Handler?) {
        val inputManager = getSystemService(INPUT_SERVICE) as InputManager
        inputManager.registerInputDeviceListener(listener, null)
    }

    override fun registerKeyEventHandler(handler: (KeyEvent) -> Boolean) {
        keyListener = handler
    }

    override fun registerMotionEventHandler(handler: (MotionEvent) -> Boolean) {
        motionListener = handler
    }
}

With this, I am able to build and run and the package works as I expect.
If you are unable to replicate this, it would be helpful to know how you've integrated gamepads into your project - can you share the relevant lines from your flutter project's pubspec.yaml file?

My apologies, it seems I was experimenting with this as a system update introduced a gradle/java inconsistency or... something. This had nothing to do with gamepads - I couldn't even build the starter counter app. Resolved with steps equivalent to discussed here.
Thank you for your time.


[SE-0091] Require member operators to refer to the enclosing nominal type

What's in this pull request?

Member operators should be placed within a nominal type (or extension
thereof) that they operate on. Aside from being good style, enforcing
this in the type checker can help with dependency tracking.
Resolved bug number: (rdar://problem/27536066)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

…

@swift-ci please smoke test and merge

@jrose-apple, these feel like reasonable restrictions to aid dependency analysis
@rudkx, can you review?

I haven't thought through 100% whether this restriction is sufficient. We still do overload resolution on the full set of operators, and sometimes that helps select the types of the operands.

@shahmishal The linux failure here looks like either a configuration problem or perhaps something spurious.

@jrose-apple Can you give an example of where this might be overly permissive?

@DougGregor LGTM. I'll cherry-pick to swift-3.0-branch, fill out the template, and kick off a new PR.


Support graphql-core<2

The current Graphene version does not support graphql-core>=3 and it will take some more time ... It seems like the AST part we're using is not much difference between these versions and it should be easy to support both versions of graphql-core

I don't think it is necessary anymore - the old version is deprecated, and supporting both increases the maintenance cost.


Changing marketplace publisher from ms-vscode to ms-dotnettools causes extension recommendations to repeat ad nauseum

Steps to reproduce

Have an old version of the extension installed that uses ms-vscode.csharp as the extension ID.
Be upgraded to ms-dotnettools.csharp.
Have a project that uses extension recommendations, set up to install the extension under the new ms-dotnettools.csharp

Expected behavior
Everything keeps working normally.
Actual behavior
Constantly keep being pestered with the need to install a not-yet-installed recommended extension, even though the extension is already installed.

NOTE
Issue can be worked around by

going into the VSCode extensions folder (%USERPROFILE/.vscode/extensions/ on Windows)
opening the ms-dotnettools.csharp extension folder holding the locally installed extension.
opening the package.json file.
editing the publisher field to the ms-dotnettools value.

Moral of the story;
Probably don't just change the publisher on the marketplace without also correctly updating the package manifest?

Your workaround does not work for me, @rjgotten, but my situation may be slightly different (perhaps we have different situations or encountered different repo states)

Steps To Reproduce

Open an existing workspace for which the "csharp" extension is "recommended", that was created before the release of the latest extension version 1.21.13 (I have Azure Functions app workspaces that were built prior to March 5, 2020)
Let VS Code update the csharp extension to the latest available version (automatic in my case, but could be manual for other users)

Expected behavior

Everything keeps working normally.

Actual behavior

VS Code still recommends the extension referencing the old publisher attribute and shows the following warning notification after the latest extension has been upgraded to v1.21.13:
"The below 1 extension(s) in workspace recommendations have issues:
ms-vscode.csharp (not found in marketplace)"
NOTE: In this case, Investigating %USERPROFILE%/.vscode/extensions/ms-dotnettools.csharp-1.21.13/package.json reveals that it already has the new publisher attribute ("publisher": "ms-dotnettools")

Workaround

Because the existing workspace references the old publisher attribute value ("ms-vscode.csharp"), the only workaround I know of is to edit every such workspace in the following way:

Edit each workspace's extensions file: .vs-code\extensions.json
Change the recommendation line from "ms-vscode.csharp" to "ms-dotnettools.csharp"
Load/Reload the given workspace(s)
Result: No more publisher mismatch for the recommended plugin, so no warning notification.

This really should not happen with such a broadly used and recommended extension :(
Hope this helps some folks

@rjgotten Sorry about that. There was a gap time between the publisher switch and we when we were able to publish a new build with the updated publisher id. We published a new release this morning so things should be back in a good state. Note you will likely see a suggestion to try the C# extension but that should go away with the next insiders release 3/6 and the next stable release 3/9.

Totally understand and appreciate the reply, hopefully others can avoid the nagging message using my quick workaround or perhaps somebody has another suggestion for the interim end-user workaround.
From the extension upgrade side of things, is there not a means (formal SOP or similar) to update the publisher attribute at the repo/asset level without such issues resulting?  I'm thinking super simply here, but some manifest attribute that can list historical publisher values.  I'm not sure how the extensions mechanisms work internally, but it would be nice if you could change things like the publisher or extension name while still keeping mappings to the previous states such that VS Code could easily detect and handle these types of changes?  (e.g. VS Code would recognize that a workspace is simply using an old reference and not render such warnings, or have a mechanism that updates recommendations automatically)

@widgetcollector I believe on the backend, requests for the package under the old publisher will get forwarded correctly. The issue is that the suggestion engine is just doing a string comparison and not aware of the package forwarding.

Unfortunately, it also seems to prevent other extensions from loading:

@widgetcollector
my situation may be slightly different (perhaps we have different situations or encountered different repo states)

Yes; you seem to have had the dual of my problem; a recommendation setting for the old publisher ID and an installation that according to:

NOTE: In this case, Investigating %USERPROFILE%/.vscode/extensions/ms-dotnettools.csharp-1.21.13/package.json reveals that it already has the new publisher attribute ("publisher": "ms-dotnettools")

--- came with the new publisher ID. I guess that means you must've installed the new version with the fixed publisher ID that @JoeRobich referred to, while I was still served the old one.
Good to hear that everything is syncing back up and the problem will fix itself.

@videopilot
Unfortunately, it also seems to prevent other extensions from loading:

That though... Ouch. That sounds significantly more nasty than a bit of notification-nag. Probably better to roll back to the old version and disable auto-updates until the other extensions can catch up. Or maybe a compatibility package published under the old publisher ID can be issued in the mean time, to serve as a pass-through/proxy or something?

@videopilot you could try https://github.com/Unity-Technologies/vscode-unity-debug/pull/152 for a fix for unity and track https://github.com/Unity-Technologies/vscode-unity-debug/issues/153 for a related issue.  I've made the change locally.  I had rolled back to https://github.com/OmniSharp/omnisharp-vscode/releases/tag/v1.21.12 for a bit this morning which also worked.

Rolling back to 1.21.12 via uninstall and command line code --install-extension csharp-1.21.12.vsix worked for me... intellisense back up and running

I've had to stop work and waste hours searching for a solution and finally happened on this thread. Not good as I'm self-employed. I don't know where to start with "rolling back" things. Surely there was a more graceful way to have handled this change? Was it absolutely necessary to change publisher id?? Millions of devs use VSCode.

I've had to stop work and waste hours searching for a solution and finally happened on this thread. Not good as I'm self-employed. I don't know where to start with "rolling back" things. Surely there was a more graceful way to have handled this change? Was it absolutely necessary to change publisher id?? Millions of devs use VSCode.

Same here. Instructions:

uninstall c# extension (and disable any other extensions that are throwing errors because of c# not being "found").

Download csharp-1.21.12.vsix from here: https://github.com/OmniSharp/omnisharp-vscode/releases/tag/v1.21.12

Restart VS Code

Open the Command Palette in VS Code. Search for "shell command install" and run that

Open a command line terminal and go to the folder you downloaded the vsix file to

Run: code --install-extension csharp-1.21.12.vsix

Restart VS Code

Open Preferences -> Settings in VS Code. Search for "update" and uncheck Extensions: Auto Update

@JoeRobich
Note you will likely see a suggestion to try the C# extension but that should go away with the next insiders release 3/6 and the next stable release 3/9

I take it you're refering to the built-in notifications in VS Code to use a pre-determined extension to handle files of a known type?

Thank you zachstronaut!  Sorry, you lost so much time researching the issue and thank you for saving me time with your solution.  It worked perfectly.

Thank you @zachstronaut this thing's been driving me crazy. I finally found this and that worked perfectly

Having the same problem with Ubuntu 19.10

Same problem /w Ubuntu 18.04

Same problem with macOS Mojave

Rolling back to 1.21.12 via uninstall and command line code --install-extension csharp-1.21.12.vsix worked for me

Eventually did the same as well, because next to these issues the new .13 release has major performance problems too - hanging for minutes on end and clogging up all manner of other functionality in VS Code, all the way to git commit/pull/push/etc actions.
It's just completely broken and imho should be pulled.

Same problem /w Windows 10. Rolling to csharp-1.21.12 back fixed it.

Open the Command Palette in VS Code. Search for "shell command install" and run that

Thanks @zachstronaut , this worked for me as well. I didn't have to do step 4, though (as I couldn't find that specific item in the command palette).

@zachstronaut  thank bro

Thanks @zachstronaut , and I also didn't do step 4, instead I downloaded the vsix package, and seached for "Install from VSIX" to run that.

I've got an update. I've verified that the latest stable and insiders releases, which are available now, no longer suggest installing the C# extension from the old publisher. However, extensions with dependencies on the C# extension still need to be republished specifying the new publisher.

@JoeRobich Confirmed. v1.43.0 fixed the popup

Closing as the issue with the popup is resolved. #3620 is tracking the overall rename issue.

 Re-opening.

Update regarding unity: https://github.com/Unity-Technologies/vscode-unity-debug/pull/152
Pull request is merged, so once the unity debugger extension is updated, this shouldn't be an issue for folks using it. A lot of reports here came from that. Many other extensions have had PRs sent to them, but it will take time for them to incorporate those changes and publish updates. Since the name change is final (and changing it again would break things again), I think we'll just have to close this out as by design. Sorry for the horrible inconvenience here. We'll strive to do better and ensure something like this doesn't happen again.

Update regarding unity: Unity-Technologies/vscode-unity-debug#152
Pull request is merged, so once the unity debugger extension is updated, this shouldn't be an issue for folks using it. A lot of reports here came from that. Many other extensions have had PRs sent to them, but it will take time for them to incorporate those changes and publish updates. Since the name change is final (and changing it again would break things again), I think we'll just have to close this out as by design. Sorry for the horrible inconvenience here. We'll strive to do better and ensure something like this doesn't happen again.

https://code.visualstudio.com/docs/editor/extension-gallery#_install-from-a-vsix

asdf

Still having the issue with v1.23.2 and VS Code Insider 1.50.0-insider

@abouroubi Sorry you are seeing this issue. What is the message that you are seeing in VS Code?

@JoeRobich VSCode send a notification each time I open a C# project.
The 1 extension(s) below, in workspace recommendations have issues: ms-vscode.csharp (not found in marketplace)

@abouroubi Workspace recommendations are stored locally in your workspace and must be manually updated (https://code.visualstudio.com/docs/editor/extension-gallery#_workspace-recommended-extensions). You should be able to change ms-vscode.csharp to ms-dotnettools.csharp to get rid of this message.

@JoeRobich Thank you, it helped me solve it.
So if someone has the same issue, for me it was a project created with Azure Function Extension, wich had an outdated extension name.


Why etcd cant resolve mdns names with DEBUG=netdns=cgo?

I trying to use etcd for small cluster with mdns for resolving addresses of members.
I write small program on go, which resolve mdns name to ip address. With DEBUG=netdns=cgo its ok, but etcd not. Why?

@styleex etcd version? platform/os? go version? Can you provide the example program?

@heyitsanthony

@heyitsanthony
etcd version:
# etcd --version
etcd Version: 3.1.5
Git SHA: 20490ca
Go Version: go1.7.5
Go OS/Arch: linux/amd64

platform:
# uname -a
Linux host1 4.9.0-0.bpo.2-amd64 #1 SMP Debian 4.9.13-1~bpo8+1 (2017-02-27) x86_64 GNU/Linux

Example program:
package main

import (
	"net"
	"log"
)

func main() {
	ip, err := net.LookupIP("my-host.local")
	if err != nil {
		log.Fatal(err)
	}

	log.Println(ip)
}

go version:
# go version
go version go1.8 linux/amd64

Run example program with GODEBUG=netdns=go+1:
# GODEBUG=netdns=go+1 ./dns
go package net: GODEBUG setting forcing use of Go's resolver
2017/04/22 07:50:25 lookup my-host.local on <IP_ADDRESS>:53: no such host

Run example program with GODEBUG=netdns=cgo+1:
# GODEBUG=netdns=cgo+1 ./dns
go package net: using cgo DNS resolver
2017/04/22 07:50:41 [<IP_ADDRESS>]

Run etcd with GODEBUG=netdns=go+1
# GODEBUG=netdns=go+1 /usr/bin/etcd \
  --name=default \
  --data-dir /var/lib/etcd \
  --listen-peer-urls http://my-host.local:2380 \
  --initial-advertise-peer-urls http://my-host.local:2380 \
  --initial-cluster default=http://my-host.local \
  --initial-cluster-token my-etcd-token \
  --initial-cluster-state new

2017-04-22 07:51:00.580705 W | embed: expected IP in URL for binding (http://my-host.local:2380)
2017-04-22 07:51:00.580811 I | etcdmain: etcd Version: 3.1.5
2017-04-22 07:51:00.580859 I | etcdmain: Git SHA: 20490ca
2017-04-22 07:51:00.580873 I | etcdmain: Go Version: go1.7.5
2017-04-22 07:51:00.580911 I | etcdmain: Go OS/Arch: linux/amd64
2017-04-22 07:51:00.580929 I | etcdmain: setting maximum number of CPUs to 4, total number of available CPUs is 4
2017-04-22 07:51:00.580960 W | embed: expected IP in URL for binding (http://my-host.local:2380)
go package net: built with netgo build tag; using Go's DNS resolver
2017-04-22 07:51:00.587826 C | etcdmain: listen tcp: lookup my-host.local on <IP_ADDRESS>:53: no such host

Run etcd with GODEBUG=netdns=cgo+1
# GODEBUG=netdns=cgo+1 /usr/bin/etcd \
  --name=default \
  --data-dir /var/lib/etcd \
  --listen-peer-urls http://my-host.local:2380 \
  --initial-advertise-peer-urls http://my-host.local:2380 \
  --initial-cluster default=http://my-host.local \
  --initial-cluster-token my-etcd-token \
  --initial-cluster-state new

2017-04-22 07:52:19.171330 W | embed: expected IP in URL for binding (http://my-host.local:2380)
2017-04-22 07:52:19.171390 I | etcdmain: etcd Version: 3.1.5
2017-04-22 07:52:19.171399 I | etcdmain: Git SHA: 20490ca
2017-04-22 07:52:19.171407 I | etcdmain: Go Version: go1.7.5
2017-04-22 07:52:19.171415 I | etcdmain: Go OS/Arch: linux/amd64
2017-04-22 07:52:19.171422 I | etcdmain: setting maximum number of CPUs to 4, total number of available CPUs is 4
2017-04-22 07:52:19.171441 W | embed: expected IP in URL for binding (http://my-host.local:2380)
go package net: built with netgo build tag; using Go's DNS resolver
2017-04-22 07:52:19.173700 C | etcdmain: listen tcp: lookup my-host.local on <IP_ADDRESS>:53: no such host

Where <IP_ADDRESS> - dns server in my network. Why that etcd always selects the pure go resolver.
Content of /etc/nsswitch.conf:
# cat /etc/nsswitch.conf
passwd:         compat
group:          compat
shadow:         compat
gshadow:        files

hosts:          files mdns4 dns
networks:       files

protocols:      db files
services:       db files
ethers:         db files
rpc:            db files

netgroup:       nis

@heyitsanthony I rebuild etcd with CGO_ENABLED=1 and cgo resolver began to switch normally. How dangerous is it to use such an assembly?

@styleex so long as etcd's built using the vendored libraries and the same go version the release is built with, it should be OK. The release disables cgo to avoid libc dependency / surprises with containers (e.g., alpine ships with musl instead of glibc).

@heyitsanthony I understand everything,thank you!


Add dnd handle for draggable and sortable

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

feature, docs update

What is the new behavior (if this is a feature change)?

https://github.com/akserg/ng2-dnd/issues/65
https://github.com/akserg/ng2-dnd/issues/62

Other information:
Also I updated systemjs demo reflecting changes.

Hi @akserg,  This change is really useful!  Would you mind merging this?
I'm happy to test it if that helps at all.
Cheers, Joel

This patch seems like it's exactly what I need, would be great to merge it....

Done.


[FEATURE] Handle unknown interactions

Problem
The framework needs to handle incoming unknown interactions correctly. I can think of 2 scenarios at the moment. 1. the framework is in an invalid state and the interactions registered with discord do not match those of the framework. This should not happen and is always an exception, since we update the command state at every framework start. 2. an interaction was deliberately unregistered, this should mainly affect components.
Current situation
a) The CommandHandler throws an IllegalStateException.
b) The ButtonHandler just sends an ephemeral message stating that the interaction timed out.
Situation a) can be customized by using the getCommandExecutionFailedMessage method, thus I don't think we need to change anything here. However situation b) needs to be handled.
Solution
The simplest approach would be to add a getUnknownInteractionMessage method to the ErrorFactory. This would be a universal solution for buttons and all upcoming component implementations. In situation a) this message should not be used, because it always represents an unexpected error that should throw, as stated above.

+1, got this situation too

honestly this should not happen. Can you give more details?


Firebase RDB transaction returns success, but isn't committed

[REQUIRED] Step 2: Describe your environment

Operating System version: macOS 10.14.6 and Alpine Linux
Firebase SDK version: 6.9.0
Library version: java
Firebase Product: database (auth, database, storage, etc)

[REQUIRED] Step 3: Describe the problem
Cross-posted with cloud support (case 20498489). We work against Firebase Realtime Database from the Java (admin) driver. When running a rather large transaction, we change the MutableData object in significant ways, and then commit the transaction to Firebase.
The transaction returns committed=true, no error, and a value is provided as output. However, it does not appear to have applied the new data in any sense. The Firebase console does not display data as updated, and we cannot discern from the output payload that it was updated.
A few more details/things we tried:

We tried wiping all data at the transaction location. This worked, proving that the Firebase database is selected correctly, that permissions are working and not rejecting requests outright, and so on.

The database in question has no rules that might reject the write due to validation errors. In fact, there are no rules at all:

{
  "rules": {
    ".write": "auth != null",
    ".read": true
  }
}

We tried enabling debug logging over the WebSocket with Firebase. Amidst a ton of logs we found a message about ETag headers not matching for a transaction, indicating a retry, but otherwise this looks to be an innocuous error, and the stream itself displayed no other abnormalities.

When refreshing the Firebase console (to address for the >1MB trigger quota), no data is updated. Strict triggers are on anyway so it should be an error to write over 1MB, unless I am mistaken.

Steps to reproduce:
Working on this
Relevant Code:
// TODO(you): code here to reproduce the problem
# coming shortly

I believe this issue was resolved via another channel (cloud support case 20498489). If you're still having issues, please re-open with repro code, etc.  Thanks!

Indeed it was, this one was our fault! Thank you @mikelehen.


Automatically call get_cog_commands for all cogs

Changes

[x] I checked this pull request runs on Python 3.6.X.
[ ] This fixes something in Issues.

Issue:

[x] This adds something new.
[ ] There is/are breaking change(s).
[x] (If required) Document string is updated/added.
[x] This changes anything except python code. (README, docs, etc.)

About this pull request
This removes the necessity to call get_cog_commands() on each cog by automatically doing so in register_all_commands(). It also displays a temporary warning that the call is no longer required. README and docs have been updated to reflect this.
Also, first PR hype?

LTGM!

LTGM!

What's wrong?

What's wrong?

Looks good at first glance 👍

Looks good at first glance 👍

There is one problem: it can't handle cog reload.

There is one problem: it can't handle cog reload.

Okay then, new approach: overriding bot.add_cog and bot.remove_cog to register slash commands. This now makes it fully reload compatible. (docs not updated yet)

Okay then, new approach: overriding bot.add_cog and bot.remove_cog to register slash commands. This now makes it fully reload compatible. (docs not updated yet)

Merging this pull request, thank you for contributing!

Merging this pull request, thank you for contributing!


Optimizer tolerance

The tolerance of the optimizer can now be set in model.optimize()
This may break some people's existing code, if they're passing a callback and relying on the order of the arguments.

I've merged the test savings in from the fitc branch. Therefore this should be merged (by me I guess) after @jameshensman merges the passing fitc branch.

The tests are genuinely failing on this one rather than timing out @jameshensman .

should be fixed now.
On 13 April 2016 at 20:28, Alexander G. de G. Matthews <
<EMAIL_ADDRESS>wrote:

The tests are genuinely failing on this one rather than timing out
@jameshensman https://github.com/jameshensman .
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/GPflow/GPflow/pull/41#issuecomment-209611998

Current coverage is 98.36%

Merging #41 into master will increase coverage by +0.07% as of e6165f2

@@            master     #41   diff @@
======================================
  Files           20      20
  Stmts         1057    1104    +47
  Branches         0       0
  Methods          0       0
======================================
+ Hit           1039    1086    +47
  Partial          0       0
  Missed          18      18

Review entire Coverage Diff as of e6165f2

Powered by Codecov. Updated on successful CI builds.


Make auth_endpoint auto-detected from client_id

I had confirmation from grdf that client_id ending with "_grdf" belong to old auth endpoint. So instead of fallback on error auto-detect auth_endpoint from client_id.

CC @ATraineauAKA


Trouble loading plugins in Viewer demo

The maintainers of this project are doing a formidable job. 😄
I am trying to get the viewer example going (2020.06).
I installed magnum the vcpkg way, I use CMake on Ubuntu 20.04.
Then, I installed the magnum plugins too vcpkg install magnum-plugins\[\*\].
Now, I see that these plugin libraries are present in the lib dir of vcpkg, although static .a.
When I try to run the example, I run into the following issue:
PluginManager::Manager::Manager(): none of the plugin search paths in {/home/me/vcpkg/installed/x64-linux/debug/bin/magnum-d/importers, ../lib/magnum-d/importers, magnum-d/importers} exists and pluginDirectory was not set, skipping plugin discovery
PluginManager::Manager::load(): plugin AnySceneImporter is not static and was not found in

I have tried passing the vcpkg prefix to the plugin manager.
I feel like I am missing a step, how do I get dynamic plugins working the vcpkg way?
Thanks in advance for your time.

Now, it seems like the vcpkg magnum-plugins are static by default.
I tried the static plugins method and stumbled upon an issue where CMake couldn't find Assimp.
As I understand it, assimp is to be found with find_package(assimp), lower-case.
As a work-around, I copied the FindAssimp.cmake file from magnum-plugins/modules, but shouldn't this be done automatically?
Also, what's the way to use dynamic modules with vcpkg?

Now, it seems like the vcpkg magnum-plugins are static by default.
I tried the static plugins method and stumbled upon an issue where CMake couldn't find Assimp.
As I understand it, assimp is to be found with find_package(assimp), lower-case.
As a work-around, I copied the FindAssimp.cmake file from magnum-plugins/modules, but shouldn't this be done automatically?
Also, what's the way to use dynamic modules with vcpkg?

Hi!
Not exactly sure what vcpkg does on linux/macOS, on Windows at least you have an option to build a dynamic or a static version, and with the dynamic version things should magically "just work". I vaguely remember some update where vcpkg stopped offering dynamic builds on non-Windows platforms? In that case you'd be out of luck, since all examples assume dynamic plugins (as that's the default, and a much more flexible workflow since you're not tied to a particular plugin but "whatever can open PNGs" instead, for example).
For static plugins you'd need to explicitly link those (as you did), and the Find modules currently have to be copied, yes (I can't just install the 3rd party Find modules on the system to avoid conflicts, but I plan on fixing that somehow for the next release -- mosra/magnum#453).
In general, I'd suggest to avoid using vcpkg unless you're on Windows, as the other systems have far better package managers that don't try to force silly restrictions like vcpkg does, not to mention vcpkg upgrades have always been extremely painful for everybody. On Ubuntu you're encouraged to build & install the .deb packages :)

Hi!
Not exactly sure what vcpkg does on linux/macOS, on Windows at least you have an option to build a dynamic or a static version, and with the dynamic version things should magically "just work". I vaguely remember some update where vcpkg stopped offering dynamic builds on non-Windows platforms? In that case you'd be out of luck, since all examples assume dynamic plugins (as that's the default, and a much more flexible workflow since you're not tied to a particular plugin but "whatever can open PNGs" instead, for example).
For static plugins you'd need to explicitly link those (as you did), and the Find modules currently have to be copied, yes (I can't just install the 3rd party Find modules on the system to avoid conflicts, but I plan on fixing that somehow for the next release -- mosra/magnum#453).
In general, I'd suggest to avoid using vcpkg unless you're on Windows, as the other systems have far better package managers that don't try to force silly restrictions like vcpkg does, not to mention vcpkg upgrades have always been extremely painful for everybody. On Ubuntu you're encouraged to build & install the .deb packages :)

Update -- I got another report related to this today, and after digging a bit I realized what's the culprit -- for dynamic plugins to work, these two lines https://github.com/microsoft/vcpkg/blob/b759049a36728d18260963799a56e6b19cb4a2ef/ports/magnum-plugins/portfile.cmake#L102-L103 should be wrapped in if(WIN32). If you do that change locally in your vcpkg copy, dynamic plugins should get installed properly.
We have one vcpkg PR pending right now, when that one is merged I'll submit a fix for this. Will keep you updated.

Update -- I got another report related to this today, and after digging a bit I realized what's the culprit -- for dynamic plugins to work, these two lines https://github.com/microsoft/vcpkg/blob/b759049a36728d18260963799a56e6b19cb4a2ef/ports/magnum-plugins/portfile.cmake#L102-L103 should be wrapped in if(WIN32). If you do that change locally in your vcpkg copy, dynamic plugins should get installed properly.
We have one vcpkg PR pending right now, when that one is merged I'll submit a fix for this. Will keep you updated.

Very nice. Thank you.
For now, I am using the Git submodule approach combined with Conan for dependency management.
That works okay for now.

Very nice. Thank you.
For now, I am using the Git submodule approach combined with Conan for dependency management.
That works okay for now.

This is fixed as of https://github.com/microsoft/vcpkg/pull/16245, which got merged on Feb 18th. Apologies for not updating the issue sooner :)


ToDo sample fails to provision with "Failed to find Subscription"

I'm trying to use the sample todo-list-with-Azure-backend. It fails at the provisioning step. I've tried it with both the Teams Toolkit in VSC and with the TeamsFx CLI. In both cases I get the following error: "Failed to find a subscription"

I'm blocked so please help.

Hi @Rick-Kirkham  can you check the Azure Account used in your project? Whether it has a valid azure subscription. From the error, it seems cannot find a valid subscription under the signed in azure account.

this is what I see: It is a valid account and subscription. I use it all the time.

@Rick-Kirkham sorry for the late reply. Can you provide the toolkit version. Or did you try it on latest toolkit if the issue still exists?

@adashen Sorry, but I don't even remember this anymore. I don't have bandwidth right now to try it again.