docs: update coding-standard links

Description

update coding-standard repo links

Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[x] User guide updated
[ ] Conforms to style guide

Thank you, @kenjis


Fix for Issue #352

Added in fix for issue #352

This change is

@shrutir25,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@acs-bot test this please

@anhowe could you take a look at this


Basic blocks starting with "jmp" instruction cause errors in REIL cfg

Hi,
I encountered a bug during the creation of the REIL cfg.
When a basic block starts with a "jmp" instruction followed by other instructions, BinNavi creates two REIL basic blocks; yet it fails to create the appropriate edge between them.
As minimal example, the following code :
; nasm -f elf test.asm
; ld -s -o test test.o
section     .text
global      _start

_start:
    jmp label
label:
    ret

Creates this x86 basic block:

Which is translated into two REIL basic blocks:

However, the edge father -> son is missing.
Best,
Josselin
(tested on version 356eaa8)

The BinNavi project is no longer under active development.


Remove FileToInMemoryDataset

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (ddac177) 100.00% compared to head (44245e0) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #282   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          318       318
  Lines         9892      9888    -4
  Branches       952       951    -1
=========================================
- Hits          9892      9888    -4

Impacted Files
Coverage Δ

src/gravitorch/data/datasets/__init__.py
100.00% <100.00%> (ø)

src/gravitorch/data/datasets/in_memory.py
100.00% <100.00%> (ø)

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


Container Size Issue

I changed the background-color and noticed the container isn't large enough to contain the buttons on the right side.
Here is my lazy solution:
File: styles.css
.container {
max-width: 960px;
padding-right: 40px;
}

better use 35px. Fixed in #177


Set up presubmit checks

Done criteria: pull requests against main must have presubmits passing with tests run. Should be done for both CityOfDetroit/affordable-housing-app and CityOfDetroit/bloom.

I decided to use this one: https://github.com/marketplace/actions/lint-action since it seemed more popular.

I merged the lint presubmit so we shouldn't hit those errors anymore. I have an upstream PR to merge Prettier fixes. Once that happens we can do pull that commit in and then we won't have to worry about style or lint errors.

Submitted a Travis config file for running ui-components tests and building sites/public and sites/partners. We should add more tests to this as we go, but at least we won't break the build by submitting PRs.

I'm closing this task as done, the remaining tests to enable are currently broken and that work is tracked in #171


Move Reliable Distributed Systems to Advanced Systems

See Issue 459

Thanks, that's a better place for it.


add redux-token-auth to README.md

Hey @FunkyloverOne ! Just curious how this project differs from https://github.com/lynndylanhurley/redux-auth ? Going to merge this though as I see no reason not to.

Hey, @zachfeldman, to be honest - I don't know.
I just was looking for something for react in your readme, and there was only React + jToker, which doesn't seem fresh :) So I have found this project, and it has more recent activity, seems to be more up to date, hovever doesn't look popular, I didn't pay my attention at that moment...
Who knows, maybe it's a beginning of something awesome? :)
But, if you're afraid of that project - change the link to another one, my intention for this PR was to add some more recent info about react solutions to your readme.

@FunkyloverOne fair enough! I appreciate the contribution :) I hope it's the beginning of something awesome for sure!


BasicDataSource - remove obsolete inner-class reference

remove reference of PaGetConnection during initializing secure context

Unfortunately I'm unable to provide a test case where I enable and disable the SecurityManager.

I'm not sure this is the correct fix. PaGetConnection was provided explicitly to avoid problems when using a SecurityManager. I want to convince myself that removing that inner class hasn't re-introduced those problems first.

Fix confirmed. lambda expressions don't create anonymous inner classes so the need to pre-load the anonymous inner class goes away. I'll merge this shortly.


Migrate to "new" GitHub Action

GitHub announce "new" GitHub Actions. However, sadly, this new one does not have compatibility with the old (early beta) one.
By the document, we need add action.yml as a new metadata. We may need more change to migrate to a new format. But I'm not sure about it.
See also
https://github.com/actions/container-template

Good catch, sane-san!
Now I don't have the environment to test github actions (my github account is still in wait list of beta program). I'll report this issue to the member of abema organization.


Sensors open with delayed alarm

In my configuration I set the alarm via keypad with a 10 second delay to be able to leave the house. After 10 seconds the alarm is activated. However, if I have an open sensor, I am only notified when Alarmo tries to activate the alarm when I have already left the house. Wouldn't it be possible in these cases (when an activation delay is chosen) to immediately check the sensors so as to immediately notify the problem?

You can configure per sensor if it is allowed to be active during the exit delay or not.
Depending on the setting, arming would fail immediately or after the delay ends.


[8.0] Accept callable and listener class names as batch handler functions

Batch is using closures for then, catch, finally handlers. Because I was afraid serializing closures might have some limitations, I thought it is not a bad idea to allow to users the feature without closure serializations.
This PR is trying to let users to use php-serializable callables  as well as listener classes for handlers:

Bus::batch([
    new ProcessPodcast(Podcast::find(1)),
    new ProcessPodcast(Podcast::find(2)),
    new ProcessPodcast(Podcast::find(3)),
])->then(SomeListener::class)
->catch([SomeClass::class, 'staticFunction'])
->finally('someFunctionName')
->dispatch();

class SomeListener
{
    function __construct(Batch $batch) {
        //...
    }
    function handle() {
        //...
    }
}
class SomeClass {
    static function staticFunction(Batch $batch, Throwable $e) {
        //
    }
}
someFunctionName(Batch $batch, Thrwoable $e = null) {
    //...
}

Let me know if you agree with this.

I think that's probably fine. Do you know why tests are failing?

No. But I will take a look at it soon.

This invoke also works:
    protected function invoke($handler, Batch $batch, \Throwable $e = null)
    {
        if (is_callable($handler)) {
            call_user_func($handler, $batch, $e);

            return;
        }
        dispatch(new $handler($batch, $e));
    }

Hey @halaei. Can you pay attention to our coding standards with the next prs? We use FQN, extra spaces in DocBlocks and white lines after if statements. I've fixed these for you now.

@driesvints Sure thanks.

We use FQN, extra spaces in DocBlocks and white lines after if statements.

FTR, there is https://laravel.com/docs/7.x/contributions#coding-style but except showing how code "looks", it doesn't "describe" it.
I think especially the FQN is easy to miss as this is, compared to a lot of OSS PHP code I read, rather the exception as most are using imports.
Just wanted to mention it 😄

@mfn I think if you scan the rest of the code in a file it's pretty obvious tbh. But feel free to PR something to the docs if you feel anything could be clarified 👍


⚠ Cioos Pacific ERDDAP has degraded performance

