Additional labels to apply on the pod level for monitoring and logging needs

Describe the feature request
Add labels on pod level for istio-operator and istiod. The additional labels might be required to configure logging and monitoring systems (fluentd as an example), based on pod's labels select list of pods to send logs/metrics.
Describe alternatives you've considered
Add labels to pod can be added after deployment of istio, but that will not allow to set up gitops approach. As result, for example, ArgoCD will revert any changes\mutating those labels.
[ ] Docs
[ X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

Closed by https://github.com/istio/istio/pull/34003/


do you plan to add a short instruction?

Great resource, it would be better if  you can add a short instruction based on your implementation. Thank you!!

Hi Derekabc!
You are right, I should add some instruction. As it stands, this code is part of my master's thesis which is due soon. But once I'm done writing it I'll get around to update the README

Marcus

Committed a somewhat final version of the README with instruction now

Marcus


node index.js step returns syntax error

george@debian:~/LD-ToyPad-Emulator$ node index.js
undefined:11
]5":<PHONE_NUMBER>
^
SyntaxError: Unexpected number in JSON at position 208
at JSON.parse ()
at initalizeToyTagsJSON (/home/george/LD-ToyPad-Emulator/index.js:140:25)
at Object. (/home/george/LD-ToyPad-Emulator/index.js:27:1)
at Module._compile (internal/modules/cjs/loader.js:816:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:827:10)
at Module.load (internal/modules/cjs/loader.js:685:32)
at Function.Module._load (internal/modules/cjs/loader.js:620:12)
at Function.Module.runMain (internal/modules/cjs/loader.js:877:12)
at internal/main/run_main_module.js:21:11
Any ideas what I can do to solve this? I've had a look in the index.js file for the mentioned ]5" with ctrlF but no luck, but I'm also not knowledgeable on the topic so I'm sure I'm misunderstanding something
Edit 1: Of note, I couldn't find the loader.js file at all, is that unusual?
Edit 2: Fixed the issue but leaving up as others seem to have had similar issues; The problem was at server/json/toytags.json, when making a new toy tag it messed up writing it to this file and made the syntax go bad, I went into the file and removed the bad syntax and it's now running as normal

I just Deleted The Whole Game, Worst Experience Ever.


btn-group draws first input margin #36794

fix #36794

Downside to this solution is that if someone is using <a> and <button> elements in the same button group. https://codepen.io/emdeoh/pen/WNJbvNw?editors=1100

you're right. I updated with :not(.btn-check:first-child) + .btn  selector


the homepage needs changes.

this part isnt available anymore.

I want to work on this issue under JWOC

did u make pr for this?
@taniaban2712 ?


fix tooltipAnchor behavior for different directions

This commit fixes issue #6764, see the discussion and examples there.
It's an alternative implementation for pull request #6116.
It makes tooltipAnchor behave the same like popupAnchor regardless of the tooltip's direction. With one exception: the auto direction flips the x axis of tooltipAnchor and the tooltip's offset when it switches to left depending on the position on the screen. auto therefore assumes that the icon is somewhat symmetrical to the left and the right of the y axis, which is the case for the default icon.
Example with this pull request applied
https://jsfiddle.net/emwbsr9j/
auto to the right

auto to the left

Old behavior
https://jsfiddle.net/5nycvLez/
auto to the right

auto to the left

Example Code
const map = L.map('map').setView([0.0, 0.0], 12)

const iconBase = {
  iconUrl: 'https://leafletjs.com/docs/images/logo.png',
  iconSize: [120, 31.8], // (600 x 159)/5
  iconAnchor: [60, 15.9], // center
}

const icons = {
  top: L.icon(Object.assign({
    popupAnchor   : [50, -15.9], // top right
    tooltipAnchor : [50, -15.9], // top right
  }, iconBase)),
  bottom: L.icon(Object.assign({
    popupAnchor   : [-3, 15.9], // bottom center
    tooltipAnchor : [-3, 15.9], // bottom center
  }, iconBase)),
  left: L.icon(Object.assign({
    popupAnchor   : [-60, 8], // bottom left
    tooltipAnchor : [-60, 8], // bottom left
  }, iconBase)),
  right: L.icon(Object.assign({
    popupAnchor   : [60, 0], // right
    tooltipAnchor : [60, 0], // right
  }, iconBase)),
  center: L.icon(Object.assign({
    popupAnchor   : [50, -15.9], // top right
    tooltipAnchor : [50, -15.9], // top right
  }, iconBase)),
  auto: L.icon(Object.assign({
    popupAnchor   : [60, 0], // right
    tooltipAnchor : [60, 0], // right
  }, iconBase)),
}

let offset = 0.06
L.marker([offset, 0])
  .bindPopup("default")
  .bindTooltip("default")
  .addTo(map)
  .openTooltip()
for (const k in icons) {
  L.marker([offset -= 0.02, 0], { icon: icons[k] } )
    .bindPopup("popup " + k)
    .bindTooltip("tooltip " + k, { direction: k })
    .addTo(map)
    .openTooltip()
}

Great, thank you!

Just upgraded to 1.7.1 and my marker's tooltips have moved.
I cannot figure out why the "top" position is now top right by default: https://jsfiddle.net/2kw3bdgs/1/

The tooltip opens up at the tooltipAnchor position, unaffected by the direction (with an exception for auto).
The default marker has its tooltipAnchor right of the upper circle:
https://github.com/Leaflet/Leaflet/blob/0f904a515879fcd08f69b7f51799ee7f18f23fd8/src/layer/marker/Icon.Default.js#L29
From the docs:

Note about tooltip offset. Leaflet takes two options in consideration for computing tooltip offsetting:

the offset Tooltip option: it defaults to [0, 0], and it's specific to one tooltip. Add a positive x offset to move the tooltip to the right, and a positive y offset to move it to the bottom. Negatives will move to the left and top.
the tooltipAnchor Icon option: this will only be considered for Marker. You should adapt this value if you use a custom icon.

So, you can change the point where you want the tooltip to be positioned by these two options. Once generally for all markers of an icon via tooltipAnchor or for specific tooltips individually via offset.
Before this PR, the direction to which the tooltips open was mixed up with the positioning/offsetting of the anchor point, making it impossible or harder to change the position with some of the direction values (it ignored one or both axes of tooltipAnchor completely and - judging by looking at the old code now - I think it inverted both offset axes when using the center direction).

@Istador
Could you check #5101?


favicon.ico isn't served if you navigate to the https://stroom-host

stroom nginx logs show a failure to find /stroom-nginx/html/favicon.ico
if you navigate to https://stroom-host/stroom, no failure is observed

This may be due to auth.  Its index.html has favicon located at /favicon.ico but it probably needs to look in /stroom/favicon.html.
Manifest is due to it not being reverse proxied.


[Debugger] Add BString variable support and minor fixes

Purpose
This PR introduces the following changes and fixes.

