Could not load file or assembly Microsoft.ExtendedReflection.Reasoning

Description
This error shows up in the output window and build logs for a unit test project, whether it uses MsTest nuget packages or referencing the QualityTools.UnitTestFramework assembly.
Steps to reproduce
Click [Run All] in test explorer
Expected behavior
30 tests discovered and all are executed.
Actual behavior
30 tests show in the test explorer, but only two are solid colored. The other 28 are transparent, and the output window shows the following.
[3/26/2018 4:24:59 PM Informational] ------ Load Playlist started ------
[3/26/2018 4:24:59 PM Informational] ========== Load Playlist finished (0:00:00.049947) ==========
[3/26/2018 4:25:00 PM Informational] ------ Run test started ------
[3/26/2018 4:25:02 PM Warning] MSTestAdapter failed to discover tests in class 'My.Project.Tests.DefaultTraceLoggerTests' of assembly 'c:\source\My.Project\My.Project.Tests\bin\Debug\My.Project.Tests.dll'. Reason Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
[3/26/2018 4:25:02 PM Warning] MSTestAdapter failed to discover tests in class 'My.Project.Tests.ProvisionedSerialNumbersTests' of assembly 'c:\source\My.Project\My.Project.Tests\bin\Debug\My.Project.Tests.dll'. Reason Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
[3/26/2018 4:25:02 PM Warning] MSTestAdapter failed to discover tests in class 'My.Project.Tests.Pool.NumberPoolDefinitionTest' of assembly 'c:\source\My.Project\My.Project.Tests\bin\Debug\My.Project.Tests.dll'. Reason Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
[3/26/2018 4:25:02 PM Warning] MSTestAdapter failed to discover tests in class 'My.Project.Tests.Pool.RawNumberTest' of assembly 'c:\source\My.Project\My.Project.Tests\bin\Debug\My.Project.Tests.dll'. Reason Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
[3/26/2018 4:25:03 PM Informational] ========== Run test finished: 2 run (0:00:03.4164772) ==========

Environment
Windows 10 b1709
VS 15.6.2 Enterprise
Questions
I cant find this assembly anywhere on my workstation or a mention of it on the internet. I would have figured if its needed by Pex, that it would be in the vs 2017 folder with the pex assembly but its not.  Its not embedded in the Pex assembly either. Where is this assembly so I can satisfy the complaint?
Also
This is also present on the build server from the last build executed for this solution in September.  This shares the same silent failure behavior as issue 241 (deliberate non-linkage), but the fix for that does not work for this, and it has a different symptom.
2017-09-13T03:37:40.8427749Z Executing the powershell script: D:\anothertfbainstall\tasks\VSTest\1.0.30\VSTest.ps1
2017-09-13T03:37:41.2951807Z ##[debug]Calling Invoke-VSTest for all test assemblies
2017-09-13T03:37:41.5135835Z Working folder: d:\tfs\a2\22
2017-09-13T03:37:41.5135835Z Executing C:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe "d:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\My.Project.Tests.dll" "d:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter.dll" "d:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.dll" "d:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.Interface.dll" "d:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\Microsoft.VisualStudio.TestPlatform.TestFramework.dll" "d:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\Microsoft.VisualStudio.TestPlatform.TestFramework.Extensions.dll"  /EnableCodeCoverage /logger:trx /TestAdapterPath:"d:\tfs\a2\22\s"
2017-09-13T03:37:42.1063911Z Microsoft (R) Test Execution Command Line Tool Version 14.0.25420.1
2017-09-13T03:37:42.1063911Z Copyright (c) Microsoft Corporation.  All rights reserved.
2017-09-13T03:37:42.4339953Z Starting test execution, please wait...
2017-09-13T03:37:54.7581533Z Warning: MSTestAdapter failed to discover tests in class 'My.Project.Tests.DefaultTraceLoggerTests' of assembly 'd:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\My.Project.Tests.dll' because Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
2017-09-13T03:37:54.7581533Z Warning: MSTestAdapter failed to discover tests in class 'My.Project.Tests.ProvisionedSerialNumbersTests' of assembly 'd:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\My.Project.Tests.dll' because Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
2017-09-13T03:37:54.7581533Z Warning: MSTestAdapter failed to discover tests in class 'My.Project.Tests.Range.ResultRangeTests' of assembly 'd:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\My.Project.Tests.dll' because Could not load file or assembly 'Microsoft.ExtendedReflection, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
2017-09-13T03:37:54.7581533Z Warning: MSTestAdapter failed to discover tests in class 'My.Project.Tests.Pool.NumberPoolDefinitionTest' of assembly 'd:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\My.Project.Tests.dll' because Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
2017-09-13T03:37:54.7581533Z Warning: MSTestAdapter failed to discover tests in class 'My.Project.Tests.Pool.RawNumberTest' of assembly 'd:\tfs\a2\22\s\My.Project\My.Project.Tests\bin\Release\My.Project.Tests.dll' because Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
2017-09-13T03:37:54.8205541Z Warning: Input string was not in a correct format.
2017-09-13T03:37:54.8673547Z Warning: Input string was not in a correct format.
2017-09-13T03:37:57.9561943Z Attachments:
2017-09-13T03:37:57.9561943Z   d:\tfs\a2\22\TestResults\32287f44-bcba-4241-80d5-d1b909c36b13\TfsBuild_TFSBUILDSRV02 2017-09-12 23_37_43.coverage
2017-09-13T03:37:58.1121963Z Information: Additionally, you can try specifying '/UseVsixExtensions' command if the test discoverer & executor is installed on the machine as vsix extensions and your installation supports vsix extensions. Example: vstest.console.exe myTests.dll /UseVsixExtensions:true
2017-09-13T03:37:58.1745971Z ##[warning]No results found to publish.

@StingyJack can you please provide as with a simple sample repro solution?
Also this is not a dependency of the testplatform or of mstest. Looks like your test code has a dependency on this. If that is you will have to take it up with the people who own whichever dll that has this dependency. You can simply open all your test dlls in Ilspy or some other disassembler and see who is taking this dependency and follow up with them

@StingyJack any updates here?

We have the same issue. Unfortunately if i create a new solution all tests are executed just fine.

Also this is not a dependency of the testplatform or of mstest. Looks like your test code has a dependency on this. If that is you will have to take it up with the people who own whichever dll that has this dependency. You can simply open all your test dlls in Ilspy or some other disassembler and see who is taking this dependency and follow up with them

Per ILSPY, it Is referenced by Microsoft.Pex.Framework.

@peltco - that looks like a different stack trace and symptom.

@StingyJack hi, so this is not a VsTest or MsTest issue.
It mostly has something to do with the VS installation on you machine.
I could find this dll in my VS installation on my box. Screenshot for reference.

Try reinstalling Visual Studio. (Also this is present in VS enterprise only, what flavor do you have?) And if the issue persists please log an issue in the developer community.

I have Enterprise VS.   I'm missing this file in the C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\ReferenceAssemblies\v4.5 folder, where VS is referencing Microsoft.Pex.Framework.   It is not using any of the locations you show - which I also have, and have the right file set in. I must have either not waited long enough for the search or mistyped the assembly name.  I'll try fixing the ref when I get back to this project. Thanks for the point in the correct direction.
Pex/Intellitest is part of the Microsoft Test suite, and this is the place where the issue is logged.  I cant use Dev Comm effectively anymore - not that it ever really worked.

Adding the reference to the PEX assembly by going to the Assemblies -> Extensions dialog does not fix the problem.

Adding the references by browsing to the folder you listed does address the exception and allows the tests to be discovered by VS.
It does not correct the same problem on the TFS build.
2018-04-02T16:59:42.0244113Z Executing the powershell script: D:\agent006\tasks\VSTest\1.0.32\VSTest.ps1
2018-04-02T16:59:42.5236145Z ##[debug]Calling Invoke-VSTest for all test assemblies
2018-04-02T16:59:42.7264158Z Working folder: D:\agent006\_work\59
2018-04-02T16:59:42.7264158Z Executing C:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe "D:\agent006\_work\59\s\MyProject\MyProject.Tests\bin\Release\MyProject.Tests.dll"  /EnableCodeCoverage /logger:trx
2018-04-02T16:59:43.4440204Z Microsoft (R) Test Execution Command Line Tool Version 14.0.25420.1
2018-04-02T16:59:43.4440204Z Copyright (c) Microsoft Corporation.  All rights reserved.
2018-04-02T16:59:43.9276235Z Starting test execution, please wait...
2018-04-02T16:59:52.1488762Z Warning: MSTestAdapter failed to discover tests in class 'MyProject.Tests.DefaultTraceLoggerTests' of assembly 'D:\agent006\_work\59\s\MyProject\MyProject.Tests\bin\Release\MyProject.Tests.dll'. Reason Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
2018-04-02T16:59:52.1488762Z Warning: MSTestAdapter failed to discover tests in class 'MyProject.Tests.ProvisionedSerialNumbersTests' of assembly 'D:\agent006\_work\59\s\MyProject\MyProject.Tests\bin\Release\MyProject.Tests.dll'. Reason Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
2018-04-02T16:59:52.1488762Z Warning: MSTestAdapter failed to discover tests in class 'MyProject.Tests.Pool.NumberPoolDefinitionTest' of assembly 'D:\agent006\_work\59\s\MyProject\MyProject.Tests\bin\Release\MyProject.Tests.dll'. Reason Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
2018-04-02T16:59:52.1488762Z Warning: MSTestAdapter failed to discover tests in class 'MyProject.Tests.Pool.RawNumberTest' of assembly 'D:\agent006\_work\59\s\MyProject\MyProject.Tests\bin\Release\MyProject.Tests.dll'. Reason Could not load file or assembly 'Microsoft.ExtendedReflection.Reasoning, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified..
2018-04-02T16:59:55.1752956Z Passed   ConstructorTest
2018-04-02T16:59:55.1752956Z Passed   AddRangeValueMulti
2018-04-02T16:59:59.9021259Z Results File: D:\agent006\_work\59\TestResults\TfsBuild_TFSBUILDSRV02 2018-04-02 12_59_55.trx
2018-04-02T16:59:59.9021259Z Attachments:
2018-04-02T16:59:59.9021259Z   D:\agent006\_work\59\TestResults\422cc56b-08aa-422f-b7b9-ded9f5ba14d2\TfsBuild_TFSBUILDSRV02 2018-04-02 12_59_45.coverage
2018-04-02T16:59:59.9021259Z Total tests: 2. Passed: 2. Failed: 0. Skipped: 0.
2018-04-02T16:59:59.9021259Z Test Run Successful.
2018-04-02T16:59:59.9021259Z Test execution time: 15.4480 Seconds
2018-04-02T17:00:00.2765283Z Publishing Test Results...
2018-04-02T17:00:01.8053381Z Test results remaining: 2
2018-04-02T17:00:02.6945438Z Published Test Run : https://onPremiseTfsServer.company.com/tfs/DefaultCollection/TeamProject/_TestManagement/Runs#runId=8414&_a=runCharts