In 0b1b53c, Cioos Pacific ERDDAP (https://data.cioospacific.ca/erddap/index.html) experienced degraded performance:

HTTP code: 200
Response time: 15342 ms

Resolved: Cioos Pacific ERDDAP performance has improved in 411fd27 after 5 minutes.


Enhancement: support native SQS messages in listener argument

The @SqsListener annotation should support com.amazonaws.services.sqs.model.Message as method parameter.
We want to store incoming events in DynamoDB including payload and headers, and the conversions that are done in QueueMessageUtils are making this considerably harder (String to MimeType / UUID etc.). Also we want to store events just like they come in, without any conversions or processing so there are less chances for bugs.

Let me guys discover this one
@maciejwalkowiak

Go for it @jarc0der !


0x2700 - X509 - Certificate verification failed, e.g. CRL, CA or signature check failed

ssl client connect error:
./ssl_client2 server_name=openapi.baidu.com server_port=443 ca_file=certdata.txt  debug_level=1

certdata.txt
-----BEGIN CERTIFICATE-----
MIIE0zCCA7ugAwIBAgIQGNrRniZ96LtKIVjNzGs7SjANBgkqhkiG9w0BAQUFADCByjELMAkGA1UE
BhMCVVMxFzAVBgNVBAoTDlZlcmlTaWduLCBJbmMuMR8wHQYDVQQLExZWZXJpU2lnbiBUcnVzdCBO
ZXR3b3JrMTowOAYDVQQLEzEoYykgMjAwNiBWZXJpU2lnbiwgSW5jLiAtIEZvciBhdXRob3JpemVk
IHVzZSBvbmx5MUUwQwYDVQQDEzxWZXJpU2lnbiBDbGFzcyAzIFB1YmxpYyBQcmltYXJ5IENlcnRp
ZmljYXRpb24gQXV0aG9yaXR5IC0gRzUwHhcNMDYxMTA4MDAwMDAwWhcNMzYwNzE2MjM1OTU5WjCB
yjELMAkGA1UEBhMCVVMxFzAVBgNVBAoTDlZlcmlTaWduLCBJbmMuMR8wHQYDVQQLExZWZXJpU2ln
biBUcnVzdCBOZXR3b3JrMTowOAYDVQQLEzEoYykgMjAwNiBWZXJpU2lnbiwgSW5jLiAtIEZvciBh
dXRob3JpemVkIHVzZSBvbmx5MUUwQwYDVQQDEzxWZXJpU2lnbiBDbGFzcyAzIFB1YmxpYyBQcmlt
YXJ5IENlcnRpZmljYXRpb24gQXV0aG9yaXR5IC0gRzUwggEiMA0GCSqGSIb3DQEBAQUAA4IBDwAw
ggEKAoIBAQCvJAgIKXo1nmAMqudLO07cfLw8RRy7K+D+KQL5VwijZIUVJ/XxrcgxiV0i6CqqpkKz
j/i5Vbext0uz/o9+B1fs70PbZmIVYc9gDaTY3vjgw2IIPVQT60nKWVSFJuUrjxuf6/WhkcIzSdhD
Y2pSS9KP6HBRTdGJaXvHcPaz3BJ023tdS1bTlr8Vd6Gw9KIl8q8ckmcY5fQGBO+QueQA5N06tRn/
Arr0PO7gi+s3i+z016zy9vA9r911kTMZHRxAy3QkGSGT2RT+rCpSx4/VBEnkjWNHiDxpg8v+R70r
fk/Fla4OndTRQ8Bnc+MUCH7lP59zuDMKz10/NIeWiu5T6CUVAgMBAAGjgbIwga8wDwYDVR0TAQH/
BAUwAwEB/zAOBgNVHQ8BAf8EBAMCAQYwbQYIKwYBBQUHAQwEYTBfoV2gWzBZMFcwVRYJaW1hZ2Uv
Z2lmMCEwHzAHBgUrDgMCGgQUj+XTGoasjY5rw8+AatRIGCx7GS4wJRYjaHR0cDovL2xvZ28udmVy
aXNpZ24uY29tL3ZzbG9nby5naWYwHQYDVR0OBBYEFH/TZafC3ey78DAJ80M5+gKvMzEzMA0GCSqG
SIb3DQEBBQUAA4IBAQCTJEowX2LP2BqYLz3q3JktvXf2pXkiOOzEp6B4Eq1iDkVwZMXnl2YtmAl+
X6/WzChl8gGqCBpH3vn5fJJaCGkgDdk+bW48DW7Y5gaRQBi5+MHt39tBquCWIMnNZBU4gcmU7qKE
KQsTb47bDN0lAtukixlE0kF6BWlKWE9gyn6CagsCqiUXObXbf+eEZSqVir2G3l6BFoMtEMze/aiC
Km0oHw0LxOXnGiYZ4fQRbxC1lfznQgUy286dUV4otp6F01vvpX1FQHKOtw5rDgb7MzVIcbidJ4vE
ZV8NhnacRHr2lVz2XTIIM6RUthg/aFzyQkqFOFSDX9HoLPKsEdao7WNq
-----END CERTIFICATE-----

result:
  . Seeding the random number generator... ok
  . Loading the CA root certificate ... ok (0 skipped)
  . Loading the client cert. and key... ok
  . Connecting to tcp/openapi.baidu.com/443 ... ok
  . Setting up the SSL/TLS structure... ok
  . Performing the SSL/TLS handshake...
Verify requested for (Depth 2):
cert. version     : 3
serial number     : 25:0C:E8:E0:30:61:2E:9F:2B:89:F7:05:4D:7C:F8:FD
issuer name       : C=US, O=VeriSign, Inc., OU=Class 3 Public Primary Certification Authority
subject name      : C=US, O=VeriSign, Inc., OU=VeriSign Trust Network, OU=(c) 2006 VeriSign, Inc. - For authorized use only, CN=VeriSign Class 3 Public Primary Certification Authority - G5
issued  on        : 2006-11-08 00:00:00
expires on        : 2021-11-07 23:59:59
signed using      : RSA with SHA1
RSA key size      : 2048 bits
basic constraints : CA=true
key usage         : Key Cert Sign, CRL Sign
ext key usage     : TLS Web Server Authentication, TLS Web Client Authentication, Code Signing, ???, ???
  ! self-signed or not signed by a trusted CA

Verify requested for (Depth 1):
cert. version     : 3
serial number     : 64:1B:E8:20:CE:02:08:13:F3:2D:4D:2D:95:D6:7E:67
issuer name       : C=US, O=VeriSign, Inc., OU=VeriSign Trust Network, OU=(c) 2006 VeriSign, Inc. - For authorized use only, CN=VeriSign Class 3 Public Primary Certification Authority - G5
subject name      : C=US, O=VeriSign, Inc., OU=VeriSign Trust Network, OU=Terms of use at https://www.verisign.com/rpa (c)10, CN=VeriSign Class 3 International Server CA - G3
issued  on        : 2010-02-08 00:00:00
expires on        : 2020-02-07 23:59:59
signed using      : RSA with SHA1
RSA key size      : 2048 bits
basic constraints : CA=true, max_pathlen=0
subject alt name  :
key usage         : Key Cert Sign, CRL Sign
ext key usage     : TLS Web Server Authentication, TLS Web Client Authentication, ???, ???
  This certificate has no flags

Verify requested for (Depth 0):
cert. version     : 3
serial number     : 2D:73:DA:EE:3D:44:A1:DD:F6:65:13:18:39:20:58:00
issuer name       : C=US, O=VeriSign, Inc., OU=VeriSign Trust Network, OU=Terms of use at https://www.verisign.com/rpa (c)10, CN=VeriSign Class 3 International Server CA - G3
subject name      : C=CN, ST=Beijing, L=Beijing, O=BeiJing Baidu Netcom Science Technology Co., Ltd, OU=service operation department, OU=Organization and domain(s) authenticated by iTrus China, CN=openapi.baidu.com
issued  on        : 2014-10-31 00:00:00
expires on        : 2015-11-02 23:59:59
signed using      : RSA with SHA1
RSA key size      : 2048 bits
basic constraints : CA=false
subject alt name  : openapi.baidu.com
key usage         : Digital Signature, Key Encipherment
ext key usage     : TLS Web Server Authentication, TLS Web Client Authentication, ???
  This certificate has no flags
 failed
  ! ssl_handshake returned -0x2700
    Unable to verify the server's certificate. Either it is invalid,
    or you didn't set ca_file or ca_path to an appropriate value.
    Alternatively, you may want to use auth_mode=optional for testing purposes.

Last error was: -0x2700 - X509 - Certificate verification failed, e.g. CRL, CA or signature check failed

@pjbakker whats 'top' ? Didn't understand what you meant.

@FarhanAhmad A certificate chain runs all the way from a child certificate to the 'top' (The CA certificate).
In case of the issue above, the CA Chain provided to the application contained the certificate up to (but not including) the 'top' certificate. Without that top certificate, the chain could not be verified in full.
That said. I believe the behaviour of verification has been modified in more recent version, so you might now be able to provide a 'middle' certificate as trusted instead. Maybe @sbutcher-arm can comment on that?

@pjbakker I am facing the similar error('-9984 - X509 - Certificate verification failed, e.g. CRL, CA or signature check failed') at 'mbedtls_ssl_handshake' call.
I am new to mbed tls, i dont know much fundamentals of it. Following is my scenario, please tell me if i am doing anything wrong.

I have my own ca.cert, server.cert, client.cert, server.key,client.key and ca.key files.
Following is my piece of code at server side and client side:

server partial code:
``
int ret = mbedtls_x509_crt_parse_file(&cacert, ca_file);
if (ret != 0) {
mbedtls_printf( " failed\n  !  mbedtls_x509_crt_parse_file returned %d\n\n", ret );
goto exit;
}
ret = mbedtls_x509_crt_parse_file(&srvcert, server_file);
if (ret != 0) {
    mbedtls_printf( " failed\n  !  mbedtls_x509_crt_parse_file returned %d\n\n", ret );
    goto exit;
}

ret = mbedtls_pk_parse_keyfile(&pkey,serverkey_file,password);
if (ret != 0) {
    mbedtls_printf( " failed\n  !  mbedtls_pk_parse_keyfile server returned %d\n\n", ret );
    goto exit;
}

..............
..............
**mbedtls_ssl_conf_ca_chain( &conf, &cacert, NULL );
if( ( ret = mbedtls_ssl_conf_own_cert( &conf, &srvcert, &pkey ) ) != 0 )
{
    mbedtls_printf( " failed\n  ! mbedtls_ssl_conf_own_cert returned %d\n\n", ret );
    goto exit;
}**

if( ( ret = mbedtls_ssl_setup( &ssl, &conf ) ) != 0 )
{
    mbedtls_printf( " failed\n  ! mbedtls_ssl_setup returned %d\n\n", ret );
    goto exit;
}
........................
.....................
// Performing the SSL/TLS handshake...
**while( ( ret = mbedtls_ssl_handshake( &ssl ) ) != 0 )
{
    if( ret != MBEDTLS_ERR_SSL_WANT_READ && ret != MBEDTLS_ERR_SSL_WANT_WRITE )
    {
        mbedtls_printf( " failed\n  ! mbedtls_ssl_handshake returned %d\n\n", ret );
        goto reset;
    }
}**

``
Client side code:
``
ret = mbedtls_x509_crt_parse_file(&cd->ca_cert, ca_cert_filename);
if (ret != 0) {
mbedtls_printf( " failed\n  !  mbedtls_x509_crt_parse_file of ca returned %d\n\n", ret );
exit(ret);
}
ret = mbedtls_x509_crt_parse_file(&cd->client_cert, client_cert_filename);
	if (ret != 0) {
    	mbedtls_printf( " failed\n  !  mbedtls_x509_crt_parse_file of client returned %d\n\n", ret );
    	exit(ret);
	}

ret = mbedtls_pk_parse_keyfile(&cd->client_pkey,client_key_filename,password);
if (ret != 0) {
	mbedtls_printf( " failed\n  !  mbedtls_pk_parse_keyfile client returned %d\n\n", ret );
	exit(ret);
}

   mbedtls_ssl_conf_authmode( &cd->conf, MBEDTLS_SSL_VERIFY_REQUIRED );

   **mbedtls_ssl_conf_ca_chain( &cd->conf, &cd->ca_cert, NULL  );
if( ( ret = mbedtls_ssl_conf_own_cert( &cd->conf, &cd->client_cert, &cd->client_pkey ) ) != 0 )
    {
        mbedtls_printf( " failed\n  ! mbedtls_ssl_conf_own_cert returned %d\n\n", ret );
    exit(ret);
    }**
	......................

	if( ( ret = mbedtls_ssl_setup( &cd->ssl, &cd->conf ) ) != 0 )
	{
    	mbedtls_printf( " failed\n  ! mbedtls_ssl_setup returned %d\n\n", ret );
    	freeClientTLSData(ret,cd);
	exit(ret);
	}

	**while( ( ret = mbedtls_ssl_handshake( &cd->ssl ) ) != 0 )
	{
    	if( ret != MBEDTLS_ERR_SSL_WANT_READ && ret != MBEDTLS_ERR_SSL_WANT_WRITE )
    	{
        		mbedtls_printf( " failed\n  ! mbedtls_ssl_handshake returned -0x%x\n\n", -ret );
        		freeClientTLSData(ret,cd);
		exit(ret);
    	}
	}**

``
Please let me if i am doing anything wrong at 'mbedtls_ssl_conf_ca_chain'  , 'mbedtls_ssl_conf_own_cert' calls order or if any.
And also which certificates(ca, client, server certs) are mainly required at both server and client side code.
Thanks in Advance.

My issue is resolved. I am facing it because of certificates I am using are invalid

@PravallikaKG yes, because you haven't self-signed them.


Service Tags for Windows Updates (WU) and RedHat Update Infrastructure (RHUI) suggestion

I have been using Azure Automation's Update Management for VMs that are internet facing without issue until I am required to use it on VMs that are non-internet facing (intranet) environment where I'm stumble into a lot of NSG configuration complexity.
Any chance of having these Service Tags?

AzurePlatformWU

For an example having this Service Tag created for NSG in order for consumer to configure Windows VM resources to utilise Azure Automation's Update Management feature, and allow Windows VMs to receive Windows Updates securely.
The current problem is that Windows Updates can be distributed through multiple Windows Update URL endpoints that is almost impossible to be mapped for consumer.

AzurePlatformRHUI

For an example having this Service Tag created for NSG in order for consumer to configure RedHat VM resources  to utilise Azure Automation's Update Management feature, and allow RedHat VMs to receive packages securely.
The current problem is that RedHat packages can be distributed through multiple content delivery servers that are managed and owned by Microsoft which may subject to changes in the future based on capacity requirement.
These suggested Service Tags will also enhance the cloud shared responsibility model and improve security posture for VMs connected to VNet Subnets.

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

ID: 08217244-82bb-13b5-46f7-454801d8f865
Version Independent ID: 4a8b0a32-bf57-7544-c60d-6b7f250c4477
Content: Azure service tags overview
Content Source: articles/virtual-network/service-tags-overview.md
Service: virtual-network
GitHub Login: @jispar
Microsoft Alias: jispar

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

@kiazhi As this is a feature request and not a doc request, your best place to post this request is at Azure Networking's feedback.azure.com page.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@kiazhi As this is a feature request and not a doc request, your best place to post this request is at Azure Networking's feedback.azure.com page.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Thanks, @TravisCragg-MSFT . I will bring this conversation to that channel.

@TravisCragg-MSFT  requested for an update on the channel. is the AzureUpdateDelivery Service Tag help to achieve this.


Add Social Media Account New and Edit pages in the GOV.UK Design System

Description
This ports the social media accounts new and edit pages to the GOV.UK Design System.
Something to watch out for is you cannot edit the social media type (Twitter etc.) if it's not the English translation.
Screenshots
New page (english)

Edit page (english)

Edit page (translation)

Trello card
https://trello.com/c/nnlm04DI/325-edit-social-media-page
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

LGTM


ActiveRecord::RecordInvalid

We should update this to rescue_from ActiveRecord::RecordInvalid do |e| so that errors are globally handled the same rather than only some of the methods.
https://github.com/wildland/api_me/blob/master/lib/api_me.rb#L100

No magic


Nested $ref are not getting validated

What version of Ajv are you using? Does the issue happen if you use the latest version?
"ajv": "^8.11.0"
Ajv options object

{ allErrors: true }
JSON Schema
{
  "$schema": "http://json-schema.org/draft-06/schema#",
  "$ref": "#/definitions/data",
  "definitions": {
    "data": {
      "type": "object",
      "properties": {
        "response": {
          "type": "object",
          "properties": {
            "return": {
              "$ref": "#/definitions/Response"
            },
            "arguments": {
              "type": "object",
              "properties": {},
              "required": [
                "list"
              ]
            }
          },
          "required": []
        }
      },
      "required": [
        "response"
      ]
    },
    "Response": {
      "type": "object",
      "properties": {
        "list": {
          "type": "string"
        }
      },
      "required": [
        "list"
      ]
    }
  }
}

Sample data
{
  "response": {
    "list": {
      "data": "ddd"
    }
  }
}

Your code
const ajv: any = new Ajv({ allErrors: true });
const validate = ajv.compile(schema);
const valid = validate(data);

Validation result, data AFTER validation, error messages
No Errors

What results did you expect?
The expected behavior is that all the nested schemas should get validated, in my case validation happens till data, "$ref": "#/definitions/Response" is not getting validated.
This is just a sample case, I have multiple nested $ref to be validated.

I am not sure what you mean by "#/definitions/Response is not getting validated", I don't think it should be with the combination of the schema / data you have. This ref is used in /response/return property that is not present in the data that you validate (and it is not required by the schema).

closing it for now, please re-open with any additional information if needed.


init: add sub-priority to internal ordering

Add a sub-priority field to Z_INIT_ENTRY_SECTION, which is used to
ensure that multiple drivers running at the same init priority still
run in an order that respects their devicetree dependencies.
This is primarily useful when multiple instances of the same device are
instantiated that can depend on each other, for example power domains
and i2c muxes.
After this PR, an init level now runs all SYS_INIT/DEVICE_DEFINE macros first,
then DEVICE_DT_DEFINE macros in order of devicetree ordinals.
Linker section names now look something like this:
 *(SORT_BY_NAME(SORT_BY_ALIGNMENT(.z_init_POST_KERNEL?_*)))
 *(SORT_BY_NAME(SORT_BY_ALIGNMENT(.z_init_POST_KERNEL??_*)))
 .z_init_POST_KERNEL40_0006_
                0x00000000000042b8        0x8 zephyr/drivers/gpio/libdrivers__gpio.a(gpio_stellaris.c.obj)
 .z_init_POST_KERNEL40_0017_
                0x00000000000042c0        0x8 zephyr/drivers/gpio/libdrivers__gpio.a(gpio_stellaris.c.obj)
 .z_init_POST_KERNEL40_0018_
                0x00000000000042c8        0x8 zephyr/drivers/gpio/libdrivers__gpio.a(gpio_stellaris.c.obj)
 .z_init_POST_KERNEL40_0019_
                0x00000000000042d0        0x8 zephyr/drivers/gpio/libdrivers__gpio.a(gpio_stellaris.c.obj)
 .z_init_POST_KERNEL40_0020_
                0x00000000000042d8        0x8 zephyr/drivers/gpio/libdrivers__gpio.a(gpio_stellaris.c.obj)

Resolves #22545 for multiple instances of the same device

@JordanYates could you update https://github.com/zephyrproject-rtos/zephyr/blob/main/scripts/build/check_init_priorities.py#L68 to parse the new format and compare/print it properly? (there's also some unit tests https://github.com/zephyrproject-rtos/zephyr/blob/main/scripts/build/check_init_priorities_test.py and an integration test that somehow is still working https://github.com/zephyrproject-rtos/zephyr/tree/main/tests/misc/check_init_priorities)

Done

Updated output of tests/misc/check_init_priorities and made the check actually run.

Seems reasonable, but also seems worth pointing out that this is yet more code we're adding to make up for our lack of proper dependency tracking on static devices...

Totally agree, but as a minimal internal change with no effect on the API that solves a real problem I think it is worth it

Seems reasonable, but also seems worth pointing out that this is yet more code we're adding to make up for our lack of proper dependency tracking on static devices...

Totally agree, but as a minimal internal change with no effect on the API that solves a real problem I think it is worth it

+1, the init problem® won't be easy to solve. For example, all the flexibility we've given with SYS_INIT is now hitting us hard.

Seems reasonable, but also seems worth pointing out that this is yet more code we're adding to make up for our lack of proper dependency tracking on static devices...

Totally agree, but as a minimal internal change with no effect on the API that solves a real problem I think it is worth it

+1, the init problem® won't be easy to solve. For example, all the flexibility we've given with SYS_INIT is now hitting us hard.

I'd be surprised at this point if someone comes up with a obviously perfect solution, but for now I think these progressive changes are moving in the right direction. I'm thinking, this takes care of same-level, the build time checking does multiple-level, @bjarki-trackunit were you looking into printing the init call sequence in https://github.com/zephyrproject-rtos/zephyr/pull/55654#issuecomment-1478452472? I'm thinking that having an init sequence printer could be used by subsystem tests and downstream application to compare the final init sequence with a golden sample so that any change gets caught and manually reviewed. That'd be a pretty solid system IMO.

@JordanYates I linked this up in #release on Discord and it's been pointed out that since it's been a hot topic recently, it may be worth running it through the Architecture WG before hitting the merge button (@carlescufi). Any chance you could talk about it? Happy to help out if the time zone is a problem btw.
Tagging as DNM for now.

I don't think the change is controversial enough to require me to personally argue its merits 🙂 1am is a high bar. If you are happy to introduce the topic that's good enough for me.

I don't think the change is controversial enough to require me to personally argue its merits slightly_smiling_face 1am is a high bar. If you are happy to introduce the topic that's good enough for me.

Yeah well, you never know, but I think the same. Happy to present on your behalf at the next WG then, @gmarull will you be there as well?

I don't think the change is controversial enough to require me to personally argue its merits slightly_smiling_face 1am is a high bar. If you are happy to introduce the topic that's good enough for me.

Yeah well, you never know, but I think the same. Happy to present on your behalf at the next WG then, @gmarull will you be there as well?

I'll be there, but TBH, I'd just go ahead and merge. This is not really a substantial change, just resolves some issues present in the current implementation.

Seems reasonable, but also seems worth pointing out that this is yet more code we're adding to make up for our lack of proper dependency tracking on static devices...

Totally agree, but as a minimal internal change with no effect on the API that solves a real problem I think it is worth it

Honestly, an LTO plugin seems increasingly more appealing these days ..

Architecture WG:

Incremental improvement using current infrastructure. Removes dependency on linking order for devices that have the same priority. This instead appends the Devicetree ordinal to remove any dependency from linking order.


Is Amplify support private API function of APIGW?

When we use "amplify api add" to create REST API in APIGW, the frontend use a public amplify domain to connect with APIGW.

Does it mean this traffic pass through internet to APIGW?
Can we setup a private API by leveraging VPN endpoint and resource policy with dedicated source VPC to serve our amplify frontend to have a REST API call with APIGW , which the traffic is routing within the AWS cloud?

Please advises, thanks.

yes
The Amplify CLI currently does not provide this capability. The Amazon API Gateway does allow you to set up private API that can only be accessed from your virtual private cloud in Amazon VPC by using an interface VPC endpoint
https://docs.aws.amazon.com/apigateway/latest/developerguide/apigateway-private-apis.html


conda lagging releases

Thank you for all the great work! I noticed that conda is lagging a bit behind with the release. Is that a known issue? Currently latest version on conda is 0.37.0, while pypi: 0.39.0.

Same issue here! having a problem with  slicer-0.0.7.dist-info\METADATA'  which is not allowing me to use pip.  Conda is the only way to go for now.

Also I noticed that when you compile shap from sources (for GPU) and checkout the latest tag v0.39.0, both pip show shap and conda list shap show the obsolete version 0.37.

Conda now has release 0.39.0, closing. Thanks.


Support using JsonConverterAttribute

It would be fantastic if the library supported usage of the JsonConverterAttribute as well, as for public APIs the SerializerSettings may not be under my control.
I could contribute as a pull request, if you believe it to be possible?

Hey @jonmotos,
Thank you for your involvement!
Sounds good for me, go ahead!
Looking forward to seeing POC from you 💪🏻


Compile Error on Ubuntu

I ran:
mkdir build
cd build
cmake ../src -DENABLE_TESTS=on -DENABLE_FFTW=on -DCMAKE_BUILD_TYPE=debug
make
and get errors below:
make -C build
make[1]: Entering directory '/home/jamesdang/MK-TFHE-master/build'
-- Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)
-- Could NOT find FFTW (missing: FFTW_LIBRARIES FFTW_INCLUDES)
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
FFTW_LIBRARIES (ADVANCED)
    linked by target "tfhe-fftw" in directory /home/jamesdang/MK-TFHE-master/src/libtfhe
    linked by target "unittests-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "testMkExternalProduct-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "test-gate-bootstrapping-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "test-decomp-tgsw-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "test-bootstrapping-fft-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "test-multiplication-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "test-tlwe-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "test-lwe-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "testMKbootNAND_FFT_details-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "test-addition-boot-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "test-long-run-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "testMKbootNAND-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "testMKbootNAND_FFT-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test
    linked by target "test-c-binding-fftw" in directory /home/jamesdang/MK-TFHE-master/src/test

