Zombie extension hosts running at 100%

over the last week I have seen zombie processes running at full CPU load
when trying to debug them (via SIGUSR1) they die
@deepak1556 analysed a crash dump and sees that this is because the EH is already in the process of shutting down
the previous point matches the observation that no window reports its EH as having crashed

Can you take a process sample from the OS activity monitor, I hope the native stack can give a clue to what is happening.

Just happening. This is the command line of such an EH
99701   ??  R    299:41.47 /Applications/Visual Studio Code - Insiders.app/Contents/Frameworks/Code - Insiders Helper (Plugin).app/Contents/MacOS/Code - Insiders Helper (Plugin) --type=utility --utility-sub-type=node.mojom.NodeService --lang=en-GB --service-sandbox-type=none --dns-result-order=ipv4first --inspect-port=0 --user-data-dir=/Users/jrieken/Library/Application Support/Code - Insiders --standard-schemes=vscode-webview,vscode-file --enable-sandbox --secure-schemes=vscode-webview,vscode-file --bypasscsp-schemes --cors-schemes=vscode-webview,vscode-file --fetch-schemes=vscode-webview,vscode-file --service-worker-schemes=vscode-webview --streaming-schemes --shared-files --field-trial-handle=1718379636,r,8003835085444679389,18150943907601750289,262144 --disable-features=CalculateNativeWinOcclusion,SpareRendererForSitePerProcess
99704   ??  S      0:04.19 /Applications/Visual Studio Code - Insiders.app/Contents/Frameworks/Code - Insiders Helper (Plugin).app/Contents/MacOS/Code - Insiders Helper (Plugin) --ms-enable-electron-run-as-node /Users/jrieken/.vscode-insiders/extensions/kisstkondoros.vscode-codemetrics-1.26.1/dist/server.js --node-ipc --clientProcessId=99701
99706   ??  S      0:03.39 /Applications/Visual Studio Code - Insiders.app/Contents/Frameworks/Code - Insiders Helper (Plugin).app/Contents/MacOS/Code - Insiders Helper (Plugin) --ms-enable-electron-run-as-node /Users/jrieken/.vscode-insiders/extensions/kisstkondoros.vscode-gutter-preview-0.30.0/dist/server.js --node-ipc --clientProcessId=99701
99750   ??  S      0:01.17 /Applications/Visual Studio Code - Insiders.app/Contents/Frameworks/Code - Insiders Helper (Plugin).app/Contents/MacOS/Code - Insiders Helper (Plugin) --ms-enable-electron-run-as-node /Users/jrieken/.vscode-insiders/extensions/ms-vscode.anycode-0.0.72/dist/anycode.server.node.js --node-ipc --clientProcessId=99701
16791 s034  S+     0:00.00 grep 99701

This the process sample
Uploading EH-spin-sample.txt…

@connor4312 Not sure about the --inspect-port=0 part. Does this mean we run the EH under development or not?

That means that EH is being debugged, though you'll need to find the port it's listening on if you want to attach to it. Then you could grab a performance profile using the edge/chrome devtools on edge://inspect, or by attaching to that using an attach-type launch config VS Code and using the "take performance profile" command.

