HTML5 compliant form attributes

According to W3C specification, ‘action’ is an optional attribute and can’t be empty in HTML5. So, if the form doctype is ‘HTML5’ or ‘XHTML5’, the ‘action’ attribute isn’t set as an empty string by default. Also, if either doctypes are detected, form ‘method’ attribute is not set to ‘get’ by default, as some applications may implement submit buttons that use ‘formmethod’ attribute, making form ‘method’ redundant or useless (according to W3C, “missing value default for the method attribute is also the GET state” — and have been since HTML 2.0).
Reference:

http://www.w3.org/TR/html5/forms.html#attr-fs-action
http://www.w3.org/TR/html5/forms.html#attr-fs-method

@Ocramius This was misinterpreted i guess. It would be good to revert.
http://www.w3.org/TR/html5/forms.html#form-submission-algorithm

If action is the empty string, let action be the document's address of the form document.

Angular.js has a behavior that calls preventDefault on the forms that doesn't have an action specified. It doesnt do anything if the action is empty.
https://github.com/angular/angular.js/pull/3776
My best guess is that an empty string is not considered a specification of a value, because it says , if specified, it only accepts VALID NON-EMPTY URL (but a empty string would be ok if not considered a specification).
This was working perfectly fine in zf 2.3.x. and now i must specify a url in all the actions.

because it says , if specified, it only accepts VALID NON-EMPTY URL (but a empty string would be ok if not considered a specification).

Perhaps you can describe the difference between an empty url and an empty string (with examples)?

@Martin-P I'm not saying these are different things. I'm saying the empty string might not be considered a url specification.

The specification says:

The action and formaction content attributes, if specified, must have a value that is a valid non-empty URL potentially surrounded by spaces.

If an empty url and an empty string are considered the same thing, this means action="" is not allowed in HTML5.
What you are referring to is what HTML5 should do when a particular situation is encountered, it says nothing about the fact if it is allowed or not. The complaint of previous versions of HTML was that browsers made up their own rules, because the specification did not specify what action should be taken in certain (unexpected) situations. One of the goals of the HTML5 specification is to describe what should be done in such a situation to prevent browsers making up their own rules ones again.

@Martin-P You're telling me that the specification is telling what should be done by browsers if a not valid html5 declaration is found and this behavior is to threat the issue like it wasn't there by assuming the document address is the required action?
If this document is telling browsers SHOULD use the document address when they find a EMPTY STRING, the this is a valid setting, otherwise the recommended behavior was going to be DO NOTHING BECAUSE THIS IS NOT VALID.

If this document is telling browsers SHOULD use the document address when they find a EMPTY STRING, the this is a valid setting

The document describes what procedure should be followed. If that procedure describes what should be done with certain values, this does not mean that value is valid.
An example to clarify this:
When I walk in the middle of the highway and the procedure for car drivers is to avoid me, this does not mean I am supposed to walk in the middle of the highway :wink:

A better example would be:
You're not allowed to kill people, but if you do, remember to take a jigsaw with you and cut their members so it can fit in a bag and you can bury the corpse, so nobody can see what you're doing.
It makes no sense to allow a behavior that MUST no happen.

My point is: when a situation is described it does not automatically mean that the situation is valid.

It makes no sense to allow a behavior that MUST no happen.

It does not allow a behavior, it merely describes what should happen when such a situation is encountered. Please see: 1.10 Conformance requirements for authors

Unlike previous versions of the HTML specification, this specification defines in some detail the required processing for invalid documents as well as valid documents.

However, even though the processing of invalid content is in most cases well-defined, conformance requirements for documents are still important: in practice, interoperability (the situation in which all implementations process particular content in a reliable and identical or equivalent way) is not the only goal of document conformance requirements. This section details some of the more common reasons for still distinguishing between a conforming document and one with errors.

This part clearly states processing for both valid and invalid documents is handled in the specification.

Ok, i got your point

When a form element form is submitted from an element submitter (typically a button), optionally with a submitted from submit() method flag set, the user agent must run the following steps:

If action is the empty string, let action be the document's address of the form document.

We are discussing an emtpy string as value for the action attribute, right? The part you quote says the user agent must run the following steps. One of those steps is checking whether the value of the action attribute is an empty string. If it is an empty string, act upon it. That's all. Nothing more, nothing less.

@Martin-P The specs are not even close to be clear about this. This 1.10 section is very clear, but it exemplifies the text you quoted above with problems like invalid markup, boolean attribute values, typos, etc.... Nothing even close to a empty form action.
You are assuming a empty string is a action specification, but if i'm not wrong (and i admit i can be), a empty string is not specifying anything, but the absence of a specification, and the rule says if specified.
If your interpretation of the specs is correct, the interpretation of the folks from Angular.js isn't.
@IgorMinar It might be of your interest.

but it exemplifies the text you quoted above with problems like invalid markup

An example of invalid markup: action=""

@Martin-P Where in the specs it is saying an empty action is invalid markup?

Here it says an empty action is invalid markup: http://www.w3.org/TR/html5/forms.html#attr-fs-action

The action and formaction content attributes, if specified, must have a value that is a valid non-empty URL potentially surrounded by spaces.

Its not saying that. Its saying that anything that is not considered a valid non-empty URL, potentially surrounded by spaces, is invalid markup.
But if you don't interpret that empty string is a specification, then it is valid markup.
Can we simply agree with disagreeing and wait for someone's else opinion? I don't see we moving forward with this. You could be less intolerant with other people opinion, quoting the same text over and over again. I guess we both understood the other side point of view.

Just test an empty value on W3C validator and it'll point that an action attribute must be non-empty.

@sbtoledo nice point.

W3C Validator is just a "linter". Don't forget that

https://github.com/angular/angular.js/issues/11495

Can somebody give me a tl;dr summary, please? Does anything need to change currently in ZF2? If so, what?

@weierophinney:
The question is: is action="" allowed in HTML5 or not? Looking at the HTML5 specification this PR is correct the way it was handled and action="" is not allowed in HTML5. IMO opinion the real problem is in angular.js which is not following HTML5 standards.However I think @fabiocarneiro and I agree we disagree about that part :wink:

Angular extends HTML, it doesn't care—neither have to care—with W3C standards. I suggest that if you need an empty action value, just set an empty string before rendering the form.
@Maks3w I agree, but the W3C validator is also a reliable source for checking standards conformity.

If you are already relying on javascript, another possibility is to let javascript check the forms and add an empty action value if none present. That way you do not have to change anything about the way ZF2 renders the form.

@Martin-P Yes, i'm actually doing that right now. But i guess its not necessary to prevent default submit when no action attribute is set, and this is something angular is doing. I created a issue there complaining about this. I agree with you. I'm sorry to be so insistent. zf2 is correct.

@fabiocarneiro No need to be sorry for insistence, I can be insistent as well :wink: I think it is a good thing to stand behind your standpoint :+1:


Consider styling links with rel="external"

This is just a copy of https://github.com/twbs/bootstrap/issues/4354, which was closed due to lack of a good icon (see https://github.com/twbs/bootstrap/issues/4354#issuecomment-7698773). However, the glyphicon-new-window seems like a very appropriate icon for this.
If this was easy to do yourself, I'd suggest that bootstrap not do anything here. However, the only way I could get it to work involved hardcoding some magic pixel numbers (see https://github.com/jfly/gjcomps/commit/121a1503182fd3beb97aa75a5ccafeb29bb2d7a6).

Personally I would wish to have this as part of Bootstrap. This is a common problem and solution to it is searched for by many developers:

Its an essential part of User Experience
Wordpress has a dedicated plugin for this
GitHub Atom developers are thinking about that
It is asked on Stack Overflow
Wikipedia uses it

The only thing I would note is that its burden for a developer to indicate that a link is external with class='external'. Much friendlier would be to introduce some JavaScript component into Bootstrap that can be activated. When activated then all links are scanned and external ones are detected based on URI schemes and styles (icon) applied to them.

@yursha Yes, it's a common problem and there are many perfectly fine solutions out there. So no need for us to include a new plugin in the core.


v0.6.3 and v0.1.1

fixes the rand issues
r? @rust-embedded/cortex-m (anyone)

bors r+

Works. Just published new fresh version of the microbit crate with all the latest bells and whistles (and hardware random number support).


How to use float8 to do the calculation? Especially by using tensor core.

H100 card have been release for a several mouths, but there is little kernel support for their Float8 Tensor core.

matmul operator now supports fp8:
https://github.com/openai/triton/blob/76858bd91765adccefcbad33432187f7aa16df11/python/test/unit/operators/test_matmul.py#L85

@ThomasRaoux Thank you for the information provided. But Triton seems not to support FP8 matmul in Tensor Core.


#975 Added baking_oven in preset

Please review it once actually it was done earlier I deleted my earlier repo due to which earlier pull request also got deleted...

@tyrasd sir thank you so much for reviewing and making the necessary changes...any more changes I have to do from my side please tell...Once again thank you it's very exciting contributing to the repo it was my first time but yes will involve more in it...

Fixes #975.


Updating (changing) sslcertkey fails

Following up on only send nodomaincheck on certificate change #39: I'm trying to change the certificate by uploading .cert and .key files with the same filename, but different content.
I'm using local-exec to actually upload the certificate files.
EXPECTED RESULT:
Changed sslcertkey object
ACTUAL RESULT:
Error: Error applying plan
STEPS TO REPRODUCE:
resources.tf
resource "null_resource" "destroy_certificate" {
  count = "${var.ingress["service_type"] == "SSL" ? 1 : 0}"

  provisioner "local-exec" {
    when = "destroy"
    command = "./cert_files.sh absent"
    working_dir = "${var.cwd}"
    environment = {
      TF_VAR_ns_ingress_name = "${var.ingress["name"]}"
    }
  }
}

resource "null_resource" "upload_certificate" {
  count = "${var.ingress["service_type"] == "SSL" ? 1 : 0}"

  triggers = {
    cert_hash = "${sha1(var.ingress["cert"])}"
  }

  provisioner "local-exec" {
    command = "./cert_files.sh absent"
    working_dir = "${var.cwd}"
    environment = {
      TF_VAR_ns_ingress_name = "${var.ingress["name"]}"
    }
  }
  provisioner "local-exec" {
    command = "./cert_files.sh present"
    working_dir = "${var.cwd}"
    environment = {
      TF_VAR_ns_ingress_name = "${var.ingress["name"]}"
    }
  }
}

resource "netscaler_sslcertkey" "lbsslcertkey" {
  count = "${var.ingress["service_type"] == "SSL" ? 1 : 0}"

  certkey = "TLS_${var.ingress["name"]}_${var.serial}"
  cert = "/nsconfig/ssl/${var.ingress["name"]}.cert"
  key = "/nsconfig/ssl/${var.ingress["name"]}.key"
  expirymonitor = "DISABLED"
  # notificationperiod = 90
  inform = "PEM"
  nodomaincheck = true

  depends_on = ["null_resource.upload_certificate"]
}

resource "netscaler_lbmonitor" "lbmonitor" {
  monitorname = "mon_${var.ingress["name"]}"
  type = "${var.backends_config["service_type"]}"
  interval = 350
  resptimeout = 250
}

