Monaco editor does not follow the theme

Problem/Motivation
In Node-RED v2.0.0+, you can enable the Monaco editor (known from VSCode) by editing the settings. However, when you do so that editor does not follow the theme and is instead using a light theme by default:

Theme version: v2.0.1
Node-RED version: 2.0.2
Platform/OS: Linux Mint 20.2
Browser: Firefox 90.0

Steps to reproduce
As noted in the first link above, edit your settings.js file to include
        codeEditor: {
            lib: "monaco"
        },

inside the editorTheme object and restart Node-RED.
Open the settings of a node with a large text field, such as a function or template node (or click the "..." next to a small property field).
Proposed changes
I don't know how to change the theme of the editor, but given the variety of themes available in VSCode it should be possible.

Monaco themes are also set in the settings.js file.
See https://github.com/node-red/node-red/blob/master/packages/node_modules/node-red/settings.js#L355-L361

Ah, that wasn't explained in the release notes and since I didn't regenerate my settings I did not know that option existed.
Any suggestions for a Monaco theme that works well with midnight-red?

I'm using
theme: "tomorrow-night-bright",

Yeah, that works. I think I prefer "tomorrow-night" though, which weirdly is a bit brighter than "tomorrow-night-bright".


Azure Data Factory to Data Landing Zone not supported

A client is trying setting up an Azure Data Factory to DLZ file transfer.
ADF SAS requires storage level access: "The type property must be set to AzureBlobStorage (suggested) or AzureStorage (see the following note). 1".
As per this case 2 Adobe does not support storage level access: "we can't provide storage level access to DLZ"
Could we include such explicit limit in the very same way the article includes "Private links" limit important remark?
Issue in ./help/sources/connectors/cloud-storage/data-landing-zone.md

Thank you for submitting this issue. We will review the documentation and make the necessary updates as quickly as possible. Tracking internally in PLAT-197240.

We've added guidance on how to use DLZ and ADF concurrently. Closing this issue.


Jesse: Watch pickleballopinion.com

Jesse requests the watch of the watch_keyword pickleballopinion\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

