No autocomplete with C#

The LSP only works partially.
It has to major problems:

No autocompletion

No autocompletion from the LSP only the built in from Sublime appears, even though I have explicitly disabled it in the LSP settings.

No reloading of buffer:

It does not reload when I edit the buffer (so only initial analysis). If I delete a line and then hover over a symbol beneath it, it gives me the old line, indicating again that the LSP does not accommodate to the buffer's contents.
I don't think it is an LSP problem, since it works fine in Neovim.
OS and language server
I use Arch Linux and the latest release of the OmniSharp Roslyn language server (1.34)
How you installed LSP (Package Control or from git?)
I installed the development version via cloning the git repository and then satisfying dependencies from within Sublime.
Minimal reproduction steps

Install Sublime.
Install the LSP plugin
Download the release of the OmniSharp Roslyn LSP (see above) and extract.
Add the following settings to the LSP settings:

{
	"auto_show_diagnostics_panel_level": 4,
	"clients":
	{
		"C#":
		{
			"command":
			[
				"/home/tb/prebuilt/omnisharp/OmniSharp.exe",
				"-lsp"
			],
			"enabled": true,
			"languageId": "csharp",
			"syntaxes": ["Packages/C#/C#.sublime-syntax"],
			"scopes":
			[
				"source.cs"
			]
		}
	},
	"log_payloads": true,
	"log_stderr": true,
	"show_code_actions_bulb": true,
	"show_diagnostics_count_in_view_status": true,
	"show_diagnostics_in_view_status": false,
	"show_diagnostics_phantoms": false,
	"show_references_in_quick_panel": true,
	"only_show_lsp_completions": true
}

The following is an excerpt from the log. It does not log anything when I try to trigger autocomplete.
C#: Finding descriptor for textDocument/hover
C#: Found attributes 1, ::file:///home/tb/Project/Services/ILoginService.cs
C#: Looking for handler for descriptors textDocument/hover
C#: Checking handler textDocument/hover:OmniSharp.LanguageServerProtocol.Handlers.HoverHandler
C#: Registration options (null)
C#: Document Selector (null)
C#: Handler Selected: OmniSharp.LanguageServerProtocol.Handlers.HoverHandler via (null) (targeting OmniSharp.Extensions.JsonRpc.IJsonRpcRequestHandler`2[[OmniSharp.Extensions.LanguageServer.Protocol.Models.HoverParams, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022],[OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022]])
C#: Starting: Routing Request (2) textDocument/hover
C#: Converting params for Request (2) textDocument/hover to OmniSharp.Extensions.LanguageServer.Protocol.Models.HoverParams
C#: Result was System.Threading.Tasks.Task`1[[OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022]]
C#: Response value was OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover
C#: Finished: Routing Request (2) textDocument/hover in 97ms
C#: Finding descriptor for textDocument/hover
C#: Found attributes 1, ::file:///home/tb/Project/Services/ILoginService.cs
C#: Looking for handler for descriptors textDocument/hover
C#: Checking handler textDocument/hover:OmniSharp.LanguageServerProtocol.Handlers.HoverHandler
C#: Registration options (null)
C#: Document Selector (null)
C#: Handler Selected: OmniSharp.LanguageServerProtocol.Handlers.HoverHandler via (null) (targeting OmniSharp.Extensions.JsonRpc.IJsonRpcRequestHandler`2[[OmniSharp.Extensions.LanguageServer.Protocol.Models.HoverParams, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022],[OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022]])
C#: Starting: Routing Request (3) textDocument/hover
C#: Converting params for Request (3) textDocument/hover to OmniSharp.Extensions.LanguageServer.Protocol.Models.HoverParams
C#: Result was System.Threading.Tasks.Task`1[[OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022]]
C#: Response value was OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover
C#: Finished: Routing Request (3) textDocument/hover in 23ms
no such prop font.size when matching panel_control against panel_control console_panel
no such prop font.size when matching panel_control against panel_control console_panel
no such prop font.size when matching panel_control against panel_control console_panel
C#: Finding descriptor for textDocument/hover
C#: Found attributes 1, ::file:///home/tb/Project/Services/ILoginService.cs
C#: Looking for handler for descriptors textDocument/hover
C#: Checking handler textDocument/hover:OmniSharp.LanguageServerProtocol.Handlers.HoverHandler
C#: Registration options (null)
C#: Document Selector (null)
C#: Handler Selected: OmniSharp.LanguageServerProtocol.Handlers.HoverHandler via (null) (targeting OmniSharp.Extensions.JsonRpc.IJsonRpcRequestHandler`2[[OmniSharp.Extensions.LanguageServer.Protocol.Models.HoverParams, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022],[OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022]])
C#: Starting: Routing Request (4) textDocument/hover
C#: Converting params for Request (4) textDocument/hover to OmniSharp.Extensions.LanguageServer.Protocol.Models.HoverParams
C#: Result was System.Threading.Tasks.Task`1[[OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022]]
C#: Response value was OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover
C#: Finished: Routing Request (4) textDocument/hover in 39ms
C#: Finding descriptor for textDocument/hover
C#: Found attributes 1, ::file:///home/tb/Project/Services/ILoginService.cs
C#: Looking for handler for descriptors textDocument/hover
C#: Checking handler textDocument/hover:OmniSharp.LanguageServerProtocol.Handlers.HoverHandler
C#: Registration options (null)
C#: Document Selector (null)
C#: Handler Selected: OmniSharp.LanguageServerProtocol.Handlers.HoverHandler via (null) (targeting OmniSharp.Extensions.JsonRpc.IJsonRpcRequestHandler`2[[OmniSharp.Extensions.LanguageServer.Protocol.Models.HoverParams, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022],[OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022]])
C#: Starting: Routing Request (5) textDocument/hover
C#: Converting params for Request (5) textDocument/hover to OmniSharp.Extensions.LanguageServer.Protocol.Models.HoverParams
C#: Result was System.Threading.Tasks.Task`1[[OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover, OmniSharp.Extensions.LanguageProtocol, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=6d868dff454e6022]]
C#: Response value was OmniSharp.Extensions.LanguageServer.Protocol.Models.Hover
C#: Finished: Routing Request (5) textDocument/hover in 1ms
Error handling server payload
Traceback (most recent call last):
  File "/home/tb/.config/sublime-text-3/Packages/LSP/plugin/core/rpc.py", line 149, in receive_payload
    self.response_handler(payload)
  File "/home/tb/.config/sublime-text-3/Packages/LSP/plugin/core/rpc.py", line 168, in response_handler
    handler(response["result"])
  File "/home/tb/.config/sublime-text-3/Packages/LSP/plugin/hover.py", line 76, in <lambda>
    lambda response: self.handle_response(response, point))
  File "/home/tb/.config/sublime-text-3/Packages/LSP/plugin/hover.py", line 85, in handle_response
    all_content += self.hover_content(point, response)
  File "/home/tb/.config/sublime-text-3/Packages/LSP/plugin/hover.py", line 144, in hover_content
    value = item.get("value")
AttributeError: 'NoneType' object has no attribute 'get'

No completion: I see this issue on the language server's repo: https://github.com/OmniSharp/omnisharp-roslyn/issues/1554
If you want to be really sure, we need to see what capabilities the roslyn server is replying to the window/initialize request with. If you set log_payloads to True (in addition to log_debug you should see it as soon as the server starts.
Document not updating: Perhaps you can see if the server is not logging errors in response to any didChange notifications?
Hover error: I can't see the response, but judging by the error it might be invalid - something like: { 'contents': null }. Perhaps the build you are running is still suffering from this issue : https://github.com/OmniSharp/omnisharp-roslyn/issues/1087 ?

After enabling the debug log I get this.

The log does not contain the word change. There is no new output to the log if I delete or add a line.
I guess this is also why the hover suddenly is wrong: It still thinks the buffer has the same content on line X, but in the fact that content now resides on line X+1.
There are also no occurrences of the contents being null.

The full log can be found here: https://git.sr.ht/~troels/Log/tree/master/log.log

Thanks for the log, it confirms the completion issue as well as why buffer changes are not communicated - the server turns it off (textDocumentSync: 0).
The first issue I linked explains the problem with the omnisharp server - if it turns out this package is responsible for the incomplete capabilities then I'll gladly have a look!

if it turns out this package is responsible for the incomplete capabilities then I'll gladly have a look

How can I check that?
I have just checked with the same LSP in Neovim (via CoC).
Both textDocumentSync and autocompletion works there.
To be honest I don't know much about the LSP protocol. Coc.nvim touts itself as being compatible with vscode. I don't know if that means that there is the protocol and then there is a non-standard vscode implementation.
I saw that the issue you linked to links to another issue, which has been solved. I guess it ought to be solved.

I also have this issue.

The fix for the issues was merged in a related project (csharp-language-server-protocol), but I believe the reference to that project hasn't been updated in omnisharp-roslyn itself. Hopefully a maintainer will comment in their issue (#1554) when that's been released.
LSP follows the protocol, but avoids some complexities like dynamic registration of capabilities. It's possible CoC and VSCode use dynamic registration and therefore don't experience this issue.

Any updates on this? I tried OmniSharp too, but didn't really get anywhere. Some points of interest:

OmniSharp says "don't synchronize documents please". So no textDocument/didChange events are sent from Sublime to OmniSharp. Which makes completion/signatureHelp/hover useless.
OmniSharp always responds to a textDocument/hover request with [None, None], not a valid response.
Completions seem to behave crazy.

I just tried with the latest release  1.34.2. Still the same.

A new issue was created for the TextDocumentSync here: https://github.com/OmniSharp/csharp-language-server-protocol/issues/162. Although an incomplete fix was added, new bugs appeared, and the related package also needs to be updated to the omnisharp-roslyn repo itself.
Perhaps someone wants to contribute some tests that cover non-dynamic-registration :)
I'll add some documentation for this server as I found it to be pretty quick to set up otherwise!

