Build Pipeline fails when running as a Branch Build Validation Policy

Describe the issue
I've followed all the steps in https://aka.ms/cicdhol
My build pipeline works fine when I commit any piece of code to the master branch
But when I activate the Build Validation in the Branch Policies for the master branch, the build process fails in the Remove Container step.
The error message is
 ##[error]The remote server returned an error: (404) Not Found. No published package matches the provided arguments.
At C:\Program
Files\WindowsPowerShell\Modules\BcContainerHelper\1.0.15-preview272\AppHandling\Compile-AppInNavContainer.ps1:308
char:17
+                 throw (GetExtenedErrorMessage $_.Exception)
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (The remote serv...ided arguments.:String) [], RuntimeException
    + FullyQualifiedErrorId : The remote server returned an error: (404) Not Found. No published package matches the p
   rovided arguments.
2020-11-30T12:03:33.3661772Z ##[error]PowerShell exited with code '1'.

Scripts used to create container and cause the issue
Same script found in https://aka.ms/cicdhol
trigger:
- master

pool:
  vmImage: 'Windows-Latest'

variables:
- group: BuildVariables

jobs:
- job: Build
  timeoutInMinutes: 300
  variables:
    build.clean: all
    platform: x64
    version: 'ci'
    appVersion: '1.0'
    appBuild: $(Build.BuildID)
    appRevision: 0
    skipComponentGovernanceDetection: True

  steps:
  - task: PowerShell@2
    displayName: 'Set BuildNumber'
    inputs:
      targetType: inline
      script: 'Write-Host "##vso[build.updatebuildnumber]$(appVersion).$(appBuild).$(appRevision)"'

  - task: PowerShell@2
    displayName: 'Run Pipeline'
    env:
      InsiderSasToken: '$(InsiderSasToken)'
      LicenseFile: '$(LicenseFile)'
      CodeSignCertPfxFile: '$(CodeSignCertPfxFile)'
      CodeSignCertPfxPassword: '$(CodeSignCertPfxPassword)'
    inputs:
      targetType: filePath
      filePath: 'scripts\DevOps-Pipeline.ps1'
      arguments: '-environment "AzureDevOps" -version "$(version)" -appBuild $(appBuild) -appRevision $(appRevision)'
      failOnStderr: true

  - task: PublishTestResults@2
    displayName: 'Publish Test Results'
    condition: and(succeeded(),ne(variables['TestResults'],''))
    inputs:
      testResultsFormat: JUnit
      testResultsFiles: '$(testResults)'
      failTaskOnFailedTests: true

  - task: PublishBuildArtifacts@1
    displayName: 'Publish Artifacts'
    inputs:
      PathtoPublish: '$(Build.ArtifactStagingDirectory)'
      ArtifactName: output

  - task: PowerShell@2
    displayName: 'Cleanup'
    condition: always()
    inputs:
      targetType: filePath
      filePath: 'scripts\Cleanup.ps1'
      arguments: '-environment "AzureDevOps"'
      failOnStderr: false

