ETCD is not restarted on certificate rotation from Rancher UI

bug
Steps to reproduce (least amount of steps as possible):
Deploy cluster via rancher. Rotate certificates through Rancher UI before they expires.
Result:
etcd fails with errors on expired certificate.
Other details that may be helpful:
Restarting etcd on each master fixes the problem.
Environment information

Rancher version 2.2.3
Installation option: single install

Cluster information

Cluster type: created from within rancher 2.0.x, using rancher os.
Machine type: ec2
Kubernetes version: v1.11.3
Docker version: 17.3.2

I attempted reproduction:

v2.0.15 single rancher install
Deploy ec2 cluster via Rancher UI with current RancherOS image, k8s v1.11.9
Cluster has 3 etcd+cp nodes and 2 workers
Upgrade to v2.2.3
Rotate certificates (before they expire) (all services)
Certificates  successfully deploy and cluster is operational

Note: Just because I was not able to reproduce, does NOT invalidate what was reported. There  could be many things that differ between our environments. For example, I do not know the exact v2.0.x version the upgrade was from. I had to guess and my k8s version (RKE) is slightly different. There could be other things as well that make our environments greatly differ.

Cluster was provisioned on 1st of July 2018 from Rancher 2.0.4.
Cluster has been upgraded once from within Rancher
Rancher was upgraded from 2.0.4 -> 2.1.1 -> 2.1.4 -> 2.2.3
Certificates where rotated in 2.2.3 3 days before expire.
ETCD died on expiry. docker ps showed etcd containers had been running for 4 weeks (since 2.2.3 upgrade) while other containers where running since certificate rotation. See attached screenshot from this failure.

I'm having the same problem.
Since I rotated the certificates, my nodes are unhealthy.

I'm having the same problem,when i reference the article https://rancher.com/blog/2019/kubernetes-certificate-expiry-and-rotation-in-rancher-kubernetes-clusters/, upgrade to v2.1.10 version and certificate rotation support via the API, like other compontents have been restarted ,except 3 etcd compontents ,may be it need a manual restart


Javadoc verifier does not support records

Version
23.1 - snapshot
Problem/Opportunity
The JavadocVerificationVisitor does not support records and compact constructors. This is a side-effect of Gradle using JavaParser 3.17.0 while the latest version is 3.24.X.
For now we cannot update manually due to the way Gradle serves this dependency. When Gradle updates, this issue should be resolved
Proposed solution
No response

No longer relevant after migration to Maven


Make month and day components of date validation type optional

Currently the date validation type requires that candidate values have year, month and day components (e.g. "2018-02-03"), but the ECMAScript specification for Date Time String Format indicates that both the month and day components are optional (e.g. "2018-02", "2018"). Update the date validation type to allow these components to be omitted, with assumed values of 01 in either case. Care must be taken to ensure that "2018-01-01", "2018-01" and "2018" are considered equal to each other with regards to the minimumValue, minimumValueExclusive, maximumValue and maximumValueExclusive constraints.

Addressed by #218


Adding Sidecar Injector

Desired Outcome
Add Sidecar Injector 0.2.0
Implemented Changes
Connected Issue/Story
Resolves #[relevant GitHub issue(s), e.g. 76]
CyberArk internal issue link: insert issue ID
Definition of Done
At least 1 todo must be completed in the sections below for the PR to be
merged.
Changelog

[ ] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a
CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code
changes, or
[x] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: insert issue ID
[x] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[x] No behavior was changed with this PR

Security

[ ] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[x] There are no security aspects to these changes

Should we mark this as an alpha release, given that 0.2.0 hasn't actually been released yet?


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.9 to before v5.16.10",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ac3a68d56651c3dad2c12c7afce065fe15267f44",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=f1842a8cb71de4d7eb75a86f76e88c7ee739218c"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ac3a68d56651c3dad2c12c7afce065fe15267f44",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "f1842a8cb71de4d7eb75a86f76e88c7ee739218c",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "net: dsa: ar9331: register the mdiobus under devres\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.16.10 by commit f1842a8cb71de4d7eb75a86f76e88c7ee739218c, it was introduced in version v5.9 by commit ac3a68d56651c3dad2c12c7afce065fe15267f44. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000508


API & databases seem to lost a lot of OSVs

I'm hoping you're already aware, but it seems like osv.dev is having some data issues:
osv-detector-t --cache-all-databases
  Loading OSV databases for the following ecosystems:
    npm (0 vulnerabilities, including withdrawn - last updated Thu, 09 Jun 2022 01:53:22 GMT)
    crates.io (394 vulnerabilities, including withdrawn - last updated Thu, 09 Jun 2022 01:59:03 GMT)
    RubyGems (0 vulnerabilities, including withdrawn - last updated Thu, 09 Jun 2022 01:53:21 GMT)
    Packagist (1 vulnerability, including withdrawn - last updated Thu, 09 Jun 2022 01:59:04 GMT)
    Go (156 vulnerabilities, including withdrawn - last updated Thu, 09 Jun 2022 01:59:04 GMT)
    Maven (3 vulnerabilities, including withdrawn - last updated Thu, 09 Jun 2022 01:59:05 GMT)
    PyPI (2070 vulnerabilities, including withdrawn - last updated Thu, 09 Jun 2022 01:54:28 GMT)

❯ osv-detector-t --use-api .
Gemfile.lock: found 230 packages
  Loading OSV databases for the following ecosystems:
    RubyGems (0 vulnerabilities, including withdrawn - last updated Thu, 09 Jun 2022 01:53:21 GMT)

  no known vulnerabilities found

yarn.lock: found 1164 packages
  Loading OSV databases for the following ecosystems:
    npm (0 vulnerabilities, including withdrawn - last updated Thu, 09 Jun 2022 01:53:22 GMT)

  no known vulnerabilities found

While I'm demonstrating with osv-detector, the vulnerability library on osv.dev and api are affected too, though the counts on the site are unaffected (I'm assuming they're cached or precomputed, which is good for showing something is wrong):

I was using it fine this morning.

Yep, this is an issue with the GitHub advisory repo. The automation cron seems to repeatedly delete entries at https://github.com/github/advisory-database/tree/main/advisories/github-reviewed/2022.
I've already raise this issue with them.

In the meantime I've forked the repo and reset it to its last known good state and pointed OSV to that. These should start coming back shortly.

The issues seem to be resolved. I've re linked OSV with the actual GitHub repo.


Bael 7622: Bulk and Batch APi operations

Bulk and Batch APi operations in spring with custom implementation

retest this please


(LOW) {{ View modifier }}

Simple expression to convert {{ construction }} into  form

Include into separate module or use Toolkit.

LOW

better approach found

Connect blade as process and do not worry


Create project assignment workflow

Notice the project assignment. The workflow did that... not me!

How it looks when not logged in:


Example Request: reading parquet into vectors

Hello! I was hoping to read two columns of parquet into Vec<usize>'s using the package directly. I was able to hack something together, but it would be really great to get some help to do it really well. I have been reading through the docs and examples, but I am pretty stuck.
With a little direction on the steps of how to do it, I would put together an example if you think that is useful! It would have been to me :).
This my use-case:
I work at a streaming tv / movie company and we are doing recommenders using sparse matrices. In python or Spark it requires tons of memory, but in rust I could easily handle 125m rows on my Macbook air. Completely incredible!

Hi!
Thank you for considering arrow2 :)
Have you tried this? https://github.com/jorgecarleitao/arrow2/blob/main/examples/parquet_read.rs
The idea is:

// use the example above to get a chunk (or multiple, if you want to process in chunks)
let chunk = ...;

// deconstruct it into arrays
let arrays = chunk.into_arrays();

// get the individual arrays (assuming i32, could be i64 if the parquet columns are i64)
let c0 = arrays[0]
        .as_any()
        .downcast_ref::<PrimitiveArray<i32>>()
        .unwrap();

let c1 = arrays[1]
        .as_any()
        .downcast_ref::<PrimitiveArray<i32>>()
        .unwrap();

// convert them to `Vec<usize>`
// note that the below disregards the arrays' validities.
let c0: Vec<usize> = c0.values().into_iter().map(|x| x as usize).collect();
let c1: Vec<usize> = c1.values().into_iter().map(|x| x as usize).collect();


Fixture is null while it should not be

In the following example, item_group_id shouldn't be null:
An exception occurred while executing 'INSERT INTO offer (currency, status, amount, booking_id, item_group_id) VALUES (?, ?, ?, ?, ?)' with params ["PGK", "PENDING", 102, 28, null]:

SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'item_group_id' cannot be null
 /Users/rcarree/dev/genius/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractMySQLDriver.php:112
 /Users/rcarree/dev/genius/vendor/doctrine/dbal/lib/Doctrine/DBAL/DBALException.php:128
 /Users/rcarree/dev/genius/vendor/doctrine/dbal/lib/Doctrine/DBAL/Statement.php:177
 /Users/rcarree/dev/genius/vendor/doctrine/orm/lib/Doctrine/ORM/Persisters/Entity/BasicEntityPersister.php:281
 /Users/rcarree/dev/genius/vendor/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php:1014
 /Users/rcarree/dev/genius/vendor/doctrine/orm/lib/Doctrine/ORM/UnitOfWork.php:378
 /Users/rcarree/dev/genius/vendor/doctrine/orm/lib/Doctrine/ORM/EntityManager.php:356
 /Users/rcarree/dev/genius/var/cache/test/jms_diextra/doctrine/EntityManager_5899bf64bc0d6.php:313
 /Users/rcarree/dev/genius/vendor/nelmio/alice/src/Nelmio/Alice/Persister/Doctrine.php:61
 /Users/rcarree/dev/genius/vendor/hautelook/alice-bundle/src/Alice/DataFixtures/Loader.php:236
 /Users/rcarree/dev/genius/vendor/hautelook/alice-bundle/src/Alice/DataFixtures/Loader.php:197
 /Users/rcarree/dev/genius/vendor/hautelook/alice-bundle/src/Alice/DataFixtures/Loader.php:90
 /Users/rcarree/dev/genius/vendor/liip/functional-test-bundle/Test/WebTestCase.php:572
 /Users/rcarree/dev/genius/tests/Functional/Transaction/TransactionManagerTest.php:34

Because if I remove the commented lines it is well populated:
    public function setUp() {
        $this->loadFixtureFiles([
            '../../Resources/fixtures/orm/hotel_item_group.yml',
            '../../Resources/fixtures/orm/booking.yml',
            '../../Resources/fixtures/orm/booking_provider.yml',
            '../../Resources/fixtures/orm/credit_card.yml',
            '../../Resources/fixtures/orm/offer.yml', // WORKS WITHOUT THIS
            '../../Resources/fixtures/orm/offer_payment.yml', // WORKS WITHOUT THIS
            '../../Resources/fixtures/orm/processor.yml',
        ]);
    }