I wondered if this was a recent regression, but even 1.32.9 turns off textDocumentSync for static capabilities. Either way, they merged a fix to the protocol library 3 days ago, so perhaps the next release of omnisharp-roslyn will get this working!


Connection closed

Version
Microsoft Windows [Version 10.0.19044.1586]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 18.04
Other Software
No response
Repro Steps
I  downloaded Anaconda3 but I didn't activate it. The problem occurred when I downloaded other packages or unzipped a file.I think it will happen at other times.
Expected Behavior
Successful results.
Actual Behavior
When I use WSL2 in Terminal,it writes '[process exited with code 1 (0x00000001)]'.
When I use WSL2 in Terminal MobaXterm, it writes 'Remote side unexpectedly closed network connection'.
Diagnostic Logs
No response

\logs

Hello! Could you please provide more logs to help us better diagnose your issue? You can find instructions on how to attach logs here, please make sure to post the link to the Feedback Hub item in this chat so we can see it.
Thank you!

Thanks for your reply. I've submitted it on the Feedback Hub. Here is the link:https://aka.ms/AAgny3u

/logs

https://aka.ms/AAgny3u

Sorry, it's my fault. My virtual memory is set up wrong, I reset it, and now everything is normal.


Links in package description should open in new tab

Is your feature request related to a problem? Please describe.
The longDescription of a package manifest can contain links in markdown format. A markdown link is correctly rendered as an html anchor element, but it opens in the same tab the user is currently in.
Describe the solution you'd like
Links in the longDescription should be rendered to have the target="_blank" attribute too, to make it open in a new tab.
Describe alternatives you've considered
Additional context
This description is converted from markdown to HTML on the package.html template, using the | Markdown pipe. The corresponding function Markdown is registered in the templates.go file. There, the rendering happens – probably here something has to be done.

Hi @christophenne may I take this one?


Expose the converter via GitHub Pages

GitHub Pages is a static website hosting capability built on top of GitHub. This way you could have a live version of the converter on the web, hosted and versioned by GitHub.
Good work so far! :+1:

Hi @jbjonesjr - I was using my stevecat.github.io repo and manually copying over after each update which felt very impractical. You caused me to discover that I could just create a gh-pages branch, it should be a lot simpler now. Thanks!
Every day is a :school: day!

