Multiple same name packages appeared right after the package upgrade

Brief description of your issue
I just upgraded a package and end up with multiple different versions of the package
Steps to reproduce
type winget upgrade Microsoft.SQLServerManagementStudio
and then type winget upgrade
Expected behavior
no result should include Microsoft.SQLServerManagementStudio
Actual behavior
Name                                   Id                                     Version        Available      Source
------------------------------------------------------------------------------------------------------------------
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio    2.3.28307      15.0.18410.0   winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio    12.0.40664.0   15.0.18410.0   winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio    1.36.1         15.0.18410.0   winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio    14.29.30133.0  15.0.18410.0   winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio    14.29.30133.0  15.0.18410.0   winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio    1.35.0         15.0.18410.0   winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio    11.4.7462.6    15.0.18410.0   winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio    12.0.40664.0   15.0.18410.0   winget

Environment
Windows Package Manager (Preview) v1.3.1251-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22000.613
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.18.1251.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Related to #2137
cc @JohnMcPMS
https://github.com/microsoft/winget-cli/pull/2137#issuecomment-1118884538

I suggested on the PR that we should just use the local name for list because of how they are using this in the manifest.  There are still going to be ... issues ... with targeting these for uninstall, but at least list won't be confusing.
In addition, it does appear that it supports SxS installs, so this would be the fun case for adding support for both SxS and multi-component packages.

This has been improved in v1.3.1391-preview but is still bugged
Name                                                         Id                                  Version      Available    Source
---------------------------------------------------------------------------------------------------------------------------------
Microsoft Help Viewer 2.3                                    Microsoft.SQLServerManagementStudio 2.3.28307    15.0.18410.0 winget
Microsoft Visual C++ 2013 Redistributable (x64) - 12.0.40664 Microsoft.SQLServerManagementStudio 12.0.40664.0 15.0.18410.0 winget
Azure Data Studio (User)                                     Microsoft.SQLServerManagementStudio 1.36.2       15.0.18410.0 winget
Microsoft SQL Server 2012 Native Client                      Microsoft.SQLServerManagementStudio 11.4.7462.6  15.0.18410.0 winget
Microsoft Visual C++ 2013 Redistributable (x86) - 12.0.40664 Microsoft.SQLServerManagementStudio 12.0.40664.0 15.0.18410.0 winget

I have the same issue as reported above:
Name                                   Id                                  Version      Available    Source
-----------------------------------------------------------------------------------------------------------
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio 2.3.28307    15.0.18420.0 winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio 12.0.40664.0 15.0.18420.0 winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio 1.36.2       15.0.18420.0 winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio 11.4.7462.6  15.0.18420.0 winget
Microsoft SQL Server Management Studio Microsoft.SQLServerManagementStudio 12.0.40664.0 15.0.18420.0 winget
5 upgrades available.

Winget info as follows:
C:\Users\yooak\Downloads>winget -v --info
Windows Package Manager (Preview) v1.3.1251-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22000.708
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.18.1251.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

I'm also getting too many packages marked as Microsoft.SQLServerManagementStudio, resulting in an enormous, superfluous download every time I run winget upgrade --all.
Name                                     Id                                  Version        Available      Source
-----------------------------------------------------------------------------------------------------------------
Microsoft Help Viewer 2.2                Microsoft.SQLServerManagementStudio 2.2.23107      15.0.18420.0   winget
Microsoft Help Viewer 2.3                Microsoft.SQLServerManagementStudio 2.3.28307      15.0.18420.0   winget
Microsoft SQL Server 2012 Native Client  Microsoft.SQLServerManagementStudio 11.4.7462.6    15.0.18420.0   winget

My winget info:
Windows Package Manager (Preview) v1.3.1391-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19044.1706
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.18.1391.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

@denelon I think this needs to be fixed before 1.3 is generally available, as it's a showstopper for adding AppsAndFeaturesEntries to manifests (since this will result in infinite upgrades for any manifest where all of the ARP entries don't have the same exact version number).
Steps to reproduce:

winget install Microsoft.SQLServerManagementStudio, which installs the latest version.
winget upgrade, and see that the components that were just installed need upgrades.

I proposed https://github.com/microsoft/winget-cli/issues/2186 as a possible solution, but since schema changes aren't possible this late in the game, is there an easier solution for the time being? I feel like if we know that "Azure Data Studio" exists in the AppsAndFeaturesEntries list for that package, we should be able to just compare the version from the component in the list instead of comparing to the version of the package. I haven't looked at that code in a minute though, so I can't remember if that's really easy.
(Of course, the better matching via AppsAndFeaturesEntries could also just be hidden behind an experimental flag, but I'd prefer if we didn't have to do that as then we'd have to delay it being generally available until 1.4).

Paraphrasing @johnmcpms:

Having multiple entries in the manifest enables this kind of scenario that the code was never prepared for.
Full support for having multi-component packages is probably an all of 1.4 kind of deliverable.  It might even be a 2.0 kind of feature, depending on what we would want to do with it and how much impact that would have on the rest of things.

It might be best to limit to a single entry for Apps & Features per installer node. I don't believe we gain anything with multiple entries until the client can properly reason about them.

It might be best to limit to a single entry for Apps & Features per installer node.

That's fairly similar to how it is today, except instead of the ARP name needing to equal the PackageName, we can keep the ARP name in AppsAndFeaturesEntries. An improvement, no doubt, even if it wasn't what we were hoping for (and @JohnMcPMS is totally right, multi-component packages do create a ton of questions).
Unfortunately, it won't correct the matching issues for most (all?) of the apps that needed AppsAndFeaturesEntries. JetBrains apps, for instance, where the "marketing" version number is different than the ARP version number. With the current behavior, changing the PackageVersion to the marketing version number and adding the ARP entry to AppsAndFeaturesEntries would result in buggy upgrade behavior (since version 228.x.y is less than 2022.3.1, for instance). We use the ARP version number for JetBrains apps in the PackageVersion field right now, but that causes user confusion because JetBrains doesn't refer to it with that version number in their docs.
I think my original thinking when writing up https://github.com/microsoft/winget-cli/issues/1073 was that the components themselves shouldn't be used for comparisons when figuring out upgrades, but just as another way of matching to a specific package and version. Then that package we matched to would be used for comparing for upgrades. The way it is right now, winget is treating each component like it's a separately installed version of the package.
Now that I'm writing it out, this is way more complicated than I thought before writing this comment. Anyway, long story short, I'm not certain that allowing a single AppsAndFeaturesEntry corrects any problems that we haven't already worked around (at least in the community repo).

Rustup: the Rust toolchain installer Rustlang.Rustup Unknown 1.25.5    winget
1 upgrades available.

Please check this one.

Rustup: the Rust toolchain installer Rustlang.Rustup Unknown 1.25.5    winget
1 upgrades available.

Please check this one.

That is an unrelated issue. The publisher is not setting a display version in the ARP entries. See https://github.com/microsoft/winget-pkgs/issues/67393

Unrelated, but the Windows Terminal winget version is behind the regular version, can you fix it?

I completely misunderstood the last comment, my bad. Thought they meant that Terminal had its own version of winget.
Don't try to tech right after waking up lol

I completely misunderstood the last comment, my bad. Thought they meant that Terminal had its own version of winget. Don't try to tech right after waking up lol

For me, I need to add "or having that first cup of coffee" 😊

Unrelated, but the Windows Terminal on winget version is behind the current version, can you fix it?

Windows Terminal updates their packages as they release. There is currently an issue for Windows 10 users as the latest version only support WIndows 11. The current workaround is winget show Microsoft.WIndowsTerminal --versions then one can specify the version to install. It's generally the one before the latest.

Any idea when this issue will be resolved?

Any idea when this issue will be resolved?

For SSMS it should already be resolved. You may have to clear your installed.db file by uninstalling and re-installing winget, and making sure you are on the latest stable version or higher

@Trenly, thanks for the quick reply.
Unfortunately, uninstalling it from the Apps & features is not working, is there any way you know how to deleted installed.db manually?

@Trenly, thanks for the quick reply.
Unfortunately, uninstalling it from the Apps & features is not working, is there any way you know how to deleted installed.db manually?

In powershell you should be able to run Remove-AppxPackage Microsoft.DesktopAppInstaller, or if you clone the winget-pkgs repo you can use the Powershell Script ./Tools/WingetVersionManager -Latest -Clean

Uninstall didn't work 😞

And, I deleted the installed.dbs manually from 🤯

"$env:userprofile\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\StoreEdgeFD\installed.db"
"$env:userprofile\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db"

and so far it stopped working at all 😢

and if I try to list upgrade packages

Maybe try winget source reset --force? That'll wack the source entirely and reinstall it. (you'll have to be an Administrator)

And, I deleted the _installed.db_s manually from 🤯

"$env:userprofile\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\StoreEdgeFD\installed.db"
"$env:userprofile\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db"

This should be fine, with predictable (and acceptable) results.  They should not have any effect on your ability to refresh the source.  That is likely a separate issue, and the logs from the refresh attempt (or a new one) would be helpful.

Maybe try winget source reset --force? That'll wack the source entirely and reinstall it. (you'll have to be an Administrator)
@jedieaston, I did it already.

This should be fine, with predictable (and acceptable) results. They should not have any effect on your ability to refresh the source. That is likely a separate issue, and the logs from the refresh attempt (or a new one) would be helpful.

@JohnMcPMS, I was going to send you that but had to restart, and it started to work
@Trenly, after deleting those two files, and a restart, I confirm that it fixed the issue.

Thanks all! 🤗

That said, I realized that, issue is still happening for DotNet packages as shown below.

That looks like https://github.com/microsoft/winget-cli/issues/2408

It's happening again for Pycharm Professional.

Can you post the output of winget list pycharm please?

Can you post the output of winget list pycharm please?

Name             Id                             Version  Available Source
-------------------------------------------------------------------------
PyCharm 2022.2   JetBrains.PyCharm.Professional 2022.2   2022.2.1  winget
PyCharm 2022.2.1 JetBrains.PyCharm.Professional 2022.2.1           winget

That looks like you have two versions of PyCharm installed. We need to fix this in the manifest to make sure winget knows to clean up the old version on upgrade.

In the meantime, running winget uninstall JetBrains.PyCharm.Professional --version 2022.2.1 will resolve it for you locally.

Found PyCharm Professional Edition [JetBrains.PyCharm.Professional]
Starting package uninstall...
An unexpected error occurred while executing the command:
0x800401f5 : Application not found

