Error at Invoke-ScriptInBcContainer. Exception calling "Deserialize" with "1" argument(s): "Error occurred during a cryptographic operation."

Describe the issue
Running a PowerShell script to upgrade a Business Central database to current version raises the exception "Deserialize" with "1" argument(s): "Error occurred during a cryptographic operation.".
The script is using the Invoke-ScriptInBcContainer command and inside the script block the command that is raising the exception is Invoke-NAVApplicationDatabaseConversion.
Running the same commands without the need of the Invoke-ScriptInBcContainer inside the PowerShell container prompt works smoothly.
I'm trying to convert a BC14 database to BC25. For this, inside the BC25 container that is using a BC25 database I run the Invoke-NAVApplicationDatabaseConversion command to a BC14 database hosted in a external SQL Server.
Scripts used to create container and cause the issue
This script creates the BC25 container.
$Config = Get-Content 'config.json' -Encoding UTF8 | ConvertFrom-Json

$ContainerCred = New-Object PSCredential $Config.ContainerUser,(ConvertTo-SecureString -String $Config.containerPass -AsPlainText -Force)
$DBCred = New-Object PSCredential $Config.dbUser,(ConvertTo-SecureString -String $Config.dbPass -AsPlainText -Force)

$params = @{
    containerName = $Config.vaserver
    Credential = $ContainerCred
    auth = 'NavUserPassword'
    artifactUrl = 'https://bcartifacts-exdbf9fwegejdqak.b02.azurefd.net/onprem/25.1.25873.25900/es'
    licenseFile =  'C:\temp\MSPartnerBC25.bclicense'
    databaseServer = $Config.dbServer
    databaseName = 'op251'
    databaseCredential = $DBCred
    doNotExportObjectsToText = $true
    multitenant = $false
    assignPremiumPlan = $false
    useTraefik = $false
    bakFile = ''
    useGenericImage = ''
    includeCSide = $false
    useNewDatabase = $false
    includeAL = $false
    bakFolder = ''
    accept_eula = $true
    accept_outdated = $true
    includeTestToolkit = $false
    dns = ''
    isolation = 'process'
    enableTaskScheduler = $false
    useSSL = $false
    accept_insiderEula = $true
    alwaysPull = $true
    updateHosts = $true
    TimeZoneId = 'Romance Standard Time'
    PublicDnsName = ''
}

New-BcContainer @params

This is the script that I'm running to make the conversion and raises the exception:
$Config = Get-Content 'config.json' -Encoding UTF8 | ConvertFrom-Json

Invoke-ScriptInBcContainer `
    -containerName $Config.vaserver `
    -scriptblock {
        param(
            [string]$dbServer,
            [string]$dbName,
            [string]$dbUser,
            [string]$dbPass
        )

        $cred = New-Object PSCredential $dbUser,(ConvertTo-SecureString -String $dbPass -AsPlainText -Force)

        Invoke-NAVApplicationDatabaseConversion `
            -DatabaseServer $dbServer `
            -DatabaseName $dbName `
            -ApplicationDatabaseCredentials $cred `
            -Force
    } `
    -argumentList $Config.dbServer,$Config.vf14dbName,$Config.dbUser,$Config.dbPass

Full output of scripts
Container creation output:
BcContainerHelper is version 6.0.28
BcContainerHelper is not running as administrator
Host is Microsoft Windows 11 Pro - 10.0.22631.4317
UsePsSession is True
UsePwshForBc24 is True
UseWinRmSession is never
UseSslForWinRmSession is True
Docker Client Version is 27.2.0
Docker Server Version is 27.2.0
Removing Session op251
Removing container op251
Removing entries from hosts
Removing op251 from container hosts file
Removing op251-* from container hosts file
Removing Desktop shortcuts
Removing C:\ProgramData\BcContainerHelper\Extensions\op251
Fetching all docker images
Fetching all docker volumes
Pulling image mcr.microsoft.com/businesscentral:ltsc2022
ltsc2022: Pulling from businesscentral
Digest: sha256:549ceca21454562eb552f53ac18c12fb442a373724526c5abebd45aeafc57ea5
Status: Image is up to date for mcr.microsoft.com/businesscentral:ltsc2022
mcr.microsoft.com/businesscentral:ltsc2022
Using image mcr.microsoft.com/businesscentral:ltsc2022
Creating Container op251
Style: onprem
Multitenant: No
Version: 25.1.25873.25900
Platform: 25.0.25866.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.20348.2762 (ltsc2022)
Host OS Version: 10.0.22631.4317 (23H2)
Using process isolation
Using locale es-ES
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file C:\temp\MSPartnerBC25.bclicense
Additional Parameters:
--env customNavSettings=EnableTaskScheduler=False
Files in C:\ProgramData\BcContainerHelper\Extensions\op251\my:
- AdditionalOutput.ps1
- HelperFunctions.ps1
- license.bclicense
- MainLoop.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container op251 from image mcr.microsoft.com/businesscentral:ltsc2022
Using Shared Encryption Key file
7abd823e0051655ba7a1889d3e6f3821d7dd3893c6dd52b98dc21bfdc30f132b
Waiting for container op251 to be ready
Using artifactUrl https://bcartifacts-exdbf9fwegejdqak.b02.azurefd.net/onprem/25.1.25873.25900/es
Using installer from C:\Run\240
Installing Business Central: multitenant=False, installOnly=False, filesOnly=False, includeTestToolkit=False, includeTestLibrariesOnly=False, includeTestFrameworkOnly=False, includePerformanceToolkit=False, appArtifactPath=c:\dl\onprem\25.1.25873.25900\es, platformArtifactPath=c:\dl\onprem\25.1.25873.25900\platform, databasePath=, licenseFilePath=c:\dl\onprem\25.1.25873.25900\es\database\Cronus.bclicense, rebootContainer=True
Installing from artifacts
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
c:\dl\onprem\25.1.25873.25900\platform\ServiceTier\Program Files
c:\dl\onprem\25.1.25873.25900\platform\ServiceTier\System64Folder
Copying Web Client Files
c:\dl\onprem\25.1.25873.25900\platform\WebClient\Microsoft Dynamics NAV
Copying ModernDev Files
c:\dl\onprem\25.1.25873.25900\platform
c:\dl\onprem\25.1.25873.25900\platform\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\dl\onprem\25.1.25873.25900\platform\ConfigurationPackages
Copying Test Assemblies
C:\dl\onprem\25.1.25873.25900\platform\Test Assemblies
Copying Applications
C:\dl\onprem\25.1.25873.25900\es\Applications
Copying dependencies
Importing PowerShell Modules
Skipping restore of Cronus database
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Installation took 17 seconds
Installation complete
Initializing...
Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is op251
PublicDnsName is op251
Using NavUserPassword Authentication
Import Encryption Key
Stopping local SQL Server
Creating Self Signed Certificate
Self Signed Certificate Thumbprint AD7BE7C37D5F3D1A18704B15CA26FDE7CB875BBB
DNS identity op251
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Setting EnableTaskScheduler to False
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using application pool name: BC
Using default container name: NavWebApplicationContainer
Copy files to WWW root C:\inetpub\wwwroot\BC
Create the application pool BC
Create website: NavWebApplicationContainer without SSL
Update configuration: navsettings.json
Done Configuring Web Client
Using license file 'c:\run\my\license.bclicense'
Import License
Creating http download site
Container IP Address: <IP_ADDRESS>
Container Hostname  : op251
Container Dns Name  : op251
Web Client          : http://op251/BC/
Dev. Server         : http://op251
Dev. ServerInstance : BC
Setting op251 to <IP_ADDRESS> in host hosts file

Files:
http://op251:8080/ALLanguage.vsix

Container Total Physical Memory is 31.6Gb
Container Free Physical Memory is 4.0Gb

Initialization took 157 seconds
Ready for connections!
Reading CustomSettings.config from op251
Set TimeZone in Container to Romance Standard Time
Creating Desktop Shortcuts for op251
Cleanup old dotnet core assemblies
Container op251 successfully created

Use:
Get-BcContainerEventLog -containerName op251 to retrieve a snapshot of the event log from the container
Get-BcContainerDebugInfo -containerName op251 to get debug information about the container
Enter-BcContainer -containerName op251 to open a PowerShell prompt inside the container
Remove-BcContainer -containerName op251 to remove the container again
docker logs op251 to retrieve information about URL's again
...

This is the ouput for the conversion script:
Invoke-ScriptInBcContainer: C:\Users\myuser\git\scripts\test.ps1:3
Line |
   3 |  Invoke-ScriptInBcContainer `
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Exception calling "Deserialize" with "1" argument(s): "Error occurred during a cryptographic operation."