It looks like we set that for all local EH process launches. Just having inspect-port doesn't put the process in debug mode.
What you can do is use the "Debug: Attach to Node.js Process" command and select that process. This will fail, since the debugger won't then know what port it's listening on (if --inspect-port isn't given in the args, it always listens on 9229, and js-debug assumes that is what happens). But, then you can find it. AFter doing so you could grab a performance profile using the edge/chrome devtools on edge://inspect, or by attaching to that using an attach-type launch config VS Code and using the "take performance profile" command -- if the spin sample you recorded above isn't enough for Deepak to figure out what's going on.

What you can do is use the "Debug: Attach to Node.js Process" command and select that process.

Does that use SIGUSR1? Asking cos I have tried that first and sending the signal kills the process. From the crash dumps we do know that the process already was shutting down

Attaching the symbolicated sample EH-spin-sample-symbolicated.txt
Going from the top, electron::NodeService::~NodeService() indicates the EH process is already in the shutdown phase and  has started to destroy the Node.js environment node::FreeEnvironment(node::Environment*) . At this point, connecting via a debugger is not possible since the agent host will stop receiving connections due to the following check, so we cannot get perf profile from these processes.
Moving further down most of the sample time is spent in node::NodePlatform::DrainTasks which happens before the environment gets deleted, this flushes the pending tasks in the libuv event loop but one thing to note is that entering JS via these tasks is not allowed due to the scope. But there are GC tasks being executed v8::internal::FinalizationRegistryCleanupTask::RunInternal and v8::internal::Heap::CollectGarbage which can enter JS. I think the error calls node::errors::TriggerUncaughtException are from these callsites eventually leading to node::OnFatalError. At this point, I expect the process to abort and the main process should be able to reap the utility process but that is not happening.
There is another data point looking back at the screenshot from https://github.com/microsoft/vscode/issues/194477#issue-1919108667, the zombie EH processes are titled utility-process and not extensionHost in the process explorer. We resolve the pids to respective titles based on a map maintained in the main process, the process explorer gets it periodically via the following ipc and the utility process related values are populated from here. We default to utility-process when the pid map no longer has an entry for the process which can only happen when onDidExitOrCrashOrKill got invoked which inturn suggests that the main process did receive an exit or crash signal from these EH hosts. @jrieken can you check the main process logs next time if it has any of the following entries for the corresponding zombie EH pid to validate this data

https://github.com/microsoft/vscode/blob/e6fb75347302315664c2bb8e00feb0a75c84a3b7/src/vs/platform/utilityProcess/electron-main/utilityProcess.ts#L310
https://github.com/microsoft/vscode/blob/e6fb75347302315664c2bb8e00feb0a75c84a3b7/src/vs/platform/utilityProcess/electron-main/utilityProcess.ts#L322
https://github.com/microsoft/vscode/blob/e6fb75347302315664c2bb8e00feb0a75c84a3b7/src/vs/platform/utilityProcess/electron-main/utilityProcess.ts#L408

I attempted to setup a similar flow based on the stack with base Electron app but the utility process are cleaned up as expected. I can make a change in the destruction phase of the utility process to signal termination of isolate so that tasks which require to enter JS will not do so and I hope that resolves the long running draintask.
I won't be able to make a new insiders build with this change until I am back from vacation but I can provide a custom electron build for local testing with OSS. @jrieken would that help to validate incase you face similar issue with OSS as well ?

Does that use SIGUSR1? Asking cos I have tried that first and sending the signal kills the process. From the crash dumps we do know that the process already was shutting down

Ah, my bad, skimmed over that part. Disregard.
(Yes, it sends SIGUSR1 on unix)

2023-10-06 10:26:53.084 [info] Extension host with pid 19510 exited with code: 0, signal: unknown.

I am seeing this for PID-19510 which is currently spinning forever

i think i have the same issue

Having the exact same issue. utility-process is running at 100% all the time on M1 Max, very annoying.

output.txt
Manually sampling with eu-stack running every second, for one minute.

Sorry if I'm hijacking the thread but would uninstalling Node and installing Bun solve this problem? Doesn't sound like it's a VSCode bug, right?

Thanks for the logs @taoky , the stack does look related. I wouldn't be surprised if the issue is also present on Linux since the shutdown path is same for both.

Checked log and it looks weird.
/home/username/.config/Code/logs/20231022T205712/main.log:
2023-10-24 03:44:26.258 [info] Extension host with pid 1331811 exited with code: 0, signal: unknown.

However, PID 1331811 still exists and is running at 100%.

Checked log and it looks weird.

Another hang process today with log:
> rg 1706525
20231022T205712/window9/exthost/exthost.log
1:2023-10-24 15:40:45.945 [info] Extension host with pid 1706525 started
326:2023-10-24 16:00:07.408 [info] Extension host with pid 1706525 exiting with code 0

20231022T205712/main.log
17:2023-10-24 16:00:07.409 [info] Extension host with pid 1706525 exited with code: 0, signal: unknown.

"Extension host with pid 1706525 started" => https://github.com/microsoft/vscode/blob/a8b7239d0311d4915b57c837972baf4b01394491/src/vs/workbench/api/common/extensionHostMain.ts#L171
"Extension host with pid 1706525 exiting with code 0" => https://github.com/microsoft/vscode/blob/a8b7239d0311d4915b57c837972baf4b01394491/src/vs/workbench/api/common/extHostExtensionService.ts#L271
"Extension host with pid 1706525 exited with code: 0, signal: unknown." => https://github.com/microsoft/vscode/blob/a8b7239d0311d4915b57c837972baf4b01394491/src/vs/platform/extensions/electron-main/extensionHostStarter.ts#L76

Is there any temp workaround? any script or bash command that we can run to identify and kill those pid? I am interested to do it manually, because i have a M2 and i am rebooting my macbook after some hours because get soooo slow

@NullEnt1ty @fenvalle @taoky @Maktel @ClockRide @nzhl
Please help by posting the list of installed extensions. I am looking for similarities between our setups as this is possibly caused by an extension.
This is my list
amodio.tsl-problem-matcher
chrisdias.vscode-opennewinstance
connor4312.css-theme-completions
connor4312.esbuild-problem-matchers
dbaeumer.vscode-eslint
dtsvet.vscode-wasm
eamodio.gitlens
esbenp.prettier-vscode
GitHub.codespaces
GitHub.copilot
GitHub.copilot-chat
GitHub.github-vscode-theme
GitHub.remotehub
GitHub.vscode-pull-request-github
ipatalas.vscode-postfix-ts
jrieken.browser-links
jrieken.esm-helper
jrieken.references-plusplus
jrieken.regex-notebook
jrieken.vscode-pr-pinger
jrieken.vscode-tree-sitter-query
kisstkondoros.vscode-codemetrics
kisstkondoros.vscode-gutter-preview
meganrogge.template-string-converter
ms-python.python
ms-python.vscode-pylance
ms-vscode-remote.remote-containers
ms-vscode-remote.remote-ssh
ms-vscode-remote.remote-ssh-edit
ms-vscode.anycode
ms-vscode.anycode-c-sharp
ms-vscode.anycode-cpp
ms-vscode.anycode-go
ms-vscode.anycode-java
ms-vscode.anycode-kotlin
ms-vscode.anycode-php
ms-vscode.anycode-python
ms-vscode.anycode-rust
ms-vscode.anycode-typescript
ms-vscode.hexeditor
ms-vscode.remote-explorer
ms-vscode.remote-repositories
ms-vscode.remote-server
ms-vscode.vscode-github-issue-notebooks
ms-vscode.vscode-markdown-notebook
ms-vscode.vscode-selfhost-test-provider
ms-vscode.vscode-speech
ms-vscode.vscode-typescript-next
rust-lang.rust-analyzer
wayou.vscode-todo-highlight

To get this list run code --list-extensions or code-insiders --list-extensions. Please do the same and post your results here. Thanks

> code --list-extensions
aaronduino.chisel
bbenoist.Nix
coolbear.systemd-unit-file
cschlosser.doxdocgen
eamodio.gitlens
esbenp.prettier-vscode
GitHub.copilot
GitHub.copilot-chat
GitHub.copilot-labs
golang.go
haskell.haskell
James-Yu.latex-workshop
jeff-hykin.better-cpp-syntax
justusadam.language-haskell
kokororin.vscode-phpfmt
llvm-vs-code-extensions.vscode-clangd
llvm-vs-code-extensions.vscode-mlir
luniclynx.bison
luniclynx.lex
ms-azuretools.vscode-docker
ms-python.isort
ms-python.python
ms-python.vscode-pylance
ms-toolsai.jupyter
ms-toolsai.jupyter-keymap
ms-toolsai.jupyter-renderers
ms-toolsai.vscode-jupyter-cell-tags
ms-toolsai.vscode-jupyter-slideshow
ms-vscode-remote.remote-ssh
ms-vscode-remote.remote-ssh-edit
ms-vscode.cmake-tools
ms-vscode.cpptools
ms-vscode.cpptools-extension-pack
ms-vscode.cpptools-themes
ms-vscode.hexeditor
ms-vscode.live-server
ms-vscode.remote-explorer
ms-vsliveshare.vsliveshare
ms-vsliveshare.vsliveshare-pack
nvarner.typst-lsp
prince781.vala
rust-lang.rust-analyzer
sswg.swift-lang
twxs.cmake
vadimcn.vscode-lldb
Vue.volar
wholroyd.jinja

1Password.op-vscode
bierner.github-markdown-preview
bierner.markdown-checkbox
bierner.markdown-emoji
bierner.markdown-footnotes
bierner.markdown-mermaid
bierner.markdown-yaml-preamble
bradlc.vscode-tailwindcss
Brigit.devcontainer-image-convert
charliermarsh.ruff
christian-kohler.npm-intellisense
DavidAnson.vscode-markdownlint
dbaeumer.vscode-eslint
denoland.vscode-deno
esbenp.prettier-vscode
GitHub.codespaces
GitHub.copilot
GitHub.copilot-chat
GitHub.remotehub
github.vscode-github-actions
GitHub.vscode-pull-request-github
GrapeCity.gc-excelviewer
hashicorp.terraform
hediet.vscode-drawio
jakeboone02.cypher-query-language
mikestead.dotenv
ms-azuretools.vscode-docker
ms-dotnettools.vscode-dotnet-runtime
ms-kubernetes-tools.vscode-kubernetes-tools
ms-python.autopep8
ms-python.isort
ms-python.python
ms-python.vscode-pylance
ms-toolsai.jupyter
ms-toolsai.jupyter-keymap
ms-toolsai.jupyter-renderers
ms-toolsai.vscode-jupyter-cell-tags
ms-toolsai.vscode-jupyter-slideshow
ms-vscode-remote.remote-containers
ms-vscode-remote.remote-ssh
ms-vscode-remote.remote-ssh-edit
ms-vscode-remote.vscode-remote-extensionpack
ms-vscode.azure-repos
ms-vscode.cmake-tools
ms-vscode.cpptools
ms-vscode.cpptools-extension-pack
ms-vscode.makefile-tools
ms-vscode.remote-explorer
ms-vscode.remote-repositories
ms-vscode.remote-server
ms-vscode.test-adapter-converter
ms-vscode.vscode-markdown-notebook
ms-vscode.wordcount
ms-vsliveshare.vsliveshare
mtxr.sqltools
mtxr.sqltools-driver-pg
Oracle.oracledevtools
ParallelsDesktop.parallels-desktop
pomdtr.excalidraw-editor
redhat.ansible
redhat.vscode-yaml
ritwickdey.LiveServer
snyk-security.snyk-vulnerability-scanner
Tailscale.vscode-tailscale
timonwong.shellcheck
tomoki1207.pdf
twxs.cmake
Tyriar.sort-lines
VisualStudioExptTeam.intellicode-api-usage-examples
VisualStudioExptTeam.vscodeintellicode
VisualStudioExptTeam.vscodeintellicode-completions
vscode-icons-team.vscode-icons
waderyan.gitblame

Thanks everyone. Looks like the intersection is
esbenp.prettier-vscode
GitHub.copilot
GitHub.copilot-chat

@jrieken

Thanks everyone. Looks like the intersection is
esbenp.prettier-vscode
GitHub.copilot
GitHub.copilot-chat

Yep, I'm using those extensions too. Here's the full list:
Armitxes.sqf
astro-build.astro-vscode
bungcip.better-toml
CodeQue.codeque
darkriszty.markdown-table-prettify
dbaeumer.vscode-eslint
DotJoshJohnson.xml
eamodio.gitlens
EditorConfig.EditorConfig
emmanuelbeziat.vscode-great-icons
esbenp.prettier-vscode
GitHub.copilot
GitHub.remotehub
goz3rr.vscode-glualint
KatjanaKosic.vscode-json5
MS-CEINTL.vscode-language-pack-de
ms-python.isort
ms-python.python
ms-python.vscode-pylance
ms-vscode-remote.remote-ssh
ms-vscode-remote.remote-ssh-edit
ms-vscode-remote.remote-wsl
ms-vscode.azure-repos
ms-vscode.hexeditor
ms-vscode.remote-explorer
ms-vscode.remote-repositories
ms-vsliveshare.vsliveshare
nhoizey.gremlins
nidu.copy-json-path
platformio.platformio-ide
redhat.vscode-commons
redhat.vscode-yaml
ryu1kn.partial-diff
stkb.rewrap
timonwong.shellcheck
venner.vscode-glua-enhanced
VisualStudioExptTeam.intellicode-api-usage-examples
VisualStudioExptTeam.vscodeintellicode
wmaurer.change-case
zhuangtongfa.material-theme

https://github.com/microsoft/vscode/pull/196809 is not a very good workaround: what if the process exits, but another innocent process just takes this PID?

#196809 is not a very good workaround: what if the process exits, but another innocent process just takes this PID?

It probably cannot do that as the pid is still taken? I agree it's not a great workaround, but until the root cause is clear it does solve the issue where a cpu is occupied.


conan: removing conanfiles not maintained here anymore

Removing conanfiles.py that are now maintained in the includeos/conan repository.
Removed:

binutils : conan/gnu/binutils
musl: conan/musl
llvm: conan/llvm

libunwind
libcxxabi
libcxx

Code coverage: http://vaskemaskin.includeos.org:8080/coverage_includeos/hioa-cs-org-test/IncludeOS/PR-2066

Code coverage: http://vaskemaskin.includeos.org:8080/coverage_includeos/hioa-cs-org-test/IncludeOS/PR-2066


Not working Mutation Authorization

Hello. I'am use GraphQLAuthorize for MutationFunction and authorization doesn't work. ( See Screen)

I decided to download your project and debug it. After debugging, I came to the conclusion that authorization does not work for mutations. Maybe I'm wrong, if that's the case, tell me what could be the problem. I also made edits in the local git, maybe this will help (screen below)

Thanks. I have just pushed version 1.0.3 to nuget.
Note the check you removed is still there. It checks that the user has access to the Type returned by the mutation (or field) as you can secure types as well.


Repositories: by default disable all units except code and pulls on forks

Most of the time forks are used for contributing code only, so not having
issues, projects, release and packages is a better default for such cases.
They can still be enabled in the settings.
A new option DEFAULT_FORK_REPO_UNITS is added to configure the default
units on forks.
Also add missing repo.packages unit to documentation.
code by: @brechtvl

I added a DEFAULT_FORK_REPO_UNITS setting now. That seemed easier to reason about to me than FORK_DISABLED_UNITS. Either way would break compatibility with existing configs in some sense, but hopefully not in a way that's really disruptive to anyone.
I'm not the owner of this PR and so can't change the description or mark it as ready for review, but I think both can be done now.

Thanks @brechtvl , I've updated description and marked this as ready for review from other maintainers:)

I can't edit the description of this PR, but suggest:

When forking a repository, the fork will now have issues, projects, releases, packages and wiki disabled. These can be enabled in the repository settings afterwards. To change back to the previous default behavior, configure DEFAULT_FORK_REPO_UNITS to be the same value as DEFAULT_REPO_UNITS.

LG-TM bot


Flow support

Example flow file
/* @flow */
function foo(x: string, y: number): string {
  return x.length * y;
}
foo('Hello', 42);

http://flowtype.org

we use espree to parse the files, so we can't really add support for flow unless we replace the parser..
marking it as wontfix for now, but maybe in he future we can reopen if more people asks for it and/or we change the way the formatting works.

This would be great.  Thanks!

+1

reopening since I'm planning to replace the parser to use babylon (#398) which would allow us to implement this as a plugin.

BTW, I started the plugin development 2h hours ago and got a good chunk of it done already. Should publish it later today or over the weekend. But it will only work if esformatter is using babylon + accorn-to-esprima.

Flow plugin implemented with basic features (https://github.com/millermedeiros/esformatter-flow), now just need to release<EMAIL_ADDRESS>and publish both to npm.


Write a Swift program to find sum of digits of a number

Description
Write a Swift program to find sum of digits of a number
Suppose you have a number n and you want to find the sum of its digits. For example, if n = 123, then the sum of its digits is 1 + 2 + 3 = 6.
Input  : 123
Output : 6

How to contribute

Save the solution in FindSumOfDigitsOfANumber.swift file
Add FindSumOfDigitsOfANumber.swift file in program/find-sum-of-digits-of-a-number folder

!assign

Thank you so much for the approval!


Are there plans to support tts?

Great job, is there any plan to support tts?

Yes, I'm planning to code for TTS after I supported currently unsupported ASR models, such as RNNDecoder.


panic: runtime error: index out of range [3] with length 3

Using the version installed today using brew on a MAC M1 (seems that brew version is not the latest)
kubescape scan .
[info] ARMO security scanner starting
[warning] current version 'v2.0.166' is not updated to the latest release: 'v2.0.170'
panic: runtime error: index out of range [3] with length 3

goroutine 1 [running]:
github.com/armosec/go-git-url/gitlabparser/v1.(*GitLabURL).Parse(0x140003fc1c0, {0x1400005bc20?, 0x1046a5ce0?})
        github.com/armosec/go-git-url@v0.0.15/gitlabparser/v1/parser.go:89 +0x33c
github.com/armosec/go-git-url/gitlabparser/v1.NewGitLabParserWithURL({0x1400005bc20, 0x3e})
        github.com/armosec/go-git-url@v0.0.15/gitlabparser/v1/parser.go:28 +0x98
github.com/armosec/go-git-url.NewGitURL({0x1400005bc20, 0x3e})
        github.com/armosec/go-git-url@v0.0.15/init.go:28 +0x1b0
github.com/armosec/kubescape/v2/core/cautils.metadataGitLocal({0x16d837657?, 0x3?})
        github.com/armosec/kubescape/v2/core/cautils/scaninfo.go:405 +0xe8
github.com/armosec/kubescape/v2/core/cautils.setContextMetadata(0x14000c3d3f0, {0x16d837657, 0x3})
        github.com/armosec/kubescape/v2/core/cautils/scaninfo.go:358 +0x364
github.com/armosec/kubescape/v2/core/cautils.scanInfoToScanMetadata(0x140002fc4e0)
        github.com/armosec/kubescape/v2/core/cautils/scaninfo.go:289 +0x328
github.com/armosec/kubescape/v2/core/cautils.NewOPASessionObj({0x0, 0x0, 0x0}, 0x0, 0x140002fc4e0)
        github.com/armosec/kubescape/v2/core/cautils/datastructures.go:43 +0x5c
github.com/armosec/kubescape/v2/core/pkg/policyhandler.(*PolicyHandler).CollectResources(0x14000b9b808, {0x14000611500, 0x5, 0x8}, 0x140002fc4e0)
        github.com/armosec/kubescape/v2/core/pkg/policyhandler/handlenotification.go:26 +0x40
github.com/armosec/kubescape/v2/core/core.(*Kubescape).Scan(0x103f05b60?, 0x140002fc4e0)
        github.com/armosec/kubescape/v2/core/core/scan.go:142 +0x618
github.com/armosec/kubescape/v2/cmd/scan.getFrameworkCmd.func2(0x1045bd120?, {0x14000427680, 0x2, 0x140006b1560?})
        github.com/armosec/kubescape/v2/cmd/scan/framework.go:102 +0x3ac
github.com/armosec/kubescape/v2/cmd/scan.GetScanCommand.func1(0x140000ff680?, {0x140006b1560, 0x1, 0x1?})
        github.com/armosec/kubescape/v2/cmd/scan/scan.go:45 +0x180
github.com/spf13/cobra.(*Command).ValidateArgs(...)
        github.com/spf13/cobra@v1.5.0/command.go:1018
github.com/spf13/cobra.(*Command).execute(0x140000ff680?, {0x140006b1540?, 0x1?, 0x1?})
        github.com/spf13/cobra@v1.5.0/command.go:841 +0x3a4
github.com/spf13/cobra.(*Command).ExecuteC(0x140000ff400)
        github.com/spf13/cobra@v1.5.0/command.go:990 +0x354
github.com/spf13/cobra.(*Command).Execute(...)
        github.com/spf13/cobra@v1.5.0/command.go:918
github.com/armosec/kubescape/v2/cmd.Execute()
        github.com/armosec/kubescape/v2/cmd/root.go:84 +0x34
main.main()
        github.com/armosec/kubescape/v2/main.go:9 +0x1c

This also happens with the latest v2.0.170 release as well.

origin	git@gitlab.com:foobar/machine-learning/cluster-manifests.git (fetch)
origin	git@gitlab.com:foobar/machine-learning/cluster-manifests.git (push)

This bug occurs on GitLab with versions v2.0.165-172. Version v2.0.164 seems to work, so until patch is ready will resort to use that.

@dazzag24 Have you done here any progress?

@dazzag24 Have you done here any progress?

I think Aman123lug volunteered to work on this.

@Aman123lug Any updates?

@dwertent hello. I've looked at it briefly and the offending line is: https://github.com/kubescape/go-git-url/blame/master/gitlabparser/v1/parser.go#L86
Wrapping it with a check on the "-" particular path part is enough to avoid a panic:
	if splittedRepo[index] == "-" {
		index += 1 // skip "-" symbol in URL
	}

However, I am not really sure about the consistency of the eventual result. This parser obviously is not designed in the first place to support general URLs.

@Aman123lug Any updates?

I think @dazzag24 working on it

Also I've detected that there are 2 versions of this go-git-url repo being used: one under the kubescape owner, one under the armosec owner: while the armosec version is still being pulled as a direct dependency, the other version is pulled indirectly...

I have a small patch ready if you guys are interested. This assumes a few things. I'd need a piece of advice to be sure I am heading in the right direction.  I wouldn't want to interfere with some other people's work. Let me know if you want a PR (actually 2 since 2 repos are concerned).

kubescape/go-git-url: added the simple check above. In the case of such "scp-like" git-urls, it no longer panics if the input is not exactly as expected. However, the OP-provided remote ```````````git@gitlab.com:foobar/machine-learning/cluster-manifests.git`  won't really work as expected: in this example, "foobar" is considered the owner, "machine learning" the repo and "cluster..." the branch.
With a correct origin like so  "git@gitlab.com:gitlab-tests/sample-project.git", the owner & repo are inferred correctly.
I am not sure that this is acceptable behavior.
Parsed "pseudo-URL" results in :

=== RUN   TestFred
    fred_test.go:26: remote: git@gitlab.com:foobar/gitlab-tests/sample-project.git
(*v1.GitLabURL)(0xc00030a380)({
 host: (string) (len=10) "gitlab.com",
 owner: (string) (len=6) "foobar",
 repo: (string) (len=12) "gitlab-tests",
 project: (string) "",
 branch: (string) (len=18) "sample-project.git",
 path: (string) "",
 token: (string) ""
})
--- PASS: TestFred (0.00s)
PASS

kubescape/kubescape: replaced deps on github.com/armosec/go-git-url by github.com/kubescape/go-git-url (both repos are the same right now).  I assume this is the right repo to look into. Will need to upgrade dep on go-git-url.

@dwertent  here is a proposal for a fix. @dazzag24, @Aman123lug  feel free to discard this if you've already started something better. For sure, the fix is not perfect: just assuring that wrong/unexpected input doesn't panic the CLI.


Legend items are not hidden when plottables are hidden

Issue: Legend showing all added plottables, hidden or not
ScottPlot Version: 5.0.23
Code Sample: Default code from cookbook, styled a little
ScottPlot 5.0.21

And after update to 5.0.23

I think this issue is related
https://github.com/ScottPlot/ScottPlot/issues/3541


feat: add visionOS support

Summary
Add experimental visionOS support
TODO:

[x] Update podspec
[x] Add visionOS example (since the template visionOS will change yet, it makes no sense to add example now)
[x] Add visionOS support for promises - https://github.com/google/promises/pull/224
[x] Migrate/Fix/Disable deprecated/unavailable functions that are no available in visionOS

AVPlayer:

[x] AVPlayer.allowsExternalPlayback - won't work
[x] AVPlayer.isExternalPlaybackActive - won't work
[x] AVPlayer.preventsDisplaySleepDuringVideoPlayback - won't work

AVPlayerItem: - #3449

[x] AVPlayerItem.timedMetadata

UIApplication && UIScreen: - #3448

[x] UIApplication.shared.statusBarOrientation
[x] UIApplication.shared.keyWindow
[x] UIScreen.main.bounds

AVAsset: - #3444

[x] AVAsset.tracks(withMediaType:)
[x] AVAsset.mediaSelectionGroup(forMediaCharacteristic:)

Other:

[x] AVVideoComposition.init(asset:applyingCIFiltersWithHandler:) - #3444
[x] PlayerViewController.isReadyForDisplay - won't work
[x] AVAssetResourceLoadingRequest.streamingContentKeyRequestData - skipped, will be added in future

Build are failing because of React Native issue


Automatically delete the overlay if a hotseat user gets banned

It also deletes the overlay if the user gets timed out or has a message deleted
Would close #17

I like the switch to const and let

I like the switch to const and let

They were added in a commit elsewhere. I'm not sure why they're being displayed as if I've made the changes

There I was giving you credit.
I’m on the mobile app so must have missed the fact they were already there. You must have changed the formatting slightly.

There I was giving you credit.
I’m on the mobile app so must have missed the fact they were already there. You must have changed the formatting slightly.

I’ll double check my editor maybe it’s enforcing my usual indent rules or something. Looks like these changes were actually made in #15 :)

To note this was submitted before #17 was closed, I'm not being super snarky with this PR

I don't know if this is different from the one I previously closed, but I just read the title and that's that. I won't be updating it so that it automatically deletes if the person is banned, I think a mod should just do that.


docs(tech-insights-backend): fix CHANGELOG.md typo

Hey, I just made a Pull Request!
This fixes a simple typo in the tech-insights-backend CHANGELOG.
:heavy_check_mark: Checklist

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

@colinrymer this needs a changeset in order for the Readme to be updated on npm.

Ah, figure it's alright to ship this without changeset because github is the place you see changelogs anyway

Ah, figure it's alright to ship this without changeset because github is the place you see changelogs anyway

Oh yes I confused it with the Readme. My bad :)


Jami (QT): GLib-GIO-ERROR - No GSettings schemas

Describe the bug
The Jami client crashes with this error:
GLib-GIO-ERROR : No GSettings schemas are installed on the system
Steps To Reproduce
Steps to reproduce the behavior:

Install Jami via nix-env -i jami
Run Jami: jami
Jami crashes with error.

Expected behavior
The Jami client should load as expected.
Additional context
Found references to how to make this work, unsure which if any are workable approaches:

https://discourse.nixos.org/t/failure-appimage-run-with-gsetting/4403/2
https://github.com/NixOS/nixpkgs/issues/181500

Error log:
Using Qt runtime version: 6.5.2
"notify server name: lxqt-notificationd, vendor: lxqt.org, version: 1.3.0, spec: 1.2"
"Using locale: en_US"
No accounts to migrate
Screen saver dbus interface:  "org.freedesktop.ScreenSaver"
Can't update current account info data for ""
Main window loaded using OpenGLRhi
NetworkManager client initialized, version:  1.42.8 , daemon running: yes , networking enabled: yes
primary network connection: <id truncated> default:  yes

(jami:5027): GLib-GIO-ERROR **: 10:31:21.371: No GSettings schemas are installed on the system
Trace/breakpoint trap (core dumped)

Notify maintainers
@linsui
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[jakimfett@5trix:~/]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.56, NixOS, 23.05 (Stoat), 23.05.4172.bd1cde45c778`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.5`
 - channels(root): `"nixos-23.05"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Same error on KDE Plasma:
Using Qt runtime version: 6.5.2
"notify server name: Plasma, vendor: KDE, version: 5.27.7, spec: 1.2"
"Using locale: en_US"
No accounts to migrate
Screen saver dbus interface:  "org.freedesktop.ScreenSaver"
Can't update current account info data for ""
Main window loaded using OpenGLRhi
qt.dbus.integration: QDBusConnection: name 'cx.ring.Ring' had owner '' but we thought it was ':1.74'
NetworkManager client initialized, version:   , daemon running: no , networking enabled: no
no primary network connection detected, check network settings

(jami:15941): GLib-GIO-ERROR **: 11:05:54.329: No GSettings schemas are installed on the system
Trace/breakpoint trap (core dumped)

[jakimfett@obsidian:~/hub/dev/mintotp]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.59, NixOS, 23.05 (Stoat), 23.05.4448.5550a85a087c`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.6`
 - channels(root): `"nixos-23.05.4448.5550a85a087c"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I can't reproduce the problem on GNOME. Not sure how I can test it.

Maybe a virtual machine? LXQT is one of the installer options so setting up a VM should be straightforward.

Maybe a virtual machine? LXQT is one of the installer options so setting up a VM should be straightforward.

I can't reproduce the problem on GNOME. Not sure how I can test it.

env -i HOME="$HOME" DISPLAY="$DISPLAY" WAYLAND_DISPLAY="$WAYLAND_DISPLAY" XDG_RUNTIME_DIR="$XDG_RUNTIME_DIR" QT_QPA_PLATFORM=xcb ./result/bin/jami  to get rid of gtk impurities in the env
@jakimfett you can fix by wrapping with the gtk wrapper, see the qt app example https://ryantm.github.io/nixpkgs/languages-frameworks/gnome/#ssec-gnome-common-issues-double-wrapped

@jakimfett will you submit a PR?


Improved ci

Setup coverage

Looks great! :+1:

Looks great! :+1:

Aside from merging this, is there anything the project maintainers have to do to set up coveralls/david-dm?

just log with their github account, and activate the repo :)
nothing more