Full output of scripts
Failing at Remove Contaier step
2020-11-30T11:44:54.3099030Z ##[section]Starting: Run Pipeline
2020-11-30T11:44:54.3397459Z ==============================================================================
2020-11-30T11:44:54.3397788Z Task         : PowerShell
2020-11-30T11:44:54.3398073Z Description  : Run a PowerShell script on Linux, macOS, or Windows
2020-11-30T11:44:54.3398323Z Version      : 2.177.0
2020-11-30T11:44:54.3398875Z Author       : Microsoft Corporation
2020-11-30T11:44:54.3399401Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/powershell
2020-11-30T11:44:54.3400895Z ==============================================================================
2020-11-30T11:44:55.9289735Z Generating script.
2020-11-30T11:44:55.9379830Z Formatted command: . 'D:\a\1\s\scripts\DevOps-Pipeline.ps1' -environment "AzureDevOps" -version "ci" -appBuild 27 -appRevision 0
2020-11-30T11:44:56.0335560Z ========================== Starting Command Output ===========================
2020-11-30T11:44:56.0910194Z ##[command]"C:\windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'D:\a\_temp\5d089f14-f1e7-4003-a4df-9fa1538e5d8e.ps1'"
2020-11-30T11:44:56.7957387Z Set artifact = //17.0.17126.17791/es
2020-11-30T11:44:56.8027108Z Set pipelineName = MyNewHelloWorldApp-ci
2020-11-30T11:44:56.8356718Z Set containerName = hostedagent-mynewhelloworldapp-ci
2020-11-30T11:44:56.8948469Z Set installApps = ''
2020-11-30T11:44:56.8979327Z Set previousApps = ''
2020-11-30T11:44:56.9015740Z Set appSourceCopMandatoryAffixes = ''
2020-11-30T11:44:56.9040621Z Set appSourceCopSupportedCountries = ''
2020-11-30T11:44:56.9134542Z Set appFolders = 'custApp,masterData'
2020-11-30T11:44:56.9193483Z Set testFolders = 'testApp'
2020-11-30T11:44:56.9267679Z Set memoryLimit = '6G'
2020-11-30T11:44:56.9366598Z Set additionalCountries = ''
2020-11-30T11:44:56.9393353Z Set genericImageName = ''
2020-11-30T11:44:56.9461199Z Set vaultNameForLocal = 'BuildVariables'
2020-11-30T11:44:56.9582366Z Set bcContainerHelperVersion = 'preview'
2020-11-30T11:44:56.9783584Z Set installTestFramework = True
2020-11-30T11:44:56.9963186Z Set installTestLibraries = False
2020-11-30T11:44:57.0108398Z Set installPerformanceToolkit = False
2020-11-30T11:44:57.0273505Z Set enableCodeCop = False
2020-11-30T11:44:57.0385828Z Set enableAppSourceCop = False
2020-11-30T11:44:57.0444426Z Set enablePerTenantExtensionCop = True
2020-11-30T11:44:57.0507339Z Set enableUICop = True
2020-11-30T11:44:57.0560891Z Set doNotSignApps = True
2020-11-30T11:44:57.0582759Z Set doNotRunTests = False
2020-11-30T11:44:57.0706335Z Set cacheImage = True
2020-11-30T11:44:57.0776001Z Set CreateRuntimePackages = True
2020-11-30T11:44:57.5846345Z Downloading https://bccontainerhelper.azureedge.net/public/1.0.15-preview272.zip
2020-11-30T11:45:19.1537965Z BcContainerHelper version 1.0.15-preview272
2020-11-30T11:45:21.4912408Z Determining artifacts to use
2020-11-30T11:45:21.8978296Z   _____                               _
2020-11-30T11:45:21.9001709Z  |  __ \                             | |
2020-11-30T11:45:21.9022106Z  | |__) |_ _ _ __ __ _ _ __ ___   ___| |_ ___ _ __ ___
2020-11-30T11:45:21.9075481Z  |  ___/ _` | '__/ _` | '_ ` _ \ / _ \ __/ _ \ '__/ __|
2020-11-30T11:45:21.9112779Z  | |  | (_| | | | (_| | | | | | |  __/ |_  __/ |  \__ \
2020-11-30T11:45:21.9140529Z  |_|   \__,_|_|  \__,_|_| |_| |_|\___|\__\___|_|  |___/
2020-11-30T11:45:21.9149578Z
2020-11-30T11:45:21.9170491Z Pipeline name               MyNewHelloWorldApp-ci
2020-11-30T11:45:21.9190198Z Container name              hostedagent-mynewhelloworldapp-ci
2020-11-30T11:45:21.9218006Z Image name
2020-11-30T11:45:21.9238448Z ArtifactUrl                 https://bcartifacts.azureedge.net/sandbox/17.0.17126.17791/es
2020-11-30T11:45:21.9259927Z SasToken                    Not Specified
2020-11-30T11:45:21.9708850Z Credential                  admin/Zomo4882
2020-11-30T11:45:22.0685420Z MemoryLimit                 6G
2020-11-30T11:45:22.0734659Z Enable Task Scheduler       False
2020-11-30T11:45:22.0778743Z Assign Premium Plan         False
2020-11-30T11:45:22.0829684Z Install Test Framework      True
2020-11-30T11:45:22.0881366Z Install Test Libraries      False
2020-11-30T11:45:22.0921478Z Install Perf. Toolkit       False
2020-11-30T11:45:22.0961928Z enableCodeCop               False
2020-11-30T11:45:22.1108319Z enableAppSourceCop          False
2020-11-30T11:45:22.1157859Z enableUICop                 True
2020-11-30T11:45:22.1212220Z enablePerTenantExtensionCop True
2020-11-30T11:45:22.1253326Z escapeFromCops              False
2020-11-30T11:45:22.1301196Z azureDevOps                 True
2020-11-30T11:45:22.1345198Z License file                Specified
2020-11-30T11:45:22.1381708Z CodeSignCertPfxFile         Not specified
2020-11-30T11:45:22.1433629Z TestResultsFile             D:\a\1\s\TestResults.xml
2020-11-30T11:45:22.1477698Z TestResultsFormat           JUnit
2020-11-30T11:45:22.1559546Z AdditionalCountries
2020-11-30T11:45:22.1599260Z PackagesFolder              D:\a\1\s\.packages
2020-11-30T11:45:22.1649369Z OutputFolder                D:\a\1\s\.output
2020-11-30T11:45:22.1677146Z BuildArtifactFolder         D:\a\1\a
2020-11-30T11:45:22.1726768Z CreateRuntimePackages       True
2020-11-30T11:45:22.1762290Z AppBuild                    27
2020-11-30T11:45:22.1864029Z AppRevision                 0
2020-11-30T11:45:22.1881843Z Install Apps
2020-11-30T11:45:22.1900639Z - None
2020-11-30T11:45:22.1923970Z Previous Apps
2020-11-30T11:45:22.1943938Z - None
2020-11-30T11:45:22.1990385Z Application folders
2020-11-30T11:45:22.2019463Z - D:\a\1\s\custApp
2020-11-30T11:45:22.2038026Z - D:\a\1\s\masterData
2020-11-30T11:45:22.2057157Z Test application folders
2020-11-30T11:45:22.2128405Z - D:\a\1\s\testApp
2020-11-30T11:45:22.2823859Z
2020-11-30T11:45:22.2869007Z   _____       _ _ _                                          _        _
2020-11-30T11:45:22.2932763Z  |  __ \     | | (_)                                        (_)      (_)
2020-11-30T11:45:22.2959238Z  | |__) |   _| | |_ _ __   __ _    __ _  ___ _ __   ___ _ __ _  ___   _ _ __ ___   __ _  __ _  ___
2020-11-30T11:45:22.2977016Z  |  ___/ | | | | | | '_ \ / _` |  / _` |/ _ \ '_ \ / _ \ '__| |/ __| | | '_ ` _ \ / _` |/ _` |/ _ \
2020-11-30T11:45:22.2997460Z  | |   | |_| | | | | | | | (_| | | (_| |  __/ | | |  __/ |  | | (__  | | | | | | | (_| | (_| |  __/
2020-11-30T11:45:22.3020715Z  |_|    \__,_|_|_|_|_| |_|\__, |  \__, |\___|_| |_|\___|_|  |_|\___| |_|_| |_| |_|\__,_|\__, |\___|
2020-11-30T11:45:22.3051855Z                            __/ |   __/ |                                                 __/ |
2020-11-30T11:45:22.3071965Z                           |___/   |___/                                                 |___/
2020-11-30T11:45:22.3080683Z
2020-11-30T11:45:22.3120900Z Pulling mcr.microsoft.com/businesscentral:10.0.17763.1577
2020-11-30T11:49:12.8023117Z
2020-11-30T11:49:12.8040152Z Pulling generic image took 230 seconds
2020-11-30T11:49:12.8207286Z
2020-11-30T11:49:12.8248790Z    _____                _   _                               _        _
2020-11-30T11:49:12.8276463Z   / ____|              | | (_)                             | |      (_)
2020-11-30T11:49:12.8300683Z  | |     _ __ ___  __ _| |_ _ _ __   __ _    ___ ___  _ __ | |_ __ _ _ _ __   ___ _ __
2020-11-30T11:49:12.8329223Z  | |    | '__/ _ \/ _` | __| | '_ \ / _` |  / __/ _ \| '_ \| __/ _` | | '_ \ / _ \ '__|
2020-11-30T11:49:12.8393086Z  | |____| | |  __/ (_| | |_| | | | | (_| | | (__ (_) | | | | |_ (_| | | | | |  __/ |
2020-11-30T11:49:12.8419146Z   \_____|_|  \___|\__,_|\__|_|_| |_|\__, |  \___\___/|_| |_|\__\__,_|_|_| |_|\___|_|
2020-11-30T11:49:12.8439314Z                                      __/ |
2020-11-30T11:49:12.8467510Z                                     |___/
2020-11-30T11:49:12.8477758Z
2020-11-30T11:49:13.0836316Z WARNING: Container name should not exceed 15 characters
2020-11-30T11:49:13.5962302Z BcContainerHelper is version 1.0.15-preview272
2020-11-30T11:49:13.5992964Z BcContainerHelper is running as administrator
2020-11-30T11:49:13.6090517Z Host is Microsoft Windows Server 2019 Datacenter - ltsc2019
2020-11-30T11:49:15.0062984Z Docker Client Version is 19.03.12
2020-11-30T11:49:15.0148568Z Docker Server Version is 19.03.12
2020-11-30T11:49:16.1527901Z Downloading application artifact /sandbox/17.0.17126.17791/es
2020-11-30T11:49:16.1827019Z Downloading C:\Users\VssAdministrator\AppData\Local\Temp\8d40e460-63d2-4ad9-8c7d-6c857a7538f0.zip
2020-11-30T11:49:24.5744926Z Unpacking application artifact to tmp folder using 7zip
2020-11-30T11:49:36.0585992Z Downloading platform artifact /sandbox/17.0.17126.17791/platform
2020-11-30T11:49:36.0657024Z Downloading C:\Users\VssAdministrator\AppData\Local\Temp\afe4c960-5718-469b-9bed-877908805f5e.zip
2020-11-30T11:49:45.6667706Z Unpacking platform artifact to tmp folder using 7zip
2020-11-30T11:50:11.9140034Z Downloading Prerequisite Components
2020-11-30T11:50:11.9369741Z Downloading c:\bcartifacts.cache\sandbox\17.0.17126.17791\platform\Prerequisite Components\Open XML SDK 2.5 for Microsoft Office\OpenXMLSDKv25.msi
2020-11-30T11:50:11.9910876Z Downloading c:\bcartifacts.cache\sandbox\17.0.17126.17791\platform\Prerequisite Components\IIS URL Rewrite Module\rewrite_2.0_rtw_x64.msi
2020-11-30T11:50:12.0677508Z Downloading c:\bcartifacts.cache\sandbox\17.0.17126.17791\platform\Prerequisite Components\DotNetCore\DotNetCore.1.0.4_1.1.1-WindowsHosting.exe
2020-11-30T11:50:13.2076628Z Fetching all docker images
2020-11-30T11:50:14.7068452Z Using image mcr.microsoft.com/businesscentral:10.0.17763.1577
2020-11-30T11:50:14.7079058Z Creating Container hostedagent-mynewhelloworldapp-ci
2020-11-30T11:50:14.7079919Z Version: 17.0.17126.17791-ES
2020-11-30T11:50:14.7080332Z Style: sandbox
2020-11-30T11:50:14.7080523Z Multitenant: Yes
2020-11-30T11:50:14.7080701Z Platform: 17.0.17020.17774
2020-11-30T11:50:14.7081087Z Generic Tag: <IP_ADDRESS>
2020-11-30T11:50:14.7081477Z Container OS Version: 10.0.17763.1577 (ltsc2019)
2020-11-30T11:50:14.7081739Z Host OS Version: 10.0.17763.1577 (ltsc2019)
2020-11-30T11:50:14.7082118Z Using process isolation
2020-11-30T11:50:14.7082458Z Using locale es-ES
2020-11-30T11:50:14.7082899Z Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
2020-11-30T11:50:14.7083322Z Using license file https://dl.dropboxusercontent.com/s/xv67tojh0t3c7re/BC17.flf
2020-11-30T11:50:14.7083973Z Downloading C:\ProgramData\BcContainerHelper\Extensions\hostedagent-mynewhelloworldapp-ci\my\license.flf
2020-11-30T11:50:15.5863471Z Additional Parameters:
2020-11-30T11:50:15.5890261Z --volume "D:\a\1\s:c:\sources"
2020-11-30T11:50:15.5910611Z --env customNavSettings=EnableTaskScheduler=False
2020-11-30T11:50:15.5928202Z Files in C:\ProgramData\BcContainerHelper\Extensions\hostedagent-mynewhelloworldapp-ci\my:
2020-11-30T11:50:15.6025368Z - AdditionalOutput.ps1
2020-11-30T11:50:15.6052785Z - license.flf
2020-11-30T11:50:15.6093645Z - MainLoop.ps1
2020-11-30T11:50:15.6145374Z - SetupVariables.ps1
2020-11-30T11:50:15.6263516Z - updatehosts.ps1
2020-11-30T11:50:15.6292720Z Creating container hostedagent-mynewhelloworldapp-ci from image mcr.microsoft.com/businesscentral:10.0.17763.1577
2020-11-30T11:50:16.5758613Z c23d1218f38f330acfe93ac4a4c47f24ecb2939a9b552690e134ee30187bebf3
2020-11-30T11:50:32.4836870Z Waiting for container hostedagent-mynewhelloworldapp-ci to be ready
2020-11-30T11:50:47.7129698Z Adding HOSTEDAGENT-MYN to hosts file
2020-11-30T11:50:49.1084255Z Using artifactUrl https://bcartifacts.azureedge.net/sandbox/17.0.17126.17791/es
2020-11-30T11:50:49.1106607Z Using installer from C:\Run\150-new
2020-11-30T11:50:49.1142795Z Installing Business Central
2020-11-30T11:50:49.1195118Z Installing from artifacts
2020-11-30T11:51:00.3460659Z Starting Local SQL Server
2020-11-30T11:51:00.3490488Z WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
2020-11-30T11:51:01.8440842Z start...
2020-11-30T11:51:01.8548139Z Starting Internet Information Server
2020-11-30T11:51:03.2973308Z Copying Service Tier Files
2020-11-30T11:51:06.8320930Z Copying PowerShell Scripts
2020-11-30T11:51:06.8340477Z Copying dependencies
2020-11-30T11:51:06.8357121Z Copying ReportBuilder
2020-11-30T11:51:06.8373222Z Importing PowerShell Modules
2020-11-30T11:51:26.0973231Z Determining Database Collation from c:\dl\sandbox\17.0.17126.17791\es\BusinessCentral-ES.bak
2020-11-30T11:52:25.4586402Z Changing Database Server Collation to Modern_Spanish_100_CS_AS
2020-11-30T11:52:41.6490522Z Restoring CRONUS Demo Database
2020-11-30T11:52:44.3591149Z Setting CompatibilityLevel for tenant on localhost\SQLEXPRESS
2020-11-30T11:56:57.9319180Z Exporting Application to CRONUS
2020-11-30T11:57:06.6749402Z Removing Application from tenant
2020-11-30T11:57:06.6778416Z Modifying Business Central Service Tier Config File for Docker
2020-11-30T11:57:06.6801011Z Creating Business Central Service Tier
2020-11-30T11:57:08.1467170Z Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
2020-11-30T11:57:08.1493147Z Copying Web Client Files
2020-11-30T11:57:08.1628415Z Copying Client Files
2020-11-30T11:57:08.1655975Z Copying ModernDev Files
2020-11-30T11:57:08.1742764Z Copying additional files
2020-11-30T11:57:08.1767619Z Copying ConfigurationPackages
2020-11-30T11:57:08.1792494Z Copying Test Assemblies
2020-11-30T11:57:08.1870066Z Copying Extensions
2020-11-30T11:57:08.1893416Z Copying Applications
2020-11-30T11:57:08.1921308Z Copying Applications.ES
2020-11-30T11:57:59.9499699Z Starting Business Central Service Tier
2020-11-30T11:58:02.8246434Z Importing license file
2020-11-30T11:58:02.8271669Z Copying Database on localhost\SQLEXPRESS from tenant to default
2020-11-30T11:58:02.8298679Z Taking database tenant offline
2020-11-30T11:58:25.8774123Z Copying database files
2020-11-30T11:58:25.8807671Z Attaching files as new Database default
2020-11-30T11:58:25.8889173Z Putting database tenant back online
2020-11-30T11:58:25.8961731Z Mounting tenant database
2020-11-30T12:01:08.9621463Z Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
2020-11-30T12:01:12.7797229Z Sync'ing Tenant
2020-11-30T12:01:12.7843816Z Tenant is Operational
2020-11-30T12:01:12.7868176Z Stopping Business Central Service Tier
2020-11-30T12:01:12.7934854Z Installation took 623 seconds
2020-11-30T12:01:12.7935603Z Installation complete
2020-11-30T12:01:14.2695336Z Initializing...
2020-11-30T12:01:14.2719167Z Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
2020-11-30T12:01:14.2780850Z Starting Container
2020-11-30T12:01:14.2818707Z Hostname is hostedagent-mynewhelloworldapp-ci
2020-11-30T12:01:15.8083224Z PublicDnsName is hostedagent-mynewhelloworldapp-ci
2020-11-30T12:01:15.8106771Z Using NavUserPassword Authentication
2020-11-30T12:01:18.9447684Z Creating Self Signed Certificate
2020-11-30T12:01:18.9582583Z Self Signed Certificate Thumbprint E82FA537561C50FB3BF81F5774D0A3CE1F0A3157
2020-11-30T12:01:20.5389246Z Modifying Service Tier Config File with Instance Specific Settings
2020-11-30T12:01:20.5439728Z Modifying Service Tier Config File with settings from environment variable
2020-11-30T12:01:22.0726609Z Setting EnableTaskScheduler to False
2020-11-30T12:01:42.0913174Z Starting Service Tier
2020-11-30T12:01:42.1564077Z Registering event sources
2020-11-30T12:01:53.7948326Z Creating DotNetCore Web Server Instance
2020-11-30T12:01:53.7963397Z Enabling Financials User Experience
2020-11-30T12:01:53.7970805Z Enabling rewrite rule: Don't rewrite system files
2020-11-30T12:01:53.7986469Z Enabling rewrite rule: Already have tenant specified
2020-11-30T12:01:53.7992742Z Enabling rewrite rule: Hostname (without port) to tenant
2020-11-30T12:01:53.8002186Z Using license file 'c:\run\my\license.flf'
2020-11-30T12:01:56.9013693Z Import License
2020-11-30T12:01:56.9028101Z Dismounting Tenant
2020-11-30T12:01:56.9032390Z Mounting Tenant
2020-11-30T12:02:13.5893945Z Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
2020-11-30T12:02:18.3685652Z Sync'ing Tenant
2020-11-30T12:02:18.3762587Z Tenant is Operational
2020-11-30T12:02:19.8077793Z Creating http download site
2020-11-30T12:02:19.8162270Z Setting SA Password and enabling SA
2020-11-30T12:02:19.8182451Z Creating admin as SQL User and add to sysadmin
2020-11-30T12:02:34.7168387Z Creating SUPER user
2020-11-30T12:02:34.7239748Z Container IP Address: <IP_ADDRESS>
2020-11-30T12:02:34.7435137Z Container Hostname  : hostedagent-mynewhelloworldapp-ci
2020-11-30T12:02:34.7474862Z Container Dns Name  : hostedagent-mynewhelloworldapp-ci
2020-11-30T12:02:34.7512148Z Web Client          : http://hostedagent-mynewhelloworldapp-ci/BC/?tenant=default
2020-11-30T12:02:34.7569850Z Dev. Server         : http://hostedagent-mynewhelloworldapp-ci
2020-11-30T12:02:34.7719086Z Dev. ServerInstance : BC
2020-11-30T12:02:34.7733449Z Dev. Server Tenant  : default
2020-11-30T12:02:34.7754163Z Setting hostedagent-mynewhelloworldapp-ci to <IP_ADDRESS> in host hosts file
2020-11-30T12:02:34.7786811Z Setting hostedagent-mynewhelloworldapp-ci-default to <IP_ADDRESS> in host hosts file
2020-11-30T12:02:34.7792667Z
2020-11-30T12:02:34.7817215Z Files:
2020-11-30T12:02:34.7869329Z http://hostedagent-mynewhelloworldapp-ci:8080/ALLanguage.vsix
2020-11-30T12:02:34.7877704Z
2020-11-30T12:02:34.7904133Z Container Total Physical Memory is 7.0Gb
2020-11-30T12:02:34.7952784Z Container Free Physical Memory is 2.6Gb
2020-11-30T12:02:34.7994363Z
2020-11-30T12:02:34.8017724Z Initialization took 82 seconds
2020-11-30T12:02:34.9365129Z Ready for connections!
2020-11-30T12:02:35.0963668Z Reading CustomSettings.config from hostedagent-mynewhelloworldapp-ci
2020-11-30T12:02:38.2637070Z Creating Desktop Shortcuts for hostedagent-mynewhelloworldapp-ci
2020-11-30T12:02:38.7450878Z Container hostedagent-mynewhelloworldapp-ci successfully created
2020-11-30T12:02:38.7529575Z
2020-11-30T12:02:38.7624226Z Use:
2020-11-30T12:02:38.7644095Z Get-BcContainerEventLog -containerName hostedagent-mynewhelloworldapp-ci to retrieve a snapshot of the event log from the container
2020-11-30T12:02:38.7646718Z Get-BcContainerDebugInfo -containerName hostedagent-mynewhelloworldapp-ci to get debug information about the container
2020-11-30T12:02:38.7671578Z Enter-BcContainer -containerName hostedagent-mynewhelloworldapp-ci to open a PowerShell prompt inside the container
2020-11-30T12:02:38.7788646Z Remove-BcContainer -containerName hostedagent-mynewhelloworldapp-ci to remove the container again
2020-11-30T12:02:38.7812505Z docker logs hostedagent-mynewhelloworldapp-ci to retrieve information about URL's again
2020-11-30T12:02:38.8102667Z
2020-11-30T12:02:38.8186434Z Creating container took 806 seconds
2020-11-30T12:02:38.8200430Z
2020-11-30T12:02:38.8213532Z    _____                      _ _ _
2020-11-30T12:02:38.8232227Z   / ____|                    (_) (_)
2020-11-30T12:02:38.8279422Z  | |     ___  _ __ ___  _ __  _| |_ _ __   __ _    __ _ _ __  _ __  ___
2020-11-30T12:02:38.8300872Z  | |    / _ \| '_ ` _ \| '_ \| | | | '_ \ / _` |  / _` | '_ \| '_ \/ __|
2020-11-30T12:02:38.8321476Z  | |____ (_) | | | | | | |_) | | | | | | | (_| | | (_| | |_) | |_) \__ \
2020-11-30T12:02:38.8340901Z   \_____\___/|_| |_| |_| .__/|_|_|_|_| |_|\__, |  \__,_| .__/| .__/|___/
2020-11-30T12:02:38.8365074Z                        | |                 __/ |       | |   | |
2020-11-30T12:02:38.8421858Z                        |_|                |___/        |_|   |_|
2020-11-30T12:02:38.8434324Z
2020-11-30T12:02:38.9337119Z Using Version <IP_ADDRESS>
2020-11-30T12:02:40.7611431Z Using Symbols Folder: D:\a\1\s\.packages
2020-11-30T12:02:55.6730038Z Downloading symbols: Microsoft_Application_17.0.17126.17791.app
2020-11-30T12:02:55.6993259Z Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=Application&versionText=<IP_ADDRESS>&tenant=default
2020-11-30T12:03:00.2496933Z Adding dependency to System Application from Microsoft
2020-11-30T12:03:00.2538185Z Adding dependency to Base Application from Microsoft
2020-11-30T12:03:00.2594931Z Downloading symbols: Microsoft_System_17.0.17020.17774.app
2020-11-30T12:03:00.2622155Z Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=System&versionText=<IP_ADDRESS>&tenant=default
2020-11-30T12:03:01.9824701Z Downloading symbols: Microsoft_System Application_17.0.17126.17791.app
2020-11-30T12:03:01.9844445Z Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=System Application&versionText=<IP_ADDRESS>&tenant=default
2020-11-30T12:03:03.9288329Z Downloading symbols: Microsoft_Base Application_17.0.17126.17791.app
2020-11-30T12:03:03.9334387Z Url : http://<IP_ADDRESS>:7049/BC/dev/packages?publisher=Microsoft&appName=Base Application&versionText=<IP_ADDRESS>&tenant=default
2020-11-30T12:03:06.4648178Z Compiling...
2020-11-30T12:03:06.5024421Z .\alc.exe /project:"c:\sources\masterData" /packagecachepath:"c:\sources\.packages" /out:"c:\sources\.output\ACTRMA_My MasterData Hello World App_<IP_ADDRESS>.app" /analyzer:C:\build\vsix\extension\bin\Analyzers\Microsoft.Dynamics.Nav.PerTenantExtensionCop.dll /analyzer:C:\build\vsix\extension\bin\Analyzers\Microsoft.Dynamics.Nav.UICop.dll /assemblyprobingpaths:"C:\Program Files (x86)\Microsoft Dynamics NAV\170\RoleTailored Client","C:\Program Files\Microsoft Dynamics NAV\170\Service","C:\Program Files (x86)\Open XML SDK\V2.5\lib","c:\Windows\Microsoft.NET\Assembly","C:\Test Assemblies\Mock Assemblies"
2020-11-30T12:03:19.7074819Z Microsoft (R) AL Compiler version <IP_ADDRESS>26
2020-11-30T12:03:19.7195500Z Copyright (C) Microsoft Corporation.  No published package matches the provided arguments.
At C:\Program
Files\WindowsPowerShell\Modules\BcContainerHelper\1.0.15-preview272\AppHandling\Compile-AppInNavContainer.ps1:308
char:17
+                 throw (GetExtenedErrorMessage $_.Exception)
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (The remote serv...ided arguments.:String) [], RuntimeException
    + FullyQualifiedErrorId : The remote server returned an error: (404) Not Found. No published package matches the p
   rovided arguments.
