Document how to wait for cluster availability before creating kubernetes resources

I have issues
I'm submitting a...

[x] bug report
[ ] feature request
[x] support request - read the FAQ first!
[x] kudos, thank you, warm fuzzy

What is the current behavior?
I was unable to figure out how to delay creation of kubernetes resources until the cluster is fully available. I saw a lot of search hits for people experiencing the same problem. There were comments about using a null_resource to get around the problem, mimicking what happens with the kubernetes_config_map in aws_auth.tf, but those comments were by people smarter and more capable than me.
Finally I found comments in outputs.tf as of PR #890 by @dpiddockcmp indicating an intent to make this easy for users of the module:
output "cluster_id" {
  description = "The name/id of the EKS cluster."
  value       = element(concat(aws_eks_cluster.this.*.id, list("")), 0)
  # So that calling plans wait for the cluster to be available before attempting
  # to use it. They will not need to duplicate this null_resource
  depends_on = [null_resource.wait_for_cluster]
}

So now I was able to fix the problem by adding a depends_on like so:
resource "kubernetes_namespace" "apps" {
  metadata {
    name = "apps"
  }
  depends_on = [
    module.eks.cluster_id
  ]
}

This is a really important feature! I spent most of day gnashing my teeth on a common problem experienced by users of the module because I didn't know about it.
Can you please add a description of this feature to:

the release notes for 12.1.0
the FAQ.md
the documentation for cluster_id on the Terraform Registry