Awesome! @lambtron, can you look into this?
Also, I think the badge links may actually need to be updated, since we recently moved this repo from segmentio/metalsmith to metalsmith/metalsmith.

Indeed. taking care of this now =)

Hmm, Travis seems to be unaware that the repo moved. The updated URL for that badge broke it. :confused:

Ah i'll reupdate the info in travis. Thanks for reminding me!

@lambtron Great, looks like the update worked. Now we just need Coveralls access enabled (https://github.com/organizations/metalsmith/settings/oauth_application_policy) and we can merge this.

Done!

Accidentally closed it. Feel free to merge!

Looks good.


[Extensibility Enhancement] Add Check for Max Field Length - Translation Module

I'm using the translation module and I would like to know what do you think about a check for the Translation table that returns an error if the translation text exceeds the Max. Length of the table field?
https://github.com/microsoft/ALAppExtensions/blob/5c490ab80d8f5c4ac26641604eba79dbe1b1eb47/Modules/System/Translation/src/Translation.Table.al#L33
I could create a PR, which throws an error if you agree that this is the expected behaviour.

@pri-kise interesting suggestion. Can you elaborate on that?
What are the scenarios you want to cover? What could be a useful reaction to such an error?

In my opinion the common use case for the translation module is to enable the user to store translations for specific fields that are used as an alternative for Language Specific Documents / Records.
If i look into the baseapp that current has its own translation tables (Item Translation, Payment Method Translation), they are commonly used to transfer the translations as replacement of the default value (Default Language).
Therefore I would like to throw the error early in the process - while storing the translations - instead of resulting in a shortened translated text during runtime.

I see your point! However, I'm not sure if an error maybe isn't too harsh? What if you'd like to store the proper translation, even if it doesn't fit into the field? Would that never be a scenario? Also, what about the system, where users already have stored translations which exceed the field length? How would we handle those once your rule has been implemented? Would we just truncate them?
Maybe this validation could be an optional thing you configure? Maybe it could just be a warning, which informs you that you've added a string which exceeds the field length?
I do see a valid use case here and we would love to see your PR, but you should rethink if an hard stop error is the right thing to implement :-)
Closing this issue and waiting eagerly to see your solution as a PR. Have a lovely weekend!


