VSCode c# extension not working due to not finding .NET.

Path: c:\Users\admin.vscode\extensions\ms-dotnettools.csharp-1.23.17.omnisharp\1.37.17\OmniSharp.exe
PID: 3680
    Starting OmniSharp on Windows 6.2.9200.0 (x64)

    Checking the 'DOTNET_ROOT' environment variable to find a .NET SDK

    Using the 'dotnet' on the PATH.

    DotNetPath set to dotnet

    Located 2 MSBuild instance(s)
        1: Visual Studio Community 2022 17.0.32014.148 17.0.0 - "D:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin"
        2: StandAlone 17.0.0 - "c:\Users\admin\.vscode\extensions\ms-dotnettools.csharp-1.23.17\.omnisharp\1.37.17\.msbuild\Current\Bin"

    Registered MSBuild instance: Visual Studio Community 2022 17.0.32014.148 17.0.0 - "D:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin"

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.Completion.CompletionOptionsProvider, Order: 0

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.RenameWorkspaceOptionsProvider, Order: 100

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.ImplementTypeWorkspaceOptionsProvider, Order: 110

    Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.BlockStructureWorkspaceOptionsProvider, Order: 140

    Detecting Cake files in 'd:\Users\admin\Snowboarder'.

    Could not find any Cake files

    Detecting projects in 'd:\Users\admin\Snowboarder\Snowboarder.sln'.

    Queue project update for 'd:\Users\admin\Snowboarder\Assembly-CSharp.csproj'

    Detecting CSX files in 'd:\Users\admin\Snowboarder'.

    Could not find any CSX files

    Configuration finished.

    Omnisharp server running using Stdio at location 'd:\Users\admin\Snowboarder' on host 9748.

    Loading project: d:\Users\admin\Snowboarder\Assembly-CSharp.csproj

    The reference assemblies for .NETFramework,Version=v4.7.1 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks

    Failed to load project file 'd:\Users\admin\Snowboarder\Assembly-CSharp.csproj'.

d:\Users\admin\Snowboarder\Assembly-CSharp.csproj
D:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(1217,5): Error: The reference assemblies for .NETFramework,Version=v4.7.1 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks
    Attempted to update project that is not loaded: d:\Users\admin\Snowboarder\Assembly-CSharp.csproj

    No document found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs.

    No document found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs.

    Miscellaneous file: d:\Users\admin\Snowboarder\Assets\Scripts\CrashDetector.cs added to workspace

Received response for /quickinfo but could not find request.
Received response for /quickinfo but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No document found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No document found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No document found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs.
info: OmniSharp.OmniSharpWorkspace
Miscellaneous file: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs added to workspace
Received response for /v2/blockstructure but could not find request.
Received response for /v2/getcodeactions but could not find request.
Received response for /v2/blockstructure but could not find request.
Received response for /quickinfo but could not find request.
Received response for /v2/getcodeactions but could not find request.
info: OmniSharp.Roslyn.CSharp.Services.Refactoring.V2.RunCodeActionService
Applying code action: Extract method
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 23, Column: 12.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 20, Column: 17.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 20, Column: 8.
info: OmniSharp.Roslyn.CSharp.Services.Refactoring.V2.RunCodeActionService
Applying code action: Introduce local for 'RespondToBoost()'
Received response for /v2/blockstructure but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 12, Column: 9.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 19, Column: 9.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 9, Column: 22.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 10, Column: 4.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 10, Column: 22.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 10, Column: 22.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 13, Column: 9.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 20, Column: 9.
Received response for /v2/getcodeactions but could not find request.
warn: OmniSharp.Stdio.Host
************ Request ************
{
"Type": "request",
"Seq": 2437,
"Command": "/filesChanged",
"Arguments": [
{
"FileName": "d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs",
"changeType": "Change"
}
]
}
fail: OmniSharp.Stdio.Host
************  Response (85.1780ms) ************
{
"Request_seq": 2437,
"Command": "/filesChanged",
"Running": true,
"Success": false,
"Message": ""System.IO.IOException: The process cannot access the file 'd:\\Users\\admin\\Snowboarder\\Assets\\Scripts\\PlayerController.cs' because it is being used by another process.\r\n   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)\r\n   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)\r\n   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)\r\n   at System.IO.StreamReader..ctor(String path, Encoding encoding, Boolean detectEncodingFromByteOrderMarks, Int32 bufferSize, Boolean checkHost)\r\n   at System.IO.File.InternalReadAllText(String path, Encoding encoding, Boolean checkHost)\r\n   at OmniSharp.Roslyn.BufferManager.d__8.MoveNext() in D:\\a\\1\\s\\src\\OmniSharp.Roslyn\\BufferManager.cs:line 47"",
"Body": null,
"Seq": 27889,
"Type": "response"
}
Received response for /quickinfo but could not find request.
Received response for /v2/blockstructure but could not find request.
Received response for /completion/resolve but could not find request.
Received response for /v2/getcodeactions but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 44, Column: 31.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 41, Column: 30.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 42, Column: 31.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 44, Column: 44.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 42, Column: 39.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 44, Column: 38.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 44, Column: 50.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 42, Column: 51.
info: OmniSharp.OmniSharpWorkspace
Miscellaneous file: d:\Users\admin\Snowboarder\Assets\Scripts\DustTrail.cs added to workspace
Received response for /v2/codestructure but could not find request.
Received response for /v2/codestructure but could not find request.
Received response for /v2/blockstructure but could not find request.
Received response for /v2/getcodeactions but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\DustTrail.cs, Line: 12, Column: 37.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\DustTrail.cs, Line: 19, Column: 36.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 43, Column: 36.
Received response for /v2/getcodeactions but could not find request.
Received response for /v2/blockstructure but could not find request.
info: OmniSharp.OmniSharpWorkspace
Miscellaneous file: d:\Users\admin\Snowboarder\Assets\Scripts\FinishLine.cs added to workspace
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\FinishLine.cs, Line: 12, Column: 44.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\FinishLine.cs, Line: 10, Column: 21.
Received response for /quickinfo but could not find request.
Received response for /v2/blockstructure but could not find request.
warn: OmniSharp.Stdio.Host
************ Request ************
{
"Type": "request",
"Seq": 6693,
"Command": "/filesChanged",
"Arguments": [
{
"FileName": "d:\Users\admin\Snowboarder\Assets\Scripts\FinishLine.cs",
"changeType": "Change"
}
]
}
fail: OmniSharp.Stdio.Host
************  Response (1222.0478ms) ************
{
"Request_seq": 6693,
"Command": "/filesChanged",
"Running": true,
"Success": false,
"Message": ""System.IO.IOException: The process cannot access the file 'd:\\Users\\admin\\Snowboarder\\Assets\\Scripts\\FinishLine.cs' because it is being used by another process.\r\n   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)\r\n   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)\r\n   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)\r\n   at System.IO.StreamReader..ctor(String path, Encoding encoding, Boolean detectEncodingFromByteOrderMarks, Int32 bufferSize, Boolean checkHost)\r\n   at System.IO.File.InternalReadAllText(String path, Encoding encoding, Boolean checkHost)\r\n   at OmniSharp.Roslyn.BufferManager.d__8.MoveNext() in D:\\a\\1\\s\\src\\OmniSharp.Roslyn\\BufferManager.cs:line 47"",
"Body": null,
"Seq": 75982,
"Type": "response"
}
Received response for /completion/resolve but could not find request.
Received response for /completion/resolve but could not find request.
Received response for /v2/getcodeactions but could not find request.
Received response for /v2/blockstructure but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 29, Column: 11.
Received response for /completion/resolve but could not find request.
Received response for /completion/resolve but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs, Line: 35, Column: 16.
Received response for /v2/getcodeactions but could not find request.
Received response for /completion/resolve but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\CrashDetector.cs, Line: 8, Column: 27.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\CrashDetector.cs, Line: 12, Column: 9.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\CrashDetector.cs, Line: 6, Column: 13.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\CrashDetector.cs, Line: 11, Column: 32.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\CrashDetector.cs, Line: 23, Column: 9.
warn: OmniSharp.Stdio.Host
************ Request ************
{
"Type": "request",
"Seq": 7947,
"Command": "/filesChanged",
"Arguments": [
{
"FileName": "d:\Users\admin\Snowboarder\Assets\Scripts\PlayerController.cs",
"changeType": "Change"
}
]
}
fail: OmniSharp.Stdio.Host
************  Response (4.3677ms) ************
{
"Request_seq": 7947,
"Command": "/filesChanged",
"Running": true,
"Success": false,
"Message": ""System.IO.IOException: The process cannot access the file 'd:\\Users\\admin\\Snowboarder\\Assets\\Scripts\\PlayerController.cs' because it is being used by another process.\r\n   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)\r\n   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)\r\n   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)\r\n   at System.IO.StreamReader..ctor(String path, Encoding encoding, Boolean detectEncodingFromByteOrderMarks, Int32 bufferSize, Boolean checkHost)\r\n   at System.IO.File.InternalReadAllText(String path, Encoding encoding, Boolean checkHost)\r\n   at OmniSharp.Roslyn.BufferManager.d__8.MoveNext() in D:\\a\\1\\s\\src\\OmniSharp.Roslyn\\BufferManager.cs:line 47"",
"Body": null,
"Seq": 83829,
"Type": "response"
}
Received response for /v2/blockstructure but could not find request.
Received response for /v2/blockstructure but could not find request.
Received response for /completion/resolve but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\CrashDetector.cs, Line: 30, Column: 9.
Received response for /completion/resolve but could not find request.
Received response for /v2/getcodeactions but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\CrashDetector.cs, Line: 19, Column: 28.
Received response for /completion/resolve but could not find request.
warn: OmniSharp.Roslyn.CSharp.Services.Navigation.FindUsagesService
No symbol found. File: d:\Users\admin\Snowboarder\Assets\Scripts\FinishLine.cs, Line: 14, Column: 45.
Steps to reproduce
Whenever I open a script in unity this message happens.
Expected  behavior
The c# extension works properly.
Actual behavior
No c# extension benefits to be found on my side unfortunately.
Steps I have taken to try to fix this:
Installed .NET 6.0
Installed .NET 4.7.1 developer pack
Changed omnisharp.path to latest
installed an earlier version
regenerated the .cs and .sln files in the project
Then yesterday I was advised to try installing Visual Studio, which I did but the problem persists.
Please help, thank you.