-- Configuring incomplete, errors occurred!
See also "/home/jamesdang/MK-TFHE-master/build/CMakeFiles/CMakeOutput.log".
See also "/home/jamesdang/MK-TFHE-master/build/CMakeFiles/CMakeError.log".
make[1]: *** [Makefile:308: cmake_check_build_system] Error 1
make[1]: Leaving directory '/home/jamesdang/MK-TFHE-master/build'
make: *** [Makefile:9: all] Error 2

I am new here. Please help!

Hi, I think you are missing the FFT library. In cmake you should use -DENABLE_FFTW=off

Thank you ilachill. I installed the FFTlibrary. When I make it I got this error:
In file included from /usr/include/c++/8/cassert:44,
from /home/jamesdang/MK-TFHE-master/src/libtfhe/mkTFHEkeygen.cpp:4:
/home/jamesdang/MK-TFHE-master/src/libtfhe/mkTFHEkeygen.cpp: In function ‘void MKtLweExtractKey(MKLweKey*, const MKRLweKey*)’:
/home/jamesdang/MK-TFHE-master/src/libtfhe/mkTFHEkeygen.cpp:84:12: error: ‘result’ was not declared in this scope
assert(result->params->n_extract == N);
^~~~~~
make[2]: *** [libtfhe/CMakeFiles/tfhe-core.dir/build.make:440: libtfhe/CMakeFiles/tfhe-core.dir/mkTFHEkeygen.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:255: libtfhe/CMakeFiles/tfhe-core.dir/all] Error 2
make: *** [Makefile:141: all] Error 2
Please help!