This is the error trace obtained with the $Error[0] | Format-List -Force command:
Exception             : System.Management.Automation.MethodInvocationException: Exception calling "Deserialize" with "1" argument(s): "Error occurred during a cryptographic operation."
                         ---> System.Security.Cryptography.CryptographicException: Error occurred during a cryptographic operation.
                           at Microsoft.PowerShell.ProtectedData.Unprotect(Byte[] encryptedData, Byte[] optionalEntropy, DataProtectionScope scope)
                           at Microsoft.PowerShell.SecureStringHelper.Unprotect(String input)
                           at System.Management.Automation.InternalDeserializer.ReadSecureString()
                           at System.Management.Automation.InternalDeserializer.ReadOneDeserializedObject(String& streamName, Boolean& isKnownPrimitiveType)
                           at System.Management.Automation.InternalDeserializer.ReadOneObject(String& streamName)
                           at System.Management.Automation.InternalDeserializer.ReadProperties(PSObject dso)
                           at System.Management.Automation.InternalDeserializer.ReadPSObject()
                           at System.Management.Automation.InternalDeserializer.ReadOneDeserializedObject(String& streamName, Boolean& isKnownPrimitiveType)
                           at System.Management.Automation.InternalDeserializer.ReadOneObject(String& streamName)
                           at System.Management.Automation.InternalDeserializer.ReadProperties(PSObject dso)
                           at System.Management.Automation.InternalDeserializer.ReadPSObject()
                           at System.Management.Automation.InternalDeserializer.ReadOneDeserializedObject(String& streamName, Boolean& isKnownPrimitiveType)
                           at System.Management.Automation.InternalDeserializer.ReadOneObject(String& streamName)
                           at System.Management.Automation.InternalDeserializer.ReadListContainer(ContainerType ct)
                           at System.Management.Automation.InternalDeserializer.ReadPSObject()
                           at System.Management.Automation.InternalDeserializer.ReadOneDeserializedObject(String& streamName, Boolean& isKnownPrimitiveType)
                           at System.Management.Automation.InternalDeserializer.ReadOneObject(String& streamName)
                           at System.Management.Automation.Deserializer.Deserialize(String& streamName)
                           at System.Management.Automation.PSSerializer.DeserializeAsList(String source)
                           at System.Management.Automation.PSSerializer.Deserialize(String source)
                           at CallSite.Target(Closure, CallSite, Type, Object)
                           --- End of inner exception stack trace ---
                           at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
                           at Invoke-ScriptInBcContainer(Closure, FunctionContext)
                           at System.Management.Automation.PSScriptCmdlet.RunClause(Action`1 clause, Object dollarUnderbar, Object inputToProcess)
                           at System.Management.Automation.CommandProcessorBase.Complete()
TargetObject          :
CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : CryptographicException
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at Invoke-ScriptInBcContainer, C:\Users\myuser\Documents\PowerShell\Modules\BcContainerHelper\6.0.28\ContainerHandling\Invoke-ScriptInNavContainer.ps1: line 297
                        at <ScriptBlock>, C:\Users\myuser\git\scripts\test.ps1: line 3
                        at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo : {}
PSMessageDetails      :

Additional context
As I have said previously, the commands inside the script block work perfectly in the container powershell prompt with the same configuration.
[OP251] PS C:\Run\my> $Config = Get-Content 'config.json' -Encoding UTF8 | ConvertFrom-Json
[OP251] PS C:\Run\my> $cred = New-Object PSCredential $Config.dbUser,(ConvertTo-SecureString -String $Config.dbPass -AsPlainText -Force)
[OP251] PS C:\Run\my> Invoke-NAVApplicationDatabaseConversion -DatabaseServer $Config.dbServer -DatabaseName $Config.vf14dbName -ApplicationDatabaseCredentials $cred -Force
Inplace publishing skipped. Emit version 25047 is current.

DatabaseServer      : host.docker.internal
DatabaseName        : VF14_3441
DatabaseCredentials : System.Management.Automation.PSCredential
DatabaseLocation    :
Collation           :

This is the host PowerShell version:
Name                           Value
----                           -----
PSVersion                      7.4.6
PSEdition                      Core
GitCommitId                    7.4.6
OS                             Microsoft Windows 10.0.22631
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

And this is the docker container PowerShell version:
Name                           Value
----                           -----
PSVersion                      7.4.5
PSEdition                      Core
GitCommitId                    7.4.5
OS                             Microsoft Windows 10.0.22631
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Adding the Out-Host command the script runs properly. It prints the output and there is no error.
Thanks!


Story: UsageSummary and LineItem Entities and Repositories

This story also includes a ElectricPowerUsageSummaryService, ElectricPowerUsageSummaryRepositoryTest, ElectricPowerUsageSummaryService, ElectricPowerUsageSummaryRepositoryTest, and database table(s) to support the entity class.

Although the database model calls the resource "ElectricPowerUsageSummary" the latest version of the NAESB REQ.21 ESPI standard deprecated that name and replaced it with "UsageSummary".  Please use "UsageSummary" as the Entity table name.

Closed #47


Upgrade manifestival to 0.5.0

See https://github.com/manifestival/manifestival/releases/tag/v0.5.0

/lgtm
/approve


Add deepsearch test branch

by @seambr
Added to extra branch so it triggers the preview. Noticed that the tokenization file is too big. Will check what options we have.

@seambr do you know what is causing the script.js:117 Error fetching data: TypeError: results.forEach is not a function on the test branch?

@seambr do you know what is causing the script.js:117 Error fetching data: TypeError: results.forEach is not a function on the test branch?

im not sure, im not getting that when i run it locally.

https://polished-bush-fd60.cryck.workers.dev/?input=apple&isBackwards=false
https://polished-bush-fd60.cryck.workers.dev/?input=apple&isBackwards=false&isDepth=false
Both are making request to the same endpoint so the new test branch is expecting an extra layer but its receiving the same data as the main branch

Ohhh I know why. Thats on me. Let me fix that.

@seambr Brilliant work for real! Love the deep search. Possibly need to instruct the user a little better on what Total, Start & End mean with some tooltips or improved naming, but that can come in the future.
Thank you so much for the contribution 🙏


Gracefully handle unsupported method for path defined in schema

When you have a path defined in a schema, such as:
/my/path:
  get:
    responses:
      200: ...etc...

If you issue any other request to that url except a GET it will fail due to pathObj[method] being undefined.
The changes
There were two changes necessary to support this:

A null check on pathObj[method], so it gracefully aborts looking for the schema.

Moving the response body processing to only occur when a schema is actually found; previously, it would incorrectly try to process the body of a request which wasn't being validated.

Hi @jagregory! Sorry for such late reply, but life has taken its course and I wasn't able to spend any time on the project. Have reviewed your changes and they are great! Merging it all in. Thanks for the contribution!


Devise like authentication

Hello !
I have a use case :
I need to have a authentication like rails+devise but handled directly by Elm.
How would you do that without cookies ?

Elm doesn't prevent you from using cookies.  If your server sets a cookie in the response header, the browser will include that cookie in future requests for that domain until the cookie expires.  So you can still have a server endpoint where you submit a username and password to authenticate a user, that responds with a cookie header to login the user within the current browser/user-agent.  Elm doesn't have to know anything about that.  It's all handled by the user-agent.
In order to notify your Elm app that the user was successfully logged in, you could either use appropriate HTTP statuses, and check those in your elm app (i.e. if the response code is 200, the user is logged in, if it's anything else, it failed) or return some JSON value, and read that to tell whether the user is logged in and/or what their ID is, etc.
Does that answer the question, or am I missing the focus?


Hotkey/ shortcut to focus on particular zone based on no.

Description of the new feature / enhancement
Suppose, there are 3 zones in a window, so some combination of modifier keys with a number from 1 to 3 or 0 to 2, to focus on the particular index would help:)
Scenario when this would be used?
Suppose, there are 3 zones in a window, so if we enter a number with some assigned modifier keys, as long as the number is within the zone index range, focus should shift to zone with that index number
Supporting information
No response

Currently we have a hotkey (Win+PgUp/PgDn) to switch between windows that are placed in the same zone. Would that be sufficient for you / your idea ?

Duplicate of #15067

That's what I meant by switching zones
On Mon, 20 Dec 2021 at 17:57, Manish Gowd @.***> wrote:

Yeah, I tried that, that's great. My suggestion is just focus on the
topmost window in the new zone based on numerical indexing of zone, or
something like that
On Mon, 20 Dec 2021 at 17:48, Jay @.***> wrote:

Currently we have a hotkey (Win+PgUp/PgDn) to switch between windows that
are placed in the same zone. Would that be sufficient for you / your idea ?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/15078#issuecomment-997873888,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANPESTUTDXPSEFDO5CHAUWDUR4NKBANCNFSM5KMDK66A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yeah, I tried that, that's great. My suggestion is just focus on the
topmost window in the new zone based on numerical indexing of the zone, or
something like that. That's what I meant by switching zones
On Mon, 20 Dec 2021 at 17:48, Jay @.***> wrote:

Currently we have a hotkey (Win+PgUp/PgDn) to switch between windows that
are placed in the same zone. Would that be sufficient for you / your idea ?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/15078#issuecomment-997873888,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANPESTUTDXPSEFDO5CHAUWDUR4NKBANCNFSM5KMDK66A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Suppose, there are 2 zones, and I want to switch to zone 2 (currently in zone 1), my suggestion is that as long as there is a window in that zone, switch focus to topmost (the one that is displayed) in the new zone)
So, to switch to topmost window on zone2 from zone1, some combination of modifier keys with 2, should focus window on zone 2, likewise for zone1, if no window exists in that particular zone, focus should remain in the currently focused zone


PLEASE SUPPORT PSYCH ENGINE 0.7.3 FOR P-Slice ENGINE

YES, I SAYD PLEASE SUPPORT PSYCH ENGINE 0.7.3 FOR P-Slice ENGINE

Not happening. btw P-Slice 1.1 is on 0.7.3

oh ok


Changes the method getImportAbsolutePath normalize the URL when finding the import's path

What does this PR do:

Changes the method getImportAbsolutePath to pass the url normalized to the findImportedPath to solve paths' issues on Windows coming from importers

Where should the reviewer start:

Diffs

Fixes #41

@jgranstrom now that the tests are failing, a little bit of context:

The dependency foundation-sites has changed its structure and files since version 6.5.0. If I lock the version to ~6.4.3, the tests failing related to normalize.scss (foundation-variables tests) will be fixed.

If I lock the version of the peer dependency node-sass to ^3.8.0 (ergo not supporting 4) the tests failing related to Invalid CSS (in the map-keys tests) will go away.

Would you like me to lock those dependencies' versions or would you prefer to fix the unit tests?
Although the latter would seem the most interesting approach, it is also the most complex, since it requires changes on test logic for variables and also requires to check "if node-sass's version is X assert one thing, if it's Y assert another".

Oh, and if you run the build on your master today it'll have the same problem (so, the change I did does not cause it 😉 )


LookupError: No gradient defined for operation for assign

I wish to do assign ops in a layer such as below
with tf.name_scope('conv1_1') as scope:
kernel = tf.Variable(tf.truncated_normal([3, 3, 3, 64], dtype=tf.float32,
stddev=1e-1), name='weights')
kernel = kernel[:, :, : 0:32].assign(tf.zeros([3, 3, 3, 32]))
conv = tf.nn.conv2d(images, kernel, [1, 1, 1, 1], padding='SAME')
biases = tf.Variable(tf.constant(0.0, shape=[64], dtype=tf.float32),
trainable=True, name='biases')
out = tf.nn.bias_add(conv, biases)
self.conv1_1 = tf.nn.relu(out, name=scope)
self.parameters += [kernel, biases]
But later I train again. I will have error.
LookupError: No gradient defined for operation for assign

This question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 29 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 139 days with no activity and the awaiting response label was assigned. Is this still an issue?

We are closing this issue for now due to lack of activity. Please comment if this is still an issue for you. Thanks!


Wait_template is not working correctly, it triggers final step without TRUE condition if called twice in "timeout" period.

Home Assistant release with the issue:

Last working Home Assistant release (if known):
I don't know i try this first time.
Operating environment (Hass.io/Docker/Windows/etc.):

hass.io
Component/platform:

Automation.
Description of problem:
I try to use wait_template. But it is not working correctly, it triggers final step without TRUE condition if called twice in "timeout" period.
Step by step what id does:

Main condition is triggered (device_tracker.samsunggalaxys7 is at home) - i am near my home
email is sent (that starting "XXX ...")
15 minutes countdown is started and it waits for device_tracker.itag (BLE tracker) state "home"
now i go away from home so samsunggalaxys7 state is "not-home" again
now i go back near my home so samsunggalaxys7 state is "home" again - still in 15min delay!
automation is triggered again but - no email is sent (first action is not triggered), script is called directly (third action) without wait-template condition equals to true!!

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
- alias: Kontrola příchodu domu Petr
  trigger:
    platform: zone
    entity_id: device_tracker.samsunggalaxys7
    zone: zone.home
    event: enter
  action:
    - service: notify.emailpetr
      data:
        title: XXX Petr vlezl do zóny a jdu sledovat BLE
        message: doma a jdu sledovat BLE 15min
    - wait_template: "{{ is_state('device_tracker.itag', 'home') }}"
      timeout: 00:15:00
      continue_on_timeout: 'false'
    - service: script.prichod_domu

Sorry for czech language but i thing it doesn't matter here.
Traceback (if applicable):

Additional information:
Proof of points above, relevant log messages here:
1:
2018-10-01 16:57:33 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event logbook_entry[L]: name=Kontrola příchodu domu Petr, message=has been triggered, domain=automation, entity_id=automation.kontrola_prichodu_domu_petr>
2018-10-01 16:57:33 INFO (MainThread) [homeassistant.helpers.script] Script Kontrola příchodu domu Petr: Running script
2018-10-01 16:57:33 INFO (MainThread) [homeassistant.helpers.script] Script Kontrola příchodu domu Petr: Executing step call service

2:
2018-10-01 16:57:33 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event call_service[L]: domain=notify, service=emailpetr, service_data=title=XXX Petr vlezl do zóny a jdu sledovat BLE, message=doma a jdu sledovat BLE 15min, service_call_id=8dd9d45120984f379a6bc5abbfd5e910>

3:
I thing 15min waiting has no log message..
4:
2018-10-01 17:02:42 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=device_tracker.samsunggalaxys7, old_state=<state device_tracker.samsunggalaxys7=home; source_type=gps, latitude=50.0083262, longitude=14.417486, gps_accuracy=21, battery=72.0, speed=0.0, direction=0.0, altitude=0.0, provider=network, activity=IN_VEHICLE, friendly_name=Petr, icon=mdi:mdi-gender-male, entity_picture=/local/petr.png @ 2018-10-01T16:57:33.696961+02:00>, new_state=<state device_tracker.samsunggalaxys7=not_home; source_type=gps, latitude=50.00768208013503, longitude=14.416353704863297, gps_accuracy=4, battery=71.0, speed=0.0, direction=0.0, altitude=278.0139468828492, provider=gps, activity=, friendly_name=Petr, icon=mdi:mdi-gender-male, entity_picture=/local/petr.png @ 2018-10-01T17:02:42.760000+02:00>>

5:
2018-10-01 17:04:20 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=device_tracker.samsunggalaxys7, old_state=<state device_tracker.samsunggalaxys7=not_home; source_type=gps, latitude=50.00768208013503, longitude=14.416353704863297, gps_accuracy=4, battery=71.0, speed=0.0, direction=0.0, altitude=278.0139468828492, provider=gps, activity=, friendly_name=Petr, icon=mdi:mdi-gender-male, entity_picture=/local/petr.png @ 2018-10-01T17:02:42.760000+02:00>, new_state=<state device_tracker.samsunggalaxys7=home; source_type=gps, latitude=50.00772, longitude=14.4176324, gps_accuracy=21, battery=71.0, speed=0.0, direction=0.0, altitude=0.0, provider=network, activity=, friendly_name=Petr, icon=mdi:mdi-gender-male, entity_picture=/local/petr.png @ 2018-10-01T17:04:20.742952+02:00>>

6:
2018-10-01 17:04:20 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event logbook_entry[L]: name=Kontrola příchodu domu Petr, message=has been triggered, domain=automation, entity_id=automation.kontrola_prichodu_domu_petr>
2018-10-01 17:04:20 INFO (MainThread) [homeassistant.helpers.script] Script Kontrola příchodu domu Petr: Executing step call service
2018-10-01 17:04:20 DEBUG (MainThread) [homeassistant.core] Bus:Handling <Event call_service[L]: domain=script, service=prichod_domu, service_data=, service_call_id=b32077767d674105ad457ed3f570be65>

This works as designed: turn_on on a sleeping script makes it wake up and continue execution.
I cannot say that I agree with this design, though.

I still think it is a bug. Wait-template MUST keep up condition and MUST NOT execute code after condition if condition is (still) false in any case. I think code you send is some common part which break wait-template functionality and should be fixed.
Wait-template is useless if you cannot relay on condition inside it.
And finally i think this behavior doesn't match doc here: https://www.home-assistant.io/docs/scripts/#wait

Which wording in the documentation do you think contradicts this behavior?

First - sorry for (maybe) duplicity - similar issue is here: https://github.com/home-assistant/home-assistant/issues/16985
I have answer for you, read exactly this sentence: "Wait until some things are complete. We support at the moment wait_template for waiting until a condition is true..."
First - i think it IS in documentation like as i understand it. Second - if wait-template (sometimes) breaks condition check it is practically useless.
If someone think present behavior is ok (i don't think so) someone should write this exception into docs.

Yes, the wait template is bypassed somehow when a trigger is fired twice. I don't think this is expected behavior based on logic and the docs.

Thanks for the quote from the documentation, I agree that it is at least incomplete.
In my opinion, the problem is due to the limitation that a script can either be stopped or be running but it cannot be running twice. So when we try to start it a second time, something unexpected will happen. I see these options:

The first invocation is aborted (and the second invocation is then started).
Nothing happens.
The first invocation is resumed.

So we currently have option 3 which I think is the worst of the options.
Do you agree with this analysis? If yes, what is your proposed improvement?

I think there are some constraints for right solution:

Condition MUST be used and final action cannot be fired without condition=true. Never.
If more than one trigger is fired you need at least one to be finished.

So if it is not possible to finish both/more invocations separately best solution for me is stop first invocation and finish second with condition.
I think me and as i see other people use this for "coming home" automation and solution i described is sufficient for this usecase.

So just to make sure I understand you right, that is my option 1? In other words, the script is restarted.
I think that is also my preference though I struggle to imagine whether that could break other use cases.

Yes, i think your number 1 is the right solution for me and i hope for other users.

I think it is totally broken in 0.81.2. See my original scenario. First two steps are ok:

Main condition is triggered (device_tracker.samsunggalaxys7 is at home) - i am near my home
email is sent (that starting "XXX ...")

but after {{ is_state('device_tracker.itag', 'home') }} comes to TRUE script is NOT started.
Config is still the same as it was.

after 0.81.2 my only wait_template stops being functional. It doesn't wait and immediately executes the next action in the sequence:
  - alias: 'Switch Masterbed outlet when movement'
    id: 'Switch Masterbed outlet when movement'
    #    initial_state: 'on'
    trigger:
      platform: state
      entity_id: sensor.master_bedroom_motion_sensor
      to: 'on'
    condition:
      - condition: template
        value_template: >
          {{ is_state ('sensor.activity_selection', 'Naar bed')}}
      - condition: template
        value_template: >
          {{is_state('switch.master_bed_outlet', 'off')}}
    action:
      - service: switch.turn_on
        entity_id: switch.master_bed_outlet
      - wait_template: >
          {{ is_state('binary_sensor.master_bedroom_motion_sensor_timed','off')}}
      - service: switch.turn_off
        entity_id: switch.master_bed_outlet
      - condition: template
        value_template: >
          {{ is_state('input_boolean.notify_system', 'on')}}
      - service: notify.notify
        data_template:
          message: >
            {{as_timestamp(now()) | timestamp_custom("%X") }}: You've walked safely lift. Signing off.

while the motion_sensor_timed is working as expected:
      master_bedroom_motion_sensor_timed:
        friendly_name: 'Master bedroom motion sensor timed'
        value_template: >
          {{ is_state('sensor.master_bedroom_motion_sensor','on')}}
        delay_off:
          minutes: 1
        device_class: motion

I have the same issue.  I was waiting since 80.0, still broken :(

The issue can still be reproduced in the latest HA version. I opened another issue for this
https://github.com/home-assistant/home-assistant/issues/21788


Automatic convert window path into bash path

Window Path: D:\Users\yesla\Desktop\workspace\testingRocky
Ubuntu Path: /mnt/d/Users/yesla/Desktop/workspace/testingRocky
If I copy the window path and paste it to ubuntu terminal, it should automatic convert the path from window path to ubuntu path.
Cmder terminal already have this feature.

In window terminal
When you copy the path to bash terminal, it show the exact window path rather than ubuntu path.

Feature Demonstration
Copy the window path then paste it to cmder gitbash terminal, you will see something like this. 'Auto path conversion'.

I believe this feature is already being tracked in issue #1772.

Sure is, thanks!
/dup #1772


I want to get the data

Hello,can you give me the data of the traffic light,I can not download the data from Nexar.

I can not share it without Nexar's approval. You should ask them.
Thanks


Crash loop causes many windows to spawn in quick succession

Checklist

[x] Check other issues and make sure that it is not reported yet.

Versions and Environment
Elixir:
# elixir -v
Elixir 1.9.1 (compiled with Erlang/OTP 22

Erlang:
# erl -v
rlang/OTP 22 [erts-10.4.4] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [hipe]

Scenic:
# mix deps | grep scenic
* scenic 0.10.2 (Hex package) (mix)
  locked at 0.10.2 (scenic) f4f0a00e
* scenic_driver_glfw 0.10.1 (Hex package) (mix)
  locked at 0.10.1 (scenic_driver_glfw) df7a11bc

OS:
Arch Linux (but applies to other OS as well)

Steps to reproduce

git clone https://github.com/axelson/scenic_restart_crash
mix deps.get
mix scenic.run

Expected Behavior

Scenic to crash a few times and then give up with the application exiting
Actual Behavior

Many windows spawned and died, resulting in Maximum number of clients reached being printed multiple times and a total of around 4000 lines of output (most of it stack traces).
Note: When this happens scenic tries to launch a new window 256 times in under a second which causes the OS to become sluggish and unresponsive and all the windows don't fully get created and disappear until around 10 seconds or so. It's 256 times due to the current supervision hierarchy which has 4 supervisors allowing 3 restarts (so 4 starts/crashes) which is 4^4 = 256.
Stack Trace

07:23:49.391 [error] GenServer :main_viewport terminating
** (UndefinedFunctionError) function ScenicRestartCrash.Scene.Home.start_dynamic_scene/4 is undefined (module ScenicRestartCrash.Scene.Home is not available)
    ScenicRestartCrash.Scene.Home.start_dynamic_scene(#PID<0.2017.0>, nil, nil, [vp_dynamic_root: #PID<0.2014.0>, viewport: #PID<0.2014.0>, styles: %{}])
    (scenic) lib/scenic/view_port.ex:532: Scenic.ViewPort.handle_cast/2
    (stdlib) gen_server.erl:637: :gen_server.try_dispatch/4
    (stdlib) gen_server.erl:711: :gen_server.handle_msg/6
    (stdlib) proc_lib.erl:249: :proc_lib.init_p_do_apply/3
Last message: {:"$gen_cast", {:set_root, {ScenicRestartCrash.Scene.Home, nil}, nil}}
State: %{default_scene: {ScenicRestartCrash.Scene.Home, nil}, default_scene_activation: nil, driver_registry: %{}, drivers: [], dynamic_root_pid: nil, dynamic_supervisor: #PID<0.2017.0>, hover_primitve: nil, input_captures: %{}, master_graph: %{0 => %{data: {Scenic.Primitive.Group, [1]}, transforms: %{}}, 1 => %{data: {Scenic.Primitive.SceneRef, nil}}}, master_graph_key: {:graph, #Reference<0.1457681126.2594177025.129008>, nil}, master_styles: %{}, master_transforms: %{}, max_depth: 256, on_close: :stop_system, root_config: nil, root_graph_key: nil, root_scene_pid: nil, size: {700, 600}, supervisor: #PID<0.2005.0>}
Maximum number of clients reached

Additional Comments

The specific crash isn't as interesting to me (and could be relatively easily fixed), but I would like to find a way to categorically not end up in this situation either through a change in the supervision hierarchy or possibly custom supervision.
Some half-baked ideas for solutions:

Add a delay on restarting the Viewport if it crashes
a. Pro: This would give the user time to manually stop the scenic process
b. Con: Depending on the length of the delay, some of the supervisors will constantly crash because the lower-level supervisors are not crashing often enough in the default 5 second window to crash the higher-level supervisor.
Reduce the depth of the supervision hierarchy by removing supervisors or turning down the max number of crashes that they allow and possibly increasing the max_seconds configuration
a. Pro: This will mitigate the issue
b. Con: The issue will still exist, just in a less annoying format
Use some sort of custom supervision strategy
a. Pro: It is probably possible to prevent the issue
b. Con: More code complexity and likely more difficulty understanding the system as a whole

I am not sure which direction would be best at this point. I kind of like option 2, although depending on the necessary complexity option 3 might be best. Perhaps after enough failed restarts in a row the viewport's supervisor could somehow mark itself as "unrecoverable" and would no longer try to restart it's children.

👍  to this. Has happened a few times on MacOS and ends up crashing my computer.

Pretty sure this is finally fixed in v0.11. Reopen if you see it again please.
v0.11 will be merged into master soon


Investigate Custom styles (min width)

Custom styles that use min-width and max-width are sometimes not honored when applied to table headers and table cells. It appears that the contained content (markdown vs HTML) may be causing this inconsistency in how the styles are being applied (or ignored).

Some more info found while working on the style guide:
When a table column has max-width: 100px;, the max width is not honored sometimes. This is believed to happen when the content of the column (in the header or any of the cells below) is greater than 100px. This was found out when trying to set a max-width for the Actions column so that it won't get stretched by the responsiveness of the CSS.

I updated docs/user-docs/custom-css.md with information that was noted above about max-width as well as extra information for min-width.

@jrchudy If the min-width set is too small for the content, it is pretty much get ignored, correct? Whatever width that will fit the content will be used instead?

If the content is longer than the min-width, the width will be at minimum the value set for min-width and get wider to accomodate the content.

Yes that's correct. Min-width will be ignored if the content is longer than the min-width.


Its asking for a password on an unlocked wallet

help

The only way an unencrypted (no password) wallet can be encrypted is by the user choosing the "File | Add password' option and adding a password twice.
You may want to have a look at which files are generated for a wallet which is specified here:
https://multibit.org/en/help/v0.5/help_fileDescriptions.html
Have a look at the wallet backups as they are stored separately for unencrypted and encrypted wallets.
If your wallet only has encrypted backups then it IS an encrypted wallet and you'll need a password to spend your bitcoin.

do you recovered your old multibit wallet?
if you have the wallet files, there is still chance to recover it
u can contact https://btc2doge.com/


[V4] NativeWindStyleSheet has no property to set output for web / default

During upgrade from v2 to v4 I discovered that our app used back in v2 NativeWindStyleSheet.setOutput for selecting the correct output for web and mobile. NativeWindStyleSheet has no longer this prop so I'm a bit confused here why It got removed.
To Reproduce
[v2] -> [v4] upgrade:
NativeWindStyleSheet.setOutput({   web: 'css',   default: 'native' })
Expected behavior
Property 'setOutput' does not exist on type 'typeof import("/Users/ritmillio/Developer/expo-router-starter-kit/node_modules/nativewind/dist/index")'.ts(2339)

NativeWind v4 no longer supports changing the output per platform. Web is always CSS and native and always StyleSheet.create. This only only existed as React Native Web's CSS support was very young and not many projects had upgraded


Experiencing "Error: JSON schema not found at file://"

When updating our dependencies from 5.2.1 to 5.2.2 today, we experienced this error:
Error: JSON schema not found at file://

Ref https://github.com/wp-cli/wp-cli/pull/4393 and https://travis-ci.org/wp-cli/wp-cli/jobs/282809129
Let me know what situational details I can provide to help reproduce it. justinrainbow/json-schema is a dependency of Composer, which we use within our application.

Thanks for the heads-up. Could you please let me know the URI of the schema that's triggering the error, and how that schema is loaded? Also, does that schema contain an id, and / or any relative $ref?

Could you please let me know the URI of the schema that's triggering the error, and how that schema is loaded? Also, does that schema contain an id, and / or any relative $ref?

I'm not entirely sure, but this looks like it could be it: https://github.com/composer/composer/blob/af519de0de087acb11450c2c10225377c8a909ff/res/composer-repository-schema.json
From https://github.com/composer/composer/search?utf8=✓&q=json-schema&type=
I also see https://github.com/composer/composer/blob/e12e0335a4d467878093fc76e3dea592ff3e1ce3/src/Composer/Json/JsonFile.php#L150

@danielbachhuber
I'm having difficulty reproducing this. I've done the following:

clone wp-cli/wp-cli locally into /tmp/wp-cli
Run composer install
Update justinrainbow/json-schema to 5.2.2
Inside /tmp/wp-cli, run ./bin/wp package install danielbachhuber/wp-cli-reset-post-date-command

Step 4 (i.e. the first failing travis test) seems to work perfectly without error.
Are you able to provide a minimal test case, or instructions to reproduce?

steve@neith /tmp/wp-cli $ ./bin/wp package install danielbachhuber/wp-cli-reset-post-date-command
Installing package danielbachhuber/wp-cli-reset-post-date-command (dev-master)
Updating /home/steve/.wp-cli/packages/composer.json to require the package...
Using Composer to install the package...
---
Loading composer repositories with package information
Updating dependencies
Resolving dependencies through SAT
Dependency resolution completed in 0.001 seconds
Analyzed 441 packages to resolve dependencies
Analyzed 50 rules to resolve dependencies
Package operations: 1 install, 0 updates, 0 removals
Installs: danielbachhuber/wp-cli-reset-post-date-command:dev-master b8fee09
 - Installing danielbachhuber/wp-cli-reset-post-date-command (dev-master b8fee09)
Writing lock file
Generating autoload files
---
Success: Package installed.

I've triggered a build to reproduce: https://github.com/wp-cli/wp-cli/pull/4396

I suspect I know what the issue is (now that I've managed to compare 5.2.1 and 5.2.2).
WP-CLI is built and distributed as a Phar archive. As such, prepending file:// to a path is erroneous because the file protocol inside a Phar is phar://.
I don't know enough about the json-schema codebase to offer an alternative at this point :/

@danielbachhuber It shouldn't be prepending file:// to anything unless there's no protocol provided at all. If phar:// is present in the base URI, then it will be left alone and relative URI will also be resolved as phar://.
I'll investigate to see if there's a possibility that something might be coming through from a phar archive that does not have a phar:// URI prefix on the base, as that would certainly trigger the issue.

I think it's caused by this. Nonsensical behavior, and apparently not a bug (although noting that phar archives are supposed to be filesystem-equivalentl it sure seems like a bug to me). I'll refactor this code to avoid the use of realpath().

@danielbachhuber This is fixed in #455.

👌 Thanks for your help with this, @erayd

No problem. Thanks for reporting the issue :-).
I'll update this issue once the fix is merged (@bighappyface needs to do that part). It'll go to 6.0.0-dev first, then backported for v5.2.5.

@danielbachhuber Now merged in 6.0.0-dev, and backported for 5.2.5 in #457.

👍 Looking forward

https://github.com/justinrainbow/json-schema/releases/tag/5.2.5

Created a build in https://github.com/wp-cli/wp-cli/pull/4433

5.2.5 fixes our issue. Thanks for your help @erayd @bighappyface :)


A problem occurred configuring project ':irondash_engine_context'

Looks like a required ndkVersion is missing in a grade file:
* Where:
...../pub.dev/irondash_engine_context-0.4.0/cargokit/gradle/plugin.gradle' line: 140

* What went wrong:
A problem occurred configuring project ':irondash_engine_context'.
> Failed to notify project evaluation listener.
   > Please set 'android.ndkVersion' in 'app/build.gradle'.
   > Please set 'android.ndkVersion' in 'app/build.gradle'.
   > Please set 'android.ndkVersion' in 'app/build.gradle'.

Version 0.4.0

You need to set ndkVersion inside your root project app/build.gradle. It should be set to flutter.ndkVersion by default.

I'm getting the same error for this library and flutter.ndkVersion is set in the build.gradle.

How>?
irondash_engine_context-0.5.2/cargokit/gradle/plugin.gradle' line: 140

What went wrong:
A problem occurred configuring project ':irondash_engine_context'.

Failed to notify project evaluation listener.
Please set 'android.ndkVersion' in 'app/build.gradle'.

Can u just bother add some setup steps in README avoid so many new comers's issues?

android.sdkVersion is by default set for newly created Flutter project. And the error message clearly says what is missing. PR for improved documentation is always welcome.

it;s ndkVersion ...

I meant NDK version.
android {
    ...
    ndkVersion flutter.ndkVersion

This should be present by default in any Flutter generated project file.


CI: Don't run CompatHelper on the docs project

Otherwise CompatHelper tries to make these PRs to add compat entries for FHIRClient itself: https://github.com/JuliaHealth/FHIRClient.jl/pull/142

Closing in favor of #141


Support for custom labels

Fixes parts of #215
Only for labels 😅
AI summary
This pull request introduces new functionality to add labels to pull requests created by the wrapper upgrade process. The changes include updates to the Options class to support labels, modifications to the pull request creation logic to apply these labels, and test updates to verify the new functionality.
Key changes:
Adding support for labels in pull requests:

src/main/java/org/gradle/wrapperupgrade/UpgradeWrapper.java: Modified the gitCreatePr method to add labels to the pull request if they are specified in the options.
src/main/java/org/gradle/wrapperupgrade/WrapperUpgradeDomainObject.java: Added a new labels property to the Options class and provided a getter method for it. [1] [2]

Test updates:

src/test/groovy/org/gradle/wrapperupgrade/GradleWrapperUpgradePluginFuncTest.groovy: Updated the functional test to include labels in the options for the wrapper upgrade.

Can you also document this new option in the REAMDE.md?

Documented here https://github.com/gradle/wrapper-upgrade-gradle-plugin/pull/263/commits/43987d61419c376121acf6c94ec34e37e9ece040


Can you add that CIS Benchmark Market Place Virtual Machines are not supported

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.
[ ] A link to the published documentation article that you have feedback about.
[x] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[x] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/forums/34192--general-feedback.

Can you add that CIS Benchmark Market Place Virtual Machines are not supported. Due to the nature of a CIS Benchmark it is not possible to act as a Syslog Forwardedr Can we get a clear statement for this type of installations?

@roflemin Can you please provide us with the URL of the Azure Doc you are having issues with? We look forward to your reply.

@roflemin Could you please share the document you are referring to?

https://docs.microsoft.com/en-us/azure/sentinel/connect-common-event-format
the list of supported operating systems. The CIS Benchmark builds do not work, but we have had customers try to install it on them and it does not work.

@roflemin Thanks for bring this to our attention. This is being assigned to the content owner for review.
@rkarlin & @yelevin To review and update as needed.

@roflemin Could you be more specific? Are these CIS Hardened Virtual Machines in Azure? In another cloud? Are they Level 1 or Level 2?
Thanks

Hi @roflemin, we haven't seen any response to this issue in some time.
I'm going to close this issue for now, but feel free to continue commenting here if you have additional feedback.
Thanks for sending your question into Microsoft docs!
#please-close


[Bug] The source code installation of version 0.9.0 failed

Expected behavior
Successful installation
Actual behavior
Error following the documentation to get this far
cd python; python setup.py install --user; cd ..
Check setup.py and find that the following code causes an error
# Add tvmc configuration json files
for name in lib_path:
    candidate_path = os.path.abspath(os.path.join(os.path.dirname(name), "..", "configs"))
    if os.path.isdir(candidate_path):
        libs.append(candidate_path)
        break

Environment
ubuntu18.04
tvm==0.9.0

Closing this as a duplicate of #12065, but we definitely do want to get this fixed before the v0.9.0 release so thanks for flagging it


lacrimal gland myoepithelium

Please add:
abnormal lacrimal gland myoepithelium morphology
DEF: any structural anomaly of the stellate-shaped contractile cells with flattened nuclei and long slender processes that surround the secretory acini and intralobular ducts of the lacrimal gland, located between the epithelium and the basement membrane; myoepithelial cells not only exhibit characteristics of other epithelial cells, such as expression of cytokeratin, but also exhibit properties of smooth muscle cells, such as expression of alpha-smooth muscle actin; their contraction might aid in the expulsion of fluid out of the secretory tubules or help maintain the patency of acinar and ductal lumens
SYN: abnormal lacrimal gland myoepithelial cell morphology
SYN: abnormal lacrimal myoepithelium morphology
SYN: abnormal lacrymal gland myoepithelium morphology
SYN: abnormal lacrymal gland myoepithelial cell morphology
SYN: abnormal lacrymal myoepithelium morphology
https://books.google.com/books?id=fjZjN_xGfGAC&pg=PA69&lpg=PA69&dq=mouse+lacrimal+gland+duct&source=bl&ots=NCMoeK0H4c&sig=nl2vgDM5Yi0Sp80clEABJvNTRbM&hl=en&sa=X&ei=ynL4VNClBYONyATAxIDICA&ved=0CE8Q6AEwBjgK#v=onepage&q=mouse lacrimal gland duct&f=false
https://books.google.com/books?id=me3Yb2H2KeIC&pg=PT208&lpg=PT208&dq=mouse+lacrimal+gland+and+lobe+and+lobule&source=bl&ots=Gij2SWFLQ4&sig=sIux57r77qpiR5MsK9_Qc-CS6bk&hl=en&sa=X&ei=IGX4VKrtL8GAygTF-YLoAQ&ved=0CC0Q6AEwBQ#v=onepage&q=mouse lacrimal gland and lobe and lobule&f=false
https://books.google.com/books?id=93s6aes5ob0C&pg=PA29&lpg=PA29&dq=lacrimal+gland+myoepithelial+cell+and+duct&source=bl&ots=AybwQh86W4&sig=93MFZ_oFG4mYwVqhuLJ7LI3MaVY&hl=en&sa=X&ei=bID4VPKJFNOmyATdpYHYCA&ved=0CB0Q6AEwADgK#v=onepage&q=lacrimal gland myoepithelial cell and duct&f=false
Thanks
Anna
Reported by: anna_anagnos
Original Ticket: obo/mammalian-phenotype-requests/2037

abnormal lacrimal gland myoepithelium morphology  MP:0014156


[Filled TextField] Disabled field underline should not change on hover

When a filled textfield is disabled, the underline should not change color on hover as that implies interactivity.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Underline on a disabled filled textfield darkens on mouse hover.
Expected Behavior 🤔
Underline on a disabled filled textfield should not change on mouse hover.

Steps to Reproduce 🕹
Steps:

Create and Filled TextField and set it to disabled.
Hover to see underline change. You may need to zoom is as it is a subtle color change.

See the filled textField in this example: https://codesandbox.io/s/material-demo-9vqbl
(this example is just the first example from TextField docs, with "disabled" added to the filled textField)
Context 🔦
I've found  a css workaround:
   '&.Mui-disabled:hover::before': {
      borderColor: 'inherit',
    },

But I suspect this is a bug as the underline should not change color on hover as that implies interactivity. Standard and Outlined textFields follow this practice, as neither of them change on hover when disabled.

Your Environment 🌎

Tech
Version

Material-UI
v4.10.2

React
16.13.1

Browser
Chrome

TypeScript

etc.

Yeah, I can reproduce that.
diff --git a/packages/material-ui/src/FilledInput/FilledInput.js b/packages/material-ui/src/FilledInput/FilledInput.js
index 99c433918..bf3589d1f 100644
--- a/packages/material-ui/src/FilledInput/FilledInput.js
+++ b/packages/material-ui/src/FilledInput/FilledInput.js
@@ -78,7 +78,7 @@ export const styles = (theme) => {
         }),
         pointerEvents: 'none', // Transparent to the hover style.
       },
-      '&:hover:before': {
+      '&:hover:not($disabled):before': {
         borderBottom: `1px solid ${theme.palette.text.primary}`,
       },
       '&$disabled:before': {

How about this fix?

@tchmnn Great fix, do you want to get it to the finish line :)?


[BUG] query vertex's neighbors occurs bug occasionally

Describe the bug
A clear and concise description of what the bug is.
An example is g.V().has("symbol", "symbol_id", xx).in("road_symbol").out("road_symbol").values("type"), which outputs unexpected results.
This bug seems related to vertex shuffling, as it outputs the following error info occasionally:

2022-10-10 17:34:36.254279560 ERROR (/work/interactive_engine/executor/engine/pegasus/server/src/rpc.rs:180) [tokio-runtime-worker] submit job 126 failure: Build(user defined error: Not supported: get server id failed in graph_partition_manager in source op QueryStoreError("get server index by given partition id failed in Vineyard"))

Closed as caused by the non-terminated java process and the issue has already been resolved in user's environment.


Image upload support

any plan to implement upload?
happy to contribute if u can give some pointers. thanks and great work!
PS. I'd add signed S3 upload + auto fill final url to image block

I do have plans to do it. For the package, I will expose an API something like onImageUpload or onImageDrop, and let the user do whatever they want. But for the app, I need to think differently because image upload requires some amount of money. So let's see, I'm closing it for now and will be back with a solution.


Update Heroku deploy docs

Our Heroku deploy docs were outdated and in the wrong place. This PR moves the Heroku deploy section from the "Deploy tracking troubleshooting page" to the main "Deploy tracking" page, corrects the deploy hook URL and rake task, and adds info about setting the repository URL.
The diff is a little messy, so it's probably easier to review each commit one at a time.

Updated, PTAL 👀


Data Size To Bytes function and associated code cleanup

Motivation for changes:
Many search plugins use the same math for parsing the data size.  Centralizing this math into a function will help to keep code clean
Detailed changes:
Added data_size_to_bytes to tools.py and then changing search plugins to use

I think MB vs MiB needs some work. Looks like it always divides by 1000 even in the case of MiB.

I think our 'content_size' entry field is supposed to be in MiB, rather than bytes. Not sure what the best fix is, we can either:

Return MiB from the parse function
divide the result of the parse function by 1024 when setting content_size field
add a parameter to the parse function so that return unit can be specified (might be overkill, as we'll probably always use the same unit)
change the definition of content_size field to be bytes (I don't like this one, bytes are less useful unit and it would introduce backward compatibility issues)

We should also add a docstring to the new parse function which explains what it does, returns, and what the base2 param does.

Consensus on IRC is to go with returning MiB from the parse function. That's been our standard so far, so it should keep everything working the same.


Update package.json to include the repository key

With the rise in supply chain attacks and OSS dependencies being used as a attack vector, Microsoft is working with our ecosystem partners, such as the Linux Foundation's OpenSSF, to enable OSS consumers to track packages back to their public sources.
We've identified that the following packages published to NPM do not report where sources can be found, typically accomplished by including a link to your GitHub repository in your package.json REPOSITORY field. This PR was created to add this value, ensuring future releases will include this provenance information.
Published NPM packages with repository information:

utf7

Is there anything that is needed to land this pull request ?

ls there anything else I can do to land this pull request ?

Is there anything that is needed to land this pull request?


Convert some utility classes to individual functions

We have some utility classes such as ContainerManager and MetadataController which handle some generic functionality that is needed in multiple places. This way these can easily be replaced with other classes fulfilling the same interface (actually MetadataController doesn't even have an interface but that should change if we keep it).
ContainerManager handles getting the identifier of a container that contains the input resource. This was made in a separate class in case other behaviour was needed than just slicing off the last part before the previous slash. But since there are already several places where the trailing slash is used to identify containers vs documents, I have my doubts we would ever have anything else there?
MetadataController handles de(serializing) metadata, and adding the LDP metadata (resource/container and containment). Same thing here: it seems unlikely suddenly a new ontology would be needed.
Removing them as classes would simplify the dependency injection configuration as the number of parameters would reduce for several classes..

But since there are already several places where the trailing slash is used to identify containers vs documents

Let's definitely make that an isContainer function then.
No objection to the rest.

Let's definitely make that an isContainer function then.

Utility function or part of the ContainerManager class? Second one would make sense if at some point we ever want to support another way of doing containers but would require injecting it in many classes.

I'm fine with utility; my main concern is that the piece of knowledge is only encoded in one place.


Disable Crop & Lock forced usage of Always on Top

Description of the new feature / enhancement
Change Crop & Lock to no longer cause forced usage of always-on-top.
Scenario when this would be used?
Sometimes I don't need the cropped Window to be permanently stuck in the top of the z-order and I just want it to behave like any other window.
Supporting information
And if I did want it always-on-top, there is already a PowerToy just for that optional case. It shouldn't be a forced requirement that you can't work around.
optionally the context menu of the cropped window could contain a "AOT" option, although again there's already a PowerToy for it so no need to build anything new to accomplish the same result.

/dup #29733

Closing as dupe.


Add a saner default for non-Lisps

I've added some configs so that default config for lisp and non-lisp are different, which is often the way we want.
This makes seamless interop with nvim-treesitter/nvim-treesitter-textobjects.

I've found that the non expression based languages may actually requires a new different type of evaluation, so maybe add a new command, evaluate statements to it maybe better.

I may close and reopen this with some other modifications, how do you think about evaluation for non-lisps?

Hmm I'm not sure if I'm on board with the idea of splitting into "lisp" and "non-lisp" unless it's absolutely the very last resort. Any split adds potentially double maintenance cost and worry as I break one and not another without realising. So I try to avoid special casing outside of the clients as much as possible.
I'd rather add hooks into the client interface that allows the individual clients to add their own selection logic that isn't based on well defined form indicators (like parens in lisps). So a hook you can define for a client that returns true if what you have can be evaluated as an expression or if the TS code should continue to walk further up.
That's just one idea off the top of my head, but I'd much rather extend the client design / interface than add custom things outside of that in the shared layer since I want that to remain as generic as possible, I don't want the outer shell of Conjure to know anything about the languages it's talking to, that's the point in the client interfacer really (which is only defined at the bottom of :h conjure which isn't great, I should codify it so it can be verified programmatically).

Well, I'll rework and you can see if it meets your requirements


Allow users to select between PDB2PQR versions

Alluded to here, we'd need to add options on the PDB2PQR configuration page to switch between PDB2PQR versions 2.1.1 and 3.1.0 (latest)

This can be in the form of a dropdown.
Adjust additional options based on version (e.g. removing PDB2PKA fields when PDB2PQR 3 is selected)

@sobolevnrm do you think version 3.1.0 is stable enough that we should just support it and drop version 2.1.1?
The reason I ask is that 2.1.1 requires Python 2.7 and I was trying to get us to strictly 3.X to make maintenance and deployments easier.
Is there anything in 2.1.1 that got dropped moving to 3.1.0?
Were there things added to 3.1.0 that @Eo300 would have to add features or GUI elements to support it?

@sobolevnrm do you think version 3.1.0 is stable enough that we should just support it and drop version 2.1.1?
The reason I ask is that 2.1.1 requires Python 2.7 and I was trying to get us to strictly 3.X to make maintenance and deployments easier.
Is there anything in 2.1.1 that got dropped moving to 3.1.0?
Were there things added to 3.1.0 that @Eo300 would have to add features or GUI elements to support it?

I think we should drop it soon but after we've confirmed that the AWS deployment works the same way as the current Azure deployment.
The change will require removal of the PDB2PKA radio button on the PDB2PQR web form.

The PDB2PKA option was removed as part of #26
@sobolevnrm now that we have PDB2PQR 2.1.1 working correctly on AWS, I think we could swap it out for version 3.1.0.
From the release notes of 3.1.0, I don't think there is anything we would have to change on the existing web interface (e.g., adding new options).
Do we need to support things like psize or dxcube command line options from the web interface?
If not, do you know of any changes that we would need to make to support version 3.1.0?

Sounds good to me.  Didn't we drop PDB2PKA in 3.*?  That would require a web interface change.

Due to Electrostatics/apbs-aws#62, we removed the PDB2PKA option from the interface already.  So we could try swapping in PDB2PQR 3 if the CLI is consistent (w/ the exception of PDB2PKA)

We dropped PDB2PQR 2.X so this is no longer needed.


Links to lower sections from top nav e.g. 'what...' and 'contribute...' cloudevents.io

What would you like to be added:
Links to lower sections from top nav e.g. 'what...' and 'contribute...'
Why is this needed:
A great design, key content could be easier to find.

@kitmacleod you want to see the menu that we have here:

In here:

?

Apologies @ricardozanini I was commented on the wrong site.


Question - How to replace multiple notifications with a single one?

Hi,
Many thanks for working on such a great framework! I've managed to get react-native-fcm working (react-native version 0.48.4) with remote notifications, both on iOS and Android devices (versions from iOS 9-11, Android 5-8), when the app is in foreground, background and not-running.
Sometimes it turns out I send multiple notifications to the user before they check them. And I was wondering if it's possible to replace multiple notifications with a single one, saying that there's multiple updates? This would ideally work in background, when the app is not running, and when it's been killed by the user (is that the same as not running?).
This is the format of the remote notifications I'm sending:
iOS:
{
            title: title,
            body: body,
            sound: "default",
            priority: "high",
            show_in_foreground: showinfg
}

Android:
{
            "custom_notification": {
                "body": body,
                "title": title,
                "icon":"ic_notif",
                "priority":"high",
                "sound":"default",
                "show_in_foreground": showinfg
                }
}

Actually collapse_key seems to be what I'm looking for
references:
https://github.com/evollu/react-native-fcm/issues/94
https://stackoverflow.com/questions/39782438/can-fcm-notification-on-android-overwrite-previous-one

or you can pass same id property, it should override the old one

Oh cool thanks, does the id property work for iOS and Android, and for remote notifications?

may be not for iOS. but sure for custom_notification

many thanks will give it a try


Understanding the executeCommand workflow

Hi,
Apologies if this is the wrong place to ask this, but the documentation is not clear on this. I'm trying to add a command to the VSCode Command Palette (or any other similar set up). I understand that the command "lives" client side, and simply when the user invokes it, the server picks it up and can respond on executeCommand. That works fine. But I want, when the user invokes the command, for a dialogue box (similar to a Git commit asking you for your username and password) to pop up and ask for additional information.
It seems, on the face of it, to be connection.sendRequest but I'm unsure how to use it, and the documentation, such as it is, around it is unclear.
Any help pointing me in the right direction would be appreciated.

You might want to create some custom methods?
https://stackoverflow.com/questions/51041337/vscode-language-client-extension-how-to-send-a-message-from-the-server-to-the

Hmm. That's kind of unfortunate -- that really chains the LSP to a specific client. Thanks though!


slow training speed even 20 steps

Hi,
I am running run_lm_finetuning.py code on a 1 million sentences .
It's taking more than 20 hours even for each epoch.
where as run_pretraining.py code from google-research/bert takes very less time.
What can be the reason?
How to resolve this?

We have new lm_fintetuning scripts thanks to @Rocketknight1 PR #392.
Maybe you can try these ones.
They are in ./examples/lm_finetuning/


Migrate job not showing up in ArgoCD

There is an issue with OpenFGA helm chart when it's used with ArgoCD.
Migrate job doesn't show up on ArgoCD and is not created by ArgoCD.
This is due to the default helm hooks in place in the migrate job defined in the default values.yaml of this helm charts:
https://github.com/openfga/helm-charts/blob/main/charts/openfga/values.yaml#L269-L271
migrate:
  annotations:
    helm.sh/hook: "post-install, post-upgrade, post-rollback, post-delete"
    helm.sh/hook-weight: "-5"
    helm.sh/hook-delete-policy: "before-hook-creation"

The current workaround is to disable them like this in one of your child charts:
migrate:
  annotations:
    helm.sh/hook: null
    helm.sh/hook-weight: null
    helm.sh/hook-delete-policy: null

Could we remove them ?

It seams that ArgoCD doesn't understand post-rollback. Removing it should fix the issue:
migrate:
  annotations:
    helm.sh/hook: "post-install, post-upgrade, post-delete"

Hello.
I found a way to make this work using ArgoCD's built-in sync waves.
I use Helm in Kustomize like this:
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization

namespace: openfga

helmCharts:
- name: openfga
  repo: https://openfga.github.io/helm-charts
  version: 0.2.3
  releaseName: openfga
  namespace: openfga
  valuesFile: values.yaml

And the full helm values:
replicaCount: 1

datastore:
  engine: postgres
  uriSecret: pg-uri

serviceAccount: # Requires existing before the migrations job
  annotations:
    argocd.argoproj.io/hook: PreSync
    argocd.argoproj.io/sync-wave: "-2"

migrate:
  annotations:
    argocd.argoproj.io/hook: PreSync
    argocd.argoproj.io/sync-wave: "-1"

For reference I'm using CNPG (postgres 14.11) as postgres operator and external-secrets-operator to create a shared secret for the pg username, password and uri.


track_published webhook event works incorrectly

Describe the bug
It is necessary to publish audio and video tracks in the system always, even if the microphone is muted at the entrance, and then, depending on the state, mute the track. Implemented subscription to the RoomEvent.LocalTrackPublished event, inside the handler - mute tracks depending on the state. Case: in state - mute microphone and video. So, the webhook receives information that the audio track is published, but the video track does not reach.
Server

Version: [1.1.0]
Environment: local env

Client

SDK: JS
Version: [1.4.3]

To Reproduce
Steps to reproduce the behavior:

Client join the conference with audio: false, video:false state
Backend recieve webhook track_published event

Expected behavior
webhook sends information about the publication of audio and video tracks
My guess is that the message processing for the hook happens after the RoomEvent.LocalTrackPublished event is dispatched. Have to hack with settimeout to mute a track after publishing

It's not possible for livekit to receive a muted track, due to the way pion handles incoming tracks. filed #1434 to keep track instead.


trace/dns: Add NumAnswers and Addresses to DNS trace events

Parse and return the IP address answers from A and AAAA DNS records.

Since BPF events are fixed-length, set an arbitrary limit on the number
of addresses. Include the total number of answers in the event so
the user can see when addresses were discarded.

For now, we are parsing IP addresses only for DNS answers
that use message compression
(see https://datatracker.ietf.org/doc/html/rfc1035#section-4.1.4).
This makes it much easier to calculate the size of each answer,
since the answer name is always two octets.
I believe most DNS servers use message compression to avoid
repeating the domain name in both the question and answer
sections.

Signed-off-by: Will Daly<EMAIL_ADDRESS>Signed-off-by: Mauricio Vásquez<EMAIL_ADDRESS>
All credit goes to @wedaly that did almost all the work as mentioned in https://github.com/inspektor-gadget/inspektor-gadget/issues/1415#issuecomment-1462937412, I only handled some nits and used a percpu map to increase the number of addresses we can send to user space.
Discussion
As mentioned in the commit message, only compressed DNS replies are supported. I tried to support non-compressesed replies but the verifier didn't like any of the ideas I had. I think we can move ahead and handle this in the future if needed.
Fixes #1415

It seems that I got a (double) chicken and egg situation on this one.

This modifies the ebpf-builder to use clang-15, but the ebpf-objects-checks test uses the pushed image that is different and the test fails.
This also modifies the dnstester, that image is pushed using a different tag #PR-merged but the tests use :latest by default. We should find a way for the tests to pick the right image automatically.

This also modifies the dnstester, that image is pushed using a different tag #PR-merged but the tests use :latest by default. We should find a way for the tests to pick the right image automatically.

Let me have a look into this but would it be possible to open a separate (smaller) PR for this as well?

I am not sure I understand.
Why not opening a PR dedicated to bump this?

This is more relevant for the dnstester given that the push is automated. However, I feel we should also push this image automatically (#1420) because it's not clear when we should push that image after changing it.

Before merging this one, shouldn't we deal with #1450?

IMHO we should not create a dependency there. It seems to me a proper solution for #1440 will require some time, so I'd prefer to move this forward.

Also, the TestTraceDns failed locally. What am I missing or doing wrong?

I believe you have to specify DNSTESTER_IMAGE from the PR while running the tests locally:
$ INTEGRATION_TESTS_PARAMS="-test.run TestTraceDns" make -C integration/ig/non-k8s DNSTESTER_IMAGE="ghcr.io/inspektor-gadget/dnstester:1448-merge" test-docker

Does it helps?

I tested it, and it never showed the answers, bpfEvent.Anaddrcount was always 0. I tested it in an AKS cluster and minikube and never saw the Address field:

Right, I wasn't able to get the addresses also when coredns is used because replies aren't compressed in all cases -> https://github.com/coredns/coredns/pull/2333.
Unfortunately it's something difficult to implement as I mentioned in the description of the PR, I'm wondering if in this case adding a "Compresseded bool" field to the event will help users understand what's going on.


Move shared resources to ./src/shared

And try to make them just plain functions of possible.

No need.


Add context parameter to methods that perform IO

Creating sessions, sending, and receiving messages all perform IO.
Removed context parameter from Prefetched() as it performs no IO.

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

Based on the following API review https://apiview.dev/Assemblies/Review/48a2525bba2f4784ab175622e2b40636

Fixes https://github.com/Azure/go-amqp/issues/147


Update CircleCI config

List dependencies separately.
Uninstall check merged in upstream meson, so it can be simplified.
libsolv doesn't need to be updated any more with Fedora 31.

This looks fine. Can you please just rename the commit?

Done.


Is it possible to create "template" field in array of elements?

I have some schema like this:
  schema: {
  title: "TOPICS AND RULES",
  disable_edit_json: true,
  disable_properties: true,
  disable_array_reorder: true,
  disable_collapse: true,
  type: "array",
  name: "some name",
  format: "tabs",
  items: {
            type: "object",
            format: "grid",
            uniqueItems: true,
            title: "RULE",
            properties: {
              path: {
                title: "ROOT Topic",
                type: "string",
                minLength: 4,
                defaut: "/hepek/"
              },
              type: {
                type: "string",
                enum: ["rc-switch", "ir-switch"],
                default: "rc-switch"
              },
              name: {
                type: "string"
              },
              message: {
                type: "string"
              },
              code: {
                type: "string"
              },
              path_would_be: {
                type: "string",
                description: "command for mqtt",
                template: "{{path}}",
                watch: {
                    path: 'item.path',
                }
              }
            }
          },
      "default": [{
        path: "/somepath/",
        name: "somename",
        code: "somecode"
      }],
    }

But i can't figure out if there is a way for "path_would_be" to be combination of previous values in same node?
Any idea???

You need to pass the index to your watch statement. For example, if you are going to watch the first element in the item for the path, you should use:
watch: {
     path: 'item.0.path',
 }

As a follow up, is there anyway to reference the particular item in the array that's doing the watching?
i.e.
watch: {
     path: 'item.i.path'
}


Separate Dockerfiles for development and production.

As discussed in issue #219, we may maintain only one official Docker image in DockerHub which is primarily catered towards development needs.
However, to make it easier for end users variable needs, we may provide two Dockerfiles. Namely, Prod.Dockerfile to build production Docker image and  Dev.Dockerfile to build development Docker image. The Readme.md file has been updated to reflect this idea.
The official hosted Docker image shall only be built from Dev.Dockerfile.
Actually, the only difference between Prod.Dockerfile and Dev.Dockerfile is whether we use RUN pip install smarts or COPY . /src.

This idea was in response to the "Update documentation for the use cases" checkbox in issue #219. Instead of simply explaining how to modify the Dockerfile to suit production, I thought of providing a sample production Dockerfile for reference.
What changes do you suggest?
We can drop this idea, if we think that multiple Dockerfiles bloat the codebase.


Traffic lights ZMQ API

Types of PR

[x] New Features
[x] Upgrade of existing features
[ ] Bugfix

Description
Adds ZMQ API for traffic lights control.

@hakuturu583 Typos should be fixed now. Sorry for that :bow:


Is there any way to generate a more user friends oid from Noray?

Right now, Noray gives an oid string like: 11abcdefABcdAbcAbC1AB
Is there any way to generate one with only 4 character? or 5? What is the algorithm used by noray to generate this? If it would be 4 characters only letter and numbers, that would be a lot of combinations I feel like.
If we had numbers + big letters: 36. 36^4 = 1679616
If we had numbers + big and small letters, 62. 62^4 = 14776336
And if these are reused, then it's more than enough for my use case for example and would be easier for users also to transmit this shorter code. The only thing would be to make absolutely sure that these are generated without duplicates unless they all run out(eg 1,2,3..99 and not 1,2,2,5,2,..,99). Even though not great if it were in order, wouldn't be the worst also.

Upon closer investigation, it seems the code that generates the id is in: host.entity.mjs and it calls nanoid. From their docs it seems nanoid accepts a parameter that tells it what length to generate the id(knowing that you might have collisions). Maybe we can have a config variable that goes into .env that defaults to 21(which is what nanoid uses by default) and also one that exposes the letters used by nanoid(which would default to useandom-26T198340PX75pxJACKVERYMINDBUSHWOLF_GQZbfghjklqvwyzrict which is what nanoid uses right now).
This way it would be backward compatible. Or we can make it if the env vars are empty, we set default values, this way is even safer that its backwards compatible.
And like this, users could easily override the settings and put something like:
length: 4
letters:<PHONE_NUMBER>ABCDEFGHIJKLMNOPQRSTUVWXYZ

Done in #48, thanks for the contribution!


Retrain RWCNet using Oasis Data set from Learn2Reg

Hi Vignesh,
@vigsivan @mrhardisty
Thank you for this nice work to solve the registration problem in medical imaging.
Basically, I was trying to retrain this model for Oasis data to reproduce the result. The steps in each stages are set based on the paper.
I have two questions.

I used NVIDIA TITAN RTX with 24 GB RAM , which took around 4 weeks to train the whole model.
How you managed to train on NVIDIA A100 - 32 GB in around 30 hours ? Can you please share the configuration/request hardware
allocations ?

After 4 weeks of training, I was not able to get the expected DSC value quoted in the paper.

Here are some of the tensor board graph and validation results.

Please feel free to write so as to find and avoid possible errors, I am making in this training and validation.
FYI,
The used training syntax for this repo is :
python l2r_train_eval.py   /home/rranabhat/OASIS/OASIS_dataset.json   train_config.json
and for validation  :
python eval.py  oasis_files_paper/checkpoints/data.json  eval_config.json
Later on all metrices were measured based on the Lear2Reg evaluation repo.
Thank you.
Best  regards
Raj

Hi @bhatrana , can you please share your config?

Re: gpu and training speeds, if you trained on Bender, it could be that it has slow filesystem access (maybe its a different issue). I was also able to leverage a faster GPU (A100 64GB) which made training a lot faster for me.

@vigsivan :
.
It seems, I made some change in https://github.com/vigsivan/RWCNet/blob/d90b48f7377b0b69dd655c625772be0328bf166b/networks.py#L354 to resolutions: List[int]=[4,4] due to ongoing cuda error in compute Canada, but later on forgot to put default one while training in bender.
I am training again the network with default value, will update the result by next week.
Is  GPU (A100 64GB) allocation based in compute Canada ?
Thanks. !!

@animesh-007 : This repo is based on Learn2Reg , https://learn2reg.grand-challenge.org/ .
You can do evaluation/test using eval.py which allows to store displacement field of each pair.
To quantify the results using several metrices, you can use the following repo which is also based on L2R.
https://github.com/MDL-UzL/L2R/tree/main/evaluation
Thanks

Thanks @bhatrana. The link you shared was very helpful. I was able to calculate metrics offline. But the evaluation script doesn’t give the Dice metric as reported in the RWCNet paper. Can you guide me on how I can calculate the Dice metric?

Please see the weights in the train_oasis folder regarding reproducibility of this issue. A100s are available on Narval.
I'm closing this issue, but feel free to follow-up if you come across anything else


Datacenter Outage

We have noticed some services are slowly coming back online by themselves, an ISP site visit is required by End of Day.
7:03 AM BST - 07/06/2019

We have noticed some services are slowly coming back online by themselves, an ISP site visit is required by End of Day.
7:03 AM BST - 07/06/2019

One Datacenter trip later and SkyHive is now back up and running


Flow cannot use a string enum as a string for polymorphic types

The basic gist is there's weird places where Array<EnumOfStrings> cannot work as a Array<string>.
Here's the try.
And a copy for easy reading:
type Enum =
  | 'foo'
  | 'bar'

function a(b: Array<string>): string {
  return b.join(', ')
}

const enumValue: Enum = 'foo'

type Nested = {
  e: Array<Enum>,
}

const nested: Nested = {
  e: [enumValue],
}

const enumValues = [enumValue]
// works alone
a([enumValue])
a(enumValues)
// does not work, but has no material difference
a(nested.e)
// Appears to be a workaround for some cases:
const extracted = a.e
a(extracted)
// I'm not sure why this is special and prevents the workaround.
// Perhaps a non-root context?
function doSomething(aa: Nested) {
  const extracted2 = aa.e
  return a(extracted2)
}

It's really weird that the workaround works outside of the function, but fails inside of the function.
My basic expectation here is that if I have something like a Array<'foo' | 'bar'> that I can still treat it like a Array<string> because all of the union's members are string literals.

This is because arrays are invariant. You need to use $ReadOnlyArray instead, which is covariant: https://flow.org/try/#0C4TwDgpgBAogdgVwLZQLwCgpQD5QOQBmA9kXpjvgEYCGATmegQnAMbACWRcU1AFJQC4oAEgBKEagBMA8nAA2IAIK1a1EAB4AzsFrs4AcwB8ASiHbdBqAG9ytCMAS1ulAHQArInt54ANPmNQ6AC+6OgsXNpQEIhIAGrUcggQQvDIaPjEpKGgkFAAchDaEJLpNljJUMqqGqlIhj7BoeFwkXCFwMVCBUUlqNbkFQDa0cjxiRAAug0hYRHAUTFjSZrpw4sJSRPoAPTbUADuRLQA1isJXBDofGujG5PGV7wjcXeaD7tQkkSFUHBE84cTn5KAh5gALagrP5QJDUDq6BKfdgEAgQOysS58No9FwQd57RRgSB0FbAIhQSjQagHI7HOhEZglYi0KCaIhIaAsSGFASzFrzCAADx01DYxXS1FxjyFIrFknxUAAkngUH95ppHNB9mCQFBgGD2CtDazICx2IjqHASmA7AA3aLAUlgrW0+mMlw7PYABTRELAZ1+XAAtLQSPNmh1hQB+RjMNicbhfADK7PsBoMvGo1C67WKATKUGakRlqjlACYJZLLlg7A4nDwnsLSx1JGWHiEgA


Update to CODEOWNERS

Overview/Summary
This PR updates the CODEOWNERS file as discussed internally over the past few weeks to allow all teams to be able to contribute and approve changes easily to required parts of the repo.
This PR fixes/adds/changes/removes

Updates CODEOWNERS file

Breaking Changes
None
Testing Evidence
Unable to test however this is just an update to a file that uses the standard Git path/file matching patterns and is in-line with what existed before and aligns as per GitHub docs: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation. - Not required as backend change and non-impacting to customers or ESLZ
[ ] Updated the "What's New?" wiki page (located: /docs/wiki/whats-new.md) - Not required as backend change and non-impacting to customers or ESLZ

Closing and will re-create as changing my downstream fork repo.


Include latest lat/lon based definition for forecast outputs

Update produce_op_forecasts.sh to allow new lat/lon based region and north-facing plot arguments via this script as introduced in this PR https://github.com/icenet-ai/icenet/pull/282.
https://github.com/icenet-ai/icenet/blob/9a70ce688e1196aaebf60df5829e5fbb722f32c5/icenet/plotting/forecast.py#L1393-L1401

While icenet-ai/icenet#283 is not fixed, the script should avoid attempting to output metric plots when lat/lon region bounds are specified, which are not currently implemented for metric computation.
Hence, as it stands, these metric plot outputs are based on either pan-Arctic or pan-Antarctic rather than user defined lat/lon region.
Add logic to skip these outputs for now.

Resolved by above PR.

Wanting to opt for user-specified Coordinate Reference System (CRS) instead of hard coding the CRS to use, want to allow specifying it when running the produce_op_forecasts.sh script.


improve error logging, and fix defualt casting

Signed-off-by: orweis<EMAIL_ADDRESS>

In case the user tries to override a default with a bad value (that fails to parse), OPAL should exit immediately with an error. Running with the default value in that case is certainly not what the user expects.

Sure that's fair; I'll fix accordingly.

Isn't it simpler to just set the correct types for the default values? (rather than basically extending decouple's behavior from the outside). (e.g. for POLICY_REPO_WEBHOOK_PARAMS: default=GitWebhookRequestParams({...}) instead of just the dict)

You are correct, but only when it comes to pydantic models - you forget other types that are being cast from strings: lists, enums, booleans, numbers, etc. etc.
Also I think it makes the config file itself less readable and cumbersome, you'd to mention the type 3 times (it's already 2 times now)


Check Achievement

client.achievement.activate('ACHIEVEMENT') seems to return true even if the achievement was previously activated. It would be awesome if this either a) had a way of only returning true if the achievement was newly activated, or if b) there was a way to check beforehand if the achievement had already been obtained.

I added an client.achievement.is_activated in 0.0.17 https://github.com/ceifa/steamworks.js/commit/f623ac5b5f767406e52a0876b25af47446b84648


ROX-25621: Add multi-arch Konflux builds to main image

Description
Per title, replaces https://github.com/stackrox/stackrox/pull/12054.
Tracking build issues in:

https://issues.redhat.com/browse/KFLUXBUGS-1523
https://redhat-internal.slack.com/archives/C04PZ7H0VA8/p1723056835366329

User-facing documentation

[x] CHANGELOG is updated OR update is not needed
[x] documentation PR is created and is linked above OR is not needed

Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing
No changes to automated tests.
How I validated my change
Only Konflux CI.

/retest main-build

/retest main-build

Quay flakes in https://console.redhat.com/application-pipeline/workspaces/rh-acs/applications/acs/pipelineruns/main-build-tv86k

/retest main-build

/retest main-build

Except for quay upload of clamav-scan results everything succeeded in https://console.redhat.com/application-pipeline/workspaces/rh-acs/applications/acs/pipelineruns/main-build-9cl8k
It's really a pity that I need to rerun and wait for it again.

/retest main-build

/retest operator-build

/retest main-build

Quay flakes in https://console.redhat.com/application-pipeline/workspaces/rh-acs/applications/acs/pipelineruns/main-build-g9892


updated seed.command.ts to use importUtils from typeorm

Updated loadDataSource function with import file utils from typeorm github repo, supports datasource promise from file

Currently seeing the owner is working on v6, so closing this pull request


Update asp-net-mvc.md

<!--
For Work In Progress Pull Requests, please use the Draft PR feature,
see https://github.blog/2019-02-14-introducing-draft-pull-requests/ for further details.
 For a timely review/response, please avoid force-pushing additional
 commits if your PR already received reviews or comments.

 Before submitting a Pull Request, please ensure you've done the following:
 - 📖 Read the Source Pocket Contributing Guide: https://github.com/sdtrdev/source-pocket/blob/main/CONTRIBUTING.md
 - 📖 Read the Source Pocket Code of Conduct: https://github.com/sdtrdev/source-pocket/blob/main/CODE_OF_CONDUCT.md
 - 👷♀ Create small PRs. In most cases this will be possible.
 - ✅ Provide tests for your changes.
 - 📝 Use descriptive commit messages.
 - 📗 Update any related documentation and include any relevant screenshots.

-->
What type of PR is this? (check all applicable)

[x] Added new link to category
[ ] New category added
[ ] Fixed broken link
[ ] Typo fix
[ ] Readme update
[ ] Optimization
[ ] Documentation update
[ ] Frontend update

Description
[optional] Related Issues & Documents
[optional] QA Instructions, Screenshots, Recordings
Please replace this line with instructions on how to test your changes, as well
as any relevant images for UI changes.
[optional] Anything Else?

Asp.NET Yeni Kaynaklar


Installer script possible sets up RDS incorrectly

Install Script
Summary
The install script seems to setup the RDS 'Roles' table with an incorrect type. roleId is set to an BIGINIT when i believe roleId is actually created as an UUID.
db.sql:

UserRolesServiceImpl.createUserRole:

Reproduce steps
Run the install script creating a fresh environment and see what the type of roleId is.
You can see a failure if you attempt to create a new Role in the web application.
Expected Results
I'm assuming it should be a VARCHAR() but maybe I'm missing something.
Actual Results
roleId is set to a BIGINT

We will check this and get back. The issue seems to be with the incorrect table mapped.

Ya it comes with the readonly and admin role but we were also trying to create a role to see if we could limit it further. It looks like an role either has read or write permissions and that is it. We mainly tried to create a role because we wanted to see if we could then look at limiting that role to say a certain asset group.

Role based access to an asset group is already in our pipeline , we shall be releasing it in upcoming versions.

Role based access to an asset group is already in our pipeline , we shall be releasing it in upcoming versions.

Awesome that is really good to hear! As Anil pointed out above this bug isn't really blocking anything because of the two roles already created so its not a big deal but we just saw it in investigating of role based access. Thanks for the information!

The table has been updated as part 1.1 release and so closing it


I found init.py always download face_yolov8n.pt ,if else function is not working. so I comment it .

Hello,
I comment download_model function in the  lastest version
commit<PHONE_NUMBER>b9661d4d3433bd6d4c098df5043cdf
Date:   Fri Dec 8 06:34:23 2023 -0300
Because  face_yolov8n.pt model is already here.Place in Both of these folders :
ComfyUI_windows_portable\ComfyUI\custom_nodes\DZ-FaceDetailer\yolo\face_yolov8n.pt
ComfyUI_windows_portable\ComfyUI\models\DZ-FaceDetailer\yolo\face_yolov8n.pt
BUT,  else function always download the model  at start comfyui run.bat
My Suggestion it just tell users where to download & put it to which folder ,is OK
no necessary to make it automatically ,keep it simple is OK.
console.log： just show all about this plugin
[tinyterraNodes] Loaded
FaceDetailer: Model dir already exists
FaceDetailer: Model doesnt exist
FaceDetailer: Downloading model
Downloading: 100%|███████████████████████████████████████████████████████████████| 5.92M/5.92M [00:02<00:00, 2.16MiB/s]
FaceDetailer: Model dowload finished
Searge-SDXL v4.3.1 in E:\AI\ComfyUI\ComfyUI-webui\ComfyUI_windows_portable\ComfyUI\custom_nodes\SeargeSDXL

@oliverban
Once you download model put this in yolo folder, just comment it in __init__py file , comfyui will not show red warnning again .

reboot comfyui, loading this plugin will skip model checking after this.
This is a simplistic, palliative approach,  not the best way I think.

thanks for the warning! I hadn't noticed it, I forgot to change the path in the download logic when checking if the model file exists.
In theory now it's working. I pushed the fix.