After looking in the proj file it has added relative hint paths for these assemblies. which is not going to work as the build server uses a different working directory. Changing those hint paths to use the absolute paths does seem to work but is clearly not optimal.
2018-04-02T17:16:25.1264414Z Total tests: 30. Passed: 30. Failed: 0. Skipped: 0.
2018-04-02T17:16:25.1264414Z Test Run Successful.

So the bug here appears to be that the place where visual studio and Tfs build are loading the Microsoft.Pex.Framework.dll from is not the correct location. I don't know if its an extra file, a bad GAC reference, or what, but that appears to be the defect.


GetSongPosition() and GetPropertyClass(?) doesn't work (0.5.2h)

Describe your bug here.
So if you try and get song position with GetSongPosition() or  getPropertyClass('Conductor', 'songPosition') will result a error, "mods/data/opposition/script.lua:3: attempt to call global 'GetSongPosition' (a nil value)" and if you use getPropertyClass, its the same error, I might be stupid or it's not just me.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Yes, I made a little changes in playstate for when you press 7 in the first song, you get loaded into a another.

you didn't show your script but my guess is you are calling a function before it's loaded, and getPropertyFromClass is misspelled

hha

Its getSongPosition() and not GetSongPosition()
I know I'm annoyed too...

Make sure you don`t capitalize the first letter...

I didn't, doesn't really matter because I downgraded to psych engine 0.4.2.


Profile GUID is not recognized by defaultProfile if hidden

Environment
Windows Build Version: 10.0.19041
Windows Terminal Version: 1.3.2651.0

Steps to Reproduce

Set hidden to true in profile list

{
  "guid": "{6b342e18-c43b-45ef-9ea9-149badcf55b9}",
  "name": "Ubuntu",
  "source": "Windows.Terminal.Wsl",
  "hidden": true
}

Set the default profile guid to hidden profile guid

"defaultProfile": "{6b342e18-c43b-45ef-9ea9-149badcf55b9}",

Save settings.json

Expected Behavior
Use the default profile and hide the profile in dropdown menu list
Actual Behavior
Unable to find GUID and return error below:

Additional information
Full settings.json
// This file was initially generated by Windows Terminal 1.3.2651.0
// It should still be usable in newer versions, but newer versions might have additional
// settings, help text, or changes that you will not see unless you clear this file
// and let us generate a new one for you.

// To view the default settings, hold "alt" while clicking on the "Settings" button.
// For documentation on these settings, see: https://aka.ms/terminal-documentation
{
    "$schema": "https://aka.ms/terminal-profiles-schema",

    "defaultProfile": "{6b342e18-c43b-45ef-9ea9-149badcf55b9}",

    // You can add more global application settings here.
    // To learn more about global settings, visit https://aka.ms/terminal-global-settings

    // If enabled, selections are automatically copied to your clipboard.
    "copyOnSelect": false,

    // If enabled, formatted data is also copied to your clipboard
    "copyFormatting": false,

    // A profile specifies a command to execute paired with information about how it should look and feel.
    // Each one of them will appear in the 'New Tab' dropdown,
    // and can be invoked from the commandline with `wt.exe -p xxx`
    // To learn more about profiles, visit https://aka.ms/terminal-profile-settings
    "profiles":
    {
        "defaults":
        {
            // Put settings here that you want to apply to all profiles.
        },
        "list":
        [
            {
                "guid": "{6b342e18-c43b-45ef-9ea9-149badcf55b9}",
                "name": "Ubuntu",
                "source": "Windows.Terminal.Wsl",
                "hidden": true
            },
            {
                // Make changes here to the powershell.exe profile.
                "guid": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",
                "name": "Windows PowerShell",
                "commandline": "powershell.exe",
                "hidden": false
            },
            {
                // Make changes here to the cmd.exe profile.
                "guid": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}",
                "name": "Command Prompt",
                "commandline": "cmd.exe",
                "hidden": false
            }
        ]
    },

    // Add custom color schemes to this array.
    // To learn more about color schemes, visit https://aka.ms/terminal-color-schemes
    "schemes": [],

    // Add custom actions and keybindings to this array.
    // To unbind a key combination from your defaults.json, set the command to "unbound".
    // To learn more about actions and keybindings, visit https://aka.ms/terminal-keybindings
    "actions":
    [
        // Copy and paste are bound to Ctrl+Shift+C and Ctrl+Shift+V in your defaults.json.
        // These two lines additionally bind them to Ctrl+C and Ctrl+V.
        // To learn more about selection, visit https://aka.ms/terminal-selection
        { "command": {"action": "copy", "singleLine": false }, "keys": "ctrl+c" },
        { "command": "paste", "keys": "ctrl+v" },

        // Press Ctrl+Shift+F to open the search box
        { "command": "find", "keys": "ctrl+shift+f" },

        // Press Alt+Shift+D to open a new pane.
        // - "split": "auto" makes this pane open in the direction that provides the most surface area.
        // - "splitMode": "duplicate" makes the new pane use the focused pane's profile.
        // To learn more about panes, visit https://aka.ms/terminal-panes
        { "command": { "action": "splitPane", "split": "auto", "splitMode": "duplicate" }, "keys": "alt+shift+d" }
    ]
}

Thanks! This is actually a symptom of /dup #4139 :smile:

Eu estou com esse problema e não consigo me livrar e estou tentando de tudo.


Updating some dependencies

Just going through and updating some outdated dependencies

Codecov Report
Patch coverage has no change and project coverage change: -0.02 :warning:

Comparison is base (847a40d) 87.64% compared to head (ecd03c0) 87.63%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #382      +/-   ##
==========================================
- Coverage   87.64%   87.63%   -0.02%
==========================================
  Files          41       41
  Lines        8570     8570
==========================================
- Hits         7511     7510       -1
- Misses       1059     1060       +1

see 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Create a workflow to run e2e tests with hyperv provider on regular basis

We are now running pd-e2e tests with podman-installation using default wsl provider and pd e2e with podman nightly version also with wsl.
Extend or add new workflows/axes and run the tests with hyperv virtualization provider.

There are differences when running e2e tests with hyperv and wsl virtualization provider.
As for example, I am looking at test job: https://github.com/containers/podman-desktop-e2e/blob/e9f9af557dcea0a204cd3a016ed283f9221e915f/.github/workflows/desktop-e2e-test-job-windows.yaml.

In order to run podman with hyperv, we must use 0.0.2 version of pde2e-* images.
Podman needs to be installed from setup.exe (https://github.com/containers/podman-desktop-e2e/actions/runs/11322784457/workflow#L27)
Podman installed must be at least 5.2.3 (5.2.4)
On contrary of running the tests with wsl, the podman desktop and powershell session preparing the podman machine must be run with admin privileges (should be handled in pde2e-runner/podman)
Env_vars, as are now, are not passed into e2e tests due to above reason, same for secrets.
User mode networking is not relevant for hyperv

I am suggesting to create new workflow that would run on nightly basis using particular podman version not latest nightly so we test podman-desktop from main.


Add social media icons in the page and also align the text in the landing page

@rahulkhandait-sde
@anuragverma108
@PriyaGhosal
I want to work on this issue can you assign me this issue

In the Landing page the text and the button is not aligned properly
In the footer section terms and use and privacy policy has been repeated
Social media links has not been add yet in the footer of the page.

Thanks for creating the issue in WildGuard! The issue has been assigned to you.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Attach before & after screenshots in your PR for clarity.
Include the issue number in your PR description for better tracking.

Don't forget to follow @anuragverma108 – Project Admin & @rahulkhandait-sde - Project Collaborator  – for more updates!
We're excited to see your contribution as part of Hacktoberfest 2024! 🎉
Happy open-source contributing!


Support 3 12 & Update GHA Versions

This PR is relatively straightforward, just adding support for Python 3.12 in the testing matrix as well as updating the versions of various actions used in the test/lint workflows. While they could be two separate PRs, I did them in one because they both touch GitHub Actions, and because I think of them as both updating versions. The only thing I'm unsure about is whether gmpy2 should be explicitly required to be at least 2.2.0 for Python 3.12 or not, as technically 2.2.0 is the first version that supports 3.12/3.13.

Adding a constraint for python3.12 seems like it might be a good idea, particularly if the unconstrained gmpy2 runs into errors due to pip trying to use some older incompatible version. We can always tweak constraints later if we start to run into gmpy2 versioning issues.
That said, I think pip will try to use the latest compatible version so it might not make a difference in practice since pip will already try to use a fairly new version.  We probably would benefit more from running our tests against a continually-latest gmpy2 version instead of constraining it in the long run.
So, I think this looks good as-is (unconstrained) if it's passing w/out errors already.


Add requestTimeout transOption for grpc request

#204 grpc的实现，请审阅

Codecov Report

Merging #228 (153fd44) into main (ec72eed) will increase coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #228      +/-   ##
==========================================
+ Coverage   94.46%   94.65%   +0.18%
==========================================
  Files          48       48
  Lines        2513     2527      +14
==========================================
+ Hits         2374     2392      +18
+ Misses        109      107       -2
+ Partials       30       28       -2

Impacted Files
Coverage Δ

dtmgrpc/dtmgimp/utils.go
100.00% <100.00%> (ø)

dtmsvr/trans_class.go
88.13% <100.00%> (+0.20%)
:arrow_up:

dtmsvr/trans_status.go
96.58% <100.00%> (+0.18%)
:arrow_up:

dtmsvr/trans_type_saga.go
98.60% <0.00%> (+2.79%)
:arrow_up:

Continue to review full report at Codecov.

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


[bug] Ayaneo Air 1S oxp-sensors implementation broken

This was reported in my fork as a PR here https://github.com/aarron-lee/PowerControl/pull/2
Just re-posting the bug here for awareness

So i've been digging into this a bit. it seems the steam distro I use, chimeraos has an incomplete backport of oxp-sensors for my air 1s until kernel 6.12. I'd love to fix this for real, is there an established pattern for blacklisting a config dynamically? like based off of possibly falling back if oxp-sensors fails to the ec methods? or based off kernel version? Here's a bit of my research into the issue

It looks like there is a problem with oxp-sensors. There are too many uncertain factors in the proactive judgment, and it feels like there is no very effective method


Flyway fails to migration migration containing loopback linked server

Which version and edition of Flyway are you using?
6.1.2
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
We are not able to upgrade to the latest version due to another issue that is actively being worked on by the flyway team.
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using? (Type & version)
MSSQL 2014
Which operating system are you using?
Linux
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
In an environment where linked servers are setup to the same server (loopback, meaning there are multiple databases on a single instance with linked servers to eachother on the same instance) we executed flyway migrate with a repeatable migration containing a stored procedure that interacted with the the linked server.
What did you expect to see?
The migration to successfully apply
What did you see instead?

Note that logged-in as any user with permission to alter the stored procedure, including the user flyway executes as, we were able to alter the stored procedure without issue.  It is only when flyway does it that the issue manifests.
We dug deeper and found the following error in SQL Profiler
Cannot obtain the schema rowset “DBSCHEMA_TABLES_INFO” for OLE DB provider “SQLNCLI11" for linked server “[REDACTED]”.
The provider supports the interface, but returns a failure code when it is used.

Does this only occur for the repeatable migration in question?
Do the contents of this repeatable migration execute successfully through SSMS?
Do the contents of this repeatable migration execute successfully through JDBC outside of Flyway? (This will help identify if the issue is JDBC or Flyway related)

Could you try the first solution in this post which suggests copying the contents of the stored procedure into the end of the script
See also this post which mentions that distributed transactions don't work with loopback linked servers, so if your scripts contain these then that might be the cause
Could you also provide a script that reproduces this, along with the exact setup of your linked server?

Does this only occur for the repeatable migration in question?

No.  It occurred with every migration interacting with the linked servers.

Do the contents of this repeatable migration execute successfully through SSMS?

Yes

Do the contents of this repeatable migration execute successfully through JDBC outside of Flyway? (This will help identify if the issue is JDBC or Flyway related)

How does one go about testing this?

How does one go about testing this?

If you've used Java migrations, then that might be the quickest way. Java migrations give you access to the underlying JDBC connection, meaning you can execute statements directly over that connection. There is a sample Java migration in our documentation here. This will let you see which statements execute successfully and their exceptions when they fail
If you don't want to use a Java migration then you'll need a Java project that will achieve the same result as the Java migration. Here's a quick tutorial on setting one up. It will give you access to a Connection object, and then you can proceed as described in the Java migration case

See also this post which mentions that distributed transactions don't work with loopback linked servers, so if your scripts contain these then that might be the cause
Wrapping in a transaction appears to be the issue.  If we wrap it in a transaction through SSMS we get

Msg 3910, Level 16, State 2, Line 3
Transaction context in use by another session.

which highlights another flavor of flyway not really showing the actual error as mentioned in #3157.
Adding a script configuration file to run this outside of a transaction allows it to succeed.  So that aspect appears to be a SQL limitation and not flyway.


Initializing the Application

GIVEN a text editor web application
WHEN I run npm run start from the root directory
THEN I find that my application should start up the backend and serve the client

The application is able to boot up successfully (for localhost)!


[mkFit] Mid-way support for Phase2 initialStep in CMSSW

PR description:
This PR adds the ability to run mkFit on Phase-2 data / geometry. Only initialStep iteration is supported so far.

Add CMSSW Phase-2 hit and seed Converters

Properly support binary readout for Phase-2 strips where cluster-charge cut can
not be applied.

Improve standalone performance monitoring & debugging tools.

It requires also a new JSON file in cmssw/data repository:
https://github.com/trackreco/RecoTracker-MkFit/pull/3
PR validation:
In TTbar events, with PU:

Phase-2 (PU200, where mkFit is only used for initialStep track building): http://uaf-10.t2.ucsd.edu/~mmasciov/MIC/Phase2/MTV_TTbarPU200_mcRun4_withTiming/
Phase-1 (2024; no change, as expected): http://uaf-10.t2.ucsd.edu/~mmasciov/MIC/Phase2/MTV_TTbarPU_Run3_2024/

there is a merge conflict here now. Configuration/PyReleaseValidation/python/relval_2026.py
is it just a matter of resyncing the fork?

there is a merge conflict here now. Configuration/PyReleaseValidation/python/relval_2026.py is it just a matter of resyncing the fork?

Solved.

Closing in favor of 'compressed' PR #146.


IA -> Add Response ByteCount & Wait(sec) implementation

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Implementation of the Add Response - Wait(sec).vi
Implementation of the Add Response - ByteCount.vi
Implementation of the Test VIs

Why should this Pull Request be merged?

Implements the "Add Response API" and and corresponding tests

What testing has been done?

Implemented test VIs for the created API
Testing on the local DEV machine has been done

There are some tests where the "TestCase out" is unwired. Please wire them accordingly.

For improving readability, please wire the same constant to these checks (as far as I understood, they should have the same value), instead od using two different ones (see below):

Please remove the extra space at the end in the VI names for:

Resolve Seconds Wait .vi
Resolve ByteCount Wait .vi

[Recommend] Wherever possible, I suggest scaling the constant arrays being used, so that all elements are visible.

Parse Format String.vi -> this functionality is already present in the IA (Find Equation Variables.vi, in SE library). Please remove one of them, so that we don't have duplicated code.

Parse Format String.vi -> typo here:

Please adjust the names for the below marked inputs. Please use one of the below options:

"Item Ref In" and "Item Ref Out"
"Message In" and "Message Out"

Resolve Response Linkaging.vi -> I must admit I'm having a hard time understanding the VI, mostly because of the multiple nested Case Structures. To improve readability, I would like you to make the following changes:

the check for "empty Response" can be put in a separate subVI (throws error when empty), called right before Parse Format String, thus relying on the error chain to ensure correct execution
a similar solution for checking that the array size from Parse Format String is 1
the innermost functionality (see below) can be extracted in a subVI:

Taking into consideration the fact that Add Response is linking variables with channel refnums (which is different from the UI logic), is there a way to utilize this scripting node (or some of its subVIs) directly in the UI part(SE lib)? In the long run, it will be very beneficial to remove as much duplication as possible, yielding a more lean and robust code.

For improving readability, please wire the same constant to these checks (as far as I understood, they should have the same value), instead od using two different ones (see below):

Also please correct the typo in the constant names: "Response Parameters".

Please remove the extra space at the end in the VI names for:
* Resolve Seconds Wait .vi

* Resolve ByteCount Wait .vi

This is not recommended ?

Please remove the extra space at the end in the VI names for:
* Resolve Seconds Wait .vi

* Resolve ByteCount Wait .vi

This is not recommended ?

I don't know about phutanu, so I only speak for myself now, but unless there's a specific reason for that extra space (I thought it was a typo), I definitely like to see it fixed.

Resolve Response Linkaging.vi -> I must admit I'm having a hard time understanding the VI, mostly because of the multiple nested Case Structures. To improve readability, I would like you to make the following changes:
* the check for "empty Response" can be put in a separate subVI (throws error when empty), called right before Parse Format String, thus relying on the error chain to ensure correct execution

* a similar solution for checking that the array size from Parse Format String is 1

* the innermost functionality (see below) can be extracted in a subVI:
  ![image](https://user-images.githubusercontent.com/61490253/93198103-33b4f700-f755-11ea-93eb-fdc52615ece0.png)

Sure thing - I will refactor it to make it easier to understand

Please remove the extra space at the end in the VI names for:
* Resolve Seconds Wait .vi

* Resolve ByteCount Wait .vi

This is not recommended ?

I don't know about phutanu, so I only speak for myself now, but unless there's a specific reason for that extra space (I thought it was a typo), I definitely like to see it fixed.

Sure

Taking into consideration the fact that Add Response is linking variables with channel refnums (which is different from the UI logic), is there a way to utilize this scripting node (or some of its subVIs) directly in the UI part(SE lib)? In the long run, it will be very beneficial to remove as much duplication as possible, yielding a more lean and robust code.

There are ways, but this would not be a simple copy&paste operation. I think extensive refactoring would be required in the SE and this would be a separate task/US.

[Recommendation] VI naming.
In order to improve API code reuse and readability inside project libraries I would suggest not using shorting in VI names.
VI in question: "Check LC&RF Size Equality.vi"

[Recommendation] VI naming.
In order to improve API code reuse and readability inside project libraries I would suggest not using shorting in VI names.
VI in question: "Check LC&RF Size Equality.vi"

For example this VI can be reused in the "Add Command" node. And that's why I would like it to have a verbose name.


ENOENT errors unable to execute code until manually creating directories

Hi, I'm trying this plugin before it's added to the community repo! If this might be a persistent issue, I'd love to just bring it up before others run into it.
Env

MacOS 12.3.1
Obsidian 1.1.9
bb 1.0.169
obsidian-babashka 1.0.1

Repro steps

Download main.js and manifest.json
Copy both to .obsidian/plugins/obsidian-babashka relative to vault root
Enable plugin and set paths, /usr/local/bin/bb etc
Write a codeblock and execute from command palette

The electron console gives this error:
node:fs:585 Uncaught (in promise) Error: ENOENT: no such file or directory, open '/Users/kees/Library/Mobile Documents/iCloud~md~obsidian/Documents/Rink/.babashka/gen/vault_bindings.cljc'
    at Object.openSync (node:fs:585:3)
    at Object.func [as openSync] (node:electron/js2c/asar_bundle:5:1812)
    at writeFileSync (node:fs:2153:35)
    at writeVaultBindingsNs (plugin:babashka:95:31)
    at Object.editorCallback (plugin:babashka:170:11)
    at Object.e.mobileOnly.Mw.isMobile.e.checkCallback (app.js:1:1503548)
    at XN (app.js:1:1502943)
    at e.executeCommand (app.js:1:1504357)
    at eval (plugin:obsidian-kanban:77:14184)
    at e.o [as executeCommand] (plugin:obsidian-kanban:23:77313)

I tried a variety of things, changing the babashka dir to absolute, installing the plugin from the repo code and generating main.js with npm run build, moving the dir around, pre-populating files.
Solution
The minimum steps to make the plugin work were creating .babashka/gen ahead of time. Then it works as expected. So, seems like permissions and being unable to create the parent folders?
Could this still be an issue if users install the plugin within obsidian, instead of manually like I did?
If not, happy to close this and move on

Or, looking further, I'm not fluent with js but I don't see fs creating parents before calling (0, import_fs.writeFileSync)(bindingsNsPath, src);, if this is necessary..?

Oh wow, I must have created it manually and never thought about it further! Can you try creating it manually there and seeing if that solves it?

Final test, adding this snippet somewhere in the code resolves the issue!
const bindingsPath = import_path.resolve(vaultPath, settings.bbDir, "gen")
import_fs.mkdirSync(bindingsPath, {recursive: true});

Just pushed a 1.0.2 that essentially does that, can you check if it works for you please?

Btw, thank you so much for testing and finding this problem!

Yes, cleaned out the vault and works great! Thanks!


page.window.frameless = True After that, the window does not have rounded corners

Duplicate Check

[X] I have searched the opened issues and there are no duplicates

Describe the bug

Code sample
import flet as ft
def main(page: ft.Page):
page.window.frameless = True
page.window.width = 320
page.window.height = 480
page.theme_mode= ft.ThemeMode.SYSTEM
page.padding= 0
page.add(
    ft.WindowDragArea(content= ft.Container(border_radius=20),expand=True),
    ft.IconButton(ft.icons.CLOSE, on_click=lambda _: page.window.close())
)
page.update()

ft.app(target=main, assets_dir="assets")
To reproduce
aa
Expected behavior
No response
Screenshots / Videos
No response
Operating System
Windows
Operating system details
win11
Flet version
0.24.1
Regression
Yes, it used to work in a previous Flet version (please specify the version in additional details)
Suggestions
No response
Logs
No response
Additional details
No response

Can you share with which version of flet this used to work and share a screenshot of the resulting window?

Hello! May I chime in?
I opened an issue some time ago and it was closed, but the issue wasn’t fixed. It was 9 months ago. I didn’t reply because I missed the response somehow and forgot about it, terribly sorry. Issue number was #1110.
I think we’re dealing with the same thing here.
Here is the code to reproduce
import flet as ft

def main(page: ft.Page):
    page.window.width = 450
    page.window.height = 450
    page.window.bgcolor = ft.colors.TRANSPARENT
    page.bgcolor = ft.colors.TRANSPARENT
    page.window.frameless = True
    page.window.center()

    def open_dialog(e):
        page.dialog = ft.AlertDialog(
            modal=True,
            title=ft.Text('Alert Dialog'),
            content=ft.Text('This is an alert dialog'),
            actions=[
                ft.TextButton('OK', on_click=close_dialog),
                ],
            )
        page.dialog.open = True
        page.update()

    def close_dialog(e):
        page.dialog.open = False
        page.update()

    def close_app(e):
        page.window.close()

    page.add(
        ft.Container(
            content=ft.Column(
                controls=[
                    ft.ElevatedButton('Open dialog', on_click=open_dialog),
                    ft.ElevatedButton('Close app', on_click=close_app),
                    ],
                alignment=ft.MainAxisAlignment.CENTER,
                spacing=10,
                ),
            bgcolor=ft.colors.SURFACE_CONTAINER_HIGHEST,
            border_radius=30,
            alignment=ft.alignment.center,
            expand=True,
            ),
        )

ft.app(target=main)

How it looks on my end. Using python 3.12.6 and flet 0.24.1.


Easier workflow for port forwarding

Adding something similar to https://github.com/coreos/coreos-vagrant/blob/master/config.rb.sample#L90 would make it much easier to access services exposed via nodePort.

I assume this is related to the issue I was having - I spun up the single-node Vagrant instance, and went through the whole multi-tier app configuration, without much issue. However I was unable to access the guestbook app in my browser, and I assume it's related to Vagrant port forwarding. I tried a few different changes to the Vagrantfile networking, but am unable so far to be able to access it.
Some documentation (or perhaps there's some way to handle this automatically) would be helpful. Here's my kubernetes services, if someone could provide a pointer on how I need to update vagrant:
mwarkentin@Michaels-iMac kubernetes (master) $ kubectl get services
NAME           LABELS                                    SELECTOR                IP(S)       PORT(S)
guestbook      app=guestbook                             app=guestbook           <IP_ADDRESS>   3000/TCP
kubernetes     component=apiserver,provider=kubernetes   <none>                  <IP_ADDRESS>    443/TCP
redis-master   app=redis,role=master                     app=redis,role=master   <IP_ADDRESS>   6379/TCP
redis-slave    app=redis,role=slave                      app=redis,role=slave    <IP_ADDRESS>   6379/TCP

I tried the following changes, but was still unable to view it on http://localhost:3000:

config.vm.network "forwarded_port", guest: 3000, host: 3000
config.vm.network "forwarded_port", guest_ip: "<IP_ADDRESS>", guest: 3000, host: 3000

You should be able to reach the guestbook via the service NodePort on your vagrant node (no additional port forwarding needed).
The example should have set up a service with a NodePort, which you should be able to see with:
$ kubectl describe service guestbook
Name:                   guestbook
Namespace:              default
Labels:                 app=guestbook
Selector:               app=guestbook
Type:                   LoadBalancer
IP:                     <IP_ADDRESS>
Port:                   <unnamed>       3000/TCP
NodePort:               <unnamed>       30635/TCP
Endpoints:              <IP_ADDRESS>:3000,<IP_ADDRESS>:3000,<IP_ADDRESS>:3000
Session Affinity:       None
No events.

The NodePort will be randomly assigned from the range 30000-32676, so in your case the nodeport will likely be different.
Then from your local machine's browser, you should be able to see the guestbook via http://<IP_ADDRESS>:<nodeport>.

Also, for more detail on NodePort's & external services: http://kubernetes.io/v1.0/docs/user-guide/services.html#external-services

@aaronlevy Thank you, I'll give that a shot!

That worked! Might be helpful to have something about that in the docs. :smile:

Glad it worked. I just went through the upstream guestbook docs, and it does seem a little confusing because there is no mention of the nodeport (which is necessary for platforms that do not support cloud-provider loadbalancers). I'll open a ticket to add some documentation around this.


[AutoPR sql/resource-manager] Updating databases for new read scale property

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6902

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/5676


Quotations in user names raises errors

When projecting a user with the structure level y"x this error appears:
Error updating projector slides in the data cache: decoding projector elements json: error calling MarshalJSON for type json.RawMessage: invalid character 'y' after object key:value pair

I observed the error in our prod-logs. It seems, that the username is not json-encoded here. Are there more places where raw values are put into a json structure?

I am not sure if there are more places like it.
I was assuming, that usernames only contain letters that are valid in a JSON string.
Do we have a document that says what letters are allowed? Do we have a documentation like this for OS4?

Only implicit: Everything is allowed.

I used different search patterns to find similar errors and could only find the username. I hope #171 fixes all of them


Backpropagation of the loss function

Hi，
May I ask why torch.ones_like(loss) is used when doing backpropagation of the loss function ？
Thank you！

That makes no difference to directly use 'loss' for propagation in this project where loss is a number. However, the function torch.ones_like() is needed if loss is a tensor with size bigger than one. I am accustomed to such usage in that the preceding occasion occurs in some other tasks.


Provide an open source license

We should provide an open source license with this project so that usage cannot be contested.

Fixed in c4d0b579d0209b90357bfaade8bc0017c451dcd6.


Implement sparse SVD with function handles

In this PR we will use function handles to perform the iterative SVD of the half-infinite environment in CTMRG. To that end, a struct will be implemented that stores all necessary tensors for the environment contraction and on which tsvd! can dispatch.
Before doing that, a long-standing issue will be resolved where IterSVD would lead to singular vectors which, when supplied to FixedSVD, would not lead to element-wise converged environments and thereby making it not possible to use the iterative SVD during PEPS optimization.

I think if you change the initial guess to be a slight deviation from the first singular vector, you'll have better performance. I think you need a small component orthogonal to that vector to start generating new vectors in the krylov subspaces.

So in principle this should be a relatively complete implementation of "function handle CTMRG", however it's still untested and there are some things to discuss/fix:

somehow the way the adjoint action of HalfInfiniteEnv is defined leads to errors when calling tsvd! and I haven't figured out why
implementing this took quite a bit of extra compatibility methods and some rewriting, so I'm sure one can improve on some of the design choices

Maybe we could discuss, review and fix these things next week or so @lkdvos.

Had to add some extra renormalize_corner methods to make EnlargedCorner work with CTMRG. For some reason CTMRG doesn't want to be differentiated anymore (...well when does AD ever work right away anyway?) Tried to find the problem, but no success yet

Wow, only now I figured out that the problem with the iterative SVD and the :fixed mode leading to non-converging environments was due to wrong daggers in the relative gauge fixing contractions. (Big thanks to Bram on this one!)
Weirdly enough, the TensorKit tsvd and iterative SVDs with weird initial guesses retain some sort of symmetry, where the relative phases are just $\pm 1$; if, however, the initial guess is random, the relative phases will be complex which led the gauge fixing procedure to fail since the daggers were in the wrong place... Anyways, this is quite an important fix!

This should be finished up to the function handle SVD aspects - but those we'll anyway implement later on. Once the KrylovKit/TensorKit SVD interface is implemented, the function handles should also be very easy to add.


Create CONTRIBUTE.md

Is your feature request related to a problem? Please describe.
This is feature which help new contributor to understand how they can contribute project and how we can help this project more effect and do understand what this project really is and how we can get started. Let's get started with making this project more better 👍
Describe the solution you'd like
Build a Document CONTRIBUTE.md that help everyone to understand what this project is and how the collaborator get started with this and understand what the current project status and Where they need to take.

Hello @vipuldcoder, you can refer to cod-ed.github.io/simulate/contributing for the same:)


How to monitor memory usage

Hello!
In my project I need to monitor memory usage inside the Wasm VM's store. Is there a corresponding API to do this?
Thank you.

In the Rust wasmtime crate the type for this is the ResourceLimiter trait. That API is not bound in the C API which this repository uses, however, so currently this can't be done through Go. If you're interested though adding bindings for that API to the C API would be much appreciated

@alexcrichton Hello! Thank you for your answer
It seems we misunderstood. I read about ResourceLimiter.

It does the same as ResourceLimiter? https://github.com/bytecodealliance/wasmtime-go/pull/171/files
We need not only limiting but monitoring also. We want monitor :

how much memory was reserved (is about it memory.DataSize()???),
how much memory was consumed (consumed of reserved),
when will the next increase of memory.DataSize occur,

Yes, indeed, my question was not about how to limit the memory, but how to know at runtime how much memory is occupied, free, etc right now.

Ah so the API implemented in #171 is a less-powerful version of the ResourceLimiter trait. Despite its name that trait in Rust enables "listening" effectively to changes in memory. For example the trait gets an accurate snapshot of all memory used within a Store and notifications whenever memory is grown. The trait can optionally disallow some growth (hence the "limiter" in the name) but it could also only be there for accounting purposes (which I think is what you want here).
In that sense the ResourceLimiter trait is the way to do this, but that's only exposed in a limited fashion in the C API right now because otherwise it'd involve callbacks and such in C.


Using f-strings to make code more readable

I highly suggest the use of the f-string methods for string formatting in this notebook, it will make code such as

keys = get_s3_keys(bucket_name, prefix = product_name+'/'+
                                         str(year) + '/' + str(day_of_year).zfill(3)
                                         + '/' + str(hour).zfill(2) + '/OR_'+
                                         product_name + '-M3C' + str(band).zfill(2))

more readable with
keys = get_s3_keys(bucket_name, prefix=f"{product_name}/{year}/{day_of_year:03.0f}/{hour:02.0f}/OR_{product_name}-M3C{band:02.0f}")

Thanks for the great suggestion @ClariNerd617. I updated the notebook.


[stable/openebs] Update charts to 1.4.0 release

Updated image tags as per 1.4.0 release
Updated values.yaml
Updated README
Updated Chart.yaml
Updated clusterrole.yaml
Updated deployment-admission-server.yaml
Removed service-admission-server.yaml and validationwebhook.yaml

Signed-off-by: ChandanSagar<EMAIL_ADDRESS>[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/hold

/unhold

/ok-to-test

/test pull-charts-e2e

/hold

/test pull-charts-e2e


CIT Resewn is not detected for 1.19.1

When I updated my modpack to 1.19.1, MMC-export no longer found CIT Resewn from Modrinth and included the JAR instead.
Notably the JAR has not been updated on CurseForge or Modrinth during that period, so I don't know what could be the cause.
Using v2.5.1 of MMC-export.

I guess the problem is CIT Resewn doesn't count 1.19.1 as supported version, that's why mmc-export ignored it.

I can change the behavior to check only for major and minor version, and ignore patch, but I don't know if there will be any negative consequences

I would assume that most mods don't actually mark 1.19.1 on the jar yet in Modrinth. How is this one different?

I would assume that most mods don't actually mark 1.19.1 on the jar yet in Modrinth.

You think so?

Well, it will be easier to figure out what's the problem if I can reproduce the bug. Can you send me a link to modpack you're exporting?

Examples of mod versions that worked but don't mark as 1.19.1:

https://modrinth.com/mod/notenoughcrashes/version/4.1.6+1.19-fabric
https://modrinth.com/mod/fabrishot/version/1.8.0
https://modrinth.com/mod/yosbr/version/0.1.1

Modpack version where it worked: https://modrinth.com/modpack/fabulously-optimized/version/4.0.0-beta.12 (from https://www.curseforge.com/minecraft/modpacks/fabulously-optimized/files/3900792)
and where it didn't (I manually removed the jar and re-added CIT Resewn to manifest for now, but automatic conversion doesn't work): https://modrinth.com/modpack/fabulously-optimized/version/4.1.0-alpha.1 (from https://www.curseforge.com/minecraft/modpacks/fabulously-optimized/files/3904191)

Well, I was right. Examples you provided was successfully found by hash but, cit-resewn wasn't, just like lambabettergrass. But lambdabettergrass have 1.19.1 in game versions, so it was added automatically, but cit-resewn won't.

Must be fixed now

Works. I've asked the devs why the files are different.

Now https://modrinth.com/mod/notenoughcrashes/version/4.1.6%2B1.18.2-fabric got bundled in 1.18.2... :(

Lol

Must be fixed


Try to get WCT to actually retry on failures; report sauce connect issues

WIP

Closing; will pursue this another time.


Split MavenPublishResolvedVersionsJavaIntegTest

So we have smaller tests which we can run better in parallel

@bot-gradle test QFL

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test and merge

OK, I've already triggered a build for you.


Mac OS notification icons and labels are still 'Bitcoin-Qt' and orange 'B' image.

In one case both the icon and the application name are incorrect:

In another case, the app name is 'Omni Core' but the icon is still the orange Bitcion logo:

@msgilligan: is this still an issue with v0.2?

After moving on top of Bitcoin Core 0.13, this may no longer be an issue.


Chains of dependent transactions are going to be slow

At the moment if we have a (predictable) series of dependent transactions then we potentially incur a lot of overhead by waiting for each one to be validated in turn, i.e. one slot per step in the chain.
Multi-currency examples fall foul of this. At the moment we can have:

Create the tx out that we're going to spend to start the currency.
Create a first output locked by the monetary policy script.
Forge some tokens using the monetary policy script.
Actually set up our contract that uses the tokens!

That could be a 4-slot delay before things kick off, which is quite a bit.
It would be great if we could submit these as a batch and have them all validated together, although this might require support from the chain.

https://github.com/input-output-hk/cardano-ledger-specs/issues/768


README example segfaults

julia> autodiff(f1, Active(1.0))
julia: /workspace/srcdir/Enzyme/enzyme/Enzyme/Enzyme.cpp:103: void HandleAutoDiff(llvm::CallInst*, llvm::TargetLibraryInfo&, llvm::AAResults&): Assertion `0 && "illegal diffe metadata string"' failed.

