Module is imported partially

Describe the Bug
There is a module called PSLiteDB . Import-Module command imports module to Pode server. But some functions of module are absent.
Steps To Reproduce
Steps to reproduce the behavior:

Install-Module PSLiteDB
Start pode server as per script below

[cmdletbinding()]
param(
   [int] $port = 8090,
   [switch] $debugging
)
$PodeServerParams = @{"Quiet" = !$debugging; "DisableTermination" = !$debugging }

Import-Module Pode -Force #Does not work here

Start-PodeServer @PodeServerParams {
   Add-PodeEndpoint -Address localhost -Port $port -Protocol Http
   #Import-PodeModule -Name "PSLiteDB" #Does not work here either

Add-PodeRoute -Method Get -Path '/awake' -ScriptBlock {
   #Import-Module PSLiteDB -Force #Works here!!!
   Get-Module PSLiteDB | out-string | Write-PodeTextResponse -StatusCode 200
   Get-Command -Module PSLiteDB | out-string | Write-PodeTextResponse -StatusCode 200
}
}

Open the route in browser. Check that ConvertTo-LiteDbBSON function is not available in the workspace of the route. It will be missing in the response.

Same happens if I put Import-Module inside the Start-PodeServer's scriptblock or if I use Import-PodeModule.
But it works if I Import-Module right inside the Add-PodeRoute's scriptblock.

Expected Behavior
Function should be available if module imported before Start-PodeServer's scriptblock.
Platform

OS: Windows Server 2019
Browser: any
Versions:

Pode: 2.5.1
PowerShell: 7.2

Additional info
Module is installed for all users in powershell 7.

Also noticed that on the first screenshot (when module is imported globally) it shows version <IP_ADDRESS>. I don't have such version installed.

Odd! I just installed PSLiteDB (never had it before) - globally, so into C:\Program Files\PowerShell\Modules and it works for me without needing an explicit import:

Code:
[cmdletbinding()]
param(
[int] $port = 8090,
[switch] $debugging
)
$PodeServerParams = @{"Quiet" = !$debugging; "DisableTermination" = !$debugging }

Start-PodeServer @PodeServerParams {
	Add-PodeEndpoint -Address localhost -Port $port -Protocol Http

	Add-PodeRoute -Method Get -Path '/awake' -ScriptBlock {
		Get-Module -Name PSLiteDB | out-string | Write-PodeTextResponse -StatusCode 200
		Get-Command -Module PSLiteDB | out-string | Write-PodeTextResponse -StatusCode 200
	}
}

Using Pode version 2.5.1.

Strange. Just created fresh server 2019. Installed powershell 7 and both Pode and PSLiteDB modules. If I open terminal window I can immediately run Pode commands with autocomplete (means module is automatically loaded), but PSLiteDB requires explicit Import-Module to run and to autocomplete. Same goes on my Windows 10 laptop.
Is it possible that some kind of policy blocks it from autoloading?

My test was on my Windows 10 desktop. Repeating today from Terminal (and PS 7.2) as you can see autocomplete works, even before importing the module.

Historically autocomplete used to be managed by a CommandAnalysis cache populated on PS startup, but this has changed to ModuleAnalysisCache.
If you clear all ModuleCacheAnalysis files from $env:LOCALAPPDATA\Microsoft\Windows\PowerShell and restart PowerShell, does this help at all? Note: I did this and had to wait around 15 seconds for the cache file to reappear. It also took a few seconds more for the autocomplete to work.
My understanding is that you can also force a refresh of the cache with Get-Module -ListAvailable -Refresh.
Something else that springs to mind is that I noticed the module contained a number of DLL files. Could any of these be blocked on your systems? (right click > properties is a quick way to check this).

Hi @robinmalik, thanks for help. This didn't work, but I managed to find solution.
CmdletsToExport with explicit names ('*' won't do) should be used to allow module to be imported automatically. I'll report this to original repository. Have no issue with Pode then!

Hi @MaxFalcone,
You've actually stubbled over the same bug someone on Discord found, but with the MicrosoftTeams module :)
There is actually a bug in Pode that causes this to happen; for Teams it was a case that letting PowerShell auto-import the module worked as the CmdletsToExport was set properly. While in PSLiteDB, you're right that * actually does nothing 😂
The bug is in Pode's auto-import logic, to load imported modules into its runspaces. It goes off the module's Path, which for Teams was the .psm1 and for PSLiteDB it appears to be a .dll. Since the .psd1 isn't being imported, a lot of extras are missed - such as the entire psm1 for PSLiteDB! 😳
I have a fix for it, but didn't have chance to write it up with it being holidays; but it's to change the auto-import logic to look for <module_name>.psd1 otherwise fallback.
I'll re-open this issue, since it's the same, and contains another module that highlights the problem :)

I think I will follow this bug since I'm the one with trouble in microsoftteams :) The workaround of not loading the module dunamically work but I ran on another problem. I can't load the module and run it in more than 4 or 5 scriptblock. After that, MS consider that I reach the concurent connetion limit... Using diconnect-microsoftteams dosen't change anything... For now, I think I'm going to create two or three users to connect...

Hi @MaxFalcone and @Veers01,
I've pushed a fix in the above commit for importing modules, I've testing with both and for me they now both seem to be importing properly. Would either of you be able to test the changes locally and verify?
@Veers01, I had a quick look on the MicrosoftTeams connection limit, and looks like it suffers from similar Session issues to AWS (though AWS did it a little more sensibly). It seems the MT module creates a new session every time, and disconnecting then reconnecting doesn't properly close down and re-use existing connections, and you have to manually call Remove-PSSession. I've thought about seeing if it's possible to build to session re-use logic in Pode, make it simpler, but will need to looking into.
For AWS, the fix was to create a session outside of Pode, and Import-PSSession (I think it was Import) into each of Pode's runspace using State/Middleware and then check the PSSession state. If the session is closed, Remove it and Connect again - but you have to do all of this with locking, etc. so 2 runspaces don't try to create the same session/use it while closed. It's quite painful! 🙈

I can help test, if you're able to specify where we should be putting our Connect-MicrosoftTeams related code?
(As an aside, I think you've another issue open re: ExchangeOnlineManagement and sessions, which I did test and it failed ambiguously after about 5-6 hours but I had logging off so didn't collect any useful information - will repeat it soon).

In the example @Veers01 sent me, there was a Connect-MicrosoftTeams within the Start-PodeServer itself and then another Connect-MicrosoftTeams call within a route, with the Import-Module MicrosoftTeams outside of Start-PodeServer. The server level call worked, but the route one failed. (it was actually in a Pode.Web page, but a route did the same).
Ah yes! There was that issue with sessions as well. It's a tricky one with sessions because they limit you to one that you have to share across all runspaces 😕 getting the thread-safety down to a T is quite awkward, which is why I'm wondering there's anything I could write to make it much easier.

I can test it but I'm not sure how to use the dev version of pod. Sorry, I'm not very used to github and built tools...

