fix offline node tests

longhorn/longhorn#1303
Signed-off-by: Mohamed Eldafrawi<EMAIL_ADDRESS>

@meldafrawi mentioned this PR needs more test.

@yasker PR is ready for review

Need to revisit this once we moved the infra to AWS.


Crash on Windows 1903

Been trying this out, but it crashes. Been debugging a bit, looks like this part is the culprit:
$decoder = AwaitOperation ([Windows.Graphics.Imaging.BitmapDecoder]::CreateAsync($inputStream)) ([Windows.Graphics.Imaging.BitmapDecoder])

Will try to investigate further.

I installed / purchased the correct extensions in the store.

Please share the OS version, command line, and exception details.

I tried catching the exception, but the whole powershell host crashes (so not sure how to get the exception message since the host console disappears).

Command line:
.\ConvertTo-jpg.ps1 "c:\users\some path with spaces\some file\something.heic"

This script tries to catch and display any managed-code exceptions that occur while running. If something is bypassing that and/or crashing PowerShell, maybe there’s an issue with the codec or Windows configuration. Have you tried opening that image in Windows Photo viewer? Or converting it on another machine?

I can open it fine in the Photos app. Maybe I will investigate this by converting it to a C# (UWP) console app tomorrow to see why it crashes.

Great idea!

On my laptop with vs 2019, it works (as UWP). Testing script alone on this laptop.

Says "unsupported", let me check if I need the codec on this laptop as well (not only the photo viewer extension).

Also installing HEVC Video Extensions from the store.

System.Management.Automation.MethodInvocationException: Exception calling "Wait" with "0" argument(s): "One or more err
ors occurred." ---> System.AggregateException: One or more errors occurred. ---> System.UnauthorizedAccessException: Ac
cess is denied. (Exception from HRESULT: 0x80070005 (E_ACCESSDENIED))
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at CallSite.Target(Closure , CallSite , Object )
   --- End of inner exception stack trace ---
   at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception ex
ception)
   at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
At .\ConvertTo-Jpeg.ps1:103 char:13
+             throw $_.Exception.ToString()
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (System.Manageme...tedFrame frame):String) [], RuntimeException
    + FullyQualifiedErrorId : System.Management.Automation.MethodInvocationException: Exception calling "Wait" with "0
   " argument(s): "One or more errors occurred." ---> System.AggregateException: One or more errors occurred. ---> Sy
  stem.UnauthorizedAccessException: Access is denied. (Exception from HRESULT: 0x80070005 (E_ACCESSDENIED))
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at CallSite.Target(Closure , CallSite , Object )
   --- End of inner exception stack trace ---
       at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Excepti
   on exception)
   at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)

Here are the exception details in powershell:
System.Management.Automation.MethodInvocationException: Exception calling "Wait" with "0" argument(s): "One or more errors occurred." ---> System.AggregateException: One or more errors occurred. ---> System.ArgumentException: Value does not fall within the expected range.
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at CallSite.Target(Closure , CallSite , Object )
   --- End of inner exception stack trace ---
   at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
   at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)

Looks like a completely different error on this laptop, this is the code:
            try
            {
                # Get SoftwareBitmap from input file
				Write-Host 1
                $inputFile = AwaitOperation ([Windows.Storage.StorageFile]::GetFileFromPathAsync($file)) ([Windows.Storage.StorageFile])
				Write-Host 2
                $inputFolder = AwaitOperation ($inputFile.GetParentAsync()) ([Windows.Storage.StorageFolder])
				Write-Host 3
                $inputStream = AwaitOperation ($inputFile.OpenReadAsync()) ([Windows.Storage.Streams.IRandomAccessStreamWithContentType])
				Write-Host 4
                $decoder = AwaitOperation ([Windows.Graphics.Imaging.BitmapDecoder]::CreateAsync($inputStream)) ([Windows.Graphics.Imaging.BitmapDecoder])
				Write-Host 5
            }
            catch
            {
                # Ignore non-image files
                Write-Host " [Unsupported]"
				Write-Host $_.Exception

                continue
            }

Output is:
1
 [Unsupported]
System.Management.Automation.MethodInvocationException: Exception calling "Wait" with "0" argument(s): "One or more errors occurred." ---> System.AggregateException: One or more errors occurred. ---> System.ArgumentException: Value does not fall within the expected range.
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at CallSite.Target(Closure , CallSite , Object )
   --- End of inner exception stack trace ---
   at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
   at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)

On my other machine it gets to 4...

Failing after 1 above is weird - StorageFile is built-in and has nothing to do with the extra codecs. Maybe this is a Windows 1903 problem? (I know of other .NET scenarios  they broke from similar reports.) If so, I’ll find out after I upgrade because I use this script every week. Until then, sorry for the trouble! Nobody else has reported this issue so far.