signal (6): Aborted
in expression starting at REPL[5]:1
gsignal at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
abort at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
unknown function (ip: 0x7fdadb292748)
__assert_fail at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
HandleAutoDiff at /workspace/srcdir/Enzyme/enzyme/Enzyme/Enzyme.cpp:103
Allocations: 37717581 (Pool: 37701286; Big: 16295); GC: 43
[1]    20710 abort (core dumped)  julia

Do I need to build Enzyme locally first?

No, you shouldn't need to. I'm not quite sure why this is happening since the Julia binaries haven't been rebuilt, but there was a change in Enzyme proper updating the calling convention slightly that hasn't been reflected in Julia yet.
#22 should update the convention to match Enzyme proper, but we'll also need to push a new Enzyme binary packed for Julia @vchuravy

Should be fixed by https://github.com/JuliaRegistries/General/pull/24154

@vchuravy @wsmoses still getting this
(@v1.5) pkg> add Enzyme
   Updating registry at `~/.julia/registries/General`
######################################################################## 100.0%
  Resolving package versions...
Updating `~/.julia/environments/v1.5/Project.toml`
  [7da242da] + Enzyme v0.1.0
Updating `~/.julia/environments/v1.5/Manifest.toml`
  [7da242da] + Enzyme v0.1.0
  [7cc45869] + Enzyme_jll v0.0.1+5
  [61eb1bfa] + GPUCompiler v0.4.1