Ok, I manage to have it working (just replace the 3 modified files on my test computer) and it seem to work. I still have a error when I use connect-microsoftteams inside the route:
Level: Error
ThreadId: 1
Server: DESKTOP-OUPM2GU
Category: InvalidOperation: (System.Collections.…oint,System.String]:Dictionary`2) [ConvertTo-Json], InvalidOperationException
Message: The type 'System.Collections.Generic.Dictionary`2[[Microsoft.TeamsCmdlets.Powershell.Connect.Common.Endpoint, Microsoft.TeamsCmdlets.PowerShell.Connect, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null],[System.String, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]' is not supported for serialization or deserialization of a dictionary. Keys must be strings.
StackTrace: at Write-PodeJsonResponse, C:\Users\BD\Documents\PowerShell\Modules\Pode\2.5.1\Public\Responses.ps1: line 717
at <ScriptBlock>, <No file>: line 11
at Invoke-PodeScriptBlock, C:\Users\BD\Documents\PowerShell\Modules\Pode\2.5.1\Public\Utilities.ps1: line 697
at <ScriptBlock>, <No file>: line 114

Date: 2022-01-02 22:28:08
Level: Error
ThreadId: 1
Server: DESKTOP-OUPM2GU
Category: System.Management.Automation
Message: The type 'System.Collections.Generic.Dictionary`2[[Microsoft.TeamsCmdlets.Powershell.Connect.Common.Endpoint, Microsoft.TeamsCmdlets.PowerShell.Connect, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null],[System.String, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]' is not supported for serialization or deserialization of a dictionary. Keys must be strings.
StackTrace:    at System.Management.Automation.MshCommandRuntime.ThrowTerminatingError(ErrorRecord errorRecord)```

For the max session and the problem with deconnection, I check if I have any session open with get-pssession when I use connect-microsoftteams and I don't. I'm not sure that the new version of connect-microsoftteams (vs the skypeforbusiness module) still use a pssession or a web API in the background.

I still hope that Microsoft release more functions in the MS Graph API for Teams (There is almost noting for the voice part) because it so much easier to use and way more fast that the PSmodule.

I copied @Veers01's approach here as I was unable to get Invoke-Build to work, but with the latest updates this works for me. Full code for ease of following (note that Connect-LUMicrosoftTeams is just a custom wrapper function that ultimately calls Connect-MicrosoftTeams - we have a number of these for different MS services):
server.ps1:
Import-Module MicrosoftTeams

Start-PodeServer {

    New-PodeLoggingMethod -Terminal | Enable-PodeErrorLogging -Level Error, Debug, Verbose

    Set-PodeViewEngine -Type Pode

    Enable-PodeSessionMiddleware -Duration 1200 -Extend

    ########################################################################################
    #Region Server Level Thread Safe 'state' objects
    Lock-PodeObject -ScriptBlock {
        Set-PodeState -Name 'ApplicationRoot' -Value @{ 'ApplicationRoot' = $PSScriptRoot } | Out-Null
    }
    #EndRegion Server Level Thread Safe Objects

    Connect-LUMicrosoftTeams -Tenancy abc123 -Verbose

    ########################################################################################
    #Region Additional state objects
    #EndRegion Additional state objects

    ########################################################################################
    #Region Listeners
    Add-PodeEndpoint -Address (Get-PodeConfig).Url -Port $(Get-PodeConfig).Port -Protocol Http
    #EndRegion Listeners

    ########################################################################################
    #Region Schedules
    #EndRegion Schedules

    ########################################################################################
    #Region Routes
    Use-PodeRoutes -Path "routes"

} -Threads 5

routes/team.ps1:
Add-PodeRoute -Method Get -Path '/team/:id' -ScriptBlock {
	Import-Module MicrosoftTeams
	$Value = $WebEvent.Parameters['id']
	$Data = Get-Team -MailNickName $Value | Where-Object { $_.MailNickname -eq $Value }
	Write-PodeJsonResponse -Value $Data
}

Yesterday this didn't work, so the changes made in #892 seem to have resolved things for me 👊🏻 The only downside is that version 3.0.0 of MicrosoftTeams is so incredibly slow to connect now and so the server takes an age to start now 😁 (and earlier version will be deprecated at some point!).

A few observations/questions:

The server is now incredibly slow to start. I've noticed MicrosoftTeams version 3.0.0 is much slower to initiate but it doesn't appear to be with the connection part. The wrapper function outputs:
VERBOSE: 03/01/2022 11:16:28: Selected account: o365ga1-svc
VERBOSE: 03/01/2022 11:16:28: Connecting.
VERBOSE: 03/01/2022 11:16:31: Connected.

... and then it's a fair while longer until the Listening on the following 1 endpoint(s) [5 thread(s)]: appears.

Is there any way we can call Disconnect-MicrosoftTeams when the server terminates? If I terminate the server with CTRL+C and run a Get-Team, it's still connected. I haven't seen any detrimental impact at the moment when running server.ps1 again, but generally it'd be good to clean up if we can.

A few observations/questions:

The server is now incredibly slow to start. I've noticed MicrosoftTeams version 3.0.0 is much slower to initiate but it doesn't appear to be with the connection part. The wrapper function outputs:
VERBOSE: 03/01/2022 11:16:28: Selected account: m365ad2-svc
VERBOSE: 03/01/2022 11:16:28: Connecting.
VERBOSE: 03/01/2022 11:16:31: Connected.

... and then it's a fair while longer until the Listening on the following 1 endpoint(s) [5 thread(s)]: appears. I don't think we can do much about this though. Using the Graph API directly where possible is much faster but for some things, the Teams module is needed.

Is there any way we can call Disconnect-MicrosoftTeams when the server terminates? If I terminate the server with CTRL+C and run a Get-Team, it's still connected. I haven't seen any detrimental impact at the moment when running server.ps1 again, but generally it'd be good to clean up if we can.

@robinmalik: Yes,MS Teams module is verrrryyyy slow to connect, This have noting to do with pode, I'it slow in the command line too. I have two tenant and I have the same excruciating slow speed to connect.
You can call disconnect, but like @Badgerati said, this is not working as intented.
For my part, I get around by making another small pode script who act as a REST API. The script connect to teams with connect-microsoftteams and listen to a localhost socket for a url (like localhost:8080/users/ID). A each request, I do a small "get-tenant" to check if the connection is still open. On error, I reconnect teams. This is the only way I can have a reasonable response time, at the cost of more data transfomation).
Maybe there is a way to do that with a runspace or other technique, but this is not something that I'd work with so that could take some time :)

@Veers01 Multiple tenancies too and we have the same. Pretty sure it's started happening since v2.5+ but given v3 will be the only supported path going forwards there's little we can do but work with/around it.
Interesting approach re: separate Pode server for the Teams bit. There's some code in #841 regarding Exchange sessions and reconnection that might help integrate the Teams code into your "main" Pode server. So far I've not done anything to deal with Teams disconnections... nor have I hit any kind of maximum connection limits for our tenancy, but given the Connect-MicrosoftTeams is within Start-PodeServer, I assume it's just connecting once anyway.

@Veers01, I was using v3.0.0 in PS7.2.1, and when I did a Get-PSSession I was getting sessions back for MicrosoftTeams 😮 It wasn't in 5.1, which is weird, possibly 3.0.1 is different 🤔
With regards to the Write-PodeJsonResponse error, it looks like you're passing a Dictionary with non-string keys? ConvertTo-Json requires all keys in a Dictionary/Object/Hashtable to be strings :)
@robinmalik Ooh yeah, as you've both mentioned that MicrosoftTeams module is slooow. There's not much I can do about it directly, however! I've been looking into seeing if it's possible to start all runspaces async, rather than sync - so that should "speed" it up a bit. It does look like it's possible, just need to play/test with it.
And yep, there is! If you register an event for server "Terminate" or "Stop", then you should be able to run Disconnect-MicrosoftTeams  and/or Remove-PSSession there 😄

@Badgerati

@Veers01, I was using v3.0.0 in PS7.2.1, and when I did a Get-PSSession I was getting sessions back for MicrosoftTeams 😮 It wasn't in 5.1, which is weird, possibly 3.0.1 is different 🤔

Weird, I do have the same version:

ModuleType Version    PreRelease Name                                ExportedCommands
---------- -------    ---------- ----                                ----------------
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Management     {Add-Content, Clear-Content, Clear-Item, Clear-It…
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Ob…
Script     3.0.0                 microsoftteams                      {Add-TeamChannelUser, Add-TeamUser, Connect-Micro…
Script     2.1.0                 PSReadLine                          {Get-PSReadLineKeyHandler, Get-PSReadLineOption, …
Script     0.0                   SetMSTeamsReleaseEnvironment

PS C:\Users\benjamin.duval.BUREAU2> Get-PSSession
PS C:\Users\benjamin.duval.BUREAU2> connect-microsoftteams

Account                          Environment Tenant                               TenantId
-------                          ----------- ------                               --------
<EMAIL_ADDRESS>AzureCloud  xxxxxxxxxxx-xxxx-xxxx-xxxxxxxxxxxx xxxxxxxxxxx-xxxx-xxxx-xxxxxxxxxxxx

PS C:\Users\benjamin.duval.BUREAU2> Get-PSSession
PS C:\Users\benjamin.duval.BUREAU2>

With regards to the Write-PodeJsonResponse error, it looks like you're passing a Dictionary with non-string keys? ConvertTo-Json requires all keys in a Dictionary/Object/Hashtable to be strings :)

I dont parse anythig, this is the output of "connect-microsoftteams" :(

After some more testiing, I think that the teams session is still valid in the podewebpage. I change my test code for that:
Import-Module Pode
Import-Module Pode.Web
import-module microsoftteams

Start-PodeServer {

	Use-PodeWebTemplates -Title 'Example' -Theme Dark

	Add-PodeEndpoint -Address * -Port 80 -Protocol Http

    # Logging dans la console
    New-PodeLoggingMethod -Terminal | Enable-PodeErrorLogging

    $user =<EMAIL_ADDRESS>    $pass = ConvertTo-SecureString -String 'xxxxxxxxxxxxxxxxxxxxx' -AsPlainText -Force
    $cred = New-Object System.Management.Automation.PSCredential($user,$pass)

    #### this one is working #####
    Connect-MicrosoftTeams -Credential $cred
    $test = Get-CsOnlineUser -identity<EMAIL_ADDRESS>    write-host $test.SipAddress
    ##############################

    Add-PodeWebPage -Name "Liste" -Icon 'phone' -ScriptBlock {

        new-PodeWebContainer -Id 'searchlist' -NoBackground -Content @(
            New-PodeWebForm -Name 'Recherche' -ID 'recherchePS' -AsCard -ScriptBlock {

                $searchValue = $WebEvent.Data.Recherche

                if($searchValue -ne $null){
					get-module | out-default
					#export-podeModule MicrosoftTeams
                    #import-module microsoftteams
					#$user =<EMAIL_ADDRESS>                    #$pass = ConvertTo-SecureString -String 'PNFdP@0E%0N%iJnr#V8r!blSbjav!dEp2#4J^%bd1sysGBD' -AsPlainText -Force
                    #$cred = New-Object System.Management.Automation.PSCredential($user,$pass)

                    # This one was not working but work with change from 892 #####
                    #Connect-MicrosoftTeams -Credential $cred | out-null
                    $test = Get-CsOnlineUser -identity<EMAIL_ADDRESS>                    write-host $test.SipAddress
                    ###############################

                    ## code for updating the table
                }

            } -Content @(
                New-PodeWebTextbox -Name 'Recherche' -Type Text
            )

            New-PodeWebTable -Id 'listeposte' -DataColumn 'Nom' -Name 'Liste des postes' -AsCard -Paginate -PageSize 20 -ScriptBlock {

            } `
            -Columns @(
                Initialize-PodeWebTableColumn -Key "Nom" -Alignment Left
                Initialize-PodeWebTableColumn -Key "Téléphone" -Alignment Left
                Initialize-PodeWebTableColumn -Key "Extension" -Alignment Center
                Initialize-PodeWebTableColumn -Key "Carnet" -Alignment Left
                Initialize-PodeWebTableColumn -Key 'Plan de num.' -Alignment Center
                Initialize-PodeWebTableColumn -Key 'Actions' -Alignment Center
            )
        )
    }
}

And the get-csonlineuser inside the podewebpage work! I don't have to reconnect. I will make a more elaborate script with some connection check (for timeout) and get back to you.

@Veers01 Hey, just a quick reply to say I think there's a password and username in this above code about half way down and if so, you might want to delete the comment and re-reply! And change the password for good measure perhaps :) If I'm wrong, apologies.

@robinmalik : You are right, I forgot this one :) Thank you for report that! At least this is just a simple account on our test tenant!

I wonder if it creates a session elsewhere, like how az/kubectl do theirs. I just tried something as well, the session isn't created in the main terminal, but when Connect-MicrosoftTeams and Get-CsOnlineUser are called in the runspace/route that does create a session 🤔
I'm going to push the main fix for the module loading issue, and close this ticket. Probably worth moving the sessions talk to Discord 😄


low-frequency-mpi

Investigate the impact of lowering power limit inside of MPI regions which are expected to be long running.  For example lower power for MPI_Send if payload is larger than some minimum, but do not lower power for MPI_ISend or other non-blocking calls.  This could improve energy efficiency.

This is already a feature of the EE agent; still need this issue?

I think with the EE agent and Countdown this is good.


Broken/Missing Setup Doc Links

Setup Doc links are not available or there are broken links
Expected Behavior
User should be able to navigate to the doc links
Current Behavior
User is not able to navigate to the doc links due to broken links or unavailable links
Possible Solution
Please update the doc links
Steps to Reproduce
Navigate to

https://github.com/hyperledger/cello/blob/master/docs/tutorial.md
https://github.com/hyperledger/cello/blob/master/docs/setup/server.md#21-clone-source-code
https://github.com/hyperledger/cello/blob/master/docs/setup/server.md#22-prepare-environment
https://github.com/hyperledger/cello/blob/master/docs/setup/server.md#23-start-service

I would like to fix this issue. Can I go ahead?


kubelet: Add container readiness manager.

Move the readiness managing logic into pkg/kubelet/container package
to facilitate the pluggable container runtime.
This is similar to https://github.com/GoogleCloudPlatform/kubernetes/pull/5982
This PR will enable us to move killContainer into container runtime. Why moving it into container runtime? Because by that, the docker container runtime can take care of the pod infra container restart process. (Take a glance of the coming patch here):
https://github.com/yifan-gu/kubernetes/blob/run_container_in_pod/pkg/kubelet/dockertools/docker.go#L1074
I feels maybe we can put these two guys together(RefManager and ReadinessManager). But I can't think of a good name :(
Thank you in advance for reviewing!
@vmarmol @dchen1107