No problem at all, just wanted to report this. I am using CopyTrans HEIC for Windows at the moment, which you can also call as command line.
I needed it because I was working on a video / photo diary (see https://github.com/GeertvanHorrik/DiaryGenerator). Maybe in the future, if this script works again, I will replace it again. Would save another dependency users will have to install at the moment.

The issue of failing after step 1 should be fixed by https://github.com/DavidAnson/ConvertTo-Jpeg/commit/db8f7d0fd6b448c85ca6098ce2fff341bfc0e17e.


Go To Definition broken for TypeScript 2.4.1 project

Type: Bug
With one of the recent updates the "Go To Definition" feautre is broken for a TypeScript 2.4.1 project. We sadly can't update the project to a newer TypeScript version.
The developer console shows the following error:
ERR <main> TypeScript Server Error (2.4.1)
undefined
undefined: Error: <main> TypeScript Server Error (2.4.1)
undefined
undefined
	at n.create (c:\Users\REDACTED\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:2:901269)
	at h.w (c:\Users\REDACTED\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:2:896478)
	at h.u (c:\Users\REDACTED\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:2:895543)
	at c:\Users\REDACTED\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:2:895175
	at f.invoke (c:\Users\REDACTED\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:63:145)
	at g.deliver (c:\Users\REDACTED\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:63:2066)
	at h.fire (c:\Users\REDACTED\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:63:1704)
	at m.j (c:\Users\REDACTED\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:2:903875)
	at Socket.<anonymous> (c:\Users\REDACTED\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\typescript-language-features\dist\extension.js:2:903654)
	at Socket.emit (node:events:526:28)
	at addChunk (node:internal/streams/readable:315:12)
	at readableAddChunk (node:internal/streams/readable:289:9)
	at Readable.push (node:internal/streams/readable:228:10)
	at Pipe.onStreamRead (node:internal/stream_base_commons:190:23)

The problem also occurs with the TypeScript nightly plugin.
VS Code version: Code 1.76.0 (92da9481c0904c6adfe372c12da3b7748d74bdcb, 2023-03-01T10:22:44.506Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: Yes
Extensions (32)

Extension
Author (truncated)
Version

pascal
ale
9.6.0

pascal-formatter
ale
2.8.1

asciidoctor-vscode
asc
2.9.8

vscode-intelephense-client
bme
1.9.5

laravel-blade
cjh
1.1.2

vscode-svgviewer
css
2.0.0

githistory
don
0.6.19

xml
Dot
2.5.1

gitlens
eam
13.3.1

EditorConfig
Edi
0.16.4

code-runner
for
0.12.0

gc-excelviewer
Gra
4.2.56

todo-tree
Gru
0.0.224

rainbow-csv
mec
3.5.0

git-graph
mhu
1.30.0

vscode-docker
ms-
1.24.0

vscode-language-pack-de
MS-
1.76.2023030109

isort
ms-
2022.8.0

python
ms-
2023.4.0

vscode-pylance
ms-
2023.3.10

jupyter
ms-
2023.2.1000592019

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

remote-wsl
ms-
0.76.1

vscode-typescript-next
ms-
5.1.20230302

debugger-for-chrome
msj
4.13.0

vscode-commons
red
0.0.6

vscode-yaml
red
1.11.0

vscode-icons
vsc
12.2.0

php-debug
xde
1.32.0

markdown-pdf
yza
1.4.4

markdown-all-in-one
yzh
3.5.0

Are you using VS Code's built-in TS version or lower project's local from node_modules?

I have explicitly selected the project's lower 2.4.1 version.

TypeScript 2.4 is approaching 6 years old and no longer officially supported by VS Code's TS support. Some features may not work if you select it for tooling

It's understandable, but also disappointing. Is there any way to find it in what version this still works?

We only officially support the most recent stable TS version but you can likely use TS 3.x+ without issues for now


Enable interactive C# on the What's new topics

The topics under "What's new" should leverage the C# interactive window for (almost) all the samples. This issue should be split into a few PRs:

[ ] What's new in C# 7.2
[ ] What's new in C# 7.1
[ ] What's new in C# 7.0
[x] What's new in C# 6

In addition, the What's new in C# 6 should link to the string interpolation interactive quick start.

This part of the task:

In addition, the What's new in C# 6 should link to the string interpolation interactive quick start.

Has been done in #8238

Moving to March. New engineering features are needed for the 7.x interactive experience.

The work planned for this is being replaced by #12642 and #12645
Closing.


CDN seems to be down and cannot get mirrors working

Hey,
Been trying to builds over the last few hours and I keep getting hung up on here:
Step 2/7 : RUN apk add --no-cache ca-certificates make curl git
 ---> Running in 5530e241112
fetch http://dl-cdn.alpinelinux.org/alpine/v3.5/main/x86_64/APKINDEX.tar.gz
WARNING: Ignoring http://dl-cdn.alpinelinux.org/alpine/v3.5/main/x86_64/APKINDEX.tar.gz: temporary error (try again later)
fetch http://dl-cdn.alpinelinux.org/alpine/v3.5/community/x86_64/APKINDEX.tar.gz
WARNING: Ignoring http://dl-cdn.alpinelinux.org/alpine/v3.5/community/x86_64/APKINDEX.tar.gz: temporary error (try again later)
ERROR: unsatisfiable constraints:
  curl (missing):
    required by: world[curl]
  git (missing):
    required by: world[git]
  make (missing):
    required by: world[make]
The command '/bin/sh -c apk add --no-cache ca-certificates make curl git' returned a non-zero code: 3

Anyway around this? Anyway I can get a mirror working?

See the end of this issue https://github.com/gliderlabs/docker-alpine/issues/155 for a solution. Seems to be working for me until they resolve issues with the other mirrors

Encountered same thing use above to fix ^

Thanks @willtrking


Glitch occurs when using hotkeys against Cerulean Bell

When playing against the Cerulean Bell boss blind that forces a card to be selected at all times, using the hotkeys in certain sequences can cause the forced card to be de-selected. Or sometimes pseudo-deselected, where it appears unselected but is in a glitched state where it is unselected and selected simultaneously and contributing to the current hand. Even after the boss is defeated, this and other cards glitch in your hand, sometimes appearing flipped over, invisible, or altogether unselectable.
Huge fan of this mod, just thought I'd bring this to your attention. For the time being, I'll just select my cards with the mouse like a caveman during Cerulean Bell. Thanks for all you do!

Thank you for your feedback!
I pushed this commit that should fix the issue. Though I couldn't test it yet. Gonna close the issue after I finally find Cerulean Bell.


Midi Gate Make Note

I'm think of a useful node.
I remade my sampler patch, to just loop through the sample in the phasor gen fashion.
Now, I'd like to trigger the loop infinitely with Make Note, but Make Note has a duration and I would rather have a Gate Note, with Trig On, Trig Off, in order to determine myself when to gate on and off I'm wondering if it could be a good option to add in the plugin?
Here's a screenshoot of a version I made and I'm joining my nasty patcher.
SeqSample.maxpat.zip


205 replace anonymous sessions by reader sessions in dashboard sharing viewer

Feature or Bugfix

Feature

Detail

Added a configuration parameter in cdk.json. This configurable parameter can take 2 values ["reader", "anonymous"]
When a Dashboard that is SHARED is embedded in the UI we read the value of this configuration and depending on the setup we use the corresponding Quicksight API calls:
For the anonymous configuration, we use a Quicksight ANONYMOUS session to display the Dashboard in the UI. No Quicksight user is created in the Dashboard Quicksight account.
For the reader configuration, we create a Quicksight user and Quicksight group (if they don't exist yet) for this user that got access to a shared dashboard. We grant "read" Quicksight permissions to the specific Dashboard to the Quicksight GROUP. Then in the UI we use a READER session to embed the Dashboard in data.all UI

Recommendations
This implementation tries to satisfy different customer scenarios. The decision to use 'anonymous' or 'reader' sessions depends on your constraints. With reader sessions you have more control over the users accessing the dashboard and you can apply data filter and permissions on those users, however if you expect thousands of users accesing a single view of a dashboard, anonymous sessions might scale better. Check the embedding documentation and the pricing to understand the differences and find which solution fits better to your needs.
Relates

#205

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

Changes done


Remove BigOven API as it is no longer free

Per https://support.bigoven.com/hc/en-us/articles/208189893-Do-you-offer-free-access-to-the-API-
BigOven API is only available for free for 2 weeks
Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[x] Your submissions are formatted according to the guidelines in the contributing guide
[x] Your additions are ordered alphabetically
[x] Your submission has a useful description
[x] The description does not end with punctuation
[x] Each table column should be padded with one space on either side
[x] You have searched the repository for any relevant issues or pull requests
[x] Any category you are creating has the minimum requirement of 3 items

Thanks!


[SPARK-4340] [Core] add java opts argument substitute to avoid gc log overwritten

add substitute args for while forking java process.
So we can set extra java opst like this:
-XX:+PrintGCDetails -XX:+PrintGCDateStamps -verbose:gc -Xloggc:/tmp/spark.executor.{{HOSTNAME}}.{{APP_ID}}.{{EXCUTOR_ID}}.gc.log

to avoid gc log to be overwriten

Jenkins this is ok to test.
@haitaoyao this looks like a good feature to have. The PR is somewhat old at this point and I think there may be some updates you'll need to make after you rebase to master. Once you do that @srowen and I will take a closer look again.

ok to test

Test build #27852 has started for   PR 3205 at commit 9f8db81.

This patch merges cleanly.

Test build #27852 has finished for   PR 3205 at commit 9f8db81.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27852/
Test PASSed.

Test build #27933 has started for   PR 3205 at commit 1a103c9.

This patch merges cleanly.

Test build #27933 has finished for   PR 3205 at commit 1a103c9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27933/
Test PASSed.

@haitaoyao are you able to address the comments? If not, can we close this issue for now?

Can one of the admins verify this patch?

Mind closing this PR?


Allow positioning of mobile video poster

Mobile video poster is displayed cover. So it needs to be positionable.
Redmine #15022

Coverage remained the same at 99.156% when pulling fb01bef3b3d23869c194b0e7cea713047de7c738 on tf:mobile-poster-pos into 8b0e8da3b434234beddbb05f03e1c8498a76bba5 on codevise:master.


Bug: Application error: Uncaught (in promise): CloseEvent: {"isTrusted":true}

Error message: Uncaught (in promise): CloseEvent: {"isTrusted":true}
Error stack:
Error: Uncaught (in promise): CloseEvent: {"isTrusted":true}
    at ve (altair://-/polyfills.js:1:35936)
    at altair://-/polyfills.js:1:34973
    at t (altair://-/main.js:1:2601541)
    at _ (altair://-/main.js:1:2601744)
    at x.invoke (altair://-/polyfills.js:1:25842)
    at Object.onInvoke (altair://-/main.js:1:107574)
    at x.invoke (altair://-/polyfills.js:1:25781)
    at x.run (altair://-/polyfills.js:1:20882)
    at altair://-/polyfills.js:1:36771
    at x.invokeTask (altair://-/polyfills.js:1:26527)

When did this error occur? And does it re occur, or it happened only once?
Please provide some helpful screenshots of the bug
Version 4.4.1

Closing this issue as I believe this is already fixed in v4.4.2


esy install failing on Windows.

Operating System: Windows 10 Pro
Revery Version: Trying to install
OCaml version: trying to install
I do have visual studio 2017 installed

After various false starts I got as far as typing
"esy install" in an admin prompt in the revery cloned directory and it got as far as:
.... fetching<EMAIL_ADDRESS>internal error, uncaught exception:
Unix.Unix_error(Unix.EACCES, "rename", "C:\local\revery\_esy\default\tmp\esy-ce1b63")
Raised at file "esy-lib/RunAsync.re", line 13, characters 2-51
Re-raised at file "esy-lib/RunAsync.re", line 13, characters 2-51
Re-raised at file "src/core/lwt.ml", line 2999, characters 20-29
Called from file "src/unix/lwt_main.ml", line 26, characters 8-18
Called from file "esy-lib/Cli.re", line 264, characters 9-28
Called from file "cmdliner_term.ml", line 25, characters 19-24
Called from file "cmdliner.ml", line 25, characters 27-34
Called from file "cmdliner.ml", line 116, characters 32-39
Before that I installed npm
found that "npm install -g windows-build-tools" failed when my antivirus reports "PowerShell tried to load a malicious resource detected as Heur.BZC.ZFV.Boxter.6008.E5DBFDF9 and was blocked. Your device is safe." - and changing its settings won't stop it from blocking.
But then I noticed that you had a an alternative way to install.
"choco install visualstudio2017buildtools visualstudio2017-workload-vctools -y"
I tried that and it installed visualstudio2017buildtools but balked at visualstudio2017-workload-vctools.
So I tried again from an admin prompt but it claimed that visualstudio2017-workload-vctools was already installed.
So I tried "choco upgrade -y visualstudio2017-workload-vctools --force"
then I went back to the normal prompt and tried. "esy install"
That failed with something I don't see in the scrollback about tar not being able to create a symlink.
So I tried it again in the admin prompt and here I am at
.... fetching<EMAIL_ADDRESS>internal error, uncaught exception:
Unix.Unix_error(Unix.EACCES, "rename", "C:\local\revery\_esy\default\tmp\esy-c99678")
Raised at file "esy-lib/RunAsync.re", line 13, characters 2-51
Re-raised at file "esy-lib/RunAsync.re", line 13, characters 2-51
Re-raised at file "src/core/lwt.ml", line 2999, characters 20-29
Called from file "src/unix/lwt_main.ml", line 26, characters 8-18
Called from file "esy-lib/Cli.re", line 264, characters 9-28
Called from file "cmdliner_term.ml", line 25, characters 19-24
Called from file "cmdliner.ml", line 25, characters 27-34
Called from file "cmdliner.ml", line 116, characters 32-39

I tried installing an older version of reperf, same error
I tried leaving reperf out of the install - same error on the NEXT include
I tried uninsalling esy and installing it again - same error
I tried turning off the virus scanner so that I could use "npm install -g windows-build-tools"

once again I got the same error.

I tried installing an older version of reperf, same error
I tried leaving reperf out of the install - same error on the NEXT include
I tried uninsalling esy and installing it again - same error
I tried turning off the virus scanner so that I could use "npm install -g windows-build-tools"

once again I got the same error.

I've ran into this error before; I removed the _esy (in the project) and .esy (in your home folder) directories and it worked after that. Also make sure you're running esy as admin; for some reason it never fully finishes for normal users.


Bring in new utils for new requirements-freeze command

Looks good, but can we hold off deploying any changes to Search API to prod until we get a reply from PaaS - thanks.


Add xhr, async, etc call detection feature for headless

Please describe your feature request:

Describe the use case of this feature:

Done


Overview doc about Azure Function settings

There should be a doc that gives an overview about settings in Azure Functions that covers these topics:

The distinction between settings for local deployments (local.settings.json) and Azure deployments (Application Settings in the Azure Portal). They use different settings mechanisms.
How Azure Function settings compare with .NET core web services settings. Currently, developers like me wonder "I configure my .NET core web service with appsettings.json and appsettings.<environement>.json. Can I do the same with my Azure Function?".
Describe how objects need to be flattened (e.g. { "Foo": { "Bar": 2 } } becomes { "Foo:Bar": 2 })
Describe how arrays need to be flattened (e.g. { "Foo": [2] } becomes { "Foo:0": 2 })

See issue #39109 for additional suggestions about what kind of information is missing from the current settings docs.
After reading this new doc, I'll still be left with questions specific to settings for local deployments and Azure deployments.
Settings for local deployments

Should local settings (local.settings.json) be checked into git? local.settings.json is in the default .gitignore which leads me to believe I shouldn't check it in. If that's the case, how do we share the settings amongst members of our team?
Is the @ KeyVault syntax supported in local settings? If not, what should I do instead? There's an unanswered GitHub issue about this

Settings for Azure deployments

Can I manage my settings for Azure deployments in git? I'd like them to be version controlled. It seems like I can't because they are managed through the Azure Portal.

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

ID: d4f40a37-fa0f-6d54-57ad-8d58c6c3531b
Version Independent ID: 3b2178f9-f6a8-8771-f7f5-7a9952b45b2b
Content: Configure function app settings in Azure
Content Source: articles/azure-functions/functions-how-to-use-azure-function-app-settings.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@rigdern Thank you or making us aware of this suggestion. I am having an integration engineer evaluate the feedback and update the documentation where appropriate.

@rigdern I have assigned this to content author for further review and update as appropriate.

@rigdern sorry it took so long to follow-up on your suggestions here. I think that we've addressed all of these by this point, but in the following articles (where the info seemed to go best):

https://learn.microsoft.com/en-us/azure/azure-functions/functions-develop-local#local-settings-file
https://learn.microsoft.com/en-us/azure/azure-functions/functions-host-json#override-hostjson-values
https://learn.microsoft.com/en-us/azure/azure-functions/functions-how-to-github-actions?tabs=dotnet

#please-close


Rounds do not generate

When using /rounds generate, the application does not respond.

That is a good point, I'll have to reimplement that. The fact is that it ties into the awesomescibo api, which I think would be best just to merge into this repo.

Actually, I think this is due to timing out, which was fixed in #23. The rounds are non-functional anyways since the API and the bot have to be running with the same database, which will hopefully be made easier with #30.


Can't install native package into two or more win32 projects in same solution

To reproduce:

In VS Community, Update 3, with everything up to date
Create a new empty solution, add two C++ win32 console applications (call them bogus1 and bogus2, for example).
From solution line select "Manage nuget packages...", search nuget.org for the ROOT native package.
Select "bogus1" as the target, and install. Works fine.
Select "bogus2" as the target, and install. Fails with a "Could not install package 'ROOT 534.36.13'. You are trying to install this package into a project that targets 'native,Version=v0.0', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author."
Uninstall from bogus1, select bogus2, install. Works fine.
Select bogus1, install. Fails with the same error as above.

I have not figured a work around yet. I have version <IP_ADDRESS>4 of nuget installed according to the extensions and updates dialog box. Here is a complete log of the attempted installation when I selected two of my projects (selecting bogus1 and bogus2 at the same time).
Attempting to gather dependency information for package 'ROOT.534.36.13' with respect to project 'PlotSingleLimit', targeting 'native,Version=v0.0'
Gathering dependency information took 85.67 ms
Attempting to resolve dependencies for package 'ROOT.534.36.13' with DependencyBehavior 'Lowest'
Resolving dependency information took 0 ms
Resolving actions to install package 'ROOT.534.36.13'
Resolved actions to install package 'ROOT.534.36.13'

Attempting to gather dependency information for package 'ROOT.534.36.13' with respect to project 'ExtrapolateByBeta', targeting 'native,Version=v0.0'
Gathering dependency information took 0.76 ms
Attempting to resolve dependencies for package 'ROOT.534.36.13' with DependencyBehavior 'Lowest'
Resolving dependency information took 0 ms
Resolving actions to install package 'ROOT.534.36.13'
Resolved actions to install package 'ROOT.534.36.13'
Retrieving package 'ROOT 534.36.13' from 'nuget.org'.
Adding package 'ROOT.534.36.13' to folder 'C:\Users\Gordon\Documents\Code\calratio2015\Extrapolation\packages'
Added package 'ROOT.534.36.13' to folder 'C:\Users\Gordon\Documents\Code\calratio2015\Extrapolation\packages'
Added package 'ROOT.534.36.13' to 'packages.config'
Successfully installed 'ROOT 534.36.13' to PlotSingleLimit
Executing nuget actions took 300.44 ms
Found package 'ROOT 534.36.13' in 'C:\Users\Gordon\Documents\Code\calratio2015\Extrapolation\packages'.
Install failed. Rolling back...
Package 'ROOT.534.36.13' does not exist in project 'ExtrapolateByBeta'
Executing nuget actions took 11.33 ms
Could not install package 'ROOT 534.36.13'. You are trying to install this package into a project that targets 'native,Version=v0.0', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.
========== Finished ==========
Time Elapsed: 00:00:00.7735402

Workaround: uninstall nuget 3.5 and install 3.4.4 (which is most up to date in the gallery). Given 3.4.4. is the most up to date that is sitting in the extension gallery, I'm not sure how I ended up with 3.5 on my machine. Perhaps it came down when I upgraded to update 3?

hi, not sure what happened with your nuget version, but I'm glad you fond the issue.

Yikes! I'm so sorry - I must have been working without sleep. This should have been in the nuget issue feed, not yours!! :(


Cohttp client

Fixes the tests to support cohttp
Add equinoxe-cohttp which relies on cohttp-lwt-unix to work
Update README.md with equinoxe-hlc and equinoxe-cohttp
Fixes #60

LGTM thanks!


Support for multi-GPU Parallelism

It would be great to have support in this package for multi-GPU parallelism.  I devised a very hacky sort of way to accomplish this and wrote it up here.  This particular one is for the CUSPARSE package, but the implementation would be nearly identical for CUBLAS.  I'm not certain if my approach was necessarily the best.  I'd be happy to work a bit to get this added to the package.  But, since I've never contributed to a package before and am not certain how good my approach is, it would be helpful to correspond a bit before just putting up a pull request.
How does the linked implementation look?  Any comments, thoughts, suggestions?  Obviously, what's posted there is just the rudiments of an implementation, just for a single function and without even all of the functionality for that.
See also this discussion of much this same issue on the Julia CUSPARSE GitHub page here.

The people looking at the issues for JuliaGPU are all the same, so raining it in on place should be enough ;)
JuliaGPU/CUSPARSE.jl#23


Unit tag should not be stored as 'np.int32'

From this line, the last item u.tag is stored as np.int32. It is also converted again as np.int32 here.
From StarCraft II protobuf, the unit tag should be uint64. And they DO use tag beyond int32. Even in simple games like MoveToBeacon, the actual tag value differs from the stored tag. I have tried simply changing those two occurrences of int32 to int64 and it worked (so far). I believe this is a bug.

This should be fixed.


Add CustomMetadata to top-level of pipeline spec

Move annotations out of service spec into top-level custom_metadata field

closes #4598

PTAL @jdoliner

name updated, I believe I've updated the relevant docs, but because this is api breaking you might want to take a pass @svekars

@jdoliner feel free to take another pass, restore is now working

CI passing, do you want to take one final pass @jdoliner


Cannot move to splits/panes

Setup

init.lua config:

-- Configuration
require('Navigator').setup({
    auto_save = 'current',
    disable_on_zoom = true
})

-- Keybindings
local map = vim.api.nvim_set_keymap
local opts = { noremap = true, silent = true }

map('n', "<c-h>", "<CMD>lua require('Navigator').left()<CR>", opts)
map('n', "<c-k>", "<CMD>lua require('Navigator').up()<CR>", opts)
map('n', "<c-l>", "<CMD>lua require('Navigator').right()<CR>", opts)
map('n', "<c-j>", "<CMD>lua require('Navigator').down()<CR>", opts)
map('n', "<c-p>", "<CMD>lua require('Navigator').previous()<CR>", opts)

$XDG_CONFIG_HOME/.config/tmux/tmux.conf:

set -g @plugin 'tmux-plugins/tpm'
set -g @plugin 'christoomey/vim-tmux-navigator'

set-environment -g TMUX_PLUGIN_MANAGER_PATH '~/.local/share/tmux/plugins'
if "test ! -d ~/.local/share/tmux/plugins/tpm" \
   "run 'git clone https://github.com/tmux-plugins/tpm ~/.local/share/tmux/plugins/tpm && ~/.local/share/tmux/plugins/tpm/bin/install_plugins'"
# Initialize TMUX plugin manager (keep this line at the very bottom of tmux.conf)
run '~/.local/share/tmux/plugins/tpm/tpm'

nvim --version (:checkhealth) shows no issues:

NVIM v0.5.0-dev+1226-g370469be2
Build type: RelWithDebInfo
LuaJIT 2.0.5

tmux version: 3.1_c-1 (archlinux)
echo $TMUX | cut -d, -f 1: /run/user/1000/tmux-1000/default, permissions: srwxrwx---, ownership: my user

Issue description
When pression <c-l>/<c-j>/<c-k>/<c-h> to move to another nvim split / tmux pane, I get the following error message and the action fails:
E5108: Error executing lua .../pack/packer/start/Navigator.nvim/lua/Navigator/tmux.lua:28: bad argument #1 to 'read' (invalid option)

which points to an issue in this line:
https://github.com/numToStr/Navigator.nvim/blob/ff87ccf64be8edb0cc783131ceba1d44a87a5f81/lua/Navigator/tmux.lua#L28
I get no such issue with the regular vim-tmux-navigator plugin.

Thanks for the detailed report. I am seeing your LuaJIT version is 2.0.5 which is different than mine which is
NVIM v0.5.0-dev+1221-g9b2d4ff62
Build type: Release
LuaJIT 2.1.0-beta3

Can you try to update neovim and see if that fixes the issue?

Thanks @numToStr , using luajit 2.1.0-beta3 and rebuilding neovim solves the issue.
I'm using luajit 2.05 because it's the one in arch repos and thus is the one pulled when installing neovim-git from the AUR.
I would suggest either:

changing that handle:read() to something supported on luajit 2.0.5 (which is the latest stable version)
putting a warning on the README that neovim needs to be compiled with luajit 2.1.0-beta3

The first one would be preferable of course :wink:

I am glad that updating fixes your issue. I'll add a note on about LuaJIT 2.1.0 or use HEAD.

or use HEAD.

It has nothing to do with HEAD, I'm always using HEAD. It's entirely related to the version of luajit neovim was compiled with.

Ohh i see. I was not aware of that as I always build from the repo.

Ohh i see. I was not aware of that as I always build from the repo.

Yeah, if you build manually from the repo you end up using their bundled dependencies which includes that luajit version, but other package managers which offer HEAD (arch's AUR/ nixOS/ brew) automatically build HEAD from the repo but often use the system versions of neovim's dependencies to avoid duplication.
Ref: https://github.com/neovim/neovim/wiki/Building-Neovim#third-party-dependencies


Second Bot not running?

ver amongusdiscord-2.3.2
AmongUsCapture-2.4.1
I don't think my second bot is connected to the game.
When I start the bot in discord (au -n), I get a direct message with a link from my first bot. I do not get a direct message from the second bot.
My output from the amongusdiscord window follows. I replaced private info with asterisks .
You can see in line 2, amongusdiscord recognizes that the final.txt has a 2nd Discord Bot Token.
In the last line, amongusdiscord indicates that it is waiting 1 secs to update...
Does that "waiting" message indicate that the second bot is not running?
I reference prior thread #154 Which seems to be unresolved.
2020/10/17 17:53:05 2.3.2-Prerelease
2020/10/17 17:53:05 You provided a 2nd Discord Bot Token, so I'll try to use it
2020/10/17 17:53:05 [Info] Invalid or no particular PORT (range [1024-65535]) provided. Defaulting to 8123
2020/10/17 17:53:05 [Info] No valid SERVER_URL provided. Defaulting to http://localhost
2020/10/17 17:53:05 [Info] No EXT_PORT provided. Defaulting to PORT
2020/10/17 17:53:05 Using ./ as the base path for config
2020/10/17 17:53:05 Success in initializing the local Filesystem as the Storage Driver
2020/10/17 17:53:05 Bot is now running.  Press CTRL-C to exit.
2020/10/17 17:53:06 Successfully loaded config from storagedriver for *************
2020/10/17 17:53:06 Added to new Guild, id *************, name *************
2020/10/17 17:53:06 [This is not an error] No explicit guildID provided for emojis; using the current guild default
2020/10/17 17:53:06 Serving at localhost:8123...
2020/10/17 17:54:04 D9FD4EF3
2020/10/17 17:54:04 Added self game state message
2020/10/17 17:54:18 connected: 1
2020/10/17 17:54:18 Received connection code: "D9FD4EF3"
2020/10/17 17:54:18 Associated websocket id 1 with guildID ************* using code D9FD4EF3
2020/10/17 17:54:18 Waiting 1 secs to update the status message to not be rate-limited
2020/10/17 17:54:18 phase received from capture:  0
2020/10/17 17:54:18 Pushing phase event to channel
2020/10/17 17:54:18 Received PhaseUpdate message for guild *************
2020/10/17 17:54:18 Detected transition to Lobby
2020/10/17 17:54:18 player received from capture:  {"Action":4,"Name":"Loren","IsDead":false,"Disconnected":false,"Color":5}
2020/10/17 17:54:21 Received PlayerUpdate message for guild<PHONE_NUMBER>10357268
2020/10/17 17:54:21 Added new player instance for Loren
2020/10/17 17:54:21 Waiting 1 secs to update the status message to not be rate-limited
2020/10/17 17:54:21 lobby: {"LobbyCode":"SWZXYQ","Region":0}
2020/10/17 17:54:21 Received room code {"LobbyCode":"SWZXYQ","Region":0} for guild *************from capture
2020/10/17 17:54:21 Received PhaseUpdate message for guild<PHONE_NUMBER>10357268
2020/10/17 17:54:42 phase received from capture:  3
2020/10/17 17:54:42 Pushing phase event to channel
2020/10/17 17:54:42 Received PhaseUpdate message for guild *************
2020/10/17 17:54:42 Detected transition to Menu
2020/10/17 17:54:42 Waiting 1 secs to update the status message to not be rate-limited

As I understand, the second bot only works as a helper for the first bot, to help muting/unmuting people to avoid being ratelimited so that mutes and unmutes dosnt take forever when playing more than 7-8 players.
If the second bot is online in your discord, it is working as it should

^ @mLgz0rn is right, the second bot is only a "worker" bot that issues 1/2 the mute/deafen requests. Not sure what the functionality would entail for having 2 bots that both respond to .au new.... Why would you want a second bot that also acknowledges all the commands as the first bot, with 100% overlap?

@mLgz0rn Sounds good.
How would you test if the second bot is working correctly?
Create a game with 9+ players and watch for ratelimiting? How do I know if I am being rate limited?
Looking at my log now, it says, "Waiting 1 secs to update the status message to not be rate-limited."
Is 1 sec a short amount of time? (it sounds short.)
How long would the delay be if I had 9+ players and no second bot?

@OriginalFlavor
Yes, testing with 9+ players is the easiest way to test. If you see users being muted/deafened or unmuted/undeafened later than others, by a couple seconds, that's usually the cause.
And firstly, I believe those logs are from an older version... I recommend updating. But they also have nothing to do with the rate limit for muting/deafening. The logs are purely for indicating if the status message for the bot should be updated right away; I slowed down the updating by 1 second (which actually makes a huge difference) to prevent it from blowing past the limit, when the status message really doesn't need to update more than once a second, anyways.
And the delay for 9+ players w/ no 2nd bot is sometimes less than 2-3 seconds, but can be as bad as 5-10 seconds for some players. Discord dynamically adjusts their rate limits, so there's no way of saying for certain


Alter disclaimer text for unauthenticated access.

@caesar2164 Do we need a copy of this in the other themes? Will the include break w/out it?

hmm, we could just use the optional inlcude

I'd say we rename this file course_info_extra_sneakpeek.html and we add it with an optional include...

I believe we said F2F that this is no longer needed.


Atom command line always creates new files in a new window

Description
atom new_file will open the new file (that does not exist) in a new window instead of a new tab, creating a new Atom window. This does not happen if the file already exists.
Steps to Reproduce
atom some_new_file
Expected behavior: [What you expect to happen]
Open the new file in a new tab
Actual behavior: [What actually happens]
Opens the new file in a new window.
Reproduces how often: [What percentage of the time does it reproduce?]
Always
Versions
Atom    : 1.17.2
Electron: 1.3.15
Chrome  : 52.0.2743.82
Node    : 6.5.0
apm  1.17.0
npm  3.10.5
node 6.9.5 x64
python 2.7.10
git 2.13.0
MacOS 10.12.5
Note: adding the -n=false didn't change the behaviour in any way.

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/8030. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Be more explicit in the README about the root privileges requirements

When the overall testing is invoked
$ python run_algtest.py all

it requires root privileges (asks for them twice, I think). The first prompt for root password is at the start of the execution, so the tester is likely able to spot that and fill it in. However, the second prompt is later and it is easy to not fill that in, which leads to incomplete results.
I can see two options: be more explicit about the need to babysit the execution and fill in the root password (and specify how many times it will be) or suggest running as sudo (the user is running it at some point as sudo anyway).

Thanks for bringing this up, fixed in c1a5092db1bf944fdc01cdc66a197737c83a2aca.


Traffic shaper ACK packet selection is too wide and should be narrowed down.

As discussed here: https://forum.opnsense.org/index.php?topic=17372.0
the drop-down entry 'tcp (ACK packets only)' in the 'Edit rule' dialog of the Traffic Shaper will match all packets with 'tcpflags ack'.
Since this will in effect match almost all traffic, it is not particularly useful.
So far as I could see this would only need changes in service/templates/OPNsense/IPFW/ipfw.conf
and the suggested changes are (so far) to replace 'tcpflags ack' by either 'tcpflags ack,!psh' or 'tcpflags ack iplen 52'.
It might also be important to communicate any modifications as it will change current behavior.
Note: the original issue that prompted the addition of the feature was https://github.com/opnsense/core/issues/528

Have you had a chance to make this change and test? I'm making the commit to my current fork and may set up a VM if no one else has had a chance to try yet.

Have you had a chance to make this change and test? I'm making the commit to my current fork and may set up a VM if no one else has had a chance to try yet.

While I have not changed and tested the template, I have run a production firewall with 'tcpflags ack iplen 52' for some time now - I edited the /usr/local/etc/ipfw.rules file directly, though.

Note: There was no consensus in the discussion in the forum as to what the appropriate rule should be, only that the current rule is to generic and therefore not very useful.

Could you point me to a resource that explains iplen 52? or give me a quick and dirty as to why you chose that instead of !psh?

I just made the change you suggested in my ipfw.rules and reloaded all. looks like it did the trick!

Sure. From what I can recall, 40 bytes is what you need for tcp+ip(v4) header, e.g. what you need for an ACK or RST packet. An ACK with timestamp needs 52 bytes. (See http://www.faqs.org/rfcs/rfc1323.html - 1.3 Using TCP options).

You're the best, I'm going to change the template on my test device and verify that it works for the drop down option and then submit a pull.

You were correct, The change in the template is tested and working for both tcp ack and non-ack

One more thing, I am not very firm on IPv6 stuff, but if memory serves me right, IPv6 has (or can have) larger headers.
The following page on bufferbloat.net says (https://www.bufferbloat.net/projects/bloat/wiki/ACK_prioritization/):
"Since ack packets are very small (less than 72 bytes in ipv4 and less than 140 bytes in IPv6, depending on encapsulation), shaping methods that depend more on packets than bytes tend to suffer."
So, I guess there will be situations where iplen 52 will be too narrow and not capture ACK packets.
(Btw., the less than 72 bytes for IPv4 is true because other (useful) options could be set in an ACK packet and thus increase the size.)
But, for me, iplen of 52 has always had the desired effect (with IPv4 at any rate).

maybe it's better to add iplen as an option in the model and form instead of shape it into the tcp_ack selection.https://github.com/opnsense/core/issues/528 wasn't very great and if someone else wants to set iplen in the future we're going to run into all new kinds of races again.

@AdSchellevis I like it. It would probably be useless to attempt to find a one-size-fits-all scenario, so adding the iplen flag as an option for defining rules sounds like a good idea!

I agree, having the option to set the length separately in the gui makes more sense.

I'll take a look at this when I have some time this weekend, @AdSchellevis which files are you aware of that would need to be changed? I can dig through and find them if need be.
I imagine the iplen flag could be toggled and the field would take a numerical value. The help text would indicate some of the standard maximum sizes for packet types. What would a safe default be? Is the mtu available as a variable?

@maxfield-allison it shouldn't be very hard (I think iplen is compatible with all other options, which should avoid the need for referential constraints).
These are the files that come to mind:
https://github.com/opnsense/core/blob/master/src/opnsense/mvc/app/models/OPNsense/TrafficShaper/TrafficShaper.xml
https://github.com/opnsense/core/blob/master/src/opnsense/mvc/app/controllers/OPNsense/TrafficShaper/forms/dialogRule.xml
...and the ipfw template you already found.
The default should be empty (omit option) in which case it's still compatible with existing setups.
Our docs contain quite some content on how these models work by the way, see https://docs.opnsense.org/develop.html

Or something like this as an example:
https://github.com/opnsense/core/pull/2794/files
forms = UI
model = whats written/loaded in config.xml
template = you already know :)

Thanks for all of this information, You guys have all been great. This will be my first real contribution to a project so I'm very excited to learn and contribute more in the future.

Quick question, In TrafficShaper.xml is the underscore in the <tcp_ack> parsed as a space when it's written to the ipfw.conf?

So I've made the changes to dialogueRules.xml and TrafficShaper.xml and loaded them onto a test system while setting the originals to .bak but I'm getting page not found when navigating to the traffic shaper pipes, queues and rules pages. Is there a checksum that OPNsense uses to validate files?

case sensitivity? no checksum is being used while serving files.

That's what I thought too, double checked and all is well. I'm not building from source, just replacing the file, could that be the issue? After replacing the bak for each and reloading all services I'm still seeing page not found.

what are you trying to access and what exact message is returned?

Firewall>Shaper>Rules, Queues, Pipes
"Page not found" in the title where each would normally be displayed.
Console shows this
Failed to load resource: the server responded with a status of 404 (Not Found)

browser network inspector? what is it trying to access?

Request URL: https://<IP_ADDRESS>/ui/trafficshaper
Request Method: GET
Status Code: 404 Not Found
Remote Address: <IP_ADDRESS>:443
Referrer Policy: same-origin

something in /tmp/PHP_errors.log ? if the controller is there it should just pass the template, nothing special

Looking at backend, webgui, and general logs, I'm seeing the Web GUI starts, I'm seeing the traffic shaper statistics calls `
configd.py: [0e2a7c0d-49e8-4703-9575-374b83dbbc2b] request dummynet stats
`
but nothing from when I try to access the rules, queues, or pipes. pages. I'll check the php errors log

I'm not even seeing a php_errors.log in /tmp/ or anywhere else for that matter

figured it out.

Ok one easy question, How should I set the default to omit? I'm currently using
                <iplen type="IntegerField">
                    <MinimumValue>1</MinimumValue>
                    <MaximumValue>65535</MaximumValue>
                    <default></default>
                    <Required>N</Required>
                    <ValidationMessage>The absolute limitation for packet size is 64K (65535 bytes)</ValidationMessage>
                </iplen>

just leave the tag out?

ok, even with:
    <field>
        <id>rule.iplen</id>
        <label>IP Packet Length</label>
        <type>select_multiple</type>
        <advanced>true</advanced>
        <style>tokenize</style>
        <allownew>true</allownew>
        <help>Matches packets whose length is in the set. Specified in the same way as ports, Examples: 52-72, 1500, 1474</help>
    </field>

My thoughts were that since Iplen can take a list this would be the best way to implement. Open to suggestions though of course.

the integerfield expects a single value. question is if there are use-cases where you would like to match exact packet lengths, maybe a max_iplen is more logical in that case.
not required, default empty?
https://github.com/opnsense/core/blob/6b6e3ed32d6c3b54aec5fadd49ee66c0824c99fe/src/opnsense/mvc/app/models/OPNsense/TrafficShaper/TrafficShaper.xml#L36-L41

You're probably right. In the case where a single maximum iplen is set, users could simply create additional rules. I'm probably overcomplicating it.

@AdSchellevis I'm taking a look at copying and modifying PortField.php to create PacketSizeField.php. I think the ability to specify a range of packet lengths to match would be beneficial in this case. After looking over the documentation for IPFW dummynet, it appears that setting one specific size would necessitate the creation of many rules to accomplish one primary task. By allowing a range, we could cut down on the complexity of implementing shaping rules.

I think you can simplify this by specifying a max and writing 1-[max] in your ipfw.conf. If it needs a field type, it should remain in the module (https://docs.opnsense.org/development/frontend/models_customfields.html) since packetsize isn't something generic in my opinion.
Maybe it's better to discuss the scope of the feature first, I'm a bit worried we're overcomplicating things here if we can't think of a reason why multiple ranges would make sense. (besides that max_iplen can easily be migrated into a single range in the future).

That makes perfect sense. I had gotten through the XML and started looking at the ipfw.conf so was not aware that I could simply use 1-[max] to set the value as a range while only taking a single integer in the model and form.

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/core/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.


How to locate vm root?

For now, hypervisor related stuff(socket/pidfile etc.) exist in /var/run/hyper/ dir, and named with generated vm name vm-xxxxxxxx. This vm name can't be got through any runv command or option, could we offer a mechanism to get this vm specified path?

add additional field in runv state result to include vm id.
Or add a symbol link in container root dir, ie /run/runv/<container_id>/vm-root --> /var/run/hyper/<vm_id>

/cc @laijs

you can set the hypervisor.BaseDir to the containerd when runv containerd is just started.
and the vm_id can be fetched from vm.Id.
could you make a pr for it please?

@laijs yep, will do.

Fixed by #505


[meta] Acquire old binary dfm files and document the format

Currently only the plain text dfm format is supported as I'm lacking examples of the older binary dfm format used on older version of Borland Delphi.

I made some progress on this issue https://github.com/kaitai-io/kaitai_struct_formats/pull/707


chore: update i18n change

Please verify the following:

[x] yarn test jest tests pass with new tests, if relevant
[x] README.md has been updated with your changes, if relevant

Describe your PR

Hi @luvyouso, would you mind editing the PR comment and writing up a description for your PR as to why it is needed and what it does?
Thank you for your contribution!

Hi @luvyouso, would you mind editing the PR comment and writing up a description for your PR as to why it is needed and what it does?
Thank you for your contribution!

hi @frankcalise , i update PR.

Update:
I updated the example
I will update the instructions once I finish the IOS as well.
for now you can just install packages and run example, and start toggling the locale, it will restart the app, also if you fully close the app and re-run it agian, you will see that the latest locale is synched from the locale storage successfully, and it's working like charm! with no hooks required, no second render, no Async on JS.

Update:
I updated the example I will update the instructions once I finish the IOS as well.
for now you can just install packages and run example, and start toggling the locale, it will restart the app, also if you fully close the app and re-run it agian, you will see that the latest locale is synched from the locale storage successfully, and it's working like charm! with no hooks required, no second render, no Async on JS.

thanks for your support. @ahmedrowaihi

my pleasure! @tunv8zzz
I just released an initial PoF, with Docs on how to use it.
v1.2.0

Thanks for the contribution but we are not looking to pull this into Ignite at this time. I believe if we changed languages from LTR to RTL it wouldn't be reflected still.


feat: move library to typescript code generation

Something is wrong with the header system test:

I think you may need to get a service account key and debug it locally :(

cool, system-test fails locally, thought it was because of some credential, but yes, I will get account and debug it. @alexander-fenster


[task] support percpu run-queue and cpu affinity for axtask

scheduler : https://github.com/arceos-org/scheduler/pull/1

Modifications to arceos-apps: https://github.com/arceos-org/arceos-apps/pull/1/files


Error Page Added

i have added the Error page which match the website . @Anupkjha2601  @goodchai0

@goodchai0


build(deps): remove some unused dependencies

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
It takes quite long time to build the project on my laptop, so I tried to remove
some dependencies to see whether it will improve the compile time.
From my local test with

command cargo build --timings after cargo clean the cache
8 threads
16GB memory (about 15GB free)
clang version 16.0.6
reboot system and compile directly to reduce the affect of other stuffs

before:

after:

However, I only observed very little improvement except units reduce from 1229
to 1219. The compilation will drain all my memory and sometimes crash my GUI,
so maybe better hardware can give more improvements?
Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.
[x]  This PR does not require documentation updates.

However, I only observed very little improvement except units reduce from 1229
to 1219.

Most of the unused deps you removed in this patch are still dependent in other places so the overall compile time doesn't change a lot. But still appreciate it 💯

Only test cargo build locally 😅, try to test others. Or this job has already been done?

Have you tried the mold? It only takes ~4GiB memory for cargo build.

Test it and time reduces from 461.4s (7m 41.4s) to 374.0s (6m 14.0s), really boost a lot. Thanks ❤!
What do you mean ~4GiB memory for cargo build? As compilation still drain almost all of my free memory.

Have you tried the mold? It only takes ~4GiB memory for cargo build.

Maybe we should mention this somewhere in our document. In my experience mold helps in many environments.
cc @tisonkun do you have any suggestions for the place?

Maybe we should mention this somewhere in our document. In my experience mold helps in many environments.

@waynexia it can be a subtask of a revision of CONTRIBUTING.md. You can create an issue and ping me. I schedule to handle the CONTRIBUTING.md file in April; any feedback is welcome :D

Maybe we should mention this somewhere in our document. In my experience mold helps in many environments.

@waynexia it can be a subtask of a revision of CONTRIBUTING.md. You can create an issue and ping me. I schedule to handle the CONTRIBUTING.md file in April; any feedback is welcome :D

Ticket: https://github.com/GreptimeTeam/greptimedb/issues/3587


Add TLSA types that will be used to implement DANE

I put these in a separate module to avoid clutter, but that can be moved if you prefer to use a different style for this.
One thing I'm not totally sure about is this paragraph at the bottom of https://datatracker.ietf.org/doc/html/rfc6698#section-2.1.1:

The certificate usages defined in this document explicitly only apply
to PKIX-formatted certificates in DER encoding [X.690].  If TLS
allows other formats later, or if extensions to this RRtype are made
that accept other formats for certificates, those certificates will
need their own certificate usage values.

For the sake of simplicity in constructing the TlsaRecord I've encoded the data field as Der, but the above suggests that perhaps the CertificateUsage variants should become "load bearing" members that have Der data inside them.
Thoughts?
refs: https://github.com/rustls/rustls/issues/816

A couple of problems with this, if TlsaRecord is to be embedded in rustls::ServerName:

ServerName is Hash+Eq but Der is not. I can make Der Hash+Eq, seemingly without consequence. I'm not sure that it is valid to relax ServerName to be just PartialEq; I'm not sure how/where ServerName is hashed.
The lifetime in Der<'a> either needs to infect ServerName<'a> or requires the use of 'static when embedded in there.

A couple of problems with this, if TlsaRecord is to be embedded in rustls::ServerName:
* `ServerName` is `Hash+Eq` but `Der` is not. I can make Der `Hash+Eq`, seemingly without consequence. I'm not sure that it is valid to relax `ServerName` to be just `PartialEq`; I'm not sure how/where `ServerName` is hashed.

Try removing the Hash bound and see what fails? I think it's probably okay to make Der Hash, too.

* The lifetime in `Der<'a>` either needs to infect `ServerName<'a>` or requires the use of `'static` when embedded in there.

I think we'll want to let it infect ServerName? IIRC ServerName so far is a rustls concept, might be worth seeing how we represent this in rustls-webpki (where, IIRC, we avoid allocating on the common paths).

The path forward here feels uncertain enough that it might make sense to make this a draft PR and consider it for merging only after having used it in downstream draft pull requests that specify this branch as a dependency override. I suspect there pursuing the webpki and rustls updates will affect the shape of the common types.


Reactivate atoms and bonds valence checks

This partially re-implement issue #185 . It reimplements valence checks for atoms and bonds only.
Reactivating valence checks for angles and torsions will require larger modifications, but I'd like to take a decision about #216 first in order to minimize the total time I'll spend on this, and implement it in a separate PR.

I've also renamed ParameterHandler.get_matches() to find_matches() for consistency with ToolkitWrapper.find_smarts_matches() because I thought it was more representative of what it is actually doing, but I can revert that commit if you don't agree.

This pull request fixes 1 alert when merging 6fbe300b40d8187e41b245a7323c24ce8afc406e into 8ea7a09e9fdba58a0e2567da7be37e94a675aa7c - view on LGTM.com
fixed alerts:

1 for Unused import

Comment posted by LGTM.com

Codecov Report

:exclamation: No coverage uploaded for pull request base (topology@8ea7a09). Click here to learn what that means.
The diff coverage is 81.48%.

@@             Coverage Diff             @@
##             topology     #217   +/-   ##
===========================================
  Coverage            ?   72.97%
===========================================
  Files               ?       16
  Lines               ?     4911
  Branches            ?        0
===========================================
  Hits                ?     3584
  Misses              ?     1327
  Partials            ?        0

Impacted Files
Coverage Δ

...enforcefield/typing/engines/smirnoff/forcefield.py
68.4% <0%> (ø)

...enforcefield/typing/engines/smirnoff/parameters.py
75.26% <83.01%> (ø)

Continue to review full report at Codecov.

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

After conversation in #216, I've decided to implement Topology.angles/propers/impropers properties for now returning tuples of Atoms, similar to what's currently implemented in Molecule. This check will be important to include in the release. In the future, we may want to have Angle/Proper/Improper classes similar to Bond to make things a little more consistent.

Please remember to add a note to the docstring for any methods or properties where the API may not be final!

@j-wags can correct me if I'm wrong, but I'm not sure we're at that level yet, meaning that I think we're planning to generally label the API as unstable for the upcoming release.
I think it's still hard to make guarantees about what's not going to change signature in the next versions except maybe for a few methods such as ForceField.create_openmm_system() and Molecule.from_anything(), which are needed for the most common use cases of the toolkit. We're planning to take a systematic pass at the API in the short future though.

Reactivating the valence terms caused some tests to fail. Most of them complain about improper torsions that are not parametrized, but also some proper torsions as well. I'll check a few specific cases to see if this is a bug or something that we want to ignore.

This pull request introduces 6 alerts and fixes 1 when merging 426e345e0843623a3f54082cc5696782eb99325c into 8ea7a09e9fdba58a0e2567da7be37e94a675aa7c - view on LGTM.com
new alerts:

6 for Property in old-style class

fixed alerts:

1 for Unused import

Comment posted by LGTM.com

I think we're planning to generally label the API as unstable for the upcoming release.

Right. I get the sense that we're applying "unstable API" warnings to 80% of functions right now, with no clear pathway to un-labeling them as such. If we define a process for removing the label, then it makes more sense.
Before I started typing this, I had probably agreed with @andrrizzi -- Everything will be labeled as "unstable", and then the day we do the stable release, all the labels are removed. However, maybe it would make more sense to have a meeting after the initial release, and select different chunks of the API to "stabilize" as part of different milestones. This way, we could have focused discussions about individual areas of the API and remove the "unstable" labels one at a time.
tl;dr - Let's apply "unstable API" labels to individual functions. Later, we can come through and remove them one-by-one as we "stabilize" different areas.

Impropers not parameterized is OK -- impropers are optional, essentially. But propers not parameterized is concerning.

This pull request introduces 6 alerts and fixes 1 when merging 20ea3c02ec3c3df2ad75ddec6ac0ad4f8ad2860e into 8ea7a09e9fdba58a0e2567da7be37e94a675aa7c - view on LGTM.com
new alerts:

6 for Property in old-style class

fixed alerts:

1 for Unused import

Comment posted by LGTM.com

This pull request introduces 6 alerts and fixes 1 when merging a21ae75248c1fd042c22faac8f3e32584d7c0498 into 8ea7a09e9fdba58a0e2567da7be37e94a675aa7c - view on LGTM.com
new alerts:

6 for Property in old-style class

fixed alerts:

1 for Unused import

Comment posted by LGTM.com

Thanks for the fix, @j-wags! Good to know that our tests caught that. This is ready for merging on my end.

This pull request introduces 6 alerts and fixes 1 when merging 826716bf21f7716414834cc181726b8787b7928c into 8ea7a09e9fdba58a0e2567da7be37e94a675aa7c - view on LGTM.com
new alerts:

6 for Property in old-style class

fixed alerts:

1 for Unused import

Comment posted by LGTM.com

If we are not certain that most of the API will be stable at release, we really should NOT be calling this a 1.0 alpha release, since it's absurd to have to go through some crazy gymnastics of a few dozen alpha bugfix, feature, and API breaking releases and then switch to semantic versioning. Let's stick with 0.y.z releases until we feel that much of the API is stable, and not worry about whether or not the April release is "1.0 anything". It's just the first release with RDKit support and a more coherent object model for the parameter set, but it can be 0.y.z. Feature and API breaking releases can increment y, and bugfixes can increment z. If we release 1.0.0 in year one, I think we will still be in great shape.
In addition, we want to label the docstrings of classes and functions that have unstable APIs with standard warnings. We can remove those as parts of the API stabilize.
TensorFlow has been in a similar situation for a while, and we are a much smaller operation, so there is nothing to be ashamed about here. We just need to be very clear in how we communicate this to our users

I agree with @jchodera that the upcoming release should be 0.2.0, and that we can build toward 1.0.0. It's OK if the majority of the API is labeled as unstable for 0.2.0, but we should have a plan to collect API feedback after the initial release, and have a schedule that will have us finalizing it in manageable-sized chunks.

Sounds like a great plan, @j-wags.
We should also be sure to craft a release announcement for the News page that lays out some details of where we are, what feedback we'd like (and how), and where we're going (and how to check the Milestones).

we really should NOT be calling this a 1.0 alpha release

Absolutely agree.
Thank you all for the reviews!


T22162 Patch Eigen for ninja support

didn't see earlier but there is the same fix: https://github.com/MITK/MITK/pull/132

Closing this one as #132 fixes the same issue.


Listeners remove html attributes

Test case: https://codepen.io/David-Brooks/pen/yLGbKeG
It's all there in my test case but the crux of it is that as soon as I add the listener code (for example let events = listeners({ log });) certain attributes are arbitrarily removed from the rendered html. My test case shows that the class has been removed from the spans and also some of the data- attributes are removed but weirdly some data- attributes remain.
I'm happy to submit any more information if required, just let me know.

Hi again. I realise that is project is maintained by one guy (@cferdinandi) so I understand that fixes can sometimes take a long time and that's fine, it's "free" software after all. I was just wondering if this project is still being developed?
It's fine if it isn't I just need to know if I should look elsewhere or hold out for a fix.
Thank you.

@davidbrooksio it is, but I've had 0 time to look into this issue. Thanks for the reduced test case. I can clearly see the issue in action. I'll try to debug this soon.

Hi @cferdinandi, thank you for your reply. I know what it's like maintaining projects and juggling everything else so I hope you don't think I was being pushy. Please let me know if you need me to test anything.

@davidbrooksio Alright, just pushed version 12.3.2, which fixes this issue. Thanks for letting me know about it!


NullReferenceException in SessionLocator.DisposeSession()

Hi recently we are seeing NRE in SessionLocator.DisposeSession(). As we use Debug binaries, we managed to pinpoint the exact line that causing the issue:
_contentManagerSession.Clear();
I looked at all references to _contentManagerSession and it seems to me that the only possible explanation is that RequireNew()/RequireNew(...)/Cancel()/Dispose() was called before GetSession() is called. I'm not exactly sure how this (sequence of calls) would happen, but it's happening to us. I'm proposing a simple fix by using the new C# 6.0 syntax:
_contentManagerSession?.Clear();
Please let me know your thoughts.

I think it's a good catch.
There are some other issues related to this one. See #6555 where, because i couldn't repro myself, i just also proposed this null check _session.Transaction != null. (may also be useful)
But i didn't think about _contentManagerSession, maybe because it seemed to me as a normal IDependency... This could explain some issues where sessions are not correctly disposed...
But how can _contentManagerSession be null? Calling _contentManagerSession.Clear() before doesn't explain this. But your debugging proves it.
Right now, just seen that it is not directly injected. It's an auto generated (by autofac) delegate factory func _contentManagerSessionFactory that is injected and then used to construct _contentManagerSession when creating a new session.
Maybe a thread safety problem when using this factory...
Best.

@jtkech
To help anyone understand how we found this issue, here is the call stack when exception was thrown:
2016-04-07 00:00:53,433 [125] Orchard.Tasks.BackgroundService - (null) - ERROR Error while processing background task "Orchard.Indexing.Services.IndexingBackgroundTask" on tenant "Default".
System.NullReferenceException: Object reference not set to an instance of an object.
   at Orchard.Data.TransactionManager.DisposeSession() in C:\projects\orchard\src\Orchard\Data\SessionLocator.cs:line 96
   at Orchard.Data.TransactionManager.RequireNew(IsolationLevel level) in C:\projects\orchard\src\Orchard\Data\SessionLocator.cs:line 68
   at Orchard.Data.TransactionManager.RequireNew() in C:\projects\orchard\src\Orchard\Data\SessionLocator.cs:line 64
   at Orchard.Tasks.BackgroundService.Sweep() in C:\projects\orchard\src\Orchard\Tasks\BackgroundService.cs:line 38

To answer "how can _contentManagerSession be null", here is my proposed explanation:

_contentManagerSession can only be set when EnsureSession() is called, which is in turn called by GetSession(), Demand(), and RequireNew(...). However, Demand() is not called by anyone other than GetSession(), and RequireNew(...) would call DisposeSession() first and throw an NRE inevitably, the only caller that will safely set _contentManagerSession is GetSession().
I analyzed the calls to GetSession(), and my conclusion is that it is called only when anyone is trying to touch data (get/update/remove). There are framework references from these classes:

DefaultContentManager
DefaultDataMigrationInterpreter
SessionLocator
TransactionFilter

On the other hand, DisposeSession(), which calls _contentManagerSession.Clear(), is called by RequireNew()/RequireNew(...)/Cancel()/Dispose().
If for somehow, any of the above is called before GetSession(), say ITransactionManager.RequireNew(), _contentManagerSession will be null when executing _contentManagerSession.Clear() from DisposeSession(). Looking at the code in BackgroundService.Sweep() and its (theoratical) call stack starting from SweepGenerator.Elapsed(), it seems to me that this is destined to happen, if no other code will be involved.

Does anyone think of a reason that this BackgroundService.Sweep() thing shouldn't be happening this way? Otherwise, I will prepare a fix based on 1.10.x if that's the branch this issue should target.

PR #6722 is prepared

Anyway your fix is good but, if possible, would be good to know the root cause.
So, i try to understand your analysis. When DisposeSession() is called, if _contentManagerSession is null (not already set), "normally", _session is also null, so _contentManagerSession.Clear() is not called...
Except if there are concurrent accesses of the same instance (but it would not be normal), one having set _session but not yet _contentManagerSession, and another executing DisposeSession()...
Best

@jtkech Ah, now I see your point and why you would think of race condition. However, yet following your insights, I found this exception in the log, which will well explain why the code path has ever gone down to _contentManagerSession.Clear() when _session is not null but _contentManagerSession is.
2016-04-07 00:18:53,613 [163] Orchard.Tasks.BackgroundService - (null) - ERROR Error while processing background task "Orchard.Alias.Implementation.Updater.AliasHolderUpdaterTask" on tenant "Default".
NHibernate.TransactionException: Begin failed with SQL exception ---> System.InvalidOperationException: Timeout expired.  The timeout period elapsed prior to obtaining a connection from the pool.  This may have occurred because all pooled connections were in use and max pool size was reached.
   at System.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at System.Data.SqlClient.SqlConnection.TryOpenInner(TaskCompletionSource`1 retry)
   at System.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)
   at System.Data.SqlClient.SqlConnection.Open()
   at System.Data.SqlClient.SqlConnection.Open()
   at NHibernate.Connection.DriverConnectionProvider.GetConnection()
   at NHibernate.AdoNet.ConnectionManager.GetConnection()
   at NHibernate.Transaction.AdoTransaction.Begin(IsolationLevel isolationLevel)
   --- End of inner exception stack trace ---
   at NHibernate.Transaction.AdoTransaction.Begin(IsolationLevel isolationLevel)
   at NHibernate.Impl.SessionImpl.BeginTransaction(IsolationLevel isolationLevel)
   at Orchard.Data.TransactionManager.EnsureSession(IsolationLevel level) in C:\projects\chichard\src\Orchard\Data\SessionLocator.cs:line 114
   at Orchard.Data.TransactionManager.RequireNew(IsolationLevel level) in C:\projects\chichard\src\Orchard\Data\SessionLocator.cs:line 69
   at Orchard.Data.TransactionManager.RequireNew() in C:\projects\chichard\src\Orchard\Data\SessionLocator.cs:line 64
   at Orchard.Tasks.BackgroundService.Sweep() in C:\projects\chichard\src\Orchard\Tasks\BackgroundService.cs:line 38

Now looking at EnsureSession():
        private void EnsureSession(IsolationLevel level) {
            // ...
            // not so important code, ellipsed
            // ...
            _session = sessionFactory.OpenSession(new OrchardSessionInterceptor(_interceptors.ToArray(), Logger));
            _session.BeginTransaction(level);
            _contentManagerSession = _contentManagerSessionFactory();
        }

If _session.BeginTransaction(level) throws exception, as indicated from the log, _contentManagerSession will not be set and remain null, yet _session is assigned. So a more logical fix would put a try-catch block around _session.BeginTransaction(level) (or even including the line above) to set _session back to null if anything goes wrong.

You saved my night because i was trying to find another source, e.g around dependency injection...
Indeed, this is one of the possible reason i thought, but because nobody reported this exception... What's good with you is that you report real facts with which we can progress...
So, let me think about this, maybe we can enhance your related PR, e.g with a try / catch. I also think about a null check  on _session.Transaction before checking _session.Transaction.IsActive. This e.g when not in a try block as in Cancel()...
Best

@jtkech
I have updated the PR with more changes around accessing _session and _session.Transaction. Please let me know your thoughts. How would you suggest me doing the testing? I'm going to make sure the normal case is still working, but I'm not sure if I should test against the exact situation we are preventing from (say DB connection pool full)

Ok.
There would be so many cases to check. So, the goal is to let things as much as possible as they was. We still have to let nhibernate surface its exceptions but prevent our own exceptions. This to not mix problems.
So, maybe the best is to only do your first null check. But it was so important to know the root cause. I'll take a look directly on the PR.
See also #5653, it may help you to configure retries logic for connections.
Best.

Hi @jtkech I did update the PR to reflect your latest comments. I'm taking the advice on new syntax as it makes sense to enable people who use VS2013 to build it. Also now the fix relies on global exception handling to dispose the session object when exception is thrown from BeginTransaction.


dart.exe consuming my all ram.how to fix?

Thank you for taking the time to file an issue!
This tracker is for issues related to:

Analyzer
Common Front End (CFE) and kernel
Dart core libraries ("dart:async", "dart:io", etc.)
Dart VM
dart2js
dev_compiler

Some other pieces of the Dart ecosystem are maintained elsewhere. Please
file issues in their repository:

Dart language: https://github.com/dart-lang/language
Dart website: https://github.com/dart-lang/site-www
Flutter: https://github.com/flutter/flutter
Linter: https://github.com/dart-lang/linter
dart format: https://github.com/dart-lang/dart_style

If you aren't sure, file the issue here and we'll find the right home for it.
In your issue, please include:

Dart SDK Version (dart --version)
Whether you are using Windows, MacOSX, or Linux (if applicable)
Whether you are using Chrome, Safari, Firefox, Edge (if applicable)

Missing some or all of the above might make the issue take longer or be
impossible to act on.
If you simply have a question, consider starting with Stack Overflow:
https://stackoverflow.com/questions/tagged/dart
For a real-time response, consider our Gitter channel:
https://gitter.im/dart-lang/sdk

Marking with needs-info. The bot will autoclose the issue if no information is provided within few days. I also took a liberty to prettify the title a bit.


Not able to recover the wallet using seed words.

https://github.com/bithyve/bitcoin-keeper/assets/62699947/c31e4bfd-5085-4c87-9ce4-06b38e1fafde

Getting error message as "Failed to get app image"

Not able to reproduce this issue again in 1.1.7(262)


cannot import name '_jpegpy' from 'lib.data_preprocess.utils.jpegpy'

Hi，I have a question.can you help me, please?

You can refer to the following closed Issues.
"How do I generate so files"


Create layout mockups for Design System website

Looks like this is complete. Feel free to reopen as needed.


Ping ranges aren't filled in with defaults on new configurations

When a user uses the bot for the first time and a new configuration is generated, the ping ranges aren't filled with default values, the same default values that get assigned by pushing the Reset to Defaults button in the config window.

Resolved in 7f46e4830cda27cf8b810a0f76faf24bd80953b8.


Fixing response status line parsing

Fixing response status line parsing, so that status line is parsed correctly even if Reason-Phrase is missing. HTTP spec says the Reason-Phrase is optional.
Doneness:

[X] Build is okay - I made sure that this change is building successfully.
[X] No Bugs - I made sure that this change is working properly as expected. It doesn't have any bugs that you are aware of.
[ ] Branching - If this is not a hotfix, I am making this request against master branch

:white_check_mark: Build Titanium-Web-Proxy 3.0.646 completed (commit https://github.com/justcoding121/Titanium-Web-Proxy/commit/0ef5ba34f7 by @)


Regarding Multilabel classification

Hello,
I wanted to know about multi label classification implementation. Did you implement Hierarchical attention mechanism with multi label classification? The reason why I am asking is that I searched a lot through internet and did not find any implementation. I have not looked through the code but if you give positive reply, I will start looking at it because this is exactly what I wanted to achieve.
Next question is I looked at your data folder. There is a csv file with labels and another with data. Right now I have a excel file that I am attaching here. The excel file has multiple columns with 1 indicating the topic the article belongs to and 0 indicating not. Could you please suggest me with your code how can i implement it?
train.xlsx

Upon downloading, I figured it out. It indeed is a multi label classification.........


CSS does not load in Chrome when HTML report is hosted

Describe the bug
After creating a Kover HTML report, we host it on our build server for later viewing. However, Chrome is blocking the CSS from loading when it is opened from this hosted location.
Errors

Refused to apply inline style because it violates the following Content Security Policy directive: "style-src 'self'". Either the 'unsafe-inline' keyword, a hash ('sha256-8EpQVk5wr2+8kqUUsnmqEFhEfR2/zDth+L/PopUdkMo='), or a nonce ('nonce-...') is required to enable inline execution.

Expected behavior
CSS is able to be loaded when viewing the HTML page.
Environment

Kover Gradle Plugin version: 0.7.3
Gradle version: 8.0
Kotlin project type: Kotlin/Android
Coverage Toolset (if customized in build script): Kover
Other context important for this bug: Google Chrome 116.0.5845.187

Relates #272


Harvester Plugin

Supersedes #6210.
I've reviewed previous changes and made more which should be reviewed by @mantis-toboggan-md
Briefly this includes further work on splitting out store, handling pkg routing and removal of loadVirtual action.
More specifically...

The harvester store now fully lives in the harvester plugin
loadVirtual has been removed. The harveter pkg now uses the new standard loadCluster --> pkg loadCluster flow
Fixed a lot of routing issues

general updates for c-cluster-product --> pkg-c-cluster. Includes ensuring harvester resource models have correct references to routes via base abstract class
harv landing page large resource cards
Fix RBAC binding/project/namespace create/edit/cancel/done flows. This isn't perfect (contains 'if harvester' style switches in models for project, namespace and management.cattle.io.clusterroletemplatebinding).

Fixed some merge conflicts
Added harverster cluster actions back in (woops)
Fixed css import in pkg/harvester/components/SerialConsole/index
I've reverted a previous change to isHarvesterCluster which might make harvester clusters not show up in the virtualisation manager cluster list that have been deployed via the instructions at https://docs.harvesterhci.io/v1.0/dev/dev-mode. This can be fixed by applying the same change as in Step 4 from those instructions to the local cluster's mgmt cluster harvester resource

I spoke with the harvester team and they're offered to help. At some point I'd like them to look at areas marked with // FIXME: Harvester. At the moment these are just area's that i couldn't test.
Other general areas it would be good for the harvester team to test at some point

Upgrade harvester notification/features
WebVNC & Serial Console features
Navigating between harvester clusters in the multi-cluster environment

In terms of local testing, i've tried both multi and single cluster builds and nothing obvious fails.
The package builds, but didn't get a change to test it. Hopefully some of the changes in this PR result in the navigation working correctly.
Still to do (in this PR)

Fix the failing check plugin gate (this looks like similar issues that we see over at https://github.com/rancher/ui-plugin-examples/pull/1)

Still to do (in another PR)

Split out l10n
Split out harvester manage into it's own plugin
Address styling issues

Ensure the pkg can be loaded into the dashboard instead of bundled with it

I tried running the plugin with a standalone harvester instance and rancher w/ two imported harvester clusters, testing both the bundled plugin mode and built-then-imported plugin mode. For the most part things worked w/ the harvester plugin bundled, I just found one issue: I see console errors when navigating from a harvester cluster back to the virtual management product via top level nav. It looks like we're trying to fetch some harvester resources after the harvester steve store is reset? I wasn't able to track down the cause today, though.

The package builds, but didn't get a change to test it. Hopefully some of the changes in this PR result in the navigation working correctly.

Alas, no. I see the same loading error, Type harvesterhci.io.dashboard not found ...I was able to track down the differences between this branch and the pre-rebase work: there we have list/harvesterhci.dashboard.vue and in this PR we have list/harvesterhci.io.dashboard/index.vue. I clicked around a little more and noticed that nested dynamically imported resource files aren't working generally, e.g. the VM custom edit page is not detected.

I fixed the "unknown schema..." error screenshotted above, as well as one other issue I found in the process- we were running the harvester loadCluster action twice on initial navigation into an imported cluster.


too slow when downloading tensorflow

I use Thunder to accelerate when downloading software. I just want to know, if I download tensorflow from Kitematic,  how can I use this accelerator? 3x

Unfortunately the image for Tensorflow is quite large 1-2GB and there isn't much we can do on our side.
You should open an issue with their repo to have them slim it down, as many improvements could be made.
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/docker/Dockerfile