Hugo doesn't include new helm release when created.

When a new release is cut, the github action that generates the documentation won't include the latest changes, I need to manually create a commit in the gh-pages to force hugo to recompile with the new helm charts.
User experienced the issue with #12.
Posssibly need to reorder and reorganize some of the actions so it is integrated in a more robust way.

We encountered a similar issue in another project. We didn't want to order the GitHub Actions for the Helm release after the documentation because it didn't make sense to depend on the Helm release from the documentation release.
Here the workaround we used:
Publish the documentation on GitHub Pages into a complete subfolder, such as "/docs," to prevent the Helm "index.yaml" file from being deleted. Then, manually commit an "index.html" file at the root level with HTML redirect:
<meta http-equiv="refresh" content="0; url=https://pier-oliviert.github.io/phonebook/docs/index.html">

Perhaps you'll find a better solution!


doing: init at 1.0.10pre

Motivation for this change
Adds the doing application.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

It looks like this is missing an addition to all-packages.nix, or how do you build this application?

right... I've it in my own overlay... Will fix it...


Feedback for exercise 3

Depending on the knowledge that you think everyone has, maybe it makes sense to explain what the manual injection in istio is.
In "Install the guestbook app with Istio" I would move step 4 after step 1, so that you verify the pods right after you created the app, and then you can concentrate on the service deployment and verifying that the service is up and running.
At the end, you add the watson tone analyzer. But in the beginning it is not mentioned that this is something that you would do in this lesson.
Question: Can you use the watson tone analyzer without binding it to the cluster?
It might make sense to explain what guestbook is, maybe just one sentence, so that people understand why you want to analyze the tone.
You also might need to add that you must have the developer role (CF) for the space where you provision the service, to actually be able to do all that.

@szihai can you check this out?  ^^

Closing per @nilesh32's request


Passing Active Directory Objects into a Runspace Stripts All Non-Default Properties

For those of you who frequently work with the ActiveDirectory module, you'll know using Get-ADUser or Get-ADComputer will only return a few of the many properties of the object from the database (10 and 9 respectively). If you wish to retrieve additional properties of an AD object, you must specify the -Properties parameter and provide one or more property names.
When passing an ADObject that contains one or more non-default properties to a runspace as an argument, the property values are not accessible. They can not be accessed with dot-notation, nor used in a Where-Object filter (among other things).
Interestingly enough, if you just pass the object through the runspace script and output it again in the current context, it will now have the missing properties again, as if they were never lost.
I can reproduce the exact same behavior with Start-ThreadJob, ForEach-Object -Parallel (which of which uses runspaces) as well as the popular 3rd party module PoshRSJob. When using either of these method, it makes no difference to which method I use to pass the object into the job. The problem occurs if I use the -ArgumentList or the $using: syntax.
This problem however does NOT occur with the standard Start-Job which spawns an entire new process rather than a new thread/runspace. The downside to using standard background jobs is that the objects are serialized upon passing to the job.
Steps to reproduce
In the example below the Name property of an ADUser object is "default", but the City property must be requested specifically. I output the two properties three times each. Once before, once from within the runspace, and once using the object that was passed through the runspace and back into the current context.
$ADUser = Get-ADUser -identity mander -Properties City

Write-Output "Before Runspace - The value of name is: $($ADUser.Name)"
Write-Output "Before Runspace - The value of city is: $($ADUser.City)"

$Runspace = [runspacefactory]::CreateRunspace()
$PowerShell = [powershell]::Create()
$PowerShell.runspace = $Runspace
$Runspace.Open()

[void]$PowerShell.AddScript({
    Param ($ADUser)
    $a = "Inside Runspace - The value of name is: $($ADUser.Name)"
    $b = "Inside Runspace - The value of city is: $($ADUser.City)"
    $c = $ADUser

    return @($a,$b,$c)
})
[void]$PowerShell.AddArgument($ADUser)

$a, $b, $c = $PowerShell.Invoke()

$a
$b

Write-Output "After Runspace  - The value of name is: $($c.Name)"
Write-Output "After Runspace  - The value of city is: $($c.City)"

Expected behavior
Before Runspace - The value of name is: Max Anderson
Before Runspace - The value of city is: Atlanta
Inside Runspace - The value of name is: Max Anderson
Inside Runspace - The value of city is: Atlanta
After Runspace  - The value of name is: Max Anderson
After Runspace  - The value of city is: Atlanta

Actual behavior
Before Runspace - The value of name is: Max Anderson
Before Runspace - The value of city is: Atlanta
Inside Runspace - The value of name is: Max Anderson
Inside Runspace - The value of city is:
After Runspace  - The value of name is: Max Anderson
After Runspace  - The value of city is: Atlanta

Environment data