2020-11-30T12:03:33.3661772Z ##[error]PowerShell exited with code '1'.
2020-11-30T12:03:33.4263163Z ##[section]Finishing: Run Pipeline

Additional context

It only happens when building through a Pull Request. When the Build is triggered through a commit on the master branch it works fine

It looks like you are building: ACTRMA_My MasterData Hello World App_<IP_ADDRESS>.app
Publisher: ACTRMA
and then looking for the symbols with a different publisher:
Active Business & Technology_My MasterData Hello World App_<IP_ADDRESS>.app
Publisher: Active Business & Technology
Looks like your dependencies are off.
Have no idea why this is related to a branch - maybe it is related to app.json in that branch.

You're right, I have a difference in that branch.
Thank you Freddy


Add FrameworkCapability factory

On further testing, I discovered that this extra line was missing. I now have successfully tested getting a Mesomatic framework to connect to a cluster with the GPU_CAPABILITY.

Hey, I pushed a version of this which accounts for the new way to map keywords to record builders.

Thank you!


Style cleanup

Mostly cleaning up extraneous spaces and little PEP8 mistakes.
Merge at will. This is just intended to make review of future PRs easier by separating the cleanup from the real code. There's plenty more style cleanup to do, but this improves a few files that I plan to work on the in the near future.

thanks. Will merge.


Delacy Icon

Is this a finished asset?

[X] Finished asset
[ ] Asset request

What type of asset is this?

[X] Icon
[ ] Concept art

**Delacy Icon

**
Does the following exist?

[ ] SVG file
[X] PNG file
[ ] JPG file