I too am having similar issues with omnisharp, vsc (both insiders and regular) and unity. I have both 5.0.400 and 6.0.101 installed

you need to mess a bit with Custom Analyzers to make it work fully, you need to download the nupkg file of Microsoft.Unity.Analyzers, open it with 7zip or winrar as a zip, and navigate to -> analyzers/dotnet/cs/ and add to Omnisharp as a custom analyzer the Dll that is in there. This should fix the problem, it did for me

A lot has changed with O# project loading since this issue was opened. If you are still having problems, please open a new issue at https://github.com/omnisharp/omnisharp-roslyn/issues


cli: small refactor to align the other products

This is a small refactor to align it with other company product like:
https://github.com/bblfsh/bblfshd/blob/master/cli/bblfshctl/
Also will be nice add some tests to the commands.

@mcuadros  cmd is way more common than cli in the Go community. Shouldn't we change it the other way around? cli -> cmd.
There was some previous discussion here: https://github.com/src-d/guide/pull/196#discussion_r181054490

Its already, in many places, and don't worth to changed, and cli is a more meaningful name that cmd.
But this changes are more related to the scaffolding than the naming.

If we are not going to have more sub-commands I would dump them. By default it starts a server and --version prints the version number.


Can't install plugin via Visual Studio

When I try to import this plugin directly from Visual Studio 2015 RC I get this error message:
Visual Studio was unable to parse plugin.xml. Please ensure plugin exist and contains no errors.
Do you have any idea ?
Thank you.

This is the same as issue #52 and #46. Fix for that hasn't been released yet.
See the related pull request with issue 52 on how to fix it by hand.
BTW VS 2015 is RTM now.

Thank you for your help

Hey has this fix been made public

I have same problem but the string show in other issue is already present.

Fix was merged


Make Test Compare view easier to use

In Test Compare view, I think we could just reduce to 3 fields Test Job URL A, and Test Job URL B, and test target name (which could be optional) and cut'n'paste from the browser to compare (rather than 8 fields that have to be painstakingly filled out, and if one or both of the test jobs are not in that instance of TRSS, just report as such.

This will be similar to how Perf Compare is already, with the additional optional test target field

@LongyuZhang could you take a look at this one? Thanks

Close this issue as the PR is delivered.


BUG: No such file or directory , this error thrown on package utils/any-in-by while running benchmark tests

Description

@kgryte @Pranavchiku Can you please take a look at this?
Related Issues
Related issues # , # , and # .
Questions
No.
Demo
No response
Reproduction
No response
Expected Results
No response
Actual Results
No response
Version
No response
Environments
N/A
Browser Version
No response
Node.js / npm Version
No response
Platform
No response
Checklist

[X] Read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.

@Pratik772846 There shouldn't be a Julia benchmark in utils/any-in-by.

@kgryte Similarly I think we do not need Julia benchmark in utils/every-own-by. But these packages with Julia benchmark have been merged , should I remove Julia benchmark from every-own-by also?

@Pratik772846 No, that was an oversight. Those packages (e.g., utils/every-own-by, etc) should not have Julia benchmarks. This was a copy-paste bug introduced by copying files from, e.g., utils/every, where having Julia benchmarks is appropriate.

cc @Planeshifter

Thanks for the information, I will remove the Julia files while pushing my commit.


Confusing permissions behaviour when logged in as low-privilege user

When you're logged in as E.g. an author, the admin panel isn't good at telling you why you can't edit a post. I've found two specific cases - the /edit URL and search, but I think they are both very similar.
To Reproduce
Case 1: search

In admin search for a post you can't edit
See the post listed in the results
Click on the post
You are taken to the posts list with no explanation

Case 2: /edit

In the frontend, find that post you're trying to edit
Add /edit to the end of the URL
Watch the post screen load for a while
You are taken to the posts list with no explanation

In both cases, there should be IMO a banner error saying you don't have permission.
In the case of search, we probably shouldn't be showing post the user can't edit at all?
Technical details:

Ghost Version: latest

I'm a first time contributor and would like to work on this. How do i get started?

We have a contributing guide as per most repositories. We don’t have anyone to coach you through issues, so you may want to start with an issue marked help wanted as these are the ones we think are suitable for contributors.

Good to know @ErisDS thanks, I’ll find one of those.

@peterzimon would need a copy advice and an opinion on:

In the case of search, we probably shouldn't be showing post the user can't edit at all?

Banner/copy part of the solution
For reference, currently displayed error notification coming from server error data if the author user tries to edit not their own post (disabled redirect in code):
Permission error, cannot edit post. You do not have permission to edit posts.

At the moment when the post/page is loaded we have both user and post information and the redirect is done on the frontend side. This means we can construct and display any error using this information (there's no server involved). I'm thinking of something like this displayed as warning notification:
Permission error, cannot edit selected post "{post title here}". Only authors or administrator users can edit this post.

Would look like:

This message would make sense to display in both situations described in the issue.
Not showing posts to users that can't edit
This approach makes sense to me,  will check how easy it is to filter these records on the client. @peterzimon would be good to know if you have different opinion.

@peterzimon to expand on the showing only editable information in search results. Similarly to showing posts that don't belong  to current author (and cause redirect), search results show "tags", "pages" and "users" to user logged in as "author", which it probably should not be showing?

Another possible bug that have discovered while investigating these issues is that author user can't create "page" resources. This is not documented anywhere and needs more investigation.

@matthanley not sure how to proceed with this bug. The linked issue TryGhost/Team#391 is a holistic solution improving workflow for contributors. This issue is somewhat related, but could be partially solved through copy updates and smarter filtering of shown search results for all low-privilege users (contributors and authors).
My thinking is following. Ideally we'd make an audit of behavior in Ghost-Admin for "authors" users as well and solve each inconsistency separately (like the contributor's workflow issue). If there's not much appetite for holistic approach like that, we could solve inconsistencies that have been discussed in this issue separately from contributor's improvements.
Let me know what are you thoughts.

@matthanley ping to check above message ;)

@matthanley ping to check above message ;)

If there's not much appetite for holistic approach like that, we could solve inconsistencies that have been discussed in this issue separately from contributor's improvements.

@naz what would be your proposal for a small iterative improvement on what we currently have for this?

If there's not much appetite for holistic approach like that, we could solve inconsistencies that have been discussed in this issue separately from contributor's improvements.

@naz what would be your proposal for a small iterative improvement on what we currently have for this?

@matthanley would break the permissions and Admin UI mismatches into following stages.
Small change that can be done immediately: implement the solutions that have been initially proposed in the issue description:

Display a banner in Admin UI with a warning notification proposed above when permission redirect happens. Blocked by review of proposed above copy.
Filter out search results for authors/contributors to only show the content they can edit. Blocked by confirmation on the improved search behavior.

Note, the linked team issue that improves usability for posts list view for contributors does not address the issues raised here! It's only related because it touches same topic - "mismatch of what we show in the UI and what logged in user can actually do"
Bigger follow up change would be auditing how search (for posts, pages, tags, users) and  post/page list views behave for all lower privilege user roles. An example of such inconsistencies is described here and here, which was found during research for current issue. Note, this audit might be related and should account for the problem&solution proposed in posts list view improvements for contributors.

@matthanley would break the permissions and Admin UI mismatches into following stages.
Small change that can be done immediately: implement the solutions that have been initially proposed in the issue description:

Display a banner in Admin UI with a warning notification proposed above when permission redirect happens. Blocked by review of proposed above copy.
Filter out search results for authors/contributors to only show the content they can edit. Blocked by confirmation on the improved search behavior.

Note, the linked team issue that improves usability for posts list view for contributors does not address the issues raised here! It's only related because it touches same topic - "mismatch of what we show in the UI and what logged in user can actually do"
Bigger follow up change would be auditing how search (for posts, pages, tags, users) and  post/page list views behave for all lower privilege user roles. An example of such inconsistencies is described here and here, which was found during research for current issue. Note, this audit might be related and should account for the problem&solution proposed in posts list view improvements for contributors.

This is now resolved with the introduction of new designs for authors and contributors.


Test Bug 87

Summary
When I retrieve the source based on package.xml using SFDX: Retrieve Source in Manifest from Org

then despite of window being wide enough (as I am using 4k display with 150% scaling factor) "xml" file extension is moved to next line which results in additional lines added which is not needed.

Steps To Reproduce:

Retrive source from Package.xml using the context menu SFDX: Retrieve Source in Manifest from Org
Check results window

Expected result
"xml" is at the end of the line as there is enough space:
CustomLabels                                                                             CustomLabels              force-app\main\default\labels\CustomLabels.labels-meta.xml
Actual result
Text is wrapped at last dot (.) and "xml" is in new line:
CustomLabels                                                                             CustomLabels              force-app\main\default\labels\CustomLabels.labels-meta.
xml
Additional information
Salesforce Extension Version in VS Code:v60.11.0
Salesforce CLI<EMAIL_ADDRESS>win32-x64 node-v20.12.2
OS and version:Widnows 11
VS Code version:1.89.0
Most recent version of the extensions where this was working:None

Hello @daphne-sfdc :wave: None of the versions of sf you shared match the latest release.
Shared: 2.39.6
Latest: 2.42.6
Update to the latest version of Salesforce CLI (docs) and confirm that you're still seeing your issue.
You can also try the rc and nightly releases! (docs)
After updating, share the full output of sf --version.

Hello @daphne-sfdc :wave: None of the versions of the Salesforce Extension Pack you shared match the latest release.
Shared: 60.11.0
Latest: 60.13.0
Update to the latest version of the Salesforce Extension Pack in the VSCode Extensions view and confirm that you're still seeing your issue.
After updating, share your current extensions version.