@stevecat Always happy to spread the pages (and markdown) love. I got referred to this the other day, and was really impressed. Really nice tool, and happy to see if get on your pages site (and it's own site :smile: )
cc/ @benbalter as he is my Pages and Markdown guru, he may be interested as well.

Thank you! I enjoy coding little tools and doing this is great for helping me learn the proper GitHub flow.
Just added HTML conversion too, hurrah!


Not able to add event handler- any1 facing same issue?

Hi,
I tried to add event handler using swagger . i was trying to register event handler using POST /event api. but giving me below error.
"code": "INTERNAL_ERROR",
"message": "INTERNAL_ERROR - String index out of range: -1",
json used
{
	"name": "event1",
	"event": "event_task1",
	"condition": "number>2",
	"actions": [
	{
		"action": "start_workflow",
		"start_workflow": {
			"name": "decision_workflow",
			"input": {
				"number": "${workflow.input.number}"
			}
		}

	}
	],
	"active": true
}

now im able solve internal error by changing "event": "conductor:wokflow_test:event_1", where wokflow_test is main workflow name and event_1 is event task in main wf.
now my workflow completes but i dont think so any event is getting triggered. --> need help with this.
Found 1 more article on event and event handlers.
http://techblog.netflix.com/2017/04/netflix-conductor-inversion-of-control.html?

#125 . solved using this link.


Additional hotel attributes, offerings, specialities

wellnes, pool, tennis, jogging, hiking, bars...

This issue was moved to joachimprinzbach/ng2-camp2#8


change default user.php path.

(app\User.php to app\Modules\User.php)
1.change app\User.php;
namespace App\Models;
2.change config\auth.php;
'model' => App\Models\User::class,
3.fix this branch.

The default class name for Laravel's User model is App\User.
https://github.com/laravel/laravel/blob/master/app/User.php


Version does not accept "."

When the version string does contain a "." the resulting report url does not work.
Example:
https://dashboard.stryker-mutator.io/reports/github.com/richardwerkman/stryker-net/my.test
It could be that the . causes the path to look like a file.
https://github.com/stryker-mutator/stryker-dashboard/blob/3369498044517a3fc0893831b499f778df6c03dd/packages/website-backend/src/middleware/spaMiddleware.ts#L13

Good catch!

I had some spare time. Fixed and released in v0.5.1 of the dashboard.


[loc][Query][AzureStorageExplorer] Localizability of  0; "ErrorInactiveFileDownload"

Hi,
please help localization team to answer question from translator.
In case translator is not sure about context/meaning/localizability of string we raise a question to Core team.
Can you help, please?
Thanks!
Source:
Internal error: The specified file download is not currently active.
File Name:
loc\en\src\Standalone\resources\Activities.json
Resource ID:
0; "ErrorInactiveFileDownload"
Query:
Please clarify if "file" is singular or plural here. Thanks.

@PDostalek "File" is an adjective here. The subject of the sentence is "file download", that is, a singular file download.

Perfect, thanks


add recusive call if an error is detected.

(note: this has not been tested!! [I didn't want to make any changes to the data, and I don't have the env])

ALSO WHO TF HAS THE VERCEL TO THIS REPO ??


Incorrect Path Resolution for _components Folders in VS Code Extension

Version
1.96.0
Describe the bug

Cody incorrectly resolves paths when generating or modifying code for components located in a folder named _components. Instead of using the correct path, Cody generates an invalid one, making the "Apply in Editor" feature unusable.
Steps to Reproduce

Use a Next.js project with the App Router setup.
Place components in a folder named _components alongside a page.tsx file, like this:src/app/(board)/tableau-de-bord/organisations/ajouter/_components/organization-form.tsx

Ask Cody to modify the component or generate code.
Observe the incorrect path Cody produces, such as:src/app(board)/tableau-de-bord/organisations/ajouter_components/organization-form.tsx

Expected behavior
Cody should generate the correct path:
src/app/(board)/tableau-de-bord/organisations/ajouter_components/organization-form.tsx

Additional context
The issue prevents applying changes directly via the “Apply in Editor” feature.
This behavior has been observed consistently with _components folders in App Router projects.

Hey @tiger-githubb
Thank you for providing the detailed bug report.
The team will have a look at it.
Sorry for the inconvenience.


New data from 210317

Signed-off-by: vng<EMAIL_ADDRESS>

I think we can wait one week and update data in April.

No problem here. But this is the first data, generated with fast_math, so feel free to test ..

The earlier we start testing the better.

@vng @rtsisyk please deploy it


Rework node types config, renamed "expression" to "node type"

Rework node types config, renamed "expression" to "node type"
Closes #145
Closes #103
Closes #54
Description
Refactors the nodeTypesConfig.json. We now no longer have a direct mapping between node type and metric, but a "category" to which a node type belongs that might be relevant for one or multiple metrics. This is especially helpful when defining new metrics or refining the definition of old metrics. For example, there are now distinct categories for "class_definition", "interface_definition", "if", "logical_binary_expression", etc., so that we can redefine e.g. the "classes" or "complexity" metric quite easily.
Changed the Queries in src/parser/queries/QueryStatements.ts and all their uses accordingly. By reading all information out of a NodeTypesConfig-object by default, using these queries should be less error prone now, too.
This also introdues a more consistent naming by changing "expression" to "node type" or "type name" where "expression" was used as a synonym to these terms to avoid confusion. The obsolete field type was removed.
The file src/parser/helper/Model.ts now includes documentation about the meaning of the fields in the nodeTypesConfig.json.
This does not touch the issue #213 and #222. Intries for node types that touch these issues are either unchanged if there is no mapping or mapped as "interface_definition".
Definition of Done
A PR is only ready for merge once all the following acceptance criteria are fulfilled:

[x]  All TODOs related to this PR have been closed
[x]  There are automated tests for newly written code and bug fixes
[x]  All bugs discovered while working on this PR have been submitted as issues (if not already an open issue)
[x]  Documentation (README.md, UPDATE_GRAMMARS.md) has been updated

NOTE on activated_for_languages:
This does not change that we cannot specify a mapping between some languages to one category (metric before) and other languages to another category for one syntax node type I discussed in https://github.com/MaibornWolff/metric-gardener/issues/145#issuecomment-1961626890. We currently do not have the issue that we consider a syntax node type for different things in different languages. To support this, we would have to change the field "category" to a list of objects that contain a pair of the category name itself and a (optional) "activated_for_languages" field for this single category. We also would have to take care that we do not add multiple queries for one node type which has multiple categories when building a query that should cover the node types of more than one category, e.g. for the "complexity" metric. As that is not trivial and would negatively impact the performance, I am not sure if we should implement such a feature now for the case that this issue could occur in the future.


switch image uploads to text fields & support avatar

@asood123  LGTM

merging this to unblock us.

What's the rationale for doing this?


fix(contact-row): add space between presence and status text

fix(contact-row): add space between presence and status text

:hammer_and_wrench: Type Of Change

[x] Fix
[ ] Feature
[ ] Refactoring
[ ] Documentation

:book: Description
Adds space between status and presence text.
:bulb: Context
related to this DP 2.0 issue: https://dialpad.atlassian.net/browse/DP-79063
For some reason this is only a problem on vue 3.
:pencil: Checklist

[x] I have reviewed my changes

:crystal_ball: Next Steps
Let DP 2.0 team know
:camera: Screenshots / GIFs
Before:

After:

:link: Sources

Hmm, can you check if the dt-leftbar-row__meta-custom is being applied?
Maybe it has some left margin, but dt-emoji-text-wrapper has inheritAttrs: false and the class is not binding correctly.

they are certainly both being applied but I don't see any styles actually being applied by the classes. I'm a bit confused by this code in leftbar_row.less, but it seems like this class will not be applied unless the element with dt-leftbar-row__meta-context is within dt-leftbar-row__meta-custom? Currently they are siblings
&__meta-context {
    //
  }

  &__meta-custom:not(:empty) {
    &:deep(.dt-leftbar-row__meta-context ~ &:before) {
      content: ' • ';
      color: var(--dt-theme-sidebar-status-color-foreground);
    }
  }

I compared the CSS with app framework (https://dialpad.github.io/app-framework/inbox.html) and it seems the logic is not really correct.. seems kinda backwards. Seems to have broken around the same time we implemented :deep
see: https://github.com/dialpad/dialtone-vue/commit/7e27e9c1b5ef5a7c77a9957d09b26c4fde9d38b3#diff-ac651f54cd1f090ca715d5a[…]836e4453c905af06240aa734dR276
@juliodialpad I'm not sure exactly what this change was made for in the first place, please verify I'm not breaking anything else by changing this.
I think I will have to make this same change on the vue 2 branch now as well.

Now correctly shows the dot


Fix error in api review

Fix error in api review: https://apiview.dev/Assemblies/Review/2ebd7c87eaf047aaa9f13359a6fd38b2#

Hi, @JonathanGiles .
Please help to review this PR.
Relates api-reviews:

original one: https://apiview.dev/Assemblies/Review/2ebd7c87eaf047aaa9f13359a6fd38b2#
new one:https://apiview.dev/Assemblies/Review/32fa8bce24ef4edeb3769dbb3d529a0c

/azp run java - keyvault - tests

Hi, @zhichengliu12581 , please help to review this PR.

/azp run java - keyvault - tests

LGTM

LGTM!


⚠ ClamAV has degraded performance

In a1f38fe, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 432 ms

Resolved: ClamAV performance has improved in 0d23441.


Portalicious: profile page debit cards

AB#29986
Describe your changes

Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I have added tests wherever relevant
[ ] I have made sure that all automated checks pass before requesting a review
[ ] I do not need any deviation from our PR guidelines

Closing this in favour of #5870


bug: gjs and gts files are not parsed when environment is glimmerx

When trying to run glint on a glimmerx project it does not parse gjs or gts files.

glimmerx config

ember-template-imports

When parsing files setting glimmerx isAnalyzableFile comes back as false for any gjs or gts files.

Technically not a bug, because historically that was an unsupported configuration of GlimmerX. However, if/as we change that (and we should! We should 100% align them!) this should definitely update to support it. I'm marking this as an enhancement accordingly. Thanks for filing it!

Related: https://github.com/typed-ember/glint/issues/678 :see_no_evil:

We've dropped support for GlimmerX in https://github.com/typed-ember/glint/pull/716, so the next major of Glint will not have GlimmerX support.
See also this general tooling plan with the language servers: https://github.com/emberjs/rfcs/pull/1014


BUG: Duplicate msg on subscription

Is there an existing issue for this?

[X] I have searched the existing issues

What went down?
Duplicate messages are getting displayed when clicked the subscribe button

Record

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

hey @Amruta7203 I have been observing for some time. You are raising issues again and again. That's not a good way to work on open source. Why don't you raise and describe everything in a single issue?

@mdazfar2 I would have raised a same issue if the issues were on a single page.. But since the issues are on different pages I thought of raising separate issues.

Also if you clearly observe there are ppl who have raised multiple issues like I have raised

@Amruta7203 I have no animosity towards you. You can raise another issue. I will be happy to assign you. :)

Don't forget to join our official Discord server that's why we can discuss more about to enhance HelpOps-Hub.
And also follow us on Linkedin
And please ⭐ our repo

@Amruta7203 We've actually added validation to check if the user is already subscribed. Please see the screenshot below.

That's alright @mdazfar2 .. The issue is about the duplicate msg been displayed.. Once in black below the input field and the other in red..


After 3.6 Update

Ever since I updated to 3.6 I have not got an update for the snapshots. I am going to do a reset of my HomeBridge to start from scratch and see if that’s what the problem is. If not just wanted to let you know.
Sent with GitHawk

Odd...I've been keeping mine up to date and it's working fine.  Let me know how the rest goes.  What model of camera do you have?

I have the ring doorbell pro and the floodlight


Revive docker_test

Currently, the test_docker is non-functional and has been since we upgraded to rust 1.56.
As we plan to move to test on physically distributed peers, this is not a priority, but reviving this test (and figuring out how to do such tests in the future) might come in handy.

@appetrosyan could you provide a bit more context on what exactly is wrong with the test? Actually, a containerized application/node should be considered as a target deployment too.

When we first introduced rust-1.56 the docker test in our CI stopped working. Since then, we've stopped running the docker-test on any of our pull requests, either before or after merge. We wanted to get to it, and fix the thing, but RC1 stuff got in the way and so we shelved it for the time being.
Now that the Iroha 2 tutorial is almost done, and the first release candidate is up, we have the means and the time to focus on making a docker test that checks a bit more of what we need.


Indent the generated GLSL properly

Currently we don't and it makes it hard to read. Fix this :)

This belongs in varjo


Finish ReadMe

[ ] Update building instructions for new cmake system
[ ] Explain code structure so other people can create their own projects

This issue is replaced by #27 and #28

This issue is replaced by #27 and #28


[interop][SwiftToCxx] Annotate emitted declarations with Clang's external_source_symbol attribute

Each emitted declaration is annotated with the external_source_symbol with its own USR, to allow Clang's indexer to recognize this declaration as a Swift declaration with a specific USR

@swift-ci please smoke test

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please test source compatibility

@swift-ci please test source compatibility


Fix code formatting issues in sdk

Fix code formatting issues in sdk

Codecov Report

Merging #776 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #776   +/-   ##
=======================================
  Coverage   53.74%   53.74%
=======================================
  Files          56       56
  Lines        2203     2203
  Branches      216      216
=======================================
  Hits         1184     1184
  Misses        968      968
  Partials       51       51

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 d31b959...53374d2. Read the comment docs.


Custom action controller

Hi, we have some issue for intgration, currently when login we need 3 information data like username, password, and code. 2 step authentication is work and cool. but jus only validate username and password, and our need is also to validate the code to select group user. so we need custom action after submit token. is there solustion for this issues ?

Hello, I'm not quite sure what you're asking for here. Could you explain the full user flow you are hoping to build and I'll see what I can suggest.

Hi, so in our product we need 3 paramatter for login username, password, and code. the code here use for filter group of company user. you can see on this link https://www.greenhcm.com/users/sign_in?locale=en , on devise, we set custom action for decalre session code when sign in. but after we implement authy, code is not setting up into session, so i need custom action after success login with authy.

How were you storing the code before you implemented Authy?

like this code
`class Devise::SessionsController < DeviseController
POST /resource/sign_in
def create
self.resource = warden.authenticate!(auth_options)
set_flash_message(:notice, :signed_in) if is_flashing_format?
sign_in(resource_name, resource)
yield resource if block_given?
# for active or inactive user
# respond_with resource, location: after_sign_in_path_for(resource) # default redirect sign_in function
company = ActiveRecord::Base.connection_pool.with_connection do
  Company.find_by_code(params[:company_code])
end

# company = Company.joins("LEFT JOIN user_companies ON user_companies.company_id = companies.id JOIN users ON users.id = user_companies.user_id").where("companies.code = ? and users.id = ?", params[:company_code], resource.id).first || {}
if company.present?
  user_company = ActiveRecord::Base.connection_pool.with_connection do
    UserCompany.find_by_user_id_and_company_id(resource.id, company.id)
  end
end

company_active = (company.valid_datetime > DateTime.now) if company.present?
if company.present? and user_company.present? and user_company.is_active and company_active
  if params[:user][:remember_me] == "1"
    cookies.permanent[:regcom] = company.auth_token
  else
    cookies[:regcom] = company.auth_token
  end
  respond_with resource, location: after_sign_in_path_for(resource)
end

end
end
`

hi, now i can customize GET_verify_authy on devise_authy_controller.rb, but the sign_in parameter not included on there, as you see my system have Company code paramaeter on sign in like this link https://www.greenhcm.com/users/sign_in?locale=en , can i get sign_in paramater on GET_verify_authy ?

This library is no longer actively maintained. The Authy API has been replaced with the Twilio Verify API. Twilio will support the Authy API through November 1, 2022 for SMS/Voice. After this date, we’ll start to deprecate the service for SMS/Voice. Any requests sent to the API after May 1, 2023, will automatically receive an error.  Push and TOTP will continue to be supported through July 2023.
Learn more about migrating from Authy to Verify.
Please visit the Twilio Docs for:

Verify + Ruby (Rails) quickstart
Twilio Ruby helper library
Verify API reference
Coming soon: Look out for a new Devise plugin to use Twilio Verify with Devise

Please direct any questions to Twilio Support. Thank you!


Question consultation

Dear authors:
Recently I  have a looking at your this project and feel interested in it !  I am interested in this project, but I also face some challenges.  and I wish I could receive your suggestion from your guide..
The question is that I could not find the dataset used in this project?  And I konw,
"Kaggle BR35H dataset for training
 used another dataset from Kaggle itself having just 253 images    "
And how could I get them?  And My e-mqil is<EMAIL_ADDRESS>Any suggestion or recommendation is appreciated.
Thank you. I am looking forward to your reply Sincerely!
Best regards.

Hey, you can find the dataset over this link: https://www.kaggle.com/datasets/ahmedhamada0/brain-tumor-detection
Just see in the summary and you can see the 1500 images over there. You just need to download the dataset, that's it :)
Hope this helps!


Updated docstrings for base module

Updated docstrings for the base module.

Added returns documentation to all functions
Added consistency across documentation
Clarified parameter type and return type

Here is a pdf of the updated documentation:
matplotcheck package — matplotcheck 0.0.10 documentation.pdf
This should resolve issue #102

@lwasser could you take a look at this? Looks like the appveyor test is failing for the same reason as before, but I only changed the comments on one file, so this pull request shouldn't affect testing results anyway. Thanks!

this is great. If you update from this fork / master branch @ryla5068 it may fix things. when you are working again on this please give this a shot! it seemed to work for me so far but ... let's see!

thanks @ryla5068 !! merging this!


Format RBD volumes with nodiscard formatting options.

Currently rbd CSI plugin uses formatAndMount of
mount.SafeFormatAndMount. This does not allow to pass or use
specific formatting arguments with it. This patch introduce
RBD specific formatting options with both xfs and ext4,
for example: -E no-discard with ext4 and -k option with
XFS to boost formatting performance of RBD device.
Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>(cherry picked from commit 0e6617e1ffff4dc93e7606bfdf0dcaf60f3c327d)

@Madhu-1 @ShyamsundarR PTAL.


Feature request: Interrupt loading Data Viewer

Feature: Notebook Editor, Interactive Window, Python Editor cells

Description
VScode Data Viewer launches quickly and shows the top part of the large data frame relatively fast, but it takes a long time to finish loading. It freezes intermittently while it is loading the data.
Below is pandas dataframe with 1M rows and 113 columns.

I want to grasp a quick overview of the dataframe and browsing only some part of them usually fine. In that sense, I think implementing 'interrupt loading' would be beneficial for some users.
Fast dataframe viewer is what I'm really missing feature for python IDEs for data science. I've been using a python package called gtabview that did the job very well. For example, with this package, it takes less than a second to show this data frame - which is even faster than PyCharm dataframe viewer. Problem is, this package does not work well with the current version of jupyter, so in order to use this in vscode I have to run pure python on the terminal.
I'm hoping that VScode can make a fast Data Viewer like above package in the future!
Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

Thank you for the feedback @matthewgson. We'll look into this shortly.

We should consider using one of the 3rd party widgets (ipywidgets) with support for lazy landing data from the backend.


arm64 VS CMake integration with vcpkg manifest file failing upon configure

Describe the bug
I'm trying to configure a CMake project in Visual Studio. My toolset is msvc_arm64_x64 and I have a vcpkg.json with 4 packages that support arm64-windows. The configure step is failing.
Environment

OS: Windows
Compiler: msvc arm64 (latest installed with VS 2019 16.9 Preview 3)

To Reproduce

Create CMake project in Visual Studio 2019 version 16.9 Preview 3. Ensure the latest arm64 build tools are installed
Add a new arm64-debug CMake configuration to CMakeSettings.json
Add a vcpkg manifest file to the root of the CMake project with glad, glfw3, imgui, gtest
Make the arm64-debug configuration the active CMake configuration in Visual Studio
Attempt to configure the project
See error, failure logs below

Expected behavior
vcpkg will automatically build and install the arm64-windows triplet of the 4 libraries listed in vcpkg.json and the arm64-debug configuration of my CMake project will successfully configure.
Failure logs
Detecting compiler hash for triplet arm64-windows...
Unsupported toolchain combination. Target was: arm64 but supported ones were:
x86,amd64,x86_amd64,amd64_x86

Logs from cmake -B build-compilertracking -S scripts/detect_compiler -A arm64:
-- Building for: Visual Studio 16 2019
-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19042.
-- The C compiler identification is MSVC 19.28.29812.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Preview/VC/Tools/MSVC/14.28.29812/b
in/Hostx64/arm64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- The CXX compiler identification is MSVC 19.28.29812.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Preview/VC/Tools/MSVC/14.28.29812
/bin/Hostx64/arm64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
#COMPILER_HASH#9306dc21c1ae74a9f79ebb3310835a3ba81392ca
#COMPILER_C_HASH#5a57d8a31bb0d527d01ebe06fe7438085538c548
#COMPILER_C_VERSION#19.28.29812.0
#COMPILER_C_ID#MSVC
#COMPILER_CXX_HASH#5a57d8a31bb0d527d01ebe06fe7438085538c548
#COMPILER_CXX_VERSION#19.28.29812.0
#COMPILER_CXX_ID#MSVC
-- Configuring done
-- Generating done
-- Build files have been written to: C:/Users/ersweet/vcpkg/build-compilertracking

so clearly the ARM64 compilers work. cc @ras0219

Logs from cmake -B build-compilertracking -S scripts/detect_compiler -A arm64:
-- Building for: Visual Studio 16 2019
-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19042.
-- The C compiler identification is MSVC 19.28.29812.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Preview/VC/Tools/MSVC/14.28.29812/b
in/Hostx64/arm64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- The CXX compiler identification is MSVC 19.28.29812.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Preview/VC/Tools/MSVC/14.28.29812
/bin/Hostx64/arm64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
#COMPILER_HASH#9306dc21c1ae74a9f79ebb3310835a3ba81392ca
#COMPILER_C_HASH#5a57d8a31bb0d527d01ebe06fe7438085538c548
#COMPILER_C_VERSION#19.28.29812.0
#COMPILER_C_ID#MSVC
#COMPILER_CXX_HASH#5a57d8a31bb0d527d01ebe06fe7438085538c548
#COMPILER_CXX_VERSION#19.28.29812.0
#COMPILER_CXX_ID#MSVC
-- Configuring done
-- Generating done
-- Build files have been written to: C:/Users/ersweet/vcpkg/build-compilertracking

so clearly the ARM64 compilers work. cc @ras0219

We likely selected a different visual studio than CMake does by default. You can see the list of visual studio instances we detected via
vcpkg x-vsinstances

sorted by our preference. If you want us to use a different one, you can set that in the environment variable VCPKG_VISUAL_STUDIO_PATH: https://github.com/microsoft/vcpkg/blob/master/docs/users/config-environment.md#vcpkg_visual_studio_path

We likely selected a different visual studio than CMake does by default. You can see the list of visual studio instances we detected via
vcpkg x-vsinstances

sorted by our preference. If you want us to use a different one, you can set that in the environment variable VCPKG_VISUAL_STUDIO_PATH: https://github.com/microsoft/vcpkg/blob/master/docs/users/config-environment.md#vcpkg_visual_studio_path

That helped a bit, I'm now hitting this:
Detecting compiler hash for triplet arm64-windows...
Error occurred while detecting compiler information. Pass --debug for more information.

That helped a bit, I'm now hitting this:
Detecting compiler hash for triplet arm64-windows...
Error occurred while detecting compiler information. Pass --debug for more information.

What does it say when you pass --debug?

What does it say when you pass --debug?

CMake is invoking vcpkg for me. How do I pass --debug? Should I run vcpkg install from cmd?

CMake is invoking vcpkg for me. How do I pass --debug? Should I run vcpkg install from cmd?

The easiest way to repro this issue will be to switch to just using the cli directly to install something simple, like vcpkg install zlib:arm64-windows.

The easiest way to repro this issue will be to switch to just using the cli directly to install something simple, like vcpkg install zlib:arm64-windows.

vcpkg install zlib:arm64-windows --debug
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'manifests' = on
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Using vcpkg-root: C:\Users\ersweet\vcpkg
[DEBUG] Using installed-root: C:\Users\ersweet\vcpkg\installed
[DEBUG] Using buildtrees-root: C:\Users\ersweet\vcpkg\buildtrees
[DEBUG] Using downloads-root: C:\Users\ersweet\vcpkg\downloads
[DEBUG] Using packages-root: C:\Users\ersweet\vcpkg\packages
[DEBUG] Using scripts-root: C:\Users\ersweet\vcpkg\scripts
Your feedback is important to improve Vcpkg! Please take 3 minutes to complete our survey by running: vcpkg contact --survey
[DEBUG] Default binary cache path is: C:\Users\ersweet\AppData\Local\vcpkg/archives
Computing installation plan...
Error: There are invalid field types in the CONTROL or manifest file of C:\Users\ersweet\vcpkg\ports\boost-locale\vcpkg.json
The following fields had the wrong types:

features was expected to be an array of feature definitions

Error: Failed to load port boost-locale from C:\Users\ersweet\vcpkg\ports\boost-locale
[DEBUG] C:\Users\ersweet\vcpkg\toolsrc\src\vcpkg\portfileprovider.cpp(153)
[DEBUG] Exiting after 100639 us (53426 us)

vcpkg install zlib:arm64-windows --debug
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'manifests' = on
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Using vcpkg-root: C:\Users\ersweet\vcpkg
[DEBUG] Using installed-root: C:\Users\ersweet\vcpkg\installed
[DEBUG] Using buildtrees-root: C:\Users\ersweet\vcpkg\buildtrees
[DEBUG] Using downloads-root: C:\Users\ersweet\vcpkg\downloads
[DEBUG] Using packages-root: C:\Users\ersweet\vcpkg\packages
[DEBUG] Using scripts-root: C:\Users\ersweet\vcpkg\scripts
Your feedback is important to improve Vcpkg! Please take 3 minutes to complete our survey by running: vcpkg contact --survey
[DEBUG] Default binary cache path is: C:\Users\ersweet\AppData\Local\vcpkg/archives
Computing installation plan...
Error: There are invalid field types in the CONTROL or manifest file of C:\Users\ersweet\vcpkg\ports\boost-locale\vcpkg.json
The following fields had the wrong types:

features was expected to be an array of feature definitions

Error: Failed to load port boost-locale from C:\Users\ersweet\vcpkg\ports\boost-locale
[DEBUG] C:\Users\ersweet\vcpkg\toolsrc\src\vcpkg\portfileprovider.cpp(153)
[DEBUG] Exiting after 100639 us (53426 us)

That looks like an out-of-date vcpkg issue; Did you re-run ./bootstrap-vcpkg.bat after running git pull?

That looks like an out-of-date vcpkg issue; Did you re-run ./bootstrap-vcpkg.bat after running git pull?

Oops, no. Sorry about that. I re-ran it and it's working correctly now. Thank you both!

Oops, no. Sorry about that. I re-ran it and it's working correctly now. Thank you both!

Resolved.

Resolved.


Can I use react-native for Android now if I don't use the native UI stuff ?

Hi guys, I'm thinking building the UI with famo.us -- which is awesome, and combining the power of react-native -- which is also awesome : ), so if I don't use the native UI stuff, can I use react-native for Android now ? What native modules does it currently have ? How do I integrate with famo.us ? Curious ~

@ShawnOceanHu - None of the react-native Android project is open source currently, it is still about 6 months off from release. Sorry, there's no way around it, I wish there were!

6 months ... that's long ~
closing this ~


mobile: c++ explicit flow control tests and fixes

Handling reentrant callbacks for data, and slow uploads for explicit flow control mode properly.
Testing: e2e tests
Docs Changes: n/a
Release Notes:n/a
[Optional Runtime guard:] n/a

/retest


Show dependent of missing type in error

When the schema provider cannot find  a type, it reports an error like this:

No schema type found for dotnet type TypeName. Make sure you add it or add a type mapping

It would be helpful for the error message to display the type and field that referenced the missing type.
An a minor detail on the same error: The second sentence in the error message could use a period at the end.

Good idea. We don't always have that context though (e.g. you can just ask the provider for a type). But when we do we should show it. Certainly help find the types not set up.


Element subclasses can't clear dirty flag

Custom elements that are subclassing Element directly cannot clear the dirty [1] flag because the backing property _dirty is not accessible to them outside of the framework.dart file. To properly clear that flag in custom Elements, one must extend one of the provided subclasses of Element (e.g. ComponentElement), which do implement proper clearing logic.
We should either document that Element shouldn't be extended directly or provide API to properly deal with the dirty flag in subclasses.
[1] https://master-api.flutter.dev/flutter/widgets/Element/dirty.html

@goderbauer has a decision been made here?
Are we leaning towards documentation or API change?

I fixed this recently in https://github.com/flutter/flutter/pull/109401, but apparently forgot to link it to this issue.


Option to mute channels/servers

What do you want
When selecting a channel, guild, or channel group, hitting the "m" key (or something else) should toggle mute for that channel/guild/group. There should be a mute icon so we know what channels are muted.
Why

This would be great because it would let me use discord without having to log in to the official website/app every time I want to change the mute setting of a channel.

ooh yeah that would be nice

It would also be cool if the mute icon was an emoji in front of the channel, but you could also just change the color that it's displayed in. Hiding muted channels/groups could also be a neat feature, I have a lot of servers that I only use a few channels of.

Okay, there's a problem with how this is implemented, where if you press m to jump to a channel, it mutes the currently selected one instead of taking it as input

Jump to a channel?

Oh I see

Probably we can get around this by adding a modifier for muting

1ed6525


⚠ Samsung has degraded performance

In 586a0cd, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2634 ms

Resolved: Samsung performance has improved in 64097ab.


Add optimum[onnx]

Closes #58

Codecov Report
Attention: 34 lines in your changes are missing coverage. Please review.

Comparison is base (0fc2b61) 87.94% compared to head (4162359) 87.07%.

Files
Patch %
Lines

...nity_emb/infinity_emb/transformer/utils_optimum.py
68.33%
19 Missing :warning:

...b/infinity_emb/transformer/crossencoder/optimum.py
80.55%
7 Missing :warning:

...y_emb/infinity_emb/transformer/embedder/optimum.py
91.48%
4 Missing :warning:

...inity_emb/infinity_emb/transformer/acceleration.py
50.00%
2 Missing :warning:

...ibs/infinity_emb/infinity_emb/transformer/utils.py
77.77%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
- Coverage   87.94%   87.07%   -0.88%
==========================================
  Files          24       27       +3
  Lines        1070     1222     +152
==========================================
+ Hits          941     1064     +123
- Misses        129      158      +29

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


Don't include site title in Twitter tag by default or make it overrideable

This doesn't look good in some use cases, e.g.,

Should just be "Donate to Freedom of the Press Foundation" instead of "Donate to Freedom of the Press Foundation — Freedom of..."
It also can't be changed without overriding the template tags.html. Probably these values should be generated from a page method that can be overridden in a more targeted way. I also think the default behavior should be not to include the site title, but that's potentially more subjective.

Yeah I agree, adding site_name to the meta_title shouldn't be the default. It's what we usually do internally but it should be done using an override on our side.

Is there any update? Or do you have a temporary workaround to prevent this behavior?

Just released version 3.2.0 that includes a fix for this.
If anyone wants the old behavior back they can override the get_meta_title  method to return the title + sitename

Is this now something you can regulate via settings or the like? Because some folks really need the site name in there still =)