Can I at least blacklist JetBrains products? JetBrains products already update automatically nicely, winget just f's it up.

The manifest can have the "requireExplicitUpgrade" tag set which would exclude it from 'winget upgrade -all' by default, but a user could still "--force" an install.

requireExplicitUpgrade

Can You add this tag on all Jetbrains products manifest? [Assuming you guys are responsible for maintaining it]

Even after upgrading, I'm getting them again the list
winget upgrade --all
Name                                            Id                             Version    Available   Source
------------------------------------------------------------------------------------------------------------
Android Studio                                  Google.AndroidStudio.Canary    2<IP_ADDRESS> 2<IP_ADDRESS> winget
Eclipse Temurin JDK with Hotspot 17.0.4+8 (x64) EclipseAdoptium.Temurin.17.JDK <IP_ADDRESS>   <IP_ADDRESS>  winget

Name                                                Id                             Version          Available    Source
-----------------------------------------------------------------------------------------------------------------------
Android Studio                                      Google.AndroidStudio.Canary    2<IP_ADDRESS>       2<IP_ADDRESS>  winget
Eclipse Temurin JDK with Hotspot 17.0.4+8 (x64)     EclipseAdoptium.Temurin.17.JDK <IP_ADDRESS>         <IP_ADDRESS>   winget
Windows Software Development Kit - Windows 10.0.19… Microsoft.WindowsSDK           < 10.0.22000.832 10.0.22621.1 winget
3 upgrades available.
1 package has a version number that cannot be determined. Using "--include-unknown" may show more results.

After the update, I got 2 versions of JDK inside the folder.

I've been getting these same two .NET packages every time I run upgrade for several weeks:
PS C:\> winget upgrade --all
Name                                       Id                           Version  Available Source
-------------------------------------------------------------------------------------------------
Microsoft .NET SDK 6.0.302 (x64)           Microsoft.DotNet.SDK.6       6.0.302  6.0.401   winget
Microsoft .NET Core Runtime - 2.1.30 (x64) Microsoft.DotNet.Runtime.3_1 < 3.1.25 3.1.29    winget
2 upgrades available.
2 package(s) have version numbers that cannot be determined. Use "--include-unknown" to see all results.

(1/2) Found Microsoft .NET SDK 6.0 [Microsoft.DotNet.SDK.6] Version 6.0.401
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://dotnetcli.azureedge.net/dotnet/Sdk/6.0.401/dotnet-sdk-6.0.401-win-x64.exe
  ██████████████████████████████   196 MB /  196 MB
Successfully verified installer hash
Starting package install...
Successfully installed

(2/2) Found Microsoft .NET Runtime 3.1 [Microsoft.DotNet.Runtime.3_1] Version 3.1.29
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://dotnetcli.azureedge.net/dotnet/Runtime/3.1.29/dotnet-runtime-3.1.29-win-x64.exe
  ██████████████████████████████  25.1 MB / 25.1 MB
Successfully verified installer hash
Starting package install...
Successfully installed

6 package(s) have version numbers that cannot be determined. Use "--include-unknown" to see all results.

Removing the older (6.0.302) version of the .NET SDK stopped it from continually trying to reinstall the 401 version. It does seem like winget should be smart enough to skip installing versions that are already installed.

I'm back up to five packages that winget wants to reinstall every time I run it.
Name                                 Id                                     Version       Available     Source
--------------------------------------------------------------------------------------------------------------
Microsoft Edge                       Microsoft.Edge                         106.0.1370.52 107.0.1418.24 winget
WinMerge <IP_ADDRESS> x64               WinMerge.WinMerge                      <IP_ADDRESS>     <IP_ADDRESS>     winget
Visual Studio Build Tools 2022 (2)   Microsoft.VisualStudio.2022.BuildTools 17.3.5        17.3.6        winget
Microsoft .NET SDK 6.0.305 (x64)     Microsoft.DotNet.SDK.6                 6.0.305       6.0.402       winget
NVIDIA GeForce Experience <IP_ADDRESS> Nvidia.GeForceExperience               <IP_ADDRESS>    <IP_ADDRESS>    winget
5 upgrades available.

Still getting this problem with Microsoft.SQLServerManagementStudio
Nome                                                     ID                                  Versão   Disponível Origem
-----------------------------------------------------------------------------------------------------------------------
Microsoft Visual C++ 2013 Redistributable (x64) - 12.0.… Microsoft.SQLServerManagementStudio < 16.5.3 18.12.1    winget
Microsoft Visual C++ 2013 Redistributable (x86) - 12.0.… Microsoft.SQLServerManagementStudio < 16.5.3 18.12.1    winget
2 Atualizações disponíveis.
2 package(s) have version numbers that cannot be determined. Use "--include-unknown" to see all results.
PS C:\Users\user> winget --version
v1.4.2161-preview

I don't even have the SSMS installed anymore but it still suggests as an update for some MS components. Strangely this doesn't happen on my other win 11 machine.

I don't even have the SSMS installed anymore but it still suggests as an update for some MS components. Strangely this doesn't happen on my other win 11 machine.

That's probably because some old versions of Microsoft.SQLServerManagementStudio manifests inappropriately list all dependent bundled packages ARP entries (i.e. Microsoft Visual C++ 2013 Redistributable in this case) like in this pr
These inappropriate entries were removed later. So if the winget source is updated to latest, winget should not recognize Microsoft Visual C++ 2013 Redistributable as Microsoft.SQLServerManagementStudio.

That's probably because some old versions of Microsoft.SQLServerManagementStudio manifests inappropriately list all dependent bundled packages ARP entries (i.e. Microsoft Visual C++ 2013 Redistributable in this case) like in this pr
These inappropriate entries were removed later. So if the winget source is updated to latest, winget should not recognize Microsoft Visual C++ 2013 Redistributable as Microsoft.SQLServerManagementStudio.

So I've just updated winget to the latest preview version, but I'm still getting SSMS as an upgrade.
Microsoft Visual C++ 2013 Redistributable (x64)… Microsoft.SQLServerManagementStudio < 16.5.3      18.12.1       winget
Microsoft Visual C++ 2015-2022 Redistributable … Microsoft.VCRedist.2015+.x64        14.34.31823.3 14.34.31931.0 winget
Microsoft Visual C++ 2013 Redistributable (x86)… Microsoft.SQLServerManagementStudio < 16.5.3      18.12.1       winget
3 Atualizações disponíveis.
2 package(s) have version numbers that cannot be determined. Use "--include-unknown" to see all results.
PS C:\Users\user> winget --version
v1.4.3132-preview

What else do I need to upgrade in order to fix it?

I meant to update winget source by doing winget source update
If that still does not solve the issue, I'd suspect the tracking catalog under C:\Users<user>\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db contains old manifests.

I meant to update winget source by doing winget source update
If that still does not solve the issue, I'd suspect the tracking catalog under C:\Users\AppData\Local\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Microsoft.Winget.Source_8wekyb3d8bbwe\installed.db contains old manifests.

I've deleted the installed.db file ranwinget source update and it worked, thank you.
Do you if the problem could return in case I install ssms again?

Do you if the problem could return in case I install ssms again?

It should not. This issue is caused by inappropriately authored manifests. This should not happen again as long as the manifests are correct (and they are today).

I still have an issue with side-by-side installation of MSSQL Server Management studio.
Winget upgrade --all installed the new version 19.1 but left the old version 18.12.1 as-is. When I re-issue Winget upgrade --all it installs Version 19.1 again.
Winget upgrade comes down to this:
Name                                   ID                                  Version          Verfügbar      Quelle
-----------------------------------------------------------------------------------------------------------------
Microsoft SQL Server Management Studi… Microsoft.SQLServerManagementStudio 18.12.1          19.1           winget

I wanted to remove the old version so I searched first usind winget list --name SQL.
Name                                             ID                                     Version     Verfügbar Quelle
--------------------------------------------------------------------------------------------------------------------
Microsoft SQL Server Management Studio - 18.12.1 Microsoft.SQLServerManagementStudio    18.12.1     19.1      winget
Microsoft SQL Server Management Studio - 19.1    Microsoft.SQLServerManagementStudio    19.1                  winget

I issued the command winget uninstall Microsoft.SQLServerManagementStudio -v 18.12.1 --verbose and now both versions have been uninstalled. This was the output:
Gefunden Microsoft SQL Server Management Studio [Microsoft.SQLServerManagementStudio]
Paket-Deinstallation wird gestartet...
Erfolgreich deinstalliert

Now winget list --name SQL does not list anything any more.
Should I create a separate issue for this? I could not find a matching one.

We've got several specific issues related to side-by-side installed packages, and we're working on this class of problems in the WinGet 1.8 preview.
This issue was resolved originally with updates to the manifest. I'm closing this issue since most of the comments are unrelated to the original issue, and we have many other open issues that more clearly describe specific instances of behaviors we need to fix.


Problems testing with Ransack

Hello,
Thanks for the awesome project, it's a lot of fun to build components in this way, and so far I'm a big fan. I've been running into a testing issue, though, and I'm not entirely sure if I'm just doing something wrong (probably), but when I try to use Ransack in a component, and then test that component, I get a ActionController::UrlGenerationError error raised. A simple example is the following component:
module Clients
  class TableComponent < ApplicationComponent
    def initialize(clients:, q:)
      super()
      @clients = clients
      @q = q
    end

    def template
      render(NewTableComponent.new(items: @clients)) do |table|
        table.column do |column|
          column.title { helpers.sort_link(@q, :name) }
          column.item(&:name)
        end

        # More attribute columns removed for brevity...
      end
    end
  end
end

...and the following test:
require 'test_helper'
require 'phlex/testing/capybara'
require 'phlex/testing/rails/view_helper'

module Clients
  class TableComponentTest < ActiveSupport::TestCase
    include Phlex::Testing::Capybara::ViewHelper
    include Phlex::Testing::Rails::ViewHelper

    setup do
      @q = Client.ransack({})
      @clients = create_list(:client, 2)
    end

    test 'renders a table with clients' do
      render TableComponent.new(clients: @clients, q: @q)
    end
  end
end

When I get to the point where it renders the component, I get the following error:
ActionController::UrlGenerationError: No route matches {:q=>{"s"=>"name asc"}}

Are there some helpers that I could use in my tests to solve this issue or will I need to workaround this in some other way?
I appreciate any guidance you can provide. Thanks!

