Undefined index: $servers and $ttl in neo4j driver

Describe the bug
I upgraded to version 2.1.2 and now the neo4j protocol doesn't work. I can use bolt or http directly but with neo4j i get an error:
PHP Notice:  Undefined index: servers in /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php on line 101

Notice: Undefined index: servers in /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php on line 101
PHP Notice:  Undefined index: ttl in /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php on line 101

Notice: Undefined index: ttl in /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php on line 101
PHP Fatal error:  Uncaught TypeError: Argument 1 passed to Laudis\Neo4j\Neo4j\RoutingTable::__construct() must be iterable, null given, called in /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php on line 122 and defined in /vendor/laudis/neo4j-php-client/src/Neo4j/RoutingTable.php:28
Stack trace:
#0 /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php(122): Laudis\Neo4j\Neo4j\RoutingTable->__construct()
#1 /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php(58): Laudis\Neo4j\Neo4j\Neo4jConnectionPool->routingTable()
#2 /vendor/laudis/neo4j-php-client/src/Bolt/Session.php(151): Laudis\Neo4j\Neo4j\Neo4jConnectionPool->acquire()
#3 /vendor/laudis/neo4j-php-client/src/Bolt/Session.php(136): Laudis\Neo4j\Bolt\Session->acquireConnection()
#4 /vendor/laudis/neo4j-php-client/src/Bolt/Session.php(76): Laudis\Neo4j\Bolt\Session->beginInstantTransaction()
#5 /vendor/lau in /vendor/laudis/neo4j-php-client/src/Neo4j/RoutingTable.php on line 28

Fatal error: Uncaught TypeError: Argument 1 passed to Laudis\Neo4j\Neo4j\RoutingTable::__construct() must be iterable, null given, called in /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php on line 122 and defined in /vendor/laudis/neo4j-php-client/src/Neo4j/RoutingTable.php:28
Stack trace:
#0 /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php(122): Laudis\Neo4j\Neo4j\RoutingTable->__construct()
#1 /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php(58): Laudis\Neo4j\Neo4j\Neo4jConnectionPool->routingTable()
#2 /vendor/laudis/neo4j-php-client/src/Bolt/Session.php(151): Laudis\Neo4j\Neo4j\Neo4jConnectionPool->acquire()
#3 /vendor/laudis/neo4j-php-client/src/Bolt/Session.php(136): Laudis\Neo4j\Bolt\Session->acquireConnection()
#4 /vendor/laudis/neo4j-php-client/src/Bolt/Session.php(76): Laudis\Neo4j\Bolt\Session->beginInstantTransaction()
#5 /vendor/lau in /vendor/laudis/neo4j-php-client/src/Neo4j/RoutingTable.php on line 28

The code is here:
    /**
     * @param ConnectionInterface<Bolt> $driver
     *
     * @throws Exception
     */
    private function routingTable(ConnectionInterface $connection): RoutingTable
    {
        if ($this->table === null || $this->table->getTtl() < time()) {
            /** @var Bolt */
            $bolt = $connection->getImplementation();
            $protocol = $connection->getProtocol();
            if ($protocol->compare(ConnectionProtocol::BOLT_V43()) >= 0) {
                /** @var array{rt: array{servers: list<array{addresses: list<string>, role:string}>, ttl: int}} $route */
                $route = $bolt->route();
                ['servers' => $servers, 'ttl' => $ttl ] = $route['rt']; // <-- $servers and $ttl not defined before this point
            } elseif ($protocol->compare(ConnectionProtocol::BOLT_V40()) >= 0) { // not sure what this is checking but it returns true in my case
                $bolt->run('CALL dbms.routing.getRoutingTable({context: []})');
                /**
                 * @var iterable<array{addresses: list<string>, role:string}> $servers
                 * @var int                       $ttl
                 */
                ['servers' => $servers, 'ttl' => $ttl] = $bolt->pullAll(); // <-- $servers and $ttl not defined before this point
            } else {
                $bolt->run('CALL dbms.cluster.overview()');
                /** @var list<array{addresses: list<string>, role: string}> */
                $response = $bolt->pullAll();

                /** @var iterable<array{addresses: list<string>, role:string}> $servers */
                $servers = []; // <-- $servers first time being set
                $ttl = time() + 3600;// <-- $ttl first time being set
                foreach ($response as $server) {
                    $addresses = $server['addresses'];
                    $addresses = array_filter($addresses, static fn (string $x) => str_starts_with($x, 'bolt://'));
                    /**
                     * @psalm-suppress InvalidArrayAssignment
                     *
                     * @var array{addresses: list<string>, role:string}
                     */
                    $servers[] = ['addresses' => $addresses, 'role' => $server['role']];
                }
            }

            $this->table = new RoutingTable($servers, $ttl); // this is where the fatal error occurs since these variables are not set
        }

        return $this->table;
    }

It is an easy enough problem to fix but i dont know what the value of $servers or $ttl need to be.  If i knew that i could do a pull request and fix it myself which may bring me to another bug. who knows?
To Reproduce
Steps to reproduce the behavior:
I guess figure out how to make this true when establishing a connection and run it:
($protocol->compare(ConnectionProtocol::BOLT_V40()) >= 0)

All i have to do to create it is run the neo4j driver.
Expected behavior
connection successful to the DB
Desktop (please complete the following information):

Library version: 2.1.2
Neo4j Version: 4.2.0
PHP version: 7.4.3
OS: Linux, Ubuntu

ok, i learned about  setting array values into another array since PHP 7.1 (https://wiki.php.net/rfc/short_list_syntax) so I now have more knowledge in my toolbox but, It was not working correctly and the variables were never declared.
I tried to trace down what would cause the following to be true:
($protocol->compare(ConnectionProtocol::BOLT_V40()) >= 0)

and i hit a dead end. From there I decided to see how the query inside of that conditional works against my DB.
I got this:

and then i figured that it would pick up the keys and assign them correctly.
it did not...
From there i wanted to see what this produced:
$bolt->pullAll();

it did this:
Array
(
    [0] => Array
        (
            [0] => 300
            [1] => Array
                (
                    [0] => Array
                        (
                            [addresses] => Array
                                (
                                    [0] => {ip address}:7687
                                )

                            [role] => WRITE
                        )

                    [1] => Array
                        (
                            [addresses] => Array
                                (
                                    [0] => {ip address}:7687
                                    [1] => {ip address}:7687
                                )

                            [role] => READ
                        )

                    [2] => Array
                        (
                            [addresses] => Array
                                (
                                    [0] => {ip address}:7687
                                    [1] => {ip address}:7687
                                    [2] => {ip address}:7687
                                )

                            [role] => ROUTE
                        )

                )

        )

    [1] => Array
        (
            [bookmark] => {some weird looking key here}
            [type] => s
            [t_last] => 1
            [db] => {DB Name}
        )

)

I realized that the whole structure was all wrong.
I made some adjustments to the code and it worked:
changed this on file /vendor/laudis/neo4j-php-client/src/Neo4j/Neo4jConnectionPool.php
/**
     * @param ConnectionInterface<Bolt> $driver
     *
     * @throws Exception
     */
    private function routingTable(ConnectionInterface $connection): RoutingTable
    {
        /** @var Bolt */
        $bolt = $connection->getImplementation();
        $protocol = $connection->getProtocol();
        if ($protocol->compare(ConnectionProtocol::BOLT_V43()) >= 0) {
            /** @var array{rt: array{servers: list<array{addresses: list<string>, role:string}>, ttl: int}} $route */
            $route = $bolt->route();
            ['servers' => $servers, 'ttl' => $ttl] = $route['rt'];
            $ttl += time();
        } elseif ($protocol->compare(ConnectionProtocol::BOLT_V40()) >= 0) {
            $bolt->run('CALL dbms.routing.getRoutingTable({context: []})');
            /**
             * @var iterable<array{addresses: list<string>, role:string}> $servers
             * @var int                       $ttl
             */
            ['servers' => $servers, 'ttl' => $ttl] = $bolt->pullAll();
            $ttl += time();
        } else {
            $bolt->run('CALL dbms.cluster.overview()');
            /** @var list<array{addresses: list<string>, role: string}> */
            $response = $bolt->pullAll();

            /** @var iterable<array{addresses: list<string>, role:string}> $servers */
            $servers = [];
            $ttl = time() + 3600;
            foreach ($response as $server) {
                $addresses = $server['addresses'];
                $addresses = array_filter($addresses, static fn (string $x) => str_starts_with($x, 'bolt://'));
                /**
                 * @psalm-suppress InvalidArrayAssignment
                 *
                 * @var array{addresses: list<string>, role:string}
                 */
                $servers[] = ['addresses' => $addresses, 'role' => $server['role']];
            }
        }

        return new RoutingTable($servers, $ttl);
    }
}

To this
/**
     * @param ConnectionInterface<Bolt> $driver
     *
     * @throws Exception
     */
    private function routingTable(ConnectionInterface $connection): RoutingTable
    {
        if ($this->table === null || $this->table->getTtl() < time()) {
            /** @var Bolt */
            $bolt = $connection->getImplementation();
            $protocol = $connection->getProtocol();
            $servers = []; // moved this out of the conditional so it is always defined
            $ttl = time() + 3600; // moved this out of the conditional so it is always defined. still dont know why this is the default value
            if ($protocol->compare(ConnectionProtocol::BOLT_V43()) >= 0) {
                /** @var array{rt: array{servers: list<array{addresses: list<string>, role:string}>, ttl: int}} $route */
                $route = $bolt->route();
                ['servers' => $servers, 'ttl' => $ttl ] = $route['rt'];
            } elseif ($protocol->compare(ConnectionProtocol::BOLT_V40()) >= 0) {
                $bolt->run('CALL dbms.routing.getRoutingTable({context: []})');

                $response = $bolt->pullAll();

                $ttl = $response[0][0]; // i set this since it comes back from the server. no idea if this is correct
                foreach ($response[0][1] as $server) { // looping through the servers since the original keys were lost with pullAll() function
                    $servers[] = ['addresses' => $server['addresses'], 'role' => $server['role']]; // and then set the results
                }
            } else {
                $bolt->run('CALL dbms.cluster.overview()');
                /** @var list<array{addresses: list<string>, role: string}> */
                $response = $bolt->pullAll();

                /** @var iterable<array{addresses: list<string>, role:string}> $servers */
                foreach ($response as $server) {
                    $addresses = $server['addresses'];
                    $addresses = array_filter($addresses, static fn (string $x) => str_starts_with($x, 'bolt://'));
                    /**
                     * @psalm-suppress InvalidArrayAssignment
                     *
                     * @var array{addresses: list<string>, role:string}
                     */
                    $servers[] = ['addresses' => $addresses, 'role' => $server['role']];
                }
            }
            $this->table = new RoutingTable($servers, $ttl);
        }

        return $this->table;
    }