LGTM, thanks @yifan-gu! I think RefManager, ReadinessManager, and PodStatusManager may merge into one or two managers one day.


Collapsible columns or possible keyword hiding column causing issue

My application uses collapsible columns universally throughout the app and on this specific table that we're having an issue with, we are also using the keyword option _hc to hide a "Notes" column. This table by default has 2 filters set within the view settings. Please watch this video to see the issue.
https://www.loom.com/share/5530a70b084d40489c38ce0ee5839c57?sid=a0f285a9-f8b5-4a51-b7c9-62d7c748eaef

Hi DeeJay,
This is going to be a tough one, I think.
Can you give me access to that page please?  It would save me a lot of time replicating your setup.
Send the info at the same email as usual.
Norm


Spectrum 2

Spectrum 2 is a major update to Adobe's design system, currently in pre-release. Designed to support our growing suite of products, Spectrum 2 aims to work seamlessly across experiences by balancing personality and function.
Spectrum 2 in React Spectrum is built on React Aria Components and a new styling foundation powered by Spectrum Tokens and Parcel macros. This gives developers access to Spectrum design fundamentals such as colors, spacing, sizing, and typography in applications and custom components. Spectrum 2 also brings new features such as t-shirt sizing, improved form layout, dynamic new press interactions, and more.
Note: the PR will be merged rather than squashed to preserve history in the main branch.

Build successful! 🎉

View the storybook
View the storybook-19
View the storybook-17
View the storybook-16
View the S2 storybook
View the documentation

Spectrum 2 dependency doesn't looks like it is deployed on npm. Additionally version 1 starts with @adobe prefix while in the new documentation it doesn't.

Spectrum 2 is a major update to Adobe's design system, currently in pre-release. Designed to support our growing suite of products, Spectrum 2 aims to work seamlessly across experiences by balancing personality and function.
Spectrum 2 in React Spectrum is built on React Aria Components and a new styling foundation powered by Spectrum Tokens and Parcel macros. This gives developers access to Spectrum design fundamentals such as colors, spacing, sizing, and typography in applications and custom components. Spectrum 2 also brings new features such as t-shirt sizing, improved form layout, dynamic new press interactions, and more.
Note: the PR will be merged rather than squashed to preserve history in the main branch.

https://github.com/adobe/react-spectrum/pull/6777#issuecomment-2255786843

@tofiqquadri This will need to be merged, then published to npm before it is available to install via npm.

The S2 storybook link shows only a subset of the components and stories available in S1 storybook.  Is it correct to assume that the underlying components have already been migrated, but the storybook stories for all S2 components do not exist yet?

@cheema-corellian some components are still in development (as seen in open PRs) and some are awaiting finalized designs.

@cheema-corellian some components are still in development (as seen in open PRs) and some are awaiting finalized designs.

Thank you for clearing that up.

@cheema-corellian Not at the moment.


Allow BaseLocalCache to, optionally, only allocate storage for caching of references (PR 12034 follow-up)

Yet another instalment in the never-ending series of things that you think of after a patch has landed.
Since Functions are only cached by reference, we thus don't need to allocate storage for names in LocalFunctionCache instances. Obviously the effect of these changes are really tiny, but it seems reasonable in principle to avoid allocating data structures that are guaranteed to be unused.

/botio test

Thanks!


Image from phone in portrait appears incorrectly on PC during an AppRTC call

https://github.com/GoogleChrome/webrtc/issues/293
braveyao commented on Dec 10, 2014
Steps to reproduce:
On a PC(any latest chrome), open an apprtc.appspot.com room.
On the phone(N5 in test, any latest chrome) in portrait position, navigate to the same room.
Observed behavior:
The image from the phone will appear overly large on the PC, cutting off much of the picture, making it difficult to show all of a person's face when holding the device, and it's stretched with obvious aliasing.
On the phone side, same situation, no aliasing observed though.
Expected behavior:
Apprtc should display as much of the image coming from the device as possible.
And as discussed, it should be better to "when the aspect of the incoming video is substantially different than the display, we should not try to stretch - instead we should crop to square and letterbox".
braveyaobraveyao added the apprtc label on Dec 10, 2014
Justin Uberti
Owner
juberti commented on Dec 10, 2014
This means we want to remove the object-fit:cover CSS from remoteVideo when we detect that remoteVideo.width/height is substantially different than remoteVideo.videoWidth/Height.
Proposed metric for "substantially different": width/videoWidth > 16/9 or < 9/16

Closing due to age.


Added a note about ElevationOfPrivilege retirement in New-ActivityAlert.md

https://github.com/MicrosoftDocs/office-docs-powershell/issues/5941

Docs Build status updates of commit b3e7ab4:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 8f251a3:
:clock10: Full build: the average full build time is 17 min(s) 55 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 8f251a3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit b3e7ab4:
:clock10: Full build: the average full build time is 17 min(s) 55 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit b3e7ab4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@chrisda Please merge this PR. Thanks

Docs Build status updates of commit 92e5a5c:
:clock10: Full build: the average full build time is 17 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 92e5a5c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 6dc412c:
:clock10: Full build: the average full build time is 17 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 6dc412c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 5caed15:
:clock10: Full build: the average full build time is 17 min(s) 54 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 5caed15:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-ActivityAlert.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[Triple] More detailed errors if getting the triple from the compiler should fail

This fails spuriously on Windows. It would be nice to get some more detailed output when that happens.
CC @compnerd

See https://forums.swift.org/t/error-tscutility-triple-swift-fatal-error-could-not-determine-host-triple-malformed/47775

@swift-ci please test

Yeah the comments are mostly just to break up what would otherwise be an unreadable glob of do-catches. They could also be blank lines, I suppose.


How to get digits of PyLong/PyInt instance?

In Python C API we can simply access digits of int with ob_digit slot like
PyLongObject* value = ...;
Py_ssize_t digits_count = Py_ABS(Py_SIZE(value));
if (digits_count != 0) {
  digit highest_digit = value->ob_digit[digits_count - 1];
  ...
}

is there a way to do the same thing using given library?
I need this to add conversion between custom big int class and int one.
There is a workaround by converting to string and parsing it again, but copying digits should be easier & faster than roundtrip to string and back.

Can you use _PyLong_AsByteArray for this?
That is what our conversions for num_bigint use (if that is what you (can) use, you can simply use that conversion instead)
See https://github.com/PyO3/pyo3/blob/main/src/num_bigint.rs

It worked, thanks @mejrs!


ZWave automation has become slow

Home Assistant release (hass --version):
0.32.0.dev0
Python release (python3 --version):
3.5.2
Component/platform:
zwave, automations
Description of problem:
I have a couple of zwave double relay switches (qubino) and an older wall socket (fibaro).
Until earlier versions (I think 0.30) the automation described below in the configuration used to work instantly (<1s) but with the latest versions it takes arround 15s.
Expected:
Automation should work faster (<=1s)
Problem-relevant configuration.yaml entries and steps to reproduce:
- alias: "Bedroom lights off at night"
  #hide_entity: True
  trigger:
    platform: state
    entity_id: switch.qubino_zmnhbdx_flush_2_relays_switch_26_0
    to: 'off'
  condition:
    condition: and
    conditions:
      - condition: sun
        after: sunset
      - condition: state
        entity_id: switch.fibaro_system_fgwpe_wall_plug_switch_3_0
        state: 'off'
  action:
    - service: notify.webos_tv
      data:
        message: "Bedroom lights of, lamp on"
    - service: switch.turn_on
      entity_id: switch.fibaro_system_fgwpe_wall_plug_switch_3_0

Additional info:
I haven't checked yet but as you can see the automation triggers two services and I would like to add that the webos notification fails because webos can't be detected but I'll submit this bug as well.

Disable the notify, and test again. I don't think the following service will be executed before notify is done.

Well...thats still a bug, and here the async things @balloob I

Funny, changing the order doesn't even trigger the automation and in plus I've noticed another bug mention here (don't remember the number, but someone said that it adds an additional group tab in the UI); in my case the route itself to the group is not available anymore and I get a 404.
Very strange behaviour; in the logs I have this:
16-10-30 01:18:25 aiohttp.web: Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/aiohttp/server.py", line 261, in start
yield from self.handle_request(message, payload)
File "/usr/local/lib/python3.5/dist-packages/aiohttp/web.py", line 93, in handle_request
match_info.handler, type(resp), self._middlewares)
AssertionError: Handler <function HomeAssistantWSGI.register_static_path..serve_file at 0x7f15347a2d90> should return response instance, got <class 'NoneType'> [middlewares []]
16-10-30 01:18:26 aiohttp.web: Error handling request
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/aiohttp/server.py", line 261, in start
yield from self.handle_request(message, payload)
File "/usr/local/lib/python3.5/dist-packages/aiohttp/web.py", line 93, in handle_request
match_info.handler, type(resp), self._middlewares)
AssertionError: Handler <function HomeAssistantWSGI.register_static_path..serve_file at 0x7f153409ad08> should return response instance, got <class 'NoneType'> [middlewares []]

Is this issue still a problem?

Not to me

Alright then 😊


chore: add license check

Codecov Report

Merging #6 (8a708bb) into main (424e032) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   45.81%   45.81%
=======================================
  Files           7        7
  Lines         179      179
=======================================
  Hits           82       82
  Misses         91       91
  Partials        6        6

Impacted Files
Coverage Δ

internal/conf/conf.go
0.00% <ø> (ø)

internal/conf/nacos.go
76.92% <ø> (ø)

internal/core/comm/query.go
90.00% <ø> (ø)

internal/core/comm/watch.go
0.00% <ø> (ø)

internal/discoverer/discoverer.go
86.95% <ø> (ø)

internal/utils/msg.go
0.00% <ø> (ø)

internal/utils/validate.go
68.18% <ø> (ø)

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 424e032...8a708bb. Read the comment docs.


According to https://github.com/rrag/react-stockcharts/issues/310, I should change border with
.react-stockcharts-grabbing-cursor,
.react-stockcharts-crosshair-cursor

However, looking into html, a chart is created with "g" without a css class attached. I want to have a border-bottom at the end of each chart. So actually, I can use
<XAxis axisAt="bottom" orient="bottom" showTicks="false"/> to simulate a border at the bottom of a chart.
I want to use something simpler:
border-bottom: 1px solid black;
How can I do this?
The "Chart" has no property such as "borderStyle".