Thanks! But it's already present on manufacturers page


Add Cleaner

This PR adds a Cleaner.
A Cleaner periodically removes expired entries. Its interval is specified by the user.  It runs in a separate goroutine.

Codecov Report

Merging #13 (cf49afb) into master (0312335) will increase coverage by 0.42%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   96.80%   97.22%   +0.42%
==========================================
  Files           6        7       +1
  Lines         313      361      +48
==========================================
+ Hits          303      351      +48
  Misses         10       10

Impacted Files
Coverage Δ

cache.go
100.00% <100.00%> (ø)

cleaner.go
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Handle when resp_vol4s are empty

From an evaluation standpoint, if a system returns an invalid response, the test harness should fail the evaluation rather than exit with an exception.

We have a very strict linter; to satisfy it, please run make format in the root of the repo.


Draw Map error

I have this issue as well, with a world created earlier and opened/saved in 1.3.5, as well as a world created in 1.3.5.


Remove tag, use only image digest.

Konflux doesn't support it.
See https://github.com/containers/image/issues/1736.

Obsolete. See #46.


refactor(payment): remove flow from Operation

Type of Change

[ ] Bugfix
[ ] New feature
[ ] Enhancement
[x] Refactoring
[ ] Dependency updates

Description
This change splits the operation into to parts, and provides a clear separation between how data is handled in the payments core and in the call connector function.

Additional Changes

[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context
N/A

How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

This change, is not valid in the current state of the entire payments core, in the case when non-flow specific function in the operation increases this approach could be brought into action. For now closing this PR


LibWeb/CSS: transform rotate function not properly rotating around Z axis

Using the transform: rotate(...) or rotateZ(...) functions do not behave as expected:

Ladybird Screenshot
Arc (Chromium) Screenshot

Reproducible html:
<!DOCTYPE html>

        <meta charset="utf-8" />

Painter that uses LibGfx does not support CSS rotation transform. we are working on a new Skia painter that is currently available under a flag --enable-skia-painting where attached reduction does display correctly.


Calendar FreeBusy Anonymous sharing must be enabled to use Bookings with me.

In the article, we recommend:
To enable anonymous sharing, use the following command.
PowerShellCopy
Set-SharingPolicy "Default Sharing Policy" -Domains @{Add="Anonymous:CalendarSharingFreeBusyReviewer"}
It has been shared with me by Bookings with me team, that any anonymous sharing policy will work, so users could use Anonymous:CalendarSharingFreeBusySimple.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7bad8864-5f08-90ae-a6d1-a23e58fc1843
Version Independent ID: 7bad8864-5f08-90ae-a6d1-a23e58fc1843
Content: Bookings with me
Content Source: microsoft-365/bookings/bookings-in-outlook.md
Service: bookings
GitHub Login: @kwekuako
Microsoft Alias: kwekua

@dkotars Thank you for your feedback. This has been address in PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9473
Once the author approves, the changes will be live.
Thanks
Sri


🔍 feat: enhance SEO and accessibility with meta tags

Summary
This PR improves the meta tags to improve SEO and accessibility.
Features

Open graph title: Use the page title, if available, for the Open Graph og:title property. Resolves #148.

Improved multilingual support: Included locale and hreflang tags for multilingual support.

Image Metadata: Introduced Open Graph image width and height metadata.

Conditional Open Graph URL: Made the og:url tag conditional based on whether current_url exists (so it doesn't get added for the 404 page).

This PR was not applying the multilingual tags on sections (like the main index). Commit 465bc55b9ab8263f55e0f83b9468a669c81b6ce8 fixes that.


onNotify never calls its callback

Hello, I'm writing the bluetooth communication part of an app with several other people, and I've been struggling for quite some time now so that's why I'm opening this issue here hoping someone can help !
I'm connecting to a peripheral that has a UART over BLE service, so I'm subscribing to notifications on the TX characteric of the UART service (and the logs correctly reports that notify is TRUE) but when I write values on the RX characteristic, I never receive anything while I should be receiving something (I'm sending an HEX command that works perfectly in native android code or also on cordova...).
Here is how I subscribe to notifications (notifications subscription works as I said earlier as I'm correctly seeing in the console JS: --- notifying - this is on Android, but works also on iOS):
public subscribeTXnotification = function () {
    //Subscribe to TX notifications
    console.log("----- Starting TX subscription");
    bluetooth.startNotifying({
      peripheralUUID: this.connectedPeripheral.UUID,
      serviceUUID: this.uartService,
      characteristicUUID: this.txUuid,
      onNotify: this.onData
    }).then(function() {
      console.log("----------- subscribed for notifications of TX Channel");
    });
  }

and the onData callback is here:
public onData = function (data) {
    console.log("read: ", JSON.stringify(data));
  }

You should note that I'm coding using TypeScript and those functions are part of a class.
Example of write call is here:
bluetooth.write(
  {
    peripheralUUID: that.connectedPeripheral.UUID,
    serviceUUID: that.uartService,
    characteristicUUID: that.rxUuid,
    value: '0xAA0400002955', //AA010400C555
  }
).then(
  function(result) {
    console.log("Done: ", JSON.stringify(result));
  },
  function (err) {
    dialogs.alert({
      title: "Whoops!",
      message: err,
      okButtonText: "Hmmkay"
    });
  }
);

Also, I tried on both iOS and Android and I get the same behaviors. Write works without printing any errors in the console but I never receive anything.

Okay, I understood the problem while looking at how your library works... it cannot work with 2 different characteristics at the same time, which is quite a problem for UART over BLE.

@farfromrefug I have this issue too. Doing it on iOS, with last version of this plugin. Everything works, except startNotify - I never get onNotify callback. I have tried to call .startNotify when button is clicked, then write value. Also tried to write value then .startNotify. Also tried right after .onConnected ... nothing.
My BLE device has notifications, because I test it with other free apps in store - there I see "Notify received ...." (right after writing).
Here is my method, that is called on button tap:
`startNotify(event) {
var service = this.connectedDevice.services[0];
  var _that = this;

  bluetooth.startNotifying({
    peripheralUUID: _that.connectedDevice.UUID,
    serviceUUID: service.UUID,
    characteristicUUID: service.characteristics[1].UUID,
    onNotify: function (result) {
      // see the read example for how to decode ArrayBuffers
      console.log("read: " + JSON.stringify(result));
    }
  }).then(function() {
    console.log("subscribed for notifications");
  });

}`

@maximedm indeed have it locally but not finished :s sorry. Will do that when i have the time.
Sorry!

@farfromrefug no problem the project is really nice the other features are working good soo keep up the good work!

@maximedm i release a beta 13. It should work my multiple char startNotifying. Let me know
Also i released a new plugin @nativescript-community/ble which will replace this one (the repo changed already) for N 7.0


[ADAM-2159] Add load methods for data frames

Fixes #2159

Coverage remained the same at ?% when pulling 86d58afd3d5296e4643b2590e70a3152cb598649 on heuermh:load-from-dataframe into 38f916f6147547545d73a74680474dd2ba21f6ee on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/3007/
Test PASSed.


Finding sets by set code pulls in all sets

The following code returns 448 results where it should only return one.
from mtgsdk import Set
sets = Set.find('ktk').all()
len(sets)

I obtained this snippet from the code examples on the repo readme https://github.com/MagicTheGathering/mtg-sdk-python/blob/master/README.md

Don't use .all() with Set.find(), or Card.find() for that matter.
The instructions on the readme say to use Set.find('ktk')
When you call .all() on the returned object it gets all sets.
Here's some IDLE output:
`>>> khans = Set.find('ktk')

khans.name
'Khans of Tarkir'
khans.code
'KTK'
len(khans.all())
448`


WiP: Support for running NTO e2e tests.

This is the first attempt at leaveraging this repo to run NTO e2e tests.
Examples:
Run NTO's e2e tests automatically based on clusterversion.
run nto e2e

Run NTO's e2e 4.7 tests.
run nto e2e origin/release-4.7

Run NTO's e2e tests for a specific NTO commit.
run nto e2e 14bdf4d4

This is the first step.  Ideas how to improve or approach this in a different way more than welcome.  Thanks!

Hello @jmencak, thanks for this first PR on this repo ;-)
the way we usually organize the code/entry path is:

ci_entrypoint_nto.sh calls a toolbox script, eg toolbox/nto/test-e2e.sh
toolbox/nto/test-e2e.sh prepare the test parameters (here commit=origin/release-$OCP_VERSION) and calls ansible
ansible performs the test steps (here, git clone + make test)

I would suggest that you follow these 3 levels, or at least 1 and 2, for consistency of the repo organization.
I think you (we) would benefit from following 3 as well, to have the logs properly organized.
By calling an ansible playbook, you'll get a dedicated directory for the test artifacts + a parsable log file of the playbook tasks:
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/logs/periodic-ci-openshift-psap-ci-artifacts-release-4.7-gpu-operator-e2e-master/1401675477935984640/artifacts/gpu-operator-e2e-master/nightly/artifacts/002__entitlement__deploy/
it's hard to say if it's really necessary for running 2 tasks, what do you think?
and we can already make it grow 50% bigger by adding a task storing the current git commit, to know immediately what was tested:
git show --quiet > {{ artifact_extra_logs_dir }}/git_commit

you can also call toolbox/cluster/capture_environment.sh to store the OCP version + ci-artifacts commit, again to know what was tested

Thank you for the valuable input, Kevin!  Hopefully adjusted closer to what you expect.  Those three "levels" seem overkill at first, but makes sense on second look.  If I missed any of your comments, happy to adjust.

Also, quick question, why did you decide to use ci-artifacts instead of directly using NTO repo as the base of the testing?
when I prepared the Prow PR it made sense to me not to use ci-artifacts, as you have only one toolbox command here. The main point of the repo is to orchestrate multiple "complex" steps required for the testing, while with NTO, you have only one "simple" (= make test) step
anyway, when we'll progress with the ci-dashboard, you'll have additional benefits for free ;-)

That's the plan. Some of the reasons I wanted to use the ci-artifacts:

I wanted to keep the openshift/release for NTO as simple as possible given the issues we had with the first version of https://github.com/openshift/release/pull/17020
I wanted to test NTO on nightly/ART builds.  I do not believe this can be easily done with openshift/release as is at the moment as there are still pending some ART Jira tickets for that
You've made a lot of progress on the ci-artifacts/ci-dashboard repos; the dashboards look cool and I'd like to leaverage that for NTO too in the future, as you say nearly "for free".

ack,

I wanted to keep the openshift/release for NTO as simple as possible given the issues we had with the first version of openshift/release#17020

indeed, makes sense

I wanted to test NTO on nightly/ART builds. I do not believe this can be easily done with openshift/release as is at the moment as there are still pending some ART Jira tickets for that

FYI, for what I see in the current code of the PR, using openshift/release would test the exact same thing
ci-artifacts is just a man in the middle, but Prow 1/ clones a git repo 2/ triggers a command side, that's what we have here

You've made a lot of progress on the ci-artifacts/ci-dashboard repos; the dashboards look cool and I'd like to leaverage that for NTO too in the future, as you say nearly "for free".

:-)
FYI again, the current dashboard works with any Prow test, I only parse the file-artifacts generated by Prow
but in the future, I would like to take into account a) the toolbox steps (000_, 001_) to show which step step failed,
b) the _ansible_logs.json, first to show "X test succeeded, Y test skipped, Z test failed", and maybe also render the logs task by task