it now works as expected but i dont know how to troubleshoot the first conditional or if this is even correct for the rest of the code.
if this looks good i can do a merge request. let me know.

I see the pull request with the corrected code. thank you so much @transistive! I will close this bug now.


[FAILED] use ceph-volume lvm batch to create bluestore osds

Bug Report
What happened: ceph-ansible can be installed.
What you expected to happen: ceph-ansible tries to make excceed volume group
like this:
Running command: /usr/sbin/lvcreate --yes -l 572318 -n osd-block-0c40c23f-54ee-4ec9-86e3-65b2a88e6c78 ceph-53c2ffbe-921b-4416-900f-2181b78b8efb
stderr: /dev/sdj: open failed: No medium found
stderr: /dev/sdj: open failed: No medium found
stderr: Volume group "ceph-53c2ffbe-921b-4416-900f-2181b78b8efb" has insufficient free space (572317 extents): 572318 required.
So I tried to "/usr/sbin/lvcreate --yes -l 572317 -n osd-block-0c40c23f-54ee-4ec9-86e3-65b2a88e6c78 ceph-53c2ffbe-921b-4416-900f-2181b78b8efb" not 572318
It does make it!
How to reproduce it (minimal and precise):

Share your group_vars files, inventory and full ceph-ansibe log
Environment:

OS (e.g. from /etc/os-release): Ubuntu 20.04.2 LTS
Kernel (e.g. uname -a): Linux 5.4.0-65-generic #73-Ubuntu SMP Mon Jan 18 17:25:17 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Docker version if applicable (e.g. docker version): 19.03.8 (non-containered installation, only metric)
Ansible version (e.g. ansible-playbook --version): 2.9.6
ceph-ansible version (e.g. git head or tag or stable branch): commit 9359ee913fb6ca42dd9cb7cc30c0072bc0cb652d (HEAD -> stable-5.0, origin/stable-5.0)
Ceph version (e.g. ceph -v): ceph version 15.2.8 (bdf3eebcd22d7d0b3dd4d5501bee5bac354d5b55) octopus (stable)

Sorry. I find Same Issue.


Asynchronous test failures cause cascading bugs in asynchronous tests

If a test fails asynchronously, or at least in a Future that's expected to complete, it causes all sorts of havoc for future tests. Consider the following example:
    Future get async {
      var completer = new Completer();
      new Timer(0, (_) => completer.complete(null));
      return completer.future;
    }
    test('test 1', () {
      expect(async.transform((_) {
        throw 'error 1';
      }), completes);
    });
    test('test 2', () {
      expect(async.transform((_) {
        throw 'error 2';
      }), completes);
    });
When run, only test 1 reports an error. Test 2's asynchronous code is apparently never run at all.
Consider also the following code:
      setUp(() => print("setUp"));
      tearDown(() => print("tearDown"));
      test('test 1', () {
        expect(async.transform((_) {
          throw 'error 1';
        }), completes);
      });
      test('test 2', () {
        print("in test 2 synchronously");
        expect(async.transform((_) {
          print("in test 2 asynchronously");
        }), completes);
      });
      test('test 3', () {
        print("in test 3 synchronously");
        expect(async.transform((_) {
          print("in test 3 asynchronously");
        }), completes);
      });
This prints:
    setUp
    tearDown
    setUp
    in test 2 synchronously
    tearDown
    setUp
    in test 3 synchronously
    in test 2 asynchronously
    in test 3 asynchronously
    tearDown
Note that test 2's asynchronous callback isn't run until after its tearDown method has been called and after test 3's setUp method is called and test 3 has been run synchronously.

This issue has been moved to dart-lang/test#163.


Cant change Icon Layout

Using this mod you cannot change the Icon Layout when the dropdown overlaps the recipe types. It should have the focus on the dropdown but focus is still on the recipe types when you get down to that area.

Thanks for reporting. I will check this.

Fixed in v1.0.7


os/exec: Cmd.Args not passed to the command as environmental variables

What version of Go are you using (go version)?

$ go version go1.20.4 darwin/arm64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/karthikeyan_s/Library/Caches/go-build"
GOENV="/Users/karthikeyan_s/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/karthikeyan_s/go/pkg/mod"
GONOPROXY="github.com"
GONOSUMDB="github.com"
GOOS="darwin"
GOPATH="/Users/karthikeyan_s/go"
GOPRIVATE="github.com"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.20.4"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/wn/dhtfqqk17y95h22m4lh2_n780000gn/T/go-build605004963=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I was going through the ffmpeg docs and found a way to create a log file by setting an env variable FFREPORT. Initially, I couldn't find any doc on how to set a command specific environment variable in golang(This is important for me because I run multiple ffmpeg commands inside the program and want different env vars for them).  Then, I found this SO answer and tried it out.
What did you expect to see?
The env variable FFREPORT being recognised and used by the ffmpeg command.
What did you see instead?
It didn't look like the env var was being read.
The command ran fine, but it didn't look like ffmpeg recognized the env var FFREPORT I set. So, I tried to run a simple shell script to check if it was working fine and I found this. The env vars set before calling the Cmd.Start aren't actually present (or accessible) to the command being run.
Reproduce the issue
You can easily reproduce this issue with a simple shell script:
#!/usr/bin/env bash
echo ${KEY1}
echo ${KEY2}

printenv

Now, try running this sh script from this go program on playground locally.
We set 2 env vars, KEY1 and KEY2 in the go program and ideally, you should see the output:
val1
val2
// then, all the env vars

But instead, the output you will see will be 2 empty lines and then all the env vars. And you won't find those 2 env vars set from the program in them anywhere.

The environment is Env, not Args.
The Go project doesn't use the issue tracker for questions. See https://golang.org/wiki/Questions.


Create service.html

@SSR12345678
Awesome contribution to Hacktoberfest'24! 🎉 Your work is super valuable. Keep up the great work! Just a quick note — please remember to mention the issue number from next time. Thanks!


Multiple double tap events firing?

Problem/Motivation

After a recent update, when double pressing my Jasco ZWave Dimmers the system registers four 'double presses' milliseconds apart.  These events show up identically other than the unique id.  This wouldn't be an issue, except that I use multiple double presses to trigger different actions in my system.

Expected behavior

Ideally there would only be one event sent per double press.

Actual behavior

Four events appear of the type: event_type: zwave_js_value_notification

Steps to reproduce

Bind device to double tap on the controller. Double tap light switch.

Proposed changes

Unknown.

Hi there @lkroeker 👋
It seems like you are experiencing an issue with Z-Wave JS itself, not the add-on specifically.
Add-ons wrap existing applications, in this case Z-Wave JS UI. If you experience issues with the software itself, you should report it to the upstream project. Once it gets resolved and a release with the fixes is made publicly available from upstream, the add-on will update as well.
Thanks 👍
../Frenck


Update shutdown-k3s.service

Some little changes to shutdown-k3s.service. Mostly to prevent it from running if no k3s-killall script is available (e.g. in ISOs or before provisioning).
However the relevant fix is at https://build.opensuse.org/request/show/1068496 to ensure this service is enabled by default.
Related to #587

How does shutdown-k3s interact with RKE2 nodes ?!

How does shutdown-k3s interact with RKE2 nodes ?!

Good point. According to https://github.com/k3s-io/k3s/issues/2400#issuecomment-1040795816 I bet there should be an analog process for rke2
In addition I am still noticing some weird behavior related to this service on my env. On  a provisioned cluster if I am run in the shell:
k3s-killall.sh && shutdown -r now

it reboots without issues, no errors during shutdown. However if shutting down directly and relaying on shutdown-k3s.service to stop k3s I don't see #587 (it is gone), however I consistently see an error umounting /usr/local/bin... this is weird. Trying to figure out the difference...
Marking it as draft for now, in any case I believe this is needed. At least to prevent service execution at shutdown before provisioning. In fact I believe this should be provided as part of the provisioning process...

This is are the logs I get with a for a shutdown call including the changes in this PR:
[  OK  ] Unmounted /var/lib/rancher.
[  OK  ] Stopped Flush Journal to Persistent Storage.
         Unmounting /etc...
         Unmounting /var/lib/kubelet...
         Unmounting /var/log...
[  OK  ] Unmounted /etc.
[  OK  ] Unmounted /var/lib/kubelet.
[  OK  ] Unmounted /var/log.
         Unmounting /usr/local...
         Unmounting /var...
[FAILED] Failed unmounting /usr/local.
[  OK  ] Unmounted /var.
         Unmounting /run/overlay...
[  OK  ] Unmounted /run/overlay.
[  OK  ] Stopped target Preparation for Local File Systems.
[  OK  ] Stopped target Swaps.
         Stopping Monitoring of LVM…meventd or progress polling...