Thank you for filing this issue.  We appreciate your feedback and will review the issue as soon as possible. Remember, however, that GitHub isn't a mechanism for receiving support under any agreement or SLA. If you require immediate assistance, contact Salesforce Customer Support.


using ballots to determine unjudged projects

if ballots.length == 0, the project is unjudged.
the flaw is that some projects may have >= 1 ballots cast, but still need to be judged more times, but we don't know how to determine that the project still needs to be judged when ballots.length >= 1

Good work!


MailChimp Proxy

I set up a Mailchimp test account, see Slack convo.
Create an API endpoint that takes an email as a parameter and sends a query to the MailChimp API.
Endpoint is https://us16.api.mailchimp.com/3.0/lists/{listId}/members/
JSON Payload:
{
    "email_address": "...",
    "status": "pending"
}

Auth by basic auth (API key as password) or OAuth2
Ref.
http://developer.mailchimp.com/documentation/mailchimp/guides/manage-subscribers-with-the-mailchimp-api/

https://github.com/Vizzuality/trase-api/pull/46


Not obvious if get_requires_for_build_wheel will modify installed packages state, or how - and other questions

I am very interested in the following code, since it is what I want to use in my project:
import os
from pep517.wrappers import Pep517HookCaller

src = 'path/to/source'  # Folder containing 'pyproject.toml'
hooks = Pep517HookCaller(src)
print(hooks.build_sys_requires)  # List of static requirements

config_options = {}   # Optional parameters for backend
# List of dynamic requirements:
print(hooks.get_requires_for_build_wheel(config_options))

However, I have a couple of questions, which I suggest should be possibly answered in the README:

What happens when the package doesn't provide the new functionality to get this metadata without building the wheel? If it builds the wheel in that case, can I possibly configure it to error out instead?
Can I somehow prevent any sort of state modification outside of the package dir (installed site packages) by this command? Will it do such modifications ever at all, e.g. when building the wheel as fallback or in other situations. e.g. to satisfy setup_requires?
If the package examined with this command is already installed in the site-packages, will this in any way interfere with obtaining the information from it in the exact version as found at the src path?

Contribution to the docs gratefully accepted ;-)
If the backend doesn't supply get_requires_for_build_wheel, then the wrapper returns []. See https://github.com/pypa/pep517/blob/master/pep517/_in_process.py#L40
What get_requires_for_build_wheel does is entirely handled by the backend. PEP 517 says nothing on the matter, so a generic wrapper like this can't do anything. There's a config_options argument, but they are totally backend-defined. So (a) you'd have to consult the backend docs, and (b) relying on config options will make your code backend specific.
At the moment, there's essentially no real-world experience with what backends can or should do - there are too few alternative backends to draw any real conclusions. Questions like you're asking are likely to need input from backend authors, and may need documenting in PEP 517. But they aren't going to be something that's covered by this project - which will always be based on what the PEP says.

With backend you're referring to e.g. setuptools/distutils I assume?

Whichever one is specified in pyproject.toml.

Sorry, I realize this might be the wrong place for this feedback. Where is the best place for that, the python packaging bugtracker?

Well, as far as I'm concerned, this (at least wrappers.py) is a library to handle all of the bookkeeping needed to call PEP 517 hooks as specified by the PEP. So if you don't use this code, you'll have to write it yourself. If you want isolated build environments, the envbuild.py code is basically doing what you'd otherwise have to implement yourself - although there's a proviso in that case, that the code hasn't been used in production, so it may have bugs (but even so, why not contribute fixes here rather than write your own version from scratch?)
It sounds like you don't want to call PEP 517 hooks, but want something else. So it's quite possible this library isn't for you.
However, I will say:

each setup.py provider is required to implement --install_requires and --setup_requires to print out the according information, and this function has to not modify the site-packages

I'm not sure how you intend to enforce that. You'd need it in a PEP if you are going to rely on it. And furthermore, it's unlikely that any backend other than setuptools will even use a file called setup.py, so your interface won't make sense for a general backend.

Am I missing something? Why is this all so horribly complicated? Am I wrong thinking that everyone involved should meet up and figure out a better solution than the current mess?

To be blunt, we did that, spending months thrashing out the details of PEP 517. Calling the specification a "mess" is rather dismissive of all the work that went into producing it. Maybe we missed whatever your use case is, but that's likely only because no-one participating in the discussions (which were open to the public, on distutils-sig) mentioned it.

Sorry, I realize this might be the wrong place for this feedback. Where is the best place for that, the python packaging bugtracker?

Honestly, I don't know. I don't actually understand what you're trying to do. On the one hand you seem to want a generic way of getting package information, but on the other hand, you are talking about setup.py, as if you're only interested in setuptools.
If you are only interested in working with setuptools, then you should probably bring your requirements up on the setuptools tracker. If you want to work with general backends, then the packaging tracker might be better - but you should be prepared to explain what you're trying to do in terms that don't assume setuptools is involved. That's hard, because there aren't many good examples of alternative backends. I'd recommend looking at flit as an example, but it doesn't handle anything other than pure Python code, so it may be of limited help.

Sorry for calling it a mess, I am pretty clueless I have to admit (e.g. I wasn't aware there are other install methods in real world use not using setup.py). So I'm mostly talking of that because I have no idea what I'm talking about :grimacing:
For completeness sake, my use case is just to get the dependencies without installing (because I'm in a cross compile environment, so installing at that stage doesn't make any sense) or touching anything, and to query them for longer deps-of-deps chains (so a throaway virtualenv for each query could become a perf issue).
Thanks for all your responses, I hope I didn't thrash too much on things. As written above I am just surprised at how complicated a simple "list the package dependencies" can get, but I shouldn't be judging given I have no clue about the python packaging ecosystem as a whole. Sorry about that

If you are only interested in working with setuptools, then you should probably bring your requirements up on the setuptools tracker

Forgot to answer that part: I did, and they said they weren't interested, in part because they want to get such setuptools-specific logic out and into generic tools instead (like this one)

:-)
I think that your biggest problem is that getting dependencies without installing is perfectly possible. But what's not possible is getting dependencies without building. The problem is that setup.py is executable code, and can to anything at all in the process of doing a build. We'd all like things to be more static, but backward compatibility prevents us from making assumptions - so we're left in a situation where of necessity, general tools have to be prepared to run arbitrary code, and execute arbitrary build steps, to work out how to install a Python package.
Most real-world situations are far easier than that, and 99.9% of the capability of setuptools is mostly unused. But generic tools have to cope with the worst case.
I don't have a good answer for you. You may be able to make simplifying assumptions that mean you can handle the problem relatively simply. But I don't know your requirements well enough to suggest how - and from what you say, I suspect you don't have the background in packaging to know how either. Sorry there's no better answer for you here...

As an example of the kind of complexity there is, one of the goals in PEP 517 was to allow the build process to determine extra requirements - e.g. extension modules which build on numpy might need to be installed with the same version of numpy they were built against.
As @pfmoore says, it really depends how general you need to make it. For lots of popular packages, it's probably sufficient to download a wheel from PyPI and extract requirements from its metadata - this can be done without installing anything or running any unknown code. But the closer you want to get to 100% coverage, the more complexity you'll need. It's a classic 80/20 situation - you can get 80% of the result with 20% of the effort (or even less...)

Thanks for the very extensive responses, this was all extremely helpful! :heart: I forgot to close this so I will do so now


Benchmark is correct?

I am doing tests since its benchmark seems to me too high, I am doing the benchmark and they give me very different results to those that appear in the image:
CPU: Intel (R) Core (TM) i7-6700 CPU @ 3.40 GHz
japronto
docker run williamyeh/wrk -c 200 -t 1  -d 4 http://<IP_ADDRESS>:8080/
Running 4s test @ http://<IP_ADDRESS>:8080/
1 threads and 200 connections
Thread Stats   Avg      Stdev     Max   +/- Stdev
Latency     0.85ms  307.50us   3.17ms   70.66%
Req/Sec   227.29k     3.84k  232.01k    75.00%
904138 requests in 4.01s, 79.33MB read
Requests/sec: 225703.67
Transfer/sec:     19.80MB
meinheld
docker run williamyeh/wrk -c 200 -t 1  -d 4 http://<IP_ADDRESS>:8080/
Running 4s test @ http://<IP_ADDRESS>:8080/
1 threads and 200 connections
Thread Stats   Avg      Stdev     Max   +/- Stdev
Latency     5.03ms    9.90ms 215.48ms   99.25%
Req/Sec    46.38k   768.92    48.44k    75.00%
184537 requests in 4.01s, 31.15MB read
Requests/sec:  46049.91
Transfer/sec:      7.77MB

Hi kianxineki you are benchmarking speed of non pipelined client (like a regular browser) which is different from what I benchmarked. To do pipelined benchmark you need to grab a lua script from misc/pipeline.lua. Then you can edit the script to get different number of pipelined requests. The benchmark from the article and README.md tests pipelined performance.

Still without request pipelining Japronto does 400,000 RPS on the same hardware.

The performance with meinheld is very similar to its graphics, I expected to have 400,000 rps on my server without a pipeline

Maybe try without docker then because it adds an extra layer of indirection between kernel and userspace. I didnt run the benchmarks inside docker. Also be sure to run them on Python 3.6 which gives some gains over 3.5

pipeline on:
docker run -v /tmp/wrk/scripts/pipeline.lua:/pipeline.lua williamyeh/wrk:4.0.2 -c 200 -t 1  -d 4 -s /pipeline.lua http://<IP_ADDRESS>:8080/
Running 4s test @ http://<IP_ADDRESS>:8080/
1 threads and 200 connections
Thread Stats   Avg      Stdev     Max   +/- Stdev
Latency     0.95ms  423.28us   4.10ms   66.73%
Req/Sec   499.02k     5.61k  508.05k    77.50%
1986543 requests in 4.01s, 174.30MB read
Requests/sec: 494990.77
Transfer/sec:     43.43MB

