DX expression on required doesn't refresh warning message on rule change

Using properties below and trying to have required based on the other two options. When leaving the first field if it's blank the required message stays. Is this a bug or is there another way to handle validation like this?
https://gfycat.com/ConsciousLimitedAmericanindianhorse
        first_name: {
          type: 'string',
          rules: {
            required: 'dx: {{$root.last_name}} == "" && {{$root.email}} == ""',
          },
        },
        last_name: {
          type: 'string',
          rules: {
            required: 'dx: {{$root.first_name}} == "" && {{$root.email}} == ""',
          },
        },
        email: {
          type: 'email',
          rules: {
            required: 'dx: {{$root.last_name}} == "" && {{$root.first_name}} == ""',
          },

Yeah, it's a bug, i will fix it as soon as possible

After testing, this is not a BUG, you need to cooperate with linkFields to achieve the desired feature
{
  "type": "object",
  "properties": {
    "first_name": {
      "type": "string",
      "rules": {
        "required": "dx: {{$root.last_name}} == \"\" && {{$root.email}} == \"\""
      }
    },
    "last_name": {
      "type": "string",
      "ui": {
        "linkFields": [
          {
            "fieldPath": "first_name",
            "rules": [
              "required"
            ]
          }
        ]
      },
      "rules": {
        "required": "dx: {{$root.first_name}} == \"\" && {{$root.email}} == \"\""
      }
    },
    "email": {
      "type": "email",
      "rules": {
        "required": "dx: {{$root.last_name}} == \"\" && {{$root.first_name}} == \"\""
      }
    }
  }
}


Allow multiple IPs for external IP

This exposes the list nature of the external IP to the configuration, so that in setups where multiple IPs are desired (e.g. for a HA-ish setup) that can be used.
The old-style single-IP configuration is still supported

Forgot to test upgrades; please don't merge this yet.

Upgrades seem to have no issues for me.

This LGTM but I'm unsure how I should test the backwards compatibility. I can edit the generated values.yaml but I don't know if that's what the real scenario is. For the vagrant box, we're generating it, so I don't see how there would be a mismatch. Is the customer reusing their old values.yaml a valid scenario?
What happens if they use our new file but have an old --set kube.external_ip=<IP_ADDRESS> in the command line of helm? Is that something we need to handle?
I naively changed:
vagrant@vagrant-kube:~/scf> diff -u output/helm/values.yaml{.orig,}
--- output/helm/values.yaml.orig 2018-02-23 14:48:34.454369557 -0500
+++ output/helm/values.yaml 2018-02-23 14:45:31.979082455 -0500
@@ -933,8 +933,7 @@
 services:
   loadbalanced: false
 kube:
-  external_ips:
-  - "<IP_ADDRESS>"
+  external_ip: "<IP_ADDRESS>"
   storage_class:
     persistent: "persistent"
     shared: "shared"

But make run returned an error:
Error: render error in "cf-opensuse/templates/tcp-router.yaml": template: cf-opensuse/templates/tcp-router.yaml:237:53: executing "cf-opensuse/templates/tcp-router.yaml" at <.Values.kube.externa...>: wrong type for value; expected []interface {}; got interface {}

I believe the correct edit is:

In values.yaml, have external_ips: []
In the custom config file, have external_ip: <IP_ADDRESS>

Alternatively, say --set kube.externap_ip=<IP_ADDRESS> on the command line.  If you want to do the new thing, it's --set kube.external_ips[0]=<IP_ADDRESS>.
But that does mean we should remove the default value in the external_ips, though.

Ok, LGTM. I assume there will be a corresponding change to the make/run scripts in UAA and SCF to set the external IP when fissile is bumped there.


Request entity too large

Platform:

[x]  iOS
[ ]  tvOS
[ ]  MacOS
[ ]  watchOS

Swift:

[x]  Yes ->  3.1
[ ]  No

sentry-swift installed with:

[x] Cocoapods
[ ] Carthage
[ ] Manually

Version of sentry-swift:
2.1.9

I have following issue:
Cannot upload dsyms
Actual result:
[DEBUG] sentry_cli::api headers:
[DEBUG] sentry_cli::api   Server: nginx
[DEBUG] sentry_cli::api   Date: Mon, 08 May 2017 06:38:03 GMT
[DEBUG] sentry_cli::api   Content-Type: text/html
[DEBUG] sentry_cli::api   Content-Length: 192
[DEBUG] sentry_cli::api   Connection: keep-alive
[DEBUG] sentry_cli::api   X-Served-By: app-59a746
[DEBUG] sentry_cli::api   Strict-Transport-Security: max-age=31536000; includeSubDomains; preload
[DEBUG] sentry_cli::api body:
<title>413 Request Entity Too Large</title>

413 Request Entity Too Large

nginx

Is this something i am doing wrong? I am using sentry-cli 1.8.0 and my app.dSYM.zip is 36MB

I seem to have posted this in the wrong repo, sorry for the inconvenience


Adds a groupcache-base cache, sort of.

This is more of a PoC for discussion than an actual submission!
GroupCache has the concept of a backfill, whereby it will pull from another source on cache miss, helping reduce hotspots and distributing the pull load. As-is, this isn't compatible with the []Cache concept, so the backfiller is nil, which will work, but results in worse than usual performance as a cache miss will result in all nodes being checked instead of the correct member pulling it through opportunistically.

Thanks for this.  I looked at groupcache from your previous comment but it's not something I'd have gotten to until I have more of the "knowns" taken care of. I do like the idea of having a proper distributed near-cache available without extra infrastructure requirements.
I tried running with this but kept getting "cache not found." Am I configuring it wrong or is that expected? I get it'd need refactoring to work well but wasn't sure if that means it's expected to work but not well?
For the backfill functionality, the high level idea is I'd need to rework the main layer flow such that checking slower caches and then calling the provider happens inside the "fillfunc"?  And that fillfunc might happen on a different server than the one the request originally came in on depending on who groupcache considers the "owner."  I can see that being workable in a generic manner, in most other cache implementations they'd just trigger the "fillfunc" on a cache miss like a callback function.

Closed since https://github.com/Michad/tilegroxy/pull/25 now exists


Xplat build of native code working

This is a big change because it includes cached binaries and headers from the release versions of Mac and Linux LLVM along with the cached build output for Windows. The default behavior of the pipeline is now to use the cached binaries, resulting in about a 20 min build time, while the full 3 hour build of LLVM is still an option. Lots of clean-up on the scripts is still left to do, as well as creating a multistage pipeline that can actually produce and test the fully formed nuget package, but this is a big chunk of the way there.

This PR is more for FYI. I'm going to go ahead and merge into xplat to continue working on cleaning up the overall diff.

This PR is more for FYI. I'm going to go ahead and merge into xplat to continue working on cleaning up the overall diff.


train speed get more and more slowly

Hi, I am trying to train the model in dataset, however I find that the speed is getting more and more slowly. I noticed that the embedding size is getting bigger, so whether we can change it? Thanks.

In my opinion the training procedure of this model is intented to train for one epoch only. The minimax-facility location (kCenterGreedy here as approximation) doesn't make sense for more than one epoch (imo).

So your mean is the only thing we should do is train one epoch for embedding file when we detect in new dataset?

yes, because the is not trained as i mentioned before. Procedure is as follows:

get embedding vectors of layer 2 and 3 with inferencing the pretrained network
concat these to vectors (with remaining w and h from layer 2)
do this for all train_images and you get the Memory Bank M (refering to the paper)
do a Sparse Random Projection (dimensionality reduction) for reduction of computation time
do a minimax facility location to reduce the memory bank to the embedding_coreset
Training Finished

Test:

load the embedding coreset
get knn with input image as queue vector and embedding coreset as "train" set of the knn
compute anomaly score with formula given from the paper
Finished

I hope this helps you

Hi, @SDJustus Thanks for your help, I will test it.


Canonical Config

The current FluentBitConfig controller in the operator owns Kubernetes Secret objects, and watches for the custom types, Input, Output, Parser, and Filter. Within the Reconcile we gather existing objects, and then performs a merge and re-render all the configuration options. However several types (especially in filter plugins) have map[string]string as parameters. Additionally when we load the objects, I think the order is not guaranteed to be the same every time. Due to these conditions, as we increase the number of Input, Filter, and Output objects created that may use map parameters, the config rendering becomes increasingly likely to be non-deterministic. This results in essentially an infinite loop in the reconciler...

An event causes Reconcile(), for example a new Output.
We gather all the objects and render the config.
Configuration Secret object is updated, which causes the Update Event to fire and notify our operator.
Reconcile() is called again.
We gather all the objects again and render the config.

This time around we should be generating the same config since nothing has truly changed, and effectively ending the reconcile loop.
However, since the configuration rendering is non-deterministic, the rendered configuration file is likely to be different than the original. Order of Add or Set params in Modify filter may be different for example. Or order of plugins may be different. When we save the configuration in the Secret object, the Secret is actually updated, with a new resource version, and the Update Event is fired, and we get notification to reconcile again...

The issue worsens with the more fluentbit config objects are present. The fewer objects, the likelihood is greater we end up with the same permutation and stop the loop. More objects, and likelihood decreases and we are constantly reconciling.
This results in two main problems:

Operator starts using a lot of CPU constantly
We're constantly updating the config file, which is causing fluent bit watcher to constantly restart fluent bit process.

The problem is easily reproducible by creating a lot of objects, especially Modify filter with Set and/or Add, and Output objects with Headers property.
This PR attempts to fix this issue and improve the operator by the following changes:

Move the KVs type into its own new params package within plugins package so that it can be used within plugins
Add InsertStringMap function to params package to be used to canonically insert a map
Sort items in all the plugin Load() functions
Update dependencies. This was mainly done so we can use more optimized controllerutil.CreateOrPatch() function. However it seems updating sigs.k8s.io/controller-runtime package caused update of everything.
Update the build to Go 1.16. It would not compile with the new updated dependency. I get the following error:

 > [builder 10/10] RUN CGO_ENABLED=0 GO111MODULE=on go build -a -o manager main.go:
#17 60.75 # sigs.k8s.io/controller-runtime/pkg/manager
#17 60.75 /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.6/pkg/manager/manager.go:53:2: duplicate method Start
#17 60.75 note: module requires Go 1.16
------
error: failed to solve: executor failed running [/bin/sh -c CGO_ENABLED=0 GO111MODULE=on go build -a -o manager main.go]: exit code: 2

An alternative, perhaps somewhat hacky, solution is to use an event filter in the controller, and in Update predicate catch that we are doing an update to the secret and return false indicating that this Update event should not be processed. This does not seem like a correct approach to me.

Redo of PR #130 to fix the signoff breakage.

@bojand Have to say this is a great enhancement to existing logic!
You mentioned you got compile errors after upgrading to go 1.16, is it solved now?
You also mentioned An alternative, perhaps somewhat hacky, solution is to use an event filter in the controller, and in Update predicate catch that we are doing an update to the secret and return false indicating that this Update event should not be processed. This does not seem like a correct approach to me., it seems that your PR doesn't use this alternative, right?

@benjaminhuo That's correct. This solution only performs deterministic consistent rendering.
The alternative would look something like this:
return ctrl.NewControllerManagedBy(mgr).
	For(&logging.FluentBitConfig{}).
	Owns(&corev1.Secret{}).
	Watches(&source.Kind{Type: &logging.Input{}}, &handler.EnqueueRequestForObject{}).
	Watches(&source.Kind{Type: &logging.Filter{}}, &handler.EnqueueRequestForObject{}).
	Watches(&source.Kind{Type: &logging.Output{}}, &handler.EnqueueRequestForObject{}).
	Watches(&source.Kind{Type: &logging.Parser{}}, &handler.EnqueueRequestForObject{}).
	WithEventFilter(predicate.Funcs{
		UpdateFunc: func(e event.UpdateEvent) bool {
			sOld := e.ObjectOld.(*corev1.Secret)
			sNew := e.ObjectNew.(*corev1.Secret)

			if sOld != nil && sNew != nil {
				return false
			}

			return true
		},
	}).
	Complete(r)

But I don't think this is what we want really...

@benjaminhuo That's correct. This solution only performs deterministic consistent rendering.
The alternative would look something like this:
return ctrl.NewControllerManagedBy(mgr).
	For(&logging.FluentBitConfig{}).
	Owns(&corev1.Secret{}).
	Watches(&source.Kind{Type: &logging.Input{}}, &handler.EnqueueRequestForObject{}).
	Watches(&source.Kind{Type: &logging.Filter{}}, &handler.EnqueueRequestForObject{}).
	Watches(&source.Kind{Type: &logging.Output{}}, &handler.EnqueueRequestForObject{}).
	Watches(&source.Kind{Type: &logging.Parser{}}, &handler.EnqueueRequestForObject{}).
	WithEventFilter(predicate.Funcs{
		UpdateFunc: func(e event.UpdateEvent) bool {
			sOld := e.ObjectOld.(*corev1.Secret)
			sNew := e.ObjectNew.(*corev1.Secret)

			if sOld != nil && sNew != nil {
				return false
			}

			return true
		},
	}).
	Complete(r)

But I don't think this is what we want really...

Agree, Thank You!


gravatar doesn't show on android

the gravatar avatar doesn't display on android, it works just fine on ios, but doen't work at all for android.
is there any fix for that?
  <ScrollView>
        <View style={{
            borderBottomWidth:1,
            borderColor:'#ddd'
            }}>
            <Gravatar style={{
                width:60,
                borderWidth: 3,
                height:60,
                borderRadius:30,
                margin:20,
                backgroundColor:'#333',
                padding:30
            }}
                options={{
<EMAIL_ADDRESS>                     secure: true
                }}/>
        </View>
        <DrawerItems {...props}/>
    </ScrollView>

the avatar should be above "today"

That is quite odd. There shouldn't be any platform specific code. Did you check if the request itself was successful. It's not enough to go on.

i fixed it, it had nothing to do with gravatar, sorry


How debug this error AWS EMR API Error (ValidationException ) while running EmrEtlRunner

Below is the error i am getting while running EmlEtlRunner.
While running this command.
./snowplow-emr-etl-runner run --config snowplow/4-storage/config/emretlrunner.yml --resolver snowplow/3-enrich/config/iglu_resolver.json --enrichments snowplow/3-enrich/config/enrichments/ --skip staging
D, [2017-09-22T11:26:11.077000 #17797] DEBUG -- : Initializing EMR jobflow
D, [2017-09-22T11:26:14.981000 #17797] DEBUG -- : EMR jobflow j-3ERLMNX8JB01B started, waiting for jobflow to complete...
F, [2017-09-22T11:36:17.039000 #17797] FATAL -- :
Snowplow::EmrEtlRunner::EmrExecutionError (EMR jobflow j-3ERLMNX8JB01B failed, check Amazon EMR console and Hadoop logs f        or details (help: https://github.com/snowplow/snowplow/wiki/Troubleshooting-jobs-on-Elastic-MapReduce). Data files not ar        chived.
Snowplow ETL: TERMINATING [STEP_FAILURE] ~ elapsed time n/a [2017-09-22 11:34:04 +0000 - ]

Elasticity S3DistCp Step: Raw S3 -> Raw HDFS: FAILED ~ 00:00:19 [2017-09-22 11:34:06 +0000 - 2017-09-22 11:34:25 +0        000]

Elasticity S3DistCp Step: Shredded S3 -> Shredded Archive S3: CANCELLED ~ elapsed time n/a [ - ]

Elasticity S3DistCp Step: Enriched S3 -> Enriched Archive S3: CANCELLED ~ elapsed time n/a [ - ]

Elasticity S3DistCp Step: Raw Staging S3 -> Raw Archive S3: CANCELLED ~ elapsed time n/a [ - ]

Elasticity S3DistCp Step: Shredded HDFS _SUCCESS -> S3: CANCELLED ~ elapsed time n/a [ - ]

Elasticity S3DistCp Step: Shredded HDFS -> S3: CANCELLED ~ elapsed time n/a [ - ]

Elasticity Spark Step: Shred Enriched Events: CANCELLED ~ elapsed time n/a [ - ]

Elasticity Custom Jar Step: Empty Raw HDFS: CANCELLED ~ elapsed time n/a [ - ]

Elasticity S3DistCp Step: Enriched HDFS _SUCCESS -> S3: CANCELLED ~ elapsed time n/a [ - ]

Elasticity S3DistCp Step: Enriched HDFS -> S3: CANCELLED ~ elapsed time n/a [ - ]

Elasticity Spark Step: Enrich Raw Events: CANCELLED ~ elapsed time n/a [ - ]):
uri:classloader:/emr-etl-runner/lib/snowplow-emr-etl-runner/emr_job.rb:572:in run' uri:classloader:/gems/contracts-0.11.0/lib/contracts/method_reference.rb:43:in send_to'
uri:classloader:/gems/contracts-0.11.0/lib/contracts/call_with.rb:76:in call_with' uri:classloader:/gems/contracts-0.11.0/lib/contracts/method_handler.rb:138:in block in redefine_method'
uri:classloader:/emr-etl-runner/lib/snowplow-emr-etl-runner/runner.rb:92:in run' uri:classloader:/gems/contracts-0.11.0/lib/contracts/method_reference.rb:43:in send_to'
uri:classloader:/gems/contracts-0.11.0/lib/contracts/call_with.rb:76:in call_with' uri:classloader:/gems/contracts-0.11.0/lib/contracts/method_handler.rb:138:in block in redefine_method'
uri:classloader:/emr-etl-runner/bin/snowplow-emr-etl-runner:41:in <main>' org/jruby/RubyKernel.java:979:in load'
uri:classloader:/META-INF/main.rb:1:in <main>' org/jruby/RubyKernel.java:961:in require'
uri:classloader:/META-INF/main.rb:1:in (root)' uri:classloader:/META-INF/jruby.home/lib/ruby/stdlib/rubygems/core_ext/kernel_require.rb:1:in '

Please help me to resolve this error.

Please direct your support request to our forum: https://discourse.snowplowanalytics.com


Way to adjust panel height

Version
v1.12.5
Development Environment
MacOS safari
Current Behavior
Currently when viewing the week view with many events, only 3 are shown. Then a + button is available to enlarge the panel.
Expected Behavior
I would like to overwrite the https://github.com/nhn/tui.calendar/blob/188b1846eb1023660e0776ef0fcfc1a4846e9b64/src/js/factory/weekView.js#L47 DEFAULT_PANELS so I have more configuration flexibility?

@haringsrob
We will discuss with the team whether or not to provide an API to allow the user to override the vertical panel of the weekly view.
If it's going to work, it'll take some time.
If you are in a hurry, I recommend using a fork.

Hi, thanks, it is not that urgent since I found a better solution for my case.
(I have 3 types of appointment, now I just show one type (time, allday or task) at the time.).

@haringsrob
Yeah :)
In the future, the TOAST UI Calendar team is considering providing it as a template.


Support for Docker

Support for Docker

I don't use Docker so I'm not sure exactly what files and directories need to be ignored.

It appears that there is no specific "Docker" template.  There was an idea I had before for Stacks like LAMP and MEAN which could be created based on certain container configurations.

May be


fixing ruby code in dev docs

found by merchant who emailed in
@jackfranklin Hi Jack, can you please take a look?

:+1:


Use with construct to compile  one time use blocks of code

In some projects I find the need to compile some "one time use" block of code, so i need to package it into a function and then call it.
I propose to use the with  statement to directly compile some block of code and execute it.
Example:

...some code...

with jit(arguments):
    ...code to be compiled...

Unfortunately, this isn't possible. with blocks aren't real entities in the output bytecode produced by the Python compiler and it does certain rearrangements of the code, so what you specified as "to be compiled" may be moved to the wrong side of the boundary.
Functions are the only entities Python treats as real entities Numba can work with. For this kind of code, the best option is to do some kind of:
...some code...

@numba.njit
def jitted_fun(p1, p2, ...):
    ...code to be compiled...
    return (mutated variables)

(v1, v2, ...) = jitted_fun(arguments)

Didn't know that. Thanks for the explanation.


Ensure nix-build works in codespaces

Fixes #12

I am thinking whether putting fix into docker-entrypoint.sh https://github.com/xtruder/nix-devcontainer/blob/main/src/docker-entrypoint.sh would be better? This will make sure that devcontainer.json can be kept as it is.

I think codespaces ignores the entry point from the container anyway, so maybe that won't work (and maybe that will cause other issues)? I'll try it.

Regarding testing, I have created devcontainer.json for codespaces testing, that uses src/Dockerfile directly and you can use it without pushing docker image. You will need to select .devcontainer/codespaces/devcontainer.json when creating codespace:

I assume this option is also available if you fork this repo and do it on your personal accounts.

@dsyer entrypoint seems to work for me. In what cases did you saw issues with entrypoint?


[SD-App] please add @Mystereon's M5ELite

https://github.com/Mystereon/M5ELite

The app has been scheduled for rebuild

Compilation of this package failed, it has been labeled as invalid until the source of the problem is identified and addressed

pre-hook:
inofile=M5ELite.ino; inofile_forced=$inofile; mv M5elite.ino $inofile
post-hook:


Prerendering server component with <component :is="prop">

Environment
Stackblitz
Reproduction
https://stackblitz.com/edit/nuxtissues18855
Describe the bug
When using a .server.vue component which uses a prop as tag via <component :is="">, pre-rendering via nuxi generate fails.
Example component:

<template>
  <component :is="tag">Szia!</component>
</template>

nuxt.config.ts
export default defineNuxtConfig({
  experimental: {
    componentIslands: true,
  },
  nitro: {
    prerender: {
      routes: ['/'],
    },
  },
});

Additional context
No response
Logs
No response

I tried replicating this one. The specific issue seems to be if you provide an undefined value to :is, which understandably throws an error. Providing an actual value seems to work correctly.
https://stackblitz.com/edit/nuxtissues18855-zw16uy?file=app.vue
Could you confirm or have I missed something

@harlan-zw My bad - played too much with it 😂 Fixed!

@manniL cannot reproduce as there is a syntax bug in the provided stackblitz snippet.
<template>

    <TheHeading :tag="h1" /> // in the snippet
    <TheHeading tag="h1" /> // as it should be

</template>

Also works for me like this
// app.vue
<template>

    <TheHeading :tag="h1" />

</template>

Please, clarify the issue

@alex-key Indeed, thanks. Fixed that in the reproduction ☺
As written above, run pnpm generate to see the error


Do Tachyon run on Node JS 8?

Will tachyon run on Node JS 8?
The node-tachyon branch
https://github.com/humanmade/tachyon/tree/def01d3aa546e9709db9ea04eee58f237ca31528

It should, yes. Try it and find out. :)


Update user guide 'Securely purchasing and storing Monero'

Closes #922

^ two previews because one comes from the new netlify account, which is paid and the other one is the old free account, which will be removed soon


no certs in /etc/istio-certs mount

Hi,
I applied below annotations to my pod in order to use use istio provided certs. But there are no certs listed in /etc/istio-certs volumeMount. If I install istio using istioctl, then it works .. I can see certs in my pod's istio side car volumeMount. But same does not work in banzai helm based istio installation. I tried patching mountMtlsCerts to true, but did not work.
Can you please suggest where do I need to enable in banzai istio CR.
Banzai Istio version is 1.6.4
proxy.istio.io/config: |
proxyMetadata:
      OUTPUT_CERTS: /etc/istio-certs
      SECRET_TTL: 10m
  sidecar.istio.io/userVolume: '[{"name": "istio-certs", "emptyDir": {"medium": "Memory"}}]'
  sidecar.istio.io/userVolumeMount: '[{"name": "istio-certs", "mountPath": "/etc/istio-certs"}]'

I do not see global.mountMtlsCerts set to true  in the istio-sidecar-injector map even though I applied patch succesfully and I can see it set to true CR. Is this the reason?

Hi,
Yeah, that seems to be the issue, let me verify and prepare a quick fix.

The banzaicloud/istio-operator:0.6.9 docker image should fix this problem, could you please update your istio-operator statefulset to use this version and verify that it solves the problem for you?

@Laci21  thank you . I will apply and update here soon

@Laci21
Now istio-certs volume and volumeMount as /etc/certs are getting added to istio side car as part of autoInjection.
But istio-certs volume points to istio-default instead of emptyDir medium : memory
So when I do
exec -it  -n  -c istio-proxy -- ls -ltr "/etc/certs"
total found files is 0

@Laci21
I attached service account name to my pod so that side car injector will attach istio-certs volume to istio.mypodserviceaccount secret. But should not this istio.mypodserviceaccount secret be automatically created by istio side injector?
Can you please help with fix


A11y_AzureDataStudio_Welcome_ExtendyourdataStudio_ScreenReader : The "SQL Admin Pack" is not announced as heading element for screen reader users.

"Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags:#A11y_AzureDataStudioJuly2020;#A11y_July27_2020_TestPass;#A11yMAS;#A11yTCS;#SQL Azure Data Studio;#Benchmark;#MAC;#Screenreader;#VoiceOver;#A11ySev2;#Benchmark;#WCAG1.3.1;#WCAG2.4.6;#FTP;
Environment Details:
Application Name: Azure Data Studio
Application Version: 1.21.0-insider
Commit: eccf3cf
Date: 2020-07-24T09:28:31.172Z
VS Code: 1.48.0
Electron: 9.1.0
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0
Operating system: macOS Catalina (Version 10.15.6 (19G73)
Screen Reader: VoiceOver
Repro Steps:

Launch Azure Data Studio Insiders application.
If Welcome screen is not shown on application launch, then navigate to Help --> Welcome.

Start screen reader, navigate to the "SQL Admin Pack" text and observe if it is announced as a heading element.

Actual:
When screen reader users navigate to the "SQL Admin Pack" text it is announced as a plain text but visually it is appearing as a heading element.
Expected:
The "SQL Admin Pack" text should be announced as a heading element with level 3 for screen reader users.
User Impact:
If the text that is visually appearing as a heading element but the same is not announced to the screen reader users then the screen reader users will have difficulty in understanding the structure of the screen content.
Attachment link for Reference:
11550_A11y_AzureDataStudio_Welcome_ExtendyourdataStudio_ScreenReader The "SQL Admin Pack" is not announced as heading element for screen reader users.zip

Issue is fixed Hence closing the bug and added attachment.
Githubtags:  #Closed;
11550_Fixed.zip


Create struct for generic 1d time series data

We want a simple structure for a time series z(t) with the following fields:

z  the time series data
t the time values measured from T_start
z_units
time_zone
start_time

We could have a second struct which adds an extra field for the uncertainties in the time series measurements

add another struct allowing for multiple sources of uncertainty, i.e.

instrument
variogram
representativeness

Perhaps we should make a type for uncertainties that we can then just pass into our current time series structs, e.g.
AbstactUncertainty
AbstractIntsrumentUncertainty
AbstractRepresentativenessUncertainty
specific types specialized on the data type, e.g. Float64
We could then form a CompositeUncertainty type


Fix how it checks if Daily Notes are enabled

Description
It seems that the Obsidian API for reading configuration has changed. In the API, there does not seem to be any configuration object available. Instead, there is a configDir property exposed on the Vault object.
https://github.com/obsidianmd/obsidian-api/blob/b0aa06eab03d8c39c0733949da5aa1edc2084b01/obsidian.d.ts#L2936-L2941
Although I could not find any reference to it, it seems that Obsidian stores the list of the active core plugins under ${configDir}/core-plugins.json.
Obsidian API reference:
https://github.com/obsidianmd/obsidian-api/blob/master/obsidian.d.ts
This fixes https://github.com/shichongrui/obsidian-rollover-daily-todos/issues/17
Note to the reviewer
I checked these changes with my local vault and everything seems to be working here. I can now see the plugin settings and I confirmed it works as expected.
Please check these changes against your vault as well to make sure these changes are somehow portable 🙏
It's the first time for me to work on Obsidian plugin.
You reference path.json in  checkDailyNotesEnabled, but path is not defined and throws an error.
I've submitted PR #20 which also attempts to correct this issue.

These changes don't correct the problem.
You reference path.json in checkDailyNotesEnabled, but path is not defined and throws an error.
I've submitted PR #20 which also attempts to correct this issue.

@trentonTrama I'm sorry, what path.json are you referring to? The only file referenced in these changes is core-plugins.json.
That file exists on my computer and the plugin works completely fine with these changes. But as I wrote in the description, there is no reference to core-plugins.json in the API so I would consider this more like a hack than an actual solution 😅 We should get in touch with the providers of the API to get an official way for what we are trying to achieve.

This is fixed by #21


Demag GUI tests crash on my machine

Here is an example of the output I'm getting before the tests crash:
test_VGP_viewer (pmagpy_tests.test_demag_gui.TestDemagGUI) ... -I- can't find zeq_gui_preferences file, using defualt default
data model: 2.5
....
-E- invalid parameters cannot assign to fit test0 for specimen SI3-59.8-D - was given: {'er_specimen_name': 'SI3-59.8-D', 'specimen_comp_name': u'test0', 'specimen_dang': 0}
Could not calculate dec or inc for specimen SI3-59.8-D component test0 with bounds 7.5mT and     145.0mT in coordinate_system geographic, component not added
-E- no measurement data for specimen SI3-59.9-D in coordinate system geographic
-E- invalid parameters cannot assign to fit test0 for specimen SI3-59.9-D - was given {'er_specimen_name': 'SI3-59.9-D', 'specimen_comp_name': u'test0', 'specimen_dang': 0}
Could not calculate dec or inc for specimen SI3-59.9-D component test0 with bounds 7.5mT and 145.0mT in coordinate_system geographic, component not added
libc++abi.dylib: terminating with uncaught exception of type std::out_of_range: basic_string::erase
Abort trap: 6

Breaks similarly with: test_check_sample_orientation_bad_good (pmagpy_tests.test_demag_gui.TestDemagGUI)
and:  test_read_write_pmag_tables (pmagpy_tests.test_demag_gui.TestDemagGUI)
The rest of the tests run correctly.  They all have this line in common:  self.ie_add_n_fits_to_all(n_fits), but I haven't done any digging.
@Caoimhinmg ?  I've been just skipping these tests on my machine, which is fine, but this might be useful information.

@moonshoes87 This is one of those completely mind boggling errors. It's some kind of low level error in a C++ library native to OSX and used by python on macs, which is why I've never noticed this. After digging on the internet I found one thread that mentioned an association with importing pylab so I removed the depreciated pylab imports from DemagIE which seems to be causing the error. I'm not sure it will fix it but right now it's my best hunch. Let me know if the error persists.

Yeah, I hate stuff like this :\  Unfortunately, I just pulled and the error does persist.
Is it possible that this error is affecting actual use of Demag GUI for Mac users, as well as testing?  Is there any kind of sequence I should try with Demag GUI to see if I can get it to crash under actual use conditions?

Yeah, of course, the tests only simulate user input. I would suppose that if it's crashing on ie_add_n_fits_to_all which is a helper method adding n fits using the DemagIE add all function. So I'd say open DemagGUI on a dataset without a pmag_specimens file (the test uses $PmagPyDir/pmagpy_tests/examples/demag_test_data), then open the DemagIE using command+e or the tools menu, put some info into the fit name/color and bounds boxes, and finally hit the add all button. It should add a fit with all the attributes in those boxes to every specimen where those bounds and name make sense, if this doesn't crash try opening the DemagIE again while it's open as that could be happening in the tests as well it should just bring the DemagIE to the top. I'll look into this a bit more in the mean time. Thanks @moonshoes87.

I tried out this sequence and it seems to work fine.  One slight glitch is that the ShowModal "ok" and "cancel" buttons (with issues with a particular specimen, after clicking 'add new fit to all specimens') both seem to do the same thing, or if they have a different effect it is unclear in the UI.
Seems like this might just be a weird testing environment issue.  Let me know if you want me to try anything else.

Hey @moonshoes87 I've done more digging on this and it appears that the library in question libc++abi.dylib is possibly a part of XCode on mac so updating those libraries might help fix the problem. I can't be totally sure as I can't debug and I can't seem to find anyone with exactly this problem on the internet. If this doesn't work I'll try restructuring the tests to prevent the scenario that's causing the error from ever occurring.

I just updated everything I could, and I still have the same error as before:
libc++abi.dylib: terminating with uncaught exception of type std::out_of_range: basic_string::erase
Abort trap: 6
It’s your call as to how much time you want to put into this — I’m ok with just skipping these tests.  Let me know if you want me to try any other debugging steps...

On Nov 3, 2016, at 3:35 PM, Kevin Gaastra<EMAIL_ADDRESS>wrote:
Hey @moonshoes87 https://github.com/moonshoes87 I've done more digging on this and it appears that the library in question libc++abi.dylib is possibly a part of XCode on mac so updating those libraries might help fix the problem. I can't be totally sure as I can't debug and I can't seem to find anyone with exactly this problem on the internet. If this doesn't work I'll try restructuring the tests to prevent the scenario that's causing the error from ever occurring.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/88#issuecomment-258294599, or mute the thread https://github.com/notifications/unsubscribe-auth/ABwGIhymcdTLzL1wLmIPKJ_gbEckBRQqks5q6mGwgaJpZM4KgNjN.

I am closing this issue, since the tests work on Linux/Windows, and the tested functionality works on OS X.


Reduce repository size (documentation images inflate the repo size)

Unfortunately, we maintain 2 copies of documentation images (one
for the documentation source; another for the generated documentation).
We need to solve this as it makes the repository size unproportionally
large.

Github Pages supports publishing the site from a separate branch called gh-pages. See here.
I think that might be a solution to this problem, in that the html files don't need to be on the master branch that users check out. But I haven't researched enough to understand what it means to make this change.

When users clone the repository, they clone all branches (and unfortunately all of the history diffs), so a different branch doesn't really help with that.  Keeping source and documentation in different branches is counter-intuitive as we want to manage code and docs in unison.

Hmm, a bit of Git newbie-ism from my side there. Shame on me :)
BTW, I cloned a clean repo now and excluding .git the repository is ~51 MB, with the documentation images a little over 5 MB (2.6 MB per copy). So doesn't look like this is the main thing driving up the repo size. There's another set of images used in the README and wiki which is another ~5MB, and there are the SSL checkpoints and wikitext data that are ~13MB each.

OK, cool.  I wince whenever I add images to the documentation.  Now it won't hurt as much.
The checkpoints and wikitext data are something we should be able to separate pretty easily.


Wrong paths on S3 after composer update

Hello,
After updating composer, keys of files on S3 are wrong. It's taking absolute path instead of relative. i.e. instead of making public folder in my bucket as root folder, it's creating var as root folder. Full path of files are like /var/www/app/public/css/app.css instead of public/css/app.css.
Issue started happening after Symfony/finder got updated to 3.4.7 from 3.4.6.
After little debugging, I found that $file->getPathName() returns absolute path after updating.

Any update? we are still facing this issue.

Yes. Whenever I do Cdn:push

@rishiyadav31 You still have this issue ?


[Bug] xct:SemanticEffect.Hint not being read on IOS voiceover

Description
xct:SemanticEffect.Hint not being read on IOS voiceover when is implemented
complete stack trace (if applicable) -->
Steps to Reproduce

Add any element
Add following lines:
AutomationProperties.IsInAccessibleTree="True"
AutomationProperties.Name="{Binding SomeText}
AutomationProperties.HelpText="{Binding SomeText2}
xct:SemanticEffect.Hint="{Binding SomeText3}
xct is not read

Expected Behavior
When you add semantics and automation properties both options should be read.
Actual Behavior
Only AutomationProperties of the native xamarin are read.
Basic Information
Version 2.04

@MarianHristov92
Could you attach a small repro?


Ferdi system tray single left mouse click problem

Bug description:
I'm using KDE Plasma and when Ferdi is minimized and i single click on it, it won't maximize Ferdi but i'm getting the same options like when i'm making a right mouse click: Show Ferdi, Disable notifications & audio, Quit Ferdi.
I wan't when i left click with mouse to restore Ferdi window to my previous state.
Environment:

Operating System: Arch Linux
Ferdi Version: 5.6.2-nightly.2

I am facing the same issue, whether I do a right click or left click on the system tray icon, it always shows the menu. And not just that, for some reason a lot of things behave oddly in the Linux version compared to the Windows version of Ferdi.

Any news on this? I always have to click the tray icon, then click on "Show Ferdi" why 2 clicks when a single left mouse click should show Ferdi? If I want the context menu I can right click the tray icon.

There's a lot of conversation going on in #2147 - and we would appreciate some code contributors to step in and help solve the problem.

There's a lot of conversation going on in #2147 - and we would appreciate some code contributors to step in and help solve the problem.

The issue reported here is not related with #2147 . The issue is that at least in Linux when clicking on the Ferdi tray icon using the left mouse button a context menu is opened. 2 clicks are required to show Ferdi window. If I want context menu I can click with the right mouse button.

This should now be fixed via #2234 - please try out the latest nightly (should appear in a couple of hours from now) and close this ticket.

Thanks a lot, now hiding/showing works again.

@vraravam this bug can be closed I guess.

what did you do to make it work?

Hey norbusan, what did you do to make it work?

@tonypachino Install the latest version, it is fixed.

@tonypachino Install the latest version, it is fixed.

i'm on Arch and i installed it from AUR, ferdi 5.6.5-1, not bin, not git, and doesn't work, but something is interesting, it was working on the beginning and then i think i updated my system with some package python.something, and then again i'm getting the options, show ferdi, disable notifications & audio, quit ferdi.
I don't know where is the problem?

@tonypachino Install the latest version, it is fixed.

i'm on Arch and i installed it from AUR, ferdi 5.6.5-1, not bin, not git, and doesn't work, but something is interesting, it was working on the beginning and then i think i updated my system with some package python.something, and then again i'm getting the options, show ferdi, disable notifications & audio, quit ferdi.
I don't know where is the problem?


FAQ: other TRE archetypes and data tiers

:white_check_mark: Checklist

[x] This pull request has a meaningful title.
[ ] If your changes are not yet ready to merge, you have marked this pull request as a draft pull request.

:ballot_box_with_check: Maintainers' checklist

[x] This pull request has had the appropriate labels assigned
[x] This pull request has been added to the SATRE backlog project board
[ ] This pull request has been assigned to one or more maintainers

:arrow_heading_up: Summary
Follow-up from https://github.com/sa-tre/satre-specification/pull/252#pullrequestreview-1646110635
:closed_umbrella: Related issues
:raising_hand: Acknowledging contributors

[x] All contributors to this pull request are already named in the table of contributors in the README file.
[ ] The following people should be added to the table of contributors in the README file:

I wonder if there's a pre-commit plugin that can split sentences on . for us (the only ones I'm aware of split on a fixed line length which is annoying)

I wonder if there's a pre-commit plugin that can split sentences on . for us (the only ones I'm aware of split on a fixed line length which is annoying)

I would be very happy if there were a tool to enforce semantic line breaks.
I think the trouble might be distinguishing between sentences ending in full stops and other uses like initials or decimals.


Revoke refresh token if password is changed

If the user changes his/her password, the old refresh token can still be used to generate new access tokens.
How to make refresh tokens invalid if username or password is changed

In my opinion refresh token should be blacklisted in this case.
Also RefreshToken should be blacklisted in case of new authentication IMHO

It's not the best UX, but you could delete the tokens from local storage and force the user to log in again.
Something like:
localStorage.removeItem('refresh_token');

This is my custom implementation of blacklist on consequent logins:
from rest_framework_simplejwt.token_blacklist.models import BlacklistedToken, OutstandingToken

def logout(user):
    for token in OutstandingToken.objects.filter(user=user).exclude(
            id__in=BlacklistedToken.objects.filter(token__user=user).values_list('token_id', flat=True),
    ):
        BlacklistedToken.objects.create(token=token)

class BaseTokenObtainPairSerializer(simplejwt_serializers.TokenObtainPairSerializer):

    @classmethod
    def get_token(cls, user):
        logout(user=user)
        token = super(BaseTokenObtainPairSerializer, cls).get_token(user)
        return token

But the above method will blacklist all the refresh tokens of the user. What should I do to invalidate the current refresh token only?

@Aniket-Singla You can get the current token from request.user  I think. Then you can use the corresponding outstanding token to blacklist it.
However IMHO, all tokens should be blacklisted if user changes her password/username. I made a signal for this .

@adwait-thattey  I already tried getting user from the request. But I did not find any way to only logout user from current device. Please suggest how to delete the current device token.

I've ended up posting the refresh token to a DRF APIView while also removing it from the backend. If the token is in the headers, that may be a more straightforward approach.
// authAPI.js

static async logout() {
        try {
            const refresh_token = localStorage.getItem('refresh_token');
            const response = await axiosInstance.post('/api/user/token/blacklist/', {
                refresh_token: refresh_token
            });
            axiosInstance.defaults.headers['Authorization'] = null;
            return response
        } catch (error) {
            throw error;
        }
    }

And the Django code:
from rest_framework.views import APIView
from rest_framework_simplejwt.token_blacklist.models import OutstandingToken
from rest_framework import permissions, status

class LogoutAndBlacklistRefreshTokenForUserView(APIView):
    permission_classes = (permissions.AllowAny,)
    authentication_classes = ()

    def delete(self, request):
        # find all tokens by user and blacklists them, forcing them to log out.
        try:
            tokens = OutstandingToken.objects.filter(user=request.user)
            for token in tokens:
                token = RefreshToken(token.jti)
                token.blacklist()
        except:
            token = RefreshToken(request.data.refresh_token)
            token.blacklist()
        return Response(status=status.HTTP_205_RESET_CONTENT)  # 204 means no content, 205 means no content and refresh

    def post(self, request):
        # Post is for logging out in current browser
        try:
            refresh_token = request.data["refresh_token"]
            token = RefreshToken(refresh_token)
            token.blacklist()
            return Response(status=status.HTTP_205_RESET_CONTENT)
        except:
            return Response(status=status.HTTP_400_BAD_REQUEST)

Thanks for sharing the code. I will try to delete the token by finding it from the header.

I reached at the following logout method:
class LogoutView(views.APIView):
    permission_classes = (permissions.IsAuthenticated,)

    def post(self, request, *args, **kwargs):
        return self.logout_current_session(request)

def logout_current_session(self, request, *args, **kwargs):
        # Post is for logging out in current browser
        try:
            refresh_token = request.data["refresh_token"]
            token = OutstandingToken.objects.get(token=refresh_token)
            if token.user == request.user:
                BlacklistedToken.objects.create(token=token)
            return Response(status=status.HTTP_205_RESET_CONTENT)
        except:
            return Response(status=status.HTTP_400_BAD_REQUEST)

I am taking the refresh_token in post body. Token will be deleted only if the submitted token belongs to the current logged in user. Permission classes is also set to IsAuthenticated .
Thanks for the inputs.

Closing the issue as no update in 2 years and @Toruitas has given a good working solution


Adds a check for proposal data on proposal detail page to prevent from crashing

Summary of changes

Adds a check for proposal data on proposal detail page to prevent from crashing. This crash is caused as a result of indexing of unknown proposals fix in this PR on webb-graphql.
Currently, when a proposal is added (created later, meaning, created not as a result of DKGProposalHandlerSection.ProposalAdded event but created when proposal gets removed i.e as a result of DKGProposalHandlerSection.ProposalRemoved event), the event data is missing which causes the crash on stats dapp.
To prevent this, stats dapp now displays {expired: true} in proposal details page when the data is not present and was removed, just like in the tangle event logs shown below.

NOTE: We should further investigate and add a fix to display the missing data. Link to the issue will be added below once created.
Link to issue on webb-graphql: https://github.com/webb-tools/webb-graphql/issues/86
Proposed area of change

[ ] apps/bridge-dapp
[x] apps/stats-dapp
[ ] apps/webbsite
[ ] apps/faucet
[ ] apps/tangle-website
[ ] libs/webb-ui-components

Tangle event logs:

Stats dapp proposal detail:

In the follow-up issue, is there no way we can cross-reference an existing item in the stats dapp with the removal option?

Maybe we can augment the event to show the data if it'll help. Certainly we would like to show what proposal was removed and if the data isn't there that might defeat the transparency of it. What do you think @devpavan04 ?


Incomplete description of impacted services (All applications using Azure AD)

Please specify which applications are using Azure AD (or a link to an article describing these services.
https://docs.microsoft.com/en-us/azure/active-directory/fundamentals/concept-fundamentals-security-defaults#protecting-all-users

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

ID: cfac0ecf-e5a9-c5ff-f8c2-ce27789708f5
Version Independent ID: 0f5e0cc9-a54f-a5c8-192d-d60a6a4db3ff
Content: Azure Active Directory security defaults
Content Source: articles/active-directory/fundamentals/concept-fundamentals-security-defaults.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@yvesrosius Thanks for your question. We are checking on this and will respond to you soon.

@yvesrosius are you referring to where it says "This functionality protects all applications registered with Azure AD"?
This means that any application registered in the Azure portal can be protected with Azure MFA. You can also use conditional access to apply MFA to certain actions or access by users. https://docs.microsoft.com/en-us/azure/active-directory/authentication/tutorial-enable-azure-mfa
Azure MFA prerequisites are listed here: https://docs.microsoft.com/en-us/azure/active-directory/authentication/howto-mfa-getstarted

I'm closing this out because I believe it describes the services in the article, but feel free to tag me in the comments or open a new issue if I misunderstood!


Add support for preview links for public sharing

Add 3 new server configuration options:

PREVIEW_IMAGE
PREVIEW_TITLE
PREVIEW_DESCRIPTION

that would set according index.html meta tags to generate proper preview links during public sharing.

Hi, I'd like to work on this. I completed the changes on my local


Version 0.6.4

@tshemsedinov can you update jstp to v2.1.0 and release impress v0.7.0 with it?

@belochub I am going to publish 0.6.4 without jstp 0.2 (this branch rebased) but will not land this to master. Then I'll publish 0.7 with jstp 3.0, ok?

Published to npm


Colour Stitching:  Seam Awareness

because each chunk is only aware of itself.  it cannot access the noisemap values within another chunk. this is what causes the colour inconsistencies along the edges of each chunk like this:

let's explain why how the colour calculation works, and from there we will pinpoint where the problem has occurred.
the colour of a vertex of the mesh is decided by the gradient of the vertex. the gradient of the vertex is calculated by taking the height of all the neighbours and subtracting the minimum height from the maximum height found. this value (the range) represents the steepness of this vertex.
the problem here. is that a vertex on the edge of the mesh only has 5 neighbours instead of the usual 8. a vertex on the corner of the mesh will only have 3 neighbours.
this means two things.
firstly, vertices with fewer neighbours will have a less accurate result since there is a smaller sample size.
secondly. vertices on the left of one chunk would need to access vertices on the right side of their intersecting chunk to output an accurate result.
let's see this depicted:

the red line shows a chunk boundary.
the green dots show the vertices along that boundary. (these points will be rendered twice, once for each chunk)
let's take the lime point with the dark green circle around it as an example:
this point belongs to the chunk on the left
in order for the green point to have accurate colour, it needs to access the points that are blue. but it cannot as these points exist within a separate chunk.
one fix would be to allow this chunk to access its neighbours. but there are two issue with that.
firstly. this will slow the map generation considerably because one chunk will have to access 8 other chunks for accurate colours
secondly, if we try and access a chunk that doesn't exist (has not been generated yet) then there will be an error.

i think i have come up with a solution to this problem,
if I add padding around the noisemap, border just 1 layer thick then it should fix this issue.
let me explain. if I have a mesh that is 3x2 . then I should generate noise that is  5x4:

the mesh is still 3x2. but the padding around the edge of the noise should in theory fix the seam awareness because now, a chunk doesn't need to access another chunk to get the values.  because the values are already there, they just arent rendered in the mesh.
essentially, all chunks will have a slight overlap. but the overlap is never rendered.
then the overlap is used to calculate the colours around the seams


Added unit tests to cover assertion methods with both required and op…

Added unit tests for assertion methods, covering scenarios with and without optional parameters.

Thanks for the contribution!


Runtime Warning in the Leap IDE

When the demo is run in the Leap IDE there's runtime warning.

This issue no longer reproduces.  Here is a screenshot of current state.

Recommend closing this issue as the warning no longer appears.
Note, the problem bqm is now printed where the warning used to appear.  That may not be ideal as it appears to be a bit of debug code and its size distracts from the smaller results text.  But that is a separate issue.

Agreed, I've just removed the print statement in #12


[CGAL] build failure

Host Environment

OS: Windows 11 version 21H2
Compiler: Powershell 5.1.22000.832

To Reproduce
Steps to reproduce the behavior:
./vcpkg.exe install cgal
Failure logs
error: building liblzma:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest port files with `git pull` and `vcpkg update`.
Then check for known issues at:
    https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+liblzma
You can submit a new issue at:
    https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[liblzma]+Build+error
Include '[liblzma] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
    vcpkg-tool version: 2022-07-21-a0e87e227afb536c62188c11ad029954f28fdb22
    vcpkg-scripts version: 48369f2a2 2022-08-01 (3 hours ago)

You can also use the prefilled template from C:\dev\vcpkg\installed\vcpkg\issue_body.md.

PS C:\dev\vcpkg> git pull https://github.com/microsoft/vcpkg.git
From https://github.com/microsoft/vcpkg
 * branch                HEAD       -> FETCH_HEAD
Already up to date.
PS C:\dev\vcpkg>

This is the file contaning the entire console output in PowerShell, which include the error above and the package install details:
PS CGAL error log.txt
Since the issue lies with error: building liblzma, I’m also including the log files found in the following directories:

"C:\dev\vcpkg\buildtrees\liblzma\stdout-x86-windows.log"
-- Downloading https://github.com/xz-mirror/xz/archive/v5.2.5.tar.gz -> xz-mirror-xz-v5.2.5.tar.gz...
[DEBUG] To include the environment variables in debug output, pass --debug-env
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'manifests' = off
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
[DEBUG] Downloading https://github.com/xz-mirror/xz/archive/v5.2.5.tar.gz
[DEBUG] Download failed -- retrying after 1000 ms.
[DEBUG] Download failed -- retrying after 2000 ms.
[DEBUG] Download failed -- retrying after 4000 ms.
Error: Failed to download from mirror set:
https://github.com/xz-mirror/xz/archive/v5.2.5.tar.gz: WinHttpReceiveResponse() failed: 12002
https://github.com/xz-mirror/xz/archive/v5.2.5.tar.gz: WinHttpReceiveResponse() failed: 12002
https://github.com/xz-mirror/xz/archive/v5.2.5.tar.gz: WinHttpReceiveResponse() failed: 12002
https://github.com/xz-mirror/xz/archive/v5.2.5.tar.gz: WinHttpReceiveResponse() failed: 12002

[DEBUG] D:\a\_work\1\s\src\vcpkg\base\downloads.cpp(766):
[DEBUG] Time in subprocesses: 0 us
[DEBUG] Time in parsing JSON: 4 us
[DEBUG] Time in JSON reader: 0 us
[DEBUG] Time in filesystem: 116 us
[DEBUG] Time in loading ports: 0 us
[DEBUG] Exiting after 1.535 min (92125656 us)

CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):

      Failed to download file with error: 1
      If you use a proxy, please check your proxy setting. Possible causes are:

      1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
         to `https://address:port`. This is not correct, because `https://` prefix
         claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
         , etc..) is an HTTP proxy. Try setting `http://address:port` to both
         HTTP_PROXY and HTTPS_PROXY instead.

      2. You are using Fiddler. Currently a bug (https://github.com/microsoft/vcpkg/issues/17752)
         will set HTTPS_PROXY to `https://fiddler_address:port` which lead to problem 1 above.
         Workaround is open Windows 10 Settings App, and search for Proxy Configuration page,
         Change `http=address:port;https=address:port` to `address`, and fill the port number.

      3. Your proxy's remote server is out of service.

      In future vcpkg releases, if you are using Windows, you no longer need to set
      HTTP(S)_PROXY environment variables. Vcpkg will simply apply Windows IE Proxy
      Settings set by your proxy software. See (https://github.com/microsoft/vcpkg-tool/pull/49)
      and (https://github.com/microsoft/vcpkg-tool/pull/77)

      Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:273 (z_vcpkg_download_distfile_show_proxy_and_fail)
  scripts/cmake/vcpkg_from_github.cmake:106 (vcpkg_download_distfile)
  ports/liblzma/portfile.cmake:1 (vcpkg_from_github)
  scripts/ports.cmake:147 (include)

"C:\dev\vcpkg\buildtrees\liblzma\x86-windows.vcpkg_abi_info.txt"
 add_support_ios.patch 3955592d52b4c4fae8e02929c9f23ccbe70e55c1350227355291e49d9c67fcd6
build-tools.patch d630583ae687b126805edb078fcfc7b3fa7cd04895c3a4d4432f5eb4c7edd844
cmake 3.22.2
enable-uwp-builds.patch 2aa2f0734a4b02479d286fddb6f4e0163c7084d5e36c4fc43f3df01fb2f3fb6a
features core
fix_config_include.patch 34073e84d9c184b72f6270da26a049276f2881aeb990943697cfb9b70332b583
portfile.cmake eeb1c9efb5ead615727db0424ace610146236335bb376ee65f0b8966698ec592
ports.cmake cb19b7173c7d50b26a59a2b43239f2da55ca9fbbf1e78599ae3eba5bd1e8c458
post_build_checks 2
powershell 7.2.5
triplet x86-windows
triplet_abi 3e71dd1d4afa622894ae367adbbb1ecbd42c57c51428a86b675fa1c8cad3a581-c0600b35e024ce0485ed253ef5419f3686f7257cfb58cb6a24febcb600fc4b4c-249e2da0fced5777e3f907c3dc10380ddd2c9c98
usage 7e2c8d81961ea4c76c4478cf3c4ba6b7fa4e84e414f26a498ee413347d8dc082
vcpkg-cmake 50f0a77ead2a9591797c31ab8b2ae11e65630bf12e1c9966dc8fea4d68b1fd5e
vcpkg-cmake-config 14e5298107e69da30c91c6513ddfd2fd01ef55f65e51d3240e1aa8b3e2c40b6e
vcpkg-cmake-wrapper.cmake a4611673d837ec6e94f0f0f9a85a1859b85ef169e762b48a99946b9436647b39
vcpkg.json cb990bef9b33eb1d02df1d4771c1cdf274f5629a3a5d227a9003d8b0441f7039
vcpkg_check_features 3cdc889b7e6965ad5ddab803060aed2019c0177b6f314c7a5a947c01efa4db60
vcpkg_copy_pdbs d57e4f196c82dc562a9968c6155073094513c31e2de475694143d3aa47954b1c
vcpkg_copy_tools 3d45ff761bddbabe8923b52330168dc3abd295fa469d3f2e47cb14dce85332d5
vcpkg_fixup_pkgconfig 433d0d235b4e9f6c7bb8744a2f06a9f534b355da67d0d85b7c2daf15c8c77671
vcpkg_from_git 0aab20e34e84d52ba4763f009e539bfa8f418c41c918c8cf700156f1a8551a10
vcpkg_from_github b743742296a114ea1b18ae99672e02f142c4eb2bef7f57d36c038bedbfb0502f
vcpkg_replace_string d43c8699ce27e25d47367c970d1c546f6bc36b6df8fb0be0c3986eb5830bd4f1
win_output_name.patch ef21ba4b07a1bf358117fe7e847a3e4fd50a852800c24fba2402107230a826b1

Additional context
As stated in the logfile some of the possible causes for the error are related to the usage of proxy but in this context I’m not using any proxy.
Also I don’t know if this is relevant but there was initially an error while bootstrapping vcpkg using C:\dev> .\vcpkg\bootstrap-vcpkg.bat as shown in the Failure logs so I solved it by adding "-disable metric". So upon rerunning the same command there’s no error prompted and I’m getting Validating signature... done.

Please move this file to VCPKG_ROOT/downloads then try again.
The common workaround is you need to use proxy.
Thanks.

I've moved the xz-mirror-xz-v5.2.5.tar file into the the vcpkg/downloads. And I tried to remove the cgal library and reinstall using the following lines:
./vcpkg.exe remove cgal
./vcpkg.exe install cgal

A different list of packages were installed as compared to the first installation. Here're the file containing the console output:
PS CGAL error log (ii).txt
Also the error still persists but this time with [zlib] Build error instead of [liblzma] Build error.

Show error log
error: building zlib:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest port files with `git pull` and `vcpkg update`.
Then check for known issues at:
    https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+zlib
You can submit a new issue at:
    https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[zlib]+Build+error
Include '[zlib] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
    vcpkg-tool version: 2022-07-21-a0e87e227afb536c62188c11ad029954f28fdb22
    vcpkg-scripts version: 48369f2a2 2022-08-01 (8 hours ago)

You can also use the prefilled template from C:\dev\vcpkg\installed\vcpkg\issue_body.md.

@Alison-97 Yeah, that's why I said The common workaround is you need to use proxy.
Please read the error log carefully:
      Failed to download file with error: 1
      If you use a proxy, please check your proxy setting. Possible causes are:

      1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
         to `https://address:port`. This is not correct, because `https://` prefix
         claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
         , etc..) is an HTTP proxy. Try setting `http://address:port` to both
         HTTP_PROXY and HTTPS_PROXY instead.

      2. You are using Fiddler. Currently a bug (https://github.com/microsoft/vcpkg/issues/17752)
         will set HTTPS_PROXY to `https://fiddler_address:port` which lead to problem 1 above.
         Workaround is open Windows 10 Settings App, and search for Proxy Configuration page,
         Change `http=address:port;https=address:port` to `address`, and fill the port number.

      3. Your proxy's remote server is out of service.

      In future vcpkg releases, if you are using Windows, you no longer need to set
      HTTP(S)_PROXY environment variables. Vcpkg will simply apply Windows IE Proxy
      Settings set by your proxy software. See (https://github.com/microsoft/vcpkg-tool/pull/49)
      and (https://github.com/microsoft/vcpkg-tool/pull/77)

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


Missing Parent: MONDO:0010799	 mitochondrial non-syndromic sensorineural deafness with susceptibility to aminoglycoside exposure

from
https://github.com/monarch-initiative/mondo/issues/1587
I do have one more query though:
mitochondrial non-syndromic sensorineural deafness with susceptibility to aminoglycoside exposure
is not a descendant of
mitochondrial non-syndromic sensorineural deafness

Yes, this omission is odd. This is a definite anti-pattern (lexical pattern implies subclass when in fact they are siblings).
Scratching a bit deeper it's a bit more complex
These two terms are equivalenced to two different OMIMs:

https://omim.org/entry/500008 DEAFNESS, NONSYNDROMIC SENSORINEURAL, MITOCHONDRIAL
* mitochondrially inherited nonsyndromic sensorineural deafness can be caused by mutation in any 1 of several mitochondrial genes, including MTRNR1 (561000), MTTS1 (590080), MTCO1 (516030), MTTH (590040), MTND1 (516000), and MTTI (590045).
https://omim.org/entry/580000 DEAFNESS, AMINOGLYCOSIDE-INDUCED

aminoglycoside-induced deafness is associated with mutations in at least 2 mitochondrial-encoded genes, including MTRNR1 (561000) and MTCO1 (516030).
* = MONDO:0010799 mitochondrial non-syndromic sensorineural deafness with susceptibility to aminoglycoside exposure

The superclass-suggesting name in mondo for MONDO:0010799 comes from ORDO (https://www.ebi.ac.uk/ols/ontologies/ordo/terms?iri=http%3A%2F%2Fwww.orpha.net%2FORDO%2FOrphanet_168609) (note ORDO doesn't xref the OMIM but we made the call that they are equivalent)
Now, most cases we would not have one OMIM subsume another, because usually this is an error since OMIM entities are usually distinct representing gene-specific etiologies, but perhaps mt diseases are handled differently. In fact we should have a check set up to detect these.
Now it may be the case that in this case one of these OMIMs subsumes the other but I would be inclined to be conservative here and keep them siblings. I would however use the OMIM name rather than the ORDO name, which will resolve the apparent contradiction in there being no superclass relationship.
(We can ask Ada if there are plans to split the Mt diseases into gene-specific entries, but I just want to check we haven't asked this in another ticket before bothering her).

I would however use the OMIM name rather than the ORDO name, which will resolve the apparent contradiction in there being no superclass relationship.

I will revise the name of MONDO_0010799 to aminoglycoside-induced deafness


Regenerate server_objects.rb with latest from pyvmomi.

This file was originally generated from the pyvmomi Python SDK by a script
that is no longer in the bosh repo (see commit 4a23c5).
This version was generated using https://github.com/mmb/pyvmomi_to_ruby.
core_types.rb was also regenerated but had no changes.
Passes vSphere lifecycle with one cluster.
Not an official toolsmiths PR (sent as an individual).

Awesome thanks!

Merged into develop.

Updated. Thanks!


Animation Questions

Hi @drcmda. I tweeted to you earlier about some questions I may have. Thanks a lot for taking the time to response and offer to help. I got a few different questions here. I would really appreciate it if you help me with those. So here goes:

Say I have 2 buttons next to each other, each 50% of the width. Whenever I click on one, I want it to animate to replace the other button and take the full width of the container. It seems like a common use case for me, but I'm not sure how to do it. Here's a codesandbox I set up for it: https://codesandbox.io/s/5k4pzymqon.

If I'm writing a UI library or a reusable React component (publish to npm), would you recommend using react-spring as the animation framework for it, or should I try to do everything with transition and animation CSS? I'm not entirely sure what would be best.

If I'm abstracting the animation layer out of a UI component, would it still be a good idea to use animated.element to every elements in the UI component so I can make use of native rendering? For example, I have an AnimatedModal component that use Modal component. Should Modal be comprised of animated.div so that AnimatedModal can use native rendering, although maybe DifferentModal may not have any animation at all?

Thank you very much for offering to help. I've been using react-spring for a while and absolutely love it. If there's any issue that may be a bit more user friendly, may be I can help out.

@alexluong  Let's see ...

There are dozens of ways. You can spring your elements individually or do something with the parent container: https://codesandbox.io/s/186yqz18n7 A transition and a simple flex layout (each item on flex:1) would probably be the best.

It depends on the use case. When css is enough use css. If it isn?t or if it gets increasingly hard, use RS. I ship two libs myself with a RS dependency: animated-tree and mauerwerk, i don't think i'd like to do this with css because it would be messy if not impossible in some cases. Also, users can use their own configurations and effects easily.

I'd always use native + animated.el when possible. Unless you animate text, numbers or other foreign components. Native is faster and doesn't re-render the component. Imagine someone uses your lib and wraps a huge tree of components inside. If react would attempt to render this 60fps, it would be very slow.


Fails to install on nodejs

Expected Behavior

npm ERR! node v7.5.0
npm ERR! npm  v4.1.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node ./install.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node ./install.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the dlib-build package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs dlib-build
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls dlib-build
npm ERR! There is likely additional logging output above.
npm ERR!  (C:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:279:16)
181 verbose stack     at emitTwo (events.js:106:13)
181 verbose stack     at EventEmitter.emit (events.js:192:7)
181 verbose stack     at ChildProcess. (C:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:40:14)
181 verbose stack     at emitTwo (events.js:106:13)
181 verbose stack     at ChildProcess.emit (events.js:192:7)
181 verbose stack     at maybeClose (internal/child_process.js:890:16)
181 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
182 verbose pkgid<EMAIL_ADDRESS>183 verbose cwd C:\Users\IReznikov\Documents\FindMe
184 error Windows_NT 10.0.16299
185 error argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install" "dlib-build"
186 error node v7.5.0
187 error npm  v4.1.2
188 error code ELIFECYCLE
189 error<EMAIL_ADDRESS>install: node ./install.js
189 error Exit status 1
190 error Failed at the<EMAIL_ADDRESS>install script 'node ./install.js'.
190 error Make sure you have the latest version of node.js and npm installed.
190 error If you do, this is most likely a problem with the dlib-build package,
190 error not with npm itself.
190 error Tell the author that this fails on your system:
190 error     node ./install.js
190 error You can get information on how to open an issue for this project with:
190 error     npm bugs dlib-build
190 error Or if that isn't available, you can get their info via:
190 error     npm owner ls dlib-build
190 error There is likely additional logging output above.
191 verbose exit [ 1, true ]
Steps to Reproduce
npm install dlib-build

I don't know anything about the npm dlib package. You should ask this question to whoever is the package maintainer for it. Or whoever set it up.


Deconstify GrContext

When rust-azure uses GrContext, it needs a non-const pointer. This
allows it to stop trying to cast away the constness.

@larsbergstrom This is the first of two PRs to fix the rust-azure build. The second will be for rust-azure.

@bors-servo: r+

:pushpin: Commit 439ba06 has been approved by larsbergstrom

:hourglass: Testing commit 439ba06 with merge 62a452b...

:sunny: Test successful - travis


the meaning of keystore_keypass

The Question
What is the point of the keystore_keypass variable?
The Context
I'm trying to use the playbook with custom certs, but having a hard time making it work.
I have no previous experience with Java's trust- and keystores so assume ignorance on my part.
using the custom_certs play, I do get the trust- and keystores. But the confluent-server service fails with:
java.security.UnrecoverableKeyException:
Get Key failed:
Given final block not properly padded.
Such issues can arise if a bad key is used during decryption.

THE INTERNET tells me that this can be caused by several things, but likely is a bad password.
So on a whim I set listener.name.external.ssl.key.password to the same value as listener.name.external.ssl.keystore.password and the service now starts.
I then had a look at roles/confluent.kafka_broker/templates/listener.j2 where I see that kafka_broker_keystore_keypass is used.
Going back to roles/confluent.ssl/tasks/custom_certs.yml, keystore_keypass is nowhere to be found.
the use of a hard-coded key password is adding to my confusion.
-passout pass:mykeypassword
Please excuse my ignorance, but I'm thoroughly confused.

Hello @smedegaard thanks for your question.
When you generate keys and certificates there are generally three passwords you can set, one on each storage container (Keystore, Truststore) and a third on the private key.  Private key passwords on generally mandatory to set.
If you look at our self signed tasks:
https://github.com/confluentinc/cp-ansible/blob/5.4.1-post/roles/confluent.ssl/tasks/self_signed_certs.yml
You can see that even when we do the initial import of the CA cert into the truststore, we set a password on the key with the -keypass switch:
- name: Create Truststore and Import the CA Cert   shell: |     keytool -noprompt -keystore {{truststore_path}} \       -storetype pkcs12 \       -alias CARoot \       -import -file /var/ssl/private/generation/{{ssl_self_signed_ca_cert_filename}} \       -storepass {{truststore_storepass}} \       -keypass {{truststore_storepass}} {{extra_args}}
So to confirm, there is a password set on the Containers which contain your certificates, and there is password on the private key.
Hope that helps explain.

Thanks for the answer @JumaX !
I really appreciate you taking the time to explain this.
Sadly, I must admit that I'm still confused.
The flag you refer to -keypass {{truststore_storepass}} uses truststore_storepass, not keystore_keypass.
When searching for keystore_keypass in this repo here on Github I find it 6 times. One for each of the roles that use the SSL role. This makes sense.
But the search does not reveal a single place where the keystore_keypass is used in a play.
Not In the tasks in the SSL role. Nowhere. So either there's some Ansible trickery that I'm missing (likely 😀) or this variable is redundant since only the component specific keystore_keypass are used.
For example:
confluent.kafka_broker/templates/server.properties.j2 refers to the specific kafka_broker_keystore_keypass.
Assuming I'm correct, this will make the property files incorrect since it's getting <some_component>.ssl.key.password={{<some_component>_keystore_keypass}}. Where <some_component>_keystore_keypass is not used in any of the SSL tasks.
Again, I really do appreciate you (or anyone) taking time to answer this. And if I'm wrong about this please see this as an opportunity to share your knowledge. 😅
Regards,
Anders

My understanding is that for a java keystore you can have a password on the keystore itself, and on a key within it. Those passwords may be the same thing, which you've pointed out in the code.
Can you provide a copy of you inventory file? Also the scripts that generated the certificates/keys is helpful

Hi @domenicbove ,
I can't share the repo, but here's the gist of it.
I'm using a slightly modified version of the custom_certs. The CA, CA key and signed cert is copied to each vm by a company Ansible job. So I've removed the plays to copy those, and specified their paths in the respective variables.
Defined variables in the inventory are
 kafka_broker_default_listeners:
      external:
        name: EXTERNAL
        port: 9093
        ssl_enabled: true
        ssl_mutual_auth_enabled: false
        sasl_protocol: none
    kafka_broker_custom_listeners:
      internal:
        name: INTERNAL
        port: 9092
        ssl_enabled: false
        ssl_mutual_auth_enabled: false
        sasl_protocol: none

and in group_vars/all.yml
self_signed: false
ssl_custom_certs: true
regenerate_ca: true

ssl_ca_cert_filepath: "my_ca.crt"
ssl_signed_cert_filepath: "my_signed_cert.cer"
ssl_key_filepath: "my_key"

In desperation I've set all <component>_keystore_keypass to <component>_keystore_storepass in confluent.variables_handler/defaults/main.yml
for example:
kafka_broker_keystore_keypass: "{{ vault.kafka_broker_keystore_storepass }}"
The confluent-server service works with this config.
schema-registry is failing to start, but that's another topic (maybe?.)

I think you are missing the ssl_key_password var. Which would be the key password for ssl_key_filepath
https://github.com/confluentinc/cp-ansible/blob/5.4.1-post/roles/confluent.ssl/tasks/custom_certs.yml#L39
What branch of cp-ansible are you using?

Oh ya I just saw your link, it appears you don't pass a key password here:
https://gist.github.com/smedegaard/a7b8c85312a5ba6065ad0ec6187e05e7#file-custom_certs-yml-L19
And I think whats happening is the subsequent code updates the keypass within the keystore created. So maybe thats the fix, you need a keypass var

I branched out from 5.4.0-post, but I've merged 5.4.1-post.
There's no password for the ssl key (I know, but it's not my decision).
That's why I don't have  -passin pass:{{ssl_key_password}}.
And the keystore_keypass is not the same as ssl_key_password. One of my initial reasons for confusion 😄

yep this change was recent- someone pointed it out to me that there wasnt a password on the key
https://github.com/confluentinc/cp-ansible/issues/221
https://github.com/confluentinc/cp-ansible/pull/222
"merge 18 days ago"

Yeah, I saw the PR and thought my problems where solved for a happy 10 minutes. 😄
But  the CA key that's provided by my company has no password. That's what we are providing with  -passin pass:{{ssl_key_password}}, right?
So I don't actually need  -passin pass:{{ssl_key_password}}.
Please correct me if I'm wrong

This is pretty hard to do over comments, I am available for a call if you'd like.
It sounds like:

your custom certs do not have passwords

the custom certs tasks are successfully creating a keystore with this code (the deststorepass and destkeypass are set to the same thing):

- name: Create Keystore
  shell: |
    keytool -importkeystore \
      -srckeystore /var/ssl/private/generation/client.p12 \
      -srcstoretype pkcs12 \
      -srcstorepass {{ keystore_keypass }} \
      -destkeystore {{keystore_path}} \
      -deststoretype pkcs12 \
      -deststorepass {{ keystore_storepass }} \
      -destkeypass {{ keystore_storepass }} {{extra_args}}

But then in server.properties, these vars are not set correctly:

listener.name.{{listener.value.name|lower}}.ssl.keystore.location={{kafka_broker_keystore_path}}
listener.name.{{listener.value.name|lower}}.ssl.keystore.password={{kafka_broker_keystore_storepass}}
listener.name.{{listener.value.name|lower}}.ssl.key.password={{kafka_broker_keystore_keypass}}

Another important line is here: https://github.com/confluentinc/cp-ansible/blob/5.4.1-post/roles/confluent.variables_handlers/defaults/main.yml#L116
Which essentially says that unless you provide a keystore (not your scenario, make sure ssl_provided_keystore_and_truststore is not set to true) then the keypass var is set to the storepass var
Can you provide the server.properties ansible creates and what it should be? (you can paste in dummy passwords if there are security concerns just make sure the differences are clear)

@dominicbove Are you available in an hour?
Where can I reach you ? Slack?

    kafka_broker_custom_listeners:
      internal:
        name: INTERNAL
        port: 9091
        ssl_enabled: false
        ssl_mutual_auth_enabled: false
        sasl_protocol: none
      external:
        name: EXTERNAL
        port: 9092
        ssl_enabled: true
        ssl_mutual_auth_enabled: false
        sasl_protocol: none

make a PR for something like this
- name: Put Key and Signed Cert into pkcs12 Format
  shell: |
    openssl pkcs12 -export \
      -in /var/ssl/private/generation/signed.crt \
      -inkey /var/ssl/private/generation/key.pem \
      -passin pass:{{ssl_key_password}} \
      -out /var/ssl/private/generation/client.p12 \
      -name kafkassl \
      -passout pass:mykeypassword
  when: ssl_key_password is defined

- name: Put Key and Signed Cert into pkcs12 Format
  shell: |
    openssl pkcs12 -export \
      -in /var/ssl/private/generation/signed.crt \
      -inkey /var/ssl/private/generation/key.pem \
      -out /var/ssl/private/generation/client.p12 \
      -name kafkassl \
      -passout pass:mykeypassword
  when: ssl_key_password is not defined

This can be closed because it was all resolved in this pr: https://github.com/confluentinc/cp-ansible/pull/269


Window border not present after restoring from maximised fullscreen

Windows Terminal version (or Windows build number)
1.8.1521.0
Other Software
Also effects Command Prompt and Powershell at the time of writing this (Windows 10.0.22000.0).
Steps to reproduce

Open terminal
Maximise terminal window (Win+Up)
Enter fullscreen (F11)
Restore terminal window (Win+Down)

Expected Behavior
The terminal window should restore with a border.
Although it could be argued that the window should return to its maximised state, as this behaviour is used by windows explorer.

Actual Behavior
The terminal window restores to its previous location and size, however it does not have a border.

 I'd imagine that win+down shouldn't work at all from fullscreen, or if it does, it should at least toggle off the "fullscreen" window styles.
Out of curiosity, do you know if this repro'd in Windows 10? I wouldn't be surprised if this was a Win11 regression, but I'd be equally unsurprised if this was just always how the console worked.

I first saw the issue a few years ago in Windows 10 with cmd, and initially reported it via the feedback hub, but I don't think it ever got picked up.
Out of curiosity I re-enabled my insider account to test the issue on Windows 11 and it was still present.
Then, I tested it on terminal, and it was also effected.

initially reported it via the feedback hub, but I don't think it ever got picked up.

 At least if it did repro in Windows 10, then I don't need to go chase down the DWM team to figure out how it regressed. Now we just need to fix our code 😅

I attempted to find a solution myself, but was unsuccessful.
From what I found, win+down invokes onResize however onResize doesn't know the previous fullscreen state.
This means that if we were to check the value of _fullscreen in onResize, it would be true when entering fullscreen as well as when win+down is pressed (as we never actually exit fullscreen), so we cannot use the value of _fullscreen as a test.
I thought maybe onRestore may be invoked but that is only when transitioning from a minimized state.
This is my first time looking at the terminal source, so I'm probably looking in the wrong places :smile:. I look forward to seeing the solution :smile:.


Validate entire message instead of parts of it

permissions-request, protocols-configure, and protocols-query all validate like so:
Message.validateJsonSchema({ descriptor, authorization: { } });

we should consider constructing the entire message and then validating unless there's a reason not to. many other message classes already do this

One system I built validated parts of the message as well as the whole message (which you have to do even if the message contains only previously validated parts).  The idea was that often many messages share the bulk of the content with only minor changes, and we could validate the reusable parts once.  After doing it that way, I didn't see much of an advantage.

Done.


Allow pulp_pull to act as an exit plugin

This is required for arrangement version 4.
This is required for arrangement version 4, in which pulp_pull runs as an exit plugin. It must run in that phase because it has to run after pulp_publish, also an exit plugin -- pulp_publish has work to do when a build is failing, i.e. remove v1 content.
For arrangement versions earlier than 4, pulp_pull must still be able to run as a post-build plugin.
Signed-off-by: Tim Waugh<EMAIL_ADDRESS>

The reason to make it an exit plugin is that it has to run after pulp_publish, which is exit plugin

Updated commit message.

Inheriting from ExitPlugin allows it to run as an exit plugin. Additionally inheriting from PostBuildPlugin is only done for clarity.

Anyone have objections to merging?

Release notes updated.


npm install --save react-navigation - peer<EMAIL_ADDRESS>not installed

Description
When attempting to install react-navigation I run into npm warnings about peer<EMAIL_ADDRESS>not being installed.
Expected Behavior
What action did you perform, and what did you expect to happen?
Actions were as follows:
1.) create-react-native-app myAppName
2.) cd myAppName/
3.) npm install --save react-navigation
4.) Expecting successful installation of react-navigation...
Observed Behavior
npm install --save react-navigation
I get the following errors:
npm install --save react-navigation
<EMAIL_ADDRESS>/Users/myaccount/react_native/tests/navigation
└──<EMAIL_ADDRESS>
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of react@> 15.0.0 but none was installed.
USBUR60WB147306:navigation $

Environment

npm ls react-native-scripts<EMAIL_ADDRESS>npm ls react-native<EMAIL_ADDRESS>npm ls expo<EMAIL_ADDRESS>node -v: v8.2.1
npm -v: 4.6.1
yarn --version: 1.2.1
watchman version: command not found

Also specify:

Operating system: Mac OS X - 10.11.6
Phone/emulator/simulator & version: Android

peer dependency warnings are essentially meaningless given the strange semver policy that npm has chosen. you can ignore them. if you're curious, see more info at: https://github.com/npm/npm/issues/8854


Update sql-server-powershell.md

Missed telling the reader the actual piece of code to use (#NOSQLPS).

@SQLvariant : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Can we have a second option of champion pick in case of ban of the main one?

Hello, is it possible as the title?

Hello,
I don't know if it's possible. I need to analyze this.

Can skills be automatically upgraded? I really need it. Thank you

Can skills be automatically upgraded? I really need it. Thank you

I don't understand this question.
Can you be more explicit please ?

zdzd


[#868] feat(tez): Tez Shuffle Read supporting RemoteShuffle related shuffle and reader

What changes were proposed in this pull request?
Tez Shuffle Read supporting RemoteShuffle related shuffle and reader
Why are the changes needed?
Fix: #(868)
Does this PR introduce any user-facing change?
No.
How was this patch tested?
We will add test cases in a few days

Codecov Report

Merging #875 (86c92b7) into master (ec9c278) will increase coverage by 2.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #875      +/-   ##
============================================
+ Coverage     56.58%   58.70%   +2.11%
+ Complexity     2175     2174       -1
============================================
  Files           327      307      -20
  Lines         15981    13621    -2360
  Branches       1263     1263
============================================
- Hits           9043     7996    -1047
+ Misses         6430     5186    -1244
+ Partials        508      439      -69

see 22 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Enter on OSS

Name: Jose Lopez
GitHub: @gollodev
Twitter: @gollodev
Linkedin: https://www.linkedin.com/in/joselopezcv/
Website: joselopezweb.com

Completed Courses
Name of the Section

Course
Files

Course Name
link

@gollodev please, add your profile in the issue Students Profile #6.
Thank you! :smile:


Support Kytea python3

This PR makes to use Kytea with Python3.
close #2

Thank you for your PR. I checked. I think.... you might forget putting some files in your commit.
Considering your changes, following files should exist, however they are not in your PR.
JapaneseTokenizer/kytea_wrapper/kytea_wrapper_python3.py
and
test/test_kytea_wrapper_python3.py
Are they just copied one of python2 version? Or did you add special process for python3?

Oops... I added forgotten files.

Thanks ;)


Fail build on warnings

I have a case when I need to fail the build when there are any warnings in powershell. Is it possible?

Going to close this now as #21 should solve your issue 😄

@SharpeRAD So could you give me a hint how to fail a build on any warnings?


Update the URL to the QA server

The URl was updated in the main branch and that alteration was merged back to this branch.  Closing PR.


fix: switching in project tree is lagging

Description
Switching in project tree is causing too much re-rendering (including some full re-rendering), up to 8 times in a simple switch. This fix some simple refactoring that saves us at least 4 full rendering times, leaves us to 3-4 partial rendering. Will continue investigate the other unnecessary rendering.
The goal eventually is to have only one rendering in one switch.
Root cause

We always set local state in useEffect when location change.
We are setting < breadcrumb /> when switching the item, but the state only used by breadcrumb component and put it in the < DesignPage />  will make the page full re-render.
The < App /> will listen to the url to initialize the logger, the location change will make the whole app re-render

Fix

add value check, if no change, do nothing
move the < Breadscrum/> to a single component, and let it's own to manage the state
The same as 2

next step

split the < DesignPage />
investigate the reach/router's state management, There is still a full re-render caused by Router

Task Item
#minor

Screenshots
before

after

Coverage increased (+0.2%) to 55.065% when pulling 9ef8cf187b0a559ea3cd8731bb425b80db24f6c1 on lei9444:switch into 70c0d7b85a9c895c9cc688e6c959f7da1b59f84c on microsoft:main.

Coverage increased (+0.2%) to 55.065% when pulling 9ef8cf187b0a559ea3cd8731bb425b80db24f6c1 on lei9444:switch into 70c0d7b85a9c895c9cc688e6c959f7da1b59f84c on microsoft:main.


Add Carbonize to README

After peppering you with Safari related issues 😅, I've finally released my macOS wrapper for Carbon. It's currently climbing the ranks on Product Hunt. I'm pretty stoked about how it turned out and would love to know what you think.

I thought it might be cool to add it to your README as well. That's where the PR comes in.
I've done my best to attribute your work wherever I can. Let me know if you think it's sufficient.
The app is free and available on the Mac App Store.
Description
Carbonize wraps carbon.now.sh in a WKWebView and adds a bunch of features you'd expect from a native app:

Dragging and dropping code onto the Dock icon to load it automatically.
Using the Services menu to easily send selected text to Carbon.
Bespoke floating preview windows that allow you to drag images into other apps or sites.

By leaving your editor intact I'm confident the desktop wrapper can retain feature parity with the site fairly easily.
(Btw. I noticed the commit hook mangled the contributors overview, so I committed without activating the hooks. I also tried running yarn contrib:add boyvanamstel, but it failed with is/are invalid contribution type(s). Even on a clean branch it threw the same error. 🤷♂)

@boyvanamstel cool! I'll run the contribution script for you.
However, I want to update the copy added to the README since Carbon already has drag-and-drop... is it not working for you on Carbon?

BTW I really appreciate you calling out our company on your site btw 🙏

However, I want to update the copy added to the README since Carbon already has drag-and-drop... is it not working for you on Carbon?

I should’ve been more specific. With drag and drop I meant dragging and dropping of the exported image into other apps through the new save preview windows that pop up, and dragging and dropping code onto the Dock icon.
How about:
a macOS wrapper that natively integrates with the OS and adds cool features like dragging and dropping exported images into other apps

@boyvanamstel gotcha! 🙂
I would like it to be more succinct. What about: "A MacOS wrapper with extended native features"

@boyvanamstel gotcha! 🙂
I would like it to be more succinct. What about: "A MacOS wrapper with extended native features", just to keep a similar tone with the rest of the examples

That’s alright. Please make sure you put the caps in the right places though, or Apple will find you. ;)
It’s macOS.

I had no idea! Thanks! 👍


fix: normalize windows paths in config-manager

Discord user reported error on save: https://discord.com/channels/835168149439643678/1096106617184985240
Issue was due to config-manager not transforming the windows path separators

Yeah, I tried unsuccessfully to figure that out. I had to use the node debugger to see where it was failing.


CUDA out of memory

Dear author:
I ran into this issue while testing on my own dataset. And hope to get your help.My configuration is as follows：
The model is depthformer_swinl_22k_nyu.pth and config is  depthformer_swinl_22k_w7_nyu.py. In addition，I set sampler to:samples_per_gpu=1
,workers_per_gpu=1. How can I configure it to reduce cuda memory during testing?

And I would like to know how to test large scale images？Thanks

The best solution is to change to a GPU with more memory. Since you have set samples_per_gpu=1, the batch_size is the minimum for the model inference on GPU. While I do not have much knowledge on saving memory, I wonder if you can run the model on CPU as introductions in our inference as
CUDA_VISIBLE_DEVICES=-1 python tools/test.py configs/depthformer/depthformer_swinl_22k_w7_nyu.py path/to/your/model --eval x

Testing large-scale images needs more consideration of model size. That's tough for our depthformer w/ swinL. Your can look for more light-weight methods, like the recent paper SideRT: A Real-time Pure Transformer Architecture for Single Image Depth
Estimation.

最好的解决方案是更换为具有更多内存的 GPU。由于您已设置samples_per_gpu=1，因此batch_size是 GPU 上模型推理的最小值。虽然我对节省内存知之甚少，但我想知道您是否可以在 CPU 上运行模型作为我们推理中的介绍
CUDA_VISIBLE_DEVICES=-1 python tools/test.py configs/depthformer/depthformer_swinl_22k_w7_nyu.py path/to/your/model --eval x

测试大规模图像需要更多考虑模型大小。这对我们的带 swinL 的 depthformer 来说很难。您可以寻找更轻量级的方法，例如最近的论文SideRT: A Real-time Pure Transformer Architecture for Single Image Depth Estimation。

Thank you very much for your reply.

The best solution is to change to a GPU with more memory. Since you have set samples_per_gpu=1, the batch_size is the minimum for the model inference on GPU. While I do not have much knowledge on saving memory, I wonder if you can run the model on CPU as introductions in our inference as
CUDA_VISIBLE_DEVICES=-1 python tools/test.py configs/depthformer/depthformer_swinl_22k_w7_nyu.py path/to/your/model --eval x

Testing large-scale images needs more consideration of model size. That's tough for our depthformer w/ swinL. Your can look for more light-weight methods, like the recent paper SideRT: A Real-time Pure Transformer Architecture for Single Image Depth Estimation.

Thank you very much for your reply. I have tested on 3090 and I'm afraid it won't work on cpu because I have a lot of pictures. Now my solution is to crop the original image.

OK! That will be interesting to explore how to infer from high-resolution images effectively. Close this issue for now, but feel free for more discussions!