[  OK  ] Stopped Create Static Device Nodes in /dev.
[  OK  ] Stopped Create System Users.
[  OK  ] Stopped Remount Root and Kernel File Systems.
[  OK  ] Stopped Monitoring of LVM2… dmeventd or progress polling.
[  OK  ] Finished Kill containerd-shims on shutdown.
[  OK  ] Reached target System Shutdown.
[  OK  ] Reached target Unmount All Filesystems.
[  OK  ] Reached target Late Shutdown Services.
[  OK  ] Finished System Reboot.
[  OK  ] Reached target System Reboot.
[  124.714236][    T1] watchdog: watchdog0: watchdog did not stop!
[  124.730204][    T1] systemd-shutdown[1]: Syncing filesystems and block devices.
[  124.732986][    T1] systemd-shutdown[1]: Sending SIGTERM to remaining processes...
[  124.740777][ T1163] systemd-journald[1163]: Received SIGTERM from PID 1 (systemd-shutdow).
[  124.756717][    T1] systemd-shutdown[1]: Sending SIGKILL to remaining processes...
[  124.763001][    T1] systemd-shutdown[1]: Using hardware watchdog 'iTCO_wdt', version 0, device /dev/watchdog
[  124.767621][    T1] systemd-shutdown[1]: Unmounting file systems.
[  124.770725][ T4434] [4434]: Remounting '/usr/local' read-only in with options '(null)'.
[  124.835080][ T4434] EXT4-fs (vda5): re-mounted. Opts: (null). Quota mode: none.
[  124.844964][ T4435] [4435]: Unmounting '/usr/local'.

/usr/local fails to unmount during the first attempt, however it is successfully remounted to ro and unmounted afterwards in later stages. I don't think this is a relevant issue, but still a sign we might not properly shutdown k3s for a reboot. I pretty much convinced this is a race issue between umount.target and shutdown-k3s.service. See the shutdown service finishes attempting to umount
Interesting enough I don't see the issue if I increase the systemd verbosity to debug shutdown:
systemctl log-level debug
systemctl log-target console
shutdown -r now

Shutsdown without issues.
I have spent whole day running trials in Rke2 and k3s trying to find a way to include killall scripts as part of the shutdown and make sure they are finalized before umount.target pulls all dependencies (or conflicts to mountpoints). I could not find a way. In any case in both cases, for RKE2 and K3S, the shutdown is quick (with errors on /opt and /usr/local respectively) and reboots without errors. I'd say we can leave it like that for the time being. In all cases running the killall script just before calling shutdown works smooth.

IMHO this is an issue for rancher provisioning, we should be providing such a service file, this should be provided as part of the provisioning, so the same logic that provides the k3s.service should provide a service or a way to reboot without having unclean umounts.


Create Read-Only Admin Role for Northstar API Users

First thought up for the Quasar platform, it would be useful to have OAuth role that allow Admin Read-Only access so they can access all data, but not destroy/modify it.

I think this is a super-reasonable thing to have. 👍

This is done in #726!


[bug] NextJS | Lazyloading Components

Describe the bug
I have Header, Video, Footer coming as part of this.childComponents... Is there a way to lazy load my footer  component or have more control on the list of components without breaking in-context editing?
https://github.com/adobe/aem-react-editable-components/blob/d01b307cf6726399ba9a13d2abb37cb2bc004adc/src/components/Page.tsx#L91
Package version
^1.1.6
To Reproduce
N/A
Expected behavior

Would be great if there are some useful examples / documentation around these packages.

Screenshots
N/A
Additional context
N/A