Exception from Format Document

Did Format Document from Command Palette:

[Error - 3:00:00 p.m.] Please report this issue to https://github.com/microsoft/al/issues including information on how to reproduce it, if possible.
Processing of message 'textDocument/formatting' failed with error: 'One or more errors occurred.'
Details:
System.AggregateException: One or more errors occurred. ---> System.Threading.Tasks.TaskCanceledException: A task was canceled.
--- End of inner exception stack trace ---
at System.Threading.Tasks.Task.WaitAll(Task[] tasks, Int32 millisecondsTimeout, CancellationToken cancellationToken)
at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Formatting.AbstractFormatEngine.BuildContext(FormattingContext context, TokenStream tokenStream, NodeOperations nodeOperations, CancellationToken cancellationToken) in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Formatting\Engine\AbstractFormatEngine.cs:line 467
at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Formatting.AbstractFormatEngine.d__12.MoveNext() in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Formatting\Engine\AbstractFormatEngine.cs:line 100
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.LanguageServices.SyntaxFormattingService.d__11.MoveNext() in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\LanguageServices\SyntaxFormattingService.cs:line 147
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.LanguageServices.SyntaxFormattingService.d__8.MoveNext() in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\LanguageServices\SyntaxFormattingService.cs:line 91
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Formatting.Formatter.d__24.MoveNext() in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Formatting\Formatter.cs:line 293
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Formatting.Formatter.d__16.MoveNext() in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Formatting\Formatter.cs:line 225
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.EditorServices.Protocol.Utility.FormattingHelper.d__0.MoveNext() in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\Utility\FormattingHelper.cs:line 23
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.EditorServices.Protocol.LanguageServer.FormatDocumentRequestHandler.d__3.MoveNext() in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\LanguageServer\FormatDocumentRequestHandler.cs:line 35
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.EditorServices.Protocol.MessageProtocol.RequestHandlerBase`1.d__9.MoveNext() in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\MessageProtocol\RequestHandlerBase.cs:line 43
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Dynamics.Nav.EditorServices.Protocol.RequestRegistry.d__6.MoveNext() in F:\a\760\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\Endpoints\RequestRegistry.cs:line 64
---> (Inner Exception #0) System.Threading.Tasks.TaskCanceledException: A task was canceled.<---

This code gives the error:
trigger OnAfterGetRecord()
var
Job: Record Job;
begin
DescriptionIndent := Indentation;
StyleIsStrong := "Job Task Type" <> "Job Task Type"::Posting;
InPlanningMode := false;
CalcFields("Bid Count");
case "Job Task Type" of
"Job Task Type"::Posting:
begin
if "Bid Count" = 0 then
DescriptionStyle := 'Attention'
else
DescriptionStyle := 'Standard';
end;
    if Job.GET(Rec."Job No.") then begin
        if Job."Alair Status" < Job."Alair Status"::Construction then
            InPlanningMode := true;
    end;
end;

Adding the missing end; (to close the case of statement allows correct formatting.

Thanks for the great repro! We'll take this as a bug.

The fix for this issue has been checked in to the master branch. It will be available in the bcinsider.azurecr.io/bcsandbox-master Docker image starting from platform build number 38380 and VS Code Extension Version 5.0.203252.
If you don’t have access to these images you need to become part of the Ready2Go program: aka.ms/readytogo
For more details on code branches and docker images please read:
https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/
https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/


chore(main): release 1.7.1

:robot: I have created a release beep boop
1.7.1 (2023-05-29)
Bug Fixes

nulls: Minor fixes to address null values (#52) (ec5c9be)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/andrewthetechie/gha-repo-manager/releases/tag/v1.7.1 :sunflower:


[Nitpick] No preview when editing detailed stats

Give an example of what is wrong
A follow up from https://github.com/TeamRizu/OutFox/issues/160. Is it intentional that the whole display preview is gone now?

Given an example of what it should look like
There should be a display preview on the right.

Are you able to provide the logs from the session where this happens? I was able to identify an issue on my end and fixed it, however I need to confirm that your issue is the same as I encountered.

userlog.txt
info.txt
log.txt

Seems to be fixed in the hotfix.


Can LibRdKafka.LoadLibrary use the path of the currently executing assembly?

Hello, I was wondering i this is something you would consider.
Running NUnit (without shadow copying), unless i force NUnit to use a different working dir, RdKafka fails to find the native dlls. This is because the following code uses relative paths.
Would you consider replacing this:
LibRdKafka.LoadLibrary(flag ? "x64/zlib.dll" : "x86/zlib.dll");
LibRdKafka.LoadLibrary(flag ? "x64/librdkafka.dll" : "x86/librdkafka.dll");
with this:
var baseDirectory = Path.GetDirectoryName(new Uri(Assembly.GetExecutingAssembly().GetName().CodeBase).LocalPath);
LibRdKafka.LoadLibrary(Path.Combine(baseDirectory, flag ? "x64/zlib.dll" : "x86/zlib.dll"));
LibRdKafka.LoadLibrary(Path.Combine(baseDirectory, flag ? "x64/librdkafka.dll" : "x86/librdkafka.dll"));
Note, this is not an RdKafka problem, but maybe the above is more appropriate anyway?

That's much better, could you test the latest CI build to confirm it works for you?

Yepp, works as expected. Thank you for the quick turnaround!

Cool, thanks for the patch!


incident 116 fix and version tweak

Current dependencies on/for this PR:

main

PR #114   👈

This comment was auto-generated by Graphite.


Change to actually alive netmap repo

Update reference to correct github repository as the original one has been removed. That way one can still get the code to compile.


Is there an example of being able to access a Minecraft Bedrock Edition server?

For example, obtain the following information:

this raknet impl is buggy and outdated, I recommend to use another

你好,我目前不在，如果你只是看到论坛邮件就不用回复了


v model value format

i see that there are props to format value on the input and on the action row but is there a way to format the value before it is assigned to the v-model.. cant seem to find that info anywhere in the docs or here in the issues.. im using latest version 5.4.0 in nuxt 3.6.5 project
btw keep up the good work on this package :)

@djixadin Yes, take a look at model-type prop

so this doesnt work for regular date picker? is there any other way for me then to round the date im receiving (something like moment.startOf('day'))


view script detects App.test.js App.css and icon.svg from Create-React-App

Need to ensure that only component and container files are logged in the console to import components/containers into a view.

Fixed in commit: https://github.com/blixjs/blix/commit/abf30b5266a78d5147d941e0dd2ac3c6373924d3


A few bugs in app.R

Hi team,
There are a few bugs in our app.R:

[x] The ratings and profits plots have duplicated legends.
[x] The directors for ratings and profits plots are not filtered by genre.
[x] The director dropdown menu does not update after choosing a genre.

I will see if I can fix those bugs tonight. Otherwise, we can do it together in lab together.

Bugs are fixed now!
I translated our plots from ggplot to plotly, which makes it easier to arrange subplots.


Update README.rst

aiohttp.web_urldispatcher.UrlDispatcher does not have add_get method

It does have in master branch


Makes StorageTank Not Explode for PC Tide

About the PR

#28606
Why / Balance

#28606
Technical details

Comments out the trigger even for the tank to explode
Media

https://github.com/space-wizards/space-station-14/assets/83733158/33c42ec7-e5fe-485f-9fb9-b9b4739b3dd5

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

remove: Welder Tanks no longer explode(Temporary)

gah, too late, someone did it already


[feauture] non async , and from curl_cffi import requests

from curl_cffi import requests
s = requests.Session(impersonate="safari15_3")
#s.proxies = genProxy()
r = s.get("https://kick.com/xqc")
print(r.text)
works fine with Ja3 just let you :)

A sync client won't happen, though I have thought about switching to that curl library as it supports async. It's just that I haven't taken the time to read up on how to use it

You could also try to use"tls client" library as it supports okhttp tls fingerprints.


Change regex to ignore custom descriptions

Check at start of string, since custom descriptions start with ''

Add to add-tests branch to avoid conflicts.


Type checks inconsistent when dealing with a -> b function types

from Standard.Base import all

t1 (f : Function) =
    f "x"

t2 (f : (Text -> Any)) =
    f "x"

t3 (f : (Integer | Function)) =
    f "x"

t4 (f : (Integer | (Text -> Any))) =
    f "x"

main =
    f x = "|" + x + "|"
    IO.println (t1 f)
    IO.println (t2 f)
    IO.println (t3 f)
    IO.println (t4 f)

Running the example above yields:
|x|
|x|
|x|
Execution finished with an error: Type error: expected `f` to be Integer, but got Function.
        at <enso> inconsistent-function-type.t4(inconsistent-function-type.enso:12-13)
        at <enso> inconsistent-function-type.main<arg-1>(inconsistent-function-type.enso:20:17-20)
        at <enso> inconsistent-function-type.main(inconsistent-function-type.enso:20:5-21)

Whereas we'd expect all 4 cases to behave the same way.
Weirdly, if the type is 'alone' - both Function and Text -> Any behave the same and accept a function input.
However, once we construct a sum type using |, for some weird reason the Text -> Any seems as if it disappears.

However, once we construct a sum type using |, for some weird reason the Text -> Any seems as if it disappears.

Text->Any actually disappears all the time. Just in case of t4 the Integer is left in the check. While in case of t2 there is no type check at all. Anyway #7298 fixes it all.


purge click

Codecov Report

Merging #822 (39a6d32) into main (f1ec7e3) will decrease coverage by 0.37%.
The diff coverage is 10.41%.

@@            Coverage Diff             @@
##             main     #822      +/-   ##
==========================================
- Coverage   41.72%   41.34%   -0.38%
==========================================
  Files         216      216
  Lines       11372    11379       +7
==========================================
- Hits         4745     4705      -40
- Misses       6627     6674      +47

Impacted Files
Coverage Δ

lnbits/commands.py
71.64% <ø> (-2.65%)
:arrow_down:

lnbits/server.py
0.00% <0.00%> (ø)

lnbits/wallets/cln.py
22.11% <8.10%> (ø)

lnbits/wallets/__init__.py
100.00% <100.00%> (ø)

lnbits/wallets/lndrest.py
18.51% <0.00%> (-20.38%)
:arrow_down:

lnbits/db.py
80.80% <0.00%> (-9.61%)
:arrow_down:

lnbits/wallets/macaroon/macaroon.py
27.53% <0.00%> (-5.80%)
:arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

i think there is nothing wrong with click, do you still want to remove it?


XLNet Embeddings

How can I retrieve contextual word vectors for my dataset using XLNet ?
The usage and examples in the documentation do not include any guide to use XLNet.
Thanks.

I'm currently finishing to add the documentation but just use XLNetModel instead of BertModel in the usage example with BertModel

Thanks a lot, @thomwolf for the quick reply. I'll try it out.

Here is an example now: https://huggingface.co/pytorch-transformers/model_doc/xlnet.html#pytorch_transformers.XLNetModel

@thomwolf, I tried the following snippet. The similarity between apple and orange comes out to be 0.36.
I am using only the last layer weights. This score doesn't look too good since the words are being used in the same context. Any ideas?
`config = XLNetConfig.from_pretrained('xlnet-large-cased')
tokenizer = XLNetTokenizer.from_pretrained('xlnet-large-cased')
model = XLNetModel(config)
input_ids = torch.tensor(tokenizer.encode("The apple juice is sour.")).unsqueeze(0)
input_ids_2 = torch.tensor(tokenizer.encode("The orange juice is sweet.")).unsqueeze(0)
outputs = model(input_ids)
outputs_2 = model(input_ids_2)
last_hidden_states = outputs[0]
last_hidden_states_2 = outputs_2[0]
apple = last_hidden_states[0][1]
orange = last_hidden_states_2[0][1]
x = apple
y = orange
cos_sim = dot(x.detach().numpy(),y.detach().numpy())/(norm(x.detach().numpy())*norm(y.detach().numpy()))
print(cos_sim)`

For me logits values changes as well ... using exactly the same settings as mentioned in the example.
Have you found a way to fix that?

@Oxi84 put model.eval() before you make the predictions. This fixed the problem of changing weights for me.

Thanks. For me it works when call like that:
 tokenizer = XLNetTokenizer.from_pretrained("xlnet-large-cased")
 model = XLNetLMHeadModel.from_pretrained("xlnet-large-cased")
 model.eval()

However accuracy seems to be much lower that for Bert - with the code i wrote here: https://github.com/huggingface/pytorch-transformers/issues/846
Did you find that the accuracy is good or bad? I compared with Bert on few examples for masked word prediction and its pretty much completely off.

@kushalj001 hi, how can I get the sentence vector

Hi, so it seems that creating a model with a configuration is primarily the problem here:
model = XLNetLMHeadModel.from_pretrained("xlnet-large-cased")
yields consistent outputs, but
config = XLNetConfig.from_pretrained("xlnet-large-uncased")
model = XLNetModel(config)
does not at all.
My question is, how is it possible to set configuration states (like getting hidden states of the model).  I have run the glue STS-B fine tuning code to customize the model which is stored at ./proc_data/sts-b-100, but when I load the model using code like this to get hidden states:
config = XLNetConfig.from_pretrained('./proc_data/sts-b-110/')
config.output_hidden_states=True
tokenizer = XLNetTokenizer.from_pretrained('././proc_data/sts-b-110/')
model = XLNetForSequenceClassification(config)
I get results that vary wildly across runs.

Looking at run_glue, it seems that actually outputs[1] is used for prediction?  This is confusing because all the examples use [0] and the documentation is not very clear.
outputs = model(**inputs)
tmp_eval_loss, logits = outputs[:2]
From run_glue.py

Ok, I figured the logits and loss issue out - the issue is that for XLNetForSequenceClassification, the second index does in fact have logits while the first has loss.

@thomwolf @Oxi84 while calculating word-embeddings of a document, i.e multiple sentences, is it necessary to pass the document sentence-wise? For my dataset, I removed punctuation as a part of the pre-processing step. So now, my whole document goes into the model. Does this hurt the model's performance? Does it make a lot of difference in terms of capturing the context of words?
Thanks

It should improve acuracy if the text is longer, but still for me Bert is way better ... on 20-40 words long text.

It should improve acuracy if the text is longer, but still for me Bert is way better ... on 20-40 words long text.

Yeah, even for my experiments, BERT simply outperfoms XLNet. Still don't know why though.
When you say "it should improve accuracy", you mean that feeding sentences to calculate word-vec would be better, right?

Did you managed to try tensorflow version of XLNet, there is a chance it might be different from the pytorch version?

Maybe there is some bug, but its unlikely since the bechmark results with the XLnet pytorch are the same. But I gues this would the first thing to try to recheck.

Did you managed to try tensorflow version of XLNet, there is a chance it might be different from the pytorch version?

Any simple way of doing this?

any updates regarding this issue?

@kushalj001 why remove the punctuation ? Is it domain specific or to improve accuracy?

My dataset had a lot of random punctuation, ie misplaced single and double-quotes.
But also, do punctuations add any valuable information to the text? Apart from the period (which can be used to break a large para into sentence), does keeping other punctuation symbols make sense?

I will close this issue which dates back before we had the clean documentation up here: https://huggingface.co/pytorch-transformers/
Please open a new issue with a clean explanation of your specific problem if you have related issues.


Watson crash report: CLR_EXCEPTION_DETOURS_e0434352_ReactiveUI.dll

Hi, I have some Watson crash details for you. Is there an email I could send something to that is more private than this forum? You can reach me at<EMAIL_ADDRESS>

Here is fine!

Okay. Here's the auto-generated full Watson title: [Watson] clr20r3: CLR_EXCEPTION_DETOURS_e0434352_ReactiveUI.dll!ReactiveUI.RxApp+__c__DisplayClass3._.cctor_b__1
And here are two sample callstacks:
1.KERNELBASE!RaiseException
2.clr!RaiseTheExceptionInternalOnly
3.clr!IL_Rethrow
4.System_Reactive_Core!System.Reactive.Concurrency.AsyncLock.Wait
5.unknown!System.Reactive.Concurrency.DefaultScheduler+__c__DisplayClass9`1[[System.__Canon, mscorlib]]._SchedulePeriodic_b__6
6.System_Reactive_PlatformServices!System.Reactive.Concurrency.ConcurrencyAbstractionLayerImpl+PeriodicTimer.Tick
7.mscorlib_ni!System.Threading.TimerQueueTimer.CallCallbackInContext
8.mscorlib_ni!System.Threading.ExecutionContext.RunInternal
9.mscorlib_ni!System.Threading.ExecutionContext.Run
10.mscorlib_ni!System.Threading.TimerQueueTimer.CallCallback
11.mscorlib_ni!System.Threading.TimerQueueTimer.Fire
12.mscorlib_ni!System.Threading.TimerQueue.FireNextTimers
13.mscorlib_ni!System.Threading.TimerQueue.AppDomainTimerCallback
14.clr!CallDescrWorkerWithHandler
15.clr!MethodDescCallSite::CallTargetWorker
16.clr!AppDomainTimerCallback_Worker
17.clr!ManagedThreadBase_DispatchInner
18.clr!ManagedThreadBase_DispatchMiddle
19.clr!ManagedThreadBase_DispatchOuter
20.clr!ManagedThreadBase_FullTransitionWithAD
21.clr!AppDomainTimerCallback
22.clr!ThreadpoolMgr::AsyncTimerCallbackCompletion
23.clr!UnManagedPerAppDomainTPCount::DispatchWorkItem
24.clr!ThreadpoolMgr::ExecuteWorkRequest
25.clr!ThreadpoolMgr::WorkerThreadStart
26.clr!Thread::intermediateThreadProc
27.kernel32!BaseThreadInitThunk
28.ntdll!__RtlUserThreadStart
29.ntdll!_RtlUserThreadStart
1.ReactiveUI!ReactiveUI.RxApp+__c__DisplayClass3._.cctor_b__1
2.System_Reactive_Core!System.Reactive.Concurrency.Scheduler.Invoke
3.System_Reactive_Core!System.Reactive.Concurrency.ScheduledItem_2[[System.TimeSpan,_mscorlib],[System.__Canon,_mscorlib]].InvokeCore
4.System_Reactive_Core!System.Reactive.Concurrency.CurrentThreadScheduler+Trampoline.Run
5.System_Reactive_Core!System.Reactive.Concurrency.CurrentThreadScheduler.Schedule[[System.Reactive.Concurrency.Scheduler+Pair_2[[System.__Canon,_mscorlib],[System.__Canon,_mscorlib]],_System.Reactive.Core]]
6.System_Reactive_Core!System.Reactive.Concurrency.LocalScheduler.Schedule[[System.Reactive.Concurrency.Scheduler+Pair_2[[System.__Canon,_mscorlib],[System.__Canon,_mscorlib]],_System.Reactive.Core]]
7.ReactiveUI!ReactiveUI.WaitForDispatcherScheduler.Schedule[[System.Reactive.Concurrency.Scheduler+Pair_2[[System.__Canon,_mscorlib],[System.__Canon,_mscorlib]],_System.Reactive.Core]]
8.System_Reactive_Core!System.Reactive.Concurrency.Scheduler.Schedule[[System.__Canon,_mscorlib]]
9.System_Reactive_Core!System.Reactive.ScheduledObserver_1[[System.Reactive.Unit,_System.Reactive.Core]].EnsureActiveSlow
10.System_Reactive_Core!System.Reactive.ObserveOnObserver_1[[System.Reactive.Unit,_System.Reactive.Core]].OnNextCore
11.System_Reactive_Core!System.Reactive.ObserverBase_1[[System.Reactive.Unit,System.Reactive.Core]].OnNext
12.System_Reactive_Linq!System.Reactive.Linq.ObservableImpl.Select_2+[[System.Reactive.Unit,_System.Reactive.Core],[System.Reactive.Unit,System.Reactive.Core]].OnNext
13.System_Reactive_Linq!System.Reactive.Linq.ObservableImpl.Where_1+[[System.Reactive.Unit,System.Reactive.Core]].OnNext
14.System_Reactive_Linq!System.Reactive.Linq.ObservableImpl.Throttle_1+[[System.Reactive.Unit,_System.Reactive.Core]].Propagate
15.System_Reactive_Core!System.Reactive.Concurrency.DefaultScheduler+__c__DisplayClass4_1[[System.UInt64,_mscorlib]]._Schedule_b__3
16.System_Reactive_PlatformServices!System.Reactive.Concurrency.ConcurrencyAbstractionLayerImpl+Timer.Tick
17.mscorlib_ni!System.Threading.TimerQueueTimer.CallCallbackInContext
18.mscorlib_ni!System.Threading.ExecutionContext.RunInternal
19.mscorlib_ni!System.Threading.ExecutionContext.Run
20.mscorlib_ni!System.Threading.TimerQueueTimer.CallCallback
21.mscorlib_ni!System.Threading.TimerQueueTimer.Fire
22.mscorlib_ni!System.Threading.TimerQueue.FireQueuedTimerCompletion
23.mscorlib_ni!System.Threading.QueueUserWorkItemCallback.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem
24.mscorlib_ni!System.Threading.ThreadPoolWorkQueue.Dispatch
25.mscorlib_ni!System.Threading._ThreadPoolWaitCallback.PerformWaitCallback

