Malformed filter: 'CommonName -eq '

Hello,
I saw something similar that had been closed.  I read through it but it seems that my code is different.  I apologize if this is, in fact, an issue that has already been reported.
I'm using PSPKI and I'm getting the error below.
Malformed filter: 'CommonName -eq '
At C:\Program Files\WindowsPowerShell\Modules\PSPKI\server\Get-RequestRow.ps1:17 char:17

throw "Malformed filter: '$line'"

CategoryInfo : OperationStopped: (Malformed filter: 'CommonName -eq ':String) [], RuntimeException
FullyQualifiedErrorId : Malformed filter: 'CommonName -eq '

My code was working until the number of certificates expiring in 30 days grew, it's around 5950 now.  I think I may have overloaded it.
Can you tell me what might be going wrong and how I might get past this?  I would welcome ideas if there's something I can do in my script to correct the issue.
I think it's failing fails somewhere around the PRODGoal function.
This is the last output in my logfile:
Count of PROD certs expiring in 30 days:
5950
Count of PROD certs expriring in 30 days for which CN has been renewed in the last 8 months
Here's the complete script:
$logFile = "C:\program files\tsom\Patrol3\log\check_certs_30Day.ps1_"+(get-date).ToString("yyyy_MM_dd")+".log"
$date = get-date
echo ">>> Started capture_cert_info.ps1 at ${date}" >> $logFile

Remove Expiring_Certs_Reported files older than 3 days

foreach ($File in Get-ChildItem -Path F:\Expiring_Certs_Reported_30Day*.txt)
{
if ($File.CreationTime -lt ($(Get-Date).AddDays(-3)))
{
echo "Removing $File.FullName older than 3 days"
echo "Removing $File.FullName older than 3 days" >> $logFile
Remove-Item $File.FullName -force
}
}

Only runs on the active node of the CA cluster.
Verify that F: exists.  If F: does not exist, exit.

if (Test-Path F:)
{
$dateStrWithTime = (get-date).ToString("yyyy_MM_dd HH:mm:ss")
echo "$dateStrWithTime check_certs.ps1: This is the active cluster node: F:\ drive exists... continue processing."  >> $logFile
}
else
{
echo "$dateStrWithTime check_certs.ps1: This is NOT the active cluster node: F:\ drive does not exist, exiting check_certs_30day.ps1." >> $logFile
exit
}
$expiringCertsReported = "F:\Expiring_CERTS_Reported_30Day_"+(get-date).ToString("yyyy_MM_dd")+".txt"

Get Exclusion Lists:
F:\IOU_Exclude_TEST.txt is for TEST.
F:\IOU_Exclude_PROD.txt is for PROD.

Get contents of IOU_Exclude_TEST.txt

if (Test-Path F:\IOU_Exclude_TEST.txt)
{
$IOU_Exclude_TEST = Get-Content F:\IOU_Exclude_TEST.txt
}
else
{
$IOU_Exclude_TEST = $null
}
echo "TEST Exclusions:" >> $logFile
echo $IOU_Exclude_TEST >> $logFile
echo $IOU_Exclude_TEST.Count >> $logFile

Get contents of IOU_Exclude_PROD.txt

if (Test-Path F:\IOU_Exclude_PROD.txt)
{
$IOU_Exclude_PROD = Get-Content F:\IOU_Exclude_PROD.txt
}
else
{
$IOU_Exclude_PROD = $null
}
echo "PROD Exclusions:" >> $logFile
echo $IOU_Exclude_PROD >> $logFile
echo $IOU_Exclude_PROD.Count >> $logFile
$IOU_ExcludeList = $IOU_Exclude_TEST

Get contents of Expiring_CERTS_Reported.txt files

if (Test-Path F:\Expiring_CERTS_Reported_30Day*)
{
$Expiring_CERTS_Reported = Get-Content F:\Expiring_CERTS_Reported_30Day*
}
else
{
$Expiring_CERTS_Reported = $null
}
echo "Expiring CERTS  Reported:" >> $logFile
echo $Expiring_CERTS_Reported.Count >> $logFile

TEST

$CATEST = "WVSFZCHY.Support.Statefarm.Org"
Import-Module PSPKI

Get list of TEST certs expiring in 30 days.

$TESTcerts = Get-CertificationAuthority $CATEST | Get-IssuedRequest -Filter "CertificateTemplate -eq Web_Server_Internal_SSL_TEST_SHA256_v2" , "NotAfter -le $((Get-Date).AddDays(30))"
echo "Count of TEST certs expiring in 30 days:" >> $logFile
echo $TESTcerts.count >> $logFile
$count = $TESTcerts.count
$PromFile = "C:\Program Files\windows_exporter\textfile_inputs\TEST_Certificates_Expiring_In_30_days.prom"
$CoreText=@"
#HELP test_certificates_expiring_in_30_day
#TYPE test_certificates_expiring_in_30_days gauge
test_certificates_expiring_in_30_days
"@
$Combo = $CoreText, $count
$Combo + "rn" | Set-Content $PromFile -NoNewline

Make sure there are $TESTcerts to process

if ($TESTcerts.count -eq 0)
{
$OK_To_Continue = $false
echo "No TEST certs expiring in 30 days, bypassing TEST"  >> $logFile
}
else
{
$OK_To_Continue = $true
}
if ($OK_To_Continue)
{

Get list of TEST certs expiring in 30 days for which CN has been renewed in the last 8 months

Function TESTGoal
{
    foreach ($TESTserver in $TESTcerts)
    {
        get-CertificationAuthority "$CATEST" | Get-IssuedRequest -Filter "CommonName -eq $($TESTserver.CommonName)", "NotBefore -ge $((Get-Date).AddMonths(-8))"
    }
}

echo "Count of TEST certs expriring in 30 days for which CN has been renewed in the last 8 months:" >> $logFile
echo (TESTGoal).count >> $logFile
$count = (TESTGoal).count

$PromFile = "C:\Program Files\windows_exporter\textfile_inputs\TESTGoal.prom"

$CoreText=@"
#HELP TESTGoal
#TESTGoal gauge
TESTGoal
"@
$Combo = $CoreText, $count
$Combo + "rn" | Set-Content $PromFile -NoNewline

Make sure there are $TESTcerts to process

if ((TESTGoal).count -eq 0)
{
    $OK_To_Continue = $false
    echo "No TEST certs expiring in 30 days for which CN has been renewed in the last 8 months, bypassing TEST" >> $logFile
}
else
{
    $OK_To_Continue = $true
}

}

Compare the list of TEST certs renewed in 8 months to the list of certs expiring in 30 days.

if ($OK_To_Continue)
{
$xxx = Compare-Object -ReferenceObject ((TESTGoal).CommonName) -DifferenceObject $TESTcerts.CommonName
echo "Count of comparison of certs renewed in 8 months to certs expiring in 30 days:" >> $logFile
echo $xxx.count >> $logFile

Make sure there are $xxx to process

if ($xxx.count -eq 0)
{
    $OK_To_Continue = $false
    echo "There are no TEST certs renewed in 8 months to certs expiring in 30 days... bypassing TEST"  >> $logFile
}
else
{
    $OK_To_Continue = $true
}

}
if ($OK_To_Continue)
{

Get list of TEST certs expiring in 30 days that that have NOT been renewed.

$yyy = echo $xxx | findstr ">"

echo "TEST certs expiring in 30 days that have NOT been renewed:" >> $logFile
echo $yyy >> $logFile
echo "Count of TEST certs expiring in 30 days that have NOT been renewed:"  >> $logFile
echo $yyy.count >> $logFile
$count = $yyy.count

$PromFile = "C:\Program Files\windows_exporter\textfile_inputs\TEST_Certificates_Not_Renewed.prom.prom"
$CoreText=@"

#HELP test_certificates_not_renewed
#TYPE test_certificates_not_renewed gauge
test_certificates_not_renewed
"@
$Combo = $CoreText, $count
$Combo + "`r`n" | Set-Content $PromFile -NoNewline

if ($yyy.count -eq 0)
{
    $OK_To_Continue = $false
    echo "No TEST certs expiring in 30 days that have NOT been renewed... bypassing TEST"  >> $logFile
}
else
{
    $OK_To_Continue = $true
}

}
if ($OK_To_Continue)
{

Get unique list of TEST certs expiring in 30 days that that have NOT been renewed.

$zzz = echo $xxx | findstr ">" | select-string -pattern ">" | Sort-Object | Get-Unique

$zzz = $zzz -replace '=>',""
echo "Unique TEST certs expiring in 30 days that have NOT been renewed:" >> $logFile
echo $zzz >> $logFile
echo "Count of unique TEST certs expiring in 30 days that have NOT been renewed:"  >> $logFile
echo $zzz.count >> $logFile
$count = $zzz.count

$PromFile = "C:\Program Files\windows_exporter\textfile_inputs\Unique_TEST_Certificates_Not_Renewed.prom"
$CoreText=@"

#HELP unique_test_certificates_not_renewed
#TYPE unique_test_certificates_not_renewed gauge
unique_test_certificates_not_renewed
"@
$Combo = $CoreText, $count
$Combo + "`r`n" | Set-Content $PromFile -NoNewline

if ($zzz.count -eq 0)
{
    $OK_To_Continue = $false
    echo "No unique TEST certs expiring in 30 days that have NOT been renewed... bypassing TEST"  >> $logFile
}
else
{
    $OK_To_Continue = $true
}

}
if ($OK_To_Continue)
{

Loop through the unique list of certs that have not been renewed and build complete list of non-renewed certs for each and execute msend to generate an incident

foreach ($cn in $zzz)
{
    $cn = $cn.Trim()
    echo "Getting nonrenewed certs for ${cn}:" >> $logFile
    $cns = certutil -view -config WVSFZCHY.SUPPORT.STATEFARM.ORG\PKTESTCLSTR256NextGen -restrict "Issued Common Name=$cn,disposition==20" -out "Certificate Expiration Date, Issued Organization Unit, Issued Organization, Issued Common Name, Serial Number" csv | Select -Skip 1

Get details from first in the set (one or more certificates in the set).

    $FirstLine = $cns -split [Environment]::NewLine | Select-Object -first 1
    $CertificateExpirationDate,$IssuedOrganizationalUnit,$IssuedOrganization,$IssuedCommonName,$IssuedSerialNumber = $FirstLine.Split(',')

    if ($IssuedOrganizationalUnit -like "*+*")
    {
        echo "IssuedOrganizationalUnit contains +, stripping everything after +: $IssuedOrganizationalUnit" >> $logFile
        $WG = $IssuedOrganizationalUnit.Split('+')[0]
        $WG = $WG.ToUpper()
        $WG = $WG -replace('"','')
        echo "Resulting Issued Organizational Unit: $IssuedOrganizationalUnit" >> $logFile
    }
    else
    {
        $WG = $IssuedOrganizationalUnit
        $WG = $WG.ToUpper()
        $WG = $WG -replace('"','')
    }

    $ICN = $IssuedCommonName
    $IO = $IssuedCommonName

    if ($Expiring_CERTS_Reported -notcontains "${IssuedCommonName}|${CertificateExpirationDate}")
    {
        if ($IOU_ExcludeList -notcontains $WG)
        {
            if ($WG -NotMatch '^WG\d{4,5}$')
            {
                $wg_found = $WG -imatch "(WG\d{4,5}$)"
                if ($wg_found)
                {
                    $xxx = $WG
                    $WG = $matches[0]
                    echo "Bad Issued Organizational Unit: Replace $xxx with $WG" >> $logFile
                }
                else
                {
                    echo "Bad Issued Organizational Unit: Replace $WG with WG1505" >> $logFile
                    $WG = "WG1505"
                }
            }

            $msg = "CERT-30DAY: The certificate(s) on <$IssuedCommonName> is expiring on Certificate Expiration Date, <$CertificateExpirationDate>.  Please request a new certificate and replace it within 30 days.  Use procedures located here to request a new certificate: https://techguide.opr.statefarm.org/index.php/Requesting_Digital_Certificates.  Please also check your certificate repository for <$IssuedCommonName> for any other unbound certificates. Please send a Request for Service to WG1505 with the serial numbers below along with any others found to have them revoked when they are no longer in use. Certificate Owner Responsibilities and Expectations are documented here:  Server Type/Product Owner SSL Certificate Responsibilities <https://collab.sfcollab.org/sites/WSS005492/Pages/Certificate%20Owner%20Responsibilities.aspx?mode=none>.
"

            foreach ($line in $cns -split [Environment]::NewLine)
            {
                $CertificateExpirationDate,$IssuedOrganizationalUnit,$IssuedOrganization,$IssuedCommonName,$IssuedSerialNumber = $line.Split(',')
                $WG1 = $IssuedOrganizationalUnit
                $WG1 = $WG1.ToUpper()
                $WG1 = $WG1 -replace('"','')

                if ($IOU_ExcludeList -notcontains $WG1)
                {
                    $msg = $msg + "
The Configuration Item entered for this certificate was <$IssuedOrganization>.  The serial number of the certificate is <$IssuedSerialNumber>. The Issued Organizational Unit is <$IssuedOrganizationalUnit>.  The certificate is expiring on <$CertificateExpirationDate>."

                    $msg = $msg.Replace('"','')

echo "Certificate Expiration Date: $CertificateExpirationDate" >> $logFile
echo "Issued Organizatinal Unit:   $IssuedOrganizationalUnit" >> $logFile
echo "Issued Organization:         $IssuedOrganization" >> $logFile
echo "Issued Common Name:          $IssuedCommonName" >> $logFile
echo "Issued Serial Number:        $IssuedSerialNumber" >> $logFile
                    echo "$CertificateExpirationDate,$IssuedOrganizationalUnit,$IssuedOrganization,$IssuedCommonName,$IssuedSerialNumber" >> $logFile
                    echo "${IssuedCommonName}|${CertificateExpirationDate}" >> $logFile
                    echo "${IssuedCommonName}|${CertificateExpirationDate}" >> $expiringCertsReported
                }
                else
                {
                    echo "Issued Organizational Unit, $WG1, is on the Exclude list A" >> $logFile
                }
            }

Build msend arguments.

            $CMD = 'C:\Program Files\tsom\Agent\server\bin\msend.exe'
            $arg1 = '-v'
            $arg2 = '-n'
            $arg3 = 'local'
            $arg4 = '-a'
            $arg5 = 'MSEND_EVENT'
            $arg6 = '-r'
            $arg7 = 'WARNING'
            $arg8 = '-b'
            $arg9 = "mc_tool='check_certs.ps1';severity=WARNING;mc_tool_rule='check_certs.ps1';mc_origin_class='check_certs.ps1';application='CERT-30DAY';mc_host_class='CERT';mc_object='$WG';msg_group='CERT-30DAY';mc_host='$ICN';ci_name='$IO';send_incident='YES';wg_lookup='YES'"
            $arg10 = '-m'
            $arg11 = '"'+$msg+'"'

Execute msend for the current set (one or more certificates in the set).

            & $CMD $arg1 $arg2 $arg3 $arg4 $arg5 $arg6 $arg7 $arg8 $arg9 $arg10 $arg11
            echo "$CMD $arg1 $arg2 $arg3 $arg4 $arg5 $arg6 $arg7 $arg8 $arg9 $arg10 $arg11" >> $logFile
        }
        else
        {
            echo "Issued Organizational Unit, $WG, is on the Exclude list B" >> $logFile
        }
    }
    else
    {
        echo "${IssuedCommonName}|${CertificateExpirationDate} is on the Expiring_Certs_Reported list." >> $logFile
    }
}

}
$IOU_ExcludeList = $IOU_Exclude_PROD

