[20.09] firefox: enable pipewire+webrtc for wayland users

(cherry picked from commit 31e54cdde34073b3b83e9fca4434cb0cedcb2829)

Motivation for this change
Backports https://github.com/NixOS/nixpkgs/pull/84233
Things done

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

[ ] 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"
[ ] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@endgame This is a backport. That means you really can't request changes to already committed code without encouraging diversion which has to be forward ported.

@worldofpeace Fair enough then.

@endgame Not sure if you could submit a  review the original merged PR, though. Let's see what @andir thinks

I tried to build firefox from nixpkgs master without wayland and it broke because it couldn't find -lpipewire so  something needs to be reported: either on the original PR or as a new issue. No point pretending that wayland is an option if it breaks when turned off.

Thank you for testing and reporting @endgame.
@worldofpeace due to the quality of the patch I didn't want it in the first place but was probably too inactive to make my opinion voiced enough. At this stage I'd at least refrain from adding it to the stable release. It is a nice feature and a pity but I do not see us having the manpower to do fix and support this patch.
I'll spend a bit of time trying to clean the patch up to fix the situation in both master and this backport. If that fails I'm still in favor of skipping it or making the entire code optional (which is a mess I really dislike).

This patch is actually being used in more distros than I thought, see https://github.com/emersion/xdg-desktop-portal-wlr/wiki/Screencast-Compatibility#webrtc-aka-firefoxchromium

This patch is actually being used in more distros than I thought, see https://github.com/emersion/xdg-desktop-portal-wlr/wiki/Screencast-Compatibility#webrtc-aka-firefoxchromium

Yeah but most of them hide it behind a feature flag and/or only provide a single build for their entire distribution. Here we are trying to allow users to customize the package but didn't bake that into the derivation when this patch was added.
I just pushed c9c0b90b62bb863b56a7bdc356d38f7b143f2fa5 to this branch which does the required housekeeping for the added patch. I'll also add this to unstable once we agree on it.

This patch is actually being used in more distros than I thought, see https://github.com/emersion/xdg-desktop-portal-wlr/wiki/Screencast-Compatibility#webrtc-aka-firefoxchromium

Yeah but most of them hide it behind a feature flag and/or only provide a single build for their entire distribution. Here we are trying to allow users to customize the package but didn't bake that into the expression when this patch was added.
I just pushed c9c0b90 to this branch which does the required housekeeping for the added patch. I'll also add this to unstable once we agree on it.

Thanks, it looks good.

@andir Could you also summarize your current stance on this? It seems you've taken a few and it's not clearly linear because some of it is in the review comments.

This PR as is is probably fine. If you want to carry that into a release that sounds fine. I still worry about keeping the feature stable as we effectively now depend on Mozilla and a single fedora maintainer. Given the history with pipewire fedora will probably continue to provide patches. I just hope they'll do it on time as very often they were slower than us in updating the browser when security issues appeared.


Consider using go_image to reduce boilerplate.

See here: https://github.com/atlassian/smith/blob/63624b47d77f48cae9379528ead25d3fea00e9b5/cmd/smith/BUILD.bazel#L20-L32
This can use this:
  go_image(
     name = "smith",
     embed = [":go_default_library"],
     importpath = "github.com/atlassian/smith/cmd/smith",
     visibility = ["//visibility:public"],
 )

(the goal is to make containerizing as simple as s/lang_binary/lang_image/)
You might also TAL at rules_k8s, feel free to reach out if you have any problems.

Hey, thanks for suggestion. I new about these extra rules but I haven't tried them. I have just tried and here is what I found:

