Per the example in https://godoc.org/github.com/google/go-cmp/cmp#Diff the form should be:

if diff := cmp.Diff(want, got); diff != "" {
    t.Errorf("MakeGatewayInfo() mismatch (-want +got):\n%s", diff)
}

however, NUD tests (and possibly others) are littered with the reverse, e.g.:
https://github.com/google/gvisor/blob/8fa18e8ecb3e3da2015e5c73db9e46494dcfeff7/pkg/tcpip/stack/neighbo

I bet most users just use the same device every time they want to load NoiseTorch (I do). The last update removed the nuisance of needing to type your password, but it can get even smoother. Maybe this could be a toggle switch under the Settings section.
Think of RTX voice, which uses the last device whenever it starts.

Feature Request

There are many config generation options in talosctl cluster create which make no sense with --input-dir as they're going to be ignored.

Description

Fail with error message is any gen options are used with --input-dir. This might confuse users as arguments are silently ignored.

Since the default output is meant to be human-readable, would it make sense to add thousands separators to make the output more easily readable?

Version of Singularity:

3.0.3

Expected behavior

Some progress or update information during the creation of the SIF file.

Actual behavior

During "Creating SIF file..." step, the console does not produce any visible output for the user - but does create the resulting file eventually.

Steps to reproduce behavior

I was executing a few “larger” builds through the bui