No plans to make this a setting, this project is pretty simple with no current config, it'd be nice to keep it that way :)
If you're referencing the mentioned pull request above you could add the following to wagtailpages/pagemodels/mixin/foundation_metadata.py
class FoundationMetadataPageMixin(MetadataPageMixin):
   ...
   def get_meta_title(self):
      site = self.get_site()
      title = self.seo_title or self.title
      return f'{title} — {site.site_name}'


Dropdown appears overtop of status bar

When I have the last option selected in the dropdown and then open the dropdown the options get cut off by the top of the app.
iOS Simulator

Android

Any way to make it so that the dropdown goes down if there is not enough room at the top of the app?

Thanks for reporting, I'm planning to fix this in next release.

Fixed by adding dropdownPosition prop in #14. Released in 0.4.0.


Maven pom.xml EOF newline getting removed

TL;DR
Release-please generated PRs that update <version /> in a maven project pom.xml remove the EOF newline
Expected behavior
I'd expect the EOF newline to be retained
Observed behavior
EOF newline is removed from pom.xml, and GitHub highlights it with red symbol

Action YAML
name: release-please

on:
  push:
    branches:
      - master

jobs:
  this:
    runs-on: ubuntu-22.04
    steps:

      # Obtain a GitHub app installation token (app has necessary permissions to this repo)
      - id: auth
        uses: ...
        with:
          ...

      - uses: google-github-actions/release-please-action@v3
        with:
          release-type: maven
          package-name: my-package
          token: ${{ steps.auth.outputs.token }}

