Keyvault (AccessPolicy?) usage in 16.8 or 16.9 causes panic

I ran into this same issue with both 16.8 and 16.9 of the provider.  I'm not exactly sure what is the cause, but I just re-ran a few tests and I hit the error when I tried this:
Existing (before):
        let vault_access_policy = new azure.keyvault.AccessPolicy(vault_access_policy_name, {
            vaultName: vault_name_parsed,
            resourceGroupName: vault_resource_group_name,
            tenantId: String(azure_config.tenantId),
            objectId: uai.principalId,
            keyPermissions: ['get', 'list'],
            certificatePermissions: ['get', 'list'],
            secretPermissions: ['get', 'list']
        })

Then changed to:
        let vault_access_policy = new azure.keyvault.AccessPolicy(vault_access_policy_name, {
            keyVaultId: AzureUtils.get_keyvault_id(env_id, region, vault.name, config.get('resource_prefix')),
            tenantId: String(azure_config.tenantId),
            objectId: uai.principalId,
            keyPermissions: ['get', 'list'],
            certificatePermissions: ['get', 'list'],
            secretPermissions: ['get', 'list']
        })

With running 16.9:
    root@243674c272cf:/data/pulumi/infra/azure/core-msi# pulumi plugin ls
    NAME   KIND      VERSION  SIZE    INSTALLED  LAST USED
    aws    resource  0.16.8   155 MB  n/a        51 minutes ago
    azure  resource  0.16.9   90 MB   n/a        51 minutes ago

    TOTAL plugin cache size: 245 MB

And I get this (whole lotta muck):
https://pastebin.com/2Sakd5UP

Hi @dbilleci-lightstream! Thanks for the detailed repro, I’ll take a look into this and see what we can do.

I've been attempting to reproduce this locally, and can't get it to happen with just the latest provider. My guess is that it requires the resources to have been created the using 0.16.7, and then the provider upgraded to a later version.
The program I'm using is:
import * as azure from "@pulumi/azure";

const resourceGroup = new azure.core.ResourceGroup("resourceGroup", {
    location: "WestUS",
});

const vault = new azure.keyvault.KeyVault("vault", {
    name: "repro183vault",

    resourceGroupName: resourceGroup.name,
    location: resourceGroup.location,

    tenantId: <string>azure.config.tenantId,

    sku: {
        name: "standard",
    },

    networkAcls: {
        defaultAction: "Deny",
        bypass: "AzureServices",
    },

    tags: {
        Purpose: "Repro azure#183"
    },
});

const vaultAccessPolicy = new azure.keyvault.AccessPolicy("policy", {
    keyVaultId: vault.id,
    tenantId: <string>azure.config.tenantId,
    objectId: "xxxx", // redacted, my user's object ID
    keyPermissions: ['get', 'list'],
    certificatePermissions: ['get', 'list'],
    secretPermissions: ['get', 'list']
});

The output from pulumi plugin ls (apologies, I have a lot...):
 ▲ Code/pulumi-repro/azure-183 pulumi plugin ls
NAME        KIND      VERSION                          SIZE    INSTALLED     LAST USED
aws         resource  0.16.8                           154 MB  6 days ago    now
aws         resource  0.16.7                           143 MB  2 weeks ago   now
aws         resource  0.16.5                           138 MB  1 month ago   now
aws         resource  0.16.4-dev.1544849085+gfa42da3b  135 MB  2 months ago  now
aws         resource  0.16.3                           135 MB  2 months ago  now
azure       resource  0.16.9                           89 MB   1 hour ago    now
azure       resource  0.16.8                           89 MB   6 days ago    now
azure       resource  0.16.7                           88 MB   2 weeks ago   now
docker      resource  0.16.4                           34 MB   2 weeks ago   now
docker      resource  0.16.2                           34 MB   2 months ago  now
f5bigip     resource  0.1.2-dev.1549067585+g0637c1d    34 MB   6 days ago    now
gcp         resource  0.16.8                           61 MB   6 days ago    now
gcp         resource  0.16.4                           61 MB   1 month ago   now
kubernetes  resource  0.20.2                           45 MB   6 days ago    now
kubernetes  resource  0.18.0                           43 MB   1 month ago   now
openstack   resource  0.16.5-dev.1550091790+g35ebcbd   40 MB   6 days ago    now
packet      resource  0.1.0                            31 MB   6 days ago    now
random      resource  0.4.0                            28 MB   6 days ago    now
random      resource  0.2.0                            35 MB   1 month ago   now
vsphere     resource  0.16.4                           53 MB   6 days ago    now

And finally the output from pulumi up:
 ▲ Code/pulumi-repro/azure-183 pulumi up                                                                                1m
Previewing update (dev):

     Type                            Name           Plan
 +   pulumi:pulumi:Stack             azure-183-dev  create
 +   ├─ azure:core:ResourceGroup     resourceGroup  create
 +   ├─ azure:keyvault:KeyVault      vault          create
 +   └─ azure:keyvault:AccessPolicy  policy         create

Resources:
    + 4 to create

Do you want to perform this update? yes
Updating (dev):

     Type                            Name           Status
 +   pulumi:pulumi:Stack             azure-183-dev  created
 +   ├─ azure:core:ResourceGroup     resourceGroup  created
 +   ├─ azure:keyvault:KeyVault      vault          created
 +   └─ azure:keyvault:AccessPolicy  policy         created

Resources:
    + 4 created

Duration: 2m13s

Permalink: https://app.pulumi.com/jen20/azure-183/dev/updates/11

I'll try uninstalling the newer plugins and running through the same upgrade path.