PROD

$CAPROD = "WVSFZCH0.Support.Statefarm.Org"
Import-Module PSPKI

Get list of PROD certs expiring in 30 days.

$PRODcerts = Get-CertificationAuthority $CAPROD | Get-IssuedRequest -Filter "CertificateTemplate -eq Web_Server_Internal_SSL_PROD_SHA256_v2" , "NotAfter -le $((Get-Date).AddDays(30))"
echo "Count of PROD certs expiring in 30 days:" >> $logFile
echo $PRODcerts.count >> $logFile
$count = $PRODcerts.count
$PromFile = "C:\Program Files\windows_exporter\textfile_inputs\PROD_Certificates_Expiring_In_30_days.prom"
$CoreText=@"
#HELP prod_certificates_expiring_in_30_days
#TYPE prod_certificates_expiring_in_30_days gauge
prod_certificates_expiring_in_30_days
"@
$Combo = $CoreText, $count
$Combo + "rn" | Set-Content $PromFile -NoNewline

Make sure there are $PRODcerts to process

if ($PRODcerts.count -le 0)
{
$OK_To_Continue = $false
echo "No PROD certs expiring in 30 days, bypassing PROD"  >> $logFile
}
else
{
$OK_To_Continue = $true
}
if ($OK_To_Continue)
{

Get list of PROD certs for which CN has been renewed in the last 8 months

Function PRODGoal
{
    foreach ($PRODserver in $PRODcerts)
    {
        get-CertificationAuthority "$CAPROD" | Get-IssuedRequest -Filter "CommonName -eq $($PRODserver.CommonName)", "NotBefore -ge $((Get-Date).AddMonths(-8))"
    }
}

echo "Count of PROD certs expriring in 30 days for which CN has been renewed in the last 8 months" >> $logFile
echo (PRODGoal).count >> $logFile
$count = (PRODGoal).count

$PromFile = "C:\Program Files\windows_exporter\textfile_inputs\PRODGoal.prom"

$CoreText=@"
#HELP PRODGoal
#PRODGoal gauge
PRODGoal
"@
$Combo = $CoreText, $count
$Combo + "rn" | Set-Content $PromFile -NoNewline

Make sure there are $PRODcerts to process

if ((PRODGoal).count -le 0)
{
    $OK_To_Continue = $false
    echo "No PROD certs expiring in 30 days for which CN has been renewed in the last 8 months, bypassing PROD" >> $logFile
}
else
{
    $OK_To_Continue = $true
}

}

Compare the list of PROD certs renewed in 8 months to the list of certs expiring in 30 days.