Seems to be a common issue: https://github.com/ilachill/MK-TFHE/issues/1#issuecomment-504807211
Just comment out the assertion I guess.

I ran the following commands provided in the readme.
cd MK-TFHE
git submodule init
git submodule update
mkdir build
cd build
cmake ../src -DENABLE_TESTS=on -DENABLE_FFTW=off -DENABLE_NAYUKI_PORTABLE=off -DENABLE_NAYUKI_AVX=off -DCMAKE_BUILD_TYPE=release
make
After running make, it gives the following error.
[  1%] Building CXX object libtfhe/fft_processors/spqlios/CMakeFiles/tfhe-fft-spqlios-fma.dir/spqlios-fft-impl.cpp.o
/home/chandan/Desktop/supra/MK-TFHE/src/libtfhe/fft_processors/spqlios/spqlios-fft-impl.cpp: In function ‘void fft_model(const void*)’:
/home/chandan/Desktop/supra/MK-TFHE/src/libtfhe/fft_processors/spqlios/spqlios-fft-impl.cpp:350:22: error: passing argument 1 to restrict-qualified parameter aliases with argument 2 [-Werror=restrict]
350 |                 sub4(tmp0, tmp0, tmp3); // re2
|                      ^~~~  ~~~~
/home/chandan/Desktop/supra/MK-TFHE/src/libtfhe/fft_processors/spqlios/spqlios-fft-impl.cpp:351:22: error: passing argument 1 to restrict-qualified parameter aliases with argument 2 [-Werror=restrict]
351 |                 add4(tmp1, tmp1, tmp2); // im2
|                      ^~~~  ~~~~
/home/chandan/Desktop/supra/MK-TFHE/src/libtfhe/fft_processors/spqlios/spqlios-fft-impl.cpp:354:22: error: passing argument 1 to restrict-qualified parameter aliases with argument 3 [-Werror=restrict]
354 |                 sub4(tmp0, re0, tmp0); // re - re
|                      ^~~~       ~~~~
/home/chandan/Desktop/supra/MK-TFHE/src/libtfhe/fft_processors/spqlios/spqlios-fft-impl.cpp:355:22: error: passing argument 1 to restrict-qualified parameter aliases with argument 3 [-Werror=restrict]
355 |                 sub4(tmp1, im0, tmp1); // im - im
|                      ^~~~       ~~~~
cc1plus: all warnings being treated as errors
make[2]: *** [libtfhe/fft_processors/spqlios/CMakeFiles/tfhe-fft-spqlios-fma.dir/build.make:63: libtfhe/fft_processors/spqlios/CMakeFiles/tfhe-fft-spqlios-fma.dir/spqlios-fft-impl.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:345: libtfhe/fft_processors/spqlios/CMakeFiles/tfhe-fft-spqlios-fma.dir/all] Error 2
make: *** [Makefile:141: all] Error 2
Thanking you for any kind of help.