Adds support for BStrings and jvm integer types. (Fixes https://github.com/ballerina-platform/ballerina-lang/issues/23713)
Fixes potential NPEs when mapping jvm types to ballerina variables. (Fixes https://github.com/ballerina-platform/ballerina-lang/issues/23666)
Fixes breakpoint path issue due to new module versioning changes. (Fixes https://github.com/ballerina-platform/ballerina-lang/issues/23718)

Remarks

Related PR - #23720

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #23950 into ballerina-1.2.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.2.x   #23950   +/-   ##
================================================
  Coverage            14.59%   14.59%
================================================
  Files                   51       51
  Lines                 1411     1411
  Branches               219      219
================================================
  Hits                   206      206
  Misses                1189     1189
  Partials                16       16

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 79dd0a6...6e256bd. Read the comment docs.


More correct scoping for library functions

I can catch package function names if they're listed, however it doesn't scope correctly if you use the fully specified name.  For instance: stop(0); works fine, std.env.stop(0) appears as a normal procedure.

I made an adjustment to the standard env package declaration.  I thought about doing it to the others, but I'm a little confused now as to whether std_logic_1164 is now a standard package or a IEEE package.  So in any event, I have it in there for env which is where it's probably most likely to be used.  I'll stop there unless someone requests full library path names for other libraries.


can I applied 3D model or texture and materials on Face detection

Generally I got all face points and we have create  Layer for Face Points shape (eye, lip, eyebrow, nose).
and I want to applied Materials and texture
is that possible ??

ML Kit does not provide any utilities for drawing extra stuff on this face, but you can do this manually by (for example) applying a filter to the pixel buffers before displaying them.

y (for example) apply

Please help me, I'm new here. i tried but nothing is working.
simply i want to use materials or image in face detection points

Drawing on the face is pretty complex and outside of the scope for Firebase support. You may be able to find more help on StackOverflow.

Drawing on the face is pretty complex and outside of the scope for Firebase support. You may be able to find more help on StackOverflow.

already asked question in stackoverflow,  didin't get help please see
https://stackoverflow.com/questions/61285687/try-makeup-using-mlkit-ios-swift


Allow suppression of  PSUseSingularNouns for specific function

PR Summary
Enables suppression of PSUseSingularNouns for specific function.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .cs, .ps1 and .psm1 files have the correct copyright header
[x] Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

What's the status of this PR?? I need to be able to suppress this rule. Generally, this is a good rule. But, it's flagging the word 'Data' as "plural" even though that word is both Singular and Plural.

Waiting for review. Same as #1896.
Not sure what's going with the repo really. Only docs updates last 12 months. Would make sense if there was progress on PSSAv2, but IIRC that's also on hold. 😞

@fflaten Yup, more of a question directed at the repo owners...your changes LGTM.

Hopefully @bergmeister or @JamesWTruher can provide an update.

@fflaten  Quick question on this PR: shouldn't the use of the Target property of the SuppressMessageAttribute be used to suppress a specific function from being flagged by these violations? Isn't that what most of the other rules use?
I ask this, because it's really strange, but, I was able to successfully suppress this rule on two of my functions without the implementation of this PR. But on one function, it will simply not suppress. (And on the two functions where I was able to suppress this rule, I did not need to use the Target property, which is stranger still.....) The biggest question is, why does this sometimes not suppress and sometimes DOES???

@fourpastmidnight There's a difference: inheritance.
function Get-Elements
{
    # This will suppress the rule everywhere inside the targeted function -> both Get-Elements and Get-Containers will be suppressed
    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', '', Scope = 'Function', Target = 'Get-Elements')]
    # This will only suppress Get-Elements but still report Get-Containers
    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', 'Get-Elements')]
    # All SuppressMessageAttributes must be above a param() block, both at file level or inside a scriptblock.
    param()

    function Get-Containers
    {
        param()
    }
}

Target + Scope is ex. useful when applying the attributes at the top of your script, but it's gready. Using RuleSuppressionId/checkId will allow you exclude that specific occurrence only. Not that it's a realistic scenario but you could even do this:
# Intentionally conflicting functions

# Will be suppressed
function Get-Elements
{
    # This will only suppress Get-Elements
    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', 'Get-Elements')]
    param()

    # Will NOT be suppressed
    function Get-Containers
    {
        param()
    }
}

# Will NOT be suppressed
function Get-Elements
{
    # This will only suppress Get-Elements
    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSUseSingularNouns', 'Get-Containers')]
    param()

    # Will be suppressed
    function Get-Containers
    {
        param()
    }
}

Multiple rules already supports RuleSuppressionId which is why I provided this PR to be consistent. Ideally though, PSSA would used MessageId for this as RuleSuppressionId\checkId should've been the rule ID just like in C#/.NET. See https://github.com/PowerShell/PSScriptAnalyzer/issues/276
You already figured most of this out, but some quick comments about the samples in your post:

First sample: Nothing is suppressed, but any inner functions would be. There's no error in the first place as Get-QueueMessageMetadata is singular.
Second sample: When using Target you also need Scope (you should get an error detailing this when running Invoke-ScriptAnalyzer). The suppression also needs to occur directly above a param() block as you mentioned. I've requested an alternative to this for PSSAv2 here: https://github.com/PowerShell/PSScriptAnalyzer/issues/1483#issuecomment-721428142
Third sample: Works since attribute is above param(), but it will also apply to any child functions etc.

Especially the one that was declared at the script scope outside the function but which specified a target containing the function name!

It should work if you add it above param() at the top of your script.

Thanks for your comments. They are helpful to allow me to better understand how PowerShell, and specifically, PSSA is working with attributes.
However, to the contrary; when I did not supply a suppression attribute for Get-QueueMessageMetadata, the function was flagged. I'm running PSSA from a PowerShell Core session, which is using the Pluralization.NET library (instead of EF pluralization service), and data is both singular and plural at the same time and is flagged. In fact, IIRC for this particular rule, there is explicit code to ignore the word data! 😄.
As you pointed out for the second one, I was missing the Param () block. Is this a requirement of PSSA, or, as I suggested, an ambiguity in the PowerShell parser which therefore, makes that a requirement for PSSA? I'm simply curious at this point about the technical underpinning of that requirement, since the advanced function in and of itself does not require a Param() block.
It is interesting to see that the attributes I supplied would suppress the enclosing function and all child functions. That I did not know/understand.
Thanks again for helping me better understand PSSA (and PowerShell). I have many years of PowerShell under my belt, but I continue to learn new things about it every day!

Regarding my question on the need for the Param () block, I raised an issue about it via #1930. This is indeed some sort of PowerShell parser ambiguity because of the way the language was designed when incorporating the use of attributes. The addition of the Param () block disambiguates the parsing such that the attributes get applied to the right syntactical block. Other than that, everything else I said above stands.

It's inherited design/behavior from C#/.NET AFAIK. Attributes in general are placed above the element with, which is why you need param() to basically say "this scriptblock/file". If not it'd keep looking for the first function/class.

Using attributes
Attributes can be placed on almost any declaration, though a specific attribute might restrict the types of declarations on which it's valid. In C#, you specify an attribute by placing the name of the attribute enclosed in square brackets ([]) above the declaration of the entity to which it applies.

Source
In the issue I mentioned in the last post a comment-based suppression method is suggested for PSSAv2 to get more granular control without this limitation, similar to pragma used in other languages.

I can't find in any of my resources that the Param ( ) block is required when using the [CmdletBinding()] attribute—the function is parsed and processed by PowerShell just fine.

It is.
> function t { [CmdletBinding()] }
ParserError:
Line |
   1 |  function t { [CmdletBinding()] }
     |               ~~~~~~~~~~~~~~~~~
     | Unexpected attribute 'CmdletBinding'.

Regarding not needing Param () block: Huh! But the function ran just fine without it! I don't like that (it seems) PowerShell sometimes "lets things slide". 😉 When I first came to PowerShell, I was a C# dev. And naturally, I treated PowerShell like C# in the console. Boy was that a mistake! It wasn't long before I got burned and was wondering what the heck was going on! I've come a long way since then, but there are still lots of surprises with PowerShell--to the point where I have a love/hate relationship with it. To be honest, though, I more love it than hate it. It's a deep language.
Thanks again! I'll be more "dogmatic" when writing functions and always supply a Param () block even if no params are needed. In the end, it'll save me a lot of headaches! 😋

@sdwheeler - this will need some documentation eventually

@sdwheeler - this will need some documentation eventually

@JamesWTruher I am not sure what needs to be documented here. Can you elaborate? Can't all rules be suppressed (ie. was this a bug) or are there other rules that can't be suppressed? If there are rules that can't be suppressed, then we need a list.

@sdwheeler @fflaten  Documentation wise we already document here some examples of rules where one can suppress more specifically: https://learn.microsoft.com/en-us/powershell/utility-modules/psscriptanalyzer/using-scriptanalyzer?view=ps-modules#suppressing-rules
My recommendation would be to leave a small remark in above document as well as in the rules own documentation markdown file here. Otherwise LGTM


Not able catch exception by raise_error.

This is more of a question. I have this method i want to test.
module Test
  class InvalidPathException < Exception; end

  class Application
    def load
      raise Test::InvalidPathException.new
    end
  end
end

Now I wrote this spec.
Spec2.describe Test::Application do
  Spec2.describe "#load" do
    it "should raise error if path is not valid" do
      path = "wrong/path.yml"
      application = Test::Application.new("test", path)
      expect(application.load).to raise_error Test::InvalidPathException
    end
  end
end

I think this test should pass. But it doesn't. Am i doing something silly?

expect {
  # .. code ..
}.to raise_error(...)

Aha. Don't know how i missed that.


Add a http client implementation.

Implemented using the lwip http client.
Fixes #1386

Other comments,
1/ Passing the hostname in http_client_init means you can't reuse the same "state" to make a request to a different host. Seems a bit restrictive? The only reason I can think for this is that your tls config might be related to the host you're talking to. It's easy enough to allocate "state" for each host?
2/ Can't run multiple requests at the same time? You could do this by calling http_client_init multiple times for the number of requests you want to have running at the same time. The lwip api just allocates a new state for each request and frees this in the callback. We could do the same but it seems wasteful when you want to make multiple requests one after another (to the same host).

Yeah, perhaps have a friendly API which combines the underlying "state" init, and the actual request, but expose the underlying methods too.
I guess I need to understand what the thing we are initializing is (it's life time) to know what it should be called (is it actually a client instance, a connection, etc.?)

I guess I need to understand what the thing we are initializing is (it's life time)

It looks like the lwip api is per http request - which is a bit naff. This was just supposed to be a "helper" utility to make the lwip functionality a bit easier to use. Originally it was just part of the example code but I thought that was a bit lazy and should maybe add it to the sdk. Maybe I should forget that.

I pushed a new version. To make an http request is something like this...
PICO_HTTP_REQUEST_T req = {0};
req.hostname = HOST;
req.url = URL;
req.recv_fn = response_function;
int rc = http_client_request_sync(cyw43_arch_async_context(), &req);

I'm not sure this is adding a lot of value, so I might just move it back to pico-examples. What do you think?


Error: Entry Point Not Found

Azure Data Studio Version: 1.47.1 and 1.48.0 (Insiders build)
OS Version: Windows Server 2012 R2 Standard

Steps to Reproduce:

Install (either user or system)
Launch

Does this issue occur when all extensions are disabled?: Yes

Note: This issue DOES NOT occur with version 1.46.1
Full error message:
azuredatastudio-insiders.exe - Entry Point Not Found
The procedure entry point DiscardVirtualMemory could not be located in the dynamic link library
C:\Users<userid>\AppData\Local\Programs\Azure Data Studio - Insiders\azuredatastudio-insiders.exe.
Problem details:
Problem signature:
Problem Event Name:	APPCRASH
Application Name:	azuredatastudio-insiders.exe
Application Version:	<IP_ADDRESS>
Application Timestamp:	64ecb487
Fault Module Name:	rs\azuredatastudio-insiders.exe!DiscardVirtualMemory
Fault Module Version:	6.3.9600.21620
Fault Module Timestamp:	65175333
Exception Code:	c0000139
Exception Offset:<PHONE_NUMBER>0ed170
OS Version:	6.3.96<IP_ADDRESS>.272.7
Locale ID:	1033
Additional Information 1:	ac05
Additional Information 2:	ac0507478d1c5bd693cfc4fe3987e900
Additional Information 3:	ac05
Additional Information 4:	ac0507478d1c5bd693cfc4fe3987e900

Same problem
Azure Data Studio Version: 1.48.1
OS Version: Windows Server 2012 R2 Datacenter


Qnap Installation

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Can someone point me to a guide to install this on qnap?
Thanks
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
Latest
💻 Operating System and Arch
Qnap
🌐 Browser
Qnap
🐋 Docker Version
No response
🟩 NodeJS Version
No response

You should be able to install Docker on your Qnap. After installed Docker, come back this repo and follow the README.md.

Thanks


Support for users with multiple providers

This PR adds more realistic support for providers in the prototype. In particular how we respond to users who belong to more than one provider - about 20% of our user base.
Key changes:

Courses and records generated for a number of providers
All records now relate to a provider
The signed in user can belong to multiple providers
When adding records, we assign a provider to that record

We're still deciding whether to adopt a 'blended' or 'hat' model for users who belong to multiple providers - this pr supports both in the ui so that we can try them out.
Hat model
In this model (closest to DTTP and Publish), users must pick a provider to work as when signing in. To add trainees from different providers, they must back out and choose a different provider to work on - essentially putting on the 'hat' for the provider they're acting as.
Choosing provider

Showing provider on draft record

Blended model
In this model, users who belong to multiple providers can see all records at once - but filter by a provider should they choose. This is probably the most flexible for ways of working, but with the downside that when adding a new record they'll need to choose a provider.
Filtering by provider

Choosing a provider when adding a record

Showing provider on draft record

 Prototype: support for multiple providers

 Prototype: support for multiple providers


Package request: wio (fork)

Project description
Wayland compositor similar to Plan 9's rio (mirror).
Note that the upstream for this software was removed previously due to being unmaintained and depending on an outdated wlroots. I am proposing that this fork of that software is added to pkgs and the entry for the original in aliases.nix is amended to point to the fork. Note also that the only changes made from the original wio derivation to build the software (other then changing src) is the addition of preConfigure = ''sed -i "16 a '-Wno-maybe-uninitialized'," meson.build'';. I am pinging @AndersonTorres because they where the maintainer of the deprecated software that this addition will posthumously replace.
Metadata

homepage URL: https://github.com/Rubo3/wio
source URL: https://github.com/Rubo3/wio
license: BSD 3-Clause
platforms: linux

I am happy to know!


LSC power plug "Device did not appear with the intermediate firmware"

name, model = LSC smart power plug
This device is supported according to the wiki and also according through tasmota.
I do not know how to retrieve the firmware version.
I have not yet connected it to the official app in order to not update the firmware.
(I though this would be beter as I want to flash tasmota on it.)
I'll connect it if its necessary.
Output:
tuya-convert v2.4.4
======================================================
TUYA-CONVERT

https://github.com/ct-Open-Source/tuya-convert
TUYA-CONVERT was developed by Michael Steigerwald from the IT security company VTRUST (https://www.vtrust.de/) in collaboration with the techjournalists Merlin Schumacher, Pina Merkert, Andrijan Moecker and Jan Mahn at c't Magazine. (https://www.ct.de/)

======================================================
PLEASE READ THIS CAREFULLY!
======================================================
TUYA-CONVERT creates a fake update server environment for ESP8266/85 based tuya devices. It enables you to backup your devices firmware and upload an alternative one (e.g. ESPEasy, Tasmota, Espurna) without the need to open the device and solder a serial connection (OTA, Over-the-air).
Please make sure that you understand the consequences of flashing an alternative firmware, since you might lose functionality!

Flashing an alternative firmware can cause unexpected device behavior and/or render the device unusable. Be aware that you do use this software at YOUR OWN RISK! Please acknowledge that VTRUST and c't Magazine (or Heise Medien GmbH & Co. KG) CAN NOT be held accountable for ANY DAMAGE or LOSS OF FUNCTIONALITY by typing yes + Enter

yes
Checking for network interface wlan0... Found.
Checking UDP port 53... Available.
Checking UDP port 67... Available.
Checking TCP port 80... Available.
Checking TCP port 443... Available.
Checking UDP port 6666... Available.
Checking UDP port 6667... Available.
Checking TCP port 1883... Available.
Checking TCP port 8886... Available.
======================================================
  Starting AP in a screen...
  Starting web server in a screen
  Starting Mosquitto in a screen
  Starting PSK frontend in a screen
  Starting Tuya Discovery in a screen

======================================================

IMPORTANT
1. Connect any other device (a smartphone or something) to the WIFI vtrust-flash
   This step is IMPORTANT otherwise the smartconfig may not work!
2. Put your IoT device in autoconfig/smartconfig/pairing mode (LED will blink fast). This is usually done by pressing and holding the primary button of the device
   Make sure nothing else is plugged into your IoT device while attempting to flash.
3. Press ENTER to continue

======================================================
Starting smart config pairing procedure
Waiting for the device to install the intermediate firmware
Put device in EZ config mode (blinking fast)
Sending SSID                  vtrust-flash
Sending wifiPassword
Sending token                 00000000
Sending secret                0101
................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
.................
SmartConfig complete.
Resending SmartConfig Packets
................
SmartConfig complete.
Resending SmartConfig Packets
..............
Device did not appear with the intermediate firmware
Check the *.log files in the scripts folder
Do you want to try flashing another device? [y/N] n
======================================================
Cleaning up...

Closing AP
Exiting...

smarthack-wifi.log
smarthack-web.log
smarthack-udp.log
smarthack-psk.log
smarthack-mqtt.log

Unfortunately, while this device may have been previously supported, the model you have is not.
According to the MAC address in your wifi log, the device does not use an ESP. This means it cannot run ESP firmware such as Tasmota and is not compatible with tuya-convert.
We've been seeing this a lot lately; manufacturers are switching chipsets without changing model numbers or FCC IDs.

Hello, how can you find the chipset through the MAC address?
It is not connected to issue 483?
I really was hoping to get this plug out of the cloud...
How can I buy a new plug that I can flash if you can't trust the manufacturers to not switch chipsets.
It says on the box that it's rev 2.0 and I read on a forum that it was possible to flash this plug.

how can you find the chipset through the MAC address?

You can use an OUI lookup tool, such as this one from Wireshark

It is not connected to issue 483?

No

I really was hoping to get this plug out of the cloud...
How can I buy a new plug that I can flash if you can't trust the manufacturers to not switch chipsets.

I understand, as we all do. It is sadly not a priority for these companies but hopefully we can convince them otherwise. We need to be writing these companies and letting them know that there is a huge market for off cloud and open source IoT.

I have added a note to the wiki, so that others do not have the same issue.
I'll also try to do this on the supported devices of the tasmota wiki

@Haraldvdh thanks for the tear-down pictures. They are probably appreciated by the folks at https://github.com/openshwprojects/OpenBK7231T_App/issues/172 and if you add it to their list of Beken BK7231T teardowns in the (mostly polish) electroda forum you may as well get an SD-Card.
Here is another (dutch) forum that focuses on this Smart Plug model: https://gathering.tweakers.net/forum/list_message/63909938#63909938


Added onDeferredPurchase event

Fixes https://github.com/adaptyteam/AdaptySDK-React-Native/issues/10

Hey! Gonna test around today, stay tuned!


Sync with latest template

Synched with latest template code.
Fixed linter related changes.
closes #28

The test failures related to consul_default_policy=deny  are known and will be fixed in #32. Since the other tests have passed so I am going to merge this PR.


[StatsMigration] Remove database rows for missing partition/account/container in AccountReports

When a new StatsWrapper is generated and persisted to database, we only override the database rows where the storage value is different. Since we have a copy of previous StatsWrapper, we can easily know which database rows to update.
However, we don't remove those database rows when any partition or account or container no longer exists in the host, we just silently ignore it.
This PR would take the absence of partition/account/container into consideration and remove corresponding database rows.
We are not using batch operations for these DELETEs, since we know the delete should be a rare operation.

Codecov Report

Merging #1822 (4586cad) into master (da4f106) will decrease coverage by 28.06%.
The diff coverage is 79.79%.

@@              Coverage Diff              @@
##             master    #1822       +/-   ##
=============================================
- Coverage     73.99%   45.92%   -28.07%
+ Complexity     9536     5561     -3975
=============================================
  Files           690      690
  Lines         52213    52312       +99
  Branches       6488     6498       +10
=============================================
- Hits          38636    24026    -14610
- Misses        11553    26283    +14730
+ Partials       2024     2003       -21

Impacted Files
Coverage Δ
Complexity Δ

...m/github/ambry/accountstats/AccountReportsDao.java
66.05% <72.00%> (-33.95%)
7.00 <3.00> (+2.00)
:arrow_down:

...hub/ambry/accountstats/AccountStatsMySqlStore.java
95.84% <87.75%> (-2.24%)
69.00 <0.00> (+8.00)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaName.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...rc/main/java/com/github/ambry/rest/RestMethod.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...c/main/java/com/github/ambry/router/ByteRange.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-9.00%)

...main/java/com/github/ambry/commons/Thresholds.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-5.00%)

...main/java/com/github/ambry/config/QuotaConfig.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...in/java/com/github/ambry/frontend/IdConverter.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...in/java/com/github/ambry/router/CryptoService.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-2.00%)

...n/java/com/github/ambry/config/FrontendConfig.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

... and 403 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #1822 (6ce0537) into master (b25e907) will decrease coverage by 58.78%.
The diff coverage is 0.00%.

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


Test methods for TriggerBase

This pull request increases the code coverage of the apex class TriggerBase (79/80 lines covered).
Added test methods in TriggerBaseTest:

idToNumberOfTimesSeenBeforeUpdate
idToNumberOfTimesSeenAfterUpdate
run_false

PR comments:

[X] Tests pass: Yes, they do.
[X] Appropriate changes to README are included in PR: Not needed.

Thanks for your fast response Mitch! Let me know if you want more changes.


"Error: can not create command-line process (2)"

It seems that any format I choose gives me this error regardless of the files, is there a way to post a log?
I am using the latest version, it happens for every file I try to convert.

You have to download command-line tools https://github.com/wieslawsoltes/BatchEncoder/blob/dev/README.md#tool-downloads and Configure formats to set command-line tools executable paths.

I did what was shown in your video (in the wiki) and I have even added multiple encoders to my path on windows, somehow I don't think it's your GUI causing the issue, for some reason all the encoders under my CMD end up giving me "Error parsing input file: filename.extension" or metadata blocks errors for flac.
I've tried searching on the internet for how to fix these errors but nothing has appeared.

@dragons4life Try putting some command-line encoder in same folder as BatchEncoder or different path (or disk) and configure path from GUI and check if error occurs. The error message you are getting is that BatchEncoder  can not find encoder .exe in path you have provided. So this my be some permissions issue (no access to this directory), invalid path or some AV software is messing this up in your system.

@wieslawsoltes Actually the issue was the file I was encoding apparently was broken, after restarting my computer (probably because sometimes PATH(S) need the computer to restart to work on windows or using refreshenv) everything seem to work fine, but it's odd because I couldn't encode any other files either even while the CMD detected the encoders and using the CMD.
Oh well, a random restart worked that's all I know. thanks for helping me.


Add rule to check for irregular spaces

This PR adds a rule to check if state contains irregular spaces as discussed under #74.
Irregular spaces are usually unneeded and unwanted.
List taken from similar ESLint rule.

Thanks @sblaisot for the PR!
Maybe the non-braking space isn't the only irregular whitespace we want to look for. Would it be possible to update this rule with other characters and rename it to NonIrregularWhitespaceRule? Besides possible parsing errors the irregular whitespace also make code harder to debug in a similar nature to mixed tabs and spaces.
We might take the no-irregular-whitespace from ESLint: https://eslint.org/docs/rules/no-irregular-whitespace as an example.

ok, I updated to detect all Irregular Spaces

Thanks @sblaisot for updating the rule! I'm going the merge the pull request but might update it to include https://docs.saltstack.com/en/latest/topics/troubleshooting/yaml_idiosyncrasies.html#yaml-supports-only-plain-ascii before the next release.


Firefox missing libharfbuzz.so.0

Thanks for making these scripts! It was super helpful to be able to get up and running with Playwright quickly without having to deal with too much NixOS stuff :)
I'm having a problem running Firefox — attached is the patch log and playwright error:
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/liblgpllibs.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/plugin-container
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libmozavcodec.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/gmp-clearkey/0.1/libclearkey.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libmozgtk.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libplc4.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libsoftokn3.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/updater
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libmozwayland.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libnss3.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libplds4.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libmozsqlite3.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libnspr4.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/firefox-bin
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libsmime3.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libssl3.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libnssutil3.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/pingsender
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libfreeblpriv3.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/firefox
patchelf: not an ELF executable
patchelf: not an ELF executable
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libnssckbi.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libmozavutil.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libxul.so
/home/wesleyac/.cache/ms-playwright/firefox-1297/firefox/libmozsandbox.so

  3) [firefox] › tests/e2e.spec.ts:19:1 › misc =====================================================

    browserType.launch: Host system is missing dependencies!

      Missing libraries are:
          libharfbuzz.so.0

        at Object._baseTest.extend.browser.scope [as fn] (/home/wesleyac/code/deeplinks/.yarn/cache/@playwright-test-npm-1.16.3-3b65662da1-363f2f214d.zip/node_modules/@playwright/test/lib/index.js:221:40)
        at Fixture.setup (/home/wesleyac/code/deeplinks/.yarn/cache/@playwright-test-npm-1.16.3-3b65662da1-363f2f214d.zip/node_modules/@playwright/test/lib/fixtures.js:74:73)
        at FixtureRunner.setupFixtureForRegistration (/home/wesleyac/code/deeplinks/.yarn/cache/@playwright-test-npm-1.16.3-3b65662da1-363f2f214d.zip/node_modules/@playwright/test/lib/fixtures.js:315:5)
        at Fixture.setup (/home/wesleyac/code/deeplinks/.yarn/cache/@playwright-test-npm-1.16.3-3b65662da1-363f2f214d.zip/node_modules/@playwright/test/lib/fixtures.js:58:19)
        at FixtureRunner.setupFixtureForRegistration (/home/wesleyac/code/deeplinks/.yarn/cache/@playwright-test-npm-1.16.3-3b65662da1-363f2f214d.zip/node_modules/@playwright/test/lib/fixtures.js:315:5)
        at Fixture.setup (/home/wesleyac/code/deeplinks/.yarn/cache/@playwright-test-npm-1.16.3-3b65662da1-363f2f214d.zip/node_modules/@playwright/test/lib/fixtures.js:58:19)
        at FixtureRunner.setupFixtureForRegistration (/home/wesleyac/code/deeplinks/.yarn/cache/@playwright-test-npm-1.16.3-3b65662da1-363f2f214d.zip/node_modules/@playwright/test/lib/fixtures.js:315:5)
        at FixtureRunner.resolveParametersAndRunHookOrTest (/home/wesleyac/code/deeplinks/.yarn/cache/@playwright-test-npm-1.16.3-3b65662da1-363f2f214d.zip/node_modules/@playwright/test/lib/fixtures.js:300:23)
        at WorkerRunner._runTestWithBeforeHooks (/home/wesleyac/code/deeplinks/.yarn/cache/@playwright-test-npm-1.16.3-3b65662da1-363f2f214d.zip/node_modules/@playwright/test/lib/workerRunner.js:478:7)

which firefox reports /nix/store/dzd85wsi9wh00ndm5xazxgx6f7sdxxy1-firefox-bin-94.0.2/bin/firefox. I'm using playwright version 1.16.3 (running it in node, using @playwright/test). I'm running on NixOS 21.05.
Strangely, running ~/.cache/ms-playwright/firefox-1297/firefox/firefox-bin launches a instance of firefox that appears to work fine.
Let me know if you need any more info!

Oh, and ldd ~/.cache/ms-playwright/firefox-1297/firefox/firefox-bin reports the following:
	linux-vdso.so.1 (0x00007fff90579000)
	libpthread.so.0 => /nix/store/jsp3h3wpzc842j0rz61m5ly71ak6qgdn-glibc-2.32-54/lib/libpthread.so.0 (0x00007f02ff5d6000)
	libdl.so.2 => /nix/store/jsp3h3wpzc842j0rz61m5ly71ak6qgdn-glibc-2.32-54/lib/libdl.so.2 (0x00007f02ff5d1000)
	libstdc++.so.6 => /nix/store/9w7ra0b2nqnbly4v64vwayml255lb9bp-gcc-10.3.0-lib/lib/libstdc++.so.6 (0x00007f02ff3fa000)
	libm.so.6 => /nix/store/jsp3h3wpzc842j0rz61m5ly71ak6qgdn-glibc-2.32-54/lib/libm.so.6 (0x00007f02ff2b7000)
	libgcc_s.so.1 => /nix/store/9w7ra0b2nqnbly4v64vwayml255lb9bp-gcc-10.3.0-lib/lib/libgcc_s.so.1 (0x00007f02ff29d000)
	libc.so.6 => /nix/store/jsp3h3wpzc842j0rz61m5ly71ak6qgdn-glibc-2.32-54/lib/libc.so.6 (0x00007f02ff0dc000)
	/nix/store/jsp3h3wpzc842j0rz61m5ly71ak6qgdn-glibc-2.32-54/lib/ld-linux-x86-64.so.2 => /nix/store/jsp3h3wpzc842j0rz61m5ly71ak6qgdn-glibc-2.32-54/lib64/ld-linux-x86-64.so.2 (0x00007f02ff6bd000)

cp /nix/store/klbfc76skpzs7z807kjppqlvkxsc0w34-harfbuzz-2.8.0/lib/libharfbuzz.so.0 ~/.cache/ms-playwright/firefox-1297/firefox/ is a dumb hack that seems to fix this, but I presume the real fix is for the harfbuzz dependency to be actually detected somewhere and patchelfed in.

Thanks for the report. Are you reproducing this just by running the playwright tests, or by launching Firefox through playwright? I don't use playwright at the moment, but I would take a PR for this or happily transfer the repo over.

I'm reproducing by running playwright tests — yarn playwright test in my case.
I'll look into making a PR for this at some point, and if I get that far would be happy to take over the repo.


Feature Request: Code Reviews

Hi, thanks for the amazing utility. 💯
I was wondering if it's feasible enough to provide support for Code Reviews?
There are 4 pillars to GitHub's activity overview - Commits [supported], Pull Requests [supported], Issues [supported], and lastly Code review [unsupported]

I don't think (?) there's a straightforward GraphQL query that GitHub provides (could be wrong).
Is there any workaround (if it's even feasible)?

Thanks!
I think you can get that info by adding totalPullRequestReviewContributions here:
https://github.com/teoxoy/profile-readme-stats/blob/57da79551137662a3025d908c0e8015d6298f4f0/src/main.ts#L196
I don't have the bandwidth to add this now though. PRs are welcome


Extract data tables from markdown in migration doc

Closes: #3552
What's Changed

Extract the large data tables from upgrading-from-nautobot-v1.md into a set of separate standalone YAML files. This should make it easier to maintain/update these tables in future without introducing massive reformatting and merge conflicts as we've been encountering to date.

This adds mkdocs-table-reader-plugin as a dev dependency, which is a bit heavyweight as it actually uses pandas to load the YAML and render to a Markdown table. I'm open to recommendations for alternative packages to use here, but since it's a dev-only dependency I'm not too concerned about proceeding with this.

Restructure the YAML data a bit to mostly give us tables that fit in the rendered docs without horizontal scrolling. The exception is the "Code location changes" table which is necessarily very wide due to the presence of convert_querydict_to_factory_formset_acceptable_querydict and ensure_content_type_and_field_name_inquery_params in the table data.
Make some corrections to the data that I noticed in the process of translating the tables to YAML
Add yamllint; for now it's only pointed at these new data tables, as I didn't want to take on the work of fixing docker-compose.*.yaml and .github/workflows/*.yaml to make it happy as a part of this PR.

Please review the rendered docs at https://docs-landing-page--3553.org.readthedocs.build/projects/core/en/3553/installation/upgrading-from-nautobot-v1/#removed-serializer-fields
TODO

[x] Explanation of Change(s)
[x] Added change log fragment(s) (for more information see the documentation)
[x] Attached Screenshots, Payload Example
n/a Unit, Integration Tests
[x] Documentation Updates (when adding/changing features)
n/a Example Plugin Updates (when adding/changing features)
[x] Outline Remaining Work, Constraints from Design

Blocked by #2316.

The updated page is so much better (i.e. human friendly factor 10x by hiding the tables inside of the admonitions), outstanding work!


How to get the original raw JSON payload from channel

Bot Info

SDK Platform: .NET Framework 4.6
SDK Version: 3.15
Active Channels: Facebook Messenger, Direct Line & Emulator
Deployment Environment: Local development with Emulator

Issue Description
I'm trying to get the current raw message from the activity. For example, if Im connected to facebook messenger, and an activity gets sent to the bot, how can I get the raw json that messenger sends to botframework? is that possible?
Code Example
N/A
Reproduction Steps
N/A

@odannyc have you tried accessing the ChannelData property on the Activity? This should have the native channel's schema, i.e. for this case, the Facebook Messenger message.
You can find the ChannelData property listed here.


Add bun provider to supported package managers

What does this PR address?

Updates documentation

Fixes # (issue)
Before submitting:

[x] Make sure to follow GitHub's guide on creating PR.
[x] Did you read through contribution guidelines?
[x] Did your changes require updates to the documentation? Have you updated those accordingly?
[ ] Did you write tests to cover your changes? Did it pass locally when running cargo test?
[ ] Have you run cargo fmt, cargo check and cargo clippy locally to ensure that CI passes?

Will reopen when Bun is working on Railway again


No License

It would be fantastic if the project will be released under a license. I would like to use the well written code.
More Info

@foxos42 it's open source, i'll add a license.
In last private revision I updated the code to camelCase to be more in touch it PHP standards.
Also made it even more compact and optimized, will commit in about one week, stay tuned.

@madorin Nice to hear. :+1:  Don't hassle to much with CamelCase, there is no definitive naming convention in PHP. They differ even by framework.

Permissive license added.

Excellent, thanks.


[Breaking Change] Adding new client required parameter should be a breaking change

For example, as the following swagger, the first version only has two operations: PacketCoreControlPlaneVersions_Get and PacketCoreControlPlaneVersions_List, in the new version, PacketCoreControlPlaneVersions_GetBySubscription and PacketCoreControlPlaneVersions_ListBySubscription have been added. The new added parameter SubscriptionIdParameter is a client level parameter, which should be a breaking change.
 "/providers/Microsoft.MobileNetwork/packetCoreControlPlaneVersions/{versionName}": {
      "get": {
        "tags": [
          "PacketCoreControlPlaneVersion"
        ],
        "operationId": "PacketCoreControlPlaneVersions_Get",
        "description": "Gets information about the specified packet core control plane version.",
        "parameters": [
          {
            "name": "versionName",
            "in": "path",
            "required": true,
            "type": "string",
            "description": "The name of the packet core control plane version."
          },
          {
            "$ref": "../../../../../common-types/resource-management/v5/types.json#/parameters/ApiVersionParameter"
          }
        ],
        "responses": {
          "200": {
            "description": "Request successful. The operation returns the resulting packet core control plane version resource.",
            "schema": {
              "$ref": "#/definitions/PacketCoreControlPlaneVersion"
            }
          },
          "default": {
            "description": "Error response describing why the operation failed.",
            "schema": {
              "$ref": "../../../../../common-types/resource-management/v5/types.json#/definitions/ErrorResponse"
            }
          }
        },
        "x-ms-examples": {
          "Get packet core control plane version": {
            "$ref": "./examples/PacketCoreControlPlaneVersionGet.json"
          }
        }
      }
    },
    "/providers/Microsoft.MobileNetwork/packetCoreControlPlaneVersions": {
      "get": {
        "tags": [
          "PacketCoreControlPlaneVersion"
        ],
        "operationId": "PacketCoreControlPlaneVersions_List",
        "description": "Lists all supported packet core control planes versions.",
        "parameters": [
          {
            "$ref": "../../../../../common-types/resource-management/v5/types.json#/parameters/ApiVersionParameter"
          }
        ],
        "responses": {
          "200": {
            "description": "Get supported packet core control plane versions successful.",
            "schema": {
              "$ref": "#/definitions/PacketCoreControlPlaneVersionListResult"
            }
          },
          "default": {
            "description": "Error response describing why the operation failed.",
            "schema": {
              "$ref": "../../../../../common-types/resource-management/v5/types.json#/definitions/ErrorResponse"
            }
          }
        },
        "x-ms-examples": {
          "Get supported packet core control plane versions": {
            "$ref": "./examples/PacketCoreControlPlaneVersionList.json"
          }
        },
        "x-ms-pageable": {
          "nextLinkName": "nextLink"
        }
      }
    },
    "/subscriptions/{subscriptionId}/providers/Microsoft.MobileNetwork/packetCoreControlPlaneVersions/{versionName}": {
      "get": {
        "tags": [
          "PacketCoreControlPlaneVersion"
        ],
        "operationId": "PacketCoreControlPlaneVersions_GetBySubscription",
        "description": "Gets information about the specified packet core control plane version.",
        "parameters": [
          {
            "name": "versionName",
            "in": "path",
            "required": true,
            "type": "string",
            "description": "The name of the packet core control plane version."
          },
          {
            "$ref": "../../../../../common-types/resource-management/v5/types.json#/parameters/SubscriptionIdParameter"
          },
          {
            "$ref": "../../../../../common-types/resource-management/v5/types.json#/parameters/ApiVersionParameter"
          }
        ],
        "responses": {
          "200": {
            "description": "Request successful. The operation returns the resulting packet core control plane version resource.",
            "schema": {
              "$ref": "#/definitions/PacketCoreControlPlaneVersion"
            }
          },
          "default": {
            "description": "Error response describing why the operation failed.",
            "schema": {
              "$ref": "../../../../../common-types/resource-management/v5/types.json#/definitions/ErrorResponse"
            }
          }
        },
        "x-ms-examples": {
          "Get packet core control plane version by subscription": {
            "$ref": "./examples/PacketCoreControlPlaneVersionGetBySubscription.json"
          }
        }
      }
    },
    "/subscriptions/{subscriptionId}/providers/Microsoft.MobileNetwork/packetCoreControlPlaneVersions": {
      "get": {
        "tags": [
          "PacketCoreControlPlaneVersion"
        ],
        "operationId": "PacketCoreControlPlaneVersions_ListBySubscription",
        "description": "Lists all supported packet core control planes versions.",
        "parameters": [
          {
            "$ref": "../../../../../common-types/resource-management/v5/types.json#/parameters/SubscriptionIdParameter"
          },
          {
            "$ref": "../../../../../common-types/resource-management/v5/types.json#/parameters/ApiVersionParameter"
          }
        ],
        "responses": {
          "200": {
            "description": "Get supported packet core control plane versions successful.",
            "schema": {
              "$ref": "#/definitions/PacketCoreControlPlaneVersionListResult"
            }
          },
          "default": {
            "description": "Error response describing why the operation failed.",
            "schema": {
              "$ref": "../../../../../common-types/resource-management/v5/types.json#/definitions/ErrorResponse"
            }
          }
        },
        "x-ms-examples": {
          "Get supported packet core control plane versions by subscription": {
            "$ref": "./examples/PacketCoreControlPlaneVersionListBySubscription.json"
          }
        },
        "x-ms-pageable": {
          "nextLinkName": "nextLink"
        }
      }
    }

@tadelesh @raych1 can you share a URL of a PR that has this problem and the breaking change tool is not detecting it?

https://github.com/Azure/azure-rest-api-specs/pull/24528

@tadelesh Chenjie Shi FTE @raych1 Ray Chen FTE can you share a URL of a PR that has this problem and the breaking change tool is not detecting it?

https://github.com/Azure/azure-rest-api-specs/pull/24528


Specifying SWOOLE_CONFIGURE_OPTS does not take effect

Describe the bug

Version

[x] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[x] v2
[ ] v1

Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems
Ubuntu
PHP versions
PHP 8.1
To Reproduce
      - name: Setup PHP
        uses: shivammathur/setup-php@v2
        with:
          php-version:8.1
          extensions: redis, pdo, pdo_mysql, bcmath, swoole-5.0.3
          tools: phpize
          coverage: none
          ini-values: swoole.use_shortname=Off
        env:
          SWOOLE_CONFIGURE_OPTS: --enable-openssl --enable-swoole-curl --enable-cares --enable-swoole-pgsql --enable-brotli

Screenshots/Logs
Run shivammathur/setup-php@v2
  with:
    php-version: 8.1
    extensions: redis, pdo, pdo_mysql, bcmath, swoole-5.0.3
    tools: phpize
    coverage: none
    ini-values: swoole.use_shortname=Off
    ini-file: production
  env:
    PHP_CS_FIXER_IGNORE_ENV: 1
    SWOOLE_CONFIGURE_OPTS: --enable-openssl --enable-swoole-curl --enable-cares --enable-swoole-pgsql --enable-brotli
/usr/bin/bash /home/runner/work/_actions/shivammathur/setup-php/v[2](https://github.com/friendsofhyperf/components/actions/runs/5208302373/jobs/9396736711#step:3:2)/src/scripts/run.sh

==> Setup PHP
✓ PHP Updated to PHP 8.1.18

==> Setup Extensions
✓ redis Enabled
✓ pdo Enabled
✓ pdo_mysql Enabled
✓ bcmath Enabled
✓ swoole-[5](https://github.com/friendsofhyperf/components/actions/runs/5208302373/jobs/9396736711#step:3:5).0.1 Installed and enabled

==> Setup Tools
✓ composer Added composer 2.5.[7](https://github.com/friendsofhyperf/components/actions/runs/5208302373/jobs/9396736711#step:3:7)
✓ php-config Added php-config [8](https://github.com/friendsofhyperf/components/actions/runs/5208302373/jobs/9396736711#step:3:8).1.18

Please specify the libraries, without them, it fails to compile and fallbacks to use the extension from the ubuntu package.
- name: Setup PHP
  uses: shivammathur/setup-php@verbose
  with:
    php-version: 8.1
    extensions: redis, pdo, pdo_mysql, bcmath, swoole-5.0.3
    tools: phpize
    coverage: none
    ini-values: swoole.use_shortname=Off
  env:
    SWOOLE_LIBS: libc-ares-dev libcurl4-openssl-dev libpq-dev libbrotli-dev
    SWOOLE_CONFIGURE_OPTS: --enable-openssl --enable-swoole-curl --enable-cares --enable-swoole-pgsql --enable-brotli

downloading swoole-5.0.3.tgz ...
Starting to download swoole-5.0.3.tgz (2,066,852 bytes)
......................................................................................................................................................................................................................................................................................................................................................................................................................done: 2,066,852 bytes
471 source files, building
running: phpize
Configuring for:
PHP Api Version:         20210902
Zend Module Api No:      20210902
Zend Extension Api No:   420210902
configure.ac:22: warning: $as_echo is obsolete; use AS_ECHO(["message"]) instead
build/php.m4:2111: PHP_CONFIG_NICE is expanded from...
configure.ac:22: the top level
config.m4:334: warning: The macro `AC_PROG_CC_C99' is obsolete.
config.m4:334: You should run autoupdate.
./lib/autoconf/c.m4:1659: AC_PROG_CC_C99 is expanded from...
config.m4:334: the top level
configure.ac:165: warning: The macro `AC_PROG_LIBTOOL' is obsolete.
configure.ac:165: You should run autoupdate.
build/libtool.m4:99: AC_PROG_LIBTOOL is expanded from...
configure.ac:165: the top level
enable sockets supports? [no] : enable mysqlnd support? [no] : building in /tmp/pear/temp/pear-build-rootvB62Qw/swoole-5.0.3
running: /tmp/pear/temp/swoole/configure --with-php-config=/usr/bin/php-config --enable-sockets=no --enable-openssl=yes --enable-mysqlnd=no --enable-swoole-curl=yes --enable-cares=yes --enable-brotli=yes
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking for a sed that does not truncate output... /usr/bin/sed
checking for pkg-config... /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for cc... cc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no

env:
    SWOOLE_LIBS: libc-ares-dev libcurl4-openssl-dev libpq-dev libbrotli-dev
    SWOOLE_CONFIGURE_OPTS: --enable-openssl --enable-swoole-curl --enable-cares --enable-swoole-pgsql --enable-brotli

--enable-swoole-pgsql is lost。

@huangdijia
swoole does not have support for that parameter in its package.xml, so PECL does not add it. Please ask upstream to add support for it.
https://github.com/swoole/swoole-src/blob/master/package.xml#L2411

https://github.com/swoole/swoole-src/pull/5081 Fixed


Extremely long load time on high number of results

Something really weird is happening... Afaict it's in the JSON decoding, I'm not 100% sure though.

Found a workaround, apparently it's a bug on macOS somewhere in reqwest/hyper.
Workaround in f47d19e (referenced the wrong issue/PR by accident).


Adding field to Login Form

Hi,
I'm in nead to add a field in the login form representing the name of the Company.
So 3 field : Username, Password and Company.
The company has to be linked to a class representing the Company Entity.
If I use the Lookup Editor it seenms not work.
How can I work around it?
Simone

Did you see this


_check_validity_of_instance

We need to think deeply about what the potential malformed inputs to OmicsIntegrator might be. This thread is for that. Beyond making sure every function is passed the right type, what else do we need to check? @divyaramamoorthy

Fixed in #75


'firstIndex == 0 || childScrollOffset(firstChild!)! - scrollOffset <=

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

Hi @chweiGitHub, when filing an issue, it is important to us that you provide the information that is specified in the template as that helps make the issue actionable for us.
Your error alone isn't enough information for us to do anything.
As such I'm going to close the issue.
If you do have an issue, kindly file a new one and fill in the required information.
Thank you


TOC Doesn't work in Markdown Preview

Table of Contents does not function when previewing a Markdown file, as reported by: https://developercommunity.visualstudio.com/content/problem/411901/-toc-doesnt-work-in-azure-devops-preview-page-for.html

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

ID: fa622358-f5bb-1f5c-177a-081a37b6acef
Version Independent ID: 72f7fd16-a822-66c8-3a34-8905645ce5f2
Content: Markdown syntax for files, widgets, and wikis in Azure DevOps - Azure DevOps
Content Source: docs/project/wiki/markdown-guidance.md
Product: devops
GitHub Login: @chcomley
Microsoft Alias: chcomley

Hi @mchelen, thanks for reaching out. I've engaged our PM, so please stay tuned. Thanks!

Hi @mchelen, Good news! Per our PM, we are in the process of fixing this bug! Please let me know whether you have any other questions. Have a fantastic weekend.

@chcomley Great to hear, thanks you too!

Any updates on this?

Hi @lennybacon and @mchelen, this fix should be deployed by the end of this week! Thanks.


Resizing a component under special conditions causes it to move up

Hello,
I have a really wired behavior using Chrome (Version 52.0.2743.82) with Win10 and Win7.
If I try your Demo in a fullscreen browser window and resize the "Event Console" it wants to move one row up.
I just have to grap the resize handle and then this happens:

If I resize the whole browser window, reload the demo and try the same. The "Event console" does not move up and it works as expected.
I tested with Edge and Firefox and they work fine.
Thanks,
Michael

Can reproduce it any more.


Feature/aql subquery all rows fetcher shadow rows

Make AllRows Fetcher aware of ShadowRow API.
This PR will NOT use the API yet, it will just prepare everything to be wired together.

New Jenkins after long weekend:
http://jenkins.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/6606/

Jenkins finally blue \o/


Extract index from the high hash bits instead of the low bits.

Some Hashers, and FxHasher in particular, mix the low bits poorly; so only use the high bits for both hashes.
Instead of introducing bucket_shift next to bucket_mask in the top-level struct, derive them both from capacity_log2. This avoids exacerbating struct size in #69.
This change is also a prerequisite for rayon-based parallel collect, which requires that the high bits of the index are always in the same place regardless of table size.
name                     old ns/iter  new ns/iter  diff ns/iter   diff %  speedup
find_existing            0            0                       0     NaN%    x NaN
find_existing_high_bits  84,320       3,442             -80,878  -95.92%  x 24.50
find_nonexisting         0            0                       0     NaN%    x NaN
get_remove_insert        25           29                      4   16.00%   x 0.86
grow_by_insertion        205          209                     4    1.95%   x 0.98
grow_by_insertion_kb     290          180                  -110  -37.93%   x 1.61
hashmap_as_queue         25           26                      1    4.00%   x 0.96
insert_8_char_string     18,038       17,491               -547   -3.03%   x 1.03
new_drop                 0            0                       0     NaN%    x NaN
new_insert_drop          45           50                      5   11.11%   x 0.90

Unfortunately, the 2 benchmarks which actually matter (find_existing and find_nonexisting) are currently broken. I will prepare a PR to fix then, and then we can evaluate the impact that your change has on performance.
Also, could you explain what you have in mind for parallel collect? I am not sure if it is possible to do better than the implementation we have now (parallel collect into a Vec and then normal collect that into a HashMap).
I personally feel that we would be better off fixing the hash function to provide better output in the lower bits, but that is something to decide with benchmarks.

I implemented a parallel collect for the std HashMap, and it got a nearly linear speedup with more cores: https://github.com/edre/rayon-hash/commit/f94751a3d0ad87ff483b5d8369f000883c991a69
The gist of the algorithm is:

Hash and partition items into shard buckets in parallel.
Insert shard buckets into their segment of the hash table in parallel.
Fix up any entries that needed to overflow into the next segment.

Just this change of using the high bits for the index in the std HashMap made serial collect a couple percent faster. Probably because table resize writes to one spot in the new table instead of two. Adding a bit to the front of the index splits the new address into two possible spots.

Can you retry the benchmarks with the latest master? They should give a better picture of the performance now that #72 is merged.

This change does a lot worse in the new benchmarks. My rust dev setup is in a VM, and there's about 10% noise, but this still looks bad.
 name              old ns/iter  bigstruct ns/iter  diff ns/iter  diff %  speedup
 insert_erase_i32  38,723       39,912                    1,189   3.07%   x 0.97
 insert_erase_i64  36,316       41,743                    5,427  14.94%   x 0.87
 insert_i32        31,225       29,544                   -1,681  -5.38%   x 1.06
 insert_i64        28,472       31,279                    2,807   9.86%   x 0.91
 iter_i32          2,091        2,375                       284  13.58%   x 0.88
 iter_i64          2,096        2,541                       445  21.23%   x 0.82
 lookup_fail_i32   3,813        5,684                     1,871  49.07%   x 0.67
 lookup_fail_i64   3,524        3,324                      -200  -5.68%   x 1.06
 lookup_i32        4,846        4,467                      -379  -7.82%   x 1.08
 lookup_i64        5,420        9,067                     3,647  67.29%   x 0.60

My conclusion from this is that Fx is not a very good general hash function. These benchmarks use keys that are closely packed small integers, and it would be difficult for a hash function to mess up the perfect low-bit entropy handed to it here. I think you should reintroduce some form of the find_existing_high_bits benchmark, which highlights where Fx is problematic.
Using fnv makes the benchmarks more similar, but is a bit slower.

We've switched the hash function to AHash, which solves this issue.

See #97


Revert "Add Cassandra Reaper Spec in Mariner. (#3178)"

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[X] The toolchain has been rebuilt successfully (or no changes were made to it)
[X] The toolchain/worker package manifests are up-to-date
[X] Any updated packages successfully build (or no packages were changed)
[X] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[X] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[X] All package sources are available
[X] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json)
[X] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[X] All source files have up-to-date hashes in the *.signatures.json files
[X] sudo make go-tidy-all and sudo make go-test-coverage pass
[X] Documentation has been updated to match any changes to the build system
[X] Ready to merge

Summary

Reverting #3178. We forgot to upload the tarballs during that PR and the cache-building script is having issues I couldn't quickly resolve, so it needs a few more tweaks anyway.
Change Log

Reverted #3178.

Does this affect the toolchain?

No.
Test Methodology

N/A - reverting breaking change.

I failed to upload reaper src tarball since the Publish Source pipeline was timing out in "install antivirus" phase.
Rest cache tarballs were already uploaded.
All tars are now uploaded for reaper.


increase mem and limit parallelism for pre-merge

Signed-off-by: Peixin Li<EMAIL_ADDRESS>Increase memory request to avoid potential OOM kill for pre-merge pod run during the parallel pytest run
And limit parallelism to make sure if we change GPU type w/ larger GPU mem, it won't be OOM of the pod limit unless we manually change both GPU and adjust host memory

build


MACS2 can't find python files after fresh conda install

I install macs2 using conda and which can find it, but MACS2 throws errors when I try to run it about not being able to find files.
balter@server:~$ conda install macs2
Fetching package metadata ...........
Solving package specifications: ..........

Package plan for installation in environment <server path>/users/balter/miniconda2:

The following NEW packages will be INSTALLED:

    macs2: <IP_ADDRESS>60309-r3.2.2_0 bioconda

Proceed ([y]/n)? y

Linking packages ...
[      COMPLETE      ]|###########################################################################################| 100%
balter@server:~$ which macs2
<server path>/users/balter/miniconda2/bin/macs2
balter@server:~$ macs2 -h
Traceback (most recent call last):
  File "<server path>/users/balter/miniconda2/bin/macs2", line 4, in <module>
    __import__('pkg_resources').run_script('MACS2==<IP_ADDRESS>60309', 'macs2')
  File "<server path>/users/balter/miniconda2/lib/python2.7/site-packages/setuptools-23.0.0-py2.7.egg/pkg_resources/__init__.py", line 719, in run_script
  File "<server path>/users/balter/miniconda2/lib/python2.7/site-packages/setuptools-23.0.0-py2.7.egg/pkg_resources/__init__.py", line 1496, in run_script
pkg_resources.ResolutionError: No script named 'macs2'
balter@server:~$ macs2 --version
Traceback (most recent call last):
  File "<server path>/users/balter/miniconda2/bin/macs2", line 4, in <module>
    __import__('pkg_resources').run_script('MACS2==<IP_ADDRESS>60309', 'macs2')
  File "<server path>/users/balter/miniconda2/lib/python2.7/site-packages/setuptools-23.0.0-py2.7.egg/pkg_resources/__init__.py", line 719, in run_script
  File "<server path>/users/balter/miniconda2/lib/python2.7/site-packages/setuptools-23.0.0-py2.7.egg/pkg_resources/__init__.py", line 1496, in run_script
pkg_resources.ResolutionError: No script named 'macs2'

I'm wondering if it has something to do with my python path settings. Here they are:
balter@server:~$ grep -i python* .bashrc
# Python environment variables
export PYTHONLIB="<server path>/users/balter/miniconda2/lib"
export PYTHONPATH="<server path>/users/balter/miniconda2/lib/python2.7/site-packages"
export PYTHONUSERBASE="<server path>/users/balter/miniconda2"

balter@server:~$ grep -i path .bashrc
# Custom PATH variable
export PATH="/bin:/usr/local/bin:/usr/bin:/opt/condor/bin:<server path>/CompBio/bin"
export PATH="<server path>/users/balter/miniconda2/bin:$PATH"

Cross-posted here:
https://www.biostars.org/p/216312/
and here:
http://stackoverflow.com/questions/40008344/macs2-python-files-not-found-after-clean-install-with-conda

Hi @abalter, could you find out where the 'conda' installed MACS2? Especially, where is the 'egg' for MACS2? According to my knowledge of 'easyinstall', it will package actual python scripts (macs2) and the library files (such as .pyc or .o files)  in 'egg' then put it somewhere (such as /users/balter/miniconda2/lib/python2.7/site-packages/). Then it will create a virtual script (such as /users/balter/miniconda2/bin/macs2) and instruct the system where the egg is. To find the egg, it will look for a file named 'easy-install.pth' and in that file there must be a line like 'somepath/MACS2-<IP_ADDRESS>60309-py2.7-xxxxxxxxx.egg' (the egg is in fact a directory). Python will try to insert 'somepath/MACS2-<IP_ADDRESS>60309-py2.7-xxxxxxxxx.egg' to sys.path so that it will locate the actual 'macs2' script and its library files. The actual 'macs2' script should be in 'xxx.egg/EGG-INFO/scripts/macs2', and library files should be in 'xxx.egg/MACS2/'. Yes. It's so complicated and I really hate that :) The conclusion is... if you installed python package in this way -- through easyinstall (the setup tools), your hack on PYTHONPATH will not be that useful anymore! So, try not to use PYTHONPATH if your python software is installed through this 'conda' script. Instead, try to figure out where the .pth and .egg are and to check if conda successfully write a line about 'MACS2' in the .pth file...

@abalter Disclaimer: I have never used miniconda so the actual .pth filename might be different. But the mechanism of this setuptools should be the same...

I didn't use conda but still got the same error.

This is what I got:
Traceback (most recent call last):   File "/usr/local/bin/macs2", line 4, in <module>     __import__('pkg_resources').run_script('MACS2==2.1.2', 'macs2')   File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 658, in run_script     self.require(requires)[0].run_script(script_name, ns)   File "/usr/lib/python2.7/dist-packages/pkg_resources/__init__.py", line 1429, in run_script     .format(**locals()), pkg_resources.ResolutionError: Script 'scripts/macs2' not found in metadata at '/home/ubuntu/tools/MACS2-2.1.2/MACS2-2.1.2.egg-info'

I would recommend installing with conda. Seems to work perfectly now.
(base) balter@spectre3:~/deseq-learn/taxa_filtered_data$ conda create -n macs macs2
(base) balter@spectre3:~/deseq-learn/taxa_filtered_data$ conda activate macs
(macs) balter@spectre3:~/deseq-learn/taxa_filtered_data$ which macs2
/home/balter/conda/envs/macs/bin/macs2
(macs) balter@spectre3:~/deseq-learn/taxa_filtered_data$ macs2 -h
usage: macs2 [-h] [--version]
             {callpeak,bdgpeakcall,bdgbroadcall,bdgcmp,bdgopt,cmbreps,bdgdiff,filterdup,predictd,pileup,randsample,refinepeak}
             ...

macs2 -- Model-based Analysis for ChIP-Sequencing

positional arguments:
  {callpeak,bdgpeakcall,bdgbroadcall,bdgcmp,bdgopt,cmbreps,bdgdiff,filterdup,predictd,pileup,randsample,refinepeak}
    callpeak            Main MACS2 Function: Call peaks from alignment
                        results.
    bdgpeakcall         Call peaks from bedGraph output. Note: All regions on
                        the same chromosome in the bedGraph file should be
                        continuous so only bedGraph files from MACS2 are
                        accpetable.
    bdgbroadcall        Call broad peaks from bedGraph output. Note: All
                        regions on the same chromosome in the bedGraph file
                        should be continuous so only bedGraph files from MACS2
                        are accpetable.
    bdgcmp              Deduct noise by comparing two signal tracks in
                        bedGraph. Note: All regions on the same chromosome in
                        the bedGraph file should be continuous so only
                        bedGraph files from MACS2 are accpetable.
    bdgopt              Operations on score column of bedGraph file. Note: All
                        regions on the same chromosome in the bedGraph file
                        should be continuous so only bedGraph files from MACS2
                        are accpetable.
    cmbreps             Combine BEDGraphs of scores from replicates. Note: All
                        regions on the same chromosome in the bedGraph file
                        should be continuous so only bedGraph files from MACS2
                        are accpetable.
    bdgdiff             Differential peak detection based on paired four
                        bedgraph files. Note: All regions on the same
                        chromosome in the bedGraph file should be continuous
                        so only bedGraph files from MACS2 are accpetable.
    filterdup           Remove duplicate reads at the same position, then save
                        the rest alignments to BED or BEDPE file. If you use
                        '--keep-dup all option', this script can be utilized
                        to convert any acceptable format into BED or BEDPE
                        format.
    predictd            Predict d or fragment size from alignment results.
                        *Will NOT filter duplicates*
    pileup              Pileup aligned reads with a given extension size
                        (fragment size or d in MACS language). Note there will
                        be no step for duplicate reads filtering or sequencing
                        depth scaling, so you may need to do certain pre/post-
                        processing.
    randsample          Randomly sample number/percentage of total reads.
    refinepeak          (Experimental) Take raw reads alignment, refine peak
                        summits and give scores measuring balance of
                        waston/crick tags. Inspired by SPP.

optional arguments:
  -h, --help            show this help message and exit
  --version             show program's version number and exit

For command line options of each command, type: macs2 COMMAND -h


Don't set onChange validation to "valid" for field with async rule with no sync rules

What
I suggest we don't mark field as valid if it has asyncRule, but doesn't have any sync rules applied.
Why
When you edit async field, it gets marked as valid, and then validation happens on blur. This is confusing in case there is no sync validation to indicate as valid.
How

Check if field has any sync rules during the validation.
If it's async, and has no sync rules, don't mark it as valid unless async validation happens.

The root of the issue lies in the fieldUtils.shouldValidate() method.
Basically, it resolves whenever the validation is needed, regardless of which validation type was requested to be checked.
onChange > SyncValidation > shouldValidate? Sure, has "asyncRule" > Requests new validity state

Changelog

fieldUtils.shouldValidate() was removed
Its implementation was modified and added as ValidationType.shouldValidate()

TODO

[ ] Write test(s)
[ ] asyncRule Promise resolving with timeout throws the following error:

Uncaught (in promise) TypeError: resolved.getIn is not a function
    at Sequence.sequenceIterator [as iterator] (http://localhost:6006/static/preview.bundle.js:11701:27)
    at Sequence._callee$ (http://localhost:6006/static/preview.bundle.js:58391:28)
    at tryCatch (http://localhost:6006/static/preview.bundle.js:27681:40)
    at Generator.invoke [as _invoke] (http://localhost:6006/static/preview.bundle.js:27915:22)
    at Generator.prototype.(anonymous function) [as next] (http://localhost:6006/static/preview.bundle.js:27733:21)
    at step (http://localhost:6006/static/preview.bundle.js:58294:191)
    at http://localhost:6006/static/preview.bundle.js:58294:361
    at <anonymous>

The issue is covered and published under 0.18.3. Closing.


Split cilium log out from test-output.log

In local testing, I found that spinning up a simple test that runs Cilium, then Expect(true).To(BeFalse()), the test-output.log contained 4500 lines of cilium logs, and 200 lines of actual test output.
It would be great to split the cilium logs out and put them in a separate file so that the test-output.log can reflect exactly what's going on in the test. This can be cross-referenced against the logs separately.

I'm open to this, but for me this is a feature. That cilium logs are the logs that happens during the test, and have all in the same file help me a lot to do a simple search instead of looking for a bunch of files.
I think @ianvernon want to have this one in another file too, so as you prefer.

I think one of the main problems I have with this approach is that I can't easily find the actual end of the test output. IMO this would be the best approach to solve this problem.
That said, if we covers the various use cases better, then why not both? We can have one file with test-output, one with the logs, and one with both.

This proposal above improves the ability to find where the error occurs, so that could work for the case where I just want to find the end of the test output. That would be a positive step forward.
More generally though, I think that we often hit this general question of the library vs. the catalog. Some people just find it easier to browse through a list of files (like books in a library), picking which information they think looks relevant or interesting based on the filename, while others find it easier to remember the names of things they are looking for and search within a single file (like a catalog). I've had similar discussions with others regarding the cilium debuginfo output that's used in bugtool, which also aggregates a bunch of information in a single file.
Personally I often don't know the names of what I'm looking for, so the ability to "browse" the files is easier for me. I like the ability to see an overview of the information available, pick file by name and glance through, decide whether the contents are relevant, then move on to the next file. If I don't think that some files are relevant, then I will often just skip them. Also, when I look in a file called test-output.log, I expect it to be the output of the test, not some other state from Cilium or somewhere else.
It seems to me like it's easier to have everything split out in separate files, then run cat * >> all_output.txt or for f in *; do echo "### $f" >> all_output.txt; cat "$f" >> all_output.txt; done to look at all the output at once, rather than to start with a giant file and try to split it out into separate files for the browsing use case.


[WIP] Added SegmentManager interface.

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/518/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3109/

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/519/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3110/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3139/

Build Failed with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/547/

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/37/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/34/


🚀 Feature: support for node < 18

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
Hi, my project uses<EMAIL_ADDRESS>and this script fails with
import { parseArgs } from "node:util";
         ^^^^^^^^^
SyntaxError: The requested module 'node:util' does not provide an export named 'parseArgs'

Would be great if this package would support lower node version
Additional Info
No response

👍 agreed, Node 14 is still in maintenance for a few more months and this package should support it. Thanks for filing - accepting PRs!
Thinking for the future: it'd be nice to keep using something that looks like parseArgs, because after Node 16 leaves maintenance support later this year we'll be able to switch back to using parseArgs. Perhaps there's a polyfill for parseArgs that works well? https://github.com/pkgjs/parseargs?
If that doesn't work for Node 14 (or in general) I suppose we can just use something else. 😄

@JoshuaKGoldberg opened a PR, tested it in my node 14.20 repo, all works fine

thank you for so fast response!


exhaust request body

Hi, we're using nsq_to_http as a source for dark traffic testing--it's great!
Would you guys be open to consuming the request body? It adds the overhead of the round trips incurred by body bytes, but otherwise causes all types of transport errors on the other side should the endpoint expect to return a body.
This doesn't change functionality at all (I think.)

Might consider using ioutil.Discard. Not sure opening /dev/null would work on some OS's plus its a little cleaner and guaranteed not to return an error.

@wtolson good call. I was searching around in ioutil, but didn't see iotul.Discard.
Patched

Makes sense, thanks!


Migrating Changes

Rebase error occurred with original branch. Attempting to migrate changes to new branch.

Originally approved PR: https://github.com/triton-inference-server/common/pull/66


ami_variables: update aws variables for releng account

follow the change of ami creation in releng account the ami valiables should be updated accordingly.
Should be merged together with: https://github.com/scylladb/scylla-pkg/pull/3392

rebased with master and removed the subnet_id variable
image-build-subnet* subnet:tag was updated also in our account
Verification: https://jenkins.scylladb.com/view/master/job/scylla-master/job/releng-testing/job/ami/129/


Merge alternate MSVC build branch into our fork of trust wallet-core

Use git to pull in and merge this repo's branch into our wallet-core fork.

Create develop branch on our wallet-core fork from master latest

setup this branch as another git origin, i.e.
    git remote add upstream-msvc https://github.com/kaetemi/wallet-core-windows.git

Create develop branch
    git pull origin master

Pull windows-msvc branch into our branch
    git pull upstream-msvc pr/windows-msvc

resolve conflicts with develop branch

get it to compile on windows.

Document what has to be done on Windows, i.e. install rust, execute others.

make sure dependencies like Boost and protoc use our thirdparty/grpc/thirdparty/proto for instance.  The find_package for Windows of grpc/proto, should be used for all platforms, instead of downloading inside the wallet-core source directory.

To make this work, checkout thirdparty branch temp-cross-build and do
git submodule update --init --recursive to pull the 'cross-build' branch of wallet-core as well.


Release mentioned was not found - 1.25.0

File: compose/install.md
docker compose - current stable build referred in the document 1.25.0 was not found.
only 1.24.1 (stable) and 1.25.0-rc1 was available.
Thanks.

Thanks @usha-mandya, @ollypom .
Are we changing the install.md documentation as well?

The instructions below outline installation of the current stable release (v1.25.0) of Compose

Hey @jskumar, I updated the variables file (_config.yml). All pages pull from that variable. So once the cache clears all page (including install) will include the 1.24.1 variable.

Thanks @ollypom. It works :)


Feat: new UI add page params

Questions
Answers

Description?
Add hardcoded required parameters for new UI on controller and updating twig according new parameters.

Type?
new feature

BC breaks?
no

Deprecations?
no

Fixed ticket?
N/A

Sponsor company
@PrestaShopCorp

How to test?
For dev : you can change parameters on controller and reload according page to see changes. For test : you can just check if all new ui pages work fine.

PR merged, well done!
Message to @PrestaShop/committers: do not forget to milestone it before the merge.


nested helm chart using checksum/configmap for Roll Deplopyments throws error

Output of helm version:
Client: &version.Version{SemVer:"v2.15.1", GitCommit:"cf1de4f8ba70eded310918a8af3a96bfe8e7683b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.15.1", GitCommit:"cf1de4f8ba70eded310918a8af3a96bfe8e7683b", GitTreeState:"clean"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.0", GitCommit:"2bd9643cee5b3b3a5ecbd3af49d09018f0773c77", GitTreeState:"clean", BuildDate:"2019-09-18T14:36:53Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.0", GitCommit:"2bd9643cee5b3b3a5ecbd3af49d09018f0773c77", GitTreeState:"clean", BuildDate:"2019-09-18T14:27:17Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
Kubeadm based k8s installation on BM Server
What Happened:
chart/
-values.yaml
-templates/
--configMap.yaml
-charts/
--chart2/
--values.yaml
--templates/
---deployment1.yaml
--chart3/
--values.yaml
--templates/
---deployment2.yaml

Any Change in the configMap in root chart. I would like the Deployments in the subchart (deployment1 & deployment2) to have a Rolling Update.
Deployment1.yaml:
kind: Deployment
spec:
  template:
    metadata:
      annotations:
        checksum/config: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}
[......]
        envFrom:
          - configMapRef:
              name: e2eo

Command Run:
helm install chart
Error Message:
Error: render error in "e2eo/charts/chart1/templates/deployment1.yaml": template: charts/chart1/templates/deployment1.yaml:15:28: executing "charts/chart1/templates/deployment1.yaml" at <include (print $.Template.BasePath "/configmap.yaml") .>: error calling include: template: no template "charts/chart1/templates/configmap.yaml" associated with template "gotpl"

Hi, @ganchandrasekaran - apologies for the delayed reply. I found this issue with a search of issues referring to Helm v2. Can you let us know if you can reproduce this issue with Helm 3? We are no longer providing support for Helm 2 as of today: https://helm.sh/blog/helm-2-becomes-unsupported/

@bridgetkromhout I just tried to reproduce it on helm "v3.3.1". I dont see the same error as above.
Thank you for your response.


ACPI tracking issue

I've started work on ACPI.

[x] RSDP V1/V2
[x] RSDT/XSDT
[x] MADT
[x] HPET
[ ] FACP
[ ] AML
[ ] (Other tables... add pls)

How much of this should we merge into master before 0.2.0?

Re:

How much of this should we merge into master before 0.2.0?

Perhaps not AML, but what more tables?

What would AML be required for? If not too much, then yeah, we should probably leave it for later.

The main reason I pose this question is because ACPI is big. The spec is a thousand words long. AML is the bytecode of AML, and tbh I don't see myself completing it in the same timeframe as of PS/2, interrupts, and memory management.

In that case, we should definitely merge in before it's fully complete. It's not worth spending long waiting for ACPI and letting that hold everything else up.

Yeah. It's in another (local) branch currently.

Maybe let's merge when HPET is done?

Cool. @Gegy1000 did hpet, but now I'm thinking we should do FACP before we merge, so we can tell if PS/2 controller is there for the ps2 branch. I can do that sometime, or maybe gegy can, I don't particularly mind.

Hi, we're also currently working on an ACPI implementation at rust-osdev/acpi. Maybe we can combine forces?
cc @rust-osdev/acpi

@phil-opp Sounds great! :)

Yeah! What can we do to help? Perhaps we can add the HPET code to the crate. What needs to be done, otherwise? I was looking at AML the other day, so I could try help with that perhaps. I've started a bit of work on a local branch.

Cool! I sent you two an invitation to the organization.

What can we do to help?

The two people currently working on it are @IsaacWoods and @Sh4d1, so they should be able to answer it best.

Update: I am working on integrating the crate into flower now in the acpi-crate branch.


Worldpay v2 - capture flag

The capture status flag was the wrong way around.

@mattbrailsford bug fix for the capture flag