if ($OK_To_Continue)
{
$xxx = Compare-Object -ReferenceObject ((PRODGoal).CommonName) -DifferenceObject $PRODcerts.CommonName
echo "Count of comparison of certs renewed in 8 months to certs expiring in 30 days:" >> $logFile
echo $xxx.count >> $logFile

Make sure there are $xxx to process

if ($xxx.count -le 0)
{
    $OK_To_Continue = $false
    echo "There are no PROD certs renewed in 8 months to certs expiring in 30 days... bypassing PROD"  >> $logFile
}
else
{
    $OK_To_Continue = $true
}

}
if ($OK_To_Continue)
{

Get list of PROD certs expiring in 30 days that that have NOT been renewed.

$yyy = echo $xxx | findstr ">"

echo "PROD certs expiring in 30 days that have NOT been renewed:" >> $logFile
echo $yyy >> $logFile
echo "Count of PROD certs expiring in 30 days that have NOT been renewed:"  >> $logFile
echo $yyy.count >> $logFile
$count = $yyy.count

$PromFile = "C:\Program Files\windows_exporter\textfile_inputs\PROD_Certificates_Not_Renewed.prom"
$CoreText=@"

#HELP prod_certificates_not_renewed
TYPE prod_certificates_not_renewed gauge
prod_certificates_not_renewed
"@
$Combo = $CoreText, $count
$Combo + "`r`n" | Set-Content $PromFile -NoNewline

if ($yyy.count -le 0)
{
    $OK_To_Continue = $false
    echo "No PROD certs expiring in 30 days that have NOT been renewed... bypassing PROD"  >> $logFile
}
else
{
    $OK_To_Continue = $true
}

}
if ($OK_To_Continue)
{

Get unique list of PROD certs expiring in 30 days that that have NOT been renewed.

$zzz = echo $xxx | findstr ">" | select-string -pattern ">" | Sort-Object | Get-Unique

$zzz = $zzz -replace '=>',""
echo "Unique PROD certs expiring in 30 days that have NOT been renewed:" >> $logFile
echo $zzz >> $logFile
echo "Count of unique PROD certs expiring in 30 days that have NOT been renewed:"  >> $logFile
echo $zzz.count >> $logFile
$count = $zzz.count

$PromFile = "C:\Program Files\windows_exporter\textfile_inputs\Unique_PROD_Certificates_Not_Renewed.prom"
$CoreText=@"

#HELP unique_prod_certificates_not_renewed
#TYPE unique_prod_certificates_not_renewed gauge
unique_prod_certificates_not_renewed
"@
$Combo = $CoreText, $count
$Combo + "`r`n" | Set-Content $PromFile -NoNewline

if ($zzz.count -le 0)
{
    $OK_To_Continue = $false
    echo "No unique PROD certs expiring in 30 days that have NOT been renewed... bypassing PROD"  >> $logFile
}
else
{
    $OK_To_Continue = $true
}

}
if ($OK_To_Continue)
{

Loop through the unique list of certs that have not been renewed and build complete list of non-renewed certs for each and execute msend to generate an incident

foreach ($cn in $zzz)
{
    $cn = $cn.Trim()
    echo "Getting nonrenewed certs for ${cn}:" >> $logFile
    $cns = certutil -view -config WVSFZCH0.SUPPORT.STATEFARM.ORG\PKMANCLSTR256NEXTGEN -restrict "Issued Common Name=$cn,disposition==20" -out "Certificate Expiration Date, Issued Organization Unit, Issued Organization, Issued Common Name, Serial Number" csv | Select -Skip 1

Get details from first in the set (one or more certificates in the set).

    $FirstLine = $cns -split [Environment]::NewLine | Select-Object -first 1
    $CertificateExpirationDate,$IssuedOrganizationalUnit,$IssuedOrganization,$IssuedCommonName,$IssuedSerialNumber = $FirstLine.Split(',')

    if ($IssuedOrganizationalUnit -like "*+*")
    {
        echo "IssuedOrganizationalUnit contains +, stripping everything after +: $IssuedOrganizationalUnit" >> $logFile
        $WG = $IssuedOrganizationalUnit.Split('+')[0]
        $WG = $WG.ToUpper()
        $WG = $WG -replace('"','')
        echo "Resulting Issued Organizational Unit: $IssuedOrganizationalUnit" >> $logFile
    }
    else
    {
        $WG = $IssuedOrganizationalUnit
        $WG = $WG.ToUpper()
        $WG = $WG -replace('"','')
    }
    $ICN = $IssuedCommonName
    $IO = $IssuedCommonName

    if ($Expiring_CERTS_Reported -notcontains "${IssuedCommonName}|${CertificateExpirationDate}")
    {
        if ($IOU_ExcludeList -notcontains $WG)
        {
            if ($WG -NotMatch '^WG\d{4,5}$')
            {
                $wg_found = $WG -imatch "(WG\d{4,5}$)"
                if ($wg_found)
                {
                    $xxx = $WG
                    $WG = $matches[0]
                    echo "Bad Issued Organizational Unit: Replace $xxx with $WG" >> $logFile
                }
                else
                {
                    echo "Bad Issued Organizational Unit: Replace $WG with WG1505" >> $logFile
                    $WG = "WG1505"
                }
            }

            $msg = "CERT-30DAY: The certificate(s) on <$IssuedCommonName> is expiring on Certificate Expiration Date, <$CertificateExpirationDate>.  Please request a new certificate and replace it within 30 days.  Use procedures located here to request a new certificate: https://techguide.opr.statefarm.org/index.php/Requesting_Digital_Certificates.  Please also check your certificate repository for <$IssuedCommonName> for any other unbound certificates. Please send a Request for Service to WG1505 with the serial numbers below along with any others found to have them revoked when they are no longer in use. Certificate Owner Responsibilities and Expectations are documented here:  Server Type/Product Owner SSL Certificate Responsibilities <https://collab.sfcollab.org/sites/WSS005492/Pages/Certificate%20Owner%20Responsibilities.aspx?mode=none>.
"

            foreach ($line in $cns -split [Environment]::NewLine)
            {
                $CertificateExpirationDate,$IssuedOrganizationalUnit,$IssuedOrganization,$IssuedCommonName,$IssuedSerialNumber = $line.Split(',')
                $WG1 = $IssuedOrganizationalUnit
                $WG1 = $WG1.ToUpper()
                $WG1 = $WG1 -replace('"','')

                if ($IOU_ExcludeList -notcontains $WG1)
                {
                    $msg = $msg + "
The Configuration Item entered for this certificate was <$IssuedOrganization>.  The serial number of the certificate is <$IssuedSerialNumber>. The Issued Organizational Unit is <$IssuedOrganizationalUnit>.  The certificate is expiring on <$CertificateExpirationDate>."

                    $msg = $msg.Replace('"','')

echo "Certificate Expiration Date: $CertificateExpirationDate" >> $logFile
echo "Issued Organizatinal Unit:   $IssuedOrganizationalUnit" >> $logFile
echo "Issued Organization:         $IssuedOrganization" >> $logFile
echo "Issued Common Name:          $IssuedCommonName" >> $logFile
echo "Issued Serial Number:        $IssuedSerialNumber" >> $logFile
                    echo "$CertificateExpirationDate,$IssuedOrganizationalUnit,$IssuedOrganization,$IssuedCommonName,$IssuedSerialNumber" >> $logFile
                    echo "${IssuedCommonName}|${CertificateExpirationDate}" >> $logFile
                    echo "${IssuedCommonName}|${CertificateExpirationDate}" >> $expiringCertsReported
                }
                else
                {
                    echo "Issued Organizational Unit, $WG1, is on the Exclude list C" >> $logFile
                }
            }

Build msend arguments.

            $CMD = 'C:\Program Files\tsom\Agent\server\bin\msend.exe'
            $arg1 = '-v'
            $arg2 = '-n'
            $arg3 = 'local'
            $arg4 = '-a'
            $arg5 = 'MSEND_EVENT'
            $arg6 = '-r'
            $arg7 = 'WARNING'
            $arg8 = '-b'
            $arg9 = "mc_tool='check_certs.ps1';severity=WARNING;mc_tool_rule='check_certs.ps1';mc_origin_class='check_certs.ps1';application='CERT-30DAY';mc_host_class='CERT';mc_object='$WG';msg_group='CERT-30DAY';mc_host='$ICN';ci_name='$IO';send_incident='YES';wg_lookup='YES'"
            $arg10 = '-m'
            $arg11 = '"'+$msg+'"'

Execute msend for the current set (one or more certificates in the set).

            & $CMD $arg1 $arg2 $arg3 $arg4 $arg5 $arg6 $arg7 $arg8 $arg9 $arg10 $arg11
            echo "$CMD $arg1 $arg2 $arg3 $arg4 $arg5 $arg6 $arg7 $arg8 $arg9 $arg10 $arg11" >> $logFile
        }
        else
        {
            echo "Issued Organizational Unit, $WG, is on the Exclude list D" >> $logFile
        }
    }
    else
    {
        echo "${IssuedCommonName}|${CertificateExpirationDate} is on the Expiring_Certs_Reported list." >> $logFile
    }
}

}
echo " " >> $logFile
$date = get-date
echo ">>> Finished check_certs_30day.ps1 at ${date}" >> $logFile

Unfortunately, I can't debug your own scripts. However, the problem looks like in the missing filer qualifier. I see you are calling Get-IssuedRequest here:
foreach ($TESTserver in $TESTcerts)
    {
        get-CertificationAuthority "$CATEST" | Get-IssuedRequest -Filter "CommonName -eq $($TESTserver.CommonName)", "NotBefore -ge $((Get-Date).AddMonths(-8))"
    }

some of elements in TESTcerts collection have missing, null or empty CommonName property. You need to ensure that this property exist and is non-null.