resource "netscaler_server" "backends" {
  count = "${length(var.backends)}"

  name = "srv_${format("%s_%s", var.ingress["name"], replace(var.backends[count.index], "/[:.]+/", ""))}"
  ipaddress = "${var.backends[count.index]}"
}

resource "netscaler_servicegroup" "backend" {
  servicegroupname = "sg_${var.ingress["name"]}"
  lbvservers = ["${netscaler_lbvserver.lbvserver.name}"]
  servicetype = "${var.backends_config["service_type"]}"
  servicegroupmembers_by_servername = ["${formatlist("%s:%s", netscaler_server.backends.*.name, var.backends_config["port"])}"]
}

resource "netscaler_lbvserver" "lbvserver" {
  name = "lbvs_${var.ingress["name"]}"
  ipv46 = "${var.ingress["ip"]}"
  port = "${var.ingress["port"]}"
  servicetype = "${var.ingress["service_type"]}"
  lbmethod = "ROUNDROBIN"
  persistencetype = "COOKIEINSERT"
  # ugly workaround to conditionally reference a possible 0 count resource:
  sslcertkey = "${element(concat(netscaler_sslcertkey.lbsslcertkey.*.certkey, list("")), 0)}"
}

variables.tf
variable "ns_user" {
  default = "nsroot"
}

variable "ns_password" {
  default = "nsroot"
}

variable "ns_hostname" {
  default = "ns.domain.tld"
}

variable "cwd" {
  default = "."
}

variable "serial" {
  default = ""
}

variable "ingress" {
  type = "map"
  "description" = "name, vip, port and servicetype for the load balancer"
  default {
    port = 443
    service_type = "SSL"
  }
}

variable "backends_config" {
  type = "map"
  "description" = "port and servicetype for the backend services"
  default {
    clttimeout = 40
    backend_port = 80
    service_type = "HTTP"
  }
}

variable "backends" {
  type = "list"
  description = "List of backend services [ipaddress:port]"
}

ingress = {
  name = "ademoapp"
  service_type = "SSL"
  ip = "<IP_ADDRESS>"
  port = 443

  cert = <<EOF
-----BEGIN CERTIFICATE-----
<snip>
-----END CERTIFICATE-----
EOF

}

backends_config = {
  clttimeout = 40
  port = 80
  service_type = "HTTP"
}

backends = [
  "<IP_ADDRESS>",
  "<IP_ADDRESS>",
  "<IP_ADDRESS>",
]

Create initial setup
export TF_CLI_CONFIG_FILE=terraformrc
export TF_VAR_cwd=/home/user/netscaler/terraform
export TF_VAR_ns_hostname=ns.domain.tld
export TF_VAR_ns_ingress_name=ademoapp
export TF_VAR_ns_password=Pssssh
export TF_VAR_ns_user=my_username
export TF_VAR_serial=9b464bfc0efeb1fd

terraform apply -parallelism=1 -refresh=true -no-color \
  -state app_states/ademoapp.tfstate \
  -var-file=app_vars/ademoapp.tfvars

Resulting state
terraform show app_states/ademoapp.tfstate
netscaler_lbmonitor.lbmonitor:
  id = mon_ademoapp
  action = Not applicable
  alertretries = 0
  application =
  attribute =
  basedn =
  binddn =
  customheaders =
  database =
  destip =
  destport = 0
  dispatcherip = <IP_ADDRESS>
  dispatcherport = 0
  domain =
  downtime = 30
  evalrule =
  failureretries = 0
  filename =
  filter =
  group =
  hostipaddress =
  hostname =
  httprequest = HEAD /
  inbandsecurityid = NO_INBAND_SECURITY
  interval = 350
  iptunnel = NO
  kcdaccount =
  lasversion =
  logonpointname =
  lrtm =
  metric =
  metrictable =
  metricthreshold = 0
  metricweight = 0
  monitorname = mon_ademoapp
  mssqlprotocolversion =
  netprofile =
  oraclesid =
  originhost =
  originrealm =
  password =
  productname =
  query =
  querytype =
  radaccountsession =
  radapn =
  radframedip = <IP_ADDRESS>
  radkey =
  radmsisdn =
  radnasid =
  radnasip = <IP_ADDRESS>
  recv =
  respcode.# = 1
  respcode.0 = 200
  resptimeout = 250
  retries = 3
  reverse = NO
  rtsprequest = HEAD /
  scriptargs =
  scriptname =
  secondarypassword =
  secure = NO
  send =
  servicegroupname =
  servicename =
  sipmethod = OPTIONS
  sipreguri =
  sipuri =
  sitepath =
  snmpcommunity =
  snmpoid =
  snmpthreshold =
  snmpversion = V1
  sqlquery =
  sslprofile =
  state = ENABLED
  storedb = DISABLED
  storefrontacctservice = NO
  storefrontcheckbackendservices = NO
  storename =
  successretries = 1
  tos = NO
  tosid = 0
  transparent = NO
  trofsstring =
  type = HTTP
  units1 = SEC
  units2 = SEC
  units3 = SEC
  units4 = SEC
  username =
  validatecred =
  vendorspecificvendorid = 0
netscaler_lbvserver.lbvserver:
  id = lbvs_ademoapp
  appflowlog = ENABLED
  authentication = OFF
  authenticationhost =
  authn401 = OFF
  authnprofile =
  authnvsname =
  backuplbmethod =
  backuppersistencetimeout = 2
  backupvserver =
  bypassaaaa = NO
  cacheable = NO
  comment =
  connfailover = DISABLED
  cookiename =
  dbprofilename =
  dbslb =
  disableprimaryondown = DISABLED
  dns64 = DISABLED
  dnsprofilename =
  downstateflush = ENABLED
  hashlength = 0
  httpprofilename =
  httpsredirecturl =
  icmpvsrresponse = PASSIVE
  insertvserveripport = OFF
  ipmask = *
  ippattern = <IP_ADDRESS>
  ipset =
  ipv46 = <IP_ADDRESS>
  l2conn = OFF
  lbmethod = ROUNDROBIN
  lbprofilename =
  listenpolicy = NONE
  listenpriority = 0
  m = IP
  macmoderetainvlan = DISABLED
  mssqlserverversion =
  mysqlcharacterset = 0
  mysqlprotocolversion = 0
  mysqlservercapabilities = 0
  mysqlserverversion =
  name = lbvs_ademoapp
  netmask =
  netprofile =
  newname =
  newservicerequest = 0
  newservicerequestincrementinterval = 0
  newservicerequestunit = PER_SECOND
  oracleserverversion =
  persistencebackup = NONE
  persistencetype = COOKIEINSERT
  persistmask = <IP_ADDRESS>
  port = 443
  pq =
  processlocal = DISABLED
  push = DISABLED
  pushlabel = none
  pushmulticlients = NO
  pushvserver =
  recursionavailable =
  redirectfromport = 0
  redirectportrewrite = DISABLED
  redirurl =
  redirurlflags = false
  resrule =
  retainconnectionsoncluster = NO
  rhistate = PASSIVE
  rtspnat = OFF
  rule =
  sc =
  servicename =
  servicetype = SSL
  sessionless = DISABLED
  skippersistency = None
  sobackupaction =
  somethod = NONE
  sopersistence = DISABLED
  sothreshold = 0
  sslcertkey = TLS_ademoapp_9b464bfc0efeb1fd
  sslprofile =
  state =
  tcpprofilename =
  timeout = 2
  tosid = 0
  trofspersistence = ENABLED
  v6netmasklen = 0
  vipheader =
  weight = 0
netscaler_server.backends.0:
  id = srv_ademoapp_1042421
  comment =
  delay = 0
  domain =
  domainresolvenow = false
  domainresolveretry = 0
  graceful =
  internal = false
  ipaddress = <IP_ADDRESS>
  ipv6address = NO
  name = srv_ademoapp_1042421
  newname =
  state = ENABLED
  translationip = <IP_ADDRESS>
  translationmask = <IP_ADDRESS>
netscaler_server.backends.1:
  id = srv_ademoapp_1042422
  comment =
  delay = 0
  domain =
  domainresolvenow = false
  domainresolveretry = 0
  graceful =
  internal = false
  ipaddress = <IP_ADDRESS>
  ipv6address = NO
  name = srv_ademoapp_1042422
  newname =
  state = ENABLED
  translationip = <IP_ADDRESS>
  translationmask = <IP_ADDRESS>
netscaler_server.backends.2:
  id = srv_ademoapp_1042423
  comment =
  delay = 0
  domain =
  domainresolvenow = false
  domainresolveretry = 0
  graceful =
  internal = false
  ipaddress = <IP_ADDRESS>
  ipv6address = NO
  name = srv_ademoapp_1042423
  newname =
  state = ENABLED
  translationip = <IP_ADDRESS>
  translationmask = <IP_ADDRESS>
netscaler_servicegroup.backend:
  id = sg_ademoapp
  appflowlog = ENABLED
  autodisabledelay = 0
  autodisablegraceful =
  autoscale = DISABLED
  cacheable = NO
  cachetype = SERVER
  cip = ENABLED
  cipheader = X-Forwarded-For
  cka = NO
  clttimeout = 180
  cmp = NO
  comment =
  customserverid =
  delay = 0
  downstateflush = ENABLED
  dupweight = 0
  graceful = NO
  healthmonitor = YES
  httpprofilename =
  includemembers = false
  lbmonitor =
  lbvservers.# = 1
  lbvservers.4129248370 = lbvs_ademoapp
  memberport = 0
  monconnectionclose = NONE
  monitornamesvc =
  netprofile =
  newname =
  pathmonitor = NO
  pathmonitorindv = NO
  port = 0
  riseapbrstatsmsgcode = 0
  rtspsessionidremap = OFF
  sc =
  serverid = 0
  servername =
  servicegroupmembers.# = 0
  servicegroupmembers_by_servername.# = 3
  servicegroupmembers_by_servername.1839026872 = srv_ademoapp_1042423:80:1
  servicegroupmembers_by_servername.2797699357 = srv_ademoapp_1042422:80:1
  servicegroupmembers_by_servername.542503859 = srv_ademoapp_1042421:80:1
  servicegroupname = sg_ademoapp
  servicetype = HTTP
  sp = OFF
  state = ENABLED
  svrtimeout = 360
  tcpb = NO
  tcpprofilename =
  useproxyport = YES
  usip = NO
  weight = 0
netscaler_sslcertkey.lbsslcertkey:
  id = TLS_ademoapp_9b464bfc0efeb1fd
  bundle =
  cert = /nsconfig/ssl/ademoapp.cert
  certkey = TLS_ademoapp_9b464bfc0efeb1fd
  expirymonitor = DISABLED
  fipskey =
  hsmkey =
  inform = PEM
  key = /nsconfig/ssl/ademoapp.key
  linkcertkeyname =
  nodomaincheck = false
  notificationperiod = 0
  ocspstaplingcache = false
  passplain =
  password = false
null_resource.destroy_certificate:
  id =<PHONE_NUMBER>656250296
null_resource.upload_certificate:
  id =<PHONE_NUMBER>869952010
  triggers.% = 1
  triggers.cert_hash = 66b0e9969c211e5a824b504d092802e57d9f3367