Feature: Exploration Sidebar

Description

Explore Sidebar

Sidebar Action

Rename

Each explore result sidebar item can be editable

Copy Link

Copy the web URL with an exploration result ID

Delete

Call API to delete the explore result
Modal to confirm delete

Add New exploration button, click will redirect to the Explore index page

UI

Click an exploration result and redirect to the page

Sidebar Action

Copy Link

Delete

Rename

Issue ticket number
closes #xx
Additional Context

Hi @andreashimin,
The code has been changed, please code review if you have available time.
Thanks,
Freda

Hi @andreashimin ,
The code has been changed, please code review if you have available time.
Thanks,
Freda


How do I create a new profile inside a project?

Am using pylxd 2.3.1 and I've creating a new project and then attempting to clone in the default profile into the project.
However I've not sure how this is done via pylxd as neither profiles.create or project classes appear to have a 'project' argument.      Any help would be appreciated..
if not self.lxd.projects.exists(self.project):
    # Could put in configrable options here like description,cpu/memory limits HERE
    self.lxd.projects.create(self.project)
    project=self.lxd.projects.get(self.project)

    # create a new project and copy over the default profile
    dflt=self.lxd.profiles.get('default')
    project.profiles.create('default', config=dflt.config, devices=dflt.devices)

Ok I solved the issue with two Client connections the first detects if the project exists and creates it if not.
then open a second and that one specifies the project and then I can create/modify profiles using that 2nd object.


Add Charter Communications to LF-Edge lanscape

Add logo and metadata for Charter Communications

added as a member org


winget is localized for my region, even when my UI-language is set to en-US

Brief description of your issue

I am running an English version of Windows, but live in The Netherlands. So my UI culture and my region settings are different. All my apps should be defaulting to showing their UI in English, but the currency and date formats in Dutch.
winget shows most of its text as Dutch:

Steps to reproduce
Windows Country or region: The Netherlands
Windows Regional format: English (Europe)

Windows Display Language: English

Expected behavior

the text output from winget is English
Actual behavior
the text output is Dutch

Environment
Windows Package Manager v0.1.41331 Voorbeeld
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19041.264
Package: Microsoft.DesktopAppInstaller v1.0.41331.0

Found the reason. Because this is a store app,  my Microsoft Account display language is used. This is pretty counterintuitive.
To see what your current language is, check the display language

Then open the Microsoft store app and download updates. It may take a while for the setting to sync.
I consider this a bug, my Windows installation has a UI language set, applications should follow it, but I'm guessing this is a WinRT remnant from the phone era we're now stuck with. I'd like a way to override the display language through configuration.
See also:

https://support.office.com/en-us/article/Display-language-doesn-t-change-in-Mail-and-Calendar-apps-for-Windows-10-3df52f6d-e9fe-4a65-9446-1b0a2f213368

This mess is very common in Microsoft apps, and was a major annoyance for years in Windows 10. Please don't bring it back.

Thanks for reporting this.  I am struggling a little bit with the repo however.  I am finding that the language for the client is responding to the Windows display language.
IMO, the language used by the client should come from Windows display language, so it is correct.  Did it eventually update per the link you provided?
We can investigate adding a language override in options.
Thanks

Yeah, the appstore language change eventually synced to windows. I'm happily on English now.
It's funny how much of Windows assumes I will constantly deliberately switch between languages.

I think I found out what my problem was: my Windows preferred language order was English (Sweden), Swedish (Sweden), English (United States).
This probably meant that Winget ignored English (Sweden) and went straight to Swedish. I don't understand the reasoning behind the separation between preferred language and Windows display language, but my expectation would be that if your app doesn't support English (whatever), just do English (US/UK/AUS/CAN), cause the differences are small enough that it doesn't matter.
Still didn't fix the Windows Store app.

Is this issue resolved?

Is this issue resolved?

Seems to work, adhering to the order of languages in the "Preferred languages" list, as output by Get-WinUserLanguageList.
This is probably correct, though as a user I can't really understand why there is a setting for "Windows display language" that is separate from "Preferred language"

Seems to work, adhering to the order of languages in the "Preferred languages" list, as output by Get-WinUserLanguageList.
This is probably correct, though as a user I can't really understand why there is a setting for "Windows display language" that is separate from "Preferred language"

This issue appears to be related to how Windows handles language priority.

We can investigate adding a language override in options. Thanks

Language override by options will be great. The default font of cli \ powershell doesn't support my windows display language so I like to set only winget to english.

Set-WinUserLanguageList solved the issue.
https://docs.microsoft.com/en-us/powershell/module/international/set-winuserlanguagelist?view=windowsserver2019-ps

Thank you, this is great.