I’m having issues with displaying a border below the chart so that it also appears under Y Axis.
I’ve tried using StraightLine as you suggested and even made a tmp solution so that it accepts the width as a prop, but looks like it just stops rendering where Chart ends, while I need it to be drawn after it, below Y Axis
I wish I could render it inside ChartCanvas, not a Chart, but it seems to be impossible.
What I want (notice the gray lines between charts and below Y Axis-es):

Turned out it's fixable by passing clip={false} to GenericChartComponent rendered in StraightLine. Opened a pr for it https://github.com/rrag/react-stockcharts/pull/538

Ended up creating a custom component for that purpose:
import React, { Component } from 'react'
import PropTypes from 'prop-types'
import {
  hexToRGBA,
  strokeDashTypes,
  getStrokeDasharray,
} from 'react-stockcharts/lib/utils'
import GenericChartComponent from 'react-stockcharts/lib/GenericChartComponent'
import { getAxisCanvas } from 'react-stockcharts/lib/GenericComponent'

class ChartHorizontalLine extends Component {
  constructor(props) {
    super(props)
    this.renderSVG = this.renderSVG.bind(this)
    this.drawOnCanvas = this.drawOnCanvas.bind(this)
  }

  drawOnCanvas(ctx) {
    const {
      stroke,
      strokeWidth,
      opacity,
      strokeDasharray,
      width,
      y,
      x,
    } = this.props
    ctx.beginPath()

    ctx.strokeStyle = hexToRGBA(stroke, opacity)
    ctx.lineWidth = strokeWidth

    const y1 = y
    const y2 = y
    const x1 = x
    const x2 = x + width

    ctx.setLineDash(getStrokeDasharray(strokeDasharray).split(','))
    ctx.moveTo(x1, y1)
    ctx.lineTo(x2, y2)
    ctx.stroke()
  }

  renderSVG() {
    const {
      width,
      className,
      x,
      y,
      stroke,
      strokeWidth,
      opacity,
      strokeDasharray,
    } = this.props

    return (
      <line
        className={className}
        strokeDasharray={getStrokeDasharray(strokeDasharray)}
        stroke={stroke}
        strokeWidth={strokeWidth}
        strokeOpacity={opacity}
        y1={y}
        y2={y}
        x1={x}
        x2={x + width}
      />
    )
  }

  render() {
    const { clip } = this.props

    return (
      <GenericChartComponent
        svgDraw={this.renderSVG}
        clip={clip}
        canvasDraw={this.drawOnCanvas}
        canvasToDraw={getAxisCanvas}
        drawOn={['pan']}
      />
    )
  }
}

ChartHorizontalLine.propTypes = {
  className: PropTypes.string,
  /* eslint-disable react/boolean-prop-naming */
  clip: PropTypes.bool,
  stroke: PropTypes.string,
  strokeWidth: PropTypes.number,
  strokeDasharray: PropTypes.oneOf(strokeDashTypes),
  opacity: PropTypes.number,
  y: PropTypes.number.isRequired,
  x: PropTypes.number.isRequired,
  width: PropTypes.number.isRequired,
}

ChartHorizontalLine.defaultProps = {
  className: 'line ',
  clip: true,
  stroke: '#000000',
  opacity: 0.5,
  strokeWidth: 1,
  strokeDasharray: 'Solid',
}

export default ChartHorizontalLine

@rrag do you want me to open a pr with this? I can name it HorizontalLine.


TypeError: '<' not supported between instances of 'dict' and 'float'

Added print statement to see what is happening.. push_needed is called twice once each time with different time:
drive push checking for changes in 'reconstruction' .... sync_time<PHONE_NUMBER>.5466192 local_time<PHONE_NUMBER>.5294328 sync_time {'time':<PHONE_NUMBER>.9402184} local_time<PHONE_NUMBER>.205921 Traceback (most recent call last):
sync_time seems to be assigned a dictionary rather than a value in drive push.  drive status runs ok and reports files to push but drive push fails with the following error:
drive push checking for changes in 'reconstruction' .... Traceback (most recent call last):   File "/home/peter.iabra/.local/bin/drive", line 11, in <module>     load_entry_point('drive-cli', 'console_scripts', 'drive')()   File "/home/peter.iabra/.local/lib/python3.8/site-packages/Click-7.0-py3.8.egg/click/core.py", line 764, in __call__     return self.main(*args, **kwargs)   File "/home/peter.iabra/.local/lib/python3.8/site-packages/Click-7.0-py3.8.egg/click/core.py", line 717, in main     rv = self.invoke(ctx)   File "/home/peter.iabra/.local/lib/python3.8/site-packages/Click-7.0-py3.8.egg/click/core.py", line 1137, in invoke     return _process_result(sub_ctx.command.invoke(sub_ctx))   File "/home/peter.iabra/.local/lib/python3.8/site-packages/Click-7.0-py3.8.egg/click/core.py", line 956, in invoke     return ctx.invoke(self.callback, **ctx.params)   File "/home/peter.iabra/.local/lib/python3.8/site-packages/Click-7.0-py3.8.egg/click/core.py", line 555, in invoke     return callback(*args, **kwargs)   File "/home/peter.iabra/localbuilds/drive-cli/drive_cli/actions.py", line 311, in push     utils.push_content(cwd, fid)   File "/home/peter.iabra/localbuilds/drive-cli/drive_cli/utils.py", line 494, in push_content     if(push_needed(drive_lis[item], item_path)):   File "/home/peter.iabra/localbuilds/drive-cli/drive_cli/utils.py", line 173, in push_needed     if sync_time < local_time: TypeError: '<' not supported between instances of 'dict' and 'float'

fixed with the following patch
def push_needed(drive, item_path): ...     if isinstance(sync_time, dict):         sync_time = sync_time['time']     if sync_time < local_time:

