Comments on commits that can't be seen.

This is either an issue with GopherCI or with GitHub, I can't immediately tell. Reporting here for investigation.
I'm seeing some notifications and signs of comments left on commits, yet when I visit that commit, I can't see any comments. Here's an instance that happened just now.
In my GitHub notifications:

At https://github.com/shurcooL/github_flavored_markdown/commits/master:

Note it shows there are 2 comments on that commit.
Yet the actual commit shows 0 comments:

This symptom, comments on a commit that aren't visible, usually occurs when the commit comment API is used on a file or line that hasn't changed. So the commit does have two comments, but the UI only shows changed lines so the comments are missing.
So I'm going to suggest it looks like a bug on my end, placing comments on lines that didn't change. This has been tested extensively, though, so I do find this a little surprising.
It appears golint and apicompat raised the issues (based on logs), apicompat may have complained about an exported function being removed (although this is a main package, so it shouldn't have). I can't see immediately why golint would have had an issue with this change.
I'll need to look into this a little later, unfortunately the logging isn't extensive for this area. For my records:
{"installationID":5912,"level":"info","logger":"gci","msg":"analysing","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:09+09:30"}
{"analysisID":1049,"installationID":5912,"level":"info","logger":"gci","msg":"created new analysis record","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:09+09:30"}
{"analysisID":1049,"installationID":5912,"level":"info","logger":"gci","msg":"Setting https://api.github.com/repos/shurcooL/github_flavored_markdown/statuses/cccd3ce4f8e394ae9f87de0bd8b37e00625913d9 state: \"pending\", context: \"ci/gopherci/push\", description: \"In progress\"","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:09+09:30"}
{"area":"docker","containerID":"f1c55b9b8d6ccb0d5401c28543b2534d7134d2a0e16b9021e6486929630d7b20","level":"info","logger":"gci","msg":"created exec for cmd: \u0026{9ca119cab4faef64553c96870ec33b1479b3a51ba3e01d53efd7be5d32e4531c}%!(EXTRA []string=[bash -c ulimit -v 1572864 \u0026\u0026 cd $GOPATH/src/github.com/shurcooL/github_flavored_markdown; git checkout cccd3ce4f8e394ae9f87de0bd8b37e00625913d9])","server_name":"www1","time":"2017-09-19T03:48:10+09:30"}
{"area":"docker","containerID":"f1c55b9b8d6ccb0d5401c28543b2534d7134d2a0e16b9021e6486929630d7b20","level":"info","logger":"gci","msg":"created exec for cmd: \u0026{cb88d577c385b96d6f41395a3e14923606ceb4fe34a0936a82e586b4b0c8754c}%!(EXTRA []string=[bash -c ulimit -v 1572864 \u0026\u0026 cd $GOPATH/src/github.com/shurcooL/github_flavored_markdown; git diff cccd3ce4f8e394ae9f87de0bd8b37e00625913d9~1...cccd3ce4f8e394ae9f87de0bd8b37e00625913d9])","server_name":"www1","time":"2017-09-19T03:48:11+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"No (or unsupported) vendoring tool, using go get","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"install-deps.sh","time":"2017-09-19T03:48:18+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"ran tool","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"go vet","time":"2017-09-19T03:48:18+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"revgrep found 0 issues","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:18+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"ran tool","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"golint","time":"2017-09-19T03:48:19+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"revgrep found 1 issues","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:19+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"main.go was not generated","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"isFileGenerated","time":"2017-09-19T03:48:19+09:30"}
{"area":"docker","containerID":"f1c55b9b8d6ccb0d5401c28543b2534d7134d2a0e16b9021e6486929630d7b20","level":"info","logger":"gci","msg":"created exec for cmd: \u0026{a72149de8bc6976a3b6b49dc26338260904a6684b52ef677d6c65b0581fc9b10}%!(EXTRA []string=[bash -c ulimit -v 1572864 \u0026\u0026 cd $GOPATH/src/github.com/shurcooL/github_flavored_markdown; apicompat -before cccd3ce4f8e394ae9f87de0bd8b37e00625913d9~1 ./...])","server_name":"www1","time":"2017-09-19T03:48:19+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"ran tool","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"apicompat","time":"2017-09-19T03:48:19+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"revgrep found 1 issues","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:19+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"main.go was not generated","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"isFileGenerated","time":"2017-09-19T03:48:19+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"ran tool","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"gosimple","time":"2017-09-19T03:48:26+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"revgrep found 0 issues","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:26+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"ran tool","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"staticcheck","time":"2017-09-19T03:48:34+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"revgrep found 0 issues","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:34+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"ran tool","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"unused","time":"2017-09-19T03:48:40+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"revgrep found 0 issues","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:40+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"ran tool","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","step":"unparam","time":"2017-09-19T03:48:46+09:30"}
{"analysisID":1049,"area":"analyser","installationID":5912,"level":"info","logger":"gci","msg":"revgrep found 0 issues","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:46+09:30"}
{"analysisID":1049,"installationID":5912,"level":"info","logger":"gci","msg":"Setting https://api.github.com/repos/shurcooL/github_flavored_markdown/statuses/cccd3ce4f8e394ae9f87de0bd8b37e00625913d9 state: \"success\", context: \"ci/gopherci/push\", description: \"Found 2 issues\"","owner":"shurcooL","pr":0,"ref":"cccd3ce4f8e394ae9f87de0bd8b37e00625913d9","repo":"github_flavored_markdown","server_name":"www1","time":"2017-09-19T03:48:46+09:30"}

apicompat may have complained about an exported function being removed (although this is a main package, so it shouldn't have)

It's not a main package (a command), it's package github_flavored_markdown (a library). The commit is indeed a breaking API change. It removes a (deprecated) exported function.

I can't see immediately why golint would have had an issue with this change.

I suspect it's:
github_flavored_markdown $ golint
main.go:11:1: don't use an underscore in package name
sanitize_test.go:1:1: don't use an underscore in package name

It'd probably try to leave that comment on the package clause line.
(The underscore is unfortunate, but it's too expensive to change now. The package is very old and predates me being aware of best practices for Go package names.)

I see the 2 comments at via the API:
https://api.github.com/repos/shurcooL/github_flavored_markdown/commits/cccd3ce4f8e394ae9f87de0bd8b37e00625913d9/comments
[
  {
    "url": "https://api.github.com/repos/shurcooL/github_flavored_markdown/comments/24381322",
    "html_url": "https://github.com/shurcooL/github_flavored_markdown/commit/cccd3ce4f8e394ae9f87de0bd8b37e00625913d9#commitcomment-24381322",
    "id": 24381322,
    "user": {
      "login": "gopherci[bot]",
      "id": 24670144,
      "avatar_url": "https://avatars3.githubusercontent.com/in/889?v=4",
      "gravatar_id": "",
      "url": "https://api.github.com/users/gopherci%5Bbot%5D",
      "html_url": "https://github.com/apps/gopherci",
      "followers_url": "https://api.github.com/users/gopherci%5Bbot%5D/followers",
      "following_url": "https://api.github.com/users/gopherci%5Bbot%5D/following{/other_user}",
      "gists_url": "https://api.github.com/users/gopherci%5Bbot%5D/gists{/gist_id}",
      "starred_url": "https://api.github.com/users/gopherci%5Bbot%5D/starred{/owner}{/repo}",
      "subscriptions_url": "https://api.github.com/users/gopherci%5Bbot%5D/subscriptions",
      "organizations_url": "https://api.github.com/users/gopherci%5Bbot%5D/orgs",
      "repos_url": "https://api.github.com/users/gopherci%5Bbot%5D/repos",
      "events_url": "https://api.github.com/users/gopherci%5Bbot%5D/events{/privacy}",
      "received_events_url": "https://api.github.com/users/gopherci%5Bbot%5D/received_events",
      "type": "Bot",
      "site_admin": false
    },
    "position": 0,
    "line": null,
    "path": "main.go",
    "commit_id": "cccd3ce4f8e394ae9f87de0bd8b37e00625913d9",
    "created_at": "2017-09-18T18:18:46Z",
    "updated_at": "2017-09-18T18:18:46Z",
    "author_association": "NONE",
    "body": "golint: don't use an underscore in package name"
  },
  {
    "url": "https://api.github.com/repos/shurcooL/github_flavored_markdown/comments/24381323",
    "html_url": "https://github.com/shurcooL/github_flavored_markdown/commit/cccd3ce4f8e394ae9f87de0bd8b37e00625913d9#commitcomment-24381323",
    "id": 24381323,
    "user": {
      "login": "gopherci[bot]",
      "id": 24670144,
      "avatar_url": "https://avatars3.githubusercontent.com/in/889?v=4",
      "gravatar_id": "",
      "url": "https://api.github.com/users/gopherci%5Bbot%5D",
      "html_url": "https://github.com/apps/gopherci",
      "followers_url": "https://api.github.com/users/gopherci%5Bbot%5D/followers",
      "following_url": "https://api.github.com/users/gopherci%5Bbot%5D/following{/other_user}",
      "gists_url": "https://api.github.com/users/gopherci%5Bbot%5D/gists{/gist_id}",
      "starred_url": "https://api.github.com/users/gopherci%5Bbot%5D/starred{/owner}{/repo}",
      "subscriptions_url": "https://api.github.com/users/gopherci%5Bbot%5D/subscriptions",
      "organizations_url": "https://api.github.com/users/gopherci%5Bbot%5D/orgs",
      "repos_url": "https://api.github.com/users/gopherci%5Bbot%5D/repos",
      "events_url": "https://api.github.com/users/gopherci%5Bbot%5D/events{/privacy}",
      "received_events_url": "https://api.github.com/users/gopherci%5Bbot%5D/received_events",
      "type": "Bot",
      "site_admin": false
    },
    "position": 0,
    "line": null,
    "path": "main.go",
    "commit_id": "cccd3ce4f8e394ae9f87de0bd8b37e00625913d9",
    "created_at": "2017-09-18T18:18:46Z",
    "updated_at": "2017-09-18T18:18:46Z",
    "author_association": "NONE",
    "body": "apicompat:  breaking change declaration removed"
  }
]

The comments are as suspected. One from golint, another from apicompat.
It seems they're missing position information:
"position": 0,
"line": null,
"path": "main.go",

It looks to me that there are 2 separate things happening here:

GitHub should either display comments that have missing line information both on commits list page, and on the individual commit. Or not display them anywhere and don't send notifications. But not the inconsistent state now where you get a notification, see number of comments on commits list page, but can't actually see the comments. (Alternatively, creating such a comment should return an error.)

You may or may not want to create such comments in GopherCI.

I've reported the GitHub-facing issue to them via<EMAIL_ADDRESS>

OK, there are two issues here:

revgrep should have excluded the issue main.go:11:1: don't use an underscore in package name, this was caused by revgrep incorrectly detecting the file as new, and I'll patch that shortly.
Although apicompat correctly reported the issue, it shouldn't have, and that was due to #86 (and possibly #63). It reported the issue for the same reason above, revgrep incorrectly detected the file main.go as being a new file.

I'll fix revgrep now.

No problem. GitHub support (h/t @izuzak) got back to me, confirming the following:

Comments on a diff line with the index 0 are invalid. Diff lines start at 1. GopherCI should not be creating such invalid comments.

GitHub's API is missing the input validation to catch the error. They've raised an internal issue and will hopefully add the missing input validation, so that trying to create the same invalid comment in the future would return a failure rather than succeeding (and causing a bad user experience on github.com).

Thanks for reporting this, Dmitri -- that does look strange. I agree that looks strange.
So, as far as I can tell -- the reason why that's happening is that the comments were created on an invalid diff line with the index 0. Notice the values of the "position" parameter for those comments in the API response -- they have the value 0. That's not a valid position in the diff -- diff lines start at 1.
So, the problem here is that the API is missing a validation -- the API shouldn't have allowed those comments to be created since they can't be displayed. But due to the missing validation -- the comments were created and exist, but they can't be displayed. This is why the count says that there are two comments, but none are shown.
I hope this explains things, but let me know if it doesn't or you have any other questions. The integration you're using should be able to fix this by making sure comments are created with valid permissions. I'll also open an internal issue so that the team can consider adding more validations there and rejecting calls which don't have a valid position.

I agree 100% with GitHub's response and great to see this'll be a validation error too, thanks.

Resolve upstream and rebuilt/deployed GopherCI, thanks again for spotting and raising the issue.

Seeing this issue happen again, both on GopherCI side (it produces an impossible line number value of 0) and GitHub API side (it doesn't prevent an inline comment from being made on an impossible line number, causing a notification to show up, but comment is not visible).
https://gci.gopherci.io/analysis/1247

Line 0 is impossible because lines are one-indexed.
I got a notification on GitHub about the comment:

But can't see the comment:
https://github.com/shurcooL/users/commit/3500fa32fc1c1bce424c831ce06357dd3b2ddab5

Worth noting, the output from apicompat states line 49, not 0. It seems it wasn't parsed correctly:
$ apicompat -before 3500fa32fc1c1bce424c831ce06357dd3b2ddab5~1 ./... 245ms
HEAD:users.go:49: breaking change members removed
	type User struct {
		UserSpec
		Elsewhere	[]UserSpec
		Login	string
		Name	string
		Email	string
		AvatarURL	string
		HTMLURL	string
		CreatedAt	time.Time
		UpdatedAt	time.Time
		SiteAdmin	bool
	}
	type User struct {
		UserSpec
		Elsewhere	[]UserSpec
		Login	string
		Name	string
		Email	string
		AvatarURL	string
		HTMLURL	string
		SiteAdmin	bool
	}


Combining MSBuild and nswag.json variables might create invalid json

I was trying to use Variables and NSwag.json from MSBuild like this:
<Exec Command="$(NSwagExe_Core20) webapi2swagger run nswag.json /variables:"assembly=$(TargetPath)" />

This will end in Invalid JSON, since $(TargetPath) naturally contains "" which is needed to be escaped in JSON.
My proposals to solve this are:

Either provide a way to pass overrides for nswag.json (e.g: /overrides:"path:in:json=newValue") (which is still problematic, because multiple overrides have to be separated by something, which might also be part of newValue)
Or allow automatic string escaping in nswag.json (e.g: nswag.exe run nswag.json /escapeVars:true /variable:"foo=b\ar")

I assume the last one would be easier to implement in the variable Substitution, and with some advice where to look, i'd also provide a PR for that.

I'd say that variables are only allowed in string properties, so they should be always escaped?
The code is in:
https://github.com/RSuter/NSwag/blob/master/src/NSwag.Commands/NSwagDocumentBase.cs#L183
https://github.com/RSuter/NSwag/blob/master/src/NSwag.Commands/NSwagDocumentBase.cs#L271

Hm.. there's a solution for the Environment variables, which might also work for user provided variables here:
https://github.com/RSuter/NSwag/blob/3a780d1e55844eccd7162d5b0d77271e2ecb68d6/src/NSwag.Commands/NSwagDocumentBase.cs#L181
Can you think of any valid use case for "" in the nswag.json currently?

Hmm, backslashes are allowed in paths on Windows... Maybe we should instead of replacing this particular case just JSON escape the env vars and other vars?

Windows uses / and \ interchangeably as far as I know (that's why the replacement in line 181 also solves the problem). So problaby it'd be sufficient to use p.Value.Replace("\\", "/") instead of p.Value during the Substitution.
But since i'm more or less new to nswag, I cannot forsee use cases for \ in variables ..

.Replace("\\", "/") avoids the problem of escaping \ in the vars - but it seems that we have to escape anyway, so we can just remove the replace and instead of proper escaping...
"path": "sample/$(myPath)/foo"

Here my path can be "foo\bar" or "foo/bar" but \ must be escaped in order to have valid JSON...

Changed two lines of Code, all Tests (not using something remote) were green:
https://github.com/RSuter/NSwag/pull/1252


Added Audi UI Kit

Audi Ui Kit is a interface design system within the renewed Audi Brand Guidelines.

Looks awesome, thanks for adding!


Add sockets support

Would it be possible to add support for using sockets in ucode?
I am especially interested in ethernet sockets, I want to communicate with the management interface of a vdsl modem.

We've been tinkering with sockets over FFI a little here: https://github.com/jow-/ucode/pull/121
Far from ideal, but maybe just enough for you

In the meanwhile OpenWrt also gained an ucode binding for libuclient, which provides support for doing HTTP requests:
https://github.com/openwrt/uclient/commit/e611e6d0ff0ba31070abacdb53037d3e9a4115a6

See also https://github.com/jow-/ucode/pull/196

Fixed by #196


Added "Check for Updates" feature in Thunderbird Android

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
8.0b2
Problem you are trying to solve
Added "Check for Updates" feature in Thunderbird Android。When you check that there is a new version, you can directly click Update. Firefox android doesn't have an update check function, and then if I want to update the new version of Firefox, I have to go to google play to download the update, but I'm in China, and I need to use a vpn to use Google play, which is very troublesome, so in order to avoid this from happening to thunderbird, I need to add the check update function
Suggested solution
Thunderbird needs to automatically detect updates like Firefox windows and download updates if they are available ...
Screenshots / Drawings / Technical details
No response

Note that for F-Droid Tracking Anti-Feature tag needed if the update checker will be enabled default.

Checking for updates without your knowledge or permission

Examples of where it would not be applied - any of the above, if the functionality is opt-in (i.e. you are asked before it happens) and disabled by default.

And auto-updater/downloaders are strictly against F-Droid' inclusion policy.

The software must not download additional executable binary files (e.g. addons, auto-updates, etc.) without explicit user consent. Consent means it needs to be opt-in (it must not be harder to decline than to accept or presented in a way users are likely to press accept without reading) and structured in a way that clearly explains to users that they’re choosing to bypass F-Droid’s checks if they activate it.

You can simply use FFUpdater
https://f-droid.org/packages/de.marmaro.krt.ffupdater/

I've made a pull request to FFUpdater to support this. I don't think we should be maintaining a separate update check feature within Thunderbird for Android.
https://github.com/Tobi823/ffupdater/pull/642


Could you add a function with pull-up-load-more?

Thank you for a nice tool! if it has a function with pull-up-load-more, It will be perfect! I am looking forward it! Thanks again!

@278956952 Thanks for kind words :) But unfortunately we are not planning to add this functionality at the moment. Furthermore pull-up-load-more feature doesn't fit the design.


New, not flashable bulb, help?

hi all,
i recently bought a pair of smart light bulbs (LOHAS E14 from Amazon https://www.amazon.de/dp/B0796MX6DP/), that seems to be not compatible.
flashing over wifi with tuya convert fails, like many others, because of new firmware/ssl?
after i opened one bulb i found out, that it's kind of tricky, if not impossible (for me), to solder and flash it. the TX/RX/GPIO0 pads are hidden on the lower side of the PCB.
if anyone has an idea, where to measure or solder, i can try. but i guess, it's not compatible. :(
best regards

Do a Google search for "TYWE2S pinout".  There are a bunch of little modules like you have, that are used in these lights, and they all have similar names.
https://docs.tuya.com/en/iot/device-development/module/wifi-module/wifie2smodule?id=K9605u79tgxug
You'll also find a lot of these modules by searching for the FCC page.
https://fccid.io/2ANDL-TYWE2S/Users-Manual/Users-Manual-3596121
Sometimes they keep the pinout restricted.  I have a couple lights here with the TYWE2L chip, and they didn't have the manual available for the first year.  That just expired a few days ago.  Whoohoo!
For yours, viewing where there are 6 silver and 5 copper pins, the pins you're interested in are...
1 - 3.3V
3 - GND
5 - RX
7 - TX
There are just a few GPIO pins on these.  It'll save you a lot of guessing.  They are 4, 5, 12, 13, 14.
If this is like some I've played with, just start setting each of those to be PWM, and see what lights up.
If that doesn't work, you'll need to use your multimeter, and see what is connected to the GPIO pins with no resistance, and then check the specs on the chip they are attached to.
Good luck.

I managed to flash a couple of LOHAS GU10 bulbs with tuya-convert, and they have very similar internals based on your photos. In case it's useful, here is my esphome config for the GPIO:
output:

platform: esp8266_pwm
id: output_red
pin: GPIO5
platform: esp8266_pwm
id: output_green
pin: GPIO4
platform: esp8266_pwm
id: output_blue
pin: GPIO13
platform: esp8266_pwm
id: output_warm_white
pin: GPIO12
platform: esp8266_pwm
id: output_cold_white
pin: GPIO14

I hope this is useful, if you haven't got it working already.

thank you both, but i meant, that it's not possible to remove the blue module (TYWE2S) from the white PCB. it's soldered flat (reflow?) and i dare not solder it. i would need a hot air gun / hot air solder station and heat the hole module to ~ 300°C. i'm afraid that i damage something...
so, this post was more a "warning" to others, who might buy these modules :) as long as tuya convert doesn't work with the new firmware, it's not possible or at least really difficult to flash theses bulbs.

That's a shame - I also have a couple of LOHAS bulbs's which were unflashable OTA due to the new firmware, and I was going to look into flashing the TYWE2S - but as you say it appears like it would require additional equipment.

You should be able to solder onto the fingers sticking down from the module.  The part with 6 silver and 5 gold parts.  It may be necessary to scrape some paint.  It may be necessary to drill through just the module's PCB (not the PCB holding the LEDs), but only on 1,3,5,7.
If you look up the other ICs on the opposite side, you can probably find the one responsible for handing the low voltage power over to it.  Clip on those for 3v3 and GND with your TTL adapter.  With that, you'd only need connect to pins 5, and 7 on the module.
You can accomplish all that with a regular soldering iron, TTL adapter, razor knife, and maybe a dremel.
Don't give up.  You already voided the warranty.  You may as well make it work.  You're almost there.

Thanks for your suggestion, but I still need GPIO0 BC it must be hold LOW for flashing :) or at least to enter flash mode, mustn’t it?
We already know the design of the TYWE2S, so we know that the “silver” pins are 3V3, GND, TX and RX. If I manage to connect to TX and RX, it should be easy to get 3V3 and GND too. But what about GPIO0? :(
Somehow I can’t imaging that it’s used to control one of the PWMs. And I can’t find any information about the LED driver. Just that it uses a PWM from the esp. my guess is that 5 PWMs (one per color and one per white (warm and cold)) are used. And as we have 7 possible GPIOs on the lower side of the module, It would be to easy if GPIO0 would have been used to control one PWM ....
Maybe it’s best to wait till WiFi flash is possible again :)

Just a quick update:
A friend of mine managed to desolder the esp-Module from The LED PCB.
If anyone is interested, here are two pictures.
As thought, GPIO0 isn’t used to control a pwm. So one must desolder the PCBs or the metal shield from the esp directly.

Thanks for a useful update - gives me some confidence that I could attempt this. Was it a particularly difficult task to de-solder, just a conventional soldering iron used?

Hey,
I've asked my friend. They've used a RD-500V rework machine. So nothing one has at home. but he said, it's possible to do similar things with an hot air rework station. it may need some training/testing, so nothing that you would like to test on your first bulb :)
a conventional soldering iron is not sufficient.

Definitely not then! - thanks very much for enquiring - will wait until tuya-convert hopefully is able to flash bulbs with the newer firmware.

Hi thought I would ask this here if that is all good.
Any idea if these A19 Lohas light bulbs can be flashed over wifi or will they run into the same issue as the E14 bulbs?
https://www.amazon.com/LED-LOHAS-2700K-6000K-Daylight-Equivalent/dp/B07T7W7ZMW
Thanks

I just bought the A19 Lohas in the link above and I can't flash with Tuya convert either of course. Here are the internal pics:

I haven't started probing for voltages yet but I'm about to if no one else has done this yet.

I just bought the A19 Lohas in the link above and I can't flash with Tuya convert either of course. Here are the internal pics:

I haven't started probing for voltages yet but I'm about to if no one else has done this yet.

Darn... Thanks for the update

Hey @chrisphotonic did you probe for voltages with the Lohas A19. Do you think it will be possible to create a template for these?
Thanks.

@chrisphotonic / @jdubz666 Any luck with manually flashing these? 3/4 worked with tuya convert but I have one I need to manually flash...

As thought, GPIO0 isn’t used to control a pwm. So one must desolder the PCBs or the metal shield from the esp directly.

Actually the cover pries off fairly easy.  I just used a scribe on mine, though on another one I did have to use a small pair of needle nose to wiggle one side completely loose.

For the original Lohas Candelabra bulb question, I can confirm that with some moderate soldering ability you can flash the ESP chip without desoldering the TYWE2S module from the LED board.  I'll add pictures later when I flash another one, but in the meantime here's the instructions:

First you need to do like I mentioned in my previous post, and remove the TYWE2S module cover.  Again it pries off fairly easily, and so far I found the top of the module (wifi antenna side) has the best gap to start prying from.  Don't be afraid to give it a little effort, though stay mindful of the sensitive components that are present underneath it.
Next you need to remove the white-ish silver enamel / solder mask between top left of GPIO13 to top right of GPIO05 front copper pads.  This will expose the (albeit tiny) pin points for the TX, RX, Ground, & 3.3v on the front of the TYWE2S module.  They're directly inline with the neighboring visible pin points on the copper pads.  If you have the LED board completely removed from the candelabra light base, then you only need to uncover the middle two located to the left and right of the GPIO13 pad.  As @JWSmythe pointed out above, pin 6 on 1F8JK (this designation varies slightly from bulb to bulb) / MP2359DJ-LF on the back of the LED board is the 3.3v.  You don't have to use pin 2 for ground though, as you can just use the negative lead that normally powers the board (i.e. the white wire / pad that connects the LED board to the light's socket base).
Now you need some fine single strand wire to carefully solder to these newly exposed points.  It can be a bit daunting if you're not used to soldering things this small, but it's a lot easier and safer to solder here than it is to the ESP chip directly.  Be sure to check with a DMM that you're newly soldered wires are not inadvertently shorting to the neighboring copper pads.  Afterwards you'll need to attach the fine wire strands to your normal serial flashing wires.  We're not going to the Moon with this thing, so a light quick and dirty solder job is fine... you don't exactly want to be tugging or overheating your fragile wire strands.
Lastly you'll need a fine test lead to momentarily tough and ground out GPIO0 (pin 15 on the ESP chip as shown in my previous post) while you plug in your USB serial flasher.  If you don't have one, you can fashion one from a sewing needle, stranded wire, and some electrical tape.  Remember to be gentle when touching the ESP pins, as they're just soft solder pin legs, and they will deform and get damaged if you press too hard.

That's pretty much it.  When you're done flashing desolder your fine wire strands and reassemble the light.  For those that are curious, there is no need (nor should you really try) to reattach the TYWE2S module cover.  It's just there to protect the components during manufacturing and distribution.  Also I recommend a magnifying visor, glass, or scope to accomplish all of this... the pins and soldering tasks are pretty small, and you don't want to short or touch the wrong things while working on this.  Visors go for as little $15 on Amazon, and they make this project a heck of a lot easier!

Hi, sorry to bump this year old issue
I just have to ask, for anyone who did desolder the chip using hot air gun similar to led bulb posted by @spcqike
How difficult can it be without melting/damaging leds surrounding the chip?

FYI he didn't use a "hot air gun," but rather a professional level solder rework station.  Industrial/commercial ones like he listed can easily cost thousands of dollars, and sometimes tens of thousands!  Using a heat gun or entry level consumer rework set is pretty much guaranteed to damage the components, especially if you're inexperienced at it.  Where this one is more difficult than say the Costco dimmer switches that also uses this module that some desolder, is that it has a few pads soldered down on the back of the board.  You're just super likely to damage something while solder flowing it off AND then back on.
The method I listed above is doable by even novice electronic techs, and only requires a decent soldering iron, a magnifying glass/headset/scope, some fine wire, and a very steady hand.  I've done nearly a dozen of these bulbs this way, and haven't lost one yet (though came close once while removing the ESP chip cover... broke a smd cap off, but was able to find/solder it back on).  If you remove the board from the bulb housing (which you would have to do if you planned to desolder the entire TYWE2S module as you alluded to), then you only need to uncover the TX and RX points, as you can use the voltage regulator IC pin 6 (3.3v) and the white negative power lead (ground) as listed above.
Obviously feel free to wait for someone else to chime in here if you'd like, but unless you have the equipment and skills to do this professionally, then my professional opinion is you're going to absolutely damage the components.

Hot air rework stations aren't expensive.  Please don't try to talk with authority about topics you know nothing about.
It is perfectly practical for him to desolder and resolder any component shown in the pictures.   I can.   It only cost me about $137, and I use it all the time.
This is the hot air rework station. It was only $90 USD.  I could change every single smd component in the light with no problems.  There are a bunch of these, sold under different brand names.  They are all the same thing, from the same factory.    Don't try to compare reviews between them, one that looks the same isn't any better or worse than another.  It's just the randomness of who reviewed on which.  Buy the cheapest one you can find.
https://www.amazon.com/gp/product/B07SCPZJYS
My only problem is when I start working with the really small components, like the resistors around the ESP8285 chip in the photos.   So I use this microscope.  It cost $47 USD.  The same thing about different brands selling the same one.  They're all the same.  Buy the cheapest one you can find.
https://www.amazon.com/gp/product/B081L4Y98G
There is a flaw with that microscope.  For some reason, the board stays on after it's shut off, and won't let it turn back on until the battery dies.  If you get it, pop the top off, and disconnect the battery.  You don't need it if you use it with USB power.  I either plug it into my computer, and use the Windows Camera App to view, or I pull it up on my phone or tablet.    You can clearly see the smallest traces, so it's more than you'll need.   It isn't like a biology microscope, so you won't see cells and microbes. :)
There are other small tools and consumables that you will need.

a flux pen (I use this the most)
flux paste and/or flux liquid in a syringe.  The one for electronics, not plumbing.
solder (wire).  For electronics, not plumbing.
solder paste.
solder sucker (the blue one, it's cheapest)
desoldering wick (braided fluxed copper in a spool)
tweezers (like, the sharp beauty tweezers, and the blunt angled beauty tweezers, from Walmart or whatever)
pick (like a dental pick)
electronics repair toolset, with all the bits.   You don't need it for this project, but you will for others.  Walmart stores stock a decent starter kit for $15.  I have a bigger kit that cost closer to $100, with all kinds of obscure bits that you'll need once ever.
91% or higher isopropyl alcohol, for cleaning boards, and cleaning up flux after soldering.  Health/beauty kind.
q-tips for cleaning, the beauty/ear  kind.
head mounted magnifier.  I use this a lot, if the work is small, but not small enough to require the microscope.

On the ones I noticed not as plumbing, they use a different flux.  Their flux will make a mess of your electronics, and it will corrode pretty soon after you've done the work.
And, finally, don't bump unrelated posts to ask new questions.  You could have asked this on reddit or somewhere, and gotten more useful answers.   I just didn't want you thinking technowizard was right, and this is an impossible as a hobby.  I bought some of that to fix a blown diode on an ESP8266 dev board.   I bought the rest to expand my ability.  Now I will fix most any electronics that I come across, and I have been designing new boards, and having them manufactured.
At some point, I will be making a replacement module for lights like this, to replace the Ali/Tuya module that we can't flash.  I want to make those with a USB port on them too, but it might just get marked header pins.  It'll depend on how small I can make it.  I have designed some really small electronics, and that isn't beyond my ability, I just haven't gotten around to it.

Hot air rework stations aren't expensive. Please don't try to talk with authority about topics you know nothing about.

You're right, I've only been working with this stuff on multi-million dollar electronics for 30 years... what do I know?
Seriously though, I didn't say "hot air rework stations" were expensive... I said the one that spcqike used (a Den-On RD-500V unit) was an industrial grade one, and those type are in fact quite expensive (at least for a regular consumer/hobbyist).  Also I didn't say it was "impossible" to accomplish the removal of the module with one of the sub-$100 cheapy hot air stations... no I just determined that the fact the poster was even asking the question, let alone calling the RD-500V a "hot air gun," that he was likely very inexperienced with this type of work, and that he was "pretty much guaranteed" and "super likely" to melt/damage components.
@JWSmythe I'm glad to hear that people like yourself are getting proficient with more advance tools, and passing that knowledge on to others.  That said, don't presume I know nothing about this topic just because I cautioned an inexperienced electronics person (who let's be honest is probably just wanting to up their smart home game, and nothing more) towards an alternative and novice friendlier method.  As I said above I do this professionally for a living, and I made and posted the above directions for this device and others after reading/seeing numerous ESP based module boards and various components getting destroyed by cheap Amazon hot air setups, and overall lack of skill/experience by smart home hobbyists.

Hot air rework stations aren't expensive. Please don't try to talk with authority about topics you know nothing about.

You're right, I've only been working with this stuff on multi-million dollar electronics for nearly 30 years... what do I know?
Seriously though, I didn't say "hot air rework stations" were expensive... I said the one that spcqike used (a Den-On RD-500V unit) was an industrial grade one, and those type are in fact quite expensive (at least for a regular consumer/hobbyist). Also I didn't say it was "impossible" to accomplish the removal of the module with one of the sub-$100 cheapy hot air stations... no I just determined that the fact the poster was even asking the question, let alone calling the RD-500V a "hot air gun," that he was likely very inexperienced with this type of work, and that he was "pretty much guaranteed" and "super likely" to melt/damage components.
@JWSmythe I'm glad to hear that people like yourself are getting proficient with more advance tools, and passing that knowledge on to others. That said, don't presume I know nothing about this topic just because I cautioned an inexperienced electronics person (who let's be honest is probably just wanting to up their smart home game, and nothing more) towards an alternative and novice friendlier method. As I said above I do this professionally for a living, and I made and posted the above directions for this device and others after reading/seeing numerous ESP based module boards and various components getting destroyed by cheap Amazon hot air setups, and overall lack of skill/experience by smart home hobbyists.

Thanks guys, for clearing that up
I don’t have a hot air workstation yet
I’ve been using hot air gun to desolder beken chips and replace with esp chip to flash tasmota.
It’s been quite an easy task especially when the beken chip is far away from any fragile components that might get damaged from high temperature.
But recently, i have 2-3 switches with the beken chip sitting right next to the button switch.
If i were to desolder this and not damage/melt the button switch next to it.
What would be the best approach with hot air rework station?


DO NOT MERGE YET: Updating gitignores of skeletons

Any other files that should be put into the skeletons' .gitignore files?

db.json  and resource-db.json are changed during build - imho they should stay in the .gitignore

But they are contained in the *-output - folders anyways, aren't they? So stating them seems redundant. Or did I overlook something?

You are right..

Closing because issue is already taken care of.


Design Issue - Helper text in tabs

As we move stuff into tabs, we are starting to add more and more helper text to explain the fields in the tabs.
Can we have a font/color to be used in these tabs as helper text?
Currently, we have it in Namespace edit/detail like this:

And in ingress it's like a warning/helper text and looks like this:

@westlywright  Could you review the service pages to change the helper text to this styling?

Helper texts in Service pages are updated to the banner style.


unmet peer dependency react-dom

cannot install material-ui v1.0.0-beta.12 when I use npm install material-ui@next I get message:
-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>|| ^15.0.0 -- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>|| ^16.0.0-beta.5

This issue is most likely not on our end. Please provide a reproduction example otherwise.


Tip 235 is VNC not RDP

I updated this tip to correctly reference the method of access as VNC, not RDP. VNC is not the same as Remote Desktop, and calling it such will confuse people. I updated the article to correctly be titled "VNC on Azure Linux VMs".
If you're interested in Remote Desktop on Azure Linux VMs, then please check out these articles:

https://build5nines.com/how-to-setup-an-ubuntu-linux-vm-in-azure-with-remote-desktop-rdp-access/

https://docs.microsoft.com/en-us/azure/virtual-machines/linux/use-remote-desktop

perfect ! thanks for the pr and your time


customize the time param format in SQL log

[x] Do only one thing
[x] Non breaking API changes
[x] Tested

What did this pull request do?
Add feature to allow customization of the time parameter format in the SQL log.
User Case Description

Please resolve the conflict.

Use a global variable to customize time format sounds tricky ;(
Maybe we need to figure out another solution...

Use a global variable to customize time format sounds tricky ;(
Maybe we need to figure out another solution...

@jinzhu
Thanks for your feedback. I agree that using a global variable is a quick but inelegant solution. If you have any alternative agree in mind, pls let me know and I want to improve this implementation.


Update WebCamCapture.js

Replaced deprecated createObjectURL with srcObject with fallback

Hey @beije I used this as the basis for some motion capture I did but recently it stopped working, createObjectURL is now deprecated in chrome in favour of .srcObject, this PR fixes that

@VibroAxe thanks!


Security Risk

If Some delete the file from it, then whole application will be stop working,
Please see this one - https://prnt.sc/q2b4ot, https://prnt.sc/q2badv

Hi.
You're deleting the necessary files of application - index.php/vendor dir. After deleting these files whole application will stop to work, not only package

Added possibility to set a list of forbidden commands in the release:
https://github.com/alkhachatryan/laravel-web-console/releases/tag/2.2


Allow request headers to be specified in double creation.

We can now specify required headers. If these headers are in the request
then the double that matches these headers will be returned.

Looks like this is not needed after all


[1.3.z] Pinning external application branches to 3.2

Summary

Pinning usages of Camel on Quarkus and Quarkus quickstarts to 3.2 release stream branches.

Please check the relevant options

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Dependency update
[x] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[ ] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[x] Example scenarios has been updated / added
[x] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

run tests

What about TodoDemoIT. That should probably have some pinning too @mjurc ?

aha, @Disabled("Disabled until Quarkus 3 is released and TODO application migrates to it")

Sadly last time that app got an update was with 2.7.1.

Run OpenShift Tests in JVM are red, @mjurc can you investigate?

@rsvoboda The job needs to be ran on JDK17 executors. I will update it, as JDK11 is deprecated anyway - wdyt?

What requires JDK 17?

@rsvoboda The Apache Camel extension for Quarkus quickstart has compilation target set to JDK17. Since we run the tests on JDK 11, the application gets built locally into container image, deployed and referenced from OpenShift deployment.
Sadly, since the executor is on JDK 11, it cannot compile the quickstart:
https://github.com/apache/camel-quarkus-examples/blob/3.2.x/file-bindy-ftp/pom.xml#L41


Vite build warning with @iconify/tailwind package

[plugin:vite:resolve] [plugin vite:resolve] Module "node:fs" has been externalized for browser compatibility, imported by "/project/node_modules/.pnpm/@iconify+tailwind<EMAIL_ADDRESS>See https://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.

Why are you bundling plugin?


Connnect to E-Mail Service and prepare Design

We need to prepare E-Mail Templates at SendInBlue and need to integrate it into the Azure Functions.

resolved with #15


Cannot read property 'dependencyLib' of undefined

Error on submit form.
Uncaught TypeError: Cannot read property 'dependencyLib' of undefined     at HTMLInputElement.ev (jquery.inputmask.js?1596795273:1451)     at HTMLFormElement.dispatch (jquery-3.1.1.min.js?20191214:3)     at HTMLFormElement.q.handle (jquery-3.1.1.min.js?20191214:3)     at Object.trigger (jquery-3.1.1.min.js?20191214:4)     at HTMLFormElement.<anonymous> (jquery-3.1.1.min.js?20191214:4)     at Function.each (jquery-3.1.1.min.js?20191214:2)     at r.fn.init.each (jquery-3.1.1.min.js?20191214:2)     at r.fn.init.trigger (jquery-3.1.1.min.js?20191214:4)     at r.fn.init.r.fn.<computed> [as submit] (jquery-3.1.1.min.js?20191214:4)     at order.js?1596793641:3384
Windows 10
Edge Browser
I used jquery.inputmask.js, lastest version

@yogi12314 ,
Can you create a codepen or jsfiddle for it.  I cannot reproduce here.

https://codepen.io/yogi12314/pen/QWNbxjR?editors=1010
Here's a little part of my code. Hope that help you

Ok i finded solution. When i have defined inputmask for input by class selector frist, and  in next step add next inputmask by id, then problem occurs.

@RobinHerbots  I am getting the same error but on a completely different use case, and I was able to reproduce in codesandbox [React] - https://codesandbox.io/s/inputmask-undefined-error-yqug4
In order to get the error you need to

Click on button 'Enable the Mask'
Then Click on button 'Remove the Mask'
Enter any value (you can even leave it blank)
Click on Submit.

You will be getting this error - Uncaught TypeError: Cannot read property 'dependencyLib' of undefined
Upon investigation, I found that without enabling the Mask, the Input element don't have the property inputmask. However after enabling the mask we have new property for the input with key inpuptmask
After clicking on Remove mask this property is set to undefined. inputmask: undefined
Which is getting evaluated on the code as undefined when we are submitting the form.
The below piece of code is producing error and its from node_modules/inputmask/dist/inputmask.js
   var EventRuler = {
            on: function on(input, eventName, eventHandler) {
                 var $ = input.inputmask.dependencyLib, ev = function ev(e) {
                    e.originalEvent && (e = e.originalEvent || e, arguments[0] = e);
                    var that = this, args, inputmask = that.inputmask, opts = inputmask ? inputmask.opts : void 0,
   {/*error line*/}  $ = inputmask.dependencyLib;
                 <...>
             }
 } }

I am not sure but undefined check on the inputmask could resolve this issue hopefully.

Below is the screenshot of error in dev tools.

Please let me know if you need any additional details.

@yogi12314 ,
The issue should be fixed in version 5.0.6-beta.19.  Also see #2403


Update-database not finding migrations when solution is moved to a different computer

Describe what is not working as expected.
I have a solution that I created along with migrations, successfully created the database and then uploaded to dropbox. It's using latest EF Core 2.0.1.
I pulled the solution down to another computer..running the same version of Windows 10 (FCU) and same version of VS2017 (15.5.2), the same version of SQL Server LocalDB and the same (latest) version of .NET Core installed.
When I ran update-database to create the database on this machine, I kept getting the "migrations already run" message. The new database got created along with the update-history table, but no migrations got run and nothing was added to update history.
After trying a variety of troubleshooting steps, I finally deleted migrations folder completely as well as the database, ran add-migration, and then update-database and the database got created correctly.
I can't seem to find a related issue but wanted to be sure this wasn't a duplicate (or familiar) before I added more information to this issue.

hey @bricelam ! I'm having the problem again ..testing when someone reported the problem from downloading my samples for Pluralsight. I will put the solution on GH and give you a link.
This is using EF COre 2.0.3, Visual Studio 2017 15.5.6 on windows 10 latest updates.

Repo: https://github.com/julielerman/EFCore2MigrationsProblemForBrice
Here's a screenshot so you can see the WebApp is startup, Data is default project, there's stuff in the initial migration and response to update-database "no migrations applied. The database is already up to date."
Is there something I've done wrong with the setup by including the migrations?

I cloned the project and got repro.

is there anything obvious about my solution that would create this problem?  Since it's the 2nd one I've seen this with ...but haven't tried others ...the only common denominators are 1) Julie 2) VS2017.

There wasn't anything obviously wrong with the project. When i tried adding a new migration I found out what is causing this issue.
When I added new Migration it added 2 files to my project (beside existing migrations), migration file & designer file. Both existing migrations do not have designer files associated with them.
Looking at the structure of designer file & Code to retrieve migrations, each designer file (which is partial class of migration file) has DbContextAttribute & MigrationAttribute. DbContextAttribute tells which Context it belongs to. So in this case we saw the migration but deduced they don't belong to our context and ignored them. Hence Database is up-to-date. When I copied over both the attributes to existing migrations (and updated the MigrationAttribute to correct Id), they were applied correctly to server.
(MigrationAttribute tells the MigrationId to tools. Missing MigrationAttribute was throwing, @bricelam converted it to warning in #9920)

oh! I wonder if those are being trashed by gitignore?

nothing in gitignore. I wonder if git -fxd is removing them. These were solutions to share so I was cleaning them. Will see if I can replicate that...

nope that isn't doing it (I didn't think so , but trying to figure out how they are not making it up to github)

happy to close this issue now if you got what you needed from it


[benchmark] DataCreateSmall - Fix legacyFactor

Adjust the multipliers for legacy factor and inner loop to account for the improved performance of the Data implementation.

ok, no problem


ACM-14858-Streaming-data-stops-on-unfocused-browser-tab-2.12

Signed-off-by: John Swanke<EMAIL_ADDRESS>

/lgtm


Is this line necessary?

https://github.com/lyhue1991/torchkeras/blob/125bdc3707aebd6505e24c2de6bb85332e497c17/torchkeras/kerasmodel.py#L166

Of course, to load  the best weight.

Okay, thank you very much, can I think that after executing the fit function, if there is no subsequent code execution, it is not necessary, but it is necessary if you continue to execute methods such as evaluate？

Okay, thank you very much, can I think that after executing the fit function, if there is no subsequent code execution, it is not necessary, but it is necessary if you continue to execute methods such as evaluate？

You got it!


Incorrect link to header generated when header contain link.

Hi,
thanks for great LSP server. I noticed that when I use link in header like in the example bellow the server autocompletes incorrect link like in the example:
# Google

test

# [Google link](https://google.com)

test

[google link](#google)
[google link with link](#google-linkhttpsgooglecom)

I believe the second link should omit the url part like this#google-link.
Here is screenshot from neovim how I see autocomplete options for this example:

Hi @johnybx! Thanks for reporting.
Generally, with anchor completions, the anchor text is a slug of the respective header's text. With that said:

I think when a header contains a single link we could make the completion and reference resolution handle that in a special way. But then the question is: does Github understands this kind of anchor links?
What if a header is some text + a link. What do you expect the behavior to be this way?

As a workaround I could suggest using link definitions, e.g.
# Google

# [Google Link]

[google link]: https://google.com

Would that work for you or you specifically need full inline links in headings? If so, I'm curious what's the use-case?

Hi,
yes I tested the links in gitlab and also with nvim extension and links work only if anchor slug ignores the link text. Even if heading looks like this
# [test](link) something

I believe the link should be #test-something.
Basically my report was based on the fact that the generated links did not work :).
Currently my use case is developer documentation where heading is most often link to relevant folder or file which is relevant for section.
The workaround looks good, I actually did not even know about that, I will try that in the following days and if it works than I am more than happy to close this 👍.
Thanks for quick response.

Thanks for the info @johnybx! I do care for the generated slugs to be readable by GitHub/GitLab so if they strip out urls from links in the heading, so should Marksman 🙂 I'll look into it. Meantime, yes do try the workaround.

I just tried the workaround and it works but when I define links using link definition than the generate table of contents command is broken
Test table of contents

<!--toc:start-->
- [[Google Link]](#google-link)
<!--toc:end-->

# [Google Link]

[google link]: https://google.com

Should I create another issue for this or is it related enough to have it in single issue?

I have a similar issue when the header contains special characters such as underscores.
For instance:
## Table:some_function

If I create a TOC via code actions, it will create a TOC with anchor reference #tablesomefunction. When trying out on GitHub, it seems that the anchor that would work is actually #tablesome_function. Hence it drops the : correctly, but incorrectly also drops the _.
I am aware this does not perfectly match the name of this issue, but it is a very related issue I guess. Would you want me to create a separate issue about this?


Version 5.2.1

Landed in https://github.com/metarhia/metautil/commit/8b58aa3e31dee0fd5922d4ee0a7dcb321f6b5dcb


Auto-generate .md docs from files

For each properties member of language properties:

start a file with that property name (.md)
start it with descriptions from:

properties class' jsdoc
each command that uses it

step one: tag in the jsdoc?
step two: find all reference?

auto-generate table from jsdocs, then each language's

Meh, not worth it.


Switch to JUnit version 5 for unit tests #4955

migrate unit tests in sdk/eventhubs/*

Can one of the admins verify this patch?

Thanks again @mclay!


[#1604] remove cleanDb() override and fixed error

fixes #1604

removed cleanDb() override
changed Set.of(....) to List.of(....)

The Set isn't ordered so the values order wasn't guaranteed in the deleteEntities() call.   For some reason the Oracle test failed and the other DB's didn't.

Good catch.

The Set isn't ordered so the values order wasn't guaranteed in the deleteEntities() call.

Could you add  this to message in the commit, please?
Something like:

Replaced the Set with a List because entity deletions must happen in the specified order


Ackermann.py added in recursion

Added Ackermann function in recursion. Please Kindly check and merge the pull request.

Hi @anjalibarnwal31, it'd be great if you could add this function to the README.md (under recursion) as well.

I have updated it in the readme section ,should i create a new pull request for it.

Hey, @anjalibarnwal31. This looks good! Merging it in.


Issue while import sonnet

ConversionError: converting <function BaseBatchNorm.call at 0x00000268C3EF7950>: AttributeError: module 'gast' has no attribute 'Constant'
My gast version is 0.3.3
Tensorflow 2.3.1
graph-nets 1.1
How I installed graph-nets
pip install "graph_nets>=1.1" "tensorflow>=2.1.0-rc1" "dm-sonnet>=2.0.0b0" tensorflow_probability

Hi @pshrini I think this is a TF issue not a Sonnet issue (we do not depend on or use gast directly but TF does).
I would suggest making sure you have the latest version of TensorFlow (2.4.1 has been released recently) and all other deps. If you continue to have issues after upgrading I would suggest filing an issue on TensorFlow.


AzimuthalEquidistant to WGS84

I can not deal with a strange bug. At the input: the coordinate of the WGS84 point and the radius in meters. The task is quite simple, you need to build a buffer around this point so that it is correctly displayed in WGS84 coordinates.
Solution:

create an Azimuthal Equidistant projection with the center at the specified point
using the GDAL library and the standard "buffer" function, create a polygon
reproject the polygon back to WGS84

Where is the mistake:

coordinates are truncated
there is an incorrect transformation of the polygon

For reprojection, the PROJ library and GeographicLib::AzimuthalEquidistant were used. (+proj=aeqd +lon_0=129.4943 +lat_0=64.2402 +x_0=0 +y_0=0 +a=6371000 +b=6371000 +units=m +no_defs +type=crs) The result is the same.
Moreover, if the task is completed in ArcGIS, then the ideal option is obtained.
I also did a step by step test and found the following: the error occurs when the coordinates are reprojected back to WGS84.
The polygon built by me and ArcGIS in the projection are identical.
Code example:
void convert_wgs84_to_aeqd(double lonCentr, double latCentr, double lonInp, double latInp, double& lonRes, double& latRes, bool isToWgs)
{
    Geodesic geod(Constants::WGS84_a(), Constants::WGS84_f());
    AzimuthalEquidistant proj(geod);
    const double lat0 = latCentr, lon0 = lonCentr;
    if (!isToWgs) {
        proj.Forward(lat0, lon0, latInp, lonInp, lonRes, latRes);
    }
    else {
        proj.Reverse(lat0, lon0, latInp, lonInp, latRes, lonRes);
    }
}

void create_buffer(double lonCentr, double latCentr, double bufferInMeters)
{
    const double buffer_distance = bufferInMeters;
    const int points_per_circle = 360;

    using coordinate_type = double;
    boost::geometry::strategy::buffer::distance_symmetric<coordinate_type> distance_strategy(buffer_distance);
    boost::geometry::strategy::buffer::join_round join_strategy(points_per_circle);
    boost::geometry::strategy::buffer::end_round end_strategy(points_per_circle);
    boost::geometry::strategy::buffer::point_circle circle_strategy(points_per_circle);
    boost::geometry::strategy::buffer::side_straight side_strategy;

    double xr, yr;
    convert_wgs84_to_aeqd(lonCentr, latCentr, lonCentr, latCentr, xr, yr, false);

    boost_point px(xr, yr);
    boost_mulpolygon polyx;

    bg::buffer(px, polyx, distance_strategy, side_strategy, join_strategy, end_strategy, circle_strategy);

    OGRLinearRing* glr = new OGRLinearRing;
    auto rng = bg::exterior_ring(polyx[0]);

    for (auto vp : rng) {
        double laty = bg::get<1>(vp);
        double lonx = bg::get<0>(vp);
        double lonRes, latRes;
        convert_wgs84_to_aeqd(lonCentr, latCentr, lonx, laty, lonRes, latRes, true);
        glr->addPoint(lonRes, latRes);
    }

    OGRPolygon* pdff = new OGRPolygon;
    pdff->addRingDirectly(glr);

    char* wktStrings;
    pdff->exportToWkt(&wktStrings);
    std::string r(wktStrings);
    std::cout<< r <<std::endl;
}

For some reason I can't add pictures with examples :(

Can you pare down your problem to show

a simple calling sequence of a GeographicLib class
the output that you got
the output that you expected

I'd prefer not to have to wade through your calls to boost::geometry!
Thanks.

Hello! Thank you for your reaction.
For some reason I can't attach pictures, GitHub throws an error on all browsers :(((((
Therefore, I apologize for the inconvenience of the format for providing examples.
This function performs projection to Azimuthal Equidistant and vice versa depending on the isToWgs flag:
void convert_wgs84_to_aeqd(double lonCentr, double latCentr,
                           double lonInp, double latInp, double& lonRes, double& latRes,
                           bool isToWgs)
{
    Geodesic geod(Constants::WGS84_a(), Constants::WGS84_f());
    AzimuthalEquidistant proj(geod);

    if (!isToWgs) {
        proj.Forward(latCentr, lonCentr, latInp, lonInp, lonRes, latRes);
    }
    else {
        proj.Reverse(latCentr, lonCentr, latInp, lonInp, latRes, lonRes);
    }
}

Stages:

I take a point with coordinates and convert to Azimuthal Equidistant projection.
Example: longitude: -46.5733 latitude: -63.8239

With the help of boost (or GDAL, there can be anything) I build a buffer
Example: buffer in meters: 8406747

I get a polygon in  Azimuthal Equidistant projection. If you compare it with the ArcGIS output, it matches, i.e. buffer is built correctly
example image, WKT polygon , JSON object

There is currently an error here.
I go through each point of the polygon and use the convert_wgs84_to_aeqd function to convert from Azimuthal Equidistant to WGS84 latitude/longitude.
result: result image, JSON result, WKT result polygon

This is what the result should actually look like: correct image

Also a few more examples in pictures:
example 2, example 3, example 4

From your pictures, it seems that the errors concern how polygons are
plotted when the longitude range spans longitude = +/- 180d.  You can
check this by dividing the buffer radius by 10 in your first example.
Or add 180d to the longitude of the starting point in example 4.
Most likely geographiclib is performing correctly and you need to
investigate the software you use to display the polygons.
I'm closing this issue for now.  Feel free to reopen if you have
questions about what geographiclib is doing.


CSS minifies badly when there are nested selectors

Hello, I detected a bug when minifying CSS files with nested selectors. I don't know exactly what the problem is, but files containing nested selectors don't work and I had to replace them with the non-minified version. I understand from what I observed that it removes certain parts of the code.

nested selectors don't work? it should work, from what I know. I'll make a update sometime this week.

@TalkNetApps Hello, could you provide some examples. I was unable to recreate such issue, I need some examples to work with.


Add segment tracking

Remove Google Analytics, segment will send that same data over

:+1:

@drapergeek This is great! Thanks. How do we access Segment for Bourbon? Is it in our 1Password?

@drapergeek Ping ^

@tysongach Seems to be invite only on the thoughtbot account. I sent you an invite, sorry for missing for the first notification!


Added the idempotent flag to the kafka Input Operator

Belongs to MLHR-1692

Updated the code as per the comments.


Availability of assoc command

At the present moment, only server owner have access to assoc command.
It's done for security reasons, so @MrShadowMaster can't pretend to be a GreatMan11
And can't get op for this nick (he still can log in with this name, btw).
But, what if play in-game could have an assoc request with dialog, like this:
Hey, GreatMan11, guy from Discord (MrShadowMaster) tell's us, that you're the same person, is it True?
[YES] / [NO]
and click on each button will insert text like
%yes I'm MrShadowMaster and %no I'm not
So from discord you as default (non-admin) user use %assoc GreatMan11 command
(yup, I mean your id used if no-one else mentioned) and you get request in-game, so after confirmation the process will end up ast it was with admin rights :)

At the present moment, only server owner have access to assoc command.

After 8 month... 😄
You can now set role Administrator for bot and add selected members, so I think this problem solved now.

Actually this is different features :)


Cleared Properties not reflected in UI

Consider the sevenguis CRUD example.
The person detailView shows the values of person .name and .surname.
If I call person.name$.clear(), or person$.clear() the detailView is not updated.
Is this expected behaviour?
If I run:
this.person.name$.clear();
undefined
this.person.name;
undefined
I see that it is clearing the person's name property, but the UI isn't updating, so there must be a bug in the property/UI binding when it comes to 'undefined'.  I'll fix.

Fixed.


Can't remove influxdb port

Hello. Ask for help.
Tried to remove row - influxDBPort (I don't need that, because influxdb pod has such settings inside our cluster) but it returns anyway. I tried from UI, tried to remove it from jmx file itself. It was successful - I can't see value for port in jmx file, but it still exists in UI. Don't know if it really removed or not.
If there a way to actually remove this value from test? Or maybe I can use another solution to exclude port from final url?

Hi @Samolevar, what about HTTP OR another port?  It's no possibility that you have no HTTP port to connect to influx  :)  suppose you can try 80 or 443? Please try them ..
BR,
Mike


yarn install failing

yarn install is giving me trouble.
$ yarn install
yarn install v1.22.19
info No lockfile found.
[1/4] Resolving packages...
warning @reactunity/scripts > @svgr/webpack > @svgr/plugin-svgo > svgo ><EMAIL_ADDRESS>Modern JS already guarantees Array#sort() is a stable sort, so this library is deprecated. See the compatibility table on MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#browser_compatibility
[2/4] Fetching packages...
error An unexpected error occurred<EMAIL_ADDRESS>Request failed \"404 Not Found\"".
info If you think this is a bug, please open a bug report with the information provided in "...\\full-sample\\react\\yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Any thoughts on how to resolve this error?

I just tried and it worked for me. I suspect your issue might have been caused by a temporary internet connection loss, or a temporary error in the yarn registry. Running the command again may work. If it does not work, try running npm or setting yarn registry to npm registry. (Note: delete yarn.lock and node_modules just in case)
yarn config set registry https://registry.npmjs.org

Adding the npm registry did the trick. Thank you!


Spring-controller-tests public API

private & public API for Spring-controller-tests module

This change is

Coverage remained the same at 63.183% when pulling e9ac977fb74e9da20b580029409babd2e850904e on private-public2 into 4e926c5f606dc5b3b32e9fb869fd04ff0139b25c on master.

Reviewed 26 of 34 files at r1, 7 of 8 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Coverage remained the same at 63.115% when pulling 2fc580f088798619efea90af4cfe4fca1bdc6fad on private-public2 into eb0e19e997d4a12b85e88bbfb6b70db77faa0ff6 on master.


Where do you add b_{lab(i,j)} to your value term (see eq{2} in the paper)

Hi Authors,
Thanks for releasing the code! This is very helpful.
I was wondering where do you add the bias term $b_{lab(i,j)}$ as mentioned in equation 2 of your paper in this code.
I was looking at Line 155 here:
https://github.com/linjieli222/VQA_ReGAT/blob/master/model/graph_att_layer.py#L155
It seems like the attention weights are directly multiplied to Value vector. If you can point us to where in the code are you adding the bias term $b_{lab}$, that will be great.
Thanks!

Hi,
Thanks for your interests in this project. Sorry about the confusion, our implementation is slightly different from the equation in our paper
Please take at look at this line:
https://github.com/linjieli222/VQA_ReGAT/blob/9f6fe5bcda169c268eb1c92ef00df9f61d540081/model/graph_att_layer.py#L144


README instruction incorrect

Update
To update Bash-it, simply run:
bash-it update
that's all.

When we do this:
○ → bash-it update
bash-it             bash-it help and maintenance
parameters:
                    1: verb [one of: help | show | enable | disable ]
                    2: component type [one of: alias(es) | completion(s) | plugin(s) ]
                    3: specific component [optional]
examples:
                    $ bash-it show plugins
                    $ bash-it help aliases
                    $ bash-it enable plugin git
                    $ bash-it disable alias hg

Now here's a nice chicken/egg problem. It looks like you have an older version of Bash-it locally that does not have the update functionality yet. But you can't update using bash-it update either...
Can you check whether you have the latest version, e.g. your lib/helpers.bash file should have this line: https://github.com/Bash-it/bash-it/blob/master/lib/helpers.bash#L35
In theory, you should be able to do this:
cd $BASH_IT
git pull origin master

This will only work if you have cloned Bash-it from the original remote (the official Bash-it/bash-it). If you have your own fork and are using that, you will have merge the changes from the official repo into your fork first.
Once you have done all of that, you should be able to open a new Terminal session and run bash-it update there.

@nwinkler Sorry for the slow response! Unfortunately the VM I was on got blown away. I'll try to reproduce this error and post here if it returns. Apologies!

Sure, feel free to open a new issue in that case.


Explain the project's philosophy

I tried to explain the project's philosophy in 2 short sentences, so that its aim is clear to people who visit the repo. @deviantony what do you think?
The reference to "elastdocker" is optional. I though it would be nice to provide an example of project that forked this repo and added some bells and whistles to it (lots of opinionated parameters, etc.).

I wasn't 100% sure either, but then Google returned this thread where the author mentions:

This project is a collection of everything I added on deviantony/docker-elk to start a small scale dockerized elk stack.
-- https://www.reddit.com/r/devops/comments/db3cw7/comment/f1zajgw

Yes IMO it's fine and it's a good example. You can merge it.

Thanks for the review!


Add possibility to configure the concurrency for sharing and userlog

Description
This PR adds the possibility to configure the concurrency for sharing and userlog services
Related Issue

https://github.com/owncloud/ocis/issues/10647

Motivation and Context
Adjustabliity
How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[x] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[x] Documentation generated (make docs) and committed
[ ] Documentation ticket raised:
[ ] Documentation PR created:

@d7oc I also reflected https://github.com/owncloud/ocis/issues/10647#issuecomment-2498338980 in this PR.
Basically with oCIS 5, the sharing concurrency was 0, meaning unlimited. So setting it to 20 should be a more sensible default than 1 and also was backed by @butonic 's loadtests on Kubernetes.


Call to a member function all() on null in Mpociot/BotMan/Drivers/BotFrameworkDriver.php:172

I'm using say() function to send notifications via skype. After 1.5 update I get this error.
Judging by code, when using say() getPayload() always is null, so getPayload()->all() will no work at all that way. Temporary fixed that by removing ->all() from BotFrameworkDriver.php.

Oh right. This happens only when originating messages. Would you like to create a pull request for this?
So we check if getPayload is a collection/parameter bag?


Add project manager / editor initial screen settings, implement get_keyboard_focus_screen method.

Implements DisplayServer.get_keyboard_focus_screen (on macOS, X11 and Windows) method to get screen containing the window with the keyboard focus.
Adds new DisplayServer.SCREEN_* constants.
Adds support for new values to the Window and Project Settting initial position option.
Adds new project settings to specify project manager and editor initial screen.

Fixes https://github.com/godotengine/godot/issues/75038

Thanks!


Bug on network down

I noticed that when I run the radio and lose the connection, the audio session does not come back ... I made an update in the code using reacheability, I would like to know if I have the possibility to be a collaborator of the project or just show the code for you. Thank you!

Hello @bartollo- anyone can make a pull request. Thank you!

Hello @swiftcodex , I will do the pull request ... thanks!

hello) i try to convert Swift4 and no work)
i leave Swift3 but no work JSON )
oh and ah Use the Xcode9

@sdvigov you can check the xcode 9 branch : https://github.com/swiftcodex/Swift-Radio-Pro/tree/xcode9


Disable the parallax effect while scrolling up

I would like to disable the parallax effect while scrolling up.
I set segmentedPager.bounces = false to disable parallax when scrolling down but I don't know how to disable when scrolling up.
How can I do it?
Thank you for any help

I managed to do it by changing the method:  layoutContentView inside MXParallaxHeader.m
the result:
- (void)layoutContentView {
    CGFloat minimumHeight = MIN(self.minimumHeight, self.height);
    CGFloat relativeYOffset = self.scrollView.contentOffset.y + self.scrollView.contentInset.top - self.height;
    CGFloat relativeHeight  = -relativeYOffset;

    CGFloat div = self.height - self.minimumHeight;
    CGRect frame = (CGRect){
        .origin.x       = 0,
        .origin.y       = -self.height,
        .size.width     = self.scrollView.frame.size.width,
        .size.height    = self.height
    };
    self.contentView.frame = frame;
    self.progress = (self.contentView.frame.size.height - self.minimumHeight) / (div? : self.height);
}

Any update on this feature?


The only thing is that the Symbols move to the middle...
Is there a bugfix on the road?
Thanks

Whats your W10 build?
If it's 1709 or lower then you need to update.

I have 1909 :)

Facing this issue as well!
1909 build

Taskbar runs on both PCS (1909). Only doesn't work on Notebook (1909)

I have a similar issue. The taskbar is transparent, but after a while it goes back to being opaque. Happens randomly.

my laptop has the same problem,restart pc transparent works 2~5second, then being opaque.

@chendijun527 can I ask how you did that, I have never used task scheduler before

I fixed mine by simply going to the GitHub link I got it from and downloading the newest version, when I ran the install it prompted me to update instead, I did and It now works fine.
This is where I got it:
https://falcon-x.en.softonic.com/download

I fixed mine by simply going to the GitHub link I got it from and downloading the newest version, when I ran the install it prompted me to update instead, I did and It now works fine.
This is where I got it:
https://falcon-x.en.softonic.com/download
This worked for me too, its funny how an older version works.

@83y Yeah.. there some weird things going on. FalconX supports the older way of taskbar styling which should not work on 1803 and above. In 1803 Microsoft made some changes to the SetWindowCompositionAttribute API so in TaskbarX i had to adapt to the changes. But according to some reports and your comments it seems to be all mixed up 😩. Maybe something is forgotten inside the Windows updates...


Add simple CLI commands

There are two very simple Azure CLI commands to create an app insights instance and get the instrumentation key. I'd love to see those documented here, or in a "CLI Quickstart" sister doc.
From this github thread: https://github.com/Azure/azure-cli/issues/5543
az resource create \
                  --resource-group $RESOURCE_GROUP \
                  --resource-type "Microsoft.Insights/components" \
                  --name $NAMESPACE_PREFIX-appinsights \
                  --location $PRIMARY_LOCATION

az resource show -g $RESOURCEGROUPNAME -n $APPLICATION_NAME --resource-type "Microsoft.Insights/components" --query properties.InstrumentationKey

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

ID: 4c8783dc-d0fd-8f1f-88a3-5db2495a1428
Version Independent ID: 076efce2-ff9e-2d2a-ba9b-7d760c5eea0b
Content: Automate Azure Application Insights with PowerShell
Content Source: articles/azure-monitor/app/powershell.md
Service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@alexkarcher-msft
Thanks for your feedback! We will investigate and update as appropriate.

@alexkarcher-msft Thank you very much for the feedback!  We're assigning this to the content author so they can update the documentation accordingly.

@alexkarcher-msft thanks for pointing this out, I will get it added this week.

Hey, sorry this stayed open so long. We have internally merged the following section to the create resource doc. The change will go-live between 3-4 PM (UTC-7).
Azure CLI (preview)
To access the preview Application Insights Azure CLI commands you first need to run:
 az extension add -n application-insights

If you don't run the az extension add command you will see an error message that states: az : ERROR: az monitor: 'app-insights' is not in the 'az monitor' command group. See 'az monitor --help'.
Now you can run the following to create your Application Insights resource:
az monitor app-insights component create --app
                                         --location
                                         --resource-group
                                         [--application-type]
                                         [--kind]
                                         [--tags]

Example
az monitor app-insights component create --app demoApp --location westus2 --kind web -g demoRg --application-type web

Results
az monitor app-insights component create --app demoApp --location eastus --kind web -g demoApp  --application-type web
{
  "appId": "87ba512c-e8c9-48d7-b6eb-118d4aee2697",
  "applicationId": "demoApp",
  "applicationType": "web",
  "creationDate": "2019-08-16T18:15:59.740014+00:00",
  "etag": "\"0300edb9-0000-0100-0000-5d56f2e00000\"",
  "flowType": "Bluefield",
  "hockeyAppId": null,
  "hockeyAppToken": null,
  "id": "/subscriptions/{subid}/resourceGroups/demoApp/providers/microsoft.insights/components/demoApp",
  "instrumentationKey": "00000000-aaaa-bbbb-cccc-dddddddddddd",
  "kind": "web",
  "location": "eastus",
  "name": "demoApp",
  "provisioningState": "Succeeded",
  "requestSource": "rest",
  "resourceGroup": "demoApp",
  "samplingPercentage": null,
  "tags": {},
  "tenantId": {tenantID},
  "type": "microsoft.insights/components"
}

For the full Azure CLI documentation for this command, and to learn how to retrieve the instrumentation key consult the Azure CLI documentation.

#please-close


reportMissingModuleSource in apps.py of an empty Django project

This issue has been moved from a ticket on Developer Community.

[severity:I'm unable to use this version]
I've just installed Visual Studio 2022 and created an empty Django project. I'm getting a lot of error messages when creating the virtual environment (see attached screenshot). Are these messages "normal"? However, I can run the server.
When I create my first app (HelloDjangoApp), when updating views.py, I have, among others (see screenshot),
reportMissingModuleSource Import "django.ashortcuts" could not be resolved from source.
After having followed the differents steps of the tutorial, I can't run the server anymore. message: "cannot imprt 'url' from 'django.conf.urls'" (see sceenshot)
It seems there is some problem regarding packages resolution. But, I do not see what to do...

Original Comments
Feedback Bot on 4/20/2023, 07:47 PM:
(private comment, text removed)

Original Solutions
(no solutions)

Duplicate of https://github.com/microsoft/PTVS/issues/7068. Fix is here: https://github.com/microsoft/PTVS/pull/7401
The issue should be fixed in the next version.


hardware.pulseaudio.enable conflicts with sound.mediaKeys.enable

Describe the bug
I was using ALSA just fine but recently enabled PulseAudio for bluetooth support and noticed my media keys on my laptop no longer work. I checked the output of nixos-option services.actkbd.bindings to ensure that the key bindings were still set and indeed they are. In fact if I run the amixer commands manually from the cmdline they still work just fine. Somehow PulseAudio is interfering in a way I don't quite understand. I tried to fix myself but cannot pinpoint the issue.
To Reproduce
Steps to reproduce the behavior:

enable sound.mediaKeys.enable
open alsamixer from command line and use your media keys on your keyboard and ensure they work to manipulate the volume
enable hardware.pulseaudio.enable
repeat step 2 and notice that the media keys no longer manipulate the alsamixer volume

Expected behavior
I checked the module declarations for PulseAudio and it does not set or declare actkbd in any way shape or form so it should not interfere with its operation when set by sound.mediaKeys.enable. In other words, the media keys option should work with or without PulseAudio
Screenshots
N/A
Additional context
if I run the commands amixer set Master 1%- unmute manually it works fine even with PulseAudio enabled. This is the same command set by actkbd through the sound.mediaKeys options binded to the media keys. Seeing this I am a little perplexed as to why there is an issue at all.
Metadata

system: "x86_64-linux"
host os: Linux 5.2.9-rt3, NixOS, 19.09.git.a2bd8da (Loris)
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.2.2
channels(): ""
channels(root): ""
nixpkgs: /home/<user>/git/nixpkgs

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
- sound.mediaKeys.enable
- hardware.pulseaudio.enable
# a list of nixos modules affected by the problem
module:
- alsa
- pulseaudio

The problem is that mediaKeys use system-wide daemon to change system-wide volume, and PulseAudio doesn't work like that - it has user instances and changing volume has any sense only in environment with running PulseAudio. I don't think it can be fixed - rather we need to document it.

I believe I have a way to solve this in a reasonably sane way. I'll submit a pull request when I have a working proof of concept, but I've got most of the code written I'm just out of time today. Basically, I'm thinking to add a record to the bindings attribute set to toggle an actkbd user service instead of a system wide service if both sound and pulseaudio are enable. In theory, I can just define a system d user service using the existing system that way the addition to complexity is minimal. Then I can use pactl to change the volume. If just alsa is enabled the regular behavior prevails.
This way the functionality of the service doesn't change. A user doesn't need to enable for desktop environments. But people running window managers can have the proper functionality and volume control will still work properly outside of x11 e.g. running retroarch directly in a tty shell.


Update ext-solarwinds_server to add tag

Small change to add an extra tag:
npm.Environment
Relevant information
Describe what you have done and any details that you think are relevant or that you might want to discuss with us.
Checklist

[ x] I've read the guidelines and understand the acceptance criteria.
[ x] The value of the attribute marked as identifier will be unique and valid.
[ x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

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


960 grid for web development

Use case

Proposal

Just a suggestion, if you could add a feature for displaying 960 grid for web development.
example: https://960.gs/demo.html
Something similar to debug label where it is displayed during development. Also, it would be nice if there is a possibility to align elements on those grid lines (or guide lines) by code.

If we add this, it will not be exclusive to the web.

@HansMuller @goderbauer is this something we want to support out-of-the-box? This could be a pub package too.


Generate 4-5x oversampled PSFs

Left is original PSF, right is PSF after decorrelation.
Mapping is log scale, so it looks worse than it is.
(original notes from @rknop)

This can be done in galsim easily--https://github.com/matroxel/roman_imsim/blob/v2.0/roman_imsim/utils.py#L88
it's just a kwarg in the function we use.

When you say "it looks worse", what's the quantitative thing you're trying to minimize?
I think it's unlikely we will actually know the PSF that much better than a few times smaller than a pixel.  You shouldn't expect that a decorrelated image will look like an image.  When you're operating near the sampling limit, there's just stuff you can't recover.

@thomasvrussell @thomasvrussell
Even 2x would help, we think!

What's the metric for improvement or success?

On the phrosty side, the following branch now accepts additional kwargs for get_imsim_psf that will be passed on to roman_imsim getPSF_Image, which then passes that on to drawImage.
Next is to
[ ] 1. Add a test to verify this behavior
[ ] 2. Add change, configuration to diff-img preprocess.py to pass something like oversampling_factor=4 to get_imsim_psf.
Do we always want it to be something constant?  Should be aa default in diff-img to be 4, should it be specific on the command line?


feat(bindings/cpp): expose writer

Thanks for the PR. But our writers API is not ready for exposing yet. Would you like to skip this part first?

Would you like to skip this part first?

It's OK.


tests/pgwire: randomly force binary transfer

@seut please review

there a still some test which a forcing binary always, is this on purpose?

yes prepareThreshold -1 also causes the client to use server-side prepared statements immediately and the tests which use it only make sense if that is the case.

lgtm


test: set kubeProxyReplacement=probe for upstream k8s tests

Running with kube-proxy-replacement=disabled fails e2e tests for k8s
conformance tests. Therefore we should enable kpr until the underlying
issue is fixed.
Signed-off-by: André Martins<EMAIL_ADDRESS>porting from https://github.com/cilium/cilium/pull/16150

test-upstream-k8s

test-upstream-k8s


ssbench can't benchmark performance of large object

Ssbench works well, but it doesn't when the size of object is larger than 5GB. The reason is that swift has a limit on the size of a single uploaded object. http://docs.openstack.org/developer/swift/overview_large_objects.html

@Fuge2015 As you point out, the largest single, individual  object you can put into Swift is 5GB. However, you can take a 100GB (or any size really) and split it into 5GB (or smaller chunks), using Swift's SLO or DLO functionality. When using SLO or DLO on a larger than 5GB object, to Swift it just looks like many small objects. Thus, if you ran ssbench with lots of objects that emulate the chunk size you've chosen, it will perform operations that mimic exactly what it would look like uploading an object larger than 5GB.


Implement Spanish translation

Once we get the Spanish translations for the interview content, we need to implement a feature to switch languages in the interview.
@JCCChris I've more or less mocked up a yaml structure that I think will work in the UAT playground: https://mycitations.uat.courts.ca.gov/playground. The components are:

a2p-entrypoint.yml
a2p-en.yml
a2p-es.yml

See it on the playground here: https://mycitations.uat.courts.ca.gov/interview?i=docassemble.playground1%3Aa2p-entrypoint.yml#page1
I think the remaining tasks are:

Pull the code blocks out of a2p-en and a2p-es into a code.yml (so that we don't have to change the logic in both files when we need to make changes.)
Insert all the spanish translations into the appropriate places in a2p-es.yml
For messages that appear in code.yml, for example errors like this one, use a lookup table to get strings in the correct language. I.e. we need a function like get_e18n_message('ineligible_citation_error', user_language) that returns the ineligible_citation_error text in the appropriate language.

@bennlich
I've created a translation.yml file and switched interview.yml to use the dictionary lookup.  I think we need to adjust the language selection mechanism still.

Currently working in Spanish on production.


[WIP] AF5 Feature - Redirects Per Request

This PR is a work in-progress, but currently adds the ability to AF5 to control redirect behavior per-request instead of only at the SessionDelegate level.
Problem
The general issue with redirect handling is that you want to allow most endpoints to redirect, and deny several very specific ones. We recently ran into this issue and had to go to some extreme lengths to work around the lack of per-request redirect control in Alamofire. Commonly, the place where you are implementing the SessionDelegate or subclass of it will not know about all the URLs that are going to be piped through it, nor would you want it to. However, the SessionDelegate only has a single callback point for all requests hitting a redirect, so it would be awesome if Alamofire could automatically provide a way to control redirects per-request rather than having to do this through the SessionDelegate.
Solution
Per-request redirect support! By adding a Request.redirect API that allows clients to specify redirect behavior for a specific request, it greatly simplifies the redirect logic. This gives clients full control over all redirects at the Request level, making it VERY easy to disable redirects for a single call without having to subclass the SessionDelegate.
Callouts
@jshier I haven't added tests or anything for this behavior yet b/c I wanted to run the idea by you first to make sure this is going in the right direction. Feedback welcome!

Given that this is an API that the redirect removed due to the removal of the closure API, I think it would be better to offer the same API on SessionManager and Request at the same time. A closure or protocol encapsulation should work, but users should be able to build a single redirect handler for all requests, or one per request using the same API, similar to how the retriers and adapters will eventually work out.

Definitely, that's a great callout. I'll take another pass with that consideration in mind.

i just wanna thank you so much i installed your version of alamofire and my resumedata problems fixed really reallyy thank you man love u

Closing this PR out since it was replaced by #2699 and merged into master.