With NextJS I don't believe we have an example like this currently.
We got lazyloading examples currently with react universal component on vanilla react, in our archetype.
This is how I would try to approach it:
Try to leverage [custom webpack config]( https://nextjs.org/docs/api-reference/next.config.js/custom-webpack-config to integrate the react universal component) somehow in combination with react universal component.
Now, this is a third party plugin. Once React 18 is released (not in alpha) then we can move into using React.Lazy() which will be better and might actually solve this, provided we can use React.Lazy with NextJS.


[Storage] Remove validator column family

Motivation
Right now the way to find validator set changes in storage is that we use the version-to-epoch index to find the versions where an epoch change happened, then use the versions to find the corresponding ledger infos. These ledger info objects will have the validator set. No code uses this validator set column family right now.
Have you read the Contributing Guidelines on pull requests?
Y.
Test Plan
CI.
Related PRs
None.

@bors-libra r+

:pushpin: Commit ef181db has been approved by lightmark

:hourglass: Testing commit ef181db30c2643487298960de58cce90085cabf3 with merge 6c8a534b1f1ddd5eab28e6f2c3a738d8c74a623f...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit ef181db30c2643487298960de58cce90085cabf3 with merge 06b41d07622b635c2545da359de8b20926d584e2...

:broken_heart: Test failed - checks-circle_commit_workflow

@bors-libra retry

:hourglass: Testing commit ef181db30c2643487298960de58cce90085cabf3 with merge 16653b388ae7b5d1247c8d89fef74d554bbaaad7...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: lightmark
Pushing 16653b388ae7b5d1247c8d89fef74d554bbaaad7 to master...


Table of Assessments

User story
As an analyst,
I want to see a table of the assessments,
So that I can get an overview of the progress of the assessment
Acceptance Criteria

[x] Clicking on the 'Assessment' link in the left frame will now show the table of assessments

[ ] The order  of the "list of assessments type" is as is displayed in the wireframe if the screens exists.
Screening
GIS
Technical
Project Management
Financial Risk
Permitting

[x] Clicking on a row in the Assessment Table takes the user to the corresponding assessment tab

[x] If an assessment has no values filled, (i.e. the default, empty state) then in the table:
Progress = Not started
Analyst = Not assigned
Target date = <empty>
Decision = <empty>

[ ] If an assessment has an assignee in the dropdown AND does not have a radio button selected in the 'Progress' group
Progress = Assigned
Analyst = <analyst name>
Target date = <empty> (assuming it's not filled out)
Decision = <empty> (assuming none are selected)

[ ] If an assessment has a decision AND assessment progress = "Assessment complete" then:
Progress = Complete
Decision = <decision>

[ ] If "Progress" = "RFI" or "Needs 2nd review" then in the table
Progress = <progress>
Analyst = <analyst name>
Target date = <target date>
Decision = <decision>

[ ] display  Target date with the format [Mon, Jan 9]

Colors

Pill colors
https://www.figma.com/file/nhOquOVmWD1P3BxEY0KIh5/CCBC-Intake?node-id=8545%3A41032&t=hErmLZEzmRwCJ0fl-4

Wireframes
Figma
https://www.figma.com/file/nhOquOVmWD1P3BxEY0KIh5/CCBC-Intake?node-id=7039%3A32767
Progress pills

Outcome pills

Frontend Complexity:

Needs some extra work for the different pills based on different Assessment Type

Require creating or updating a component:

Yes

Back-end Work:

Will require function or computed column to return every kind of assessment

Changes in DB Structure:

No

Tests:

DB Test and Integration test

Impacts on DevOps:

None

Risks and Side effects:

None that I can think of

Documentation Update:

Also none that I can think of

Ultra Refinement :

[ ] #1061

DB:

Create a query to list all assessment form data in the DB for a specific application

Frontend:

Reuse: statusColors.ts for pill colours
New: Make the table for Assessments
Rows will have onClick to assessment - check figma for hover and cursor (should be the same as other dashboards)
Map existing assessments to expected set on the table

Assume there will be only one assessment record per type of assessments
Implement logic for pill and decision mapping

Tests:

pgTap tests for new tables and computed columns
e2e testing with happo screenshots
Unit tests for Table and assessments/index.tsx page (landing assessments page)
Design review on PR

Documentation:

For the new DB table we will need to update miro board

@elliottSlee @eazeredo  want to review the AC?

I'll add the tag needs refinement and get to it closer to the sprint 19

I'll add the tag needs refinement and get to it closer to the sprint 19

Acceptances Criteria updated.

Looks good!
This acceptance criteria doesn't behave as expected: If an assessment has an assignee in the dropdown AND does not have a radio button selected in the 'Progress' group
Progress = Assigned --> SHOWS AS 'Not Started'
Analyst =
Target date =  (assuming it's not filled out)
Decision =  (assuming none are selected)


Make dragPreview: null correspond to "empty image", fix it in IE

I noticed that on IE the hack with setting a transparent pixel image as dragPreview does not work for some reason.
We need to

fix it for IE (assuming it is possible at all);
make it easy to achieve the same without having consumer write getEmptyImage.

Maybe, adding an empty DOM node to the document and using it as dragPreview will work. I'm not sure.
It any case, I suggest we do this when dragPreview is explicitly set to be null rather than undefined. We'll just do whatever we need to hide it in all browsers including IE.

With 1.0, it might be a better idea to expose empty image from HTML5 backend.
And we need to look into possible hacks for IE.

Here's what we're doing in 1.0:

Expose getEmptyImage helper from HTML5 backend;
Expose and explain a workaround for making custom drag layers work in IE11.

Here's the relevant commit: https://github.com/gaearon/react-dnd/commit/de264ff2bcc666fa3a9d82543f311a78d2133791.
I added captureDraggingState to drag preview options. By default it's true and that's what you usually want, but you can pass false and then the browser will screenshot the drag source with isDragging already equal to true. This is our only chance to hide the drag preview on IE by hiding the node itself with CSS.
If you don't care about IE support or custom drag layers, nevermind what I just said ;-)
This is out in 1.0.0-alpha.1 so I'm closing out this issue. The final 1.0 release is coming soon.


Nice

all works finely

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

ID: 16b31202-eae5-59aa-3de4-7e1fb897ed4f
Version Independent ID: 9f595797-0c89-63d1-e8d8-9a67e2cc7e19
Content: Tutorial: Create calculated columns in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-tutorial-create-calculated-columns.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Great! Glad to hear it, thanks for posting.


Add page about selects

This PR adds a page that explains all types of select menus.
Preview: # testing > 💬 (message in the discord.py server) or: https://github.com/AbstractUmbra/abstractumbra.github.io/assets/33902984/59714760-c042-4590-a05e-315c02e04653
⚠ Depends on https://github.com/Rapptz/discord.py/pull/9577 for the section about default_values.

This PR cannot be merged until you rectify the divergent commits.
This repo auto updates every 12 hours via a workflow so committing between this window may cause issues.
Ideally just take out your file, reset to HEAD and add the file again.


cannot export client from UI

pressing the export button from the interface does not export the file, it does not return any errors even in the console.
Maybe it's a problem with javascript?
if you use the link the url bar, it works:

console/dev/realms/{realm}/apps/{client_id}

fix with b20e4ef


Question: Do we have cargo support ?

Hello,
I trying out the version from here https://marketplace.visualstudio.com/items?itemName=vosen.VisualRust
Wanted to know does this support cargo project structure?
2nd Question. I get this error when compiling the Visual rust code

Any help on how to resolve this? I am on master branch

About the first question: No, the version of VisualRust currently available at the marketplace does not yet support Cargo.
Second quesiton: I've never seen this error. Is Java installed on your system? Antlr requires it.

Thanks I forgot to reopen visual studio after reopening it that error went but I have now build failure

That Phx.FatalError looks like it's coming from FxCop (http://stackoverflow.com/questions/30490358/error-in-fxcop-phoenix-analysis-engine). Do you have a special configuration there? I've never seen an error like this.

Nice! That fixed it. Thanks for your help.


Nick: Watch singnutri.com

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

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

Approved by Makyen in Charcoal HQ


[Improve][Connector-V2] Change read excel util from POI to EasyExcel

#8040
Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

https://github.com/apache/seatunnel/runs/33188901598 @dwave Please open ci workflow

https://github.com/apache/seatunnel/runs/33188901598 @dwave Please open ci workflow

Okay, it's already opened

Thanks @dwave ! Currently only support change engine when read excel, I thing we should support it when write excel.

I participated in the attempt, but it will take a while for both surnames to be present

I participated in the attempt, but it will take a while for both surnames to be present

@dwave Thank you for your contribution,If you want to provide the engine selection for reading from the source first, you should update the documentation to remove the sink capability and modify all related documentation about Excel support. Thank you again for your contribution. The sink engine selection capability will be provided in future updates.:>


Remove legacy "lib/constants/colors.js"

lib/constants/colors.js was there before introducing styled-components with a theme to the codebase. We don't want to use it anymore, and it has an history of confusing contributors. We should:

[ ] Replace the occurrences with existing colors from the design system (props.theme.colors)
[ ] Remove the file

Hey, @Betree. I'm interested in taking this one on; seems like a quick entry into learning something about the codebase.

@ericyoungson Absolutely! Feel free to ask if you need anything.

Thanks, @Betree. I think I'm on the right track but I do have a question about tests. I eliminated the old colors in Location.js and the tests pass with npm run. I am attempting to run the full test suite but running into an issue building the api. Do I need to do the full test or are pages and components enough?

@ericyoungson running all tests locally is tedious, you can submit your PR directly and let the CI run them for you :wink:

Great! Thanks.

@Betree Looks like most of the remaining references are for darkgray. Seems to pass tests with color: props.theme.colors.darkgray but I'm trying to verify whether the color exists as the same value '#797D7F' in colors.js.
Line 13 in CollectiveThemeProvider.js says the colors are defined in collective.settings.collectivePage.primaryColor. I can't seem to find it, though.

@Betree Looks like most of the remaining references are for darkgray. Seems to pass tests with color: props.theme.colors.darkgray but I'm trying to verify whether the color exists as the same value '#797D7F' in colors.js.
Line 13 in CollectiveThemeProvider.js says the colors are defined in collective.settings.collectivePage.primaryColor. I can't seem to find it, though.

@Betree Looks like most of the remaining references are for darkgray. Seems to pass tests with color: props.theme.colors.darkgray but I'm trying to verify whether the color exists as the same value '#797D7F' in colors.js.
Line 13 in CollectiveThemeProvider.js says the colors are defined in collective.settings.collectivePage.primaryColor. I can't seem to find it, though.

What's defined in collective.settings.collectivePage.primaryColor is the primary color of the collective, which serves as a seed to generate the theme (if it's defined). Colors themselves are defined lib/theme/colors.js


Update os resources file.'

This change is

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

Comparison is base (fc59f33) 66.97% compared to head (4d87182) 67.61%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1142      +/-   ##
==========================================
+ Coverage   66.97%   67.61%   +0.64%
==========================================
  Files          51       52       +1
  Lines        6755     7044     +289
  Branches     6755     7044     +289
==========================================
+ Hits         4524     4763     +239
- Misses       1819     1872      +53
+ Partials      412      409       -3

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


Could not restore project backup

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce
Click on project
click on create from backup
upload zip file having size 40GBCreatethe
Expected Behavior
should restore the project and all annotations tasks
Possible Solution
should restore the project and all annotations tasks annotation
Context
Dear team I am trying to upload a backup from another server while doing so i am getting file size exceeds the max limit of 25GBPlease help, as I have annotated videos for the last 4 to 5 months and it is not working.Please help me solve this problem. It requires me to be ready to pay for support.
Environment
install using guide

Thank you very much @bsekachev
Although it now not showing a limit size error, it has been running for 1 hour.
How long does it normally take to upload a 40GB backup?

@bsekachev after running for several hours:
I am getting the following error:
Could not restore project backup: ValueError: No media data found

Please help, my 4-5-month-old work
is at stake, @bsekachev

@kailash-delvify
Have you tried to backup/restore tasks from the project individually?

@bsekachev
I am having trouble restoring a project backup as I'm encountering a media data entry error.
When attempting to restore individual tasks, I am also getting a "task.json not found" error, even though the file is present in the folder structure. Here's the layout:
├── data
│ └── {user uploaded data}
├── task.json
└── annotations.json

@kailash-delvify
It's possible to share the archive so we can reproduce the issue on our side?

@bsekachev , could you please share your email address? so i can share archive

@bsekachev , could you please share your email address? so i can share archive

<EMAIL_ADDRESS>

@bsekachev , I have sent an email with an attachment.

Without data we can't reproduce the problem, so I'm closing the issue for now.


fix: component arg parse

Summary
This is the problem https://github.com/lacework/go-sdk/blob/main/cli/cmd/component_args.go#L83-L86.
		// We're actually a bit stuck here as we don't know if this flag
		// takes an argument or not, so we don't know whether or not to consume
		// the next arg.  What we'll do is peek ahead, and if the next arg does
		// not start with - then we'll take it.

Cobra strips the surrounding double quotes and when the go-sdk sees a - prefix, it treats the argument as a 'shorthand' (e.g. -vvvv).
The Fix
Simple, we don't need to parse the flags.  Use -- to terminate flag parsing and pass them as arguments to the component, and let the component handle the parsing.
We benefit by having less code to maintain and instead lean upon the Cobra library.  We make use of the flag terminator --, like the K8 CLI.
How did you test this change?
Manually.  Create a component using lacework component dev test --scaffolding Golang --type CLI_COMMAND.  Add the following patch to the component code.
diff --git a/cmd/placeholder.go b/cmd/placeholder.go
index 10bc5ee..75e41ae 100755
--- a/cmd/placeholder.go
+++ b/cmd/placeholder.go
@@ -30,12 +30,24 @@ var (
                Long:  `This is just an example of a command.`,
                RunE: func(_ *cobra.Command, _ []string) (err error) {
                        fmt.Println("Your new component 'jontest' is ready!")
+
+            fmt.Println(TestState.time)
+
                        return nil
                },
        }
+    TestState = struct {
+        time string
+    }{}
 )

 func init() {
        // add the placeholder command
        rootCmd.AddCommand(placeholderCmd)
+
+        placeholderCmd.Flags().StringVar(
+        &TestState.time,
+        "time", "-t",
+        "start time for query",
+    )
 }

> bin/lacework --nocache test --json -- placeholder --time -24h
Your new component 'test' is ready!
-24h

Issue
https://lacework.atlassian.net/browse/GROW-2677

We are going to go with https://github.com/lacework/go-sdk/pull/1512


Request details in exceptions

I can examine uploader.request.response_content type to retrieve information from response in case of  tusclient.exceptions.TusUploadFailed. However, if creation of uploader fails from  tusclient.exceptions.TusCommunicationError I cannot determine why. Moreover, accessing properties of exception in case of an error is easier than looking into request property. I want to use:
class SomeUnitTest(unittest.TestCase):
    # ...
    def test_something(self):
        # ...
        try:
            tus_client = tusclient.client.TusClient(myurl)
            uploader = my_client.uploader(path, chunk_size=512*1024)
        except tusclient.exceptions.TusCommunicationError as e:
            self.assertEqual(e.response_content,  'MyErrorMessage')
            self.assertEqual(e.status_code,  400)

        # ...

        try:
            uploader.upload()
        except tusclient.exceptions.TusUploadFailed as e:
            self.assertEqual(e.response_content,  'MyErrorMessage')
            self.assertEqual(e.status_code,  400)

Thank you for the work! 👍


BREAKING CHANGE: update to eye v9.0.0

BREAKING CHANGE: update to eye v9.0.0

Cancelling as eye v9 has been deleted.


code-minimap: 0.5.0 -> 0.5.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/wfxr/code-minimap/releases.
meta.description for code-minimap is: "A high performance code minimap render"
meta.homepage for code-minimap is: "https://github.com/wfxr/code-minimap"
meta.changelog for code-minimap is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.5.1 with grep in /nix/store/nvx8yl6cd915nw0ij46llz1h11pmi4p7-code-minimap-0.5.1

directory tree listing: https://gist.github.com/a73c736aa4576b99a7a155b12acd5c91

du listing: https://gist.github.com/b869a8587545a7d883b69000f817878e

Rebuild report (if merged into master) (click to expand)

5 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
code-minimap
vimPlugins.minimap-vim

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nvx8yl6cd915nw0ij46llz1h11pmi4p7-code-minimap-0.5.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A code-minimap https://github.com/r-ryantm/nixpkgs/archive/0d4e75fcea645948aa543e4276bd4785fbca263c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nvx8yl6cd915nw0ij46llz1h11pmi4p7-code-minimap-0.5.1
ls -la /nix/store/nvx8yl6cd915nw0ij46llz1h11pmi4p7-code-minimap-0.5.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

code-minimap
vimPlugins.minimap-vim

Maintainer pings
cc @bsima for testing.

Looks good to me


added navbar 11

https://user-images.githubusercontent.com/115935687/198815367-4f927044-871e-43e6-9b6e-4135f7c8320b.png
link for the screenshot of the navbar
added the file called navbar-11

Good job @Hasika-11
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .
Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


Implement composable uid/gid generators

This cleans up the code a bit and provides the necessary uid/gid functionality for implementing user/groups for rkt fly, see #2439

@alban @tmrts PTAL

I just recognized ... now that I factored that jazz out I really should introduce unit tests ;-)