Also, just received another one, though it looks like a potential dupe of the first:
Bug 236364:[Watson] clr20r3: CLR_EXCEPTION_e0434352_ReactiveUI.dll!ReactiveUI.RxApp+__c__DisplayClass3._.cctor_b__1
Callstack:
reactiveui!ReactiveUI.RxApp+__c__DisplayClass3._.cctor_b__1
2.system_reactive_core!System.Reactive.Concurrency.Scheduler.Invoke
3.system_reactive_windows_threading!System.Reactive.Concurrency.DispatcherScheduler+__c__DisplayClass1_1[[System.__Canon,_mscorlib]]._Schedule_b__0
4.windowsbase_ni!System.Windows.Threading.ExceptionWrapper.InternalRealCall
5.windowsbase_ni!System.Windows.Threading.ExceptionWrapper.TryCatchWhen
6.windowsbase_ni!System.Windows.Threading.DispatcherOperation.InvokeImpl
7.windowsbase_ni!System.Windows.Threading.DispatcherOperation.InvokeInSecurityContext
8.mscorlib_ni!System.Threading.ExecutionContext.RunInternal
9.mscorlib_ni!System.Threading.ExecutionContext.Run
10.mscorlib_ni!System.Threading.ExecutionContext.Run
11.windowsbase_ni!System.Windows.Threading.DispatcherOperation.Invoke
12.windowsbase_ni!System.Windows.Threading.Dispatcher.ProcessQueue
13.windowsbase_ni!System.Windows.Threading.Dispatcher.WndProcHook
14.windowsbase_ni!MS.Win32.HwndWrapper.WndProc
15.windowsbase_ni!MS.Win32.HwndSubclass.DispatcherCallbackOperation
16.windowsbase_ni!System.Windows.Threading.ExceptionWrapper.InternalRealCall
17.windowsbase_ni!System.Windows.Threading.ExceptionWrapper.TryCatchWhen
18.windowsbase_ni!System.Windows.Threading.Dispatcher.LegacyInvokeImpl
19.windowsbase_ni!MS.Win32.HwndSubclass.SubclassWndProc