Changes

contents of certificate files : app_vars/ademoapp.cert, app_vars/ademoapp.key
cert var variable
name of lbsslcertkey to trigger change ("taint"), see TF_VAR_serial

app_vars/ademoapp.tfvars

ingress = {
  ....
  cert = <<EOF
-----BEGIN CERTIFICATE-----
MIIDmTCCAoGgAwIBAgIJAJtGS/wO/rH9MA0GCSqGSIb3DQEBCwUAMGsxCzAJBgNV
<snip>
-----END CERTIFICATE-----
...
}

Execute terraform again

export TF_CLI_CONFIG_FILE=terraformrc
export TF_VAR_cwd=/home/user/netscaler/terraform
export TF_VAR_ns_hostname=ns.domain.tld
export TF_VAR_ns_ingress_name=ademoapp
export TF_VAR_ns_password=Pssssh
export TF_VAR_ns_user=my_username
export TF_VAR_serial=123456789abcdef
terraform apply -parallelism=1 -refresh=true -no-color \
  -state app_states/ademoapp.tfstate \
  -var-file=app_vars/ademoapp.tfvars

null_resource.upload_certificate: Refreshing state... (ID:<PHONE_NUMBER>869952010)
null_resource.destroy_certificate: Refreshing state... (ID:<PHONE_NUMBER>656250296)
netscaler_server.backends[2]: Refreshing state... (ID: srv_ademoapp_1042423)
netscaler_server.backends[1]: Refreshing state... (ID: srv_ademoapp_1042422)
netscaler_server.backends[0]: Refreshing state... (ID: srv_ademoapp_1042421)
netscaler_lbmonitor.lbmonitor: Refreshing state... (ID: mon_ademoapp)
netscaler_sslcertkey.lbsslcertkey: Refreshing state... (ID: TLS_ademoapp_9b464bfc0efeb1fd)
netscaler_lbvserver.lbvserver: Refreshing state... (ID: lbvs_ademoapp)
netscaler_servicegroup.backend: Refreshing state... (ID: sg_ademoapp)

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  ~ update in-place
-/+ destroy and then create replacement

Terraform will perform the following actions:

~ netscaler_lbvserver.lbvserver
      sslcertkey:                                   "TLS_ademoapp_9b464bfc0efeb1fd" => "TLS_ademoapp_123456789abcdef"

  ~ netscaler_servicegroup.backend
      servicegroupmembers_by_servername.1839026872: "srv_ademoapp_1042423:80:1" => ""
      servicegroupmembers_by_servername.213590207:  "" => "srv_ademoapp_1042423:80"
      servicegroupmembers_by_servername.2796742708: "" => "srv_ademoapp_1042421:80"
      servicegroupmembers_by_servername.2797699357: "srv_ademoapp_1042422:80:1" => ""
      servicegroupmembers_by_servername.3020376026: "" => "srv_ademoapp_1042422:80"
      servicegroupmembers_by_servername.542503859:  "srv_ademoapp_1042421:80:1" => ""

  ~ netscaler_sslcertkey.lbsslcertkey
      certkey:                                      "TLS_ademoapp_9b464bfc0efeb1fd" => "TLS_ademoapp_123456789abcdef"
      nodomaincheck:                                "false" => "true"

-/+ null_resource.upload_certificate (new resource required)
      id:                                           "4257515104869952010" => <computed> (forces new resource)
      triggers.%:                                   "1" => "1"
      triggers.cert_hash:                           "66b0e9969c211e5a824b504d092802e57d9f3367" => "80c5376a34359caf831c1bef1724bb296764d1cd" (forces new resource)
Plan: 1 to add, 3 to change, 1 to destroy.

null_resource.upload_certificate: Destroying... (ID:<PHONE_NUMBER>869952010)
null_resource.upload_certificate: Destruction complete after 0s
null_resource.upload_certificate: Creating...
  triggers.%:         "" => "1"
  triggers.cert_hash: "" => "80c5376a34359caf831c1bef1724bb296764d1cd"
null_resource.upload_certificate: Provisioning with 'local-exec'...
null_resource.upload_certificate (local-exec): Executing: ["/bin/sh" "-c" "./cert_files.sh absent"]
null_resource.upload_certificate: Provisioning with 'local-exec'...
null_resource.upload_certificate (local-exec): Executing: ["/bin/sh" "-c" "./cert_files.sh present"]
null_resource.upload_certificate: Creation complete after 12s (ID:<PHONE_NUMBER>478337533)
netscaler_sslcertkey.lbsslcertkey: Modifying... (ID: TLS_ademoapp_9b464bfc0efeb1fd)
  certkey:       "TLS_ademoapp_9b464bfc0efeb1fd" => "TLS_ademoapp_123456789abcdef"
  nodomaincheck: "false" => "true"
netscaler_sslcertkey.lbsslcertkey: Modifications complete after 0s (ID: TLS_ademoapp_9b464bfc0efeb1fd)

Error: Error applying plan:

1 error(s) occurred:

* netscaler_lbvserver.lbvserver: netscaler_lbvserver.lbvserver: diffs didn't match during apply. This is a bug with Terraform and should be reported as a GitHub Issue.

Please include the following information in your report:

    Terraform Version: 0.11.13
    Resource ID: netscaler_lbvserver.lbvserver
    Mismatch reason: attribute mismatch: sslcertkey
    Diff One (usually from plan): *terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff{"sslcertkey":*terraform.ResourceAttrDiff{Old:"TLS_ademoapp_9b464bfc0efeb1fd", New:"TLS_ademoapp_123456789abcdef", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}}, Destroy:false, DestroyDeposed:false, DestroyTainted:false, Meta:map[string]interface {}(nil)}
    Diff Two (usually from apply): *terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff(nil), Destroy:false, DestroyDeposed:false, DestroyTainted:false, Meta:map[string]interface {}(nil)}

Also include as much context as you can about your config, state, and the steps you performed to trigger this error.

I verified the bug you reported.
The root cause is the fact that the netscaler_sslcertkey resource in the planning phase appears to be updated with only the name changing.
The actual code in the module does nothing when only this attribute is changed, so in the application phase the update was a noop.
Subsequently the key referenced by the netscaler_lbvserver resource was non existent so the diff was different from the plan and that caused the terraform framework to fail with the error message you reported.
The fix is implemented in #44 where we mark the certkey attribute with the ForceNew flag to force a deletion and recreation of the resource, which is the correct behavior.
The fix is also present in the latest release 0.11.13
For your terraform plan to work you will also need to add the lifecycle option create_before_destroy in the netscaler_sslcertkey resource.
This is  needed to avoid a cyclical dependency between the sslcertkey and the lbvserver resource.
Without this option terraform will first try to delete the old sslcertkey which will fail since it is referenced by the existing lbvserver.
Adding this lifecycle optoin avoids this problem since it performs all the updates before deleting the old certificate.
I have a sample resource script below that I used to verify the correctness of this operation.
resource "netscaler_sslcertkey" "lbsslcertkey" {
  certkey = "certificate_mock_1"
  cert = "/nsconfig/ssl/certificate_mock.pem"
  key = "/nsconfig/ssl/key_mock.pem"
  expirymonitor = "DISABLED"
  nodomaincheck = true
  inform = "PEM"
  lifecycle {
    create_before_destroy = true
  }

}

resource "netscaler_lbvserver" "lbvserver" {
  name = "ssltest_lbvserver"
  ipv46 = "<IP_ADDRESS>"
  port = "443"
  servicetype = "SSL"
  lbmethod = "ROUNDROBIN"
  persistencetype = "COOKIEINSERT"
  sslcertkey = "${netscaler_sslcertkey.lbsslcertkey.certkey}"
}

@ajsiersema let us know if this works out for you.

@giorgos-nikolopoulos ,
Thanks for the speedy reply and the explanation.
I've changed the resource script to include the lifecycle option, in fact, that's the only thing I've changed in my resources.tf.  However, it still fails, see below. I can't spot the difference with your test setup, or am I missing something else ?
resource "netscaler_sslcertkey" "lbsslcertkey" {
  count = "${var.ingress["service_type"] == "SSL" ? 1 : 0}"
  certkey = "TLS_${var.ingress["name"]}_${var.serial}"
  cert = "/nsconfig/ssl/${var.ingress["name"]}.cert"
  key = "/nsconfig/ssl/${var.ingress["name"]}.key"
  expirymonitor = "DISABLED"
  inform = "PEM"
  nodomaincheck = true

  lifecycle {
    create_before_destroy = true
  }

  depends_on = ["null_resource.upload_certificate"]
}

resource "netscaler_lbvserver" "lbvserver" {
  name = "lbvs_${var.ingress["name"]}"
  ipv46 = "${var.ingress["ip"]}"
  port = "${var.ingress["port"]}"
  servicetype = "${var.ingress["service_type"]}"
  lbmethod = "ROUNDROBIN"
  persistencetype = "COOKIEINSERT"
  sslcertkey = "${element(concat(netscaler_sslcertkey.lbsslcertkey.*.certkey, list("")), 0)}"
}

It still fails:
1 error(s) occurred:

* netscaler_lbvserver.lbvserver: netscaler_lbvserver.lbvserver: diffs didn't match during apply. This is a bug with Terraform and should be reported as a GitHub Issue.