Hello Vadims,
Thank you for your analysis.  I’m working to implement your suggestion.  It didn’t occur to me that some of the fields would be NULL or mal-formed.  I’m winding my way through the code putting in conditional logic that you suggested.
Thank you again for your help with this.
Garland Smith
From: Vadims Podans @.>
Sent: Monday, June 26, 2023 4:31 AM
To: PKISolutions/PSPKI @.>
Cc: Garland Smith @.>; Author @.>
Subject: [EXTERNAL] Re: [PKISolutions/PSPKI] Malformed filter: 'CommonName -eq ' (Issue #191)
Unfortunately, I can't debug your own scripts. However, the problem looks like in the missing filer qualifier. I see you are calling Get-IssuedRequest here:
foreach ($TESTserver in $TESTcerts)
{

    get-CertificationAuthority "$CATEST" | Get-IssuedRequest -Filter "CommonName -eq $($TESTserver.CommonName)", "NotBefore -ge $((Get-Date).AddMonths(-8))"

}

some of elements in TESTcerts collection have missing, null or empty CommonName property. You need to ensure that this property exist and is non-null.
—
Reply to this email directly, view it on GitHubhttps://github.com/PKISolutions/PSPKI/issues/191#issuecomment-1607276317, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BAZVKT22CANVWP66FR6P3NDXNFXIZANCNFSM6AAAAAAZSWTFBU.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Closing as a non-code-issue.


Fix identify track

Okay so we were not sending user data and track events in accordance to Kahuna's Docs.
I also got on call with 4 of their engineers to comb through this PR. They gave it a green light.
I added a lot of in line comments for why I'm doing certain things.
Notable Changes

Removal of username was requested by Kahuna. I think we originally thought this was an important trait to send to Kahuna but they said it doesn't do anything really. They also said it won't break reports.

Changing the name of the event to from start to user_created_or_updated_segment was also requested by Kahuna since the name doesn't really have any meaning or feature dependency and doesn't break reports since these are .identify() calls.

One key thing is that we are using Kahuna's Intelligent Events feature for .track() calls which is only available via their REST api and NOT with the mobile SDKs.

So that should be noted in our docs.
Let me know what you guys think. I'd like to get this merged by end of Friday and then deployed next Monday :)
@sperand-io @f2prateek

ping?

LGTM


Installation link in the README is broken

The link "installation documentation" in the readme file does not load.

Thanks a lot for catching that.
I'll correct it first thing in the morning ;-)

The link in the third paragraph See the installation documentation for more information is still broken. #documentationreview ;)

My bad, 3 other links to the installation documentation were broken too...


Unable to use proxyRequest in router eventHandler

Environment
"h3": "^1.0.1",
"nuxt": "^3.0.0",

Reproduction
Probably not a bug but a problem with my implementation, but help would be appreciated.
I am using h3 to create a proxy server - to send some /api requests to a back-end server. The API server is not publicly accessible.
I've created a router (createRouter()) and am adding routes via router.use()
It works fine but the body of a POST request is dropped. I've been using sendProxy() and saw that proxyRequest() is recommended to ensure headers and POST body are sent on to the back-end server.
However, within the eventHandler() returning a proxyRequest() results in none of my /api requests working.
eg
router.use(
    '/api/foo/**',
    eventHandler((event) => {
        if (isRequestAuthenticated(event)) {
            return proxyRequest(
                event,
                'https://backend.example.com/api/bar'
            )
        }
    })
)

Describe the bug
Replacing sendProxy with proxyRequest results in a 404 error from the router
Additional context
No response
Logs
No response

Any advice on this please? I want to proxy all HTTP methods (GET, POST, PUT, DELETE) to a back-end server and can only proxy GET at the moment. Other methods are changed to GET requests plus any body payload is lost.

@chrisnoden
Would you provide a reproduction in which vanilla h3 app without Nuxt is conneted to a publically accesible backend server, so that we could find where the root cause is: h3, Nuxt, or your own infra?

By the way, as far as I tested instantly, h3's proxyRequest works as expected and request information is sent to the backend server without loss (e.g. request body).

I don't think the issue is reproducible without Nuxt v3 in the equation.

After much testing, I think the issue is to do with my local setup. Node fetch or H3 does not like that I am proxying the requests (for dev) to a local docker server which has a Lets Encrypt SSL. Even with NODE_TLS_REJECT_UNAUTHORIZED=0 set the proxyRequest requests don't hit the server - at best I get a 404. I can curl the same request from the dev machine just fine.
I'm still trying out different permutations - but if I modify the target hostname to webhook.site for example it works just fine - but changing only the hostname of the target to a locally resolvable host (my /etc/hosts has the entry) and I get a 404. As mentioned, testing via curl or just using GET requests via sendProxy and it it works.

I have been able to replace H3 with Express and it works, so I'll stick with that.


Popover: add component docs/demos

We should add component docs/demos when Popover is released. Blocked by https://github.com/driftyco/ionic/issues/5420

[x] Component Docs
[x] Component Demo
[ ] Theming Default Mode Configuration (http://ionicframework.com/docs/v2/theming/platform-specific-styles/)
Defaults will be here: https://github.com/driftyco/ionic/blob/2.0/src/config/modes.ts

Added the demo on the beta 8 branch in the preview-app: https://github.com/driftyco/ionic-preview-app/tree/feature/beta-8

Component docs added.

It is done.
https://github.com/driftyco/ionic-site/commit/f8973d380e02502d7aacffac61fc2f25ba704ae3


The flow with example files fails

verifyBamID used in the docker container seems to be  verifyBamID2, which is known to be much less tolerant to low depth. The workflow with default  sample files fails at the contamination detection stage.

Also seeing the workflow failing on CheckContamination.
Get warnings for insufficient markings and then errors during delocalization, although not sure if those are related. The following is from the end of CheckContamination.log:
[...]
NOTICE - Process chr22:50745507-50745507...
NOTICE - Process chr22:50774185-50774185...
NOTICE - Number of marker in Reference Matrix:99976
NOTICE - Number of marker shared with input file:331
NOTICE - Mean Depth:1.003021
NOTICE - SD Depth:0.054882
NOTICE - 330 SNP markers remained after sanity check.
WARNING -
Insufficient Available markers, check input bam depth distribution in output pileup file after specifying --OutputPileup
2019/08/21 22:33:10 Starting delocalization.
2019/08/21 22:33:11 Delocalizing output /cromwell_root/NA12878_PLUMBING.preBqsr.selfSM -> gs:////work/WholeGenomeGermlineSingleSample/795f578d-3944-406f-b15e-933c4043f65e/call-UnmappedBamToAlignedBam/UnmappedBamToAlignedBam/7197d913-456d-411d-bf05-1164c1549aeb/call-CheckContamination/NA12878_PLUMBING.preBqsr.selfSM
2019/08/21 22:33:14 rm -f $HOME/.config/gcloud/gce && gsutil   cp /cromwell_root/NA12878_PLUMBING.preBqsr.selfSM gs:////work/WholeGenomeGermlineSingleSample/795f578d-3944-406f-b15e-933c4043f65e/call-UnmappedBamToAlignedBam/UnmappedBamToAlignedBam/7197d913-456d-411d-bf05-1164c1549aeb/call-CheckContamination/ failed
CommandException: No URLs matched: /cromwell_root/NA12878_PLUMBING.preBqsr.selfSM
2019/08/21 22:33:14 Waiting 5 seconds and retrying
2019/08/21 22:33:20 rm -f $HOME/.config/gcloud/gce && gsutil   cp /cromwell_root/NA12878_PLUMBING.preBqsr.selfSM gs:////work/WholeGenomeGermlineSingleSample/795f578d-3944-406f-b15e-933c4043f65e/call-UnmappedBamToAlignedBam/UnmappedBamToAlignedBam/7197d913-456d-411d-bf05-1164c1549aeb/call-CheckContamination/ failed
CommandException: No URLs matched: /cromwell_root/NA12878_PLUMBING.preBqsr.selfSM
2019/08/21 22:33:20 Waiting 5 seconds and retrying
2019/08/21 22:33:26 rm -f $HOME/.config/gcloud/gce && gsutil   cp /cromwell_root/NA12878_PLUMBING.preBqsr.selfSM gs:////work/WholeGenomeGermlineSingleSample/795f578d-3944-406f-b15e-933c4043f65e/call-UnmappedBamToAlignedBam/UnmappedBamToAlignedBam/7197d913-456d-411d-bf05-1164c1549aeb/call-CheckContamination/ failed
CommandException: No URLs matched: /cromwell_root/NA12878_PLUMBING.preBqsr.selfSM
2019/08/21 22:33:28 Delocalizing output /cromwell_root/stdout -> gs:////work/WholeGenomeGermlineSingleSample/795f578d-3944-406f-b15e-933c4043f65e/call-UnmappedBamToAlignedBam/UnmappedBamToAlignedBam/7197d913-456d-411d-bf05-1164c1549aeb/call-CheckContamination/stdout
2019/08/21 22:33:32 Delocalizing output /cromwell_root/stderr -> gs:////work/WholeGenomeGermlineSingleSample/795f578d-3944-406f-b15e-933c4043f65e/call-UnmappedBamToAlignedBam/UnmappedBamToAlignedBam/7197d913-456d-411d-bf05-1164c1549aeb/call-CheckContamination/stderr
2019/08/21 22:33:37 Delocalizing output /cromwell_root/rc -> gs:////work/WholeGenomeGermlineSingleSample/795f578d-3944-406f-b15e-933c4043f65e/call-UnmappedBamToAlignedBam/UnmappedBamToAlignedBam/7197d913-456d-411d-bf05-1164c1549aeb/call-CheckContamination/rc

Yes this is exactly the error I had. Most likely, veryfyBamID exist abnormally without creating the expected output - *.selfSM file. What you see is an error to copy the file back to the bucket.

Any updates on this? I get the same error.


refactor: ♻ use slash commands and JSON files for channelmanager

replace !csvparse with /csvparse
merge !deleteclassroles and !deleteclass into /deleteclasses
remove prefix from code and docs
change /csvparse to use JSON from compsci-class-scraper and rename it to /importclasses
improve performance of /deleteclasses and /importclasses (about a 2x speedup)
/deleteclasses and /importclasses now print display how many classes and roles where created and deleted
remove the old csv files and quotes.json from git

Chiara says I can merge now :)


Push Data

Hello, kudos for the work .
I have some free time to help, specially with pushing data to the device (user) to be honest. Do you have something or can i just open a PR ?

Hi @spinolacastro can you pls provide the PR for creating users?


did not work with phasmophobia

Tried playing phasmophobia and was stuck on a black screen. tried it a few times then reverted back to the original .exe and worked fine.

you need to launch steamvr first via launching oculusdash, aka plug your headset in and load into steamvr

I did launch it from inside steamvr, but ill try again today.

@Jamesdabeast If you are still experiencing this problem, did you set your default OpenXR runtime to SteamVR?

@UnusualNorm yes, im not sure how to change the runtime, but Vrchat and Pavlov work fine.

@Jamesdabeast You can find the settings by showing advanced settings, then under the "developer" tab you find "Current OpenXR Runtime"
You might want to just double check.

@UnusualNorm Thank you, that worked!


Adds Mesh Tutorial

Signed-off-by: Franco Cipollone<EMAIL_ADDRESS>🎉 New feature
The mesh tutorial here created is migrated from Google Doc to an actual tutorial in ign-gazebo
Tutorial Author: @RafiAbdullah
Some pics:

Test it
I share here the generated HTML  in case you want to take a look:
model_and_optimize_meshes_tutorial.zip
After extracting the files you can use your favorite browser by doing: (e.g. using google chrome)
google-chrome model_and_optimize_meshes.html

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[x] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Thanks for pushing the changes! @chapulina


Close #1575: management calls on server entities to configure stat collector

Also, upgraded Ehcache versions.
Depends on Terracotta-OSS/terracotta-platform#182 (and a 5.0.11.beta release of tc-platform)

test this please

test this please

test thi please

test this please

test this please
an integration test (HA) failed in JDK7

test this please


"svelte resolve configuration that has conflicting results", but svelte exports condition is used

Describe the bug
I am receiving a warning and a link to a doc to resolve it, but the steps to resolve it were already taken.
package.json:
	"exports": {
		".": {
			"types": "./dist/index.d.ts",
			"svelte": "./dist/index.js",
			"default": "./dist/index.js"
		},
		"./api": {
			"types": "./dist/api/index.d.ts",
			"svelte": "./dist/api/index.js",
			"default": "./dist/api/index.js"
		},
		"./components": {
			"types": "./dist/components/index.d.ts",
			"svelte": "./dist/components/index.js",
			"default": "./dist/components/index.js"
		},
		"./helpers": {
			"types": "./dist/helpers/index.d.ts",
			"svelte": "./dist/helpers/index.js",
			"default": "./dist/helpers/index.js"
		},
		"./i18n": {
			"types": "./dist/i18n/index.d.ts",
			"svelte": "./dist/i18n/index.js",
			"default": "./dist/i18n/index.js"
		},
		"./layouts": {
			"types": "./dist/layouts/index.d.ts",
			"svelte": "./dist/layouts/index.js",
			"default": "./dist/layouts/index.js"
		},
		"./types": {
			"types": "./dist/types/index.d.ts",
			"svelte": "./dist/types/index.js",
			"default": "./dist/types/index.js"
		}
	},
	"svelte": "./dist/index.js",
	"types": "./dist/index.d.ts",
	"type": "module"
}

Reproduction URL
N/A
Reproduction

Create a Svelte library with ~9141 lines of code
Follow https://kit.svelte.dev/docs/packaging
Import that library

Logs
10:25:30 a.m. [vite-plugin-svelte] WARNING: The following packages use a svelte resolve configuration in package.json that has conflicting results and is going to cause problems future.

Please see https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#conflicts-in-svelte-resolve for details.

System Info
System:
    OS: Linux 6.2 Pop!_OS 22.04 LTS
    CPU: (12) x64 AMD Ryzen 5 5600X 6-Core Processor
    Memory: 20.78 GB / 31.26 GB
    Container: Yes
    Shell: 5.8.1 - /usr/bin/zsh
  Binaries:
    Node: 20.3.1 - /usr/bin/node
    npm: 9.7.2 - ~/.local/share/pnpm/npm
    pnpm: 8.6.5 - ~/.local/share/pnpm/pnpm
  Browsers:
    Chrome: 114.0.5735.198
  npmPackages:
    @sveltejs/adapter-auto: ^2.1.0 => 2.1.0
    @sveltejs/adapter-node: ^1.2.4 => 1.2.4
    @sveltejs/adapter-static: ^2.0.2 => 2.0.2
    @sveltejs/kit: ^1.20.2 => 1.20.2
    svelte: ^3.59.1 => 3.59.1
    vite: ^4.3.9 => 4.3.9

We'd need a repro to investigate this. Can you also show the logs for DEBUG="vite:vite-plugin-svelte" <your start command here>? That should log out this for us to investigate.

Hi, it looks like the issue went away on its own. I suspect an older version was in my node_modules, but I can't explain why or how.
In any case, I can't reproduce it today, so I'll close this. If I get that warning again, I'll be sure to post it with DEBUG="vite:vite-plugin-svelte". Thanks!


[Oracle] Table Index cannot be deleted

System information:

Operating system (distribution) and version Win10
DBeaver version 7.1.2EA

Connection specification:

Database name and version Oracle

Steps to reproduce, if exist:

Open Indexes tab for a table (e.g. in schema HR)
Create an index
Drop Index via UI

Actual result:

Error appears ORA-01418: specified index does not exist
Wrong data in Declaration tab
Index is not deleted

It works for current user schema only. Seems to be a specific of test environment.


Why require Ord on map and mapR?

Sorry if I'm missing something obvious, but why do you have the Ord constraint on map and mapR?

Data.Map.mapKeys has an Ord constraint.

To expand, a Bimap is a pair of maps, one from a to b, and the other from b to a. Whenever a key or value type is changed, it must have an Ord constraint for the map keyed on that type to work.

Thanks, I understand.  I was thinking, stupidly, that you wouldn't need to map across the keys for some reason since you store both maps.


Feature/add cookies pref

What
Updated Regex
Added cookies-pref.js
Updated .yml files to use more recent version of node
How to review
LGTM
Who can review
Anyone

Just one last thing...pls can you update the .nvmrc file to match the node version used in CI? Then devs and CI will all be using the same version :)

Done :)


DependencyCI badge links to broken page

(I have clicked the request approval button)
Does it make sense to have it in the base? (Especially since we don't have dependencies for the example.)

Fixed the link.  Now how we should handle / document is another question entirely :)


[types] Cluster client ByID returns error

When using the management client to retrieve a cluster ByID the following error is returned.
FATA[0000] Resource type [cluster] can not be looked up by ID
https://github.com/rancher/types/blob/master/client/management/v3/zz_generated_cluster.go#L90