No Source for Boma (#1983) in older converted games

Older converted games give the warning that there is no source for Boma. This goes away by the 1.6 version save.

Solving it in the next update. Boma will be added from the EU4 province of Loango to the missing mappings, like in next versions.

Fixed in https://github.com/Idhrendur/paradoxGameConverters/issues/291

And implemented in cace4b779a5d6dd0f163b16bb4e4bbbbe0fe2625 .


CP4I cluster is not usable

Multiple installs have not been cleaned up properly and the cluster is in an unusable state.
The cluster needs to be rebuilt
Long term we need to make sure the component can be installed together (and the modules should clean up properly)

CP4I Cluster has been rebuilt.
Cloud UI: https://cloud.ibm.com/kubernetes/clusters/c8sd03ow03qhumt19l50/overview?region=us-east&resourceGroup=2895876252b04bcb8af608662e0ad20b
OpenShift UI:
https://console-openshift-console.toolkit-dev-cp4i-2ab66b053c14936810608de9a1deac9c-0000.us-east.containers.appdomain.cloud/
Includes GitOps configuration and Portworx storage.


feat: --run support

Issue https://github.com/foundpatterns/torchbear/issues/148

Coding in Rust for two days like nobody's business!!1
Please, review the code, not sure if it's not total crap.

updating this change's name to fit the work
use that semantic message pattern for patch names (commit messages)

Force pushed new changes, please review again.

fixed commit message
moved --run value handling to src/lib.rs
fixed error messages

We need different implementation to support CLI mode, see https://github.com/foundpatterns/torchbear/issues/148#issuecomment-446250572


fix fragile proof for bignum squaring

🙇♂

@msprotz, yes, just needed to provide a more detailed proof using the distributive property of multiplication over addition..


options in query string not supported

I am getting this error while adding options field in query string in rule file.
"Error running query: TransportError(400, 'search_phase_execution_exception', '[query_string] query does not support [options]'
Can anyone help me

Could you post your query? Or link to the documentation for options? The normal format I've used is something like
filter:
- query:
     query_string:
        query: "foo: bar AND _exists_: baz"


Small corrections to documentation

What version of React-Table are you using?
Your bug may already be fixed in the latest release. Run yarn upgrade react-table!
6.7.4
What bug are you experiencing, or what feature are you proposing?
One small definite error, one small potential error in documentation:

In your section entitled "Custom Cell, Header and Footer Rendering", the first two items in your list are:

row - Original row from your data
row - The post-accessed values from the original row
The first is a typo - it should be
original - Original row from your data
row - The post-accessed values from the original row

In your list of props passed to the renderers - in particular, the props that are passed to the Cell renderer (and maybe others), it seems the props also include a column property that contains the entry from theColumns array for that column.  I think some of your examples use this prop, and I seem to be using it successfully in my code.  It contains much useful information, such as props.column.id, which is indispensable for callbacks.  This should be included in the documentation, unless I am misunderstanding something.

Use https://codesandbox.io/s/X6npLXPRW (by clicking the "Fork" button) to reproduce the issue.
What are the steps to reproduce the issue?
export const Cell = (cellInfo) => {
    let colInfo = cellInfo.column;
    let colId = colInfo.id;
    ....

There are lots of issues with the doco that need to fixed - they are on the radar (including the 'row/original' one).

Closing due to issue age. If you think this issue needs to be reopened, please check the latest version of React-Table for the issue once more and reopen if appropriate. Thanks!


bug fixes for date format

There's a bug of NSDateFormatter, it happens with the first day of daylight saving time every year, the date will be nil which will crash your code.
More info at here.

Thanks merging 👍


client-keystone-auth: can no longer setup KUBECONFIG file with 'kubectl config ...' command

/kind bug
What happened:
With OLD version of client-keystone-auth, one could use
kubectl --kubeconfig=/etc/kubernetes/admin.conf config set-credentials openstackuser --auth-provider=openstack
to update the KUBECONFIG file with the following lines ... to enable use of client-keystone-auth:
-name openstackuser
user:
auth-provider:
config: {}
name: openstack
With the NEW version of client-keystone-auth, there does NOT seem to be any 'kubectl config ...' options for setting up the NEW lines required in the KUBECONFIG file.
e.g.
- name: openstackuser
user:
exec:
command: "/etc/kubernetes/client-keystone-auth"
apiVersion: "client.authentication.k8s.io/v1beta1"
What you expected to happen:
Expected a command option like
kubectl --kubeconfig=/etc/kubernetes/admin.conf config set-credentials openstackuser --command="/etc/kubernetes/client-keystone-auth"
... in order to enable client-keystone-auth in the KUBECONFIG file ... writing the appropriate lines.
How to reproduce it (as minimally and precisely as possible):
% kubectl config set-credentials
.... shows the help and valid options.
Only valid options listed are:
Options:
--auth-provider='': Auth provider for the user entry in kubeconfig
--auth-provider-arg=[]: 'key=value' arguments for the auth provider
--embed-certs=false: Embed client cert/key for the user entry in kubeconfig
--password='': password for the user entry in kubeconfig
--username='': username for the user entry in kubeconfig
Anything else we need to know?:
Environment:

openstack-cloud-controller-manager version:
client-keystone-auth being used is 1.13.1
‘kubectl’ @ Kubernetes 1.13.5

OS (e.g. from /etc/os-release):
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"

CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"

Kernel (e.g. uname -a):
Linux controller-1 3.10.0-957.1.3.el7.1.tis.x86_64 #1 SMP PREEMPT Wed Mar 13 09:48:27 EDT 2019 x86_64 x86_64 x86_64 GNU/Linux

Install tools:
kubeadm

Others:

/assign @Fedosin
Mike, can you please take a look?

cc @lbragstad

cc @adisky

/assign @hogepodge

@gwaines  --kubeconfig file is a parent command option, you can still use it with

kubectl config set-credentials
do you want to report something else?? if so, please add the error output also.

The issue is not that there is an error on the 'kubectl config set-credentials' command.
The issue is that there does not seem to be an option to setup the the 'openstackuser' User in the KUBECONFIG file as required by the new version of client-keystone-auth.
I.e. what is the exact 'kubectl config set-credentials ...' command syntax for adding the 'openstackuser' User in KUBECONFIG file, for the new version of client-keystone-auth ?
i.e. the 'kubectl config set-credentials ...' command to insert the following lines in KUBECONFIG file ?
- name: openstackuser
user:
exec:
command: "/etc/kubernetes/client-keystone-auth"
apiVersion: "client.authentication.k8s.io/v1beta1"

I'm wondering if it is a feature request to kubectl?  Here https://github.com/kubernetes/kubernetes/pull/73230

Yeah this looks like what I'm looking for :-)
So it looks like

this feature is coded,
this is targeted for kubernetes v1.15,
'dims' just recently asked for the feature to be reviewed,
so we are just waiting on the appropriate reviewers to approve ?
... then this will be available in Kubernetes v1.15 .

Am I reading the linked feature status correctly ?

@gwaines I think you are right :-)

Thanks for raising this @gwaines

@gwaines, given it's actually not a cloud-provider-openstack related issue, do you mind we close it for now?

Yeah I'm fine with closing.

/close


Minimap: remember zoom

Sets zoom amount when reaching the login_screen state and saves it to the config to reload the value after loading client again.

Cleaned up relevant feedback. A zoom changed event would require an internal change.

The hub plugin Default Minimap Zoom covers this


Add a link to manage presenters

Coverage remained the same at 52.482% when pulling dc7b157872236bd98ed280a936e37e8f2ec4f721 on manage_link into 140394b12c8d02c18fa1816f77f829fc0ab99acf on master.


Add CKEditor to post content field

The CKEditor should appear in the content field when creating and editing posts.

I know this is older, but I'd be happy to take this if you want.


asynchronous function not calling callback on ubuntu

it was creating the directory but not calling the callback
.sync() worked tho

Hey.. has this issue been fixed? I kinda need sync to work with the callback.

@its2mc mkdirp.sync doesn't take a callback because it is synchronous

@mikesmullin, @its2mc: If you still think this is an issue, can you supply source code to show the case you think isn't working properly? Otherwise I recommend this issue be closed.


Link to Constrained decoding blog does not work

The link to the blog post "Constrained Decoding is Posterior Inference by Saibo-creator" does not work

@nielstron  Thanks for flaging this! I updated the link :)


Don't populate settings during read

There's a fair bit of complexity around handling computed settings. I'll look at adding this back in a follow up PR, but want to get this resource in a working state so we can make a new release.
Fixes https://github.com/elastic/terraform-provider-elasticstack/issues/767

@tobio , some other fields also use Default together with Computed in the same scheme (e.g. mappings, deletion_protection and few others). Shall we consider removing Default for them as well?

Just as a side note - kibana data view also uses that combination of Default and Computed.


Missing Counties in drop down box when creating a station location

This is for reporting bugs with the code, if this issue is regarding setting up on your local server please use the discussions area
At least for Kansas, not all counties show up in the drop down box - I suspect there's a limit somewhere in the number of items available in the drop down box.  Kansas has 105 counties total, and the last 4 alphabetically are not shown:
Wichita
Woodson
Wilson
Wyandotte
Wyandotte is one of the most heavily populated counties in Kansas, so I'm surprised this has never been noticed before.
Steps to Reproduce

Go to Create Station Location
Select United States, and the state of Kansas
In the drop list for counties, the last one shown is Washington, 4 are missing

thx for reporting this :) fixed soon and will be in the next release

fixed


remove useless no-report-written test summary output