pickleballopinion\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Fix typo in url stream notebook (#395)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.78%. Comparing base (90b2f0c) to head (798c8ce).
Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #396   +/-   ##
========================================
  Coverage    89.78%   89.78%
========================================
  Files          120      120
  Lines         7653     7653
  Branches       675      675
========================================
  Hits          6871     6871
  Misses         651      651
  Partials       131      131

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


can we assert text in pdf file downloaded after clicking link on page

Selenium can not interact with pdf files.  You will need to find a library that can and use it in your tests after you download the file.

thanks, i will try to find any such library and will update here as well so that in future if anybody need can use that library.

Sounds good!


Add btcpayserver

Motivation for this change
Adds package btcpayserver and its dependency, nbxplorer
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

there are some impurities when I try to run it:
jb55@monad> cd ./result/bin
jb55@monad> ./btcpayserverbitcoin/result/bin
which: no btcpayserver in (/usr/local/heroku/bin:/home/jb55/.ruby/1.8/gems/bin:/home/jb55/.npm/bin:/home/jb55/dev/gocode/bin:/home/jb55/.ruby/1.8/gems/bin:/home/jb55/.npm/bin:/home/jb55/dev/gocode/bin:/home/jb55/.local/bin:/home/jb55/bin:/run/wrappers/bin:/home/jb55/.nix-profile/bin:/etc/profiles/per-user/jb55/bin:/nix/var/nix/profiles/default/bin:/run/current-system/sw/bin:/home/jb55/.rvm/bin:/home/jb55/.fzf/bin)
readlink: missing operand
Try 'readlink --help' for more information.
dirname: missing operand
Try 'dirname --help' for more information.
./btcpayserver: line 4: dotnet: command not found

Fwiw, I can build it on archlinux with sandbox and I can successfully play around with btcpayserver after installing dotnet-sdk_3. Is that dotnet-sdk_3 runtime dependency necessary? The wasabibackend package doesn't seem to require this.

I think it is since this happened https://github.com/NixOS/nixpkgs/pull/95104#discussion_r471132008

Is it true that the nbxplorer package is independent and is not a dependency of btcpayserver?

It's possible to run nbxplorer without btcpayserver as stated in the nbxplorer readme though I don't think there are many that are doing so.

I think it is since this happened #95104 (comment)

Nice, the fix works for me.

Very nice improvements @erikarvstedt. Agree that they should be added to this PR. I also like that you kept the common-updater very general and simplified the build phase.
My generated deps.nix differs in the order of the listed dependencies because this sort on my machine puts . before /. Perhaps need to set LC_ALL? I tried a few values but couldn't reproduce your order.

The order was produced by en_US.UTF-8. Fixed, thanks for catching this!

@erikarvstedt I'll just cherry-pick your squashed commits here if that's ok

@kcalvinalvin, sure, that's what I had prepared the squashed branch for :cat:
Could you also add the latest changes? Just cherry-pick the squashed branch again.

The order was produced by LANG=en_US.UTF-8. Fixed, thanks for catching this!

Fix works for me. I can reproduce exactly the same deps.nix now.

I rebased my module on @erikarvstedt's changes. The speed and size improvements are great.
Everything compiles fine, but when I try to connect to the BTCPayServer URL I get the following error:
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]: fail: Microsoft.AspNetCore.Server.Kestrel: Connection id "0HM2JQQG2QAKN", Request id "0HM2JQQG2QAKN:00000002": An unhandled exception was thrown by the application.
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]: System.NullReferenceException: Object reference not set to an instance of an object.
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at BundlerMinifier.FileHelpers.NormalizePath(String path)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at BundlerMinifier.FileHelpers.TrimTrailingPathSeparatorChar(String path)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at BundlerMinifier.FileHelpers.DemandTrailingPathSeparatorChar(String path)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at BundlerMinifier.TagHelpers.BundleTagHelper.GetSrc(String path)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at BundlerMinifier.TagHelpers.BundleTagHelper.Process(TagHelperContext context, TagHelperOutput output)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Razor.TagHelpers.TagHelper.ProcessAsync(TagHelperContext context, TagHelperOutput output)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Razor.Runtime.TagHelpers.TagHelperRunner.RunAsync(TagHelperExecutionContext executionContext)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at AspNetCore.Views_Shared_Header.ExecuteAsync() in /tmp/nix-build-btcpayserver.drv-0/source/BTCPayServer/Views/Shared/Header.cshtml:line 19
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Razor.RazorView.RenderPageCoreAsync(IRazorPage page, ViewContext context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Razor.RazorView.RenderPageAsync(IRazorPage page, ViewContext context, Boolean invokeViewStarts)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Razor.RazorView.RenderAsync(ViewContext context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.TagHelpers.PartialTagHelper.RenderPartialViewAsync(TextWriter writer, Object model, IView view)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.TagHelpers.PartialTagHelper.ProcessAsync(TagHelperContext context, TagHelperOutput output)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Razor.Runtime.TagHelpers.TagHelperRunner.<RunAsync>g__Awaited|0_0(Task task, TagHelperExecutionContext executionContext, Int32 i, Int32 count)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at AspNetCore.Views_Shared__LayoutSimple.<ExecuteAsync>b__10_0()
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Razor.Runtime.TagHelpers.TagHelperExecutionContext.SetOutputContentAsync()
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at AspNetCore.Views_Shared__LayoutSimple.ExecuteAsync() in /tmp/nix-build-btcpayserver.drv-0/source/BTCPayServer/Views/Shared/_LayoutSimple.cshtml:line 2
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Razor.RazorView.RenderPageCoreAsync(IRazorPage page, ViewContext context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Razor.RazorView.RenderPageAsync(IRazorPage page, ViewContext context, Boolean invokeViewStarts)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Razor.RazorView.RenderLayoutAsync(ViewContext context, ViewBufferTextWriter bodyWriter)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Razor.RazorView.RenderAsync(ViewContext context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.ViewFeatures.ViewExecutor.ExecuteAsync(ViewContext viewContext, String contentType, Nullable`1 statusCode)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.ViewFeatures.ViewExecutor.ExecuteAsync(ActionContext actionContext, IView view, ViewDataDictionary viewData, ITempDataDictionary tempData, String contentType, Nullable`1 statusCode)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.ViewFeatures.ViewResultExecutor.ExecuteAsync(ActionContext context, ViewResult result)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.ViewResult.ExecuteResultAsync(ActionContext context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeNextResultFilterAsync>g__Awaited|29_0[TFilter,TFilterAsync](ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.Rethrow(ResultExecutedContextSealed context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.ResultNext[TFilter,TFilterAsync](State& next, Scope& scope, Object& state, Boolean& isCompleted)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.InvokeResultFilters()
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]: --- End of stack trace from previous location where exception was thrown ---
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeNextResourceFilter>g__Awaited|24_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.Rethrow(ResourceExecutedContextSealed context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.InvokeFilterPipelineAsync()
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]: --- End of stack trace from previous location where exception was thrown ---
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeAsync>g__Awaited|17_0(ResourceInvoker invoker, Task task, IDisposable scope)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Session.SessionMiddleware.Invoke(HttpContext context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at BTCPayServer.Hosting.BTCPayMiddleware.Invoke(HttpContext httpContext) in /tmp/nix-build-btcpayserver.drv-0/source/BTCPayServer/Hosting/BTCpayMiddleware.cs:line 83
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Diagnostics.StatusCodePagesMiddleware.Invoke(HttpContext context)
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at BTCPayServer.Hosting.HeadersOverrideMiddleware.Invoke(HttpContext httpContext) in /tmp/nix-build-btcpayserver.drv-0/source/BTCPayServer/Hosting/HeadersOverrideMiddleware.cs:line 29
Sep 08 09:07:39 nix-bitcoin btcpayserver[6206]:    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpProtocol.ProcessRequests[TContext](IHttpApplication`1 application)

Could this have to do with nbxplorer: only install output of 'dotnet publish' and the fact that we are no longer copying over all the .cs files in BTCPayServer/Hosting?

@nixbitcoin I am also getting the same issue. Was able to fix the issue by reverting the installPhase back to the one I have right now.
In the btcpayserver repo, they just have a 3 line shell script which I just copied. I think this should just be followed?

Pushed new commits that are based off of @erikarvstedt's work

Merged the new commits and checked that the webfront works

@kcalvinalvin, thanks!
Please add this final change, which sets DOTNET_CLI_TELEMETRY_OPTOUT=1 in the btcpayserver run script. (This isn't needed for nbxplorer because it's built with dotnet publish.)

@erikarvstedt done :)

@erikarvstedt I can do that. But I'm not seeing any version changes with git diff. Do let me know if commit hashes 2f55d6f697396e18f6ae644754013e47d9c647d2 and 2724f7be543ebc57a01f7fc4fa42efb0a13e24cc are the ones you intend for me to pull in.
-  pname = "btcpayserver";
+  name = "btcpayserver";
   version = "<IP_ADDRESS>";

   src = fetchFromGitHub {
-    owner = pname;
-    repo = pname;
+    owner = name;
+    repo = name;
     rev = "v${version}";
     sha256 = "11h1nrmb7f44msbhhiz9ddqh5ss2kz6d8ysnvd070x3xj5krgnxz";
   };
@@ -48,10 +48,10 @@ stdenv.mkDerivation rec {

   installPhase = ''
     cd ..
-    share=$out/share/$pname
+    share=$out/share/${name}
     mkdir -p $share
     mv -T source $share
-    install -D -m500 <(echo "$runScript" | sed "s|@@SHARE@@|$share|") $out/bin/$pname
+    install -D -m500 <(echo "$runScript" | sed "s|@@SHARE@@|$share|") $out/bin/${name}
   '';

   dontStrip = true;
diff --git a/pkgs/applications/blockchains/nbxplorer/default.nix b/pkgs/applications/blockchains/nbxplorer/default.nix
index 76ca21ef9aa..3c87de8d7b3 100644
--- a/pkgs/applications/blockchains/nbxplorer/default.nix
+++ b/pkgs/applications/blockchains/nbxplorer/default.nix
@@ -14,7 +14,7 @@ let
 in

 stdenv.mkDerivation rec {
-  pname = "nbxplorer";
+  name = "nbxplorer";
   version = "2.1.42";

   src = fetchFromGitHub {
@@ -35,11 +35,11 @@ stdenv.mkDerivation rec {
     nuget init ${linkFarmFromDrvs "deps" deps} $TMP/nuget

     dotnet restore --source $TMP/nuget NBXplorer/NBXplorer.csproj
-    dotnet publish --no-restore --output $out/share/$pname -c Release NBXplorer/NBXplorer.csproj
+    dotnet publish --no-restore --output $out/share/$name -c Release NBXplorer/NBXplorer.csproj
   '';

   installPhase = ''
-    makeWrapper $out/share/$pname/NBXplorer $out/bin/$pname \
+    makeWrapper $out/share/$name/NBXplorer $out/bin/$name \

Yes, these changes are correct.
With pname the derivation name is automatically set to ${pname}-${version}.

With pname the derivation name is automatically set to ${pname}-${version}.

TIL. Changes applied

Thank you, great work!


loading intellisense status

Type: Bug
I dont't know why
VS Code version: Code 1.70.0 (Universal) (da76f93349a72022ca4670c1b84860304616aaa2, 2022-08-04T04:38:55.829Z)
OS version: Darwin arm64 21.3.0
Modes:

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
18, 18, 13

Memory (System)
16.00GB (0.14GB free)

Process Argv
--crash-reporter-id 9d661fd0-fcaa-484a-a564-4cc48ee598b2

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

vscode-tailwindcss
bra
0.8.6

vscode-eslint
dba
2.2.6

gitlens
eam
12.2.2

prettier-vscode
esb
9.8.0

python
ms-
2022.14.0

vscode-pylance
ms-
2022.9.10

vs-code-prettier-eslint
rve
5.0.4

code-spell-checker
str
2.7.2

vscode-js-console-utils
wht
0.7.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411cf:30557515
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vscaac:30438847
pylanb8912:30545647
vsclangdf:30486550
c4g48928:30535728
hb751961:30553087
dsvsc012:30540252
azure-dev_surveyone:30548225
2144e591:30553903

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.71.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

What language is this for?

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Docs for relation manager: Add missing "?"

When copying the example for "Customizing the relation manager title" the following error occurs:
Type of ...\RelationManagers\ContributionsRelationManager::$title must be ?string (as in class Filament\Resources\RelationManagers\RelationManager)

This just adds the missing "?" before string in the docs.

Thanks


sysutils/monit: Allow source address to be specified for Ping check

I want to ping the remote firewall on a IPsec tunnel.
I need to specify a source address as it is not possible to ping directly from one firewall to another over IPsec (https://github.com/opnsense/core/issues/1624)

monit is now part of core :D

Moved to https://github.com/opnsense/core/issues/2632


Write design doc for azure-spring-cloud stream

Parent issue: #23402
Sample design doc: https://github.com/Azure/azure-sdk-for-java/wiki/Spring:-Design-doc-of-level-1-azure-spring-cloud-starter

Close this issue for design doc has been finished: https://github.com/Azure/azure-sdk-for-java/wiki/Spring-Cloud-for-Azure-Messaging-design


0.3.0 Able to play one-shots but not select Soundsets

Describe the bug
Using module 0.3.0, Foundry v9 on Forge in Chrome on Mac, the Soundset pull-down shows all available soundsets, but when a soundset is selected, the drop down closes and returns to the default "-no soundset-" selection. Moods remain unselectable.
Global one-shots load in the Ambient Sound browser and can be played, so API key appears to be functioning.
Details

OS: macOS 12.2
Chrome Version 99.0.4844.51
Foundry VTT v9 on Forge
Module version: 0.3.0
Does not work in safety mode
[X] I did read FAQ - https://github.com/frondeus/fvtt-syrin-control/wiki/Frequently-Asked-Questions
-->

Can you select a soundset/mood in the playlist?
Is it only related to scene config?
Anyway: That sounds not right. Definitely. Could you send me what the logs say?

Hello,
I am having the same issue. This is in the main sound tab, though the scene ambiance dialog box acts the same.
I tried downloading Firefox and it appears to work as intended, however Chrome behaves as described above.
I can start / stop a previously saved playlist and the global oneshots.
There is no entry in the dev tools when I select the drop down item.

Hi, thanks for extra details. I will look into it soon.

I am also seeing this issue running on forge-vtt.com. If I can help please let me know.

I am having same issue.. if its related I am using their subscription package so my soundset options are huge

I am using Foundry app btw

Okay, sorry for the long waiting time.
Lets make a summary of #35, #38 and #42.
It seems to me that there is one bug occurring on Chrome/Chromium/Edge (edge is using chrome engine so it makes sense), and also probably only on macOS.
The problem is, I am unable to replicate the bug, however, I'm using Linux & Windows only.
First of all I am going to close other issues and keep this one only.

When you select a soundset, the SyrinControl sends a request to SyrinscapeAPI asking what moods are part of the set.
While waiting for the response, it blocks mood selection since it might be updated at any second.
Then it seems that the response never arrives, or it's invalid, or there is another nasty bug causing it to enter the invalid state.
Because of that, it never enables the dropdown again.
That's all that I know. I would have to know what the response was (if there was any response).
I'm sure your API keys are correct since you were able to retrieve a list of soundsets. That's something.
But I'm not sure why, oh why another request is causing problems only on Chrome.

On my Chrome it looks like this:

I am getting response for the request called ?soundset__id=... and the status is as expected: 200

I released https://github.com/frondeus/fvtt-syrin-control/releases/tag/0.3.1-alpha1-3
Could someone install it manually (not via ForgeVTT and not via Foundry update mechanism, just Drag n Drop into the foundry modules dir) and see if there are more logs? And paste them here :)?
It should look like this:

This link also should work if you want to install it via Foundry or Forge manager:
https://github.com/frondeus/fvtt-syrin-control/releases/download/0.3.1-alpha1-3/module.json

I just released version 0.3.1 it should fix the issue.


[BEAM-10402] Move @Nullable annotations adjacent to the type they annotate

This separates out a trivial piece of #13134, fixing annotations so they will not produce warnings when type checking is enabled.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Noting that this is an uninteresting commit from an approved PR.


Added ability to obtain "clientX" and "clientY" from mouse events

This change exposes the "clientX" and "clientY" properties of a DOM mouse event.
I need these for a game I'm working on (currently using unsafeEventNumberProp but this seems useful enough to include in this package).

Thank you :1:


DOCS: tidy up wording, spelling.

Looking good! Please squash the commits in to one and accurately document the areas that have changed.
This is because multiple PRs are open with the same/similar commit names but do not touch on what parts of the docs have been updated.

@SpiritLevel looks like you deleted your fork and so a lot of your PRs have closed and can't be merged :(

Will sort this. :)


Bump worker count of UI and API to 2

This allows for traffic to route to the second worker if the first
worker goes down, which is a typical HA setup.


[BUG] AI + MSAL in SPA throws "string value is too long" exception after login.

Description/Screenshot
When logging in to a SPA / JS application with MSAL (3.0.2) it passes the "code" back in the address bar as a GET parameter. Since upgrading to 3.x branch of ApplicationInsightsJS this now throws an uncaught exception where AI can't process the request as the URL is too long and it says it is truncating this to 1024 characters.

I'm ok with the string being truncated but having the uncaught exception thrown after every login is a bit annoying.
Steps to Reproduce

OS/Browser: Any
SDK Version [e.g. 22]: v3.0.3
How you initialized the SDK:

Expected behavior
Doesn't throw an error or allows us to configure a longer string length.

The error is being thrown because you have enabled the "enableDebug" setting turned on (it's off by default).
With this setting it will cause all calls to throwInternal to be thrown rather than just logged, if this setting to disabled it will just "log" the warning.
This affects not only v3.x but also v2.x because when we did not truncate the string it cause the logged event to be dropped.
I assume, you are requesting that we also provide the ability to selectively define at what level we should throw when the enableDebug is enabled?

Hey @MSNev ,
Thanks for that.

Yes please to the length if that's possible
Even if it is triggered by enableDebug, I wouldn't expect this to throw an "uncaught exception" - especially one I'm not sure if I'm meant to try and catch or not? This feels more of a warning kind of thing rather than an Exception?

For the length, that not possible as they are generic messages so they will need to continue calling the function they do now which (depending on the configuration) cause the SDK to emit events with the warning.
On the enableDebug its whole purpose (as currently defined) is to always throw with the value it would normally send as an event, so that it causes the uncaught exception. ie. the enableDebug SHOULD ONLY be used during development and shoudl not be enabled for a production deployment.
I'll tag this issue as an enhancement to provide an additional configuration to enable defining the minimum level to throw @ so if it doesn't throw it will continue to log the event.


Gawr Gura 4 Million Subscriber Celebration Project

Adapts last years tanabata celebration project for reuse.
Important things to note:

You'll have to update the gura3mil flag to be guratanabata in the prod database.
Make sure that the new project is id 14, if it isn't let me know and I'll change the code, because it makes that assumtion.
Sephi said they were going to create a new congratulations message for the bottom of the 4 million submissions, so don't merge this immediately.

If there's anything that needs changing, let me know.

Already done, it has both flags atm
Next id would be 16, so that needs to be changed

Will review this in two days prob, busy tmr.

Oh, also, the redirect also will need to change whenever the move to Payload happens, which should be fairly soon.


"control" error when processing ZNG file (regression at f56d310)

Repro is with Zed commit f56d310.
A community zync user was trying to perform a Zed lake migration which ran into a problem processing the attached 1.zng.gz which is a GZIP-compressed copy of a file from the branches/ directory for one of their pools.
$ zq -Z 1.zng.gz
{
    entry: {
        ts: 2022-12-13T21:56:05.933051284Z,
        name: "main",
        commit: 0x0000000000000000000000000000000000000000 (=ksuid.KSUID)
    } (=branches.Config)
} (=journal.Add)

At Zed commit 8dd27d9, it could be processed successfully with the following pipeline.
$ zq -version
Version: v1.2.0-130-g8dd27d98

$ zq -f text 'yield entry.name | sort | uniq' 1.zng.gz
main

However, at the next commit f56d310 which is associated with the changes in #4213, it produces some output but also an error message and ultimately a non-zero exit code.
$ zq -version
Version: v1.2.0-131-gf56d3102

$ zq -f text 'yield entry.name | sort | uniq' 1.zng.gz
main
1.zng.gz: control

$ echo $?
1

As the user also noticed, dropping the uniq makes it work ok.
$ zq -f text 'yield entry.name | sort' 1.zng.gz
main

$ echo $?
0

The problem is still with us at the current tip of main, which is currently commit e01e9f5.
$ zq -version
Version: v1.6.0-17-ge01e9f54

$ zq -f text 'yield entry.name | sort | uniq' 1.zng.gz
main
1.zng.gz: control

I just realized that for now I can work around this by invoking zq twice.
$ zq -z 'yield entry.name | sort' 1.zng.gz | zq -f text 'uniq' -
main

$ echo $?
0

Verified in Zed commit 0d949c4.
Now the input data that previously caused an error can be read without problems.
$ zq -version
Version: v1.7.0-64-g0d949c48

$ zq -f text 'yield entry.name | sort | uniq' 1.zng.gz
main

$ echo $?
0

Thanks @nwt!


How to handle "simple" retina images

On some website it is sometimes useful to have an image preset with just a retina version. For example a profile thumb with default of 100x100 and a retina version of 200x200.
Ideally we should be able to return something like:

It seems that currently Timmy only output the image 'w'. So I tried to create 2 image presets and use that to render a picture element. The presets would look like this:
'card' => [
    'resize' => [100, 100],
  ],
  'card@2x' => [
    'resize' => [200, 200],
  ],

And then render them with something like this:
return '<picture>
  <source ' . get_timber_image_responsive_src($image, $preset . '@2x') . ' media="(min-device-pixel-ratio: 1.5), (min-resolution: 144dpi)">
  <source ' . get_timber_image_responsive_src($image, $preset) . '>

</picture>';

This gives us:
<picture>
  <source src="unsplash-200x200-c-default.jpg" media="(min-device-pixel-ratio: 1.5), (min-resolution: 144dpi)">
  <source src="unsplash-100x100-c-default.jpg">

</picture>

Notice that the source elements have an src element and not an srcset. Eventually chrome and firefox and probably other browser doesn't seem to support src attributes on the source element. Only the standard image is loaded, even on retina display.
The use of src instead of srcset if there is only one url is defined here: https://github.com/mindkomm/timmy/blob/master/functions-images.php#L319

Not sure yet about the best solution and if this would cover all situations but ideally the image preset would look like this:
'card' => [
  'resize' => [100, 100],
  'srcset' => [2],
],

And we could use something like this, with a new method if needed:

Hey @idflood
Yeah, picture elements are not that well supported with Timmy yet, because they are quite hard to automate. You can go different paths with it and I thought it might be easier to just write out the markup you want instead of adding a lot of configuration to Timmy.
I guess as a workaround for now you could use the get_timber_image_src() function instead of get_timber_image_responsive_src():
return '<picture>
    <source src="' . get_timber_image_src($image, $preset . '@2x') . '" media="(min-device-pixel-ratio: 1.5), (min-resolution: 144dpi)">
    <source src="' . get_timber_image_src($image, $preset) . '">

</picture>';

Coming back to your proposal. When you have a configuration like the following:
'card' => [
   'resize' => [100, 100],
    'srcset' => [2],
],

You would expect it to output the srcset with a 2x descriptor, right? I think we could do that, because it’s not the same as 'srcset' => [ [ 2 ] ], which would double the size of image.
Or maybe we could just use 2x, or 1.5x, to make it more explicit what you want to do:
'card' => [
    'resize' => [100, 100],
    'srcset' => [ '2x', '1.5x' ],
],

Adding get_timber_image_srcset() function is also a good idea 😎! It would then work like this:

get_timber_image_responsive() – Gives you the automatic responsive markup.
get_timber_image_srcset() – Gives you the value for a srcset attribute.
get_timber_image_src() – Gives you the value for a src attribute.

Would you agree? Would that make sense?

Hi @gchtr ,
Thanks for the super quick feedback and obviously this great little library : )
I think you are right, the proposal is probably too simple to cover all use cases and doesn't provide enough informations to automatically render the desired markup.
Your proposition looks great to me, at least it will provide all the tools needed to build most picture markup 👍

Maybe one missing element is the ability to retrieve the single src of a size within srcset. Something like get_timber_image_srcset_url($image, 'card', 2). So for my initial proposition it would look like this:

What do you think?

Hey @idflood
Sorry for answering so late. You know how it is… There’s always lots of other stuff that is more pressing. I finally had a similar use case in a project, so I could work on this.
I made a new pull request in #25 that adds a new get_timber_image_srcset() function. In addition with the newly supported notation for srcset in your image configuration, it would work like this for you.
Take the following image configuration:
'card' => [
    'resize' => [ 100, 100 ],
    'srcset' => [ '1x', '2x' ]
],

And the following output:
echo '';

That would result in the following HTML:

So I think you wouldn’t actually need a get_timber_image_srcset_url() function. Does that make sense? Would that work for you?

Fixed in #25.


Should raise a warning/error or something when methods already exist

class MyClass < ApplicationRecord
  def blue?
    'blue!'
  end

  enumbled_to :color
end

my = MyClass.new(color: Color.blue)
my.blue? #=> true (not "blue!")

The Enumbler will just silently overwrite a method that is already defined.  Rails has a nice way of warning you when it is overwriting a method.  We should do the same.

This was fixed by #7


Enhancement: Allow prediction on numpy arrays present as local variable

The current model only allows us to predict on images by providing a path. We can extend the functionality and flexibility of out module by allowing numpy arrays as arguments to predict.

You're right, I'm planning to add all of these functionalities through the InferenceEngine class found here
So a more sensible thing to do now is to modify all of the current models so that they're  a subclass of the InferenceEngine class.

Okay

So first we can do that and then continue with this.

yes


NETOBSERV-613: drop messages when they accumulate in the exporter

The following PRs need to be merged first: https://github.com/netobserv/netobserv-ebpf-agent/pull/61 and https://github.com/netobserv/netobserv-ebpf-agent/pull/62. Then, this PR needs to be rebased into master.
This PR adds a flow's dropper before the exporter to avoid memory growing indefinitely or synchronized channel operations blocking previous parts of the system.
It also logs the number of dropped flows, suggesting the customer to take reactive actions.
The new architecture results in:
flowchart TD

    E(ebpf.FlowFetcher) --> |"pushes via
RingBuffer"| RB(flow.RingBufTracer)
    E <--> |"polls
PerCPUHashMap"| M(flow.MapTracer)
    RB --> |chan *flow.Record| ACC(flow.Accounter)

    ACC --> |"chan []*flow.Record"| DD(flow.Deduper)
    M --> |"chan []*flow.Record"| DD

    subgraph Optional
        DD
    end

    DD --> |"chan []*flow.Record"| CL(flow.CapacityLimiter)

    CL --> |"chan []*flow.Record"| EX("export.GRPCProto
or
export.KafkaProto")

Codecov Report

Merging #63 (1f06575) into main (f63d104) will increase coverage by 0.24%.
The diff coverage is 37.77%.

@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
+ Coverage   33.04%   33.28%   +0.24%
==========================================
  Files          21       22       +1
  Lines        1392     1433      +41
==========================================
+ Hits          460      477      +17
- Misses        913      936      +23
- Partials       19       20       +1

Flag
Coverage Δ

unittests
33.28% <37.77%> (+0.24%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/agent/agent.go
14.22% <0.00%> (-0.55%)
:arrow_down:

pkg/flow/limiter.go
51.51% <51.51%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Does Flow has alternative for this parameters?

TypeScript allows to type annotate the this context of a function, does flow has something similar to this?
https://www.typescriptlang.org/docs/handbook/functions.html#this-parameters

https://github.com/niieani/typescript-vs-flowtype#declarable-arbitrary-this-in-functions-outside-of-objects


Update to Windows 10 Anniversary Update - October 2016 Update (10.0.14393.0)

Fixes #811

All should be fine now, you can merge if you want.
Thanks

@amerkoleci  Cool.
I would like to avoid copying wincrypt.h, so you can workaround this by adding a #define SORTPP_PASS in top level Mapping.xml in the include_prolog section to avoid compilation error.
After could you rebase to remove wincrypt.h from the commits?

@xoofx will do so later and commit in my branch, just dont merge now.
Thanks

Should be fine now, you can merge if you want :)

Cool! Thanks!
@xoofx
How soon could you release new version to nuget?


feat: add load_git, to load Module from a git committish

Since you mentioned liking the module-from-git approach in griffediff.  Here's a PR for your consideration that adds it here.  This will create a temporary git worktree with a specific commit or tag checked out, then add it to the search_paths passed to the loader
from griffe.git import load_git

module = load_git('my_module', commit='v0.1.0', repo='path/to/repository')

~leaving in draft for now, as i'm sure there's some stuff to discuss (and I haven't run all the checks yet).~

Fixed up docstrings and checks... and created a better test fixture strategy.


Too many DB connections

Unable to connect to database - too many connections. Attempted from pgAdmin, rstudio server psql tool, and spatial_summaries.Rmd.

Max DB connections allowed:

Summary of open connections:

Seems to be a single problem query. 89 instances of the following query which is just compiling a list of tables in the database:
SELECT NULL AS schema, table_name AS table FROM INFORMATION_SCHEMA.tables WHERE (table_schema = ANY(current_schemas(true))) AND (table_schema <> 'pg_catalog') UNION ALL SELECT DISTINCT table_schema AS schema, NULL AS table FROM INFORMATION_SCHEMA.tables
Most instances were created on 2021-06-29 or 2021-06-30.
Not sure why this query is being executed, or what for. Do we need connection pooling? - like  PgPool.

Temporary fix to remove sessions:
select pg_terminate_backend(pid) from pg_stat_activity where pid != pg_backend_pid() and datname = 'gis' and usename = 'admin' and state = 'idle';

Yep, I had a similar problem and used pool::dbPool() here (so far just reports-v2 branch):
https://github.com/marineenergy/apps/blob/1a52acf702322768aec8f62b162bbf02f3bfea15/scripts/db.R#L38

Now that it's merged, report-v2 should be working without this issue popping up


Read Composite support for timestamped data.

Question
Would it be possible for Leshan to support  timestamps in Read Composite operation? Currently  the presence of timestamp in the response causes an exception:
"Unable to decode node [node] : value should not be timestamped"

We would like to support to be able to have 1 timestamp like for Simple (not composite) Read and Observe e.g.
[
    { "n":"3/0/1","v":11234, "bt":1706888656 },
    { "bn":"/6/0/","n":"0","v":-21.0 },
    { "n":"1","v":141.0 },
    { "n":"5","v":1706888656 }
]

I preparred following: PoC  supporting described above feature. However, I don't know if my idea of impelementing this is correct.

I looked at this quickly, I think this is the idea.
When you think your code is ready for review, create a PR and let me know.

I looked at this quickly, I think this is the idea. When you think your code is ready for review, create a PR and let me know.

PR (#1639) created.

#1639 is now integrated in master so I close this issue.
(will be available in 2.0.0-M17)


Coupa (Independent Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[X] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[X] I attest that the connector works and I verified by deploying and testing all the operations.
[X] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[X] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[X] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[X] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[X] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[X] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[X] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[X] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

@microsoft-github-policy-service agree

[[certify-connector]]

Hello @NovaGL,
I hope you are doing well.
Congratulations, your pull request is approved. We will move forward with the certification process. We are preparing your connector for production deployment and will queue up for the next deployment schedule. Once your connector onboards the next deployment schedule, it starts to deploy your connector in our production environments that typically takes 3-4 weeks.
Please let us know if you have any questions.
Thank you very much for working with us.

@Amjed-Ayoub It says 1 change requested. Am I missing something?

@Amjed-Ayoub It says 1 change requested. Am I missing something?

Hello @NovaGL no you are not missing anything. your connector will start the deployment process soon.

@Amjed-Ayoub Maybe I did something wrong as it says Amjed-Ayoub was requested for review

@Amjed-Ayoub Maybe I did something wrong as it says Amjed-Ayoub was requested for review, merging is blocked

My mistake, I left one comment open.

The connector will start the deployment on Friday.

@Amjed-Ayoub
Still says Requested changes must be addressed to merge this pull request.

@NovaGL don't worry. the process is moving forward for your connector. I will approve the PR once the connector is deployed. we will keep the PR open to communicate with you if we have any issues during the deployment process.

@Amjed-Ayoub you have to buy coupa. It's like SAP. The people using this connector would know how to login. I can't give URLs as that is provided by their administrator.
Can you explain why this is a problem

@Amjed-Ayoub you have to buy coupa. It's like SAP. The people using this connector would know how to login. I can't give URLs as that is provided by their administrator.
Can you explain why this is a problem
EDIT: Although I don't believe this is required. I have added login at instance.coupahost.com

Ohh, that is why then. Unfortunately, we don't buy subscriptions for connectors. do you think it is possible to provide us with a valid subscription and client credentials for this API?

@Amjed-Ayoub can you contact me privately so we can discuss

Yes please send us an email to<EMAIL_ADDRESS>

@Amjed-Ayoub I've sent you an email. I would like this resolved as soon as possible.

@Amjed-Ayoub I've sent you an email. I would like this resolved as soon as possible.

@NovaGL Thank you so much for reaching out and sharing this information with us.
I would suggest you change the apiProperties file to let the users enter their client credentials. Please see an example here PowerPlatformConnectors/apiProperties.json at dev · microsoft/PowerPlatformConnectors (github.com).

@Amjed-Ayoub That is not applicable to this connector as my apiProperties are dynamic

@Amjed-Ayoub My api matches that

[[certify-connector]]

@NovaGL the connector will start deploying tomorrow. Thank you so much for your patience and understanding.

Hello @NovaGL,
I hope you are doing well.
Your connector is in production deployment and queued up for the next deployment schedule. This process typically takes 3-4 weeks. Please note that the documentation for your connector will not be generated until the deployment process is in the final steps.
Please let us know if you have any questions.
Thank you very much for working with us.

Hello @NovaGL,
I hope you are doing well.
I want to let you know that we are preparing the connector for deployment. Please expect it to be fully deployed and published within the next week.
Thank you so much again for your patience and understanding.
Best regards,
Amjed Ayoub

Hello @NovaGL,
I hope you are doing well.
Congratulations, the deployment of your connector and documentation page was already completed, We have a short survey on the connector certification process that we would like for you to complete. Your feedback is appreciated and will help us improve the program.
Thank you very much for working with us.


Revert mill to 0.11.7 to fix release?

Looks like I can't make the release pass and this is the only major change that could have changed anything
https://github.com/scalacenter/bloop-config/actions/runs/10160570327

Actually, it seems a bigger issues and mill is unable to get released either. :/


Javalin and Project Loom

I've created a example project for running Javalin on Jetty with Loom: https://github.com/tipsy/loomylin
Would love some feedback on the project and suggestions for how to improve it (what endpoints to include, what benchmarks to run).

Great work.  I'll have to have a think about suggestions.
One comment about the last case (waiting async), my suspicion is that this closely matches loom partly because the amount of work moved/switched to the other thread pool at 100 millis is a relatively decent amount of work and specifically is high cost relative to the thread switching cost. If that 100 millis of work drops then we should start to see the relative cost of looms virtual thread park/unpark vs the cost of taking that work and executing on a different thread pool.

FYI: For Postgres we ideally want resolution of https://github.com/pgjdbc/pgjdbc/issues/1951

Thanks @rbygrave ! I've just started looking at Loom (and performance testing in Java in general), so I'm trying to gather some input on what should be tested and how.
I'm guessing we should move away from hey and use something like JMH, so we get a better overview of what's going on?

FYI: For Postgres we ideally want resolution of pgjdbc/pgjdbc#1951

Nice, so I guess you've looked at bit at Loom before! Do you know if there are other changes we need to do to Jetty in order to get the full benefit of Loom? Currently I'm just replacing the threadpool.

Currently I'm just replacing the threadpool.

Yes, as far as I know that should be all we need to do. Loom is NOT expected to initially support synchronized as it is a OS level lock so the current recommendation as I see it is to replace use of synchronized with ReentrantLock. I don't think Jetty or Javalin need to do anything there so that is just an FYI (but it is a task for Postgres JDBC drivers and other libraries for example).
The only task for Javalin I see would be at some point to perhaps make it easier for people to switch to using the loom threadpool (like a config option as it is in Jex now).
I've added a Jex module for the VirtualThreadPool (same as Loomylin) and Jex now has a boolean config option to just turn on use of VirtualThreadPool. The limitation we need to be using a JDK 17 Loom build in order to use it.
Jex is using Java module-info so there was a bit of a trick (via moditech-maven-plugin https://github.com/avaje/avaje-jex/blob/master/avaje-jex-jetty/pom.xml#L52) such that we have the Java 17 compiled module with VirtualThreadPool with all the other modules just need Java 11 (but that said I don't think Javalin will need that trick because Javalin doesn't have module-info's).
I think thats it / all the interesting bits.  I've got 2 demos of Loom that I'll show to a JVM group shortly - a HttpClient based one and a Jetty/Jex one.  Ideally I patch the Postgres JDBC Drivers and get to incorporate that into the demo but we shall see.
Cheers, Rob.

Thanks @rbygrave! I went for a slightly different approach, so Javalin will just use Loom if it's there, and fall back to the QueuedThreadPool if it's not: https://github.com/tipsy/javalin/pull/1256.


fix: vpc export

:tada: This PR is included in version 1.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Doge wallet 1.10 Ubuntu 12,14 & 16 w/all lib's install error

configure: error: libdb_cxx headers missing, Dogecoin Core requires this library for wallet functionality (--disable-wallet to disable wallet functionality)
I have all the lib's installed. Tested on 2 VPS 1 full server and 4 VM's. Ubuntu 12,14 and 16, Mint and they all have the same error. All have different Lib releases. And all have the required lib's listed in the readme file and then some. And all tested on clean servers.
The small work around listed in the tickets here does not work.
If you have also ran in this. Please reply.
Thank you.

I remember seeing this error in the past...
I have a fresh ubuntu 16 install. I will try to reproduce this.
I am mostly using docker when I want to use dogecoind (running 1.10). Maybe this can help you :
https://github.com/BitcoinAmiens/docker-dogecoin
Will report tomorrow with more info.

Err, which version of Berkeley DB do you have installed?

I believe I have 5.3

Nope my bad. I actually wasn't installed. But 5.1 doesn't seems available like in the documentation however I found 5.3
So I did :
sudo apt install libboost-all-dev libdb5.3-dev libdb5.3++-dev

but now I have this error :
checking for Berkeley DB C++ headers... default
configure: error: Found Berkeley DB other than 5.1, required for portable wallets (--with-incompatible-bdb to ignore or --disable-wallet to disable wallet functionality)

So I believe I have to download and build 5.1

Actually no need to download 5.1 with incompatible db flag.
I have done a short gist. I hope it helps @ultra-pool
https://gist.github.com/rllola/0e46feae5a41cb7d29352a84fb388304
@rnicoll If you think it is needed I can update the documentation ?

I'm trying to decide if we want to update the docs, or move to 5.3/later. Probably yes can you raise a documentation update against the 1.14-branding branch, and we'll move to a later BDB for 1.15+.

@rnicoll I believe this issue can be close now.


RTL

First pass at adding the Sass needed for RTL support into the sass-tools

Also cleaned up a few functions
Added triple and third math functions

To Do

[x] Tests
[x] Docs
[x] Bump

👍

Looks great. 👍


[FR] Add support for custom dataloaders

Is your feature request related to a problem? Please describe.
Add support for custom pytorch dataloaders so that not entirely dependent on zarr stores
Describe the solution you'd like
Modularise the training function so that it can take in a zarrstore or custom dataloader from the user

Not necessarily about writing lots of custom ones ourselves, just support for any user wanting to implement their own - but need to give an example of what the dataloader needs to look like


Support Cabal 3.6 (fixes #484)

Attempt to resolve #484. I'm not sure if this is a proper fix?
Side-note, when forking this repo on github, I instead got this... any idea why?

Side-note, when forking this repo on github, I instead got this... any idea why?

Likely because you forked from https://github.com/yamadapc/hpack-convert which forked from this, and github seems to allow only one fork per subject/organization.

@bergmark I took the liberty to amend your commit in https://github.com/sol/hpack/pull/489.

Thanks!


Update CI

This PR tries to address https://github.com/zsteve/OptimalTransport.jl/issues/18#issuecomment-735473875.
It:

sets up CI with Github Actions for Linux, MacOS, and Windows (it worked for my package and recently Travis introduced quite strict limits even for public projects)
adds TagBot
updates CompatHelper
builds documentation

Note that you have to add a deploy key to the repo such that CompatHelper is allowed to run tests and TagBot is allowed to rebuild the documentation on a new release: https://juliadocs.github.io/Documenter.jl/stable/man/hosting/#Authentication:-SSH-Deploy-Keys

It is unfortunate that the package states that it is compatible with Julia 1.0 but it can't be tested on Julia 1.0 since CUDA is not available for Julia 1.0. Maybe it would be good to separate the CUDA tests and only run them on more recent Julia versions or just drop support for Julia < 1.3. For now I only removed tests with Julia 1.0.


slow loading time to view forms w lots of submissions

when a form has a large number of submissions. it takes a very long time to load, even with a decent internet connection. slow connection makes it almost impossible. 3,694 submissions with the buildings form is an example. maybe it can load only 100 and then have option to load the rest. this is an issue with cloud-hosted instances of omk server.

Yes, it makes sense it gets slow at that point. Implementing pagination will resolve that.

Have you tried it with @ahmedOpeyemi JSON Stream work?

Another possible bottleneck is the file system. Many file systems' performance degrades when a directory has a certain number of files. The solution is simple: break up the submissions into more directories.
I've seen Who's on First, NGINX cache directories, Flickr, and others to have done this.
I'd have to get the data and do some profiling to determine what is contributing to your problem. Most likely pagination will resolve it.

So, this is one of the issues https://github.com/AmericanRedCross/OpenMapKitServer/pull/85 aimed to resolve at the server end. We had about 8500+ submissions and this issue here https://github.com/AmericanRedCross/OpenMapKitServer/issues/78 was occurring at the server end.
Calling the endpoint directly from your browser http://<server_url>/omk/odk/submissions/<form_id>.json using the fix in the PR would show that it returns appropriate result in good time but flattening it out at the client end is where the bottleneck currently is.

Flattening? You're saying that it's just too much for one table in a browser to handle?

@hallahan
Here is where the delay is coming from
Below is the flattening fx i was talking about
function doCSV(json) {
    // 1) find the primary array to iterate over
    // 2) for each item in that array, recursively flatten it into a tabular object
    // 3) turn that tabular object into a CSV row using jquery-csv
    var inArray = arrayFrom(json);

    var outArray = [];
    for (var row in inArray)
        outArray[outArray.length] = parseObject(inArray[row]);

    var csv = $.csv.fromObjects(outArray);
    // excerpt and render first 10 rows
    renderCSV(outArray);
    showCSV(true);

    // show raw data if people really want it
    $(".csv textarea").val(csv);

    // download link to entire CSV as data
    // thanks to http://jsfiddle.net/terryyounghk/KPEGU/
    // and http://stackoverflow.com/questions/14964035/how-to-export-javascript-array-info-to-csv-on-client-side
    var uri = "data:text/csv;charset=utf-8," + encodeURIComponent(csv);
    $("#downloadCsv").attr("href", uri).attr("download", getParam('form') + ".csv");
    $("#downloadJson").attr("href", OMK.jsonUrl()).attr("download", getParam('form') + ".json");
}

Oh right. We do need to "flatten" the JSON so that it works in table form.

After spending some time examining this problem with a large dataset of 33,578 submissions, I'm seeing the slowness of the submissions page from two things:

The submssions.json response takes quite some time, as the server has to open 33,578 directories and aggregate the data.json. This is unavoidable if you want all of the submissions in 1 go.
As @ahmedOpeyemi explains, the doCSV function takes some time, because it has to iterate through each submission and flatten them to create the table and the CSV object.

Ideally we would have pagination in the REST api that would give back separate responses for smaller chunks of the JSON from the server. The nice table we see with the search and sort functionality, however, would have to be thrown away and be replaced.
Right now, the CSV download and text area are done in the browser, which is slow. The next step is to create a .csv REST endpoint where this is done on the server instead.
I think with what we have right now, we should just create this .csv endpoint and check to see how many submissions there are. If it is above a certain threshold, we should hide the table view and provide links for the user to download CSV and JSON.
Then, we can later implement pagination.


Fix flaky tests

Those tests sometimes fail on CI
https://github.com/foundry-rs/starknet-foundry/blob/c1049ecde2e907d5b59939228d1fd542d1ca9186/crates/forge/tests/e2e/running.rs#L388
https://github.com/foundry-rs/starknet-foundry/blob/c1049ecde2e907d5b59939228d1fd542d1ca9186/crates/forge/tests/e2e/fuzzing.rs#L169
https://github.com/foundry-rs/starknet-foundry/blob/61a6e62d22b3452b8cb0dd19e22097d8c52e19ae/crates/forge/tests/e2e/running.rs#L327

As a result of #862


Invoke-CMClientAction is missing

The cmdlet Invoke-CMClientAction is missing. Please add this!

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

ID: 5e2f0b6d-c7ca-cb30-3de9-e50eae55c24d
Version Independent ID: 01e64fb6-4385-6864-7430-38ac98632aaf
Content: ConfigurationManager
Content Source: sccm-ps/ConfigurationManager/ConfigurationManager.md
Service: powershell

It's there now: https://docs.microsoft.com/en-us/powershell/module/configurationmanager/invoke-cmclientaction?view=sccm-ps
This content is mostly auto generated from what's in the module, but what's currently published is only as of version 1902. I have a pending update to bring it up to the latest version, and then a new process to keep it up to date going forward.


Display a notice when loading an autosave

Cross-posted to garglk https://github.com/garglk/garglk/issues/662
When I try running Aisle in iplayif.com's Parchment or the latest Gargoyle, I don't see the "splash screen". It's a text box that appears when you start the game and gives an introduction that starts with "You are about to read a story". Instead, it jumps straight to the prompt. (In Gargoyle, there's a blank screen first)
It appears fine when I use Frotz 2.53. Is this a regression?

You're probably just seeing the autosave. If you enter "restart" then it will take you back to the splash screen.

Ah, yes, that's the problem. Sorry for the false alarm. I didn't know of this feature - I wonder if there should be some tooltip saying an existing game was restored, with an option to delete it.
When I tried it in Firefox's Private Browsing mode, it didn't work at all. I think it's because a certificate problem in https://proxy.iplayif.com/ -- Chrome also gives an error on it, in both regular and incognito mode.

Yes, I agree there should be a notice.
There shouldn't be an certificate errors for proxy.iplayif.com, it's just using Cloudflare certificates.

It looks like my ISP is blocking some access to the proxy website - maybe because of its name. Oddly, Firefox isn't affected.


Level API stuff

Added Levels APIs (post and get)
Added base code for calling APIs. Doesn't have AWS link yet. Will probably need to resolve that.

Verbal lgtm


$WEEKSTART comparison fails on non-English locales.

We have had a few devs at our group being unable to use git-standup on Mondays due to the hardcoded comparison for $WEEKSTART. I've forked the repo and made a quick fix for internal use and was wondering if there may be interest in implementing it upstream as well.
gimi-org/git-standup@c74aa71fa1d2f4f221c31b5544e4e52cfd976628
I'm fairly new to writing bash scripts with cross-platform compatibility so any sort of feedback would be appreciated. (And yes there is a typo in that commit but that has since been fixed)

This still seems to be an issue for me, every monday I need to remember to run git standup -d3 instead of git standup otherwise it tells me I did nothing last day ;)


type-safe enums

What do you think of this solution to make the properties that are (essentially) enum values type-safe? (I.e. have proper enums for those)
I moved these enums to the core package (as they are used in core now). I put them all in one file. I could also put them into the individual *Layer.kt files or something if you think this is clearer.

Type checker happily accepts this example with no assertion required, nice!
LineLayer(
  id = "amtrak-routes",
  source = routeSource,
  color = const(animatedColor),
  width = const(4),
  cap = case(
    const(false) then const(LineCap.Round),
    fallback = const(LineCap.Square),
  ),
)


External stations cargo customers

An external station may have freight customer the same way that module stations can have.

Implemented in release 1.1.0.


Can not export COCO, Pascal VOC, YOLO...

Describe the bug
I have 13 annotated images with rotated bounding boxes and I am not able to export the project. There is error in javascript console and also in command line.
To Reproduce
Steps to reproduce the behavior:

Go to Project > annotated images
Click on Export
Message "Files are being prepared. It might take some time." is displayed, but the export will never finish.

Expected behavior
Should export zip with images and annotations.
Screenshots

Environment (please complete the following information):

OS: Windows 11
Label Studio Version: 1.15.0

Additional context
I did installation using pip into fresh virtual environment of python 3.10.

It has to be something with the project, not the installation, because older projects can be exported.

The error is on this line only in browser.

If I open export URL on new tab, the zip file is actually downloaded.

Try to export using LS console: https://labelstud.io/guide/export.html#Export-using-console-command


Cloudflare IP not updating

I have gotten this script to run without any errors but when I manually change one of my records on the Cloudflare dashboard, the script that is meant to change it does not. I have tried running it multiple times but to no avail. What should I do to find out more about why this is happening or how to fix it?
Here are the variables that I have set in the shell script file. Nothing else I have changed:
<EMAIL_ADDRESS>
auth_method="global"
auth_key="REDACTED"
zone_identifier="8b073b276e41fd6b61a6658678a83981"
record_name="node1"
ttl="3600"
proxy="true"
sitename="artichokemc.net"
slackchannel=""
slackuri=""
discorduri="https://discord.com/api/webhooks/REDACTED"

So uhhhh... What exactly isn't working?
Did you mean that the script cannot update your record's IP after you manually changed something on the dashboard?

Yes. That

So uhhhh... What exactly isn't working? Did you mean that the script cannot update your record's IP after you manually changed something on the dashboard?

Yes that is what I mean. When I change a record manually, the script doesn't update it.

I see, alright. I would suggest you to look into the audit log on Cloudflare to check if the request is getting passed or not.
In addition, please get this debug script from here. Then, run it with ./dev-ipv4.sh --debug --force

I am running into the same issue, IP is not getting updated, I ran the scripting using bash -x ./cloudflare.sh and I can see it is an authentication error which is strange as I checked everything several times and everything is correct.
{"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}
Maybe Cloudflare made some change that broke the script?

It turned out to be an issue with auth_method, I was using a global key and did not set the value of it to be global. After setting that I can see the updated IP 👍🏼

I'm glad it worked for you. My auth method is on global and when I change it to something else it just throws an error.

I still have the problem. Does anybody know why this might be happening?

@Smartich0ke I'm sorry, but it's really hard for anyone to help with the lack of debug information.
So, let me get this straight. When you say the script stopped working, do you mean it by temporarily or endlessly?
Uhhh, and you said you changed one of your records. What changes did you made?
And FYI, auth method only has two valid options. One is token, another one is global. When you use one that isn't matching to your Cloudflare API key type, of course it's going to throw an error.

When I manually change a record and run the script to see if it changes that record back to the public IP of the machine, it does not do so. The script appears to complete successfully with no errors and it doesn't matter how many times I run the script, the IP address for that record does not change.

Thank you for the explanation. Hmmm, I tried to reproduce your issue, but everything worked fine.
Anyways, can you uhhh do a bash -x ./your-script-name.sh as mentioned by @shairyar. And what's the script actually doing?

Here is the output of  bash -x ./script-name.sh:
+ auth_email=REDACTED
+ auth_method=global
+ auth_key=REDACTED
+ zone_identifier=8b073b276e41fd6b61a6658678a83981
+ record_name=REDACTED
+ ttl=3600
+ proxy=true
+ sitename=REDACTED
+ slackchannel=
+ slackuri=
+ discorduri=REDACTED
+ ipv4_regex='([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])\.([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])\.([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])\.([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])'
++ curl -s -4 https://cloudflare.com/cdn-cgi/trace
++ grep -E '^ip'
+ ip=ip=<IP_ADDRESS>
+ ret=0
+ [[ ! 0 == 0 ]]
++ echo ip=<IP_ADDRESS>
++ sed -E 's/^ip=(([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])\.([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])\.([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])\.([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5]))$/\1/'
+ ip=<IP_ADDRESS>
+ [[ ! <IP_ADDRESS> =~ ^([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])\.([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])\.([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])\.([01]?[0-9]?[0-9]|2[0-4][0-9]|25[0-5])$ ]]
+ [[ global == \g\l\o\b\a\l ]]
+ auth_header=X-Auth-Key:
+ logger 'DDNS Updater: Check Initiated'
++ curl -s -X GET 'https://api.cloudflare.com/client/v4/zones/8b073b276e41fd6b61a6658678a83981/dns_records?type=A&name=REDACTED' -H 'X-Auth-Email: REDACTED' -H 'X-Auth-Key: REDACTED' -H 'Content-Type: application/json'
+ record='{"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}'
+ [[ {"success":false,"errors":[{"code":10000,"message":"Authentication error"}]} == *\"\c\o\u\n\t\"\:\0* ]]
++ echo '{"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}'
++ sed -E 's/.*"content":"(([0-9]{1,3}\.){3}[0-9]{1,3})".*/\1/'
+ old_ip='{"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}'
+ [[ <IP_ADDRESS> == {"success":false,"errors":[{"code":10000,"message":"Authentication error"}]} ]]
++ echo '{"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}'
++ sed -E 's/.*"id":"(\w+)".*/\1/'
+ record_identifier='{"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}'
++ curl -s -X PATCH 'https://api.cloudflare.com/client/v4/zones/8b073b276e41fd6b61a6658678a83981/dns_records/{"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}' -H 'X-Auth-Email: REDACTED' -H 'X-Auth-Key: REDACTED' -H 'Content-Type: application/json' --data '{"type":"A","name":"artichokemc.net","content":"REDACTED","ttl":"3600","proxied":true}'
+ update=
+ case "$update" in
+ logger 'DDNS Updater: REDACTED REDACTED DDNS updated.'
+ [[ '' != '' ]]
+ [[ REDACTED != '' ]]
+ curl -i -H 'Accept: application/json' -H Content-Type:application/json -X POST --data-raw '{
      "content" : "artichokemc.net Updated: REDACTED'\''s new IP Address is REDACTED"
    }' REDACTED
HTTP/2 204
date: Sat, 14 May 2022 02:29:41 GMT
content-type: text/html; charset=utf-8
set-cookie: __dcfduid=b5980eb4d32d11ecaf27c609b3e326c9; Expires=Thu, 13-May-2027 02:29:41 GMT; Max-Age=157680000; Secure; HttpOnly; Path=/
strict-transport-security: max-age=31536000; includeSubDomains; preload
x-ratelimit-bucket: 3cd1f278bd0ecaf11e0d2391374c011d
x-ratelimit-limit: 5
x-ratelimit-remaining: 4
x-ratelimit-reset:<PHONE_NUMBER>
x-ratelimit-reset-after: 2
x-envoy-upstream-service-time: 127
via: 1.1 google
alt-svc: h3=":443"; ma=86400, h3-29=":443"; ma=86400
cf-cache-status: DYNAMIC
expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
report-to: {"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report\/v3?s=rk9V9x%2FTyNlWI0yHbkXzWlecAp7A%2FsaU64q6l3E8NVYkZj6N7pDvJ5loePtXO4a85uw95pIQ7aDXdtLWKpKPqmSd%2B0Th6QxIDa9mAWkUaU9phTIxPpm32zSQvjOt"}],"group":"cf-nel","max_age":604800}
nel: {"success_fraction":0,"report_to":"cf-nel","max_age":604800}
x-content-type-options: nosniff
set-cookie: __sdcfduid=b5980eb4d32d11ecaf27c609b3e326c98710cb9a882eb906a4e32d4c983cefedcdf01f5954f6bfa524489f8132f640a3; Expires=Thu, 13-May-2027 02:29:41 GMT; Max-Age=157680000; Secure; HttpOnly; Path=/
set-cookie: __cfruid=1fd1a71996c3993317256ae9ff2c7fcc484178dd-1652495381; path=/; domain=.discord.com; HttpOnly; Secure; SameSite=None
server: cloudflare
cf-ray: 70b03524f9813772-MEL

+ exit 0

Thanks so much for the log! That really helped the debug process!!!
Aha! I think I knew why now. After carefully looking through your log, it seems that the script failed to find the record identifier. The reason is shown as '{"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}'
I would suggest you to double check all information in the variables are correct, mostly...

auth_email
auth_key
zone_identifier

Also, I have noticed another issue. Let's say your domain is named artichokemc.net. And you're trying to use this script to update a subdomain, let's say node1.artichokemc.net.
You need to setup the script like record_name="node1.artichokemc.net", or else you will get an error.

Ah ok thanks for your help! I will check through those things and let you know if I get it working.

Yep, I re-entered the API token and fixed the record_name and now everything is working! Thanks @Sailboat265 for your help!

@Smartich0ke Glad that you fixed it! Also, You're welcomed, anywhere, anytime!


Add timestep  support for CloudGAN

The previous PR #50 was getting quite large, so this is just to finish some more of it, and flesh out a few of the GANS, especially adding support for predicting individual timesteps instead of just all the outputs being concatenated together as a huge multi-channel image.

Other time-based GANs include https://arxiv.org/pdf/1801.09710.pdf and https://papers.nips.cc/paper/2019/file/c9efe5f26cd17ba6216bbe2a7d26d490-Paper.pdf, of which there is this PyTorch TimeGAN implementation https://github.com/d9n13lt4n/timegan-pytorch and Tensorflow implementation of TempoGAN here https://github.com/thunil/tempoGAN -> The TempoGAN might not be as useful, but seems like it could have some decent ideas

Relates to #4

Current work  is making CloudGAN work for both predicting all timesteps at once, like Pix2Pix, or conditioning on a future timestep, like MetNet does. The slides originally talk about only prediciting for a single timestep in the future, 15minutes in their case, so conditioning on time makes the model work a bit more like in the slides. In that implementation that now exists, the generator is given the previous timesteps in (T, C, H, W) format, runs a recurrent model on it, and returns a single image in the future, and the discriminator takes a single timestep for each and compares them. This might have the downside of less temporal consistency that the discriminator looks at though, although means the discriminator doesn't have a huge 288 or so channel set of images to choose between.

CloudGAN works, although runs out of memory quite quickly. Getting it more efficient will be in a different PR.


No "lib_LiteMORT.so" file!!!

Only windows version now. The Linux version will be released next month.


Promise.all() does not work with IE11

It seems that Promise object is present but the .all() isn't.
Instead, it works with Edge as expected.

Edge is not IE11. We should test it with IE10 and IE11 (unfortunately).


Disable _FusedConv2D generation in grappler if XLA is on (GPU only).

XLA does not recognize the _FusedConv2D node, resulting in multiple
clusters during auto clustering. Furthermore, XLA performs this
particular optimization as part of the fused convolution rewriter

@bas-aarts  Can you please resolve conflicts? Thanks!

@bas-aarts  Can you please resolve conflicts? Thanks!

@bas-aarts  Can you please check @cheshire's comments and keep us posted ? Thanks!

@bas-aarts  Can you please check @cheshire's comments and keep us posted ? Thanks!

@bas-aarts  Any update on this PR? Please. Thanks!

@bas-aarts  Any update on this PR? Please. Thanks!

It has been 24 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

@cheshire, per @sanjoy this will not work for auto-clustering.


Add --nodes= to kube-spawn up

Capturing an OOB discussion last night with @blixtra.
We should choose reasonable defaults for cluster configuration options in kube-spawn up, but we should be able to configure cluster properties.
--nodes= is a cluster property so we should add it as an option for kube-spawn up.
Things like --image= can be left to the other commands.

Thx @iaguis.
I see a difference in cluster configuration and cluster properties. Cluster configuration are things like what version of k8s to run, which image to use, etc. But a cluster property is something like how many nodes to have.
Perhaps we should group options like this to make it clear? Then, each time we add something it will be clear what to add where.

I tried implementing this but we get blocked by https://github.com/spf13/viper/issues/377


adding hover effect to home page cards

##issues
##hover effect to home page cards

please assign me to this issues

pls assign me this issue under GSSoC'24 Extd.


Implementations for operators and async functions on AsyncSequence per SE-0298

This has implementations for the following operators and asynchronous functions on AsyncSequence that match selected parity with Sequence. This adds the following operators (conversions from an existing asynchronous sequence to a new transformed asynchronous sequence applying a given functional operation): map, filter, dropFirst, drop(while:), prefix, prefix(while:), compactMap, and flatMap. Additionally this adds the following asynchronous functions as extensions to AsyncSequence: reduce, allSatisfy, contains, first(where:), min, min(by:), max, max(by:).
It is worth noting that a number of operators contain closures. In all cases these should allow async functions to be called so they will always be marked as async. In a few cases these will dramatically improve the power and ease of use of these operators. There is one key difference between the lazy counterparts and the async ones; the production of failures is possible since it is inheritly asynchronous - this is facilitated by the concept of protocol conformance rethrows. That conformance rethrows cannot interact with blocks since they dont store the "throw-y-ness" or "non-throw-y-ness" in their signature since the closure is stored; this means we need to bifurcate the type system at the concrete return types to ensure the proper throws versus rethrows concepts.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

the linux crasher makes no sense...
all the other tests pass yet:
17:55:15 FAIL: ["collect", "collect throws upstream", "collect empty"]

from a crash no less?!
if the Mac side passes I will push a disable for linux for that test for now

@swift-ci please smoke test and merge

Weird, now it's the test right after collect that fails… Like it's hitting a limit?

@swift-ci please smoke test

@phausler Quick summary of the stdlib team's discussion around throwing — we came to a consensus on two points:

Just like returning nil continuously once reaching the end of a sequence, throwing from an AsyncIterator should mark the end of iteration, and all subsequent calls to next() should return nil. A primary motivation here is that the only way to know whether an iterator is finished is to call next() — without this guarantee, if you're passed an iterator you basically have no way of knowing whether it's valid or not, and can't do anything with an iterator you've passed somewhere else temporarily. In terms of code impact, this should only affect iterators that store a throwing closure of their own. Any iterator that just calls through to a base iterator (like the prefix operator) can rely on this guarantee and doesn't have to track state for whether the base iterator has thrown.
This should be a semantic requirement of AsyncIteratorProtocol, as should the returning nil forever requirement. Neither was addressed in the evolution discussion that I can find, so we may need to add a short addendum to the proposal. (Note that the nil guarantee was added to IteratorProtocol, née IteratorType, in SE-0052.)

Iterator implementations should prefer to discard closures as soon as they're finished with them, to free up resources. This doesn't need to be an explicit guarantee, but should be a matter of course for the built-in iterators. Note that this works nicely with (1), since you can use an optional closure property to handle the "has thrown" state without increasing the size of the iterator.


Tighten security groups

This PR is based on the feedback from @chriskl, who rightfully pointed out that we were too generous in our open egress ports from the bastion host.
Ports for db ingress and egress have been tightened.

LGTM! You can merge this PR.


Add the ability for SQLTransform to accept a raw SQL statement in addition to inputURI

For discussion.
Given we allow the ipynb (Python Notebook) file to be executed directly numerous users have commented that it would be much easier to allow 'inline' SQL. Whilst this goes somewhat against the original intention it would reduce delivery friction significantly - whilst still having the benefit of text based version control (albeit a bit poor due to the ipynb format).

FYI the api from the notebook point of view:
%inlinesql outputView=stream1 environments=production,test persist=true sqlParams=inputView=${INPUTVIEW_ARGUMENT}
SELECT *
FROM ${inputView}

merged into 2.8.0


fix: Fetch Headers and Bodies separately #4167

Resolves #4167.
WIP

Benchmark results
Metrics with a significant change:

note_successful_decrypting_time_in_ms (8): 322 (-25%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit e63f0226 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,956
181,604
724,196

l1_rollup_calldata_gas
227,180
888,812
3,532,616

l1_rollup_execution_gas
320,371
1,007,260
3,752,994

l2_block_processing_time_in_ms
1,195
4,493 (+1%)
17,731

note_successful_decrypting_time_in_ms
:warning: 322 (-25%)
983 (+1%)
3,633 (+1%)

note_trial_decrypting_time_in_ms
33.3 (-21%)
117 (+11%)
181 (+3%)

l2_block_building_time_in_ms
18,242
72,428 (+1%)
291,121 (+1%)

l2_block_rollup_simulation_time_in_ms
13,521
53,618 (+1%)
216,043 (+1%)

l2_block_public_tx_process_time_in_ms
4,689
18,729 (+1%)
74,817 (+1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
16,241 (+2%)
31,500 (+3%)

note_history_successful_decrypting_time_in_ms
2,329 (+2%)
4,571 (-1%)

note_history_trial_decrypting_time_in_ms
106 (+21%)
244 (-2%)

node_database_size_in_bytes
17,842,256 (+3%)
34,267,216 (+3%)

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
327
44,528
25,313

private-kernel-ordering
196
43,633
16,193

base-rollup
1,463 (+1%)
129,774
933

root-rollup
82.6
4,192
729

private-kernel-inner
437
70,819
25,313

public-kernel-private-input
243 (+1%)
33,663
26,945

public-kernel-non-first-iteration
240 (+1%)
33,705
26,945

merge-rollup
7.84
2,712
933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
2 leaves
8 leaves
16 leaves
32 leaves
128 leaves
64 leaves
512 leaves
1024 leaves
2048 leaves
8192 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.2 (-2%)
10.7 (-3%)
12.6 (-1%)
17.0 (-2%)
23.1 (+1%)
64.4 (+1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.9
17.5
23.0
31.6
47.0
143
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.592 (-2%)
0.594 (-4%)
0.536 (-1%)
0.524 (-2%)
0.482 (+1%)
0.444
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
N/A
N/A
N/A
74.9 (-1%)
47.5 (-2%)
239
461 (-2%)
906 (+1%)
3,557

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
N/A
N/A
N/A
159
96.0
543
1,055
2,079
8,223

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.462 (-1%)
0.487 (-2%)
0.436
0.431 (-2%)
0.431 (+1%)
0.428

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
N/A
N/A
N/A
105 (+1%)
56.8
354
688 (-1%)
1,367
5,433

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
N/A
N/A
N/A
197
104
691
1,363
2,707
10,771

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.496
0.495
0.480
0.472 (-1%)
0.472
0.471

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
56.9
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
94.1
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
0.577
N/A
N/A
N/A
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
16,859
35,484

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
552
1,443

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.512 (+1%)
579 (+1%)


fi(ci): warnings

Upgrade actions to the latest version (it fixes warning about deprecated set-output command)
Added pre-commit hook setup to gradle

I propose to merge this one after https://github.com/hCaptcha/hcaptcha-android-sdk/pull/67 . There some update will be required after


refactor: Re-order stuff

based on plugins v3

Close as per https://github.com/R2Northstar/NorthstarLauncher/pull/593#discussion_r1382657441


views: addition of custom error handlers 401, 403, 404

Added custom error handlers to errors 401,
403 and 404. (Closes #13)

Signed-off-by: Javier Delgado<EMAIL_ADDRESS>

@lnielsen I have created the tests overriding the variable THEME_ERROR_TEMPLATE in order to avoid the assets fight. I am also creating a "fake.html" in a temp dir. I don't think this is the best way.

s/Added/Adds/

@jirikuncar Corrected.

@JavierDelgadoFernandez PR needs to be rebased.

@jirikuncar Done.

@lnielsen Done.


Adding contactme page #5

fixes #5
This page  have a form through which users can contact you.

#5 issue closed


vscode doesn't resolve gradle import dependencies macOS

I am using Java 11 and gradle 5.3.1 (via macports) on macOS.  When I am viewing Java source for a gradle project, many of the Java language extension features don't function properly.  It complains about unresolved symbols even though my build works properly.  I also get this error message in the problems:
Could not create an instance of Tooling API implementation using the specified Gradle installation '/opt/local/share/java/gradle'.
Failed to load native library 'libnative-platform.dylib' for Mac OS X x86_64.
The Gradle installation directory it reports does not exist.  I have tried setting GRADLE_HOME and running Electron from the command line with no luck.  FYI - the only instance of the file libnative-platform.dylib on my mac is in ~/.gradle/native/25/osx-amd64/ which does appear to be in use by IntelliJ if I fuser the file.
Vscode was working so well with Maven projects - all the Java language features worked beautifully so I am hoping this issue can be resolved soon/easily as I have moved on to Gradle and Java 11.

VSCode Version: 1.34.0-Insider (and latest stable Mac OS version)
OS Version: Mojave 10.14.4

Steps to Reproduce:

Add gradle project folder to workspace
You will see the error message regarding libnative-platform.dylib
Open a Java file and witness most/all Java language dynamic features don't work

The issue doesn't happen with extensions disabled but that's not surprising since my issue is with the extensions functioning properly.

Please file the issue against the extension that you use for Java.
https://github.com/redhat-developer/vscode-java ?


ValueError: Input contains NaN.

When I run the program on Amazon Book to 11 epochs, the loss becomes nan.
All the settings are default. The GPU is A100 *1.
Can you give some suggestions?

BTW, can we access the pre-trained weights?

BTW, can we access the pre-trained weights?

I will upload the pre-trained weights when I am free.

When I run the program on Amazon Book to 11 epochs, the loss becomes nan. All the settings are default. The GPU is A100 *1.
Can you give some suggestions?

What learning rate are you using? You might try tuning it down.

Thanks for your suggestions!