Log output
No response
Additional information
We use a GitHub app installation token, because we need to trigger other workflows on release events.
However, I don't think this configuration has anything to do with the observed bug, as everything else works fine.

This would need to be fixed on the upstream library: https://github.com/googleapis/release-please

This would need to be fixed on the upstream library: https://github.com/googleapis/release-please

@chingor13 Do I need to create a separate issue there?

Yes, it cannot be fixed here.

Yes, it cannot be fixed here.

Created upstream issue here: https://github.com/googleapis/release-please/issues/1894

This was fixed in 15.10.1 and will be included in #757


Is there support for NetworkBehaviour?

the readme says to replace monobehaviour with betterbehaviour, nothing is mentioned at all when it comes to multiplayers / networkbehaviour, so I assume there is no current support for that?

This has been asked multiple times, see VFW thread post #439. Due to the way (inflexibility) the Unity editor system works, you have to duplicate code to get it to work. Ideally you'd want to mark your class with [VFW] or something without having to inherit from BaseBehaviour etc. But that will require a revamping of the entire Editor class to work this way.


HookMap: replace .tap() with .for().tap()

Tapable will deprecate HookMap's tap tapAsync tapPromise API, so I make this PR to replace the HookMap instance use .tap() API with .for().tap() API.
What kind of change does this PR introduce?
This PR is refactoring the HookMap API.
Did you add tests for your changes?
Yes.
Does this PR introduce a breaking change?
No.
What needs to be documented once your changes are merged?
The pr has no perception of the user, do not need to be documented.

