Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

Keyboard telemetry #1619

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

d-loose
Copy link
Member

@d-loose d-loose commented Mar 14, 2023

Collect keyboard layout and variant

@jpnurmi
Copy link
Contributor

jpnurmi commented Mar 14, 2023

LGTM 👍

Let's double-check the keys:

@jpnurmi
Copy link
Contributor

jpnurmi commented Mar 14, 2023

Looking at the ubuntu-report example again, I see that "Language" and "Timezone" are outside the "Install" section which is the only part written by the installer. I guess the same should apply to the keyboard layout so we might have just wasted your time on this, sorry. :(

{
  "Version": "18.04",
  "OEM": {
    "Vendor": "Vendor Name",
    "Product": "4287CTO"
  },
  "BIOS": {
    "Vendor": "Vendor Name",
    "Version": "8DET52WW (1.27)"
  },
  "CPU": {
    "OpMode": "32-bit, 64-bit",
    "CPUs": "8",
    "Threads": "2",
    "Cores": "4",
    "Sockets": "1",
    "Vendor": "Genuine",
    "Family": "6",
    "Model": "158",
    "Stepping": "10",
    "Name": "Intius Corus i5-8300H CPU @ 2.30GHz",
    "Virtualization": "VT-x"
  },
  "Arch": "amd64",
  "GPU": [
    {
      "Vendor": "8086",
      "Model": "0126"
    }
  ],
  "RAM": 8,
  "Disks": [
    240.1,
    500.1
  ],
  "Partitions": [
    229.2,
    479.7
  ],
  "Screens": [
    {
      "Size": "277mmx156mm",
      "Resolution": "1366x768",
      "Frequency": "60.02"
    },
    {
      "Resolution": "1920x1080",
      "Frequency": "60.00"
    }
  ],
  "Autologin": false,
  "LivePatch": true,
  "Session": {
    "DE": "ubuntu:GNOME",
    "Name": "ubuntu",
    "Type": "x11"
  },
  "Language": "fr_FR",
  "Timezone": "Europe/Paris",
  "Install": {
    "Media": "Ubuntu 18.04 LTS \"Bionic Beaver\" - Alpha amd64 (20180305)",
    "Type": "GTK",
    "PartitionMethod": "use_device",
    "DownloadUpdates": true,
    "Language": "fr",
    "Minimal": false,
    "RestrictedAddons": false,
    "Stages": {
      "0": "language",
      "3": "language",
      "10": "console_setup",
      "15": "prepare",
      "25": "partman",
      "27": "start_install",
      "37": "timezone",
      "49": "usersetup",
      "57": "user_done",
      "829": "done"
    }
  }
}

@d-loose
Copy link
Member Author

d-loose commented Mar 14, 2023

No worries, this was just a quick no-effort thing after the ActiveDirectory PR :)

"Language" is also collected within "Install", while "Timezone" is not. I'm not sure how valuable it is to collect those things during the installation - they might not be terribly relevant later on.

@elioqoshi elioqoshi added this to the Lunar Design Polish milestone Mar 17, 2023
@elioqoshi
Copy link

It would be great to collect all of that anonymous data as we can base design decisions on these things later on instead of assumptions.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants