quietlight/skraakCLI - Change DD3LCTLZFDIPVXXSG7RDINZCQ7NGKVG3X52OFVGVZIISD5VPF35QC

tidy up lat lng timezone api for calls classify and push certainty

Created by quietlight on May 4, 2026

DD3LCTLZFDIPVXXSG7RDINZCQ7NGKVG3X52OFVGVZIISD5VPF35QC

Dependencies

In channels

main

Change contents

Deletion in tools/calls_clip.go at line 10 [3.574094]
B:BD[3.574177] → [2.87:98]
```
	"strconv"
```

Replacement in tools/calls_clip.go at line 75 [3.574094]

B:BD[2.334] → [2.334:392]

		lat, lng, timezone, err = parseLocation(input.Location)

[2.334]

[2.392]

		lat, lng, timezone, err = utils.ParseLocation(input.Location)

Deletion in tools/calls_clip.go at line 124 [3.574094]

B:BD[4.9215] → [2.703:1531]

}
// parseLocation parses a "lat,lng[,timezone]" string into its components.
// Timezone is optional and defaults to "" (UTC).
func parseLocation(location string) (lat, lng float64, timezone string, err error) {
	parts := strings.Split(location, ",")
	if len(parts) < 2 || len(parts) > 3 {
		return 0, 0, "", fmt.Errorf("--location must be \"lat,lng\" or \"lat,lng,timezone\" (got %d parts)", len(parts))
	}
	lat, err = strconv.ParseFloat(strings.TrimSpace(parts[0]), 64)
	if err != nil {
		return 0, 0, "", fmt.Errorf("--location: invalid latitude: %s", parts[0])
	}
	lng, err = strconv.ParseFloat(strings.TrimSpace(parts[1]), 64)
	if err != nil {
		return 0, 0, "", fmt.Errorf("--location: invalid longitude: %s", parts[1])
	}
	if len(parts) == 3 {
		timezone = strings.TrimSpace(parts[2])
	}
	return lat, lng, timezone, nil

Deletion in cmd/calls_push_certainty.go at line 7 [3.1103981]
B:BD[3.1104034] → [3.1104034:1104045]
```
	"strconv"
```

Replacement in cmd/calls_push_certainty.go at line 21 [3.1103981]

B:BD[3.1104856] → [3.1104856:1105407]

	fmt.Fprintf(os.Stderr, "  --night               Only act on solar-night recordings (requires --lat and --lng)\n")
	fmt.Fprintf(os.Stderr, "  --day                 Only act on solar-day recordings (requires --lat and --lng)\n")
	fmt.Fprintf(os.Stderr, "  --lat <float>         Latitude in decimal degrees (required with --night or --day)\n")
	fmt.Fprintf(os.Stderr, "  --lng <float>         Longitude in decimal degrees (required with --night or --day)\n")
	fmt.Fprintf(os.Stderr, "  --timezone <zone>     IANA timezone ID (e.g. Pacific/Auckland)\n")

[3.1104856]

[3.1105407]

	fmt.Fprintf(os.Stderr, "  --night               Only act on solar-night recordings (requires --location)\n")
	fmt.Fprintf(os.Stderr, "  --day                 Only act on solar-day recordings (requires --location)\n")
	fmt.Fprintf(os.Stderr, "  --location <lat,lng[,tz]> GPS coordinates and optional IANA timezone\n")
	fmt.Fprintf(os.Stderr, "                        e.g. --location \"-36.85,174.76\" or --location \"-36.85,174.76,Pacific/Auckland\"\n")
	fmt.Fprintf(os.Stderr, "                        Required with --night or --day. Timezone defaults to UTC.\n")
	fmt.Fprintf(os.Stderr, "                        Not needed for AudioMoth data (UTC from WAV comment).\n")

Replacement in cmd/calls_push_certainty.go at line 29 [3.1103981]

B:BD[3.1105538] → [3.1105538:1105660]

	fmt.Fprintf(os.Stderr, "  skraak calls push-certainty --folder ./data --species Kiwi --night --lat -45.5 --lng 167.4\n")

[3.1105538]

[3.1105660]

	fmt.Fprintf(os.Stderr, "  skraak calls push-certainty --folder ./data --species Kiwi --night --location \"-45.5,167.4\"\n")

Deletion in cmd/calls_push_certainty.go at line 38 [3.1103981]
B:BD[5.24554] → [5.24554:24571]
```
	timezone string
```
Replacement in cmd/calls_push_certainty.go at line 40 [3.1103981]
B:BD[5.24601] → [5.24601:24667]
```
	lat      float64
	lng      float64
	latSet   bool
	lngSet   bool
```
[5.24601]
[5.24667]
```
	location string
```

Replacement in cmd/calls_push_certainty.go at line 64 [3.1103981]

B:BD[3.1106971] → [3.1106971:1106987]

B:BD[3.1106987] → [5.25026:25084]

∅:D[5.25084] → [3.1107281:1107297]

B:BD[3.1107281] → [3.1107281:1107297]

B:BD[3.1107297] → [5.25085:25143]

∅:D[5.25143] → [3.1107591:1107612]

B:BD[3.1107591] → [3.1107591:1107612]

B:BD[3.1107612] → [5.25144:25188]

		case "--lat":
			f.lat = requireFloat(arg, args, &i)
			f.latSet = true
		case "--lng":
			f.lng = requireFloat(arg, args, &i)
			f.lngSet = true
		case "--timezone":
			f.timezone = requireValue(arg, args, &i)

[3.1106970]

[3.1107759]

		case "--location":
			if f.location != "" {
				fmt.Fprintf(os.Stderr, "Error: --location can only be specified once\n")
				os.Exit(1)
			}
			f.location = requireValue(arg, args, &i)

Deletion in cmd/calls_push_certainty.go at line 93 [3.1103981]

B:BD[5.25486] → [5.25486:25808]

// requireFloat parses the next argument as a float64, or exits with an error
func requireFloat(flag string, args []string, i *int) float64 {
	s := requireValue(flag, args, i)
	v, err := strconv.ParseFloat(s, 64)
	if err != nil {
		fmt.Fprintf(os.Stderr, "Error: %s must be a number\n", flag)
		os.Exit(1)
	}
	return v
}

Replacement in cmd/calls_push_certainty.go at line 105 [3.1103981]

B:BD[3.1108251] → [5.26000:26053]

∅:D[5.26053] → [3.1108296:1108379]

B:BD[3.1108296] → [3.1108296:1108379]

	if (f.night || f.day) && (!f.latSet || !f.lngSet) {
		fmt.Fprintf(os.Stderr, "Error: --night/--day requires both --lat and --lng\n\n")

[3.1108251]

[3.1108379]

	if (f.night || f.day) && f.location == "" {
		fmt.Fprintf(os.Stderr, "Error: --night/--day requires --location\n\n")

Insertion in cmd/calls_push_certainty.go at line 138 [3.1103981]

[3.1108868]

	var lat, lng float64
	var timezone string
	if f.location != "" {
		var err error
		lat, lng, timezone, err = utils.ParseLocation(f.location)
		if err != nil {
			fmt.Fprintf(os.Stderr, "Error: %v\n", err)
			os.Exit(1)
		}
	}

Replacement in cmd/calls_push_certainty.go at line 157 [3.1103981]

B:BD[5.26696] → [5.26696:26758]

		Lat:      f.lat,
		Lng:      f.lng,
		Timezone: f.timezone,

[5.26696]

[3.1109103]

		Lat:      lat,
		Lng:      lng,
		Timezone: timezone,

Replacement in cmd/calls_classify.go at line 40 [3.1141693]

B:BD[3.1143399] → [3.1143399:1144090]

	fmt.Fprintf(os.Stderr, "  --night               Only review solar-night recordings (requires --lat and --lng)\n")
	fmt.Fprintf(os.Stderr, "  --day                 Only review solar-day recordings (requires --lat and --lng)\n")
	fmt.Fprintf(os.Stderr, "  --lat <float>         Latitude in decimal degrees (required with --night or --day)\n")
	fmt.Fprintf(os.Stderr, "  --lng <float>         Longitude in decimal degrees (required with --night or --day)\n")
	fmt.Fprintf(os.Stderr, "  --timezone <zone>     IANA timezone ID (e.g. Pacific/Auckland). Required for non-AudioMoth\n")
	fmt.Fprintf(os.Stderr, "                        recorders whose filenames embed local time (e.g. DOC AR4).\n")

[3.1143399]

[3.1144090]

	fmt.Fprintf(os.Stderr, "  --night               Only review solar-night recordings (requires --location)\n")
	fmt.Fprintf(os.Stderr, "  --day                 Only review solar-day recordings (requires --location)\n")
	fmt.Fprintf(os.Stderr, "  --location <lat,lng[,tz]> GPS coordinates and optional IANA timezone\n")
	fmt.Fprintf(os.Stderr, "                        e.g. --location \"-36.85,174.76\" or --location \"-36.85,174.76,Pacific/Auckland\"\n")
	fmt.Fprintf(os.Stderr, "                        Required with --night or --day. Timezone defaults to UTC.\n")
	fmt.Fprintf(os.Stderr, "                        Not needed for AudioMoth data (UTC from WAV comment).\n")

Deletion in cmd/calls_classify.go at line 73 [3.1141693]
B:BD[6.8074] → [6.8074:8092]
```
	timezone  string
```
Replacement in cmd/calls_classify.go at line 77 [3.1141693]
B:BD[6.8154] → [6.8154:8224]
```
	lat       float64
	lng       float64
	latSet    bool
	lngSet    bool
```
[6.8154]
[6.8224]
```
	location  string
```
Deletion in cmd/calls_classify.go at line 110 [3.1141693]
∅:D[6.8880] → [3.1147592:1147602]
B:BD[3.1147592] → [3.1147592:1147602]
B:BD[3.1147603] → [3.1147603:1147624]
B:BD[3.1147624] → [6.8881:9024]
```
			i += 2
		case "--timezone":
			a.timezone = a.requireValue(args, i, "--timezone")
			i += 2
		case "--lat":
			a.lat = a.requireFloat(args, i, "--lat")
			a.latSet = true
```

Replacement in cmd/calls_classify.go at line 111 [3.1141693]

B:BD[6.9034] → [6.9034:9113]

		case "--lng":
			a.lng = a.requireFloat(args, i, "--lng")
			a.lngSet = true

[6.9034]

[3.1147760]

		case "--location":
			a.location = a.requireUniqueValue(args, i, "--location", a.location)

Deletion in cmd/calls_classify.go at line 166 [3.1141693]
B:BD[3.1149225] → [6.10202:10407]
∅:D[6.10407] → [3.1149341:1149358]
B:BD[3.1149341] → [3.1149341:1149358]
B:BD[3.1149358] → [6.10408:10471]
∅:D[6.10471] → [3.1149532:1149548]
B:BD[3.1149532] → [3.1149532:1149548]
B:BD[3.1149548] → [6.10472:10484]
∅:D[6.10484] → [3.1149548:1149549]
B:BD[3.1149548] → [3.1149548:1149549]
```
// requireFloat parses the next arg as float64.
func (a classifyArgs) requireFloat(args []string, i int, flag string) float64 {
	val := a.requireValue(args, i, flag)
	v, err := strconv.ParseFloat(val, 64)
	if err != nil {
		fmt.Fprintf(os.Stderr, "Error: %s must be a number\n", flag)
		os.Exit(1)
	}
	return v
}
```

Replacement in cmd/calls_classify.go at line 182 [3.1141693]

B:BD[6.11010] → [6.11010:11146]

	if (a.night || a.day) && (!a.latSet || !a.lngSet) {
		fmt.Fprintf(os.Stderr, "Error: --night/--day requires both --lat and --lng\n\n")

[6.11010]

[6.11146]

	if (a.night || a.day) && a.location == "" {
		fmt.Fprintf(os.Stderr, "Error: --night/--day requires --location\n\n")

Insertion in cmd/calls_classify.go at line 270 [3.1141693]

[3.1151628]

	// Parse location into lat/lng/timezone
	var lat, lng float64
	var timezone string
	if a.location != "" {
		var err error
		lat, lng, timezone, err = utils.ParseLocation(a.location)
		if err != nil {
			fmt.Fprintf(os.Stderr, "Error: %v\n", err)
			os.Exit(1)
		}
	}

Replacement in cmd/calls_classify.go at line 301 [3.1141693]

B:BD[6.12572] → [6.12572:12661]

		Lat:               a.lat,
		Lng:               a.lng,
		Timezone:          a.timezone,

[6.12572]

[3.1152351]

		Lat:               lat,
		Lng:               lng,
		Timezone:          timezone,

Replacement in README.md at line 250 [3.1177482]

B:BD[3.1188592] → [3.1188592:1188685]

./skraak calls push-certainty --folder ./data --species Kiwi --night --lat -45.5 --lng 167.4

[3.1188592]

[3.1188685]

./skraak calls push-certainty --folder ./data --species Kiwi --night --location "-45.5,167.4"

Replacement in CHANGELOG.md at line 5 [3.1197933]

B:BD[2.3274] → [2.3274:3370]

## [2026-05-05] `calls clip`: replace --lat/--lng/--timezone with --location; remove --wav-only

[2.3274]

[2.3370]

## [2026-05-05] Replace --lat/--lng/--timezone with --location across CLI; remove --wav-only

Replacement in CHANGELOG.md at line 7 [3.1197933]

B:BD[2.3371] → [2.3371:3437]

**Breaking CLI change.** Two flag changes to `skraak calls clip`:

[2.3371]

[2.3437]

**Breaking CLI change.** The `--lat`, `--lng`, `--timezone` flags are replaced
by a single `--location "lat,lng[,timezone]"` flag in three commands:

Replacement in CHANGELOG.md at line 10 [3.1197933]

B:BD[2.3438] → [2.3438:3742]

1. **`--lat`, `--lng`, `--timezone` → `--location`** — The three GPS/timezone
   flags are replaced by a single `--location "lat,lng[,timezone]"` flag.
   Timezone is optional (defaults to UTC; not needed for AudioMoth).
   This makes invalid states unrepresentable (you can't pass lat without lng).

[2.3438]

[2.3742]

- `skraak calls clip`
- `skraak calls classify`
- `skraak calls push-certainty`

Replacement in CHANGELOG.md at line 14 [3.1197933]

B:BD[2.3743] → [2.3743:4094]

   Before:
   ```bash
   skraak calls clip --folder ./data --output ./clips --prefix kiwi \
     --species Kiwi --night --lat -40.85 --lng 172.81 --timezone Pacific/Auckland
   ```
   After:
   ```bash
   skraak calls clip --folder ./data --output ./clips --prefix kiwi \
     --species Kiwi --night --location "-40.85,172.81,Pacific/Auckland"
   ```

[2.3743]

[7.27627]

Timezone is optional (defaults to UTC; not needed for AudioMoth).
This makes invalid states unrepresentable (you can't pass lat without lng).

Replacement in CHANGELOG.md at line 17 [3.1197933]

B:BD[7.27628] → [2.4095:4275]

2. **`--wav-only` removed** — This flag skipped spectrogram PNG generation.
   Removed as unnecessary; the tool's primary purpose is generating training
   data (PNG+WAV pairs).

[7.27628]

[2.4275]

Before:
```bash
skraak calls clip --folder ./data --output ./clips --prefix kiwi \
  --species Kiwi --night --lat -40.85 --lng 172.81 --timezone Pacific/Auckland
```
After:
```bash
skraak calls clip --folder ./data --output ./clips --prefix kiwi \
  --species Kiwi --night --location "-40.85,172.81,Pacific/Auckland"
```
Additionally, `--wav-only` was removed from `skraak calls clip`. This flag
skipped spectrogram PNG generation; removed as unnecessary since the tool's
primary purpose is generating training data (PNG+WAV pairs).

Insertion in CHANGELOG.md at line 32 [3.1197933]

[2.4276]

A shared `utils.ParseLocation` helper was added to avoid duplicating the
`lat,lng[,tz]` parse logic across commands.

Replacement in CHANGELOG.md at line 36 [3.1197933]

B:BD[2.4295] → [2.4295:4515]

- `cmd/calls_clip.go` — flag parsing, validation, usage text
- `tools/calls_clip.go` — `CallsClipInput` struct, `parseLocation` helper,
  removed `wavOnly` from function signatures, unwrapped spectrogram conditional

[2.4295]

[2.4515]

- `utils/location.go` — new `ParseLocation` helper
- `cmd/calls_clip.go` — `--location` flag, removed `--wav-only`
- `cmd/calls_classify.go` — `--location` flag
- `cmd/calls_push_certainty.go` — `--location` flag
- `tools/calls_clip.go` — `CallsClipInput` struct, removed `wavOnly`/`parseLocation`