Code window only shows placeholder UI on start, no other controls

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: nightly-20210612, and nightly-20210616. Currently on 1.58.0-insider 3e0c442a8ee9fdc037d627d70fe291f72af04cfd x64
OS Version: Windows 10 Pro Insider (21390)

Steps to Reproduce:

Open VS Code
Window appears, but seemingly stops loading forever. It only displays the placeholder monaco-parts-splash elements and no other controls. No resize handles, but I can grab the window by the title bar area and resize it with Aero Snap (either dragging or keyboard shortcuts). Window immediately responds to Alt-F4. Process is idle (0-0.2% CPU usage, ~18MB RAM). See screenshot:

A few things I've tried with no resolution:

Deleting %appdata%\Code - Insiders\Code Cache as suggested in https://github.com/microsoft/vscode/issues/114459
I tried launching from the terminal with combinations of --open-devtools, --disable-features=CalculateNativeWinOcclusion, --disable-extensions, --disable-gpu, with no change.
Electron Chrome devtools do not show any messages in the console.
Tried opening a new path in case it was an issue with restoring another session.
Tried updating from previous build (2021-06-12) but still encounter the issue on current nightly (2021-06-16).

Some more context:

VS Code is installed via scoop install vscode-insiders
I've successfully run VS Code (2021-06-12) before on this OS build earlier this week.
I can run a fresh install of the official release version of Code (1.57.0) just fine, even if I copy over the settings and symlink the extension folder.
Logs from %appdata%\Code - Insiders\logs are very sparse. All launches today show only a main.log with these messages below:

[2021-06-16 14:17:57.744] [main] [info] window: using vscode-file:// protocol and V8 cache options: bypassHeatCheck
[2021-06-16 14:17:58.015] [main] [info] update#setState idle

Sometimes the next two lines are also included:
[2021-06-16 14:18:28.022] [main] [info] update#setState checking for updates
[2021-06-16 14:18:28.199] [main] [info] update#setState idle

Keywords for search:

blank window
empty window
blank status bar, blank menu bar

@lediur sorry to hear that, as a first step can you make sure to backup the entire %appdata%\Code - Insiders folder so that anything we try out here does not result in loss of information to diagnose this issue further?
The Code Cache folder is no longer being used for caching, it will now be inside %appdata%\Code - Insiders\CachedData. This folder will contain a folder per version (commit) of VSCode, so there must be a 3e0c442a8ee9fdc037d627d70fe291f72af04cfd folder in there. Within you should find a chrome folder, which is the location of the code cache.
Here are things to try:

delete that folder (or the entire CachedData folder) and see if that helps
open %userprofile%\.vscode-insiders\argv.json and add the following line [1] before the closing }

[1]
	, "enable-browser-code-loading": false

I would still like to see if we can find an exception though, so with the old user data contents that do not work, try to run like this: code-insiders --verbose --open-devtools
This should open the developer tools and on the "Console" tab maybe something is printed?

Hey @bpasero thanks for the reply. Since I opened the ticket, I did try deleting all of the folders that started with “Cache” in case it was causing things to get stuck in a weird state. The folder you mentioned was one of the folders I deleted. Unfortunately, there was no change in behavior after removing those folders.
I also tried looking into the dev tools console earlier, but saw absolutely no logging output at all (no errors, no tracing, nothing).

Okay, I added the flag as you suggested, and the editor loaded up properly as normal. To confirm, I removed the flag and restarted, and it freezes at the blank placeholder screen again.

@lediur can you then do the code-insiders --verbose --open-devtools launch again without the enable-browser-code-loading?

Yep, sure thing. Here's the log from the terminal:
> code-insiders --verbose --open-devtools

[main 2021-06-17T06:59:45.926Z] Starting VS Code
[main 2021-06-17T06:59:45.928Z] from: c:\Users\dliu\scoop\apps\vscode-insiders\nightly-20210616\resources\app
[main 2021-06-17T06:59:45.928Z] args: {
  _: [],
  diff: false,
  add: false,
  goto: false,
  'new-window': false,
  'reuse-window': false,
  wait: false,
  help: false,
  'list-extensions': false,
  'show-versions': false,
  version: false,
  verbose: true,
  status: false,
  'prof-startup': false,
  'prof-v8-extensions': false,
  'disable-extensions': false,
  'disable-gpu': false,
  telemetry: false,
  debugRenderer: false,
  logExtensionHostCommunication: false,
  'skip-release-notes': false,
  'skip-welcome': false,
  'disable-telemetry': false,
  'disable-updates': false,
  'disable-keytar': false,
  'disable-workspace-trust': false,
  'disable-crash-reporter': false,
  'crash-reporter-id': 'ac0b41a1-e1bf-4d5e-b907-5ba3e31632f8',
  'skip-add-to-recently-opened': false,
  'unity-launch': false,
  'open-url': false,
  'file-write': false,
  'file-chmod': false,
  'driver-verbose': false,
  force: false,
  'do-not-sync': false,
  trace: false,
  'force-user-env': false,
  'force-disable-user-env': false,
  'open-devtools': true,
  __sandbox: false,
  'no-proxy-server': false,
  nolazy: false,
  'force-renderer-accessibility': false,
  'ignore-certificate-errors': false,
  'allow-insecure-localhost': false,
  logsPath: 'C:\\Users\\dliu\\AppData\\Roaming\\Code - Insiders\\logs\\20210616T235945'
}
[main 2021-06-17T06:59:45.930Z] Resolving machine identifier...
[main 2021-06-17T06:59:45.930Z] Resolved machine identifier: c9ec33d18138ccc2c3c9cfd4f41d6ef2f126138cdd5c397cc267c12cc36ce240
[main 2021-06-17T06:59:45.931Z] Main->SharedProcess#connect
[main 2021-06-17T06:59:45.950Z] StorageMainService: creating global storage
[main 2021-06-17T06:59:45.951Z] lifecycle (main): phase changed (value: 2)
[main 2021-06-17T06:59:45.952Z] windowsManager#open
[main 2021-06-17T06:59:45.952Z] windowsManager#open pathsToOpen [
  {
    workspace: { id: '34396725fd6acd65b70c27fa15c9d72f', uri: [g] },
    exists: true
  }
]
[main 2021-06-17T06:59:45.954Z] IPC Object URL: Registered new channel vscode:9c0ca6b6-35bf-4772-844d-865f4103f307.
[main 2021-06-17T06:59:45.955Z] window#validateWindowState: validating window state on 2 display(s) { mode: 1, x: 1094, y: 0, width: 1284, height: 1400 }
[main 2021-06-17T06:59:45.955Z] window#validateWindowState: multi-monitor working area { x: 0, y: 0, width: 2560, height: 1400 }
[main 2021-06-17T06:59:45.955Z] window#ctor: using window state { mode: 1, x: 1094, y: 0, width: 1284, height: 1400 }
[main 2021-06-17T06:59:45.956Z] window: using vscode-file:// protocol and V8 cache options: bypassHeatCheck
[main 2021-06-17T06:59:46.028Z] StorageMainService: creating workspace storage (34396725fd6acd65b70c27fa15c9d72f)
[main 2021-06-17T06:59:46.028Z] windowsManager#open used window count 1 (workspacesToOpen: 0, foldersToOpen: 1, emptyToRestore: 0, emptyToOpen: 0)
[main 2021-06-17T06:59:46.110Z] lifecycle (main): phase changed (value: 3)
[main 2021-06-17T06:59:46.119Z] update#setState idle
[main 2021-06-17T06:59:46.120Z] resolveShellEnv(): skipped (Windows)
[main 2021-06-17T06:59:46.483Z] [File Watcher (node.js)] [CHANGED] c:\Users\dliu\AppData\Roaming\Code - Insiders\User\globalStorage
[main 2021-06-17T06:59:46.485Z] [File Watcher (node.js)] [CHANGED] c:\Users\dliu\AppData\Roaming\Code - Insiders\User\globalStorage
[main 2021-06-17T06:59:46.627Z] resolveShellEnv(): skipped (Windows)
[main 2021-06-17T06:59:46.694Z] [File Watcher (node.js)] >> normalized [CHANGED] c:\Users\dliu\AppData\Roaming\Code - Insiders\User\globalStorage
[24288:0616/235946.950:VERBOSE1:CONSOLE(1)] "Main._createAppUI: 43.784912109375 ms", source: devtools://devtools/bundled/main/main.js (1)
[24288:0616/235946.971:VERBOSE1:CONSOLE(1)] "Main._showAppUI: 19.7060546875 ms", source: devtools://devtools/bundled/main/main.js (1)
[24288:0616/235947.102:VERBOSE1:CONSOLE(1)] "Main._initializeTarget: 11.714111328125 ms", source: devtools://devtools/bundled/main/main.js (1)
[main 2021-06-17T06:59:47.117Z] [File Watcher (node.js)] [CHANGED] c:\Users\dliu\AppData\Roaming\Code - Insiders\User\globalStorage
[24288:0616/235947.246:VERBOSE1:CONSOLE(1)] "Main._lateInitialization: 0.370849609375 ms", source: devtools://devtools/bundled/main/main.js (1)
[main 2021-06-17T06:59:47.327Z] [File Watcher (node.js)] >> normalized [CHANGED] c:\Users\dliu\AppData\Roaming\Code - Insiders\User\globalStorage

As before, there's no output in the devtools console.

@lediur can you try this:

from devtools click on Sources
click on "Pause" button [1]
hit Ctrl+R from devtools to reload window
does it break somewhere?

If I toggle the "Pause on exceptions" button to "on" and reload, I don't get anything. If I also check "pause on caught exceptions" I get some stuff. Do you want info on those?

Yeah if possible.

The thing that puzzles me a bit is that you can reproduce when CachedData is deleted, but in that case no code cache is generated and thus I do not think this is exactly the same issue as https://github.com/microsoft/vscode/issues/114459
To clarify, you can even reproduce when you do code-insiders --user-data-dir <some empty folder>?

Here's one:
Error: Cannot find module 'graceful-fs' Require stack: - electron/js2c/renderer_init