Paging @shana to the white courtesy ☎

/me picks up the :phone:
Is this on GHfVS? I don't see anything in the traces above that would point to us, is there more data?

@shana https://github.com/github/VisualStudio/issues/367#issue-160504601 was my clue

Right, it's clearly VS blowing up, but hard to know if it's us or some other extension using RxUI :/

/shrug - some of these also look like rethrown exceptions, so we need the InnerException to find out the real culprit

Well, we just shipped a slightly unstable beta, so it could be that.
@kevinfansler when did these start showing up?

236634 just landed on my plate as of June 30 and only has 100+ crashes against VS Update 3.
The "older" one looks like it was created in mid-June and has 50+ crashes against Update 3 and only a handful against some CTP before that (between Update 2 and Update 3).
Neither crash report is actually high enough to meet our bug bar, but I thought you might want to know about the crashes.

Ok, so it's not the beta (that came out July 5), and I doubt it's our hotfix release for https://github.com/github/VisualStudio/issues/367, there's not a lot in it and none of it is RxUI-related. If it's us, it's something that's been around for a while.
We can't really know for sure what's causing it without inner exception details, so that sucks :/
It could be any code using RxUI and not handling exceptions properly. It could be multiple extensions doing different things all using RxUI and triggering exceptions that end up with these reports.
We're currently working to ship GHfVS 2.0, and maybe those exceptions are the results of crashes during development (the number is certainly low enough). Let us know if these crash reports increase dramatically, it might be a hint of some edge case we're not catching in our tests.

This might be completely unrelated, but I noticed that after installing VS 2015 update 3 together with .Net Core tooling preview 2, the IDE started crashing on opening any file ("Visual Studio is restarting..."). I managed to fix it by uninstalling the SaveAllTheTime extension, which indeed uses RxUI.

Got it. Thanks for looking into it. I'll let you know if I see a dramatic rise in numbers. For this week, they both are holding steady (slight uptick, but within a margin of error). I'll also make a note of what pmbanka found.

@kevinfansler any uptick? FYI - The GitHub team for Visual Studio maintain their own private fork of ReactiveUI.

It's holding steady. 277 crashes total. We've closed the bug on our end as
the hit count doesn't really meet our new bug bar. But there's nothing new
in the auto-reporting.
On Mon, Aug 29, 2016 at 7:34 PM, Geoffrey Huntley<EMAIL_ADDRESS>wrote:

@kevinfansler https://github.com/kevinfansler any uptick? FYI - The
GitHub team for Visual Studio maintain their own private fork of
ReactiveUI.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactiveui/ReactiveUI/issues/1131#issuecomment-243316232,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI82_4dLVKIC8fh0bm4vDX7RQxDQiApNks5qk5bCgaJpZM4JGWI4
.

--
Kevin Fansler<EMAIL_ADDRESS>Technical Writer, Rooster Park
206 412-2671


No pointer when permission to show pointer is deactivated

As a GM I don't want my players to see my default pointer all the time (in case I have to move invisible creatures around) so I deactivated the showing of the GM pointer (see screenshot). But this has the side-effect of disabling the pointer from this module, too.
Can this be fixed?

I just tested this and it does not seem to be the case for v10 or v11. What Foundry version and module version are you on?

Thank you for the answer. I'm using v11.
I have just checked it again. It's probably not a permission problem as I thought before. When I created a new world and only activated your module, everything worked as expected. So I experimented a bit and found out that it doesn't work in conjunction with Monk's Common Display.
On the GM side the pointer is visible but not to connected clients.
I previously used PnP that worked together with Monk's Common Display.
Thanks for looking into it!

I will look at common display and try to work out what is happening.

Fixed in 0.0.4. Ironmonk was aggressively hiding any #hud elements that were not of his modules. I went one step further in aggressively showing my pointers.


NPE in language server when executing "isDynamicCommandRegistrationSupported()"

Description:
NPE in language server for default monaco client initialize request in below source line
https://github.com/ballerina-platform/ballerina-lang/blob/2.0.0-beta3-dev/language-server/modules/langserver-core/src/main/java/org/ballerinalang/langserver/util/LSClientUtil.java#L116
Steps to reproduce:

Start language server
initialize client LS connection

Affected Versions:
beta3-rc

Closing since the fix has been merged


RSASVE usage does not fill in output len when out is not null