julia> using Enzyme; f1(x) = x * x; autodiff(f1, Active(1.0))
julia: /workspace/srcdir/Enzyme/enzyme/Enzyme/Enzyme.cpp:103: void HandleAutoDiff(llvm::CallInst*, llvm::TargetLibraryInfo&, llvm::AAResults&): Assertion `0 && "illegal diffe metadata string"' failed.

signal (6): Aborted
in expression starting at REPL[4]:1
gsignal at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
abort at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
unknown function (ip: 0x7f0e656cc748)
__assert_fail at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
HandleAutoDiff at /workspace/srcdir/Enzyme/enzyme/Enzyme/Enzyme.cpp:103
Allocations: 33246359 (Pool: 33230666; Big: 15693); GC: 41
[1]    3814 abort (core dumped)  julia

with
Julia Version 1.5.2
Commit 539f3ce943* (2020-09-23 23:17 UTC)
Platform Info:
  OS: Linux (x86_64-linux-gnu)
  CPU: Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-9.0.1 (ORCJIT, skylake)

Where did you get your version of Julia from?
On Thu, Dec 10, 2020, 23:32 McCoy R. Becker<EMAIL_ADDRESS>wrote:

@vchuravy https://github.com/vchuravy @wsmoses
https://github.com/wsmoses still getting this
(@v1.5) pkg> add Enzyme
Updating registry at ~/.julia/registries/General######################################################################## 100.0%
Resolving package versions...
Updating ~/.julia/environments/v1.5/Project.toml
[7da242da] + Enzyme v0.1.0
Updating ~/.julia/environments/v1.5/Manifest.toml
[7da242da] + Enzyme v0.1.0
[7cc45869] + Enzyme_jll v0.0.1+5
[61eb1bfa] + GPUCompiler v0.4.1
julia> using Enzyme; f1(x) = x * x; autodiff(f1, Active(1.0))
julia: /workspace/srcdir/Enzyme/enzyme/Enzyme/Enzyme.cpp:103: void HandleAutoDiff(llvm::CallInst*, llvm::TargetLibraryInfo&, llvm::AAResults&): Assertion `0 && "illegal diffe metadata string"' failed.signal (6): Abortedin expression starting at REPL[4]:1gsignal at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)abort at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)unknown function (ip: 0x7f0e656cc748)__assert_fail at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)HandleAutoDiff at /workspace/srcdir/Enzyme/enzyme/Enzyme/Enzyme.cpp:103Allocations: 33246359 (Pool: 33230666; Big: 15693); GC: 41[1]    3814 abort (core dumped)  julia
with
Julia Version 1.5.2
Commit 539f3ce943* (2020-09-23 23:17 UTC)
Platform Info:
OS: Linux (x86_64-linux-gnu)
CPU: Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
WORD_SIZE: 64
LIBM: libopenlibm
LLVM: libLLVM-9.0.1 (ORCJIT, skylake)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wsmoses/Enzyme.jl/issues/21#issuecomment-742961995,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABDO2UZ2DIO2KWBGAOX27DSUGOG7ANCNFSM4TJIIUEA
.