Please include the following information in your report:

    Terraform Version: 0.11.13
    Resource ID: netscaler_lbvserver.lbvserver
    Mismatch reason: attribute mismatch: sslcertkey
    Diff One (usually from plan): *terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff{\"sslcertkey\":*terraform.ResourceAttrDiff{Old:\"TLS_ademoapp_ae137539706f54de\", New:\"TLS_ademoapp_9b464bfc0efeb1fd\", NewComputed:false, NewRemoved:false, NewExtra:interface {}(nil), RequiresNew:false, Sensitive:false, Type:0x0}}, Destroy:false, DestroyDeposed:false, DestroyTainted:false, Meta:map[string]interface {}(nil)}
    Diff Two (usually from apply): *terraform.InstanceDiff{mu:sync.Mutex{state:0, sema:0x0}, Attributes:map[string]*terraform.ResourceAttrDiff(nil), Destroy:false, DestroyDeposed:false, DestroyTainted:false, Meta:map[string]interface {}(nil)}

I changed my terraform version to match yours 0.11.13 and I added the dependency to the null_resource in case this was creating some different kind of flow.
It still works correctly for me.
The only thing I can think of is that the netscaler provider in the .terraform folder inside the configuration scripts folder is not updated to the 0.11.13 version.
Check the lock.json inside the .terraform folder to have a matching sha256sum with the binary from the release for the netscaler provider.
If this still fails I would need the whole output from the terraform apply to see if I can find any other possible sources of error.

Duh, that was it. I could have sworn I updated the provider to 0.11.13, but apparently I forgot to in my latest test environment.
I've now successfully changed the certificate twice.
Thanks !


Some Thumb functions in raw binary decoded as ARM despite specifying -a thumb

I am using retdec to decompile some Cortex-M0 firmware images in raw binary format. One thing I noticed is that sometimes retdec will mistakenly treat some valid Thumb functions as ARM functions and output nonsensical disassembly and decompilation results despite specifying -a thumb. Also when this happens, there's no way I know to manually override them as Thumb functions (e.g. by tweaking the .config.json file).
The command I used was roughly like:
retdec-decompiler -m raw -a thumb -b 32 -e little \
    --raw-section-vma <flash mapping address> \
    --raw-entry-point <reset vector address> \
    <firmware>

For confidentiality reasons the binary cannot be shared, but I can try to replicate the behavior with a minimal test case.

If you replicate the behavior with a minimal test case that would be awesome. I may then look if the problem is in the decoder and perhaps try to fix it.


Change list of Orgs to from Hard Code to DB

We need to change of the list of orgs in tag generator from hard code and instead retrieve from the data base.

created PR


Interoplate for waypoints, extrapolate for obstacles

If successful may help us relax the rate requirement. If nothing less it'll give the teams the benefit of doubt for waypoints, and not for obstacles, which is what we probably want.

Semi-blocked on #174

What's the ETA for this?


add badgest to readme and also to the docs index page

ref:
The project repository front page and/or website MUST identify and hyperlink to any achievements, including this best practices badge, within 48 hours of public recognition that the achievement has been attained.

Done.

Fixed by #219, closing this @v0lkan. Thanks!!


issue with Router.go() after upgrade to 1.0.8 from 1.0.7

updated iron router to 1.0.8 from 1.0.7 and get a problem:
I call Router.go('application', {_id: response}); in a call back of a Meteor method
where the definition for application route is:
Router.route('/application/:_id', {
name: 'application'
}
and response variable is response in callback from Meteor method
with 1.0.7 route goes properly to /application/2t6uD9ausB9MovToW
with 1.0.8 route goes incorrectly to /application/undefined2t6uD9ausB9MovToW
in 1.0.8 there is word 'undefined' added in front of the _id in the route

Update again to 1.0.9.

On Jun 2, 2015, at 12:49 PM, e9<EMAIL_ADDRESS>wrote:
updated iron router to 1.0.8 from 1.0.7 and get a problem:
I call Router.go('application', {_id: response}); in a call back of a Meteor method
where the definition for application route is:
Router.route('/application/:_id', {
name: 'application'
}
and response variable is response in callback from Meteor method
with 1.0.7 route goes properly to /application/2t6uD9ausB9MovToW
with 1.0.8 route goes incorrectly to /application/undefined2t6uD9ausB9MovToW
in 1.0.8 there is word 'undefined' added in front of the _id in the route
—
Reply to this email directly or view it on GitHub.

1.0.9 worked great, thanks!


pd: test upgrading a validator node

I've run through this process several times now. It's very important to use the --genesis-start flag added in https://github.com/penumbra-zone/penumbra/pull/3491, as well as the re-initialized priv_validator_state.json added in same. With those changes, I'm able to restart a chain post-upgrade by upgrading the two (2) genesis validators.

Closing as complete, as we're quite familiar with and confident about this part of the upgrade story.


fix: typos and tighten a sentence

Proposed Changes

Minor typos helps the documentation flow.
Remove implied reference to the option being described, helps readability. Less text more clarity.
Wrap reserved keywords in code backticks.

Thanks for this 😄


commit offset explicitly  when AutoCommit is disabled and added Test Code for Manual Commit

context

We should commit offset explicitly then AutoCommit disabled by donmikel · Pull Request #13 · ThreeDotsLabs/watermill-kafka

I took over his pull request and added test code for it.

I followed  his suggestion

Can you add a test for that case in that file: https://github.com/ThreeDotsLabs/watermill-kafka/blob/master/pkg/kafka/pubsub_test.go

I think that it is not needed to run entire tests.TestPubSub. I think that it should be enough to use TestPublishSubscribe from the test suite.

could you confirm when you have time?

@roblaszczak Hi. We disabled autocommit and experienced error which is fixed in this PR. Is it real to merge it? It's very necessary for us. Thanks.


Instrument dds callback time

513
Steps towards addressing https://github.com/microsoft/FluidFramework/issues/9618.
Description
Implements telemetry around processing operations/messages at the DDS level. Two things are instrumented:

Processing the operation/message (which includes code in the processCore() implementation of the DDSs themselves)

This scenario is not measured for local operations, since in that case the callbacks from DDS events don't run (they execute in a different codepath), the execution times are lower, and they would skew the actual numbers as seen when processing remote ops.

Executing callbacks from events emitted by the DDSs.

The first scenario "wraps" the second one, but the second one gives us more direct visibility into the time introduced by consumer callbacks, mostly without time spent by the Fluid Framework (if a DDS used the event emitter for internal purposes and added callbacks internally, that will currently be included in the measurements).
Also of note, the current infrastructure formats the duration field of performance telemetry event by applying Math.floor(), thus making it 0 for very small measurements, which show an actual value in the maxDuration field captured for these new events:
    "duration": 0,
    "count": 10,
    "minDuration": 0,
    "maxDuration": 0.09999999403953552,

Might be a good idea to change that formatting to round to 2 or 3 decimal places.
PR Checklist

Use the check-list below to ensure your branch is ready for PR. If the item is not applicable, leave it blank.

[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My code follows the code style of this project.
[x] I ran the lint checks which produced no new errors nor warnings for my changes.
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change.

Does this introduce a breaking change?

[ ] Yes
[x] No

Any relevant logs or outputs
These are some (trimmed) examples of the generated performance telemetry events (with a threshold of 10 instead of 100, as the PR has), with the ddsOpProcessing event also generated for local ops. Note the totalDuration in that case reports 0, while the corresponding event in the remote client reports 2463.
Local client
Interactive container
fluid:telemetry:FluidDataStoreRuntime ddsEventCallbacks
{
    "category": "performance",
    "duration": 1240,
    "count": 10,
    "minDuration": 0,
    "maxDuration":<PHONE_NUMBER>9403954,
    "clientType": "interactive",
}

fluid:telemetry:FluidDataStoreRuntime ddsEventCallbacks
{
    "category": "performance",
    "duration": 1023,
    "count": 10,
    "minDuration": 0,
    "maxDuration":<PHONE_NUMBER>089407,
    "clientType": "interactive",
}

fluid:telemetry:FluidDataStoreRuntime ddsOpProcessing
{
    "category": "performance",
    "duration": 0,
    "count": 10,
    "minDuration": 0,
    "maxDuration": 0.10000000894069672,
    "clientType": "interactive",
}

Remote client
Interactive container
fluid:telemetry:FluidDataStoreRuntime ddsEventCallbacks
{
    "category": "performance",
    "duration": 1431,
    "count": 10,
    "minDuration": 0,
    "maxDuration":<PHONE_NUMBER>1192093,
    "clientType": "interactive",
}

fluid:telemetry:FluidDataStoreRuntime ddsEventCallbacks
{
    "category": "performance",
    "duration": 1030,
    "count": 10,
    "minDuration": 0,
    "maxDuration": 207.5,
    "clientType": "interactive",
}

fluid:telemetry:FluidDataStoreRuntime ddsOpProcessing
{
    "category": "performance",
    "duration": 2463,
    "count": 10,
    "minDuration": 201,
    "maxDuration":<PHONE_NUMBER>089407,
    "clientType": "interactive",
}

Summarizer container
fluid:telemetry:FluidDataStoreRuntime ddsEventCallbacks
{
    "category": "performance",
    "duration": 0,
    "count": 10,
    "minDuration": 0,
    "maxDuration": 0,
    "clientType": "noninteractive/summarizer",
}

fluid:telemetry:FluidDataStoreRuntime ddsEventCallbacks
{
    "category": "performance",
    "duration": 0,
    "count": 10,
    "minDuration": 0,
    "maxDuration": 0.09999999403953552,
    "clientType": "noninteractive/summarizer",
}

fluid:telemetry:FluidDataStoreRuntime ddsOpProcessing
{
    "category": "performance",
    "duration": 0,
    "count": 10,
    "minDuration": 0,
    "maxDuration": 0.29999999701976776,
    "clientType": "noninteractive/summarizer",
}

@anthony-murphy @vladsud particularly interested in comments you might have.
Tangential to the change itself, the perfTelemetryAggregator module is my attempt at a generic component that can be used anywhere we want to capture aggregated telemetry, from a chat between Tony and myself. If it seems weird or it needs more work, we could capture the same information in-line in SharedObjectCore for now.

@vladsud @anthony-murphy , some updates already there, in case you have any feedback while I continue working on these other things (and coming up with better names for some things I'm not super happy with):

Making the sampling rate configurable.
Add details like the DDS type and/or event names to the telemetry event (with the TelemetryDataTag.CodeArtifact approach Mark mentioned above). The PR in its current state already produces separate telemetry events for each DDS instance (both for op processing and for callbacks), but we don't know which DDS type each one of those instances is.
Support more than a plain string for adding custom dimensions to tag the telemetry events. Probably an object with arbitrary properties.
Splitting the measured emit() function for child DDSes from the one SharedObjectCore's uses for its own events.

@vladsud probably a good time for another look if you have a chance.
This is how the events look right now. For the op-processing one the custom property is localOp: true/false (measurements for each are tracked separately). I added ddsType to the logger at the DDS level per @anthony-murphy 's suggestion so all DDS telemetry will have it.


Update readme

Fixme! :shipit:

@humo: r+

@fkaa You are awesome!

@homu: r+

:pushpin: Commit 7cf57a0 has been approved by cmr

:hourglass: Testing commit 7cf57a0 with merge fee4489...

The hidden arrow on the logo is a nice touch!

:broken_heart: Test failed - status

Hm, something is wrong with our travis config. Doc generation is broken.

@cmr yes, it would be great to have you taking a look at it.

@homu retry

:hourglass: Testing commit 7cf57a0 with merge 20c7277...

:sunny: Test successful - status


bugfix: fix checkstyle plugin can't exclude single file

Signed-off-by: slievrly<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
bugfix: fix checkstyle plugin can't exclude single file
fixs #2985
http://maven.apache.org/plugins/maven-checkstyle-plugin/checkstyle-mojo.html#encoding
https://checkstyle.sourceforge.io/config_filters.html
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2986 into develop will decrease coverage by 0.10%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #2986      +/-   ##
=============================================
- Coverage      50.26%   50.15%   -0.11%
  Complexity      3060     3060
=============================================
  Files            600      600
  Lines          19507    19480      -27
  Branches        2404     2359      -45
=============================================
- Hits            9806     9771      -35
+ Misses          8715     8701      -14
- Partials         986     1008      +22

Impacted Files
Coverage Δ
Complexity Δ

...io/seata/core/rpc/netty/RmNettyRemotingClient.java
39.13% <100.00%> (ø)
10.00 <1.00> (ø)

...io/seata/core/rpc/netty/TmNettyRemotingClient.java
66.10% <100.00%> (ø)
10.00 <1.00> (ø)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (ø%)

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterTMRequestConvertor.java
94.73% <0.00%> (-5.27%)
3.00% <0.00%> (ø%)

... and 27 more


Moon phases

Icon Request

Icon name: moon-new moon-waxing-crescent moon-first-quarter moon-waxing-gibbous moon-full moon-waning-gibbous moon-last-quarter moon-waning-crescent
Use case: displaying the current phase of the moon in applications and sites that usually display it, e.g. calendar, astrology, and weather services.
Screenshots of similar icons: https://static.thenounproject.com/png/11882-200.png

To avoid using a fill, perhaps a hatched, dotted, or lined pattern could be used to provide the shading for the unlit portion of the moon.

A personal first run as an idea: full-moon can be represented with the existing circle. I don't think the size of the gap in the two gibbous icons is considered ok since it creates a very small white space, and rotating the lined pattern for the shadow didn't help much either. Perhaps another approximation of shading could work, or replacing shading with a dotted line pattern for the unlit side.
Another option would be to not have a shadow at all, although this would create ambiguity between waxing and waning. Ultimately, I'd rather the guidelines be followed and the icons not exist within the pack.
moon-new

moon-waxing-crescent

moon-first-quarter

moon-waxing-gibbous

moon-waning-gibbous

moon-last-quarter

moon-waning-crescent

@dsvoid See also eclipse in https://github.com/lucide-icons/lucide/pull/1280#issuecomment-1549433462.

Hey @dsvoid,
Thank you for submitting this icon request. The thing is, we do not currently support fills (please refer to #458), crosshatching and similar patterns mimicking them are sadly not an acceptable alternative, since they do not scale well with either icon size nor stroke width. Nonetheless, I think we would happy to add these as soon as we add support for filled icons, up until then, I'm closing this as not planned, but marking them with a dedicated tag so they can be reopened later on).

I'm also leaving this Figma file here with some initial drafts for future filled versions, once we can finally support these: https://www.figma.com/file/A7K6e8DjdQDZOEHwry3yNl/lucide%2Fmoon-phases?type=design&node-id=0%3A1&mode=design&t=Mq1L0fEUkpPn4Mkt-1

An option could be to not show the part that's in the shadow

An option could be to not show the part that's in the shadow

I did think of that, unfortunately that still would not allow to differentiate between a full moon and a new moon.

I did think of that, unfortunately that still would not allow to differentiate between a full moon and a new moon.

circle-dashed -> moon-crescent -> moon-half -> moon-gibbous -> circle

It's not quite that easy, since a moon not only waxes but also wanes, but I guess in principle that could kinda work, although it's a bit sub-optimal, I don't like the lack of consistency between different stages & new moon, but using dashed parts is a bit funky as well:

but using dashed parts is a bit funky as well

Looks pretty good IMO.
We don't need the waning ones, they are just flipped, and we don't create flipped versions.

Looks pretty good IMO.

As the one who designed them I disagree. 😄

We don't need the waning ones, they are just flipped, and we don't create flipped versions.

Um... sure we do, we have all kinds of "up/down/right/left" variants for a whole lot of icons.

Hello, sorry for commenting but I was also looking into moon icons and where theses added?

Hello, sorry for commenting but I was also looking into moon icons and where theses added?

They weren't and are very unlikely to be. The dashed icons are simply not up to par, and we cannot yet support fills, I'm closing this issue again until we do.

@karsa-mistmere can you post the svgs for the dashed version so people can still use theme by taking them from here?

@karsa-mistmere can you post the svgs for the dashed version so people can still use theme by taking them from here?

I've added them to the Figma file above.


Youtube-dl only download specific codec if it has best resolution

Checklist

[X ] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

OK so I've been using youtube-dl --cookies cookies.txt -f '(bestvideo[vcodec*=av01]/bestvideo[vcodec=vp9]/bestvideo)+(bestaudio[acodec=opus]/bestaudio[acodec=vorbis]/bestaudio[acodec=aac]/bestaudio)/best' --sub-lang en --embed-subs --add-metadata %video% -o '%(title)s.%(ext)s' to download videos and have a question, when such a video has an av1 encode that is below the best resolution (let's say it has av1 144p but vp9 1080p) how would I tell it to download the best (less preferred) codec (in the case of the previous example vp9), instead of the lower quality (more preferred) codec?

There is sadly no easy way to do this. You can either make a list of all format codes like this guy, or you could use a large format selection with seperate codec check for each resolution (Something like: ((bestvideo[vcodec*=av01]/bestvideo[vcodec=vp9]/bestvideo)[height>2160]/(bestvideo[vcodec*=av01]/bestvideo[vcodec=vp9]/bestvideo)[height>1080]/**...**/(bestvideo[vcodec*=av01]/bestvideo[vcodec=vp9]/bestvideo)[height>144]/(bestvideo[vcodec*=av01]/bestvideo[vcodec=vp9]/bestvideo))+bestaudio**...**

Sorry to hear that, but thanks for the workarounds

https://github.com/ytdl-org/youtube-dl/pull/25959 now automatically selects the best codec av01>vp9>avc1...


add Fictiverse Redream Stable Diffusion UI 0.10

/verify

@StarsbySea could you suggest?
already manifest contains checkver and hashed
    "architecture": {
        "64bit": {
            "url": "https://github.com/Fictiverse/Redream/releases/download/1.10/Fictiverse_Redream_1.10.7z",
            "hash": "6bfdc4e0d058f99ada4563a95c6d8f68cffe76ad6e7c094c1e3bec6c8b6b674c"
        }
    },

    "checkver": "github",

why validation is not passed?

/verify

@StarsbySea could you review this PR and merge?

It requires stable-diffusion-webui

Automatic1111 stable-diffusion-webui
Make sure--xformers and --api commandline arguments are set in your "webui-user.bat":
set COMMANDLINE_ARGS=--xformers --api

which doesn't belong to known buckets and couldn't be added to suggest/depends.
Suppose it would be helpful to mention this point at least in notes section.


feat: enable custom provider for federated logins

Description of changes

Removed the filtering logic to check for supprted providers

Issue #, if available

Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

What's the motivation behind this change?


Add Accept:html header

Barba should imitate regular document requests. Here I set the Accept header sent by Chrome, excluding image related additions.
I noticed this because browser-sync stopped working when I added barba: https://github.com/BrowserSync/browser-sync/issues/1519

Coverage decreased (-0.1%) to 37.743% when pulling 016b456d01298e694f19b78f6e8d32160fa194b7 on bfred-it:patch-1 into da17b49be8eb741ff3a9673652353b45c35b8e12 on luruke:master.


Gramps.Gramps does not allow uninstall

after running winget install -e --id Gramps.Gramps
when running

winget uninstall -e --id Gramps.Gramps

winget uninstall Gramps

will return

No installed package found matching input criteria.

however the items is still found in control pannel uninstall apps

winget list gramps
Name        Id                Version
GrampsAIO64 GrampsAIO64 5.1.3 5.1.3

winget uninstall grampsAIO64   Does work
winget uninstall gramps  does not work

winget search gramps
Name   Id            Version
Gramps Gramps.Gramps 5.1.3

This issue should be solved now:
Name                                      Id                                        Version
-----------------------------------------------------------------------------------------------------
GrampsAIO                                 Gramps.Gramps                             5.1.3


Avoid caching the target directory during CI

The travis caches are currently between 5 and 10Gb and the build spends a lot of time (about 6 mins x 2) downloading and unpacking before the build and packing and uploading afterwards.

This seemed to speed things up, and reduced the cache size!


Cannot find module 'styled-components'

1.0.0 was working fine but when i updated it to 1.0.1 this error occur

styled-components is moved to peerDependencies in 1.0.1 along with react, react-dom, prop-types. So user is expected to install styled-components separately in their project.
Can you try installing styled-components in your project.

solved. but it should be pre install in styled-loaders-react package.

Nice that it's working now. If it is pre-installed, then the user might get conflicts in styled-components versions if they already have styled-components installed and will cause problems. Please see https://github.com/saisandeepvaddi/styled-loaders-react/pull/1

Then it should be mention in docs. Thanks for your time


Update README.md with ribbon icon ordering guide

Added CSS snippet workaround for Obsidian's ribbon icon order bug

Thank you for the contribution!


Broken Link

This link https://guides.getalby.com/user-guide/alby-lightning-account/connect-your-alby-lightning-account-to-other-apps  currently opens a 404 page which is not suppose to be the case

@itstomekk the link popped up when I tried searching for how to connect zeus wallet to alby on google


net/url: add tohex

Use to hex instead string like "0123456789ABCDEF", the former is more faster and readable.

@googlebot I signed it!


[ARPA P&E Reporting] EC 2.15 Picklist Mismatch Error

Submitted Description:
Error looks like:
(Type_of_Capital_Expenditure__c) : Type of Capital Expenditure: bad value for restricted picklist field: "Affordable housing, supportive housing, or recovery housing"
Field: Type_of_Capital_Expenditure__c
See screenshot for more. When this happened last quarter it was because the actual dropdown in UST portal was slightly different than the Excel-provided options (a comma difference for instance) but I can't spot the difference in this case.
(Optional) Uploaded file - click to download:
https://www.jotform.com/uploads/grants.dev/212007960331041/5414838234518850775/112.PNG

From @trumanc :

This is actually being caused by a bug within the Treasury portal with how they handle values that include a comma within the text value. Their bug is causing them to incorrectly add quotation marks around the value. We aren't able to fix it in our render tool, but we did report this bug to Treasury about a month ago. From your side, you'll just have to delete the quotation marks in the text box asking you to edit the value and then it should pass validation.


feat(questDirectives): a table type series of questions

This is useful for series of many likert type questions.
We probably need a parent questions that creates several sub questions with the questController.
The sub questions are very similar to the selectOne questions.

How about something like this:
var gridQuest = {
	type: 'grid',
	stem: 'How tasty are:',
	columns:['Very', 'Medium', 'Not'],
    advancedColumns: [{text:'High', value:7}, {text:'Low', value:1}],
	rows: [
		{name:'a01', stem:'Bannanas'},
		{name:'a02', stem:'Apples', required: true},
		{name:'a03', stem:'Grapes'}
	]
};

It should create a table with check-boxes.
Allow randomization of rows (using the mixer, same as selectOne).
Maybe we can keep track of the sum of results? If so it is critical to have an explicit flip property.
In the future we may want different types of columns (n/a, other=text).

@eladzlot I think it makes sense to implement this feature with a question type that has sub questions as properties. Researchers will probably easily understand that there is a type of question that includes a series of sub-questions. So, I recommend adding it with AddQuestionsSet.
Regarding the actual feature:

I think that it will look best if the check boxes are implemented as a selection of a rectangle in the row (perhaps with a V sign, or coloring the selected cell in black [can also add properties to change colors]).

Another possible way to indicate selection is to print the response label of the selected cell in a large bold font inside the selected cell.

Allow researchers show the response labels in each row (will appear by default only at the top row).

Based on this question type, you can also create two other useful types:

Matching (don't allow selecting the same response option in two different rows of the same table): for instance, if the response options are words in French and the questions are words in English, and the participants should match each word to the correct translation.
Ranking (based on a Matching question, but participants match rank to each item, and an additional button re-order the items by rank).

Different types of columns is a good idea.


May I contribute code to this repo?

I was wondering if there would be any chance that you could kindly allow me to create PRs for this repo?
I am planning to push 1) a fix to keep linkopts and 2) a unit test to check if the output format can match the regex used in clusterfuzz, and I would really appreciate it if I could create PR directly : )
If that is inconvenient, I am still happy to create CLs as before (which might take me a bit more time).
Thanks!

Hi Dongge,
As Kostya said in another thread, we aren't set up to easily backport PRs into our master version control. OTOH, any internal CLs that changes Centipede code base will auto-create a GitHub PR, so it's an easy workflow (internally, to be fair).
--Sergey

Sure, no problem.
I will submit CL to g3.


The "loading..." sign does not disappear after adding azure databases connection with a SQL account

OS: All
Build Version: 20201013.1
Repro Steps:

Create a new Linux Web App  and select Node.js as the runtime.
Expand Deployments node -> Click "Connect to a GitHub Repository..." -> Choose organization -> Choose repository with "node-todo" -> Choose "master" branch.
Browse website.
Expand Databases node -> Click "Add Azure Databases Connection..." -> Select a SQL account -> Select a database.
Wait 10 minutes -> Browse website again.

Expect:
The "loading..." sign disappears.

Actual:
The "loading..." sign does not disappear.

More Info:
This issue doesn't reproduce when adding azure databases connection with a MongoDB account

Could you share the project that you're using for this?

You probably just need to use the "cosmos" branch of the node-todo repo. The default "master" branch doesn't work with SQL

@EricJizbaMSFT Thanks for your guidance, this issue doesn't reproduce when using the "cosmos" branch of the node-todo repo.


Fixed DSM radio bug in rc_balance

Added call to rc_initialize_dsm() to rc_balance to make DSM radio work. Tested on EduMiP with an OrangeRX satellite receiver.

pulled into testing, thank you Auggie!


Tweaks to produce a functional NPM project page.

R: @addyosmani @wibblymat @gauntface
I had to change a few things with our template and build process to make the NPM project page look like it should.
Here's what the page looks like with these changes in place: https://www.npmjs.com/package/sw-appcache-behavior

LGTM. Sorry for the late review!


Not detecting BSD-2 license on parpd

The problem I think is that the lines are wrapped earlier so that extra prefixes, such as code comments markers, can be inserted in front of each line.
Here is the full text of the license, which matches most NetBSD derived files.
parpd: Proxy ARP Daemon
Copyright (c) 2008-2017 Roy Marples<EMAIL_ADDRESS>All rights reserved.

Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions
are met:
1. Redistributions of source code must retain the above copyright
   notice, this list of conditions and the following disclaimer.
2. Redistributions in binary form must reproduce the above copyright
   notice, this list of conditions and the following disclaimer in the
   documentation and/or other materials provided with the distribution.

   Specifically:

The addition of the project name on the first line
Numbers instead of bullets
Two different types of quotes around "as is"

Computers can't tell if those changes are legally significant or not. If you update the license to follow the official license text (regardless of wrapping) it should be detected.

Removing the project name fixed it, thanks for the tip.


Caching Fallback mechanism

Is your feature request related to a problem? Please describe.
I was wondering how I could configure the IDistributedCache so that it would not blow up the entire application should the cache be 'temporarily' unavailable. Say that I use RedisDistributedCache, and the redis server crashed. This does not mean that our application does not work at all anymore but only the caching layer. However, exceptions get thrown now that we are not always able to catch, e.g.: when using IdentityServer4.AccessTokenValidator with caching enabled. ref.
Is this something that can be configured right now, or would I have to implement my own version of RedisDistributedCache that does not throw exceptions.
Describe the solution you'd like
The ability to make the application still work without blowing up, while being able to detect the issue / trigger warnings.
Or temporarily switch to another caching mechanism (e.g. InMemoryCache, or 'dummy' cache)
Or being able provide multiple layers of caching (e.g. Memory -> Redis -> Mysql)
Describe alternatives you've considered
Writing my own implementation of RedisDistributedCache that is able to gracefully handle failures

If you can add dependency on Polly, then probably using the Fallback would be a best option.

This isn't something we intend to add to the cache directly. Retry logic and dependency management is a common cross-cutting concern and I would suggest something like Polly, or custom retry code to achieve what you're looking for.


WIP Use Travis CI to test the book build and then deploy it to the site, take 2

Continuation of https://github.com/algorithm-archivists/algorithm-archive/pull/284.
Don't merge this yet, because things need to be added.
The last PR failed in three ways:

The site went down because the CNAME for custom domain redirection was not present. After checking the log (which I saved), if you're curious, I noticed that there are several other files present in algorithm-archivists.github.io that do not come from either the main repo or building the book. These can persist because update_site.sh only copies files over, rather than wipe and move. This is personal preference, but I think the website repo should be entirely reproducible from the main repo via this script. You can see the difference by comparing your website repo with mine. Let me know which path you'd rather take, and I'll either need to add those files to the main repo and have them copy over during deploy, or have the deploy copy the build rather than wipe the book repo.

The guard for when the book should be deployed was wrong. Once the initial PR was merged, every Travis pull_request build (see below) was going to try deploy. This is before the merge actually happens. I don't think Travis would continue to this step if the build failed earlier, but it's still wrong.

A bunch of files got deleted. I still don't know how this happened, because I think they should have been present in the book build. I don't think they were extra files left over from before the flattening, either.

For reference, and because I didn't understand it for a while, there are two kinds of Travis builds:

push:

Under the status dialog, this is called Travis CI - Branch and continuous-integration/travis-ci/push.
"This is a normal build for the  branch. You should be able to reproduce it by checking out the branch locally."
What it does is test the branch as-is. By default, this will run for everyone who has a fork of the main repo every time they push to their own fork.

pull_request:

Under the status dialog, this is called Travis CI - Pull Request and continuous-integration/travis-ci/pr.
"This is a pull request build. It is running a build against the merge commit, after merging . Any changes that have been made to the master branch before the build ran are also included."
What is does is it simulates merge specified by the PR, then runs the Travis config. This only runs after creating a pull request.

The reason each type of CI run appears twice is that the former leads to the pretty GitHub summary, and the latter leads to the actual Travis log.
The config is now set up so that only push commits to master will deploy to algorithm-archivists.github.io. I checked, and this works in all cases: when there's a merge commit, a rebase and FF merge, a squash and merge, or a regular commit. Any time the history of master is updated, the book will be built and deployed.

Note that I changed the names of some environment variables you must set in Travis to be more explicit. Hopefully things are clearer now.

Now that Travis has been activated, you can see that the pr/pull_request build didn't deploy to the book, which would have broken the site...again...

Ok. I think we are ready to give this another go, but I would like for @berquist to be available when we merge this PR just in case... So maybe we should set up a time to give this a shot on discord?

Sure. I won't be around Monday EST, but any time from Tuesday EST on is fine.
In the meantime, read the first paragraph again. This PR needs to be updated before merging, otherwise the site will break.

Mind if I add a WIP to the title to reflect this?

Go right ahead.
Just to be clear, what changes I make next depend on your decision about
point 1.
On Sun, Aug 5, 2018 at 6:00 PM James Schloss<EMAIL_ADDRESS>wrote:

Mind if I add a WIP to the title to reflect this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Falgorithm-archivists%2Falgorithm-archive%2Fpull%2F316%23issuecomment-410551241&data=01|01|erb74%40pitt.edu|1bf6a44a39fb4ff9241a08d5fb1ec997|9ef9f489e0a04eeb87cc3a526112fd0d|1&sdata=O9%2Fs6l4M3dfoAHqXWuGhce5w1udGE0mwDXS7v%2FJFLn0%3D&reserved=0,
or mute the thread
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAsaEwWpU7RASZP8wluRvqoSjTT08AQ3ks5uN2regaJpZM4VljIQ&data=01|01|erb74%40pitt.edu|1bf6a44a39fb4ff9241a08d5fb1ec997|9ef9f489e0a04eeb87cc3a526112fd0d|1&sdata=mvDjO1E2EabpmUy96YZAIS0roUoTgkNQiqE%2F41kumww%3D&reserved=0
.

So to be clear: you are arguing there are a lot of extra files because...

update_site.sh only copies files over, rather than wipe and move. This is personal preference, but I think the website repo should be entirely reproducible from the main repo via this script.

I'm up for updating the update_site command. What do we need to do with the CNAME stuff?

Uncommenting this line will take care of CNAME. I still need time to check the rest of the files.
However, there is a new problem: https://travis-ci.org/algorithm-archivists/algorithm-archive/builds/409608474#L427. If you go to https://docs.travis-ci.com/user/pull-requests/#pull-requests-and-security-restrictions:

Travis CI makes encrypted variables and data available only to pull requests coming from the same repository. These are considered trustworthy, as only members with write access to the repository can send them.

Pull requests sent from forked repositories do not have access to encrypted variables or data.

If your build relies on encrypted variables to run, for instance to run Selenium tests with BrowserStack or Sauce Labs, your build needs to take this into account. You won’t be able to run these tests for pull requests from external contributors.

To work around this, restrict these tests only to situations where the environment variables are available, or disable them for pull requests entirely, as shown in the following example:

I need to figure out if the AAA will be affected by this. I think so, because I am on another org's repo, but for some reason this PR ran fine? It will take some investigation. Maybe later this week is best.

Hey @berquist, quick question on the state of this PR. Obviously, no pressure to finish, but I just wanted to make sure it was still on the table.

 I don't understand the sudden failure. https://travis-ci.org/algorithm-archivists/algorithm-archive/builds/432252148

I'll be honest, I have no idea how to do that.

Is there a button labeled "Restart job" or something similar in the area where I have whitespace on the right?

No, I see what you see

@leios are you signed in to Travis?

Yeah, alright. That was it. I logged in and could restart it. Sorry.


Can't connect to remote docker instance?

I'm testing Portainer and currently trying to add an new endpoint...

The remote dockerd exposes its API via port 2375
All firewalls allow traffic
I'm able to query the API from a shell on the box where Portainer is running.
I see traffic generated by Portainer to the dockers (tcpdump)
GET /_ping
GET /v1.24/info

replies

However, I'm constantly seeing in the logs:
2019/02/23 15:22:49 background schedule error (endpoint snapshot). Unable to create snapshot (endpoint=xxxxx, URL=tcp://xxxxx:2375) (err=Cannot connect
to the Docker daemon at tcp://xxxxx:2375. Is the docker daemon running?)
What could be wrong?

Hi @xme
From the box where Portainer is running, can you connect to the remote docker instance using the Docker CLI ? via docker -H ...

Confirmed working...
docker -H xxxxx:2375 ps

And what about running this container on the same host:
docker run -d -p 9000:9000 portainer/portainer -H tcp://xxxxx:2375

And what about running this container on the same host:
docker run -d -p 9000:9000 portainer/portainer -H tcp://xxxxx:2375

When running the container on the same host, it works...
I'm lost :)

Ok, problem solved!
I took more time to deep dive and it was not a problem related to Portainer. It was a dump MTU issue (mismatch of MTU size between the Docker bridge used by Portainer and the VLAN interface used to connect to the remote Docker instance... Sorry for bugging you!


Where are the devs?

Abandoning people isn’t cool.  Why not let Curve or Convex take this over??
Update needed

When v2? Let's goooo!


Obscuring MWS & Client credentials in config UI

Obscures credential fields with sensitive data in Magento Admin > System > Configuration > Payment Methods > Amazon Payments UI

Did you close this pull request intentionally?

Yes, apologies, fired it off a little too soon.
On Fri, May 29, 2015 at 12:32 PM, Nathan Grice<EMAIL_ADDRESS>wrote:

Did you close this pull request intentionally?
—
Reply to this email directly or view it on GitHub
https://github.com/amzn/amazon-payments-magento-plugin/pull/167#issuecomment-106865157
.

Will finish this off probably Monday and pull request again. All
credentials in Magento config are usually obscured for security reason and
we found that Amz Payments isn't doing this
On Fri, May 29, 2015 at 3:48 PM, David Z. Lerner<EMAIL_ADDRESS>wrote:

Yes, apologies, fired it off a little too soon.
On Fri, May 29, 2015 at 12:32 PM, Nathan Grice<EMAIL_ADDRESS>wrote:

Did you close this pull request intentionally?
—
Reply to this email directly or view it on GitHub
https://github.com/amzn/amazon-payments-magento-plugin/pull/167#issuecomment-106865157
.


Added sentry.io integration with fpvout org DSN

This pull request adds unhandled exception logging to the fpvout orgs sentry.io account.
Any caught exceptions that still need logging to sentry can use:
Sentry.captureException(e);
or
Sentry.captureMessage("Something went wrong");
Default PII scrubbing is on and server side PII scrubbing is also enforced.

will check sentry a bit more before merging. Adding this to the BETA millestones, so we can get crash feedbacks ! thanks !

commit 5322ed5 on branch feature/data-collection-agreement-popup add user agreement popup at first launch (so we can enable/disable sentry depending on user answer)
@j005u  Could you please rebase your PR on top of this branch so I can merge it ?

This has been implemented in 5322ed5 . Thanks @j005u :)


Prevent "Invalid variant playlist" error

Currently stripAds will remove the whole content of the playlist if a SCTE35 tag is found, because .split('\\n') will not split by newline (should be '\n') but instead return an array with one entry (the whole m3u8). The DISCONTINUITY and DATERANGE checks will then match and thus that line will be filter'ed and the join just returns an empty string, causing the "Invalid variant playlist" error and a lot of retry requests (#9). Also the for loop isn't needed because the RegExp g modifier replaces all occurrences anyway - and \\s doesn't match spaces (or newlines with m modifier), should be \s.
The problem is that in case of MIDROLL Ads, even with this PR, the stream will just stop and the buffering circle appears forever (only page refresh helps).

Hey, thanks a lot for the PR.
Looks good to me. But before I merge just to make sure, did you test it with a few streams (including some preroll ads)?
Got any tips on how to actually get ads to show up, as funny as it may sound?
Regarding the midroll problem, I'll open an issue to track this.

Was only able to test with Midroll Ads yet, even watched /twitchpresents because they play Midrolls in breaks. Prerolls are really sporadic for me - they seem to limit them based on the IP because I tested from different Browsers.
I keep an eye on how THA behaves with this PR in the coming days and will report back.

I've done a few tests myself on /twitchpresents now and it seems to work well. So I'm ready to pull. Thanks again for the PR.
I've also encountered the midroll issue and it seems it happens when all we have is ads and not a single chunk from the real stream. Basically after filtering we end up with this:
#EXT-X-VERSION:3
#EXT-X-TARGETDURATION:6
#EXT-X-MEDIA-SEQUENCE:22385
#EXT-X-TWITCH-ELAPSED-SECS:45964.239
#EXT-X-TWITCH-TOTAL-SECS:45994.380
#EXT-X-DATERANGE:ID="17e8a4e96a......[CUT]...."

My idea is to just reuse the last successful chunk (that contains a playable ts file) until Twitch gives us more chunks from the real stream. This will be glitchy, but I hope that it only lasts a few seconds.


Document behavior of constraints on a global mode service

File: engine/reference/commandline/service_create.md, CC @mstanleyjones
The behavior seems to be fairly clear if you find the original pull request that enables the current behavior: https://github.com/docker/swarmkit/pull/1570
The current way that this document page reads, it is unclear what the exact intended behavior is: global mode services do respect constraints.

Issue moved to docker/cli #592 via ZenHub


Fixed huge memory leak issue with $popovers

The $popovers array defined outside of the directive, was never cleared when popovers are removed from the DOM.
This causes a huge memory leak in large lists which greatly slows down the application. Since $popovers is actually never used - I removed it entirely. No more memory leaks!

@nohros Could you publish a new version to npm/etc. with this?

Btw I think I found some more. I'll update you guys when I'm certain.
On Thu, Jan 28, 2016, 6:11 AM Anna Henningsen<EMAIL_ADDRESS>wrote:

@nohros https://github.com/nohros Could you publish a new version to
npm/etc. with this?
—
Reply to this email directly or view it on GitHub
https://github.com/nohros/nsPopover/pull/108#issuecomment-175961784.

Just encountered this bug as well - good catch - is a new release planned?

@dsommerville - hey no problem! @nohros  - any chance you can publish the new release?


[core/TPLinear] Fix breaking change

Addresses: https://github.com/huggingface/peft/pull/1435#discussion_r1479110527
Fixes a breaking change introduced by #1435
since is_paralle_a is a public attribute, it is a breaking change to completely remove it as some users might be using it already
cc @pacman100 @BenjaminBossan

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Thank you. I do not speak Python.


Split Conv2d to Conv2d and ConvTranspose2d

Rebase to webnn-native latest commit as well.
Commit id: 0d5517e45532f5950a5ffddd316570032d58841f

@huningxin, PTAL, thanks!

@Honry , thanks. Code LGTM. My question is whether we should merge it into the webnn_delegate branch first?

I'm OK to merge it into webnn_delegate branch first, thus we could start rebasing webnn-native in old emscripten and tflite-support.

@Honry , you are right.
webnn-delegate-tfjs-tflite-node is for node usage which has less dependency.
webnn_delegate branch is more for wasm usage which depends on emscripten and tflite-support.
I am fine to merge it to this branch and work on webnn_delegate for wasm usage later.
Thanks!


Add most of the remaining CloudRail files

@PatrickStoklasa  can you update this PR to be updated from the base branch?

Updated with the latest commits from main branch


Include User's object id in Application claims

When configuring Application Claims issued by B2C you shouuld include User's Object Id (in addition to first/last name). Otherwise, APIM will not allow the user to sign up - it will display the signup screen after signing in in from B2C but then show 'Server error. Unable to send request. Please try again later.' when submitting the Sign Up page.

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

ID: 18a8a3ac-153d-90b9-4c66-cc6207eaafac
Version Independent ID: 35488468-4002-093e-1155-00c7fd063d13
Content: Authorize developer accounts by using Azure Active Directory B2C - Azure API Management
Content Source: articles/api-management/api-management-howto-aad-b2c.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

@mrochon Thank you for your feedback! We will review and update as appropriate.

Had this issue as well. Can confirm that the solution @mrochon suggests works.

Hello @mrochon @martinhenrysson - I believe the requirement for the user's Object Id is already documented in the article. However, it's not shown in the screenshot, which is truncated because of its length.

I'll see if I can make the requirements a little clearer in the article. It seems like they are easy to overlook. Thanks for raising the issue.


fix dependalot issues and warnings

Reading through #339 #377 #383 #387 and some choice warnings from compilation.

(needs #386  first, updating this in a second ... ;))

@louis993546 and @alisen I took the liberty of merging all the recent updates into one, so we fix interdependencies and make a huge PR. Bigger is better, right?


Fix typo in EnableControlPlane config option

This PR changes EnableControlPane ➡ EnableControlPlane.

I checked kube-vip/website for any config documentation, but looks like there are no references.

ah, I guess auto complete in vscode masked this..


Create Eclipse Deploy

Eclipse Deploy

eclipse deploy

Thanks for your contribution! We made a major refactor which is why we hadn't accepted this yet. Please feel free to try the new version and port your improvements there.
We just merged our new version to main.


Reintroduce imports

Imports were temporarily removed while I changed how we track and identify subscriptions. Now that design is settled, we can bring them back.

Needs some fine-tuning, but this looks like it might be working.


Hotfix for reinstantiating prediction_history if a miner registers

Could you also update the version in predictionnet/init.py to 2.0.1?


Update SPICE integration to include ctrl+alt+delete button

Description
There is no easy way to send the ctrl+alt+delete signal through SPICE.

At the moment the only way is to do that is to go to the hypervisor node and issue the command
virsh send-key one-<vm_id> KEY_LEFTCTRL KEY_LEFTALT KEY_DELETE

Use case
This is important when using Windows VMs when triggering the task manager.
Interface Changes
A button should be added like VNC does.

Additional Context
The feature is already supported by SPICE.

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

won't fix as we'll deprecate SPICE and RSunstone in 7.0


feat: arith_mult: an aesop tactic for proving IsMultiplicative statements

Add a tactic that proves multiplicativity of arithmetic functions using Aesop by applying structural lemmas, akin to the continuity and measurability tactics.

bors r+


daft cannot handle _metadata

Describe the bug
when there is _metadata, daft treat it as normal data file and raise error about missing row groups.
v0.2.3
To Reproduce
import shutil
import os
import pyarrow as pa
import pyarrow.parquet as pq
t1 = pa.Table.from_pydict({"field1": ["a", "b", "c"]})
t2 = pa.Table.from_pydict(
    {
        "field1": ["d", "e", "f"]
    }
)
path = "/tmp/test3.pq"
shutil.rmtree(path, ignore_errors=True)
schemas = []
metadatas = []
for _t in [t1, t2]:
    schemas.append(_t.schema)
    pq.write_to_dataset(
        _t,
        root_path=path,
        metadata_collector=metadatas
    )
pq.write_metadata(
    schemas[-1],
    f"{path}/_common_metadata",
)
pq.write_metadata(
    schemas[-1],
    f"{path}/_metadata",
    metadata_collector=metadatas
)
import daft

daft.read_parquet(path).to_pydict()

ValueError: DaftError::External Unable to create arrow chunk from streaming file reader/tmp/test3.pq/_metadata: External format error: File out of specification: Invalid thrift: protocol error
Expected behavior
should just work
Screenshots
If applicable, add screenshots to help explain your problem.

Hi @braindevices! Thanks for making an issue.
I just ran your code and it looks like /tmp/test3.pq is a directory. is that expected?
In [7]: ls /tmp/test3.pq/
1db50d2da7324203be646a84a97ef1bf-0.parquet  _common_metadata
65c175493f274598bdc2c1c88d66d2c5-0.parquet  _metadata

Currently we don't filter out files by extension so this
daft.read_parquet("/tmp/test3.pq/**.parquet").to_pydict() works

Oh, is this because the folder was created by Spark? I think Spark does some shenanigans with dumping a _metadata file by default.

I found this write up  from pyarrow

Oh, is this because the folder was created by Spark? I think Spark does some shenanigans with dumping a _metadata file by default.

Yes, so far spark and dask are doing this. The example just tries to mimic the results without other dependcies.
Since you mentioned the wildcard works then it should be fine for a while. But I guess this is kind of standard, maybe we can skip them by default.

The separated metadata has some advantage when you have lots of partition files. Then it can just obtain the info from a single file instead of crawling through the FS.


Added untriaged issue workflow.

Coming from https://github.com/opensearch-project/.github/issues/133, create a GitHub workflow for labelling untriaged issues. Please ensure that an untriaged label with color  exists before merging.

@dblock please fix line endings in the file \r\n (CRLF) to \n (LF). And in all other PRs were you're introducing this workflow too. Maybe: publish a patch for already merged ones.

@dblock please fix line endings in the file \r\n (CRLF) to \n (LF). And in all other PRs were you're introducing this workflow too. Maybe: publish a patch for already merged ones.

I copied a file we already had in the repos and I guess it had the wrong CRLF :(
Updated.
You might want to add a linter to the repo to catch these.


fixed non-port elements being added as ports for custom footprints

/claim #302
/closes #302
/claim #308
/closes #308
the silkscreen element used to be seen as a fifth port in the test

looks like you can't claim two bounties in one pr


re: dot 3x on telegram polkadot events

hey, polkadot{.js} detected phishing for https://t.me/c/1371228565/18396
https://polkadotlive.network/
https://polkadotlive.network/block-assets/index.html

@Tbaut Can you take a look at this one please in the extension. It is on the blocklist, but in some way it gets somewhat bypassed. So what happens when I click it is that the extension blocks, but the page opens a new tab anyway. So you end up with the blocked tab and then a new tab on this page.
Would need a work-around here, they seem to use something to get around the extension.

@Tbaut Can you take a look at this one please in the extension. It is on the blocklist, but in some way it gets somewhat bypassed. So what happens when I click it is that the extension blocks, but the page opens a new tab anyway. So you end up with the blocked tab and then a new tab on this page.
Would need a work-around here, they seem to use something to get around the extension.

seems like a scam, if you could confirm, would be great! thanks!

seems like a scam, if you could confirm, would be great! thanks!

No, there are official channels (and unofficial ones) everywhere - however in this case it is a clear scam. It was first reported on 19 Jan.
There are NO DOT airdrops ongoing, nor planned anytime into the future. If plans do change on the network, there will be ab announcement on the official https://polkadot.network page.

No, there are official channels (and unofficial ones) everywhere - however in this case it is a clear scam. It was first reported on 19 Jan.
There are NO DOT airdrops ongoing, nor planned anytime into the future. If plans do change on the network, there will be ab announcement on the official https://polkadot.network page.

thanks:)

thanks:)

@tannertanner1 Thanks for logging.
@Tbaut Can you take a look at this one please in the extension. It is on the blocklist, but in some way it gets bypassed.
So what happens when I click it is that the extension blocks, but the page opens a new tab anyway. So you end up with the blocked tab and then a new tab on this page.
Would need a work-around here, they seem to use something to get around the extension.

Its a cell phone mobile issue, I have noticed

@tannertanner1 Thanks for logging.
@Tbaut Can you take a look at this one please in the extension. It is on the blocklist, but in some way it gets bypassed.
So what happens when I click it is that the extension blocks, but the page opens a new tab anyway. So you end up with the blocked tab and then a new tab on this page.
Would need a work-around here, they seem to use something to get around the extension.

Its a cell phone mobile issue, I have noticed

I followed the 3x DOT scam through the blockchain
Some seems to go here: 13wj7TPehUG1stdyWouo4nZVai39rARmz2RGB1MTs22qgVCm
Which is then immediately sent to: 14oEqqqCabUQdsdq1zP1qdb8b4uhEnc2mL1YYHKf8xYbh8FB
And finally ends up here: 1exaAg2VJRQbyUBAeXcktChCAqjVP9TUxF3zo23R2T6EGdE
When you search this final address, you find it's the 20th largest wallet on the network, with 8.7 million DOT.
Trying to find it's origin. But I noticed it dumped 350,000 DOT to 0x1c3b9d3ce0630c922e67ea0f5883613877b5172db47cd8c159c05f75d55d04ed
Unfortunately that's where my search ended, because I can't view the wallet.

The final is the Binance address (sadly blocked that one befor, not quite such a good idea). will check the rest, thank you.

The final  1exa... is the Binance address (I sadly blocked that one before, which is not quite such a good idea :). Will check the rest, thank you.

Has some one at Web3 contacted Binance directly requesting they block scam victoms from sending their dot into that Binance address ¿
Perhaps contacting the exchanges direct to block any user on their platform sending dot into those addresses on regualar basis is one short term possibility¿

does that mean its moved into a generic Binance address for selling on
their exchange or into the wallet of the actual scammer?
On Tue, Feb 9, 2021 at 5:16 AM Jaco Greeff<EMAIL_ADDRESS>wrote:

The final is the Binance address (sadly blocked that one before, not quite
such a good idea :). will check the rest, thank you.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/polkadot-js/phishing/issues/72#issuecomment-775342720,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQAE4UGOOSSNFYS3RNBCDG3S6AS7ZANCNFSM4WUAQKCA
.

They moved it to Binance to sell.

In some good news one Dot holder questioned why a phishing link was showing
up in one of the scam dot groups. The fake admin using the name Urban said
"their tec" would look into it. Some of them know the net is closing in.
Now we just need someone to tell Bianance ( and other exchanges) of the
Phishing links and keep them updated so they can black list the addresses.
I would be happy to help with that.
.
On Fri, Feb 12, 2021 at 9:24 AM Jaco Greeff<EMAIL_ADDRESS>wrote:

They moved it to Binance to sell.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/polkadot-js/phishing/issues/72#issuecomment-777832688,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQAE4UHLDO2ZSWWTHB22YKTS6RKINANCNFSM4WUAQKCA
.

At least one person saved. Well, on their side they probably changed the address, so the game of whack-a-mole continues...

Closing this. The current version of the extension now correctly detects links opened in a new tab. (Previously it was only in the current window). Additionally, as mentioned, the addresses are on the list.


WIP: Initial pipeline build

Making PR to test pipeline build on sandbox
Signed-off-by: Lisa Rashidi-Ranjbar<EMAIL_ADDRESS>

This build is finally working on the sandbox and is ready for review. I ran everything except the push tag command.

@ernestojeda @dweomer @lenny-intel @tsconn23


Remove readme field from Cargo.toml

Since 1.46 (rust-lang/cargo#8277), cargo can automatically detect the value of the readme field.

bors r+


refmt splits labelled args to new line causing parse error when seeing cons operator

Input:
let make ::label=(String.concat "foo" ["bar", "baz", "this", "list", "needs", "to", "be", "longer", "than", "your", "length", "limit"]) () => label;

Output (cannot be compiled or refmt'd again):
let make
    ::
      label=(
        String.concat
          "foo"
          [
            "bar",
            "baz",
            "this",
            "list",
            "needs",
            "to",
            "be",
            "longer",
            "than",
            "your",
            "length",
            "limit"
          ]
      )
    () => label;

Expected output:
let make
    ::label=(
      String.concat
        "foo"
        [
          "bar",
          "baz",
          "this",
          "list",
          "needs",
          "to",
          "be",
          "longer",
          "than",
          "your",
          "length",
          "limit"
        ]
    )
    () => label;

Interesting. I was trying to repro this and couldn't.
Looks like it's being avoided by https://github.com/facebook/reason/blob/master/src/reason_pprint_ast.ml#L2162-L2174

Interesting. Looks like I was using an old version of refmt! Thanks for checking.


confusing performance between gdscript and c#

Godot version
4.0.1 stable
System information
system: windows 10, ryzen 6 3600, rx 590, 16gb RAM, android device: OPPO A57, CPH2387, android version 12
Issue description
I did a small test to measure performance between c# and gdscript
process: Instantiate a child scene in loop then add instantiated scene as child of a node.
(actual purpose it to building cells for an inventory in my game)
gdscript code:

c# code:

Result:
loop count 40
c# - 8ms
gd- 6ms
loop count 1000
c#- 167ms
gd- 150ms
loop count 10000
c#-4212ms
gd-1538ms
According to my test result, gdscript is faster.
But if you search the topic in internet, Even official docs saying that c# is faster.
The reason for this test:
My aim is to develop a large scale survival game which involves various type of game mechanics such.

player able to place objects such as floor,wall, and furniture.
different enemy systems
crafting system. player able to craft different items in different machines and able to track its progress (timer)
procedurally generated terrain system

Im confused whether gdscript can handle this game mechanics because my target platform is mobile devices, so decent performance is my first priority.
If gdscript can handle the performance, then im very happy with it because personally i don't like too much coding.
Also using gdscript i can easily maintain my code (thanks to contributors for this super easy language)
Many article suggest that you can use both language in a project. But if we stick on a single language,its more easy to maintain code. And if i hire more programmers later, then its easy to provide training in only on language.
(Note that its not a problem of the engine, i am posting this here to get best suggestions and replies from the peoples who contributing the engine code, because im seriously working on my project any wrong result will destroy future of my project.
Im also interested to contribute to engine code, but c++ is like a hell for me. So im testing the engine in all ways i can possible.)
Steps to reproduce
create a project, instantiate some scenes via code,
Minimal reproduction project
none

But if you search the topic in internet, Even official docs saying that c# is faster.

Well, who are you going to believe, "The Internet", or your own personal experience and some actual testing? 🙃
Jokes aside, there is a lot of nuance to the performance that gets lost when you try to put it into a nice box "Which language would be faster?" Because a fair and correct answer would be "It depends". With GDScript you get tighter engine integration, so less overhead when calling into the engine API, but the language itself can be slower as it is executed in a virtual machine and it's not a very optimized one, at least at the moment. We generally consider GDScript to be "fast enough" for most scenarios, even in Godot 3. But the runtime got faster in Godot 4, with more room to improve too.
With C# and .NET you get a faster runtime by itself, but you get more overhead when calling into the engine API, because it's a third-party language and it needs to hook into our executable in a more generic way. So you may count to 1 million faster in pure C# than in "pure" GDScript, but in practice, as your real life code will inevitably use a lot of engine APIs, it evens out. So do your own testing, just like you did.
I would also recommend coming with these questions to our community platforms, such as Discord. GitHub is not a support forum, and it's not exactly an issue to report that GDScript is actually pretty fast :) So I'm going to close your report.


chore: storybook migrate segment, timepicker, select, toast

Changes looks good, we can merge it after the #139 I think including those changes as well.


fixing wrong model sizes

as suggested by http://code.opencv.org/issues/3637
I am guessing a conversion from older models went wrong

@vpisarev i think this is kind of priority since it actually breaks down the detection using those models. Same for https://github.com/Itseez/opencv/pull/3661. Could you handle with priority?

:+1:


Correct Outlook mobile appid

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
95
23
link
:white_check_mark:

msteams-platform/tabs/how-to/authentication/tab-sso-register-aad.md
95
15
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

#assign-reviewer:@nwojod-MSFT

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
95
23
link
:white_check_mark:

msteams-platform/tabs/how-to/authentication/tab-sso-register-aad.md
95
15
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
95
23
link
:white_check_mark:

msteams-platform/tabs/how-to/authentication/tab-sso-register-aad.md
95
15
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.