This should be hopefully much closer to the final PR, Kevin, right? :)

OutOfMemoryKilled: The build pod was killed due to an out of memory condition.
/test sro-images

@jmencak , I tested locally, good news is that the test ran, bad news is that the test failed!
• Failure [305.058 seconds]
[basic][rollback] Node Tuning Operator settings rollback
/tmp/nto/test/e2e/basic/rollback.go:18
  Tuned settings rollback
  /tmp/nto/test/e2e/basic/rollback.go:26
    net.ipv4.tcp_tw_reuse set [It]
    /tmp/nto/test/e2e/basic/rollback.go:40

    Unexpected error:
        <*errors.errorString | 0xc0006d21e0>: {
            s: "sysctl net.ipv4.tcp_tw_reuse=2 in Pod tuned-vx7bx, expected 1: command [sysctl -n net.ipv4.tcp_tw_reuse] outputs 2 (leading/trailing whitespace trimmed) in Pod tuned-vx7bx, expected 1: <nil>",
        }
        sysctl net.ipv4.tcp_tw_reuse=2 in Pod tuned-vx7bx, expected 1: command [sysctl -n net.ipv4.tcp_tw_reuse] outputs 2 (leading/trailing whitespace trimmed) in Pod tuned-vx7bx, expected 1: <nil>
    occurred

    /tmp/nto/test/e2e/basic/rollback.go:71
------------------------------
SSSSSSSS

Summarizing 1 Failure:

[Fail] [basic][rollback] Node Tuning Operator settings rollback Tuned settings rollback [It] net.ipv4.tcp_tw_reuse set
/tmp/nto/test/e2e/basic/rollback.go:71

Ran 1 of 9 Specs in 305.058 seconds
FAIL! -- 0 Passed | 1 Failed | 0 Pending | 8 Skipped
--- FAIL: TestNodeTuningOperator (305.06s)
FAIL
FAIL	github.com/openshift/cluster-node-tuning-operator/test/e2e/basic	305.073s
FAIL
make: *** [Makefile:91: test-e2e] Error 1

apart from that, looks good !

@jmencak , I tested locally, good news is that the test ran, bad news is that the test failed!
• Failure [305.058 seconds]
[basic][rollback] Node Tuning Operator settings rollback
/tmp/nto/test/e2e/basic/rollback.go:18
  Tuned settings rollback
  /tmp/nto/test/e2e/basic/rollback.go:26
    net.ipv4.tcp_tw_reuse set [It]
    /tmp/nto/test/e2e/basic/rollback.go:40

    Unexpected error:
        <*errors.errorString | 0xc0006d21e0>: {
            s: "sysctl net.ipv4.tcp_tw_reuse=2 in Pod tuned-vx7bx, expected 1: command [sysctl -n net.ipv4.tcp_tw_reuse] outputs 2 (leading/trailing whitespace trimmed) in Pod tuned-vx7bx, expected 1: <nil>",
        }
        sysctl net.ipv4.tcp_tw_reuse=2 in Pod tuned-vx7bx, expected 1: command [sysctl -n net.ipv4.tcp_tw_reuse] outputs 2 (leading/trailing whitespace trimmed) in Pod tuned-vx7bx, expected 1: <nil>
    occurred

    /tmp/nto/test/e2e/basic/rollback.go:71
------------------------------
SSSSSSSS

Summarizing 1 Failure:

[Fail] [basic][rollback] Node Tuning Operator settings rollback Tuned settings rollback [It] net.ipv4.tcp_tw_reuse set
/tmp/nto/test/e2e/basic/rollback.go:71

Ran 1 of 9 Specs in 305.058 seconds
FAIL! -- 0 Passed | 1 Failed | 0 Pending | 8 Skipped
--- FAIL: TestNodeTuningOperator (305.06s)
FAIL
FAIL	github.com/openshift/cluster-node-tuning-operator/test/e2e/basic	305.073s
FAIL
make: *** [Makefile:91: test-e2e] Error 1

apart from that, looks good !

Interesting, tested locally 3 times, no failures. :)

thanks @jmencak looks good :D
\lgtm

\approve

thanks @jmencak looks good :D
/lgtm
/approve


Master to MRLearningSharing

Docs Build status updates of commit ffe3751:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/mixed-reality, working directory: W:\smps-s, target branch: MRLearningSharing, target branch commit: be528346676546e0397779918f5c856ab55956d5, source commit: ffe3751342b2bd127218753f4699221a4d95f7db.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit ff20042:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/mixed-reality, working directory: W:\tp2a-s, target branch: MRLearningSharing, target branch commit: be528346676546e0397779918f5c856ab55956d5, source commit: ff20042e76ae0c1fe9f203f72213bdf912a25f51.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 7a14180:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/mrlearning-asa-ch4.md
:warning:Warning
View
Details

mixed-reality-docs/Configure-Unity-Project.md
:white_check_mark:Succeeded
View

mixed-reality-docs/about-this-design-guidance.md
:white_check_mark:Succeeded
View

mixed-reality-docs/design.md
:white_check_mark:Succeeded
View

mixed-reality-docs/direct-manipulation.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze-and-commit.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hands-free.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hololens-hardware-details.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/Module2chapter1step1im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step10aim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step10bim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step10cim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step10dim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step10eim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step10fim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step11im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step12im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step3im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step4im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step5aim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step5bim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step5cim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step6aim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step7im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter1step8im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter2step1im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter2step2aim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter2step2bim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter2step2cim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter2step5im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter2step6aim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter2step6bim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter3step1im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter3step2im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter3step3im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter3step4im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter3step5im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter3step6im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter3step7im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter3step8im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter3step9im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter4note2im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter4note3im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter4noteim.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter4step10im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter4step11im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter4step12im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter4step13im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter4step15im.PNG
:white_check_mark:Succeeded
View

mixed-reality-docs/images/module2chapter4step16im.PNG
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.

mixed-reality-docs/mrlearning-asa-ch4.md

Line 172: [Warning] Invalid file link:(~/images/Module2chapter4step24im.png).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


RSA example

enable 52-, 40-, 28- and 16-bit range checks with Gadgets.rangeCheck64()
expose Provable.witnessFields()
add RSA example

I doubt this will land soon, while the core functionality is there, this needs to be polished, tested and documented. Wdyt @mitschabaude

Is there anything here in particular that you want landed @emlautarom1?
Note that RSA is just an example, not part of the library


DBReff

Hello,
How can I implement DBRef mechanism using laravel-mongodb?
And for ex. if I have array of DBRef in Document how can I add/remove it using Laravel 5 mechanisms?

:+1:  I have the same question.
I  think you mean http://docs.mongodb.org/manual/reference/database-references/#dbref-explanation
I would use the same feature. But think this feature is not available in this module.

Link


Poor handling of non-number characters

I accidentally pasted (3/46.879/22.164 into the zoom/lat/long field in Maperture. After pressing enter, it looked like this (which is fine):

The problem is that after that, clicking change doesn't do anything, so there's no way to correct it and enter the location I want. FWIW, pressing copy puts the string (NaN/46.879/22.164) in the clipboard but doesn't change the text to copied.

Ah, good catch here. We should definitely validate the input before setting it and revert if it's invalid!


Database_URL on Yarn Build

[    ] info  - Generating static pages (0/7)(node:20666) UnhandledPromiseRejectionWarning: Error: error: Environment variable not found: DATABASE_URL.
  -->  schema.prisma:3
   |
 2 |   provider = "postgresql"
 3 |   url      = env("DATABASE_URL")
   |

Validation Error Count: 1
    at Object.loadEngine (/home/pi/6tbdata/Void/node_modules/@prisma/client/runtime/index.js:35582:19)
    at async Object.instantiateLibrary (/home/pi/6tbdata/Void/node_modules/@prisma/client/runtime/index.js:35511:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:20666) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:20666) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
info  - Generating static pages (7/7)
info  - Finalizing page optimization

In the config.toml the database url is set, but I get this when running yarn build, using node 14.19.3 and yarn 1.22.18

Just ignore it, that does not matter. It's a bug which is fixed in v1.

it does matter as I cannot login because of it
when I try to login I get this error:
PrismaClientInitializationError:
Invalid `prisma.user.findFirst()` invocation:

  Database `voiduploader.public` does not exist on the database server at `localhost:5432`.
    at Object.request (/home/pi/6tbdata/Void/node_modules/@prisma/client/runtime/index.js:39815:15)
    at async PrismaClient._request (/home/pi/6tbdata/Void/node_modules/@prisma/client/runtime/index.js:40637:18)
    at async handler (/home/pi/6tbdata/Void/.next/server/pages/api/auth/login.js:30:16)
    at async Object.apiResolver (/home/pi/6tbdata/Void/node_modules/next/dist/server/api-utils.js:101:9)
    at async Server.handleApiRequest (/home/pi/6tbdata/Void/node_modules/next/dist/server/next-server.js:775:9)
    at async Object.fn (/home/pi/6tbdata/Void/node_modules/next/server/next-server.ts:985:24)
    at async Router.execute (/home/pi/6tbdata/Void/node_modules/next/server/router.ts:346:21)
    at async Server.run (/home/pi/6tbdata/Void/node_modules/next/server/next-server.ts:1239:20)
    at async Server.handleRequest (/home/pi/6tbdata/Void/node_modules/next/dist/server/next-server.js:292:20) {
  clientVersion: '3.10.0',
  errorCode: undefined
}

Try prisma migrate deploy and then prisma db seed. Use npx or yarn, depending on your PM.

Closed due to inactivity. Feel free to reopen if you encounter other issues.


(PAR): move all paths to a signle section

This is an early rfc PR, so do not merge.
Rust paths are tricky, because there are several different flavours of them. This tries to consolidate all paths in a single place and avoids

using path_expr in place of path
reimplementation of qualified paths in several places

The code is mostly aligned with parser.rs .
It totally breaks current resolving :)
@Atsky @alexeykudinkin what do you think?