@dramich is this fixed already?

@deniseschannon Yes, this looks to be fixed.

Ability to return clusterClient ById is used my --add-node command which is able to provide docker run command needed to register host for a given clusterid.


fix: Default answer disappearing in authoring [PT-185100790]

The default answer was disappearing in open response authoring when the required checkbox was selected as it was not specified in the UI order AND the authoring iframe useAutoHeight() hook was not updating the height when the prediction feeback field showed after required was selected.
This change sets the UI order and makes the prediction feedback always show at the bottom of the form so it is not cut-off from view.

Merged after reply from @scytacki on Slack:

Yeah it looks fine. :+1:


Unknown simulation failure (with VCell)

This run failed with VCell for no clear reason
https://run.biosimulations.dev/simulations/60f13090c9fb54c29d9f0cc3

StatusReason: Checking status for job with id 3176413 for simulation 60f13090c9fb54c29d9f0cc3: Status is FAILED
The job on the hpc failed. I will debug further

The actual job on the hpc passed, so must have been an error connecting to HPC to check status.  Unfortunately, no errors were thrown so there is no log

The rerun worked, but there seem to be issues with the results: https://run.biosimulations.dev/simulations/60f5883b72ff0120ba5ce11e#tab=viz

API is returning results correctly, so seems to be frontend issue:
https://run.api.biosimulations.dev/results/60f5883b72ff0120ba5ce11e?includeData=true

I'll take a look at the rerun.

The Vega visualization for the rerun is failing because https://run.api.biosimulations.dev/results/60f5883b72ff0120ba5ce11e/simulation_1.sedml%2Freport?includeData=true is failing.

I created three issues for VCell about this. Turns out that VCell doesn't save plot data consistently. The result is different, depending on whether reports are also defined or not. I set VCell to validated=False until this is fixed. This will avoid users encountering a simulation tool that produces inconsistent data.

This should be handled automatically by the submission of the simulator


Modify TapeView to allow SetNextNode to be yielded after everything

As of now, it seems that SetNextNode can be after ToolCalls, but not after Call.
It would be nice to make it consistent and allow SetNextNode after Call as well.

Can you please create a short example illustrating what happens now when we generate SetNextNode after the Call and what the intended behavior looks like?
I've changed the computation of the next node in #73, and I have some doubts that these changes could already solve the problem, but I need a concrete example to check.

I was looking at the Financial Analyst example at the end of the intro notebook. Particularly inside the generate_steps function of the ActNode. I was wondering if we can change the order of SetNextNode() and AssistantStep(). Just tried it again and it seems to be working regardless of the order...


Rebuild Python 3.9 and add macOS ARM

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Closes #19
Closes #18

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.


mps support for mac users