After the modifications I commented plus tests, LGTM

blocked by #2515 for providing functional tests for fly, this is too short notice, rescheduling for the next release.

TODO: backport fix from https://github.com/coreos/rkt/pull/2508

@tmrts @alban @iaguis PTAL

LGTM

Do you wanna squash the commits?

@iaguis squashed

I didn't go very deep reviewing this but LGTM on the surface.


update go sdk core version

:tada: This PR is included in version 2.0.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[immer] Add export CMake ConfigVersion file

Describe the pull request

What does your PR fix?
Fixes #28864

Isn't this a type of PR which should be presented to upstream first?

It is not a vcpkg bug. It is upstream which doesn't support find_package(immer VERSION 0.8.0).
Upstream must decide if immer 0.8.1 satisfies find_package(immer VERSION 0.7.5) or  find_package(immer VERSION 0.8.0)  or just find_package(immer VERSION 0.8.1).

The upstream PR https://github.com/arximboldi/immer/pull/250, I converted this PR to draft waiting for the response of upstream.

Thanks!


[#3864] feat(lakehouse-iceberg): add custom catalog name to  Iceberg catalog backend

What changes were proposed in this pull request?
add custom catalog name to Iceberg catalog backend
Why are the changes needed?
Fix: #3864 #3865
Does this PR introduce any user-facing change?
yes, add configuration document
How was this patch tested?

create a jdbc catalog with custom iceberg-catalog-name
check it takes effects in MySQL
spark connector could successfully operate under the jdbc catalog.

Why do you need to backport to branch-0.5?

Why do you need to backport to branch-0.5?

because there is a bug in #3865,  it's also about the iceberg catalog name,  spark connector could operate on Gravitino Iceberg catalog with JDBC backend.

I don't think there will be a 0.5.2 release anymore.

I don't think there will be a 0.5.2 release anymore.

There are known critical bugs in 0.5.1, why not release a bug-fix version?

Sure, you can create a release if you really want.

Sure, you can create a release if you really want.

ok, I think it's necessary and I'd like to do the release

@jerryshao please help to review this pr again when you have time


TypeError: urlparts is null when endpoint is set to http://<IP_ADDRESS>:8000

When endpoint is set to "http://<IP_ADDRESS>:8000", getStatements() appends "statements" to it, so URL becomes "http://<IP_ADDRESS>:8000statements" and this regexp doesn't match.

is the endpoint supposed to be "http://<IP_ADDRESS>:8000/statements? or where can you get your statements from for your lrs?

The correct endpoint is "http://<IP_ADDRESS>:8000/xapi", I used the wrong URL in https://github.com/adlnet/xapi-statement-viewer and had this TypeError again and again on page reload until I manually removed saved config from localStorage.

It looks like the readme already outlines proper LRS endpoint syntax:
conf['endpoint'] = "http://localhost:8000/xapi/";

Even if it didn't, I've seen some wrappers require the / and others omit it.  /xapi/statements is the correct path for retrieving xAPI statements though, is there an issue here I'm not seeing?


Home Assistant: Allow swiping to switch triggers, standardize spellings

Allow swiping to switch triggers
Standardize spellings

Neat - thanks!


HUB-143: Add description to service sign in pages

Add a description paragraph that will be displayed under the title on each of the 4 services using the service sign in pattern, each description includes the service specific text. This "description" is already present in the example yaml, the schema and on the page template on government-frontend but was hitherto unused. Should this prove to be a good pattern according to the base line test we will add a description to the welsh pages as well.

solo: @rachelthecodesmith

Can you add a link to the card please?


OPS-1466 - Adds config to add the ganglia port to the security group

https://datatamr.atlassian.net/browse/OPS-1466
Ganglia access is a requirement for this OPS ticket. Put it behind a flag since some customers are sensitive about non-SSL traffic, so they can follow the guides to access via ssh tunnel:
https://docs.aws.amazon.com/emr/latest/ReleaseGuide/view_Ganglia.html
Relevant config to set to make the web ui accessible:
applications          = ["Spark", "Ganglia"]
enable_http_port      = true

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update block gas target formatting

On the London screen, show the percentage of the gas usage relative to the gas target, like "18,450,975 (+23.01%)" and remove the gas target column.
Use two decimal places when showing the basefee.

will merge this one, although I've hid the london link from homepage, I think we'll probably remove this page in future.


fix: Remove validation of super in use paths

Fixes #10770

This still allows e.g. use std::io::copy::super, but I don't think we can spot that copy is a function and not a module at this level.

This still allows e.g. use std::io::copy::super, but I don't think we can spot that copy is a function and not a module at this level.


Added logo

Pull Request
Proposed Changes

Added Elevate Lab logo in two themes dark and light and in both svg and png formats.

Types of changes made

[ ] Fixed Bug
[ ] Added new Features
[x] Edited Documentations
[ ] Enhanced the working of code
[ ] Additional Changes

@all-contributors add @stygian-96 for Design


sgt-puzzles: bump to version 20210409

Couple of notes:

Simon switched to cmake
There are now .desktop files in the manifest!

You'll want to add cmake make to depends.
I have no idea what the source tree looks like, but icons generally go in /usr/share/icons - if you'd like to do it. I'm not sure how much it matters.

Added cmake depends.  Forgot about that.

sed -i 's/cmake depends/cmake make/' sgt-puzzles/depends ;)

oops, I was in a hurry to play slay the spire and wasn't paying attention.  I'll fix it when I get back to my kisslinux laptop.


TensorFlow with CUDA broken as of c3255fe8

Issue description
Trying to install tensorflowWithCuda fails with the message below. This is broken on latest nixpkgs master, and git bisect pointed me to c3255fe8ec326d2c8fe9462d49ed83aa64d3e68f as the offending commit.
____Loading package: tensorflow/tools/pip_package
____Loading package: @local_config_cuda//crosstool
____Loading package: @local_config_xcode//
____Loading package: @local_jdk//
____Loading package: @bazel_tools//tools/cpp
____Loading package: @local_config_cc//
____Loading complete.  Analyzing...
____Loading package: tensorflow/contrib/cluster_resolver
____Loading package: tensorflow/core
____Loading package: @nccl_archive//
____Loading package: tensorflow/python/saved_model
____Loading package: @gemmlowp//
____Loading package: @grpc//
ERROR: /tmp/nix-build-python3.5-tensorflow-1.3.1.drv-0/source/tensorflow/tools/pip_package/BUILD:100:1: no such package '@farmhash_archive//': BUILD file not found on package path and referenced by '//tensorflow/tools/pip_package:licenses'.
ERROR: Analysis of target '//tensorflow/tools/pip_package:build_pip_package' failed; build aborted.
____Elapsed time: 16.221s
builder for ‘/nix/store/aj3dnip64sinvm0pkgncjjjaklq4zlip-python3.5-tensorflow-1.3.1.drv’ failed with exit code 1
error: build of ‘/nix/store/aj3dnip64sinvm0pkgncjjjaklq4zlip-python3.5-tensorflow-1.3.1.drv’ failed
/run/current-system/sw/bin/nix-shell: failed to build all dependencies

Steps to reproduce
Execute:
nix-shell -p pkgs.python35Packages.tensorflowWithCuda

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 4.9.56, NixOS, 17.09.2055.c19136c4c9 (Hummingbird)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 1.11.15`
 - channels(root): `"nixos-17.09.2055.c19136c4c9, nixpkgs-17.09pre109651.6916f24e84"`
 - channels(hodapp): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs`

Closing. I saw that https://github.com/NixOS/nixpkgs/issues/31492 is also open and this looks like a duplicate.


Fix crypto/bn/bn_prime.pl and apps/progs.pl to generate correctly formatted source (1.0.2)

It was still generating EAY style source.
This is an alternative to #2888

Yes, I like this approach as well.
Could you also update the progs.pl from trunk ?

Uhmm, why does progs.pl need an update?

If I touch apps/progs.pl and build everything, then apps/progs.h looks also totally different.
So I guess that would not happen on master...

I see what you mean.  I've added a commit for that one as well.
@richsalz, could you reconfirm, please?

reconfirm

Merged.  f4675a10f8f42d7e1505ac98e85d2350123e6666 to acdf0814234c8a20bcbdab7f1cf0b4d452adb415


Enhance the instrumentation for a corner case where the query doesn't go through DocIdSetOp

Codecov Report

Merging #10729 (c9795ad) into master (c47e7e5) will decrease coverage by 54.71%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #10729       +/-   ##
=============================================
- Coverage     68.53%   13.82%   -54.71%
+ Complexity     6452      439     -6013
=============================================
  Files          2122     2068       -54
  Lines        114285   111803     -2482
  Branches      17253    16955      -298
=============================================
- Hits          78322    15458    -62864
- Misses        30401    95082    +64681
+ Partials       5562     1263     -4299

Flag
Coverage Δ

integration1
?

unittests1
?

unittests2
13.82% <0.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

...perator/query/DictionaryBasedDistinctOperator.java
0.00% <0.00%> (-76.09%)
:arrow_down:

... and 1635 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Rename the module and assumed file path.