providers/implementations/kem/rsa_kem.c
*outlen is not set if out != NULL
This means callers to EVP_PKEY_decapsulate that have provided a buffer of sufficient size (i.e. some large buffer) get informed that the length is whatever was passed in to collect the result which is incorrect (the length is left unchanged).
The documentation makes it clear this is an error  (and even if it didn't it would still be an error).

If secret is not NULL and the call is successful then the decapsulated secret data is written to secret and the amount of data written to secretlen.

Separately, this indicates that the testing of these APIs has a small gap that should also be addressed to ensure this sort of bug does not make it into the code. This should be backfit to all open branches in 3.x
Note this bug is not present in the corresponding DHKEM code.

Note: Task here is to do the following:

Modify rsasve_recover to check the input value of outlen to ensure that it is at least RSA_size(prsactx->rsa) bytes long if out != NULL
Assign a stack variable to the return value of RSA_private_decrypt and assign that value to *outlen if the value is non-negative
Add a test set to ensure that setting the value of outlen to various 'bad' values (0 >= *outlen > RSA_size(prsactx->rsa) ) results in appropriate error code, and that valid values returns an expected appropriate outlen value


IntegrityError: Duplicate entry for key 'publisher_public_id'

I am unable to publish a page.  Not sure what's causing this issue.
Here's the stack trace:
Environment:

Request Method: GET
Request URL: en/admin/cms/page/11/en/publish/

Django Version: 1.6.5
Python Version: 2.7.7
Installed Applications:
('djangocms_admin_style',
 'djangocms_text_ckeditor',
 'django.contrib.auth',
 'django.contrib.contenttypes',
 'django.contrib.sessions',
 'django.contrib.admin',
 'django.contrib.sites',
 'django.contrib.sitemaps',
 'django.contrib.staticfiles',
 'django.contrib.messages',
 'filer',
 'easy_thumbnails',
 'cmsplugin_cascade',
 'cmsplugin_filer_file',
 'cmsplugin_filer_folder',
 'cmsplugin_filer_image',
 'cmsplugin_filer_teaser',
 'cmsplugin_filer_video',
 'cms',
 'mptt',
 'menus',
 'south',
 'sekizai',
 'djangocms_style',
 'djangocms_column',
 'djangocms_file',
 'djangocms_flash',
 'djangocms_googlemap',
 'djangocms_inherit',
 'djangocms_link',
 'djangocms_picture',
 'djangocms_teaser',
 'djangocms_video',
 'reversion',
 'foo',
 'django_extensions')
Installed Middleware:
('django.contrib.sessions.middleware.SessionMiddleware',
 'django.middleware.csrf.CsrfViewMiddleware',
 'django.contrib.auth.middleware.AuthenticationMiddleware',
 'django.contrib.messages.middleware.MessageMiddleware',
 'django.middleware.locale.LocaleMiddleware',
 'django.middleware.doc.XViewMiddleware',
 'django.middleware.common.CommonMiddleware',
 'django.middleware.clickjacking.XFrameOptionsMiddleware',
 'cms.middleware.user.CurrentUserMiddleware',
 'cms.middleware.page.CurrentPageMiddleware',
 'cms.middleware.toolbar.ToolbarMiddleware',
 'cms.middleware.language.LanguageCookieMiddleware')

Traceback:
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/core/handlers/base.py" in get_response
  112.                     response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/utils/decorators.py" in _wrapped_view
  99.                     response = view_func(request, *args, **kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/views/decorators/cache.py" in _wrapped_view_func
  52.         response = view_func(request, *args, **kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/contrib/admin/sites.py" in inner
  198.             return view(request, *args, **kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/transaction.py" in inner
  431.                 return func(*args, **kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/reversion/revisions.py" in do_revision_context
  300.                     return func(*args, **kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/cms/admin/pageadmin.py" in publish_page
  1049.             published = page.publish(language)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/cms/models/pagemodel.py" in publish
  572.                 public_page.save()
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/cms/models/pagemodel.py" in save
  455.                 super(Page, self).save(**kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/mptt/models.py" in save
  717.         super(MPTTModel, self).save(*args, **kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/models/base.py" in save
  545.                        force_update=force_update, update_fields=update_fields)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/cms/models/pagemodel.py" in save_base
  473.         ret = super(Page, self).save_base(*args, **kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/models/base.py" in save_base
  573.             updated = self._save_table(raw, cls, force_insert, force_update, using, update_fields)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/models/base.py" in _save_table
  654.             result = self._do_insert(cls._base_manager, using, fields, update_pk, raw)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/models/base.py" in _do_insert
  687.                                using=using, raw=raw)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/models/manager.py" in _insert
  232.         return insert_query(self.model, objs, fields, **kwargs)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/models/query.py" in insert_query
  1514.     return query.get_compiler(using=using).execute_sql(return_id)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/models/sql/compiler.py" in execute_sql
  903.             cursor.execute(sql, params)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/backends/util.py" in execute
  69.             return super(CursorDebugWrapper, self).execute(sql, params)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/backends/util.py" in execute
  53.                 return self.cursor.execute(sql, params)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/utils.py" in __exit__
  99.                 six.reraise(dj_exc_type, dj_exc_value, traceback)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/backends/util.py" in execute
  53.                 return self.cursor.execute(sql, params)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/django/db/backends/mysql/base.py" in execute
  124.             return self.cursor.execute(query, args)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/MySQLdb/cursors.py" in execute
  205.             self.errorhandler(self, exc, value)
File "/home/foo/.virtualenvs/foo/lib/python2.7/site-packages/MySQLdb/connections.py" in defaulterrorhandler
  36.     raise errorclass, errorvalue

Exception Type: IntegrityError at /en/admin/cms/page/11/en/publish/
Exception Value: (1062, "Duplicate entry '11' for key 'publisher_public_id'")

Any help would be appreciated. Thanks!

@yakky We just experienced the same issue again, coming from
https://github.com/divio/django-cms/blame/develop/cms/models/pagemodel.py#L716
What kind of info would help?

I got this error too after a migration or maybe after the publisher_publish django command.
It's like the draft version is losing the link to his published version and vice-versa.
I got this:
mysql> SELECT id, publisher_is_draft, publisher_public_id FROM cms_page where reverse_id='services';+------+--------------------+---------------------+
| id   | publisher_is_draft | publisher_public_id |
+------+--------------------+---------------------+
|   53 |                  1 |                NULL |
| 1444 |                  0 |                NULL |
+------+--------------------+---------------------+
when I think I should have this:
+------+--------------------+---------------------+
| id   | publisher_is_draft | publisher_public_id |
+------+--------------------+---------------------+
|  53 |                  1 |                1444 |
| 1444 |                  0 |                 53 |
+------+--------------------+---------------------+

I tried to redo the migration and everything went fine. So I guess the problem is really to do all the commands after the migration in the right order:

migrate
cms moderator on
publisher_publish

Is this still relevant for you @granteagon in 3.1 or 3.2?

I think @avoine was right.  It has to do with command execution order.  I haven't spun up a new version of 3.1 or 3.2 recently, but I'm okay with closing this issue.

Hello.
We've experienced this sporadically in recent weeks in Django 1.7, Django-CMS 3.1. Unfortunately, it can basically bring a site down due to unhandled exceptions in cms.api.get_page_draft.
I've written a Django management command (tested on Python 2.7, Django 1.8.8, Django-CMS 3.2.0) to fix broken cms_page.publisher_public_id columns:
from django.core.management.base import BaseCommand

from cms.api import get_page_draft
from cms.models import Page

class Command(BaseCommand):
    help = 'Fix CMS pages (publisher_public_id foreign key)'

    def __init__(self, *args, **kwargs):
        self._fixed_count = 0
        super(Command, self).__init__(*args, **kwargs)

    def handle(self, *args, **options):
        self._fixed_count = 0
        self.stdout.write('Checking all non-draft pages ...')
        ps = Page.objects.filter(publisher_is_draft=False)
        for p in ps:
            try:
                get_page_draft(p)
            except Page.DoesNotExist:
                self._fix_cms_page(p)
        self.stdout.write('Fixed {} page(s).'.format(self._fixed_count))

    def _fix_cms_page(self, p):
        self.stdout.write('Fixing non-draft page id {} with no publisher_public_id ...'.format(p.id))
        draft_page_id = None

        if Page.objects.filter(id=p.publisher_public_id, publisher_is_draft=True).exists():
            draft_page_id = p.publisher_public_id
        elif Page.objects.filter(id=p.id - 1, publisher_is_draft=True).exists():
            draft_page_id = p.id - 1

        if draft_page_id is None:
            self.stdout.write('Cannot find a draft page corresponding to non-draft page id {}.'.format(p.id))
        else:
            # fix draft page
            draft_page = Page.objects.get(id=draft_page_id, publisher_is_draft=True)
            draft_page.publisher_public_id = p.id
            draft_page.save()

            # fix non-draft page (if necessary)
            if p.publisher_public_id is None:
                p.publisher_public_id = draft_page.id
                p.save()

            # success
            self._fixed_count += 1

Note that when it's looking for the draft equivalent of a non-draft page, if it can't find it from the publisher_public_id foreign key, it assumes the draft page id will be one less than the non-draft page id. This may not necessarily be the case if successive draft pages were created without non-draft equivalents, and therefore could mess up the cms_page table. Use with caution.
I've really only posted this here to demonstrate that this issue (at least in Django 1.7, Django-CMS 3.1) hasn't gone away, and is serious enough for us to have had to come up with this management command.
Unfortunately, the issue is intermittent and I cannot reproduce the problem (otherwise, I'd run it through a debugger and try to provide better information).

By the way, we're using MySQL 5.6.17 in production.

Is anyone watching this to reopen it? Or should I enter another issue?

@mikemcgowan please open a new one, as the underlying  issue is completely different from the original

Okay. Will do.

I experienced the same on Django 1.8.9 / Django-CMS 3.2.0 / Postgres 9.1
The management command helped! Thanks a bunch @mikemcgowan


Ability to send chat messages to self

This is an easy way to take quick notes.
All the mattermost/slack/hipchat/etc. apps support this feature.

This is already possible, see #3137

So it is. Thanks.


remove newlines from comment block

Code is often formatted to 80 columns or some similar width.
It would be nice to have a flag so that a word-wrapped comment block like
# The data that we are interested in is made of 8x8 images of digits.
# Let's have a look at the first 4 images, which is of course
# stored in the `images` attribute of the dataset.

appears like this, where Jupyter gets to wrap to the width of the cell:
The data that we are interested in is made of 8x8 images of digits. Let's have a look at the first 4 images, which is of course stored in the `images` attribute of the dataset.

Note that the underlying Markdown would still have the line breaks, in case the user wants to edit it to restore any intended line break.
The data that we are interested in is made of 8x8 images of digits.
Let's have a look at the first 4 images, which is of course
stored in the `images` attribute of the dataset.

Currently I do this by postprocessing.  You can correct the output file output.ipynb by removing the line breaks
:
perl -pi -e 's/(?=\n")//g' output.ipynb
Here's a better version that preserves a line break before an indented line.
perl -pi -e 's/(?=\n", "\S)//g' output.ipynb
Either version will preserve blank lines, e.g., between paragraphs.
However, this solution isn't bulletproof because it just does local pattern matching.  It would be better to have a command-line option to p2javoid inserting ` in the first place (when a flag is set.  That's also more user-friendly.
the generated file with.  (It's not bulletproof.)

Hi, Jason. I’m currently not actively maintaining this so it might take
quite some time for me to get gather some context.
Would you like to make a PR for this?
On Sun, 19 Nov 2023 at 6:05 AM, Jason Eisner @.***>
wrote:

Reopened #15 https://github.com/remykarem/python2jupyter/issues/15.
—
Reply to this email directly, view it on GitHub
https://github.com/remykarem/python2jupyter/issues/15#event-11002557398,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUDL46KLBWQLL4QHQLJT6DYFEWI7AVCNFSM6AAAAAA7RET3QKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGAYDENJVG4ZTSOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Looking at your p2j.py code and specifically at the part that adds
, it looks like you already included some logic that tries to determine where to add line breaks?
I might disagree with your choices there, but I'm afraid that if I just got rid of
, I would be breaking some cases where you deliberately wanted (which I may not have encountered).

I would be also interested in this feature 👍🏼


allow renderable slotable components

This is also related to https://github.com/msaraiva/surface/issues/265.

This is also related to #189 as this PR makes Slotable component renderable.

@Malian we'll still have the same restrictions around directives for slotable components, and at least as is this PR doesn't add any additional support or error logging for unknown directives

@lnr0626 I'll review this one over the weekend. I'll release v0.3.2 leaving this one out. It will probably be better to include it in v0.4.0 as it introduces important features with possible breaking changes.

Alright! @lnr0626 feel free to merge this and let us know what you have in mind about the docs and if you think we should improve/change the vocabulary around renderless/slotable in any way.
Thanks for working on this issue. Great stuff! ❤

I'll try to find some time to look at an update the docs this week


update contributor emails

@rc0x03 @teleological please make sure your commits have a valid email address associated with them which can be used to reach you. This is important in the event of security disclosures or follow up emails not on github. If both of you have valid emails on your commits then this issue can be closed.
Thanks!

I can confirm.

I've updated my global git settings to include my email address.

Thanks!


Pass enum as argument in foreign function

I am looking for a way to pass the enum as  argument in foreign function through node-ffi.
I have already installed enum module using npm install enum
var myEnum = require('enum');
var myTestenum = new myEnum({'key1':1,'key2':2});

var myFF= ffi.Library('someLib', {
     'someFn': [ 'int', ['myTestenum '] ]
   });

myFF.someFn(myTestenum .key1);

but it does not work. I think I am not passing correct argument data type during declaration.

That enum module doesn't work with ref/node-ffi. I wonder if they'd be willing to accept a PR making it a proper "type"...
As a workaround, just use "int" as the type.

@TooTallNate  have you create any enum prototype in ffi module .


More descriptive names

As a newcomer to Affjax, I found the names of the various types and functions to be a bit confusing.  As an example of what I found confusing, there is a type named Response and a typed named AffjaxResponse. They sound like the same thing. Furthermore, Response isn't actually a response. It's a description of how to interpret the body of a response.
In this PR I've tried to rename things in ways that, at least to me, makes more sense. Obviously naming is somewhat of an opinion. But, below is a list of the things that the PR renames with my rationale for the renaming.

Response to ResponseFormat. This type isn't actually a response. It's a description about which format the body of a response should be interpreted as.
AffjaxResponse to Response. This is the actual type of a response.
Request to RequestBody. This type is not a request but the type of things that can be used as the body of a request.
AffjaxRequest to RequestOptions. This type represents a record of options specifying how a request should be made.
affjax to request. The name "affjax" doesn't really communicate anything. The name request is appropriate since this is the function used to make requests.
Affjax removed. Affjax a is equivalent to Aff (Response a). IMO the Affjax type only obfuscated the types and made it harder to see that the functions in the library return responses in the Aff monad.

As an example this is the type of post today:
post :: forall a. Response a -> URL -> Request -> Affjax a

And this is the type of post in this PR:
post :: forall a. ResponseFormat a -> URL -> RequestBody -> Aff (Response a)

I'd argue the latter is easier to understand. I find it confusing that the first argument has the type Response. Why do I need to give a response to a function that is supposed to make a request and return a response? Also, the name RequestBody makes it explicit which argument is the actual payload for the request.

I like all of these changes 👍 Of course this will cause a breaking change, any plans for that soonish @garyb ?

I like them too.
I had no particular plans for doing anything on this at the moment, but now this has been opened there's a few things I can think of trying out, so will see how they go :wink:. Otherwise I'm fine with making a new breaking release just for this anyway.

I'm happy that you like the changes and thanks for merging :+1:


Failing to install on Proxmox 8.0.3: Frontend not Built

Script
nginx-proxy-manager
Describe the bug
✘ Frontend not Built
[ERROR] on line 281 with exit code 128 while executing command yarn install --silent --network-timeout=30000 > $__OUTPUT
System info:

Proxmox Version: 8.0.3
Alpine Version: 3.16 & 3.18

Screenshots

I join the problem is the same
System info:
Proxmox Version: 7.4.3
Alpine 3.16, 3.18 and debian 11

I also got this error on line 281 but with statuscode 1 instead.
I managed to fix it by creating the /app/node_modules folder myself and running sudo chmod -R 777 node_modules/ afterwards.

I had some struggles using the script today (installing over an already-existing Alpine 3.18 LXC on Proxmox 8.1.3).
A few hurdles I had to clear:

I didn't realize it, but I kept running out of disk space while trying to install.  I think one of the install steps requires a large download, and I kept running out of room
It appeared to me like the yarn executable wasn't getting found, so I added it to the PATH

The command that finally worked for me was:
rm -rf /usr/local/share/.cache ; export PATH="$PATH:/usr/local/bin" ; sh -c "$(wget --no-cache -qO- https://raw.githubusercontent.com/ej52/proxmox/main/install.sh)" -s --app nginx-proxy-manager
Hope that helps someone!

I have one "extra" error, just before the error from line 281:
error Couldn't find any versions for "sass-loader" that matches "^12.11.1"
Maybe that helps debugging.

Same issue here, tried installing on Alpine 3.16, Debian 11.17 with the same result. Including sass-loader issue.
Proxmox 8.1.4

I have one "extra" error, just before the error from line 281: error Couldn't find any versions for "sass-loader" that matches "^12.11.1" Maybe that helps debugging.

same to me.
pfa some more details:

running
root@proxy:~# wget --no-cache -qO- https://raw.githubusercontent.com/ej52/proxmox/main/install.sh >install.sh root@proxy:~# sh -x ./install.sh  --app nginx-proxy-manager > log
will provide no more helpful details.

had same issue also can reproduce it

The command that finally worked for me was: rm -rf /usr/local/share/.cache ; export PATH="$PATH:/usr/local/bin" ; sh -c "$(wget --no-cache -qO- https://raw.githubusercontent.com/ej52/proxmox/main/install.sh)" -s --app nginx-proxy-manager
Hope that helps someone!

this worked for me, thanks very much! i joined github today to mention that!

had same issue also can reproduce it

The command that finally worked for me was: rm -rf /usr/local/share/.cache ; export PATH="$PATH:/usr/local/bin" ; sh -c "$(wget --no-cache -qO- https://raw.githubusercontent.com/ej52/proxmox/main/install.sh)" -s --app nginx-proxy-manager
Hope that helps someone!

this worked for me, thanks very much! i joined github today to mention that!

Thanks bud! That worked for me!

had same issue also can reproduce it

The command that finally worked for me was: rm -rf /usr/local/share/.cache ; export PATH="$PATH:/usr/local/bin" ; sh -c "$(wget --no-cache -qO- https://raw.githubusercontent.com/ej52/proxmox/main/install.sh)" -s --app nginx-proxy-manager
Hope that helps someone!

this worked for me, thanks very much! i joined github today to mention that!

Thanks bud! That worked for me too!

This worked for me as well.
@ej52 you may want to append those two commands to the beginning of the script
Also, please note that it seems like others are still experiencing this issue. See https://github.com/ej52/proxmox-scripts/issues/176#issue-2274237473 .


Added OmitFromUI tag to operators without input and output ports

Needs to be fixed in the app builder.


gha: fossa license scanning

Please reference for a complete summary: https://github.com/kubeflow/model-registry/issues/323
Description
Resolves #323 by introducing FOSSA license scanning, a tool supported by CNCF, in the way recommended by the tool documentation and as implemented by other projects.
How Has This Been Tested?
n/a
Merge criteria:

All the commits have been signed-off  (To pass the DCO check)

[x] The commits have meaningful messages; the author will squash them after approval or in case of manual merges will ask to merge with squash.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work.
[x] Code changes follow the kubeflow contribution guidelines.

If you have UI changes

[ ] The developer has added tests or explained why testing cannot be added.
[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Verify that UI/UX changes conform the UX guidelines for Kubeflow.

/lgtm


Is it possible bypass renormalize by check if simplify_types made any changes?

Is it possible to bypass renormalize() in step_simplify_types() by check the return value of simplify_types() make any changes to any node? If no changes made in simplify_types(), then renormalize() can be bypassed for better performance.

It's pretty much always going to change something in a realistic scenario (any use of a data structure that's not a tuple and any use of keyword arguments produce cruft that's removed here). Regardless, the goal is to eventually never run renormalize and update the information incrementally instead. It's already almost possible to remove this particular renormalize: only 8 tests don't pass if that line is removed. Presumably there's not a lot to do to fix simplify_types so that renormalize isn't needed at all.

Thanks for your comments. As renormalize is costly so if it can be removed it will be good for performance.


update readme

Codecov Report

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

@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   98.17%   98.17%
=======================================
  Files          21       21
  Lines         929      929
  Branches      148      148
=======================================
  Hits          912      912
  Misses         17       17

Continue to review full report at Codecov.

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


Promo updates

Important: Do not merge this PR, it's supposed to go live on Monday (14.6.).

Putting this to bnb/btc pool because it will be released then too


Revert "Check Nix flake workflow (#46)"

related to

#46

cc/ @IogaMaster
Description of changes
this PR reverts #46.
without any justification for this, i'd rather revert the changes :yum:

This makes sure the flake is functional and doesn't error when someone wants to enter the devshell or build nufmt with nix.

we'll decide the fate of that PR from https://github.com/nushell/nufmt/pull/52 :wink:

#53 Happens anyway because the flake syntax is valid but the package itself built for macos needs Iokit to build.

So if we dont merge #52 then I will write a check that runs on macos to test builds there as well.

#46 has been sorted out, thanks @IogaMaster for the explaining :pray:


[WIP] Reference architecture one

do not merge, this is only open so we can collect comments

I know the code is a rough draft, but I experimented with it locally and found some things to change. I'll add some line comments.

This is very nice overall. I mostly marked some typos and linguistic things. Feel free to selectively ignore them.
Main points:

Change the JSONB example to something where the JSONB is in the raw data
Introduce the rollup_thresholds table before the function
Maybe add a "psql ... | xargs ..." type shell command as an alternative for the rollup function, since dblink won't work on Citus Cloud

There are two places in the top-level table of contents where Technical Solutions might fit in better. It depends on what we consider the purpose of the section to be.

If the section is meant as a bridge from the tutorials to bigger, more realistic problems then it might be good to put it between Tutorials and Installation.
If however the section is intended as a place to walk people through esoteric situations they may encounter after using Citus for a while then maybe it should go down between Administration and Reference.

@lithp -- I'm adding my notes to this pull request instead of #153.

@byucesoy Just pushed a commit which I think handles all of these.

@lfittl -- I just checked with @begriffs. Putting this as a section after Cloud and before Administration sounds good.

@ozgune I assume you meant @lithp?

Apart from end-to-end tests, I also tried some failure test as @onderkalaci's suggestion.
While we ingest data, I failed one of the workers.
Before restarting failed node;

Remaining worker continued to get new data. :white_check_mark:
Rollups are done in remaining worker without any problem. :white_check_mark:
SELECT queries are answered from remaining shard :white_check_mark:
Placement on failed worker does not marked as invalid :x:

I think this one is problematic because when we restart failed worker, we will get wrong/old data from this worker
After restarting failed node;

There is no sync between workers :question:

I think this would be nice to have, but it may be not in the scope of this reference architecture

New data are sent to restarted worker :question:

After restart, user intervention may be required to continue to send data (like manualy syncing workers)

Rollups also run in restarted worker :x:

We end up incomplete/wrong aggregations

I'm merging this change based on request of team @ozgune amd @craigkerstiens. We will fix remaining items on iteration basis.

@craigkerstiens I merged this change but what should I do to generate temporary (and eventually permanent deployment)?


How to add certSANs

Hi
I am considering to use microk8s, but I need to change/add to certSANs
Do you know if you can do that and how?

Great thanks, it certainly helps!


Chat log from previous session displays without "encoding" the messages

Observed on Foundry 0.6.6 and Polyglot 1.2.
If a user is logged in and the GM sends a message in a language the character doesn't understand, the message is appropriately encoded.
If that user logged out and back in, they can then read the unencoded message in the chat log.

Never mind.  I think this was happening because I was speaking as the gamemaster rather than as a particular token.