To clarify, you can even reproduce when you do code-insiders --user-data-dir <some empty folder>?

I haven't tried this yet. I can try that now.

Okay, just tried this:

Now the window is totally blank, and doesn't respond to dragging anymore. Still can aero snap it around so it's responsive.

@lediur uh oh, that looks more like the issue we had...
Back to your normal user data dir and devtools, I recorded something to try out to see specifically where the error could happen by setting two breakpoints and only enabling break on error after the first one is hit, can you try? In your case I would expect the first breakpoint to get hit but not the second one. The first one will stop where that splashscreen is rendered (the shapes in the window before the workbench loads) and the second one is for successfully loading the workbench:

Okay, just tried it. I set the two breakpoints at :6 and :17 and reloaded. I think I've hit the second breakpoint according to the call stack:

uh oh, that looks more like the issue we had...

Actually I take that back. The reason that no placeholder UI shows is just a consequence of starting with a fresh user data dir where we did not cache the placeholder yet, that is why the window is empty.

@lediur yeah, do you hit the one above too or does that end up in an error when you break on error + caught exceptions?

It hits the error that I screenshotted above before it gets to the first breakpoint.

@lediur ok very confusing for me why this would be hit. To rule out corrupt installation issues can you try to reproduce from:

the same build: Download
todays build: Download

Okay let me give those a shot.

Okay both builds seem to launch fine.

@lediur ok next step to reinstall your main VSCode, not sure if you use system wide install or user install?

User: Download
System: Download

Just to confirm: you are NOT running with that argv.json change I had asked you initially right?

Yeah, I removed the argv flag you mentioned earlier.
I used the scoop package manager to install vscode (via scoop install vscode-insiders) which I believe uses the portable version. https://github.com/lukesampson/scoop-extras/blob/master/bucket/vscode-insiders.json indicates it downloads the archive at https://update.code.visualstudio.com/latest/win32-x64-archive/insider.
Do you still want me to go through the setup process with the installer? I could also try uninstalling and reinstalling via scoop, or unpacking the install from that archive link over my installation.

@lediur oh yeah, I am not entirely sure what scoop does. Let's keep the version on disk as it is for now to figure out more, there is one additional idea we had when I brought this up with team today:

for once bring back the argv.json flag again for this test
from the command line run code-insiders --no-cached-data
Does this show the same symptoms?

Okay this is super bizarre but I just used rclone check to compare the two folders (my broken installation, at ~/scoop/apps/vscode-insiders/current, and the working download of yesterday's build). rclone found them to be identical (apparently using a hash check as well 😵)

Ok thanks maybe I need to try out scoop myself to see if I can reproduce.

I'll try doing that. Happy to be of help tracking this issue down - it's the least I can do :P

Oh yeah that would be awesome!

Yep, I was able to repro with a fresh scoop install on a "fresh" Windows Sandbox image.
Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.
                                                                                Install the latest PowerShell for new features and improvements! https://aka.ms/PSWindows                                                                                PS C:\Users\WDAGUtilityAccount> Set-ExecutionPolicy RemoteSigned -scope CurrentUser
Execution Policy Change
The execution policy helps protect you from scripts that you do not trust. Changing the execution policy might expose
you to the security risks described in the about_Execution_Policies help topic at
https:/go.microsoft.com/fwlink/?LinkID=135170. Do you want to change the execution policy?
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "N"): y
PS C:\Users\WDAGUtilityAccount> iwr -useb get.scoop.sh | iex
>>
Initializing...
Downloading scoop...
Extracting...
Creating shim...
Downloading main bucket...
Extracting...
Adding ~\scoop\shims to your path.
'lastupdate' has been set to '2021-06-17T02:10:08.4943169-07:00'
Scoop was installed successfully!
Type 'scoop help' for instructions.
PS C:\Users\WDAGUtilityAccount> scoop bucket add extras
Git is required for buckets. Run 'scoop install git' and try again.
PS C:\Users\WDAGUtilityAccount> scoop install git
Installing '7zip' (19.00) [64bit]
7z1900-x64.msi (1.7 MB) [=====================================================================================] 100%
Checking hash of 7z1900-x64.msi ... ok.
Extracting 7z1900-x64.msi ... done.
Linking ~\scoop\apps\7zip\current => ~\scoop\apps\7zip\19.00
Creating shim for '7z'.
Creating shortcut for 7-Zip (7zFM.exe)
'7zip' (19.00) was installed successfully!
Installing 'git' (2.32.0.windows.1) [64bit]
PortableGit-2.32.0-64-bit.7z.exe (44.4 MB) [==================================================================] 100%
Checking hash of PortableGit-2.32.0-64-bit.7z.exe ... ok.
Extracting dl.7z ... done.
Linking ~\scoop\apps\git\current => ~\scoop\apps\git\2.32.0.windows.1
Creating shim for 'git'.
Creating shim for 'gitk'.
Creating shim for 'git-gui'.
Creating shim for 'tig'.
Creating shim for 'git-bash'.
Creating shortcut for Git Bash (git-bash.exe)
Creating shortcut for Git GUI (git-gui.exe)
'git' (2.32.0.windows.1) was installed successfully!
PS C:\Users\WDAGUtilityAccount> scoop bucket add extras
Checking repo... ok
The extras bucket was added successfully.
PS C:\Users\WDAGUtilityAccount> scoop install vscode-insider
Couldn't find manifest for 'vscode-insider'.
PS C:\Users\WDAGUtilityAccount> scoop install vscode-insiders
WARN  This is a nightly version. Downloaded files won't be verified.

Linking ~\scoop\apps\vscode-insiders\current => ~\scoop\apps\vscode-insiders\nightly-20210617
Creating shim for 'code-insiders'.
Creating shortcut for Visual Studio Code - Insiders (Code - Insiders.exe)
Running post-install script...
'vscode-insiders' (nightly-20210617) was installed successfully!
Notes
-----
Add Visual Studio Code as a context menu option by running:
"C:\Users\WDAGUtilityAccount\scoop\apps\vscode-insiders\current\vscode-install-context.reg"
PS C:\Users\WDAGUtilityAccount>

However, if I download the zip from the link in the manifest and unzip it directly, it works:

If I navigate to the scoop "installation" folder for vscode and launch the executable (avoiding the scoop shim), it does not work.

Oh. I think I might have found the issue.
Scoop maintains older versions and uses a symlink called "current" to point to the folder containing the most recent version:

If I open current/Code - Insiders.exe, it hangs with the issue.
If I open the original folder nightly-20210617/Code - Insiders.exe, it launches properly.

Can confirm that launching the non-symlinked folder nightly-20210616/Code - Insiders.exe works on my machine.

@lediur great findings! I will try to reproduce as well.
In the debugger when you see that exception raised about "gracefulfs" not being found, if you press play to continue, are there more exception breakpoints hit after, or is that the only one? I wonder if that one is expected in node.js but another one is not.

It throws another graceful-fs exception after the first one. Seems like it's in the same place so maybe being called twice?
exception 1:

exception 2:

Yeah looks like it, thanks a ton, now that I should be able to reproduce I can investigate further. I hope this is addressable by us in some way...

This is very easy to reproduce:

be on Windows 10
create a folder vscode-insiders
unzip vscode as archive into a folder nightly-20210617
open a command prompt as admin
run mklink /D current nightly-20210617
close the command prompt
open a normal command prompt
run ".\current\Code - Insiders.exe" --open-devtools

=> 🐛  nothing loads, reload from devtools with break on error ends up in the exception that gracefulfs cannot be found
I get a feeling that this may be ASAR related.

Here is something interesting, when running with our loader loading node.js modules, the paths include the node_modules.asar thing:

But when running with vscode-file and the node.js standard loader the paths are only this:

And here we go, when I do run without symlink I see the asar file is back on the paths!

Further looking into it, it falls apart here:
https://github.com/microsoft/vscode/blob/30f0d188c71400fbb9a1e2b00b4f20ebe1256d6d/src/bootstrap.js#L47-L47
appRoot is actually the resolved symlink (nightly-20210617)
And then here:
https://github.com/microsoft/vscode/blob/30f0d188c71400fbb9a1e2b00b4f20ebe1256d6d/src/bootstrap.js#L73-L73
This check is never true because for node.js the paths are the symlink'ed form (current).
The property appRoot is coming all the way from here:
https://github.com/microsoft/vscode/blob/30f0d188c71400fbb9a1e2b00b4f20ebe1256d6d/src/bootstrap.js#L73-L73
Which seems to be using require.toUrl which I think is a method our loader defines. But I fail to understand where the loader gets its root path from.

I wonder if a pragmatic fix could be to just always add the node_modules.asar file to the lookup paths and then remove this change again once the filesystem has moved off the renderer because then gracefulfs wouldn't be needed anymore.
@alexdima thoughts?

Nice debugging. Out of curiosity and to narrow things down I downloaded the Linux archive version and set up a similar symlink structure on WSL, and that launched fine, so seems like it's Windows specific. I have very little knowledge of the internals of the editor startup process but thought this might help save a bit of time.

@lediur @bpasero Thank you for figguring this out! Very impressive analysis! I'm working on a fix.

/verified

It works in 1.58.0-insider 🎉


CV-1241998: Fix unitialized member in NumericType

This is by design, just need to make coverity be quiet.

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/411/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/422/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/533/

AU check successful! https://ci.trafficserver.apache.org/job/autest-github/407/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/2105/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/1999/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/665/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/447/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/457/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/570/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/2036/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/444/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/2140/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/701/


Removed package com.sun.tools.javac.util

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

removed [package com.sun.tools.javac.util


Remove warnings

Not a big deal but I have those warnings which rise when I compile my app:
warning: found quoted keyword "coveralls" but the quotes are not required. Note that keywords are always atoms, even when quoted. Similar to atoms, keywords made exclusively of Unicode letters, numbers, underscore, and @ do not require quotes
  /app/deps/parallel_stream/mix.exs:17
==> parallel_stream
warning: Stream.chunk/4 is deprecated. Use Stream.chunk_every/4 instead
  lib/parallel_stream/producer.ex:16
Generated parallel_stream app

Trying to get rid of it.

+1

@beatrichartz Would it be possible to release the current master to hex to get rid of this warning?

I'm probably going to get to this sometime in the next days. Will have to move CI from travis so it might take a bit.


Escape filePath and args passed to the phpcs command

Fixes issue #144
Tested on Mac.
Need to be tested on Windows.

@aslamdoctor thanks for the PR. Of course, both the filePath and the standard would need to be escaped if they're both in the same location :facepalm: I'm annoyed I missed that :grin:
One question: in my original PR, to fix for just the filePath, instead of running the string through a string replace, I wrap the string in double quotes. This comes from my experiences in bash scripting, where one should wrap any variables in double quotes, to prevent scripts from breaking when input contains spaces, line feeds, glob characters and such.
I will freely admit I don't know a lot about the node child_process spawn method used to execute the phpcs command, but it seems to me that following the same process for escaping strings in the terminal made sense here. However, as it's a JavaScript code calling a method to execute the command, perhaps your solution is the better option.
Do you have any opinions on this?
@valeryan, as the original developer, I'd also appreciate any thoughts you might have on which solution is "better"; thanks.

Seems like a good solution. Makes me wish I had time to write unit test though...

@jonathanbossenger I was also confused why adding the double quote didn't work. It was supposed to work. It's quiet tedious to debug this. But I believe when you pass anything as argument to a command on terminal, it shouldn't have space and needs to be escaped with \.

Ok, thanks for the confirming folks, let's go ahead and get this merged.

Makes me wish I had time to write unit test though...

@valeryan I hear you, but in my experience open source tools often don't get tests until someone comes along with enough time to add them. I will see if I can put some time aside for this as well.

Need to be tested on Windows.

I can test this on Windows, hopefully tomorrow.

Hey @jonathanbossenger no probs. Thanks for the heads up. Broken windows boot is always a pain 😔

This is pretty annoying bug, any chance its getting merged/released soon?

+1


restructure command line arguments

currently they are mixed up with commands and options.
possible structure
basic commands:

message - delete messages
bot - delete bot messages
files - delete files
info - dump information about users and channels

extras:

invert the logic add ad --dry-run instead of --perform
don't separate groups and channels but unify them

in favor of the develop branch following a python module approach


missed part for list blobs

[Enter feedback here]
As document said in list blobs part: The third approach uses --num-results and --marker parameters to limit the retrieval of all blobs within a container.
But there is no related example

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

ID: 2f1bd1c5-0eb0-5b25-aad9-e8a236b8e47b
Version Independent ID: 370c7591-bf3e-6ef9-5dbf-660b805b090c
Content: Manage block blobs with Azure CLI - Azure Storage
Content Source: articles/storage/blobs/blob-cli.md
Service: azure-blob-storage
GitHub Login: @stevenmatthew
Microsoft Alias: shaas

@XiaoYue2-22
Thanks for your feedback! We will investigate and update as appropriate.

@XiaoYue2-22 - We've updated this article to add the missing sample. Thanks for your patience.

thanks tamaram


Create a new "Customizing Gitea" page under Advanced

Create a new "Customizing Gitea" page under Advanced

For now, the only customization documented is the /robots.txt one.
Fixes #66 (backport)

Mention to restart Gitea when changing custom/robots.txt

LGTM

Please rebase

done

LGTM


[Bug]:  highlighting broken after lsp format

Neovim version
v0.10.0-dev-510+g3c4890d1e
Language affected
clojure
Query
No response
Strategy
No response
Description
In a clojure file, using the clojure-lsp formatting, the rainbow delimiters become out of sync until the file is edited again.
This can be seen in the below asciicast (which I hopefully formatted correctly).

I can reproduce the issue using the following clojure file, deps.edn, and minimal config by opening the file with nvim -u init.lua minimal.clj and running :lua vim.lsp.buf.format()
init.lua
local lazypath = "/tmp/rainbow/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({
		"git",
		"clone",
		"--filter=blob:none",
		"https://github.com/folke/lazy.nvim.git",
		"--branch=stable", -- latest stable release
		lazypath,
	})
end
vim.opt.rtp:prepend(lazypath)
vim.cmd.colorscheme("retrobox")

require("lazy").setup({
	{
		"nvim-treesitter/nvim-treesitter",
		build = ":TSUpdate",
		event = { "BufReadPost", "BufNewFile" },
		opts = {
			ensure_installed = {
				"clojure",
			},
			highlight = {
				enable = true,
			},
			rainbow = {
				enable = true,
			},
		},
		config = function(_, opts)
			if type(opts.ensure_installed) == "table" then
				---@type table<string, boolean>
				local added = {}
				opts.ensure_installed = vim.tbl_filter(function(lang)
					if added[lang] then
						return false
					end
					added[lang] = true
					return true
				end, opts.ensure_installed)
			end
			require("nvim-treesitter.configs").setup(opts)
		end,
	},
	"HiPhish/nvim-ts-rainbow2",
	{
		"neovim/nvim-lspconfig",
		event = { "BufReadPre", "BufNewFile" },
		dependencies = {
			"mason.nvim",
			"williamboman/mason-lspconfig.nvim",
		},
		autoformat = true,
		config = function(_, opts)
			require("lspconfig").clojure_lsp.setup(opts)
		end,
	},
	{
		"williamboman/mason.nvim",
		cmd = "Mason",
		config = true,
	},
	{
		"williamboman/mason-lspconfig.nvim",
		opts = {
			ensure_installed = { "clojure_lsp" },
		},
	},
})

minimal.clj
(defn a-func [x y]
  (let [a 1]
  (let [b 2]
  (+ a b x y))))

(a-func 2 3)

deps.edn
{
 :paths ["."]
 }

It's possible this may be the same bug as this in which case the linked fixes might help?

I can replicate the issue in Fennel with fnlfmt as the formatter. It looks like the extmarks are are not being moved because the old highlighting stays in place, painting the wrong characters. You should be able to make it go away by executing :edit on the buffer. I will have to investigate what is going on.

Cool. Thanks for the workaround. I'll give it a shot. Let me know if there's anything else I can do to help narrow it down.
Also fyi the issue also occurs with the local strategy sometimes leading to some pretty dramatic highlighting issues.

It looks like a Neovim issue with extmarks. Here is a test which does not involve Tree-sitter, rainbows or LSP at all. Disable the rainbow plugin, then take this Snippet of Fennel code:
(fn foo []
  (if (and (= :a :a)
        (not= :a :b))
      (print "It just works!")))

Now let's create a new namespace and add highlighting to the second to last closing parenthesis on the third line.
let g:nsid = nvim_create_namespace('derp')
call nvim_buf_add_highlight(0, nsid, 'WarningMsg', 2, 19, 20)

This will highlight the parenthesis (yellow in my case). Now shell out to fnlfmt to format the entire buffer. This assumes we have the compiled fnlfmt program in the same directory.
!./fnlfmt --fix %

This is the result:
(fn foo []
  (if (and (= :a :a) (not= :a :b))
      (print "It just works!")))

Now the s in just should be highlighted yellow because it is where the parenthesis used to be.
I don't know if this is even a bug or if it works as intended. Extmarks move with their text, but this is not really a case of text moving. Should the extmark have been deleted?
It raises questions for this plugin too. Should I delete extmarks in the changed region? What if one delimiter of the pair is inside the formatted region and one is outside? I don't know, I will have to think about this.

Quick heads-up: it looks like the parser is parsing the old buffer from before it was formatted. Or at least it is applying the highlighting to the old positions. This then puts extmarks in weird places, causing the highlighting to be wrong. I have not yet been able to figure out why this is happening.
There is another issue: callbacks don't get cleared for a parser and multiple callbacks can start building up if the buffer is edited multiple times. Maybe this is part of the problem? I'll try and see if I can fix this problem upstream in Neovim.

I have mixed news, both good and bad.
Let's start with the bad: I don't think this issue can be fixed in this plugin at all. This plugin is implemented as a module for nvim-treesitter and it relies on nvim-treesitter to register (attach) and unregister (detach) autocommands for buffers. Unfortunately the nvim-treesitter way seems to be what is causing this problem and a number of other problems as well.
The mixed news is that nvim-treesitter has deprecated the module system and will remove it in the future (source). This means that sooner or later I will have to fork this plugin yet again and make a new standalone plugin anyway.
As for the good news, I have had my first success. The new version does not suffer from lingering extmarks, but it still does not insert new ones. There is still some fine tuning to be done to get the autocommands correct and make the configuration work without nvim-treesitter. I will be able to reuse most of my existing code, so the amount of effort needed will be reasonable.
The configuration will have to be done differently, that's why I want to make it a new standalone plugin and leave this one here for legacy. The configuration format will be the same, but it will have to be specified somewhere else now.

Wow. Thanks for the updates. If you need beta testers for the new version I'm happy to give it a spin.
I appreciate all your work. It would be a sad world without a good rainbow parens plugin.

The new rainbow plugin is here: rainbow-delimiters.nvim (GitHub mirror). Please file an issue over there if the problem still persists.


infinite loop when comparing state.__pc and gadgetAddress

state.__pc and gadgetAddress will never be equal and causes infinite loop in iOS12.

I also encountered this problem


[PLUGIN] Accordion

Is this a unique plugin request?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Proposed Plugin Solution
Implement Accordion Plugin
Screenshots
No response
Do you want to work on this issue?
Yes
If "yes" to the above, please explain how you would technically implement this (issue will not be assigned if this is skipped)
No response

Related https://github.com/Darginec05/Yoopta-Editor/issues/131

Done in v4.5.0
Examples - https://yoopta.dev/examples/withBaseFullSetup


Adding two binary numbers

🚀 Feature
(A clear and concise description of what the feature is.)
Have you read the Contributing Guidelines on Pull Requests?
yes
Motivation
This is a must do question, we add two binary numbers, which involves a hole lot of implementations as well.
Pitch
This is a must do question, we add two binary numbers, which involves a hole lot of implementations as well.