In case user requests that there is no JSON report file actually written (and instead all the collected report information is just kept stored in the plugin object - typical scenario when doing customized reporting), there is no need to add extra:
---------------------- JSON report ----------------------
no JSON report written.

filler content to the end of the pytest terminal output.

The idea is to convey whether the plugin has been doing any work. Even if no outfile is written, I wanted to give some indication that the plugin has been busy compiling a report object. I didn't want there to be a scenario where the plugin slows down a test suite by generating large reports in memory without informing the user who thus assumes it's been idle.
For that, I agree the message isn't very useful, so what do you think about changing it along the lines of JSON report generated (not written to file) or JSON report generated in memory?
(Thanks for your previous well-written PRs.)

hmm, I'm coming at this from a place where:

we consciously use the JSON report plugin for collecting report data and then after the test run is done, read it from our code to construct our own customized report and distribute it through various channels
we want to trim down the main console output, especially the part at the end, so it's easy to see a test result summary at a glance, and this line is just 'filler' which we then 'learn to ignore', which is not a good habit to get in if not needed

so I know we have JSON report running and do not need extra information in the output just to tell me 'yes, you used the JSON report plugin;
if you feel that such output is really useful in other scenarios, would it be ok if we add some option to make JSON report be 'less verbose' and not output such filler summary output?

I tied the summary output verbosity to pytest's default --verbosity. This may already work the way you need it, but you can also use the new --json-report-verbosity=LEVEL to force a level, e.g. -1 to suppress all terminal output by the plugin.

thanks, works nicely for us; added one more test case in https://github.com/numirias/pytest-json-report/pull/68


auto-editor 21.19.1 crash Win10 to BSOD

pip version = python39\lib\site-packages (21.1.1)
when executing for silence cut, would falsely prompt, despite running both pip & auto-editor at latest rev:
"Auto-Editor is out of date. Run:
pip3 install -U auto-editor

to upgrade to the latest version."
Win10 crash to BSOD in a few seconds after the error prompt.
after several tries Win10 was corrupted required reformat (system installed since 2018).
issue reproduced after Windows 10 reinstallation.
Reverting back to auto-editor-21.16.1, eliminated crashing.
system config: i8700k + Nvidia GTX 1070, 32GB RAM.

I also noticed that the upgrade message was being displayed even though it shouldn't, so I fixed that.
I ran the latest version on my Windows Installation and it didn't crash the os, so I'll need more details before I can look into that.

upgraded to 21.19.2, when attempt silence cut, the following returned, good news is it now longer crash windows, but yield no video editing output nor XML:
H:> auto-editor ".\2021-05-09 02-08-48.mp4" -exr
Warning! exportMediaOps options are not used when exporting  as an XML.
.\2021-05-09 02-08-48.mp4solve XML file.
Warning! Could not get number of VRF Frames.
Traceback (most recent call last):
File "c:\users\wkwee\appdata\local\programs\python\python39\lib\runpy.py", line 197, in _run_module_as_main
return run_code(code, main_globals, None,
File "c:\users\wkwee\appdata\local\programs\python\python39\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\Users\wkwee\AppData\Local\Programs\Python\Python39\Scripts\auto-editor.exe_main.py", line 7, in
import tempfile
File "c:\users\wkwee\appdata\local\programs\python\python39\lib\site-packages\auto_editor_main.py", line 557, in main
sampleRate, audioData = read(f'{TEMP}{sep()}{args.cut_by_this_track}.wav')
File "C:\Users\wkwee\AppData\Local\Programs\Python\Python39\Lib\site-packages\auto_editor\wavfile.py", line 551, in read
data = _read_data_chunk(fid, format_tag, channels, bit_depth,
File "C:\Users\wkwee\AppData\Local\Programs\Python\Python39\Lib\site-packages\auto_editor\wavfile.py", line 453, in _read_data_chunk
data = data.reshape(-1, channels)
ValueError: cannot reshape array of size 14811097 into shape (2)


Add units

I can hear my science teacher in the back of my head: "Always add units!"

Thanks for reviewing this PR. I'm going to go out on a limb here and say that I think that you interpreted that this PR was adding units to minSize (which as you correctly point out already states the units) rather than adding units to maxSize.
Please let me know if that's not the case, or how this PR can be otherwise improved.

Hi @aryzing
Unfortunately i still think that we dont want to add units everywhere where the option is mentioned. When we would speak about timeout we wouldnt add ms e.g. "After the timeout milliseconds is triggered, X is called"

We can keep the PR in current shape and wait for more contributors reviews if you like

we dont want to add units everywhere

I very much agree with you, which is why this PR adds units once. I think we can both agree that each variable's units should be stated at least once.
Bear in mind that the docs aren't necessarily for a seasoned devs like yourself that are already familiar with the config, but rather for folks like me just trying to figure out how to set up webpack properly.
The experience I had while browsing this section of the documents was not great: I was left puzzled not knowing exactly how to make this option work. I then invested some time using trial-and-error builds, and finally, by chance, stumbled upon the minSize section, all of which could have been avoided by stating the option's units.
It's a single line PR that does make a positive difference.

Thanks for elaborating, still i would suggest adding it here once: https://webpack.js.org/plugins/split-chunks-plugin/#splitchunksmaxsize

This is a good addition, it only says "number", but it could be anything.


Delete symbol in exchange-core

Hi:
Is there any class to delete the existing symbol in exchange core. Actually here we have BatchAddSymbolsCommand class through we can add symbol dynamically but how to delete that symbol dynamically? I am search through the code but not getting something like. Pls help me.
Thanks

Hi @mzheravin
Pls give me some hints to get out of this issue.
Many Thanks

Not as far as I know but you can fork the code and add a new command to remove symbols

Hi @esanchezros
Thanks for your response. In BatchAddSymbolsCommand, we put the CoreSymbolSpecification object as value and symbolId as key in IntObjectHashMap . After that,  SerializationUtils.marshallIntHashMap mathod which is serialize the IntObjectHashMap object.
So now If I try to create BatchDeleteSymbols command to delete the created symbol how would I get that object. I mean the key-pair which was stored in  IntObjectHashMap. This is the issue.
Many Thanks

Hi @mzheravin  Sir / @esanchezros  Sir:
             Whatever I have figured out that symbol is stored as byte format through WriteBytesMarshallable. So how  to read that byte and get the symbol back.

Thanks

Hi @mzheravin  Sir / @esanchezros  Sir:
             Whatever I have figured out that symbol is stored as byte format through WriteBytesMarshallable. So how  to read that byte and get the symbol back.

Thanks

@mzheravin  Sir ,
Pls help, so that I can add a new feature  to the existing system. So we can delete the symbol dynamically.

Hey @shanhashcah,
Not sure if the repo is maintained any longer but adding this functionality yourself should not be difficult.
Have a look at https://github.com/exchange-core/exchange-core/blob/2f8548749839e9095c8dc597e4b61521d259fa5d/src/main/java/exchange/core2/core/processors/MatchingEngineRouter.java#L218 where the BatchAddSymbolsCommand is processed. You need to do something similar, adding a BatchDeleteSymbolsCommand which will delete the order book by symbolId

Hi @esanchezros  Sir,
              I am trying to create a class like below


feat: replace thecodingmachine/class-explorer with kcs/class-finder

Main issue is to let type mappers find types in vendor packages which class-explorer and maintainer is not updating the project.
Symfony and Laravel bundles have to be updated too. Fixes: #657

It seems to me that because of this catch block here, one of my use cases (loading non class files) would be solved.
However the other would still be broken - anything that requires autoloading won't function as expected. One example of that - our pre-8.0 enums require special autoloader to call ::initialize() on them to initialize all static variables with enum instances, and there are about 150 of them in our codebase. Excluding them is doable of course, but I'm just not entirely sure not using autoloading is the way to go, as it feels like an incredibly large hack to me.
I've taken a look at the issue that's mentioned in alekitto's package to see what was the root of the problem that made them use include instead of loading the class through autoloading. They mention this issue: https://github.com/composer/composer/issues/6987. In short, the include is there to avoid loading the same file twice IF it's present in a PSR-0/PSR-4 namespaces mapping AND manually loaded via files.
If you just do
class_exists('Aws\functions');
class_exists('Aws\functions');

it breaks. So the worst thing here is it does indeed look like Composer's bug that they don't want to fix :/ Now it makes sense to me why both Symfony and Spiral parse the file to see if it contains a class before trying to autoload it.
To be honest I don't like either solution. We could still try two things - re-reporting this as a bug with a clearer example to composer, or attempt to fix class-finder in a different way - by first checking whether given file is amongst Composer's autoload_files.php file.
That solution would satisfy both needs without compromises if it works :/

I've submitted an issue on the class-finder side. Maybe we can get the fix done on their side 🤷🏻

Codecov Report
Attention: Patch coverage is 83.33333% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 95.17%. Comparing base (53f9d49) to head (f66e9e5).
Report is 70 commits behind head on master.

Files
Patch %
Lines

src/Utils/Namespaces/NS.php
73.68%
5 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #664      +/-   ##
============================================
- Coverage     95.72%   95.17%   -0.56%
- Complexity     1773     1824      +51
============================================
  Files           154      171      +17
  Lines          4586     4847     +261
============================================
+ Hits           4390     4613     +223
- Misses          196      234      +38

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@fogrye this looks good - thanks!  It's a BC breaking PR.  That's fine though, as we're targeting a new major version for the next tag.


Removed redundant callback.

The original purpose of the removed code was if user returns false from
handler, then inform user the next read is stopped.
However, this code passed not only th user hander return false, but also
internal error happens. In the internal error case, func is called
twice.
So I removed the code. The user that returns false at the handler, user
alread knows the next read never happen.

Codecov Report

Merging #309 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
- Coverage   85.48%   85.47%   -0.01%
==========================================
  Files          33       33
  Lines        4842     4840       -2
==========================================
- Hits         4139     4137       -2
  Misses        703      703


Update deployment.yaml

For newer releases this will be supported/needed. K8 v.1.15.* can already handle the versionApi: "apps/v1".

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/mychartname])