@vchuravy The Julia store :)
No - that's a binary I grabbed from releases.
Note that I just tried this ToT and got:
ERROR: AssertionError: rt <: Union{AbstractFloat, Nothing}
Stacktrace:
 [1] fspec
   @ ~/.julia/packages/Enzyme/Bmd0F/src/compiler.jl:88 [inlined]
 [2] thunk(f::typeof(f1), tt::Type{Tuple{Active{Float64}}})
   @ Enzyme.Compiler ~/.julia/packages/Enzyme/Bmd0F/src/compiler/thunk.jl:51
 [3] autodiff(f::Function, args::Active{Float64})
   @ Enzyme ~/.julia/packages/Enzyme/Bmd0F/src/Enzyme.jl:41
 [4] top-level scope
   @ REPL[4]:1

with
Julia Version 1.6.0-DEV.1442
Commit f2eb09e5da (2020-11-06 22:36 UTC)
Platform Info:
  OS: Linux (x86_64-linux-gnu)
  CPU: Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-11.0.0 (ORCJIT, skylake)

Different issue entirely I suspect. I'll try on 1.5.3 later.

(@v1.6) pkg> add Enzyme
   Resolving package versions...
Updating `~/.julia/environments/v1.6/Project.toml`
  [7da242da] + Enzyme v0.2.1