I am a gssoc21 participant and want to contribute to this in cpp.

/assign


Plan DIDComm Transports

https://github.com/roots-id/rootswallet/wiki/Roots-Wallet-Implementation-Proposal#layer-2-didcomm-peer-to-peer-protocol

We are making significant progress on DIDComm.  @rodolfomiranda will be generating specific tickets as we progress beyond https


File System

There needs to be a File system abstraction that allows the following functionality:

Manipulating file names and paths
opening, closing, and read/write of individual files
Async file IO for streaming

A good reference for how to manage this is in Game Engine Architecture chapter 7

Added support for some basic file system functionality, which is probably all that we will need.


Feature/2231 Improve Eligibility Report functionality

What's included?
Closes #2231.

Fix the "Recommendation" dropdown and "Reason for recommendation" textbox.

Remove "Reject Non-declaration" from the issue status options.

Add a checkbox at the top with the text "Display only candidates with Eligibility issues". This checkbox toggles between all candidates being displayed and just the ones that don't meet some or all of the criteria.

Add a "Download SCC Annex report link" at the top of the page. The functionality will be implemented in #2300.

Show the information as shown below with Met / Not met returned for each criterion (not met would require further manual consideration by the SET).

Note: This PR requires changes in digital-platform: Feature/admin 2231 Eligibility Report.
Who should test?
✅ Product owner
✅ Developers
✅ UTG
How to test?
Example exercises:

Legal exercise:
https://jac-admin-develop--pr2332-feature-2231-improve-hpvso0a0.web.app/exercise/pmhY2nNEQFg4OXzSDPYw/reports/eligibility-issues
Non-legal exercise:
https://jac-admin-develop--pr2332-feature-2231-improve-hpvso0a0.web.app/exercise/mof6mdC6grPuF8xQYwUA/reports/eligibility-issues

Check if the "Recommendation" dropdown and "Reason for recommendation" textbox work properly.
Check if "Reject Non-declaration" has been removed from the issue status options.
Check if the checkbox "Display only candidates with Eligibility issues" works properly.
Check if each application's information with Met/Not Met shows correctly.

Risk - how likely is this to impact other areas?
🟢 No risk - this is a self-contained piece of work
Additional context
Include screen grabs, video demo, notes etc.
Related permissions
Have permissions been considered for this functionality?

No permission changes required
Permissions have been added / updated. Details:

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

@HalcyonJAC This is looking really good, Ryan. I've been playing around with data in a couple of the apps. Under Shakin' Stevens, the PJE (sitting days) appears to work, but with Jennifer Saunders I have played around with the two entries; they now total 60 sitting days, but PJE is flagged as met even though 300 days are required.
I'm also wondering about on Non-legal exercises, if no professional registrations are requested, would those candidates disappear when the 'Display only..' tickbox was ticked?

@HalcyonJAC Another thought - suggest we need a disclaimer on the PQE calculations; candidate inputs only month and year, but we are calculating years, months, days. Obviously there are potential inaccuracies there, even if we fed back years and months only. For example, I'm assuming if a candidate entered, e.g. March 2001 - April 2001, the platform would calculate this as 31 + 30 = 61 days. Thoughts? What's the maximum range of error? Should we consider some kind of flag for candidates that are very close to the required PQE?

@HalcyonJAC This is looking really good, Ryan. I've been playing around with data in a couple of the apps. Under Shakin' Stevens, the PJE (sitting days) appears to work, but with Jennifer Saunders I have played around with the two entries; they now total 60 sitting days, but PJE is flagged as met even though 300 days are required.
I'm also wondering about on Non-legal exercises, if no professional registrations are requested, would those candidates disappear when the 'Display only..' tickbox was ticked?

I have redeployed the cloud function. Could you test it again, please?
For now the checkbox "Display only candidates with Eligibility issues" will not take "Professional Registration" into account. Hope it makes sense.

@HalcyonJAC On the first point, I think it's quite a good illustration of my second point above about how the PQE is calculated, in this case it is Jan 2007-Dec 2008 and Jan 2009-Dec 2009. That is being presented as 2y10m. I'm not saying that's wrong, but it could also be displayed as 3y, do you see what I mean? Can you clarify how the calculation is done pls?
On point 2, it seems obvious, but can you confirm that it only applies to non-legal exercises?

@nickaddy Here is an example of how the range will be calculated:

Jan 2007-Dec 2008: the range is calculated from 2007-01-01 00:00:00 to 2008-12-01 00:00:00 which is 1 year and 11 months.
Jan 2009-Dec 2009: the range is calculated from 2009-01-01 00:00:00 to 2009-12-01 00:00:00 which is 11 months.
So the total length is 2 years and 10 months.

On point 2, the filter only applies to non-legal exercises.

@HalcyonJAC On this basis, it seems that the maximum 'extra days' the platform could calculate would be 1 month, e.g. candidate states January as a start date but actually started on 31 January; we don't need to worry about 'under-calculation' on the end date. As a solution, could we subtract 1 month from the total calculated before feeding it into the Met/Not met calculation? Or just have the platform flag as Not Met if it is within 1 month of the PQE requirement? In reality, this will happen in a minority of cases, but we should cover all bases to give SETs confidence in the feature.

@nickaddy Yes, I could subtract 1 month from the total calculated before feeding it into the Met/Not met calculation.


Check Angular-translate versions compatibility

Subject of the issue
How can I know which is the latest angular-translate version that works with AngularJS 1.0.7? Where can I check compatibilities?
Your environment
Please read this stackoverflow post for details: https://stackoverflow.com/questions/50245390/performance-issues-in-angularjs-1-0-7-and-angular-translate
Steps to reproduce
Tell us how to reproduce this issue. Please provide a working demo, you can use this template as a base.
Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead

@samal84  is right, that is obviously out-of-scope.
We are still running our test suite against different setups (local bower environment, scopes defined via bower.json files), see here https://github.com/angular-translate/angular-translate/tree/ee641d8f2c586364c48507af617a033ca33f1a1e/test_scopes Of course, only covered things within tests. There are obvious changes between 1.2 and 1.6.
For 1.7: I've pushed 080cd04, but we will what happened.

For now I am unblocked by using the Selective dependency resolutions feature of Yarn, and just added this to package.json:
"resolutions": {
  "angular": "1.7.0"
}

I'm not seeing any issues with angular-translate 2.17.1 when forcing it to use angular 1.7.0 like this.

Yeah, the tests have been run successfully too. Stick with this until released.

Let's summarize:

We have 2018, we don't care about AJS 1.0.7.
Support for AJS 1.7.0 is in<EMAIL_ADDRESS>officially (but 2.17 works also)

you should also look at bower-angular-translate-storage-cookie, there is still a wrong dependency in bower.json:
"angular-cookies": ">=1.2.26 <1.7"

Arh, sorry about that. 2.18.1 is your friend.


refactor prototype datasets to inherit from IterDataPipe

This adds initial support for simplifying our prototype dataset, by letting them inherit directly from IterDataPipe.

Blocked by pytorch/data#237.

As discussed offline with @NicolasHug , we are going to keep the changes in a feature branch for now and send PRs against that. This way we can still go one dataset at a time without worrying about the intermediate phase were some datasets are ported and some are not.


Are there any webhook registration limits for criteria?

Hi! We're looking to using webhooks for our project.
The webhook registration details registering criteria for subscription, such as players and teams.
Are there any limits to the amount of criteria you can add to a particular webhook?
For example, would it be possible to add hundreds OR thousands of player ids for a single subscription?
Or is this something limited to 'Premium Tier' access.
Thanks so much!

I wouldn't advise using webhooks except for hobby projects. It's no longer well maintained/supported. However, I don't think we have set limits on how many things you can subscribe to.

I see. However the API is still being maintained I presume?
Does odota expose any kind of webhook for parsed matches or is the only other alternative to poll the API through the players endpoint to get their matches?

If you're only looking for parsed matches, there's a "parsedMatches" endpoint that only lists IDs of recently parsed matches. However most of these would not be matches of the players you're interested in.


[$250] Xero - Xero action modal appears below the button and overlaps with the composer

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.8-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/44170
Action Performed:
Precondition:

Workspace is connected to Xero
Delayed submission is enabled in Workflows

Go to staging.new.expensify.com
Go to workspace chat
Submit an expense
In the main chat, click on the pay elsewhere dropdown.the composer.
Pay elsewhere the report
Click on the Xero action dropdown

Expected Result:
Xero action modal will appear above the button and does not overlap with the composer
Actual Result:
Xero action modal appears below the button and overlaps with the composer
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/c20d38e5-f41b-46b0-8c69-384acca67b66
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019da83e276a5e9487
Upwork Job ID:<PHONE_NUMBER>967508952
Last Price Increase: 2024-07-18

Issue OwnerCurrent Issue Owner: @aimane-chnaif

We think that this bug might be related to #wave-collect - Release 2

@arosiclair do you agree this isn't a blocker?
I agree the popover should be in the same place as the pay selector options, so let's fix it. Also, we should be using circle Xero avatars in the popover, not squares. Just like QBO:

Certainly not a web blocker, removing that label minimally now.

@arosiclair do you agree this isn't a blocker?

Yeah this looks pretty minor to me I agree it shouldn't be a blocker.

Proposal looks okay though having the dropdown always open upward feels off to me tbh. Also you could argue that the dropdown covering the preview is just as bad (or even worse) than covering the composer.
Played around with Slack and it looks like it only opens the context menu upwards when there isn't enough space below regardless of the composer:

Down
Up