Oh, that's interesting. When you test a component like this, it's tested with a test controller. The test controller probably doesn't let you generate links like that. It might be better to use a traditional rails view test for this instead of testing the component directly.

I was able to do some more digging after I realized that it seems to work in ViewComponent. I think they're extending the test controller in a way to trick it into working. I made some minimal changes to my test here:
    def with_url(url, &)
      old_request_path_info = test_request.path_info
      old_request_path_parameters = test_request.path_parameters
      old_request_query_parameters = test_request.query_parameters
      old_request_query_string = test_request.query_string
      old_controller = defined?(@test_controller) && @test_controller

      path, query = url.split('?', 2)
      test_request.path_info = path
      test_request.path_parameters = Rails.application.routes.recognize_path_with_request(test_request, path, {})
      test_request.set_header('action_dispatch.request.query_parameters', Rack::Utils.parse_nested_query(query))
      test_request.set_header(Rack::QUERY_STRING, query)

      yield
    ensure
      test_request.path_info = old_request_path_info
      test_request.path_parameters = old_request_path_parameters
      test_request.set_header('action_dispatch.request.query_parameters', old_request_query_parameters)
      test_request.set_header(Rack::QUERY_STRING, old_request_query_string)
      @test_controller = old_controller
    end

    def controller
      @controller ||= ActionView::TestCase::TestController.new.tap do |controller|
        controller.request = test_request
      end.extend(Rails.application.routes.url_helpers)
    end

    def test_request
      @test_request ||=
        begin
          out = ActionDispatch::TestRequest.create
          out.session = ActionController::TestSession.new
          out
        end
    end

I got that code mostly from copying the code in ViewComponent's test helper module. I don't fully understand the code just yet, although I think I get the gist of it. Would this be something you'd be willing to add to the Phlex Rails gem? If not, I don't think it's a big deal, I know it's somewhat niche. I could just keep my own module in my code.
Thanks for the insight and I appreciate the work!

It might help to look at how the phlex-rails test helper does this. https://github.com/phlex-ruby/phlex-rails/blob/main/lib/phlex/testing/rails/view_helper.rb


Running "make containerized-test" always fails

Describe the bug
I can not run make containerized-test successfully at this time. It appears that shellcheck is checking a script ./tmp/get_helm.sh that is not under control of the "ceph-csi" source control.
I have never run the tests non-containerized so perhaps that is why I am hitting this? I do realize that the make rule in question is fairly new.
Environment details
n/a
Steps to reproduce
Run make containerized-test CONTAINER_CMD=podman and it fails every time for me, both on master and my working branch.

@nixpanic I am not sure why I am hitting this 100% but I am assuming you did not when you added this.

Looks like a simple fix might be to exclude the "./tmp" directory assuming nothing else is expected to appear there. Let me know if you want to try and figure out why this didn't affect you or if you want me to just go and update the script regardless.

hmm, I do not have the ./tmp/get_helm.sh script here... Running the containerized tests for all things I do with this repository.
I guess you ran ./deploy.sh in the repository? That is a script that you do not need to run :smile: You should be able to remove the ./tmp directory completely. If it was caused by deploy.sh, the script should be corrected to clean up after itself.

I don't remember running deploy.sh, at least not w/in the last 6 months. It might be old, stale, stuff from when I last worked on ceph-csi many months ago. Deleting the ./tmp dir prevented the failure and it did not appear to get re-created. Hopefully @Madhu-1 change will prevent it from happening to other people in the future.


Support for ChangeListener

Hi @mikebaum,
I implemented support for ChangeListener's for: JTabbedPane,  JSlider, JSpinner, SpinnerModel, AbstractButton, ButtonModel and JViewport. Would be nice to have that soon in RxSwing.
About the implementation:
Since there is no "common" interface for the two common methods addChangeListener and removeChangeListener of classes like JTabbedPane, JSlider, etc... I decided to use internally reflection to avoid duplicated code. Using reflection gives a little performance penalty, but in the seldom case of registering / unregistering change listeners, this should not be an issue.
I know this PR is competing the work and the two PR's from @samuelgruetter. It's based on the typical style of RxSwing and does not introduce refactorings, but comes "ready to use" with tests. So it's a quick solution to have support for ChangeListener now, without considering design improvements. What do you @samuelgruetter and @mikebaum think?
Best regards from Lucerne in Switzerland,
Peti

Opps... just saw that there is another issue regarding that topic in https://github.com/ReactiveX/RxSwing/issues/27 from @whymarrh.

@Petikoch @samuelgruetter
I think there are elements from both these pull requests #27 that make sense. However since this pull request is complete and I don't have a lot of time to work on the one started by @samuelgruetter I will review this one.
What I like about #27 is the addition of a generalization of the creation of an event stream from an event source. I think that could be a PR in and of itself and we could refactor all event sources to use this pattern. It removes a lot of code duplication. Would be even better with lambdas, but since we're bound to java 6 it's not worth discussing.

Hi @mikebaum, thanks for pointing me to https://docs.oracle.com/javase/tutorial/uiswing/events/eventsandcomponents.html (didn't know it). I just updated the PR. Best regards, Peti

Okay I finished my review. After addressing the comments, perhaps you can squash all your commits into one patch.

Thanks a lot @mikebaum for the review. I'll update the PR tonight or tomorrow. Best regards, Peti

@Petikoch Sounds good, thanks for the PR.

@mikebaum, the PR is updated.

Okay I finished my review. Just a few minor things to change. Once you make those updates, please squash all the commits into one commit and I'll merge it.
:+1:
Although I'm not a big fan of reflection, in this case, for the reasons you stated it is a reasonable compromise. We could potentially generalize and use this approach if we discover other event sources that do not share a common interface. It's a shame we don't have java 8, because then we could have used method references rather than reflection.

@Petikoch So everything seems fine... Can you just squash the 5 commits into one. Not sure if you've done that before on github, but it works and magically updates the PR. Here's a link that describes what to do. There are probably others.

Hi @mikebaum, I finally managed  to squash all changes into one commit and update my feature branch :-)

@mikebaum, after sleeping one night... the following idea came up... what do you think about putting just one fromChangeEvents method with a single parameter of type Object into SwingObservable?    I think this would be better.

@Petikoch I think it's okay the way it is. This way the users of SwingObserverable don't have to search which classes support ChangeListeners.

@Petikoch merged, good work!

@mikebaum thanks again a lot for the review!

Could we have a release that contains this 0.26 maybe?

This library doesn't seem to be under active development. It is setup to support one-button release though. Let me see if I can get a release out but the version-tool doesn't seem to like 0.x version patterns.

@kittinunf I've released 0.26.0 without problems. It may take some hours to show up on maven central.

Thanks @akarnokd for releasing!

@akarnokd Thanks for releasing that update. Just wondering where is the "release" button? I would have released it long ago, but alas life distracted me.
@Petikoch poke
@kittinunf @Petikoch I have outstanding PRs on this project, would one of you like to review them. Also, I laid out a bunch of issues that I consider the minimum to release a v.1 of this project. It's not particularly challenging code, but necessary. Would any of you be interested in spending some hours finish this up? I'll work on it too, I could knock out a few of them quickly.

There is a release tab on GitHub where if you create a new release with tag vx.y.z, the CI will perform the release automatically.

Sorry for being late to the party. Thanks @akarnokd for releasing the 0.26.0, it works like a charm.
@mikebaum I am not sure I am qualified enough as I am writing Kotlin a lot these days. I'll do what I can do.


adds marker interfaces for composed types

related https://github.com/microsoft/kiota/issues/1411

@andrueastman solved conflicts


Moving modules into ouroboros-network-framework

This PR moves some modules into the ouroboros-network-framework package. Some are moved up, some moved down, but in each case the argument is that this is the more natural layer. The ouroboros-network-framework package is intended to cover opinionated decisions about how were are doing the lower levels of our network stack, without covering anything specific to the Ouroboros application level protocols. So we move generic functionality from ouroboros-network down. We also move overly-specific functionality from typed-protocols upwards, which makes those lower level libs more reusable.

@dcoutts judging from commit messages I agree with the proposed changes, but moving Ouroboros.Network.Socket module now will cause conflicts in @avieth connection manager and my coot/snockets stuff.  Can we wait with this PR until we resolve these?

Updated now that the snockets abstraction was merged. The snockets modules also move into the framework lib.


Multiple devices

It would be great if it were possible to use the same account on multiple devices.
I'd like to use my tablet when I'm at home, and my phone on the road.

There are 2 main technical concerns:

Synchronization. The game is not designed to be played in multiple device simultaneously. How do you solve that?
If the solution to point 2 is by logging out people, how to logout them? There's no indicative way that user has been logged out unless there are no activity for 3 days (default; configurable)

I hadn't thought about it that way.
If i remember correctly, with the original transfer system, the old device would be logged out instantly after importing on the new device.
Maybe such a system would work?
Unless I'm missing something, it's currently entirely impossible to switch to a different device, as SIFID is the only option, which obviously doesn't work anymore.

Right, which is the default behavior of NPPS4. How you can then differentiate between user who just simply want to transfer and user who wants to link with multiple devices?
That being said, you can transfer your account by going to Settings -> Transfer Passcode. To generate Transfer Passcode for your account, open the SIF2 Game Data Transfer banner in the in-game home screen.

Closing due to impossibility to implement this feature.


Always use varchar(max) or nvarchar(max) for SQL Server string command parameters

Fixes #5975

While the title and referencing issue says to use nvarchar(max) why all the test changes are converting to nvarchar(4000). In certain cases, its fine since it would be key. Is it case for all though?

@smitpatel It seems that SQL Client uses varchar(4000) or nvarchar(4000) if the data fits

Can we add a test for nvarchar(max) too?

Updated

providers fyi/beware?


Support Edge

Support for Edge would be nice. There's apparently a way to convert Chrome extension for Edge.

I tried to make it work using Microsoft's Edge Extension Toolkit but without success. I don't want to spend too much time and effort supporting Edge. I will close this issue but if anyone wants to give it a try, feel free to do so and submit a PR.


Issue Loading Mod

So after some testing, and fiddling with my mods as of latest update for dance of masks this mod will freeze during initial loading of the game and gives an endless load screen. No other mod seems to be doing this, but the moment i try to load this mod it will cause the endless load screen. So it seems to be this mod in question causing the issue, i can only assume it needs to be updated to the latest patch BUT thought i would mention it at least.