How this is possible that item_group_id stops to be populated?
Would you be able to see what happens?
The yml/entities files are located here: https://dl.dropboxusercontent.com/u/57999777/item_group_id-is-null/item_group_id-is-null.zip

Just in case, to make it easier, I created a new Symfony project with the related files:
https://github.com/Kwadz/alice-test

I am still investigating. May be the problem would come from doctrine/data-fixtures.

In fact a @ORM\JoinColumn name was wrong. The issue came from the Offer entity configuration.


Update configure-wave-processing.md

These changes are valid for 10.0.17, which should be GA within a couple of weeks.

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.

.


Implement markdown mutating checkboxes through CommonMarkViewer::show_mut

Closes #38
This pull request neglects to implement the feature for the comrak backend. I haven't tested what happens when trying to use show_mut with the comrak backend.
Since the comrak backend's future is unclear, this is acceptable.

Can you add an example that demonstrates the feature? Just copy how hello_world.rs does it.

I'll try cooking something up

Let me know if you need any other change.

Thanks. This is great!


Test failure: HttpsClientCert_GetCertInformation

 Failed   Microsoft.AspNetCore.Server.IIS.FunctionalTests.ClientCertificateTests.HttpsClientCert_GetCertInformation(variant: Server: IISExpress, TFM: netcoreapp2.2, Type: Portable, Arch: x64, ANCM: V2, Host: InProcess)
  Error Message:
   Assert.Equal() Failure
             (pos 0)
  Expected: Enabled;BAF8529DF729239A8D4E871846FD4AA08···
  Actual:   <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML ···
             (pos 0)
  Stack Trace:
     at Microsoft.AspNetCore.Server.IIS.FunctionalTests.ClientCertificateTests.ClientCertTest(TestVariant variant, Boolean sendClientCert) in /_/test/Common.FunctionalTests/ClientCertificateTests.cs:line 77
  --- End of stack trace from previous location where exception was thrown ---
  Standard Output Messages:
   | [0.001s] TestLifetime Information: Starting test HttpsClientCert_GetCertInformation-Server: IISExpress, TFM: netcoreapp2.2, Type: Portable, Arch: x64, ANCM: V2, Host: InProcess at 2018-09-20T19:04:44
   | [0.002s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Deploying [Variation] :: ServerType=IISExpress, Runtime=CoreClr, Arch=x64, BaseUrlHint=https://localhost:44300/, Publish=False
   | [0.002s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Executing: dotnet C:\projects\iisintegration\test\WebSites\InProcessWebSite\bin\x64\Release\netcoreapp2.2\InProcessWebSite.dll
   | [0.031s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: ContentRoot: C:\projects\iisintegration\test\WebSites\InProcessWebSite
   | [0.032s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Attempting to start IIS Express on port: 44300
   | [0.034s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Debug: Saving Config to C:\Users\appveyor\AppData\Local\Temp\1\tmp831.tmp
   | [0.034s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Executing command : C:\Program Files\IIS Express\iisexpress.exe /site:HttpTestSite /config:C:\Users\appveyor\AppData\Local\Temp\1\tmp831.tmp /trace:error /systray:false
   | [0.035s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Removing environment variable ASPNETCORE_ENVIRONMENT
   | [0.035s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: SET ASPNETCORE_DETAILEDERRORS=true
   | [0.035s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: SET ASPNETCORE_MODULE_DEBUG=console
   | [0.035s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: SET LAUNCHER_PATH=dotnet
   | [0.035s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: SET LAUNCHER_ARGS=C:\projects\iisintegration\test\WebSites\InProcessWebSite\bin\x64\Release\netcoreapp2.2\InProcessWebSite.dll
   | [0.035s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: SET ASPNETCORE_CONTENTROOT=C:\projects\iisintegration\test\WebSites\InProcessWebSite
   | [0.036s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress Process 6052 started
   | [0.104s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: Starting IIS Express ...
   | [0.118s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: Initializing the W3 Server Started CTC = 919656
   | [0.122s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: W3 Server initializing WinSock.  CTC = 919671
   | [0.122s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: W3 Server WinSock initialized.  CTC = 919671
   | [0.122s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: W3 Server ThreadPool initialized (ipm has signalled).  CTC = 919671
   | [0.140s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: [aspnetcorev2.dll] Initializing logs for 'C:\projects\iisintegration\test\IISExpress.FunctionalTests\bin\Release\netcoreapp2.2\x64\aspnetcorev2.dll'. Process Id: 6052.. File Version: 12.2.18263.0. Description: IIS ASP.NET Core Module V2. Commit: 2f4ebec7b5c08a307c934d3a53abf2efc78a2d1b.
   | [0.140s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: Start listenerChannel http:0
   | [0.142s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: Successfully registered URL "https://localhost:44300/" for site "HttpTestSite" application "/"
   | [0.142s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: Registration completed for site "HttpTestSite"
   | [0.143s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: AppPool 'IISExpressAppPool' initialized
   | [0.143s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: InitComplete event signalled
   | [0.144s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: IIS Express is running.
   | [0.144s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Started iisexpress successfully. Process Id : 6052, Port: 44300
   | [0.146s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Application ready at URL: https://localhost:44300/
   | [1.717s] HttpTestSite Debug: Method: GET, RequestUri: 'https://localhost:44300/GetClientCert', Version: 2.0, Content: <null>, Headers:
   |                              {
   |                              }
   | [1.729s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: IncrementMessages called
   | [1.837s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress stdout: Request ended: https://localhost:44300/GetClientCert with HTTP status 403.16
   | [1.850s] HttpTestSite Warning: StatusCode: 403, ReasonPhrase: 'Forbidden', Version: 1.1, Content: System.Net.Http.HttpConnection+HttpConnectionResponseContent, Headers:
   |                                {
   |                                  Cache-Control: private
   |                                  Server: Microsoft-IIS/10.0
   |                                  X-SourceFiles: =?UTF-8?B?QzpccHJvamVjdHNcaWlzaW50ZWdyYXRpb25cdGVzdFxXZWJTaXRlc1xJblByb2Nlc3NXZWJTaXRlXEdldENsaWVudENlcnQ=?=
   |                                  X-Powered-By: ASP.NET
   |                                  Date: Thu, 20 Sep 2018 19:04:46 GMT
   |                                  Content-Type: text/html; charset=utf-8
   |                                  Content-Length: 5218
   |                                }
   | [1.850s] HttpTestSite Warning: <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

  <title>IIS 10.0 Detailed Error - 403.16 - Forbidden</title>

HTTP Error 403.16 - Forbidden

Your client certificate is either not trusted or is invalid.

   <fieldset>
Most likely causes:

The client certificate used for this request is not trusted by the Web server.

   </fieldset>

   <fieldset>
Things you can try:

The client may have an old certificate selected for client authentication to this Web site. Close all open client windows, open a new browser window, and then select a valid certificate for client authentication.

Verify that the client certificate is trusted by the Web server.

Verify that the root certificate is properly installed and trusted on the Web server.

Check the failed request tracing logs for additional information about this error. For more information, click here.

   </fieldset>

   <fieldset>
Detailed Error Information:

Module	   IIS Web Core

Notification	   BeginRequest

Handler	   aspNetCore

Error Code	   0x800b0109

Requested URL	   https://localhost:44300/GetClientCert

Physical Path	   C:\projects\iisintegration\test\WebSites\InProcessWebSite\GetClientCert

Logon Method	   Not yet determined

Logon User	   Not yet determined

Request Tracing Directory	   C:\Users\appveyor\Documents\IISExpress\TraceLogFiles\HTTPTESTSITE

   </fieldset>

   <fieldset>
More Information:

    A Secure Sockets Layer (SSL) client certificate identifies you as a valid user of the resource. This error can occur if you choose a client certificate created by a Certificate Authority (CA) that is not trusted by the Web server.

View more information »

   </fieldset>

   | [1.853s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Attempting to cancel process 6052
   | [1.915s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: iisexpress Process 6052 shut down
   | [1.915s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Host process shutting down.
   | [1.917s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: Successfully terminated host process with process Id '6052'
   | [1.917s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Debug: Deleting applicationHost.config file from C:\Users\appveyor\AppData\Local\Temp\1\tmp831.tmp
   | [1.918s] Microsoft.AspNetCore.Server.IntegrationTesting.IIS.IISExpressDeployer Information: [Time]: Total time taken for this test variation '1.9160615' seconds

cc @jkotalik

WindowsAuthTest failed with about the same error on master.
This comment was made automatically. If there is a problem contact ryanbrandenburg.

HttpsNoClientCert_NoClientCert failed with about the same error on master.
This comment was made automatically. If there is a problem contact ryanbrandenburg.

All other tests failures here are irrelevant to the original bug. Seems to be a net461 + Kestrel issue with packages.

There were 16 failures with about the same error on master:

HttpsHelloWorld
HttpsNoClientCert_NoClientCert
HttpsClientCert_GetCertInformation
HttpsHelloWorld
WindowsAuthTest
HttpsNoClientCert_NoClientCert
HelloWorld
HelloWorld
HttpsClientCert_GetCertInformation
WindowsAuthTest
(And 6 more test failures in this build)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

There were 16 failures with about the same error on master:

HelloWorld
WindowsAuthTest
HttpsNoClientCert_NoClientCert
HttpsClientCert_GetCertInformation
HttpsHelloWorld
HttpsClientCert_GetCertInformation
HelloWorld
HttpsNoClientCert_NoClientCert
HttpsHelloWorld
WindowsAuthTest
(And 6 more test failures in this build)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

There were 16 failures with about the same error on master:

HelloWorld
HttpsClientCert_GetCertInformation
HelloWorld
HttpsClientCert_GetCertInformation
HttpsHelloWorld
HttpsHelloWorld
HttpsNoClientCert_NoClientCert
HttpsNoClientCert_NoClientCert
WindowsAuthTest
WindowsAuthTest
(And 6 more test failures in this build)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

There were 16 failures with about the same error on master:

HttpsNoClientCert_NoClientCert
HelloWorld
WindowsAuthTest
HttpsNoClientCert_NoClientCert
WindowsAuthTest
HttpsClientCert_GetCertInformation
HttpsHelloWorld
HttpsHelloWorld
HttpsClientCert_GetCertInformation
HelloWorld
(And 6 more test failures in this build)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

There were 16 failures with about the same error on master:

HttpsNoClientCert_NoClientCert
HttpsClientCert_GetCertInformation
WindowsAuthTest
HttpsHelloWorld
WindowsAuthTest
HttpsNoClientCert_NoClientCert
HttpsHelloWorld
HelloWorld
HttpsClientCert_GetCertInformation
HelloWorld
(And 6 more test failures in this build)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

There were 16 failures with about the same error on master:

HelloWorld
HttpsHelloWorld
WindowsAuthTest
HttpsClientCert_GetCertInformation
HttpsNoClientCert_NoClientCert
HttpsHelloWorld
WindowsAuthTest
HttpsClientCert_GetCertInformation
HttpsNoClientCert_NoClientCert
HelloWorld
(And 6 more test failures in this build)

This comment was made automatically. If there is a problem contact ryanbrandenburg.

This comment was made automatically. If there is a problem contact ryanbrandenburg.
There were 1 failures with about the same error on 2.2 at 1:53:25 AM:

HttpsClientCert_GetCertInformation

@jkotalik last failure is real:

HTTP Error 403.16 - Forbidden

Your client certificate is either not trusted or is invalid.

 <fieldset>
Most likely causes:

The client certificate used for this request is not trusted by the Web server.

 </fieldset>

 <fieldset>
Things you can try:

The client may have an old certificate selected for client authentication to this Web site. Close all open client windows, open a new browser window, and then select a valid certificate for client authentication.

Verify that the client certificate is trusted by the Web server.

Verify that the root certificate is properly installed and trusted on the Web server.

Check the failed request tracing logs for additional information about this error. For more information, click here.

 </fieldset>

Something in how we generate cert?

The HRESULT is CERT_E_UNTRUSTEDROOT, so somehow the machine doesn't trust the cert. I doubt it's an issue with how we generate the cert, but it's possible.

Will reopen if test start's failing again.


OSError: Could not open stream for reading

I am trying to run the "Offline ASR using Kaldi Models" example using Docker + Jupyter Notebook. When trying to run the script, I am getting the following error on MacOS:
OSError                                   Traceback (most recent call last)
<ipython-input-23-1def4b3ace20> in <module>
     19 asr = NnetLatticeFasterRecognizer.from_files(
     20         model_path, graph_path, symbols_path,
---> 21         decoder_opts=decoder_opts, decodable_opts=decodable_opts)
     22
     23 # Extract the features, decode and write output lattices

/usr/local/lib/python3.5/dist-packages/pykaldi-0.1.2-py3.5-linux-x86_64.egg/kaldi/asr.py in from_files(cls, model_rxfilename, graph_rxfilename, symbols_filename, allow_partial, decoder_opts, decodable_opts, online_ivector_period)
   1052             NnetLatticeFasterRecognizer: A new recognizer.
   1053         """
-> 1054         transition_model, acoustic_model = cls.read_model(model_rxfilename)
   1055         graph = _fst.read_fst_kaldi(graph_rxfilename)
   1056         if not decoder_opts:

/usr/local/lib/python3.5/dist-packages/pykaldi-0.1.2-py3.5-linux-x86_64.egg/kaldi/asr.py in read_model(model_rxfilename)
    887     def read_model(model_rxfilename):
    888         """Reads model from an extended filename."""
--> 889         with _util_io.xopen(model_rxfilename) as ki:
    890             transition_model = _hmm.TransitionModel().read(ki.stream(),
    891                                                            ki.binary)

/usr/local/lib/python3.5/dist-packages/pykaldi-0.1.2-py3.5-linux-x86_64.egg/kaldi/util/io.py in xopen(xfilename, mode, write_header)
    250         binary = True
    251     if reading:
--> 252         return Input(xfilename, binary)
    253     else:
    254         return Output(xfilename, binary, write_header)

/usr/local/lib/python3.5/dist-packages/pykaldi-0.1.2-py3.5-linux-x86_64.egg/kaldi/util/io.py in __init__(self, rxfilename, binary)
     48         self.binary = False
     49         if rxfilename is not None:
---> 50             self.open(rxfilename, binary)
     51
     52     def __del__(self):

/usr/local/lib/python3.5/dist-packages/pykaldi-0.1.2-py3.5-linux-x86_64.egg/kaldi/util/io.py in open(self, rxfilename, binary)
     80             success = self._open_text_mode(rxfilename)
     81         if not success:
---> 82             raise IOError("Could not open stream for reading.")
     83         self._read = _base_io.read if binary else _base_io.read_text
     84         self._readline = _base_io.readline if binary else _base_io.readline_text

OSError: Could not open stream for reading.

Hello,
Did you, by any chance, downloaded the models before running the example?
Maybe take a look at the zamia example to get a better idea of how to obtain the models.

Is there any update on this topic? I have installed pykaldi from source and downloaded the models as it is described in https://github.com/pykaldi/pykaldi/tree/master/examples/setups/zamia . Trying to run any of the python files in https://github.com/pykaldi/pykaldi/tree/master/examples/scripts/asr still prints the same error
Traceback (most recent call last):
File "nnet3-rnnlm-recognizer.py", line 24, in
decodable_opts=decodable_opts)
File "/home/daniel/repository/pgchill2/Entwicklung/Speechtest/pykaldi/tools/env/local/lib/python2.7/site-packages/pykaldi-0.1.2-py2.7-linux-x86_64.egg/kaldi/asr.py", line 1054, in from_files
transition_model, acoustic_model = cls.read_model(model_rxfilename)
File "/home/daniel/repository/pgchill2/Entwicklung/Speechtest/pykaldi/tools/env/local/lib/python2.7/site-packages/pykaldi-0.1.2-py2.7-linux-x86_64.egg/kaldi/asr.py", line 889, in read_model
with _util_io.xopen(model_rxfilename) as ki:
File "/home/daniel/repository/pgchill2/Entwicklung/Speechtest/pykaldi/tools/env/local/lib/python2.7/site-packages/pykaldi-0.1.2-py2.7-linux-x86_64.egg/kaldi/util/io.py", line 252, in xopen
return Input(xfilename, binary)
File "/home/daniel/repository/pgchill2/Entwicklung/Speechtest/pykaldi/tools/env/local/lib/python2.7/site-packages/pykaldi-0.1.2-py2.7-linux-x86_64.egg/kaldi/util/io.py", line 50, in init
self.open(rxfilename, binary)
File "/home/daniel/repository/pgchill2/Entwicklung/Speechtest/pykaldi/tools/env/local/lib/python2.7/site-packages/pykaldi-0.1.2-py2.7-linux-x86_64.egg/kaldi/util/io.py", line 82, in open
raise IOError("Could not open stream for reading.")
IOError: Could not open stream for reading.


No list of commands

There isn't a docs article with a comprehsnvie list of commands, or proper syntax. Each of the commands has a basic man article in its script, but thats not super clear docs wise.

The docs are very unclear indeed, and not just in this area.
It would be great to have docs for every command beyond their man page.
Also all the commands could be listed via the man command itself.
Having options like man --list and man -l could be a great addidion.

Because list isn't likely a commonly used flag I think -L would be better but --list would be good yeah.

I quickly added the list option.
It showed me that it would be best if the rework of GDShellCommandParser and GDShelCommandRunner, would include the GDShellCommandDB because the way of handling aliases is really awkward.


Support switching to frame by index

Please improve the switchToIFrame method with frame/iframe switching support by an index. Currently, in the W3C-compliant mode, it doesn't work.
Please see https://github.com/minkphp/Mink/issues/866 for more info.
I would happily create PR similar to https://github.com/minkphp/MinkSelenium2Driver/pull/378 as well.

Please improve the switchToIFrame method with frame/iframe switching support by an index. Currently, in the W3C-compliant mode, it doesn't work.

It should already work, and actually the PR you linked to looks broken - it only handles index (id), or?
    public function switchToIFrame(?string $name = null)
    {
        $this->getWebDriverSession()->frame(array('id' => $name));
    }

vs
    public function switchToIFrame(?string $name = null): void
    {
        $frameQuery = $name;
        if ($name && $this->getWebDriver()->isW3cCompliant()) {
            try {
                $frameQuery = $this->getWebDriver()->findElement(WebDriverBy::id($name));
            } catch (NoSuchElementException $e) {
                $frameQuery = $this->getWebDriver()->findElement(WebDriverBy::name($name));
            }
        }

        $this->getWebDriver()->switchTo()->frame($frameQuery);
    }

I'd be fine with replacing ?string type-def with null|string|int PHPDoc type-hint though.

Looking at the json-wire link you provided it mentions:

JSON Parameters:
id - {string|number|null|WebElement JSON Object} Identifier for the frame to change focus to.

It doesn't say anything about an index just "identifier", however it also mentions the number type, so if I understand correctly, this means:
->switchToIFrame('xx');    // non-W3C mode, <iframe name=xx>
->switchToIFrame('xx');    // W3C mode, <iframe id=xx>
->switchToIFrame(1);       // W3C mode, first <iframe> element <- this is broken case we're discussing now, right?

@uuf6429 , I've confirmed (by running a live test), that providing a number does switch to a window.frames[that_number] frame.
I'm only concerned by the frame index switching, because in Selenium 3 using MinkSelenium2Driver frame switching by name doesn't work at all.

I'm only concerned by the frame switching by index, because in Selenium 3 using MinkSelenium2Driver frame switching by name doesn't work at all.

I guess that's because MinkSelenium2Driver is only using id()? This driver actually does check selenium 3, and it works. For me this looks like a new feature; to support switching by index (which is also ok).


Why is only arm64 supported on Mac?

CMake Error at CMakeLists.txt:50 (message):
  Only arm64 is supported on macOS

❯ uname -m
x86_64

I was so excited about this. What are the limitations to running on x86_64? Can I help?

Hey @jcaffey, I don't have an Intel Mac so I didn't test it, so not a terribly good reason. You're welcome to add support for it with a PR.
Builds of CEF (Chromium Embedded Framework) support it, so it should work 🙂
Currently uses these builds:
https://cef-builds.spotifycdn.com/index.html
You'll need to update the CMake files to support it. Thanks for your interest in my little project.

Hey @jcaffey, I don't have an Intel Mac so I didn't test it, so not a terribly good reason. You're welcome to add support for it with a PR.
Builds of CEF (Chromium Embedded Framework) support it, so it should work 🙂 Currently uses these builds: https://cef-builds.spotifycdn.com/index.html
You'll need to update the CMake files to support it. Thanks for your interest in my little project.

I hacked up CMakeLists just to see if it worked. I got it to work in kitty, but it was terribly slow / sluggish on scroll.
https://github.com/jcaffey/awrit/blob/main/CMakeLists.txt
I also tried in wezterm (which I think supports the correct graphics protocol .... maybe not) but had no luck there.
Cool project, I hope to see it grow!

@jcaffey I'm not sure if it's slow because of poor rendering performance or not, since it seemed fine on both of my machines.
If it's not stuttering, but it just seems like it isn't scrolling very much relative to your input, you can try adjusting the amounts here:
https://github.com/chase/awrit/blob/main/awrit/input_event_handler.cc#L330-L337
I'm sure there's a better way to scale mouse wheel input, but it's basically transmitted as a series of "mouse wheel scrolls down/up" but without any sense of velocity I just set it to 10 :sweat_smile:
wezterm needs to be configured to support Kitty graphics protocol and key protocol, it doesn't work out of the box.


Revert "Fix regressions related to cuDF changes in handline of end-of…

This reverts commit 3398daaba3513a55e08d09601f502396c189751b.
It looks like my PR maybe broke the build. Testing this.

build


texlive.combine: call faketime from within fmtutil

Description of changes
Calling 'faketime fmtutil' fails on some non-NixOS systems, due to fmtutil calling system, and implicitly /bin/sh, which may be incompatible with faketime's LD_PRELOAD (e.g. using a different glibc). This change moves faketime within fmtutil so that LD_PRELOAD is applied only to the final binary.
Fix #230519.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[X] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual) both with and without sandbox
[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg build tests.texlive


[JAV-147]RestTemplate支持扩展

Coverage increased (+0.07%) to 82.062% when pulling 9b31c6f89f700e4f86d2fcbfd3533321e3ae5e2b on liubao68:discover into 908593188e1e0f1438ecc2fc4081bd1d1e52040e on ServiceComb:master.

Coverage increased (+1.5%) to 83.462% when pulling 415236a140b645ad24e376b150aa376f6e623967 on liubao68:discover into 908593188e1e0f1438ecc2fc4081bd1d1e52040e on ServiceComb:master.


added ivy's details

Added my details to the page
I also forgot to create a branch and got lazy ...

Hi @Ivy8127, did you look at the comments above?

Hi @Ivy8127, did you look at the comments above?

Hi, @RuthNjeri  I've made corrections to the comments you made. About the width and height, the width and height style attributes aren't changing to 600 px though when I locally view the image. Is it the bootstrap? How do I change that?

Hi @Ivy8127, can you share a screenshot of what's happening on your end?

Hi @Ivy8127, can you share a screenshot of what's happening on your end?

@RuthNjeri  Sure :

Hi Ivy, what would happen if you use inline styling and add the ! important rule?

Feedback implementation took too long. Closing the PR.


Seekable Azure Blob Stream

Provide a leaky abstraction (via IAzureBlobStorageNativeOperations) to open azure block blob as a seekable stream.
The seekable stream will allow to navigate to any part of a massive blob without downloading it locally.

hey,
is there any update regarding this?

@HassanBadir write to local disk first, then upload from disk to azure blob. As explained before the issue is azure blob SDK, not parquet library.

Thank you, yea I figured it out eventually .. although it would be awesome to have this feature.


Provide user manual directly from product

This is subject to change, but we should make the awesome help in the user manual available to users right in the product at the point they're entering data (particularly the notes and services instructions).

Wow, I never knew this document existed! It's really cool to see all the different things that Notes/Services get used for and to see what the features we're working on look like from the teacher's perspective.
It is cool if I share this with new folks? (Looks like it might have a bit of real student data in it).

@erose It's brand new!  :) Good eyes on that, thanks!  I thought it was fully scrubbed so I removed the document and will re-scrub so we can share with everyone.

This will need to be scrubbed.

Steps!

[ ] Scrub user manual, add to docs as Markdown
[ ] Design: "someone sitting in a meeting, not sure what they should"
[ ] Implement

Design mockup. Here's one thought: (clicking on a question-mark bubble would open a help tooltip).

And another take: adding some friendlier text to the "Which service?". Clicking on it would open the same help tooltip.

@kbweaver Any design feedback on this?

Note to self: a similar thing exists on the roster page.

@erose Nice! I like how the question marks are prominent and purple, draws the eyes. And I like the "help me decide" text a lot too. You can ignore a floaty question mark but you can't ignore "help me decide."


[TF 2] Expose tf.layers as alias for issue

In #26176, @MarkDaoust said about this
It's been some time since then, so I thought I'd go ahead and send the PR. Let me know if anything needs changing!

Can one of the admins verify this patch?

@pavithrasv , can you please review this change

#29197 is the origin of these changes, these are 100% copy-pasted


update: Third-party categorization

Description
WIP, mockups for third-party plugins.
https://konghq.atlassian.net/browse/DOCU-3509
Testing instructions
https://deploy-preview-6236--kongdocs.netlify.app/hub/
https://deploy-preview-6236--kongdocs.netlify.app/hub/?support=third-party
Partner plugin: https://deploy-preview-6236--kongdocs.netlify.app/hub/imperva/imp-appsec-connector/
Community plugin: https://deploy-preview-6236--kongdocs.netlify.app/hub/amberflo/kong-plugin-amberflo/
Checklist

[x] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

In the URL support=third-party still implies some level of support. Can we change to support=none?
The “Not Supported” phrasing makes me uneasy. It implies that they won’t work, when we actually mean “functional, but no support contract”. Can we rephrase to something like “No Support Available”?

In the URL support=third-party still implies some level of support. Can we change to support=none?
Yep, I can do that.

The “Not Supported” phrasing makes me uneasy. It implies that they won’t work, when we actually mean “functional, but no support contract”. Can we rephrase to something like “No Support Available”?

Maybe "not supported by Kong"? "No support available" is also not quite right, since all of these plugins have support links, eg https://deploy-preview-6236--kongdocs.netlify.app/hub/moesif/kong-plugin-moesif/

The more I think about it, the more it sounds to me like "Not supported by Kong" is going to sound like "Not supported by Kong Gateway", since Kong is still an unofficial short-form for Gateway in... well, everywhere. Going with your suggestion as the lesser of two evils here :)


Add created and deleted counters

Co-authored-by: marthehagen<EMAIL_ADDRESS>

Co-authored-by: marthehagen<EMAIL_ADDRESS>


1.18 broken dependencies

Geckolib has no 1.18 version for forge making it impossible to use this mod on minecraft 1.18

Can you tell me what is this geckolib jar file?


/usr/bin/env: "node\r" no such file or directory found

I have this problem under ubuntu 16.04
After installing globally with sudo npm install -g angular-material-tools, I type angular-material --help, and I get this error /usr/bin/env: "node\r" no such file or directory found
Any workaround? Thanks!

Thanks for the issue. We have fixed this with SHA https://github.com/angular/material-tools/commit/3dbea5610d831311a31058fd22aa87bc2811726b. A new release beta.6 is also published.


how to copy the font files to public folder when compiling a css file?

Laravel Mix Version: 0.8.9 (npm list --depth=0)
Node Version (node -v): 6.10.2
NPM Version (npm -v): 4.5.0
OS: Ubuntu 16.04.2 LTS

Description:
When I copy a css file to public folder, the font source url cannot be copied to /fonts/ folder or /public/ folder, but the font files in bootstrap and font-awesome can be copied correctly and automatically, what need I do?
Steps To Reproduce:
mix.css('node_modules/element-ui/lib-theme-default/index.css', 'public/css/element-ui.css');

I see. Only the url in less or sass file will be handled automatically, css file will not.

It's probably mix.styles() rather than mix.css()


Token icons are requested every time the app is started (privacy issue)

When a provider gives an icon URL in the otpauth scheme, FreeOTP uses it in the list of tokens.
The issue is that the icon is not cached and is requested every time the app is loaded. That can lead to a privacy issue, as this provider will be able to track when someone opens up the app and from which IP it comes from (and possibly more? I didn't check what info is sent in the HTTP request).

This project is abandoned.  Please contribute to an alternative https://github.com/freeotp/freeotp-android/issues/207

On the iOS side there has been some progress. The Android project is in need of someone to take over active development.
I am currently looking after the Android PRs, but don't have time to take on the big feature work in the Renaissance project.


TensorFlow MiniLM segfaults for GPU cases on A100.

To reproduce:
pytest tank/tf/MiniLM-L12-H384-uncased_tf_test.py -k "gpu"

fixed ?


Propagate single-sourced projectId to namespace and filepaths

User request from SCE, see b/156327862

marking as done till i hear otherwise


need a strong validation on unit file

When I try to do a side kick for a unit, I made a mistaken on the unit file for the key word "BindsTo"
I wrote it as "BindTo" without "s"
But fleet/fleetctl doesn't report any error.
But sometimes, it is blocked during start this wroing unit and there is no error logs in etcd, or fleet or systemd
So I hope fleet can do some strong validation on the uint file at the submit stage.

fleet is not going to validate the content of systemd options it does not control. We cannot chase systemd releases with validation code. Asserting the format using github.com/coreos/go-systemd/unit is all we can reasonably do for now.


Whether to release fine-tuning code

It's a training-free framework, isn't it?

Is the Complementary Regional Diffusion also training-free?


Match doesn't narrow tagged union type when matching on union's tag

Environment data

Language Server version: Pylance language server 2022.4.3 (pyright 88f2a424)
OS and version: Ubuntu 20.04.2 LTS
Python version (& distribution if applicable, e.g. Anaconda): Python version 3.10 (condaforge)

Expected behavior
When matching on a tagged union's tag should narrow the type of the union
Actual behavior
When matching on a tagged union's tag, the type of the union class is not narrowed
Code Snippet / Additional information

import typing as t

class Dog:
    type: t.Literal['dog']
    favorite_snack: str

class Cat:
    type: t.Literal['cat']
    favorite_activity: str

Animal = Dog | Cat

def fun(a: Animal):
    match a.type:
        case 'dog':
            print(a.favorite_snack)  # <- Should narrow type to Dog, but a stays Animal so this is flagged as an error
        case 'cat':
            print(a.favorite_activity) # <- Should narrow type to Cat, but a stays Animal so this is flagged as an error

Yes, this form of type narrowing is not currently supported by pyright, the type analyzer that underlies pylance. It's also not supported by mypy, another popular Python type checker.
We could add support for tagged unions in match statements, but we'd probably do so only if we received sufficient signal from users that this pattern is commonly used. In the meantime, here are a few straightforward workarounds.
Both pyright and mypy do support tagged union type narrowing for if/elif statements:
def fun2(a: Animal):
    if a.type == "dog":
        reveal_type(a)  # Dog
    elif a.type == "cat":
        reveal_type(a)  # Cat

You can also use class pattern matching, which would be a more idiomatic way to use match in this case:
def fun3(a: Animal):
    match a:
        case Dog():
            reveal_type(a)  # Dog
        case Cat():
            reveal_type(a)  # Cat

Thanks for the reply. That all makes sense!
In my situation, classes can have multiple tags, e.g.:
import typing as t

class Dog:
    type: t.Literal['dog']
    favorite_snack: str

class Cat:
    type: t.Literal['cat', 'feline'] # Add a second tag
    favorite_activity: str

Animal = Dog | Cat

def fun(a: Animal):
    match a.type:
        case 'dog':
            print(a.favorite_snack)
        case 'cat':
            do_something()
            print(a.favorite_activity)
       case 'feline':
           do_something_else(a.favorite_activity)

I tried to do class pattern matching, but it doesn't detect the following as exhaustive:
import typing as t

class Dog:
    type: t.Literal['dog']
    favorite_snack: str

class Cat:
    type: t.Literal['cat', 'feline'] # Add a second tag
    favorite_activity: str

Animal = Dog | Cat

def fun(a: Animal):
    match a:
        case Dog():
            print(a.favorite_snack)  # <- Should narrow type to Dog, but a stays Animal so this is flagged as an error
        case Cat(type='cat'):
            do_something()
            print(a.favorite_activity) # <- Should narrow type to Cat, but a stays Animal so this is flagged as an error
       case Cat(type='feilne'):
           do_something_else(a.favorite_activity

Are if statements the only way to go, or is there a better approach?

How about this?
def fun(a: Animal):
    match a:
        case Dog():
            print(a.favorite_snack)
        case Cat():
            match a.type:
                case "cat":
                    do_something()
                    print(a.favorite_activity)
                case "feline":
                    do_something_else(a.favorite_activity)

gotcha, that works too. The indent level starts to become a bit much (esp when there are Dog subtypes, etc.), but I still like it better aesthetically than the if statements + assert_never. I'll use this for now, and then if this feature ever gets implemented I'll be able to knock out that extra level of indent. I realize structural pattern matching is still pretty new for python so my fingers are crossed.
thanks again for the replies!


Help users avoid accidentally opening Scope to the whole Internet

We do warn users not to run the Scope app exposed to the Internet, but apparently some people carry on.
Perhaps Scope should default to a simple basic-auth scheme with a randomly-generated password so it is a little safer?

Maybe as a first step we could document how to do that in a very basic way?

Do you mean "we could expand the instructions on how to use a reverse proxy"?
Since this issue is about changing the code to behave less open, we wouldn't document that specifically.

I have some code working to add basic auth support to protect weave. I can create a pr if people are interested.

https://github.com/weaveworks/scope/pull/3393


No RancherOS images for RPI since 1.5.6?

RancherOS Version: (ros os version)
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
I noticed that RPI images are no longer being released since v1.5.6? I spent some time googling, but haven't found any relevant information. Is this by design?

Duplicate of #3050


Add remaining support for AsyncBufRead

The remaining encoders/decoders that are yet to be supported through AsyncBufRead:

[ ] gzip
[ ] brotli
[ ] zstd

Help would be appreciated!

This should probably be a wrapper around AsyncRead instead, right?

I suppose it could be, although I originally went off of @Nemo157's original design, which was that the associated reader for a given AsyncBufRead encoder/decoder would be a type of AsyncBufRead. Though I don't imagine it'd take too long to refactor this, I wonder what he thinks.

Adding AsyncRead support would be pretty trivial, just wrap the incoming AsyncRead into a BufReader so that it supports AsyncBufRead. Since the en/decoders operate on in-memory buffers you need a buffering reader.


test: Cypress - mock api url update + Flaky fixes

Description

Modified mock api url and refactored specs accordingly

Flaky fixes below:

Listv2_BasicChildWidgetInteraction_spec.js
QueryPane/Mongo_Spec.js
CreateNewApp_spec.js

Type of change

Flaky test fix

How Has This Been Tested?

Cypress test runs

Checklist:
QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=2ebb278

/ok-to-test sha=4ed3b5f

/ok-to-test sha=3e9cc22

/ok-to-test sha=dc8f574

/ok-to-test sha=163fa60

/ok-to-test sha=a9a41aa

/ok-to-test sha=a9a41aa


open_mfdataset for merging of netCDF files with different time values

What is your issue?
In xarray version 0.16 I could combinine multiple netCDF files with different values on the time coordinate along a new dimension in a straight forward way:
    def rename_time(ds: xr.Dataset) -> xr.Dataset:
        return ds.rename_dims({'time': 'sim_time'})

    ds = xr.open_mfdataset(ncfiles, concat_dim='simulation', preprocess=rename_time,
                                           combine='nested', coords=['time'])

The merged dataset got a new time dimension called "sim_time" which had dimensions of simulation and time, so a 2D array.:

When upgrading xarray to 2022.9.0, this code no longer works:
    ds_allsims = xr.open_mfdataset(coarser_files, concat_dim='simulation', preprocess=rename_time,
  File "/home/jorgens/mambaforge/envs/era_acute_post_processor/lib/python3.10/site-packages/xarray/backends/api.py", line 1003, in open_mfdataset
    combined = _nested_combine(
  File "/home/jorgens/mambaforge/envs/era_acute_post_processor/lib/python3.10/site-packages/xarray/core/combine.py", line 365, in _nested_combine
    combined = _combine_nd(
  File "/home/jorgens/mambaforge/envs/era_acute_post_processor/lib/python3.10/site-packages/xarray/core/combine.py", line 239, in _combine_nd
    combined_ids = _combine_all_along_first_dim(
  File "/home/jorgens/mambaforge/envs/era_acute_post_processor/lib/python3.10/site-packages/xarray/core/combine.py", line 275, in _combine_all_along_first_dim
    new_combined_ids[new_id] = _combine_1d(
  File "/home/jorgens/mambaforge/envs/era_acute_post_processor/lib/python3.10/site-packages/xarray/core/combine.py", line 298, in _combine_1d
    combined = concat(
  File "/home/jorgens/mambaforge/envs/era_acute_post_processor/lib/python3.10/site-packages/xarray/core/concat.py", line 243, in concat
    return _dataset_concat(
  File "/home/jorgens/mambaforge/envs/era_acute_post_processor/lib/python3.10/site-packages/xarray/core/concat.py", line 569, in _dataset_concat
    combined_idx = indexes[0].concat(indexes, dim, positions)
  File "/home/jorgens/mambaforge/envs/era_acute_post_processor/lib/python3.10/site-packages/xarray/core/indexes.py", line 327, in concat
    new_pd_index = cls._concat_indexes(indexes, dim, positions)
  File "/home/jorgens/mambaforge/envs/era_acute_post_processor/lib/python3.10/site-packages/xarray/core/indexes.py", line 307, in _concat_indexes
    raise ValueError(
ValueError: Cannot concatenate along dimension 'simulation' indexes with dimensions: 'sim_time'

Looking around, I found this answer on stack overflow for a similar question: https://stackoverflow.com/a/55500434
I had to modify the preprocessor in the answer to make it work:
    def rename_time(ds: xr.Dataset) -> xr.Dataset:
        ds['sim_time'] = ds.time
        ds = ds.drop('time').rename({'time': 'ntime'})
        return ds

However, this creates a dataset structure which is less useful in my case:

Here, time is lost as a coordinate, and I have to explicitly work the new "sim_time" variable to make sense of my data variables.
This is not impossible to work with, but I preferred the previous output.
My questions are: 1) is the change in behavior in xarray as intended and 2) is there a way for me to get the old dataset structure out of a new version of xarray?

Thanks @kmuehlbauer! That does give me the same dataset structure as before which can go directly into my pipeline.
However, I'd like to note that the previous method offloaded more of the work to xarray through the API which made the code more clear to understand. It was more explicit since one could specify that concatenation should happen for the time-coordinate, and the post-processor was just one short operation.
If this API will be changed again in the future and someone comes across this use-case could be kept in mind.


Generalized range operators in for-inloops and other purposes

Generalized range operators in for-in loops and other purposes
Introduction
Departing from C-stype forloops, the Swift for-in loops use a range operator ... for specifying the bounds of the range of values to be covered. In practice, a more useful operator ..< which excludes the upper bound is used, typically for accessing array elements given the starting value 0 and the element count. We propose a generalisation of these interval operators, offering more complete description of the bound values, as well as enumeration order control when this is needed, typically when used in for-in loops.
Our goal is to extend the available range operators using a simple and consistent logic for most useful tasks when it comes to enumerating items in for-in loops and other purposes, with simple control of enumeration order and bound exclusion. While we preserve the behavior of the ..< operator and model the newcomers logically from it, we also extend the scope of the ... operator in a useful and consistent way.
Proposed Approach
We propose that ranges could be described by the following operators:
..< , .<., <.., >.., .>., ..>, ...
The various notations specifies if one of the boundaries should be excluded, and also specifies the eventual enumeration order (ascending or descending) that would typically be used in for-inloops.

Ascending order:

..<: Ascending, start value included, end value excluded (Already exists).
[0 ..< 4] = [0, 1, 2, 3]
[2 ..< 2] = []
[4 ..< 0] = []
.<.: Ascending, start and end values included.
[0 .<. 4] = [0, 1, 2, 3, 4]
[2 .<. 2] = [2]
[4 .<. 0] = []
<..: Ascending, start value excluded, end value excluded.
[0 <.. 4] = [1, 2, 3, 4]
[2 <.. 2] = []
[4 <.. 0] = []

Descending order

..>: Descending, start value included, end value excluded.
[4 ..> 0] = [4, 3, 2, 1]
[2 ..> 2] = []
[4 ..> 0] = []
.>.: Descending, Start and end values included.
[4 .>. 0] = [4, 3, 2, 1, 0]
[2 .>. 2] = [2]
[0 .>. 4] = []
>..: Descending, start value excluded, end value included.
[4 >.. 0] = [3, 2, 1, 0]
[2 >.. 2] = []
[0 >.. 4] = []

Adaptive order

...: Ascending or descending according to the relative start and end values, which are included (Specification changed).
[0 ... 4] = [0, 1, 2, 3, 4]
[2 ... 2] = [2]
[4 ... 0] = [4, 3 ,2 ,1, 0]
Usefulness
Although there are simple algorithmic ways of reproducing the effect of these operators through the use of existing operators and bound value offsets, these variations nevertheless help improve code simplicity and clarity in many situations where for-in are used.
Hence, while navigating through the indexes of an array would be done as follows:
for i in 0 ..< myArray.count {
	ProcessArrayElement(i)
}

Doing the same on a descending order gets a little more complicated:
for i in 0 ..< myArray.count {
	ProcessArrayElement(myArray.count-1-i)
}

Using the proposed generalised range operators, we can restore the simplicity for the descending order:
for i in myArray.count >.. 0 {
	ProcessArrayElement(i)
}

Another example, when processing needs to be done between two bounds regardless of their ordered value. Currently, it has to be done as such:
if startValue <= endValue {
	for i in startValue ... endValue {
		Process(i)
	}
} else {
	for i in endValue ... startValue {
		Process(startValue+endValue-i)
	}
}

With the new approach, this could simply be done as follows:
for i in startValue ... endValue {
	Process(i)
}

When ordering is not significant, these operators should be consistently equivalent. For example:
if case 10 ..< 100 = value {
	print(value, " has two digits")
}

Should be equivalent to:
if case 100 >.. 10 = value {
	print(value, " has two digits")
}

Another motivation to change the behavior of the ... range operator is to alleviate the current inconsistency between the existing ...and ..< behaviors, Hence, currently, if y is smaller than x, x ..< y+1is an empty range while x ... yreturns an error condition. With the proposed scheme, we take the opportunity to remove the redundancy between these two operators by extending the scope of ... in a useful and logical way.
Impact on existing code
This extension should allow numerous minor simplification in existing code, mostly by easing the specifications of bound values in for-inloops. Code clarity would be improved.
Allowing explicit statement of element ordering also helps analysis of code consistency.
Compatitility with existing code
The ordered operators are a generalisation of the ..<range operaton which already exists in the current language specification. There is no significant compatibility issue for using them or not.
The only significant departure from the existing specification is the ability of the ... operator to describe a range if both ascending or descending order. Existing code which would create an error using x ... y when y is smaller than x would here not cause any error. This could have an impact when migrating old code if this error condition is used as a signal for inconsistent values during the processing, particularly as unreviewed code would not cause any error at compile time. Yet, upon code review, adding boundary order conditions for restoring the original limitations of the ... range operator is fairly trivial.
At a broader range in the language specification, the explicit inclusion on enumeration order (ascending, descending, adaptive) has to be evaluated and handled consistently anywhere where ranges can be used. Some other specification changes could be welcome for ensuring proper consistency generality and orthogonality, although we expect these to be minor and have no broad implication as a whole.
Alternative options

No changes, if we consider that the specification changes outweigh the benefits of having these range operators available for rewriting algorithms in a simplier way.

Preserving the old behavior of the ... operator. This would preserve code compatibility and reduce possible migration woes, the ordered operators being additions that could be used or not. However, this would alter the consistency of the proposed enumeration order scheme, while loosing the benefit of being able to specify a border value order-agnostic range.

Restricting the proposed scheme for the range operators in for-inloops. This would alleviate the possible need to reconsider ordering for all language features using ranges, at the expense of loosing generality and orthogonality in the language specification.

Extend the scheme with additional operators allowing the exclusion of both bounds of the range (for example, with <.< and >.> operators). We don't expect this to have any significant practical use.

Please follow the proposal template: among other things, we need a link to the mailing list discussion (which is still ongoing), it should be a ".md" file, etc.


PageFormExecutionEnvironmentSelect Update

Updated PageFormExecutionEnvironmentSelect to use the PageFormSingleSelectAwxResource component.

Are we going to do those extensions also in multi versions?

Are we going to do those extensions also in multi versions?

Yes

When I go to the Execution Environment form field and select Browse, I see the following:

When I click into the Edit form for the Job Template, it shows the execution environment's ID instead of its name. Can we change this so the name is displayed?


ClaimsPrincipalConverter potential problem when deserializing ClaimsPrincipal

In Thinktecture.IdentityServer.Core solution ClaimsIdentity is created using such statement
var id = new ClaimsIdentity(claims, authenticationType, Constants.ClaimTypes.Name, Constants.ClaimTypes.Role);

whereas in Thinktecture.IdentityServer.EntityFramework for ClaimsPrincipalConverter is used such statement
var id = new ClaimsIdentity(claims, source.AuthenticationType);

For that reason Name property of ClaimsPrincipal is null. I believe it should have value of claim with type name.

So you're getting Name of null somewhere in your code (perhaps in the user service) due to this?

Nope it doesn't cause any errors right now. I had that error because I've started reusing that converter.

Ah ok. I was trying to determine where it mattered. I think it'll be ok to make this change if it help you.

Would be great. Thanks.

Do you want to send a PR for this? It'll get done sooner if you do it :)

Ok, this was quick -- I added it on dev.


HSEARCH-3468 Restore the ability for the Elasticsearch schema validation to report errors across multiple indexes

https://hibernate.atlassian.net//browse/HSEARCH-3468

Pull Request Test Coverage Report for Build 1

101 of 107 (94.39%)  changed or added relevant lines in 15 files are covered.
16 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.05%) to 87.858%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

backend/elasticsearch/src/main/java/org/hibernate/search/backend/elasticsearch/logging/impl/ElasticsearchEventContexts.java
5
6
83.33%

backend/elasticsearch/src/main/java/org/hibernate/search/backend/elasticsearch/index/admin/impl/ValidationErrorCollector.java
35
40
87.5%

Files with Coverage Reduction
New Missed Lines
%

backend/elasticsearch/src/main/java/org/hibernate/search/backend/elasticsearch/orchestration/impl/AbstractElasticsearchBarrierWorkOrchestrator.java
1
79.55%

backend/elasticsearch/src/main/java/org/hibernate/search/backend/elasticsearch/index/impl/ElasticsearchIndexManagerBuilder.java
6
78.57%

backend/elasticsearch/src/main/java/org/hibernate/search/backend/elasticsearch/index/admin/impl/ValidationContextElement.java
9
27.27%

Totals

Change from base Build 138:
-0.05%

Covered Lines:
14530

Relevant Lines:
16538

💛  - Coveralls

Merged, thanks!


test: toast 补充测试

Coverage increased (+0.4%) to 76.272% when pulling b43d4018bcca0cdca003eb97bb8e9f5843cce0df on Yang03:toast into 70d1b80fa824f77b41a2419333599c2dfab36c94 on ZhonganTechENG:dev.

Coverage increased (+0.4%) to 76.272% when pulling b43d4018bcca0cdca003eb97bb8e9f5843cce0df on Yang03:toast into 70d1b80fa824f77b41a2419333599c2dfab36c94 on ZhonganTechENG:dev.


PulsarSpout emits only to default stream

Is your feature request related to a problem? Please describe.
We have an application, currently using another streaming ingestion method, that we'd like to replace with either Kafka or Pulsar. Either would be a candidate, except the spout needs to emit to different output streams (to be processed by different bolts) based on the content of individual messages.
The Kafka spout supports this. Topologies implement RecordTranslator, an interface that returns the output stream and the Values for the tuple.
The equivalent in PulsarSpout is MessageToValuesMapper, but that returns only the Values. There's no way to specify an alternate stream; everything goes to "default."
Describe the solution you'd like
Simply put, a way for MessageToValuesMapper to specify the output stream for a tuple. (It already has a declareOutputFields() method so can declare the appropriate output streams.)
Two possible solutions, both of which would require a backwards-compatible extension of MessageToValuesMapper. The first I think is a bit cleaner, but both would work.

Change PulsarSpout to use a new method in MessageToValuesMapper that returns a PulsarTuple interface similar to Kafka's KafkaTuple. It would return the Values and output stream. A default implementation of the new method would just call the existing method (which probably should be deprecated) and return a PulsarTuple with stream "default."
Add a new method to MessageToValuesMapper: String outputStream(Message msg, Values values) to return the output stream given the original Message and extracted Values. The default implementation would just return "default."

Describe alternatives you've considered
All of these would require collector to be not private, and probably other things. None are very satisfying. All would require an application to extend PulsarSpout, which can cause compatibility issues with new versions.

Change the existing collector.emit() calls with a call to a non-private doEmit(Message msg, Values values) call. Applications could override that and figure out the output stream as needed.
Make mapToValueAndEmit() at least protected so it can be overridden.
Override nextTuple() and replicate most of the logic except the emit() calls.

Or, just use Kafka, which is what we'd actually do.
Additional context
One application receives Avro-encoded events with a header that identifies the schema type and version. (Also some other information not relevant here.) The schema type determines the output stream.
Another application has a fixed Avro schema with two fields, an event type and a serialized protobuf. The event type determines the output stream.
Changing those formats isn't an option.

Thanks @dawillcox for the help on this issue and PR

Any guidance on what I I need to do with this going forward would be appreciated. For example, should a do a separate PR to master?

I'll note that branch-1.20 has quite a bit more extensive unit tests for pulsar_storm. They really ought to be brought forward.

@dawillcox The Storm tests were moved to the integration tests, because of issues with shading dependencies. https://github.com/apache/pulsar/tree/master/tests/pulsar-storm-test


SVG Element can't dynamic animate.

Version
2.5.3
Reproduction link
https://codepen.io/shockw4ver/pen/Nwjowa
Steps to reproduce
Enter the page, the animate will be triggered once, CLICK SVG, change the value of isOpen to switch between two , however nothing happend, svg just keeps its state changed from the page entering.
What is expected?
svg should transform when data(isOpen) change.
What is actually happening?
only transform once at entering the page.

If I got a wrong way when using ?
 I think you would be better off trying to use CSS transforms when you want dynamic animations.

I don't think the repro clearly demonstrates that the problem is related to Vue or not, because it involves potential problems in the way you use SVG. Please provide a repro with your intended animation working in plain SVG so that we know this is actually a Vue issue.
/cc @sdras

Hey there and welcome!
This problem isn't with Vue or the SVG, but rather the logic. You're asking it to fire if the condition of isOpen returns true, which it does on initial page visit.
I'm sorry to say, but as a side note I don't suggest you continue using SMIL's <animateTransform>.Chrome is pausing deprecation for the moment, but it's on the chopping block. Microsoft says they will never ever support it for IE/Edge browsers.
For better support, I highly recommend using GreenSock's TweenMax- you'll eventually run into cross browser inconsistencies on these transform origins otherwise. You can check this demo, a fork of yours, for an example:
https://codepen.io/sdras/pen/c2483edf1630550c2e70f4fd39a67b19?editors=1010
I threw it together from yours pretty quickly, so you'd probably need some tweaks here and there. Let me know if you have further questions!

Thanks for the explanation Sarah.

I've just hit this myself trying to use https://codepen.io/ghenry/pen/LrGwgW in a Spinner component. Will look at GreenSock too.


Include sigma-clipping in mean calc for noisy search

This PR changes the code such that sigma-clipping is used when computing the mean and standard deviation of the standard deviation values across the input slope images.

@KevinVolkSTScI, is this the change you envisioned? The script was already using scipy's sigma clipping function, so I thought it would be easiest to start by using that again.

This is similar to what I was doing, but I prefer not to use sigmaclip because there is no control over the number of iterations that it does and the results can become very sensitive to noise in cases where the main distribution is not Gaussian.  I would just have to see what the results look like compared to the other cases that I coded in for the specific example of NIRISS darks.  It may not make a big difference in practical terms.

@KevinVolkSTScI I just updated the code to use astropy's sigma_clip. The maximum number of iterations is now a user-settable parameter.

I took a look at the code changes and it all looks good.  I have to test it and see what the results are, but I may not have time this week.

I made a trial run with the changes and it flagged 130000 noisy pixels.  That is roughly similar to what I got myself.  So it looks like it is working, although I need to do some analysis on the results when I have time.

I'm glad the results are more in agreement than before. I'm going to merge this so that others can test with the sigma clipping as well.


ScintillaNET 4.0 64-bit SCNotification modificationType is 0

Hi,
ScintillaNET 4.0 which hasn't been released yet appears to have a bug when the platform target is 64-bit.
ScnModified method in LineCollection.cs receives a parameter NativeMethods.SCNotification. When the text is changed SCNotification.modificationType is always 0 and therefore TrackInsertText and TrackDeleteText do not get called and this causes Lines.Count to always be 1.
In order to reproduce make sure to set your platform target in your WinForms project to x64. Goto Project-->Properties-->Build tab.
Let me know if you are able to reproduce the bug.
Regards,
leojazz

This is fixed by #447.


Made package selection more minimal and concrete

Unwanted packages will not install anymore (systemd, qemu-guest etc.)
Package count and size reduced from 165PKG, 149MB  to 101 PKG and
104MB
Subscription Manager and its dependencies did not include. Therefore,
it is more lightweight than the RHEL variant (103pkg/103MB)

Signed-off-by: Elkhan Mammadli<EMAIL_ADDRESS>

@LKHN Are you suggesting to add systemd package to minimal? When I try build your changes it results total 165 packages in container. Can you please add your build log?

@LKHN Are you suggesting to add systemd package to minimal? When I try build your changes it results total 165 packages in container. Can you please add your build log?

I separated the PRs for simplicity :) please, merge PR#18 and build again.

Current almalinux:minimal in dockerhub uses libcurl-minimal current changes replaces it with libcurl to be compatable with redhat/ubi8-minimal. Number of packages goes from 96 to 101. Packages added are
brotli
libpsl
libssh
libssh-config
publicsuffix-list-dafsa


Happy to take over maintenance

I only just noticed the "Maintainer Needed" notice in the readme.
Since I use both Emacs and Mocha on an almost-daily basis, maintaining this package shouldn't be a hassle for me. 😉 Hell, my setup even contains Mocha-specific additions that are better off being added to this rather than my ~/.emacs.d folder…

Sounds good. I made you a contributor, so you should now be able to commit directly and merge PRs.
Not sure the best way to transfer ownership or give you gull admin of the repository -- maybe that would be just transferring it to your github account, or making an org for it and moving it there?

It's up to you if you'd like to transfer full ownership or not. Simply having write/push privileges is enough for me. 👍


blur effect or dim screen effect when show dialog

hello. Thanks you for great library.
I think it would be better when implment blur effect or dim screen effect when show dialog

Hi, there's a property named backgroundViewColor that you can use to set the background color for the view behind the picker. I'm also working on making it possible to create the picker as a separate view - when it's done you can use it and create your own blur effect if necessary.


FIx the return type of config.getConfigVarSchema()

Change-type: patch
Signed-off-by: Thodoris Greasidis<EMAIL_ADDRESS>
Resolves: #
HQ:
See:
Depends-on:
Change-type: major|minor|patch

Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

@balena-ci I self-certify!


Error in SimonSaysArea Tests, toModel test.

This test fails for two reasons:
The test currently includes an additional parameter that does not exist on the SimonSays type (occupantsByID) but it also cannot succeed yet since we are still learning how to use a JSON file for data storage and retrieval.
A temporary band-aid fix will be applied because the function toModel() itself is correct, but a dependency (JSON read/write) is causing the failure.

Fix provided in 6871a72 but the issue shall remain open as a reminder to come back and clean up as soon as we get the JSON read/write working as intended.

I created a new label: Technical debt for this and similar issues.
To keep track of things we should change/improve .


Make observing LiveData more readable

I do not like how LiveData is observed and I've also found a lot of people using this as lifecycleOwner in fragments.
I did these extension functions to solve the first and reduce the likelihood of the second.
If you like them the same mechanism could be applied to your EventSource.observe.

The idea of hiding the viewLifecycleOwner is sound, but I do not like the approach.
I have 1 nitpick and 1 real concern about it.
/**
 * Utils to observe liveData in a cleaner way from any LifecycleOwner.
 *
 * This should not be used with Fragment as lifecycleOwner
 */
inline fun <T> LifecycleOwner.observeLifeCycle(
    liveData: LiveData<T>,
    crossinline observer: (T) -> Unit
) {
    liveData.observe(this, Observer { observer(it) })
}

Nitpick: it's called Lifecycle in the lifecycle api, not LifeCycle, this can easily throw off users of the code.
Real issue: if this shouldn't be called on Fragment, then I should not have an extension function that makes it possible (in this case, LifecycleOwner.observe()).
Adding 1 extension for a AppCompatActivity, and 1 extension for a Fragment is sufficient, and more importantly safer to use.

This also shows that I should remove observeOnce() because it is not actually used, it is left-over from a previous attempt when I was trying to initialize the ViewModel from onCreate() (without luck, initially moving it to onCreateView, then through lazy access to onViewCreated).

I'm slightly conflicted about adding this though because making the viewLifecycleOwner explicit in the sample might help people know that that's what they should be using, especially considering that even the Google Dev Advocates tend to screw this up.

True that it should not be LifeCycle, my bad.
In my case I have the comments mainly to educate about using the viewLifecycleOwner, and I left them to not loose that in case someone didn't realize why there are separate methods for fragment and activity. They could be removed if not helpful, or changed for a link to a good resource about the fragment lifecycle.
The LifecycleOwner extension could be made private. I left it for the rare occasions where you are not in a fragment or activity and you have a lifecycleOwner to use. But in those cases it is probably better to use the official api and reduce the risk of people using the wrong lifecycle in fragment.
And I saw you were not using the observeOnce. I thought you left it to use soon so I added the equivalent (I sometimes use it myself). If it won't be used it can be removed.
About what is better, whether to leave the viewLifecycleOwner to be more explicit or have it in a utils, it's hard to tell. In my case I preferred the second one but I also had doubts about that... My reasoning was that people who care about details would notice it with or without the extension, and people who do not would not research it enough so they would probably forget about it. Therefore, I preferred readability, hiding slightly lower level details, and making mistakes easier to detect in a code review (I think).
I fixed the nitpick and real issue. Whether it should be added or not is up to you :) Maybe I gave you more trouble than help with this PR

This won't be merged. Thanks for the PR though 😊

This won't be merged. Thanks for the PR though 😊

inline fun <T> LifecycleOwner.observe(     liveData: LiveData<T>,     crossinline observer: T.() -> Unit ) = liveData.observe(this, Observer { observer(it) })
is it correct


Development

Added project channel check when new projects added, the bot automatically creates a channel under the category.

Plues .env settings


How to make cdb use a secure symbol server with authentication?

Answered: https://stackoverflow.com/questions/79114343/provide-symbol-server-auth-to-windows-debugger/79116006#79116006


BUG: various errors in weight calculations in orthogonal.py

Some of these bugs came with gh-3941, some (c_roots, s_roots at least) predate that work.
Fixes gh-4683.

Can you modify verify_gauss_quad so that it calculates integral for some test function using the roots and weights, and compares the result to integrate.quad.
This probably catches the normalization error in the he_root weights, which does not seem to be currently detected because the result is normalized...

can you mark this as a possible backport candidate to fix 0.15.x ?
I have gaussian quadrature code using he_roots  and h_roots that never made it into statsmodels. We might have a GSOC project that needs gaussian quadrature weights, and it would be nicer to have correct he_roots across scipy versions.
(We can use h_roots instead of he_roots in statsmodels, now that we know.)

@pv, requested test added.

This needs rebased since #4587 was merged.  @pv, do you have any thoughts on what to do here? The two commits here ought to back port cleanly, but if I rebase them on the current master branch they won't.

Probably easiest to open a separate pull request against the 0.15.x branch.

Rebased.

PR against maintenance/0.15.x here: #4700

This has had a month to cook.  I'm going to merge it now.  This is a gotta-have for 0.16.

Thanks @ewmoore.
Anyone have an objection to hitting the green button on the 0.15.x backport of this PR now?


Adding Gen2 VM support for SR-IOV tests

New mlx5_core depends on mlx5_ib not mlx5_en. It depends on the ib device availability also.
Fixed the VF name fetch steps with another way.
TEST RESULTS
[LISAv2 Test Results Summary] with Gen2 VM
Test Run On           : 10/25/2019 17:34:56
VHD Under Test        : XXXXXXXXXXXXXXXXXXXXXXRHEL76-GEN2-1014-Upgrade.vhd
Initial Kernel Version: 3.10.0-957.36.1.el7.x86_64
Final Kernel Version  : 3.10.0-957.36.1.el7.x86_64
Total Test Cases      : 3 (3 Passed, 0 Failed, 0 Aborted, 0 Skipped)
Total Time (dd:hh:mm) : 0:0:30

   ID TestArea             TestCaseName                                                                TestResult TestDuration(in minutes)
-------------------------------------------------------------------------------------------------------------------------------------------
    1 SRIOV                SRIOV-DISABLEVF-ON-GUEST                                                          PASS                  5.5
    2 SRIOV                SRIOV-RELOAD-MODULE                                                               PASS                 5.66
    3 SRIOV                SRIOV-DISABLE-ENABLE-PCI                                                          PASS                 7.12

Logs can be found at C:\LISAv2\ZR88\TestResults\2019-25-10-10-34-47-5363

[LISAv2 Test Results Summary] with Gen1 VM
Test Run On           : 10/25/2019 18:21:08
ARM Image Under Test  : RedHat : RHEL : 7.6 : latest
Initial Kernel Version: 3.10.0-957.21.3.el7.x86_64
Final Kernel Version  : 3.10.0-957.21.3.el7.x86_64
Total Test Cases      : 3 (3 Passed, 0 Failed, 0 Aborted, 0 Skipped)
Total Time (dd:hh:mm) : 0:0:19

   ID TestArea             TestCaseName                                                                TestResult TestDuration(in minutes)
-------------------------------------------------------------------------------------------------------------------------------------------
    1 SRIOV                SRIOV-DISABLEVF-ON-GUEST                                                          PASS                 3.07
    2 SRIOV                SRIOV-RELOAD-MODULE                                                               PASS                 2.83
    3 SRIOV                SRIOV-DISABLE-ENABLE-PCI                                                          PASS                  7.3

Logs can be found at C:\LISAv2\RZ83\TestResults\2019-25-10-11-21-03-5834

@juhlee-microsoft Looks good, could you please merge all the commits into one?
Refer [here] (https://www.freecodecamp.org/forum/t/how-to-squash-multiple-commits-into-one-with-git-squash/13231
)

@juhlee-microsoft Looks good, could you please merge all the commits into one?
Refer [here] (https://www.freecodecamp.org/forum/t/how-to-squash-multiple-commits-into-one-with-git-squash/13231
)

Done.


RLP v2 CRD

Fixes #179
From RFC 0001

Group
Version
Kind

kuadrant.io
v2beta1
RateLimitPolicy

Example with all fields:
apiVersion: kuadrant.io/v2beta1
kind: RateLimitPolicy
metadata: {}
spec:
  targetRef:
    group: gateway.networking.k8s.io
    kind: HTTPRoute
    name: <HTTPROUTE-NAME>
  limits:
    <limit-name>:
      routeSelectors:
      - hostnames:
         - "*example.com"
        matches:                # []HTTPRouteMatch
        - path:                      # HTTPPathMatch
            type: PathPrefix
            value: "/toys"
          method: GET          # HTTPMethod
          headers:                # []HTTPHeaderMatch
          - type: Exact
            name: "X-Canary"
            value: "1"
          queryParams:      # []HTTPQueryParamMatch
          - type: Exact
            name: "q"
            value: "1"
      rates:
      - limit: 50
        duration: 1
        unit: minute
      counters:
      - auth.identity.username
      - context.request.http.path
      when:
      - selector: auth.identity.group
        operator: neq
        value: admin

this PR targets https://github.com/Kuadrant/kuadrant-operator/tree/rlp-v2-base

ready for a new review @guicassolato


Dark theme

Description

As a student,
I want the ability to switch between light and dark themes,
So that I can maintain a cohesive experience with other websites and applications on my device.

See https://css-tricks.com/easy-dark-mode-and-multiple-color-themes-in-react/
Requirements

Scenario 1: No theme preference

Given I have no light/dark theme preference set in my browser,
When I visit the site,
Then the site displays in light theme.

Scenario 2: Light theme preference

Given I have my browser set to prefer a light theme,
When I visit the site,
Then the site displays in light theme.

Scenario 3: Dark theme preference

Given I have my browser set to prefer a dark theme,
When I visit the site,
Then the site displays in dark theme.

Scenario 4: Change light to dark

Given the site is displayed in light theme,
When I click/tap the theme control switch,
Then the site displays in dark theme.

Scenario 5: Return to the site

Given the site was in dark theme the last time I was there,
When I visit the site again,
Then the site displays in dark theme.

Tasks

TODO

Hey i would like to work on this. I give user multiple color theme option if you would allow me to :)


Allow availability items with only deprecated versions.

Bug/issue #, if applicable: 138034711
Summary
Symbols in the symbol graph can specify a deprecated availability version without an introduced version. Currently, DocC excludes these availability items entirely. This change allows that items without an introduced version but with a deprecated version are retained instead of being filtered out.
This adjustment is necessary because the @available attribute allows defining only a deprecated version without an introduced version, a scenario that is not currently reflected in the final documentation.
Dependencies
Describe any dependencies this PR might have, such as an associated branch in another repository.
Testing
Availability.docc.zip
Steps:
1.Preview the attached doc catalog.
2.Assert that the platform iOS is showing as deprecated in the symbol Foo.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[X] Added tests
[X] Ran the ./bin/test script and it succeeded
[ ] Updated documentation if necessary

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test


Adds the ability to specify the appserver host address by an environment variable.

Checks the DEV_APPSERVER_DEFAULT_ADDRESS environment variable for a default appserver address.

Addressed comments.

Cool, I'll merge it when I'm back in the office

this will go out in the next release.


Implementation of DOM tree diffing (fixes ListView, et. al.)

The original problem is that ListView updates just didn't work: the ordering was all wrong after scrolling around because we didn't get any updates from ViewGroup.OnHierarchyChangedListener.
We've known for awhile that we need to have some kind of support for DOM tree diffing because every location in the Android UI tree seems to require a different protocol for subscribing to change notifications (if it's even possible), some of which have require brittle, reflection-based hacks (yuck).
DOM tree diffing necessitates the use of a "shadow DOM" which contains a copy of the DOM so can keep track of what we've sent to Chrome and compare it to what the DOMProvider says the tree should look like now.
DOM tree diffing is split into 3 phases:

Collecting changes to the shadow DOM. We currently brute-force this in two ways:

AndroidDOMProvider does not currently have the ability to detect when (or where!) changes are made to the Android UI tree. Instead, it has a timer that fires every second and it just announces to its DOMProvider.Listener that it "changed."
Since DOMProvider.Listener.onChanged() takes no parameters, this means that DOM's only option is to read the entire tree from mDOMProvider. Later on we could make it so onChange() takes more information about what changed.

Sending the changes over to Chrome. This is accomplished in DOM.applyShadowDOMUpdate() with the participation of a few other friends.
Merging the changes back into the shadow DOM. This is implemented in ShadowDOM.Update.commit().

While the first phase is brute-forced and takes linear time with respect to the number of elements in the tree, the other two phases are not brute-forced. Their time and space requirements should be roughly proportional to the number of elements that have changed. Disconnected elements are included in that equation: if 1 element is removed but it has a sub-tree of 50 elements, then all 51 of those elements must be considered.
Closes #198

The algorithm looks solid to me though I'll need to spend a bit more time reading through the details to make sure I actually understand it :)

Okay there's another round of changes. Deleted a bunch of code, added some significant comments explaining some important stuff, ...

I'm fine to punt on the more philosophical issues (adding classes and the like), but please do take a pass through the feedback.  Otherwise this looks good to me and I see no major reason we can't land it.  Just remember to squash and merge when you're ready.


Break blocks in gamemode 0 with protection

If a player is in a safe zone (with protection where he cannot break) but tries to break, even if he does not give him the block, he adds money.
Obviously this is a bad thing, I don't know if it can be fixed

What do you mean?
I think you may need to read here

I sent u a video how its work

https://gofile.io/d/RrIUK3

Are you sure about the PurrPerms Groups permissions?
You need to add the permission to join jobs and earn money by jobs only in the specified world
You can read here

Are you sure about the PurrPerms Groups permissions?
You need to add the permission to join jobs and earn money by jobs only in the specified world
You can read here

Just check in the Event if the BlockBreakEvent is cancelled

Guest at all worlds

jobui.earn.break
jobui.job.*
jobui.job.asesino
jobui.job.cazador
jobui.job.excavador
jobui.job.minero
jobui.job.talador
jobui.progress.break

Oh
You need to give the permissions only in a specified world(or how many you want) then they won't be able to earn in every world

Just check in the Event if the BlockBreakEvent is cancelled

I really don't understand what do you mean...

The problem is, that world is for Survival, so the spawn is what is protected so that they do not break it and die or do pvp

So, if you can put something so that that does not happen, it would be fine, in plan, that when breaking the object record that it is broken, and then when picking it up confirm the payment

I'm really sorry for that
You can't disable earning in a zone in a world
You can disable it only in a world not in a zone
But I suggest you creating another world for Regenerating Mine
That would be better...


Administrate Access to Admin Tools

This request is related to this article:
https://easyengine.io/docs/admin-tools-22222/
Currently there is one HTTP Auth user added to htpasswd for access to any of the administrative tools in any directory under Port 22222.
This is unfortunate, because for example VimbAdmin should be accessible without giving also permissions to access any of the other admin tools.
VimbAdmin allows users to change their mailbox passwords. It allows to setup administrators for each Domain pointing to the server. So they can setup mailboxes and alias-emailaddresses for domains they have been assigned to.
Currently, even if I set up an additional htpasswd user and password combination for Vimbadmin Access, those users could if they wanted, also access phpmyadmin and all the other admin tools, if they navigated to server.com:22222 instead of server.com:22222/vimbadmin
Request:
Please improve this :)

We will keep this for V4 later releases. Let's see how does this fits naturally once the first v4 goes out! 🤞

Being handled in https://github.com/EasyEngine/easyengine/issues/1119


Logging in using superuser raise django.core.urlresolvers.NoReverseMatch Exception

When trying to login to the regular site (not the admin) using the superuser account i get this error:
NoReverseMatch: Reverse for 'show' with arguments '('',)' and keyword arguments '{}' not found. 1 pattern(s) tried: [u'users/(?P[\w\-]+)$']

Found the issue,
when running migrate the profiles tables were not created causing an error when creating a new super user
django.db.utils.ProgrammingError: (1146, "Table 'test.profiles_profile' doesn't exist")
Solution was to run from the src directory:
python manage.py makemigrations
and then run migrate again:
python manage.py migrate
then create another superuser and everything should be ok.
You can delete the old user and recreate it.

The underlying problem is that slug attribute for user profile is not created, if migrations were executed incorrectly, as described above.
The solution in edge could be somethin like this in src/profiles/templates/profiles/show_profile.html
{% if show_user.profile.slug %}

        Permalink {{ show_user.profile.get_absolute_url }}
{% else %}
            No permalink. See this issue.
{% endif %}

Thanks to @slmjy we now have a error message which points to this issue.


Support for joins

Mirrors https://github.com/firebase/FirebaseUI-Android/pull/276.
Still missing a few things:

[x] Documentation
[ ] Actual error handling
[ ] Data source tests, tests for edge cases

cc @puf, @ianbarber

Seems like github is bugging out and not updating this PR correctly.

@ianbarber ready for more review