What are the benefits? Beyond unification.
P.S. It can't break resolve -- you can't break something that doesn't work.

I guess unifications are the main benefit: unification within rust.bnf and also unification with the reference parser.
I also don't like current overloading of path_exr for patterns and struct expressions.

For me looks reasonable. But I don't have fully understanding of Rust grammar to see pitfalls.

changes are added in https://github.com/alexeykudinkin/intellij-rust/commit/5ad0850083fc3e3c5916bd05ed8c1650b3631c30


[SPARK-45464][CORE] Fix network-yarn distribution build

What changes were proposed in this pull request?
Exclude an apache license in the packaged jar to fix the build for network-yarn. Also shade the native libraries so they can be used.
Why are the changes needed?
Making a distribution build using the following command currently fails after https://github.com/apache/spark/pull/43164:
./dev/make-distribution.sh --tgz  -Phive -Phive-thriftserver -Pyarn

Failures look like
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:3.1.0:run (unpack) on project spark-network-yarn_2.13: An Ant BuildException has occured: Error while expanding /Users/hasnain.lakhani/spark/common/network-yarn/target/scala-2.13/spark-4.0.0-SNAPSHOT-yarn-shuffle.jar
[ERROR] java.nio.file.FileSystemException: /Users/hasnain.lakhani/spark/common/network-yarn/target/exploded/META-INF/license/LICENSE.aix-netbsd.txt: Not a directory
[ERROR] around Ant part ...<unzip src="/Users/hasnain.lakhani/spark/common/network-yarn/target/scala-2.13/spark-4.0.0-SNAPSHOT-yarn-shuffle.jar" dest="/Users/hasnain.lakhani/spark/common/network-yarn/target/exploded/" />... @ 5:198 in /Users/hasnain.lakhani/spark/common/network-yarn/target/antrun/build-main.xml
[ERROR] -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:3.1.0:run (unpack) on project spark-network-yarn_2.13: An Ant BuildException has occured: Error while expanding /Users/hasnain.lakhani/spark/common/network-yarn/target/scala-2.13/spark-4.0.0-SNAPSHOT-yarn-shuffle.jar
java.nio.file.FileSystemException: /Users/hasnain.lakhani/spark/common/network-yarn/target/exploded/META-INF/license/LICENSE.aix-netbsd.txt: Not a directory
around Ant part ...<unzip src="/Users/hasnain.lakhani/spark/common/network-yarn/target/scala-2.13/spark-4.0.0-SNAPSHOT-yarn-shuffle.jar" dest="/Users/hasnain.lakhani/spark/common/network-yarn/target/exploded/" />... @ 5:198 in /Users/hasnain.lakhani/spark/common/network-yarn/target/antrun/build-main.xml

After much trial and error (I added a debugging section for the next person who runs into this) I realized the problem is that inside the jar, the META-inf folder looks like this:
META-INF/LICENSE
META-INF/LICENSE.txt
META-INF/license/
META-INF/license/LICENSE.aix-netbsd.txt
META-INF/license/LICENSE.boringssl.txt
META-INF/license/LICENSE.mvn-wrapper.txt
META-INF/license/LICENSE.tomcat-native.txt

Since the filesystem on macosx is case insensitive, this causes issues. Unzipping this fails, as we unzip the file first, then do not create the directory, and fail to unzip the files after that.
This PR proposes to fix it by removing the LICENSE file, as suggested, by adding it to the exclusion list, so the folder gets copied. The contents are the same as LICENSE.txt (which is also just the apache2 license) so this should be fine. We also shade and copy over the new native libraries similar to the existing ones.
Does this PR introduce any user-facing change?
No
How was this patch tested?
CI
I reran
./dev/make-distribution.sh --tgz  -Phive -Phive-thriftserver -Pyarn

and it succeeded
Was this patch authored or co-authored using generative AI tooling?
No
Appendix: debugging
I was able to debug the contents of the jar using jar tf $jar and print out the files using unzip -q -c $jar $path.
To make iterations go faster I went to /Users/hasnain.lakhani/spark/common/network-yarn/target/antrun and edited the ant file by hand then ran ant -f build-main.xml

cc: @wangyum @LuciferYang please let me know if this fix works for you

Another question is, are we sure that network-yarn needs to shade the netty-tcnative-boringssl-static related jar packages? If indeed necessary, should we rename libnetty_tcnative_* in a similar way to libnetty_transport*?

I am not sure if it needs to shade them. I'll admit I'm not the most familiar with this aspect of things, happy to try it without. I will say that even with the shading it does seem to work fine (tests in https://github.com/apache/spark/pull/42685 pass) so aside from this issue I am not sure about the harm of keeping it shaded for consistency

This depends on whether the YarnShuffleService needs the tcnative-boringssl related functions. If so, it indeed needs to be shaded, and libnetty_tcnative_* should likely be renamed similarly; if not, these jars can be directly excluded from the shade.

This depends on whether the YarnShuffleService needs the tcnative-boringssl related functions. If so, it indeed needs to be shaded, and libnetty_tcnative_* should likely be renamed similarly; if not, these jars can be directly excluded from the shade.

If it's not provided, the functionality will still work, it'll just be slower (falling back to the JDK implementation). I think for now for simplicity's sake (especially since I don't have a great setup for testing yarn beyond unit tests) I think it makes sense to exclude from the shading, that seems simpler. Will try that out

@hasnain-db I suggest following @LuciferYang's suggestion to follow the existing rule to rename (the prefix must exactly match the relocated package name) the newly introduced native library, it's actually a standard way of creating artifacts including shaded Netty classes and native libraries.

I've updated to rename the libraries + use the exclusion, PTAL

we may consider adding these renaming rules to the SBT build.
$ ./build/sbt -Pyarn network-yarn/assembly
$ jar tf common/network-yarn/target/scala-2.13/spark-network-yarn-4.0.0-SNAPSHOT-hadoop3.3.6.jar | grep libnetty
META-INF/native/libnetty_transport_native_epoll_aarch_64.so
META-INF/native/libnetty_transport_native_epoll_x86_64.so
META-INF/native/libnetty_transport_native_kqueue_aarch_64.jnilib
META-INF/native/libnetty_transport_native_kqueue_x86_64.jnilib

libnetty_transport_native_kqueue_x86_64

Yes, this needs to be fixed. However, inconsistencies between sbt assembly and maven shaded already existed before adding boringssl , and there are likely more. The main goal of this pr is to fix make-distribution.sh, so I personally prefer to address the differences between sbt assembly and maven shaded in a series of separate PRs.

@LuciferYang the docker integration tests are consistently failing with
[info] *** 2 SUITES ABORTED ***
[error] Error during tests:
[error] 	org.apache.spark.sql.jdbc.v2.OracleIntegrationSuite
[error] 	org.apache.spark.sql.jdbc.OracleIntegrationSuite

I don't know if they are flaky but it seems to be unrelated to this PR

@LuciferYang the docker integration tests are consistently failing with
[info] *** 2 SUITES ABORTED ***
[error] Error during tests:
[error] 	org.apache.spark.sql.jdbc.v2.OracleIntegrationSuite
[error] 	org.apache.spark.sql.jdbc.OracleIntegrationSuite

I don't know if they are flaky but it seems to be unrelated to this PR

https://github.com/apache/spark/pull/43123#issuecomment-1752118564
friendly ping @dongjoon-hyun, it seems like this is an unstable case? I noticed that you have also encountered similar failures.
@hasnain-db Can you manually re-trigger Run Docker integration tests?

Yes, re-trigger will help. I saw the same situation before after upgrading the recent Oracle image, but didn't check the root cause yet, @LuciferYang .

Thank you for your confirmation @dongjoon-hyun  ~

failed again, retrying once more

Merged into master for Spark 4.0, thanks @hasnain-db @srowen @dongjoon-hyun @wangyum @cfmcgrady

np - thanks for merging!


Use (hopefully) fixed version of sidekiq-unique-jobs

Currently we're seeing a problem where job IDs are not removed from the
uniquejobs hash in Redis, leading to Redis memory running out.
The change in my version of the gem is to add the following method and call it from Unlockable#after_unlock:
    def ensure_job_id_removed
      Sidekiq.redis { |redis| redis.hdel("uniquejobs", jid) }
    end

If this works in integration, we can raise a pull request against the sidekiq-unique-jobs gem.
Alternative solutions:

Ensure job uniqueness via some other means (Sidekiq Pro has it built in)
Run a nightly job that clears out the uniquejobs hash.

Looks good :) but lets test this on integration before merging.


[WIP] Introduce writeLoop

Description
To solve slow-reader problem, writeLoop (goroutine) has been introduced.
This writeLoop routine is responsible for calling a.send() for all outgoing packets.
Reference issue
Fixes #40

@backkem can you try this branch with your libp2p test on CI?

Fantastic work @enobufs
Anything I can do just tell me @backkem @albrow!
getting ipfs unblocked (then announcing v2.0.0) is going to be huge. Would love to have more active users! been getting a lot of issues/asks all the sudden but not many more contributors

Hey @backkem would you mind checking out this PR? I would love to know when you feel good about it so we can release v2.0.0 I have pushed it back about a month at this point.
Or we can decouple the release from these changes.
thanks

I really hope that this could land in v2.0.0 as soon as possible because the performance with the current one on master is pretty bad. (storm of control packets with slow reader causing a huge backlog in ice buffer)... my heart racing...


Devel

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.592%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
1536

Relevant Lines:
1677

💛  - Coveralls


fw_pos_control_l1 move to new uORB::Subscription

Tested on Pixhawk 1 V3 fixed wing
Modes Tested:
mission Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.
Notes:
No issues were noted, good flight in general.
log:
https://review.px4.io/plot_app?log=52beff5d-2b7b-4a80-aba2-9dec1c1489b5


Use core libraries and remove dependencies

I am not too much familiar with lunny/log and jehiah/go-strftime but it seems that we could drop them and rely on core libs.

Thanks!


Preferred Phone saving invalid setting

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
User is allowed to save incorrect state:
Enable Enhanced Preferred Phone Functionality - unchecked
Disable Preferred Phone enforcement - checked

What is the expected behavior?
If "Enable Enhanced Preferred Phone Functionality" is unchecked, user cannot save "Disable Preferred Phone enforcement" checked.

There are multiple possible ways to prevent this, including:

automatically unchecking second checkbox when first checkbox is checked
preventing save by giving an error message
...

**lurch: add

Tracking W-036340


Added support to custom JSON logging

Added support to custom JSON
Added support to tags
Added support to configure console errors
Bug fixes

Hey @ajbrown
Reverted all the modifications, then pulled your changes and my changes.

@psquickitjayant looks like the tests are failing.  Can you take a look?

@ajbrown  Seems all worked well.