Are you actually running Japronto inside docker ro only wrk? Where did you get the binary release of Japronto from, did you compile them yourself?

japronto run in docker python:3.6
docker run -it python:3.6 bash
pip install japronto
git clone https://github.com/squeaky-pl/japronto.git
python3 japronto/benchmarks/japronto/micro.py

Running benchmarks in docker is not a great idea.
You can follow this script for Ubuntu https://github.com/squeaky-pl/japronto/blob/master/misc/bootstrap.sh to setup the environment exactly the same I did in the article.

Result is similar
without docker (python 3.6):
meinheld
./wrk -c 200 -t 1 -d 4 -s scripts/pipeline.lua  http://localhost:8080/
Running 4s test @ http://localhost:8080/
1 threads and 200 connections
Thread Stats   Avg      Stdev     Max   +/- Stdev
Latency     8.35ms   19.08ms 258.92ms   98.07%
Req/Sec    75.44k     1.42k   79.40k    70.00%
300155 requests in 4.01s, 50.67MB read
Requests/sec:  74921.39
Transfer/sec:     12.65MB
japronto
./wrk -c 200 -t 1 -d 4 -s scripts/pipeline.lua  http://localhost:8080/
Running 4s test @ http://localhost:8080/
1 threads and 200 connections
Thread Stats   Avg      Stdev     Max   +/- Stdev
Latency     0.93ms  386.70us   3.13ms   62.77%
Req/Sec   534.89k    72.99k  639.61k    65.00%
2129517 requests in 4.00s, 186.84MB read
Requests/sec: 531982.26
Transfer/sec:     46.68MB

I don't know what's happening then. I can start a machine tomorrow that is exactly like the machine I used for benchmarking and let you login to play with it.

@squeaky-pl nice, i testing japronto -c100 -t 1 (check my after comment)
Japronto is really fast!
Currently I use bottle (https://github.com/bottlepy/bottle) as a micro-framework, it has support for many servers, such as meinheld, bjoern, tornado ... I'm thinking of trying to adapt it to take advantage of its potential. what do you think?
Thx for fast responses 👍

For bottle to work on Japronto I would need to write a WSGI adapter for it. I would expect it to be faster than Meinheld but I dont know how much. Japronto uses Request and Response objects written in C, the router is also written in C. With WSGI it's no longer possible to use those parts written in C.

I understand that performance will decrease. Since Japronto has all the basic functionalities, the only thing I see that will be out of being able to add to the wrapper over the requests as does the bottle with its plugins, thus easy to implement general functionalities.

Closing this in favor of https://github.com/squeaky-pl/japronto/issues/21 which has a nice graph for non-pipelined results.

https://www.youtube.com/watch?v=0CPCMAcs3qo
i try make benchmarks:
5 sec
8 th
5000 con
~~30-60k
1 sec
8 threads
200 connections
140-150k per sec
=\


Calendar - Calendar Conflicting plugin commands & names

I first wanted to thank you for writing a awesome plugin. It is by far the best calendar plugin/package/addon that I have found in the editor world. It makes life so much easier.
Second, as you probably know it conflicts with matsumoto's calendar plugin. To reiterate the situation:
Both are named calendar.vim, and when inside of vim you call :Calendar either your plugin or matsumoto's plugin loads depending where it is located inside of your vimrc. Whichever plugin you are able to execute, the other you are not. Since org-mode for vim and journal.vim use matsumoto's plugin, it makes it impossible to have both enabled inside of your vimrc and/or use both at the same time.
I have attempted to find information concerning creating a location specific key map, that is a keymap that refers to a location inside of the load path. ex. /home/user/.vim/bundle/Itchyny'sAwesomeCalendarplugin/plugin/calendar.vim = <map>something. But have been unable to find information about such a keymap, or how to do it. I have even searched the plugins for vim. I usually use emacs, but your plugin is sweet. I figured it would be easier to trouble you since your plugin is cleanly written and organized, and a modification or some form there of, might be a requirement. But, I could be wrong.
Thanks,

I should apologize on this.  We cannot use both my plugin and mattn's calendar plugin. This plugin may not be able to be used with a plugin which interact with mattn's plugin.
This problem was actually supposed to appear since I was developing the plugin. I actually renamed my plugin multiple times (neo-calendar, ultimate-calendar, zcalendar or something like that) but I couldn't be satisfied with writing such function prefix again and again (like zcalendar#foo#bar). So after all I decided to name this plugin just calendar and make it a replacement of mattn's calendar. Some people blamed me for creating a conflict intentionally, and claim that how the users choose plugins should be decided by the users. I agree with the opinion but I could not come up with a better command/function prefix than calendar; this plugin is actually a calendar in the editor too.

Whoa, smoking fast response time...
No worries and no need for an apology. Great plugin.


Quick Accent Remove Toolbar

Description of the new feature / enhancement
Unsure if this is a bug, user error or a feature request...
Quick Accent is fantastic! I often get left with the Toolbar open when it's no longer needed. So far the only workaround I've discovered to resolve this is to quit PowerToys.
Scenario when this would be used?
It would be useful as it would remove the toolbar when it's not required.
Supporting information
No response

Same issue here.  Sometimes the toolbar gets stuck with no way to close it other than disabling the feature.

What version are you (both) using?

PowerToysReport_2022-12-17-14-44-37.zip

PowerToysReport_2022-12-15-12-30-33.zip

PowerToysReport_2022-12-18-09-54-22.zip

Fwd @damienleroy ?

I seen that several times. I think I have a solution about that, but need time to put it in place and test it. Hope it will be ok for the new year :)

PowerToysReport_2024-01-23-10-51-01.zip
I do have the same problem.
Latest PowerToys running v0.77.0
Character set is set to German only
The problem: Sometimes the toolbar position gets stuck, showing the last used character (ü, ä, or similar ones) for ever and ever, ever.
Solution so far: I just type another character (ö or the same one again ä)
What I wish for: Get rid of the bug, or make an option to just not showing the toolbar at all.

I seen that several times. I think I have a solution about that, but need time to put it in place and test it. Hope it will be ok for the new year :)

Any news @damienleroy ?

still same issue on v0.80.0

Same issue on x 0.80.1


fix printing error message to file  when cannot resolve highlighters …

Thanks @jsoizo, I've merged this with a tweaked log message.    Also thanks @phy1729 for reviewing/approving over IRC.


Add an initial draft for a Resin TypeScript skeleton project.

To go along with the PR review for resin-io/hq#612.
This is a draft proposal, so lots of comments required.

What do you think about adding an .editorconfig, to make keeping indentation, line endings and such consistent automatically?
For Example:
# editorconfig.org
root = true

[*]
indent_style = space
indent_size = 2
end_of_line = lf
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = true

[Makefile]
indent_style = tab

[*.{md,mkd,markdown}]
trim_trailing_whitespace = false

[*.ts]
indent_size = 4

@jhermsmeier I think an .editorconfig is a very good idea. Let's sort out the TS style conversation and then I'll add one based on the result.

A quick note on @petrosagg's last 3 comments: if we do wrap this up with Yeoman (as somebody mentioned in flowdock), we could very easily do dynamic config that'd make swapping out package.json values extremely easy. I'd just blank those out for this PR I think, but it'd be a good extension later.


[WIP] Conditionally add Content-MD5 in S3 when not sigv4

This PR conditionally adds a Content-MD5 to several S3 operations when the following constraints are met:

Not using signature version 4 (because a Content-SHA256 header is already used).
When a Content-MD5 is not already present.

I ran into a few issues with this commit because a file is being passed into the "body" parameter of a request dict, but six.b requires a buffer or string: md5.update(six.b(params['body']))

Superseded by https://github.com/boto/botocore/pull/583


RS3 only: Public port for the IIS is not accessible outside Azure even though the task is healthy

This transient failure was reproduced in the following Jenkins jobs runs:

https://mesos-jenkins.westus.cloudapp.azure.com/job/dcos-testing/541/console
https://mesos-jenkins.westus.cloudapp.azure.com/job/dcos-testing/553/console
https://mesos-jenkins.westus.cloudapp.azure.com/job/dcos-testing/558/console
https://mesos-jenkins.westus.cloudapp.azure.com/job/dcos-testing/560/console
https://mesos-jenkins.westus.cloudapp.azure.com/job/dcos-testing/561/console

This issue turned out to be a known Windows RS3 networking issue, which has been fixed in RS4.

We are moving away from RS3 because issue like this one only got fixed in the RS4


Release

@davidlmobley has requested a beta release and conda package for pymbar3.  If there are no objections, I will roll that out ASAP.  One question is whether #186 is ready for merge, in which case I can merge that before the release.

Does anyone know how to support pymbar3?
There's probably no harm in releasing a beta release, but we do need a way to handle the bugs that are found this way.  I'm still slogging through the BAR code in #186 trying to understand what it does and what the cause of instability is.

What do you mean by "does anyone know how to support pymbar3"?

Sorry, that was poorly phrased: What is our model for supporting pymbar3 once it's out in the wild?  Do we have enough expertise from the Shirts and Mobley side?  Would you have time to help deal with bugs?  Or am I going to be the one it falls to in order to fix things?
I'm just trying to avoid the situation where I'm the only one tasked with supporting a beta release where I don't have a good working familiarity of the current codebase---that's all!  If we're covered, then I have no problems with cutting a 3.0.0-beta release!  If I have to support it myself, then I'd want us to wait a bit longer.

So I think Michael has some familiarity with many of the changes.  The changes he is less familiar with mostly live here: https://github.com/choderalab/pymbar/blob/master/pymbar/mbar_solvers.py.  IMHO, these changes actually bring about substantial improvements in readability.
I think Levi has some familiarity as well.
Regarding support: I imagine I should have time for bugfixes outside of normal working hours.  That could mean minor delays on the hours to day timescale, but IMHO that's still better than most other projects.