Changes the module name to "GoogleCloud" from "GoogleCloudPowerShell". The
trailing "PowerShell" is redundant. (Although AWS' cmdlets are called
"AWSPowerShell".)
Changes the assumed location of the cmdlets.
Changes references to the product as a whole to "Cloud Tools for PowerShell", since "Google Cloud PowerShell" was the old name while we were still in alpha.
Previously the Cloud SDK installer
would the files in "${Cloud SDK}\platform\GoogleCloudPowerShell" and add
"${Cloud SDK}\platform\GoogleCloudPowerShell" to PSModulePath.
While this allowed PowerShell to quickly find GoogleCloudPowerShell.psd1,
it broke the ability to type Import-Module GoogleCloudPowerShell and do other
sorts of lookups.
Internal cl/133881461 will update the Cloud SDK to use a new folder for
PSModulePath: "${Cloud SDK}\platform\PowerShell". And the PowerShell
bits will be installed into "${Cloud SDK}\platform\PowerShell\GoogleCloud".
Testing:

Manual testing that PowerShell behaves the way Microsoft tells us it does.
Manually built the new installer, ran it, and manually replaced the "powershell"
component bits ($INSTALLDIR/platform/) with the new bits.
I did not test the PatchFromGcs.ps1 or AppendModulePath.ps1 scripts.

Fixes #314 and #315.

PTAL

LGTM


Support any separator

This PR makes gomailer support any type of separator in its user input file;
The first element is treated as the e-mail and captured with a regular expression, whatever is not the e-mail or the separator is then considered the user name;
Example of possible user input file:
<EMAIL_ADDRESS>Surname One,Name One
<EMAIL_ADDRESS>Surname Two;Name Two
gomailer@mailinator.org,Surname Three, Name Three
gomailer@mailinator.dm.nl;Surname Four Name Four
<EMAIL_ADDRESS>Five-Name Five

Will be parsed as
2022/09/15 12:20:01 Starting mailer
2022/09/15 12:20:01 Looking for recipients.tsv file...
2022/09/15 12:20:01 Found 5 users
2022/09/15 12:20:01 > Name: Surname One,Name One E-mail<EMAIL_ADDRESS>2022/09/15 12:20:01 > Name: Surname Two;Name Two E-mail<EMAIL_ADDRESS>2022/09/15 12:20:01 > Name: Surname Three, Name Three E-mail<EMAIL_ADDRESS>2022/09/15 12:20:01 > Name: Surname Four Name Four E-mail<EMAIL_ADDRESS>2022/09/15 12:20:01 > Name: Surname Five-Name Five E-mail<EMAIL_ADDRESS># ...

Thanks for the suggestion @amjjbonvin 🚀


Schemas management failed when schemas are outside execution context

When topology-builder runs in docker context, schema management feature failed if schemas are outside of the docker execution context.

java -Dlog4j.configuration=file:/var/lib/jenkins/workspace/myProject/tmp/log4j.properties -cp /var/lib/jenkins/workspace/myProject/dev/topology/configuration.yaml /var/lib/jenkins/workspace/myProject/dev/topology/test-value.avsc -jar /usr/local/kafka-topology-builder/bin/kafka-topology-builder.jar --clientConfig /var/lib/jenkins/workspace/myProject/dev/kafka-admin.properties --topology /var/lib/jenkins/workspace/myProject/dev/topology/configuration.yaml --brokers kafka-dev.mydomain.com:9094
Error: Could not find or load main class .var.lib.jenkins.workspace.myProject.dev.topology.test-value.avsc

Let me know if you want more information

Hi @szanlucchi thanks a lot for your report, this was for sure a bug present before the commit in master of f51e14d047d9f05a215df49b31b969e67fb8aef5, so is present in all currently released version till now.
I hope to release a new set of artifacts soon, they will contain this commit and per se the fix.
For now, I updated the latest version in docker hub and you could as well build the jar from the master branch, I know is not much, but this will contain the fix necessary for this.
Can you check with either of this two options? If you need a new artifact I'll try to release one in the next couple of days, sadly very busy with work currently :-(.
-- Pere

closing this for now as f51e14d been already merged. the code has been merge to master, if the problem persist please feel free to reopen this bug.


Unexpected results for sunspot/sunspot

Is there any way to support a repo like this where the there are actually three separate gems in subfolders?

Not at the moment, no. :-1:
I don't currently have any plans to support this kind of structure as it isn't incredibly common, but it has come up a couple times. My concern is keeping the management UI as simple as possible and I don't have any good ideas for simple ways to make this happen yet. Please let me know if you have any ideas but in the meantime, I'm going to close the issue. Feel free to reopen similar issues as they occur. I'm interested in the frequency of this. Thanks!


[Windows]locales error: Path starts with "" is preserved by the browser.

npm run dev fails on win11, what should i do to fix it?
win11
npm run dev error
> npm run dev

><EMAIL_ADDRESS>dev
> extension dev

✖✖✖ my-new-extension-32 compiled with errors in 1193 ms.
ERROR in [webpack-extension-target]
Path starts with "_" is preserved by the browser.
The browser will refuse to load this extension.
Please adjust your webpack configuration to remove that.
File(s) starts with "_":
    _locales\en\messages.json
    _locales\zh_CN\messages.json

but it works with npm run build ok
> npm run build

><EMAIL_ADDRESS>build
> extension build

►►► my-new-extension-32 compiled successfully in 249 ms.
►►► Building my-new-extension-32 extension using Chrome defaults...

.
├─ _locales\zh_CN\messages.json (2.16KB)
├─ _locales\en\messages.json (2.15KB)
└─ manifest.json (0.19KB)

Version: 1.0
Size: 4.50KB
Build Target: Chrome
Build Status: Success

Build completed in 0.25 seconds.
►►► No errors or warnings found. Your extension is ready for deployment.

ubuntu
npm run dev ok
> npm run dev

><EMAIL_ADDRESS>dev
> extension dev

►►► my-new-extension-32 compiled successfully in 7876 ms.

npm run build ok
> npm run build

npm run build

><EMAIL_ADDRESS>build
> extension build

►►► my-new-extension-32 compiled successfully in 913 ms.
►►► Building my-new-extension-32 extension using Chrome defaults...

.
├─ _locales
│  ├─ zh_CN
│  │  └─ messages.json (2.16KB)
│  └─ en
│     └─ messages.json (2.15KB)
└─ manifest.json (0.19KB)

Version: 1.0
Size: 4.50KB
Build Target: Chrome
Build Status: Success

Build completed in 0.91 seconds.
►►► No errors or warnings found. Your extension is ready for deployment.

hi @xesam could you share the folder structure you are using/sample extension so I can try to replicate locally? I did some changes in the locales field, seems worth adding more tests to it

this is a message from an upstream dependency, so unless you do not have a folder starting with _ the message is expected. it works in production because the plugin works in dev mode only.

hi @xesam could you share the folder structure you are using/sample extension so I can try to replicate locally? I did some changes in the locales field, seems worth adding more tests to it

sorry for the late reply, i put a new sample in this tmp repo: https://github.com/xesam/extension_path_test_20240923. hope this helps.


Configuration file for netcdf2vic.py

There appears to be a mismatched between the configuration file (example) and the code. It gives a key error ("Options" not found).

@amandalehr - thanks for the report.  @mbaptiste also ran into this recently and has a few code changes that fix the problem.  I'm asking her to share those :wink:

@amandalehr - take a look here and let us know if these changes fix your problem: #33.
It looks like #33 could be cleaned up a bit and that netcdf2vic could be generalized a bit further. I don't use this script anymore so any contribution you can make would be welcomed.


DO NOT SUBMIT: testing proxy sha 7792fbdd71cc2c070a3080c4a279b61378be…

…9715

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: qiwzhang
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: hklai
Assign the PR to them by writing /assign @hklai in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@qiwzhang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@qiwzhang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests-v1alpha3.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@qiwzhang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests-v1alpha3.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test e2e-bookInfo

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
8087e1df2251136c21e450176e3c9d47015a5244
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Simplify public API

This PR is related to #59.
So here, I refactor the names of the modules to simple ones.
In that journey, I noticed that we've completely forgot to check some files while working on refactoring the toolbox to accept both Chamber of Deputies and Federal Senate.

As we see in the image, it still keeps the format from before Federal Senate introduction  👀
But I don't know if we just wait for the #87 to fix it, or something.
Help will be accepted <3

Ok, I think that I find a way to update the sphinx documentation.
What I've done was:
make rst into the /docs folder
So, I am not sure if it was the right way, I'll be waiting for reviews :)

Ok, I guess that now everything is right!

Ok, I guess that now everything is right!

Running integration tests and merge it as soon as it finishes ; )


docs(agent): Regenerate PNG images from updated PUML files

Description:
ATL-7050: Regenerate PNG images from updated PUML files
Checklist:

[x] My PR follows the contribution guidelines of this project
[x] My PR is free of third-party dependencies that don't comply with the Allowlist
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked the PR title to follow the conventional commit specification

Thank you, @bvoiturier, for picking it up. What do you think about moving these puml diagrams to the markdown code with a mermaid format? I haven't tried this, but it would be easier for us to manage these pics later.

@yshyn-iohk Yep, that would be great. It seems Mermaid is the only format both Github and Docusaurus can natively render. I'll add a tech debt ticket for this 👍


On remove of "single-page-app" and "static-website": S3 Bucket deletion is triggered, when the bucket is not yet empty

Description
Recently an error startet to appear, which was not there before: whenever a Stack deletion is triggered through serverless remove the lift still tries to empty the S3 bucket of the single-page-app or static-website construct
But for some reason doesn't wait until the cleanup is finished and triggers the removal of a Bucket itself, which results in an error like:
Resource handler returned message: "The bucket you tried to delete is not empty (Service: S3, Status Code: 409, Request ID: -, Extended Request ID: -)"
The whole output (omitting the hashes and project info):
npx serverless remove --stage=poc