it is cool that I don't need to maintain distroless base layer sha
Gazelle still generates go_binary rule and if go_image has name=smith then they conflict.
If I change name=smith_container on go_image to avoid the conflict then it works but now I have two rules - just like before :)
I need an ability to run the binary. I usually build it and then run it (I don't use bazel run for this). This becomes bazel-bin/cmd/smith/smith_container.binary vs bazel-bin/cmd/smith/smith. It's a bit uglier and, which is more important, more magical - it is not obvious what is that .binary and there is no way to know it exists without looking at the rule implementation.

So given the above I think it is unlikely I'll use it unless there is a way to improve the user experience somehow.

Yeah, I have a bug open against Gazelle.  This came up here so I may augment the surface of language rules that don't benefit from layering to also accept the binary target directly.
(to elaborate a bit) For Go there is less benefit than with languages like Python and Java where we can intelligently factor layers (you can see some commentary on this here and here).  Eventually I'd like to make this more intelligent and less manual.
I'll open an FR on rules_docker to track this for now.

it is cool that I don't need to maintain distroless base layer sha

Even if you don't use go_image, you can get the benefit of this by using the following symbol for your base image: https://github.com/bazelbuild/rules_docker/blob/master/go/image.bzl#L59-L64  This has the benefit of putting busybox into containers built with -c dbg :)
Mostly I wanted to reach out and make sure you were aware of the option.  If you run into any problems, please don't hesitate to reach out.

I think this should get you what you want.
Let me know if you run into any trouble using it.


Client.on_message, None author_name

I assume this doesn't happen with every Facebook account, for it would create some problems.  However, each time I try to display author_name, in the Client.on_message method, I get None.  I have the ID, so I could ask for the author in the list of users, but that would add a query.  Is it something that Facebook doesn't support anymore?
Thank you for your work,

That's exactly the reason, Facebook doesn't provide this information anymore when you receive a message. Perhaps try calling getUserInfo with the author_id, look at the raw data from that function, and grab the username from there (I'd recommend that you implement some sort of caching, too)


Point lights don't seem to be working on Android

I have been trying to get point lights working on Android with no luck. I added a simple piece of code to the sample-gltf-viewer project and I can see that the point light does not show up:
private fun createIndirectLight() {
        val engine = modelViewer.engine
        val scene = modelViewer.scene
        val ibl = "default_env"
        readCompressedAsset("envs/$ibl/${ibl}_ibl.ktx").let {
            scene.indirectLight = KtxLoader.createIndirectLight(engine, it)
            scene.indirectLight!!.intensity = 30_000.0f
        }
        readCompressedAsset("envs/$ibl/${ibl}_skybox.ktx").let {
            scene.skybox = KtxLoader.createSkybox(engine, it)
        }

        val light = EntityManager.get().create()

//TRY ADDING A POINT LIGHT

        // Create a color from a temperature (5,500K)
        val (r, g, b) = Colors.cct(5_500.0f)
        LightManager.Builder(LightManager.Type.POINT)
                .color(1f, 0f, 0f)
                // Intensity of the sun in lux on a clear day
                .intensity(110_000.0f)
                // The direction is normalized on our behalf
                .direction(0.0f, -0.5f, -1.0f)
                .position(0f, 1f, 0f)
                .castShadows(true)
                .build(engine, light)

        // Add the entity to the scene to light it
        scene.addEntity(light)
    }

However if I change the light type to DIRECTIONAL then I can see the red light in the scene.

I also tried adding the following line in onCreate just to be sure that this isn't due to the light being clipped somehow but I still can't see the effect of the point light:
        modelViewer.view.setDynamicLightingOptions(0.1f, 100f)

Smartphone (please complete the following information):

Device: Huawei P20
OS: Android 10

A color temperature of 5,500K shouldn't produce red. Is your code snippet above correct? You may also want to check the light intensity vs the camera exposure. Maybe first try disabling all the other lights (IBL and directional).

Sorry I'm not actually using that color temperature, I just copied and pasted it across from a different sample. The color passed is just 1.0f, 0.0f, 0.0f in the code above. I just tried removing all other lights from the scene including the IBL and the light added by the model viewer class and I now just see a black screen.

To be clear all I have done in the above example is taken the example project at android/samples/sample-gltf-viewer/ and added the above code to: android/samples/sample-gltf-viewer/src/main/java/com/google/android/filament/gltf/MainActivity.kt

I just realized you didn't give your point light a radius. It's required for point and spot lights.

Ah yes sorry that works, sorry for my mis-understanding. Thank you for your help.


grunt fails to complete build

Hi,
I do not know if something is wrong with my network or not!
BEST,
.
.
.
Running "noflo_browser" task
Running "noflo_browser:main" (noflo_browser) task
Verifying property noflo_browser.main exists in config...OK
Files: component.json -> browser/noflo-ui.js
Options: require, development=false, debug=false, ide="https://app.flowhub.io", signalserver="https://api.flowhub.io", concurrency=10, plugins=["component-json","component-coffee","component-fbp"], remotes=[], graph_scripts=[], heads=[]
Reading /opt/test/noflo-ui/node_modules/grunt-noflo-browser/templates/graph.html...OK
install noflo/noflo@master
install noflo/noflo-strings@master
install noflo/noflo-ajax@master
install noflo/noflo-localstorage@master
install noflo/noflo-interaction@master
install noflo/noflo-objects@master
install noflo/noflo-groups@master
install noflo/noflo-routers@master
install noflo/noflo-dom@master
install noflo/noflo-core@master
install noflo/noflo-polymer@master
install noflo/noflo-indexeddb@master
install noflo/noflo-github@master
install noflo/noflo-graph@master
install noflo/noflo-runtime@master
install noflo/noflo-runtime-webrtc@master
install the-grid/flowhub-registry@master
install jonnor/microflo@master
install gjohnson/uuid@master
install bergie/octo@master
install<EMAIL_ADDRESS>install jashkenas/underscore@master
install flowbased/fbp-spec@master
exists noflo/noflo-graph@master
Warning: connect ETIMEDOUT Use --force to continue.
Aborted due to warnings.

Did you ever get the build do work? When I run grunt build it fails with: TypeError: key must be a string or number. undefined in lru-cache.js:18:11

Which versions of nodejs and NPM are you using? The build works on Travis (Saucelab tests sometimes fails though). https://travis-ci.org/noflo/noflo-ui/builds/94453497

...
install flowbased/fbp-spec@master
Warning: connect ECONNREFUSED <IP_ADDRESS>:443 Use --force to continue.

Please attach full log from clean rebuild (after say git clean -dfx).
Do you use a proxy?

Hi,
No I don't use proxy!
grunt-build.txt

root@biz1:/# node -v
v4.2.3
root@biz1:/# npm -v
2.14.7

Hi,
Something should be wrong in the site that I am trying to install. That IP address is a private one. Probably a router / firewall. The thing that I can't understand is how the build process has acquired it and tries to connect!

Hi,
I also have various failed. Some random due to what seemed a temp issue with github CDN or maybe grunt failing randomly its network connections? After retrying couple of times it ends up getting the files it was missing before.
However I have this error that is related to this commit in underscore repo:
Step 13 : RUN grunt build
 ---> Running in 76dc0957efa5
Running "inlinelint:all" (inlinelint) task
>> 44 files lint free.

Running "noflo_manifest:update" (noflo_manifest) task
File "component.json" updated.

Running "bower-install-simple:deps" (bower-install-simple) task
Executing Bower (Command: install)

Running "noflo_browser:main" (noflo_browser) task
install noflo/noflo@master
install noflo/noflo-strings@master
install noflo/noflo-ajax@master
install noflo/noflo-localstorage@master
install noflo/noflo-interaction@master
install noflo/noflo-objects@master
install noflo/noflo-groups@master
install noflo/noflo-routers@master
install noflo/noflo-dom@master
install noflo/noflo-core@master
install noflo/noflo-polymer@master
install noflo/noflo-indexeddb@master
install noflo/noflo-github@master
install noflo/noflo-graph@master
install noflo/noflo-runtime@master
install noflo/noflo-runtime-webrtc@master
install the-grid/flowhub-registry@master
install jonnor/microflo@master
install gjohnson/uuid@master
install bergie/octo@master
install<EMAIL_ADDRESS>install jashkenas/underscore@master
install flowbased/fbp-spec@master
Warning: failed to fetch https://raw.githubusercontent.com/jashkenas/underscore/master/component.json, got 404 "Not Found" Use --force to continue.

Aborted due to warnings.
Removing intermediate container 76dc0957efa5
The command '/bin/sh -c grunt build' returned a non-zero code: 6

If I install underscore beforehand with npm install underscore and then use grunt build --force I now have:
Step 13 : RUN grunt build --force
 ---> Running in 06e8e4d83eb8
Running "inlinelint:all" (inlinelint) task
>> 44 files lint free.

Running "noflo_manifest:update" (noflo_manifest) task
File "component.json" updated.

But then I don't have any index.html file (it is supposed to be generated, right?):
# ls -la
total 136
drwxr-xr-x 14 root root  4096 Jan  7 18:27 .
drwxr-xr-x 11 root root  4096 Jan  7 18:27 ..
drwxrwxr-x  8 root root  4096 Jan  7 17:31 .git
-rw-rw-r--  1 root root   297 Jan  7 17:31 .gitignore
-rw-rw-r--  1 root root   901 Jan  7 17:31 .travis.yml
-rw-rw-r--  1 root root  7270 Jan  7 17:31 CHANGES.md
-rw-rw-r--  1 root root   680 Jan  7 18:22 Dockerfile
-rw-rw-r--  1 root root 12092 Jan  7 17:31 Gruntfile.coffee
-rw-rw-r--  1 root root  1072 Aug 18  2014 LICENSE-MIT
-rw-rw-r--  1 root root  3494 Jan  7 17:31 README.md
drwxrwxr-x  2 root root  4096 Jan  7 17:31 app
-rw-rw-r--  1 root root   771 Jan  7 17:31 bower.json
drwxr-xr-x 20 root root  4096 Jan  7 18:26 bower_components
-rw-rw-r--  1 root root  7367 Jan  7 18:27 component.json
drwxrwxr-x 44 root root  4096 Jan  7 18:28 components
-rw-rw-r--  1 root root  1460 Aug 18  2014 config.dist.xml
drwxrwxr-x  2 root root  4096 Jan  7 17:31 css
drwxrwxr-x  2 root root  4096 Jan  7 17:31 elements
-rw-rw-r--  1 root root   147 Aug 18  2014 examples.json
drwxrwxr-x  2 root root  4096 Jan  7 17:31 graphs
-rw-rw-r--  1 root root  4173 Jan  7 17:31 index.dist.html
-rw-rw-r--  1 root root  1104 Jan  7 17:31 manifest.dist.json
-rw-rw-r--  1 root root  1125 Jan  7 17:31 manifest.dist.webapp.json
drwxr-xr-x 27 root root  4096 Jan  7 18:26 node_modules
-rw-rw-r--  1 root root  4286 Aug 18  2014 noflo.ico
-rw-rw-r--  1 root root  1225 Jan  7 17:31 package.json
drwxrwxr-x  2 root root  4096 Jan  7 17:31 spec
drwxrwxr-x  2 root root  4096 Aug 18  2014 src

Forgot the versions:
# node --version
v4.2.4
root@e60df4449e06:/# npm --version
2.14.12

Well, looking at the output of your CI build, it seems in my case grunt build does not execute the remaining tasks.
But if I execute them manually I get the following:
# grunt copy:themes
Running "copy:themes" (copy) task
Copied 2 files

Done, without errors.
root@f8d0c7a1a5ce:/usr/src/app# grunt vulcanize:app
Running "vulcanize:app" (vulcanize) task
Warning: ENOENT: no such file or directory, open '/usr/src/app/browser/noflo-noflo-polymer/noflo-polymer/noflo-polymer.html' Use --force to continue.

Aborted due to warnings.

Can't do much from here, the "browser" folder really do not exists...

That underscore issue is different from that by original reporter here, it is caused by a recent upstream change. Propose we keep that in https://github.com/noflo/noflo/issues/355

Yes, I initially though it was linked but after further investigation I opened noflo/noflo#355, indeed ;).

We're now using webpack and npm for deps, so this should no longer be an issue

I am getting this error while doing
npm run build

Running "exec:vulcanize" (exec) task

'node_modules' is not recognized as an internal or external command,
operable program or batch file.
Exited with code: 1.
Error executing child process: Error: Process exited with code 1.
Warning: Task "exec:vulcanize" failed. Use --force to continue.

@Mrinaldo
I got the same issue trying npm run build.... it seems is related on how windows OS handles the following command: 'node_modules/.bin/polymer-bundler index.dist.html > index.html' inside Gruntfile.coffe
Here is what I did in order to make it work

Open Gruntfile.coffe file and search for vulcanize
Replace command: 'node_modules/.bin/polymer-bundler index.dist.html > index.html' with
command: 'vulcanize.bat'
Create vulcanize.bat file in the same folder where Gruntfile.coffe resides including the following line
./node_modules/.bin/polymer-bundler index.dist.html > index.html
(it should include ./  prefix)
save vulcanize.bat
Run again: npm run build

I did above and I got my noflo ui working
I  think the issue is related with windows OS having problems to execute commands the way Gruntfile.coffe implements them, I assume build process will work just fine on any linux OS... just guessing
Hope this helps you moving forward


[BUG] ScheduledEnqueueTime is not respected

Library name and version
Azure.Messaging.ServiceBus v7.14.0
Describe the bug
ScheduledEnqueueTime is not respected, at least for small (10s of seconds) time periods.
Expected behavior
Scheduled enqueue time is respected:
Setting ScheduledEnqueueTime to         17/05/2023 1:01:23 PM
Sending message at                      17/05/2023 1:01:03 PM
Recieved message at                     17/05/2023 1:01:03 PM
Expected delay                          00:00:20
Actual delay                            00:00:20
----------------------------------------------
Setting ScheduledEnqueueTime to         17/05/2023 1:01:23 PM
Sending message at                      17/05/2023 1:01:03 PM
Recieved message at                     17/05/2023 1:01:03 PM
Expected delay                          00:00:20
Actual delay                            00:00:20
----------------------------------------------
Setting ScheduledEnqueueTime to         17/05/2023 1:01:23 PM
Sending message at                      17/05/2023 1:01:03 PM
Recieved message at                     17/05/2023 1:01:03 PM
Expected delay                          00:00:20
Actual delay                            00:00:20
----------------------------------------------

Actual delay should be ~20 seconds, += a few hundred milliseconds each way
Actual behavior
Setting ScheduledEnqueueTime to         17/05/2023 1:01:23 PM
Sending message at                      17/05/2023 1:01:03 PM
Recieved message at                     17/05/2023 1:01:03 PM
Expected delay                          00:00:20
Actual delay                            00:00:00.5169477
----------------------------------------------
Setting ScheduledEnqueueTime to         17/05/2023 1:01:23 PM
Sending message at                      17/05/2023 1:01:03 PM
Recieved message at                     17/05/2023 1:01:03 PM
Expected delay                          00:00:20
Actual delay                            00:00:00.1924140
----------------------------------------------
Setting ScheduledEnqueueTime to         17/05/2023 1:01:23 PM
Sending message at                      17/05/2023 1:01:03 PM
Recieved message at                     17/05/2023 1:01:03 PM
Expected delay                          00:00:20
Actual delay                            00:00:00.1067586
----------------------------------------------

Reproduction Steps
using Azure.Messaging.ServiceBus;

const int PadSize = -40;
const int EnqueueDelaySeconds = 20;
var connectionString = "<service bus connection string>"
var queue = "<queue name>";
var client = new ServiceBusClient(connectionString);
var sender = client.CreateSender(queue);
var processor = client.CreateProcessor(queue);

processor.ProcessMessageAsync += Processor_ProcessMessageAsync;
processor.ProcessErrorAsync += Processor_ProcessErrorAsync;

await processor.StartProcessingAsync();
await SendMessageAsync();

Console.ReadLine();

async Task Processor_ProcessMessageAsync(ProcessMessageEventArgs arg)
{
    try
    {
        Console.WriteLine($"{"Recieved message at",PadSize}{DateTime.UtcNow}");
        Console.WriteLine($"{"Expected delay",PadSize}{TimeSpan.FromSeconds(EnqueueDelaySeconds)}");
        Console.WriteLine($"{"Actual delay",PadSize}{DateTime.UtcNow.Subtract(SendTime)}");
        Console.WriteLine($"----------------------------------------------");
        await SendMessageAsync();
    }
    catch (Exception ex)
    {
        Console.WriteLine(ex.ToString());
    }
}

async Task SendMessageAsync()
{
    var message = new ServiceBusMessage();
    var scheduledEnqueueTime = DateTime.UtcNow.AddSeconds(EnqueueDelaySeconds);
    Console.WriteLine($"{"Setting ScheduledEnqueueTime to",PadSize}{scheduledEnqueueTime}");
    message.ScheduledEnqueueTime = scheduledEnqueueTime;
    SendTime = DateTime.UtcNow;
    Console.WriteLine($"{"Sending message at",PadSize}{DateTime.UtcNow}");
    await sender.SendMessageAsync(message);
}

Task Processor_ProcessErrorAsync(ProcessErrorEventArgs arg)
{
    throw new NotImplementedException();
}

public partial class Program
{
    public static DateTime SendTime;
}

Environment

.NET Version: .NET 6, but can repro on .NET 7.0 as well.
OS: Windows

dotnet --info
.NET SDK:
 Version:   7.0.400-preview.23225.8
 Commit:    c63317749e

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.22621
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\7.0.400-preview.23225.8\

Host:
  Version:      7.0.5
  Architecture: x64
  Commit:       8042d61b17

.NET SDKs installed:
  6.0.202 [C:\Program Files\dotnet\sdk]
  6.0.203 [C:\Program Files\dotnet\sdk]
  6.0.311 [C:\Program Files\dotnet\sdk]
  7.0.100 [C:\Program Files\dotnet\sdk]
  7.0.400-preview.23225.8 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 3.1.31 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.16 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 7.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 7.0.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.1.31 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 7.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 7.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.31 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.4 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.16 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 7.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 7.0.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

Other architectures found:
  x86   [C:\Program Files (x86)\dotnet]
    registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]

Environment variables:
  Not set

global.json file:
  Not found

Learn more:
  https://aka.ms/dotnet/info

Download .NET:
  https://aka.ms/dotnet/download

Hi @JakeYallop.  Thank you for reaching out and we regret that you're experiencing difficulties. The client library passes the desired scheduled time to the service as it was specified and does not inspect or transform it.  The decision for when a message should be available to read is made by the Service Bus service; the client has no influence nor control over that decision.
Unfortunately, this repository is focused on the Azure SDK for .NET - we're unable to assist with other Azure issues.  Since the behavior that you're inquiring about is related to the Service Bus service, your best path forward would be to open an Azure support request.
I'm going to close this out; if I've misunderstood what you're describing, please let us know in a comment and we'd be happy to assist as we're able.


[CELEBORN-1607] Fix openapi-client send worker event issue

What changes were proposed in this pull request?
For SendWorkerEventRequest eventType, using String instead of Enum.
Why are the changes needed?
I met exception when sending worker event with openapi sdk.
Exception in thread "main" ApiException{code=400, responseHeaders={Server=[Jetty(9.4.52.v20230823)], Content-Length=[491], Date=[Fri, 20 Sep 2024 23:50:27 GMT], Content-Type=[text/plain]}, responseBody='Cannot deserialize value of type `org.apache.celeborn.rest.v1.model.SendWorkerEventRequest$EventTypeEnum` from String "DecommissionThenIdle": not one of the values accepted for Enum class: [DECOMMISSION_THEN_IDLE, GRACEFUL, NONE, DECOMMISSION, IMMEDIATELY, RECOMMISSION]
 at [Source: (org.glassfish.jersey.message.internal.ReaderInterceptorExecutor$UnCloseableInputStream); line: 1, column: 14] (through reference chain: org.apache.celeborn.rest.v1.model.SendWorkerEventRequest["eventType"])'}
    at org.apache.celeborn.rest.v1.master.invoker.ApiClient.processResponse(ApiClient.java:913)
    at org.apache.celeborn.rest.v1.master.invoker.ApiClient.invokeAPI(ApiClient.java:1000)
    at org.apache.celeborn.rest.v1.master.WorkerApi.sendWorkerEvent(WorkerApi.java:378)
    at org.apache.celeborn.rest.v1.master.WorkerApi.sendWorkerEvent(WorkerApi.java:334)
    at org.example.Main.main(Main.java:22)

The testing code to re-produce:
package org.example;

import org.apache.celeborn.rest.v1.master.WorkerApi;
import org.apache.celeborn.rest.v1.master.invoker.ApiClient;
import org.apache.celeborn.rest.v1.model.ExcludeWorkerRequest;
import org.apache.celeborn.rest.v1.model.SendWorkerEventRequest;
import org.apache.celeborn.rest.v1.model.WorkerId;

public class Main {
    public static void main(String[] args) throws Exception {

        String cmUrl = "http://localhost:9098";
        WorkerApi workerApi = new WorkerApi(new ApiClient().setBasePath(cmUrl));
        workerApi.excludeWorker(new ExcludeWorkerRequest()
                .addAddItem(new WorkerId()
                        .host("localhost")
                        .rpcPort(1)
                        .pushPort(2)
                        .fetchPort(3)
                        .replicatePort(4)));
        workerApi.sendWorkerEvent(new SendWorkerEventRequest()
                        .addWorkersItem(new WorkerId()
                                .host("<IP_ADDRESS>")
                                .rpcPort(56116)
                                .pushPort(56117)
                                .fetchPort(56119)
                                .replicatePort(56118))
                .eventType(SendWorkerEventRequest.EventTypeEnum.DECOMMISSION_THEN_IDLE));
    }
}

Seems because for the EventTypeEnum, the name and value not the same and then cause this issue.
Not sure why the UT passed, but the integration testing failed.
For EventTypeEnum, because its value is case sensitive, so we meet this issue.
https://github.com/apache/celeborn/blob/8734d1663884345477257d2f10f4c19732c6f1c3/openapi/openapi-client/src/main/java/org/apache/celeborn/rest/v1/model/SendWorkerEventRequest.java#L47-L83
Related issue in jersey end I think, https://github.com/eclipse-ee4j/jersey/issues/5288
Not sure how to resolve it, to unblock, in this PR, I use string type instead of enum.
Does this PR introduce any user-facing change?
Yes, using string for SendWorkerEventRequest eventType.
How was this patch tested?
Existing UT & IT.

cc @pan3793 @SteNicholas @akpatnam25 @mridulm

For EventTypeEnum, because its value is case sensitive, so we meet this issue.

As mentioned in the PR description, if it is not case sensitive, it should be fine:
Solution 2:
https://github.com/apache/celeborn/pull/2754

close this one and prefer #2754


Issue 571: Catch develop-2.0.0 up to develop

This is a maintenance, catch-up Pull Request, incorporating no new features. Once CI passes, it can be merged.

@ucoProject/maintainers-uco : This is ready for merge.


FSharpMemberOrFunctionOrValue.Overloads does not return overloads

For example the following code:
type JustOverloads() =
  member x.Overload(i:int) =
    i + 42
  member x.Overload(s:string) =
    int s + 42

The symbol Overload is a MemberFunctionOrValue but is assigned a ValRef type so the Overloads code doesnt ever return an item:
    member x.Overloads matchParameterNumber =
        checkIsResolved()
        match d with
        | M m ->
            match item with
            | Item.MethodGroup (_name, methodInfos) ->
                let methods =
                    if matchParameterNumber then
                        methodInfos
                        |> List.filter (fun methodInfo -> not (methodInfo.NumArgs = m.NumArgs) )
                    else methodInfos
                methods
                |> List.map (fun mi -> FSharpMemberOrFunctionOrValue(cenv, M mi, item))
                |> makeReadOnlyCollection
                |> Some
            | _ -> None
        | _ -> None

Im not sure why the members are created as ValRef rather than MethInfo is it the case that only C# types with overloads are assigned that way?

I checked this with last build and now Overloads() returns an item, but it's always the same method without references to other overloads.
In above example, calling [JustOverloads Entity].MembersFunctionsAndValues.[1].Overloads(false) will give Some(seq [Overload(i:int)]) but you don't get information about Overload(s:string).
Moreover, if you call Overloads(false) on a FSharpMemberOrFunctionOrValue taken from a typed expression you still don't get any item.


split: correct filename creation algorithm

(Sorry for the large diff here. I tried to find a way to minimize the changes but there were several coupled changes and the algorithm is very unintuitive. I tried to add plenty of comments to help make sense of it.)
Fix two issues with the filename creation algorithm. First, this
corrects the behavior of the -a option. This commit ensures a
failure occurs when the number of chunks exceeds the number of
filenames representable with the specified fixed width:
$ printf "%0.sa" {1..11} | split -d -b 1 -a 1
split: output file suffixes exhausted

Second, this corrects the behavior of the default behavior when -a
is not specified on the command line. Previously, it was always
settings the filenames to have length 2 suffixes. This commit corrects
the behavior to follow the algorithm implied by GNU split, where the
filename lengths grow dynamically by two characters once the number of
chunks grows sufficiently large:
$ printf "%0.sa" {1..91} | ./target/debug/coreutils split -d -b 1 \
>   && ls x* | tail
x81
x82
x83
x84
x85
x86
x87
x88
x89
x9000

Wow! Great work on figuring this out! That's a really intriguing algorithm. At first, it indeed seemed really unintuitive, but I feel like it make sense looking at it iteratively. So I'm wondering whether it might make sense to embrace that iterative nature and build the next prefix from the last one. For example, I think this works correctly too:
fn main() {
    // The characters in the prefix, but reversed.
    let mut prefix = vec![b'a', b'a'];
    let mut zs = String::new();
    let mut len = 2;

    print_str(&zs, &prefix);
    for _ in 1..26*26 {
        for c in prefix.iter_mut() {
            *c += 1;
            // If we reach the character after 'z', we want to wrap back to 'a' and carry the increment to the next char
            // else we can stop.
            if *c > b'z' {
                *c = 'a';
            } else {
                break;
            }
        }
        // We have reached the final name for this length ("za", "zaa", etc.)
        // So set it to 'a' and append an 'a'. Also add a 'z' to the front.
        if prefix[len-1] == b'z' {
            prefix[len-1] = b'a';
            zs.push('z');
            prefix.push(b'a');
            len += 1;
        }
        print_str(&zs, &prefix);
    }
}

// This should be implemented by writing the bytes directly to stdout.
fn print_str(zs: &str, prefix: &[u8]) {
    println!(
        "{}{}",
        zs,
        String::from_utf8(prefix.iter().map(|&x| x).rev().collect::<Vec<_>>()).unwrap()
    )
}

(I also put this in a Rust Playground link, if you want to test it). We could then make this into an iterator of filenames in your Factory struct.
Granted, it's not immediately obvious how it works, but it sidesteps some of the mathematical reasoning necessary in your approach. What do you think?

Yes, there's something to that idea. My approach of computing the filename directly from the chunk index is more general than necessary, since we always visit chunks in order (that is, index 0, then 1, then 2, etc.). So using an iterator and just computing the successor to the current filename seems sensible. Thanks for coming up with the code for it, the output looks correct to me.
I can work on implementing this as an iterator, but in the meantime don't hesitate to merge this as-is if you decide it is acceptable, since it may take me some time.

I have made the requested change and rebased on master.


Use parameters in get_product_trades()

The parameters had no effect since they were never passed to
self._send_paginated_message()
Fixes #414

This isn't quite complete since I don't know what to do about results, but at least parameters like after are working now.

Are you sure these parameters aren't deprecated? I don't see them in the official documentation, and when I tried something like what you pushed here, I got
{'message': 'Invalid positive integer value for parameter - before'}
When parsing the output from the request... were you able to get actual data?

after works which is the only thing I tried.

It's definitely possible this needs to be rewritten completely.

Hi, what do you think about this approach?
    def get_product_trades(self, product_id, **kwargs):
        return self._send_paginated_message('/products/{}/trades'
                                            .format(product_id), params=kwargs)

It is the same approach that other paginated endpoints on authenticated_client.py


Setup github workflows

Towards Fix #69
Following a discussion with @gengliangwang , I tried to setup a CI that runs tests on every commit.

Hi @Pranavchiku, thank you for your contribution. The GitHub workflow might involve setting up certain GitHub secrets, which you might not have access to. I'm planning to set up this soon so that more people can contribute.
Also, it's worth noting that pyspark-ai does not use pytest. It would be great if we could use pytest instead of unittest in our codebase.

Exactly, I was wondering about secrets GitHub secrets and hence decided not to make more progress over it.

@Pranavchiku Thanks for the contribution. Since https://github.com/databrickslabs/pyspark-ai/pull/71 is merged, I am closing this one.


License for knightadjuster?

I see you haven't declared which license you're releasing this under.   I have a couple projects that I'd like to use this in, but basically if no licensing is declared, I just assume that it's not redistributable, as do most projects.

Thanks for getting in touch about this. I am not a licensing expert that's why I have not set one. In what kind of projects would you like to use it? What license would you think is appropriate for this project?

I personally want to package it for use in openSUSE, but this is a requirement for plenty of projects, to get software into their "official" respositories.
As far as licenses go, that sort of depends on your philosophy on FOSS.    If you want a "Hey, take this code, and use it, and don't worry about the details" Something like the Apache-2.0, MIT, or BSD-3-Clause licenses might fit.
If you're wanting a more "copyleft" style license, where it theoretically assures that any derivative works are also licensed in a "Free" manner, you might look at GPL-2.0-or-later or GPL-3.0-or-later.
Or you know, if you just don't want to worry about anything, there's always the WTFPL, which in short boils down to "Go ahead, take this code, and do whatever you want with it, I don't care, and stop bothering me about it"
Any of the ones I listed are suitable for inclusion in the majority of FOSS projects, but if you want to have a look at the list of SPDX licenses, they can be found at https://spdx.org/licenses/
Github has Choosealicense which walks you through a few questions to help you decide, if you're just not sure what you want to do.

Thanks a lot for the overview. I will add an MIT license file.
You can package it and it would be nice to reference it here.

Committed 02d8df2.


Add homology similarity scores

Solves issue: #280
But still most of the homologous relations don't have a similarity score (Such as https://github.com/obophenotype/brain_data_standards_ontologies/blob/issue-280/src/patterns/data/default/CCN202002013_class_homologous.tsv).
After updating centralized data following subcluster of changes occurred that need to be confirmed:
Marmoset individuals (src/templates/CCN201912131.tsv)

old : PCL:0015579	RNAseq 117 - CS201912131_79	Oligo L2-6 OPALIN MAP6D1	PCL:0015640|PCL:0015647|PCL:0015651
new : PCL:0015579	RNAseq 117 - CS201912131_79	Oligo L2-6 OPALIN MAP6D1	PCL:0015640|PCL:0015647

old : PCL:0015617	RNAseq 116 - CS201912131_117	Exc L5-6 FEZF2 LPO	PCL:0015639|PCL:0015647
new : PCL:0015617	RNAseq 116 - CS201912131_117	Exc L5-6 FEZF2 LPO	PCL:0015639

old : PCL:0015618	RNAseq 118 - CS201912131_118	OPC L1-6 PDGFRA COL20A1	PCL:0015647|PCL:0015651|PCL:0015683
new : PCL:0015618	RNAseq 118 - CS201912131_118	OPC L1-6 PDGFRA COL20A1	PCL:0015647|PCL:0015683

old : PCL:0015647	RNAseq 116-120 - CS201912131_147	Oligo-OPC	PCL:0015682
new : PCL:0015647	RNAseq 117-120 - CS201912131_147	Oligo-OPC	PCL:0015651

@hkir-dev - dont quite understand what the last column (with the changes) are?

ah ok, the changes arent wrong - seems like oligo-opc got mapped as a subcluster_of non-neuronal instead of a intermediary node (I think)

most of the homologous relations don't have a similarity score

hmm are you using these:
https://raw.githubusercontent.com/AllenInstitute/MOp_taxonomies_ontology/main/mouseMOp_CCN202002013/Mouse_CrossSpecies_Similarity.csv
https://raw.githubusercontent.com/AllenInstitute/MOp_taxonomies_ontology/main/marmosetM1_CCN201912132/Marmoset_CrossSpecies_Similarity.csv
https://raw.githubusercontent.com/AllenInstitute/MOp_taxonomies_ontology/main/humanM1_CCN201912131/Human_CrossSpecies_Similarity.csv
cause they look like they have all the scores regardless of if there's a relation, and we want to only represent those with the relation


Disable client operator's noobaa subscription management

Set a configuration for client operator's configmap to not manage noobaa's subscription while in provider mode

/retest

/lgtm

Do we want this change in 4.17 branch? If so, please create a bug and backport the fix.
Thanks.


⚠ Samsung has degraded performance

In 02f2fc1, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2472 ms

Resolved: Samsung performance has improved in c5ce12e.


Fix center view button appears outside the  GenericTilePolygonEditor

Fixes https://github.com/godotengine/godot/issues/89377
Also add a tooltip to it like the center button we have in tile atlas view.

Thanks!


Change communication protocol for DDC's dart:developer implementation

Currently DDC logs to the JS console whenever it needs to make a message available. This appears to have some performance overhead, especially if the Chrome inspector is open. A potential better alternative is to provide hooks on the page which DDC can detect and call directly.
cc @jakemac53
cc @vsmenon

Agreed.  Do you have a sense of whether we need to do this now, or whether it can wait till later (post 2.3)?

This likely can wait post 2.3. I only notice performance issues when the Chrome Inspector is open. Surprisingly it is made even worse if you set the logging level to Default (which hides these type of messages).  This implies that it is potentially a Chrome Inspector issue. Nonetheless the alternative I outlined is better for a number of reasons.

Ok.  We may want to add an abstraction layer here:
https://github.com/dart-lang/sdk/blob/master/pkg/dev_compiler/tool/input_sdk/patch/developer_patch.dart#L27
(and the other 4-5 call sites).  E.g., a hook that allows webdev bootstrap to override, but otherwise falls back on theconsole.debug("dart.developer...", ...) call.

That makes sense to me. Is this something @nshahan should pick up?

(unassigned because someone else may pick this up)
FWIW, it was @jacob314 who told me to use console logging, I think because it's easy to subscribe to those events from the Chrome debug protocol. But yeah, since a debugger is listening, we could basically do anything.
What sort of thing is easiest for the debug service to subscribe to? Are there other sorts of events besides console logging?

I think adding the ability to provide our own function to call instead of console.debug would be the most straightforward and efficient solution, as opposed to firing events through some other mechanism.
This is mostly only an issue because of how many events flutter fires (one for each frame), if you have an app that does constant animations (like the spinning square) it is really noticeable (and ironically during animations is the worst possible time to be causing jank...).

SGTM. The way we've handled this with similar things (such as "ignore whitelisted errors", hot restart) is to have a method on dart. that can be called to set things up. So you could pass in the callback for DDC to use.
Personally, I would remove the "console.debug" calls if they're slow. If the callback isn't registered, those calls can ignored.

I agree that we should switch off of console.debug for this. The chrome devtools implementation is far slower than I expected. It is strange it hasn't been optimized more.
Sorry for recommending it!

Adding to 2.3 for now as a p2.  Depending on perf, we can bump up or out.

Bumping to 2.4 - if we're hitting perf issues, we can pull forward.

Since y'all need to run JS code anyway, feel free to assign functions directly to the developer library object. For example:
let developer = dart_sdk.developer; // note: import Dart's SDK module somehow
// Intercept "dart:developer" inspect...
developer.inspect = function(object) {
  // ... your JS code here ...
  return object;
};

The current definition is simply:
developer.inspect = function(object) {
  console.debug("dart.developer.inspect", object);
  return object;
};

We can provide a cleaner hook for you if needed. Normally I wouldn't advise mucking with SDK internals, but I think the debug service protocol needs to do that anyway, e.g.  to find and call extensions.

Per offline conversation with @jakemac53 bumping from this milestone.  We can override in webdev today as well.


Generate warning message to try alternate MIR for degenerate cases

Randy Settgast had a test dataset for something he was working on in Ale3d. I've attached it here. It is 3 hexs in a row. The two outer hexes have material A and the middle hex has a mix of materials B & C. So, the one and only mixing zone doesn't have any materials in common with the (few) neighboring zones. Default MIR gets it wrong and displays the middle hex as a clean zone. PLIC gets it right. I am thinking it would be good to add logic to MIR to check that if MIR did not cut any zones but that the input mesh does indeed have mixed zones (with significant volume fractions) then issue a warning that it is a possible degenerate case and to attempt an alternate MIR.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1650
Status: Pending
Project: VisIt
Tracker: Feature
Priority: Normal
Subject: Generate warning message to try alternate MIR for degenerate cases
Assigned to: -
Category: -
Target version: -
Author: Mark Miller
Start: 11/07/2013
Due date:
% Done: 0%
Estimated time:
Created: 11/07/2013 06:53 pm
Updated: 11/26/2013 02:32 pm
Likelihood:
Severity:
Found in version: 2.12.3
Impact: 2 - Low
Expected Use: 3 - Occasional
OS: All
Support Group: Any
Description:
Randy Settgast had a test dataset for something he was working on in Ale3d. I've attached it here. It is 3 hexs in a row. The two outer hexes have material A and the middle hex has a mix of materials B & C. So, the one and only mixing zone doesn't have any materials in common with the (few) neighboring zones. Default MIR gets it wrong and displays the middle hex as a clean zone. PLIC gets it right. I am thinking it would be good to add logic to MIR to check that if MIR did not cut any zones but that the input mesh does indeed have mixed zones (with significant volume fractions) then issue a warning that it is a possible degenerate case and to attempt an alternate MIR.
Comments:

1657_feusion05_001.00001.zip


macOS TouchBar Button Swapped

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

VS Code Version: 1.57.0
OS Version: macOS Big Sur 11.4

Steps to Reproduce:

Tap icon on the right, code will run w/o debugger (tested for Python, Java and LLDB)
Tap icon on the left, code will run w/ debugger

Looks like the "Start Debugging" button and "Run Without Debugging" button swapped in latest version, but the icon apperance didn't swap.

Context: https://github.com/vadimcn/vscode-lldb/issues/462

@isidorn I can reproduce this issue, so the buttons seem to be swapped.

Actually the contents of the icons got updated:
the icon "continue-without-debugging-tb.png" became:

which is clearly wrong.

@isidorn I've renamed the icon to "run-with-debugging-tb.png" and fixed the code referencing the icons.
@0f3eacd3 you can try the fix in the next Insider build.

@weinand thanks for fixing this 👏
fyi @misolori since this was probably caused by accident when we added color to our touch bar icons

Thank you, @weinand!


add version information to rukpak built binaries

This PR cleans up the Makefile to make it more consistent, and adds a version flag to the binaries that are produced in rukpak: the unpack binary and the plain-v0-provisioner binary. This version information will be helpful as development accelerates and encoding version information in the binary will be useful when debugging.
$ ./bin/plain-v0 --version
Rukpak version: ba0e6da
Git commit: ba0e6da94d140e7e8b2780bc4f9a541d7f2ccdc8

Looks like this is still running into an error when running make build inside the container:  fatal: HEAD is neither a commit nor blob. I think it's important to fix since we want the version to come through on the deployed container image as well.
Right now the container builds but the version output is empty:
$ docker run -it --rm quay.io/operator-framework/plain-v0-provisioner:latest --version
Rukpak version:
Git commit:

/hold

Looks like adding the entire git tree resolves the issue.
/hold cancel

I think Go 1.18 makes this much simpler: https://tip.golang.org/doc/go1.18#go-version
Is it worth waiting for that?

Going to play around with this locally before approving.

Ran into an error locally:
make build-container
docker build -f Dockerfile -t quay.io/operator-framework/plain-v0-provisioner:latest .
Sending build context to Docker daemon    404MB
Step 1/20 : FROM golang:1.17-buster AS builder
 ---> fce131d0083a
Step 2/20 : WORKDIR /workspace
 ---> Using cache
 ---> a501ae717111
Step 3/20 : COPY go.mod go.mod
 ---> Using cache
 ---> 84221af3ddae
Step 4/20 : COPY go.sum go.sum
 ---> Using cache
 ---> 73155e76c4cf
Step 5/20 : RUN go mod download
 ---> Using cache
 ---> 0bc4d486549c
Step 6/20 : COPY Makefile Makefile
 ---> 6527f4cf18ad
Step 7/20 : COPY cmd cmd
 ---> 808ab6ce3c1a
Step 8/20 : COPY api api
 ---> ab3f6067d45a
Step 9/20 : COPY internal internal
 ---> 8bb7e882daf0
Step 10/20 : COPY provisioner provisioner
 ---> c60a98a84d8c
Step 11/20 : COPY pkg pkg
COPY failed: file not found in build context or excluded by .dockerignore: stat pkg: file does not exist
make: *** [Makefile:116: build-container] Error 1

Not sure why CI didn't catch this?

Figured it out: https://github.com/operator-framework/rukpak/pull/121.


Achievements and time not tracking with possible solutions

Hi, i just noticed that achievements and game time are not tracking. I did some modifications to the code based of off this pull request #28 and realized it is sending the wrong time format for game_time, it requires to use an epoch object instead of a string.
I already fixed that issue currently in my local build of the plug. It obtains the game time from the emulator so even if you play your games outside of gog it will track it the next time you sync it.
Now, the achievements are another thing, currently i made some big modifications and refactoring of the achievement object (for example i send the achievement id instead of None). I checked the gogo api documentation and made sure i sent the correct achievement object, i even compared the logs of the plugin to the official xbox. They send the objects the same way, the only difference is that xbox sends all data to galaxy.api.jsonrpc and this plugin sends it to root.
Here are some examples of achievement data:
Xbox
2022-07-27 21:34:09,552 - galaxy.api.jsonrpc - INFO - Sending notification: method=game_achievements_import_success, params={'game_id': '2043073184', 'unlocked_achievements': [Achievement(unlock_time=1639011428, achievement_id='1', achievement_name='First Contact')]}
rpcs3
2022-07-27 21:41:10,980 - root - INFO - Sending notification: method=game_achievements_import_success, params={'game_id': 'BLUS31396', 'unlocked_achievements': [Achievement(unlock_time=1577048967, achievement_id='001', achievement_name="Devil's Daughter")]}
The weird thing is that there is an osu plug in that tracks achievements (or tries), in my case it doesnt track them, but that plug in does send the data to the ga;axy.api.
Any thoughts, i really want to see my achievements :c

Hey, Got any solutions?


Add UC config for disabling storage hash validation

[x] I confirm this contribution is my own and I agree to license it with Apache 2.0.
[x] I confirm the moderators may change the PR before merging it in.
[x] I understand the release model prohibits detailed Git history and my contribution will be recorded to the list at the bottom of CONTRIBUTING.md.
Summary of changes

This PR tries to add one Update Client configuration option to disable the flow of storage hash validation for PAL using like PSA FWU API which doesn't support storage read of firmware candidate (no psa_fwu_read()).
@marcuschangarm

@ccli8 Thank you! I'll take a look!

@marcuschangarm I merge the suggestion and original #91 into this PR.

@ccli8 good catch with disabling ARM_UC_FEATURE_DELTA_PAAL!
I've sent you an email with an alternative solution to disabling hash calculations, since that would have a negative impact on our security story.

I've sent you an email with an alternative solution to disabling hash calculations, since that would have a negative impact on our security story.

@marcuschangarm I merge the in-transmit hash validation alternative into this PR. It's fine per my test.

@marcuschangarm Added PSA implementation update-client-hub/modules/pal-psa. It follows update-client-hub/modules/pal-flashiap-mcuboot for firmware metadata management assuming secure bootloader is TF-M integrated MCUboot, and invokes PSA FWU API to install firmware candidate.
Currently, only non-secure firmware update is supported. To support secure or combined secure/non-secure firmware update, it is necessary to read header (and TLV) of active located in SPE, which is prohibited by TF-M.


Invalid configuration object. Webpack has been initialized using a configuration object that does not match the API schema.

`λ npm run-script build

<EMAIL_ADDRESS>build D:\workplace\podium\web-portal
cross-env PUBLIC_URL='./' node scripts/build.js

(node:13432) [DEP_WEBPACK_SINGLE_ENTRY_PLUGIN] DeprecationWarning: SingleEntryPlugin was renamed to EntryPlugin
Creating an optimized production build...
Failed to compile.
Invalid configuration object. Webpack has been initialized using a configuration object that does not match the API schema.

configuration.module.rules[2] should be one of these:
["..." | object { compiler?, dependency?, descriptionData?, enforce?, exclude?, generator?, include?, issuer?, loader?, mimetype?, oneOf?, options?, parser?, realResource?, resolve?, resource?, resourceFragment?, resourceQuery?, rules?, sideEffects?, test?, type?, use? }, ...]
-> A rule.
Details:

configuration.module.rules[2].oneOf[6].loader should be a non-empty string.
-> A loader request.

configuration.node should be one of these:
false | object { __dirname?, __filename?, global? }
-> Include polyfills or mocks for various node stuff.
Details:

configuration.node has an unknown property 'dgram'. These properties are valid:
object { __dirname?, __filename?, global? }
-> Options object for node compatibility features.
configuration.node has an unknown property 'fs'. These properties are valid:
object { __dirname?, __filename?, global? }
-> Options object for node compatibility features.
configuration.node has an unknown property 'net'. These properties are valid:
object { __dirname?, __filename?, global? }
-> Options object for node compatibility features.
configuration.node has an unknown property 'tls'. These properties are valid:
object { __dirname?, __filename?, global? }
-> Options object for node compatibility features.
configuration.node has an unknown property 'child_process'. These properties are valid:
object { __dirname?, __filename?, global? }
-> Options object for node compatibility features.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: cross-env PUBLIC_URL='./' node scripts/build.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\bhara\AppData\Roaming\npm-cache_logs\2020-10-14T14_49_39_006Z-debug.log`

@Blari  problem still not fixed... could you please tell me solution if you can

@Blari problem still not fixed... could you please tell me solution if you can
Excuse me.
Have you tried updating the project and installing it from scratch? We've made a lot of edits and corrections in the meantime.

@Blari could i have updated project link to download ?

@Blari problem still not fixed... could you please tell me solution if you can

Excuse me.
Have you tried updating the project and installing it from scratch? We've made a lot of edits and corrections in the meantime.

https://github.com/flatlogic/light-blue-react-template/releases/tag/v.1.2.8


Random Port Mapping - Multi-protocol random ports

Rancher Version:
1.1.0.
Docker Version:
Docker version 1.11.2, build b9f10c9.
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 16.04 - bare metal dedi.
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node on same machine as controller, internal DB.
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Spawn a container which exposes the same port on UDP and TCP, IE: 27015/udp + 27015/tcp. Put in the internal ports but leave host blank to get random port mapping.
Results:
Two different port numbers, IE: 6123/udp 8370/tcp.
Expected:
The same random port for each protocl, IE: 6123/udp 6123/tcp.

All ports are assigned randomly regardless of what's the exposed port, so if you'd want ports of the same port number but different protocol to be randomly assigned to the same port, that would be an enhancement to how we handle random port assignment.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Slice by list

Just reinventing more of NumPy (probably this is a bad sign).
Will be useful for dask.
In [1]: from blaze import *
x =
In [2]: x = np.arange(20)

In [3]: d = Data(x)

In [4]: compute(d[[5, 3, 5]])
Out[4]:
array([5, 3, 5])

should this work for table-like things?
In [11]: from blaze import *

In [12]: df = pd.DataFrame({'a': np.random.randn(5)})

In [13]: t = symbol('t', discover(df))

In [14]: compute(t, df)
Out[14]:
          a
0 -1.058250
1 -0.306955
2  0.988621
3 -2.285035
4  1.309147

In [15]: compute(t[[0, 1]], df)  # NotImplementedError

Presumably it should work for all things.  I'm being pretty narrow about this at the moment.


how to build?

how to build in linux? how to add my domain as email website domain, and replace domain as domain use to send email?

The readme should answer these questions. Have a read through and let me know if you still have more.

how to config? whee is config file?

As per the readme. Config is defined via environment variables.


[Enhancement]Remove Jindo sdk.

Why I'm doing:
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[X] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[X] I have checked the version labels which the pr will be auto-backported to the target branch

[X] 3.2
[X] 3.1
[X] 3.0
[ ] 2.5

[Enhancement][branch-3.2]Remove Jindo sdk.(backport #42864) #42934

[Enhancement][branch-3.1]Remove Jindo sdk.(backport #42864) #42935

[Enhancement][branch-3.0]Remove Jindo sdk.(backport #42864) #42937


add metrics for application allocations

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@smorgrav FYI - didn't want to gate this on you but I think you're the one who knows which of the app, applicationId dimensions should be and are actually used?

I've lost track of which dimensions that we necessary and used at various places. I imagine that it is important for alerting (ie alert rules) and for metric dashboard (e.g list metrics on the application dashboard). @ldalves or @yngveaasheim are better candidates for this knowledge

@andreer please take a look here: https://git.ouroath.com/vespa/vespa-yahoo/blob/28fdd9be1c10cb5ef8ead89a9bae316cee99753a/hosted/host-admin/src/main/java/com/yahoo/vespa/hosted/hostadmin/task/docker/HostedNodeAgent.java#L575


lib.UI.fileBrowser no longer has PathItem

https://github.com/gferreira/hTools2_extension/blob/9e5150082a0a39847c1078aac3dc38d914a44f83/hTools2.roboFontExt/lib/init-RF-extension.py#L8
I just installed hTools2 in the latest RF3 beta and found a traceback, lib.UI.fileBrowser no longer has a PathItem but I'm not sure what the proper replacement is...

RFPathItem:
https://github.com/frankrolf/hTools2_extension/commit/66aa5043acbca4c35dee10dc32dab13cd19dfbd5


fix(app): Show only the credential matches the submission to multi credential shared

Signed-off-by: Talwinder kaur<EMAIL_ADDRESS>

Codecov Report
Patch and project coverage have no change.

Comparison is base (139404b) 90.19% compared to head (4087e2a) 90.19%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #417   +/-   ##
=======================================
  Coverage   90.19%   90.19%
=======================================
  Files          75       75
  Lines        2724     2724
=======================================
  Hits         2457     2457
  Misses        186      186
  Partials       81       81

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


update chart

@thiennv-qng Instead of a direct api call, I highly recommend that you should use redux to build and interface to get data from backend. Let's have a look at pool.reducer.js for practice.


Synthesized assemble model

SAT-1629
We use a naïve approach to our assemble model
We need a better one, based on program synthesis

Outdated.


Cryptographic APIs misuses

I'm a PhD student interested in finding security vulnerabilities in open source projects.
We found a total of 9  warnings (indicating potential vulnerabilities) when running the CogniCrypt static analyzer (*) on price-converter (or its library dependencies). We documented each one of these issues in private gists for the sake of confidentiality (non-disclosure).
Can you please let us know whether we can share these gists with you? We are eager to evaluate the perception of developers (e.g. severity of these warnings) and improve price-converter's security, and the quality of the reports of static analysis tools.
(*) https://github.com/CROSSINGTUD/CryptoAnalysis

Hello @hank-milliken,
Are you interested in receiving these warnings privately? If don't, should I close this issue?
Thanks for your collaboration.


chore: Switch to ruff

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

Comparison is base (f1b098d) 100.00% compared to head (1430482) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #92   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines         2620      2620
  Branches       490       490
=========================================
  Hits          2620      2620

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


Fix so setting of NC_FORMATX_NC3 in parallel is kept

If the user is opening a existing file for appending (NC_WRITE) in parallel and the file is in CDF5 format, the NC_interpret_magic_number() routine clears the model->impl setting of NC_FORMATX_PNETCDF which was set in NC_omodeinfer (See lines following the done: label in that routine which specifically set the impl if useparallel is true.)
This setting then gets overwritten when NC_interpret_magic_number is called which sets the model->impl back to NC_FORMATX_NC3.  This can (did) cause problems with parallel output as the NC3 format does not correctly handle parallel writing but the PNETCDF does.
Not sure if this is the best place for the test, but it did fix the parallel write issues I was seeing...
If you need more details on what is happening, let me know.  But a restatement at a higher level is that I was calling nc_open_par with NC_WRITE and NC_64BIT_DATA mode and the existing file has CDF5 for the magic number.  However, the dispatcher was being set to NC3_dispatch_table instead of NCP_dispatch_table which is the dispatcher which had been chosen for the original creation of the file being appended to.
I was then getting zeroes in the data being written to the vars since NC3 wasn't correctly handling multiple MPI ranks writing to different parts of the same variable...

I think the underlying problem is that nc_open_par is not leaving some flag set that
tell later code that the entry point was nc_open_par (or nc_create_par).
The NC_MPIIO etc flags are deprecated in netcdf.h, which is ok for external
callers, but we should still be using one of them internally as a mode flag that is set in nc_open_par so e.g. nc_interpret_magic_number sets the proper dispatch table.

I think the 'use_parallel' means it came in from the _par functions.

Yes, but I see the piece of code below in dinfermodel.c
and apparently it is not being used. Any idea why not?
/* Look at the magic number */
   if(NC_interpret_magic_number(magic,model) == NC_NOERR
   && model->format != 0) {
       if (model->format == NC_FORMAT_NC3 && use_parallel)
           /* this is called from nc_open_par() and file is classic */
           model->impl = NC_FORMATX_PNETCDF;
       goto done; /* found something */
   }

This isolation is really getting to me; In any case, this seems like the right solution.

Do I need to add anything to this to get it merged?  We need this or something like it to enable appending to existing netCDF files in our parallel runs...

Do I need to add anything to this to get it merged?  We need this or something like it to enable appending to existing netCDF files in our parallel runs...

Thanks @gsjaardema it is being merged now!

Thanks @gsjaardema it is being merged now!

Thanks.  that will help.

Thanks.  that will help.


Adding more examples to Readme, caveats and inEnvironment info

I hope these examples are helpful to people. I think your examples repo is helpful, but the examples are so complicated it doesn't give you an at-a-glance view of what the library can do. I was hoping to show people right away things like from, filter, reduce, catch and if they need it, inEnvironment.
I also wanted to make it clear that Node.js support was limited (I didn't realize that when I started). I think in the future Node users could be a major consumer of this library.  Sorry I didn't have time to submit a PR to make Node.js work.
I'm happy to make modifications or additions to this PR if you want

Thank you for the PR.
I will take a look soon when I have more time at hand. I think most is well explained with a potential to formulate some statements a little bit from a more positive perspective ;)


Cannot locate Mongo shell console under IntelliJ Tools

I'm using IntelliJ v15, and the I've configured the settings to point to my mongo.exe. However, the Mongo shell console option is not appearing under the Tools menu item, as shown in the GitHub instructions. Is the shell currently unavailable in this release?

Hi,
You may setup several replica sets in the server configuration. The console is unavailable because detecting the master replica set is quite difficult.
To fix that you could setup one host.
Tell me if it works and check if you can make some writing operations.
Thanks

Thanks for your reply, David. I attempted to register (alone, as you suggested) the 3 different MongoDB hosts I use (localhost included, which does not have multiple replica sets), and none allowed the MongoDB Console option to appear under the Tools menu.
I'd very much like to use this inside IntelliJIs, as the other features you've built into the plugin are superior to any other GUI I've used. Is there anything else I can do?

From: David Boissier<EMAIL_ADDRESS>Sent: Monday, November 23, 2015 2:44 AM
To: dboissier/mongo4idea
Cc: Aaron Jessen
Subject: Re: [mongo4idea] Cannot locate Mongo shell console under IntelliJ Tools (#125)
Hi,
You may setup several replica sets in the server configuration. The console is unavailable because detecting the master replica set is quite difficult.
To fix that you could setup one host.
Tell me if it works and check if you can make some writing operations.
Thanks
—
Reply to this email directly or view it on GitHubhttps://github.com/dboissier/mongo4idea/issues/125#issuecomment-158877838.

All suggestions about UX or features are welcomed :)

I'm sorry, but I may have been unclear in my last post. The issue is still unresolved.
For the sake of this discussion, let's just say that I have a single MongoDB instance, hosted at localhost:27017. It is a new database with only one collection in it, and the database does not have multiple replica sets. When I configure that server within your plugin's settings, there still isn't a Console option appearing under the Tools menu, as you detailed in the instructions.
Is there anything else you can suggest that I try in order to see the Console option?

Sorry for misreading your post.
The documentation is outdated :(.
Actually, you need to select a database under your mongo server node and the button will appear in the toolbar.

Okay, I see it now. Thanks, again, for such an awesome plugin!


Increase cluster name character limit

We should increase the cluster name character limit to the max allowed for each cloud provider (or the max of the shortest one if a simpler, faster solution suffices).  The schema check is currently capped at 13 chars which causes an exception to be thrown.
_An exception occurred during task execution. To see the full traceback, use -vvv. The error was:     'TheMotherOfAllClusters'
fatal: [localhost]: FAILED! => {"changed": false, "failed": true, "module_stderr": "Traceback (most recent call last):\n  File "/tmp/ansible_UvKuR3/ansible_module_jsonschema.py", line 184, in \n    main()\n  File "/tmp/ansible_UvKuR3/ansible_module_jsonschema.py", line 180, in main\n    module.fail_json(changed=False, msg="Validation failed", result=result)\n  File "/tmp/ansible_UvKuR3/ansible_modlib.zip/ansible/module_utils/basic.py", line 1993, in fail_json\n  File "/tmp/ansible_UvKuR3/ansible_modlib.zip/ansible/module_utils/basic.py", line 1973, in return_formatted\n  File "/tmp/ansible_UvKuR3/ansible_modlib.zip/ansible/module_utils/basic.py", line 414, in remove_values\n  File "/tmp/ansible_UvKuR3/ansible_modlib.zip/ansible/module_utils/basic.py", line 414, in \n  File "/tmp/ansible_UvKuR3/ansible_modlib.zip/ansible/module_utils/basic.py", line 414, in remove_values\n  File "/tmp/ansible_UvKuR3/ansible_modlib.zip/ansible/module_utils/basic.py", line 414, in \n  File "/tmp/ansible_UvKuR3/ansible_modlib.zip/ansible/module_utils/basic.py", line 425, in remove_values\nTypeError: Value of unknown type: <class 'jsonschema.exceptions.ValidationError'>, 'TheMotherOfAllClusters' is too long\n\nFailed validating u'maxLength' in schema['properties']['deployment'][u'properties'][u'clusters'][u'items'][u'properties'][u'name']:\n    {u'description': u'Name of this cluster.',\n     u'maxLength': 13,\n     u'minLength': 1,\n     u'type': u'string'}\n\nOn instance['deployment'][u'clusters'][0][u'name']:\n    'TheMotherOfAllClusters'\n", "module_stdout": "", "msg": "MODULE FAILURE", "rc": 0}

PR: https://github.com/samsung-cnct/k2/pull/733

fixed by pr


[FEATURE] Distinct Number OF Islands Using DFS Algorithm

Detailed description
Given a boolean 2D matrix grid of size n * m. You have to find the number of distinct islands where a group of connected 1s (horizontally or vertically) forms an island. Two islands are considered to be distinct if and only if one island is not equal to another (not rotated or reflected).
Context
This algorithm opens a wide variety of questions and this is basic need to other questions. This Question is the upgradation of the number of islands problem and is helpful in getting in-depth clarity
Possible implementation
Using the DFS algorithm and traversing every possible islands to get the number of islands
Additional information
No response

already done in previous  issue

@harshagrawal523  this is different question from that kindly read the desc properly

@Panquesito7 please review my PR for this question

please assign this issue to me

Please assign this task ,
I would really love to work at this

hi sir/madam
can you please assign me i want to contribute

Please, assign this problem to me.

hello sir can you assign this issue to me i want to solve this issue.


Rewrite vdslib tests from cppunit to gtest.

@vekterli please review

Travis failures in Java modules, unrelated to this PR
[ERROR] Failures:
[ERROR]   ModelContextImplTest.testModelContextTest:75
Expected: <false>
     but: was <true>
[ERROR]   HostedDeployTest.testAccessControlIsOnlyCheckedWhenNoProdDeploymentExists:272 expected:<3> but was:<4>
[ERROR]   HostedDeployTest.testDeployMultipleVersions:100 expected:<3> but was:<4>
[ERROR] Errors:
[ERROR]   ConfigServerBootstrapTest.testBootstrap:65 Â» InvalidApplication Invalid applic...
[ERROR]   ConfigServerBootstrapTest.testBootstrapWithVipStatusFile:98 Â» InvalidApplication
[ERROR]   HostedDeployTest.testCreateLatestMajorOnPreviousMajorIfItFailsOnMajorVersion8:174->deployWithModelForLatestMajorVersionFailing:207 Â» InvalidApplication
[ERROR]   HostedDeployTest.testCreateOnlyNeededModelVersions:129 Â» InvalidApplication In...
[ERROR]   HostedDeployTest.testCreateOnlyNeededModelVersionsNewNodes:159 Â» InvalidApplication
[ERROR]   HostedDeployTest.testThatConfigChangeActionsAreCollectedFromAllModels:334 Â» InvalidApplication


In String compareTo docs, "<" doesn't display in Android Studio tooltip

In String compareTo docs, "<" doesn't display in Android Studio tooltip.
I first reported this issue to Android Studio, but they claim the docs simply need to escape the character, so I'm making this issue here. See the screenshot there for what I'm talking about.
Here's the relevant comment.
Dart SDK version: 2.16.1 (stable) (Tue Feb 8 12:02:33 2022 +0100) on "windows_x64"; I'm using Android Studio.

Seems like a Language Server issue.
The source code is valid Markdown (CommonMark). The < in the comment does not need to be escaped, and indeed cannot be escaped inside a code fence (any escape character would also be included literally in the output).
If Android Studio is parsing the DartDoc comment itself, it's doing it wrong. Most likely it isn't.
If It's getting the comment from the language server, and the language server is providing HTML, then it needs to escape < characters inside code fences in the HTML code block (probably also & characters, although code like var x = foo  is likely rare).


[Feature Request] Add resolve options to worker and ssr configurations

Description
It is sometimes very useful to differentiate between resolution in different contexts (server, browser, worker and why not tests)
An example is a shared library that should work in node, client and workers but with some differences. A common thing is to put the differing parts in separate files for each platform.
Suggested solution
import core from './core'
# resolves based on environment and config
# core.server.js
# core.browser.js
# core.worker.js

With the following configuration
export default {
  resolve: { extensions: ['.browser.js', '.js'] },
  ssr: { resolve: { extensions: ['.server.js', '.js'] } },
  worker: { resolve: { extensions: ['.worker.js', '.js'] } },
  // test: { resolve: { extensions: ['.vitest.js', '.js'] } },
}

Alternative
The alternative would be to explicitely check for the current environment and use dynamic imports which is quite a gymnastic.
Additional context
A part of this pattern (Platform-specific extensions) is quite common and useful in react-native  https://reactnative.dev/docs/platform-specific-code#platform-specific-extensions
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Related: https://github.com/vitejs/vite/pull/13487


Rules & Memorandom Change

Issue 15475: /bcgov/entity###
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Change the comment in the ticket so it actually links

Need a resync to fix the conflicts

Update package.json and package-lock.json - whenever you're introducing new changes into business-edit-ui

Picking up this work in big PR.


Messageboxes when installing components

Display messages with info about components and with possibility to proceed/interrupt by pressing yes/no when installing.

Fixed in Master.

Fixed in Prod.


links no longer work after calling goTo

everything works fine in my app until I called goTo:
jQT.goTo('#rest', 'flip');
It takes you to the correct page, but then none of the links on that page work.
I was able to do a workaround, by adding an id to a link that already went to my '#rest' page, and then triggering the click/tap event on it like so:
$('#go-to-rest').trigger(clickEvent);
I'm pretty sure there is a bug at play with the goTo function... If anyone knows if/where I am going wrong, let me know.

I have found fix for this problem.
it works without animation :)
jQT.goTo('#rest');

Thanks hiddenpearls. It does work. After debugging at filled a bug: https://github.com/senchalabs/jQTouch/issues/306
which explains why this happens.


Could this please be added to Kirby's official plugins list?

It's not in Kirby's official plugin list? Could it be added there?
Process is described on the bottom of this page.
Thanks!

This issue can be closed: https://getkirby.com/plugins/jongacnik/fields-block

Hey thanks for the help 😅 It's often the smallest things I don't get around to.


Allow insecure HTTPS/TLS connection for HttpWaitStrategy

Some containers (fe OpenSearch, but there are many other examples) expose the connection endpoints over HTTPS protocol using self-signed certificates. The HttpWaitStrategy uses the default HttpsURLConnection / SSLSocketFactory which perform certificate validation and reject connections to such HTTPS endpoints. It would be great to provide the option to support HTTPS connection to servers that use self-signed certificates (basically, trust all certificate chains, no validation).
To be noted, in case of HttpsURLConnection, there is a way to disable certificates validation on per-JVM level (using HttpsURLConnection::setDefaultSSLSocketFactory) but it would affect every single HTTPS connection out there which is not desirable.
This feature nicely complements https://github.com/testcontainers/testcontainers-go/issues/45
@kiview would love to hear your opinion, it is not directly related to https://github.com/testcontainers/testcontainers-java/issues/4782 but popped up while implementing the integration, thank you.

Hey @reta, thanks for raising this PR, I can definitely see the need for this in certain scenarios. I also agree, setting this at the JVM level is generally a nogo, so it is very desirable to avoid it.
To understand your use case with regards to OpenSearch better, does it provide no way to run via HTTP? And how would you interact with it after container startup, if it uses HTTPS with self-signed certificates?

Hey @reta, thanks for raising this PR, I can definitely see the need for this in certain scenarios. I also agree, setting this at the JVM level is generally a nogo, so it is very desirable to avoid it.

:+1: totally agree, thank you

To understand your use case with regards to OpenSearch better, does it provide no way to run via HTTP?

As of now, the default images does HTTPS only but there are discussions on the subject, https://github.com/opensearch-project/OpenSearch/issues/1598 (in case your are interested)

And how would you interact with it after container startup, if it uses HTTPS with self-signed certificates?

To generalize, there are 2 ways right now: using HTTP(S) or TCP (transport level), for my particular case I am intended to use OpenSearch-provided Rest Client, but others do you TCP transport (in this case HTTP/HTTPS endpoints are useful for health / liveness checks).
Thank you!

So in case of using their provided REST client, can you configure it to allow insecure HTTPS as well? Just to understand if this helps this use case sufficiently, or if you just get stuck at another point (which would mean allowing HTTP upstream in the image could be the better approach).

So in case of using their provided REST client, can you configure it to allow insecure HTTPS as well? Just to understand if this helps this use case sufficiently, or if you just get stuck at another point (which would mean allowing HTTP upstream in the image could be the better approach).

Oh, got it, sorry, OpenSearch Rest Client use CloseableHttpAsyncClient (httpasyncclient) under the hood and it allows to configure SSL context with own trust managers / host verifiers, thanks!

Hey @kiview , my apologies, absolutely no pressure, just wondering if anything is holding this change back from being integrated? Thank you!

I have a similar use case and I would love for this feature to be merged.

My apologies for not responding earlier to the PR. Can we please add a (integration) test for the new functionality? Ideally using a minimal Docker image.

Thanks @kiview , the integration test has been added into existing Elasticsearch test suite (cuz it is also applicable there), please take a look at [1]
[1] https://github.com/testcontainers/testcontainers-java/pull/4951/files#diff-a6f3081bf09c2a39c5a6337b2d9754d7d51ed63f6ce655ad0661832163d0926aR288

Thanks for your contribution @reta ! this is now in master branch and will be part of the next version

Thanks for your contribution @reta ! this is now in master branch and will be part of the next version

Thanks a mill, @eddumelendez and @kiview !


original_dst: use localaddr if sock is transparent

Commit Message: original_dst - use localaddr if sock is transparent
Additional Description: original_dst to use localAddr if socket is transparent
Risk Level: low
Testing: n/a
Docs Changes:  n/a
Release Notes: n/a
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

bump @kyessenov

Can you please provide the motivation for this change? I think TPROXY is supported with the listener option https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/listener/v3/listener.proto#envoy-v3-api-field-config-listener-v3-listener-transparent and has been tested in the past.
Also, please clean up the diff and remove the orig files.
/wait


bstd.UnmarshalByteSlice panics

Describe the bug
bstd.UnmarshalByteSlice panics where len(slice) < 4 or returns incorrect slice with larger slice sizes.
To Reproduce
Run the following code.  Note how it panics with panic: runtime error: slice bounds out of range [4:3]
inByteSlice := []byte{1, 2, 3}
size := bstd.SizeByteSlice(inByteSlice)

buf := make([]byte, size)
bstd.MarshalByteSlice(0, buf, inByteSlice)

_, outByteSlice, err := bstd.UnmarshalByteSlice(0, buf) // <-- this panics

Expected behavior
Not to panic, and to return the correct outByteSlice.
Golang version:
go version go1.21.6 darwin/arm64
BENC version:
v1.0.5
Additional context
The fix looks to be replacing:
func UnmarshalByteSlice(n int, b []byte) (int, []byte, error) {
	// ...
	bs := b[n+4 : n+size]
	// ...
}

with:
func UnmarshalByteSlice(n int, b []byte) (int, []byte, error) {
	// ...
	bs := b[n+4 : n+4+size]
	// ...
}

A silly mistake I made, thanks for finding it. I'll fix it as soon as possible.

Fixed in v1.0.6


Remote example is not working

Looks like missing execute_batch() in the libSQL remote protocol implementation:
penberg@vonneumann libsql-js % node examples/remote/example.js
thread '<unnamed>' panicked at 'not yet implemented', /Users/penberg/.cargo/git/checkouts/libsql-00c15cfa15b9f13b/c032b9f/libsql/src/hrana/mod.rs:277:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
zsh: abort      node examples/remote/example.js

@penberg any follow ups to this?
I am doing the following and it is not working:

Spin up a simple libsql-server and inject sample data into it
curl -d '{"statements": ["CREATE TABLE IF NOT EXISTS users (username)", "INSERT INTO users VALUES (\"alice\")"]}' <IP_ADDRESS>:8082

Wrote a dead simple JS script to pull data from the table I created in step 1
// script called libsql_testing.js
const Database = require('libsql');

const url = process.env.LIBSQL_URL;
// const authToken = process.env.LIBSQL_AUTH_TOKEN;

// const opts = {
//   authToken: authToken,
// };

const opts = {};

const db = new Database(url, opts);
const row = db.prepare("SELECT * FROM users").get();
console.log(row);

I am greeted with the following error when running node libsql_testing.js
$ node libsql_testing.js

/app/node_modules/libsql/index.js:104
      throw new SqliteError(err.message, err.code, err.rawCode);
      ^
SqliteError: no such table: users
    at Database.prepare (/app/node_modules/libsql/index.js:104:13)
    at Object.<anonymous> (/app/libsql_testing.js:13:16)
    at Module._compile (node:internal/modules/cjs/loader:1218:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1272:10)
    at Module.load (node:internal/modules/cjs/loader:1081:32)
    at Module._load (node:internal/modules/cjs/loader:922:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:82:12)
    at node:internal/main/run_main_module:23:47 {
  code: 'SQLITE_ERROR',
  rawCode: 1
}

Node.js v19.3.0

I am not sure how to proceed as my setup could not be any simpler and this library is not working with the `libsql` project which it was designed upon


重叠

Issues Guideline
Following information can help us to resolve the issue faster.

Library version
Support-v4 version
Logs
Screenshots

In addition, we do not accept issues unrelated to Fragmentation.

调用loadRootFragment前进行savedInstanceState==null 。
主要解决内存重启后,Activity恢复时又add了一次


Don't reinitialize replica statuses after TFJob completes

/hold

This change is

Coverage decreased (-0.04%) to 64.826% when pulling 115893abad55535c3797d15b782cb309bcb8c0df on richardsliu:replicastatus into 451069f9798096d0b70be80809a2335f65ea313a on kubeflow:master.

After this change, when the job exits the replica statuses are:
Replica Statuses:
    PS:
      Active:  2
    Worker:
      Active:     3
      Succeeded:  1

PS pods are always shown active even if PS pods are deleted as per CleanPod policy

@johnugeorge Yes. They will be in Active state even after pods are deleted. But at least with this fix, the client can see how many workers succeeded/failed.

I think the status is still hard to understand from the user perspective. :thinking:

Should we just hide it from the output? If we do that we should make sure that JobCondition has enough information so the user can find out what went wrong (e.g. worker failed, PS died, etc).

I think we should keep it since Job resource in K8s also has the similar fields although it has conditions to tell users the status.
I think we could treat deleted pods as succeeded when the job is finished. WDYT @richardsliu @johnugeorge


Time selection and view are broken on versions 2.4.3 and 2.4.4

Hi guys,
First of all, thanks for the excellent job with Flatpickr! :)
The issue was inserted on version 2.4.3.
I use wrap and static in my config.
Please see the configuration:
this.flatpickr = this.$('.flatpickr').flatpickr({
  wrap: true,
  static: true,
  utc: true,
  enableTime: true,
  minuteIncrement: 1,
  'time_24hr': true,
  altInput: true,
  altFormat: 'F j, Y H:i'
})

      <input type="text" id="executeAt" name="executeAt" class="no-margin" data-input="" placeholder="Pick date and time" required />

    insert_invitation
    clear

And the result

Hello,
Please provide a fiddle if possible.

Here they are
v2.4.2
https://jsfiddle.net/celsomarques/sqtb7edq/#&togetherjs=Ob39cLmMCh
v2.4.4
https://jsfiddle.net/celsomarques/n9yukyh2/#&togetherjs=1GGFN2GpCw
From CDN the time selection works properly!
I'm using NPM.

Thank you for the fiddles.
So what exactly isn't working?
The npm version is identical to the CDN version.

From NPM, time selection and view are broken and from CDN, only view is broken.
In my project I build all JS (bundle.js) and CSS (bundle.css) together. I don't know if could cause this issue on time selection.

I use flatpickr in a webpack build without issues.
Are you importing the css correctly?

Yes, take a look
v2.4.4
https://jsfiddle.net/celsomarques/n9yukyh2/#&togetherjs=1GGFN2GpCw

Sorry, I'm still quite unsure what the issue is.
Everything looks fine.

The input field is not beside of group buttons, different from version 2.4.2

And I don't know why, from NPM version the time selection does not work properly
v2.4.2

v2.4.4

I'm seeing the same thing. 2.4.3 and 2.4.4 from NPM (with Yarn) won't show the time selection. 2.4.2 works as expected.

This is still happening in 2.4.7 (2.4.2 is the only one working for me)

Has this been fix? Still not able to select time in 2.4.7 only in 2.4.2.
@celsomarques are you able to select the time?
Do I need to change anything? @chmln

I'm still using 2.4.2


Timestep Window

Adding high-level tasks finalized from dev meeting.

Decided that we will not report frequency in the harness, since it is simple to report and does not require any calculation. It will still be reported in the overall NeuroBench leaderboard.