IMHO, these changes actually bring about substantial improvements in readability.

Agreed, this is really nice.  That should be unproblematic to support.

Regarding support: I imagine I should have time for bugfixes outside of normal working hours.

That sounds great.  As long as we can bug you if we can't figure how to fix things, I'm totally fine with moving forward with the release.

And just to chime in - currently your last release is a 2.1 beta release,
which I'd argue that in some sense is probably worse to support than a 3.0
beta release (and it also effectively means you have to support something
"old" as well as the new version).
(This of course is why I'm bringing it up to begin with - harder to support
two versions of an API at once).
On Wed, May 27, 2015 at 9:26 AM, John Chodera<EMAIL_ADDRESS>wrote:

IMHO, these changes actually bring about substantial improvements in
readability.
Agreed, this is really nice. That should be unproblematic to support.
Regarding support: I imagine I should have time for bugfixes outside of
normal working hours.
That sounds great. As long as we can bug you if we can't figure how to fix
things, I'm totally fine with moving forward with the release.
—
Reply to this email directly or view it on GitHub
https://github.com/choderalab/pymbar/issues/188#issuecomment-105984847.

--
David Mobley
<EMAIL_ADDRESS>949-385-2436

For some reason the 3.0.0beta conda linux release was building on Jenkins for 1.5 hours.  Not sure what's going on there.
I killed it and will try again later.

I think it's running the tests, which take a bit longer than other
libraries.  If it doesn't work this time, we can disable the tests on the
recipe and update the build.
On Wed, Jun 10, 2015 at 3:59 PM, John Chodera<EMAIL_ADDRESS>wrote:

For some reason the 3.0.0beta conda linux release was building on Jenkins
for 1.5 hours. Not sure what's going on there.
I killed it and will try again later.
—
Reply to this email directly or view it on GitHub
https://github.com/choderalab/pymbar/issues/188#issuecomment-110941341.


bug: Cannot go to register or top-level menu when using Fedora & Wayland

Did you check docs and existing issues?

[X] I have read all the which-key.nvim docs
[X] I have searched the existing issues of which-key.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
> nvim -v
NVIM v0.10.0
Build type: RelWithDebInfo
LuaJIT 2.1.1713773202
Run "nvim -V1 -v" for more info

> bat ~/.config/astronvim/lazy-lock.json | rg which
  "which-key.nvim": { "branch": "main", "commit": "a86e9d3c0023f985344014bf2b7cb15e0be8fa2b" },

Operating system/version
Fedora 40 w/ Wayland
> fastfetch --logo none
dannyfritz@raven-fedora
-----------------------
OS: Fedora Linux 40 (Workstation Edition) x86_64
Host: 20HDCTO1WW (ThinkPad T470)
Kernel: Linux 6.9.7-200.fc40.x86_64
Uptime: 5 days, 8 hours, 14 mins
Shell: fish 3.7.0
Display (AUO243D): 1920x1080 @ 60Hz [Built-in]
DE: GNOME 46.3.1
WM: Mutter (Wayland)
Terminal: tmux 3.4

Describe the bug
When using Fedora 40 and Gnome Wayland, which-keys dismisses itself after hitting <leader> + backspace. This is intended to show the top-level keymappings. This also happens when " is hit, the UI is immediately dismissed.
wl-clipboard might be related?
https://github.com/user-attachments/assets/edde2bc2-06b3-4106-a17e-7dd351b41d5a
Steps To Reproduce

Hit <leader>
Hit backspace

Expected Behavior
I expect to see the which-keys menu showing top-level keymappings.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  { "folke/which-key.nvim", opts = {} },
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Potentially related: https://github.com/folke/which-key.nvim/issues/598

That's weird indeed. I'm also using wayland and don't have this issuse.
I'll push a debug mode later that will write to a log file. Will let you know when that's ready to further debug.

Can you enable debug with opts.debug = true and report back the contents of  ./wk.log?

I have the exact same issue. This is the log after pressing ":
on_key: "
State(start): { "Mode(n)", 'Node(")', { keys = '"', update = true, waited = 0 } }
  getchar
  on_key: <Esc>
  got: <Esc>

And this is after first pressing ', which works fine, and then <BS> to go to the top level.
on_key: '
State(start): { "Mode(n)", "Node(')", { keys = "'", update = true, waited = 0 } }
  getchar
  on_key: <BS>
  got: <BS>
  getchar
  on_key: <Esc>
  got: <Esc>
  reattach: { "<Esc>", "n" }
  feedkeys: { "Mode(n)", "<Esc>" }

Damn ok, I think I know what's causing this.
There's a bug/feature with wl-copy indeed where if you get its contenst it steals focus.
which-key closes on focus lost.
I'm using hyprland, and there the issue doesn't exist, but I had a similar issue at some point with yanky on Gnome.
Will see if I can improve this.

Should be fixed. Would be great if you can confirm

It is, thanks a lot! :zap: speed :D

Also confirmed it is fixed. Thanks!


Remove RES and REF typedefs in favor of spelled out Ref<>

These typedefs don't save much typing compared to the full Ref<Resource> and Ref<RefCounted>, yet they sometimes introduce confusion among new contributors.
See https://github.com/godotengine/godot/issues/33027#issuecomment-1110387546.

I don't understand why there is a REF either. So this makes sense to remove confusing features.

Thanks!


Publish to npm

Want to use this with webpack via npm. The current version on npm is 0.0.3. Could you publish an update please?

Just noticed the version on npm is not yours, but someone else's. Could you please publish your library to npm? Thanks!

ah.. someone got the name on npm.. !! you can do a bower install.

I know I can do a bower install. But my project is using npm and I'd prefer to only have one package manager in my project. If the name matters that much to you, perhaps @keyvanfatehi would be willing to give it up. The repo isn't even there anymore. @keyvanfatehi would you be willing to unpublish your package in favor of @a5hik's?

will do

@a5hik @kentcdodds
name released1

Thanks @keyvanfatehi! :-D

Thanks @keyvanfatehi ., @kentcdodds it is published now.

https://www.npmjs.com/package/ng-sortable

Thanks @a5hik!


Prometheus Operator Support and Change Socket Path

Fixes #7
Adds Prometheus Operator support so scraping and alerts can be managed through Kubernetes resources. Disabled by default so shouldn't affect existing installations.
I've also added the option to change the containerd path which is needed to run on Amazon Bottlerocket. Happy to move that to an separate PR if needed.

I've forked the repo and am hosting my own Helm repository with this change and #11 along with Github actions builds, for the time being.

Please merge this PR, it's useful


For uv pip compile don't output requirements to stdout when output file selected

Or at least have an option for this? Maybe it already exists and I missed it?
My scenario is I am building a set of requirements with uv pip compile via a script and have an output file selected. uv gives some potentially useful warnings, e.g.
warning: The requested Python version 3.10.6 is not available; 3.11.4 will be used to build dependencies instead.
Resolved 360 packages in 124ms
warning: The package `google-api-core==2.8.2` does not have an extra named `grpcgcp`.

However it's easy to miss these warnings because the screen is quickly then filled with the 360 packages of the output requirements, however I don't need these output requirements on the screen because they are written to a file.

Does -q work for you?
I believe we omitted this output by default initially but added it to match pip compile's behavior.

Does -q work for you?

With -q I see no messages from uv, including warnings.

I believe we omitted this output by default initially but added it to match pip compile's behavior.

Fair enough, I'm not that familiar with pip compile's specific behaviors.

IDK I'm torn on it. I'd change it back. I find it strange but yes we did it for compatibility.

FYI I tested with -q using pip-compile and it displays some, but not all, warnings.  Though it's docs state "-q, --quiet                     Give less output", where as uv's state "-q, --quiet Do not print any output".
uv's use of -q makes more sense to me, I'm sure some users will not want any output.

Oh I think the workaround people have used in the past is to pipe stdout to a /dev/null since warnings are on stderr and this output is on stdout?
Maybe we should just have a --no-stdout flag? hm.

Oh I think the workaround people have used in the past is to pipe stdout to a /dev/null since warnings are on stderr and this output is on stdout?

Ah thanks, this is sufficient for me!
Feel free to close this issue if you would rather keep compatibility / not add any more flags.


⚠ Bofrost.at has degraded performance

In 9869138, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2390 ms

Resolved: Bofrost.at performance has improved in bf61c82.


Support for [Fuga melodies of steel 2] CyberConnect2Co.Ltd.FugaMelodiesofSteel2_zpv0gf6t8hz5r

Game name: Fuga melodies of steel 2

CyberConnect2Co.Ltd.FugaMelodiesofSteel2_zpv0gf6t8hz5r
Game package name: CyberConnect2Co.Ltd.FugaMelodiesofSteel2_zpv0gf6t8hz5r

wgs.zip

The newest release now support this game. Please try it out and report back here if it works or not.
https://github.com/Z1ni/XGP-save-extractor/releases


Memory leak in xds_proxy