Name                           Value
----                           -----
PSVersion                      7.1.0
PSEdition                      Core
GitCommitId                    7.1.0
OS                             Microsoft Windows 10.0.19041
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I also would like to note that this issue is not only occuring in PowerShell core. I've also observed the same behavior in Windows PowerShell 5.1. This would tell me that it's not an issue with any abstraction layer (such as WindowsCompatibility making the ActiveDirectory module work within PowerShell Core.
Name                           Value
----                           -----
PSVersion                      5.1.19041.546
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.546
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

I also would like to note that this issue is not only occuring in PowerShell core. I've also observed the same behavior in Windows PowerShell 5.1. This would tell me that it's not an issue with any abstraction layer (such as WindowsCompatibility making the ActiveDirectory module work within PowerShell Core.
Name                           Value
----                           -----
PSVersion                      5.1.19041.546
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.546
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Since AD module is not source open only MSFT team can investigate the issue. I guess the problem can be related with how PowerShell clones variables. If so we could try to fix this in the repo. Otherwise the issue should be reported by Windows support channels, UserVoice site and MSFT forums.
/cc @PaulHigin Could you please make an conclusion?

Since AD module is not source open only MSFT team can investigate the issue. I guess the problem can be related with how PowerShell clones variables. If so we could try to fix this in the repo. Otherwise the issue should be reported by Windows support channels, UserVoice site and MSFT forums.
/cc @PaulHigin Could you please make an conclusion?

This is definitely a weird one. When testing against a remote Runspace I can see the CLIXML returned contains the property and accessing $obj.City works just fine
<Obj RefId="0">
  <TN RefId="0">
    <T>Microsoft.ActiveDirectory.Management.ADUser</T>
    <T>Microsoft.ActiveDirectory.Management.ADAccount</T>
    <T>Microsoft.ActiveDirectory.Management.ADPrincipal</T>
    <T>Microsoft.ActiveDirectory.Management.ADObject</T>
    <T>Microsoft.ActiveDirectory.Management.ADEntity</T>
    <T>Microsoft.ActiveDirectory.Management.ADPropertyCollection</T>
    <T>System.Object</T>
  </TN>
  <ToString>CN=vagrant-domain,CN=Users,DC=domain,DC=test</ToString>
  <Props>
    <S N="City">My City</S>
    <S N="DistinguishedName">CN=vagrant-domain,CN=Users,DC=domain,DC=test</S>
    true
    <Nil N="GivenName" />
    <S N="Name">vagrant-domain</S>
    <S N="ObjectClass">user</S>
    <G N="ObjectGUID">bdfa699e-dfb9-41ba-864d-6a2f162f579d</G>
    <S N="SamAccountName">vagrant-domain</S>
    <S N="SID">S-1-5-21-3849754078-2265128455-4044759769-1104</S>
    <Nil N="Surname" />
    <S<EMAIL_ADDRESS>  </Props>
</Obj>

When using a PowerShell pipeline that is targeted against the current runspace (no serialization occurs) I can replicate this problem. What's even more interesting is that the property itself still exists when checking $obj.PSObject.Properties and it has the value
$ps = [PowerShell]::Create()

$null = $ps.AddScript(@'
Get-ADUser -Identity vagrant-domain -Property City
'@)

$user = $ps.Invoke()

"Direct: '$($user[0].City)'"
"By Prop: '$(($user[0].PSObject.Properties | Where-Object Name -eq City).Value)'"

Direct: ''
By Prop: 'My City'

The same thing applies when running it other ways like through Invoke-Command but interestingly not ForEach-Object -Parallel even though no serialization occurs through that method. I believe remote runspaces are fine because when the ADUser object is serialized the type isn't preserved so the object that is returned is just a generic property bag.
I'm honestly not sure why this is happening so interested to hear what others may have to say.

This is definitely a weird one. When testing against a remote Runspace I can see the CLIXML returned contains the property and accessing $obj.City works just fine
<Obj RefId="0">
  <TN RefId="0">
    <T>Microsoft.ActiveDirectory.Management.ADUser</T>
    <T>Microsoft.ActiveDirectory.Management.ADAccount</T>
    <T>Microsoft.ActiveDirectory.Management.ADPrincipal</T>
    <T>Microsoft.ActiveDirectory.Management.ADObject</T>
    <T>Microsoft.ActiveDirectory.Management.ADEntity</T>
    <T>Microsoft.ActiveDirectory.Management.ADPropertyCollection</T>
    <T>System.Object</T>
  </TN>
  <ToString>CN=vagrant-domain,CN=Users,DC=domain,DC=test</ToString>
  <Props>
    <S N="City">My City</S>
    <S N="DistinguishedName">CN=vagrant-domain,CN=Users,DC=domain,DC=test</S>
    true
    <Nil N="GivenName" />
    <S N="Name">vagrant-domain</S>
    <S N="ObjectClass">user</S>
    <G N="ObjectGUID">bdfa699e-dfb9-41ba-864d-6a2f162f579d</G>
    <S N="SamAccountName">vagrant-domain</S>
    <S N="SID">S-1-5-21-3849754078-2265128455-4044759769-1104</S>
    <Nil N="Surname" />
    <S<EMAIL_ADDRESS>  </Props>
</Obj>

When using a PowerShell pipeline that is targeted against the current runspace (no serialization occurs) I can replicate this problem. What's even more interesting is that the property itself still exists when checking $obj.PSObject.Properties and it has the value
$ps = [PowerShell]::Create()

$null = $ps.AddScript(@'
Get-ADUser -Identity vagrant-domain -Property City
'@)

$user = $ps.Invoke()

"Direct: '$($user[0].City)'"
"By Prop: '$(($user[0].PSObject.Properties | Where-Object Name -eq City).Value)'"

Direct: ''
By Prop: 'My City'

The same thing applies when running it other ways like through Invoke-Command but interestingly not ForEach-Object -Parallel even though no serialization occurs through that method. I believe remote runspaces are fine because when the ADUser object is serialized the type isn't preserved so the object that is returned is just a generic property bag.
I'm honestly not sure why this is happening so interested to hear what others may have to say.

[ ]<PHONE_NUMBER>593420********

[ ]<PHONE_NUMBER>593420********

Does this only occur for an AD object?  I cannot run the repro, and it would be interesting to know if there is a general type of object/property that causes this problem.  For example a script property on an object may not run correctly when passed to a different runspace, due to script block runspace affinity.

Does this only occur for an AD object?  I cannot run the repro, and it would be interesting to know if there is a general type of object/property that causes this problem.  For example a script property on an object may not run correctly when passed to a different runspace, due to script block runspace affinity.

@PaulHigin I have not run into another type of object that this issue occurs on. I've tested your theory using a file object and it does NOT occur.
$File = Get-Item C:\windows\system32\drivers\etc\hosts | Select-Object Mode, LastWriteTime, Length, Name, @{N="LengthInKB";E={$_.Length/1KB}}

Write-Output "Before Runspace - The value of name is: $($File.Name)"
Write-Output "Before Runspace - The value of lengthInKB is: $($File.LengthInKB)"

$Runspace = [runspacefactory]::CreateRunspace()
$PowerShell = [powershell]::Create()
$PowerShell.runspace = $Runspace
$Runspace.Open()

[void]$PowerShell.AddScript({
    Param ($File)
    $a = "Inside Runspace - The value of name is: $($File.Name)"
    $b = "Inside Runspace - The value of lengthInKB is: $($File.LengthInKB)"
    $c = $File

    return @($a,$b,$c)
})
[void]$PowerShell.AddArgument($File)

$a, $b, $c = $PowerShell.Invoke()

$a
$b

Write-Output "After Runspace  - The value of name is: $($c.Name)"
Write-Output "After Runspace  - The value of lengthInKB is: $($c.LengthInKB)"

@PaulHigin I have not run into another type of object that this issue occurs on. I've tested your theory using a file object and it does NOT occur.
$File = Get-Item C:\windows\system32\drivers\etc\hosts | Select-Object Mode, LastWriteTime, Length, Name, @{N="LengthInKB";E={$_.Length/1KB}}

Write-Output "Before Runspace - The value of name is: $($File.Name)"
Write-Output "Before Runspace - The value of lengthInKB is: $($File.LengthInKB)"

$Runspace = [runspacefactory]::CreateRunspace()
$PowerShell = [powershell]::Create()
$PowerShell.runspace = $Runspace
$Runspace.Open()

[void]$PowerShell.AddScript({
    Param ($File)
    $a = "Inside Runspace - The value of name is: $($File.Name)"
    $b = "Inside Runspace - The value of lengthInKB is: $($File.LengthInKB)"
    $c = $File

    return @($a,$b,$c)
})
[void]$PowerShell.AddArgument($File)

$a, $b, $c = $PowerShell.Invoke()

$a
$b

Write-Output "After Runspace  - The value of name is: $($c.Name)"
Write-Output "After Runspace  - The value of lengthInKB is: $($c.LengthInKB)"

I can't speak for @MaxAnderson95 but I've only just tested this with Get-ADUser and the ADUser type. Here are the details:
> $user.PSObject.Properties | Where-Object Name -eq City

BaseObject      : CN=vagrant-domain,CN=Users,DC=domain,DC=test
Tag             :
MemberType      : Property
Value           : My City
IsSettable      : True
IsGettable      : True
TypeNameOfValue : System.String
Name            : City
IsInstance      : True

> $user.GetType()

IsPublic IsSerial Name                                     BaseType
-------- -------- ----                                     --------
True     False    ADUser                                   Microsoft.ActiveDirectory.Management.ADAccount

I haven't really checked other types but in this case it's definitely the ADUser type. What's interesting is that the City property is an actual Property and not a NoteProperty or any other PowerShell ETS one AFAIK. Maybe PowerShell sees the type as ADUser and only accesses the properties on the type itself which City is not usually present?
> [Microsoft.ActiveDirectory.Management.ADUser].GetProperties().Name

GivenName
Surname
UserPrincipalName
Enabled
SamAccountName
SID
DistinguishedName
Name
ObjectClass
ObjectGuid
PropertyNames
AddedProperties
RemovedProperties
ModifiedProperties
PropertyCount
Item

It's hard to replicate this as Add-Member only allows you to add a NoteProperty which works just fine and trying to add a PSAdaptedProperty manually through $user.PSObject.Properties.Add($adaptedProp) fails.

I can't speak for @MaxAnderson95 but I've only just tested this with Get-ADUser and the ADUser type. Here are the details:
> $user.PSObject.Properties | Where-Object Name -eq City

BaseObject      : CN=vagrant-domain,CN=Users,DC=domain,DC=test
Tag             :
MemberType      : Property
Value           : My City
IsSettable      : True
IsGettable      : True
TypeNameOfValue : System.String
Name            : City
IsInstance      : True

> $user.GetType()

IsPublic IsSerial Name                                     BaseType
-------- -------- ----                                     --------
True     False    ADUser                                   Microsoft.ActiveDirectory.Management.ADAccount

I haven't really checked other types but in this case it's definitely the ADUser type. What's interesting is that the City property is an actual Property and not a NoteProperty or any other PowerShell ETS one AFAIK. Maybe PowerShell sees the type as ADUser and only accesses the properties on the type itself which City is not usually present?
> [Microsoft.ActiveDirectory.Management.ADUser].GetProperties().Name

GivenName
Surname
UserPrincipalName
Enabled
SamAccountName
SID
DistinguishedName
Name
ObjectClass
ObjectGuid
PropertyNames
AddedProperties
RemovedProperties
ModifiedProperties
PropertyCount
Item

It's hard to replicate this as Add-Member only allows you to add a NoteProperty which works just fine and trying to add a PSAdaptedProperty manually through $user.PSObject.Properties.Add($adaptedProp) fails.

Just as an FYI you can workaround this issue by adding | Select-Object * to your cmdlet calls as that will convert the AD object to a PSObject with NoteProperties instead.

Just as an FYI you can workaround this issue by adding | Select-Object * to your cmdlet calls as that will convert the AD object to a PSObject with NoteProperties instead.

A script property was just an example of something that may not run on different runspaces.  Simple scripts may run but others may not because of affinity issues (such as accessing a variable only defined in the original runspace).
There may be other reasons why an object property might not evaluate correctly in a different runspace.
Or, it may have nothing to do with the property at all.  But the fact that it seems to only repro on AD objects makes it suspect.

A script property was just an example of something that may not run on different runspaces.  Simple scripts may run but others may not because of affinity issues (such as accessing a variable only defined in the original runspace).
There may be other reasons why an object property might not evaluate correctly in a different runspace.
Or, it may have nothing to do with the property at all.  But the fact that it seems to only repro on AD objects makes it suspect.

Definitely something up with the AD cmdlets, I cannot seem to replicate it when adding a custom adapted property to an object
$ps = [PowerShell]::Create()

$null = $ps.AddScript(@'
$obj = Get-Service -Name WinRM
$obj.PSObject.Properties.Add(($pwd.PSObject.Properties | Where Name -eq Path), $true)

$obj
'@)

$obj = $ps.Invoke()[0]

"Direct: '$($obj.Path)'"
"By Prop: '$(($obj.PSObject.Properties | Where-Object Name -eq Path).Value)'"

# Direct: 'C:\temp'
# By Prop: 'C:\temp'

I tried building my own adapted property but to get/set a value I need to supply an adapter which is hidden behind internal classes so I didn't bother to go any deeper.

Definitely something up with the AD cmdlets, I cannot seem to replicate it when adding a custom adapted property to an object
$ps = [PowerShell]::Create()

$null = $ps.AddScript(@'
$obj = Get-Service -Name WinRM
$obj.PSObject.Properties.Add(($pwd.PSObject.Properties | Where Name -eq Path), $true)

$obj
'@)

$obj = $ps.Invoke()[0]

"Direct: '$($obj.Path)'"
"By Prop: '$(($obj.PSObject.Properties | Where-Object Name -eq Path).Value)'"

# Direct: 'C:\temp'
# By Prop: 'C:\temp'

I tried building my own adapted property but to get/set a value I need to supply an adapter which is hidden behind internal classes so I didn't bother to go any deeper.

Ronald Leslie Ritchie  75 jasper ave smiths falls Ontario Canada k7a 4c8<PHONE_NUMBER>
-------- Original message --------From: Jordan Borean<EMAIL_ADDRESS>Date: 2021-01-13  2:44 p.m.  (GMT-05:00) To: PowerShell/PowerShell<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [PowerShell/PowerShell] Passing Active Directory Objects into a
Runspace Stripts All Non-Default Properties (#14604)
I can't speak for @MaxAnderson95 but I've only just tested this with Get-ADUser and the ADUser type. Here are the details:

$user.PSObject.Properties | Where-Object Name -eq City

BaseObject      : CN=vagrant-domain,CN=Users,DC=domain,DC=test
Tag             :
MemberType      : Property
Value           : My City
IsSettable      : True
IsGettable      : True
TypeNameOfValue : System.String
Name            : City
IsInstance      : True

$user.GetType()

IsPublic IsSerial Name                                     BaseType

True     False    ADUser                                   Microsoft.ActiveDirectory.Management.ADAccount
I haven't really checked other types but in this case it's definitely the ADUser type. What's interesting is that the City property is an actual Property and not a NoteProperty or any other PowerShell ETS one AFAIK. Maybe PowerShell sees the type as ADUser and only accesses the properties on the type itself which City is not usually present?

[Microsoft.ActiveDirectory.Management.ADUser].GetProperties().Name

GivenName
Surname
UserPrincipalName
Enabled
SamAccountName
SID
DistinguishedName
Name
ObjectClass
ObjectGuid
PropertyNames
AddedProperties
RemovedProperties
ModifiedProperties
PropertyCount
Item
It's hard to replicate this as Add-Member only allows you to add a NoteProperty which works just fine and trying to add a PSAdaptedProperty manually through $user.PSObject.Properties.Add($adaptedProp) fails.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

Ronald Leslie Ritchie  75 jasper ave smiths falls Ontario Canada k7a 4c8<PHONE_NUMBER>
-------- Original message --------From: Jordan Borean<EMAIL_ADDRESS>Date: 2021-01-13  2:44 p.m.  (GMT-05:00) To: PowerShell/PowerShell<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [PowerShell/PowerShell] Passing Active Directory Objects into a
Runspace Stripts All Non-Default Properties (#14604)
I can't speak for @MaxAnderson95 but I've only just tested this with Get-ADUser and the ADUser type. Here are the details:

$user.PSObject.Properties | Where-Object Name -eq City

BaseObject      : CN=vagrant-domain,CN=Users,DC=domain,DC=test
Tag             :
MemberType      : Property
Value           : My City
IsSettable      : True
IsGettable      : True
TypeNameOfValue : System.String
Name            : City
IsInstance      : True

$user.GetType()

IsPublic IsSerial Name                                     BaseType

True     False    ADUser                                   Microsoft.ActiveDirectory.Management.ADAccount
I haven't really checked other types but in this case it's definitely the ADUser type. What's interesting is that the City property is an actual Property and not a NoteProperty or any other PowerShell ETS one AFAIK. Maybe PowerShell sees the type as ADUser and only accesses the properties on the type itself which City is not usually present?

[Microsoft.ActiveDirectory.Management.ADUser].GetProperties().Name

GivenName
Surname
UserPrincipalName
Enabled
SamAccountName
SID
DistinguishedName
Name
ObjectClass
ObjectGuid
PropertyNames
AddedProperties
RemovedProperties
ModifiedProperties
PropertyCount
Item
It's hard to replicate this as Add-Member only allows you to add a NoteProperty which works just fine and trying to add a PSAdaptedProperty manually through $user.PSObject.Properties.Add($adaptedProp) fails.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

Is Microsoft.ActiveDirectory.Management.ADAccount an COM object? I see:
Item                   ParameterizedProperty Microsoft.ActiveDirectory.Management.ADPropertyValueCollection Item(str...

Maybe it is COM interaction issue? /cc @SeeminglyScience
Is an object cloned when passed to runspace? Maybe the issue related the cloning (in Engine or in the object)?

Is Microsoft.ActiveDirectory.Management.ADAccount an COM object? I see:
Item                   ParameterizedProperty Microsoft.ActiveDirectory.Management.ADPropertyValueCollection Item(str...

Maybe it is COM interaction issue? /cc @SeeminglyScience
Is an object cloned when passed to runspace? Maybe the issue related the cloning (in Engine or in the object)?

Is Microsoft.ActiveDirectory.Management.ADAccount an COM object? I see:
Item                   ParameterizedProperty Microsoft.ActiveDirectory.Management.ADPropertyValueCollection Item(str...

Maybe it is COM interaction issue? /cc @SeeminglyScience
Is an object cloned when passed to runspace? Maybe the issue related the cloning (in Engine or in the object)?

The AD module uses a custom adapter that just isn't registered in the new runspace.  Issue is fixed if you add Import-Module ActiveDirectory in the AddScript block.

Is Microsoft.ActiveDirectory.Management.ADAccount an COM object? I see:
Item                   ParameterizedProperty Microsoft.ActiveDirectory.Management.ADPropertyValueCollection Item(str...

Maybe it is COM interaction issue? /cc @SeeminglyScience
Is an object cloned when passed to runspace? Maybe the issue related the cloning (in Engine or in the object)?

The AD module uses a custom adapter that just isn't registered in the new runspace.  Issue is fixed if you add Import-Module ActiveDirectory in the AddScript block.

@SeeminglyScience thanks for the explanation, it still seems weird that the property is present through $obj.PSObject.Properties yet you can't access it through the dot notation.

It's just sorta how it's stored. The binder looks in PSObject._instanceMembers which only contains explicitly added members.  You can see the adapted property in _members if it's already been retrieved due to caching, but that's just not where the binder is looking.

@SeeminglyScience thanks for the explanation, it still seems weird that the property is present through $obj.PSObject.Properties yet you can't access it through the dot notation.

It's just sorta how it's stored. The binder looks in PSObject._instanceMembers which only contains explicitly added members.  You can see the adapted property in _members if it's already been retrieved due to caching, but that's just not where the binder is looking.


style: Change logo color to new branding

SUMMARY
Updates the color of the Superset logo to match the new branding primary color from Cartel
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
See images in diff
TEST PLAN
View images - are they ok?
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Reviewers
@mistercrunch @rusackas

@willbarrett some of the png files seem significantly smaller, did they get exported in a scaled down form?

hmmm....yeah, looks like the dimensions are off a bit. Updating shortly...

Files look about right... but they're not in use anywhere :)

Yup. This is press kit stuff.

Codecov Report

Merging #10090 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10090      +/-   ##
==========================================
+ Coverage   65.80%   65.81%   +0.01%
==========================================
  Files         587      592       +5
  Lines       31158    31191      +33
  Branches     3191     3191
==========================================
+ Hits        20503    20529      +26
- Misses      10477    10484       +7
  Partials      178      178

Flag
Coverage Δ

#javascript
59.74% <ø> (+<0.01%)
:arrow_up:

#python
70.11% <ø> (+0.01%)
:arrow_up:

Impacted Files
Coverage Δ

superset/db_engine_specs/mysql.py
78.26% <0.00%> (-13.05%)
:arrow_down:

superset/views/core.py
75.21% <0.00%> (-1.14%)
:arrow_down:

superset/models/core.py
85.42% <0.00%> (-0.88%)
:arrow_down:

superset/connectors/sqla/models.py
88.82% <0.00%> (-0.14%)
:arrow_down:

superset/views/base.py
73.41% <0.00%> (-0.07%)
:arrow_down:

superset/views/__init__.py
100.00% <0.00%> (ø)

superset/views/css_templates.py
100.00% <0.00%> (ø)

superset/views/redirects.py
70.00% <0.00%> (ø)

superset/views/health.py
100.00% <0.00%> (ø)

superset/views/key_value.py
87.87% <0.00%> (ø)

... and 6 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 987cb6e...0c7e68c. Read the comment docs.

From here it's easy to alter/copy to images/ and elsewhere


The format PRIVATE KEY is not yet implemented

I would like to load a pem file into C# and I can't find the proper tool. I was hoping pem-utils is exactly what I need, but I'm getting the following error:
Unhandled Exception: System.NotImplementedException: The format PRIVATE KEY is not yet implemented
I didn't know there are other formats besides "PRIVATE KEY". Is any other format 'implemented' in this library?
This is how my code looks like:
        RSA rsa = RSA.Create();
        string file = "/home/user/certs/ca5.pem";
        using (var stream = File.OpenRead(file))
        using (var reader = new PemReader(stream))
        {
            var rsaParameters = reader.ReadRsaKey();
            rsa.ImportParameters(rsaParameters);
        }
        Console.WriteLine("RSA: " + rsa.KeySize);

It crashes at ReadRsaKey() line.
Any help would be great.

The only private key format currently supported is RSA PRIVATE KEY. What type of key are you dealing with?
If possible, can you generate a similar private key that you can share as a test key here?

Many thanks to you, sir! I used the Keystore Explorer tool to generate private key, and for some reason this tool generates the first line as PRIVATE KEY and not RSA PRIVATE KEY. If I generate the key using openssl it all works fine.

Hi, now I'm getting the following error:
Unhandled Exception: System.InvalidOperationException: Data on the stream doesn't match the required PEM format
What does that mean?

It means your private key does not adhere to this regular expression: https://github.com/huysentruitw/pem-utils/blob/master/src/PemUtils/PemReader.cs#L56
Can you please post an example private key?

Sorry, I've also figured this out. I had both keys in the file. After removing public key from the file (and with correct format) it worked.


[platform_channel] Allow EventChannel or its broadcast stream to have a generic Stream type

Use case
In a project, I use an EventChannel to use a hardware feature on an Android device.
To do so, I attach a listener to the broadcast stream.
As I control the hardware, I am 100% sure, that the return type of the stream is String.
However, the current implementation only provides a dynamic return type. So in order to use the correct type in my onEvent function, I would have to do a type cast.
void onEvent(String message) {
    print(message);
}

const EventChannel infoChannel = EventChannel('com.example.app/information');
infoChannel.receiveBroadcastStream()
    .cast<String>()
    .listen(onEvent);

If I didn't want to do a cast, I would have to use dynamic instead of String in my onEvent function:
void onEvent(dynamic message) {
    if (message is! String) {
        throw Exception("Unexpected type received from information channel");
    }
    print(message);
}

const EventChannel infoChannel = EventChannel('com.example.app/information');
infoChannel.receiveBroadcastStream().listen(onEvent);

When I don't use the cast but still use type String, I get a Runtime exception with the following message:
════════ Exception caught by widgets library ═══════════════════════════════════
The following _TypeError was thrown attaching to the render tree:
type '(String) => void' is not a subtype of type '(dynamic) => void'

Proposal
My proposal is to either make the EventChannel class or the receiveBroadcastStream function generic. For example like this:
void onEvent(String message) {
    print(message);
}

const EventChannel infoChannel = EventChannel<String>('com.example.app/information');
infoChannel.receiveBroadcastStream().listen(onEvent);

or
void onEvent(String message) {
    print(message);
}

const EventChannel infoChannel = EventChannel('com.example.app/information');
infoChannel.receiveBroadcastStream<String>().listen(onEvent);

@bjoernahrens
don't we have pigeon for this exact purpose?


Test failure for 0.4.6 on s390x

The test rules::pyflakes::tests::preview_rules::rule_unusedimport_path_new_f401_28_all_multiple_init_py_expects is failing on s390x:
---- rules::pyflakes::tests::preview_rules::rule_unusedimport_path_new_f401_28_all_multiple_init_py_expects stdout ----
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ Snapshot Summary ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Snapshot file: crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__preview__F401_F401_28__all_multiple____init__.py.snap
Snapshot: preview__F401_F401_28__all_multiple/__init__.py
Source: crates/ruff_linter/src/rules/pyflakes/mod.rs:228
────────────────────────────────────────────────────────────────────────────────
-old snapshot
+new results
────────────┬───────────────────────────────────────────────────────────────────
    8     8 │ 5 5 | from . import unused, renamed as bees  # F401: add to __all__
    9     9 │ 6 6 |
   10    10 │ 7 7 |
   11    11 │ 8   |-__all__ = [];
   12       │-  8 |+__all__ = ["bees", "unused"];
         12 │+  8 |+__all__ = ["unused", "bees"];
   13    13 │
   14    14 │ __init__.py:5:34: F401 [*] `.renamed` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
   15    15 │   |
   16    16 │ 5 | from . import unused, renamed as bees  # F401: add to __all__
┈┈┈┈┈┈┈┈┈┈┈┈┼┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
   22    22 │ 5 5 | from . import unused, renamed as bees  # F401: add to __all__
   23    23 │ 6 6 |
   24    24 │ 7 7 |
   25    25 │ 8   |-__all__ = [];
   26       │-  8 |+__all__ = ["bees", "unused"];
         26 │+  8 |+__all__ = ["unused", "bees"];
────────────┴───────────────────────────────────────────────────────────────────

(full CI run log here: https://gitlab.alpinelinux.org/WhyNotHugo/aports/-/jobs/1403448)
I think that the order of the items here is non-deterministic somehow and varies depending on CPU architecture.

@plredmond do you want to take a look at this? It probably requires sorting the items or to use an IndexSet.

@MichaReiser The implementation creates multiple insertion edits that use the same insertion point. AFAIK, their order is deterministic. The implementation has no logic for resolving the order that ruff applies the insertions. I.e. It should be deterministic.
Otherwise I guess this ordering could change if there was a change to how multi-edit fixes get applied.

@plredmond I don't think it's about the edit ordering that is non deterministic but it is the order of the items in __all__ that is non deterministic (because it uses a hash set?)

@MichaReiser I'm not sure what you mean by "the items in __all__" because the rule implementation never creates or stores items in a data structure that represents __all__ nor uses the AST's "export" code. You might be referring to the list of unused import-bindings, which yes, could be introduce a nondeterministic order. I'll try to apply your advice there and rule it out. Is there any way for me to test this within our (astral) ci/cd infra?

@WhyNotHugo Can you check whether this issue is resolved on dcabd04caf85d7c0081ebba9f9028b9af9d686d3 or in the next release?

I applied dcabd04 onto v0.4.6 and that failed:
>>> ruff: dcabd04caf85d7c0081ebba9f9028b9af9d686d3.patch
patching file crates/ruff_linter/src/rules/pyflakes/rules/unused_import.rs
patching file crates/ruff_python_semantic/src/binding.rs

[...]

---- rules::pyflakes::tests::preview_rules::rule_unusedimport_path_new_f401_28_all_multiple_init_py_expects stdout ----
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ Snapshot Summary ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Snapshot file: crates/ruff_linter/src/rules/pyflakes/snapshots/ruff_linter__rules__pyflakes__tests__preview__F401_F401_28__all_multiple____init__.py.snap
Snapshot: preview__F401_F401_28__all_multiple/__init__.py
Source: crates/ruff_linter/src/rules/pyflakes/mod.rs:228
────────────────────────────────────────────────────────────────────────────────
-old snapshot
+new results
────────────┬───────────────────────────────────────────────────────────────────
    8     8 │ 5 5 | from . import unused, renamed as bees  # F401: add to __all__
    9     9 │ 6 6 |
   10    10 │ 7 7 |
   11    11 │ 8   |-__all__ = [];
   12       │-  8 |+__all__ = ["bees", "unused"];
         12 │+  8 |+__all__ = ["unused", "bees"];
   13    13 │
   14    14 │ __init__.py:5:34: F401 [*] `.renamed` imported but unused; consider removing, adding to `__all__`, or using a redundant alias
   15    15 │   |
   16    16 │ 5 | from . import unused, renamed as bees  # F401: add to __all__
┈┈┈┈┈┈┈┈┈┈┈┈┼┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
   22    22 │ 5 5 | from . import unused, renamed as bees  # F401: add to __all__
   23    23 │ 6 6 |
   24    24 │ 7 7 |
   25    25 │ 8   |-__all__ = [];
   26       │-  8 |+__all__ = ["bees", "unused"];
         26 │+  8 |+__all__ = ["unused", "bees"];
────────────┴───────────────────────────────────────────────────────────────────

I think the issue is that for binding_id in scope.binding_ids() is not deterministic. So we should enforce determinism elsewhere (e.g., by sorting in fix_by_reexporting or similar).

Wouldn't the nondeterminism of scope.binding_ids() result order be resolved by insertion into the BTreeMap? The only place this data comes back out is when we iterate over the BTreeMaps.
I'll try sorting in fix_by_reexporting, anyhow.

@WhyNotHugo could you try out ruff.f401.nondet2?

I'll mark this as closed. @WhyNotHugo feel free to reopen if you keep running into this issue with 0.4.8

Thanks micha

On Jun 4, 2024, at 04:50, Micha Reiser @.***> wrote:

I'll mark this as closed. @WhyNotHugo feel free to reopen if you keep running into this issue with 0.4.8. Thanks @plredmond for looking into and fixing the issue.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

0.4.7 builds okay on s390x. Thanks!


Consider @grpc/grpc-js

Hey. This is a cool library!
I was recently trying to integrate this into https://github.com/Urigo/graphql-mesh/pull/1086 and had some conflicts when passing @grpc/grpc-js CredentialsChannel to your Client constructor. See the relevant comment in https://github.com/Urigo/graphql-mesh/pull/1086

There is a conflict when using grpc-reflection-js. The CredentialsChannel type used by it is different than the CredentialsChannel type used by @grpc/grpc-js. This is why I swapped in grpc. However, on the grpc package README, it states that this package is deprecated (presumably in favor of @grpc/grpc-node? :thinking: :man_shrugging:)
I find this odd, because when using the JS protoc plugin, all of the generated gRPC server/client code uses the grpc package.
I suppose until protoc updates it's output dependency, then using grpc should be ok. Alternatively, we can submit a PR to grpc-reflection-js to update that dependency to use @grpc/grpc-js. Whew :sweat_smile:

Hey, thanks for your interest.
I have reviewed your changes. Originally, I was using grpc because I want to create some changes to uw-labs/bloomrpc.
I think we can migrate to @grpc/grpc-js since protoc also supports @grpc/grpc-js (see this for the detail).
But I will not available in a few weeks. If you're in hurry, don't hesitate to create PR for this.

Hey @tatemz, I just published your changes in version 0.0.7

:100:  Awesome!
Are the release/tagged versions in git synchronized with the versions in the NPM registry?

Not yet, I was getting problems when publishing changes to NPM registry via Github Action and ended up making many unused tags and releases. Will fix it someday later.

I just made them synchronized.


Credit usage explanations

This pull request adds initial documentation on job credit usage and estimation.
Needs further elaboration, but we want to cover a few of these aspects for SRR3/LPS.

I left out added value for now, because with what is exposed today in openEO platform, users should not encounter this, or do they?

I'm going to merge this one to have it online before the deadline. People have been notified, and we can always update afterwards.

Thanks @jdries for uploading. We confirm this document for now.


Javalin extensions

@dherges made a PR which added extensions to the Javalin class (#208). I felt sure that this was a good addition when I first saw the PR, but upon playing with it I've become less certain, so I've reverted the change in 9fa4cf98ab2eaf633f3d70224fd680d38e398c74.
The tests used a static instance of Javalin, which made them a bit confusing. To re-add this I think we need self-contained tests (each test should have a Javalin.create()), as well as some demonstrations of compelling use-cases for extensions. I have not been able to find a use case where an extension-lambda makes sense. The original example in @dherges PR was nice (register(new PersistenceLayer("jdbc://connection-url")), but I think we need to explore the feature more and come up with a few more examples before adding it back.

@dherges have you looked any more at this?

Since you can subclass Javalin and add context-extensions now, I think this can be closed.

Yes, I subclass and I haven't look into it further.


build: remove parallelization for E2E tests

SUMMARY
Follow up for #12241
Clean up Cypress parallelization for now since CI has been slow and multi-container runs are not really working anyway.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A
TEST PLAN
CI must pass
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #12290 (a91a73c) into master (d4a854e) will decrease coverage by 3.89%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12290      +/-   ##
==========================================
- Coverage   67.17%   63.28%   -3.90%
==========================================
  Files        1002      485     -517
  Lines       49231    29815   -19416
  Branches     5000        0    -5000
==========================================
- Hits        33073    18867   -14206
+ Misses      16035    10948    -5087
+ Partials      123        0     -123

Flag
Coverage Δ

cypress
?

javascript
?

python
63.28% <ø> (-0.93%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
32.65% <0.00%> (-59.19%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/db_engine_specs/hive.py
54.61% <0.00%> (-29.62%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/presto.py
70.56% <0.00%> (-12.78%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

superset/views/database/validators.py
78.94% <0.00%> (-5.27%)
:arrow_down:

... and 538 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 673dea7...a91a73c. Read the comment docs.

Codecov Report

Merging #12290 (a91a73c) into master (d4a854e) will decrease coverage by 3.89%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12290      +/-   ##
==========================================
- Coverage   67.17%   63.28%   -3.90%
==========================================
  Files        1002      485     -517
  Lines       49231    29815   -19416
  Branches     5000        0    -5000
==========================================
- Hits        33073    18867   -14206
+ Misses      16035    10948    -5087
+ Partials      123        0     -123

Flag
Coverage Δ

cypress
?

javascript
?

python
63.28% <ø> (-0.93%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
32.65% <0.00%> (-59.19%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/db_engine_specs/hive.py
54.61% <0.00%> (-29.62%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/presto.py
70.56% <0.00%> (-12.78%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

superset/views/database/validators.py
78.94% <0.00%> (-5.27%)
:arrow_down:

... and 538 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 673dea7...a91a73c. Read the comment docs.


Tracking: External ecosystem preparedness for sound null safety

Goal: We'd like to track progress on the % of flutter run sessions that are fully sound. This will help us track progress towards our next Dart major version.
Frequency: we'd like to pull these stats on a weekly basis.
How to
Here's the analytics query; must manually adjust end-date of query
Todos

[ ] Measure: % of flutter run sessions that are fully sound
[ ] Document how you pulled this information (can be in a doc or in this issue).
[ ] Automate a weekly report if possible.

@devoncarew would you be able to help with this issue, or is there someone else on the ecosystem who can?

Given that we're already tracking the data and have a query to measure what we want, I see two reasonable solutions here:

manually check in on the % of flutter run sessions every week; populate that info in a spreadsheet for historical data
use a PLX dashboard to query the analytics data and populate an automated dashboard

I suspect that 1. is something that TPgM would drive. For 2., @jayoung-lee (on the UXR team) has lots of PLX experience and could help set this up. Given the shorter timeframe we care about this data, I think 1. would be sufficient for what we need - a little more manual work per week but we'd just need to do it for a quarter or two.

I think you can use Google Sheet add-on for Google Analytics for 1. to minimize manual work. See https://developers.google.com/analytics/solutions/google-analytics-spreadsheet-add-on
Happy to help with 2. as well!

Thanks Jayoung for the link. I have created this sheet that generates Google analytics report reflecting data from last 90 days.
The report is scheduled to run daily.
On the third tab, I have also added a visualization chart that should get updated automatically as well.
https://docs.google.com/spreadsheets/d/1IBrwsMk_mkZVP4_GaCJ9hwjxV09U8OgVo955-EekaHM/edit#gid=0

Created the dashboard as well
https://dashboards.corp.google.com/_7c0b9e33_0972_4f1e_ac1c_49b1b1be337e
Who/When are we looking at these progress metrics?

@mit-mit can you please take a look at what Khyati has so far?


ktor-client: response replaces request body

Description
When using logbook-ktor-client together with a custom sink that merges request and response, the request body is always the response body, i.e. response overwrites request. I think this is caused by the State (https://github.com/zalando/logbook/blob/main/logbook-ktor-common/src/main/kotlin/org/zalando/logbook/common/State.kt) subclasses being objects, which makes them a singleton.
Expected Behavior
I expect that at the point of calling the sink, the request contains the request body and the response contains the response body.
Actual Behavior
At the point of calling the sink, the request's body is that of the response.
Possible Fix
Make State subclasses a class that has to be instanciated so that each request and response has their own State instance.
Steps to Reproduce
Please see LogbookClientWithSinkTest in my upcoming PR.
Context
I'm trying to log http requests and responses when using ktor client
Your Environment
Logbook version 3.9.0

@kasmarian thank you for approving and merging my PR!


CFRlogger accessviolation

The fix for the access violation on shutdown was a MathWorks specific fix in ServiceTracker.tpp which is not included in this pull request. While fixing the access violation a couple of small bugs were detected in CFRLogger and some inconsistencies with the way CFRLogger was being initialized in CoreBundleContext.cpp, as compared to other Service Trackers (like ServiceHook) so this PR is addressing those issues.

Codecov Report

Merging #706 (4939a81) into development (e9b190f) will increase coverage by 0.22%.
The diff coverage is 77.50%.

@@               Coverage Diff               @@
##           development     #706      +/-   ##
===============================================
+ Coverage        90.45%   90.67%   +0.22%
===============================================
  Files              248      189      -59
  Lines            13628    10139    -3489
===============================================
- Hits             12327     9194    -3133
+ Misses            1301      945     -356

Impacted Files
Coverage Δ

framework/src/util/CFRLogger.cpp
73.23% <75.00%> (-0.10%)
:arrow_down:

framework/src/bundle/CoreBundleContext.cpp
94.68% <100.00%> (-0.49%)
:arrow_down:

...endium/ServiceComponent/src/ComponentException.cpp
0.00% <0.00%> (-100.00%)
:arrow_down:

...m/ServiceComponent/src/ServiceComponentRuntime.cpp
0.00% <0.00%> (-100.00%)
:arrow_down:

...m/DeclarativeServices/src/manager/BundleLoader.cpp
63.82% <0.00%> (-6.32%)
:arrow_down:

framework/src/service/ServiceEvent.cpp
71.79% <0.00%> (-5.76%)
:arrow_down:

...mework/src/service/ServiceReferenceBasePrivate.cpp
86.45% <0.00%> (-5.64%)
:arrow_down:

...pendium/tools/SCRCodeGen/ManifestParserFactory.hpp
83.33% <0.00%> (-5.56%)
:arrow_down:

framework/src/service/ServiceRegistrationBase.cpp
77.66% <0.00%> (-4.24%)
:arrow_down:

framework/src/util/FrameworkEvent.cpp
61.81% <0.00%> (-3.94%)
:arrow_down:

... and 198 more

The fix for the access violation on shutdown was a MathWorks specific fix in ServiceTracker.tpp which is not included in this pull request. While fixing the access violation a couple of small bugs were detected in CFRLogger and some inconsistencies with the way CFRLogger was being initialized in CoreBundleContext.cpp, as compared to other Service Trackers (like ServiceHook) so this PR is addressing those issues.

Hi Patty. The CFRLogger took its structure from the SCRLogger and CMLogger classes. The code as currently presented is more inconsistent with the existing patterns now than it was originally. I'm not suggesting any change with this comment but just wanted to get some clarification as to what inconsistencies you saw given that it was meant to mimic SCR/CMLogger rather than ServiceHook.


Advise to look for existing translations

So folks don't fork the project before realising there's already someone attempting to translate the game.

Good point, thanks for this PR! 🎉 ( Also this resolves this issue: https://github.com/ncase/anxiety/issues/9 )


Range types are not supported

Let’s say we have the following .hrl file:
-type foo() :: 1..10.

Then including this file gives the following compilation error:
./foo.lfe:3: unknown type: (range 2)

The latest push to develop contains support for the range types. Check the documentation for syntax. Please test.

Thank you, it seems to work (at least no compilation errors are raised).


Question: Does OpenStreamSync Remain Blocked Indefinitely After Reaching MaxStreams Limit?

Let's say we have reached the MaxStreams limit and got blocked in OpenStreamSync. How does it unblock once one of the active streams is closed? Intuitively, that should unblock the OpenStreamSync as a closed stream freed up a stream slot. However, from what I can see, only the SetMaxStream  will unblock it, triggered by either MaxStreamsFrame or TransportParameters during handshake. Does it mean reaching the limit never resolves itself?

I didn't realize MaxStreamsFrame is sent every time an incoming stream is closed. That answer my question as no


server restart - client won't reconnect

When I restart node.js server, the client won't reconnect.
What is the preferred solution here?

@g00fy- I have the same issue. did you figure out a solution?


Use consistent naming for creating, copying and destroying an item.

Throughout the gobject/gtk/glib world there exists functions to copy something
eg. g_strdup in order to duplicate a gchar* and the copy should be g_freed. All GObject derived instances
can be copied shallowly by increasing its refcount using e.g. g_object_ref. To free those instances you can rely on g_object_unref, or g_clear_object where the instance is destroyed when the refcount drops to 0.
Creating objects/boxed types
Use psy_some_object_new()
Don't use names such as psy_some_object_create, _new is used the most.
Copying (G)object data
Copying reference types
You should use psy_some_refcounted_type_ref(instance)
You'll obtain a pointer to the object with the refcount increased, but it point to the same memory as before, hence it is the same object.
So, you'll get a copy by reference kind of behavior.
Copying plain structures/boxed types
Use psy_some_plain_struct_copy(instance)
This will result in a deep copy, you'll obtain a new pointer to a new object, whose value(s) are the same as before.
Freeing instances previously created
Freeing refcounted objects
use psy_some_refcounted_type_unref(instance)
Freeing plain structures/boxed types
use psy_some_plain_struct_free
so don't use psy_*_destroy

Additionally, I think it makes sense to create a psy_some_object_free for every object in the book, as we can check then whether the pointer is a PsySomeObject before calling g_object_unref on the object. When we have or own destructor for every object no one will add a function with a _destroy name, and makes it easier to detect traces to where the fault happened.


How about adding priority to the Record struct?

If you would like to use libdns for something like MX records it would be really nice to have some kind of way to manage priority.

True, we could add an optional Priority field to the Record struct. Do you have an actual need for this?

No current need.
But if it's gonna be one lib to rule them all then it will be necessary.

I've added a Priority field. If that looks good, I can tag v0.2.1 I guess.

Good enough for me.

I know this is closed, but having just created a provider for powerdns does this mean I (as well as all other providers) need to parse out the priority in the responses back from upstream and convert it to the new format?  Should it support both formats on the way in?  Also, since 0 is a valid priority for an MX record, knowing which format the user is giving us gets slightly tricky.  Some guidance / documentation on what providers should do in the following scenarios would be helpful:
using the MX example,
What if the caller passes in a []records slice with
0 mx.example.com
and priority of 0
what if they pass priority of 0, mx of
10 mx.eample.com
And should all response list types have the priority parsed out and no more concatenation?  What about SOA records or the like?
Sorry if this isn't the proper forum for discussion

@nathanejohnson

I know this is closed, but having just created a provider for powerdns does this mean I (as well as all other providers) need to parse out the priority in the responses back from upstream and convert it to the new format? Should it support both formats on the way in?

What "format"? Can you be more specific?

what if they pass priority of 0, mx of 10 mx.eample.com

That doesn't make any sense, what do you mean "MX of"?


Producer object

Adds new producer object to metadata object. This is to describe an event origin such as Vendor and Product.
Signed-off-by: Julian Crowley<EMAIL_ADDRESS>

Replaced with metadata.product


Feedback for "Including SAP Systems in a Formation"

https://help.sap.com/products/BTP/65de2977205c403bbc107264b8eccf4b/68b04fa73aa740cb96ed380a85a4761a.html?locale=en-US&version=Cloud
This seems to be a new step in the help tutorial, since I don't recognize this since last visit in december. It would be awesome to know what this formation does more techincally. Could there be a section added to this?
And I am a bit worried this is chosen to be on global account level. I am SAP partner, working as external consultant for customers. Often we are not allowed to be assigned roles on global level, only subaccount. This troubles this setup, as the customer often would not know what to do and needs help. It seems complicating things having these things set on such high level.

Hello @kuja11,
Thank you for your valuable feedback!
A formation is a logical grouping of SAP systems that can be extended in a single business scenario. Formations allow you to combine SAP systems, SAP applications, SAP BTP components, and runtimes to simplify the connectivity setup and to provide a unified view of all components required for the implementation of your extension scenario.
I have added this definition to the Including SAP Systems in a Formation.
The "Formations" tab in the SAP BTP cockpit has been around for some time. It was announced on May 7, 2020: release note: Extensions - Grouping SAP Solutions in a Formation.
There is also a "formation" entry in the SAP BTP Glossary.

Hello @kuja11 ,
The idea to have systems connected on a global account level has two main reasons:

Connect the system once to BTP and reuse it for extension applications residing in different subaccounts
Limit the permissions to enable the system for extensibility to the global account admin only, and also considering the goal to provide access only to a specific API/set of APIs in future.


null values are inserted as string "null"

A query query like this
await db.execute('INSERT INTO settings(name, value) VALUES($1, $2)', ['test', null])

inserts the string "null" into the value column.

Is there a workaround for this?

Is there a workaround for this?

A composed query similar to this:
    // Executes a "Insert if not exists"
    // table: Table name
    // columnNames: (Array) The table must have a 'uid' that must not be passed in.
    // fields:
    async insert({table, columnNames, fields}) {
        try {
            // `uid` must be the first column as it is referenced in the insert statement (see below).
            columnNames = ['uid', ...columnNames]
            let columns = []
            let columnReferences = []
            let values = []
            let counter = 1
            // NOTE: A composed query is required as NULL values in the values array are otherwise inserted
            // as string values 'NULL'. (2022-03-02)
            // See: https://github.com/tauri-apps/tauri-plugin-sql/issues/94 columns with NULL values must be excluded.
            columnNames.filter(attr => fields.hasOwnProperty(camelize(attr)) && fields[camelize(attr)] !== null)
                .forEach(attr => {
                    columns.push(attr)
                    columnReferences.push(`$${counter}`)
                    values.push(fields[camelize(attr)])
                    counter += 1
                })
            let query = `INSERT INTO ${table} (${columns.join(', ')})
                         SELECT ${columnReferences.join(', ')}
                         WHERE NOT EXISTS(SELECT 1 FROM ${table} WHERE uid = $1)`
            return await this.db.execute(query, values)
        } catch (error) {
            this.handleError(error)
        }
    }

Possibly related
https://github.com/tauri-apps/tauri-plugin-sql/issues/121#issuecomment-1293810650


picard-tools: 2.18.1 -> 2.18.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/picard-tools/versions.
These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 2.18.2 with grep in /nix/store/sp6gc6c6knpvpsmfdx8074qclcsgs931-picard-tools-2.18.2
directory tree listing: https://gist.github.com/114857d15222eccda94501b5a871fb24

cc @jbedo for review

Success on x86_64-linux (full log)
Attempted: picard-tools
Partial log (click to expand)
building '/nix/store/rk2qlhqhmj66m9kmdmmccp0dm89jg7x7-picard.jar.drv'...

trying https://github.com/broadinstitute/picard/releases/download/2.18.2/picard.jar
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   600    0   600    0     0   1176      0 --:--:-- --:--:-- --:--:--  1176
100 12.6M  100 12.6M    0     0  3348k      0  0:00:03  0:00:03 --:--:-- 4341k
building '/nix/store/rfcrhjpaf9xm6asm4ah2pzrpb3rq53ps-picard-tools-2.18.2.drv'...
installing
/nix/store/s3zhnncwwkafy3zvpxr9gigs5nk24fbp-picard-tools-2.18.2

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: picard-tools
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-darwin (full log)
Attempted: picard-tools
Partial log (click to expand)
building '/nix/store/pa9wln3i0xd7d5am7jhsf95mb1h844j8-picard.jar.drv'...

trying https://github.com/broadinstitute/picard/releases/download/2.18.2/picard.jar
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   600    0   600    0     0   1036      0 --:--:-- --:--:-- --:--:--  1036
100 12.6M  100 12.6M    0     0  1239k      0  0:00:10  0:00:10 --:--:-- 2462k
building '/nix/store/lss49ld8bpfzcf61b42abmaryldacv9k-picard-tools-2.18.2.drv'...
installing
/nix/store/hm21vbrjlg197x0xis5zmjgqfjn9xqn5-picard-tools-2.18.2


ResponseLoggingFilter changes xml response

From<EMAIL_ADDRESS>on September 07, 2012 12:52:09
Version: 1.6.3-SNAPSHOT
Problem: RsponseLoggingFilter changes the response instead of just logging it What steps will reproduce the problem? 1. Activate ResponseLoggingFilter: RestAssured.filters(new ResponseLoggingFilter());
2. e.g. run a test which calls a server which returns xml
3. the xml is changed such that e.g. dates cannot be parsed correctly
Example:
Reponse from server is:

2012-12-31

With logging on the response will be

2012-12-31

If a date parser is used which is not lenient then the date cannot be parsed.
After investigating this specific example I found that the logger is using the XmlNodePrinter to generate the prettified xml output. This class has a flag 'preserveWhiteSpace'. If its turned on this specific problem for xml seems to be fixed because no newlines will be inserted.
In general I think that a logger should never change the actual response.
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=199

From<EMAIL_ADDRESS>on September 07, 2012 04:05:05
I tried this approach but now I'm getting a nullpointerexception in the findContentType method of the RestAssuredResponseImpl.
Looking at the log I see this (among things)
Content-Type=application/xml
Content-Length=2391
What I got from the server was
Content-Type: application/xml
Content-Length: 2391
So it looks like the response headers are also broken.

From<EMAIL_ADDRESS>on September 09, 2012 23:33:30
It's true that the logger, by default, changes the response because pretty-printing is enabled. If you log using:
expect().log().body(false) instead then pretty printing will not be used and the response should be left intact on logging. You can also configure this for each request by using "detailed logging configuration" (see https://code.google.com/p/rest-assured/wiki/Usage#Detailed_configuration and http://rest-assured.googlecode.com/svn/tags/1.6.2/apidocs/com/jayway/restassured/config/LogConfig.html ).

From<EMAIL_ADDRESS>on March 07, 2014 03:18:33
Status: WontFix


Update install instructions + say jupyter more + mention examples

Closes: https://github.com/QuantStack/ipycytoscape/issues/49
Changes:
1. Initial description
I added more mentions of jupyter and pointed out that the examples directory exists. (on github the examples link will open the appropriate folder in the github ui)
2. Install instructions
Previously the install instructions did not actually lead to local development install of the javascript aspects of the widget. I borrowed heavily from the install instructions in the typescript cookiecutter to rewrite them.
I also added a section on how to continuously compile typescript changes and to rebuild jupyter accordingly which I based on the instructions in the jupyterlab-git README which I have found very helpful (https://github.com/jupyterlab/jupyterlab-git#install-1).

That's great :)
I added one comment, you think it makes sense?
Thanks a lot @ianhi.

Will do. I'm less familiar with the classic notebook, is there an equivalent of jupyter lab --watch for notebook? If yes then I can add that as well.

Hey @ianhi, just checked, don't think there is. I think npm run watch is the way to go.
Thank you so much =)

Cool @ianhi.
Thanks for these changes :)


Multiple Issues within E2 Editor's Highlighting and Autocomplete

Describe the bug
Multiple major issues are occurring with the E2 editor, presumably as a result of #2237.

Autocomplete is no longer working, as no menu appears after typing say entity():s...
Syntax highlighting related to block comments appears to also no longer be working, particularly after editing the text within a block comment.

To Reproduce
Steps to reproduce the behavior:

Typing entity():s should result in autocomplete list
Block comments should function properly

There may be other unforeseen issues present from these changes as well.

Agreed. Also

d

@stepa2 could you investigate?
I'll revert after a week or two if it is still broken

https://github.com/wiremod/wire/issues/2354 needs to be solved too

With the severity of these issues, it may be prudent to revert now and move the development of these enhancements to a separate branch for through testing since they affect so many pieces of the E2 editor, and I don't think that's something that can be fully resolved quickly. I have reverted my prod server to 2f5cf685b1dce80cd6d692c7aaf4e91cbe0294d6 in the meantime.

@ghostserver10 I'd suggest reverting to commit  2f5cf685b1dce80cd6d692c7aaf4e91cbe0294d6 if it's urgent until the HEAD is fixed/reverted.

If it's getting this much attention then I'll revert it

strings are broken for me too, pls revert until this is resolved

But how? I downloaded wiremod through workshop. I'm not sure how to downgrade it

"Resolved" by revert


Change string to DU

When viewing this sample, I thought it might be nice to remove the "_" match on (old) line 176.  To do so, I thought it might be nicer to use a discriminated union instead of strings to indicate state.  Hence this PR -- I hope you like it :)  All comments very much welcome -- I am a distinct newbie.

@cgravill called this "moving from string typing to strong typing", which I found very amusing.

I've found a lot of incidental programmers much happier with F#/Fable (than JavaScript) once we started using more typing features. Combined with the excellent Elm(ish) approach it lets them concentrate on mapping domain types to a visualisation without worrying they're breaking the infrastructure programming.

Sorry, missed the notification! This is a good change, I think the string remained from the original React example we converted. Thank you!

Thanks!


Cannot analyze maven project?

I have a maven project with multiple modules, like the following structure:
MyProject
|- MyModule1
|- MyModule2
|- pom.xml
I ran "infer -- mvn package", but it just printing

TODO: print error message
There was nothing to analyze, exiting

could you please report your version of java/javac?
java -version
javac -version

When I run infer with mvn package I get the same error I reported here

I get the same error in the mvn project
ec2-54-67-1-102:biz monkey$ infer -- mvn install
TODO: print error message
There was nothing to analyze, exiting

and java info
java version "1.7.0_67"
Java(TM) SE Runtime Environment (build 1.7.0_67-b01)
Java HotSpot(TM) 64-Bit Server VM (build 24.65-b04, mixed mode)

@martinoluca This is my java and javac version:
java -version
java version "1.8.0_05"
Java(TM) SE Runtime Environment (build 1.8.0_05-b13)
Java HotSpot(TM) 64-Bit Server VM (build 25.5-b02, mixed mode)
javac -version
javac 1.8.0_05

@monkeytest15 is the install target also building your source code?  Alternatively, have you tried cleaning your build with mvn clean?

Closing as a probable duplicate of #30. A workaround for now is to install java and javac at version 1.7.


Redshift Connection Failed: exception name 'password' is not defined

Getting below issue for amazon-redshift-utils:
START RequestId: 5e7221ca-06a1-46e5-8ae6-16dfb87c928e Version: $LATEST
Redshift Connection Failed: exception name 'password' is not defined
END RequestId: 5e7221ca-06a1-46e5-8ae6-16dfb87c928e
REPORT RequestId: 5e7221ca-06a1-46e5-8ae6-16dfb87c928e	Duration: 1.23 ms	Billed Duration: 2 ms	Memory Size: 128 MB	Max Memory Used: 31 MB
Can anyone help?
Thanks,
Gordon

Can you please clarify which utility you are running?

Amazon Redshift Utilities

QMRNotificationUtility

This appears to be due to an incompatibility with the latest versions of pg8000, that has changed the structure of the connect() method. Will attempt to assign to someone who can address this.

The new syntax is: conn = pg8000.connect(database=database, user=user, password=password, host=host, port=port, ssl_context=ssl). However, the Ciphertext password doesn't work properly. I have to hard code password in Lambda function. Any idea?

Connection details updated to use redshift_connector and not pg8000.

@gyuan-cars  Hi Gordon, Did this fix resolve the issue ? I am using the latest master branch with this fix. I am seeing the same issue.

Opened another ticket for this one @timjell @IanMeyers
https://github.com/awslabs/amazon-redshift-utils/issues/623


ModuleNotFoundError: No module named 'mapilio_kit.components.edit_config'

Short Information

Used Toolkit version: 1.0.2
Operating System:  Windows

Problem Definition Briefly (might be explained in steps)

Does not work when running MapSyncer instead gives me this error message.

Expected Output

MapSyncer to run without errors.

Actual Output
*Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Program Files\Python312\Scripts\RunMapSyncer.exe_main.py", line 4, in
File "C:\Program Files\Python312\Lib\site-packages\MapSyncer\app.py", line 11, in
from mapilio_kit.components.edit_config import edit_config
ModuleNotFoundError: No module named 'mapilio_kit.components.edit_config'
Extra information (can contain screen shots)

Hi @brmonaghan, due to the moduler changes on mapilio-kit-3.0.0 you're facing with this issue. You may need to update your MapSyncer version to the latest one by using pip install mapsyncer --upgrade,
Thanks for your feedback,
Best Regards 🎉


Update the title of the icon when changing the role of a team member

Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

LGTM


Test python 3.11 in CI

Related to:
https://github.com/juanep97/iop4/pull/127#issuecomment-2286868583
https://github.com/juanep97/iop4/pull/127#issuecomment-2315626056

Error in https://github.com/juanep97/iop4/actions/runs/10599496932/job/29374672641?pr=130#step:13:61, related to https://github.com/juanep97/iop4/pull/127#issuecomment-2315626056.

Docs building correctly now.


[DRAFT] dsse type

Summary

Adds a DSSE type that validated each signature on the envelope.  If the payload is an in-toto statement all in-toto subjects will be indexed.  The hash of the payload, PAE encoded payload, and public keys are also indexed.
I've tested this locally and it works but I need to write tests and add it to the e2e test script yet.  I'm opening this draft for discussion and input.
Ticket Link

Fixes #582
Release Note

Thanks for the PR and much needed cleanup!

@mikhailswift I just ran into this issue recently. Do you need any help pushing this over the finish line?

Codecov Report

Merging #596 (2b52d1e) into main (547eb3c) will increase coverage by 0.00%.
The diff coverage is 37.03%.

@@           Coverage Diff           @@
##             main     #596   +/-   ##
=======================================
  Coverage   46.56%   46.57%
=======================================
  Files          60       61    +1
  Lines        5094     5121   +27
=======================================
+ Hits         2372     2385   +13
- Misses       2447     2460   +13
- Partials      275      276    +1

Impacted Files
Coverage Δ

cmd/rekor-cli/app/root.go
20.68% <ø> (ø)

pkg/types/entries.go
6.25% <ø> (ø)

pkg/types/dsse/dsse.go
37.03% <37.03%> (ø)

pkg/types/alpine/v0.0.1/entry.go
56.25% <0.00%> (+1.25%)
:arrow_up:

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 547eb3c...2b52d1e. Read the comment docs.

This can get closed out now!

Sorry to comment on a closed PR but it seems like this was closed but never merged in to main. Was that the intention here?


Major UI Revamp

PR Fixes:

1 Improved UI
2 Optimised codebase by removing dead code
3 fixed routing issues

Resolves - a lot of issues
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

@devsargam

Checking this

@keshav-exe

This package is missing from ur package.json which is requited
pnpn i @radix-ui/react-separator

@devsargam done

@devsargam fixed all the comments

Thank u sir @keshav-exe

@devsargam Yo, I forked the repo I got Revamped UI, If I need to add some changes I need to add on that?
and there are many new issues, in that revamped do I need to make a issue and then PR for those?

Yes can add issues & pr's @MeerUzairWasHere

Of old UI or new revamp, Like I have project locally, with revamped there are many issues. Can I Make PRs for those ?


[stdlib] Failing string ordering on linux test

This testcase illustrates a problem on linux: the strings "a" and "a\0" satisfy
"a" <= "a\0"
"a" >= "a\0"
"a" != "a\0"

The testcase should succeed on OS X (@Lvt send me the OS X output of the compares - it's basically "a" < "a\0", which is reasonable).
Here is a sample output of the failure on linux (swift trunk with this pull request, ubuntu 15.10):
Command Output (stdout):
--
[ RUN      ] StringOrderRelation.StringOrderRelation/ASCII/NullByte
out>>> check failed at /home/rtreffer/work/swift/test/1_stdlib/StringOrderRelation.swift, line 19
out>>> expected: true
out>>> check failed at /home/rtreffer/work/swift/test/1_stdlib/StringOrderRelation.swift, line 22
out>>> expected: false
[     FAIL ] StringOrderRelation.StringOrderRelation/ASCII/NullByte
StringOrderRelation: Some tests failed, aborting
UXPASS: []
FAIL: ["StringOrderRelation/ASCII/NullByte"]
SKIP: []

SR-530 reports another inconsistent ordering on linux but is related to case sensitive vs. insensitive ordering. This might be another nice addition for an ordering testcase.

Can you please XFAIL this test for Linux, and raise a bug at bugs.swift.org?

Added .xfail. I'd expect it to fail on FreeBSD as well as it might be related to https://github.com/apple/swift/pull/1119
And it's SR-630

Thank you!

I'm starting to think we want .xfail(.NonApple(reason: ...)).

@jrose-apple  Here is the ```#if```` highscore throughout swift:
$ grep -ir '^#if' |sed 's/^[^:]*://'|sort|uniq -c|sort -rn|head -n12
    338 #if _runtime(_ObjC)
    128 #if SWIFT_OBJC_INTEROP
    119 #ifndef NDEBUG
     42 #if BEFORE
     38 #if arch(i386) || arch(arm)
     25 #if arch(i386) || arch(x86_64)
     19 #if os(OSX)
     14 #if os(iOS)
     13 #if FOO
     13 #ifdef __cplusplus
     13 #if AFTER
     12 #if os(OSX) || os(iOS) || os(watchOS) || os(tvOS)

Looks like the biggest chunk is "do we have an ObjC runtime or do we need a swift fallback?". It might thus be useful to classify failures due to non-objc systems :-)

Yes, please! Many of the failures I've seen when porting Swift to a new platform have been due to holes in xfails: #1090, #1032, #1119.

Here's a .noObjC case for TestPredicate: https://github.com/apple/swift/pull/1130


Improve task formatting

Current dependencies on/for this PR:

main

PR #313   👈

This comment was auto-generated by Graphite.


[Ollama AI] text selected not found

Extension
https://www.raycast.com/massimiliano_pasquini/raycast-ollama
Description
I try to use ollama with raycast and make custom commands but when I select a text and execute the command, there is an message : text selected not found
Steps To Reproduce
.
Current Behaviour
No response
Expected Behaviour
No response

Hi @D0gZys. have you give access to clipboard anche select text to raycast? #11568. Do you encounter this problem only on custom commands or on predefined commands as well?

I have the same issue.
Not working on predefined commands like translation.
I'm selecting text like I do normaly then i launch raycast and the predefined command: No selected Found text.
It works only if 'im in the Raycast Floating Note but anywhere else like a browser or a text editor :/
Clipboard option work.

Ensure you have given raycast "Accessibility" access, look at #11568.

@raycastbot close this issue


ZXEncodeHints`s property margin do not work

i set margin property in order to remove the white edge, but it not work

Can you give some more information on that, maybe example code?

It should work, but you need to use it as following:
ZXEncodeHints *hints = [ZXEncodeHints hints];
hints.margin = [NSNumber numberWithInt:0];


Fixing some URLs for hosting structurizr-lite behind a reverse proxy

You're likely going to run into other problems running Structurizr Lite behind a reverse-proxy because cdnUrl is used in a number of other places. Did you try the steps outlined in https://structurizr.com/share/76352/documentation#troubleshooting first?

Yes.
There are other problems, especially running it under some folder like this:
server {
    listen       8888;
    server_name  someservername;
    location /somedirectory {
		proxy_pass         http://structurizr-lite:8080;
    }
}

Becase then I ran in to some errors with the authorization token, and some other links like Diagrams.
So this was just fixing some easy ones, which probably usefull in the main branch as well.
The others I'm thinking how to solve without causing some troubles in other environments.

It's not designed to run underneath a folder I'm afraid, only at /.

I've tried to make it run, but it leads to some other repositories like UI and which using UI.
In this repo the back-end runs quite ok if I set the contextPath for the application something like this:
		Configuration.getInstance().setDataDirectory(structurizrDataDirectory);

		System.setProperty("server.servlet.context-path", Configuration.getInstance().getContextPath());

		SpringApplication.run(StructurizrLite.class, args);
		start();

It needs an extra setting in structurizr.properties like: structurizr.contextPath=/somedirectory.
But then embedding not working, so in the UI should change in /src/js/structurizr-content.js like this:
structurizr.ui.ContentRenderer = function(workspace, host, urlPrefix, urlSuffix, safeMode, contextPath) {

...

            } else {
                // embed the specified diagram
                var id = 'structurizrEmbed' + (++embedIndex);
                var embedUrl = contextPath + '/embed' + '?workspace=' + workspace.id + '&view=' + encodeURIComponent(diagramIdentifier) + '&perspective=' + encodeURIComponent(perspective) + '&type=' + type + '&iframe=' + id + '&urlPrefix=' + urlPrefix + '&urlSuffix=' + encodeURIComponent(urlSuffix);

                return '
' + '
';
            }

But this is a breaking change, so all others should be updated. ... and then coming the fixes for all the links...
So I understand if this is not a priority for this app at all.

So I understand if this is not a priority for this app at all.

It's not I'm afraid, sorry!


bugfix: support import of Candid composite queries

Fixes bug reported by @Dfinity-skaestle.

Thanks @crusso !


Hiding very long type annotations

I find that sometimes types generated from macros and similar get too long for the type annotations. For example:

Rather than help they make the code less readable and since they are so long they are not immediately useful as a short type that is very easily readable would be.
Perhaps there should be a limit of characters to show. Since you can always see the type by hovering the mouse over the declaration I think it would help uncluttering the view in certain types of code.
If a certain type is longer than a number of characters it would be completely hidden or perhaps abbreviated to that number of characters plus ...

There already is such a feature, the config is rust-analyzer.inlayHints.maxLength. The default value for this is unlimited(null). This config causes type hints to get truncated after reaching the number of characters by replacing any other nested hints with ...

We also have https://github.com/rust-analyzer/rust-analyzer/issues/3138 and https://github.com/rust-analyzer/rust-analyzer/issues/4151 that cover the same issue, I think we can close this one.

Thanks!


Prepare 0.3

To merge just before releasing once #8 is merged :-) Cheers!

Royal, thanks! :)

https://pypi.org/project/biscuits/0.3.0/

Awesome, that was fast :-)


Optimize mutateOnInput

In editor: when user manipulates a controller, don't send the complete mutator to the mutable, but only the changes. Otherwise, meshes maybe completely regenerated etc.

So far, I've implemented a special fix for the meshes, but the general solution described above is still desirable...

Done by temporarily degelating getMutator to getMutatorFromPath to retrieve only the changes


Package request: PACKAGENAME

Project description

Metadata

homepage URL:
source URL:
license: mit, bsd, gpl2+ , ...
platforms: unix, linux, darwin, ...

Add a :+1: reaction to issues you find important.

No package information given.


Uncaught Exception: The Internet connection appears to be offline.: (null)

An uncaught exception was reported.
Info:
OS: darwin
Arch: x64
GPMDP Version: 3.0.0
Time: Sun Mar 06 2016 11:37:29 GMT-0500 (EST)
Error:
The Internet connection appears to be offline.: (null)
Error: The Internet connection appears to be offline.: (null)
    at Error (native)

This issue was created automatically inside the uncaughtException handler

Local PC network issue


change mode until the next system change

Actually as more of us have "automatic" system setup in appearance I would like a behavior of Nightfall to force the change but only till the next system change.

Is this not working currently?
I have that setting on and Nightfall is working as intended for me, personally.


[Snyk] Upgrade koa from 2.14.1 to 2.14.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade koa from 2.14.1 to 2.14.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 9 months ago, on 2023-04-12.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5329fab) 100.00% compared to head (26e60b5) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##            master       #22   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          460       460
  Branches       121       121
=========================================
  Hits           460       460

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Implement initial support for COPY ... TO ...  statement

Which issue does this PR close?
Closes https://github.com/apache/arrow-datafusion/issues/5654
Closes https://github.com/apache/arrow-datafusion/issues/5988
Rationale for this change

I would like to to make it easier to test out datafusion (both for new comers as well as when I am trying to benchmark all the things)

What changes are included in this PR?
(I'll try and break this up into smaller pieces for easier review but I want to show it all working together)

[x] Initial parser support for COPY .. TO ... statements
[x] New LogicalPlan::CopyTo variant
[x] Planing / Runtime support for copying to Parquet files
-[ ] Parser tests
-[ ] Add end user  documentation
-[ ] Properly support writing single parquet files
-[ ] sqllogictests

I also plan to file follow on tasks (like support for other file formats, options, etc)
Are these changes tested?
Yes
Are there any user-facing changes?
Yes there is a new (documented) statement

Thank you for the heads up - I will study the PR you mention.
On Wed, May 10, 2023 at 5:02 PM Mehmet Ozan Kabak @.***>
wrote:

@.**** commented on this pull request.
In datafusion/core/src/execution/context.rs
https://github.com/apache/arrow-datafusion/pull/6313#discussion_r1190432381
:

@@ -450,6 +457,36 @@ impl SessionContext {
self.read_batch(record_batch)
}

// Execute a COPY TO statement, returning the number of rows

I think we have something you may be able to leverage, check this PR
https://github.com/synnada-ai/arrow-datafusion/pull/89 out. It extends
the ExecutionPlan approach to writing files, I think you can leverage that
work here too.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/arrow-datafusion/pull/6313#discussion_r1190432381,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADXZMJBIT634DSGELJU3XLXFQF7FANCNFSM6AAAAAAX37ZGKA
.
You are receiving this because you authored the thread.Message ID:
@.***>

i have enough feedback for now and I working to add this functionality in pieces, so no need for this PR now


Change all occurrences of "coffee-script" with "coffeescript"

Prepares for future versions of coffeescript, e.g. 2.x

From Coffeescript ^12.6, coffee-script is also being published as "coffeescript" on npm.
(https://github.com/jashkenas/coffeescript/pull/4548/commits/8ec10fa8ce9138db220844af31d9826660660513)
They published all versions back to 0.7.0 under "coffeescript"
npm view coffeescript
Coffeescript 2.x and future versions are/will only be published under "coffeescript" and "coffee-script" will stay with version 1.x forever.

@yyx990803 please, commit this change =) I have one project with browserify + vueify and i can't install last coffee with fat arrow functions support


Error Starting the Kernel

<!---   *** BEFORE SUBMITTING: PASTE CLIPBOARD HERE TO COMPLETE YOUR REPORT ***   ---!>
On startup, IPython Console shows this error message - "chap" is not recognised as an internal or external command, operable program or batch file.

Hey @AGoh2019, thanks for reporting. Are you using Mamba or micromamba 2.0+ in your base environment?

I can't tell which one I am using?
--- Andrew Goh
On Monday, 7 October 2024 at 02:27:24 am SGT, Carlos Cordoba<EMAIL_ADDRESS>wrote:

Hey @AGoh2019, thanks for reporting. Are you using Mamba or micromamba 2.0+ in your base environment?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

You need to open the Miniforge Prompt (I guess that's what you're using; if not, then the equivalent to it) and then run
mamba --version

or
micromamba --version

if that fails.

Closing due to lack of response.


Not working in vue ?

This package don't working in vue js? Why don't work this package ?
My code in script:
const WorldCities = require('worldcities');
const city = WorldCities.getByName('Tokyo');
console.log(city)
Error:

I'm not sure. I've never used Vue. I think it's trying to load the underlying data from the filesystem, and, I'm guessing Vue doesn't have access?

I'm not sure. I've never used Vue. I think it's trying to load the underlying data from the filesystem, and, I'm guessing Vue doesn't have access?

Vue have access but I don't know why don't work? But vue script almost with same js script.


Must predict lakes simultaneously!

 Now that lakes include other lakes the function to do forecasting has to be changed so that all models are then fit, and then the model predictions are done simultaneously so that lags of other lakes can be adjusted correctly!

-This has been implemented for the CNNLSTM. This chunk of code should be easy to copy and paste for the other RNNs.
-Just need to adapt it now for the GAM prediction function

Implemented and tested now for GAM predictions


[PE-D][Tester A] Adding duplicate vendors/events with extra whitespaces between names is allowed

The app allows users to add vendors/events with the same information, but with extra whitespaces between names.
Example:

create v/ n/Hong Lim Trading Pte. Ltd. p/67412943 d/Specialises in lighting effects. t/stage-crew
create v/ n/Hong␣␣␣␣␣Lim Trading Pte. Ltd. p/67412943 d/Specialises in lighting effects. t/stage-crew

Note: ␣ denotes whitespace
Expected output:
This vendor already exists in EventTory.
Actual output:
New vendor added: Hong␣␣␣␣␣Lim Trading Pte. Ltd.; Phone: 67412943; Description: Specialises in lighting effects.; Tags: [stage-crew]
If this was the intended behavior, then the user guide should have specified the constraint. See the course website for more info.

Labels: type.FeatureFlaw severity.Low
original: Gra7ityIC3/ped#4

to be addressed by #318


Use more concise EqualityMembers

Summary
I've refactored the equality members of value objects.
The tests seem to still pass.
PS: Having mistakenly deleted my copy of the repository, I closed the previous pull request attempt.
What changes did I make?

Use readonly fields in ValueObjects (932457b)

We can store the value of a ValueObject in read-only mode.

Fix possibly impure struct method called on readonly variable (4a24511)

Use more concise EqualityMembers (7fd2d29)

        public bool Equals(SSN other)
            => _text == other._text;

        public override bool Equals(object obj)
            => obj is SSN other && Equals(other);

        public override int GetHashCode()
            => _text != null ? _text.GetHashCode() : 0;

The Value Objects are more readable now. Thank you @phmatray 🍺