I can't reproduce this. Can you post your logs? Use this guide: https://github.com/cabarius/ToyBox/wiki/Diagnosing-Loading-Problems

https://gist.github.com/biship/9934b11777b89e9a9936ca4fbf24fd10
Below are my mods. All updated in the last 2 days, after the patch/DLC.
They all work fine, without ModMenu.
When I add ModMenu, then loading the game hangs at the usual ~70% progress bar on the loading screen.
Any tips or anything else you need? Thanks for the great mod BTW.
d----           6/14/2024  3:46 PM                0ToyBox0
d----           6/14/2024  4:07 PM                AlternateRacialTraits
d----           6/14/2024  1:59 PM                BubbleBuffs
d----           6/14/2024  3:39 PM                CharacterOptionsPlus
d----           6/14/2024  4:08 PM                ExpandedContent
d----           6/14/2024  2:00 PM                MicroscopicContentExpansion
d----           6/14/2024  4:07 PM                MiscTweaksAndFixes
d----           5/29/2024 10:32 AM                ModMenu
d----           6/14/2024  4:07 PM                PrestigePlus
d----           6/14/2024  2:00 PM                RandomEquipment
d----           6/14/2024  3:46 PM                RespecWrath
d----           6/14/2024  2:00 PM                TabletopTweaks-Base
d----           6/14/2024  1:59 PM                TabletopTweaks-Core
d----           6/14/2024  2:00 PM                TabletopTweaks-Flanking
d----           6/14/2024  2:00 PM                TabletopTweaks-Reworks
d----           6/14/2024  2:00 PM                WOTR_MAKING_FRIENDS
d----           6/14/2024  4:09 PM                WrathPatches

Looks like a conflict between expandedcontent and prestige plus - I just assume removing modmenu breaks prestige plus and then the game loads, but I was having the same issue the it crashed on expanded content.

Glad you like the mod @biship. Nothing obvious in the log file; I'm not very active these days but I'll try to take a look next week.
You can also look into the Expanded Content / Prestige Plus issue and let me know if that helps.

Just an update from my end---
Updated Prestige Plus, Dark Codex and Prestige Plus and seems to be working again! guess wasnt this mod specifically. Odd how those mods would load up without mod menu (wouldnt work without mod menu, but would still load up). But the updates seemed to fix it so all is good on my end.
I guess its just the pains of a new update.


Map imagery not downloaded in Firefox

The credits appear, but no map. (there should be a map down there)

Somewhat fixed. SVG/GeoJSON upload still causes a problem for Firefox.


[computer node K8S] build.sh execution error

Steps to reproduce the behavior (Required)

构建cn节点镜像

git clone https://github.com/StarRocks/starrocks
cd $your_path/starrocks/docker
./build.sh -b main

Expected behavior (Required)
build success
Real behavior (Required)
build failed
should comment these lines of the Dockerfile

yes, you are right. the script should adapt to the version branch.


Segfault using SNAPtoAdj on Twitter graph

I am trying to convert the Twitter graph to the adjacency list format but it always seems to end in a segfault.
I tried debugging with gdb and it seems to be happening in theutils/graphIO.hpp file when the words are being converted into edges.
Any help would be greatly appreciated.

This is the output I get when I try to debug using gdb
Program received signal SIGSEGV, Segmentation fault.
0x00000000004035e7 in _seq sequence::pack<long, long, identityF >(long*, bool*, long, long, identityF) ()
(gdb) bt
#0  0x00000000004035e7 in _seq sequence::pack<long, long, identityF >(long*, bool*, long, long, identityF) ()
#1  0x0000000000402863 in benchIO::stringToWords(char*, long) ()
#2  0x0000000000403e5a in edgeArray benchIO::readSNAP(char*) ()
#3  0x0000000000401cd3 in main ()

It's possible that you need to use a machine with more RAM.


Browser support (why not?)

Please (re)consider adding support for the browser.
I mean, why not? Chrome supports ANSI escape codes since version 60-something, so you don't need an alternative approach using styles or anything - the package probably just needs to be made compatible with the browser somehow?
If you try it in the browser currently, you get:
Could not find module in path: 'chalk/source/#ansi-styles' relative to '/node_modules/chalk/source/index.js'

I also tried kleur, colorette and picocolors - these can all load in the browser, though none of them actually work; they simply don't generate any color codes, which may have something to do with how color libraries detect support for colors on the console.
After trying numerous competing color libraries, I eventually discovered ansi-colors-es6 - the only library I've found that does work.
So 1 out of 5 libraries worked, heh - and it's the least popular of all of these libraries, go figure.
Perhaps this is not something most people care about?
But the first issue ever opened asked for it. It's been asked for, asked for, misunderstood, asked for, asked for and misunderstood yet again, so that's gotta make you think, right? 😏
I don't see anything in ansi-colors-es6 to suggest they've had to make any substantial concessions in terms of either bundle size or performance - it remains comparatively fast and small. (of course, it claims to be the fastest, which, oddly, most color libraries do... but either way, as long as it's not slow - if you're competing on marginal percentages - who cares.)
Bottom line, it's been solved, so there's a source of reference for the solution, and it doesn't look bad, slow, complex or ugly or anything. It appears to be a drop-in replacement for most of these color libraries, which are all pretty similar... but wouldn't it be nice if people could just install the most popular color library and have it actually work as expected?
Why not? 🙂

Looks like CodeSandbox does not support subpath imports. I would open an issue on their issue tracker instead: https://github.com/codesandbox

I think this a node-only feature? I had never heard of it.
The package doesn't load in stackblitz or codepen either, and doesn't produce a working ES6 module on skypack... so it seems whatever this feature is, it isn't very friendly to browsers.

You're better off downgrading to v4 and finding an alternative to this package. There isn't support or backwards compatibility here.

Hello,
I made a fork of chalk with node + browser support for internal projects I'm working on. See https://github.com/Jakobeha/chalk-cross. There are also a couple other features (exposes open and close escape codes)
It adds 16m escape codes to anything on the browser. If that doesn't work, you will have to change src/shims/chalk-common/index.js#getModelAnsi to handle 'custom' differently.

I eventually went with ansi-colors - it works in the browser.

CSS be like: What about me ?

I think this a node-only feature? I had never heard of it.

It's a Node.js feature, yes. But it's being adopted by build tools too.

so it seems whatever this feature is, it isn't very friendly to browsers.

It's not a browser problem, but rather these services not using the latest version of the build tools.

We cannot really remove this feature as it's required to be able to correctly provide a browser version of a sub-module we use in Chalk.
https://github.com/chalk/chalk/blob/b17d862944f9efea1ee12aaf80fe19ed5063a34d/package.json#L14-L15

I'm afraid I don't follow. You actually have color support detection code specifically for the browser - but if the module can't be loaded in the browser in the first place, what is this for?

It's not a browser problem, but rather these services not using the latest version of the build tools.