https://github.com/istio/istio/blob/efef026fa83791b917d064c1ec91d4d95b9b1a32/pkg/istio-agent/xds_proxy.go#L391 is susceptible to a memory leak. This can occur when all reader are blocked, and responsesChan has hit capacity (10).
All readers can be blocked if the consumer is slow: https://github.com/istio/istio/blob/efef026fa83791b917d064c1ec91d4d95b9b1a32/pkg/istio-agent/xds_proxy.go#L455. This can be triggered by making the downstream.Send slow (reproduced in tests, with artificial delays), or by requestsChan being full and blocking.
This is far worse in 1.8, which can also be blocked by slow upstream.Send (https://github.com/istio/istio/blob/1.8.6/pkg/istio-agent/xds_proxy.go#L294).
Every time the readers have exited (from conn.stopped closing), but we are still attempting to write on responsesChan, we will leak a goroutine+a channel with 10 discovery responses.
In real world (on 1.8, which again, is more susceptible to this), we saw roughly 200 occurrences of such a leak, leading to (presumably, no hard data for this) 2k DiscoveryResponses kept in memory, which took up 20gb of RAM and ultimately led to repeated OOMs.

@kyessenov can take a look at this ?


Background error on Firefox

In Firefox the background doesn't show correctly.

Fixed. Issue closed.


Action2 parameters are flipped

public <TArg0> StateConfiguration<S, T> onEntryFrom(TriggerWithParameters1<TArg0, S, T> trigger, final Action2<**TArg0, Transition<S, T>**> entryAction, Class<TArg0> classe0) {
But while firing, the entry action execution flips the parameters-
void executeEntryActions(Transition<S, T> transition, Object[] entryArgs) {
...
while(var3.hasNext()) {
            Action2<**Transition<S, T>, Object[]**> action = (Action2)var3.next();
            action.doIt(transition, entryArgs);
        }

I'll look into this during the upcoming weekend. Thanks for the report.

Is this something you are supporting?

Quoted code snippet does not exist in library source code.


Implement LFOs v1

This implements LFO v1 mostly.
Some opcodes still not done. (that's the depthcc, also to implement in v2)
Based on the large refactor PR.
Note that amplfo controls Volume not Amplitude. (outputs additive decibels)
All 3 LFO have behavior matched to ARIA.
EDIT(1) correct an implementation bug which would introduce a regression in v2 LFO

Merging this because it passes tests, checked against ARIA in various cases, and for avoidance of future conflicts following the refactor.


Fix typo in method name computePagingLimts

Fixes https://github.com/almighty/almighty-core/issues/1251

Codecov Report

Merging #1250 into master will decrease coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1250      +/-   ##
==========================================
- Coverage   59.74%   59.71%   -0.03%
==========================================
  Files         120      120
  Lines       10543    10543
==========================================
- Hits         6299     6296       -3
- Misses       3594     3595       +1
- Partials      650      652       +2

Impacted Files
Coverage Δ

controller/space_codebases.go
70% <100%> (ø)
:arrow_up:

controller/search.go
55.55% <100%> (ø)
:arrow_up:

controller/paging.go
100% <100%> (ø)
:arrow_up:

controller/work_item_comments.go
75% <100%> (ø)
:arrow_up:

controller/namedspaces.go
50.9% <100%> (ø)
:arrow_up:

controller/workitem.go
79.32% <100%> (-0.68%)
:arrow_down:

controller/collaborators.go
70.76% <100%> (ø)
:arrow_up:

controller/planner_backlog.go
73.33% <100%> (ø)
:arrow_up:

controller/space.go
74.74% <100%> (ø)
: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 8e825e2...34a3ca8. Read the comment docs.

[test]

[test]

[test]


Update documentation to include Go SDK

Signed-off-by: pama-ibm<EMAIL_ADDRESS>Since the Go SDK officially release v1.0.0 recently, we need to update references to it.
Type of change

Documentation update

Description
Everywhere that we talk about the available SDKs, I appended the Go SDK, including a link to the GitHub repo, README, and docs.

@Mergifyio backport release-2.3


Error [ERR_REQUIRE_ESM]: Must use import to load ES Module:

Check that this is really a bug

[X] I confirm

Reproduction link
9
Bug description
9
Expected Behavior
Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: D:\Konekt\Projects\Konekt-Dev\parkfinda-website-test\node_modules\swiper\react\swiper-react.js
require() of ES modules is not supported.
require() of D:\Konekt\Projects\Konekt-Dev\parkfinda-website-test\node_modules\swiper\react\swiper-react.js from D:\Konekt\Projects\Konekt-Dev\parkfinda-websi
te-test.next\server\pages\index.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js
files in that package scope as ES modules.
Instead rename swiper-react.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from D:\Konekt\Projects\Konekt-Dev\parkfi
nda-website-test\node_modules\swiper\package.json.
Actual Behavior
.next\server\pages\index.js (1:0) @ Object.swiper/react

1 | module.exports = require("swiper/react");;

Swiper version
9
Platform/Target and Browser Versions
Chrome,
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[X] I'm willing to open a PR

https://github.com/nolimits4web/swiper/issues/4871


[Fix #4153] Add a new cop Lint/UselessReturn

My go at https://github.com/bbatsov/rubocop/issues/4153
If you have any suggestions or see any problems, tell me and I'll update the code

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[ ] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

I think this cop is attempting to address two different problems.

The return value from initialize and setter methods is useless. ✔
Conditionally returning from these methods should be changed to a condition without the return.  @Drenmi's example in #4153 conditionally calls baz at the end, so I wouldn't call that return "useless".
(That second one seems hard 😃 )

@mikegee True enough. For the second condition actually a return may be useful (to return from the method) but the return value is not :
class A
  def val=(val)
    @val = val
    return 'WORKED'
  end
end

If we run in irb :
2.4.0 :009 > A.new.val = 5
 => 5

return value  'WORKED' is "shadowed"
Maybe return could be used but only without argument ?

I think the name opens up for some confusion with RedundantReturn, but off the top of my head I don't have any better suggestion either. 😅

ReturnInVoidContext perhaps?

I guess the best solution would be to change the cop to only allow return with no value.

That'd be my suggestion.

Thanks for all the input. I'll update the PR beginning of next week :)

@harold-s Ping :-)

Sorry, updated the pull request with all your suggestions (Thanks !) :)

@harold-s ping :-)

@harold-s Ping again :-)

Sure.


Readme notice?

Hi,
I'm trying to understand the notice in the readme, but when I do this I have a bunch of errors like this one:
 error TS2835: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Did you mean '../models/OffsetParameter.js'?

I have actually made a similar plugin for esbuild and found this repo while trying ti decipher all this.
So I'm not sure if this is still an issue or not and if not, I'm not sure of the approach?
Thanks

My bad it's actually the library I'm building with that is using an older version of TS, I though it was using the user TS version....
https://github.com/egoist/tsup/issues/733


fixed issue #754 - [closed]

In GitLab by @liyanq528 on Dec 11, 2017, 08:49
Merges 20171205 -> dev

In GitLab by @wknet123 on Nov 13, 2019, 10:17
the coverage is  7.7% check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:17
the coverage is  7.7% check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:17
closed


The first epoch downloaded has incomplete data

Description
The first epoch downloaded when running goteth will allways have the PrevState.EmptyStateRoot() flag on. This means that this epoch will be stored with incomplete data on the database.
Example from phase0 but applies to all forks:
https://github.com/migalabs/goteth/blob/3387cd458052581896abe644c1ffac79e56f27b2/pkg/spec/metrics/state_phase0.go#L40-L46

A possible solution (and probably easiest) is to start downloading from the epoch before the set one. This would require handling the case of epoch 0 as an exception.


Fix index mapping bug of contains, array_position and subscript

details in https://github.com/facebookincubator/velox/issues/6346
The cardinality of a top-level vector, such asConstantVector<ComplextType>, may be greater than its base vector, so we should use elementsDecoded and its indices to process each element.

Let's remove these functions from the skip list in velox/expression/tests/ExpressionFuzzerTest.cpp and verify that fuzzer no longer fails.

@mbasmanova I've removed contains, subscript, and array_position and the fuzzer test looks fine.

I've removed contains, subscript, and array_position and the fuzzer test looks fine.

Thanks. Let's update PR title and description to mention subscript as well?

the fuzzer test looks fine.

Would you share the command line you used to run the fuzzer? Did this command fail without the fixes? Let's run fuzzer for 40+ minutes to ensure there are no issues remaining.

Would you share the command line you used to run the fuzzer? Did this command fail without the fixes?

I used to run and reproduce the issue in my ide by running velox_expression_fuzzer_test with parameters --enable_dereference --seed 1 --enable_variadic_signatures --velox_fuzzer_enable_complex_types --lazy_vector_generation_ratio 0.2 --velox_fuzzer_enable_column_reuse --velox_fuzzer_enable_expression_reuse --max_expression_trees_per_step 2 --retry_with_try --enable_dereference --duration_sec 60 --logtostderr=1 --minloglevel=0 --repro_persist_path=./tmp/fuzzer_repro --only contains.
After removing these functions from the skip list, I rerun the test with --only contains,subscript,array_position, and the fuzzer test of the ci is green.

Let's run fuzzer for 40+ minutes to ensure there are no issues remaining.
OK, will run locally.

@mbasmanova I used the following command to run the fuzzer and all passed.
_build/debug/velox/expression/tests/velox_expression_fuzzer_test --enable_dereference --seed 0 --enable_variadic_signatures --velox_fuzzer_enable_complex_types --lazy_vector_generation_ratio 0.2 --velox_fuzzer_enable_column_reuse --velox_fuzzer_enable_expression_reuse --max_expression_trees_per_step 2 --retry_with_try --enable_dereference --duration_sec 2400 --logtostderr=1 --minloglevel=0 --repro_persist_path=./tmp/fuzzer_repro 2>&1 | tee tmp/test.log
$ head test.log
W20230906 02:07:11.387956 537432 ExpressionFuzzer.cpp:457] Skipping unsupported signature with generic: array_frequency(array(hugeint)) -> map(hugeint,integer)
W20230906 02:07:11.394208 537432 ExpressionFuzzer.cpp:457] Skipping unsupported signature with generic: combinations(array(hugeint),bigint) -> array(array(hugeint))
W20230906 02:07:11.395435 537432 ExpressionFuzzer.cpp:457] Skipping unsupported signature with generic: array_union(array(hugeint),array(hugeint)) -> array(hugeint)
W20230906 02:07:11.395814 537432 ExpressionFuzzer.cpp:457] Skipping unsupported signature with generic: array_has_duplicates(array(hugeint)) -> boolean
W20230906 02:07:11.397225 537432 ExpressionFuzzer.cpp:457] Skipping unsupported signature with generic: array_remove(array(hugeint),hugeint) -> array(hugeint)
W20230906 02:07:11.404963 537432 ExpressionFuzzer.cpp:457] Skipping unsupported signature with generic: trim_array(array(hugeint),bigint) -> array(hugeint)
W20230906 02:07:11.406005 537432 ExpressionFuzzer.cpp:469] Skipping non-deterministic function: rand(integer) -> integer
W20230906 02:07:11.406030 537432 ExpressionFuzzer.cpp:469] Skipping non-deterministic function: rand(smallint) -> smallint
W20230906 02:07:11.406049 537432 ExpressionFuzzer.cpp:469] Skipping non-deterministic function: rand(bigint) -> bigint
W20230906 02:07:11.406066 537432 ExpressionFuzzer.cpp:469] Skipping non-deterministic function: rand(tinyint) -> tinyint