Removing fe-service from stage poc (eu-central-1)
Emptying S3 bucket 'fe-serv-buildpublicbucket' for the 'build_public' static website, else CloudFormation will fail (it cannot delete a non-empty bucket)
Environment: darwin, node 20.11.0, framework 3.38.0 (local), plugin 7.2.0, SDK 4.5.1
Credentials: Local, "default" profile
Docs:        docs.serverless.com
Support:     forum.serverless.com
Bugs:        github.com/serverless/serverless/issues

Error:
DELETE_FAILED: buildpublicBucket (AWS::S3::Bucket)
Resource handler returned message: "The bucket you tried to delete is not empty (Service: S3, Status Code: 409, Request ID:-, Extended Request ID: -)" (RequestToken: -, HandlerErrorCode: GeneralServiceException)

Serverless Framework: 3.38.0
Lift: 1.28.1
Triggering the same remove command second or third time will empty the bucket and remove the stack completely.
Removing the files from the bucket using AWS Console works without issues.
Average number of files in the S3 Bucket: 1175 objects, 63.1 MB
How to Reproduce
Not sure, what the exact scenario is, but I assume, that you need a project with 1k+ files, deployed either as "single-page-app" or as "static-website".
Additional Information
No response

Fixed in #400


[RFC] #[uninit] for truly uninitialized static variables

Summary
The #[uninit] attribute will place static [mut] variables into an .uninit
section, located in RAM, that won't be initialized before main.
Motivation
Today, uninitialized (see MaybeUninit) static variables will be placed in
.bss, which means that'll be zeroed before main. This leads to unnecessary
work as leaving the memory uninitialized was the intended behavior.
Design
The #[uninit] attribute will have the following syntax.
#[uninit]
static mut FOO: u32 = {};

The initial value of uninit variables must be the placeholder {}. This
attribute will expand into:
#[link_section = ".uninit"]
static mut FOO: core::mem::MaybeUninit<u32> = core::mem::MaybeUninit::new();

#[uninit] composes with #[entry] and #[exception]; it can be used on
safe local static mut variables.
#[entry]
fn main() -> ! {
    #[uninit]
    static mut KEY: [u32; 8] = {};

    // ..

    // the name of this MaybeUninit method hasn't been decided but it writes
    // a value into the MaybeUninit and then returns a reference to the written
    // value
    let key: &'static mut [u32; 8] = KEY.insert(/* runtime value */);

    // ..
}

Implementation
Before this can be implemented, MaybeUninit must first land in the core
library. This feature will live behind a "nightly" (Cargo) feature until
MaybeUninit and its const constructor are stabilized.
Alternative syntax
Option A
#[uninit]
static mut FOO: u32 = ();

Option B
#[uninit]
static mut FOO: u32 = ..;

We can't omit the RHS of static mut because then the code won't parse and the
compiler will never reach the macro expansion phase.
Drawbacks
This is not a perfect solution.
For example, a heapless vector contains an uninitialized buffer when it's
constructed using the new method but it also contains a length field that must
be initialized to zero. Applying #[uninit] to such data structure means that
the vector will have to be initialized (assign 0 to its length field) at
runtime, which is not ergonomic. Not using #[uninit] means that the vector
length and its buffer will be zeroed, which is wasteful.
It's not possible to have partial initialization of static variables so this
is as good as it gets.

I guess I'm partial on this one as written down. On one hand it sounds like a useful feature to have, even if it just improves the ergonomics of static variables which is quite a boon. OTOH I don't buy into the expense argument of the zeroing; if wasting a few cycles once during initialisation is the biggest problem of your design you can be very proud of yourself... I'd rather highlight the ergonomics benefit than the slight performance improvement and by doing that your drawback also vanishes into thin air because it's actually a pro that you cannot abuse this to do partial initialisation.
Regarding syntax, I'd very much prefer the unit type:
static mut FOO: u32 = ();
over the other variants but I think I would like the never type even more (if that is a possibility!):
static mut FOO: u32 = !;

+1 on @therealprof for me. The zeroing does not convince me, though ergonomics is indeed a good plus.

Was this RFC accepted or not? Churning out the implementation for this should go fast.

I needed to brush up my proc macro knowledge, so I made a simple impl to do the expansion and type signature checking: https://github.com/rust-embedded/cortex-m-rt/tree/uninit_impl

I'm not terribly excited about having yet another way in which the user-specified type gets changed to something else by a macro. IMO we should require an #[uninit] static to have type MaybeUninit<T>.
Also, the MaybeUninit::write API is still unstable, so at the moment there is no ergonomics win from this. In fact, it would be impossible to use this API from fully safe code.


Script Buttons Should Not Be Added If Their Code Cannot Compile

I noticed that when I open an outline, I see buttons that I've defined in myLeoSettings.leo.  But they don't work since they contain Python code that works with Leo.  Also, the right-click goto Script does not work with them.
These buttons should not be shown.

Thanks for this. I'll look into it.
The 'goto script' that does not lead to the nodes is weird... Does it also do that for newly created buttons? (with the '+ script-button' button?) Or just with buttons created upon loading the file?

I don't know about buttons created by the "script-button" button.  I know that the goto script capability works for buttons created by @button nodes.

@tbpassin
I'm not sure I understand this issue. I'm not aware of many of Leo's possible usage, so this might be one of those instance where someone uses Leo in a way that I'm totally unaware of... and so, I don't see what is happening from such a short issue description.
Can you restate what the issue is in a much more verbose way, and provide more context, screenshots and sample file, or sample code?

@tbpassin Oh! I see what you mean: buttons in myLeoSettings file can also be child of an at buttons (plural) to be seen in all opened files. and the goto-script should open it and focus on the button.
Thanks again for this issue! :)

Yes, just so.

@tbpassin The part about goto-script for script originating in @buttons (plural) in myLeosettings is now fixed in the latest beta version  0.2.10 😄
Also, the part about not running if the code is python is being fixed in #74 (branch 'boltex/issue/74 )
I'll close this issue when #74 is released in the next version (likely 0.2.11)

@tbpassin I'll leave this opened, as you requested to try to 'compile', instead of just detecting the @language in effect. (which I added in latest 'dev' commits of #74)


Make virtctl helpers unexported and move them to testing package

What this PR does
This introduces the pkg/virtctl/testing package that provides helpers to
run virtctl in unit tests. The unit tests of virtctl are updated to use
this package instead of tests/clientcmd to avoid importing test code in
production code. Places that used clientcmd.NewVirtctlCommand before were
updated to use testing.NewRepeatableVirtctlCommand instead.
Move the helpers to run virtctl in functional tests from
tests/clientcmd/virtctl.go to tests/virtctl/cmd.go and update the
functional tests of virtctl to make use of the unexported helpers. This
prevents importing these helpers in other places where they should not
be used.
This also addresses a small bug in newVirtctlCommand, that was checking
the Value interface instead of the flag returned by flag.Lookup for nil.
Ultimately, this achieves a clear separation of production and test code in virtctl by avoiding imports of test code into the virtctl production code.
Before this PR:
virtctl production code imports test code.
After this PR:
virtctl production code and test code are clearly separated from each other.

Fixes #12891
The helpers in unit tests no longer pick up the kubeconfig and server flags. This should fix the issue mentioned in #12891.
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer
Merge after #13378 and #13384
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

None

/retest-required

/retest-required

/lgtm

/test pull-kubevirt-e2e-k8s-1.31-sig-compute-migrations


H2O performance comparison has major flaws

Hi, I saw some of the benchmarks blogged about here from a recent Strata presentation slidedeck.
There are major flaws in your benchmarking of H2O:
The point of using H2O's Sparkling Water (and rsparkling if you are using R) is to interact with data already in the Spark cluster.  When you have data on disk, then you should be using the h2o.importFile() function (to do a parallel read from disk into the H2O cluster) and the h2o package for modeling.  There is no need to use rsparkling at all.
Loading to disk into Spark, then from Spark into H2O is an unnecessary task and doing so misrepresents the computational efficiency of H2O as compared to the other tools in this benchmark.  In the interest of honest & accurate benchmarking practices, it would be great if you could revise the benchmark to reflect this.  If you have any questions on how to do this, please let me know.
All you need to do is load the data from disk using h2o.importFile() and then execute these rows
of the benchmark.  You can also compute performance directly in H2O using h2o.performace() rather than generating predicted values using h2o.predict(), however there is nothing wrong with generating the predictions and calculating performance metrics manually, it's just faster if you use H2O's h2o.performance() function.  To most efficiently write the predictions back to disk, you should be using the h2o.exportFile() function.