For the record, it didn't work with current versions of rollup and standard plugins.
I mean, I don't fully understand what's missing or whether it's on a roadmap for common bundlers somewhere in the future. I just picked a package, and it doesn't really seem to work anywhere the browser is involved.
(Maybe no one else cares? I'm building a reporter that will run both in the browser and on the CLI, so this was important to me. Either way, ansi-colors works fine and does the same thing, so... case closed as far as I care.)

Perhaps the key point in Sindre's comment was missed:
The problem you're running into has nothing to do with browsers. It has to do with your bundlers not supporting this feature of path resolution. Once whatever bundler you're using does (or you update) this will begin to work in the browser.


[v0.13.2] Incorrect GIT tag: https://github.com/crate-py/rpds/releases/tag/v0.31.2

It should be https://github.com/crate-py/rpds/releases/tag/v0.13.2, but now become https://github.com/crate-py/rpds/releases/tag/v0.31.2

Fixed.


tweet.search returning empty tweets list for some users

when using tweet.search, some users return an empty tweets list
example:
  const tweets = await rettiwt.tweet.search({
    fromUsers: ["AppFaves"],
  })

returns 0 tweets even though the account @AppFaves is active and does have tweets, this issue happens with other accounts as well but not all.

It's an issue with the search endpoint used to fetch tweets. The same behavior is displayed when trying to search for the tweets in the Twitter Web App:

You may use either the Rettiwt.user.tweets or Rettiwt.user.replies timeline to fetch the tweets, as an alternative.


fix feedback bar

Images 📸
Current situation:

Desired situation:

before someones merges this, I didn't test it, please send me a host request, this is my profile /user/joedoe


Release

Codecov Report

Merging #303 (4aa6468) into main (675e3ad) will decrease coverage by 0.24%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##             main     #303      +/-   ##
==========================================
- Coverage   33.65%   33.40%   -0.25%
==========================================
  Files          49       52       +3
  Lines        4722     4786      +64
==========================================
+ Hits         1589     1599      +10
- Misses       2982     3038      +56
+ Partials      151      149       -2

Flag
Coverage Δ

unittests
33.40% <60.00%> (-0.25%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/build/build.go
32.75% <ø> (-5.96%)
:arrow_down:

pkg/provider/pulumi/aws/aws.go
61.08% <0.00%> (-1.00%)
:arrow_down:

pkg/provider/pulumi/azure/containerapp.go
0.00% <0.00%> (ø)

pkg/provider/pulumi/common/image.go
0.00% <0.00%> (ø)

pkg/provider/pulumi/gcp/gcp.go
48.49% <0.00%> (-0.30%)
:arrow_down:

pkg/runtime/types.go
83.87% <ø> (ø)

pkg/provider/pulumi/common/build.go
55.17% <55.17%> (ø)

pkg/runtime/golang.go
75.48% <100.00%> (+0.48%)
:arrow_up:

pkg/runtime/java.go
63.01% <100.00%> (+4.44%)
:arrow_up:

pkg/runtime/javascript.go
80.43% <100.00%> (+0.65%)
:arrow_up:

... and 5 more

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


Most recent answer should remain in view

Currently, when I post for example ?- true. many times in a row, then I can no longer see the latest answers when they move out of view after a few interactions, and only exist way down below.
If possible, I would like to automatically see the latest answer, while still being able to easily post queries (i.e., keep the entrybox also in view and in focus).

Is it OK the way it is now?

I think now it is a bit more confusing than before, because we now see the most recent interaction, and then a log of all interactions below that, including the most recent interaction. In particular, when the very first query is posted, we now see the exact same interaction twice.
Since the most recent interaction is already logged as part of the entire history, would it not suffice to scroll down the history so that the most recent interaction is always in view when a new query is posted and an answer is shown?

Good idea! I removed that area and now it scrolls down automatically to the last answer, so it remains in view.

I think it's now excellent, thank you a lot!


Ports used in roles are not always configurable

All ports used in roles should be variables set by default in the role to their default standard values.
Inventories should be able to overwrite those values to custom ones when needed.
The README of the role shoul dlist the port and matching service with the default value so users know what they can change.
Thanks!

Indeed, it would be nice to define such parameters in defaults/main.yml for the roles. I would be pleased to review any PR implementing this.

Dear @strus38
Do you wish to also update other roles, or is rsyslog enough on your side ?

I did not find any other important yet... So let's move on.
Le mar. 13 oct. 2020 à 22:37, Ox<EMAIL_ADDRESS>a écrit :

Dear @strus38 https://github.com/strus38
Do you wish to also update other roles, or is rsyslog enough on your side ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bluebanquise/bluebanquise/issues/396#issuecomment-707994932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABM6DU7M42K5D5OXRKY6YHDSKS3B5ANCNFSM4SB2LJNQ
.


Investigate skipped TemplateParser tests

There are two tests in TemplateParserTests that are skipped on all tests runs because they fail.

Parse_ComplexSegment_OptionalParameterFollowingPeriod_LastSegment
Parse_ComplexSegment_OptionalParametersSeperatedByPeriod_Invalid

Both of these were fixed in #408 and are no longer skipped.

Cool beans!


Is an OS required for an XRCE client?

The examples presented in the documentation show linux and windows examples, but I am wondering if an XRCE client can be deployed bare-metal.  Is there an OS required for a client?

XRCE can be deployed bare-metal but it depends on clock_gettime() function and you should write some custom transport, check this: https://github.com/eProsima/Micro-XRCE-DDS-Client/blob/66df0a6df20063d246dd638ac3d33eb2e652fab2/CMakeLists.txt#L132

Thanks for addressing so promptly!


No support for resource tags in IAMPolicy

What happened?
I've noticed that the IAMPolicy MR has no support for tags (unlike say, IAMRole). I couldn't see an existing issue or PR for this so thought I'd note it quickly.
What environment did it happen in?
Crossplane version: 1.1
Provider: 0.17

@mf-lit I opened a PR to add this feature.
#910


So how do I implement lazy loading in the current version?

software
version

react-native
0.54

react-native-tab-view
0.77

I've looked at a lot of issues.
But I still don't know how to implement lazy this way?

I'm currently looking at this. The way I'm doing it (and I only started like 15 minutes ago) is to just re-calculate the index + routes when I get new date, and pass that in as navigationState.

Can you give me a thought?Because the page will call componentDidMount this component at the beginning.When and what do I judge?
To make an asynchronous network request.
thanks

you could you static getDerivedStateFromProps (or componentWillReceiveProps) and update your state from there. I'm not maintaining any state for the actual pager - I calculate it fresh after ever page change because it's pretty cheap.

Can you give me an example?I don't quite understand.
thanks

lazy={true}

oh no ,the current version has removed it

I'm puzzled as well as to how to achieve this. It looks like the documented function no longer works.

Justifications for removing lazy and an alternative approach are described here.
You can also use the focused prop as mentioned here along with a property you set only once:
...
renderScene = ({route, focused) => {
  ...
  return <Scene focused={focused} />;
  ...
}
...
class Scene extends Component {

  firstLoad = true;

  componentDidUpdate() {
    if (firstLoad && this.props.focused) {
      firstLoad = false;
      // do here what you used to do in componentWill/DidMount when lazy was available
    }
  }
}

Actually I found a way:
Tabs.js
_renderScene = ({ route, navigationState }) => {
    switch (route.key) {
      case 'clothes':
        return (
          <ImageGrid
            focused={navigationState.index === 0}
          />
        );
      case 'shoes':
        return (
          <ImageGrid
            focused={navigationState.index === 1}
          />
        );
      case 'other':
        return (
          <ImageGrid
            focused={navigationState.index === 2}
          />
        );
      default:
        return null;
    }
};

ImageGrid.js
export default class ImageGrid extends React.PureComponent<Props, State> {
  firstFocus = true;

  componentDidMount() {
    // for the first tab that will be in focused
    if (this.props.focused) {
      this.firstFocus = false;
      this.fetchItems(); // expensive method
    }
  }

  componentDidUpdate() {
    // when switching tabs, do this only the first time the tabs if focused to
    if (this.firstFocus && this.props.focused) {
      this.firstFocus = false;
      this.fetchItems(); // expensive method
    }
  }

  // ....
}

Why is a basic function so complicated?

I didn't understand why we have to make it complicated while this is a very basic function for TabView.

Example: https://snack.expo.io/@satya164/react-native-tab-view-lazy-load

@satya164 lazy prop is removed?


Create Packet Queue

For New Contributors
How to contribute
Description

If the bandwidth struggles to handle all the messages on one of the clients, we want to know when that happens on the server so we can pause the game and allow the client to catch up.
Intended Design

This should be done by queing the incoming packets on the client and when the queue reaches some threshold, the client sends the server a QueueFull message which will pause the game.
After a QueueFull message, when the queue becomes empty, send the server a QueueReady message to resume the game and packets.
Create a new class that will be used by CoopClient.OnNetworkReceive(). This new class will call packetManager.HandleReceive(peer, packet); using this new queue.
If another implementation would work better, please give an explanation in the discord or in the comments here. We are always open to different ideas.
Location

Create a branch based from development
Related Issues

N/A
Requirements

N/A
Additional information

N/A
Definition of Done

[ ] Class level comments exist for all new classes.
[ ] XUnit tests exist for every method that does not require the game to be ran.

Bottle neck is actually on the server side not the client side and queues already exist in the LiteNetLib library. #711 is more fit for this issue. Closing


Concurrent calls to client

Is it safe to call client.Read* methods concurrently?

yes, the client API is thread-safe: all exported methods acquire a mutex at some point.
See https://github.com/simonvetter/modbus/blob/master/client.go#L796 and https://github.com/simonvetter/modbus/blob/master/client.go#L706 for reads. Writes and other methods should all do something similar.
Let me know if you run into concurrency issues as that would definitely be a bug.

Good to know, thanks a lot!


Dev/dterletskiy/virtio integration

Could you please write comprehensive commit descriptions? Those are below any standards.
Also, @otyshchenko1, could you please take a look at the very first patch in this PR?

Hi @lorc.
Done


Update Postgresql to 9.6.21

We're currently on 9.6.15. 9.6.21 is the latest patch release for the 9.6 series. This resolves a large number of bugs, but also the following CVEs:
CVE-2020-25695
CVE-2020-25694
CVE-2020-25696
CVE-2020-14350
CVE-2020-1720

Done for 14.3


Timer/hardclock fixes

Use the RDTICK instruction instead of RDPR on sparc64 platforms: Apparently OpenBSD does allow reads from the TICK register. The RDPR instruction is only available to privileged software however so this could not work on any operating system (nonprivileged software must use the RDTICK instruction instead).
Add support for hardened systems with restricted access to the CPU cycle counter by reading system information when mbedtls_timing_hardclock is first called and using software clock if access is found to be restricted. This will still crash if access to the register is disabled afterwards. Currently supported: NetBSD/i386 and amd64
Use monotonic time instead of wall time throughout the timer/hardclock module

I don't understand why CLOCK_MONOTONIC isn't used directly on NetBSD? TSC is unreliable on a bunch of systems, it's much better to let the kernel pick the most reasonable option from the range of sources it supports (or indeed the user with sysctl kern.timecounter.)

The CI is complaining of the following errors:
timing.c:66:21: error: field ‘start’ has incomplete type
     struct timespec start;
                     ^
timing.c: In function ‘mbedtls_timing_get_timer’:
timing.c:353:9: error: implicit declaration of function ‘clock_gettime’ [-Werror=implicit-function-declaration]
         clock_gettime( CLOCK_MONOTONIC, &t->start );
         ^
timing.c:353:24: error: ‘CLOCK_MONOTONIC’ undeclared (first use in this function)
         clock_gettime( CLOCK_MONOTONIC, &t->start );
                        ^
timing.c:353:24: note: each undeclared identifier is reported only once for each function it appears in
timing.c:363:25: error: storage size of ‘now’ isn’t known
         struct timespec now;
                         ^
timing.c:363:25: error: unused variable ‘now’ [-Werror=unused-variable]
cc1: all warnings being treated as errors
Makefile:263: recipe for target 'timing.o' failed
make[1]: *** [timing.o] Error 1
make[1]: *** Waiting for unfinished jobs....
Makefile:17: recipe for target 'lib' failed
make: *** [lib] Error 2
^^^^test_gcc_opt: build/test: gcc -Os, full config: command make CC=gcc CFLAGS=-Os -std=c99 -pedantic -Wall -Wextra -Werror -> 2^^^^
(skipped because the build failed)

Full log: https://gist.github.com/bensze01/da8b24a393eebfd0066722cd30eb84e8

Thank you for pointing out the problems with _POSIX_TIMERS and _POSIX_MONOTONIC_CLOCK. I have taken a look on Debian, Solaris, NetBSD, OpenBSD and FreeBSD. Monotonic time is available on all of them but Debian sets _POSIX_MONOTONIC_CLOCK to 0 and OpenBSD (like macOS) sets _POSIX_TIMERS to -1. I concluded to implement runtime checks. The CI error was due to -std=c99 without defining _POSIX_C_SOURCE.

@niacat Could you clarify what your suggested change is?

Unfortunately, for personal reasons I won't be able to continue to work on the PR. You can close the PR now and resurrect the branch at any time.


Refactor code from class component to functional component using React hook

Now, the project was built with the class components, refactoring to using React hooks in the functional components is more preferable.

I refactored!

Totally refactored from class component to functional component!


feat: added operating_system for pools

Description
Added changes to inlcude operating system for pools.
Refer Issue-460

Release required?

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

The changes are added in one single PR here


flexible space for toolbars

Feature Request
Reading up on the material design guideline, I really want the flexible space for toolbar to be included on covalent.
Here is the link to how it acts:
https://material.io/guidelines/patterns/scrolling-techniques.html#scrolling-techniques-behavior
Basically, it is toolbar resizing the height of itself upon scroll and repositioning of the title or background along with it. Hopefully this is included in next version of your release!
Thank you.

Since we use the md-toolbar this might be a better feature request from @angular/material

Closing this since its a request for angular/material.


chore(@jest/types): mark collectCoverageOnlyFrom and coveragePathIgnorePatterns options deprecated

Reference https://github.com/facebook/jest/pull/1349#issuecomment-236773488
Reference https://github.com/facebook/jest/issues/1536#issuecomment-247236563
Related to #7185
Summary
Just thinking out loud. The documentation for collectCoverageOnlyFrom was removed in #1724. Looking at referenced comments, it seems like there was a plan to remove the option as well after adding collectCoverageFrom.
The regex / glob overlap between coveragePathIgnorePatterns and collectCoverageFrom was also questioned in the comments.
I am not against any of the options (; Only trying to understand: if they should be documented properly or should be marked @deprecated and removed with the next major?
Test plan
Green CI.

Closing in favour of #13156


Create all-new CI workflows

[x] Read CONTRIBUTING.md.

Updated existing workflows to better manage cache etc & created workflows to notify Discord server.
Tested changes:
N/A

Nice


Added 'selected' option to Legend

In V2 G2Plot exist option legendOption.selected
https://g2plot.antv.vision/en/docs/api/plots/line#legendoptionselected--

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: CAPCHIK sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for contribution @CAPCHIK .

why its not work

why its not work

Can you show, how you use it?

like this

why its not work

Can you show, how you use it?
just like this

@Henxun You should use XField value for key
On screenshot you are use 1, from type, but XField binds to year.
Try to use { "1998", false }


Errors with Angular 10 and Akita Entity Service 7

Is this a regression?
No
Description
Angular v10
Akita and Entity Service
<EMAIL_ADDRESS>/dev
└──<EMAIL_ADDRESS>
Svoboda-2:juno-admin-new patrick$ npm ls @datorama/akita-ng-entity-service
<EMAIL_ADDRESS>/dev
└──<EMAIL_ADDRESS>
Error while running ng build --prod or ng serve
ERROR in node_modules/@datorama/akita-ng-entity-service/lib/ng-entity-service-notifier.d.ts:24:21 - error TS2694: Namespace '"/project/node_modules/@angular/core/core"' has no exported member 'ɵɵFactoryDeclaration'.

24     static ɵfac: i0.ɵɵFactoryDeclaration<NgEntityServiceNotifier, never>;
                       ~~~~~~~~~~~~~~~~~~~~
node_modules/@datorama/akita-ng-entity-service/lib/ng-entity-service-notifier.d.ts:25:22 - error TS2694: Namespace '"/project/node_modules/@angular/core/core"' has no exported member 'ɵɵInjectableDeclaration'.

25     static ɵprov: i0.ɵɵInjectableDeclaration<NgEntityServiceNotifier>;
                        ~~~~~~~~~~~~~~~~~~~~~~~
node_modules/@datorama/akita-ng-entity-service/lib/ng-entity-service.loader.d.ts:22:21 - error TS2694: Namespace '"/project/node_modules/@angular/core/core"' has no exported member 'ɵɵFactoryDeclaration'.

22     static ɵfac: i0.ɵɵFactoryDeclaration<NgEntityServiceLoader, never>;
                       ~~~~~~~~~~~~~~~~~~~~
node_modules/@datorama/akita-ng-entity-service/lib/ng-entity-service.loader.d.ts:23:22 - error TS2694: Namespace '"/project/node_modules/@angular/core/core"' has no exported member 'ɵɵInjectableDeclaration'.

23     static ɵprov: i0.ɵɵInjectableDeclaration<NgEntityServiceLoader>;

Also, I see a new state management solution is recommended. Is there an equivalent to the akita entity service ?
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in
No response
Anything else?
No response
Do you want to create a pull request?
No

Downgraded to v6 and the error went away, still interested in knowing how to implement  something similar to the akita entity service in elf though


Upgrade 9.9.2963 -> 9.10.4067 causes server silent reboot

Version: 9.9.2963 -> 9.104067
Language: C#, .NET 8, AspNet Core latest stable
CP-SAT
Windows (on Azure App Services)
After upgrading from 9.9.* to 9.10.* AspNet Core Web Application Service reboots without any error message or exception. We accidentally upgraded library, considering that minor version upgrade should not affect anything. We were wrong. Downgrade make it work again.

can you send me the model that triggers the error. Windows silently crashes on floating point errors for instance.

can you check protobuf was correctly updated ?

No doubts, 3.26.1
I have the same versions on localhost and in Azure App Service. On localhost no problems, in a cloud silently crashes.

Microsoft Azure Support shared the stack trace:

Your app crashed because of System.ExecutionEngineExceptionYour app and aborted the requests it was processing when the overflow occurred. As a result, your app’s users may have experienced HTTP 502 errors.

This call stack caused the exception:
InlinedCallFrame
InlinedCallFrame
ILStubClass.IL_STUB_PInvokeGoogle.OrTools.Sat.SolveWrapper.Solve
Google.OrTools.Sat.CpSolver.Solve
<Next goes app service Solve method from the appliation>

....

Still, it works locally. So the issue is a configuration issue.

All the configuration above and still works with previous version.
Everywhere is Windows x64 platform. But I'm not sure about OS on Azure VM (App Service)

Probably I know the reason why it works on localhost. On localhost I run it always in debugger. And InlinedCallFrame never appears in debug mode as far I remember.

I cannot do anything until you send me something I can reproduce.

Sorry, I don't have anythig else. I already sent everything I have. The fact it happenes only in Cloud makes the task harder.
What is possible to do - is to compare sources of CpModel and CpSolver between abovementioned versions. Maybe something new and/or suspicious could be found in DIFF, because on the same environment all previous versions since 9.6...9.9 are working fine.

see the other issues I just closed, it was a missing updated visual studio version.

We are using Azure Pipelines to build the artifact.
Azure Agent named "windows-latest", according to the documentation doc it contains windows-2022 and visual studio 2022 (version: 17.9.34728.123). On localhost I have 17.8...
The full list of installed software on the agent is here.

Unless you can reproduce on a vanilla windows machine, I cannot do anything.
I do not have access to an azure server.
Le ven. 10 mai 2024, 09:59, Pavel Andreev @.***> a
écrit :

We are using Azure Pipelines to build the artifact.
Azure Agent named "windows-latest", according to the documentation doc
https://learn.microsoft.com/en-us/azure/devops/pipelines/agents/hosted?view=azure-devops&tabs=yaml
it contains windows-2022 and visual studio 2022 (version: 17.9.34728.123).
On localhost I have 17.8...
The full list of installed software on the agent is here
https://github.com/actions/runner-images/blob/main/images/windows/Windows2022-Readme.md
.
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4224#issuecomment-2104128708,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3IEF5FBLBGKGW32OUDZBR44RAVCNFSM6AAAAABHPLGJLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBUGEZDQNZQHA
.
You are receiving this because you commented.Message ID:
@.***>

We are using Azure Pipelines to build the artifact. Azure Agent named "windows-latest", according to the documentation doc it contains windows-2022 and visual studio 2022 (version: 17.9.34728.123). On localhost I have 17.8... (maybe this is the reason. I'll try to update) The full list of installed software on the agent is here.

I have the exact same issue with python ortools==9.10 CP-SAT on an azure pipelines runner with windows-latest (works on ubuntu-latest). Can't reproduce locally either with Visual Studio 16. Fix is also to downgrade to 9.9

https://developercommunity.visualstudio.com/t/Problems-with-publishing-after-upgrading/10587098?sort=active&topics=windows+10.0

I did many tests, please note:
I built (win-x64) in azure pipeline  and deployed to Azure Function / Azure App Service - same result: AccessViolationException.
then I built the same version (9.10) using the same toolset and deployed to linux version of azure function / azure app service - both works.
Apparently, the issue is in win-x64 platform binaries.

my 2 cents:
I've used VS 2022 Preview (up to date?) to built the binaries since few months ago VS 2022 was crashing in macro/template parsing (IIRC).
if VS 2022 Preview is shipped with an "advanced" redistributable VS runtime it main explain why azure base image can't load them...
I'll try to see If I can use a regular VS 2022 Community install to build (i.e. removed the preview from my Windows VM).

We have released 9.11. Please update.


🔧 Logging behavior to be less verbose under INFO and DEBUG

Codecov Report

Merging #163 (3854cdf) into master (ce38494) will decrease coverage by 0.03%.
The diff coverage is 81.25%.

@@            Coverage Diff             @@
##           master     #163      +/-   ##
==========================================
- Coverage   89.45%   89.41%   -0.04%
==========================================
  Files          11       11
  Lines        1195     1200       +5
==========================================
+ Hits         1069     1073       +4
- Misses        126      127       +1

Impacted Files
Coverage Δ

charset_normalizer/api.py
86.99% <80.00%> (-0.22%)
:arrow_down:

charset_normalizer/constant.py
100.00% <100.00%> (ø)

charset_normalizer/version.py
100.00% <100.00%> (ø)

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 ce38494...3854cdf. Read the comment docs.


movement-controls broken with Chrome version 97

Hi,
movement-controls broken with Chrome version 97, even with aframe 1.1.0
position is not set (always 0 0 0) and mouse is locked
work on Safari, Firefox and Edge
Thanks

Thank you for feedback @Turtleted21, can you provide a glitch please ? Perhaps i can help.

I try to reproduce my bug with Glitch without succes.
In my project bug appear when I have to many , with less   it work.
It appear only with  Chrome version 97

This seems to be an issue only on your project. I'm closing the issue. Please reopen if you reproduce your issue on a glitch example.


Update genius-lyrics extension

Description

I added a command, Search with Lyrics, to the extension. This command allows the user to find songs from the Genius Lyric API using remembered lyrics.
Screencast

This is the result of querying "strangers to love" using the new, Search with Lyrics, command:

This is the result of querying "strangers to love" using the existing, Search for Lyrics, command:

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Overall, I think this looks really good, and I like the idea of searching by lyrics. However, I think the command names are a bit confusing. Do you have any better name suggestions, or do you think the current names (with your changes) will be straightforward?
Also, would it be possible to update this with the native useFetch from @raycast/utils? I can also do it myself, but if you have the time, it would be easier. Removing an external dependency is always nice, especially when it's a large and somewhat unnecessary library like axios. Since your code doesn't appear to use any axios-specific features, a change to useFetch doesn't look too difficult.
Lastly, addressing the naming bit, do you think interweaving the results from both the lyric and title search would be a bad idea? I can likely write this, I'm just curious if it would be better to have two separate commands, or to use one unified command that searches both things.
Excellent work here! If you can make that slight change away from axios, that would be great, but otherwise I love this.

I took a look at useFetch, I should be able to rewrite the extension to use that instead. I'll get to work on that now.
As for the naming, I agree that "Search for Lyrics" and "Search with Lyrics" are a bit confusing, maybe "Find Lyrics by Title" and "Find Song by Lyrics" could be better? Let me know what you think.
About interweaving results, I find this idea interesting. After all, that's what the Genius site does;

Personally I like the two separate commands, but if you think combining them would be better, go ahead.
I also noticed some inconstancies between the two commands that missed before, so I'll clean those up as well.

For the purposes of efficiency, it might be smarter to keep them separate, so people can search for exactly what they want.
Yes, those names would certainly work much better. People can more easily differentiate those, and they can add aliases if it's needed. Personally, I think "search" should be used instead of "find," but otherwise your suggested names are perfect. I say that because none of the core Raycast commands used "find," but instead "search."
Thank you for migrating to useFetch, by the way. It's just nice to remove a dependency when you can.


CI: Install build deps to workaround internet issues

Signed-off-by: Johan Haals<EMAIL_ADDRESS>Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

This is caused by https://github.com/Automattic/node-canvas/issues/1945 so hopefully this PR will help us be able to build the node-canvas from source instead when the CDN for the binaries fails.


fix network error handling of LocalAdmissionController

What problem does this PR solve?
Issue Number: close #8074
Problem Summary:
two problems:

Grpc needs new a client context for each new reader/writer, otherwise may crash.
Network between GAC and LAC may got network error(like timeout). Need to handle it more gracefully, otherwise tiflash will crash.

two enhancements:

Delete handleBackgroundError(), because it's just a dummy wrap of LOG_ERROR
Add lock when call refill_token_callback to avoid RCQ been destroied when calling refill_token_callback

What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below)

start cluster with tiflash
continuously kill pd and check tiflash should not crash

[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests

/merge

/run-integration-test
/run-unit-test

/run-integration-test


Path must be a string. Received undefined

Not sure what I'm missing to get this to work.
Error:
app/ui/src/utils/api.js: Path must be a string. Received undefined

api.js
import { domain } from 'config'

.babelrc
{
  "env": {
   "development": {
     "plugins": [
        [ "babel-plugin-webpack-alias", { "config": "./webpack.config.js" } ]
      ]
    }
  }
}

webpack.config.js
var path = require('path')
var webpack = require('webpack')

module.exports = {
  devtool: 'eval',
  entry: [
    'webpack-dev-server/client?http://localhost:3000',
    'webpack/hot/only-dev-server',
    './src/index',
  ],
  output: {
    path: path.join(__dirname, 'dist'),
    filename: 'bundle.js',
    publicPath: '/static/',
  },
  plugins: [
    new webpack.HotModuleReplacementPlugin(),
  ],
  module: {
    loaders: [
      {
        test: /\.js$/,
        loaders: [ 'react-hot', 'babel' ],
        include: path.join(__dirname, 'src'),
      },
      { test: /\.styl$/, loader: 'style-loader!css-loader!stylus-loader' },
    ],
  },
  resolve: {
    root: path.resolve(__dirname),
     alias: {
       config: path.join(__dirname, 'config', process.env.NODE_ENV),
     },
     extensions: ['', '.js', '.styl'],
  },
  node: {
    fs: 'empty',
    net: 'empty',
    tls: 'empty',
    bufferutil: 'empty',
  },
}

I'm getting the same, oddly enough babel-plugin-webpack-alias seems to be doing the right thing: I thought line 26 was suspicious (const conf = require((0, _path.resolve)(process.cwd(), configPath));) but it's correctly resolving to the full filename of my webpack.config.js. The errors come further down, through babel-register, and even there the filename is set correctly for this and previous invocations (such as test.js). :confused:

Your problem is here:
  resolve: {
    root: path.resolve(__dirname),
     alias: {
       config: path.join(__dirname, 'config', process.env.NODE_ENV),
     },
     extensions: ['', '.js', '.styl'],
  },

I'm guessing NODE_ENV isn't defined, so path is receiving path.join(__dirname, 'config', undefined) and failing. The stacktrace lies a little. If this isn't the case, check other use of path` within your app.

Also you will get this error if any of your values are undefined, such as config:
    alias: {
      shared: path.resolve('shared/app'),
      traits: path.resolve('shared/app/styles/traits'),
      someValue: undefined // for whatever reason
    },

Error stack:
TypeError: /Users/tom/Documents/activepipe/activepipe.com/tests.js: Path must be a string. Received undefined
    at assertPath (path.js:8:11)
    at posix.isAbsolute (path.js:470:3)
    at PluginPass.CallExpression (/Users/tom/Documents/activepipe/activepipe.com/node_modules/babel-plugin-webpack-alias/build/index.js:55:51) # <<<
    at newFn (/Users/tom/Documents/activepipe/activepipe.com/node_modules/babel-traverse/lib/visitors.js:262:19)
    at NodePath._call (/Users/tom/Documents/activepipe/activepipe.com/node_modules/babel-traverse/lib/path/context.js:63:18)
    at NodePath.call (/Users/tom/Documents/activepipe/activepipe.com/node_modules/babel-traverse/lib/path/context.js:47:17)
    at NodePath.visit (/Users/tom/Documents/activepipe/activepipe.com/node_modules/babel-traverse/lib/path/context.js:93:12)

@grrowl  Thanks for taking a look. I'm going to try again later but I was able to solve my general problem by updating my test script ala http://stackoverflow.com/questions/33793504/using-webpack-aliases-in-mocha-tests
"test": "find ./src -name '*.test.js' | NODE_PATH=./src xargs mocha -r babel-core/register -r mock-local-storage"

Closing the issue as this doesn't seem to be an issue related to this plugin.


.innerText doesn't add \n newlines like browser does

Thanks for providing linkedom/worker export,  I'm using it within a Cloudflare WebWorker to separate navigation from content using Mozilla's Readability library. The last step is to get the innerText of the page content.  As far as I can tell, they do it for block level elements, <br>, and maybe others.
This is causing things like <li> lists to get merged together. My temporary fix is to append a whitespace to every text node, but this is causing issues like runon sentences.
I know converting to plaintext isn't a main use-case for your library. As you can imagine, these edge workers are very resource limited and JSDOM won't run there at all.
Thanks

I don’t understand the use case at all and I won’t work on this, but PR welcome (and tests) and also a use case maybe ….

I can see about making a PR. The use-case isn't SSR (as you've stated is the main goal of the project) it's basically web scraping, but on an edge worker with limited resources, making JSDOM a no-go.

Ok I threw together a PR. But in the end, I think what's probably best is for me to just build what I need using your TreeWalker implementation, and skip using innerText.
So feel free to close out if you prefer.

I like the PR, and it’s all green … I’ll recheck tomorrow and push it live if everything is fine, thank you 👋

it's in after minor adjustments, thanks again.


Rubocop command is not found.

[Enter steps to reproduce below:]

...
...

Atom Version: 1.0.0
System: Linux Mint
Thrown From: Atom Core
Stack Trace
Rubocop command is not found.
At When you don't install rubocop yet, Run `gem install rubocop` first.

If you already installed rubocop, Please check package setting at `Rubocop Command Path`.

undefined

Commands

Config
{
  "core": {
    "themes": [
      "atom-light-ui",
      "atom-dark-syntax"
    ],
    "disabledPackages": [
      "ruby-slim",
      "ruby-slim",
      "ruby-slim",
      "linter"
    ]
  },
  "editor": {
    "invisibles": {},
    "showIndentGuide": true,
    "showInvisibles": true,
    "softWrap": true,
    "softWrapAtPreferredLineLength": true
  }
}

Installed Packages
# User
linter-rubocop, v0.2.7
linter-ruby, v0.1.6
rubocop-auto-correct, v1.0.0

# Dev
No dev packages

Please re-open this issue in the linter-rubocop repo.

Hmmm, it's weird that this was thrown from Atom core though. @izuzak @kevinsawicki

@mnquintana Yep, already added it to https://github.com/atom/notifications/issues/66. Thanks for the :bell:

The stack is weird. Not sure by it's their message. Because the stack doesnt have any paths in it, there is incorrect attribution.

Issue still occuring now.

#2 Have issue two.  Same Operational System


Add Jest@28 to the peer dependencies

I wanted to use jest-pact in a repo that is using Jest@28, but I couldn't, because the peer dependencies don't allow it.
I reviewed the changelog, and we're able to use Jest@28 with no further changes. So I bumped the version, and confirmed that the tests still pass.
I also corrected an import line so that all versions of pact-js supported by jest-pact will work (this fixes a bug, as the latest version of pact didn't work with the current version of jest-pact - not sure when this bug was introduced).
I also fixed up some outdated dependencies.

Also, I noticed that the master build was failing sometimes - looking in to it it's because of a mistake where the beforeAll of both the pact setup and the changing the timeout were at the same level, meaning the timeout wasn't always changed during the setup hook. I fixed that too.

Thanks @TimothyJones <3
Will get this out now


How to listen to clicked event with the InlineCalendar

How to listen to the clicked event programmatically. For my purpose, I need to open a Modal when a date is clicked.

its one of the most important missing feature for this awesome calendar, the hacky way I do it is I just make a custom event from the day picker pass it to the calendar then to the calendar you are using

has there been a solution?

use this
https://github.com/ravarage/svelte-calendar


sokol_app: MacOSX (OpenGL or Metal) sapp_desc.fullscreen doesn't work

Hi,
When I set sapp_desc.fullscreen to true, I get a window that is resized to cover the whole screen, but it still has a visible title bar, and the MacOS menu is also visible. When I click the "full screen" button on the window, it does go into "real fullscreen".
I found that if I call sapp_toggle_fullscreen() from the init callback, it works as expected. So, there is an easy workaround, but I am reporting this in case others encounter it.

Hmm strange, I cannot reproduce this here, the title- and menubar only slide into view when bumping the mouse against the top border.
I'm testing with the cube-sapp sample using the sapp-osx-xcode-debug build config, and setting .fullscreen = true; in sokol_main()).
What macOS and Xcode version are you using? I'm on the latest macOS 11.5 Beta (20G5023d) and Xcode 12.5.

I tried building cube-sapp (with .fullscreen = false) and got a very surprising result. When I build it with fips (config sapp-metal-osx-make-debug), I get the correct behavior. If I build it manually as described in the readme (which is awesome, BTW), I get my "bad" result.
I am attaching a screenshot of the top left corner of the screen to get an idea of what it looks like.

I will keep digging some more to see find what the difference is.
I am doing this on Catalina with XCode 12.2.  I can believe that Catalina vs Big Sur makes a difference, but given the result above, I think it is unlikely.

That's surprising indeed! One difference that comes to mind is that the sokol-samples are built as app bundles, and also have an Info.plist file (which is generated in cmake here: https://github.com/floooh/fips/blob/master/cmake/fips_osx.cmake#L90-L134). I'm not seeing any capability in the info.plist file which seems to related to fullscreen mode though.
Looking forward to what you find out :)

This is definitely it! (Now, don't laugh at this, while I have done some Windows and X11 GUIs, I have deliberately avoided learning about MacOSX GUI development, I just use it as a more comfortable Unix... 😄)
Here is how I was able to "fix" it:
./cube-sapp # Fullscreen does not work
# OK, fix it.
mkdir -p Contents/MacOS
./cube-sapp # Fullscreen is working now!!!

I don't know what the heck is going on! Notice that there is no Info.plist and I didn't even copy the binary to Contents/MacOS/, I just created an empty subdirectory...
Whatever this is, clearly it doesn't seem like a bug in sokol-app, so closing.

Well this behaviour is new to me as well, so ;)


How can I use UUID as a primary key in MySQL

Environment
Knex version: 0.16.5
Database + version: MySQL 5.7.19
OS: Mac OS
I create this migration:
exports.up = function(knex, Promise) {
  return knex.schema.createTable('users', t => {
    t.uuid('id').primary()
    t.string('name');
};

But when insert a new user, I got this error message
Field 'id' doesn't have a default value
I think, it happens because I use MySQL instead of PostgreSQL. So I should put some .defaultTo(), but what value I put inside the function? Or, can I use uuid as primary keys in MySQL ?

Closing as usage question.
I don't know if it is possible to do with mysql, but if it is you need to give default value for 'id' column, like incremental is always adding sequential value to it. Maybe this works:
t.uuid('id').primary().defaultTo(knex.raw('UUID()'));

@elhigu, Really, it is not possible to enter a default value for the UUID in Mysql. Even with knex.raw('UUID()').
So I create these :
So, I created this "workaround" based on this answer
exports.up = function(knex, Promise) {
  return knex.schema.createTable('users', t => {
    t.uuid('id').primary()
    t.string('name');
})
.then(() => knex.raw(onInsertTrigger('users')));

and into kinexfile I add this:
  onInsertTrigger: table => `
  CREATE TRIGGER before_insert_${table}
  BEFORE INSERT ON ${table}
  FOR EACH ROW
  SET new.id = uuid();
  `

I don't know if it's a good solution, but it works.
Maybe can help other people

knex.raw('UUID()') should be: knex.raw('(UUID())') (notice that UUID() is wrapped with ()!
See this article

@DHFW  Only possible on MySQL > 8.0.


Stale balances persist when switching accounts from certain routes

I've determined how to reproduce https://github.com/blockstack/stacks-wallet-web/issues/1336

sign in
go to receive page
switch accounts while still on that page
go back home
stale data is present

I spent some time this afternoon trying to debug it, and I have a few ideas -- but this is something that should be fixed before a release. I think ultimately it's related to this issue https://github.com/blockstack/stacks-wallet-web/issues/1048
Here is a video:
https://user-images.githubusercontent.com/11803153/125535034-404bd6d7-e384-457a-874a-da235dcbb3af.mp4

@aulneau Mine is indicating a different behavior
Attaching a video recording-do kindly view till the first 20 sec as I continued with testing with the remaining testing.
https://user-images.githubusercontent.com/38872890/125536515-1c2f0373-cef5-4887-9e1c-f6912555d569.mov


Add cspyce to automated nightly tests

Once we have tests to run, add actions to run automated nightly tests on either the GH test runners or our own test runner.

This has been done using our nightly test server


sprintf is marked as deprecated for MacOS Xcode 14.2

MacOS 13.0.1 Xcode 14.2 CppMicroServices v3.7.4
Configured as
mkdir build; cd build
cmake -G 'Unix Makefiles'
-DCMAKE_INSTALL_PREFIX=/opt/local
-DCMAKE_BUILD_TYPE=Debug
-DUS_ENABLE_THREADING_SUPPORT=ON
-DUS_BUILD_TESTING=1
-DBUILD_SHARED_LIBS=1
-DUS_BUILD_EXAMPLES=1
..
build: gmake 2>&1 | tee build.log
error:
Building CXX object tools/rc/CMakeFiles/usResourceCompiler.dir///third_party/jsoncpp.cpp.o
/local/build/CppMicroServices-3.7.4/third_party/jsoncpp.cpp:3869:3: error: 'sprintf' is deprecated: This function is provided for compatibility reasons only.  Due to security concerns inherent in the design of sprintf(3), it is highly recommended that you use snprintf(3) instead. [-Werror,-Wdeprecated-declarations]
sprintf(formatString, "%%.%dg", precision);
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX13.1.sdk/usr/include/stdio.h:188:1: note: 'sprintf' has been explicitly marked deprecated here
__deprecated_msg("This function is provided for compatibility reasons only.  Due to security concerns inherent in the design of sprintf(3), it is highly recommended that you use snprintf(3) instead.")
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX13.1.sdk/usr/include/sys/cdefs.h:215:48: note: expanded from macro 'deprecated_msg'
#define deprecated_msg(_msg) attribute((deprecated(_msg)))
^
1 error generated.
gmake[2]: *** [tools/rc/CMakeFiles/usResourceCompiler.dir/build.make:104: tools/rc/CMakeFiles/usResourceCompiler.dir///third_party/jsoncpp.cpp.o] Error 1
-Bob Green  bgreen at mathworks.com

note: solution should be to replace the use of jsoncpp with rapidjson. rapidjson is used by the rest of the cppmicroservices code base and is faster than jsoncpp.

I was mistaken, jsoncpp is used in both the resource compiler and DS source code generator tools. It's non-trivial to refactor and swap out jsoncpp for rapidjson. The path of least resistance is to upgrade jsoncpp to solve this issue.

Just FYI. My quick Mac-Only fix was to just call snprintf in place of sprintf.
The code looks like it is already mostly using snprintf so this was a one line
change.
-Bob

On Dec 20, 2022, at 8:58 AM, Jeff DiClemente @.***> wrote:
note: solution should be to replace the use of jsoncpp with rapidjson. rapidjson is used by the rest of the cppmicroservices code base and is faster than jsoncpp.
I was mistaken, jsoncpp is used in both the resource compiler and DS source code generator tools. It's non-trivial to refactor and swap out jsoncpp for rapidjson. The path of least resistance is to upgrade jsoncpp to solve this issue.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.


Misc improvements

This PR makes a few changes. Let me know if you'd like me to split this up into multiple, smallers PRs.

Adds a .gitignore file to avoid checking in unnecessary changes to the Xcode project.
Adds targets to the Xcode project for the examples (keyboard/mouse).
Fixes up the keyboard example to cleanup the virtual keyboard when the program exits.
Extracts constants used in kernel and userland into foohid_user_kernel_shared.h
Adds externally callable methods for opening/closing the user-client more cleanly.
Adds some more cleanup code to the user client so it gets properly removed from registry when finished.

 This makes the diff kind of hard to read on GitHub. Appending ?w=1 to the URL hides the whitespace changes from the diff, making it easier to read.
I'll be opening a separate PR to implement the setReport stuff.

Here's a branch adding setReport support. It's based on this branch, so I can't cleanly open a PR until this one merges.

Thanks! I reviewed the changed files, but I didn't test the new kext (I don't have a macOS install available ATM). @unbit, what do you think?

The patch seems good but i need to test it :) I will check it during the week-end

@unbit Cool, thanks. Would you like me to include the setReport changes in this PR or do that in a separate PR once this one merges?

This PR ended up mostly being aesthetic changes that aren't really important. I opened another PR (https://github.com/unbit/foohid/pull/9) to implement the setReport call, which is what I'm really interested in :smile:. I'm closing this PR for now.


Update to PyDID 0.3.x

This PR updates ACA-Py to use the soon-to-be-released PyDID 0.3.x. These updates aim to address the following issues through the updated PyDID features:

Reliance on unmaintained voluptuous library. PyDID now uses pydantic.
Ugly validation errors. These errors should now be significantly easier to understand due to being more succinct and descriptive.
Extensibility. Subclassing and extending PyDID objects to tighten or loosen validation on DID Documents is far simpler.

In addition to these benefits granted by the updated PyDID version, this PR also slightly modifies the DID Resolver interface to simply return a dictionary on resolve. Thanks to the rapid development of the DID spec, there is a wide variety of DID Documents with varying levels of conformance to the DID spec in the wild. By separating the resolution and deserialization steps, we elect to give the caller the responsibility of determining how strict or otherwise the document parsing should be or whether the document is even parsed at all.
For dereference, a "least common denominator" approach is taken. The document parsing is attempted with strict rules but, if validation fails, it will fall back to a simplified NonconformantDocument. By following this approach, we can take advantage of retrieving parsed VerificationMethods or Services when the document is spec-conforming while still being able to retrieve values by ID from non-spec-conforming documents.

Marking as draft until official release of PyDID 0.3.0

Codecov Report

Merging #1225 (c7938c5) into main (b538cb9) will decrease coverage by 0.00%.
The diff coverage is 88.46%.

@@            Coverage Diff             @@
##             main    #1225      +/-   ##
==========================================
- Coverage   98.41%   98.41%   -0.01%
==========================================
  Files         457      457
  Lines       25474    25477       +3
==========================================
+ Hits        25070    25072       +2
- Misses        404      405       +1

@TimoGlastra @domwoe: I don't think this will significantly impact anything but, since you've been around this code recently, just wanted to give a heads up


Mobile (phone) UI issues

The mobile breakpoint could use some love. Small changes such as locking the height/width would do wonders. As of now, frustrating to use because of this.

We'll help out with this one.