...so people like me are more likely to find it?
(Note I'm including the rest of the information below in hopes that the detail will help other people who find this problem.)

If this is a bug, how to reproduce? Please include a code sample if relevant.
This all works fine
provider "aws" {
  region = local.region
  profile = "terraform-operator"
}

data "aws_availability_zones" "available" {
}

module "vpc" {
  source  = "terraform-aws-modules/vpc/aws"
  version = "2.6.0"
  name = "vpc-${local.cluster_name}"
  cidr = "${local.vpc_cidr}" # 192.168 , 172.16
  azs = data.aws_availability_zones.available.names
  public_subnets = "${local.vpc_subnets}"

  enable_dns_support = true
  enable_dns_hostnames = true

  tags = {
    "kubernetes.io/cluster/${local.cluster_name}" = "shared"
  }
}

data "aws_eks_cluster" "cluster" {
  name = module.eks.cluster_id
}

data "aws_eks_cluster_auth" "cluster" {
  name = module.eks.cluster_id
}

provider "kubernetes" {
  host                   = data.aws_eks_cluster.cluster.endpoint
  cluster_ca_certificate = base64decode(data.aws_eks_cluster.cluster.certificate_authority.0.data)
  token                  = data.aws_eks_cluster_auth.cluster.token
  load_config_file       = false
  version                = "~> 1.9"
}

module "eks" {
  source  = "terraform-aws-modules/eks/aws"
  version = "12.1.0"                                               # <-- The critical bit!
  cluster_name = "${local.cluster_name}"
  subnets = module.vpc.public_subnets
  vpc_id = module.vpc.vpc_id
  map_users = var.map_users
  # worker nodes
  worker_groups_launch_template = [
    {
      name                 = "worker-group-1"
      instance_type        = "t3.large"
      asg_desired_capacity = 2
      asg_max_size = 5
      asg_min_size  = 2
      autoscaling_enabled = true
      public_ip            = true
    }
  ]
}

The part that failed before 12.1.0:
# Explicitly create namespace
resource "kubernetes_namespace" "apps" {
  metadata {
    name = "apps"
  }
}

What's the expected behavior?
The creation of kubernetes_namespace.app succeeds without re-running Terraform.
(This works in 12.1.0!)
What's the actual behavior?
The creation of kubernetes_namespace.app times out because the resource attempts creation before the cluster is ready, with a dial tcp [ip-redacted]:443: i/o timeout error message.
(This is what happened in 12.0.0 and earlier.)
Are you able to fix this problem and submit a PR? Link here if you have already.
Environment details

Affected module version: 12.1.0
OS: docker pull abdennour/terraform:0.12.28-helm3.2.4-aws-iam-auth
Terraform version: 0.12.28

Any other relevant info
None, this should be plenty.

By the way, there's an upstream PR in the AWS provider which would obviate the need for any workarounds like this, but it's been languishing for 6+ months... Do the module authors have any pull to get that merged?

The intention of the change in 12.1.0 was so that your kubernetes resources don't have to explicitly depend on anything, if the kubernetes provider is configured following the Usage Example.
Creating the basic example in 12.2.0, adding your namespace:
resource "kubernetes_namespace" "apps" {
  metadata {
    name = "apps"
  }
}

Terraform apply output:
module.eks.null_resource.wait_for_cluster[0]: Creation complete after 2m31s [id=5478466492082028166]
data.aws_eks_cluster_auth.cluster: Refreshing state...
data.aws_eks_cluster.cluster: Refreshing state...
kubernetes_namespace.apps: Creating...
module.eks.kubernetes_config_map.aws_auth[0]: Creating...
kubernetes_namespace.apps: Creation complete after 1s [id=apps]
module.eks.kubernetes_config_map.aws_auth[0]: Creation complete after 1s [id=kube-system/aws-auth]

The changelog is automatically generated. This makes life a little easier but also means we can't retroactively modify it. I'll try to add something to the documentation

Does #955 help clarify the situation?


Azure Identity - support customized Default Credential

Feature Request

We use Azure Identity Client Module to address token authentication issue. Currently, NewDefaultAzureCredential creates the Credentials in a fixed order. Our requirement is to enable only part of these credentials and make the creation order of credentials sortable.
At first, we tried to rewrite our own NewDefaultAzureCredential, but we found some credentials assign values to private fields(like NewManagedIdentityCredential, NewChainedTokenCredential), which cannot be migrated to our code.
Is it possible to make credentials in NewDefaultAzureCredential selectable and sortable? Or is it possible to expose the operation of assigning values to private fields to public users so that we can write our own default credential?

At first, we tried to rewrite our own NewDefaultAzureCredential, but we found some credentials assign values to private fields(like NewManagedIdentityCredential, NewChainedTokenCredential), which cannot be migrated to our code.

You can implement the behavior of ManagedIdentityCredential.imdsTimeout with a small wrapper: https://github.com/Azure/azure-sdk-for-go/issues/19699#issuecomment-1379290680. ChainedTokenCredential.name is used only in log messages; why do you need to override its default value, "ChainedTokenCredential"?

Hi @chlowell, thank you for your solution. But it seems that azidentity.NewCredentialUnavailableError is only available in azidentity v1.3.0, which is still in beta stage. Do you have any plan to support it in v1.2.x? If not, may I know when do you plan to roll out the standard v1.3.0?


IBM.com figma white theme Kit link bug on Designing > Get started page

Bug🐛: Link bug on Designing > Get started page
Page: https://next.carbondesignsystem.com/designing/get-started#local-design-kits
Browser: Firefox
Suggestion: Under the Local design kits section for IBM.com, the "IBM.com figma white theme" row link should go to the new beta library. The link needs to be updated to this https://www.figma.com/file/RB6yqMPPsGutYnGTSvwGR4/Carbon-for-IBM.com-(White-Theme)-Beta?node-id=475%3A372 instead.
Originally posted by @laurenmrice in https://github.com/carbon-design-system/carbon-platform/discussions/1268#discussioncomment-3684294

Hey team! Please add your planning poker estimate with Zenhub @andreancardona @francinelucca @jdharvey-ibm


More criteria from "do you trust this application"?

The blog article Do you trust this application? lists some common TLS mistakes.  Are there any we should add as criteria?

Here are some potential criteria:

The project SHOULD NOT use unencrypted network communication (such as HTTP and telnet) if there an encrypted equivalent (e.g., HTTPS/SSL/TLS and OpenSSH), unless the user specifically requests/configures it.
The project MUST, if it supports SSL/TLS, perform TLS certificate verification by default when using TLS, including on subresources
The project MUST, if it supports SSL/TLS, perform certificate verification before sending HTTP headers with private information (such as secure cookies).
The project MUST, if it supports SSL/TLS, support at least TLS version 1.2.   {Maybe this should be moved into an existing criterion.}

I think these proposed criteria address these issues raised in the article:

Failure to use TLS when required (GNOME Music, GNOME Weather; note these are the only apps I mention here that do not use WebKit). This means the application has no security.
Failure to perform TLS certificate verification (Shotwell and Pantheon Photos). This means the application has no security against active attackers.
Failure to perform TLS certificate verification on subresources (Midori and Xombrero, Liferea). As sites usually send JavaScript in subresources, this means active attackers can get total control of the page by changing the script, without being detected (update: provided JavaScript is enabled). (Regrettably, Epiphany prior to 3.14.0 was also affected by this issue.)
Failure to perform TLS certificate verification before sending HTTP headers (private Midori bug, Banshee). This leaks secure cookies, usually allowing attackers full access to your user account on a website. It also leaks the page you’re visiting, which HTTPS is supposed to keep private. (Update: Regrettably, Epiphany prior to 3.14.0 was affected by this issue. Also, the WebKit 2 API in WebKitGTK+ prior to 2.6.6, CVE-2015-2330.)

I got some comments, here's an attempt to summarize.

This may exclude too many programs.  Maybe SHOULD to start instead of MUSTs?
Should we only use the term TLS?  Should we say in just one in place "TLS was formerly known as SSL".

"The predecessor of TLS was called SSL."

Similar sentiments are here: "The Most Dangerous Code in the World: Validating SSL Certificates in Non-Browser Software" by Martin Georgiev et al., http://crypto.stanford.edu/~dabo/pubs/abstracts/ssl-client-bugs.html
"We demonstrate that SSL certificate validation is completely broken in many security-critical applications and libraries... the root causes of these vulnerabilities are badly designed APIs of SSL implementations (such as JSSE, OpenSSL, and GnuTLS) and data-transport libraries (such as cURL) which present developers with a confusing array of settings and options."


Basic Auth Question

This is more of a question rather than an issue, does the current osb-broker-pack implementation provide basic auth when I interact with the broker?
For example instead of doing curl http://<brokerip>:<port>/v2/catalog I would like to do curl http://user:pass@<brokerip>:<port>/v2/catalog
Thanks!

Sounds good @pmorie ..is this something that could be implemented at the osb-starter-pack level or do you recommend implementing it in the osb-broker-lib repo?  thx

@joaquinrdzibm It would likely have aspects at both:

osb-broker-lib needs to supply the basic auth credentials
osb-starter-pack needs to be configured with the correct auth credential values somehow

@joaquinrdzibm sorry about the delay - why don't you ping me on k8s slack or via email and we'll set up a time to talk about this.


Removal of the script

Hello!
Why did you remove the script??
I really need this... Can you push the script again?

Hey there,
I'm going to change this script, so I cleaned up it some time ago. I'll recreate it soon. Maybe next week.

Can you give me the old script? I kind of need it now, it is urgent. I need to get on with my engagement ASAP.

Sorry, no. It's on my another PC that is currently under repair.

Oh, okay.

The script is rewritten and does work correctly.


Roman/feature/payment.defer inijected web3 init

Travis automatic deployment: https://84-pr-ching.surge.sh


Corp Lookup for Structures Not Working

I used Copy/Paste of D-Scan to add structures to a wormhole at https://pathfinder.lost-in-w.space/
When I try to set the Corp, it no longer auto-completes names but just says "No results found."

seems like the esi URI being used is wrong.
currently in this format:
https://esi.evetech.net/v2/search/?categories=corporation&search=CORPNAME&strict=false/datasouce=tranquility

results in 404
checking the latest esi API, the only search I could find, was within the character group.
The following URI format should be used:
https://esi.evetech.net/latest/characters/AUTHED_CHARACTER/search/?categories=corporation&datasource=tranquility&language=en&search=CORP_SEARCH_STRING&strict=true&token=ACCESS_TOKEN

this requires the additional scope:
esi-search.search_structures.v1

Hey team,
I am aware of this, I plan to fix it but I haven't had time over the past month or so. I'll do my best to get it in the next 2-4 weeks.

Has been fixed with https://github.com/goryn-clade/pathfinder/releases/tag/v2.2.1


[fix]:Optimize the quick start module used in vue3(CHINESE)

Does not provide an export named'default'will be used in vue3, but it is not explained in the document. And the quotation at the beginning of the document is not clear.

Thank you for the feed-back. We will find a different way


Fix demos on Marvell

Fix demos on Marvell
Description
Problem: Many demos were failing in Marvell board due to lack of memory in heap while establishing a TLS connection.
Root Cause: Establishing a TLS connection requires a good amount of memory in the HEAP. The tasks created with xTaskCreate() function also allocates the memory for the stack in the heap. Marvell's heap(~73KB) was running out of space due to these allocations.
Fix: Optimized the stack sizes for different tasks for the demo and verified the demos.
The demo verification CI job can be found here
https://amazon-freertos-ci.corp.amazon.com/job/im_v2_master/job/nightly_custom_job/1312/
Checklist:

[x] I have tested my changes. No regression in existing tests.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks


tailscaled: missing realtime dependency su

Problem
https://github.com/NixOS/nixpkgs/blob/30439d93eb8b19861ccbe3e581abf97bdc91b093/pkgs/by-name/ta/tailscalesd/package.nix
The newer tailscale drive command has a dependency on su as can be observed with journalctl -u tailscaled:
  Sep 27 06:36:47 elitebook tailscaled[1328]: can't find su command: exec: "su": executable file not found in $PATH
  Sep 27 06:36:47 elitebook tailscaled[1328]: user server /nix/store/gq0q640pl49hla6l00ny0r3fif5bj3x3-tailscale-1.74.0/bin/.tailscaled-wrapped stopped with error "System administrator" is root, will try again in 16ms

In tailscale the relevant code is at https://github.com/tailscale/tailscale/blob/cab2e6ea677865f50e3ccfb526603d9f36c9dce8/drive/driveimpl/remote_impl.go#L418
An issue of another NixOS user on tailscale might get solved by this too: https://github.com/tailscale/tailscale/issues/13285
I would make a PR, but I believe that su is a bit special and requires extra care, so I'd like to get a second opinion first.
Maintainer
@SuperSandro2000

That IS a special problem. Afaik pkgs.shadow provides bin/su, though it will not have it with the setuid bit. On NixOS this is usually handled by security.wrappers, and on other distros this is usually baked into their package. This poses a problem since we'd need a reference to su from outside of the /nix/store context..

Yeah, so I was thinking of maybe solving this in the module instead. There we could use the wrapper...

The su binaries with setuid bit lives in NixOS at /run/wrappers/bin/su.
So we probably want to add that path to the PATh variable.

Yeah, so I was thinking of maybe solving this in the module instead. There we could use the wrapper...
I would just add it to the PATH of the systemd service.

Correct yeah. I don't think anything can or should be done about using this package on a non-nixos distro for now. It should resolve the PATH fine anyways as long as its FHS compliant


Not detecting games with rpc

os: gnu/linux
wm: sway
node: 21.2.0

Need more info, what games, etc

factorio, faster than light

They aren't in Discord's game DB so that's expected (for now).

i'm so sorry, not understand this before and ask other thash isue

They aren't in Discord's game DB for Linux so that's expected (for now).

3% of games for linux support this


Orders sent by cert-manager using a cluster-issuer with an EAB are not RFC8555 compliant | Step-CA private ACME Server

Describe the bug:
when issuing certificates against a private ACME Server which requires the use of an External Account Binding (in this case it's step-ca) the orders created fail because the JOSE Header, containing the 'kid', is missing in the POST request.
Registration against the ACME Server works as the cluster-issuer in question is in the status "The ACME account was registered with the ACME server"
describing the failed order shows the following output:
Name:         grafana-ing-cert-5d87j-2314769740
Namespace:    default
Labels:       <none>
Annotations:  cert-manager.io/certificate-name: grafana-ing-cert
              cert-manager.io/certificate-revision: 1
              cert-manager.io/private-key-secret-name: grafana-ing-cert-gqjhz
API Version:  acme.cert-manager.io/v1
Kind:         Order
Metadata:
  Creation Timestamp:  2023-04-25T14:24:53Z
  Generation:          1
  Managed Fields:
    API Version:  acme.cert-manager.io/v1
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .:
          f:cert-manager.io/certificate-name:
          f:cert-manager.io/certificate-revision:
          f:cert-manager.io/private-key-secret-name:
        f:ownerReferences:
          .:
          k:{"uid":"4d7312bc-70da-4f64-80e8-182fe10b9919"}:
      f:spec:
        .:
        f:dnsNames:
        f:issuerRef:
          .:
          f:group:
          f:kind:
          f:name:
        f:request:
    Manager:      cert-manager-certificaterequests-issuer-acme
    Operation:    Update
    Time:         2023-04-25T14:24:53Z
    API Version:  acme.cert-manager.io/v1
    Fields Type:  FieldsV1
    fieldsV1:
      f:status:
        .:
        f:failureTime:
        f:reason:
        f:state:
    Manager:      cert-manager-orders
    Operation:    Update
    Subresource:  status
    Time:         2023-04-25T14:24:53Z
  Owner References:
    API Version:           cert-manager.io/v1
    Block Owner Deletion:  true
    Controller:            true
    Kind:                  CertificateRequest
    Name:                  grafana-ing-cert-5d87j
    UID:                   4d7312bc-70da-4f64-80e8-182fe10b9919
  Resource Version:        21309239
  UID:                     ad1c903b-e26f-4b2a-9aa0-230353c6c123
Spec:
  Dns Names:
    my-ingress-hostname (placeholder)
  Issuer Ref:
    Group:  cert-manager.io
    Kind:   ClusterIssuer
    Name:   cert-manager
  Request:  LS0tLS1CRUdJTiBDRVJU..........VVFU1QtLS0tLQo=
Status:
  Failure Time:  2023-04-25T14:24:53Z
  Reason:        Failed to create Order: 400 urn:ietf:params:acme:error:malformed: Missing JOSE header 'kid'.
  State:         errored
Events:          <none>

decoding the newOrder POST request sent by cert-manager to the step-ca ACME Server shows the following Header:
{
  "alg": "RS256",
  "jwk": {
    "e": "AQAB",
    "kty": "RSA",
    "n": "5Z-7Ghuc.......xgTvuGeQ"
  },
  "nonce": "azHv--l6Je6KyjpMH-QQSgAAAYePYE5t",
  "url": "https://acme.my-internal-acme/newOrder"
}

according to RFC8555 Section 6.2 - Request Authentication
The kid field needs to be present in a newOrder POST Request.

The "jwk" and "kid" fields are mutually exclusive.  Servers MUST
reject requests that contain both.
For newAccount requests, and for revokeCert requests authenticated by
a certificate key, there MUST be a "jwk" field.  This field MUST
contain the public key corresponding to the private key used to sign
the JWS.
For all other requests, the request is signed using an existing
account, and there MUST be a "kid" field.  This field MUST contain
the account URL received by POSTing to the newAccount resource.

described in RFC8555 Section 7.4 - Applying for Certificate Issuance  a newOrder Request should be structured like this:
   POST /acme/new-order HTTP/1.1
   Host: example.com
   Content-Type: application/jose+json

   {
     "protected": base64url({
       "alg": "ES256",
       "kid": "https://example.com/acme/acct/evOfKhNU60wg",
       "nonce": "5XJ1L3lEkMG7tR6pA00clA",
       "url": "https://example.com/acme/new-order"
     }),
     "payload": base64url({
       "identifiers": [
         { "type": "dns", "value": "www.example.org" },
         { "type": "dns", "value": "example.org" }
       ],
       "notBefore": "2016-01-01T00:04:00+04:00",
       "notAfter": "2016-01-08T00:04:00+04:00"
     }),
     "signature": "H6ZXtGjTZyUnPeKn...wEA4TklBdh3e454g"
   }

which shows the existance of a 'kid' field, expected by the ACME Server Instance.
the cluster-issuer in use:
apiVersion: cert-manager.io/v1
kind: ClusterIssuer
metadata:
  name: cert-manager
spec:
  acme:
    # The ACME server URL
    server: https://acme.my-internal-acme/directory
    caBundle: LS0tL........FURS0tLS0tCg==
    externalAccountBinding:
      keyID: "https://acme.my-internal-acme/account/SOME-ID"
      keySecretRef:
        name: eabsecret
        key: secret
    privateKeySecretRef:
      name: cert-manager-key
    solvers:
    - http01:
        ingress:
          class: traefik

Expected behaviour:
when creating orders against an ACME Server with the use of an EAB, the newOrder POST Request should only contain the 'kid' field instead of the 'jwk' field, to be RFC8555 compliant.
Steps to reproduce the bug:

install cert-manager via helm with the following IngressShim values:

ingressShim:
  defaultIssuerName: "cert-manager"
  defaultIssuerKind: "cluster-issuer"
  defaultIssuerGroup: "cert-manager.io"

create an eabsecret containing the base64URL MAC key according to the  cert-manager docs
kubectl apply the above mentioned cluster-issuer
create an ingress resource that is supplied with the following annotations:

kubernetes.io/tls-acme: true
cert-manager.io/cluster-issuer: cert-manager

verify that the order fails via kubectl describe order

Anything else we need to know?:
Environment details::

Kubernetes version: v1.22.9
Cloud-provider/provisioner: vSphere with Tanzu
cert-manager version:  v1.11.1
Install method: helm

/kind bug

/vote

/vote

after looking into this Issue with the help of @dermorz, we found out that the ACME Server in question is not responding in a RFC Compliant way when cert-manager is POSTing against the NewAccount endpoint, resulting in an Error on the ACME Server side. RFC8555 Section 7.3.1 - Finding an Account URL Given a Key
This looks more like a feature request: the acmeapi.Client can be initialized with a fixed kid which would be used on the order call instead of retrieving the kid right before (every) order.
https://github.com/cert-manager/cert-manager/blob/5035dda25e8839ef0a0737efdb3df6ff1e97f6e1/pkg/acme/accounts/client.go#L48-L57


static: fixed display no user found

issue: https://github.com/zulip/zulip/issues/17842
Tested using ./tools/test-all
GIFs or screenshots:

@Pulkit007 Please check https://zulip.readthedocs.io/en/latest/contributing/version-control.html for the guidelines on how to write commit messages.

Also, you may go through other PRs open for this issue and take a look at the discussion there. It will give you some idea of the changes you further need to make in this PR. Like in this comment on #18790, Tim asked to add the translation tags to the text and style the text in italics.

This was fixed via a cleaner approach in #19729, thanks for working on this @Pulkit007!


Second parameter to closeModalWithEvents()?

Is there any way I can do this with closeModalWithEvents()?
$this->emit('deletePost', $this->post->id);
$this->closeModal();

This doesn't seem to work, since it's expecting an array:
$this->closeModalWithEvents('deletePost', $this->post->id);

This should do the trick:
$this->closeModalWithEvents(['deletePost' => $this->post->id]);

Thank you!
Unfortunately it doesn't seem to work. deletePost is not triggered.
What I need and what is working:
$this->emit('deletePost', $this->post->id);
$this->closeModal();

Result:

Not working:
$this->closeModalWithEvents(['deletePost' => $this->post->id]);

Result:

I mean, not the end of the world to do the first option, but it's a little cleaner with all in one if possible.
Thank you!

I may be missing something entirely.
 $this->closeModalWithEvents(['returnMasterName' => $master_name_id]);
or
        $this->emit('returnMasterName', $master_name_id);         $this->closeModal();
Using both of the above methods, I get the same error back:
ErrorException Attempt to read property "fullname" on int (View: /Users/treaves/Development/projects/tbhub/resources/views/livewire/mni/master-name-search.blade.php)
what is strange, master-name-search is the child modal that I am executing the command.  It was opened by "rim-person-form", which is also a modal.
"returnMasterName" is being listened for on "rim-person-form" (RimPersonForm), but I never get anything back on it.
RimPersonForm:
`<?php
namespace App\Http\Livewire\Rim;
use App\Models\Person_type;
use App\Models\rim_admin;
use App\Models\rim_person;
use LivewireUI\Modal\ModalComponent;
use Log;
class RimPersonForm extends ModalComponent
{
protected $listeners = [
'updateField' => 'updateField',
'openMasterName' => 'openMasterName',
'returnMasterName' => 'returnMasterName'
];
public $rim_admin_id;
public $person_id;
public $master_name_id;
public $last_name;
public $first_name;

public $person_types;
public $mode;

public function mount(rim_person $rim_person, rim_admin $rim_admin)
{
    $this->person_types = Person_type::all();

...
}
public function render()
{
return view('livewire.rim.rim-person-form');
}
public function openMasterName()
{
    $last_name = $this->last_name;
    $first_name = $this->first_name;

    $params = [
        'screen' => 'rim-person-form',
        'last_name' => $last_name,
        'first_name' => $first_name,

    ];

    $this->emit('openModal', 'mni.master-name-search', [$params]);
}

function returnMasterName($master_name_id)
{
    $this->master_name_id = $master_name_id;
    $this->first_name = $master_name_id;
}

public static function closeModalOnClickAway(): bool
{
    return false;
}
public static function closeModalOnEscape(): bool
{
    return false;
}

public static function modalMaxWidth(): string
{
    return '6xl';
}

}
`
MasterNameSearch:
`<?php
namespace App\Http\Livewire\Mni;
use App\Http\Livewire\Rim\RimPersonForm;
use App\Models\MasterName;
use Livewire\WithPagination;
use LivewireUI\Modal\ModalComponent;
class MasterNameSearch extends ModalComponent
{
//    protected $listeners = ['returnScreen' => 'returnScreen'];
public $screen;
public $last_name;
public $first_name;
public $ssno;
public $dlno;
public $state_id;
public $other_id;

public $results;

use WithPagination;

public function mount(array $params)
{
    $params = (object) $params;
    $this->screen = $params->screen;
    $this->last_name = $params->last_name;
    $this->first_name = $params->first_name;

    //        Log::debug(print_r($params, true));

    $master_name = MasterName::latest();
    if ($this->last_name != '') {
        dump('inside last name..');
        $master_name->where(
            'last_name',
            'like',
            '%' . $this->last_name . '%'
        );
    }
    if ($this->first_name != '') {
        dump('inside first name..');
        $master_name->where(
            'first_name',
            'like',
            '%' . $this->first_name . '%'
        );
    }
    $this->results = [$master_name->orderBy('last_name')->paginate(10)];
}

public function render()
{
    return view('livewire.mni.master-name-search');
}

public function ReturnBack($master_name_id)
{
    //        $this->closeModalWithEvents(['returnMasterName' => $master_name_id]);
    $this->emit('returnMasterName', $master_name_id);
    $this->closeModal();
}

public static function closeModalOnClickAway(): bool
{
    return false;
}
public static function closeModalOnEscape(): bool
{
    return false;
}

public static function modalMaxWidth(): string
{
    return '6xl';
}

}
`


Rework PathCommandArray into AttributePath

Reworks PathCommandArray into an AttributePath that extends Attribute. PathCommand is in its own class now also. The reason for this is that there is a default list of commands, which should reduce the need to parsing paths multiple time.
This might help with performance, although I'm not sure it will have much of an effect. Helps advance towards solving #53.
Also refactors NumberField to not use arrows, as they just didn't seem all that useful.  The PR some visuals, makes it so number field text sticks out when it's different from the default value.
Sliders are implemented in number fields to replace arrows,but they are only optionally used when the number field has a min and max. Number fields are now a bit bigger by default when without a slider, but still smaller than before (Fixes #70)

HOLY SHIT I ACCIDENTALLY FIXED THE BIGGEST BUG


Processing crashes when trying to open or save a file

Summary
Processing 4 beta 1 is using OpenJDK 11.0.12, but it crashes when some dialog shows up. Dialogs like the ones that open when trying to save or open a file.
Triaging info
Java version:
OpenJDK Runtime Environment Temurin-11.0.12+7 (build 11.0.12+7)
OpenJDK 64-Bit Server VM Temurin-11.0.12+7 (build 11.0.12+7, mixed mode)
What is your operating system and platform?
Archlinux on x86-64
How did you install Java?
It comes with Processing.
Report log
---------------  S U M M A R Y ------------
    20
    21 Command Line: -Djna.nosys=true -Djava.library.path=lib:modes/java/libraries/javafx/library/linux64 --module-path=modes/java/libraries/javafx/library/linux64/modules --add-modules=javafx.base,javafx.controls,javafx.fxml,javafx.graphics,javafx.media,javafx.swing,       javafx.web -Xmx512m processing.app.ui.Splash
    22
    23 Host: Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz, 12 cores, 15G, Arch Linux
    24 Time: Mon Aug  9 09:17:13 2021 PDT elapsed time: 14.089794 seconds (0d 0h 0m 14s)
    25
    26 ---------------  T H R E A D  ---------------
    27
    28 Current thread (0x00007f0878b8b000):  JavaThread "ShowDialog" [_thread_in_native, id=2340, stack(0x00007f08471e1000,0x00007f08472e2000)]
    29
    30 Stack: [0x00007f08471e1000,0x00007f08472e2000],  sp=0x00007f08472dd9a8,  free space=1010k
    31 Native frames: (J=compiled Java code, A=aot compiled Java code, j=interpreted, Vv=VM code, C=native code)
    32 C  0x00007f07f1fb20cc
    33
    34 Java frames: (J=compiled Java code, j=interpreted, Vv=VM code)
    35 j  sun.awt.X11.GtkFileDialogPeer.run(Ljava/lang/String;ILjava/lang/String;Ljava/lang/String;Ljava/io/FilenameFilter;ZII)V+0<EMAIL_ADDRESS>    36 j  sun.awt.X11.GtkFileDialogPeer.showNativeDialog()V+156<EMAIL_ADDRESS>    37 j  sun.awt.X11.GtkFileDialogPeer.lambda$setVisible$0()V+1<EMAIL_ADDRESS>    38 j  sun.awt.X11.GtkFileDialogPeer$$Lambda$415.run()V+4<EMAIL_ADDRESS>    39 j  java.lang.Thread.run()V+11<EMAIL_ADDRESS>    40 v  ~StubRoutines::call_stub
    41
    42 siginfo: si_signo: 11 (SIGSEGV), si_code: 2 (SEGV_ACCERR), si_addr: 0x00007f07f1fb20cc
    43
    44 Register to memory mapping:
    45
    46 RAX=0x00007f07f1fb20cc points into unknown readable memory: 00 01 00 00
    47 RBX=0x00007f08000c3f28 points into unknown readable memory: 0x0000000000000000 | 00 00 00 00 00 00 00 00
    48 RCX=0x0 is NULL
    49 RDX=0x0000000100000036 is pointing into metadata
    50 RSP=0x00007f08472dd9a8 is pointing into the stack for thread: 0x00007f0878b8b000
    51 RBP=0x00007f08472dd9b0 is pointing into the stack for thread: 0x00007f0878b8b000
    52 RSI=0x0000000066766172 is an unknown value
    53 RDI=0x00007f08000c40b0 points into unknown readable memory: 0x0000000000000001 | 01 00 00 00 00 00 00 00
    54 R8 =0x00007f08d5e9d860: <offset 0x000000000001f860> in /home/ivanleon/Software/Processing/processing-4.0b1/java/lib/libharfbuzz.so at 0x00007f08d5e7e000
    55 R9 =0x00007f0800000080 points into unknown readable memory: 0x00007f0800179fe0 | e0 9f 17 00 08 7f 00 00
    56 R10=0x00007f08000c40f0 points into unknown readable memory: 0x0000000000000000 | 00 00 00 00 00 00 00 00
    57 R11=0x0 is NULL
    58 R12=0x0 is NULL
    59 R13=0x00007f08d5d290c0: <offset 0x000000000009e0c0> in /usr/lib/libharfbuzz.so.0 at 0x00007f08d5c8b000
    60 R14=0x00007f08472dd9c0 is pointing into the stack for thread: 0x00007f0878b8b000
    61 R15=0x00007f08000bc050 points into unknown readable memory: 0x00007f08000a8a60 | 60 8a 0a 00 08 7f 00 00

This is a duplicate of #335.


spring-cloud-reactivefeign-core module added

Fixes #4

Codecov Report

Merging #11 into master will increase coverage by 1.09%.
The diff coverage is 81.03%.

@@             Coverage Diff              @@
##             master      #11      +/-   ##
============================================
+ Coverage     75.88%   76.97%   +1.09%
- Complexity      281      363      +82
============================================
  Files            36       47      +11
  Lines          1219     1546     +327
  Branches        183      209      +26
============================================
+ Hits            925     1190     +265
- Misses          214      261      +47
- Partials         80       95      +15

Impacted Files
Coverage Δ
Complexity Δ

...java/feign/reactive/ReactiveInvocationHandler.java
36.36% <36.36%> (ø)
3 <3> (?)

...in/java/feign/reactive/client/ReactiveRequest.java
47.61% <47.61%> (ø)
5 <5> (?)

...src/main/java/feign/reactive/ReactiveRetryers.java
62.5% <62.5%> (ø)
4 <4> (?)

.../src/main/java/feign/reactive/ReactiveOptions.java
66.66% <66.66%> (ø)
2 <2> (?)

...efeign-core/src/main/java/feign/ReactiveFeign.java
80% <80%> (ø)
4 <4> (?)

...ava/feign/reactive/client/RetryReactiveClient.java
80.95% <80.95%> (ø)
6 <6> (?)

...eign-core/src/main/java/feign/reactive/Logger.java
82.14% <82.14%> (ø)
9 <9> (?)

...re/src/main/java/feign/reactive/ReactiveUtils.java
87.5% <87.5%> (ø)
1 <1> (?)

...ava/feign/reactive/ReactiveDelegatingContract.java
93.33% <93.33%> (ø)
7 <7> (?)

.../java/feign/reactive/client/WebReactiveClient.java
96.42% <96.42%> (ø)
16 <16> (?)

... and 12 more

@spencergibb A bit confused with "Some checks were not successful" as I just renamed package and reformatted code

@kptfh its just because code coverage targets were not hit.  We are not sticklers about it but if you want to try an increase the overage, that is always welcome.

@ryanjbaxter Plan to add some Unit tests but it will take some time.

@spencergibb @ryanjbaxter Renamed package, reformatted code, added tests.

We need to add some documentation.  Also I think you need to add the module to the starter pom.

Hey guys,
if you were building/testing something here's a workaround for the time being:
`public class ReactiveDecoder implements Decoder {
private final Decoder delegate;

public ReactiveDecoder(Decoder delegate) {
    Objects.requireNonNull(delegate, "Decoder must not be null. ");
    this.delegate = delegate;
}

@Override
public Object decode(Response response, Type type) throws IOException {
    boolean mono = isMono(type);
    boolean flux = isFlux(type);

    if (!flux && !mono) {
        return delegate.decode(response, type);
    }

    if (response.status() == 404) {
        if (mono) {
            return Mono.empty();
        }

        return Flux.empty();
    }

    if (mono) {
        Type enclosedType = Util.resolveLastTypeParameter(type, Mono.class);
        return Mono.just(delegate.decode(response, enclosedType));
    }

    Type enclosedType = Util.resolveLastTypeParameter(type, Flux.class);
    return Flux.just(delegate.decode(response, enclosedType));
}

private boolean isFlux(Type type) {
    if (!(type instanceof ParameterizedType)) {
        return false;
    }
    ParameterizedType parameterizedType = (ParameterizedType) type;
    return parameterizedType.getRawType().equals(Flux.class);
}

private boolean isMono(Type type) {
    if (!(type instanceof ParameterizedType)) {
        return false;
    }
    ParameterizedType parameterizedType = (ParameterizedType) type;
    return parameterizedType.getRawType().equals(Mono.class);
}

}`
and then substitute the default bean for this one:
`    @Autowired
private ObjectFactory messageConverters;
@Bean
public Decoder feignDecoder() {
    return new ReactiveDecoder(new OptionalDecoder(new ResponseEntityDecoder(new SpringDecoder(this.messageConverters))));
}`

this worked for me.
Hope it helps someone

@kptfh do you plan on picking this back up?

@ryanjbaxter Sorry, but your questions is not clear to me. Do you mean take a look at @riba2101's comment?

Sorry treated your inactivity as you've lost any interest to this PR. Seemed that I fixed all your comments and had no feedback. So I made some bunch of changes in original repository and will need to redo all this stuff with renaming of packages again. Actually I still have a strong intention to complete this work but will not be able to do it without your help and good communication.

I was waiting for you to add documentation as I asked for above.

Literally, you said:

We need to add some documentation. Also I think you need to add the module to the starter pom
So I added module to the starter pom but didn't expect that you mean me to add doc.
Finally it's not clear what kind of doc to add: java doc(which classes?), how-to doc?

We would want documentation on how to use the feature added to https://github.com/spring-cloud/spring-cloud-openfeign/blob/master/docs/src/main/asciidoc/spring-cloud-openfeign.adoc

Hi @ryanjbaxter. Added adoc to spring-cloud-reactivefeign-core. Want to remind that this module does't support Ribbon or Hystrix. This support will be added in the next "spring-cloud-reactivefeign-cloud" module

Also faced some issue with Eclipse formatter with Spring Cloud settings. You may note that some method calls are not aligned properly.

This support will be added in the next "spring-cloud-reactivefeign-cloud" module

what does this mean?

It was agreed with @spencergibb  that there are 2 modules:
spring-cloud-reactivefeign-core -- core logic
spring-cloud-reactivefeign-cloud -- cloud specific logic (ribbon/hystrix support)
To keep pull requests small decided to include in this PR only core module.
Here you can see these modules https://github.com/kptfh/feign-reactive

Got it.
I dont think the docs for this need to be its own file, it is fine to add them to the existing docs file.

@ryanjbaxter appended to spring-cloud-openfeign.adoc

Hi, @ryanjbaxter. Any comments or proposals?

Thanks @kptfh.  Right now we are going to focus on getting the Finchley RELEASE out the door.  I am going to label this for SR1 for now.  It might take me a few days to get to look at this again.

@ryanjbaxter Does it make sense to add spring-cloud-reactivefeign-cloud module (cloud specific logic - ribbon/hystrix support ) to this pull request to review it all together?

@ryanjbaxter relase is out, can you merge changes or there is additional work required?

Hopefully we can get this merged this week.  We might be doing another Finchley service release earlier than planned do to an issue that popped up in another project.  So I might wait until after that.

I am coming into this very late, and I don't want to derail this, but coming from feign, this PR contains more feign code than Spring.  In fact, with a little more refactoring, this could be a PR into feign directly.  I'm confused as to why @kptfh you did not consider submitting this to feign?

I hadn't realized that with a minor refactor it could go in an open feign module. I think that might be a good idea. Reactor could certainly be used outside of spring.

I don't see any reason why we could not create a feign-reactive-reactor sub module with the changes included in spring-cloud-reactivefeign-core module.  The refactoring would be around the Retry, as the current Retryer interface in feign would not work in this case, but I'm sure we can make it work.
This feature has been widely requested within feign as  mentioned in issues https://github.com/OpenFeign/feign/issues/622 and https://github.com/OpenFeign/feign/issues/644 and https://github.com/OpenFeign/feign/issues/361.  feign is now baselined on JDK 8, so I don't see anything really in the way here.
I'm more than willing to help out where I can with this.

@kdavisk6 This version of reactive feign depends on Spring's WebClient as reactive http transport.
Also don't see a lot of benefits to use reactive approach on Android as it suits more highly loaded server side (saves memory and CPU on reducing number of threads)

Looking over the PR again, I can see that the WebReactiveHttpClient is the only component that relies on Spring.  The rest of the PR is pure Feign.
The way this PR is constructed currently, it copies a few key core objects from feign-core verbatim into spring-cloud.  This could cause issues in the future.  For example, #34 requests that the spring-cloud update to OpenFeign 10.x, which will break this integration.  Also, @spencergibb has mentioned in the #4 that he would prefer that nothing of feign make it into spring-boot.  This PR will do just that, it will now have it's own copy of feign will be decoupled from feign without a fork.
One way to ensure that both projects can benefit is to take the Feign related items such as ReactiveHttpClient, ReactiveFeign, ReactiveEncoder, ReactiveDecoder, etc... and add them to OpenFeign under feign-reactive and keep the WebReactiveHttpClient here with Spring Cloud.  This way, both projects get the benefit.  feign will have the reactive framework and Spring is still able to take advantage of it without the duplication.  Thoughts?

@kptfh We can certainly create a repository under OpenFeign similar to what was done for Vert.x for this work.  I'm volunteering my time to help you make this happen.  To help coordinate, can we move that discussion over to https://github.com/OpenFeign/feign/issues/644?

@spencergibb  what do you think about this?

Leave PR here as is or
split into open feign part (reactive framework ReactiveHttpClient  without implementation)  and Spring part (based on WebClient)?

I think it makes the most sense to keep as much as we can in OpenFeign.

:+1:

@spencergibb @ryanjbaxter Can you comment on this issue https://github.com/OpenFeign/feign-reactive/issues/15  ?

@ryanjbaxter @spencergibb stack into dead end with openfeign repo. Don't see any benefits to keep code there. No one is reviewing the code except one guy (@kdavisk6) that don't understand how reactive approach should work. It will last for years and from my side looks like sabotage or time killing.
Please, take a look at this issue https://github.com/OpenFeign/feign-reactive/issues/15 and pull request https://github.com/OpenFeign/feign-reactive/pull/18

hello @spencergibb, could you please advise what should be done to proceed communication on this pr? obviously, there is no common vision of what should or should not be done with reactive-feign right now. maybe we could find some temporary solutions for the not agreed issues arisen during review?

@Aloren we do not want to duplicate code from openfeign in this repo.  I would rather keep this downstream.  From https://github.com/OpenFeign/feign-reactive/issues/15 it sounds like the downstream code was going to be moved to https://github.com/kptfh/feign-reactive.  Maybe the best approach then is to have a separate corresponding repo for an reactive spring cloud openfeign module that depends on that code.  I could talk to the team and see if they would mind that being a community driven project in our incubator if the community is interested in that.  Once the code in https://github.com/kptfh/feign-reactive can be moved into OpenFeign then we consider moving that incubator module into this project.

Closing, as the team has taken the decision that we will add integration only whenever there's reactive support in OpenFeign core.


ref: replaced dvc run references with dvc stage and dvc exp run in dvc dag

replaced dvc run references with dvc stage and dvc exp run
Closes #3183

😅 I should have seen this before I submitted #3223

I can close this or merge and rebase the other. WDYT @jorgeorpinel ?

If #2997 is closed this no longer seems needed (already addressed)...

Closing this, as we merged #3223

Oh haha sorry, we're working on this synchronously it seems 😬


iOS: Check target_arch and adapt framework path

Add support for x86 simulator

@EliseChouleur from my side works on these targets locally:

x86_64-apple-ios
x86_64-linux-android
i686-linux-android

Could you test with -vvv?

@qvignaud can we merge yet?


table route config and metadata config

can you talk about table route config and metadata config?can you give me an example and tell me the purpose about table route?
thanks!

Because  i want to use pinot in our project,but  i can not find encough  document  to support ? i am very glad to receive message from you.
thanks!


python3Packages.zeroconf: 0.30.0 -> 0.31.0

Motivation for this change
Update to latest upstream release 0.31.0
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@ofborg build home-assistant

@ofborg build python38.pkgs.zeroconf python39.pkgs.zeroconf

Result of nixpkgs-review pr 123567 at a70176cd run on aarch64-linux 1

1 package marked as broken and skipped:

expliot

1 package failed to build:

home-assistant

37 packages built successfully:

calibre
catt
cinnamon.warpinator
cura
curaLulzbot
dpt-rp1-py
gnomecast
home-assistant-cli
mkchromecast
octoprint
python38Packages.HAP-python
python38Packages.PyChromecast
python38Packages.aioesphomeapi
python38Packages.aiohomekit
python38Packages.devolo-home-control-api
python38Packages.libpurecool
python38Packages.libsoundtouch
python38Packages.netdisco
python38Packages.openwebifpy
python38Packages.pyatv
python38Packages.python-miio
python38Packages.pyvizio
python38Packages.webthing
python38Packages.zeroconf
python39Packages.HAP-python
python39Packages.PyChromecast
python39Packages.aioesphomeapi
python39Packages.devolo-home-control-api
python39Packages.libpurecool
python39Packages.libsoundtouch
python39Packages.netdisco
python39Packages.openwebifpy
python39Packages.pyatv
python39Packages.python-miio
python39Packages.pyvizio
python39Packages.webthing
python39Packages.zeroconf

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

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

2 packages failed to build:

esphome
platformio

39 packages built:

calibre
catt
cinnamon.warpinator
cura
curaLulzbot
dpt-rp1-py
expliot
gnomecast
home-assistant
home-assistant-cli
mkchromecast
octoprint
python38Packages.HAP-python
python38Packages.PyChromecast
python38Packages.aioesphomeapi
python38Packages.aiohomekit
python38Packages.devolo-home-control-api
python38Packages.libpurecool
python38Packages.libsoundtouch
python38Packages.netdisco
python38Packages.openwebifpy
python38Packages.pyatv
python38Packages.python-miio
python38Packages.pyvizio
python38Packages.webthing
python38Packages.zeroconf
python39Packages.HAP-python
python39Packages.PyChromecast
python39Packages.aioesphomeapi
python39Packages.devolo-home-control-api
python39Packages.libpurecool
python39Packages.libsoundtouch
python39Packages.netdisco
python39Packages.openwebifpy
python39Packages.pyatv
python39Packages.python-miio
python39Packages.pyvizio
python39Packages.webthing
python39Packages.zeroconf

platformio failure will be addressed with https://github.com/NixOS/nixpkgs/pull/123450

@ofborg build python39.pkgs.zeroconf


Depth Target change resolution in one axis will lead to bug.

On this line: https://github.com/playcanvas/engine/blob/master/src/scene/forward-renderer.js#L924
Condition assumes that changing of render target resolution have to happen with width and height at the same time, but what if resolution is changed in horizontal or vertical direction only.
Then it will not recreate render target, which will lead to a bug.
And meshInstance is not defined in a loop above, how does it checks it's blending for first iteration?:
https://github.com/playcanvas/engine/blob/master/src/scene/forward-renderer.js#L953

Fixed
https://github.com/playcanvas/engine/pull/590
https://github.com/playcanvas/engine/pull/591


Recommended best practice for maxAge minAge

What is your recommendation for using these terms in the case where a fossil specimen is not dated directly. Instead dates apply to the context. How then shall we apply minAge maxAge and chronometricAgeUncertainty? For example, a specimen derives from somewhere in the Lower Laetolil Beds at Laetoli, TZ. This deposit has K-Ar dates for the upper most and lower most marker tuffs of 3.627 +- 0.018 and 3.85 +- 0.02 respectively.  Is the maxAge 3.85 or 3.87 Ma?  If there's only a single term for uncertainty how shall it be applied when values differ? Best to use the largest uncertainty, e.g. 0.02 in this example?

@dennereed Do you feel there is a need for separate uncertainties around minimum and maximum ages? If they aren't really the same, it seems like maybe there is, to be able to be rigorous.

@tucotuco Yes, adding separate uncertainties for min and max ages would help.

There was an example case we had from Edward Davis where the specimen was dated by being sandwiched between two dated tuff layers. In reporting this, I also had problems with reporting uncertainty and the material dated. I split the uncertaintyMethods and materialDated into these subfields, but perhaps we should consider splitting these terms for max & min ages? Is there a neat way to do this, @tucotuco , where it doesn't make things so ugly when only one method was used? That is my main concern; it may make things more confusing when you want to report a more traditionally dated specimen?
minimumChronometricAgeUncertaintyInYears: 0.016 Ma
maximumChronometricAgeUncertaintyInYears: 0.05 Ma
maximumAgeMaterialDated:Double Tuff
minimumAgeMaterialDated: Hawk Rim Tuff

I think this is the way to have all of the information in the more complex case. I would fill in all fields always, even when there is one MaterialData for both.

I agree that the chronology extension would be well served by having uncertainty for both max and min dates. But verbatim and notes fields can also be used to provide more detailed information in any situation where it isn't clear what best practice should be. For e.g. deenereed your verbatim field in this case could say "Specimen derives from somewhere in the Lower Laetolil Beds at Laetoli, TZ. This deposit has K-Ar dates for the upper most and lower most marker tuffs of 3.627 +- 0.018 and 3.85 +- 0.02 respectively"
My hunch, had the specimen been mine, would be to put the max/min dates as given (3.627 and 3.85)  and then the uncertainty as the sum of the two ranges, but I think without having the verbatim information, it would be difficult then for a data-user to know what decision had been made for the input.

In terms of radiocarbon (and probably all absolute dating methods?), I’d expect most users will think of the minimum and maximum ages as the upper and lower limits of the 2-sigma range, not the 1-sigma “error” or “uncertainty.” With radiocarbon at least (and probably K-Ar?), the convention is to report with the standard error (1-sigma). But that is not conservative enough for min and max ages. For example, for a date reported as 1000 +/- 30 radiocarbon years before present we’d want to give radiocarbon ages of 940 BP (min) and 1060 BP (max), not 970 BP and 1030 BP.  I’d also wager that calibrated calendar years will be more useful for searching for than radiocarbon years, so it would be helpful to have that included as well.

How would you propose handling complex ranges derived from calibrated radiocarbon dates?
In the example below the 'easiest' range is AD 1575 to present, but the statistically 'probable' range is made up of 5 separate min-max spans.

@visead We are leaving it up to data providers to decide what range/statistical uncertainty they want to report, but providing a reference where these different ranges are reported would be advantageous so data users can decide for themselves what range they prefer for their purposes. Alternatively, you could report all these ranges with their uncertainties as different chronometricAge entries for the same specimen(s).


Test fails in physical device

Hi, Can we run the pact tests in physical device? I am getting below errors
"Verification error (check build log for mismatches): Error"
"Waited more than 1.0 seconds"
Even tested with PactSwiftExample project & getting same error.

Hi Vijay. Is PactMockService running at the time you are trying to run your tests?

Hi Marko. I am able to run in iOS simulator successfully. I have issue only when i run in Physical device. I have few doubts

How will i know that PactMockService is running?
Where this PactMockService will run? locally in device or Mac? How to implement

Here is the code snippet which i am using
let verificationService = PactVerificationService(url: "http://localhost", port: 1_234)
provider = MockService(provider: "SomeName Provider", consumer: "SomeName iOS", pactVerificationService: verificationService)

@vijay2pvk In the xcode setup instructions you probably added a pre-action step which contains the  command to start the mock service and set the port. i.e.
pact-mock-service start --pact-specification-version 2.0.0 --log "${SRCROOT}/tmp/pact.log" --pact-dir "${SRCROOT}/tmp/pacts" -p 1234
The last parameter -p 1234 specifies that it will run on port 1234. i.e. The mock server should be running on http://localhost:1234 of your mac.
When running in the simulator then the ip address of the simulator and the host computer are the same and using PactVerificationService(url: "http://localhost", port: 1234) works.
However, if running on device, then the host machine is different and localhost will not work. You must then specify the ip address of your mac, where the mock service is running.
Pact is a tool for testing the contracts between a client and server.  I would recommend that you test on a simulator, keep your pact tests localized to only testing your server calling code. There should be nothing that is device specific in your tests that warrant running on a device. This will make the tests easier to understand, breakages easier to debug, and less likely to be flaky due to device / UI issues.

If you're running the mock service on a different host, you'll also need to start it up with --host <IP_ADDRESS> so that it is externally accessible. By default, it binds to localhost.
But as Andy says, you don't typically use the mock service with real devices.  The best use of pact is to test just the code in your consumer that is responsible for calling your provider https://docs.pact.io/best_practices/consumer#use-pact-for-isolated-unit-tests

Thanks Andrew, Beth for your quick response.

Hi, I am using pact consumer in my project, real time device only able to run my project unable to run in simulator.
my local ip address is = <IP_ADDRESS>, iam getting this error
failed - Error setting up pact: The request timed out.
failed - Verification error (check build log for mismatches): The request timed out.
My code
let verificationService = PactVerificationService(url: "http://<IP_ADDRESS>", port: 1234)
mockService = MockService(provider: "Provider", consumer: "iOS", pactVerificationService: verificationService)


fix(recognition): throw result

Throw result when nothing is recognized instead of a new error.

Throwing an error and handling that somewhere seems to be the proper way of doing things, no? What good does throwing the whole result do? Please inform me

Because this is how I handle any exceptions:
.catch(error => {
        downloadUrl.value = error.recognition.audioUrl;
        downloadUrl.removeAttribute('disabled');
}

An exception can come from within the streaming operation or the throw I am changing. I'd like for a single way of handling them, instead of unwrapping the Error in the special case where nothing is recognized.


Add Heroku Kong deployment listing to Hub

Awaiting additional review from Kong Inc. colleague...

Looks good to me

Merged and deployed - thanks for your contribution @mars - you have earned the special distinction of being the first ever post-launch contribution to the Kong Hub https://docs.konghq.com/hub/ 🥇

Woo hoo! 🎉  Thanks to you folks for your continuous drive improving Kong 🙌


Add .travis.yml

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


fix: not updating layers when mjml or json source code is set to readonly

This will fix the issue mentioned here: https://github.com/zalify/easy-email/issues/325#issuecomment-1750582666
Even tho we were specifying the read-only mode, the blur event was still fired from the input. To prevent any update (like you would expect from a read-only mode), we added this check before updating the code.

Will be released in the next few days

@m-Ryan , any idea when we could get a release with the fix? Thank you!

@m-Ryan , any idea when we could get a release with the fix? Thank you!

Sorry, it has already been updated to the latest version.


feat: flag to only show matching results

Hi,
I've added a flag to only display matching results.

Hey @jon4hz, Thanks.
I think it's better to have it the other way around:
By default, we show only founded sites, and with a --all or -a flag, we show everything.
What do you think?

Yeah that makes sense. I'll update my PR :)


请问有没有c语言实现的烧录esp32 FW的例程 (IDFGH-11854)

Is your feature request related to a problem?
由于需要从host 芯片上升级esp32模块，host上不能跑python，请问有没有c语言实现的烧录esp32的代码？
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response

Please try https://github.com/espressif/esp-serial-flasher


Enable Heat in Kuttl tests

/test openstack-operator-build-deploy-kuttl

/test openstack-operator-build-deploy-kuttl

/test openstack-operator-build-deploy-kuttl

recheck

/test openstack-operator-build-deploy-kuttl

No Keystone, Rabbit, Memcached it seems:
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/openstack-k8s-operators_openstack-operator/1152/pull-ci-openstack-k8s-operators-openstack-operator-main-openstack-operator-build-deploy-kuttl/1853373947383386112/artifacts/openstack-operator-build-deploy-kuttl/openstack-k8s-operators-gather/artifacts/must-gather/quay-io-openstack-k8s-operators-openstack-must-gather-sha256-3d1f620ae3617fbd6371f55aa948b34e9daf31a461d3c557cf1a7c2db38d0385/namespaces/openstack-operators/pods/heat-operator-controller-manager-7dbdd64447-nvdb6/logs/manager-previous.log


Eval-ing modules twice when [supposedly] using relative paths in require

First reported in NativeScript/nativescript-angular#38. It appears that in Angular projects we can require rxjs/Observable twice. This leads to some issues when attaching extension operators (e.g. map to the Observable prototype):

Operator methods get attached to the first prototype definition when requiring rxjs/add/operator/map. Done in application code (as documented in the RXJS and Angular docs).
The second require (inside angular2/src/facade/async) redefines the Observable function and prototype, of course without the extension methods. That causes a crash in client code that relies on map being available.

The problem is reproducible in this project (use the login-http-get branch).
My observations so far:

This doesn't happen on Android.
The problem goes away if I change all ../Observable, ../../Observable, etc requires to rxjs/Observable in the application, rxjs/add/operator/map and the Angular async facade code.

Is this behavior observed on the latest 1.5 release, or in the master branch?

Official 1.5 release.


Fix referencing TileService on older devices.

Use a different component for 24+, and only reference this component on 24+.
Fixes #166.

for future searchers, you can use strings instead of classes for the keys as of Dagger 2.17 with the flag -Adagger.android.experimentalUseStringKeys.


Down Host Fix

Hi,
I was looking for a Cassandra plugin for New Relic and found yours. It works great. I made a small enhancement. Can you please take a look of this pull request. Thanks!
https://github.com/thoersch/new-relic-cassandra/pull/4
Jason

Closing. merged on the 11th of nov


MRG: Add shift_time method to Epochs

This adds the shift_time method to the Epochs class as discussed in #5642.
Relevant unit tests were added, flake8 and pydocstyle are happy.
Please review.

@agramfort done

Thanks @thht


IOS move map to location the map will not center on location but will move location to upper left corner

If I move to a lat lon position the position will be in the upper left corner.  Or if I put a marker on the map then I click on it the map will put the marker in the upper left where I can't even see it when it should put it in the center.   It's basically unusable in IOS.

Hi @buttonsquared
Can you please provide your flutter doctor -v your flutter run --verbose and a complete reproducible minimal code sample
Thank you


feat: embedDocuments to support progress callback

https://github.com/hwchase17/langchainjs/blob/f4b146471f967faa95071337caaf22658e90346e/langchain/src/embeddings/base.ts#L16
Would be useful for Embeddings.embedDocuments to have a second optional argument progress?: (p:number)=>void to signal the proportion of documents whose embed results have arrived. This would then be used for UI reporting in case there are many documents that take time to progress so a progress bar can be displayed.
If a maintainer thinks this is a good idea, I can contribute a PR for this

CC @ppramesi would this fit in with your changes?

Yeah I think it can fit the current embedding callback pull request. I can create something called handleEmbeddingProgress function or something like that in the callback handler/manager. @nfcampos thoughts on this?

How about a more generic handleProgress({current: number, total: number}, ...) which can be used now by embeddings and in future by some of the other run types?

You mean like adding handleProgress in callback handler/manager that can be called from other run types?

Yes I agree it should be a separate PR


Implement argMax and argMin

@huningxin @fdwr @miaobin PTAL, thanks.

@huningxin I've updated commit to address your comments, please take another look, thanks.

@fdwr I've updated commit to address your comments, please take another look. Thanks a lot!


Ml tests p

merging to add tests

Note merge done by same user who requested merge but done so with team members at in person meeting so checked off by team.


Station Power Flicker Issue

Description
Sometimes power flickers around the station, the issue seems to be with the SMES
Power grids usually goes Generator>SMES>Substation>APC
which splits the HV wire grids into two sections
The power flicker happen every time the SMES(s) drop from 100% to 99% charge, the flicker can easily fixed in character by making a power grid go Generator>SMES & Substation> APC to avoid having the SMES make a bottle neck effect, i am unsure as to why the SMES are making a bottle neck effect, but it hampers science in their artifact research heavily, since the power flicker will stop the artifact scanner, reseting the scan to 0%.
Requires sound to be able to hear the power flicker
https://github.com/space-wizards/space-station-14/assets/94384139/840d0411-e910-4e59-ae27-3bbd60026638
connecting the split HV grids before and after photos
Before:
Power flicker happens here
After:
Power flicker is fixed here by connecting the split grids which got separated by the wire terminals which are need for the SMES to charge
This issue is related to issue #22758 and possibly #20313
if you got additional info about this issue then please comment below

This is just how Pow3r is designed to work, the SMES should only "drop from 100% to 99%" if the engine suddenly cuts out. The point is that "the engine suddenly cuts out" still matters.


azure-mgmt-documentdb

Current coverage is 57.60% (diff: 91.88%)

Merging #903 into master will increase coverage by 0.29%

@@             master       #903   diff @@
==========================================
  Files          1758       1779    +21
  Lines         48752      49171   +419
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          27940      28325   +385
- Misses        20812      20846    +34
  Partials          0          0

Powered by Codecov. Last update 69da980...bab58c6

Merging even if RestAPI not fixed, package is usable and workaround available.


feat: add a check whether "Homepage" pyproject metadata is reachable

I realized that tree-sitter-cli automatically generates a "Homepage" field as github.com/tree-sitter/tree-sitter-<grammar>. This is mostly not correct when the parser is created outside of the tree-sitter organization.
We should probably reconsider how to "Homepage" fields is generated. Anyways a small check whether the Homepage in pyproject.yaml is reachable doesn't harm anyone.

hmm, you must manually enable test-python to see this. So the check will probably not be seen by anyone. It probably is placed better in the publish action.
https://github.com/tree-sitter-grammars/tree-sitter-slang/pull/27

Thank you but this is out of scope. There are several little things that need to be manually fixed by the user, until an init command is introduced. Note that the template does handle this for new projects.


package installation problem

將程式套件安載入 ‘C:/Users/user/Documents/R/win-library/4.1’
(因為 ‘lib’ 沒有被指定)
Error: could not find function "install.packages"
Execution halted
Warning in i.p(...) :
installation of package ‘C:/Users/user/AppData/Local/Temp/RtmpU5vERJ/file353091a2633/econWeb_0.1.3.tar.gz’ had non-zero exit status

請問遇到以下狀況該如何解決

devtools::install_github("tpemartin/econWeb", force = T)
WARNING: Rtools is required to build R packages, but is not currently installed.

devtools::install_github("tpemartin/econWeb", force = T)
Downloading GitHub repo tpemartin/econWeb@HEAD
These packages have more recent versions available.
It is recommended to update all of them.
Which would you like to update?

1: All
2: CRAN packages only
3: None
4: cli  (3.1.1 -> 3.2.0) [CRAN]
5: yaml (2.2.2 -> 2.3.4) [CRAN]
Enter one or more numbers, or an empty line to skip updates: 1
evaluate (NA    -> 0.15 ) [CRAN]
cli      (3.1.1 -> 3.2.0) [CRAN]
yaml     (2.2.2 -> 2.3.4) [CRAN]
Installing 3 packages: evaluate, cli, yaml
有二進位版本的，但原始碼版本是後來的:
binary source needs_compilation
evaluate   0.14   0.15             FALSE
cli       3.1.1  3.2.0              TRUE
yaml      2.2.2  2.3.4              TRUE
trying URL 'https://cran.rstudio.com/bin/windows/contrib/4.1/cli_3.1.1.zip'
Content type 'application/zip' length 1236949 bytes (1.2 MB)
downloaded 1.2 MB
trying URL 'https://cran.rstudio.com/bin/windows/contrib/4.1/yaml_2.2.2.zip'
Content type 'application/zip' length 208014 bytes (203 KB)
downloaded 203 KB
package ‘cli’ successfully unpacked and MD5 sums checked
Warning: cannot remove prior installation of package ‘cli’
Warning: restored ‘cli’
package ‘yaml’ successfully unpacked and MD5 sums checked
The downloaded binary packages are in
C:\Users\user\AppData\Local\Temp\RtmpMR279x\downloaded_packages
安裝原始碼套件 ‘evaluate’
trying URL 'https://cran.rstudio.com/src/contrib/evaluate_0.15.tar.gz'
Content type 'application/x-gzip' length 25617 bytes (25 KB)
downloaded 25 KB
錯誤發生在 library(econWeb)：不存在叫 'econWeb' 這個名稱的套件
停止執行
The downloaded source packages are in
‘C:\Users\user\AppData\Local\Temp\RtmpMR279x\downloaded_packages’
√  checking for file 'C:\Users\user\AppData\Local\Temp\RtmpMR279x\remotes1a8043ee55a5\tpemartin-econWeb-d6ee270/DESCRIPTION' ...

preparing 'econWeb':
√  checking DESCRIPTION meta-information ...
checking for LF line-endings in source and make files and shell scripts
checking for empty or unneeded directories
Omitted 'LazyData' from DESCRIPTION
building 'econWeb_0.1.3.tar.gz'

錯誤發生在 library(econWeb)：不存在叫 'econWeb' 這個名稱的套件
停止執行
Warning messages:
1: In file.copy(savedcopy, lib, recursive = TRUE) :
problem copying C:\R-4.1.2\library\00LOCK\cli\libs\x64\cli.dll to C:\R-4.1.2\library\cli\libs\x64\cli.dll: Permission denied
2: In i.p(...) :
installation of package ‘evaluate’ had non-zero exit status
3: In i.p(...) :
installation of package ‘C:/Users/user/AppData/Local/Temp/RtmpMR279x/file1a8011136706/econWeb_0.1.3.tar.gz’ had non-zero exit status

These packages have more recent versions available.
It is recommended to update all of them.
Which would you like to update?
1: All
2: CRAN packages only
3: None
4: evaluate (0.14  -> 0.15 ) [CRAN]
5: cli      (3.1.1 -> 3.2.0) [CRAN]
6: yaml     (2.3.4 -> 2.3.5) [CRAN]
Enter one or more numbers, or an empty line to skip updates: 1
evaluate (0.14  -> 0.15 ) [CRAN]
cli      (3.1.1 -> 3.2.0) [CRAN]
yaml     (2.3.4 -> 2.3.5) [CRAN]
Installing 3 packages: evaluate, cli, yaml
將程式套件安載入 ‘D:/user/Documents/R/win-library/4.1’
(因為 ‘lib’ 沒有被指定)
有二進位版本的，但原始碼版本是後來的:
binary source needs_compilation
evaluate   0.14   0.15             FALSE
cli       3.1.1  3.2.0              TRUE
yaml      2.2.2  2.3.5              TRUE
安裝原始碼套件 ‘evaluate’, ‘cli’, ‘yaml’
嘗試 URL 'https://cran.rstudio.com/src/contrib/evaluate_0.15.tar.gz'
Content type 'application/x-gzip' length 25617 bytes (25 KB)
downloaded 25 KB
嘗試 URL 'https://cran.rstudio.com/src/contrib/cli_3.2.0.tar.gz'
Content type 'application/x-gzip' length 493192 bytes (481 KB)
downloaded 481 KB
嘗試 URL 'https://cran.rstudio.com/src/contrib/yaml_2.3.5.tar.gz'
Content type 'application/x-gzip' length 94535 bytes (92 KB)
downloaded 92 KB

installing source package 'evaluate' ...
** package 'evaluate' successfully unpacked and MD5 sums checked
** using staged installation
parse(con, encoding = "UTF-8") 中的警告：
argument encoding="UTF-8" is ignored in MBCS locales
Error in parse(con, encoding = "UTF-8") :
在剖析器的第 11 列中有無效的多位元組字元
ERROR: installing package DESCRIPTION failed for package 'evaluate'
removing 'D:/user/Documents/R/win-library/4.1/evaluate'
restoring previous 'D:/user/Documents/R/win-library/4.1/evaluate'
installing source package 'cli' ...
** package 'cli' successfully unpacked and MD5 sums checked
ERROR: cannot remove earlier installation, is it in use?
removing 'D:/user/Documents/R/win-library/4.1/cli'
restoring previous 'D:/user/Documents/R/win-library/4.1/cli'
file.copy(lp, dirname(pkgdir), recursive = TRUE, copy.date = TRUE) 中的警告：
複製 D:\user\Documents\R\win-library\4.1\00LOCK-cli\cli\libs\x64\cli.dll 到 D:\user\Documents\R\win-library\4.1\cli\libs\x64\cli.dll 時出了問題：Permission denied
installing source package 'yaml' ...
** package 'yaml' successfully unpacked and MD5 sums checked
ERROR: cannot remove earlier installation, is it in use?
removing 'D:/user/Documents/R/win-library/4.1/yaml'
restoring previous 'D:/user/Documents/R/win-library/4.1/yaml'
file.copy(lp, dirname(pkgdir), recursive = TRUE, copy.date = TRUE) 中的警告：
複製 D:\user\Documents\R\win-library\4.1\00LOCK-yaml\yaml\libs\x64\yaml.dll 到 D:\user\Documents\R\win-library\4.1\yaml\libs\x64\yaml.dll 時出了問題：Permission denied

The downloaded source packages are in
‘C:\Users\user\AppData\Local\Temp\Rtmpwb3NRF\downloaded_packages’
√  checking for file 'C:\Users\user\AppData\Local\Temp\Rtmpwb3NRF\remotes4108e6cfe\tpemartin-econWeb-a5d3efb/DESCRIPTION' ...

preparing 'econWeb':
√  checking DESCRIPTION meta-information ...
checking for LF line-endings in source and make files and shell scripts
checking for empty or unneeded directories
Omitted 'LazyData' from DESCRIPTION
building 'econWeb_0.1.4.tar.gz'

將程式套件安載入 ‘D:/user/Documents/R/win-library/4.1’
(因為 ‘lib’ 沒有被指定)


Out of memory ( Custom DataSet)

@bshao001 great work!  So I am using my own dataset to build vocab.txt and run training on it.
My dataset is 500 mb which produces a vocab.txt of 15 mb . As I train on this on AWS sagemaker, I am getting out of memory errors.
ResourceExhaustedError (see above for traceback): OOM when allocating tensor with shape[2048,1024]          [[Node: dynamic_seq2seq/decoder/attention/attention_layer/kernel/Initializer/random_uniform/RandomUniform = RandomUniform[T =DT_INT32, _class=["loc:@dynamic_seq2seq/decoder/attention/attention_layer/kernel"], dtype=DT_FLOAT, seed=87654321, seed2=0, _device ="/job:localhost/replica:0/task:0/device:GPU:0"](dynamic_seq2seq/decoder/attention/attention_layer/kernel/Initializer/random_uniform /shape)]]
I am using :

vCPU
GPU
Mem (GiB)
GPU Mem (GiB)
Network Performanc

ml.p2.xlarge
4
1xK80
61
12

You do mention that vocab.txt will affect the performance , what is the optimal dataset size/vocab size for this configuration in your opinion ?

@kuberkaul
Try to reduce the batch_size and see if you can manage to train the model.
Yes, when the vocab size is too big, the existing model (1024 units and 2 layers) may not be able to accommodate it, generating a poor learned model, even if you can train it without OOM issue. A direct consequence you will see is the perplexity cannot be reduced to a reasonable value (although it is reasonably a little higher when you have a bigger vocab size and more training data, you have to try the trained the model to get a feeling of that.)
You can either increase the number of unit, say, to 1200, or even bigger, but it will require larger computing power. Or, you can try to reduce the vocab size. Refer to this script to get an idea: https://github.com/bshao001/ChatLearner/blob/master/Data/Corpus/RedditData/secondcleaner.py.
The existing implementation utilizes the computing power of the first GPU, and the memory of all GPUs.

hmm, seems to fail with out of memory even with 1500 number of units and reducing the batch_size to 128 and then 64.
I can try to limit the data/vocab but I was initially trying to tweak the hparams to make it work. Let me try that.

@bshao001 - So trimmed the data set to 100 mb, vocab to 4 mb and with 1 GPU this seems to go well but as its starts epoch 1 for training it fails with 👍
tensorflow.python.framework.errors_impl.InvalidArgumentError: pos 0 out of range for stringb'' at index 0
Any idea why that would be ?
full stacktrace :
Traceback (most recent call last):
  File "/home/ec2-user/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1323, in
 _do_call
    return fn(*args)
  File "/home/ec2-user/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1302, in
 _run_fn
    status, run_metadata)
  File "/home/ec2-user/anaconda3/envs/tensorflow_p36/lib/python3.6/site-packages/tensorflow/python/framework/errors_impl.py", line 4
73, in __exit__
    c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.InvalidArgumentError: pos 0 out of range for stringb'' at index 0
         [[Node: Substr = Substr[T=DT_INT32](arg0, Substr/pos, Substr/len)]]
         [[Node: IteratorGetNext = IteratorGetNext[output_shapes=[[?,?], [?,?], [?,?], [?], [?]], output_types=[DT_INT32, DT_INT32,
DT_INT32, DT_INT32, DT_INT32], _device="/job:localhost/replica:0/task:0/device:CPU:0"](Iterator)]]
         [[Node: Size/_93 = _HostRecv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_devi
ce="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_544_Size", tensor_type=DT_INT32, _de
vice="/job:localhost/replica:0/task:0/device:GPU:0"]()]]```

@kuberkaul I cannot be sure, however, it looks like it was caused by an empty string ''. Make sure you run the preprocessing correctly, and the vocab.txt file also does not contain empty string as a word.

Yup that was it and reducing batch size, splitting data set into multiple files helped.
Thanks!


Python : Video for Membership and Identity Operators (#477)

Description 📜
Membership and Identity Operators
Fixes #477

Type of change 📝

[x] Video

Domain of Contribution 📊

[x] Python_Domain

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

You demonstrated it quite well, looks good to merge @adityabisoi !


[DOCFIX] Update the chinese document of Running-Hive-with-Alluxio.md

Merged build finished. Test PASSed.

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


Inspect Support for policy and Tenant

Completed:
1: Tenant Inspect and Policy Inspect support.
2: Adding basic Integration tests for these two operations.
To be Done:
1:Adding number of violations stats per endpoint.
2: Adding few more Integration Tests.
3: TestTenantCreateDelete is still failing. Need to check.

@shaleman @jojimt : Could you please review this one ?

@gaurav-dalvi Thanks for taking care of this.
Is it possible to add list of all EndpointGroup, Policy, NetProfile, AppProfile, ExternalContract state in tenant inspect? Basically walk all objects under the tenant and display their config & oper states..
Basically, tenant inspect becomes one stop for dumping everything under the tenant..
Its perfectly fine to do this in stages.. i.e. multiple PRs..

Thanks @shaleman . Right now we are showing all the objects and how many of them in the tenant inspect command. Let me know if you are looking for more specific details.
netctl tenant inspect t1
Inspecting tenant: t1  {
  "Config": {
    "key": "t1",
    "tenantName": "t1",
    "link-sets": {
      "AppProfiles": {
        "t1:profile1": {
          "type": "appProfile",
          "key": "t1:profile1"
        }
      },
      "EndpointGroups": {
        "t1:g1": {
          "type": "endpointGroup",
          "key": "t1:g1"
        },
        "t1:g2": {
          "type": "endpointGroup",
          "key": "t1:g2"
        },
        "t1:g3": {
          "type": "endpointGroup",
          "key": "t1:g3"
        }
      },
      "NetProfiles": {
        "t1:netprofile1": {
          "type": "netprofile",
          "key": "t1:netprofile1"
        },
        "t1:netprofile2": {
          "type": "netprofile",
          "key": "t1:netprofile2"
        }
      },
      "Networks": {
        "t1:n1": {
          "type": "network",
          "key": "t1:n1"
        }
      },
      "Policies": {
        "t1:app2db": {
          "type": "policy",
          "key": "t1:app2db"
        }
      }
    }
  },
  "Oper": {
    "networks": [
      {
        "allocatedAddressesCount": 5,
        "allocatedIPAddresses": "<IP_ADDRESS>-<IP_ADDRESS>, <IP_ADDRESS>",
        "availableIPAddresses": "<IP_ADDRESS>-<IP_ADDRESS>",
        "dnsServerIP": "<IP_ADDRESS>",
        "endpoints": [
          {
            "endpointID": "795f123f36da74c27165162b8f171997a60d1fe9245776780e0e69dca874e697",
            "homingHost": "netplugin-node1",
            "ipAddress": [
              "<IP_ADDRESS>",
              ""
            ],
            "labels": "map[]",
            "macAddress": "02:02:14:01:01:01",
            "network": "n1.t1"
          },
          {
            "containerID": "7a299dc4a0f5b912259673c020b1ac385704b64c6edb0919915b0bb9d8a32f1e",
            "containerName": "/hungry_euclid",
            "endpointID": "868e7a3c87157af7cfd3b38a6e2bdbfbd73b1cdafbeab7830ddc499469a66d83",
            "homingHost": "netplugin-node1",
            "ipAddress": [
              "<IP_ADDRESS>",
              ""
            ],
            "labels": "map[]",
            "macAddress": "02:02:14:01:01:02",
            "network": "n1.t1"
          },
          {
            "containerID": "31a08b7271d33f1501914238b5c60c247b8a3b2684a34a5e23b937fe3ad43760",
            "containerName": "/evil_golick",
            "endpointGroupId": 1,
            "endpointGroupKey": "g1:t1",
            "endpointID": "c37cf6ca8207c883c15f95f3ac96b7f4572311cf8d3af4d6f0abc2eb11dd1ade",
            "homingHost": "netplugin-node1",
            "ipAddress": [
              "<IP_ADDRESS>",
              ""
            ],
            "labels": "map[]",
            "macAddress": "02:02:14:01:01:03",
            "network": "n1.t1",
            "serviceName": "g1"
          },
          {
            "containerID": "a68bfaa7153a96754b623c06ba4a735e77c4131b994a720f1998aa1c121af07d",
            "containerName": "/elegant_goodall",
            "endpointGroupId": 3,
            "endpointGroupKey": "g2:t1",
            "endpointID": "0b7f2686539359f344ccc124efd0c7b61fed471879b2bc6e6492e1b6cf72a9fc",
            "homingHost": "netplugin-node1",
            "ipAddress": [
              "<IP_ADDRESS>",
              ""
            ],
            "labels": "map[]",
            "macAddress": "02:02:14:01:01:04",
            "network": "n1.t1",
            "serviceName": "g2"
          },
          {
            "containerID": "e452b23f81c6a47d6620d3e4a2277035a1c3043b18ada72e2ea8607bec5d4d39",
            "containerName": "/adoring_shirley",
            "endpointGroupId": 2,
            "endpointGroupKey": "g3:t1",
            "endpointID": "ed6e42f5170c454a08ae8fc55f0d8838e56383940683784a45eee62291088673",
            "homingHost": "netplugin-node1",
            "ipAddress": [
              "<IP_ADDRESS>",
              ""
            ],
            "labels": "map[]",
            "macAddress": "02:02:14:01:01:05",
            "network": "n1.t1",
            "serviceName": "g3"
          }
        ],
        "numEndpoints": 5,
        "pktTag": 1
      }
    ],
    "totalAppProfiles": 1,
    "totalEPGs": 3,
    "totalEndpoints": 5,
    "totalNetprofiles": 2,
    "totalNetworks": 1,
    "totalPolicies": 1
  }
}

respective contivmodel PR : https://github.com/contiv/contivmodel/pull/43
Added EPG and Network in tenant inspect.


Load the schedule dynamically from redis

Add the ability to load schedules dynamically from redis on runtime,
the schedules can also be removed dynamically.

Add the ability to dynamically change schedules.
Remove an unused capistrano file.

Fix #34 and #47


~ in SASS files to include files from node_modules

                includePaths: [
                    './node_modules/',
                ]

Makes it possible to include node_modules in sass files. However...
I am not sure its a "sass feature", but its a common practice since https://github.com/webpack-contrib/sass-loader#imports to import files from /node_modules/ when using a ~ and it seems like node-sass has a way to add custom importers to make that happen. https://github.com/danderu/node-sass-custom-importer being an example.
Well-known libraries like Blueprint are using the same ~ to reference other node_modules, example:
https://github.com/palantir/blueprint/blob/master/packages/core/src/common/_font-imports.scss
Until this is fixed one cannot include the scss file from blueprint as it cannot resolve the ~ they are using.

Made a branch https://github.com/ConneXNL/fuse-box/tree/css-tilde-support that shows a potential implementation. What do you think?

@ConneXNL i think it's terrific! Could you write some tests?

Working on a test but I will require a fuse-box/fuse-box-testcase4 with a simple style.scss file to include.

You don't need to! You can use this approach
https://github.com/fuse-box/fuse-box/blob/master/test/bundle_in_a_bundle.js#L13
It creates a valid "npm" module.

@ConneXNL could you make a PR please?

http://fuse-box.org/#sassplugin

It seems like no changes were done to CSSResourcePlugin so you cannot use it to reference fonts and images.
This is however, what other libraries are doing too.


Update download link for kernel, firecracker and jailer

Signed-off-by: Royce Zhao<EMAIL_ADDRESS>Issue #, if available:
Description of changes:
Update the download link for kernel, firecracker and jailer in Makefile to keep consistent with Firecracker team.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Please remove "..." around your email address.

Please remove "..." around your email address.

Fixed it.


Update recipe

This also pins sensorml2iso to pyoos >=0.8.3.
Ping @mwengren.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


NgGridItem with button not handling clicks

I'm trying to build a tile for a dashboard and want to include a close button at the top.  Placing the close button in a header at the top of the tile is easy enough but the click event is captured by the grid and never received by the button.  Because of this, the method in the component class is never executed.
Here is an example of a button in the grid item:

    <button (click)="onRemoveClick()">
      Test
    </button>

onRemoveClick() is never called.
Note: I am using the fixed_items branch as I needed the functionality in this code base.

What version are you using? I have fixed this problem in a recent pull request - #113

OK... disregard.  I needed the fixed functionality out of the fixed_items branch so I was using it.  I didn't realize all of that had been worked into the main branch under the "fixed" property og NgGridItemConfig.
However...
I can still drag from my button.  That is, if I click and hold on my button I can drag the grid item.  And the click event still fires the onRemoveClick().  I tried to stop the click event in my code using stopPropagation but that didn't work.
Also...
In the current code base - Line 1182 of NgGrid.ts:
this._renderer.setElementStyle(this._ngEl.nativeElement, 'width', '100%');//(maxCol * (this.colWidth + this.marginLeft + this.marginRight))+'px');
This is causing the grid items to "overflow" the grid.  You can see this if you give the grid a background color.  I have auto_resize set to false because I don't want the responsiveness auto_resize=false gives.
Removing the 100% and adding back the commented code seems to fix the overflow issue.

This has become a bit tricky actually.  I'm trying to build a tile that can be closed like a typical dashboard:

Clicking the close button in the upper-right seems to work fine with Chrome.  I can click anywhere on it and the tile closes.  But with Edge and Firefox clicking in the area of the red box causes a resize event to be fired and the click on the button doesn't get through.
Clicking on the button next to the "Data goes here" works fine all the time in all browsers.
I believe because the click is within the threshold of the resize area, a resize event is capturing the click and doing its thing and the button never gets the click.

The first thing that comes to my mind is that we should move the resize function to the onMouseMove function as well - not just the drag function. @BTMorton

After some more thinking I realized that the best solution would be to stop event propagation. The solution I mentioned above would be like getting a howitzer to a pistol fight.
Try something like

    stopProp($event) {         $event.stopPropagation();     }

This is super old and probably not relevant any more, but just in case...
I think that the ngGridItem supports a config setting to set the resize selector. If you did that would it prevent the resize from getting called in that area?

I've updated the resize/drag start handlers so it registers on mouse down but doesn't actually start the drag/resize process until mouse move. Should fix this issue :)


🐞 [Bug]: Unable to deploy VM on dashboard through "Automatic" & "Manual" mode of selection due to errors.

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
When users trying to deploy VM's through dashboard it shows "No Nodes were found" when selecting Automatic Mode.

When selecting the specific Node through Manual selection it shows "Error getting node 26 Failed to send request to twinId 47 with command: zos.storage.pools, payload: TimeoutError: Didn't get a response after 20 seconds"

Steps To Reproduce
No response
which network/s did you face the problem on?
Main
version
NA
Twin ID/s
No response
Node ID/s
All
Farm ID/s
NA
Contract ID/s
NA
Relevant screenshots/screen records
Automatic

Manual

Relevant log output
NA

I'm having the same problem using the grid-cli (tried with tfgrid-sdk-go 0.15.3 & 0.15.4) so perhaps its not isolated to the dashboard. Server seems to be always returning an empty list of nodes for deployment requests.

All nodes are unhealthy according to gridproxy:
https://gridproxy.grid.tf/nodes?healthy=true

It's working again.


The problem of gen_gaussian_target

I have question in centernet_head.py as folow:
https://github.com/open-mmlab/mmdetection/blob/f78af7785ada87f1ced75a2313746e4ba3149760/mmdet/models/dense_heads/centernet_head.py#L253
https://github.com/open-mmlab/mmdetection/blob/f78af7785ada87f1ced75a2313746e4ba3149760/mmdet/models/dense_heads/corner_head.py#L455
Why gen_gaussian_target function return nothing in centernet_head.py but return values in corner_head,py.Anyone knows?

@RookieHuang It's just a matter of writing style, and both expressions have the same meaning. You can use whichever writing style you prefer.

Thanks for your reply!


PowerToys displaying incorrectly in German on Windows configured to UK English.

Microsoft PowerToys version
0.76.0
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
General
Steps to reproduce
PowerToys is displaying parts of its user interface in German, when Windows is configured to UK English.

PowerToysReport_2023-12-05-11-44-27.zip
✔ Expected Behavior
I was expecting PowerToys to display in English. See attached Windows language configuration.

❌ Actual Behavior
PowerToys incorrectly displays parts of its user interface in German. This is not new to version 0.76: it has been happening for some time.
Other Software
No response

I found that you must prioritize the first language setting for your target language.

Thank you for your comment. However, as you can see from the screen-shots that I provided, my target language (UK English) is already the top "Preferred Language", and German is not the top preferred language.

我发现您必须优先考虑目标语言的第一个语言设置。

谢谢你的评论。但是，从我提供的屏幕截图中可以看出，我的目标语言（英国英语）已经是首选语言，而德语不是首选语言。

I'm sorry I misunderstood your meaning earlier, as English is not my native language. Perhaps you can try lowering the priority of German in your preferred language.

A work-around for this issue is to make the following changes in Windows Settings -> Time & Language -> Language -> Preferred languages:
While retaining "English (United Kingdom)" as the top preferred language,
elevate "English (United States)" above "German".
The mixture of German and English in PowerToys dialogues will appear if "German" is higher than "English (United States)" in the "Preferred languages" list, even though the top preferred language is "English (United Kingdom)".

/dup https://github.com/microsoft/PowerToys/issues/25815


PowerBI reports are empty

Describe the bug
When running the script no data is returned in the powerBI reports
To Reproduce
Steps to reproduce the behavior:
Normal running of the tooling
Expected behavior
Expected to see data

Investigation shows this is a known issue with the PowerBi cmdlets: https://github.com/Microsoft/powerbi-powershell/issues/60
Seems AzureAD has a handle on Newtonsoft. [Newtonsoft.Json.JsonConvert].Module:
MDStreamVersion    : 131072
FullyQualifiedName : C:\Program Files\WindowsPowerShell\Modules\AzureAD\<IP_ADDRESS>\Newtonsoft.Json.dll
ModuleVersionId    : 814f8268-fecd-454f-84c6-f965d394a6d8
MetadataToken      : 1
ScopeName          : Newtonsoft.Json.dll
Name               : Newtonsoft.Json.dll
Assembly           : Newtonsoft.Json, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed
CustomAttributes   : {}
ModuleHandle       : System.ModuleHandle
Testing shows this should work by running the commands in a seperate runspace. Needs to be implemented.

Confirmed resolved. Code to be pushed shortly


Is the following supported?

Is it possible to do the following zstd -22 --ultra --long=31 -T0  using zstd-rs?

Hi, and thanks for the question!

zstd defaults to compressing; for example, if you want a Write that can compress, you can use the Encoder.
-22 is done by just specifying the level, for example [Encoder::new(writer, 22)].(https://docs.rs/zstd/latest/zstd/stream/write/struct.Encoder.html#method.new).
--ultra just enables levels 20+ on the CLI, it's not needed in the library.
--long=31 is done by enabling long-distance matching, and specifying the window log.
-T0 means "use as many threads as CPUs". You can specify the number of threads; detecting the number of CPUs is not provided by the library, but you can use something like num_cpus.

Thanks for the thorough explanation!
I still didn't quite understand how --long=31 should be specified using the library, here's what I wrote:
    let tar_zst = fs::File::create("archive.tar.zst");
    let buf = io::BufWriter::new(tar_zst);
    let zstd = Encoder::new(buf, 22)?;
    let mut ar = tar::Builder::new(zstd);
    zstd.finish()?;

     let tar_zst = fs::File::create("archive.tar.zst");
     let buf = io::BufWriter::new(tar_zst);
     let mut zstd = Encoder::new(buf, 22)?;
     zstd.multithread(8)?; // hardcode 8 threads here, or use num_cpus
     zstd.long_distance_matching(true)?;
     zstd.window_log(31)?;
     let mut ar = tar::Builder::new(zstd);
     zstd.finish()?;

Thanks for the amazing help and for the quick reply!
Got one last question, would it be wise to use AutoFinishEncoder in place of Encoder and remove zstd.finish()? alltogether, so that it becomes:
     let tar_zst = fs::File::create("archive.tar.zst");
     let buf = io::BufWriter::new(tar_zst);
     let mut zstd = AutoFinishEncoder::new(buf, 22)?;
     zstd.multithread(8)?; // hardcode 8 threads here, or use num_cpus
     zstd.long_distance_matching(true)?;
     zstd.window_log(31)?; // You'll need to do this too on the Decoder when decompressing.
     let mut ar = tar::Builder::new(zstd);

It's possible indeed, and can simplify integration with the tar builder. Note though that you give up error reporting during the finish phase; if something fails here, the default implementation panics, which may not be acceptable for all use cases

Thanks a lot! Closing now..


Cannot run mnist example with gpu

Hello, nni team!
I executed mnist tutorial with GPU, but none of the trials launched.
(nnictl create --config config_gpu.yml)
When I submit an experiment with non-GPU code(config.yml), it worked.
Is there any way to solve?
Thank you.
nni Environment:

nni version: 0.5.1
nni mode(local|pai|remote): local
OS: ubuntu 16.04
python version: 3.5
is conda or virtualenv used?: virtualenv
is running in docker?:no
Tensorflow version: 1.8(with GPU)

nnimanager.log
[2019-02-21T03:47:48.908Z] INFO [ 'Datastore initialization done' ]
[2019-02-21T03:47:48.933Z] INFO [ 'Rest server listening on: http://<IP_ADDRESS>:8080' ]
[2019-02-21T03:47:48.918Z] INFO [ 'RestServer start' ]
[2019-02-21T03:47:48.922Z] INFO [ 'Construct local machine training service.' ]
[2019-02-21T03:47:48.926Z] INFO [ 'RestServer base port is 8080' ]
[2019-02-21T03:47:51.191Z] INFO [ 'NNIManager setClusterMetadata, key: trial_config, value: {"gpuNum":1,"command":"python3 mnist.py","codeDir":"/tmp/sungwoo/nni/annotation/tmp4h2ywsv_"}' ]
[2019-02-21T03:47:51.207Z] INFO [ 'required GPU number is 1' ]
[2019-02-21T03:47:51.221Z] INFO [ 'Starting experiment: qx7XLBNw' ]
[2019-02-21T03:47:51.232Z] INFO [ 'Change NNIManager status from: INITIALIZED to: RUNNING' ]
[2019-02-21T03:47:51.235Z] INFO [ 'Add event listeners' ]
[2019-02-21T03:47:51.245Z] INFO [ 'Run local machine training service.' ]
[2019-02-21T03:47:51.869Z] ERROR [ 'Read GPU summary failed with error: ',
  { RangeError [ERR_CHILD_PROCESS_STDIO_MAXBUFFER]: stdout maxBuffer length exceeded
      at Socket.onChildStdout (child_process.js:343:14)
      at Socket.emit (events.js:182:13)
      at addChunk (_stream_readable.js:283:12)
      at readableAddChunk (_stream_readable.js:260:13)
      at Socket.Readable.push (_stream_readable.js:219:10)
      at Pipe.onStreamRead (internal/stream_base_commons.js:94:17) cmd: 'nvidia-smi -q -x' } ]

@pigbug419
You are using python virtualenv as your local python interpreter, so it is necessary to clain your python path in your config file.
trial:
  codeDir: .
  command: /path/to/virtualenv/python3 example.py
  gpuNum: 1

@sangloupmonoutlook When I use remote mode, I got such an error.
ERROR: Failed! Error is: {"error":"TrainingService setClusterMetadata timeout. Please check your config file."}
@leelaylay After I specified my virtualenv's python path, I got a same error 😢

@pigbug419
I guess that the local mode works successfully but remote mode does not.
The timeout error shows that there is something wrong with the network connection between your local machine and your remote cluster.
My suggestion: check the network connection between your local machine and your remote cluster. Also, check TCP 8080 port and 8081 port are open.

@SparkSnail had fixed issue in last release, @QuanluZhang to connect the PR here.

the bug "stdout maxBuffer length exceeded" is fixed by https://github.com/Microsoft/nni/pull/943

@sangloupmonoutlook When I use remote mode, I got such an error.
ERROR: Failed! Error is: {"error":"TrainingService setClusterMetadata timeout. Please check your config file."}
@leelaylay After I specified my virtualenv's python path, I got a same error 😢

Hi I met the same problem. Did you solve it?

@Guptajakala  - which version are you using? Had you tried upgrade NNI to 0.6, there is a fix in 0.6.

@Guptajakala - which version are you using? Had you tried upgrade NNI to 0.6, there is a fix in 0.6.

Thanks for your reply, could you tell me how to check the version? nni.__version does not work.

OK, from conda, I found the version is 1.0