Updating `~/.julia/environments/v1.6/Manifest.toml`
  [fa961155] + CEnum v0.4.1
  [7057c7e9] + Cassette v0.3.3
  [34da2185] + Compat v3.23.0
  [864edb3b] + DataStructures v0.18.8
  [7da242da] + Enzyme v0.2.1
  [61eb1bfa] + GPUCompiler v0.7.3
  [692b3bcd] + JLLWrappers v1.1.3
  [929cbde3] + LLVM v3.5.1
  [a759f4b9] + TimerOutputs v0.5.7
  [7cc45869] + Enzyme_jll v0.0.3+0
  [8bb1440f] + DelimitedFiles
  [37e2e46d] + LinearAlgebra
  [a63ad114] + Mmap
  [1a1011a3] + SharedArrays
  [2f01184e] + SparseArrays
  [10745b16] + Statistics

julia> using Enzyme

julia> f1(x) = x * x
f1 (generic function with 1 method)

julia> autodiff(f1, Active(1.0))
ERROR: AssertionError: rt <: Union{AbstractFloat, Nothing}
Stacktrace:
 [1] fspec
   @ ~/.julia/packages/Enzyme/Bmd0F/src/compiler.jl:88 [inlined]
 [2] thunk(f::typeof(f1), tt::Type{Tuple{Active{Float64}}})
   @ Enzyme.Compiler ~/.julia/packages/Enzyme/Bmd0F/src/compiler/thunk.jl:51
 [3] autodiff(f::Function, args::Active{Float64})
   @ Enzyme ~/.julia/packages/Enzyme/Bmd0F/src/Enzyme.jl:41
 [4] top-level scope
   @ REPL[5]:1

I suspect my other issue was "wrong Enzyme version". Anyways, this appears to be a new issue - so it's potentially better to just start a new issue for it (unless you know the answer already!)

Yeah Enzyme.jl is not yet ready for 1.6 needs it's own variant of KernelCompiler.jl or  Cassette working on 1.6, so we never tested it. We should turn that AssertionError into a proper error message though.


fix: edit messages for agent create and agent generte spec

What does this PR do?
Edits and adds a bit more info to the --help of "agent generate spec" and "agent create"
What issues does this PR fix or reference?
@W-17417430@

I'll update the --help after you implement v2.  I was writing this for the Dec milestone, in case Vivek wants to run --help on the commands in his demo.  :)


 The data are also not correctly aggregated by category. This just needs to get ripped out into a separate function that can be called on the human-modified data once that time comes.

addressed! the end user was very pleased with the report.


Implement dns-01 challenge solver

We need to implement the dns-01 challenge solver. We should be able to reuse the providers as defined in github.com/xenolf/lego/providers. Some may need to be vendored and forked into our project in order to support the options we need.

Ref #11


Unable to compile Teal code with definitions for regular Lua modules

Teal supports requiring regular Lua file given that a special definition file is created.
For instance test.lua
local M = {}

local a
local b

a = 5
b = a + 2

M.c = b

return M

Definition file test.d.tl
local record test
    c: number
end

return test

When invoking command line tl it doesn't give any errors:
TealBug % tl check main/main.tl
========================================
Type checked main/main.tl
0 errors detected -- you can use:

   tl run main/main.tl

       to run main/main.tl as a program

   tl gen main/main.tl

       to generate main.lua

tl gen also works.
But when trying to build with Defold an error appears
/tlconfig.lua
	Line 3: variable 'a' has no type or initial value
	Line 4: variable 'b' has no type or initial value

Somehow the original Lua module (test.lua) gets treated as a Teal source which results in compilation errors
Test project:
TealBug.zip

What's your tlconfig.lua have this code?
return {
     gen_target = "5.1",
     gen_compat = "off",
     include = {"**/*.tl"}
}

Does your tlconfig.lua have this code?

@vlaaad  Yes

Hmm, I can reproduce the issue. Reported to cyan: https://github.com/teal-language/cyan/issues/43 The issue can also be solved with https://github.com/defold/defold/issues/8928


Get fresh account before updating

This PR updates the RotateConsoleCredentials and RotateCredentials functions so that they pull that lastest account CR data before updating.

This PR has been tested


Keyboard events (keypress, keyup, keydown, textinput) - see here

Keyboard events (keypress, keyup, keydown, textinput) - see here http://www.quirksmode.org/dom/events/keys.html

This issue was imported from Google Moderator
Moderator votes: +4
Issue added by Deryck on 2014-07-21
To vote this issue up or down, simply include +1 or -1 in your comment.

+1

Also please add KeyboardEvent.key
References:
https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/key
https://www.chromestatus.com/feature/4748790720364544
Doesn't appear to be on IE platform status page.

+1

+1

+1

+1

Also please add KeyboardEvent.key

➡ http://caniuse.com/#feat=keyboardevent-key

IE and Edge return incorrect strings for at least navigation keys eg. "Up" instead of "ArrowUp". Therefore the status should not be "Supported"

+1

These are all covered by the MDN BCD data now, e.g. https://caniuse.com/#feat=mdn-api_document_keyup_event


bootstrap: add version for prow

Is this only removing the authentication? How do you handle the possibility that the curl fails to return something useful?

Would it be better to "teach" PROW how to pull the build-harness itself (using its own credentials/hook into our org) so that the curl will not fail as much?

Prow doesn't have GitHub credentials unless you tag the repo as "private" but that then breaks the testing of our repos for reasons I've discussed in Slack.

Well that seems silly... 🙃
Is it possible to vendorize the build harness within PROW somehow? I do not have an issue with this PR, I am just trying to see if there is a way that we can bypass the rate limiting issues that we will likely be hitting.

You'd have to:

vendorize the build harness in the multicloud-operators-channel, or
update the openshift-ci-operator and ci-tools to allow the github creds to be provided without making the test results limited to only a few people

I approve of this to unblock, but I think we might need to figure out how to teach PROW to use this as a first class citizen of its implementation... whatever that entails.


⚠ Bofrost.de has degraded performance

In 6422830, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 787 ms

Resolved: Bofrost.de performance has improved in 7d69ba8.


Token based Auth

Hi all,
First thanks for this package. It's very much appreciated.
Second, I don't seem to be able to figure out the best way to go about authentication for clients using DRF token based auth. (Django OAuth Toolkit)
Any resource I've found on auth seems to focus on session based auth which I guess makes sense from a websocket point of view.
For my REST api I am sending an Authorization header with an access token as the value. What would you recommend I look into to get this auth working with the channels_api ?
Thank you

Hi there, unfortunately the websocket spec is against you here. The websocket spec doesn't allow for Authorization. There's a variety of ways to work around this but I would think of your websocket authorization as separate from your API authorization.
What I usually do is pass a token via a query arg like so: wss://myservice.com/websocket?token={some_token}
Reference: https://devcenter.heroku.com/articles/websocket-security#authentication-authorization

@daesu Did you ever get this working? If so how did you go about it?

@storerjeremy I ended up using a third party api (pusher.com) for my needs so I can't help you here sorry.


fix: use credenza provider

Fixes #279

Maybe the CredenzaProvider can be moved to the Layout at one point. For now it's fine I think.

Yeah was thinking the same, will fix that in another PR ✅


Use Expo Router

Use Expo Router
Apply more theme

#36
#35
#34
#33  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kanziw and the rest of your teammates on  Graphite

Merge activity

May 4, 11:49 AM EDT: @kanziw merged this pull request with Graphite.


Follow-up to (PouchDB / LevelDB / LevelDown): MaxListenersExceededWarning, Error: once called more than once

Follow-up to:
https://github.com/readium/readium-desktop/issues/406
According to feedback from other developers on the PouchDB issue tracker (see https://github.com/pouchdb/pouchdb/issues/7819#issuecomment-516551035 and https://github.com/pouchdb/pouchdb/issues/7825 ), this package.json workaround is effective:
 "resolutions": { "leveldown": "5.0.2", "levelup": "4.0.2" },

As you can see, this forces specific versions of LevelDown and LevelUp, instead of what we are doing now: setting a fixed version 7.0.* for pouchdb-adapter-leveldb (we also freeze leveldown to 5.*.*):
https://github.com/readium/readium-desktop/blob/59cf6aff032f50f2d631642edb308bfcd81034aa/package.json#L172
https://github.com/readium/readium-desktop/blob/59cf6aff032f50f2d631642edb308bfcd81034aa/package.json#L263-L264
https://github.com/readium/readium-desktop/blob/59cf6aff032f50f2d631642edb308bfcd81034aa/src/package.json#L36-L37

For what it's worth, I posted an issue in the LevelDown tracker:
https://github.com/Level/leveldown/issues/686

Ah, a better workaround which allows us to use the latest leveldown (^5.3.0):
https://github.com/readium/readium-desktop/commit/480dac7a13644ea380b26fa7b181e11c53ae8120
See: https://github.com/Level/leveldown/issues/686#issuecomment-541844695
Tested successfully on MacOS with:
rm -rf dist && rm -rf node_modules && rm package-lock.json && npm install && npm run __postinstall && npm run start

Follow-up: the latest leveldown version 5.4.0 does not fix the issue, so we are keeping the workaround for now. See:
https://github.com/readium/readium-desktop/commit/3331b92529d9bdd93b064f6a464fa808837347d3
https://github.com/Level/leveldown/issues/686#issuecomment-544602542


Verify

Khurram,
I verified a few more bugs and added test cases for them. Below is the functional run URL. The failures are known failures. Can you take a look?
http://<IP_ADDRESS>:8080/job/Framework Regression/290/console
Rahul

Rahul, could you update the addremove_files.sh file to have an if/else structure to check if directory exists before you create or delete them?

Other than that, looks good.

Abhishek,
Made the suggested change and the functional run did not have any issues. Can you take a look?
Rahul

LGTM, +1


Mapping ILM phases

fixes #2727

The validator is complaining about the missing action "Freeze", but that was removed in 8.0 and should be removed from the tests.


Include Socket.IO code example

Needs a really basic "Hello World" example for a traditional Express web app, AngularJS app, React app.

Hey - I would be interested in contributing and could work on this :)