I added mps support for mac users running models on cpu is really a pain.
what i did is I added a new argument called "gpu_type" and while checking to use cuda (or there were places where it doesn't check) it also checks gpu_type and does the same for mps.

Wow, very great job. Thanks for your contribution.


Modifying robot DOF values in TSRPlanner causes weird downstream locking failures

This was provoked during development of #321 and #322.  We found a workaround for those, but this appears to be a deeper bug in PrPy, so I opened this issue.
The original bug was first reported in https://github.com/personalrobotics/prpy/pull/321#issuecomment-234697473 and https://github.com/personalrobotics/prpy/pull/322#issuecomment-234739584.  Here's what a failure looks like on the tableclearing demo:
$ rosrun table_clearing run.py --planner restart --nobowl --noglass --perception-sim
...
[INFO] [hgpc.actions.base:base.py:42]:__enter__: Validate precondition: None
[WARNING] [prpy.base.robot:robot.py:352]:PostProcessPath: Post-processing smooth paths is not supported. Using the default post-processing logic; this may significantly change the geometric path.
[INFO] [hgpc.actions.base:base.py:47]:__exit__: Validate postcondition: None
openravepy.interfaces.BaseManipulation: __del__, failed to lock environment for BaseManipulation.__del__!
[WARNING] [openravepy.interfaces.BaseManipulation:BaseManipulation.py:47]:__del__: failed to lock environment for BaseManipulation.__del__!
openravepy.interfaces.TaskManipulation: __del__, failed to lock environment for TaskManipulation.__del__!
[WARNING] [openravepy.interfaces.TaskManipulation:TaskManipulation.py:53]:__del__: failed to lock environment for TaskManipulation.__del__!
openravepy.inversekinematics: __del__, failed to lock environment for InverseKinematicsModel.__del__!
[WARNING] [openravepy.inversekinematics:inversekinematics.py:280]:__del__: failed to lock environment for InverseKinematicsModel.__del__!
openravepy.inversekinematics: __del__, failed to lock environment for InverseKinematicsModel.__del__!
[WARNING] [openravepy.inversekinematics:inversekinematics.py:280]:__del__: failed to lock environment for InverseKinematicsModel.__del__!
^\Quit

(I sent SIGQUIT after several minutes at the end because it seems to hang indefinitely.)
While debugging this, I was able to identify something interesting, which suggests a straightforward workaround for #321 and #322, but doesn't actually explain the bug.  Those changes move IK collision checking into the Python loop, instead of inside the FindIKSolutions call, so it can be done lazily.  Importantly, the current versions of both fixes don't wrap the robot posing code in a RobotStateSaver, so the initial configuration of the robot is different when the delegate planner is called.
I'm able to provoke the same weird locking errors in the above comment by adding the following code to the original (working) code which finds pre-checked IK solutions:
    for isol in range(ik_solution.shape[0]):
        robot.SetActiveDOFValues(ik_solution[isol,:])

Wrapping this in a RobotStateSaver no longer provokes the downstream issue.
I made a branch (https://github.com/personalrobotics/prpy/tree/bugfix/tsrplanner_locking_bug) which exhibits this on a simple table_clearing run:
$ rosrun table_clearing run.py --planner restart --nobowl --noglass --perception-sim

The added code is here.
I tried replacing the delegate_planner member with a planner which always fails immediately, and did not see the failure.  I also tried replacing with SnapPlanner, and I did see the failure.

This is actually a locking issue in HGPC. Omitting the RobotStateSaver caused execution to fail because the robot is not close to the starting configuration of the trajectory. Raising that exception triggered the concurrency issue.
It is fixed in this pull request: https://github.com/personalrobotics/table_clearing/pull/37.


display conn var and plugin in comment log

This is a new release v2.3 Since you are adding plugins and connections, is that any way to print out those plugins and connections listed to the PR comment? @pindge

@jayamanikharyono


External SQL Database with Database Credentials

Running into an issue using SQL Auth with an existing database.  I can use the Windows Client just fine but if the container restarts I am unable to access.
I noticed this is being logged in the container logs AFTER I restart the container:
Initializing...
Cannot find path 'C:\run\my\aes.key' because it does not exist.
at , C:\Run\SetupVariables.ps1: line 44
at , C:\Run\navstart.ps1: line 23
at , C:\Run\start.ps1: line 102
at , : line 1
If I quick grab a copy of aes.key and re-copy that back into the directory before the restart then everything works fine.   I don't think I changed anything in my setup, I'll try and trace thought through a bit better later but if I'm missing some new options or something that would be great if someone could set me straight. :)

This is a bug.
If you add another environment variable to your call:
RemovePasswordKeyFile=N
then the keyfile will not be removed, and everything should work.
Will be fixed in the next update

Version <IP_ADDRESS> has been released


feat(experience): wait for animation to collapse row span

Adds a method to ChippedContentComponent to allow animations to finish before emitting the new content displayed event. Which allows to collapse the item in the grid after animation finishes and not before.

Current dependencies on/for this PR:

main

PR #105   👈

This comment was auto-generated by Graphite.


feat: explicitly set all to false in iter_matches

Due to neovim's breaking change in treesitter, in nightly build the option all is set to true by default.
Closes #406

Thanks for the PR!


Update cached source before listing packages

This addresses #484. It does fix the simple case that I was testing against, which I'm able to reliably repeat locally, though wasn't sure how to make this into an integration test because it relies on git shenanigans.

Create go package with a release and push it to github.
package main

import "fmt"

func main() {
    fmt.Println("I'm a dependency!")
}

Create a go package that imports this dependency and then run dep init. This populates the cache with the dependency.
package main

import _ "github.com/carolynvs/tmp1"

func main () {}

[[dependencies]]
name = "github.com/carolynvs/tmp1"
version = "2.0.2"

[[projects]]
  name = "github.com/carolynvs/tmp1"
  packages = ["."]
  revision = "b3739bbc9944d9f4c82676e6b90ed161f0981351"
  version = "2.0.2"

Create another release of the dependency and push to github.

Edit the lock and manifest to update to the newest release. Note that this commit hash and tag are not in the local cache.
[[dependencies]]
name = "github.com/carolynvs/tmp1"
version = "2.0.4"

[[projects]]
  name = "github.com/carolynvs/tmp1"
  packages = ["."]
  revision = "5a5948420cfa7f3bdc2c54a9b0d31bfd19e0dede"
  version = "2.0.4"

Run dep ensure -v -n to reproduce the following error (with some debugging thrown in):
$ dep ensure -n -v
Root project is "foo"
 1 transitively valid internal packages
 1 external packages imported from 1 projects
(0)   ✓ select (root)
(1)	? attempt github.com/carolynvs/tmp1 with 1 pkgs; at least 1 versions to try
(1)	    try<EMAIL_ADDRESS>=== DEBUG: checkRequiredPackagesExist===
=== DEBUG: src.listpackages.sg.convertToRevision ===
=== DEBUG: sg.cache.toRevision -> 5a5948420cfa7f3bdc2c54a9b0d31bfd19e0dede ===
(1)	✗   Unable to update checked out version: fatal: reference is not a tree: 5a5948420cfa7f3bdc2c54a9b0d31bfd19e0dede

What makes me pause is that I don't understand why this isn't a more common bug considering my repo steps. If you can help me figure out why dep is working fine for most people without this, I'd really appreciate it!
If I understand correctly what dep is doing, getting a list of packages included in a dependency at a particular revision, and then having the latest source code seems like the correct fix. But again, I don't get why this isn't a problem for more people then? 😀

So, first...big relief that this fixes the problem. Stale caches have plagued glide ever since a central cache was introduced, and they were a problem for a while in gps until the most recent refactor. They make for a really nasty, trust-shattering experience, because the problems dep mgmt solves are hard enough to understand without the added possibility of an expired cache confusing everything further.
While I don't know the exact paths that would make this occur without some pretty exhaustive testing (the variety in input states to the solver yield a larger state space than I can hold in my head), I'm pretty sure I know the answer here. Basically, it doesn't happen more often because yes, there's a different path that's often followed: when encountering a dependency, if the solver thinks that it might need the whole list of versions for it, it will call SourceManager.SyncSourceFor() (here and here).
The basic motivation for this approach is a performance optimization - if there's already a locked version for a project, and we have it in local cache already, AND the user didn't request an upgrade for it, then then we might be able to make it through the whole solve run without needing to hit the network. If any of these conditions aren't met, though, then we try to gain a little wall time speedup by initiating the network activity in the background (via SyncSourceFor()), prior to when the solver actually needs the data.
Most of the time, this is a good guess. But not always. If it's not, then the solver will end up making more targeted calls when it needs additional data. And that's where the path with the fix you made comes in - it's an adaptive "recovery" method, utilized by a number of SourceManager methods when the version argument provided to it isn't found.
I don't know why I neglected to put that last flag on there - probably just an oversight at the time. And because it's quite arduous to test (for all the reasons you've described above), I never wrote a test for this path. But if you read the flags being required in that method, this error should make some sense - we've ensured that we have the latest list of versions from upstream, so the SourceManager thinks that version exists. So, it goes ahead and tries it...and, boom, fatal: reference is not a tree: 5a5948420cfa7f3bdc2c54a9b0d31bfd19e0dede.
Note that this is only possible with git, because in hg and bzr, sourceHasLatestVersionList entails that sourceHasLatestLocally. The latter two have to do a clone/update to get the version list, but git does not, as we can use git ls-remote to retrieve the version list without needing to clone or fetch.

I'mma follow up with an issue about how we might devise a testing apparatus for this.

Thank you for the details and pointers! ♥ I was spelunking in gps and had some of that figured out but was missing the alternate code path.

@cscatolini fyi


ASAN-compiled knowhere library got missing symbols

Fails for both gcc and clang.
I suspect that linker flags are not set properly.
The simplest way is to test it with vector db bench.
for gcc: currently, it can be fixed by adding the output of gcc -print-file-name=libasan.so into LD_PRELOAD

@alexanderguzhva is this still a problem?


Resetting Password: Automatically logs user in even if the account isn't confirmed.

When a user resets their password, it automatically logs them into their account (i.e. doesn't check if their account is confirmed).  If they log out and then try to later sign in, they get the message "Account not confirmed yet. Please follow the instructions we sent you by email."

that's a bug. maybe we should just confirm the users account when they reset their password, as we know they entered a correct email.

@manukall That sounds awesome!

I'm just evaluating this as a possible solution for an upcoming Phoenix project (so apologies for the armchair quarterbacking), but the proposed solution feels odd. If I haven't confirmed my email, it seems strange that I could log in at all. Just because I've entered an email, doesn't mean it's mine. That's why the confirmation is required in the first place.
In practice, doesn't this mean that anyone who's able to guess an unconfirmed account's email is able to log in as that user?

@clekstro In order to reset your password, you need to get the confirmation email. Hence, they (1) submit for password reset form, (2) click the link that gets emailed to them, and (3) get brought to a page where they change their password.

@ACPK Appreciate the additional context. I didn't realize from the description above that that was the flow.

@clekstro No problem.  We're using @manukall so I recommend it but I'm adding some custom features.


Old links to pages should find the new alias of the page/content

When we change the alias of a page(страница) or content(целина), the link should not become dead.
Instead if an old URL (because of changed alias) is requested by a user-agent, a redirect response 301 Moved Permanently should be made so the Search engines can update their indexes with the new Location header.

implemented in
5c22fc0


Logic for multiple groups and styling changes

Work done towards #37

Groups sidebar is always showing and no longer needs to toggle
Groups can be removed (provided the group is not selected)
Groups can be selected and calendars can be added separately to each group
Removes vertical scrollbar from application

Code styling is great and functionality is mostly there!
I just found one bug when you import a calendar using an iCal link: if you switch to another group while waiting for the calendar data to be retrieved, the new user will be created on that group rather than the one you originally had active when you created the new user.


Serialize additional class component metadata

Fixes #1872

Yes, thanks for the review. I'll have a look at optimizing things.
I wasn't sure if the serialization logic supports anything other than standard tables, but if so secondary keys should also help.

Should be a fair bit better now?
I noticed
https://github.com/larshp/abapGit/blob/b057470646a381eeecb08dc79174dcd84da094c5/src/objects/zcl_abapgit_oo_base.clas.abap#L318-L326
isn't being used at all because the two fields are always initial at this point. I assume the SAP note changes that behavior, but I cannot test it right now (because I have no idea how to configure the AS ABAP Developer Edition so that you can use SNOTE).

Even when note 2400131 is implemented this section is never executed.
https://github.com/larshp/abapGit/blob/b057470646a381eeecb08dc79174dcd84da094c5/src/objects/zcl_abapgit_oo_base.clas.abap#L318-L326
Maybe we can omit it then?

That seems like a bug though, doesn't it? Should we rely on the fields being cleared at this point or leave the logic in, should this behavior change in the future?

Scratch that, I just noticed SEO_CLASS_CREATE_COMPLETE has an optional parameter for this field, that of course is not supplied in the function call. So it will clear these fields "correctly". I think it would be better to use that parameter instead of updating standard db tables directly. But the update_description method does the same thing and there's also an optional parameter for that. Do you know if this implementation was chosen on purpose because the parameters do not exist on 7.02?

Don't know. Maybe @larshp?

not sure I understand the question.
It is better to use the API(SEO_CLASS_CREATE_COMPLETE) if possible. I'm not sure what the "update_description method" refers to

Oh it's this one:
https://github.com/larshp/abapGit/blob/b057470646a381eeecb08dc79174dcd84da094c5/src/objects/zcl_abapgit_oo_base.clas.abap#L280-L283
Since table seocompotx is updated directly there I did the same for seocompodf in this pull request. Then I noticed that there is a parameter ("attributes") that updates both of these tables in SEO_CLASS_CREATE_COMPLETE so I was just wondering if this parameter was intentionally avoided when update_descriptions was implemented (because maybe it doesn't exist on 7.02). I was unable to find it out with git blame since this seems to have been before some major refactorings.
Otherwise I'd change it so the API is used instead.

I think we should try moving it to the API, then we'll see what happens

I'll adjust the PR this weekend.

@flaiker is this change ready to be reviewed?

Sorry I forgot comment in the PR.
'SEO_CLASS_CREATE_COMPLETE' is now used instead of updating seocompodf directly. The implementation allows for interface attributes to be handled as well. Currently it has no effect though as the workflow attributes are not relevant for interfaces (I think?) and therefore not serialized.
seocompotx would be a candidate that is both relevant for classes and interfaces and currently updated directly.
Another approach would be to just serialize all of vseoattrib, that includes many questionable or redundant fields.
I wanted to add a testcase to abapGit-tests/CLAS but I am not sure how it works. Is there some automatic integration with the dangerous unit tests or are these just manual test cases?

abapGit-tests: nothing automatic, just collecting some testcases somewhere centrally. Add a new repository when needed, eg "CLAS_something"


Could not attach to process with CustompipeName 'AzureFunctionsPSWorker'

I have a Windows10 20H2 workstation suddenly giving me this error message in Visual Studio Code and without launching my function properly.
It did work for 2 weeks or so and I managed to create some PoC functions.Then it stopped working.
I then tried installing azure function core tools, PS7, SDKs 3.1/5.1 and VisualStudioCode (Azure extension) on my laptop with Win10 20H2 and that worked and it's still working this day.
My conclusion was it must be my win10 workstation which has an issue.
So I created a new VM with a Win10 20H2 and installed PS7, SDK's, AzureFuncCoreTools, VisualStudioCode (Azure extension) and then tried creating a new project, and I get the exact same error when executing.
So now I have 2 Virtual Machines running Win10 20H2 and neither of them work when trying to run local powershell functions
I have 1 Lenovo laptop with Win10 20H2 and it works fine.
For troubleshooting I have tried this on my newly created Win10 test VM:
uninstalled Powershell 7.1 and installed Powershell 7.0
uninstalled .NETcore SDK 5.1
Trying to run my function directly from PowerShell without using VSCode returns this:
PS Z:\Folder1\ProjectRoot> func host start:
Azure Functions Core Tools
Core Tools Version:       3.0.3233 Commit hash: d1772f733802122a326fa696dd4c086292ec0171
Function Runtime Version: 3.0.15193.0
Let me note, before starting the configuration of the new Win10 VM, I tried a last time executing my function on the Win10 workstation (original) and after 10-15 minutes it actually started my HTTPTrigger function and it worked. When I tried executing it once again it returned the error message.
All machines are running the exact same project, so it's the same .json files I'm referring to in the end.
Please let me know if you need more information
Thanks for your time.

Machine info:
Win10 workstation (original):
Visual Studio Code: 1.52.1
PS> dotnet --info:
`.NET SDK (reflecting any global.json):
Version:   5.0.102
Commit:    71365b4d42
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19042
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\5.0.102\
Host (useful for support):
Version: 5.0.2
Commit:  cb5f173b96
.NET SDKs installed:
3.1.405 [C:\Program Files\dotnet\sdk]
5.0.102 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 3.1.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.11 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.2 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]`
PS> func --version:
3.0.3233
PS> $PSVersionTable:
Name                           Value
PSVersion                      7.1.1
PSEdition                      Core
GitCommitId                    7.1.1
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Win10 workstation (new test machine):
Visual Studio Code: 1.52.1
PS> dotnet --info:
.NET Core SDK (reflecting any global.json):
Version:   3.1.405
Commit:    65f9d75b1c
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19042
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.405\
Host (useful for support):
Version: 3.1.11
Commit:  f5eceb8105
.NET Core SDKs installed:
3.1.405 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.App 3.1.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.11 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
PS> func --version:
3.0.3233
PS> $PSVersionTable:
Name                           Value
PSVersion                      7.0.3
PSEdition                      Core
GitCommitId                    7.0.3
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Win10 Laptop:
Visual Studio Code: 1.52.1
PS> dotnet --info:
.NET SDK (reflecting any global.json):
Version:   5.0.102
Commit:    71365b4d42
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19042
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\5.0.102\
Host (useful for support):
Version: 5.0.2
Commit:  cb5f173b96
.NET SDKs installed:
3.1.401 [C:\Program Files\dotnet\sdk]
3.1.405 [C:\Program Files\dotnet\sdk]
5.0.102 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.All 2.1.21 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.21 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.21 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.7 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.11 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.2 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
PS> func --version:
3.0.3233
PS> $PSVersionTable:
Name                           Value
PSVersion                      7.1.1
PSEdition                      Core
GitCommitId                    7.1.1
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

.vscode/launch.json:
{
"version": "0.2.0",
"configurations": [
{
"name": "Attach to PowerShell Functions",
"type": "PowerShell",
"request": "attach",
"customPipeName": "AzureFunctionsPSWorker",
"runspaceId": 1,
"preLaunchTask": "func: host start"
}
]
}

.vscode/tasks.json:
{
"version": "2.0.0",
"tasks": [
{
"type": "func",
"command": "host start",
"problemMatcher": "$func-powershell-watch",
"isBackground": true
}
]
}

.vscode/settings.json:
{
"azureFunctions.deploySubpath": ".",
"azureFunctions.projectLanguage": "PowerShell",
"azureFunctions.projectRuntime": "~3",
"debug.internalConsoleOptions": "neverOpen"
}

/host.json:
{
"version": "2.0",
"logging": {
"applicationInsights": {
"samplingSettings": {
"isEnabled": true,
"excludedTypes": "Request"
}
}
},
"extensionBundle": {
"id": "Microsoft.Azure.Functions.ExtensionBundle",
"version": "[1.*, 2.0.0)"
},
"managedDependency": {
"enabled": true
}
}

Sorry for the late response. Are you still having issues? If so, please debug your project and copy the logs from the "func host start" task here. I believe this error usually happens when there's an error in those logs. Also - how did you install the func cli (choco, npm, msi, etc.)?

Hi, yes I still have the same issue.
How would you like me to debug?
I get nowhere even when running func host start --debug
The only thing is this:
Core Tools Version:       3.0.3233 Commit hash: d1772f733802122a326fa696dd4c086292ec0171
Function Runtime Version: 3.0.15193.0

You should start debugging in VS Code, not directly with the func cli. You can press F5 or click the play button next to "Attach to PowerShell Functions"

It will start the func cli for you and you should see logs in the "Terminal" pane, which are the logs I'm talking about:

Same result as running in the Terminal, except for a dialog box when I hit ctrl+c after n minutes.
Let me just remind you, this also happens on a cleaned installed VM with same configuration, and it did work well back in december/january.

Looking forward to your next suggestion.

Hmm well that's not very helpful since it doesn't show much of anything in the logs. And that's exactly what it looks like if you run func host start manually from the command line? You could try running func host start --verbose to see if you get anything more helpful.
Also do you have this problem with a brand new, clean project with just an http trigger?

This is what it looks like if I'm running func host start from the PWSH terminal.

This is quite interesting. When I created a new folder, initialized a new project with the VSC Azure extension inside, and added a HTTPTrigger function, it would work the first time when running in Debug (VSC - F5). Stopping it and running it again will give me the same output as posted in my last post.
So, I thougt creating a new project would work the first time again, so I could send you a screendump. But nope, another brand new folder with a clean project and only a HTTPTrigger running in Debug will give me the exact same output again. Even when restarting the OS in between.
Running the project with func host start --verbose gives me this:

Do you know if it's possible for me to raise a ticket within Azure or somewhere else for getting hands-on support on this issue, I've been postponing all of my powershell function app development as long as I could in the hopes of getting a fix for this issue.
Now I'm at a point where my backlog mostly contain powershell function app development and I have no other choice than starting development on my laptop which is by no means an ideal situation.
FYI. I'm a registered Microsoft Partner and doing a SaaS product for azure hosted on azure, if that changes anything in the possibility for getting hands-on support with your open-source products.
Thanks in advance.

Well since the func cli shows the same thing in VS Code and the terminal (not much at all haha), it's probably a good idea to file an issue on the cli directly: https://github.com/Azure/azure-functions-core-tools/issues. I think the root cause is the func cli can't start up your app for some reason, and that's why VS Code can't attach.
I'm honestly not familiar with how being a "registered Microsoft Partner" works. You're more than welcome to file a ticket wherever applicable. The company probably has dedicated people to help on top of whatever discussion you get from my team on GitHub.

Oh that's my bad! I haven't even noticed this was the vscode-azurefunctions and not the core tools. For some reason I got those mixed up in my head.
I'll try to open a new issue over there refrencing to this case. So feel free to close this one.
Okay thanks for your input. I'll try open a support case in the Azure portal too.
Thank you for your time!


K-space conversion for hv scans (and normal ones as well)

Implementing k-space conversion (just for kz scans) by changing analyzer axis sampling.

PColorMesh class, although available in pyqtgraph, works incredibly slow for interactive applications.
A possible workaround would be to rescale raw dataset (solution widely used in Igor community).
Drawbacks:

rescaling can take a while (numba probably could help),
to not lose progress, rescaled dataset needs to be saved,
saved data file will be bigger, especially painful with hv scans.

All suggestions much appreciated.

The drawbacks of rescaling seem quite obvious, so I'm surprised that it's widely used with Igor. Do you have any idea of how much they rescale? Is it more a factor of 2 or a factor of 10? Naively, this would lead to 10x larger files, right?
I also gave PColorMeshItem a shot and have come to the same conclusion as you. Additionally, it would be difficult to extract slices from PColorMeshItems.
One approach I could imagine would be to do a discrete k space conversion, where we only shift channels by an integer amount of pixels. (If I understand correctly, this would be kind of equivalent to the rescaling approach, but with a scaling factor of 1) Of course, the precision of this way would be limited to the pixel size. On the bright side, all slicing functionality would be readily available and handling data on rectangular generally is generally more convenient.

From what I checked it's a bit more than 2 in size. And I think it's a limit, because I've tested it in already quite wide hv range.
For simple Femi surfaces I'm sure it will be <1.5.
I'm not sure what you mean by discrete conversion. To remain the size of dataset you'd lose some data, wouldn't you?


fix(notify): avoid panic when mobile not set

What this PR does / why we need it:

[ ] Smoke testing completed

If yes, write the release branches name in the below format and submit the related cherry-pick PR:

release/3.11

/lgtm

/lgtm


Prepare docs build

Extracted from https://github.com/citizensadvice/design-system/pull/1874
Improve yarn support
In https://github.com/citizensadvice/design-system/pull/1778 we introduced the dependency on yarn as bridgetown makes some assumptions about this being available.
Tidy things up by:

Removing the old package-lock.json file (if we're using yarn we should only have the yarn.lock file)
Updating internal rake tasks to use yarn
Adding a bin/setup script with a yarn install step

Update docs workflow
Allows for building full docs site including assets within the new CI workflow. Uses the same general approach as for the demo app.
Basic docs homepage
Adds a minimal home page and navigation structure.
Includes a fix to the builder to make sure local files are loaded so we can use the footer component.

Sooo, I'm a bit confused as I deliberately de-yarned at your request (https://github.com/citizensadvice/design-system/pull/1660)

Yeah I was confused myself, but I checked and most of the bridgetown configure commands assume the existence of yarn. Rails is a bit like this too. Also noticed that you added a yarn.lock file https://github.com/citizensadvice/design-system/blob/main/design-system-docs/yarn.lock and used yarn in the build https://github.com/citizensadvice/design-system/blob/main/.github/workflows/docs-workflow.yml#L31 so I thought it worth following through.
Given how much the ruby framework world seems to prefer yarn it's probably for the best to stick with it.

Yeah that makes sense - just wanted to check.  I guess the yarn files came from running build etc and I didn't even notice at the time! 👁


Issue 39. Create typemap needed dafus()

Fix cspyce.dafus() by creating a typemap that can handle it.

Don't approve this just yet.  Let me know what you think.
There are two ways of fixing this problem.

Create a new typemap that given an int argument, knows how to create an array of that size and pass it to the function.

Just pass an array of size DAFSIZE to the function, and then upon return, resize the array to the correct size.  We already of have typemaps that know how to do that.

(1) is "prettier".  (2) is a lot less change to the code.  And would be similar to the change for dafgda().
If you're happy with this change, go ahead and merge this in.

I'm closing this pull request.  The other way of doing it is just much less code.  I can't justify this.


fix(WebSocket): emit "error" event on bypass connections

Interceptors has a bug that it will always emit the open even on the WebSocket client, even when there are no interceptor set up for any connections (so the connection cannot physically be intercepted). This results in such connections pending forever.

This may be a big against previously established expectations of WebSocket interception always being mock-first. But this is still a bug for bypassed connections, and it has to be fixed.

Roadmap

[x] Add a failing test.
[ ] Emit an internal connection event in WebSocketOverride. In the next tick, listen if anything has prevented that event. If not, establish the connection as-is.
[ ] In the actual connection listener of an interceptor, listen to the internal connection event and prevent it.

Alternative solution
Push the WebSocket interception a layer higher, letting the interceptor decide on the connection, whether to mock it, error it, or establish as-is.
interceptor.on('connection', ({ handshake }) => {
  handshake.accept()
  // handleshake.errorWith()
  // handshake.passthrough()
})

This may also be an answer to handling the handshake (protocol upgrade) request. It's been reported that we don't do that at the moment, which may cause issues (mostly in the browser).

Released: v0.34.0 🎉
This has been released in v0.34.0!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


[BUG] samples/Basic/iOS/SkiaSharpSample canvas.DrawText cannot convert arguments

Description
To start with an iOS SkiaSharp example that matches what I would do as a nuget user, I modified samples/Basic/iOS/SkiaSharpSample/SkiaSharpSample.csproj:

target net8-ios and iOS 13.0.
Instead of referencing source projects, add nugets SkiaSharp, SkiaSharp.Views, SkiaSharp.NativeAssets.iOS 2.88.8

When build, ViewController.cs, OnPaintSurface line 41 canvas.DrawText(...) can't find a match for parameter types:

Argument 3: cannot convert from Skiasharp.SKTextAlign to float.
Argument 2: cannot convert from SkiaSharp.SKPoint to float.

Code
canvas.DrawText("SkiaSharp", coord, SKTextAlign.Center, font, paint);

Expected Behavior
The project builds and runs on an iOS 17.2 simulator.
Actual Behavior
Compile errors mentioned above.
I can probably fix this. But posting issue in case

there is some other nuget I should have included to handle iOS type conversions,
or the sample matches some preview version I have not tried,
or the sample needs to be updated.

Version of SkiaSharp
2.88.3 (Current)
Last Known Good Version of SkiaSharp
Other (Please indicate in the description)
IDE / Editor
Visual Studio (Windows)
Platform / Operating System
iOS
Platform / Operating System Version
VS 2022 17.9.7 connected to iOS 17.2 Simulator on Mac.
Devices
No response
Relevant Screenshots
No response
Relevant Log Output
No response
Code of Conduct

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

Using a different overload gets past this error:
canvas.DrawText("SkiaSharp", coord.X, coord.Y, font, paint);

If you want to use the 2.x series of NuGet packages, you need to pull the sample from the release/2.x branch: https://github.com/mono/SkiaSharp/blob/release/2.x/samples/Basic/iOS/SkiaSharpSample/ViewController.cs


ADR on security

Provide documentation and diagrams for security. This should include some documentation on Falco and Trivy

fixed typo in description


Possibly broken refmap in 0.3.0, mixin tries to find mapped classes in production

[14:51:31] [main/WARN]: Error loading class: net/minecraft/client/network/ClientPlayerEntity (java.lang.ClassNotFoundException: net/minecraft/client/network/ClientPlayerEntity)
[14:51:31] [main/WARN]: @Mixin target net.minecraft.client.network.ClientPlayerEntity was not found mixins.cotton-client-commands.json:PlayerMixin
[14:51:31] [main/WARN]: Error loading class: net/minecraft/client/network/ClientPlayNetworkHandler (java.lang.ClassNotFoundException: net/minecraft/client/network/ClientPlayNetworkHandler)
[14:51:31] [main/WARN]: @Mixin target net.minecraft.client.network.ClientPlayNetworkHandler was not found mixins.cotton-client-commands.json:NetworkHandlerMixin

The problem is something else, the build/mavenpublish tasks must be run within a single gradle invocation, after a gradle clean, at least with the current version of loom.
gradle clean build publish
gradle clean; gradle build publish works as well.
gradle clean; gradle build; gradle publish does not, because for some reason publish repeats part of the build without remapping. (gradle build twice in a row has the same problem).
You can check this (on Linux), when you do, on the finished jar file:
jar xf build/libs/cotton-client-commands-0.3.0+1.14-SNAPSHOT.jar io/github/cottonmc/clientcommands/mixin/PlayerMixin.class
strings io/github/cottonmc/clientcommands/mixin/PlayerMixin.class

A correctly built jar will have the class reference mapped MC classes, like
net/minecraft/class_2588, which appear in the strings command output. An incorrect one will have unmapped classes like net/minecraft/text/TextComponent. (Of course, picking another class will work as well, as will using a different way of viewing class files).


typescript: languages provided by plugins not synced

tsserver plugins could provide a list of languages so that when a document other than .js[x] and ts[x] is opened, the plugins would be notified.
Angular language service relies on this to parse .html files in Angular templates.
  "contributes": {
    "typescriptServerPlugins": [
      {
        "name": "@angular/language-service",
        "enableForWorkspaceTypeScriptVersions": true,
        "languages": [
          "html"
        ]
      }
    ]
  },

There seems to be a regression whereby languages provided by tsserver plugins are no longer synced (does not have buffer sync support).
This was brought to light by @andrius-pra in https://github.com/angular/vscode-ng-language-service/issues/338#issuecomment-499364844
For potential fix, please see https://github.com/microsoft/vscode/commit/a5853d6ebe8419a0e0338824cc7ecec0d7efe39e
cc: @mjbvz

Related https://github.com/microsoft/TypeScript/issues/17151
We only prototyped the support for external languages in TypeScript so this is not really an official API. You'd have to check with the TS team to see if this is still something they are interested in properly supporting as there is other work required beyond this

I merged a pr that fixes this so you can test it and provide feedback, but you must not ship anything to customers that relies on this API. It is not official and may be changed or be removed at any point in the future

@mjbvz Thanks for the update. I'll test it out locally.
I'd like to organize a meeting to finalize the design with the Typescript team, so that we can move on with the implementation.


missing a11y metadata not necessarily required

The list of missing accessibility metadata provided after the metadata table doesn't provide any indication of whether the metadata is actually required or not.
I could see people thinking that because certifierReport is listed as missing, for example, that means it has to be included, when it's just an optional field.
It might be useful to add whether the missing properties are required, recommended or optional, perhaps in parentheses after the name.

This is not specific to Ace App, I think the HTML report also needs to better present this information. In Ace App there is an indication of “severity” which is directly correlated with recommended / required, but I agree that this is unclear and should be improved.


[tensorflow] create an EagerSession per NDManager

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #888 (95f0e23) into master (0d4a708) will decrease coverage by 0.00%.
The diff coverage is 78.57%.

@@             Coverage Diff              @@
##             master     #888      +/-   ##
============================================
- Coverage     70.37%   70.36%   -0.01%
+ Complexity     5086     5085       -1
============================================
  Files           501      501
  Lines         22415    22419       +4
  Branches       2325     2326       +1
============================================
+ Hits          15774    15775       +1
- Misses         5402     5410       +8
+ Partials       1239     1234       -5

Impacted Files
Coverage Δ
Complexity Δ

...pi/src/main/java/ai/djl/ndarray/BaseNDManager.java
56.25% <ø> (ø)
23.00 <0.00> (ø)

...main/java/ai/djl/tensorflow/engine/TfDataType.java
100.00% <ø> (ø)
5.00 <0.00> (ø)

...c/main/java/ai/djl/tensorflow/engine/TfEngine.java
53.84% <ø> (-0.88%)
9.00 <0.00> (-1.00)

...ava/ai/djl/tensorflow/engine/TfEngineProvider.java
100.00% <ø> (ø)
5.00 <0.00> (ø)

...rc/main/java/ai/djl/tensorflow/engine/TfModel.java
43.37% <ø> (ø)
10.00 <0.00> (ø)

...n/java/ai/djl/tensorflow/engine/TfSymbolBlock.java
88.65% <ø> (ø)
16.00 <0.00> (ø)

...rc/main/java/ai/djl/tensorflow/zoo/TfModelZoo.java
100.00% <ø> (ø)
4.00 <0.00> (ø)

...orflow/zoo/cv/objectdetction/TfSsdModelLoader.java
100.00% <ø> (ø)
1.00 <0.00> (ø)

.../main/java/ai/djl/tensorflow/engine/TfNDArray.java
85.23% <50.00%> (+0.44%)
188.00 <0.00> (ø)

...ain/java/ai/djl/tensorflow/engine/TfNDManager.java
90.00% <80.00%> (-1.08%)
29.00 <2.00> (+1.00)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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


codeBundleId support

I have tested it and it works perfectly!

Hi @farfromrefug
What's your intended usage for this? The codeBundleId is intended to support CodePush release on React Native apps whereas our Webpack plugin is intended for use on Javascript projects.
We're interested in the problem you're trying to solve here in case there may be a better solution with our existing tools or some of our in-flight development.

I explained it in an issue here. I need it for  Nativescript. In {N} we use a mix of webpack/gradle for Android and webpack/pod for ios. This is the only part missing for me to make sure I have the right source maps for my beta builds (depending on the code bundle Id)

Thanks, I hadn't spotted the issue.
Fixes https://github.com/bugsnag/webpack-bugsnag-plugins/issues/38


[TACHYON-1423] Update - Support setting permission model in Master side

JIRA:
https://tachyon.atlassian.net/browse/TACHYON-1423,
https://tachyon.atlassian.net/browse/TACHYON-1424,
https://tachyon.atlassian.net/browse/TACHYON-1425
This PR includes 3 JIRAs which supports updating permission model in Client, Thrift, Server sides. The Shell side will be a follow-up PR.
It seems the structure in FileSystemMaster may change. I will update this patch accordingly if any.
@jsimsa , @yupeng9 , @apc999 , @sundapeng

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7672/Failed Tests: 6org.tachyonproject:tachyon-servers: 6tachyon.master.file.FileSystemMasterTest.renameTesttachyon.master.file.FileSystemMasterTest.setTTLForFileWithNoTTLTesttachyon.master.file.FileSystemMasterTest.setSmallerTTLForFileWithTTLTesttachyon.master.file.FileSystemMasterTest.createFileWithTTLTesttachyon.master.file.FileSystemMasterTest.renameToSubpathTesttachyon.master.file.FileSystemMasterTest.deleteFileTest
Test FAILed.

Jenkins, test this please.

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

@sundapeng left some comments regarding the use of options ... I also have a more high-level comment: it might makes sense to implement setting of user, group, and permission through the already existing setState API call (you ACL protobuf entry is half-way there). That way we keep the API surface small and reuse much of existing logic instead of creating repetitive boilerplate.

@jsimsa , Thanks for your comments about the options. Patch updated to use an option SetAclOptions and reduce the API to only one setAcl. About reusing setState API call, I am a little concerned that although the logic can be merge together, state and acl are different concept and putting them together may confuse API users? So in this patch not reuse it. Please feel free to let me know if needed.

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

@dongche thank you for the iteration, I left some comments

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

@jsimsa , @yupeng9 , @apc999 , thanks for your review! Patch updated to address comments and refine the methods in FileSystemMaster

@dongche thanks, left a comment for you

+1 LGTM

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7811/Failed Tests: 1org.tachyonproject:tachyon-servers: 1tachyon.master.lineage.recompute.TestRecomputeExecutor.recomputeLauncherTest
Test FAILed.

@jsimsa , thanks for the comments in #2414 about setting only one at a time. Patch updated to remove the constraints and we can set at least one now.

Jenkins, test this please.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7813/Failed Tests: 3org.tachyonproject:tachyon-servers: 3tachyon.master.file.FileSystemMasterTest.setTTLForFileWithNoTTLTesttachyon.master.file.FileSystemMasterTest.setSmallerTTLForFileWithTTLTesttachyon.master.file.FileSystemMasterTest.createFileWithTTLTest
Test FAILed.

Jenkins, test this please.

@dongche thanks for the update, a few more minor comments ... overall this looks good

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

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

@jsimsa thanks for the review. Patch updated.

@jsimsa is the latest patch good for you?

LGTM, @dongche thank you for the refinements!

+1 LGTM


BMP image Convert

Hello,
Can you include the .BMP extension among the converters?
Thanks

It is already implemented in the latest beta, check #308


Starting the agent with ``--no-pty` is a workaround, though you end up with funky build output and probably don't want to keep it enabled… so we need to figure out a recommended workflow for getting set up that Just Works without too many dramas.

same again, via @dylanfm:
Running global pre-checkout hook
Preparing build folder
Saving Git information
Running global post-checkout hook
Running global pre-command hook
Running local pre-command hook
Building Docker images
Running build script (in Docker Compose container)
$ docker-compose -p buildkited1e99f22eb8e4d83960be5d6b1f649bd run app ././script/ci
Creating buildkited1e99f22eb8e4d83960be5d6b1f649bd_db_1...


Offer a hint when bastion probe fails

Fixes #246.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 5 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 19.742%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

fabrictestbed_extensions/fablib/fablib.py
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

fabrictestbed_extensions/fablib/fablib.py
1
44.39%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
964

Relevant Lines:
4335

💛  - Coveralls


Go: fix handling empty tags-set for loading saved model

This PR fixed a crash in LoadSavedModel for empty tag-set.
The panic itself (panic: runtime error: index out of range) comes from https://github.com/tensorflow/tensorflow/blob/25251f057c476df16451599aa34a6283b9e45209/tensorflow/go/saved_model.go#L69

Is there anything I can do to help to get the PR reviewed/merged?

No problem and thanks for the help 😄


merge package template from Kronos-Tools/npm-package-template

.travis.yml

chore(travis): merge from template .travis.yml

Coverage remained the same at 83.333% when pulling e1602fb280b75d4c628add7ce4eebc9f09e43afc on template-sync-1 into b9e6d7249af467e4387a9c1625c21f2fc756008b on master.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #117   +/-   ##
=======================================
  Coverage   83.33%   83.33%
=======================================
  Files           1        1
  Lines          12       12
=======================================
  Hits           10       10
  Misses          2        2

Continue to review full report at Codecov.

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


Disable HTTP2 in webhooks

Also bumps golang.org/x/net to v0.17.0
Jira: OSPRH-468

@dprince: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name 	Commit 	Details 	Required 	Rerun command
ci/prow/cinder-operator-build-deploy-kuttl 	2d82d00 link 	true 	/test cinder-operator-build-deploy-kuttl
ci/prow/cinder-operator-build-deploy-tempest 	2d82d00 link 	false 	/test cinder-operator-build-deploy-tempest
Full PR test history. Your PR dashboard.

Looks like the KUTTL tests failed because the Rabbit server pod couldn't start and thus Cinder's transportURL could not be created (which would block Cinder from proceeding at all):
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_cinder-operator/289/pull-ci-openstack-k8s-operators-cinder-operator-main-cinder-operator-build-deploy-kuttl/1719808980391301120/artifacts/cinder-operator-build-deploy-kuttl/openstack-k8s-operators-gather/artifacts/must-gather/quay-io-openstack-k8s-operators-openstack-must-gather-sha256-9810fe9cf8d0d302413f7876267d8e3149d81e7d4b4c2b88c98f5a60f0c3f800/namespaces/cinder-kuttl-tests/pods/rabbitmq-server-0/rabbitmq-server-0-describe

/test cinder-operator-build-deploy-kuttl

/test cinder-operator-build-deploy-tempest

/test cinder-operator-build-deploy-tempest

/test cinder-operator-build-deploy-tempest