This has led to some strange behaviour. I have a folder with several files A, B, C, D, which are all tracked. After I change one file, say C, and run drive push, the content of C is saved to file A, and the rest of the files remain unchanged on the remote. This is quite worrying, but it appears that this repo is no longer maintained. Some pull requests look quite promising (#95) and it would be a shame if this project was just abandoned. Is anyone willing to take over maintenance?


Simple Fix

Discord Bot didn't respond to the prefix so I decided to fix it, nothing really crazy.

Sounds Great, It would be awesome if you rewrote this. This was meant to be just a temporary fix and I'm happy you responded.


Raspi4b makefile nor found

Compiling from sources neiter
./bootstrap
./configure
make
sudo make install

nor
cd "${XRDP_SRC_DIR}"
./bootstrap
./configure --enable-fuse --enable-mp3lame --enable-pixman
make<br>

works as there is NO Makefile.
I only find
Makefile.am and
Makefile.in

I don't think it is hardware specific. It depends on your OS. What's your OS?


[flaky ci] [Serial]SRIOV VirtualMachineInstance with sriov plugin interface [test_id:1754]should create a virtual machine with sriov interface

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind enhancement

/triage build-watcher
What happened:
Test [Serial]SRIOV VirtualMachineInstance with sriov plugin interface [test_id:1754]should create a virtual machine with sriov interface flaked before merge of #4501
PR history: https://prow.apps.ovirt.org/pr-history/?org=kubevirt&repo=kubevirt&pr=4501
Offending run: https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4511/pull-kubevirt-e2e-kind-1.17-sriov/1325727568359329793
Error stacktrace:
tests/vmi_multus_test.go:697
Unexpected error:
    <*errors.StatusError | 0xc002db5f40>: {
        ErrStatus: {
            TypeMeta: {Kind: "", APIVersion: ""},
            ListMeta: {
                SelfLink: "",
                ResourceVersion: "",
                Continue: "",
                RemainingItemCount: nil,
            },
            Status: "Failure",
            Message: "etcdserver: request timed out",
            Reason: "",
            Details: nil,
            Code: 500,
        },
    }
    etcdserver: request timed out
occurred
GOROOT/src/reflect/value.go:460

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

Possibly related:

https://github.com/kubevirt/kubevirt/issues/3777
https://github.com/kubevirt/kubevirt/issues/3742

Suggestion (if the network team and reviewers would like)
Try empirically this one, as i saw several times that a freshly created cluster might suffer in the beginning from etcd timeouts,
even that cluster-up is "done"
https://github.com/kubevirt/kubevirtci/pull/473
More possible refs in the PR

Another run
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4501/pull-kubevirt-e2e-kind-1.17-sriov/1325742799731036160

https://github.com/kubernetes/kubernetes/issues/70082#issuecomment-433604939
looks interesting, checking the metrics of the disk IO
this issue is alive 7 days ago, and lots of people reported it
we do see the prints mentioned in the issue (will add link to the PR 473 thread with the prints once github will stabilize)

I can confirm we do get ..took to long to execute warnings all over etcd pods logs on every sriov lane job, here are few for example
Right after sriov-operator deployment
1326381124888301569 etcd logs
Right after kubevirt-operator deployment, when this error occurs at this time it will affect the first test
1326105060454699008 etcd logs
We followed this issue https://github.com/kubernetes-sigs/kind/issues/717, where its suggests to manipulate inotify watcher configurations in order to fix etcd timeout errors.
I verified and on all the CI nodes that schedule sriov lane jobs we configured:
fs.inotify.max_user_watches=524288
fs.inotify.max_user_instances=8192
vm.min_free_kbytes=3000000```
On those nodes we use SAS disks so it could be low disk performance,
personally I would expect etcd to work properly on hard drives as well.

/close
Should be fixed by #4588


Using older version of Angular

Very very nice sample. However, just be aware that this is using Angular 4. Some of the latest features such as Landing Pages do not seem to be avail in this sample. The latest version of the WCH Oslo Sample (along with new features such as Landing Pages) seem to use/require Angular 6.

Thanks @bryand2
We're aware of that and we will address it as time allows.


BUG: The navbar is not alligned perfectly on the Blog page.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Navbar on the Blog page is not alligned.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@jayraj175coder   please assign me this issue I would like to work on this issue.


Dropdown value vs. highlighted item inconsistency

Bug Report
I'm working on an image viewer that depends on three interrelated dropdowns to display the desired image. There are three levels of the viewer: 1) Anatomy, 2) ROI, and 3) Overlay. Changing the value of Anatomy also updates the values and options for ROI and Overlay, changing the value of ROI also updates the value and options for Overlay, and updating Overlay only changes itself.
The problem occurs after the value of Anatomy or ROI changes, and the other updated values are set. Even though all active values seem to be updated properly, when I actually click to open the dropdown menu, the old active value is still highlighted, and closing the dropdown sets it back to the old active value.
Thanks in advance for any help!
Steps
Use this to try out the steps: https://codesandbox.io/s/3vx1vjlrwm
One example:

Update ROI (second) dropdown to "ROI2"
Update Anatomy (first) dropdown to "Body," which proceeds to update ROI and Overlay dropdowns successfully. ROI should now show "ROI1"
Open ROI (second) dropdown. Even though "ROI1" is the value, the old value "ROI2" is still highlighted. Closing the dropdown will now set it back to "ROI1"

A second example:

Update Anatomy (first) dropdown to "Body"
Update ROI (second) dropdown to "ROI2" which updates the Overlay dropdown successfully. Overlay should now show "overlay2"
Update ROI (second) dropdown to "ROI5" which proceeds to update the Overlay dropdown again. Overlay should now show "overlay1"
Open Overlay (third) dropdown. Even though "overlay1" is the value, the old value "overlay2" is still highlighted. Closing the dropdown will now set it back to "overlay2"

Expected Result
I expect the highlighted value to match the value of the dropdown.
Actual Result
The highlighted value is reflecting its previous value rather than it's current value.
Version
0.78.2 - however, the codesandbox example has the same issue, and it's using version 0.81.1
Testcase
https://codesandbox.io/s/3vx1vjlrwm

Superb report, thanks.  This is a confirmed bug.  I believe it is a duplicate but will leave open for now.  We need a good bug fix pass on Dropdown.

In case anyone else is ending up here like I just did: I had a similar Bug and I tracked it down to the use of different lifecycle methods.
This Bug here can be temporarily fixed by using the soon to be deprecated componentWillMount like the Dropdown component instead of componentDidMount OR by updating the SUI component to the new lifecycle methods like suggested here https://github.com/Semantic-Org/Semantic-UI-React/issues/2732.
@kohakukun already had a PR open in october  to update the component, but these changes seem to have gotten lost somewhere on the way https://github.com/kohakukun/Semantic-UI-React/blob/8c649dc707cfedb61f7037861dbdffdc29f21d5a/src/modules/Dropdown/Dropdown.js

@Coder-Sharon the report is great, thanks 👍

Dropdown received lifecycles update in v1.0.0., I checked this and this bug was fixed 🚀
0.81.1

1.0.0

I also update CodeSandboxes to verify:

0.81.1 https://codesandbox.io/s/semantic-ui-react-6kkbd?file=/index.js
1.0.0 https://codesandbox.io/s/semantic-ui-react-vv06k?file=/index.js


add license and copyright notices

@mbocevski @duskobogdanovski @blazhovsky This is just a copyright notice and headers. Please review it, and if it's OK merge it. If not, let me know if there's anything that should be fixed.


Add session about RSTUF Mentions

It gives users some talks and mentions about RSTUF.

:books: Documentation preview :books:: https://repository-service-tuf--479.org.readthedocs.build/en/479/

Maybe we can add a blog section as well?
I suggest we use this order of headers: Talks, Blogs and Mentions.


(Low priority) we should enable keyring sometimes

@Abirdcfly are there any potential use cases for this helm keyring inner our operator?
For chart  signature verficiation,we can do that in our rating pipeline. For chart signing with private key, the only use case for now in my mind would be publish a component in lowcode platform .

https://helm.sh/zh/docs/topics/provenance/


when cognito refresh fails, the watch appears to continue to ask in background

even though the alert popup is raised, can see repeated refresh attempts on phone.
and dismissing the popup winds up getting it raised again

duplicate of #12


Multi-threading in I/O routines

Each memory block can be read by multiple threads independently and combined later.

Not much performance improvement


make build work with phpstorm

Contact Details
PULL REQUEST
What happened?
!
Version
1.0.0
Relevant log output
No response


📚documents(README.md): Special characters

I spent half an hour to mapping key ;
I think list it in ## Configuration  with be better for newbie like me.

That's a great idea! I've made a few nitpicks to your PR. You can put both semicolons inside one key, and I've clarified the description in the README. What do you think?

That's a great idea! I've made a few nitpicks to your PR. You can put both semicolons inside one key, and I've clarified the description in the README. What do you think?

yep, my bad 😅.
i copy paste my config, put both semicolons inside one key for this case will be better 👍

        [";"] = {
            [";"] = "<esc>A;<enter>",
            [","] = "<esc>A,<enter>",
        },

Alright, I will merge!


Added czech translation

How does it look with the Czech translation today?

The Czech translation was never added, however I'm sure the devs would be open to accepting a PR for one :)


Exporting as PDF

Is there a way to export SQL notebooks as PDFs easily? Thanks!

Not at the moment. I think I've seen this feature request from others, so I'll prioritize it for future work.

@claireboyd UChicago!

Small world! I could be interested in collaborating to add it if that would help - I'm not sure how big a lift it might be, but would be happy to learn.

also my friend/fellow CAPP student @divij-sinha would be down to collab to make this work as well!

@claireboyd @divij-sinha Great! I'd be happy to accept this as a contribution. Here are some high level steps on how I think this should be accomplished

add a new command in the extension manifest, something like Export as PDF, docs on that here
when the command is invoked, generate some HTML from the notebook state (should include cells and cell "outputs"). Right now, the tabular output is formatted as markdown– this will need to be converted into HTML. The same goes for any "markdown cells". The SQL query cells should be placed in

 tags.  (maybe a future enhancement could give these blocks syntax highlighting in the final output).  Finally, some simple embedded CSS to give reasonably attractive tables that format well in the final PDF output.
convert this HTML into a PDF (pretty sure there are libraries for this) and... (write to the current directory?, maybe prompt for a filename then write?)

If you make some progress and want to open a draft PR I'll be happy to give feedback along the way or iron out any issues you're having. In general, the VS Code extension API can be a bit tricky to work with but the docs are great.

Just chipping in with a suggestion :
Exporting the whole lot to markdown might be easier & more flexible as conversion to pdf, html could be done from the complete markdown.
Even a select all, copy-as-markdown would be cool


React player subtitle

Can I custom or change the font, color, and position for subtitles?

Assuming you are just playing a file path, this should help: https://stackoverflow.com/a/33244009


Visual Studio markdown engine

Hello.
Visual Studio added markdown support in version 2022 release 17.5.
Changelog above tells decision was made based on the results of a vote on developer community forum  (topic from 08.10.2018).
But I have not succeeded in finding out what engine is working under the hood, is it commonmark-compliant? GFM support? I think there is a high probability it's based on markdig but it's only suggestion.

But I have not succeeded in finding out what engine is working under the hood, is it commonmark-compliant? GFM support? I think there is a high probability it's based on markdig but it's only suggestion.

I don't know the details, but the default Markdown Editor in VS is using Markdig (the assembly is in some sub folders in VS install)
I believe it might be a derived work from @MadsKristensen v1 version, which now has a v2 version with MarkdownEditor2022

I've just installed VS LTSC 17.4 + last MarkdownEditor2022 2.0.169 to compare with my VS 2022 17.10.3 default markdown editor.
There are Microsoft.VisualStudio.Markdown.Platform and MarkdownEditor2022 in %localappdata% respectively.
This is it at first glance on hierarchy.

Hi @Dmitresso I'm working on Markdown support in Visual Studio.
VS 17.12 uses Markdig.Signed package version 0.30.2
We're using the following pipeline configuration
            new MarkdownPipelineBuilder()
                .UseAdvancedExtensions()
                .UsePragmaLines()
                .UsePreciseSourceLocation()
                .UseYamlFrontMatter()
                .UseEmojiAndSmiley()
                .UseEmphasisExtras()
                .UseTaskLists()
                .Build();


App crashes due to too many open files

The last segment of log when app crashes.
Complete log of the app runtime.
Background of App:

Every 8 seconds from a Fragment, 5 requests are sent to a microservice (5 new threads are opened for this).
The received response from the 5 requests (5 threads) within a period of 8 seconds are collected and displayed in UI.
Apps runs fine for around 11 minutes, but then it crashes with the following error:
 06-24 07:33:04.513 24171-1946/org.loklak.android.wok E/art: ashmem_create_region failed for 'indirect ref table': Too many open files.

Interesting.  Are you doing any file manipulation inside the microservice?

I am sending a network request using microservice then parsing it and emitting the result back to Java.

@ericwlange Here is how I am using the microservice,  getScrapedTweets method receives a query, it then sends this query to Microservice, microservice sends a GET request, once the result of GET request is obtained by microservice, the result is emitted back to Fragment.
This process is repeated in method dispalyAndPostScrapedData with a time gap of 4 seconds. The app runs for around 20 minutes, and then crashes.

Thanks.  I will see if I can replicate this faster with some simpler test.  This just sounds like I didn't close something somewhere.  But it is going to take me some time to dedicate effort to this project.  I started a new job recently and am slammed.


refactor use of Gems and add Jekyll

If you add remote theme to Gemfile plugins list, you can remove it from config.
Example of my site using a remote theme, without plugins field in config
https://github.com/MichaelCurrin/dev-cheatsheets/blob/master/Gemfile
https://github.com/MichaelCurrin/dev-cheatsheets/blob/master/_config.yml
the GH Pages gem is very heavy for a ton of gems one doesn't actually need. So I was more selectively with adding Jekyll and some dependencies needed for 3.9 and Ruby 3.

@MichaelCurrin got it. I see why your changes make sense, but for now I'm going to keep the gem as it makes things simple.


Prepare release

Bumping the version in last released failed because some PRs were merged during the release.
Also add the new image added in #177 to list of single arch images.

I've killed a build since it's a no-op.


This should go in Issue4.

Essentially make the buffer for the scanner longer, such that longer messages don't overrun the buffer and cause errors/failures.

It looks like your travis build setup is incorrect as well... or that's what this:
$ $HOME/gopath/bin/goveralls -service=travis-ci -repotoken $COVERALLS_TOKEN
flag needs an argument: -repotoken
Usage: /home/travis/gopath/bin/goveralls [options]
seems to tell me.. you are missing a token required by goverals?

actually I hadn't read the godoc properly :( this change isn't necessary.
The change to the README.md should be done though.

@morrowc What did you do to get around this? My understanding is we need to read the mbox file by line?

@payaaam I think that the right answer here is to read the mbox file line by line, yes and full up a struct per 'message' so you can do whatever you want when you get done per message/file... There are clearly some bugs still in this mbox implementation, because it won't read all the way through my mbox files :( it ends up failing partway through for what seem like 'random' reasons to me.
I've not gotten time to look at the problem in quite a while though :(
A directional hint here is probably to write a parser/lexer/etc following along the rob-pike model:
https://www.youtube.com/watch?v=HxaD_trXwRE

I wrote this code for a project I dropped long time ago. So I have no experiences about my mbox implementation in production. But as my initial tests were successfull und the unit tests are passing, I decided to put this package up nonetheless.
The biggest problem I am aware of, is the fact, that you have to define a "sufficient" buffer. If mbox is not scanning your complete mbox files, you should be able to retrieve the corresponding error, maybe that error gives a hint.

@blabber the mbox file in question is pretty large (190mB) I'll see if I can get something useful from the error.


Yaml parser

Closes #93. Update configurator to use the Yaml Parser class. This replaces #94.
Dependencies are currently broken and this PR requires #76.

Hi, sorry for the delayed reponse. @tmatsuo here. @satooshi kindly gave me a collaborator access and I'm going to work on it. Can you tell me what is the purpose of the change? Is Yaml class going to be deprecated?

Yes, the use of file names as parameters was deprecated in version 2.2 and as of vesion 2.7 (I think it stared on this version) it triggers an error see code for more info.

This code should be compatible with older and newer versions of the yaml component up to v2.8.x

@onema Sounds good. I have submitted #76, which fixes the broken build on travis. Can you rebase to the current master then push it here so that we can run the test on travis?

:+1:

@onema
Sorry about the travis failure. We're working on fixing the travis build at  #114 we're almost there :)

FYI, #114 is submitted finally!
Sorry for asking many times, can you rebase and let's try the test again?

There is no need to rebase. Branch is up to date.
@tmatsuo just restart the Travis build.
Travis work in that way that he is not checking out to the lastes commit of PR, but checking out to the latest commit from target branch (master here) and then apply changes from PR.

@keradus Unfortunately I don't have a permission to re-run the build, so rebasing and re-pushing is the only option I think.

Oh wait. Actually I have a permission to re-run the build. I was confused with the travis UI change. Restarting the travis build now.

;)

nice!

damn it, it tooks Travis configuration file from PR :/

in that case rebase is needed indeed :/

Aha, so @onema, you still need to rebase. I'll appreciate it if you can.

On it.

Rebasing was giving me some strange conflicts... I just fetch and merged. No issues there.

ref #119

@keradus @onema
Thanks!
#119 seems like the same change. Which do you want me to merge?

Ok, I'm going to merge this since @onema is the original author. SG @keradus?
@onema One more ask. Can you squash the commits before merging?

look at the author at #119 ...

I squashed (actually not squashed... but make it one-commit) for you in #119 since @onema has some problems with it. The author is still @onema... https://github.com/keradus/php-coveralls/commit/df38a0af41a4bd34eb696186b772b556015bd1cc

Ah good :) So I'm going to merge #119
Thanks @keradus !


struct_array_notation misreading top-level scopes

If you have code like this
variables.CFLINT_VERSION = "1.3.0";

It will mis-report it that I need to quote it in the struct.  This should be relaxed in my opinion for top level scopes or storage scopes. Thoughts?

Is that because you’re seeing a notation
variables.CFLINT_VERSION
as something similar to a string constant in Java?

On 14/02/2018, at 11:55 AM, Luis Majano<EMAIL_ADDRESS>wrote:
If you have code like this
variables.CFLINT_VERSION = "1.3.0";
It will mis-report it that I need to quote it in the struct. This should be relaxed in my opinion for top level scopes or storage scopes. Thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/cflint/CFLint/issues/542, or mute the thread https://github.com/notifications/unsubscribe-auth/AATrRAAH51CHltUMjLVmfPlM_FOnJXdPks5tUhLNgaJpZM4SEiGA.

That was my example but it is for any case.
Luis Majano
CEO
Ortus Solutions, Corp
www.ortussolutions.com
P/F: 1-888-557-8057

From: Kai Koenig<EMAIL_ADDRESS>Sent: Tuesday, February 13, 2018 8:54:55 PM
To: cflint/CFLint
Cc: Luis Majano; Author
Subject: Re: [cflint/CFLint] struct_array_notation misreading top-level scopes (#542)
Is that because you’re seeing a notation
variables.CFLINT_VERSION
as something similar to a string constant in Java?

On 14/02/2018, at 11:55 AM, Luis Majano<EMAIL_ADDRESS>wrote:
If you have code like this
variables.CFLINT_VERSION = "1.3.0";
It will mis-report it that I need to quote it in the struct. This should be relaxed in my opinion for top level scopes or storage scopes. Thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/cflint/CFLint/issues/542, or mute the thread https://github.com/notifications/unsubscribe-auth/AATrRAAH51CHltUMjLVmfPlM_FOnJXdPks5tUhLNgaJpZM4SEiGA.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/cflint/CFLint/issues/542#issuecomment-365496088, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAIXl7DKYWJzQ3Mha1NFVDrpEiqtewUDks5tUmcfgaJpZM4SEiGA.

@lmajano , I agree.  This rule was not intended for scopes.    Fixed in dev branch.


A0-3237: sync tests

Description
Series of tests based on the synthetic-network framework (a docker tool for simulating "real-life" network conditions). All introduced tests attempts to modify nodes connectivity (e.g. creating connected subsets of nodes) and verify if both block production and finalization are still possible.
Description of how to run these tests is in e2e-tests/README.md in section #Running e2e-tests that depend on synthetic-network, e.g. sync, high_latency .
Type of change

New feature (non-breaking change which adds functionality)

Checklist:

I have added tests
I have made neccessary updates to the Infrastructure

Excellent, great to have these tests! Only some small comments.
Will this be now running in nightly tests or something?

It is now part of the nighly-pipeline.


<EMAIL_ADDRESS>Menubar Trigger is broken: Cannot destructure property 'builders' of 'getCtx$c(...)' as it is undefined.

Describe the bug
The Menubar component (and possibly others as well, though I haven't tested them all) no longer works on the latest version of svelte 5 (5.0.0-next.121), throwing the following error:
Cannot destructure property 'builders' of 'getCtx$c(...)' as it is undefined.

Reproduction
https://svelte-5-preview.vercel.app/#H4sIAAAAAAAAE2WOwWrDQAxEf0Xo7NaH3BzHUHoOgVJyyfbgxFojsLWLVm4oxv9eHIeEJOgg6TEzzIieO0pYHEaUuics8CNGzND-4vykX-qMMMMUBj3NpEwn5WiVE2fcx6AGI2xJhmOtMIHX0IPDI1t6G9jh2kmZ3y1SXqXvXyEsITcy78oJAMCNfSu3LelFOM-e6bzcZf6qebB-BjESg7rjVjYOk9VqDiFxQzvvE9lmXE3Vc9rVtlTLn7rdwaU-ZtiHhj1Tg4XpQNPP9A9_5B5kUgEAAA==
Logs
No response
System Info
Latest version of svelte 5, see repro repl above.

Severity
blocking upgrade

This appears to be a dev time only issue. Building the project for production doesn't cause any errors and everything seems to be working as expected.
Given that this appears to be an issue directly with Svelte 5, there's not much we can do ourselves. For now, I've posted a reproduction in the Svelte 5 thread for libraries here: https://github.com/sveltejs/svelte/issues/10359

This was fixed in 5.0.0-next.122


test failure at chapter 04

ERROR executing 04_Quantifiers_and_Equality.org.16.lean, for in_code_block block starting at 320,

produced output:

04_Quantifiers_and_Equality.org.16.lean:5:0: error: unifier maximum number of steps (20000) exceeded,
the maximum number of steps can be increased by setting the option unifier.max_steps
(remark: the unifier uses higher order unification and unification-hints, which may trigger non-termination

import data.nat
open nat eq.ops algebra

example (x y : ℕ) : (x + y) * (x + y) = x * x + y * x + x * y + y * y :=
have H1 : (x + y) * (x + y) = (x + y) * x + (x + y) * y, from !left_distrib,
have H2 : (x + y) * (x + y) = x * x + y * x + (x * y + y * y),
  from !right_distrib ▸ !right_distrib ▸ H1,
!add.assoc⁻¹ ▸ H2

Having set_option unifier.max_steps 40000 is a solution (but maybe not a good one). Another possibility is to introduce an intermediate step as follows:
import data.nat
open nat eq.ops algebra

example (x y : ℕ) : (x + y) * (x + y) = x * x + y * x + x * y + y * y :=
have H1 : (x + y) * (x + y) = (x + y) * x + (x + y) * y, from (left_distrib (x + y) x y),
have H2 : (x + y) * (x + y) = x * x + y * x + (x + y) * y, from !right_distrib ▸ H1,
have H3 : (x + y) * (x + y) = x * x + y * x + (x * y + y * y), from !right_distrib ▸ H2,
  !add.assoc⁻¹ ▸ H3

@avigad @soonhokong I think we should avoid examples that use ▸.
This is an anti-pattern. It generates fragile proofs that are hard to maintain.
In the future, I want to completely disallow terms such as !right_distrib ▸ !right_distrib ▸ H1.
I know that in Chapter 4, tactics were not introduced yet.
But, we can introduce just the rewrite tactic.
Another option is to provide the argument to right_distrib explicitly, and avoid the ! notation.

Closed by d9aa276de23086f70db5065297309e668baaa1a2


[1.3] Bump jackson to 2.13.2

Signed-off-by: Joshua Li<EMAIL_ADDRESS>Description
bump jackson libraries for 1.3, it's already bumped in main
Build will fail until upstream OpenSearch is bumped as well
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #633 (7a2d3b4) into 1.3 (04dcd27) will decrease coverage by 32.33%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##                1.3     #633       +/-   ##
=============================================
- Coverage     95.25%   62.91%   -32.34%
=============================================
  Files           276       10      -266
  Lines          7416      658     -6758
  Branches        544      119      -425
=============================================
- Hits           7064      414     -6650
+ Misses          298      191      -107
+ Partials         54       53        -1

Flag
Coverage Δ

query-workbench
62.91% <ø> (ø)

sql-engine
?

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

Impacted Files
Coverage Δ

...org/opensearch/sql/opensearch/client/MLClient.java

...opensearch/executor/OpenSearchExecutionEngine.java

...arch/sql/expression/aggregation/SumAggregator.java

...ava/org/opensearch/sql/utils/SystemIndexUtils.java

.../opensearch/sql/planner/logical/LogicalWindow.java

...protocol/response/format/CsvResponseFormatter.java

...org/opensearch/sql/expression/env/Environment.java

...arch/sql/expression/datetime/DateTimeFunction.java

...java/org/opensearch/sql/sql/parser/AstBuilder.java

...org/opensearch/sql/analysis/QualifierAnalyzer.java

... and 256 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 04dcd27...7a2d3b4. Read the comment docs.

@joshuali925  We need to add resolutions for jackson
> Could not resolve all dependencies for configuration ':plugin:runtimeClasspath'.
   > Conflict(s) found for the following module(s):
       - com.fasterxml.jackson.dataformat:jackson-dataformat-smile between versions 2.13.2 and 2.12.6
       - com.fasterxml.jackson.dataformat:jackson-dataformat-yaml between versions 2.13.2 and 2.12.6

@ps48 the dependency comes from upstream, they will bump jackson on 1.3 eventually


ext: hal: st: move to modules

split ext/hal/st into two modules:

hal_stm32: For the code meant to run on STM32
hal_st: For the code meant to drive ST components

Fixes #16316

Ack for the hal_stm32 part.
I'll let time to @avisconti to check the hal_st part.

@nashif , seems that hal_st leans under libs.
Is that because these are, in opposition to hal, not dedicated to a mcu family?

@nashif , last though: can we change 'stm32' to 'stm32cube'?
This will be more in line with our general naming.

@nashif , last though: can we change 'stm32' to 'stm32cube'?
This will be more in line with our general naming.

agree!

@nashif , last though: can we change 'stm32' to 'stm32cube'?

@nashif , @avisconti , actually coming back on that. stm32cube is already in hal_stm32 path (underneath), this is enough.
Let's keep as it is


Clicking on '中文' or 'English' in https://pypi.org/project/RPALite/ will result in a 404 status code

Link: https://pypi.org/project/RPALite/

This is related to the documentations. Since we are still in early stage of development I'll keep this issue as open. In future after we complete the main features we will update the documentations


Remove Location_groups, Add Stop_areas

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JSDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
This PR removes front end support for location_groups which have been removed from the GTFS flex specification in favour of stop_areas. Unlike location_groups, stop_areas uses two separate tables to define its areas (stop_areas.txt) and name them (areas.txt). As a first step, and to make this PR easier to review I have only added the stop_areas component here. I will add in functionality for naming the areas potentially after the big dev-flex to dev merge.

Adding a "blocked" label pending gtfs-lib and datatools-server dependencies.


Received parameters not in model: {extras}.

Good to see you. I'm a newbie.
I am using an Apple M2 laptop. I am going to try to train a model using lora.py.  If I run the following.
python lora.py --train --model Qwen/Qwen2-0.5B-Instruct  --data /opt/mlx/data --batch-size 2 --lora-layers 8

I get the following error:
[INFO] Quantizing
Traceback (most recent call last):
  File "/opt/mlx/mlx-examples/lora/convert.py", line 93, in <module>
    weights, config = quantize(weights, config, args)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/mlx/mlx-examples/lora/convert.py", line 18, in quantize
    model.load_weights(list(weights.items()))
  File "/Users/kasumi/.mlx/lib/python3.11/site-packages/mlx/nn/layers/base.py", line 211, in load_weights
    raise ValueError(f"Received parameters not in model: {extras}.")

Using a different model does not resolve this issue. Please help me :D

Sonoma 14.5
Python 3.11.6

Thank you!

The lora.py example only supports a limited set of models. To run with more models use MLX LM.

pip install -U mlx-lm
mlx_lm.lora --train --model Qwen/Qwen2-0.5B-Instruct  --data /opt/mlx/data --batch-size 2 --lora-layers 8

More documentation here. If that doesn't solve the problem, let us know and I can re-open this issue.


Hugging Face Free Sandbox Inference API vs Production Inference Endpoints

Weaviate's HuggingFace module is running with the Free Inference API, which is limited to 1Mb of texts on the PRO plan.
It looks like the paid inference is on  Inference Endpoints API.
The question is therefore: will Weaviate support the Inference Endpoints API?

I had a chat with Hugging Face support. Inference Endpoints API is now the production API.

@antas-marcin Can you manage to keep the current Free Inference API while introducing the new Inference Endpoints API in Weaviate's module?
It will enable users to try Weaviate with the free API, and eventually let them upgrade to the paid API.

@antas-marcin Thanks. Tested with success.


K8s version breaks when none passed through GHA

Something is going wrong with the handling of empty k8s_version and kind_k8s_version variables used by clusters.sh.
If a consumer adds a job via a GHA include statement, and doesn't specify a k8s_version (like this example from submariner-operator):
+++ b/.github/workflows/e2e-full.yml
@@ -14,6 +14,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
+        deploytool: ['operator']
         globalnet: ['', 'globalnet']
         k8s_version: ['1.20']
         lighthouse: ['', 'lighthouse']
@@ -26,6 +27,8 @@ jobs:
         include:
           # This is the oldest K8s version we try to support
           - k8s_version: '1.17'
+          - deploytool: 'helm'
     steps:

Then the resulting k8s_version is null:
2021-06-02T15:36:41.1765851Z /opt/shipyard/scripts/clusters.sh: line 27: kind_k8s_versions: bad array subscript
2021-06-02T15:36:41.1768333Z Running with: k8s_version=, olm_version=0.14.1, olm=false, globalnet=false, registry_inmemory=true, cluster_settings=/go/src/github.com/submariner-io/submariner-operator/scripts/kind-e2e/cluster_settings, timeout=5m

Which breaks the install:
2021-06-02T15:38:55.4520743Z [submariner-operator]$ [cluster1] command kubectl --context=cluster1 apply -f https://cloud.weave.works/k8s/net?k8s-version=v&env.IPALLOC_RANGE=<IP_ADDRESS>/16
2021-06-02T15:38:55.4523241Z [cluster1] error: unable to read URL "https://cloud.weave.works/k8s/net?k8s-version=v&env.IPALLOC_RANGE=<IP_ADDRESS>/16", server reported 400 Bad Request, status code=400

Adding a job like this, with a k8s_version, works:
+++ b/.github/workflows/e2e-full.yml
@@ -14,6 +14,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
+        deploytool: ['operator']
         globalnet: ['', 'globalnet']
         k8s_version: ['1.20']
         lighthouse: ['', 'lighthouse']
@@ -26,6 +27,8 @@ jobs:
         include:
           # This is the oldest K8s version we try to support
           - k8s_version: '1.17'
+          - deploytool: 'helm'
+            k8s_version: '1.20'
     steps:

Here's where I tested this and showed some details of the weirdness when adding this var: https://github.com/submariner-io/shipyard/pull/496#discussion_r601584196


GetReferencedAssemblies() returns two different entries for Unquote assembly

Minimal repro project https://github.com/vasily-kirichenko/fcs-unquote-bug
ParseAndCheckResults.ProjectContext.GetReferencedAssemblies returns duplicated entry for Unquote.dll:

VFPT cache entities by assembly full path. As a result:

the first referenced Unquote actually contains strange top-level entities:

from which we cannot get any leaf entities, and cache this empty list.

the second referenced Unquote has identical path, so we do not get entities from it, however it seems it contains proper ones:

As a final result, we cannot determine that open statements are not redundant https://github.com/fsprojects/VisualFSharpPowerTools/issues/712

FYI - as of version 3.0.0, Unquote no longer uses static linking

OK, thanks. Closing this as it is a very rare case, let's reopen if we see it again in the wild.


No indication of bug reports being exposed publicly

If a user submits a bug report on page report_bug, a new issue is created automatically, immediately.
Users are not informed anywhere that these reports are made public as soon as they are created, and might assume that only system administrators will be able to see it and therefore might include personal information. I'm also not sure if organizations using this software have consent from their respective users to publicly disclose the bug with the name of the reporter.
I think this feature should be removed. If a user is capable of submitting a github issue, we could give them a link to this repo. Otherwise, it is much easier to get the report on a more interactive platform (like e-mail), and ask for additional information.

People won’t register to GitHub just to submit a small bug. It should be enough to inform them about publicity before they send the form or alternatively offer the email option.

On 14 Feb 2024, at 10.44, horcsinbalint @.***> wrote:
If a user submits a bug report on page report_bug, a new issue is created automatically, immediately.
Users are not informed anywhere that these reports are made public as soon as they are created, and might assume that only system administrators will be able to see it and therefore might include personal information. I'm also not sure if organizations using this software have consent from their respective users to publicly disclose the bug with the name of the reporter.
I think this feature should be removed. If a user is capable of submitting a github issue, we could give them a link to this repo. Otherwise, it is much easier to get the report on a more interactive platform (like e-mail), and ask for additional information.
—
Reply to this email directly, view it on GitHub https://github.com/EotvosCollegium/mars/issues/425, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANMZKMJOZZHOWK4E77MKZILYTSBOZAVCNFSM6AAAAABDH6WELSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEZTGOJTGA4TENA.
You are receiving this because you are subscribed to this thread.


Tracking: YouTube channel

Create new sub tasks and reference this issue.

[ ] Add youtube contributors guide to repo #19
[ ] Use YouTube live content for the Rust channel #20

Closed due to inactivity


Enable possibility to generate cucumber.json with dynamic prefix

Current behavior
Currently it's possible to set desired alternative static cucumber.json name/path with json.output property
Desired behavior
Hello @badeball! What do you think about possibility to parametrize json.output property a little bit more? It would be really useful to have a possibility to set 'unique' (ex. timestamp or feature name) name for this json to avoid overwriting test results in some cases
Checklist

[ x] I've read the FAQ.
[x ] I've read instructions for logging issues.
[ x] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

Hi @elena-altuhova. I'm not opposed to the idea, but can you ellaborate a bit more the "some cases" in which this would be "really useful"?

So our current case, which basically was the reason of this suggestion, is that with an attempt to upgrade to latest Cypress version (from 10 to 12+) we switched from cy2 to cypress-cloud for parallel runs.
After switching we have noticed that cypress-cloud orchestrates the parallel run in such a manner, where it gives a command to execute one feature separately and only after it finishes - another feature and so on. (native orchestration diagram describes it here)
So when it happens - cucumber.json gets overwritten and at the end of the run we have only last feature results in there. Having possibility to have unique names for cucumber.json for each executed feature would solve this problem for us.

I'm not so sure that this requires any change from the library side. You can configure a dynamic output path during runtime, like shown below.
const { defineConfig } = require("cypress");
const { cloudPlugin } = require("cypress-cloud/plugin");

module.exports = defineConfig({
  e2e: {
    async setupNodeEvents(cypressOn, config) {
      const on = require("cypress-on-fix")(cypressOn);

      if (config.projectId) {
        config.env["jsonOutput"] = `cucumber-report-${new Date().getTime()}.json`;
      }

      await addCucumberPreprocessorPlugin(on, config);

      on(
        "file:preprocessor",
        createBundler({
          plugins: [createEsbuildPlugin(config)],
        })
      );

      return cloudPlugin(on, config);
    },
  },
});

Just tried it and it works! Thanks a lot for your help! 🥇


Inquiry about Chrome Extension integration

Hi there,
I was wondering if your GitHub repo has integrated a Chrome extension within your app? I couldn't find any information about it on your repository, so I thought I'd reach out and ask.
If it's not something that you have already incorporated, I was wondering if you could provide some guidance on how I could go about adding a Chrome extension to an app while building .
Thank you for your time and consideration.
Best regards,
SHREY RAJ

Kiwi do not bundle specific extension . Also I don't think guidance will be provided as there is a lot of resource to start from


Links on ReadMe broken

The links on the ReadMe page are broken. Eg: Clicking on Personalised Views link, only reloads the page, and does not scroll down to the "Personalized Views" header.
I would like to fix the same.

Submitted PR #333

Thank you, merged


Go mod version

When I want to obtain dependencies through go mod, I am prompted that the dependent version does not exist. The reason is that although the version number is greater than or equal to 2, the v2 suffix is not added to the go mod

Maybe https://github.com/prometheus/prometheus/issues/9302#issuecomment-914367303 can help?

Yes, please see above comment. you can do go get<EMAIL_ADDRESS>


Hide "Provisioning Profiles" group from tree

On macOS, I see an empty "Provisioning Profiles" group in every Product. It comes from xcode.qbs:
Group {
    name: "Provisioning Profiles"
    prefix: xcode.provisioningProfilesPath + "/"
    files: ["*.mobileprovision", "*.provisionprofile"]
    fileTags: ["xcode.provisioningprofile"]
}

I don't think we should display it at all.

so, do you suggest to do not display an empty groups (which are does not contains the artifacts)?

Could you please check again?


Fixed formatting in Linux install guide

Formatting issues were introduced in PR #8825, commit f30918b3694afe844990cbddc82e27e023d88856

Can one of the admins verify this patch?

Jenkins, test this please


.Net Standard 2.0 implementation of DSP FFT

Performance numbers for FFT in the digital signal processing library were based on a native implementation of the algorithms in .Net Framework. The OSS release of Trill and all of its features must be .Net Standard 2.0 compliant, and it cannot rely on any library for which there is not proper licensing. We need to do one of the following:

Implement our own FFT library over .Net Standard 2.0 (not preferable)
Find a .Net Standard library over which we can take a dependency (proper licensing, good performance)
Find a native library over which we can take a dependency and implement .Net Standard interop (proper licensing, good performance) (not preferable)

One option to consider: https://github.com/mathnet/mathnet-numerics.

I especially like that the library you mention has idiomatic F# extensions. Great idea.
For what it's worth, we've made the branch that we're developing the DSP extensions public in case the community would like to contribute or comment.


Bug: v4: Codeblocks don't update correctly in editing view

Describe the bug
If you use codeblock, navigate away, then navigate back, the codeblock will be for the previous page you navigated to.
Desktop, v. 4.0.15-beta
To Reproduce

Make a codeblock like this in editing mode:

type: tree
dir: down

Click a link on one of the links in the codeblock
Click back
Codeblock will have the tree for the previous page

Expected behavior
Should have the tree for the current page
Screenshots
n/a
Desktop (please complete the following information):

OS: windows 11
Version 4.0.15-beta

Smartphone (please complete the following information):
n/a
Additional context
Is there another way to have an updated breadcrumb list?

Looks like this is how dataview does it, making an editor extension
https://github.com/blacksmithgu/obsidian-dataview/blob/e4a6cab97b628deb22d36b73ce912abca541ad42/src/main.ts#L183
some background
https://forum.obsidian.md/t/registermarkdownpostprocessor-callback-not-called-with-live-preview-mode/56049/2
I'm bad at programming

Thank you for the issue and links :)

Here's a work-around for this:
I have it set to a stylized header on my Breadcrumbs note so I can just embed the list in the floating callout.
Still working on formatting, but it works pretty well. Bonus: if I want to change it across all my notes, I only have to update it in one place.


⚠ SceneTime has degraded performance

In a4aabaf, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1496 ms

Resolved: SceneTime performance has improved in e7c107c after 6 minutes.


Fix failure building python-ldap

Fixes /usr/bin/ld: cannot find -lldap_r error when compiling python-ldap on Fedora 36.
The single dependency was updated manually in the generated requirements.txt file.

Let me know if running pip-compile requirements.in makes more sense - but I guess this would potentially break things.

Let me know if running pip-compile requirements.in makes more sense - but I guess this would potentially break things.

I'm fine with updating the requirements.txt, I tried to figure out a solution to run pip-compile to produce a requirements file with package versions equal to the existing environment, the purpose is to install our dependencies with yum and rpms first, then generate a requirements based on what the dependencies provided by these rpms, but I didn't find a solution :(

BTW, for gating apps I started using Poetry to handle dependencies with version pinning and try to avoid using any system packages whenever possible. Dependabot can auto-update the poetry.lock file nicely though it often creates conflicting PRs (which just means I wait after merging a PR for dependabot to auto-resolve conflicts).
See: https://github.com/release-engineering/greenwave (mainly pyproject.toml, tox.ini, Dockerfile, poetry.lock)

BTW, for gating apps I started using Poetry to handle dependencies with version pinning and try to avoid using any system packages whenever possible. Dependabot can auto-update the poetry.lock file nicely though it often creates conflicting PRs (which just means I wait after merging a PR for dependabot to auto-resolve conflicts).
See: https://github.com/release-engineering/greenwave (mainly pyproject.toml, tox.ini, Dockerfile, poetry.lock)

nice, I'll take a look later :)


Allow BigIntegers to be defined in the /process/*/start/ map parameters

This incremental improvement on the object marshaling of numeric map parameters allows BigIntegers to be placed on the process instance's parameters.  These big integers can better accommodate primary keys on relational databases.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

@mrietveld could you please take a look at this one?

@psiroky Please close.
@xmrk Unfortunately, we're deprecating this code for 7.0. However, I've copied this code so that we can use it for the new code where the REST API is implemented.

closing based on @mrietveld, request


Add durable websocket client from serving.

The durable websocket client has been proven valuable in other projects too (namely an eventing websocket source), thus moving it here makes it possible to broaden its scope and make it accessible for all projects more easily.
It's coming from https://github.com/knative/serving/tree/master/pkg/websocket and contains the changes I proposed in https://github.com/knative/serving/pull/2635
/cc @josephburnett

/assign @mattmoor
If you'd do me the honors 🙏

/lgtm
/approve


Write release notes for the chart deploying the new architecture

We should write release notes for the helm chart version that introduces the new architecture.
The goals are:

Explain what changed
Provide manual instructions, if needed, for people upgrading from an older release

What's new?

Introduce a PolicyServer CRD that allow users to describe a Policy Server deployment
ClusterAdmissionPolicy has a new PolicyStatus field which can be :

unscheduled, unschedulable, pending or active

Separate helm chart for ClusterAdmissionPolicy and PolicyServer CRDs

How to upgrade?
Unfortunately helm upgrade is not supported for this release. Uninstall previous kubewarden installation and delete the ClusterAdmissionPolicy CRD:
kubectl delete --all clusteradmissionpolicies.policies.kubewarden.io
helm uninstall --namespace kubewarden kubewarden-controller
kubectl delete namespace kubewarden
kubectl delete crd clusteradmissionpolicies.policies.kubewarden.io

Then install kubewarden's new version:
 kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v1.5.3/cert-manager.yaml
 helm install --create-namespace -n kubewarden kubewarden-crds kubewarden/kubewarden-crds
 helm install --wait -n kubewarden kubewarden-controller kubewarden/kubewarden-controller

I've done some edits to your comment to fix the release notes. Good job!

Should we be even more verbose? (I like overly verbose changelogs). I think it would be nice to mention:

Drop of policyserver configmap (obviously) for configuring, moved to the CR.
Addition of spec.policyserver to ClusterAdmissionPolicy CRD, to target specific PolicyServers.
Validation of CRs with validating and mutating webhooks.
Finalizers.
New condition PolicyActive for ClusterAdmissionPolicies, useful for kubectl wait for example.

Do we include Rego, OPA, Gatekeeper policies support at some point?

Rego support is already available, starting from this release: https://github.com/kubewarden/helm-charts/releases/tag/kubewarden-controller-0.2.3.
@viccuad I've applied your suggestions

Why is this not closed ?

@kkaempf we can close this. We still need to update the Release Notes once we do the release, but there is a separated card for that.


Update Docusaurus to 2.4.3

Checklist

[x] I have read the How to Contribute.
[x] I have run yarn build after adding my changes without getting any errors.

Updating documentation or Bugfix
Update Docusaurus to 2.4.3

Fixed link warning from CIP upstream through https://github.com/cardano-foundation/CIPs/pull/599.

Fixed missing-page warning (and dead link from the Overview page) in https://github.com/cardano-foundation/developer-portal/pull/1155.

I'm creating a draft for the work to upgrade to v3 beta-0 in preparation for release of v3
https://github.com/cardano-foundation/developer-portal/pull/1157


Exit with some non-zero exit status when things aren't stellar

In the below example, blah is non-existent. I'd expect some non-zero exit status if the requested file I'm requesting be loaded doesn't exist.
grant: ~/.../whosonfirst/go-whosonfirst-clone (master)$ ./bin/wof-clone-metafiles -dest /tmp blah
[wof-clone-metafiles] 23:06:05.620065 [error] Failed to read /Users/grant/repos/whosonfirst/go-whosonfirst-clone/blah, because open /Users/grant/repos/whosonfirst/go-whosonfirst-clone/blah: no such file or directory
[wof-clone-metafiles] 23:06:05.622269 [info] processed 0 files (ok: 0 error: 0 skipped: 0) in 0.002046 seconds

grant: ~/.../whosonfirst/go-whosonfirst-clone (master)$ echo $?
0

Two things:

The code has retry hooks at the end on the grounds that AWS/S3 is expected to fail at least once during a batch:

https://github.com/whosonfirst/go-whosonfirst-clone/blob/master/clone.go#L162-L168

But yes, it should behave list a proper CLI tool assuming the retry code fails

Okay, so there is this:
https://github.com/whosonfirst/go-whosonfirst-clone/compare/exit
Can you rebuild from the exit branch and let me know if this accomplishes what you're after?

Perfect, you rock.
grant: ~/.../whosonfirst/go-whosonfirst-clone (exit)$ ./bin/wof-clone-metafiles -strict -dest ../data/ ../blah
[wof-clone-metafiles] 13:58:07.841550 [error] Failed to read /Users/grant/repos/whosonfirst/blah, because open /Users/grant/repos/whosonfirst/blah: no such file or directory
[wof-clone-metafiles] 13:58:07.841699 [error] failed to clone ../blah, because open /Users/grant/repos/whosonfirst/blah: no such file or directory
grant: ~/.../whosonfirst/go-whosonfirst-clone (exit)$ echo $?
1

Okay, I will merge with master just as soon as I finish commenting on this other issue ;-)
tasteful muzak plays

Now in master.


[Question] backtrack/snipers aim

Hello. I'd like to know what does "recoil based backtrack" mean in Osiris. I tried it on and off and i couldn't tell the difference, and there's no explanation for it in the github Readme. I'd like to know what makes backtrack stronger, having it on or off?
And one more thing, in Aimbot menu i can find smg/heavy/rifle/pistol tabs but no tab for Awp, scout, autos. I guess their settings are included in "All" tab?

snipe (rifles) are apart of the rifles tab iirc.
recoil based backtrack has something to do with the aimpunch but i haven't done enough testing to see if its benefitial.

Thanks.