@madishka Thanks, let me know if you get stuck somewhere. Although it should be pretty straightforward.
For Express web app, client side socket.io logic would go inside main.js.
For React, it would probably be App.js or Home.js component's componentDidMoiunt block.
For AngularJS, perhaps .run() block in the app.js.

Ok, I will get started working on this - I don't have a lot of XP in React so my coworker Adel is going to help out with that one. 8-)

How would authentication be handled in the websockets?
One way that seems to be common is to have a timeout on connection that closes the WebSocket if the user hasn't authenticated in a given time. The other way is to send credentials (ugh), jwt (also ugh) or a single-use token as part of the query params (lesser evil of the 3 IMO).
Thoughts?


Kotlin (KMP) source jars not recognized when viewing source

Description of the bug:
When attaching a maven dependency and including the source jar, Android Studio (and presumably Intellij) won't reference the source in the jar, even though the source jar is correctly identified.

generated intellij-info.txt file for library:
cat ./bazel-out/k8-fastbuild/bin/external/com_soywiz_korlibs_korma_korma_jvm_2_7_0/com_soywiz_korlibs_korma_korma_jvm_2_7_0--2053532790.intellij-info.txt   (output here)
This seems to work fine with java-based source jars, just not kotlin based source jars.
What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
Demo branch provided. I forked this repo and made some modifications to the examples/kotlin/simple_coroutines_project example project (this was probably unnecessary, but it works).
Branch: https://github.com/colinrgodsey/intellij/tree/kt_src_demo
(diff: https://github.com/colinrgodsey/intellij/compare/master...kt_src_demo)
Which Intellij IDE are you using? Please provide the specific version.
Android Studio Dolphin
What programming languages and tools are you using? Please provide specific versions.
Kotlin
What Bazel plugin version are you using?
bazel 6.0.0
Have you found anything relevant by searching the web?
No response
Any other information, logs, or outputs that you want to share?
No response

I did also verify the source is actually in the jar its referencing:
~/Repos/intellij$ unzip -l .aswb/.blaze/libraries/korma-jvm-2.7.0-src_a5cea6a5-src.jar
Archive:  .aswb/.blaze/libraries/korma-jvm-2.7.0-src_a5cea6a5-src.jar
  Length      Date    Time    Name
---------  ---------- -----   ----
        0  2022-03-25 02:25   META-INF/
...
        0  2022-03-25 02:07   commonMain/com/soywiz/korma/math/
      428  2022-03-25 02:07   commonMain/com/soywiz/korma/math/ArrayExt.kt
     4074  2022-03-25 02:07   commonMain/com/soywiz/korma/math/Math.kt
      111  2022-03-25 02:07   commonMain/com/soywiz/korma/Korma.kt
...
    17920  2022-03-25 02:07   commonMain/com/soywiz/korma/geom/Point.kt
...

This issue also seems to happen even with pretty vastly different important patterns (props to the team for somehow being able to pierce all that and find the source jars in the end). So the issue doesnt seem to be related to finding the source jar, it almost seems like intellij doesnt know it should be looking for .kt source files :man_shrugging:

oh.... i just now realized there's the commonMain prefix. I think this might be a KMP issue, not a kotlin issue specifically...

alright, I updated the example with okhttp, which is kotlin, but not KMP. same issue

any update?

I tried with the example you provided on IntelliJ, I noticed that the first time you try to go to a certain definition, the plugin will navigate to the .class file but if you closed that and clicked go to definition again it will take you to the .kt. If you're seeing the same behaviour then it is related to https://youtrack.jetbrains.com/issue/KTIJ-21422. We can accept a PR fixing this.

I think I'm actually not seeing this behavior specific to kt sources anymore. I was having source code resolution issues for all types recently. I'll have a PR soon that fixes that (after my PR for fixing previews!)


Consolidate support and super user into one user

Motivation
Super users can no longer make DNS record changes so they are much more like support users now. Our logic would be cleaner if these were one user instead of two
Scope of change
Investigate if we actually need the split. If not find all places we use the super flag and change it to support

@britneywright beat me to it: https://github.com/vinyldns/vinyldns/issues/641


Not working in rails 5.1

I am using rails 5 spree version 3.3.2 , i need a marketplace gem , but this gem is not working for this version kindly update it..

They keep saying it's fixed but still issue's are present but the creator just simply say's it's fixed and closes discussions.
I have been trying for a day now to try to run this gem and still nothing.
Would it finallyl work if I depreciated my rails?


fix(db-engine-spec): execute oracle DML statement bug in sqllab

when i execute oracle sql statements like update in SQLLAB, get "oracle error: not a query" error.
Refer https://www.python.org/dev/peps/pep-0249/, superset old version use
cursor.description ，because this attribute will be None for operations that do not return rows or if the cursor has not had an operation invoked via the .execute*() method yet.
SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

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

Codecov Report

Merging #10706 into master will increase coverage by 0.25%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master   #10706      +/-   ##
==========================================
+ Coverage   64.76%   65.01%   +0.25%
==========================================
  Files         789      788       -1
  Lines       37091    37175      +84
  Branches     3555     3554       -1
==========================================
+ Hits        24021    24170     +149
+ Misses      12963    12898      -65
  Partials      107      107

Flag
Coverage Δ

#cypress
55.78% <ø> (+0.17%)
:arrow_up:

#javascript
61.53% <ø> (+0.17%)
:arrow_up:

#python
60.31% <50.00%> (+0.28%)
:arrow_up:

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

Impacted Files
Coverage Δ

superset/db_engine_specs/oracle.py
81.25% <50.00%> (-7.64%)
:arrow_down:

superset-frontend/src/views/App.tsx
32.35% <0.00%> (-25.55%)
:arrow_down:

...ntend/src/dashboard/components/PublishedStatus.jsx
61.90% <0.00%> (-1.74%)
:arrow_down:

...end/src/SqlLab/components/RunQueryActionButton.tsx
68.18% <0.00%> (-1.39%)
:arrow_down:

superset-frontend/src/components/Menu/Menu.tsx
94.93% <0.00%> (-1.15%)
:arrow_down:

...ontend/src/explore/components/QueryAndSaveBtns.jsx
91.66% <0.00%> (-0.34%)
:arrow_down:

superset/views/core.py
74.46% <0.00%> (-0.05%)
:arrow_down:

superset/viz.py
57.25% <0.00%> (-0.03%)
:arrow_down:

superset/db_engine_specs/base.py
87.87% <0.00%> (ø)

superset/db_engine_specs/hive.py
83.98% <0.00%> (ø)

... and 45 more

Continue to review full report at Codecov.

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


[bug] Bucketing algorithm is incorrect in marginal cases

I have four different resolutions,  512x512, 512x768, 512x640, 768x512, all images
but the script is giving me the same lower resolution for all of them except for 1:1.
This doesn't seem reasonable.

The resolution parameter sets the max pixels that the image can have if the image size is higher it scales it down to the nearest multiple of 64 if you want to have 512x768 bucket sizes you need resolution to be at least 640 not 512 which is what it seems to be

I don't understand. My image sizes have all been pre-processed and are already in the correct size, just waiting to be bucketed. Are you saying that even though my images are already 512 in size, I need to set them to 640 or else it will scale them down to 384? Is that reasonable?

The total image pixels sets the max amount of pixels the images can have at 512*512 it's around 245'000 pixels 512x768 is more than that so it's gets scaled down

So I think the situation you described is somewhat different from what I faced.
min bucket res 512
max bucket res 1024

You need to change the resolution setting

I think I understand now. Are you saying that all the buckets must fall within a range of resolution^2?

Around there yes but the buckets use 64 pixel increments so it won't be exact

I understand now, thank you for the explanation.

I also met the same problem. Can u tell me how u set your setting or change your pic resolution?


Add FBC Makefile + doc

The Makefile will be used by operator owners to generate catalogs from templates. This file simplifies the workflow and hides any opm related commands from a user.
The commit also introduces a user documentation for a FBC workflow.
JIRA: ISV-4512

I'm still reviewing, but sorry for the delay.  I was out sick yesterday.

We've been having a bunch of onboarding/support discussions in #forum-konflux-operators and have started pinging our PM (Tony Wu) about some of the overlap. Have you had any discussions with Tony or the konflux folks about the processes?
We're thinking about spinning a #forum-fbc-support where we could cover a bunch of this ground together.

We did spin #forum-fbc-support and I invited you.  Please include anyone else I might have overlooked.


Fix nested editor cut

Null check got lost when moving to experimental copy helper. It avoids cut/copy event from being handled in non-active editor (e.g., when running cut in nested editor)

Null check got lost when moving to experimental copy helper. It avoids cut/copy event from being handled in non-active editor (e.g., when running cut in nested editor)

Do we not have test coverage here?

Thank you @fantactuka!

Null check got lost when moving to experimental copy helper. It avoids cut/copy event from being handled in non-active editor (e.g., when running cut in nested editor)

Do we not have test coverage here?

No, we don't, it's very hard to test copy-paste as we cannot mimic the browser in Playwright


Adding a python project for sending Emails in bulk. [gssoc23]

Title and Issue number

Send Templatized Dynamic Emails Automatically
This is a simple program which does its work perfectly. Nothing more, nothing less
-Send dynamic emails with unlimited variables pulling data from a CSV file.
-Supports Markdown Formatting & embed links or images.
-Supports Attaching any kind of files.
Close #4098

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

@MohitGupta121 @TusharKesarwani @pranjay-poddar Please review this PR.

@MohitGupta121 @pranjay-poddar   Why you added level 1 tag to this PR, I added a new project to the repo and it must be atleast level 2


api changes

What are you changing in this pull request and why?

Checklist

[ ] Review the Content style guide and About versioning so my content adheres to these guidelines.
[ ] Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Adding new pages (delete if not applicable):

[ ] Add page to website/sidebars.js
[ ] Provide a unique filename for the new page

Removing or renaming existing pages (delete if not applicable):

[ ] Remove page from website/sidebars.js
[ ] Add an entry _redirects
[ ] Ran link testing to update the links that point to the deleted page

@mirnawong1 I am working on a PR that touches this page. I'm updating references to the Access URL.

thanks @runleonarun - should i branch off your fix-access-URLs branch to avoid merge conflicts?

adding @runleonarun 's api access url text to this pr per meeting

Thanks @mirnawong1 ! Good to 🚢 . Looking forward to collaborating more on the use case guides soon


[UI] After starting meshery, the UI is not accessible

Current Behavior

Start meshery

$ mesheryctl system start
Starting Meshery...

Meshery deployed on Kubernetes.
Meshery is starting...
Opening Meshery (http://localhost:31611) in browser

The UI is not accessible
Also unable to get the token
$ mesheryctl system config minikube --token .
Configuring Meshery to access Minikube...
Error getting context: Authentication token not found. Login with `mesheryctl system login` or supply a valid user token using the --token (or -t) flag.

$ mesheryctl system login
authentication failed: Get "http://localhost:31611/api/providers": dial tcp [::1]:31611: connect: connection refused

Desired Behavior

Should be able to install and start using meshery
Screenshots/Logs

Environment

Browser: Chrome Safari Firefox
Host OS: Mac Linux Windows
Meshery Server Version: unavailable
Meshery Client Version:v 0.6.30
Platform: Docker Kubernetes

$mesheryctl version

      	VERSION    	GITSHA
Client	v0.6.30    	6d124bf47
Server	unavailable	unavailable
Unable to communicate with Meshery server.Get "http://localhost:31611/api/system/version": dial tcp [::1]:31611: connect: connection refused.See https://docs.meshery.io for help getting started with Meshery

Checking for latest version of mesheryctl...

  v0.6.30 is the latest release.

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Try localhost:9081

I see the same problem/behavior with localhost:9081 as well.

Yes, I have docker desktop.
But I am running minikube inside docker and running meshery inside Kubernetes cluster itself (in-cluster) mode.
Perhaps what you are asking me is to install the meshery extension inside docker and try using that. If I do that, I can see the meshery UI but it's not able to detect the k8s cluster.

there is a kubernetes option in preferences, turn that on. There is a option to set up cluster when you run docker desktop, enable it. don't run minikube just start the docker desktop after configuring this settings and then try localhost:9081

I'll try that and update here.
Any reason why it does not work with minikube or kind cluster both running inside docker? The documentation does not specify that meshery has to be exclusively used with docker k8s setup for this to work.
Also, we have been using kind cluster for all our poc's and not everyone in our team has docker desktop.
Thanks.

No, I don't know any particular reasons, I tried many times various way but only this worked, you can ask the maintainers on dev call.

Hi @bluedog13
Thanks for raising this issue.
There are a couple of pointers that do get overlooked sometimes considering it’s not explicitly stated in docs or error logs for mesheryctl.

Meshery is a multiple Kubernetes manager and irrespective of the type of Kubernetes cluster, it does auto-detect available K8s clusters in your environment from the kubeconfig in your local file system. You can always pass a custom kubeconfig from the settings page.
Meshery, as a Cloud Native manager, have some custom controllers. For these controllers to communicate within themselves, they use K8s services, one of them is meshery-broker of type LoadBalancer. That means, its external IP address needs to be readily exposed so that it can communicate well with every component of Meshery.
The challenge with Minikube and Kind clusters is the loadbalancer which usually doesn’t come out of the box and Meshery provides its users with the freedom to set up networking in their style. Hence networking for such types of deployments is more involved.
A couple of options that I prefer are:
3.1. Use minikube add-on for MetalLB that would provide you with loadbalancer. minikube addons enable metallb
3.2. Use minikube tunnel to expose your services. minikube tunnel
3.3. For kind, I prefer installing MetalLB with custom configmap.
But incase you choose docker-desktop then such networking comes out of the box. That's the reason that some contributors in the community prefer it over other options. But either way the experience is exactly same.
For mesheryctl, there is a command: mesheryctl system dashboard which is designed to automatically open Meshery UI onto the default browser. You may try that too.

I'll take a note of few things that should be improved to increase user productivity:

Improve docs with more information regarding setup installation
Improve mesheryctl err logs.

I'll open GitHub issues to track the progress of these updates.
In the meantime, would you please try some of the troubleshooting steps that I mentioned above? Do let us know if those were helpful. In case you get further doubts then feel free to reach out in the comments.
Everyone in the community would be happy to help.
I hope you have a great day  😊

@bluedog13 in short, the most likely challenge that you're facing is one of networking in which Meshery Server attempts to reach Kube API (at minikube ip), but can't. There are a handful of approaches to ensuring connectivity between the bridge network that Meshery
Server is running on. This diagram offers one (not the best solution).

Hi! @bluedog13, I think that you've not tunneled the port 9081, to your localhost port 9081, try this one:
kubectl port-forward service/meshery 9081:9081 -n meshery
I guess, this should work.

FYI:  I am also a minikube user on Windows side of my machine.  When running mesheryctl system dashboard  it redirects to :31104 and I receive ERR_CONNECTION_REFUSED . Not sure where this configuration directive is coming from. However when I am implicit to :9081 it does connects to dashboard.  I will follow a few breadcrumbs and see if I can find this on the forum to see if the issue has been fully described.


"Future Keywords" list seems out of date, many are keywords now

In https://docs.microsoft.com/en-us/sql/t-sql/language-elements/reserved-keywords-transact-sql?view=sql-server-ver15#future-keywords, the section says CAST might become a Reserved Keyword in Transact-SQL in SQL Server sometime in the future. Yet I believe CAST is already a keyword, as the article about "CAST and CONVERT" already strongly implies.  And there are other similar "future" keywords the same list that again seem keywords in SQL 2017+.

https://docs.microsoft.com/en-us/sql/t-sql/functions/cast-and-convert-transact-sql?view=sql-server-ver15

P.S.   Might be good to have an Aquent person turn all these actual T-SQL keywords into clickable LINKS to the Ref article that discusses each.
Thanks. GeneMi ( = MightyPen ).  2019/10/14.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f79ec4df-5309-60fc-1c20-f32b219fb9b9
Version Independent ID: 6ee85c34-9a2c-e007-dcda-fb6016b98119
Content: Reserved Keywords (Transact-SQL) - SQL Server
Content Source: docs/t-sql/language-elements/reserved-keywords-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @rothja
Microsoft Alias: jroth

@MightyPen  Thank you for the feedback, assigning this to the author for review of CAST.
Including links would provide a nice feature, keywords like ADD  have no equivalent page and make such a list inconsistent.

@MightyPen Sorry for the long delay. I've seen customers bring up keyword questions in the past too, but I think this is a confusing area. I believe that CAST is not a keyword in transact-SQL the way that other keywords on the list are. See this example:

You can make CAST the name of the column without escaping it. That is not the case with "official" keywords. Why you can do this is not clear to me. Maybe we need to make the article more clear about what makes a keyword a keyword?

I guess 'CAST' is more like an arbitrary function name that can be called from the T-SQL language.  Thanks.


DBZ-5395 Improve Oracle offset management when LOB enabled

https://issues.redhat.com/browse/DBZ-5395

@jpechane can you review and merge this if it seems okay to you?

@Naros Applied, thanks


[Question] Middleware to use after @Authenticated()

Hey!
I was wondering if there is a way to use middleware within the controller after an @Authenticated call. Seems like if I call a @Use, the middleware executes before the $onAuth call (I set a request.user within that call).
Thanks in advance for your help!

Hi @zoubarevm ,
Normally the order of the annotations defines the calling order of the middlewares. (Except for the @Get()/@Post() etc..., they are always the priority whatever order).
So If you make this:
@Get('/')
@Authenticated()
@Use(myMiddleware)
myMethod(){

}

// With express.js it's equivalent to
app.get('/', $onAuth, myMiddleware, myMethod);

If this is not the case then there is a bug :)
See you

Hi @Romakita ,
It seems like the @Use middleware is overwriting the @Authenticated call.
Here is what I have in my controller:
@Get('/my/:id(\\d+)')
@Authenticated()
@Use(MyController.checkPermission('retrieve'))
public getById(@Request() request: Express.Request,
                    @PathParams('id') id: number) {
    ...
}

The checkPermission method is hit, but the $onAuth is not.
When I remove the @Use, the $onAuth is hit.
Any help is appreciated!

Hello @zoubarevm ,
After code review, it's a bug. Typescript reverses the execution order of the decorators. I didn't know that XD.
I'll provide a quick fix ;)
See you.

Fixed :) v1.3.3