I think the part that triggered it for me was changing the 2 parameters in the access policy:
original with resourceGroup and vaultName params:
        let vault_access_policy = new azure.keyvault.AccessPolicy(vault_access_policy_name, {
            vaultName: vault_name_parsed,
            resourceGroupName: vault_resource_group_name,

But that gives a deprecation warning on > 16.7 now, so I had refactored it to use keyVaultId single param:
        let vault_access_policy = new azure.keyvault.AccessPolicy(vault_access_policy_name, {
            keyVaultId: AzureUtils.get_keyvault_id(env_id, region, vault.name, config.get('resource_prefix')),

If you wanted to get really precise, it would be first creating it on 16.7 with resourceGroup and vaultName and then upgrading to 16.8 or 16.9 and then changing it to use the single keyVaultId

Hmm, I also can't reproduce it that way. Here are the steps I took, in case you can notice anything different, @dbilleci-lightstream:
Remove all the plugins, using pulumi plugin rm --all and confirming.
Start with the program in the pre-0.16.8 form, with a strict dependency on version 0.16.7 of @pulumi/azure in package.json:
import * as azure from "@pulumi/azure";

const resourceGroup = new azure.core.ResourceGroup("resourceGroup", {
    location: "WestUS",
});

const vault = new azure.keyvault.KeyVault("vault", {
    name: "repro183vault",

    resourceGroupName: resourceGroup.name,
    location: resourceGroup.location,

    tenantId: <string>azure.config.tenantId,

    sku: {
        name: "standard",
    },

    networkAcls: {
        defaultAction: "Deny",
        bypass: "AzureServices",
    },

    tags: {
        Purpose: "Repro azure#183"
    },
});

const vaultAccessPolicy = new azure.keyvault.AccessPolicy("policy", {
    vaultName: vault.name,
    resourceGroupName: resourceGroup.name,
    tenantId: <string>azure.config.tenantId,
    objectId: "redacted", // my azure user object ID
    keyPermissions: ['get', 'list'],
    certificatePermissions: ['get', 'list'],
    secretPermissions: ['get', 'list']
});

Run the program, and see the resources created:
 ▲ Code/pulumi-repro/azure-183 npm install                                                                                21s  ⇡ master :: 8m :: ⬡

><EMAIL_ADDRESS>install /Users/James/Code/pulumi-repro/azure-183/node_modules/@pulumi/azure
> node scripts/install-pulumi-plugin.js resource azure v0.16.7

[resource plugin azure-0.16.7] installing
npm WARN azure-typescript@ No description
npm WARN azure-typescript@ No repository field.
npm WARN azure-typescript@ No license field.

updated 1 package and audited 498 packages in 2.491s
found 0 vulnerabilities

 ▲ Code/pulumi-repro/azure-183 pulumi plugin ls                                                                                ⇡ master :: 9m :: ⬡
NAME   KIND      VERSION  SIZE   INSTALLED      LAST USED
azure  resource  0.16.7   88 MB  2 minutes ago  4 seconds ago

TOTAL plugin cache size: 88 MB

 ▲ Code/pulumi-repro/azure-183 pulumi up                                                                                ⇡ master :: 9m :: ⬡
Previewing update (dev):

     Type                            Name           Plan
 +   pulumi:pulumi:Stack             azure-183-dev  create
 +   ├─ azure:core:ResourceGroup     resourceGroup  create
 +   ├─ azure:keyvault:KeyVault      vault          create
 +   └─ azure:keyvault:AccessPolicy  policy         create

Resources:
    + 4 to create

Do you want to perform this update? details
+ pulumi:pulumi:Stack: (create)
    [urn=urn:pulumi:dev::azure-183::pulumi:pulumi:Stack::azure-183-dev]
    + azure:core/resourceGroup:ResourceGroup: (create)
        [urn=urn:pulumi:dev::azure-183::azure:core/resourceGroup:ResourceGroup::resourceGroup]
        location: "WestUS"
        name    : "resourcegroup00f05839"
    + azure:keyvault/keyVault:KeyVault: (create)
        [urn=urn:pulumi:dev::azure-183::azure:keyvault/keyVault:KeyVault::vault]
        location         : "WestUS"
        name             : "repro183vault"
        networkAcls      : {
            bypass       : "AzureServices"
            defaultAction: "Deny"
        }
        resourceGroupName: "resourcegroup00f05839"
        sku              : {
            name: "standard"
        }
        tags             : {
            Purpose: "Repro azure#183"
        }
        tenantId         : "25024bbc-53bb-4ccd-ab0c-26a4d95ccfd5"
    + azure:keyvault/accessPolicy:AccessPolicy: (create)
        [urn=urn:pulumi:dev::azure-183::azure:keyvault/accessPolicy:AccessPolicy::policy]
        certificatePermissions: [
            [0]: "get"
            [1]: "list"
        ]
        keyPermissions        : [
            [0]: "get"
            [1]: "list"
        ]
        objectId              : "79fdd019-87fd-4f1c-92bb-c931afa49370"
        resourceGroupName     : "resourcegroup00f05839"
        secretPermissions     : [
            [0]: "get"
            [1]: "list"
        ]
        tenantId              : "25024bbc-53bb-4ccd-ab0c-26a4d95ccfd5"
        vaultName             : "repro183vault"

Do you want to perform this update? yes
Updating (dev):

     Type                            Name           Status
 +   pulumi:pulumi:Stack             azure-183-dev  created
 +   ├─ azure:core:ResourceGroup     resourceGroup  created
 +   ├─ azure:keyvault:KeyVault      vault          created
 +   └─ azure:keyvault:AccessPolicy  policy         created

Resources:
    + 4 created

Duration: 2m15s

Permalink: https://app.pulumi.com/jen20/azure-183/dev/updates/13

Then, update package.json to refer to latest (which at the time of writing is 0.16.9), and npm install again, resulting in pulumi plugin install resource azure 0.16.9 being executed:
 ▲ Code/pulumi-repro/azure-183 npm install                                                                                ⇡ master :: 12m :: ⬡

><EMAIL_ADDRESS>install /Users/James/Code/pulumi-repro/azure-183/node_modules/@pulumi/azure
> node scripts/install-pulumi-plugin.js resource azure v0.16.9

[resource plugin azure-0.16.9] installing
Downloading plugin:  26.55 MiB / 26.55 MiB [========================] 100.00% 4s
npm WARN azure-typescript@ No description
npm WARN azure-typescript@ No repository field.
npm WARN azure-typescript@ No license field.

updated 1 package and audited 498 packages in 6.867s
found 0 vulnerabilities

Run pulumi update, and see the expected deprecation warning:
 ▲ Code/pulumi-repro/azure-183 pulumi up                                                                                8s  ⇡ master :: 12m :: ⬡
Previewing update (dev):

     Type                            Name           Plan     Info
     pulumi:pulumi:Stack             azure-183-dev
     └─ azure:keyvault:AccessPolicy  policy                  2 warnings

Diagnostics:
  azure:keyvault:AccessPolicy (policy):
    warning: urn:pulumi:dev::azure-183::azure:keyvault/accessPolicy:AccessPolicy::policy verification warning: "vault_name": [DEPRECATED] This property has been deprecated in favour of the key_vault_id property. This will prevent a class of bugs as described in https://github.com/terraform-providers/terraform-provider-azurerm/issues/2396 and will be removed in version 2.0 of the provider
    warning: urn:pulumi:dev::azure-183::azure:keyvault/accessPolicy:AccessPolicy::policy verification warning: "resource_group_name": [DEPRECATED] This property has been deprecated as the resource group is now pulled from the vault ID and will be removed in version 2.0 of the provider

Modify the program such that the vaultAccessPolicy resource is defined like this:
const vaultAccessPolicy = new azure.keyvault.AccessPolicy("policy", {
    keyVaultId: vault.id,
    tenantId: <string>azure.config.tenantId,
    objectId: "redacted", // my azure user object ID
    keyPermissions: ['get', 'list'],
    certificatePermissions: ['get', 'list'],
    secretPermissions: ['get', 'list']
});

Finally, run pulumi up again, and see the expected modifications:
 ▲ Code/pulumi-repro/azure-183 pulumi up                                                                                18s  ⇡ master :: 13m :: ⬡
Previewing update (dev):

     Type                            Name           Plan        Info
     pulumi:pulumi:Stack             azure-183-dev
 +-  └─ azure:keyvault:AccessPolicy  policy         replace     [diff: +keyVaultId-resourceGroupName,vaultName]

Resources:
    +-1 to replace
    3 unchanged

Do you want to perform this update? yes
Updating (dev):

     Type                            Name           Status       Info
     pulumi:pulumi:Stack             azure-183-dev
 +-  └─ azure:keyvault:AccessPolicy  policy         replaced     [diff: +keyVaultId-resourceGroupName,vaultName]

Resources:
    +-1 replaced
    3 unchanged

Duration: 6s

Permalink: https://app.pulumi.com/jen20/azure-183/dev/updates/15

I'll keep digging here, but can you confirm whether that workflow matches the way you encountered this problem, @dbilleci-lightstream?

After some more back-and-forth with @dbilleci-lightstream on the community slack, we came to the conclusion that the reported panic is the failure mode when the supplied keyVaultId is not valid. It can be reproduced with the following program, at version 0.16.9 of the Azure resource provider:
import * as azure from "@pulumi/azure";

const resourceGroup = new azure.core.ResourceGroup("resourceGroup", {
    location: "WestUS",
});

const vault = new azure.keyvault.KeyVault("vault", {
    name: "repro183vault",

    resourceGroupName: resourceGroup.name,
    location: resourceGroup.location,

    tenantId: <string>azure.config.tenantId,

    sku: {
        name: "standard",
    },

    networkAcls: {
        defaultAction: "Deny",
        bypass: "AzureServices",
    },

    tags: {
        Purpose: "Repro azure#183"
    },

});

const vaultAccessPolicy = new azure.keyvault.AccessPolicy("policy", {
    keyVaultId: vault.id.apply(x => `${x}nope`),
    tenantId: <string>azure.config.tenantId,
    objectId: "redacted", // my user object ID
    keyPermissions: ['get', 'list'],
    certificatePermissions: ['get', 'list'],
    secretPermissions: ['get', 'list']
});

On pulumi up:
 ▲ Code/pulumi-repro/azure-183 pulumi up                                                                                ⇡ master :: 1h :: ⬡
Previewing update (dev):

     Type                            Name           Plan        Info
     pulumi:pulumi:Stack             azure-183-dev
 +-  └─ azure:keyvault:AccessPolicy  policy         replace     [diff: ~keyVaultId]

Resources:
    +-1 to replace
    3 unchanged

Do you want to perform this update? yes
Updating (dev):

     Type                            Name           Status                   Info
     pulumi:pulumi:Stack             azure-183-dev                           	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi-terraform/pkg/tfbridge/provider.go:566 +0     Type                            Name           Status                   Info
     pulumi:pulumi:Stack             azure-183-dev                           22 messages
 +-  └─ azure:keyvault:AccessPolicy  policy         **replacing failed**     [diff: ~keyVaultId]; 1 error

Diagnostics:
  azure:keyvault:AccessPolicy (policy):
    error: Plan apply failed: transport is closing

  pulumi:pulumi:Stack (azure-183-dev):
    panic: fatal: An assertion has failed: expected non-nil error with nil state during Create
    goroutine 117 [running]:
    github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi/pkg/util/contract.failfast(0xc0007ce050, 0x4c)
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi/pkg/util/contract/failfast.go:23 +0xe9
    github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi/pkg/util/contract.Assertf(0xc000685300, 0x33b8791, 0x33, 0x0, 0x0, 0x0)
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi/pkg/util/contract/assert.go:33 +0x126
    github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi-terraform/pkg/tfbridge.(*Provider).Create(0xc000696e10, 0x3713fc0, 0xc000ae6720, 0xc000b141c0, 0xc000696e10, 0x1, 0x1)
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi-terraform/pkg/tfbridge/provider.go:566 +0xbda
    github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi/sdk/proto/go._ResourceProvider_Create_Handler.func1(0x3713fc0, 0xc000ae6720, 0x3225660, 0xc000b141c0, 0x3713fc0, 0xc000ae6720, 0x372d220, 0x50acde0)
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi/sdk/proto/go/provider.pb.go:1229 +0x86
    github.com/pulumi/pulumi-azure/vendor/github.com/grpc-ecosystem/grpc-opentracing/go/otgrpc.OpenTracingServerInterceptor.func1(0x3713fc0, 0xc000ae6720, 0x3225660, 0xc000b141c0, 0xc000bb0320, 0xc000bb0340, 0x0, 0x0, 0x36feea0, 0xc0002e0c00)
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/github.com/grpc-ecosystem/grpc-opentracing/go/otgrpc/server.go:57 +0x2b0
    github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi/sdk/proto/go._ResourceProvider_Create_Handler(0x32bc200, 0xc000696e10, 0x3713fc0, 0xc000ae63c0, 0xc000be0140, 0xc00033ffe0, 0x0, 0x0, 0xc0013081e0, 0x1d5)
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/github.com/pulumi/pulumi/sdk/proto/go/provider.pb.go:1231 +0x158
    github.com/pulumi/pulumi-azure/vendor/google.golang.org/grpc.(*Server).processUnaryRPC(0xc000557080, 0x372aa00, 0xc000408c00, 0xc000af5a00, 0xc0006a75f0, 0x5080ae0, 0x0, 0x0, 0x0)
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/google.golang.org/grpc/server.go:966 +0x4a2
    github.com/pulumi/pulumi-azure/vendor/google.golang.org/grpc.(*Server).handleStream(0xc000557080, 0x372aa00, 0xc000408c00, 0xc000af5a00, 0x0)
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/google.golang.org/grpc/server.go:1245 +0xd58
    github.com/pulumi/pulumi-azure/vendor/google.golang.org/grpc.(*Server).serveStreams.func1.1(0xc00003e120, 0xc000557080, 0x372aa00, 0xc000408c00, 0xc000af5a00)
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/google.golang.org/grpc/server.go:685 +0x9f
    created by github.com/pulumi/pulumi-azure/vendor/google.golang.org/grpc.(*Server).serveStreams.func1
    	/home/travis/gopath/src/github.com/pulumi/pulumi-azure/vendor/google.golang.org/grpc/server.go:683 +0xa1

Returning a nil state and a nil error sound suspicious upstream - I'll have a dig around there and see what I can find there next.

Additionally, we confirmed that @dbilleci-lightstream can make progress by correcting the helper function being used to calculate the vault ID from components.

@jen20 thanks again for spending the time to debug this!  I'll continue to follow the ticket for info on the (upstream?) issue.

So, this is a bug upstream. I can reproduce what would cause us to fail in this manner by running this Terraform configuration:
resource "azurerm_key_vault_access_policy" "policy" {
  # note this has to at least appear to be a valid URI to a vault
  key_vault_id = "/subscriptions/<redacted>/resourceGroups/resourcegroupcbbd350e/providers/Microsoft.KeyVault/vaults/repro183vaultnope"

  # these don't matter
  tenant_id = "00000000-0000-0000-0000-000000000000"
  object_id = "11111111-1111-1111-1111-111111111111"

  key_permissions = [
    "get",
  ]

  secret_permissions = [
    "get",
  ]
}

Running terraform apply, I see this:
$ terraform apply                                                                                ⇡ master :: 1h :: ⬡

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  + azurerm_key_vault_access_policy.policy
      id:                   <computed>
      key_permissions.#:    "1"
      key_permissions.0:    "get"
      key_vault_id:         "/subscriptions/<redacted>/resourceGroups/resourcegroupcbbd350e/providers/Microsoft.KeyVault/vaults/repro183vaultnope"
      object_id:            "11111111-1111-1111-1111-111111111111"
      resource_group_name:  <computed>
      secret_permissions.#: "1"
      secret_permissions.0: "get"
      tenant_id:            "00000000-0000-0000-0000-000000000000"
      vault_name:           <computed>

Plan: 1 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

azurerm_key_vault_access_policy.policy: Creating...
  key_permissions.#:    "" => "1"
  key_permissions.0:    "" => "get"
  key_vault_id:         "" => "/subscriptions/<redacted>/resourceGroups/resourcegroupcbbd350e/providers/Microsoft.KeyVault/vaults/repro183vaultnope"
  object_id:            "" => "11111111-1111-1111-1111-111111111111"
  resource_group_name:  "" => "<computed>"
  secret_permissions.#: "" => "1"
  secret_permissions.0: "" => "get"
  tenant_id:            "" => "00000000-0000-0000-0000-000000000000"
  vault_name:           "" => "<computed>"
azurerm_key_vault_access_policy.policy: Creation complete after 0s

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

However, no state is actually created:
{
    "version": 3,
    "terraform_version": "0.11.11",
    "serial": 1,
    "lineage": "74a73fbb-d75a-af7a-c1dc-6d73cc1e9a73",
    "modules": [
        {
            "path": [
                "root"
            ],
            "outputs": {},
            "resources": {},
            "depends_on": []
        }
    ]
}

Our Terraform bridge asserts that this cannot happen, resulting in the panic we see as the presenting cause of this issue. The reason it fails is here (upstream):
https://github.com/terraform-providers/terraform-provider-azurerm/blob/master/azurerm/resource_arm_key_vault_access_policy.go#L146-L152
Likely this was to automatically remove the policy of a removed vault, but results in returning a nil state and a nil error.

Actually, I think this is a straight up bug upstream, and that this error should instead be returned if the key vault is not found: https://github.com/terraform-providers/terraform-provider-azurerm/blob/master/azurerm/resource_arm_key_vault_access_policy.go#L154

This has been fixed upstream now, in terraform-providers/terraform-provider-azurerm#2922 (in the exact form for this issue), and also in other resources susceptible to the same kind of failure (in terraform-providers/terraform-provider-azurerm#2923). It looks like they'll be in the 1.23.0 release upstream, so will get pulled in during the update when that is released.
Thanks for your patience in providing information when debugging this, @dbilleci-lightstream! I'll go ahead and close the issue as the fix will automatically be pulled in during the next update.


fix: jsonld document loader node 18

Fixes the JSON-LD document loader in node 18. For now the node document loader is copied, but if https://github.com/digitalcredentials/jsonld.js/pull/2 is merged and released we can remove the custom implementation in AFJ.

Codecov Report

Merging #1454 (d14fd4b) into main (11366e5) will decrease coverage by 23.36%.
The diff coverage is 83.33%.

@@             Coverage Diff             @@
##             main    #1454       +/-   ##
===========================================
- Coverage   85.87%   62.51%   -23.36%
===========================================
  Files         911      745      -166
  Lines       21742    17197     -4545
  Branches     3727     2872      -855
===========================================
- Hits        18670    10750     -7920
- Misses       2893     5941     +3048
- Partials      179      506      +327

Impacted Files
Coverage Δ

...e/src/modules/vc/libraries/nativeDocumentLoader.ts
33.33% <0.00%> (ø)

...ckages/core/src/modules/vc/W3cCredentialService.ts
46.04% <100.00%> (-33.37%)
:arrow_down:

... and 451 files with indirect coverage changes

Discussed on AFJ WG Call @TimoGlastra to look into this PR

As https://github.com/digitalcredentials/jsonld.js/pull/2 has been merged and I think incorporated into the new release of @digitalcredentials/jsold, now we can remove the custom implementation, right?

Yes!

PR revived. Let's hope it we can get it merged now


Update dropdown for one and many tokens

[ ] When there are many tokens, add a dropdown-caret so users can identify the dropdown

[ ] When there is only one token selected to trade on the widget, there is no need to open the select modal. Show only the token image and symbol and hide the dropdown-caret.

@sebazelonka can you add the caret?


[FLINK-7903] [tests] Add flip6 build profile

What is the purpose of the change
The flip6 build profile only runs the Flip-6 related test cases. Moreover,
all Flip-6 related test cases are excluded when not running the flip6 build
profile. This should reduce testing time when adding more and more Flip-6
test cases.
Brief change log

Introduce org.apache.flink.testutils.category.Flip6 and org.apache.flink.testutils.category.OldAndFlip6 category marker interfaces
Annotate Flip-6 test cases with Flip6 marker interface
Add flip6 build profile which only runs Flip6 and OldAndFlip6 test cases

Verifying this change

Manually tested

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for the review @zentol. Rebasing and once Travis gives green light, I'll merge it.


Doc Suggestion: Simplify Upgrade

Hi, just a suggestion for the upgrade instructions here:
https://yourls.org/#Install
As far as i can tell, Steps 4 and 5 are unnecessary.
Step 4) Overwriting the exiting install doesn't touch the config.php (in my FTP client, at least, WinSCP), so it doesn't need to be restored. Also, you don't give a step to back it up, so if restoring the config is required, then you should add a step to back it up before upgrading.
Step 5) This isn't part of the upgrade, it is just a "make sure it worked" step, and could be labelled as such, or just eliminated.
thx!

Indeed, these instruction refer to ancient versions when user config was initially found in includes/. I'll simplify.

To upgrade I just connected the git and type
git fetch --all --tags
git checkout tags/<version>

And I'm up to date :)

$ git describe --tags
1.7.6-18-gba459cb5
$ git describe --abbrev=0 --tags
1.7.6

So, I think it's more git checkout $(git describe --abbrev=0 --tags) to checkout the latest tag.


Implement Managed Streaming for Kafka (MSK) resources

What problem are you facing?
AWS Provider does not have support for MSK resources.
How could Crossplane help solve your problem?
Implement MSK resources.

Hi, I would like to contribute to this feature as I need it for a project.

Hi @supervacuum ! That's great! I think the best starting point is to go through the guide here. I'd suggest starting with only one resource and drive that to completion, and then move on to the other ones if you'd like to.

@supervacuum @marcinc
i added a MR #733  as WIP a few points open - but in general deployment and removal of MSK is successfully ;)


[1.x] Laravel Preset - include continue in blank_line_before_statement

This PR addresses an implied syntax rule of framework for the continue keyword.
When continue does not follow a control structure, a blank line is used to provide some breathing room.
Illuminate\Auth\Access\Gate
if (! $this->canBeCalledWithUser($user, $before)) {
    continue;
}

Illuminate\Collections\Arr
if (static::exists($array, $key)) {
    unset($array[$key]);

    continue;
}

For reference (see Example 3): PHPCS blank_line_before_statement

I was about to create a PR for this exact rule. However, couldn't we just use the default by PHP CS Fixer?
'blank_line_before_statement' => true,


Joysound formattedRawLyrics throws "TypeError: Cannot read properties of undefined (reading '0')"

Closed by #86


TypeError: _unmask_unattended_patched() missing 1 required positional argument: 'unmasked_value' during ONNX export LLaMa

System Info
version: latest main or release
system: any

Who can help?
No response
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction (minimal, reproducible, runnable)
Take any LLaMa model,
from optimum.exporters.onnx import main_export
main_export(args.model_dir,task="text-generation-with-past",output=args.output_dir,opset=17,dtype=args.dtype,device="cuda",framework="pt")
Expected behavior
ONNX exported, but the above error arises, due to HF removed the argument, but optimum did not.

I am also seeing this.

same here when running this code :
from optimum.onnxruntime import ORTModelForCausalLM
ORTModelForCausalLM.from_pretrained("./llama3-q", export=True, provider="CUDAExecutionProvider",)


[Feature] Remove parameter override

Prerequisites
Please answer the following question for yourself before submitting an issue.

[x] I checked to make sure that this feature has not been requested already.

1. The entire URL of the file you are using
https://github.com/tensorflow/models/tree/master/official/vision/beta
2. Describe the feature you request
Remove parameter overrides.
3. Additional context
Configuring the training from both yaml config file and parameter overrides is confusing.
4. Are you willing to contribute it? (Yes or No)
Yes

Closed by #9.


v10.2.0

Commit for the tag:

[x] Edit version in setup.py
[x] Add version in Changelog.md ; trim things
[x] Push & wait for the tests to be green
[x] tag me.
[x] build sdist + wheel packages (make package)

Back to dev commit:

[x] Edit version in setup.py
[x] Add the "master / nothing to see here" in Changelog.md
[x] Push & wait for the tests to be green

[x] Merge --ff
Github stuff

[x] Push tag in Github
[x] Edit release on Github using the changelog.
[x] Delete branch

[x] upload release on PyPI using twine
[x] (optional) Make feeback on the various PR or issues.

it's marked as closed, but it's my fault, I merged locally and deleted the branch before pushing.


Add PCL/netstandard documentation

Title
On the title describe
what you've fixed (or created) with this Pull Request (PR).
Summary
Insert a short (one or two sentence) summary here.
Fixes #Issue_Number

Note: The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.
Remove that line if you don't have issues associated with this
PR. Click on the Guidelines for Contributing link above for details.

Details
Explain your changes, and why you made them. If that
information is already available in the issue referenced
above, just referencing the issue is preferred to copying
the text.
This may not be necessary depending on the scope of the PR
changes. For example, "fix typo in introduction.md" is
sufficient to describe that PR.
Suggested Reviewers
If you know who should review this, use '@' to request a review.

Hi @rchande, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

:+1:

LGTM.  Thanks, @rchande!


Add Taint Tracking Support via Data Flow API

Takes the export of CodeQL's data flow models and leverages them in OpenRewrite
for taint tracking support.

This PR is failing because of build cache issues 😳

And also for missing License headers

Requesting that CallMatcher be moved to rewrite.java.dataflow. rewrite-java does not have a concept of call, and its API is designed for data flow.

It's not exclusively for DataFlow, it's also for general matching. I'd like to move it to the same package as MethodMatcher eventually


increased transcriptional readthrough

can
increased transcriptional readthrough
be related in some way to
abnormal transcription termination?

I can put in an "output" link.


Make increment unconditional.

Or "Always increment"... lol

This was probably a bug since iPass would have only incremented if shouldlog and (iPass >= PAYMENT_MAX_LOOPS) would have never evaluated to true below.

Included in #936.


[Feature request]: Request to Publish Ledger Live on Flatpak

Hello Ledger Live Team,
I'm a Linux user and I would like to request that Ledger Live be published on Flatpak. Flatpak is a software distribution platform that allows users to easily install and manage applications on various Linux distributions.
By publishing Ledger Live on Flatpak, you can make it easier for Linux users to manage their cryptocurrency assets on Ledger hardware wallets. Flatpak also provides a more secure and controlled environment for running applications, which can help improve the overall security of Ledger Live.
I understand that publishing on Flatpak may require additional resources from your team, but I believe it would be beneficial for both Ledger and the Linux community. I would be happy to assist in any way possible to help make this happen.
Thank you for your consideration, and I look forward to hearing from you soon.
Best regards,
Aman Singh
(Copied from https://github.com/LedgerHQ/ledger-live/issues/3317, which was closed by bot)

+1

Still important.

This initially can be easily handled by us, what has to be done is just repacking the appimage, good example is this repo https://github.com/flathub/com.lunarclient.LunarClient/blob/master/com.lunarclient.LunarClient.yml
Would be nice to get blessing from Ledger in form of "Download On FLATHUB" button on Ledger page.
Because releases get tagged on GitHub we could also make automated changelogs and updates

Still important.

Bump

Still waiting for this

Ping on maintainers: @cgrellard-ledger @abdurrahman-ledger @sprohaszka-ledger @jdabbech-ledger

Bump

Bump

Please remove the stale tag und don't close the issue.

If you prefer, you can also close the issue and I'll keep creating new ones - I just think that would be a bit stupid.

Bump
Ping on maintainers: @cgrellard-ledger @abdurrahman-ledger @sprohaszka-ledger @jdabbech-ledger

Bump
Ping on maintainers: @cgrellard-ledger @abdurrahman-ledger @sprohaszka-ledger @jdabbech-ledger

Anti stale

Anti-Stale

Bump

Still very very important. Please can this now be prioritized?

Which results in Ledger Live giving an illusion of a OSS project when practically it's just source-available, lmao.
They instead prefer to focus on adding<PHONE_NUMBER>99 useless shitcoins made by companies that probably pay them, same thing with Discover tab.
I have switched to a different HW wallet few weeks ago and I'm using my Ledger only as a backup U2F key now, recommend doing the same :)


Dev graphwork

Functionality:
Pie Chart migrated and linear progress graph added
Solution:
Created a component for data binding and reusability.
Risk level:

[ ] high
[x] medium
[ ] low

How to test:
firebase emulators:start
ng-serve

Please work on dev-angular branch


Added

Description

Pre-Submission Checklist
Common mandatory requirements

[ ] This pull request targets the master branch of freeCodeCamp.
[ ] Branch starts with either fix/, feature/, or translate/
(e.g. fix/signin-issue)
[ ] Reviewed necessary guidelines in CONTRIBUTING.md.

By submitting this pull request, I have verified the following:

[ ] Added descriptive name to pull request. (mandatory)
Your PR should NOT be called Update index.md, since it does not help the maintainer understand what has been changed.
Good Guide PR title examples:
fix(guide): Git - edit Git Commit article
feat(guide): PHP - create PHP section and add Data Structures article

Good Coding challenge PR title examples:
fix(coding): add tests to Basic HTML challenge
fix(coding): update instructions to CSS Grid challenge

Good Learning platform PR title examples:
fix(platform): fixed sign in api
fix(platform): removed css from editor

Issues Closures

[ ] Addressed currently open issue. (replace XXXXX with an issue no in next line)

Closes #XXXXX
Since this change is a copy edit or translation for Guide articles or Coding Challenges:

[ ] I confirm no plagiarized, duplicate, or repetitive content that has been directly copied from another source.
[ ] I am adding only https links to external sources.
[ ] I am not adding short links instead of full URLs to webpages.
[ ] I have followed the recomended style guides for the copy edit.

Requirements, for changes which are NOT simple copy editing or translations:
You may skip the below ONLY if your changes do not touch tests, ui logic, api-server, etc. Your PR may not be accepted, if you did not meet these mandatory requirements. Simply put, we can't assume it will work if we never saw it working.
Since this change adds, updates or removes coding challenge tests:

[ ] I completed the coding challenge locally, and the changes look good.
[ ] Other tests or user stories in instructions are not affected.

Since this change is for the Learning platform (UI, logic, etc.):

[ ] I confirm my changes look good by running freeCodeCamp locally.
[ ] All new and existing tests pass the command npm test.

(Optional) Additional Context and Screenshots:

Hi @sakshambhalla
The content seems to have been copied and pasted from wikipedia without any substantial changes to the content.
Please read the section in the Contributing Guide about copying text from other websites.
I am closing this pull request for now. Please let me know if you have any questions.


Initialize west.yml in root of the workspace instead of application/west.yml

Currently the west.yml is being created in application/west.yml this contradicts the zepyhr example application setup and also leads to the "workspace initialization from repo" bug mentioned in https://github.com/mylonics/zephyr-ide/issues/63#issuecomment-2381602140
solution: workspace initialization should create the west.yml file directly in the root of the workspace
unfortunatelly doing this manually in the repo also leads to a new issue, where west init is being executed into RepoName.git instead of the repository root :(

The IDE is designed for a multiproject workspace that shares the same west.yml. Unfortunately west does not support that well. In whatever directory the west.yml file, the parent directory of that file will contain the .west directory.
The initialize from git repository will initialize a project like this https://github.com/zephyrproject-rtos/example-application. If you want to use a workspace maintained by the zephyr ide you should do a git clone and get the repo on your computer and then open that directory with vscode.

alright thank you :) in that case i'll close the issue


UIPA Project area validator

Benefit
Dain would like/needs a tool to validate his project areas are complete. Working on a python toobox for pro that will validate based on list of parcels, source parcel layers, project area layer, municipality layer, and tax entity layers.
Acceptance Criteria

[ ]

Notes
No response
Risks
Poorly structured input data
Issue Reference
refs #

25.2.6
Haven't worked much on this, and haven't heard from Dain. Guessing that the need is not as important as originally thought. Closing for now, will reopen if we get more info/prioritization.


[client,sdl] correctly handle AuthenticateEx

handle callback with already provided credentials that did not fail (aka RDP or TLS security)

copied handling from client_cli_authenticate_ex


[SMALLFIX] clean up unused constants

Jenkins, test this please.

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

LGTM


Visualizer Defect

The Visualizer doesn't seem to work on nexus 5x when there was song played on background, the same problem was seen on samsung devices. Also i saw E/Audio record -38 on logcat.

You should read Readme file. There you will the Troubleshooting section that leads to this answer. It's possible that Visualizer will not work on certain devices. Especially if you're trying to visualize mixed output by passing 0 as audio session ID. In most cases to fix this issue you need to specify audio session ID and avoid using 0.

Reopen issue if you will have any other questions related to it.


The Command is showing a success but no file are decrypted

This issue is with the latest version, The Command is showing a success but no files are decrypted and even the output folder is still empty.

I have the same problem. Can anyone help.

I got the same thing and I think it's because this does not decrypt .LDCW files just .LYNDA ones.


[FEATURE] Apply tilting effect to all images in the website

Description
The below tilting effect has been done using Vanilla JS Tilt but these tilt effect aren't there in some of the newly created pages.
So I can add it in those pages too.
Screenshots

The tiliting effect is visible on geeksforgeeks icon

@jheel05 Hey I just created this as an issue due to some mismatch I found


feat: custom token scope

Here's a very light modification to allow a custom scope when requesting the token.
Still testing locally.

Sorry, made a mess with branches, will open again once I fix 😅


Git providers other than GitHub

Carrying this over from https://github.com/kubernetes/test-infra/issues/10146 .
@BenTheElder You mentioned on the other thread that there's been no bandwidth for this. I haven't looked at prow in quite awhile, but I'd love to jump back in if there's an appetite for it.

Prow is probably at the lowest point ever in terms of maintainer bandwidth from what I can tell, but it's still essential to the project, making sweeping changes difficult to enable.
I'm not working on it directly currently, just the SIG overall and getting prow.k8s.io fully running under SIG K8s Infra (done for the jobs now but not the services, that's next week)


Add UNDO button

Hi,
In case user accidentally assign plot incorrectly/ or want to change their interpretation after saving them
can the user or project manager could track back to the incorrect plots in the database, or would be possible to have an Undo button on the interface of the tool?
Thanks,

Changes in PR #187 allow the user to go back through plots that the same user has analyzed.  The user can then change any of the answers and save again.


autocomplete.Select2GenericForeignKeyModelField not working with proxy models

I have a proxy model in model_choice
in prepare_value we got ContentType.objects.get_for_model(...) without for_concrete_model = False
Which brings in initial data wrong content type, and when I try to save form I got invalid_choice error
Related class: ContentTypeModelFieldMixin

I believe that means the view does not return an id that the widget
understand, can you tell us more about these?
Le jeu. 21 janv. 2021 à 08:31, Maxim Fominykh<EMAIL_ADDRESS>a
écrit :

I have a proxy model in model_choice
in prepare_value we got ContentType.objects.get_for_model(...) without
for_concrete_model = False
Which brings in initial data wrong content type, and when I try to save
form I got invalid_choice error
Related class: ContentTypeModelFieldMixin
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yourlabs/django-autocomplete-light/issues/1210, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAXDLHS5DP5JIFOVLEX2ILS27J6VANCNFSM4WMMT4QQ
.

I believe that means the view does not return an id that the widget
understand, can you tell us more about these?
Le jeu. 21 janv. 2021 à 08:31, Maxim Fominykh<EMAIL_ADDRESS>a
écrit :

I have a proxy model in model_choice
in prepare_value we got ContentType.objects.get_for_model(...) without
for_concrete_model = False
Which brings in initial data wrong content type, and when I try to save
form I got invalid_choice error
Related class: ContentTypeModelFieldMixin
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yourlabs/django-autocomplete-light/issues/1210, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAXDLHS5DP5JIFOVLEX2ILS27J6VANCNFSM4WMMT4QQ
.

I believe that means the view does not return an id that the widget understand, can you tell us more about these? Le jeu. 21 janv. 2021 à 08:31, Maxim Fominykh<EMAIL_ADDRESS>a écrit :
…
I have a proxy model in model_choice in prepare_value we got ContentType.objects.get_for_model(...) without for_concrete_model = False Which brings in initial data wrong content type, and when I try to save form I got invalid_choice error Related class: ContentTypeModelFieldMixin — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1210>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAXDLHS5DP5JIFOVLEX2ILS27J6VANCNFSM4WMMT4QQ .

Will provide an example, if I can reproduce it easily myself

I believe that means the view does not return an id that the widget understand, can you tell us more about these? Le jeu. 21 janv. 2021 à 08:31, Maxim Fominykh<EMAIL_ADDRESS>a écrit :
…
I have a proxy model in model_choice in prepare_value we got ContentType.objects.get_for_model(...) without for_concrete_model = False Which brings in initial data wrong content type, and when I try to save form I got invalid_choice error Related class: ContentTypeModelFieldMixin — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1210>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAXDLHS5DP5JIFOVLEX2ILS27J6VANCNFSM4WMMT4QQ .

Will provide an example, if I can reproduce it easily myself


AngularJs Version is Synced with AngularJs Version in bowerJson file

AngularJs Version is Synced with AngularJs Version in bowerJson file.
@Rick-Anderson Please Approve this change.

@aftab10662,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@@fiyazbinhasan please review.

@AftabAnsari10662  @fiyazbinhasan  thanks!


Seemly non-recursive schema is hitting a recursive schema bug

It's my first time using the library, so I might be mis-understanding #96, however I think I've found a schema which doesn't have a recursive ref loop. But I'm seeing an error:
Type of property 'detail' circularly references itself in mapped type '{ account: { type: "primitive"; value: string; isSerialized: false; deserialized: never; }; detail: Never; "detail-type": { type: "primitive"; value: string; isSerialized: false; deserialized: never; }; ... 5 more ...; version: { ...; }; }'.

Here is my example code, .detail references a definition, however it doesn't recurse forever, should this be expected to work?
import { FromSchema } from "json-schema-to-ts";

export const s3CreateEventSchema = {
  $schema: "http://json-schema.org/draft-04/schema#",
  title: "ObjectCreated",
  definitions: {
    Bucket: {
      properties: {
        name: {
          type: "string",
        },
      },
      required: ["name"],
      type: "object",
    },
    Object: {
      properties: {
        etag: {
          type: "string",
        },
        key: {
          type: "string",
        },
        sequencer: {
          type: "string",
        },
        size: {
          type: "number",
        },
        "version-id": {
          type: "string",
        },
      },
      required: ["etag", "key", "sequencer", "size"],
      type: "object",
    },
    ObjectCreated: {
      properties: {
        bucket: {
          $ref: "#/definitions/Bucket",
        },
        object: {
          $ref: "#/definitions/Object",
        },
        reason: {
          type: "string",
        },
        "request-id": {
          type: "string",
        },
        requester: {
          type: "string",
        },
        "source-ip-address": {
          type: "string",
        },
        version: {
          type: "string",
        },
      },
      required: [
        "bucket",
        "object",
        "reason",
        "requester",
        "request-id",
        "source-ip-address",
        "version",
      ],
      type: "object",
    },
  },
  properties: {
    account: {
      type: "string",
    },
    detail: {
      $ref: "#/definitions/ObjectCreated",
    },
    "detail-type": {
      type: "string",
    },
    id: {
      type: "string",
    },
    region: {
      type: "string",
    },
    resources: {
      items: {
        type: "string",
      },
      type: "array",
    },
    source: {
      type: "string",
    },
    time: {
      format: "date-time",
      type: "string",
    },
    version: {
      type: "string",
    },
  },
  required: [
    "account",
    "detail",
    "detail-type",
    "id",
    "region",
    "resources",
    "source",
    "time",
    "version",
  ],
  type: "object",
  "x-amazon-events-detail-type": "Object Created",
  "x-amazon-events-source": "aws.s3",
} as const;

export type S3CreateEvent = FromSchema<typeof s3CreateEventSchema>;

I just tried a simpler case just to check my assumptions, I found that if I do any kind of definition I run into this problem:
const anotherTest = {
  $schema: "http://json-schema.org/draft-04/schema",
  title: "Test Schema",
  type: "object",
  definitions: {
    Larger: {
      type: "string",
    },
  },
  properties: {
    testing: {
      type: "boolean",
    },
    again: {
      $ref: "#/definitions/Larger",
    },
  },
  required: ["testing", "again"],
} as const;

type AnotherTest = FromSchema<typeof anotherTest>;
// => never

Interestingly if I drop "again" from the required arguments I still see an error but TS kind of partially infers the type as rather than just never
type AnotherTest = {
    [x: string]: unknown;
    again?: any;
    testing: any;
}

Does this mean #96 actually means any reference is unsupported?


The parallel dimensions are not collapsed in unpack + elementwsie fusion

The below snippet is dumped from unet model. There are four parallel loops in the element-wise op, and they can be collapsed into two loops. The reason to keep two loops is that it can be aligned with unpack ops. Originally, I think it is a missing pattern at flow level. However, it is also codegen's responsibility to collapse the parallel loops. Because the unpack op could be unset_encoding and get materialized in codegen. Flow does not know how to collapse dims when there is an unset_encoding op. So the CPU backend needs to handle it.
The form is very bad for codegen because the outer two dimensions are tied to tensor.unpack ops, so they must be aligned with inner tile sizes. It introduces bad tiling config (e.g., [2, 64, 1, 16]) on generic op, which leads to significant amount of IRs after vector lowering. E.g., there are 19K IRs at llvm dialect level in this case.
If we can collapse the the inner two dimensions, codegen can be much easier. The [2, 64] tile sizes would be chosen in this example.
hal.executable public @run_forward$async_dispatch_33 {
  hal.executable.variant public @embedded_elf_x86_64 target(<"llvm-cpu", "embedded-elf-x86_64", {cpu = "znver4", cpu_features = "+mmx,+popcnt,+sse,+sse2,+sse3,+ssse3,+sse4.1,+sse4.2,+avx,+avx2,+sse4a,+fma,+avx512f,+bmi,+bmi2,+aes,+pclmul,+avx512vl,+avx512bw,+avx512dq,+avx512cd,+avx512vbmi,+avx512ifma,+avx512vpopcntdq,+avx512vbmi2,+gfni,+vpclmulqdq,+avx512vnni,+avx512bitalg,+avx512bf16,+adx,+clflushopt,+clwb,+clzero,+cx16,+cx8,+f16c,+fsgsbase,+crc32,+invpcid,+rdpru,+sahf,+lzcnt,+movbe,+mwaitx,+x87,+pku,+evex512,+prfchw,+rdpid,+rdrnd,+rdseed,+sha,+shstk,+vaes,+wbnoinvd,+xsave,+xsavec,+xsaveopt,+xsaves,+fxsr", data_layout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-i128:128-f80:128-n8:16:32:64-S128", native_vector_size = 64 : i64, target_triple = "x86_64-unknown-unknown-eabi-elf"}>) {
    hal.executable.export public @run_forward$async_dispatch_33_unpack_generic_2x320x128x128_f32xf32xf32xf32xf16 ordinal(0) layout(#hal.pipeline.layout<push_constants = 7, sets = [<0, bindings = [<0, storage_buffer, ReadOnly>, <1, storage_buffer, ReadOnly>, <2, storage_buffer>]>]>) attributes {hal.interface.bindings = [#hal.interface.binding<0, 0>, #hal.interface.binding<0, 1>, #hal.interface.binding<0, 2>]} {
    ^bb0(%arg0: !hal.device):
      %x, %y, %z = flow.dispatch.workgroup_count_from_slice
      hal.return %x, %y, %z : index, index, index
    }
    builtin.module {
      func.func @run_forward$async_dispatch_33_unpack_generic_2x320x128x128_f32xf32xf32xf32xf16() {
        %c32_i64 = arith.constant 32 : i64
        %0 = hal.interface.constant.load[0] : i32
        %1 = hal.interface.constant.load[1] : i32
        %2 = hal.interface.constant.load[2] : i32
        %3 = hal.interface.constant.load[3] : i32
        %4 = hal.interface.constant.load[4] : i32
        %5 = hal.interface.constant.load[5] : i32
        %6 = hal.interface.constant.load[6] : i32
        %7 = arith.index_castui %0 : i32 to index
        %8 = arith.index_castui %1 : i32 to index
        %9 = arith.extui %2 : i32 to i64
        %10 = arith.extui %3 : i32 to i64
        %11 = arith.shli %10, %c32_i64 : i64
        %12 = arith.ori %9, %11 : i64
        %13 = arith.index_castui %12 {stream.alignment = 64 : index, stream.values = [14604672 : index, 15427712 : index,<PHONE_NUMBER> : index,<PHONE_NUMBER> : index,<PHONE_NUMBER> : index]} : i64 to index
        %14 = arith.extui %4 : i32 to i64
        %15 = arith.extui %5 : i32 to i64
        %16 = arith.shli %15, %c32_i64 : i64
        %17 = arith.ori %14, %16 : i64
        %18 = arith.index_castui %17 {stream.alignment = 64 : index, stream.values = [14605952 : index, 15428992 : index,<PHONE_NUMBER> : index,<PHONE_NUMBER> : index,<PHONE_NUMBER> : index]} : i64 to index
        %19 = arith.index_castui %6 : i32 to index
        %20 = hal.interface.binding.subspan set(0) binding(0) type(storage_buffer) alignment(64) offset(%7) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<1x5x2x64xf32>>
        %21 = hal.interface.binding.subspan set(0) binding(0) type(storage_buffer) alignment(64) offset(%8) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<2x320x128x128xf32>>
        %22 = hal.interface.binding.subspan set(0) binding(1) type(storage_buffer) alignment(64) offset(%13) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<320xf32>>
        %23 = hal.interface.binding.subspan set(0) binding(1) type(storage_buffer) alignment(64) offset(%18) flags(ReadOnly) : !flow.dispatch.tensor<readonly:tensor<320xf32>>
        %24 = hal.interface.binding.subspan set(0) binding(2) type(storage_buffer) alignment(64) offset(%19) : !flow.dispatch.tensor<writeonly:tensor<2x320x128x128xf16>>
        %25 = flow.dispatch.tensor.load %20, offsets = [0, 0, 0, 0], sizes = [1, 5, 2, 64], strides = [1, 1, 1, 1] : !flow.dispatch.tensor<readonly:tensor<1x5x2x64xf32>> -> tensor<1x5x2x64xf32>
        %26 = flow.dispatch.tensor.load %21, offsets = [0, 0, 0, 0], sizes = [2, 320, 128, 128], strides = [1, 1, 1, 1] : !flow.dispatch.tensor<readonly:tensor<2x320x128x128xf32>> -> tensor<2x320x128x128xf32>
        %27 = flow.dispatch.tensor.load %22, offsets = [0], sizes = [320], strides = [1] : !flow.dispatch.tensor<readonly:tensor<320xf32>> -> tensor<320xf32>
        %28 = flow.dispatch.tensor.load %23, offsets = [0], sizes = [320], strides = [1] : !flow.dispatch.tensor<readonly:tensor<320xf32>> -> tensor<320xf32>
        %29 = tensor.empty() : tensor<2x320x128x128xf16>
        %30 = tensor.empty() : tensor<2x320xf32>
        %unpack = tensor.unpack %25 outer_dims_perm = [0, 1] inner_dims_pos = [0, 1] inner_tiles = [2, 64] into %30 : tensor<1x5x2x64xf32> -> tensor<2x320xf32>
        %31 = linalg.generic {indexing_maps = [affine_map<(d0, d1, d2, d3) -> (d0, d1, d2, d3)>,
                                               affine_map<(d0, d1, d2, d3) -> (d1)>,
                                               affine_map<(d0, d1, d2, d3) -> (d0, d1)>,
                                               affine_map<(d0, d1, d2, d3) -> (d1)>,
                                               affine_map<(d0, d1, d2, d3) -> (d0, d1, d2, d3)>],
                              iterator_types = ["parallel", "parallel", "parallel", "parallel"]}
                             ins(%26, %27, %unpack, %28 : tensor<2x320x128x128xf32>, tensor<320xf32>, tensor<2x320xf32>, tensor<320xf32>)
                             outs(%29 : tensor<2x320x128x128xf16>) {
        ^bb0(%in: f32, %in_0: f32, %in_1: f32, %in_2: f32, %out: f16):
          %32 = arith.addf %in_1, %in_2 : f32
          %33 = arith.addf %in, %in_0 : f32
          %34 = arith.truncf %32 : f32 to f16
          %35 = arith.truncf %33 : f32 to f16
          %36 = arith.addf %35, %34 : f16
          linalg.yield %36 : f16
        } -> tensor<2x320x128x128xf16>
        flow.dispatch.tensor.store %31, %24, offsets = [0, 0, 0, 0], sizes = [2, 320, 128, 128], strides = [1, 1, 1, 1] : tensor<2x320x128x128xf16> -> !flow.dispatch.tensor<writeonly:tensor<2x320x128x128xf16>>
        return
      }
    }
  }
}

I also found that the inner reduction dims are not collapsed into a single reduction loop in the previous sprint. We can also implement it at codegen level. What we want is having a pass that collapses innermost loops as much as possible. @MaheshRavishankar what do you think?

Actually this can happen at flow level. I was wrong about tensor types. The result type of unset_encoding does not have encodings. So it should work for below both cases.
%0 = tensor.unpack .... -> tensor<2x320xf32>
%1 = linalg.generic ... ins(%0...

%0 = iree_linalg_ext.unset_encoding .... -> tensor<2x320xf32>
%1 = linalg.generic ... ins(%0...

@IanWood1 is it something you can pick up? We can improve CollapseDimensions to support the case.

Yes, I can pick this up

I looked at this more. I am not sure you can collapse the iterations here
 %31 = linalg.generic {indexing_maps = [affine_map<(d0, d1, d2, d3) -> (d0, d1, d2, d3)>,
                                               affine_map<(d0, d1, d2, d3) -> (d1)>,
                                               affine_map<(d0, d1, d2, d3) -> (d0, d1)>,
                                               affine_map<(d0, d1, d2, d3) -> (d1)>,
                                               affine_map<(d0, d1, d2, d3) -> (d0, d1, d2, d3)>],
                              iterator_types = ["parallel", "parallel", "parallel", "parallel"]}
                             ins(%26, %27, %unpack, %28 : tensor<2x320x128x128xf32>, tensor<320xf32>, tensor<2x320xf32>, tensor<320xf32>)
                             outs(%29 : tensor<2x320x128x128xf16>) {
        ^bb0(%in: f32, %in_0: f32, %in_1: f32, %in_2: f32, %out: f16):
          %32 = arith.addf %in_1, %in_2 : f32
          %33 = arith.addf %in, %in_0 : f32
          %34 = arith.truncf %32 : f32 to f16
          %35 = arith.truncf %33 : f32 to f16
          %36 = arith.addf %35, %34 : f16
          linalg.yield %36 : f16
        } -> tensor<2x320x128x128xf16>

Basically you are saying you want to collapse (d0, d1) dimensions. But since d1 is used by itself you will have to use mods to get back value of d1 from the collapsed op. That will have unintended/cascading consequences further down. None of these dimensions of the iterations space can be collapsed....

I want to collapse d2, d3, so it becomes (d0, d1) -> (d1), (d0, d1) -> (d0, d1), (d0, d1) -> (d1), (d0, d1) -> (d0, d1)

I want to collapse d2, d3, so it becomes
(d0, d1) -> (d0, d1),
(d0, d1) -> (d1),
(d0, d1) -> (d0, d1),
(d0, d1) -> (d1),
(d0, d1) -> (d0, d1)

I think it is doable?

Oh yeah, that should be doable... Sorry for the digression.
Currently CollapseDimensions pass  only handles dispatches with single operation. I tried a "very general approach" which turned out to be too difficuly, but maybe something more local can be done.
@IanWood1  lets chat next week. I can give you more context.

Hey @IanWood1 what is the status of the issue?

I was looking into the CI failure https://github.com/iree-org/iree/actions/runs/9665068317/job/26661780089?pr=17725. The fault originated in https://github.com/llvm/llvm-project/blob/6b1c51bc052ae974e89e623b3d143d010fd09222/mlir/lib/Dialect/Vector/Transforms/VectorDistribute.cpp#L1696. Running this mlir through iree-codegen-vector-reduction-to-gpu will reproduce the problem.
The pass is trying to sink the trailing scf.for out of the WarpExecuteOnLane0Op (the dump of the vector.warp_execute_on_lane_0 immediately before failure is here). The failure is because the vector, vector<1xf32>, is not distributable among threads in the warp. This is the IR when running iree before the changes https://gist.github.com/IanWood1/0070caf994707fbf84a08ff04ec43cb5.
As far as I can tell, this isn't a correctness issue with my changes. But since I'm not familiar with this area, I can't be certain if this is an issue with my implementation or not. It seems like the 1 element vector should be convertible to a scalar or just not be distributed. I was able to successfully compile when disabling distribution of the 1elem vector (but I cant speak to correctness)

Can you share the executable source artifact? I.e., the dispatch in the output of --iree-hal-dump-executable-sources-to=/tmp/dump.

https://gist.github.com/IanWood1/827aee042ff6968070d2d0c6beccae4a

https://gist.github.com/IanWood1/827aee042ff6968070d2d0c6beccae4a

It looks complicated.. @MaheshRavishankar do we expect to have elementwise -> reduction -> elementwise -> reduction -> elementwise dispatch?
@Groverkss would you be able to help fix it?
To repro: run iree-compile --output-format=vm-bytecode --compile-from=executable-sources ~/repro.mlir -o /tmp/a.vmfb.

https://gist.github.com/IanWood1/827aee042ff6968070d2d0c6beccae4a

It looks complicated.. @MaheshRavishankar do we expect to have elementwise -> reduction -> elementwise -> reduction -> elementwise dispatch?
@Groverkss would you be able to help fix it?
To repro: run iree-compile --output-format=vm-bytecode --compile-from=executable-sources ~/repro.mlir -o /tmp/a.vmfb.

Yeah this is expected. It is basically softmax fusion

Okay... I just found that it is enabled under some flags. I'm seeing that the benchmark is using additional flags (e.g., --iree-flow-enable-aggressive-fusion) which falls to experimental benchmark suites. I don't think that it should be a blocker for @IanWood1 's PR because it is experimental and it is blocking other work.
Is there a way to unblock this? I've been wanting it for three weeks. Can we make it happen by default, and add a flag to disable it in the benchmark?

https://gist.github.com/IanWood1/827aee042ff6968070d2d0c6beccae4a

To repro: run iree-compile --output-format=vm-bytecode --compile-from=executable-sources ~/repro.mlir -o /tmp/a.vmfb.

@bangtianliu Mahesh mentioned that you could help on this. Can you take a look?

https://github.com/iree-org/iree/pull/17725 fixes the issue. The GPU specific issue is tracking in https://github.com/iree-org/iree/issues/17948
closing the issue.


Medium-Energy and High-Energy Bio Reactors

Description

I think this would be another good way to encourage people to produce and use green energy. At least in Minecraft :LUL:
Btw this is not such a big addition but is useful. Also, I've been thinking about other machines though. I found two ideas interesting, but in Minecraft, they look awful.

The first one is a Tidal Turbine, that uses twice-daily tidal currents to drive the turbine.
The second one is a Geothermal Turbine or Generator, that uses hot water from geysers to drive the turbine.

Changes

Added Medium-Energy Bio Reactor
Added High-Energy Bio Reactor

Checklist

[x] I have fully tested the proposed changes and promise that they will not break everything into chaos.
[x] I have also tested the proposed changes in combination with various popular addons and can confirm my changes do not break them.
[x] I followed the existing code standards and didn't mess up the formatting.
[ ] I did my best to add documentation to any public classes or methods I added.
[ ] I added sufficient Unit Tests to cover my code.

I'm closing this. If the community suggests something similar and will be approved, I will implement it.


Add a MetaDataHeader for option GFXRECON_ALLOW_PIPELINE_COMPILE_REQUIRED

GFXRECON_ALLOW_PIPELINE_COMPILE_REQUIRED will ignore writting Create*Pipeline to file when it's with VK_PIPELINE_CREATE_FAIL_ON_PIPELINE_COMPILE_REQUIRED_BIT. But we want to leave a record in the file to convert it into .json file or print it during replay, like "application requested creategraphicspipelines but that wasn't passed on to Vulkan implementation".

From #1160


[iOS 0.29.0]TypeError: Network request failed

TypeError: Network request failed
    at XMLHttpRequest.xhr.onerror (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:24437:8)
    at XMLHttpRequest.dispatchEvent (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:9960:15)
    at XMLHttpRequest.setReadyState (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:25890:6)
    at XMLHttpRequest.__didCompleteResponse (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:25748:6)
    at http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:25823:52
    at RCTDeviceEventEmitter.emit (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:9191:23)
    at MessageQueue.__callFunction (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:7279:23)
    at http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:7183:8
    at guard (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:7131:1)
    at MessageQueue.callFunctionReturnFlushedQueue (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:7182:1)

Please tell me !  What can I do something to solve the problem ?

Had same problem, solved by adding
<key>NSAppTransportSecurity</key>
<dict>
  <key>NSAllowsArbitraryLoads</key>
  <true/>
</dict>

to Info.plist

@ksarna This is exactly the solution. Thanks a lot.

@ksarna  But it did not work for me..
How can i solve..?

didn't work for me too :(
react-native 0.30.0
node v6.3.1

@facebook-github-bot stack-overflow

Hey @liuhong1happy and thanks for posting this! @grabbou tells me this issue looks like a question that would be best asked on StackOverflow. StackOverflow is amazing for Q&A: it has a reputation system, voting, the ability to mark a question as answered. Because of the reputation system it is likely the community will see and answer your question there. This also helps us use the GitHub bug tracker for bugs only. Will close this as this is really a question that should be asked on SO.

@ksarna - not working for me either (and I don't see the Stack Overflow thread) is it possible to see a screen shot of your info.plist file?  When I edit my info.plist file in raw xml then I can't open in xcode - btw I'm using xcode 7.3, most of the suggestions appear to be a different version

Please someone help me on this issue. I am stuck with the network error. Can' do anything. I have also set the "Allows Arbitrary Loads" to YES.

here is the full strack trace.
"TypeError: Network request failed
at XMLHttpRequest.xhr.onerror (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:24386:8)
at XMLHttpRequest.dispatchEvent (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:9896:15)
at XMLHttpRequest.setReadyState (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:25901:6)
at XMLHttpRequest.__didCompleteResponse (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:25759:6)
at http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:25834:52
at RCTDeviceEventEmitter.emit (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:9127:23)
at MessageQueue.__callFunction (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:7207:23)
at http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:7108:8
at guard (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:7055:1)
at MessageQueue.callFunctionReturnFlushedQueue (http://localhost:8081/index.ios.bundle?platform=ios&dev=true&minify=false:7107:1)"

I fixed it by changing those settings via the XCode UI - it didn't work when I changed directly in the plist file

One bad news for developers using NSAppTransportSecurity.
UPDATE:
[Apple will require HTTPS connections for iOS apps by the end of 2016]
https://techcrunch.com/2016/06/14/apple-will-require-https-connections-for-ios-apps-by-the-end-of-2016/


Consume es-index-manager

Also - if you could add the pkg folder to those places as well, I missed that in my previously merged PR

Codecov Report

Merging #77 (43ec185) into main (9c503aa) will decrease coverage by 2.93%.
The diff coverage is 93.93%.

@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
- Coverage   86.28%   83.35%   -2.94%
==========================================
  Files           8        8
  Lines         853      859       +6
==========================================
- Hits          736      716      -20
- Misses         84      109      +25
- Partials       33       34       +1

Impacted Files
Coverage Δ

server/server.go
81.99% <92.59%> (-4.15%)
:arrow_down:

pkg/resource/manager.go
100.00% <100.00%> (ø)

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 9c503aa...43ec185. Read the comment docs.


Pre Instructor Setup Instructions - No c:\labfiles\setup.ps1 file

Module: 00
Lab/Demo: 00 - Pre Instructor Set Up Instructions
Task: 01
Step: 04
Description of issue
There is no C;\labfiles\setup.ps1 file to edit or run.
Repro steps:

Download the lab files folder.
Open File Explorer to c:\labfiles
No setup.ps1 file found.

The setup has changed and the PowerShell script no longer needs to be run
The setup instructions in https://github.com/MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect/blob/master/TenantSetup.md have had this step removed.
Where are you seeing Step 04?

I am using Skillable LODS as the provider. They setup the classroom and i suspect they are not following your new Instructions. I have forwarded them your email.
I have some questions about how I can best succeed delivering your course is you have time to contact me personally.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Julian Sharp @.>
Sent: Monday, June 27, 2022 2:54:49 AM
To: MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect @.>
Cc: Arnold Villeneuve @.>; Author @.>
Subject: Re: [MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect] Pre Instructor Setup Instructions - No c:\labfiles\setup.ps1 file (Issue #27)
The setup has changed and the PowerShell script no longer needs to be run
The setup instructions in https://github.com/MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect/blob/master/TenantSetup.md have had this step removed.
Where are you seeing Step 04?
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect/issues/27#issuecomment-1166951590, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIIKAPV5I4XPA5WTNP7XMTVRFF3TANCNFSM5Z4PS4IA.
You are receiving this because you authored the thread.Message ID: @.***>

Hello Julian
I am still waiting for the people at Skillable to resolve the matter and I start teaching in the morning. I have reviewed all of the material for the workshops - standard architecture stuff and group discussion to learn about the process and how it evolves - all good.
I am only concerned about the PreCourse Instructor Setup for the Domain Tenant we are issued as well as ensuring the labs work as described for the Students. I reviewed your uploaded setup instructions from a month ago and see the two lab setup files. However, that is now what I see in the Skillable VM.

The PowerShell script created separate Power Platform environments and Dataverse environments for each student
These were only for the ALM lab
You still need to perform the setup for RPA lab and APP lab

I am training right now but the best information on delivering this course in on MCT Central under the tile in the top left, Business Applications MCT Zone. In there you will find a train the trainer recording for MB-600. PL-600 is essentially the same course as MB-600

Hi Skillable Support,
Where are we with this? What is being done to fix/update the labs/instructions for our class that starts tomorrow?
Please advise asap. Thank you.
Best Regards,
Dave
David O'Neal
P: +1.703.709.6338 | M: +1.703.489.9996
From: Skillable
Sent: Monday, June 27, 2022 11:33 AM
Cc: Dave O'Neal @.>; WW PD Lab @.>; Author @.>; MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect @.>; MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect @.***>
Subject: PL-600 - c:labfilessetup.ps1 does not exist (Ticket 572703)
[ EXTERNAL EMAIL ] Do not click links or open attachments unless you recognize the sender.
##- Please type your reply above this line -##
You are registered as a CC on this support request PL-600 - c:labfilessetup.ps1 does not exist Ticket 572703
Current CCs on this ticket: WW PD Lab @.) Dave O'Neal @.) MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect @.) Author @.) MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect @.***)
[Image removed by sender.]
Zachary B (Skillable)
Jun 27, 2022, 11:33 AM EDT
Hi Arnold,
We are still researching this issue, looking for a viable path forward to get this resolved.
Thanks,
Zachary Brooks
Technical Support Engineer (Tier-3)
[Image removed by sender.]
Arnold Villeneuve
Jun 27, 2022, 11:07 AM EDT
Status please
Get Outlook for Androidhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Faka.ms%2FAAb9ysg&data=05|01|dave_o'neal%40learningtree.com|77d425e278d0418f906e08da58526135|3d7530076b6b45f28ce708d2d5a100fe|0|1|637919408103207160|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=qySDtRLfvRptijtIwSVF%2B9Li06z3%2BeQGbnJ%2Bscvpqgg%3D&reserved=0

[Image removed by sender.]
Arnold Villeneuve
Jun 27, 2022, 8:37 AM EDT
According to Github Author I think Skillable is not following the current load.
Get Outlook for Androidhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Faka.ms%2FAAb9ysg&data=05|01|dave_o'neal%40learningtree.com|77d425e278d0418f906e08da58526135|3d7530076b6b45f28ce708d2d5a100fe|0|1|637919408103207160|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=qySDtRLfvRptijtIwSVF%2B9Li06z3%2BeQGbnJ%2Bscvpqgg%3D&reserved=0

Attachment(s)
~WRD0760.jpghttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Flearnondemand.zendesk.com%2Fattachments%2Ftoken%2FGBZaneIoFlLswyh16tqbT5BuN%2F%3Fname%3D~WRD0760.jpg&data=05|01|dave_o'neal%40learningtree.com|77d425e278d0418f906e08da58526135|3d7530076b6b45f28ce708d2d5a100fe|0|1|637919408103207160|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=NCSmyDsi8UI7mXwlilgL77p8oNIbU54UqknkwIXWVhk%3D&reserved=0
[Image removed by sender.]
Justin M (Skillable)
Jun 27, 2022, 7:56 AM EDT
Hello Dave,
We do not have an update at this time. Our lab operations team is currently looking into this request and we will reply with more information as soon as possible. We appreciate your patience.
Kind Regards,
Justin M | Platform Support Agent
[Image removed by sender.]
Brook B (Skillable)
Jun 26, 2022, 8:45 PM EDT
Thank you, Arnold. We have passed this information along to our Lab Operations team and are awaiting further updates.
Thank you for your patience,
Brook B | Platform Support
[Image removed by sender.]
Arnold Villeneuve
Jun 26, 2022, 8:16 PM EDT
Here is the PL-600 Trainer Prep Guide.
It refers to the Environment that needs to be set up with PowerShell scripts yet the Pre Course Instructor Setup lab (Module 00 where you download the lab files) does not have the file detailed in Step 4 as documented.
I have set up the Azure Devops Organization using the classroom-assigned Admin email account and notified the directed email address as per Instructions. Org is: DeviceManagementVilleneuve0626 and the Project is PL-600.
But that is as far as I got.
Q1: I need to know how much of the setup Skillable Staff implemented? I suspect that as the Instructor I am expected to perform the steps in Module 00 Pre Course Instructor Setup lab. Please confirm this is true.
Q2. I need to confirm that Students are assigned to "persona email accounts" using other people's names?
Q3. Please note that users do not appear to have access to the Office Outlook email app!!! This is rather important because Azure DevOps can generate email notifications. I tried to access Office Outlook as Admin within the VM however I got an error message. Also, the Outlook Icon is not listed as an available Office app. Is this on purpose and part of the base classroom deployment? If yes, why is Office Outlook not available so Students can test app and pipeline notifications?
Thank you.
Arnold Villeneuve
613-833-0984
On Sunday, June 26, 2022, 06:47:56 p.m. EDT, Skillable @.***> wrote:
Attachment(s)
PL-600T00A-ENU-TrainerPrepGuide.pdfhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Flearnondemand.zendesk.com%2Fattachments%2Ftoken%2FKcPrg7iUxs6hruK9LmIQAeSmt%2F%3Fname%3DPL-600T00A-ENU-TrainerPrepGuide.pdf&data=05|01|dave_o'neal%40learningtree.com|77d425e278d0418f906e08da58526135|3d7530076b6b45f28ce708d2d5a100fe|0|1|637919408103207160|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=px5TdhgtZtxFpo%2BTIlxPeI0zgaBZRgy8gZsCMa6JQ5E%3D&reserved=0
[Image removed by sender.]
Daniel McKinney (Skillable)
Jun 26, 2022, 6:47 PM EDT
Arnold,
Thanks for your reply.  I see what you're seeing now.
Our team is currently reviewing, and considering our options. As soon as we have more information for you, we'll update this ticket right away.
Regards,
Daniel McKinney | Platform Support Team
[Image removed by sender.]
Arnold Villeneuve
Jun 26, 2022, 6:23 PM EDT
Hey Daniel
If you have a look at the C:\labfiles you will see there is no setup.ps1 file.
I also downloaded the current version of the GIT for this course and there is no setup file there either.
I have also checked the course Git Issues section. No outstanding Issues.  I issued a new Case there as well.
Arnold Villeneuve
613-833-0984
On Sunday, June 26, 2022, 06:17:39 p.m. EDT, Skillable @.***> wrote:
[Image removed by sender.]
Daniel McKinney (Skillable)
Jun 26, 2022, 6:17 PM EDT
Arnold,
On our end, there have been no reports that there are any issues running the lab as written, following the instructor prep guide.
We have reached out to our Lab Operations team regarding your inquiry to see if they have more information for you.
Regards,
Daniel McKinney | Platform Support Team
[Image removed by sender.]
Daniel McKinney (Skillable)
Jun 26, 2022, 5:25 PM EDT
Hello Arnold,
Thank you for contacting Skillable Support.
We are sending your inquiry to our Escalations team for review.
As soon as we have an update for you, we will issue a reply to this email.
Should you require assistance with a different issue please send a new separate support ticket.
Regards,
Daniel McKinney | Platform Support Team
[Image removed by sender.]
Arnold Villeneuve
Jun 26, 2022, 4:10 PM EDT
I have a problem with my lab
Support Ticket Subject Line: PL-600 - c:labfilessetup.ps1 does not exist
First Name: ARNOLD
Last Name: VILLENEUVE
Your email Address<EMAIL_ADDRESS>Role: Instructor
Training Organization's Name: Learning Tree - Production
How may we help you?: I have a problem with my lab
I have a problem with my lab: My issue is not listed
Lab Instance ID Number: 24277453
Detailed Description/Reproduction Steps: Hello
First Lab?
Instructor Pre Class Preparation
Application Lifecycle Management lab setup
Objectives
Create users and environments for students to run the Application Lifecycle Management (ALM) lab
Step 4: File -> Open C:LabFiles Setup.ps1
**** There is not Setup.ps1 file in the c:labfiles foder! ******* Was the Setup.ps1 file already run by Skillable and then removed?
I would like to know how much, if any, I have to do of the Instructor PreClass Set Up as you are the hosting provider. There is some indication that the Students should also be doing these steps in order to have their own Azure DevOps Pipeline parallel processing capability.
Please clarify what has and has not been configured by Skillable for this classroom so that I am not duplicating or damaging setup work you have already done.
Attachment:
Please check to agree to the Support Policy-I agree to the Support Policy.: yes
Many common support queries are addressed in our integrated help documentation sections below:
For Students, Instructors and Administrators https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.skillable.com%2Ftms%2Fhome.md&data=05|01|dave_o'neal%40learningtree.com|77d425e278d0418f906e08da58526135|3d7530076b6b45f28ce708d2d5a100fe|0|1|637919408103207160|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=%2BshJV%2BJ%2FQtT5XSzpNUT%2F6d%2Bjf%2Bi%2BXNzFWm5tHih3afo%3D&reserved=0
For Lab Developers https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.skillable.com%2Flod%2Fhome.md%3Fappid%3Dlod&data=05|01|dave_o'neal%40learningtree.com|77d425e278d0418f906e08da58526135|3d7530076b6b45f28ce708d2d5a100fe|0|1|637919408103207160|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=e%2B2%2Bs11LEj2FLWmd%2FfJcHGJzkBeLgYU5kFo%2B0%2FMOkPI%3D&reserved=0
Current CCs on this ticket:
This email is a service from Skillable.
[P932Z5-XGD3K]

I will look at that now as there is still no resolution from Skillable.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Julian Sharp @.>
Sent: Monday, June 27, 2022 2:40:31 PM
To: MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect @.>
Cc: Arnold Villeneuve @.>; Author @.>
Subject: Re: [MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect] Pre Instructor Setup Instructions - No c:\labfiles\setup.ps1 file (Issue #27)
I am training right now but the best information on delivering this course in on MCT Central under the tile in the top left, Business Applications MCT Zone. In there you will find a train the trainer recording for MB-600. PL-600 is essentially the same course as MB-600
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect/issues/27#issuecomment-1167739303, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIIKALMBGL7FDI7ONKP5JTVRHYR7ANCNFSM5Z4PS4IA.
You are receiving this because you authored the thread.Message ID: @.***>

So the ALM lab is no longer done by the students correct?
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Julian Sharp @.>
Sent: Monday, June 27, 2022 2:37:14 PM
To: MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect @.>
Cc: Arnold Villeneuve @.>; Author @.>
Subject: Re: [MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect] Pre Instructor Setup Instructions - No c:\labfiles\setup.ps1 file (Issue #27)
The PowerShell script created separate Power Platform environments and Dataverse environments for each student
These were only for the ALM lab
You still need to perform the setup for RPA lab and APP lab
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect/issues/27#issuecomment-1167734566, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIIKALYXORFFCTDZD4ST6DVRHYFVANCNFSM5Z4PS4IA.
You are receiving this because you authored the thread.Message ID: @.***>

Found it. Thank you. It is the one I attended a long time ago. Reviewing now.
Still waiting to hear from Skillable for their resolution.
Arnold Villeneuve<PHONE_NUMBER>
On Monday, June 27, 2022, 02:40:30 p.m. EDT, Julian Sharp<EMAIL_ADDRESS>wrote:

I am training right now but the best information on delivering this course in on MCT Central under the tile in the top left, Business Applications MCT Zone. In there you will find a train the trainer recording for MB-600. PL-600 is essentially the same course as MB-600
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Correct the students do not do the ALM lab, it was originally included in the course when Power Platform ALM tools were new and we wanted to expose students to ALM. It was considered that ALM is sufficiently established that students should have had experience of it themselves

Skillable has resolved this issue. You can close the case, please. Thank you very much for your help, Julian.

Resolved.

Julian
How is the new version of the course coming along?
Are you looking for a Technical Editor to assist in reviews?
Arnold

Please look at the instructions provided in the VM that you have provided!! Module00 Step 4 states run the c:\labfiles\setup.ps1.
I go by the lab Instructions inside the Skillable provided VM and not Github as Skillable is the hosting provider.
Has Skillable got the correct load?
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Julian Sharp @.>
Sent: Monday, June 27, 2022, 2:54 a.m.
To: MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect @.>
Cc: Arnold Villeneuve @.>; Author @.>
Subject: Re: [MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect] Pre Instructor Setup Instructions - No c:\labfiles\setup.ps1 file  (Issue #27)
The setup has changed and the PowerShell script no longer needs to be run
The setup instructions in https://github.com/MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect/blob/master/TenantSetup.md have had this step removed.
Where are you seeing Step 04?
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/PL-600-Microsoft-Power-Platform-Solution-Architect/issues/27#issuecomment-1166951590, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIIKAPV5I4XPA5WTNP7XMTVRFF3TANCNFSM5Z4PS4IA.
You are receiving this because you authored the thread.Message ID: @.***>

You need to get Skillable to update their instructions. Skillable should be pulling the instructions from this GitHub repo

Julian
How is the new version of the course coming along?
Are you looking for a Technical Editor to assist in reviews?
Arnold

You need to get Skillable to update their instructions. Skillable should be pulling the instructions from this GitHub repo

This is an old message.

It says it was added three days ago, strange

This is an old message.

How is the new version of the course coming along?
Are you looking for a Technical Editor to assist in reviews?

There is no work going on at the moment on this course and Trevor, the course owner is away at the moment, so any changes will wait for his return. I do think some things in the course need updating.
I do think the RPA lab is unnecessary now.  It was included because Power Automate Desktop was new at the time the course was created and we wanted to give solution architects exposure to it. The lab is quite basic. As there is a separate course, PL-500, I would skip that lab and use the time on more useful activities


Move TravisCI to OpenJDK 8

Fixes TravisCI builds.
The solution retrieved from this comment: https://travis-ci.community/t/install-of-oracle-jdk8-is-failing/4365/3
Signed-off-by: Oleksandr Porunov<EMAIL_ADDRESS>

@porunov Thanks so much for this. If the checks pass I will approve it. Though keep in mind we are moving a lot of this over to gitlab's CI. We may have even done it already for the Ferma project, I need to check. But regardless I do appreciate the help.

@freemo Great to hear that!
Looks like this PR fails also. I am not yet sure why it happens.
The build passes in my branch but fails in the PR.
My branch:
https://travis-ci.org/porunov/Ferma/builds/659321901?utm_source=github_status&utm_medium=notification
PR:
https://travis-ci.org/Syncleus/Ferma/builds/659323112?utm_source=github_status&utm_medium=notification
Looks like something with codacy

@porunov So I just checked on our gitlab mirror (this is the CI that we care about, though github stays up to day and will continue to). As you can see we already moved to JDK8 there and it is passing (though the pages CI seems to failing so I need to address that): https://git.qoto.org/Ferma/Ferma/-/jobs/386

@freemo Thank you for the update! I didn't know about that CI. I see that it is passing. I think we can close the PR in this case


VSCode is not working in Safari < 14

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: latest
OS Version: doesn't matter

The same as in https://github.com/microsoft/monaco-editor/issues/2432
Because of this https://github.com/microsoft/vscode/blob/main/src/vs/base/browser/browser.ts#L79
		this._mediaQueryList.addEventListener('change', this._listener);

And this https://github.com/microsoft/vscode/blob/main/src/vs/base/browser/browser.ts#L151
	query.addEventListener('change', callback);

And maybe some other places

@alexdima fyi

There are no plans to support Safari < 14. Safari 13 and 14 are no longer supported by Apple.


SignalR not working behind NGinx

See for possible solution:
https://alimozdemir.medium.com/deployment-of-signalr-with-nginx-daf392cf2b93

Fixed with configuration.


Feature/multilingual

Add multilingual support
Verification

[pass] Run the tests and ensure they pass python -m pytest tests/
[pass] Verify the thing does what it should: I add docs/source/translator.rst

Feature explain
You need an API key for the preferred service.

DeepL
NIM

DeepL
export DEEPL_API_KEY=xxxx

Config file
You can pass the translation service, source language, and target language as arguments.

translation_service: "nim" or "deepl", "local"
lang_spec: "ja", "ja,fr" etc. (you can set multiple language codes)

Note: The Helsinki-NLP/opus-mt-en-{lang} case uses different language formats. The language codes used to name models are inconsistent. Two-digit codes can usually be found here, while three-digit codes require a search like "language code {code}".
run:
  translation_service: {your chosen translation service "nim" or "deepl", "local"}
  lang_spec: {your chosen language code}

Examples for multilingual support
DeepL
To use the translation option for garak, run the following command:
export DEEPL_API_KEY=xxxx
python3 -m garak --model_type nim --model_name meta/llama-3.1-8b-instruct --probes encoding --translation_service deepl --lang_spec ja

If you save the config file as "garak/configs/simple_translate_config_deepl.yaml", use this command:
export DEEPL_API_KEY=xxxx
python3 -m garak --model_type nim --model_name meta/llama-3.1-8b-instruct --probes encoding --config garak/configs/simple_translate_config_deepl.yaml

Example config file:
run:
  translation_service: "deepl"
  lang_spec: "ja"

NIM
For NIM, run the following command:
export NIM_API_KEY=xxxx
python3 -m garak --model_type nim --model_name meta/llama-3.1-8b-instruct --probes encoding --translation_service nim --lang_spec ja

If you save the config file as "garak/configs/simple_translate_config_nim.yaml", use this command:
export NIM_API_KEY=xxxx
python3 -m garak --model_type nim --model_name meta/llama-3.1-8b-instruct --probes encoding --config garak/configs/simple_translate_config_nim.yaml

Example config file:
run:
  translation_service: "nim"
  lang_spec: "ja"

Local
For local translation, use the following command:
python3 -m garak --model_type nim --model_name meta/llama-3.1-8b-instruct --probes encoding --translation_service local --lang_spec ja

If you save the config file as "garak/configs/simple_translate_config_local.yaml", use this command:
python3 -m garak --model_type nim --model_name meta/llama-3.1-8b-instruct --probes encoding --config garak/configs/simple_translate_config_local.yaml

Example config file:
run:
  translation_service: local
  local_model_name: "facebook/m2m100_418M"
  local_tokenizer_name: "facebook/m2m100_418M"
  lang_spec: "ja"

Specific Hardware Examples:
Local Translation needs GPU

GPU related

Specific support required cuda
Minium GPU Memory:

Helsinki-NLP/opus-mt-en-{}: around 1500MB
facebook/m2m100_418M: around 8000MB
facebook/m2m100_1.2B: around 15000MB

I have read the DCO Document and I here by sign the DCO

Thank you, this is a significant amount of work.
This is for translation-based multilingual, rather than general multilingual support, so we should be careful to separate these two functions. The ability to select which languages garak uses is distinct from how that language is achieved (in prompts, detectors, and so on).
There are some refactorings needed to get the PR to a sustainable state. We may need a couple more hooks, either injected into base classes or added there. It might be simplest to add these hooks to the harness. Happy to set up a call or instant message chat to discuss this. Given the breadth of the changes, it is likely to be beneficial to discuss plans and get good alignment while doing the rest of the changes.

Thank you for review.
I would like to set up a call for understanding the best way to implement this function.