@XHPJ
Could you sign DCO and bump the chart version to latest please?

/ok-to-test

/assign

/lgtm


fix nil client bug in monitorAntiEntropy (and test)

also fix bug where broadcast_test tried to listen on localhost:10101
Overview
This was broken by some recent refactoring - I noticed it while trying to import a bunch of genome data into a Pilosa cluster. It was not caught by any of our current integration type tests because they use a 10 minute anti entropy interval by default.
Fixes #
Pull request checklist

[ ] I have read the contributing guide.
[ ] I have agreed to the Contributor License Agreement.
[ ] I have updated the documentation.
[ ] I have resolved any merge conflicts.
[ ] I have included tests that cover my changes.
[ ] All new and existing tests pass.

Code review checklist
This is the checklist that the reviewer will follow while reviewing your pull request. You do not need to do anything with this checklist, but be aware of what the reviewer will be looking for.

[ ] Ensure that any changes to external docs have been included in this pull request.
[ ] If the changes require that minor/major versions need to be updated, tag the PR appropriately.
[ ] Ensure the new code is properly commented and follows Idiomatic Go.
[ ] Check that tests have been written and that they cover the new functionality.
[ ] Run tests and ensure they pass.
[ ] Build and run the code, performing any applicable integration testing.

Agreed. This was sort of in the midst of a refactor. Ill add that to my
notes and the relevant ticket
On Sat, Apr 28, 2018, 9:26 AM Yuce Tekol<EMAIL_ADDRESS>wrote:

@yuce commented on this pull request.
In server.go
https://github.com/pilosa/pilosa/pull/1233#discussion_r184855367:

@@ -163,6 +163,7 @@ func OptServerGCNotifier(gcn GCNotifier) ServerOption {
func OptServerRemoteClient(c *http.Client) ServerOption {
return func(s *Server) error {
s.executor = NewExecutor(c)

s.remoteClient = c

I think we should have defaults for all Server fields, including
remoteClient in case the user doesn't call option functions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pilosa/pilosa/pull/1233#pullrequestreview-116138669,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4hczmpYMbPl9hjX15vfp-62uzCAkLmks5ttHwWgaJpZM4TrbUE
.


Refactor

Codecov Report

Merging #793 (54cabea) into main (c15d127) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #793      +/-   ##
==========================================
- Coverage   85.26%   85.26%   -0.01%
==========================================
  Files         315      315
  Lines       29082    29080       -2
==========================================
- Hits        24798    24796       -2
  Misses       4284     4284

Impacted Files
Coverage Δ

lib/ffi/src/cps/async_stack.rs
90.00% <ø> (-0.16%)
:arrow_down:

Continue to review full report at Codecov.

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


[Feature Request] Alias lat & lng to latitude & longitude

I would like to request adding alias' to OpenCage::Geocoder::Location.lat and OpenCage::Geocoder::Location.lng to be OpenCage::Geocoder::Location.latitude and OpenCage::Geocoder::Location.longitude.
def lat
  @result['geometry']['lat'].to_f
end

+ alias latitude lat

def lng
  @result['geometry']['lng'].to_f
end

+ alias longitude lng

I can open a PR, but the guide said to open an issue first.

Good idea, go ahead. I can prepare a new release next week.

@thestelz version 3.1.1 is released now

@mtmail Awesome! Thanks!


Ldap queries

Hello,
Afer I installed this bundle, I'm getting ldap connections on every single page reload and not just when a ldap connection is required, for example for authentication process. Is there a way to disable the connection on all pages I'm not using it?

Those queries are the result of a refreshUser() call on the LDAP User Provider. It reloads the user from LDAP by its GUID, along with their roles (also reloads all of the user's attributes). There a few reasons this is useful:

If a user is renamed or attributes change, then by reloading based on the GUID the rename/change is caught/handled.
If a user is removed from any groups granting them access to the site via role mappings, this is caught and they would be denied access instead of hanging onto it.
If a user account is disabled/locked out while they are using the site then that change is reflected when they try to navigate/use the site (they would be forced back to a login).

However, I'm not opposed to making this some sort of a configurable option so that it doesn't  query LDAP like that on every page load.

That would be nice @ChadSikorra. We are developing an intranet site with hundreds of users so disabling this feature would be useful to not stress our domain controller. Maybe the refreshUser() could be called every X minutes or something like that.
Thanks a lot.

Yeah, let me think about it a little bit. Forcing it every X number of minutes is probably more complicated then it would seem. I could maybe add two separate options. Maybe one that would enable/disable the refresh altogether. And another that could be an event or class with one method shouldRefreshUser($user) that returns true/false, then people could implement their own logic for it.

Sounds good. Thank you.
2016-09-21 10:37 GMT-03:00 Chad Sikorra<EMAIL_ADDRESS>
Yeah, let me think about it a little bit. Forcing it every X number of
minutes is probably more complicated then it would seem. I could maybe add
two separate options. Maybe one that would enable/disable the refresh
altogether. And another that could be an event or class with one method
shouldRefreshUser($user) that returns true/false, then people could
implement their own logic for it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ldaptools/ldaptools-bundle/issues/4#issuecomment-248614446,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUOJ8rkwe4rwZ3qaTnQbZyXT9ul5pSdlks5qsTMCgaJpZM4KCt7P
.

--
Jakob Eissler
Ingeniero Civil en Computación e Informática
<EMAIL_ADDRESS>

This has been fixed in: https://github.com/ldaptools/ldaptools-bundle/commit/7f17d027aa6d63ba83aa266a9ccbb1ddf46c16a1. It's now possible to control whether roles/attributes are queried from LDAP again on a refresh. To disable any any LDAP queries in the refresh process you can do:
# app/config/config.yml

ldap_tools:
    security:
        refresh_user_roles: false
        refresh_user_attributes: false

I documented it here. Let me know if you still have any issues. Hopefully I'll be tagging another version within a month or so.

Thank you so much Chad. My app is working much faster now.
2016-09-22 22:39 GMT-03:00 Chad Sikorra<EMAIL_ADDRESS>
This has been fixed in: 7f17d02
https://github.com/ldaptools/ldaptools-bundle/commit/7f17d027aa6d63ba83aa266a9ccbb1ddf46c16a1.
It's now possible to control whether roles/attributes are queried from LDAP
again on a refresh. To disable any any LDAP queries in the refresh process
you can do:
app/config/config.yml
ldap_tools:
security:
refresh_user_roles: false
refresh_user_attributes: false
I documented it here
https://github.com/ldaptools/ldaptools-bundle/blob/master/Resources/doc/LDAP-Authentication-Provider.md#user-refresh-settings.
Let me know if you still have any issues. Hopefully I'll be tagging another
version within a month or so.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ldaptools/ldaptools-bundle/issues/4#issuecomment-249078462,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUOJ8igU3rSCcXHur-fjVn7SrLpTUQhSks5qsy3vgaJpZM4KCt7P
.

--
Jakob Eissler
Ingeniero Civil en Computación e Informática
<EMAIL_ADDRESS>