@ajbrown  We can also add test cases for the plain text, custom JSON and console errors as I have added checks for each of these. Also we can make a separate test case for the identifier "sendConsoleErrors". You can know more about sendConsoleErrors by looking into the ReadMe file.

Thanks for this!


[BLOCKED] Update invoke task and add "requires" entry

@csadorf Is it fine to add metadata.requires entries like this? See the change here.

Please do not merge this until after today's meeting.

I am going to archive this repo, so the PR is closed.


Converting monkies to revolutionaries should not make them angrily murder you and your entire family.

Description
Using a flash to convert an NPC monkey to revolutionary as headrev will trigger the monkey's self defense protocols.
If monkies can be converted (which I believe is a fun feature), they should not murder you for doing so.
Reproduction
Be headrev
Convert monkey
screams
Additional context
I feel bad for the other botanists in this round.

I discovered this a few days ago, I found it really funny.

monkey not like brainwashing light!

I died to a monkey while trying to butcher it the other day. I think chefs are gonna have to ask sec for handcuffs now.


Bring back EntityDecoder[F, ByteVector]

As there used to be one https://github.com/http4s/http4s/blob/v0.16.6/core/src/main/scala/org/http4s/EntityDecoder.scala
Should I create a PR for individual series/x branches, or will this be propagated there from master?

The progression is series/0.22 -> series/0.23 -> main, but I can cherry-pick.

This will be in 0.22.  I'll need to adjust the constraints back to what you had when it merges forward.


Howling Blast coefficient

Describe the bug
Howling Blast AP coefficient is 0.1 in the code but should be 0.2
To Reproduce
I detailed this and other ability tests in #beta-findings on the discord but in short, make a talent spec that reaches HB with no modifiers, cast naked and in full gear noting the APs, coeff = (dmgGeared - dmgNaked) / (APgeared - APnaked), modulo dmg range.  I verified it's still the same on build 45189
Expected behavior
AP coeff 0.2 instead of 0.1

Fixed.


On Removed never called

I have this
var options = {
                cellHeight: 200,
                width: 5,
                height: 4,
                verticalMargin: 4,
                float: true,
                removable: '.remove-area',
                removeTimeout: 100
            };
            $('.grid-stack').gridstack(options);

that initialises the grid the way i want it...fine...
I have an div with a class .remove-area
the drag and drop to remove procedure is working, if i drag a widget it gets removed, but...
if i do this (as described in the docs)
$('.grid-stack').on('removed', function(event, items) {
                console.log(items);
            });

this function never gets called.
Any ideas?

Same problem here.

You can bind a jquery remove event on the child elements.
e.g $(".grid-stack-item").on("remove",function(){
console.log(this)
})
something like that should do the trick. If you are reference it a object just assign a id on each element. Which you can do with the addWidget function (last param).

I did a nasty workaround, tweaked onEndMoving() in the gridstack.js
 if (node._isAboutToRemove) {
                forceNotify = true;
                el.removeData('_gridstack_node');
                el.trigger('widget-removed') // My line
                el.remove();
            }

and then
        $('.grid-stack').on('widget-removed', '.grid-stack-item', function() {
            console.log($(this));
        });

That way it work on dynamic widgets

@MarkoIvanetic It's working,thank you ~~~~,so that i can get the data-gs-id and do my operating

@MarkoIvanetic Nice solution, i allowed me to find the problem.
.remove() in jquery doesn't trigger the .on("remove") event. you have to explicitly call .trigger('remove')
You don't need to create a new trigger. you just need to use the existing one like this.
if (node._isAboutToRemove) {                 forceNotify = true;                 el.removeData('_gridstack_node');                 el.trigger('remove').remove();             }
and then in your code do this
$('.grid-stack').on('remove', '.grid-stack-item', function() {             console.log($(this));         });
This fixes the problem without adding a new trigger. I'm going to create a pull request for this and hopefully it will be included in the next version. The fact is that i want to use this trough a cdn and don't want to host the custom .js locally. But for now it will do.
Thank you for your help.

@RobertoBiundo good job man. without creating new trigger.

Still now that it's modified i cant use it with bower.

Thanks this is a better solution :).

Just to let you guys know that i already created a pull request and hopefully this will be included soon and @MarkoIvanetic can start using it with bower.
https://github.com/troolee/gridstack.js/pull/622

Fixed in develop branch. Will go live in v0.3.0 later today.


Misspelled Material name

Mythril is not spelled with a Y. It should be spelled with an I instead.

Intentional


fix: change pointer events timing, update tests

Fixes vaadin/vaadin-context-menu#214

Does not appear to fix the issue in Firefox 66


Prepare haddock code for new Language

With the introduction of GHC2021, the Languages data type in GHC will grow. In preparation of that (and to avoid changing haddock with each new language), this change makes the code handle extensions to that data type gracefully.

@nomeata Hi! Thank you very much for making Haddock future-proof. :)
Since you've branched from ghc-head, the Github Action CI will not be triggered, and instead we will be trying to run the Travis CI one (which will most certainly fail).
Could you re-open it from the ghc-9.0 branch? Or ghc-8.10?

@nomeata Hi! Thank you very much for making Haddock future-proof. :)
Since you've branched from ghc-head, the Github Action CI will not be triggered, and instead we will be trying to run the Travis CI one (which will most certainly fail).
Could you re-open it from the ghc-9.0 branch? Or ghc-8.10?

Rebased and changed base

Rebased and changed base

Will this find its way to ghc-head, so that my patch against GHC can build on it?

Will this find its way to ghc-head, so that my patch against GHC can build on it?


Remove EXPOSE completely

Hopefully this is temporary -- apparently folks are using EXPOSE as a rough guide for doing healthchecks? (https://gitlab.com/gitlab-org/gitlab-runner/issues/4501#note_194904002)

I think this would create more fallout than even #170 did...
force-disable the gitlab team could do across the whole fleet last night, so most people never saw it https://gitlab.com/gitlab-org/gitlab-runner/issues/4501#note_194648520
for "ci failures cause buildscript starts before service is ready" there is nothing todo globally.
And given that this image does NOT have a HEALTHCHECK, whats the alternative?
exposing "both ports when one will never come up" seems kinda wrong indeed, but so does having no expose...
((and I dont think gitlab is alone here, docker run -P, portainer & co...))

There's quite a difference between docker run -P (which simply maps the EXPOSE ports to external ports so they become accessible if they're actually used) and assuming the ports will be listened on.
In most images/software, the port that's listened on is configuration, so it depends on more factors just as it is here.  With our stock configuration, we'll listen on 2376, but it's one trivial configuration later and it's 2375, or even another and it's not listening externally at all, so EXPOSE cannot possibly be more than a guideline for what ports might be useful.
Regarding HEALTHCHECK, it has similar issues -- if we did add a healthcheck by default, which port would it check?  It's not really reasonable for us to provide a generic healthcheck this (https://github.com/docker-library/faq#healthcheck).

and assuming the ports will be listened on.

sure sure ;-) - thats more regarding having no expose...

HEALTHCHECK, it has similar issues

for 18.x+ something like attempting docker system dial-stdio might be acceptable?
but I see your issue and the generally missing "ready" state in the docker-system is a well known...

to add to this from IRC, "by the book" i think there should also be a VOLUME /certs,
I agree with your "why not", but then it also gets as far as almost granting 2 images...

*-dind-tls with volume and 2376
*-dind-tcp with 2375

@tianon Yes, we use port information for doing health checks because of the nature of GitLab CI services. We define services as a network accessible service that can be use by your job.
And to make users live easier, we check if the service is listening before we start a script that is using it. In case if it's not (or we were unable to check this), we print the warning. So in case of CI job failure, users have an information about the initial service state and can use this information while deciding what caused the job to fail.
For this however we need to somehow decide how to access the service. In the past, when we've introduced the services, there was no HEALTHCHECK mechanism implemented in Docker. The configuration of services in .gitlab-ci.yml file was also very simple - you could only specify which image to use. Using information about exposed ports was the only way to handle this. And to be precise, GitLab Runner's health check isn't relaying directly on the EXPOSE. It's relaying on the ..._TCP_ADDR variables that are added to linked containers. But if the container (in this - the DinD service) doesn't specify port mapping by itself, the EXPOSE ports are mapped by default.
Nowadays we have HEALTHCHECK, which could be definitely used - in that case we'd depend fully on the image author and need to believe, that the implemented health check is really checking all that is required. We have also an extended syntax for services configuration, so we could allow users to explicitly define which ports should be used for the health check implemented in the Runner.
Usage of both of these features is already something that we're considering, please check https://gitlab.com/gitlab-org/gitlab-runner/issues/3984.
I hope that the above gives a little light on why we've decided to use this information and answers your note from this PR:
# TODO figure out why folks are using EXPOSE for health checks (https://gitlab.com/gitlab-org/gitlab-runner/issues/4501#note_194904002)

If I could ask, please postpone a while with removing the EXPOSE. It doesn't change much for you, but in case of GitLab CI it will be a huge change. Now, the service startup failure warning is presented randomly - depending on which port the user is using and which one was the first found by the health check mechanism. If you will remove the EXPOSEs, it will fail the health check constantly.

I was looking at removing EXPOSE to try and help the GitLab case and we held off when it looked like this wouldn't fix anything, so I'm going to close this instead. 😅👍

In the current case, is the code in runner looking for both ports to be open, or just one? Is it choosing one randomly, deterministically, or trying all until one is listening?

@tianon Runner is scaning environment variables, looking for the ones that have _TCP_ADDR and _TCP_PORT suffixes. The first one found is chosen. And since there is no guarantee on variables ordering and if no explicit port mapping is done all ports from EXPOSE are exposed, the finding is done totally randomly.
It's a very naive logic, but in most cases (where service images don't expose multiple ports) it just works and was the simplest thing that we could do in the past.
This case showed how risky assumption it is and I hope we'll find some time soon to work on an extended support for the health checks.
Also thank you very much for your activity in helping us resolve problems between GitLab CI and DinD image! :)

So perhaps a smaller (intermediate) solution could be to scan all available ports instead of just the first discovered and wait for any of them to open?
It's my pleasure, you folks are great to work with and I enjoy GitLab's product (even spending most of my days using your primary competitor :sweat_smile:).


fix: missing exported types

Issue:
Closes https://github.com/emilkowalski/sonner/issues/455
What has been done:
Added these exports from src/index.tsx:
export { type ToastClassnames, type ToastToDismiss, type Action } from './types';

And the interface Action is now exported from src/types.ts
export interface Action { .. }

Validation
The issue reported in #455 is now resolved for me.

Hey @emilkowalski thanks for merging, could we get a patch version pretty please? 🙏🙏🙏 I have a dependent library that can't upgrade because of this.