$ tail test.log
I20230906 03:00:31.013603 537432 ExpressionFuzzer.cpp:1131] split 177 0.05% 4920
I20230906 03:00:31.013607 537432 ExpressionFuzzer.cpp:1131] combinations 176 0.05% 6199
I20230906 03:00:31.013612 537432 ExpressionFuzzer.cpp:1131] split_to_map 99 0.03% 1837
I20230906 03:00:31.013615 537432 ExpressionFuzzer.cpp:1131] array_frequency 93 0.03% 1547
I20230906 03:00:31.013619 537432 ExpressionFuzzer.cpp:1131] json_extract 31 0.01% 895
I20230906 03:00:31.013624 537432 ExpressionFuzzer.cpp:1131] json_parse 27 0.01% 1611
I20230906 03:00:31.013628 537432 ExpressionFuzzer.cpp:1131] empty_approx_set 5 0.00% 0
[==========] Running 0 tests from 0 test suites.
[==========] 0 tests from 0 test suites ran. (0 ms total)
[  PASSED  ] 0 tests.

@laithsakka I've rebased and pulled. All ci checks passed, could you please help to reimport?


Receiving data after node deletion

If we have data watcher on some node, data changes will not be received after deleting and creating node again.
This behaviour can be reproduced with slightly changed test from aiozk.test.test_watchers
@pytest.mark.asyncio
async def test_data_watch_delete(zk, path, data_watcher):
    data = []
    ready = asyncio.Event()
    test_data = b'test'

    async def data_callback(d):
        data.append(d)
        ready.set()

    await zk.set_data(path, test_data)

    data_watcher.add_callback(path, data_callback)
    await asyncio.sleep(0.2)
    assert data == [test_data]
    ready.clear()
    await zk.delete(path)

    await asyncio.wait_for(ready.wait(), timeout=1)
    assert ready.is_set()
    assert data == [test_data, NoNode]
    # data_watcher.remove_callback(path, data_callback)

    ready.clear()
    data = []
    await zk.create(path)
    await zk.set_data(path, test_data)
    await asyncio.sleep(0.2)
    assert data == [test_data]
    ready.clear()

Is this behaviour expected?
kazoo, for example, will received data in same situation.

@kammala until now I considered it as expected behavior: path was deleted - nothing to monitor and you must handle it yourself.
But since recipes can have quite high level logic over zk basics - maybe we can change this behavior to something that can fit requirements better.
So we have several options here:

add an option, like survive_deletion
a. True by default - if it's desired behavior by default
b. False by default - to support backward compatibility, but I belive it shouldn't break something important
add an option to handle node deletions/creation separately. in case you have some logic you rely on for these events and by default handle here deletion/creation.
add one more recipe with such support

If we do 1 - we also should consider this option for other recipes too.
In general I'm ok with 1 or 2. Maybe 2nd variant even better from point of extensibility/maintanability.
So @kammala

do you have anything to add?
do you need and want to implement it yourself or maybe you want to wait when it will be implemented by me?

I think, that second option is preferable.
For example, we can add on_delete_callback to DataWatcher(or even BaseWatcher) and call it instead of calling on_data_change_callback with aiozk.exc.NoNode as data.
For API completeness we may add on_create_callback too, but I don't think that it would be very useful.
I'll try to provide pull request, but I don't know how quickly it will be ready

I'll try to provide pull request, but I don't know how quickly it will be ready

Ok. Let me know if you need something


Install dependencies on activation and update README for usage

Issue #, if available:
#2
Description of changes:

Prompts for the installation of dependencies
Updates Readme to call out requirement of AWSTemplateFormatVersion

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

thanks @Arcanemagus I will get this cleaned up.


Compiler bug for parameter declaraion in body of annotation functions