@Romakita thanks for your hard work, but it seems like I'm getting an error on the latest build:
HTTP Server error TypeError: Cannot read property 'forEach' of undefined
[1]     at Function.InjectorService.construct (/Projects/expressts2/node_modules/ts-express-decorators/lib/services/injector.js:45:13)
[1]     at /Projects/expressts2/node_modules/ts-express-decorators/src/services/injector.ts:108:37

Hi @zoubarevm,
The injectorService isn't ready on this step. But we haven't a hook for that except the $onReady.
I'll tried to move the injectorService.load() to have the services loaded before the $onMountingMiddlewares hook.

"One thing to note: I am calling the InjectorService in the mounting of middlewares for another middleware that I'm activating globally."
Ok. I'll move the InjectorService.load() to have all service ready before $onMountingMiddleware and update the documentation.

Fixed v1.3.4

@Romakita I use one of the injected services within the middleware. The latest version looks good :)
Thanks so much for your help!


staging font test

Overview

This is not meant to be merged. It is only to generate a preview version so that we can test a cors policy in staging.

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: [ABC-123]
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut@57.0.1-alpha.f8c838.0@codecademy/gamut-icons@9.30.2-alpha.f8c838.0@codecademy/gamut-illustrations@0.47.2-alpha.f8c838.0@codecademy/gamut-kit@0.6.440-alpha.f8c838.0@codecademy/gamut-patterns@0.9.15-alpha.f8c838.0@codecademy/gamut-styles@17.0.1-alpha.f8c838.0@codecademy/gamut-tests@5.1.2-alpha.f8c838.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://66d758c5d6f7df9dcc3d8bc4--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut@57.0.1-alpha.737099.0@codecademy/gamut-icons@9.30.2-alpha.737099.0@codecademy/gamut-illustrations@0.47.2-alpha.737099.0@codecademy/gamut-kit@0.6.440-alpha.737099.0@codecademy/gamut-patterns@0.9.15-alpha.737099.0@codecademy/gamut-styles@17.0.1-alpha.737099.0@codecademy/gamut-tests@5.1.2-alpha.737099.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://66d76ec9064dadb97bd733de--gamut-preview.netlify.appDeploy Logs

closing since we have completed the s3 cors change that we were testing here