Thanks for the feedback. We have updated the presentation deck (slide #61) to acknowledge that loading data using h2o.importFile() from the h2o package is likely a more efficient that the approach provided in the slide for h2o.


"Failed to send - disconnected from chat server. Retry"

MacBook Air, MacOS 10.13.2.
Using Keybase Version 1.0.38-20171220205307+f5d54bc77 (1.0.38-20171220205307+f5d54bc77)
I cannot chat, nor create a chat channel.
Keybase on my iPad, conversely, works without drama.
Any advice would be appreciated. I am developing a preference for Keybase and chat.
Thank you!

Same behavior occurs on Windows..
I quit app and restart Keybase. Fixed it.


Online Survey and Poll Creator

An extension for creating surveys and polls:
Customizable survey templates.
Real-time response tracking.
Data analysis and reporting.
Integration with data visualization tools.

Pull request received since long time. Closing this


Enable flicky scrolling on iOS Safari

This needs extensive testing for the different kinds of headers layout has.
This will not make V1.

I noticed this when browsing the docs on my iPad. Unfortunately it makes the whole experience rather painful on iOS.

+1

I'll get this manually tested on some iOS devices today. Believe @sgomes might have some time to help us with testing this patch so it lands today.

The current patch fixes the scrolling behavior for most templates except article. Also, the microsite has momentum scrolling on the components page only (and even there the sticky nav goes all kinds of crazy). We need to dig deeper here.

The components page is somewhat custom, so I'm not surprised it doesn't work out of the box. I'll give you a hand in figuring out what's happening with that and the article template.

I think I make it work.
All templates and the microsite behave correctly in both the iOS emulator and on the physical iPhone.
@addyosmani @sgomes Please look at the branch staged here: http://storage.googleapis.com/mdl-gold-surma/index.html and ACK.

No issues with desktop Safari, LGTM.

Tested on iOS8/touch. Changes lgtm.

please help me, how to install kernel missing


Smash and graaagh

Add a preference and tracking for the Zombie Slayer skill Smash & Graaagh

imo i would treat it as limited because there's no benefit to using this skill after the pickpockets stop working

Thanks for this! Merging


Is it possible to read 360 (not 720) images taken by cardboard APP

Is it possible to read 360 (not 720) images taken by cardboard APP
It look like only both mono and stereo Spherical panorama(not cubic format) images are supported for new , is 360 (horizontal  only panorama) planed to support ?

I'd also like to see this integrated as a query string parameter, so that standard Cardboard Camera photos can be displayed correctly.

What's the format of the Cardboard Camera photos? Are they equirectangular?

Yes. Also see this Cardboard Camera => over-under converter, which we recently released: http://storage.googleapis.com/cardboard-camera-converter/index.html

Awesome!

There is an issue with the convertor though - the resulting image on all my
conversions ends with a black space on the right of the top image,
compressing the top image, and making the stereoscopy a little useless.
I was delighted when it was released, but this issue hasn't been fixed (and
very few channels to provide feedback on it).
On Mon, 13 Jun 2016 5:03 am Boris Smus<EMAIL_ADDRESS>wrote:

Yes. Also see this Cardboard Camera => over-under converter, which we
recently released:
http://storage.googleapis.com/cardboard-camera-converter/index.html
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/vrview/issues/2#issuecomment-225448187, or mute
the thread
https://github.com/notifications/unsubscribe/ABlVyWQyGy4dCBgbMpPaHFcj_ObJAPsBks5qLDvsgaJpZM4IG9-3
.

Please attach an example of the cardboard camera input and converter output
to this thread.
On Sun, Jun 12, 2016 at 12:26 PM Geoff Bentley<EMAIL_ADDRESS>wrote:

There is an issue with the convertor though - the resulting image on all my
conversions ends with a black space on the right of the top image,
compressing the top image, and making the stereoscopy a little useless.
I was delighted when it was released, but this issue hasn't been fixed (and
very few channels to provide feedback on it).
On Mon, 13 Jun 2016 5:03 am Boris Smus<EMAIL_ADDRESS>wrote:

Yes. Also see this Cardboard Camera => over-under converter, which we
recently released:
http://storage.googleapis.com/cardboard-camera-converter/index.html
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/vrview/issues/2#issuecomment-225448187, or
mute
the thread
<
https://github.com/notifications/unsubscribe/ABlVyWQyGy4dCBgbMpPaHFcj_ObJAPsBks5qLDvsgaJpZM4IG9-3
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/vrview/issues/2#issuecomment-225455426, or mute
the thread
https://github.com/notifications/unsubscribe/AAQ8gM4s-9AcLLWHJBpN8l20QRggImJaks5qLF1TgaJpZM4IG9-3
.

Any change this could be opensourced @borismus? The OdsConverter looks like it would be really handy

Images below.  What's really strange is that the convertor gives back a PNG (check the file format), but calls it a JPG.

@marklundin There is a JS-based convertor you can use at https://fiddle.jshell.net/43rs5bha/50/embedded/result/?2 - it doesn't blur it, but it works.  Also see http://cctoolkit.vectorcult.com/

Ok, if there are issues with the cardboard camera converter, please file a new issue.


/nflgame is down

Hi,
It looks like http://pdoc.burntsushi.net/nflgame is currently down. Are you no longer supporting this?
Rathan

duplicate https://github.com/BurntSushi/nflgame/issues/359


Add check after cred def created in integration test

Signed-off-by: Ian Costanzo<EMAIL_ADDRESS>Adds small delay plus check that cred def was created (to address random failure of integration tests)

Codecov Report

Merging #1131 (8980f79) into main (1678458) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1131   +/-   ##
=======================================
  Coverage   98.70%   98.70%
=======================================
  Files         430      430
  Lines       24195    24195
=======================================
  Hits        23881    23881
  Misses        314      314


Spelling mistakes: buttom -> bottom

https://github.com/ProseMirror/prosemirror-tables/blob/89b87d8155ee445cad15381640af964c82488d13/index.d.ts#L297
Spelling mistakes: buttom -> bottom

Already fixed.


[Bug] Cannot setup mouse [object Event]

Device
Phone/Tablet
Device name
Samsung Galaxy S23
Android Version
14
App version
5.0.1
Script version
latest
WebView name and version
No response
Reproduction Steps
I'm testing this functionality in normal mode and in Samsung DEX:
Enable Keyboard and Mouse setting
Start the game
A short notification appears on top: "Cannot setup mouse [object Event]"
Mouse is not working

Does it work without DEX Mode?

Does it work without DEX Mode?

No, it's not.
Btw, I tried to connect my mouse via BT and via usb dongle as well.

Do you know how to get logcat of the device?

Do you know how to get logcat of the device?

Yes.

then provide it

bxc-240722-1814.txt

Did you found anything? Should I send more logs?

I didn't see anything unusual. Maybe the problem was the WebView.
Try using Thorium or Mulch: https://better-xcloud.github.io/guide/android-webview-implementation/

I didn't see anything unusual. Maybe the problem was the WebView.
Try using Thorium or Mulch: https://better-xcloud.github.io/guide/android-webview-implementation/

I tried with all the alternatives, it is the same.
Btw, this is the message:

Test again in 0.20.1

Test again in 0.20.1

it is the same :(

Same here. Tab s7

Same here with Galaxy Tab S9.

Apparently everyone who is reporting this problem has Samsung with Android 14

I've tried all versions of WebView, both the ones available within the app itself and those on the Play Store, but without success.

im using Mi 8 Pro (Android 13) but i'll try other option to fix it but unsuccess. I think, this is issue in Android 13 & higher? because i'll read other comment here, the mouse working fine in android 10 but im not sure if the android version is the problem

Working on Tab S9 now, but the mouse wheel do not work


slow down the speed

Why so fast run inference on a test video file, how slow down? I would be very grateful if you could tell me.

It could be that OpenCV's video capture is processing the frames faster (~50 when measured) than the original video fps (which is 30) or it may also depend on your system hardware. One nice fix for you is to measure the original fps of the video and supply it/adjust the frame rate during the live feed accordingly, or by introducing a delay in the feed
E.g., a quick way to test this is to introduce a delay to compensate for the faster fps:
Try changing key = cv2.waitKey(1) to key = cv2.waitKey(15) to add delay/slow down the fps
https://github.com/saimj7/People-Counting-in-Real-Time/blob/master/people_counter.py#L331

It wroks. Thank you so much.


Feature request - sub-calendar showing only selected categories

So I'm just curious to hear other user's thoughts on this.  I was asked the question so I want to pass along my thoughts and possibly help out if others agree.
I religiously use the master/sub calendar feature.  We have a multi-tenant setup on one very large website. The site is about the international main company, the company is divided up into regions, and within the regions there are local locations.
We have things setup where placing an event on the local location writes up to the region it belongs to.  Then those write up to the main calendar.  The local locations like to include everything on their calendars, a lot of them are recurring events.  All of these recurring events will be grouped into a specific category.
What I would like to be able to do is to restrict what "writes up" based on the category.  This way we don't have 300 locations all putting recurring (not important) events on the main master calendar.  But other events that they will put on their calendar need to be shown on the master calendar.
I'm curious to hear if anyone else would be able to benefit from this, or how the feature would work.  I'm just thinking about under the sub-calendar area, have the categories there with check boxes.  Then you could check off what categories you want to include.
I know how things usually work around here so I'm not asking anyone to do the work for me (unless you are just interested in it).  I'm curious to hear if I would pursue this if it would be allowed and included if I had this built into the module.  What I don't want is to have our own flavor of DNN Events module that we will constantly have to create a custom build for each time you come out with new features and updates.
Thanks in advance

Thank you for your well written and argumented feature request. I am curious is this is a scenario that more users come across. It looks like that a change like this could be implemented without functional impact on users who do not need it, which is good.
As for changes, I am very much interested in pull requests that provide fixes and general enhancements.

While I have not encountered a situation to require this, I can see how it would be more efficient and an improvement.


Added support for all DNS fields

This patch enables Skydive to capture all the DNS traffic belonging to a certain flow, parse it, and append it to the flow's data.

Hi guys,
I could use a little help here. This Orientdb test keeps failing though I didn't touch any files associated with Orientdb. Any ideas?

run skydive-functional-tests-backend-orientdb

run skydive-functional-tests-backend-orientdb

https://github.com/skydive-project/skydive/pull/1746#discussion_r272901353

https://github.com/skydive-project/skydive/pull/1746#discussion_r272901353
We can just accumulate all in a single layers.DNS. An answers packet includes the exact query sent ... so there will not be a case of conflicting values or override by a different value.

@andrekassis as you extract metadata, please add pcap traces and functional tests (dns,dhcp,vrrp) in tests/flow_test.go
we have already a dns req/resp in tests/pcaptraces/eth-ip4-arp-dns-req-http-google.pcap

@andrekassis - please let me know when you have a final version ready for testing

@nplanel  added tests for dns;)
@hunchback ready

run  skydive-functional-tests-backend-elasticsearch

run  skydive-functional-tests-backend-elasticsearch

run skydive-functional-tests-backend-elasticsearch


Fix onProgress not passed in getLoadHandlers

BUG

Fixes tensorflow/tfjs#1610.
Adds a test case to ensure onProgress works well with getLoadHandlers.

This change is

@syt123450 I see. Let's get the tfjs-core PR reviewed and checked in first. In the meantime, you can add the test, we'll run it again once your fix in tfjs-core is released.

Thanks for the fix!