For Ceylon 1.3.1 (from IntelliJ using "Rebuild project"), the following gives
Error:(22, 20) ceylon: compiler bug: annotation constructors may only contain a return statement. No error markers in the editor window are shown.
Apart from the return statement, only parameter declarations are in the method body. This should be equivalent to declaring them directly in the parameter list, and so should be allowed.
"Annotates a method to define an HTTP endpoint."
shared annotation EndpointAnnotation endpoint(path="", method=get, consumes=[], produces=[])
{
    "The path under which the endpoint is served.
     See [[de.dlkw.conjurup.core::Server.addEndpoint]] and
     [[de.dlkw.conjurup.core::EndpointScanner]]."
    String path;

    "The HTTP method for accessing the endpoint."
    AbstractMethod method;

    "A list of MIME types that the endpoint accepts. \"\\*/\\*\" can be used
     to mean any MIME type. Specifying empty (default) is equivalent
     to [\"\\*/\\*\"]."
    String[] consumes;

    "A list of MIME types that the endpoint can produce.
     \"\\*/\\*\" may of course not be used. The value used if empty (default)
     is specified depends on the HTTP method: [\"application/x-www-form-urlencoded\"]
     for POST, [\"text/plain\"] otherwise."
    String[] produces;

    return EndpointAnnotation(path, method, consumes, produces);
}

So then this is a backend bug, not typechecker? Does it work in JS?

Yes, no compile error for JS

Yeah, looks like just an over-zealous sanity check:
// compiler-java/src/com/redhat/ceylon/compiler/java/codegen/AnnotationModelVisitor.java
    @Override
    public void visit(Tree.Statement d) {
        if (annotationConstructor != null) {
            if (!(annotationConstructor instanceof Tree.MethodDefinition
                        && d instanceof Tree.Return)
                    && !d.equals(annotationConstructor)) {
                d.addError("compiler bug: annotation constructors may only contain a return statement", Backend.Java);
            }
        }
        super.visit(d);
    }

@tombentley this should be trivial to fix.


Verify "Largest Series Product" exercise implementation

There was some confusion in this exercise due to the ambiguous use of the term consecutive in the README. This could be taken to mean contiguous, as in consecutive by position, or as in consecutive numerically. The the README has been fixed (https://github.com/exercism/x-common/pull/200).
Please verify that the exercise is implemented in this track correctly (that it finds series of contiguous numbers, not series of numbers that follow each other consecutively).
If it helps, the canonical inputs/outputs for the exercise can be found here:
https://github.com/exercism/x-common/blob/master/largest-series-product.json
If everything is fine, go ahead and just close this issue. If there's something to be done, then please describe the steps needed in order to close the issue.

These test look the same as ruby and I looked over the test and I don't think the swift track is affected.


Dogfood: Use maven-publish in gradle/gradle build

Instead of using the maven plugin for publishing libraries in our own build, we should use the maven-publish plugin. See https://github.com/gradle/gradle/blob/91467bbd010be0562c6036e8d722430f878332f3/buildSrc/subprojects/plugins/src/main/kotlin/gradlebuild/publish-public-libraries.gradle.kts#L29-L31

@marcphilipp Any chance you can look into this?

I briefly looked into this.
We're using custom configurations and custom logic to generate POMs:
https://github.com/gradle/gradle/blob/e9332b4957b49538b63d2af05fa72eea27c4494d/buildSrc/subprojects/plugins/src/main/kotlin/org/gradle/plugins/publish/GeneratePom.kt#L24-L62
Since we're lacking a way to define custom software components (cf. https://github.com/gradle/gradle/issues/1842), it doesn't seem like we're ready to switch to maven-publish just yet.

If we use internal API now and not solve #1842 shortly after, we might forget about it. I think it should be a connected attempt once we've committed to #1842.

I don't think we need any of that stuff any more. It's there from the days when the tooling API was not distributed as a shaded jar. Now, we only need a way to publish the tooling API shaded jar with slf4j-api as its only dependency.
To do this, we don't necessarily need a software component instance. This only really becomes important once we start publishing Gradle metadata for the tooling API (which I wouldn't do yet). We can instead add the shaded jar (and source and javadoc jars) as artfacts directly to a publication and then tweak the pom XML to add the dependency. I wouldn't block switching to the maven-publish plugin on us adding an API to define software components.

Done: #11743


Sprites on top of each other problem

When there are two sprites on top of each other, the handleTouchDown method has a weird behaviour. The sprite that is on top is the one showing but handleTouchDown will interact with the sprite on the bottom.

Apparently the problem was in the implementation,  this is the code
public void handleTouchDown(int x,int y, int pointer){         // handle sprites created by screen         for (Sprite s:sprites) {             if(s.contain(x,y)){                 s.setDragged(true);                 dragedSprite=s;                 break;             }             else{                 s.setDragged(false);                 dragedSprite=null;             }         }     }
This code is taking the sprite that was drawn first and taking it as the one that was clicked. Iterating through the Array should be backwards. which makes sense.


Jobs sitting in enqueued

After I save a record, it adds a new job into the ts_delta queue. The queue is building up, and not running through them. I saw in the README that I need to monitor the ts_delta queue, but I don't understand what that means. I'm assuming I need some worker that has sidekiq_options :queue => :ts_delta, but what would I put in that worker?

I´m having the same issue here. I have a sidekiq.yml file like this
:concurrency: 5
staging:
  :concurrency: 10
production:
  :concurrency: 3
:queues:
  - [default, 1]
  - [mailers, 12]
  - [import, 5]
  - [ts_delta, 1]
  - [map_bbr_data_to_building, 20]
  - [touch_family_later, 2]

Tried to change the Procfile, same problem. Except the other queue is now working
worker: bundle exec sidekiq -c 10 -q default,2 -q import,10
workerdelta: bundle exec sidekiq -c 10 -q ts_delta

Out of 10 processes, seems like only 1 is working - and its really slow

Hi @casiodk - I'm guessing this is related to a ticket lodged with Heroku and Flying Sphinx (which is now assigned to me)? :)
Can you let me know which versions of the flying-sphinx and thinking-sphinx gems you're using?

yes :)
I´ll answer there
Thanks a lot

To fix the issue that @casiodk has raised: Thinking Sphinx wasn't defaulting to quiet delta output, which in turn was slowing Flying Sphinx down. TS now defaults to quiet deltas: pat/thinking-sphinx@7d97a98c3562eb970351ad8e24312f266a523769.


Error Running Test on 'Successful' Snapshot

This setup and these instructions are great! Thanks!
I went through all of the instructions a couple times, but I'm getting an error when the Lambda tries to run:
Exception  argument of type 'NoneType' is not iterable
...
[ERROR] UnboundLocalError: cannot access local variable 'VolumeId' where it is not associated with a value
Traceback (most recent call last):
File "/var/task/lambda_function.py", line 34, in lambda_handler
process_message(message)
File "/var/task/lambda_function.py", line 90, in process_message
batch_item_failures.append({"itemIdentifier": message['messageId'], "VolumeID": VolumeId, "Region": body['region']})
Any thoughts on why its having issues pulling the VolumeId?  I tried adding "DescribeVolumeAttribute" to the EC2 permissions, but received the same error.

Hi, sorry for the delay.
Here the question is why there is an Exception?
This is not related to IAM permissions, indeed, it is a problem in the lambda function input.
Are you running a test with a JSON or you created the snapshot from volume to trigger the EventBridge rule till the lambda?
Please post the full log: the actual issue is not related to volumeID, but why there is an exception.
I deployed everything from scratch, following the instructions. Then I created a snapshot from a volume with source_tag_Key="EBS-Snapshot" and source_tag_Value="LambdaRecovery" and I did not experience this issue.

I triggered the Eventbridge from a Successful snapshot.  I'm going to delete everything I created and try again.  I will update you when I've completed that.  Thanks for the response!

Deleted everything from the previous attempt and started from scratch.  Got the same error when the Lambda function is triggered.  So it looks like everything is triggering correctly, just when the Python script runs, we get this error...here's the full error:
START RequestId: 4b078ef6-d7ad-5139-8d68-d16b35c40007 Version: $LATEST
BEFORE - Source Tag  {'Key': 'xx-xxx-EBS-Recovery', 'Value': 'xxx-LambdaSnapshotRecovery'}
Exception  argument of type 'NoneType' is not iterable
LAMBDA_WARNING: Unhandled exception. The most likely cause is an issue in the function code. However, in rare cases, a Lambda runtime update can cause unexpected function behavior. For functions using managed runtimes, runtime updates can be triggered by a function change, or can be applied automatically. To determine if the runtime has been updated, check the runtime version in the INIT_START log entry. If this error correlates with a change in the runtime version, you may be able to mitigate this error by temporarily rolling back to the previous runtime version. For more information, see https://docs.aws.amazon.com/lambda/latest/dg/runtimes-update.html
[ERROR] UnboundLocalError: cannot access local variable 'VolumeId' where it is not associated with a value
Traceback (most recent call last):
  File "/var/task/lambda_function.py", line 34, in lambda_handler
    process_message(message)
  File "/var/task/lambda_function.py", line 90, in process_message
    batch_item_failures.append({"itemIdentifier": message['messageId'], "VolumeID": VolumeId, "Region": body['region']})
END RequestId: 4b078ef6-d7ad-5139-8d68-d16b35c40007
REPORT RequestId: 4b078ef6-d7ad-5139-8d68-d16b35c40007	Duration: 953.94 ms	Billed Duration: 954 ms	Memory Size: 128 MB	Max Memory Used: 90 MB

Thanks again for the response, and any thoughts on what might be going on.

The actual issue is the "None Type" which leads to the exception.
I suggest in the lambda to print the event and the message, so you have it in the log stream, to intercept this "None Type".


Preserve(archive) cleared terminal output

After clearing the terminal output (CMD+K on Mac OS) it happens from time to time that there was actually sth. in the output that you need to check again but it's lost after clearing.
It would be great if the output could be stored somewhere so that it can be examined even after clearing the terminal.
One option would be to introduce sth. like pages in the terminal. Clearing would then just create a new page but you could jump back to previous pages (even though you cannot execute commands in the previous pages).
Also the terminal search could then be extended to allow searching in previous pages.
I would be happy to implement such a feature but I'm not sure where to look at(should this even be implemented in vscode or in the underlying terminal).

/duplicate #134448

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

@meganrogge The issue you linked is a bug.
This is a feature request and unrelated to the linked bug.
When creating the issue I selected feature request but somehow the feature request label is not there and I don't have rights to add it.

I don't see a nice way to do this without overcomplicating things or introducing a feature that nobody knows about or uses as this seems extremely niche.
One related idea is to allow logging all terminal output to a file which would solve your problem, just a little less conveniently. I'm not sure we're tracking that but I know some terminals have it, it would likely be a backlog candidate though to see if there is community interest before we invest. The concerns with that feature is how and when do we start to delete those logs so we don't fill up the disk.
You may just want to stop using cmd+k to clear if you often hit this issue?

My usecase is, that I'm executing some test that produces thousands of lines of output.
If I don't clear the output before running the test it is quite easy to confuse the output of the current test run with the previous one.

You may just want to stop using cmd+k to clear if you often hit this issue?

That's not really an option for me, except I have other ways to distinguish the different test runs.

I don't see a nice way to do this without overcomplicating things

Yeah, got your point. One thing that comes to my mind is having sth. like a "mark-output-as-old" command instead of clearing the terminal.
So instead of clearing the console, the current output gets somehow grayed out (not sure about colored output) and the new output will be as usual. This would make it much easier to differentiate the different outputs (works only for one output history though).

@ninok recently we added a feature to tasks and terminals (when shell integration is on) that gives decorations for commands that get run. This allows ctrl/cmd+up/down to navigate quickly to where commands were run. So this in combination with an increased scrollback should work for you?

Thanks @Tyriar,
that is a pretty nice feature that I did not even notice before 😄
There are also small indicators in the scrollbar that show me if my search result in relation to the executed commands.
So this suffices for my use case. Closing this feature request.


Fix links for Openstack docs

What this PR does / why we need it:
Fixes the links in the Openstack docs.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

/test all


[SPARK-43541][SQL][3.4] Propagate all Project tags in resolving of expressions and missing columns

What changes were proposed in this pull request?
In the PR, I propose to propagate all tags in a Project while resolving of expressions and missing columns in ColumnResolutionHelper.resolveExprsAndAddMissingAttrs().
This is a backport of https://github.com/apache/spark/pull/41204.
Why are the changes needed?
To fix the bug reproduced by the query below:
spark-sql (default)> WITH
                   >   t1 AS (select key from values ('a') t(key)),
                   >   t2 AS (select key from values ('a') t(key))
                   > SELECT t1.key
                   > FROM t1 FULL OUTER JOIN t2 USING (key)
                   > WHERE t1.key NOT LIKE 'bb.%';
[UNRESOLVED_COLUMN.WITH_SUGGESTION] A column or function parameter with name `t1`.`key` cannot be resolved. Did you mean one of the following? [`key`].; line 4 pos 7;

Does this PR introduce any user-facing change?
No. It fixes a bug, and outputs the expected result: a.
How was this patch tested?
By new test added to using-join.sql:
$ PYSPARK_PYTHON=python3 build/sbt "sql/testOnly org.apache.spark.sql.SQLQueryTestSuite -- -z using-join.sql"

and the related test suites:
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly org.apache.spark.sql.hive.HiveContextCompatibilitySuite"

Authored-by: Max Gekk<EMAIL_ADDRESS>Signed-off-by: Max Gekk<EMAIL_ADDRESS>(cherry picked from commit 09d5742a8679839d0846f50e708df98663a6d64c)

@dongjoon-hyun @cloud-fan Could you review this backport, please.

Merged to branch-3.4 for Apache Spark 3.4.1.


PointStyle Not work with lockerService enable

Hi,
i use the Version: 2.6.0 chart js. I have set the pointStyle with an image:
var icon = new Image();
'icon.src = 'https://assets0.uvcdn.com/pkg/shared/favicons/password-d904f06a67a7b29d489489d3a35f548b.png';
and into dataset object set "pointStyle: icon".
This is not work in lightning with lockerService enable. Exist an workaround?
Thanks

@ValenteMarco this is in salesforce? Unfortunately none of the core contributors know much about it. Perhaps try asking on StackOverflow to see if you can get a better answer

Yes is salesforce. Thanks.


Any real-time deregistration notification to AVS client ?

Hi, AVS team
AVS SDK Version: 1.8
We have developed an independent companion APP for users to interact with our Alexa enabled devices.
Users have also been instructed to register/deregister our Alexa enabled products with dedicated APP rather than Amazon Alexa APP.
However, users still can deregister an Alexa device via Amazon Alexa APP. While triggering a deregistration on Amazon Alexa APP, is there any deregistered notification sent to AVS client immediately to let AVS client inform companion APP  ?
In addition, as following link indicates "a user interaction on the Amazon Alexa app would trigger  ResetUserInactivity  directive", is it a good way to detect deregistration from Alexa APP by observing ResetUserInactivity  directive ? If not, could you advice us to achieve a better way to get real-time deregistration notification ?
https://developer.amazon.com/docs/alexa-voice-service/system.html#resetuserinactivity
Best Regards,
Justy Huang

Hi @justy0714,
ResetUserInactivity is not tied to deregistration. It is used to reset the timer that would signal a user being inactive (see https://developer.amazon.com/docs/alexa-voice-service/system.html#resetuserinactivity)
We are still improving the device deregistration experience. For now, please check the answer provided in #339.
Thanks,
Kenneth