Thanks

Thank you for your amazing work!


plugin ci: bump ubuntu

This is what we use in the main plugin integration repo.
This should also fix #3143.

test-plugin is green. Merging.


ScrollBar and Main Title Color

Description of the new feature / enhancement

.EvIR4O7jOSbNmxtMdIQ0 {
background-color: var(--background-highlight);

.os-theme-spotify.os-host-transition>.os-scrollbar-vertical>.os-scrollbar-track>.os-scrollbar-handle {
border-radius: 4px;
width: 6px;
}
.os-theme-spotify.os-host-transition>.os-scrollbar-vertical>.os-scrollbar-track {
width: 6px;
}

If you like the idea, add a 👍 reaction to this post

.EvIR4O7jOSbNmxtMdIQ0 {
background-color: var(--background-highlight);

This makes the color of the top uniform, right?

.os-theme-spotify.os-host-transition>.os-scrollbar-vertical>.os-scrollbar-track>.os-scrollbar-handle {
border-radius: 4px;
width: 6px;
}
.os-theme-spotify.os-host-transition>.os-scrollbar-vertical>.os-scrollbar-track {
width: 6px;
}

But what are you doing it with the scrollbar?

Default scrollbar is too big and ugly

I believe it's subjective for everyone, as there are numerous such moments throughout the client. I highly recommend using Spicetify, where you can customize your own theme or choose from a variety of ready-made options
Cheers.


Can't support i18n?

It includes some text resources, but they are fixed in English. I would like it to be compatible with internationalization.
I want the Japanese version.
Kind regards.

Isn't it just the done button?
Set it to whatever you want https://github.com/mikaoj/BSImagePicker/blob/7afbaa1daf882b9f90e8e4a42f7b102f9d589245/Sources/Controller/ImagePickerController.swift#L42


compose: Open typeahead immediately when @ is entered.

'filter_mention_name' in 'composebox_typeahead.js' now also
accepts since there is no charater after @. Now it will return
an empty string which will be rejected by if block in
get_candidates, hence changes were made to let the empty string pass.
Changes were made to  node tests becuase now '@', '@_', ' @' and
' @_' (and similar cases with a string before), won't return false
Fixes #19142.
GIFs or screenshots:
Before:

After:

This was done via https://github.com/zulip/zulip/pull/19160, which is very similar aside from the issue noted above.  Closing, but thanks for working on this @raghupalash!  If nothing else, it does help verify that something is correct to see 2 identical implementations.

I am a new contributer, it was a good learning experience anyways, enjoyed a lot.
Thanks!


Dungeon Defenders 1 - Cannot capture scenes via RTX Remix

Describe the bug

When trying to capture a scene to begin using RTX remix on Dungeon Defenders 1, the scene simply cannot be captured, it will never do anything after pressing the capture scene button. Only additional info is that this is an Unreal Engine 3 game.
Remix version 0.4
RTX 4080
GPU Driver version 546.65
Latest on Steam

To Reproduce
Steps to reproduce the behavior:

Load up Dungeon Defenders 1 from Steam via RTX Remix
try to capture any scene, even the main menu.

Expected behavior

I expected to at least get past the very first part of the process by capturing a scene.

Hello. Thank you for reporting. Could you please attach relevant logs and screenshots.

Hi @Traslogan, the Capturer will not work if Remix is unable to prepare scene data and actually inject ray tracing.
Can you confirm whether you see a different render from the standard rasterization? Does the temporary RTX Remix banner appear on launch? Does the game look how it normally looks without remix installation?
If you are unsure about the above, feel free to attach pictures, and we can try to infer from those. Thank you in advance.

Dungeon Defenders runs on Unreal Engine 3, which is an engine that is fundamentally incompatible with Remix, it won't work.

Thank you for reaching out. As mentioned above, the title is unlikely to be supported with Remix.


chore: Added utility file to check for query compatibility

Description
This PR introduces a utility to check for whether certain migration queries can be run in an optimized way for a particular installation or not.
Type of change

Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?
No testing required, this utility is not being used yet, meant for syncing with EE.

Skipping ok to test because this util is not being consumed.


Feature suggestion: Dither removal

Hey, I'd like to suggest dither removal or undithering for waifu2x.
By the way, what prompted me to suggest this was the discovery that earlier versions of waifu2xcaffee, and I presume waifu2x were quite good at removing the dither from iDOLM@STER Cinderella Girls Starlight Stage cards using noise removal, but no so much anymore:
Original image:

2x Upscale w/ Level 2(High) Noise Removal using waifu2xcaffe 1.0.5:

2x Upscale w/ Level 2 Noise Removal using latest waifu2x (waifu2x caffe 1.0.7 gives similar results)

Actually, one interesting thing to note is that Level 2 noise removal actually removes less dither now than Level 1:

It seems that dither is not from JPEG compression. In latest filters, It is a detail.
However, I know that dither removal is very useful in this area.
I will add the level3 noise reduction filter that does not retain a detail, removing JPEG artifact and dithering noise.
You can get old filters at http://www.udp.jp/misc/waifu2x_models/v0.9/anime_style_art_rgb/ .

Actually, would it be possible for dither removal to be separate from artifact removal, or would it not make that much of a difference?

Maybe it is possible. Of course, it has its limits.
I am planning to add -undither option.

Thanks a lot, nagadomi!
Have some Starlight Stage card art to test the -undither option on https://icebluenoshunkan.kirara.ca/temp/ss/card_bg_png/200.html


Using deprecated clientSet.Extensions in Syncer in KDD

@robbrockbank could you point to the deprecation notice / lines we're using this / add some description to this issue?

Goglang was flagging this up to me too:
// Deprecated: Extensions retrieves the default version of ExtensionsClient.
// Please explicitly pick a version.

This should be really easy so adding to 2.5

we're no longer using the deprecated extensions api (libcalico-go extensions)


Controller loading State

There exists a condition whereby the controllers can get locked into a loading: true state. This state is set when the beforeSearch event kicks off, and typically is set to false when the afterStore event concludes. In the SearchController, the loading state never returns to false when the following return takes place:
https://github.com/searchspring/snap/blob/v0.61.5/packages/snap-controller/src/Search/SearchController.ts#L355
However, I am pretty sure that this return would also leave the controller in a similar state:
https://github.com/searchspring/snap/blob/v0.61.5/packages/snap-controller/src/Search/SearchController.ts#L344
There is a "catch" block below that does set the loading state back to false when errors are caught, but that does not happen in the cases above.
https://github.com/searchspring/snap/blob/v0.61.5/packages/snap-controller/src/Search/SearchController.ts#L530
The other controller types likely have similar issues. We should ensure that when a search cancels, errors out, or is prevented that the loading state is properly set back to false. Testing should be improved around these situations to validate that we have the proper state (loading: false).

Released in 0.62.0!


Prepare platform-specified renderer extensions

Each platform has different elements (like Toast on Android). Neft should has extensions for 'android', 'ios' and 'html5' platforms.

I prepared the first implementation of the native extensions for Android: https://github.com/Neft-io/neft-android . Already implemented the Toast and Button views.

Standard iOS views: https://github.com/Neft-io/neft-ios .
Currently Button and Switch have been implemented.


Game keeps crashing when i make elevators or rope pulley elevators

Describe the Bug
I think just the overall moving of blocks is breaking my game it only happens when i make the rope pulley actually move a platform and i just made an actual elevator and when i right clicked on the pulley it crashed my game and now will not let anyone rejoin the server
latest.log
Reproduction Steps
1.activating elevator contraption
2.game crashes
3.cant join back
...
Expected Result
i want the elevators to work but now my game keeps crashing after i made one
Screenshots and Videos
No response
Crash Report or Log
No response
Operating System
windows 10
Mod Version
0.5.1d
Minecraft Version
1.20.1
Other Mods
ad astra
alloy forgery
amecs
applied energistics
architectury API
armor abilities
azurelib
balm
bclib
better end
bookshelf
botany pots
botarium
cardinal components api
cavedweller
cloth config
compressed blocks
continuity
dark utilities
elytraslot
emi
enderchest
endertanks
entity culling
expanded armor enchanting
expanded axe enchanting
expanded bow enchanting
expanded enderchest
expanded trident enchanting
expanded weapon enchanting
extended drawers
fabric api
carpet modfabric language kotlin
farmers delight
feather fabric
forge config api port
gecko lib
gilded armor
gravestones
Happiness is a warm gun
inventory profiles next
iron chests
item collectors
jamlib
jei
journey map
lambdynamiclights
lanterns belong on walls
libipn
lost features
mcpitanlibarch
mob lassos
mod menu
moonlight
more mob variants
owo.lib
packed up
patchouli
physics mod
pig pen fabric
puzzles .lib
quick shulker
reinforced barrels
reinforced chests
reinforced shulker boxes
reinforced upgrade
resourceful config
resourceful lib
right click harvest
roughly enough items
runelic
shetiphiancore
shulker boxes tool tip
simple copper pipes
simply swords
smart brain lib
starlight
statement
supermartijn642configlib
supermarijn642corelib
tempad
trashcans
travelers backpack
trinkets
ubes delight
upgradable backpack
villagers plus
waystones
Additional Context
No response

Could you please send the crash log?

starlight is incompatible and unnecessary


[incubator/azuremonitor-containers] updates related to agent 03022020 release

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
This PR has the changes for azure monitor for containers agent 03022020 release
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

/ok-to-test

/approve
/lgtm


Feature suggestion: Convert hot cue loops to memory cue loops.

Currently, loops are not being converted from hot key to memory cue.
Is this something you could add?
Thank you for the great script.

I had this same problem and spent a stupid amount of time trying to figure out why (I am very new to coding). But I found a solution that mostly works. Mostly.
First problem was that looped memory cues have a different 'Type' value, so you can't make all of them 0.
Second problem was the code didn't grab the 'End' values of the cues (which all the looped cues have). Only the looped cues have an end value.
# usage: python3 hotcues-to-memory-cues.py $XML_FILENAME
# see https://djfile.com/how-import-beatgrids-cue-points-and-tags-using-rekordbox-xml for XML export/import instructions

import xml.etree.ElementTree as ET
import sys

print('converting ' + sys.argv[1])

tree = ET.parse(sys.argv[1])
root = tree.getroot()

for track in root.findall('./COLLECTION/TRACK'):
    for position in track.findall('POSITION_MARK'):
        start = position.get('Start')
        # don't create duplicate mem cues
        if track.findall('./POSITION_MARK[@Num="-1"][@Start="' + start + '"]'):
            print('skipping: ' + track.get('Name'))
        else:
            print('processing: ' + track.get('Name'))
        child = ET.Element('POSITION_MARK')
        child.set('Name', '')
        child.set('Type', position.get('Type'))
        child.set('Num', '-1')
        child.set('Start', position.get('Start'))

        temp = position.get("End")
        if temp:
            child.set('End', temp)

        track.append(child)

tree.write('output.xml')

Though this fixed the problem, I still had an issue in Rekordbox with importing it. I had to drag & play each individual song from the XML tab to 'load' it at least once for the memory cues to import. I had over 300 songs so it was a bit annoying (still a million times faster then re-creating each hot cue as a memory cue manually).
But that seems like its a but with Rekordbox not us.
Hope this helps!


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2023-06-26)
Features

add get_logs DataCollectionMode (#102) (0fbaa20)
block reward in full mode (#104) (738b8b3)
optimize db (#100) (640f5a5)
query_tool: add plot commands (#103) (bdff178)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/uzh-eth-mp/app/releases/tag/v1.1.0 :sunflower:


Change some bindings from SteamAPI_ISteam to SteamAPI_Steam.

Some bindings were missing _t_
servernetadr / gameserveritem have SteamAPI_ prefix. Added servernetadr_t_IsLessThan/servernetadr_t_Assign
Noticed this when working with the steam network messages api, and found a few other ones that were incorrect.

Looks very good, thank you for fixing this and for taking the time to PR the changes!


Fix (trait?) methods that take + return Self

fixes #40

Ah, looks like you got it!

Ah! I decided to use my lunch break to fix it up, and didn't see this pending PR, sorry!
I see you also did some other clean ups though so I will take those (:

No worries! I'll rebase this.

I didn't know about syn::parse_quote, thanks a lot!


Enable FFI in stream module; add SSL apis

This branch is a few misc things jumbled together; let me know if you'd like it as separate PRs (and which bits).
The balancer.set_ssl_ctx patch requires https://trac.nginx.org/nginx/attachment/ticket/1629/0001-add-stream_session-upstream_ssl-to-configure-per-str.patch

Note: The commit "add some of the ffi errlog API" (0d7abca) is a duplicate of the work done in #121 (which has PRs opened to meta-lua, stream-lua, and resty-core with the appropriate tests).

Closing, as this module's ssl_certby implementation is already merged, as per https://github.com/openresty/lua-resty-core/pull/205

@thibaultcha this branch contains much more than ssl_certby.
I should rebase it though.

@james-callahan I noticed. Please open different PRs against the meta-lua module if you are still proposing the other features in this patch (although I believe several were already discussed in principle in other repositories).


Reverse progress [ENHANCEMENT]

This library would be perfect, if it had an attribute <attr name="reverse" format="boolean"/>, where the progress would run counter-clockwise, if set to true.

Hi, thanks for your suggestion.
For now, it only runs counter-clockwise when resetting the current progress. I'll check that as soon as can.
Regards.


Beta release - 1.0.0-beta.8

I've rerun the check workflow as it failed due to this known issue https://github.com/ably/ably-asset-tracking-android/issues/391


Command line example doesn't work on windows cmd or powershell

The lzma command is not supplied in windows (any more) and there appears to be no easy to find safe download.
However if you install git for windows you will have bash shell and xs and you can use the given example with the simple substitution of xz --format=lzma -9 for lmza -9.
The author of lzma and xz indicates lzma has been superseded and the man page for xz indicates lzma is an alias for xz, so perhaps the example could be changed to use xz anyway?
A final detail for those using node / npm for build is that nmp always calls cmd so to invoke the command you need to use bash -c "echo -n 'hello world' ……….." or similar.
Or just use Windows WSL :)
Finally, and not windows specific, the example given only works with text. For HTML for example extra parameters are added to the generated URL. However these are easy to see in an example and then adjust the printf

Care to suggest a viable, sharable windows command? :)

Of course, sorry, I meant to then forgot
 echo -n 'hello world' | xz --format=lzma -9 | base64 -w0 | printf "https://itty.bitty.site/#%s\n" "$(cat -)"
This works on Linux, and on Windows in Git for Windows bash shell or a WSL shell. It will not work in cmd or powershell. (for these the echo -n and printf will have to be altered)
The decode example for Linux works fine on the same Windows shells but see #x for corrections to that,

If you're using node, you should probably do it with packages as opposed to calling Window's bash. I'll submit a PR with the code and runkit


Parameter mapping table

I think it would be cool to maintain a whole table of Swiper params, including the following information:

What the default value is in Swiper (maybe not necessary)
What type of param it is (Swiper has some predetermined types)
Whether it's supported in ember-swiper or not
Whether there are any mistmatches between ember-swiper and Swiper (such as a renaming of a parameter)
Any info required to make it a two-way binding that works even after initialization

Swiper has a ton of parameters and I think if we make a table like this, there is a lot of power that will come from it. We can:

Make all supported options two-way bindings
Make option bindings show the correct default value even if it was initialized by Swiper and not ember-swiper
Show in the demo page a table of which options are supported and which ones aren't
Refactor the demo page to make a "demo builder" - just choose which options to use in your slider, and handlebars will render a demo slider plus the code it takes to build it!

Thoughts?

Hey!
I like most of your ideas, with one exception: ember-swiper really should not care about default values at all. Everything that isn't part of the wrapper, will never misalign with what swiper does.
I very much like your idea of a demo-builder. As said in your PR: I will look into a basic refactoring tomorrow. Maybe this could be a basis for such a table / interactive demo.

ember-swiper really should not care about default values at all.

Fair enough. Maybe we can augment the parameters table with default values only for the test dummy app - that way the demo can show default values but it won't pollute what the addon's component itself needs.

Closed by #73


src: add templates.tgz to git

If the serverless 1.15 build can be bumped to golang 1.16, then this commit can be used to solve build problems for kn noted in https://github.com/boson-project/func/pull/340
Signed-off-by: Lance Ball<EMAIL_ADDRESS>

Closing in deference to #340


Optimize the service catalog lookup construction process

In #72, the service catalog is constructed by:

Fetching all available services from the backend CF service broker
Trying to publish all those services to the k8s ServiceCatalogEntry third party resource (ignoring errors that the entry already exists)
Fetching the catalog from the k8s ServiceCatalogEntry third party resource

Steps (2) and (3) can be merged as an optimization. Already-existing entries and newly published ones should be written to the lookup table at the same time, and we'll only have to write ServiceCatalogEntrys once.

We've split steward into steward-framework, the Go SDK for Kubernetes resources and control loops, and steward-cf, the CF broker API implementation on top of steward-framework.
steward-framework constructs the service catalog differently (according to ServiceClass described in https://github.com/kubernetes-incubator/service-catalog/pull/31), so this issue no longer applies; closing.


Trying to get machine learning models...

Just following the readme...
$ python utils/download_files.py
Downloading ml_models/bigwig_files/IC_matched_max.bw ...
Traceback (most recent call last):
  File "/home/dan/Build/genomic-data-service/utils/download_files.py", line 67, in <module>
    main()
  File "/home/dan/Build/genomic-data-service/utils/download_files.py", line 61, in main
    download_models(needs_downloading)
  File "/home/dan/Build/genomic-data-service/utils/download_files.py", line 50, in download_models
    s3.download_fileobj(BUCKET_NAME, PATH_TO_S3_KEY[file_], f)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/boto3/s3/inject.py", line 795, in download_fileobj
    return future.result()
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/s3transfer/futures.py", line 103, in result
    return self._coordinator.result()
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/s3transfer/futures.py", line 266, in result
    raise self._exception
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/s3transfer/tasks.py", line 269, in _main
    self._submit(transfer_future=transfer_future, **kwargs)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/s3transfer/download.py", line 354, in _submit
    response = client.head_object(
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/client.py", line 535, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/client.py", line 963, in _make_api_call
    http, parsed_response = self._make_request(
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/client.py", line 986, in _make_request
    return self._endpoint.make_request(operation_model, request_dict)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/endpoint.py", line 119, in make_request
    return self._send_request(request_dict, operation_model)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/endpoint.py", line 198, in _send_request
    request = self.create_request(request_dict, operation_model)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/endpoint.py", line 134, in create_request
    self._event_emitter.emit(
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/hooks.py", line 412, in emit
    return self._emitter.emit(aliased_event_name, **kwargs)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/hooks.py", line 256, in emit
    return self._emit(event_name, kwargs)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/hooks.py", line 239, in _emit
    response = handler(**kwargs)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/signers.py", line 105, in handler
    return self.sign(operation_name, request)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/signers.py", line 189, in sign
    auth.add_auth(request)
  File "/home/dan/Build/genomic-data-service/.venv/lib/python3.10/site-packages/botocore/auth.py", line 418, in add_auth
    raise NoCredentialsError()
botocore.exceptions.NoCredentialsError: Unable to locate credentials

Sorry if I'm being stuipd!

Hi Dan,
The machine learning models are in public S3 bucket. But you need your own AWS credentials for boto3 to connect to AWS. After you set up your own AWS credentials, you should have no problem to download the models.

Thanks, how do I configure the authentication?
On Fri, 27 Oct 2023, 6:55 pm Mingjie Li, @.***> wrote:

Hi Dan,
The machine learning models are in public S3 bucket. But you need your own
AWS credentials for boto3 to connect to AWS. After you set up your own AWS
credentials, you should have no problem to download the models.
—
Reply to this email directly, view it on GitHub
https://github.com/ENCODE-DCC/genomic-data-service/issues/83#issuecomment-1783297745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANKSZTXB5BEACJWWO22HMBLYBPYQVAVCNFSM6AAAAAA6TC54CKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGI4TONZUGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

If you have not created credentials, you can generate them using the AWS Management Console, which is available at https://console.aws.amazon.com/iam/. Store your credentials in a credential file locally. The credentials file is by default stored at ~/.aws/credentials. This post may help you: https://rollbar.com/blog/python-boto3-nocredentialserror/.


Feat: Recaptcha validation

Hey there 👋 I have created an implementation for Google reCAPTCHA V2 so we can start the code review process.
When we agree on the structure, let's keep the PR open and I will implement more providers before the final review and merge.
I did manual QA with solution in this PR and was able to get it working on my demo application:

Related to issue https://github.com/Basharath/FormEasy/issues/1

One concern you might have is why I named the parameter g-recaptcha-response.
This name comes from Google Docs, and while we could easily rename it to something simpler, I find it important to stay as close to the native solution as possible. That will help people experienced with Google reCAPTCHA to easily use it in FormEasy without having to scan docs.
On the other hand, this will create inconsistency between different captcha providers. This could be a valid reason to try to use specific words for all of them, like captcha. But with that, if some provider requires more than just a string response, we would make it a mess and people would have to learn how to pass it into FormEasy.

I would love to hear your feedback on this.

Hey @Basharath 👋
What do you think about PR - all good? Can I start implementing more adapters to support more captcha providers?

Hey @Meldiron
I was waiting to hear from you about that.
It is all good so far, I haven't merged yet.
Please add all the providers you want and then tell me so we can finalize the merge. Thanks!

Yeah, that's fine, you can take your time. Once done just update here.

any ETA on merging of this PR?
I am looking to use FormEasy with reCAPTCHA. Without CAPTCHA, it's just too much spam.

I won't be able to research other providers any time soon. With that said, reCAPTCHA is properly implemented and tested. This PR can be merged, in my opinion. In the future, I will open a new PR to add new providers.
Let me ping @Basharath about this.

Hey, @Meldiron I thought to ask you about this a few days ago, meanwhile, this happened.
I'll try merging this by this weekend and publish the new update.


docs: add kotlin examples for @RetroAchievements/api-kotlin

This pull request is to:

push the usage docs for @RetroAchievements/api-kotlin
Add a link to the Kotlin libary to Client Libraries in index.md

Hi @official-wizard
Looks like something may have gone wrong with the git hook responsible for automatically running Prettier. Can you try running locally:
npm run format


UI: edges should come out from PromoteSubscribers button

In the following DAG, the edges coming out of the node come from the top and bottom of the upstream node. Because of that, they are disconnected from the PromoteSubscribers button.

Since the intent of the PromoteSubscribers button is to promote to downstream, I feel the edges should originate out of the button as a better visual indicator of what will be affected..
It seems possible because Argo CD can do it (and we use the same DAG library):

Also, maybe we should use this chance to make the promote subscribers more obviously a button. I think it may not be clear.

For some reason, the subscription nodes didn't get the same treatment. Reopening


time filtering bug?

Describe the bug
https://stackoverflow.com/questions/67932711/fiware-quantum-leap-date-query

Hi @chicco785 @c0c0n3 , I have tried to reproduce this issue. In my investigation I found that the API used by the user
curl -X GET "http://localhost:8668/v2/entities/urn:ngsi-ld:Pmr_obs:001/attrs/slotStatus?dateFrom=2020-12-01T00:00:00.000Z&dateTo=2020-12-31T23:59:59.999Z" -H "Accept: application/json" -H "Fiware-Service: atos" -H "Fiware-ServicePath: /mtp"

contains field dateFrom and dateTo but as per quantumleap's API specification it should be fromDate and toDate (which worked perfectly fine).
So, the API used by the user is wrong which results in this behavior.
Kindly let me know if anything needs to be updated in code or document part.

@pooja1pathak that's great, thank you so much for getting into the bottom of this! Since the user called the API with the wrong parameter (dateTo instead of toDate) there's nothing for us to fix---yay! The user should simply replace the dateTo param name w/ toDate like you said.
I don't have a stackoverflow account so I can't comment there, but if any of you does, please just mention on stackoverflow dateTo isn't part of the API spec so it's ignored. To make the query work the user should set the toDate param instead.

@chicco785 maybe you can just link @pooja1pathak's comment in the stackoverflow ansewer?

@chicco785 maybe you can just link @pooja1pathak's comment in the stackoverflow ansewer?

done, thanks a lot @pooja1pathak !


Styling updates.

Some styles need to be updated for readability and fluidity. For example, while hovering in the footer the social links go to grey but it does not match the flow of the site.  Another example is the background images which are used in the paraxell scrolling feature is not appealing for mobile users. Other styles for the general theme could be updated and any help would be appreciated. Feel free to ask questions.

TODO:

Now the header needs to be updated so I can have a sandwich menu on screens smaller than 850 px.

-Update the table to switch earlier so the text is not microscopic.

Add an element that contains "Online Resume" in the header, and make responsive.

Add smaller background images and/or solid colors with parallel scrolling on smaller screens than 1480px.

Proofread all spelling and grammar, and final style touches (Color scheme).

Styles are content now.


hamts: typofix s/vector/trie/

Personal backlog cleanup.

Personal backlog cleanup.

What does this mean?  Was the fix not working/valid?  Should it be still applied to Guix?

Yes.

Yes.

Done.


x/mod/modfile: Parse only sets Module.Mod.Path, not Version

When using golang.org/x/mod/modfile.Parse() to parse a go.mod file into file File, I expect file.Module.Mod to have both Path and Version set. Instead, only Path is set - including the version as last element.
The go.mod contained this line: module github.com/onsi/ginkgo/v2 (the exact package being not important), running in dlv I confirmed the behavior:
(dlv) p file.Module.Mod
golang.org/x/mod/module.Version {
	Path: "github.com/onsi/ginkgo/v2",
	Version: "",}

I used file, err := modfile.Parse(fmt.Sprintf("go.mod@%v", version), fileContents, nil) to parse the file, no error was returned. The filename given as argument should be irrelevant to this problem.
Please either change the code to have Path and Version set or document this is not the case and what users should do instead.

Version details
go version go1.17.5 linux/amd64
golang.org/x/mod version v0.5.1

the Version.Version field documents the expected contents, in this case:

First, the top-level target of a build has no specific version and uses Version = ""