So I'm starting to think the dropdown covering the composer isn't really a problem and we really should just be worried about it not fitting on screen (which I think isn't an issue yet). What do you think @trjExpensify?

So I'm starting to think the dropdown covering the composer isn't really a problem and we really should just be worried about it not fitting on screen (which I think isn't an issue yet). What do you think @trjExpensify?

I think it should be consistent with the Pay button in the same preview component, so I'd say stick with how this menu opens today for that (which is upwards). If we want to make a change, we should do it overall, and I don't think it's worth the effort to revisit this one now. So I'd just fix the consistency issue with the export menu. 👍

Fair enough. We can look at it another time 👍

@nyomanjyotisa look forward to a PR for this. Can you make sure to change those Xero icons to circles instead of squares to match QBO in the PR? Thanks!

Cool, thanks! CC: @dannymcclain for vis just in case that intentionally changed somewhere.

Nope, they're supposed to be circles. We just like to upload the assets to the repo as squares and then do the rounding with code (in case we ever want to change them from circles to rounded squares or adjust the border radius or something).

Perf, thanks for confirming!

I think we're splitting hairs on requesting a bounty increase for this issue. The position of the popover was wrong appearing over the composer, and we needed to put it back to where it should be. Switching XeroSquare for XeroCircle is also so incredibly minor. Overall, $250 is already a generous bounty for a +29 line PR.

Payouts due:

[x] Contributor: $250 @nyomanjyotisa automatic offer
[ ] Contributor+: $250 @aimane-chnaif via NewDot

Upwork job is here.

@aimane-chnaif don't forget the checklist!

Thanks @aimane-chnaif! Please apply via NewDot (summary here)


Handle bootstrap errors gracefully when glasskube is not connected to the internet

Is your feature request related to a problem? Please describe.
When the user is not connected to the internet and glasskube bootstrap is used the bootstrap fails and returns a memory invalid or nil pointer deference error but instead we should print a message as glasskube is not connected to the internet. Something like this
~$ glasskube bootstrap
⚠ Glasskube is not connected to the internet, cannot bootstrap.

Describe the solution you'd like
Improve error handling for cmd/bootstrap.go
Describe alternatives you've considered
Additional context

@christophenne this could be a good-first-issue

@Baalekshan good point, but I think this is being solved already with #524, right?

No that's a different issue involving --url flag.
This one is for showing that glasskube is not connected to the internet when tried bootstrapping.This one might be status code 503 service unavailable when device is offline. #524 deals with 404 error.

@Baalekshan fair enough, good input.

Hello @christophenne. I would like to work on this issue.


Array type variable

The description on how to add a type variable in the documentation does not correspond with the screenshot provided.

@chez-charlie

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

hi @JamesJBarnett I would appreciate if you don't over look at this section of our fabric documentation, which was the intension for this pull request.
The Create activity steps given in the documentation do not reflect the steps taken in the screenshot.
That is the reason as to why I came up with something like so;


how to test nosiy images in common scenarios ?

Hi， Thank you for your efforts.
but I have question.
In your codes:
seqn = seq + noise
your input are clean data and noise which generate by (" noise = torch.empty_like(seq).normal_(mean=0, std=args['noise_sigma']).to(device)")
But common scenarios ,I got some images with noise. i dont not know the noise distribution.
how can remove image noise and get the clean images.
just like a sequences of these images

Hi,
If I understand correctly, you want to denoise photographic noise, not only Gaussian.
In this case, you can denoise your sequences with the model provided. You will probably have to test with different values for the noise map to see the one that works the best.
Else, you can retrain the model. In this case, you can choose to remove the noise map completely (blind denoising). A second option would be to use a noise estimator, and use its output as noise map.
Hope this helps

Hi,
If I understand correctly, you want to denoise photographic noise, not only Gaussian.
In this case, you can denoise your sequences with the model provided. You will probably have to test with different values for the noise map to see the one that works the best.
Else, you can retrain the model. In this case, you can choose to remove the noise map completely (blind denoising). A second option would be to use a noise estimator, and use its output as noise map.
Hope this helps

Hi,
If I understand correctly, you want to denoise photographic noise, not only Gaussian.
In this case, you can denoise your sequences with the model provided. You will probably have to test with different values for the noise map to see the one that works the best.
Else, you can retrain the model. In this case, you can choose to remove the noise map completely (blind denoising). A second option would be to use a noise estimator, and use its output as noise map.
Hope this helps

Hello @m-tassano, I am wondering what kind of noise estimator (network) will you used to get a noise map on real noisy image ? I'm working on fluorescent microscopy I have sequence of noisy images and also the sequence of the pseudo non-noisy images, but I dont have the noise map, cause it is a real noise. Thanls

Hi @Scienceseb , I have not implemented that for fastdvdnet, but I would probably start testing the one proposed with CBDNet. Hope this helps


Consecutive terraform plan always show changes to Lambda function

Description
Every time a terraform plan is run, the filename of the Lambda function is changing. "null_resource" "archive" is different every time, and thus the lambda function is destroyed and recreated on every apply.
I think the issue is related to executing this module twice in the same config. According to the docs for the Lambda module, a collision can occur if you do that and so there's an extra_hash you can add, but I don't see that as an input for this module.
⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Terraform:
v1.1.2
Provider(s):

provider registry.terraform.io/hashicorp/aws v3.71.0
provider registry.terraform.io/hashicorp/external v2.2.0
provider registry.terraform.io/hashicorp/local v2.1.0
provider registry.terraform.io/hashicorp/null v3.1.0

Module:

Reproduction

Steps to reproduce the behavior:

Define this module twice in the same config. You will need to override the lambda_function_name otherwise there will be a collision.

No

Yes

Apply once successfully
Change nothing, immediately run terraform plan and see that several resources are proposed to be replaced.

Code Snippet to Reproduce

Expected behavior
I would expect that a successful apply and subsequent plan would not have any changes
Actual behavior
What is actually happening is that 6 resources are being proposed to be added, 5 changed, and 6 destroyed.
Terminal Output Screenshot(s)

Additional context

You will need to override the lambda_function_name otherwise there will be a collision.

lambda_function_name should be unique to avoid the behavior you are seeing. I can't imagine a better workaround for this, yet.
Closing this issue.

lambda_function_name is unique, but the behavior is as I described


White-list Twitter accounts

hello,
The Chrome extension thinks some genuine Twitter accounts are fake or scam.
Is there a way to create a whitelist with accounts to make sure the extension doesn't flag genuine tweets as scam?

Hi - yep, see https://github.com/409H/EtherSecurityLookup/blob/master/README.md#whitelistingblacklisting

Go to http://gettwitterid.com/ and type your Twitter handle .
Copy your ID.
Create a PR in lists/twitter.whitelist.json following the same structure.

    "id": "twitter_handle"

Alternatively, just open an issue with the handle you want whitelisting

Hmm ok....
A simple right click to add to whitelist is not an option? :-)


Show new pets

Closing without merging as our consumer-driven workflow has changed since this PR was first created.  Will create another PR which is similar.


version push

Description
GitHub Issues

Coverage remained the same at 91.434% when pulling 99be83dbacf3c7a80ac36cc5b60b4c0f4b69d3fb on mcrowson:pushing-0.0.12 into 549f422ed127a4d7726010b50eb52cf993d19289 on refindlyllc:development.


How to mount bucket subpath?

Why is it not possible to mount bucket subpath?
Does datashim support this operation?

@Trainbow closing because this is a duplicate of #289


About visualization

Thank you for share your work.I have followed the instruction and gained the result but I met  some question in visualization steps,I don't know what the label path is,can you give me some advice?Thanks a lot.

Sorry for the late response. The label_path is where the output predictions (ply files) saved. For S3DIS, it's './test/s3dis_preds/'.

Thank you for your replying,I will try your advise


Create - 2023-11-01-KANDYKORN.md

/factcheck

@evgenydmitriev  I believe I have made all requested changes. Please let me know if I missed something.

@aholman0725, here are some reason this and other pull requests from you are not being reviewed:

You are adding multiple commits and comments in succession, so it's hard to tell when something is a work in progress and when you want us to review the work. Advice: read up on the pull request process and use draft feature.
You are leaving reviewers' comments unaddressed. Example. Make sure you mark conversations as resolved if you addressed the comment. Also, make sure you address mentioned issues in other open pull requests you have opened.
Grammar, punctuation, style, best practices. I already mentioned this multiple times, you either need to pay attention to those yourself, or ask GPT-4 or Bard to help you.
In this specific pull request, I see problems with style, markdown usage, as well as following our guidelines on target entities and losses. This wiki is focused on critical blockchain ecosystem attacks, not exploits of individual developers with no losses.

Closing this PR and giving it another week for other pull requests to be brought up to the level where they can be merged.

@evgenydmitriev Thank you for the feedback. I understand the commit issue. Would it be easier if I started the current pull requests from scratch? I'll make sure to use the draft feature moving forward. ChatGPT4 is currently unavailable. Is there another bot you suggest using? I'll address the other issues as well. I appreciate your patience and assistance.


Heading formatting broken

Configure an override doesn't show up in the navigation, I believe because the headings are incorrect.

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

ID: e1cea280-89dd-b421-0ca9-da8fc972610d
Version Independent ID: 746905a4-0549-449c-6842-d365fb7da05d
Content: Manage host networking with Network ATC - Azure Stack HCI
Content Source: azure-stack/hci/manage/manage-network-atc.md
Service: azure-stack
Sub-service: azure-stack-hci
GitHub Login: @v-dasis
Microsoft Alias: v-dasis

@jacobpedd, thank you for your feedback!  I want to make sure I'm looking at the right thing.  I currently see "Manage Network ATC" in the navigation to the left which matches the title, "Manage host networking using Network ATC".  "Configure an override" appears below as a heading and can be linked to (note the url in the screenshot.  Did I miss anything?

@jacobpedd, there was an update to the docs yesterday if I'm not mistaken.  Are you looking at the live version of this document?

@jacobpedd We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


feat: allow non-yaml $refs in scalars.

For now, only in 'description' field. However, this can be updated by adding 'resolvableScalars' field to the type definition for which the user wants to allow such scalars. 'resolvableScalars' is an array which must contain a list of scalars of the type for which $refs are allowed.
Now, we allow only file system references. However, it might make sense to allow also document-based $refs, like 'some/other/file.yaml#/some/node', although it will increase complexity of this module significantly.

This PR has to be rewritten so it uses generic resolveNode function which has support for http and filesystem refs as well as transitive or local ref.
I have other notes about this PR but merging it for now as it solves the main use case.


Window buttons on mac cause python/pyglet to crash

Description: The window buttons (close, minimize, full screen) cause python to crash in various ways. (Segmentation fault, Recursion Error, Abort Trap, Illegal Instruction).
Pyglet version(s): 1.4.1
Python version(s): 3.7
OS: OS X 10.14.5 (Seems to only affect newer Macbooks)
Related tickets: bitbucket #199 bitbucket #229
Minimal example:
import pyglet

pyglet.window.Window(resizable=True)

pyglet.app.run()

Output 1 (close):
$ python3 example.py
Traceback (most recent call last):
  File "_ctypes/callbacks.c", line 232, in 'calling callback function'
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1120, in objc_method
    args = convert_method_arguments(encoding, args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1000, in convert_method_arguments
    new_args.append(ObjCInstance(a))
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 921, in __new__
    if not isinstance(object_ptr, c_void_p):
RecursionError: maximum recursion depth exceeded while calling a Python object
Segmentation fault: 11
$

Output 2 (minimize):
$ python3 example.py
Traceback (most recent call last):
  File "_ctypes/callbacks.c", line 232, in 'calling callback function'
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1120, in objc_method
    args = convert_method_arguments(encoding, args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1000, in convert_method_arguments
    new_args.append(ObjCInstance(a))
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 921, in __new__
    if not isinstance(object_ptr, c_void_p):
RecursionError: maximum recursion depth exceeded while calling a Python object
Illegal instruction: 4
$

I cannot reproduce this on my macbook at least (10.14.5 Mojave).
Tested with:
# Current master
-e git+https://github.com/pyglet/pyglet.git@32e6bdf578db062c11ec4599365896e89f44ee32#egg=pyglet
# From PyPI
pyglet==1.4.1
# Python version
Python 3.7.3

@einarf Thanks for testing. For me, it happens consistently, on my new macbook, but I didn't use to have the problem on the old one. I'll test more and try to narrow it down.

Weird. It's at least conistent considering I did my tests on a 2013 macbook pro (old stuff!) :)

@einarf Your Mac is old but should still be 64 bit, right?

Definitely. Pretty decent specs still
Processor Name:              Intel Core i7
Processor Speed:             2,7 GHz
Number of Processors:	     1
Total Number of Cores:       4
L2 Cache (per Core):         256 KB
L3 Cache:                    6 MB
Hyper-Threading Technology:  Enabled
Memory:	                     16 GB
Graphics/Display:            NVIDIA GeForce GT 650M

I was able to reproduce this issue on my 2018 Macbook Pro 15 inch w/ touchbar. Got the same output as @olehermanse when minimizing and closing. Also, manually resizing the window by dragging causes a segfault with the same output as closing.
Pyglet version(s): 1.4.1
Python version(s): 3.7.3
OS: macOS 10.14.5

We had a pretty long session last night trying to find the root cause. The newer macbooks are indeed sending different data in some events and the current cocoa wrapper is not equipped to deal with it. @olehermanse is testing out PyCocoa now (more recent project).

@Berstad Thank you for reproducing.
cocoa-python and PyCocoa are not as compatible as I'd hoped, and their commit history is not great, but I'm still working on it.

In an effort to understand the problem better, I tried printing stack traces:
Decorator stacktrace:
  File "examples/hello_world.py", line 4, in <module>
    window = pyglet.window.Window()
  File "/usr/local/lib/python3.7/site-packages/pyglet/__init__.py", line 359, in __getattr__
    __import__(import_name)
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/usr/local/lib/python3.7/site-packages/pyglet/window/__init__.py", line 1873, in <module>
    from pyglet.window.cocoa import CocoaWindow as Window
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/usr/local/lib/python3.7/site-packages/pyglet/window/cocoa/__init__.py", line 60, in <module>
    from .pyglet_window import PygletWindow, PygletToolWindow
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/usr/local/lib/python3.7/site-packages/pyglet/window/cocoa/pyglet_window.py", line 44, in <module>
    class PygletWindow_Implementation(object):
  File "/usr/local/lib/python3.7/site-packages/pyglet/window/cocoa/pyglet_window.py", line 63, in PygletWindow_Implementation
    @PygletWindow.method(b'@'+NSUIntegerEncoding+b'@@B')
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1115, in method
    tb = ''.join(traceback.format_stack())

(Last line is the traceback I added)
Recursive stack trace:
  File "examples/hello_world.py", line 20, in <module>
    pyglet.app.run()
  File "/usr/local/lib/python3.7/site-packages/pyglet/app/__init__.py", line 144, in run
    event_loop.run()
  File "/usr/local/lib/python3.7/site-packages/pyglet/app/base.py", line 175, in run
    self._run()
  File "/usr/local/lib/python3.7/site-packages/pyglet/app/base.py", line 188, in _run
    platform_event_loop.step(timeout)
  File "/usr/local/lib/python3.7/site-packages/pyglet/app/cocoa.py", line 153, in step
    self.NSApp.sendEvent_(event)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 785, in __call__
    return self.method(self.objc_id, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 755, in __call__
    result = f(objc_id, self.selector, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1132, in objc_method
    result = f(py_self, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/window/cocoa/pyglet_window.py", line 72, in nextEventMatchingMask_untilDate_inMode_dequeue_
    mask, date, mode, dequeue, argtypes=[NSUInteger, c_void_p, c_void_p, c_bool])
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 499, in send_super
    result = objc.objc_msgSendSuper(byref(super_struct), selector, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1132, in objc_method
    result = f(py_self, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/window/cocoa/pyglet_window.py", line 72, in nextEventMatchingMask_untilDate_inMode_dequeue_
    mask, date, mode, dequeue, argtypes=[NSUInteger, c_void_p, c_void_p, c_bool])
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 499, in send_super
    result = objc.objc_msgSendSuper(byref(super_struct), selector, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1132, in objc_method
    result = f(py_self, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/window/cocoa/pyglet_window.py", line 72, in nextEventMatchingMask_untilDate_inMode_dequeue_
    mask, date, mode, dequeue, argtypes=[NSUInteger, c_void_p, c_void_p, c_bool])
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 499, in send_super
    result = objc.objc_msgSendSuper(byref(super_struct), selector, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1132, in objc_method
    result = f(py_self, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/window/cocoa/pyglet_window.py", line 72, in nextEventMatchingMask_untilDate_inMode_dequeue_
    mask, date, mode, dequeue, argtypes=[NSUInteger, c_void_p, c_void_p, c_bool])
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 499, in send_super
    result = objc.objc_msgSendSuper(byref(super_struct), selector, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1132, in objc_method
    result = f(py_self, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/window/cocoa/pyglet_window.py", line 72, in nextEventMatchingMask_untilDate_inMode_dequeue_
    mask, date, mode, dequeue, argtypes=[NSUInteger, c_void_p, c_void_p, c_bool])
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 499, in send_super
    result = objc.objc_msgSendSuper(byref(super_struct), selector, *args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1132, in objc_method

I guess the best way to focus is to understand:

nextEventMatchingMask
PygletWindow method decorator, as invoked here

It's worth noting that in the "normal" crash:
Traceback (most recent call last):
  File "_ctypes/callbacks.c", line 232, in 'calling callback function'
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1120, in objc_method
    args = convert_method_arguments(encoding, args)
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 1000, in convert_method_arguments
    new_args.append(ObjCInstance(a))
  File "/usr/local/lib/python3.7/site-packages/pyglet/libs/darwin/cocoapy/runtime.py", line 921, in __new__
    if not isinstance(object_ptr, c_void_p):
RecursionError: maximum recursion depth exceeded while calling a Python object
Segmentation fault: 11
olehermanse@OHMBP pyglet $

The last line, if not isinstance(object_ptr, c_void_p):, is not that important, it's kind of arbitrary. I was confused why isinstance would recurse / throw a recursion error, but it's just a simple function, the recursionerror is thrown because the stack is exhausted, it can't call any other function, this just happens to be the function attempted.

Right, that's makes sense.

By adding some more prints, I understand a little bit more:
calling nextEventMatchingMask(<ObjCInstance 0x109c96910: b'PygletWindow' at<PHONE_NUMBER>64016>,<PHONE_NUMBER>4, <ObjCInstance 0x10b44c550: b'__NSDate' at<PHONE_NUMBER>64336>, <ObjCInstance 0x10b44c990: b'__NSCFConstantString' at<PHONE_NUMBER>25224>, True)
receiver: <ObjCInstance 0x109c96910: b'PygletWindow' at<PHONE_NUMBER>64016>
selName: nextEventMatchingMask:untilDate:inMode:dequeue:
superclass: c_void_p(140253882060592)
super_struct: <pyglet.libs.darwin.cocoapy.runtime.OBJC_SUPER object at 0x10b44fd40>
selector: c_void_p(140734157906392)
restype: <class 'ctypes.c_void_p'>
argtypes: [<class 'ctypes.c_ulong'>, <class 'ctypes.c_void_p'>, <class 'ctypes.c_void_p'>, <class 'ctypes.c_bool'>]

calling nextEventMatchingMask(<ObjCInstance 0x109c96910: b'PygletWindow' at<PHONE_NUMBER>64016>,<PHONE_NUMBER>4, <ObjCInstance 0x10b44c550: b'__NSDate' at<PHONE_NUMBER>64336>, <ObjCInstance 0x10b44c990: b'__NSCFConstantString' at<PHONE_NUMBER>25224>, True)
receiver: <ObjCInstance 0x109c96910: b'PygletWindow' at<PHONE_NUMBER>64016>
selName: nextEventMatchingMask:untilDate:inMode:dequeue:
superclass: c_void_p(140253882060592)
super_struct: <pyglet.libs.darwin.cocoapy.runtime.OBJC_SUPER object at 0x10b44fdd0>
selector: c_void_p(140734157906392)
restype: <class 'ctypes.c_void_p'>
argtypes: [<class 'ctypes.c_ulong'>, <class 'ctypes.c_void_p'>, <class 'ctypes.c_void_p'>, <class 'ctypes.c_bool'>]

When you click the close button, PygletWindow.nextEventMatchingMask_untilDate_inMode_dequeue_ is called. This has some special logic for handling resize, and then it tries to call the same method in superclass - NSWindow. The function used to call a method in superclass is called send_super. There seems to be a bug in send_super (or the data it uses), it instead calls the method in PygletWindow again, causing infinite recursion. If I add a print statement after the send_super call I can see that none of them terminate, they just keep calling the method in the subclass (PygletWindow).

This comment is also interesting:
    # When the window is being resized, it enters into a mini event loop that
    # only looks at mouseDragged and mouseUp events, blocking everything else.
    # Among other things, this makes it impossible to run an NSTimer to call the
    # idle() function in order to update the view during the resize.  So we
    # override this method, called by the resizing event loop, and call the
    # idle() function from here.  This *almost* works.  I can't figure out what
    # is happening at the very beginning of a resize event.  The NSView's
    # viewWillStartLiveResize method is called and then nothing happens until
    # the mouse is dragged.  I think NSApplication's nextEventMatchingMask_etc
    # method is being called instead of this one.  I don't really feel like
    # subclassing NSApplication just to fix this.  Also, to prevent white flashes
    # while resizing, we must also call idle() from the view's reshape method.

It seems the author didn't fully understand why sometimes a different (class) method was called than the intended.

Is it possible to prevent the crash just by disabling the super call, for a start?

Just commenting out the send_super part cause the program to hang forever.
Commenting out the whole _nextEventMatchingMask method fixes the close button, but resize is still broken.
Commenting out both PygletView.setFrameSize_, as well as PygletWindow.nextEventMatchingMask_untilDate_inMode_dequeue_ works surprisingly well. Both closing and resizing works, except on_resize event's are not dispatched, so all my resize logic doesn't work. (But the NSView actually resizes).

Ahh interesting! After resizing, are you able to manually query the window and framebuffer sizes?

FWIW I had the issue reported here with the system Python 2.7.9 but I just installed latest Python 2.7.16 (I'm testing out an old game that's not py3k compatible) and no longer have the issue.
I install using pyenv, FWIW.

Ahh interesting! After resizing, are you able to manually query the window and framebuffer sizes?

@benmoran56 I guess that would be possible, but then we don't get the events, we would have to poll for the changes, which is not a great fix, IMO.

I forked cocoa-python here:
https://github.com/olehermanse/cocoa-python
And reproduced the issue, with a quite small example:
# Example of using ctypes with Cocoa to create an NSWindow with
# an application menu item for quitting.

import sys
from cocoapy import *

NSWindow = ObjCClass('NSWindow')
NSApplication = ObjCClass('NSApplication')
NSMenu = ObjCClass('NSMenu')
NSMenuItem = ObjCClass('NSMenuItem')
NSAutoreleasePool = ObjCClass('NSAutoreleasePool')

class MyWindow_Implementation(object):
    MyWindow = ObjCSubclass('NSWindow', 'MyWindow')

    @MyWindow.method(b'@'+NSUIntegerEncoding+b'@@B')
    def nextEventMatchingMask_untilDate_inMode_dequeue_(self, mask, date, mode, dequeue):

        print(f"nextEventMatchingMask({mask}, {date}, {mode}, {dequeue})")

        event = send_super(self, 'nextEventMatchingMask:untilDate:inMode:dequeue:',
                           mask, date, mode, dequeue, argtypes=[c_uint, c_void_p, c_void_p, c_bool])
        print("send_super returned")

        if event.value is None:
            ret = 0
        else:
            ret = event.value
        print("return value: " + str(ret))
        return ret

MyWindow = ObjCClass('MyWindow')

def create_window():
    print('creating window')
    frame = NSMakeRect(100, 100, 300, 300)
    window = MyWindow.alloc().initWithContentRect_styleMask_backing_defer_(
        frame,
        NSTitledWindowMask | NSClosableWindowMask | NSMiniaturizableWindowMask | NSResizableWindowMask,
        NSBackingStoreBuffered,
        0)
    window.setTitle_(get_NSString("My Awesome Window"))
    window.makeKeyAndOrderFront_(None)
    return window

def create_autorelease_pool():
    pool = NSAutoreleasePool.alloc().init()
    return pool

def application_run():
    app = NSApplication.sharedApplication()
    create_autorelease_pool()
    create_window()
    app.run()  # never returns

######################################################################

if __name__ == '__main__':
    application_run()

https://github.com/olehermanse/cocoa-python/commit/dc536717f0dcf9f008108a882ccb0955135160eb
So, there are many other examples where send_super() from cocoa-python works, however, in this example it doesn't. When you call send_super, it seems to call MyWindow.nextEventMatchingMaskUntilDate instead of NSWindow.nextEventMatchingMaskUntilDate.
Note: I thought setFrameSize had the same problem as nextEventMatchingMaskUntilDate, but that is just because it calls nextEventMatchingMaskUntilDate internally.

I am still experiencing this issue on 3.8 and there is no pythonw with the latest version of python which is causing issues

I have no idea how Cocoa or ObjC works, but here's a dirty hack of cocoapy that makes window operations work:
def send_super(receiver, selName, *args, **kwargs):
    if hasattr(receiver, '_as_parameter_'):
        receiver = receiver._as_parameter_
    superclass = get_superclass_of_object(receiver)
    superduperclass = c_void_p(objc.class_getSuperclass(superclass))
    if superduperclass.value is not None:
        superclass = superduperclass
    super_struct = OBJC_SUPER(receiver, superclass)

Thank you very much for this.  It's been a frustrating issue since it doesn't affect everyone's Mac, so hopeful we can get confirmation from additional affected users.

Fixed in: https://github.com/pyglet/pyglet/pull/194


qt3scan: improve scan saves

Confocal scans should be saved with more information than is currently saved.
Presently we only save the measured count rates in a simple 2D numpy array.
Ideally, we should save the following per stage position

raw number of counts
count rate

into a 2D array.
Additionally, at a minimum we should save the scan range and resolution. And we could envision the usefulness of also saving other data acquisition parameters, such as clock sample rate, and number of clock samples per position.
There is also a push to save data in hdfs format - see issue #67 - which we could consider doing in this ticket.
To accomplish these goals we would need to

modify NiDaqDigitalInputRateCounter without side-effects on qt3scope and the experiment classes such that the raw counts can be returned as well as the count rate for each scan position
modify the piezoScanner so that it holds both the raw data counts and the count rate (or perhaps, equivalently, the raw data counts and the sample time, allowing for easy computation of the count rate)
ScanImage class in piezoscan.py holds the necessary data
add h5py as requirement
update piezoscan.py save data function to package data into h5py
need some kind of documentation and/or class to define our h5py data structure(?)

If we work on this issue, then Issue #67 and Issue #35 should be closed. Or Issue #67 can be re-written just to define the work to incorporate the qt3experimentmanager tool.

An alternative is not to use h5py but instead serialize to JSON and native python data types. @mfparsons314 what are the reasons to move to h5py?

Some reasons, not sure that all are valid and happy to discuss more:

I'm not sure serialization is a great option for large numerical datasets
(imagine a 3d confocal scan with a spectrum at each point-- 4d data),
without compression they may be quite large compared to binary hdf files
HDF is supposed to be faster than serialization for large datasets
I like the metadata option and tree structure of hdf, it allows one to
nicely store parameters used to acquire data associated with results data
and also to keep multiple datasets associated with the same sample in one
file, for example, although JSON may be similar, not sure if serialization
allows for a tree structure easily?
my understanding is that HDF allows you to simultaneously read one part
of a file and write to a different part, similar to a database
hdf is portable across programming languages and operating systems-- I
think probably JSON is as well so maybe that's ok, but pickle is not
Also some discussion of options in this stackexchange
https://datascience.stackexchange.com/questions/262/what-are-the-advantages-of-hdf-compared-to-alternative-formats

On Tue, Jan 31, 2023 at 5:42 PM G Adam Cox @.***> wrote:

An alternative is not to use h5py but instead serialize to JSON and native
python data types. @mfparsons314
https://urldefense.com/v3/__https://github.com/mfparsons314__;!!K-Hz7m0Vt54!k1HS6_WEOaEY5w0cuGn8Twg0qfOQY-R5YW9G-g2XnV-PxZeH3JwaMutfo7GMGPi1MJdNC1cRwsTCyB4EMby8Kw$
what are the reasons to move to h5py?
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/qt3uw/qt3-utils/issues/76*issuecomment-1411323874__;Iw!!K-Hz7m0Vt54!k1HS6_WEOaEY5w0cuGn8Twg0qfOQY-R5YW9G-g2XnV-PxZeH3JwaMutfo7GMGPi1MJdNC1cRwsTCyB6i5iynWA$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A4B4KHBDUMPPLPV56HRM5KTWVG5QBANCNFSM6AAAAAAUNDX2E4__;!!K-Hz7m0Vt54!k1HS6_WEOaEY5w0cuGn8Twg0qfOQY-R5YW9G-g2XnV-PxZeH3JwaMutfo7GMGPi1MJdNC1cRwsTCyB7qJYxOog$
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Max Parsons
(he/him)* Director, Quantum Technologies Training and Testbed (QT3)
Laboratory https://sites.google.com/uw.edu/qt3-lab/home*
University of Washington, Dept. of Electrical and Computer Engineering
(206) 543-9689

@mfparsons314  do you envision using hdf5 file format in a way that would store ALL of the data for a single sample? That is, in a single file you'll have multiple confocal scans and multiple spectra taken over the lifetime of that sample?

Not necessarily, and this could be left up to the user.  In the past, I've
used a lot of different combinations-- 1 HDF per scan, 1 HDF per day, 1 HDF
per sample.  It all works pretty well.
On Wed, Feb 1, 2023 at 9:52 AM G Adam Cox @.***> wrote:

@mfparsons314
https://urldefense.com/v3/__https://github.com/mfparsons314__;!!K-Hz7m0Vt54!gs09Qc7CTZc5jqYud6Vs4rvl3O6I2A0seUhebwhXJJ22qg1Tz6tVxzkRBQUVG7NwS_anGmRnqjNlCKQ6_bvvTg$
do you envision using hdf5 file format in a way that would store ALL of the
data for a single sample? That is, in a single file you'll have multiple
confocal scans and multiple spectra taken over the lifetime of that sample?
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/qt3uw/qt3-utils/issues/76*issuecomment-1412480829__;Iw!!K-Hz7m0Vt54!gs09Qc7CTZc5jqYud6Vs4rvl3O6I2A0seUhebwhXJJ22qg1Tz6tVxzkRBQUVG7NwS_anGmRnqjNlCKTyav56XA$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A4B4KHB7A52BNELNEGTA5O3WVKPHPANCNFSM6AAAAAAUNDX2E4__;!!K-Hz7m0Vt54!gs09Qc7CTZc5jqYud6Vs4rvl3O6I2A0seUhebwhXJJ22qg1Tz6tVxzkRBQUVG7NwS_anGmRnqjNlCKQsrpN7rw$
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Max Parsons
(he/him)* Director, Quantum Technologies Training and Testbed (QT3)
Laboratory https://sites.google.com/uw.edu/qt3-lab/home*
University of Washington, Dept. of Electrical and Computer Engineering
(206) 543-9689

This is currently in PR #81

fixed in PR #81


Cache improvements

adds tests around cache, cache-utils, and inflight-cache
solves problem with inflight cache misgenerating cacheId

Don't ask me why bumping node got that test passing.


storage - add support for blob type 'Append'

As detailed here: http://blogs.msdn.com/b/windowsazurestorage/archive/2015/04/13/introducing-azure-storage-append-blob.aspx

Isn't this issue solved? (https://godoc.org/github.com/Azure/azure-sdk-for-go/storage#BlobType)

@preetpalS Ah, right. This is contributed in #278. Closing.


Updates README to indicate correct default for intervalDelay

I was wondering why intervalDelay seemed to be triggering much faster than 1500ms, it turns out the default in the code is actually 100ms. This just corrects the README to account for this.

Good catch, thanks @kaptron !


Poll dataset

Implementation of poll_dataset in C, C++, Fortran, Python, including test case coverage for all four languages

DO NOT REVIEW FOR NOW. I will close this PR and resubmit it when the error handling PR is complete. There are only 14 files changed for this one.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@1539461). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #174   +/-   ##
==========================================
  Coverage           ?   80.69%
==========================================
  Files              ?       50
  Lines              ?     2683
  Branches           ?        0
==========================================
  Hits               ?     2165
  Misses             ?      518
  Partials           ?        0

Replacing PR with one based on error handling PR so that it's not overly difficult to review


Camera/Video save to gallery

There is currently no first party plugin to save an image or video to a phone's gallery. Also path_provider does not even provide the path. There are some third party plugins being developed for this which currently do not work for video. Can I request this be catered for by either an existing first party plug-in or a new one?
I am a new to mobile development so apologies if there is a workaround I have not spotted (I have searched SO and Flutter Github issues before posting this).

@AriCohen8 we're unlikely to add such a new plugin to Flutter's repository, instead relying on the flutter community to provide a rich set of plugins.  As such, I'm closing this issue.


Unable to receive emails.

I configured everything correctly and used the thing to check the DNS and everything (including the MX record) says "GOOD." However, it will not receive any emails. I have a proper route set up, and it simply just won't receive the emails. They are not in spam, hold, or queue. It IS accepting the emails, because there is no error when I send the email from the other side. But it is not receiving them, like they just disappear. I have no error logs or anything it is very weird.

you can try watching the smtp_server log file to see if your message even makes it to the server, you can also verify your DNS records are pointing correctly around the internet using https://whatsmydns.net


Parser chokes on line only containing spaces

I had a file that worked with other .env parsers but failed with m1/env
The issue was a blank line containing only space characters. I agree that it's not a good style but it shouldn't break the parser.

Hi @sminnee, thank you for your reply.
Could you reproduce this issue?
Could you provide the sample code and exception result?
Thanks.

Fixed with https://github.com/m1/Env/pull/12

Thanks! :-)


Added a section on incorporating HGAMs into a Bayesian framework

I just added a section of text about Bayesian approaches including HGAMs. It could use a quick read over to make sure I didn't make any drastic mistakes (I'm not really a Bayesian at heart, and I don't want to misrepresent anything).

Sounds good! Okay, I think we basically have a finished draft then. I'm going to go through and fix a couple typos, and add a couple comments seeing if we should add/remove certain things, but I think we're basically there!

Ace! I’ll take a look ASAP!

On 7 Aug 2018, at 13:10, Eric Pedersen<EMAIL_ADDRESS>wrote:
Sounds good! Okay, I think we basically have a finished draft then. I'm going to go through and fix a couple typos, and add a couple comments seeing if we should add/remove certain things, but I think we're basically there!
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or mute the thread.


Skype bug

Found a bug with Skype where the flashing notifier thing just stays even after viewing all messages and exiting out of skype completely. Seems to be linked with viewing new messages on another device before you view them on your comp

I was unable to reproduce this bug.

Any way I can submit a log and would that help if I was able to?

If you can provide exact steps to replicate the issue, then I should be able to recreate it here. But using Skype on iOS did not result in messages remaining unread.

Only just saw this. Got an Android phone and Skype on windows. I'll try to recreate it (haven't been able to recreate it intentionally) and give a step by step with version numbers

Manage to recreate it.

Get message
View message on phone
Keep skype open on phone (idk if this is mandatory or if just viewing the message is enough) for a few minutes.
Skype on PC should be orange as though there's an unread message but without the number of unread messages
Open the chat on PC
Skype Windows 7 Ultimate 64 bit :  v<IP_ADDRESS>
Skype Android v5.0 : v<IP_ADDRESS>097
I think it might be important to mention I have view > split window view enabled on my PC.

It might be an android specific thing. When I view a message on my iPhone, it is instantly marked as read on my pc.

It's marked as read on my PC, but it still thinks I have a message to view. I'll grab someone's iPhone and test it and post results

Closed. Feel free to reopen this issue once you have more information.


Crash during Report Generation

Hello,
When I try to generate a big report ( 158 findings), the generating process begin and run but didn't finish.
No error appear on the webpage but this in the log :
pwndoc-frontend    | 2023/02/16 09:36:37 [error] 29#29: *21 upstream timed out (110: Operation timed out) while reading response header from upstream, client: <IP_ADDRESS>, server: localhost, request: "GET /api/audits/63eb9bec56689f0011a2117e/generate HTTP/1.1", upstream: "https://<IP_ADDRESS>:4242/api/audits/63eb9bec56689f0011a2117e/generate", host: "<IP_ADDRESS>:8443", referrer: "https://<IP_ADDRESS>:8443/audits/63eb9bec56689f0011a2117e/general"
I don't know what happen, other reports work fine.
Moreover, I think the application work only on 1 cpu.
Best,
Alexis

Is your installation behind a proxy or something like that ?
There is a nginx configration here: https://github.com/pwndoc-ng/pwndoc-ng/blob/master/frontend/.docker/nginx.conf#L19
Can you add proxy_read_timeout 3600;  after line 19 (api section) and let me know ?
(see https://stackoverflow.com/questions/18740635/nginx-upstream-timed-out-110-connection-timed-out-while-reading-response-hea).

There is no proxy. Just docker-compose up -d --build
I add proxy_read_timeout 3600; after line 19 and this works !
Best,
Alexis

There must be error logs somewhere. Don't you have anything in your docker compose terminal ?
Also, have you tried with an empty MS Word template ? With a minimalist one (ie. with findings name only) ?

I try with the Default template of the documentation. Same...

What about minimalist and empty template? (I'd like to know if it's due to a timeout due to the amount of findings or an infinite loop on one of your funding.)
We can get in touch on Discord if you want to Zeecka#5708.

I try :

Empty -> Work
Minimalist (General information + findings name) -> Work
Default -> Crash
My template -> Crash

On a lighter Audit :

Empty -> Work
Minimalist (General information + findings name) -> Work
Default -> Work
My template -> Work

It sounds like there's something within that audit that is causing a loop - are you using any of the newer features in your template & audit?
Would you be willing to send/show your template? If we can replicate this locally it will be much easier to fix

Hi,
I had rework me template to optimize it.
After a lot of work, the app didn't crash and I have me report.
I don't know if the problem become from me old template itself or from the backend (frontend?) that have to much work to generate the template.
The default template : crach
Me old custom template : crach but have custom data

Can you make an anonymized copy of your database / findings ?
See https://github.com/pwndoc-ng/pwndoc-ng/blob/master/docs/installation.md#backup to make a copy of your DB.


[bug]: Popover doesn't work on firefox

Describe the bug
popover doesn't work on firefox browsers
Affected component/components
popover
How to reproduce
1.open app on firefox browsers
2.create popover and add trigger on button
Codesandbox/StackBlitz link
No response
Logs
No response
System Info
Ubuntu 24.04.1,firefox developer edition 133.0b3, firefox 132.0,next 14.2.15, react 18

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

@harish-x can you provide an example?

@harish-x can you provide an example?

I wanted to clarify the issue regarding the calendar popup. The original issue mentioned that the popup wasn't working, but actually, the popup itself is functioning correctly—it’s the calendar component within the popup that isn’t working in Firefox.
To illustrate this, I’ve recorded a video showing two screens side by side. The first screen (on Brave browser) shows the calendar popup working as expected, while the second screen (on Firefox) shows the calendar not displaying within the popup.
Please let me know if you need more details or have any questions!
Screencast from 2024-11-07 23-02-29.webm

Any updates here? I'm facing the same issue! The Date picker components works on chrome, however, the popover does not open on firefox (I also tried on zen browser)

Same for me, when I tried a calendar in a popover it didn't work

Same here, it's more likely to happen if it's inside a modal.

i fixed it by adding modal={true} to the Popover Component
<Popover modal={true}>
i think its related to this #1511