Silence unused variable warning in Boost

Summary
Closes #3017
Tests
None
Side Effects
None
Release notes
Silenced an unused variable warning in Boost.
Checklist

[x] Copyright holder: Simons Foundation
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

I've opened a PR upstream with a different fix: https://github.com/boostorg/math/pull/1078
Which is just changing the assert to:
      BOOST_MATH_ASSERT(
         std::numeric_limits<T>::epsilon() < a
         && a < std::numeric_limits<T>::infinity());

I'm happy to either approve and merge this DISABLE_ASSERTS approach now, or wait until upstream comments on that fix and use that instead, just let me know

Either seems fine with me


Made it compile and work on Qt4.8, bar some tests.

The proposed changes make the library compile and work also on Qt4.8, except for some of the tests, for I didn't put much effort into understanding how to backport them. The missing tests are:

TestQFilesystemHandler
TestQHttpHandler
TestQHttpServer
TestQHttpSocket
TestQIODeviceCopier
TestQObjectHandler

I had to fix the class QIByteArray because of the change in constructor semantics between Qt 4.8 and 5.1.
The other three tests thus compile and pass.
The examples work too, as far as I could see.
I did my best to reduce the changes at a minimum, but it's the first time I work with CMake so the odds I might have done something not quite smart are not quite low.
The backport relies on two external projects which are included in the repository as submodules and automatically updated when CMake is run. These projects are:

qjson-backport
mimetypes

Awesome! I'll take a look when I get home this evening.

Thank-you so much for contributing some code. I would love to add support for Qt 4.x but there are a couple of issues with the pull request that would need to be addressed in some way before I could merge it:

Updating Git submodules from CMake is something I would like avoid since it prevents the library from building on machines that aren't connected directly to the Internet.

Using submodules breaks the daily build over on Launchpad. The code is imported from Git -> Bazaar, which doesn't support submodules. (In fact, it won't import a Git repository if any part of the history contains submodules.)

My suggestion for avoiding both of these problems is to simply copy the code from those two repositories into the source tree. They are both licensed under the LGPL. Neither has been updated in the last 2-3 years so there isn't a lot of benefit in linking upstream anyway.
As far as Launchpad goes, I think you can work around the problem by removing the submodules and squashing your commits (using git rebase -i). You will then need to use the -f flag when pushing.
Thanks again!

please, replace cmake to qmake


Default props for toasts

Description
Hey friends,
you are doing an amazing job with this library, so thank you very much first of all!
I yesterday tried to find out if there is a way to set default props for my toasts. Specifically for the placement it would be amazing to have, so I don't always have to pass that value manually.
Proposed Solution or API
I remember there is a prop in the ToastProvider for ChakraUI where you can set default values. Would be nice to have something similar here too!
Additional Information
No response

Default Toast options will be available with the next release: https://ark-ui.com/docs/react/components/toast#default-options


CSS prefix

What do you think about changing the prefix class or making it configurable.
I prefer using:
g- instead of h-.
See also our BEM concept
Also you could change the variable name from $helper-css-prefix to $grid-css-prefix

I am not sure if I understand you correctly where did we use h- ?

Overlooked the prefix option. Solves my question.


Roll Flutter Engine from effea50196ca to 35fb29b1b54a (1 revision)

https://github.com/flutter/engine/compare/effea50196ca...35fb29b1b54a
2023-06-12<EMAIL_ADDRESS>Roll Skia from c4624cc7135a to 943f70d6f836 (2 revisions) (flutter/engine#42754)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


How to get the file offset

When processing files on SSD in Bam, GPU needs to obtain theFile Offsetof the file on SSD. The File Offset may not be the same on each disk. How can I obtain the File Offset?

BaM uses physical addressing to access data on the SSD. This implies that when you write the data on the disk, you are responsible for determining where to place the data. This physical address location would be your file offset.
There are two ways to write the data that BaM supports.

You can do replication - then the physical address is constant across SSDs.
You can do striping (RAID-like) - then the physical address required to be calculated on where you have placed it accurately using the striping factor and size of the data you are writing.

Okay, thank you! If it's just different files on a single SSD, how do I get the offset of each file? Is there any tools that can obtain this offset value?
I have tried many Linux and NVMe commands to obtain file addresses, but none of them are quite accurate.

BaM requires nvme driver to be in the userspace. So none of the Linux or NVMe commands should work. Remember BaM uses NVMe as memory and NOT AS FILES. So you are in control of where you will be writing the data in the SSD.

Thanks! Using nvm-readwrite-bench allows direct writing of data to the specified location on the SSD. In addition, when writing data, it is important to avoid writing to the file header section to avoid any inconvenience during reading.


IAM: PolicyStatement: document Action or NotAction required

When you forget the Action in a policy-statement, pulumi preview doesn't fail. I didn't find out (in reasonable time) where I could add the verification or linter inside the code. Therefore at least mentioning it in the documentation is a step forward.
Questions which arose for me as a quite new contributor (also asked in #contribute Slack-channel):

I've seen the PolicyStatement interface & others are generated code. In that PR https://github.com/pulumi/pulumi-aws/pull/232 it isn't mentioned how the code-generation was done. My guess it's generated from TF's datasource aws_iam_policy_document. Is that correct? Is this procedure documented?
If it's based from these datasource: Would it make sense to add an ExactlyOneOf there (currently it's just "actions": setOfString)? Or ...
Is there an equivalent setting for ExactlyOneOf in Typescript or Pulumi's API?

At the end I would like that at least pulumi preview would fail with a proper error-message on this. Even better a pulumi lint which directly hints the bug inline in the IDE :-)

@Dominik-K Sorry for the super long delay in replying to this. I think this PR somehow slipped through the cracks. The file in question is handwritten, so you could add the documentation as you show. Are you interested in still making this docstring edit?

@leezen Sure. That short kind of documentation is better than not.
However, it would be nice if you could answer my questions in here (which might be found by other contributors, too).

I've seen the PolicyStatement interface & others are generated code. In that PR #232 it isn't mentioned how the code-generation was done. My guess it's generated from TF's datasource aws_iam_policy_document. Is that correct? Is this procedure documented?

The interfaces in documents.ts are not automatically generated and are hand authored. Generated code should include a warning at the top, e.g. https://github.com/pulumi/pulumi-aws/blob/master/sdk/nodejs/iam/getPolicyDocument.ts shows

Is there an equivalent setting for ExactlyOneOf in Typescript or Pulumi's API?

I don't think there's that exact equivalent in TypeScript. Probably we'd have to do something along the lines of:
Policy = NotActionPolicy | ActionPolicy where those then have just one of the fields.

/run-acceptance-tests


Failed to connect to peripheral (intermitent)

Hi, very nice work done! ;)
It is working, but many times I get the error "Failed to connect to peripheral"
I have the sensor close to the raspberry and here is part of the log:
2021-08-16 14:25:12 INFO     TH_IDX:23
2021-08-16 14:25:18 INFO     Temp: 28.8
2021-08-16 14:25:18 INFO     Hum: 66
2021-08-16 14:25:18 INFO     Voltage: 2.837
2021-08-16 14:25:18 INFO     Battery level: 74
2021-08-16 14:25:18 INFO     Comfort type: 1
2021-08-16 14:25:18 INFO     The request is http://<IP_ADDRESS>:8080/json.htm?type=command&param=udevice&idx=23&nvalue=0&svalue=28.8;66;1&battery=74
2021-08-16 14:25:18 INFO     The response is <Response [200]>
2021-08-16 14:30:02 INFO     ***************************************
2021-08-16 14:30:02 INFO     Start script...
2021-08-16 14:30:02 INFO     Input parameters:
Domoticz Server IP: <IP_ADDRESS>
Domoticz Server Port: 8080
Domoticz Server User:
Domoticz Server Password:
2021-08-16 14:30:12 INFO     TH_IDX:23
2021-08-16 14:30:19 ERROR    Failed to connect to peripheral A4:C1:38:1D:69:41, addr type: public
2021-08-16 14:35:02 INFO     ***************************************
2021-08-16 14:35:02 INFO     Start script...
2021-08-16 14:35:02 INFO     Input parameters:
Domoticz Server IP: <IP_ADDRESS>
Domoticz Server Port: 8080
Domoticz Server User:
Domoticz Server Password:
2021-08-16 14:35:12 INFO     TH_IDX:23
2021-08-16 14:35:23 INFO     Temp: 28.72
2021-08-16 14:35:23 INFO     Hum: 66
2021-08-16 14:35:23 INFO     Voltage: 2.837
2021-08-16 14:35:23 INFO     Battery level: 74
2021-08-16 14:35:23 INFO     Comfort type: 1
2021-08-16 14:35:23 INFO     The request is http://<IP_ADDRESS>:8080/json.htm?type=command&param=udevice&idx=23&nvalue=0&svalue=28.72;66;1&battery=74
2021-08-16 14:35:23 INFO     The response is <Response [200]>
When I "sudo hcitool lescan", I get the sensor without any issue.
Any clue what can be wrong?
Rgds,
Henry

Sorry for my late answer! Sometimes it happens if the device is too far from the Raspberry. I use a Raspberry PI Zero as a data collector near to the sensors and it sends the request to the Domoticz (Raspberry PI 4) through WIFI.


⚠ Bofrost.lu has degraded performance

In 7c61310, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1052 ms

Resolved: Bofrost.lu performance has improved in 899dfaf after 8 minutes.


chore(getComponentInfo): handle HOC default exports

This PR extends our usage of react-docgen to handle exported HOCs.

Codecov Report

Merging #2883 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2883   +/-   ##
=======================================
  Coverage   99.67%   99.67%
=======================================
  Files         161      161
  Lines        2733     2733
=======================================
  Hits         2724     2724
  Misses          9        9

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 6d6f6eb...15e7f9c. Read the comment docs.

Released in<EMAIL_ADDRESS>


Include the main free5gc apache license v2

Hi @free5gc-org,
I suggest we include the main Free5GC project's license Apache v2 to free5gc-compose, just like in all other projects of the Free5GC organization. Let me know what do you think.
Since we are at it, I also suggest we add a little description to the repo, something like "Run Free5GC NFs in docker containers" or "Build and run Free5GC easily using docker" or maybe "Docker compose to build and run Free5GC NFs in docker containers".
Cheers,

Agree. I can work on a PR for this.

@abousselmi
Agree, you can add Apache v2 license and descriptions to free5gc-compose.

@harsh-kotak thanks but I just added the file.
@free5gc-org as for the description I was referring to the about section of the repo.


Document svelte.config.js and provide examples for common setups

SCSS: https://github.com/UnwrittenFun/svelte-vscode/issues/19#issuecomment-397397168

closing in favour of #23