Skip to content

fix: make welcome.py self-contained instead of loading missing dataset - #45

Merged
Jammy2211 merged 1 commit into
mainfrom
feature/welcome-start-here-fixes
Apr 30, 2026
Merged

fix: make welcome.py self-contained instead of loading missing dataset#45
Jammy2211 merged 1 commit into
mainfrom
feature/welcome-start-here-fixes

Conversation

@Jammy2211

Copy link
Copy Markdown
Collaborator

Summary

welcome.py crashed with FileNotFoundError: dataset/example_1d/gaussian_x1/data.json
on a fresh clone, because dataset/ is gitignored and the data files
only exist after a simulator script runs. Make the welcome demo
self-contained: synthesise a noisy 1D Gaussian inline with numpy, same
spirit as the other workspaces' welcome scripts which build their demo
data in-memory.

Scripts Changed

  • welcome.py — drop from os import path and the af.util.numpy_array_from_json calls; synthesise the demo gaussian + noise inline with numpy.random.default_rng

Test Plan

  • yes "" | python welcome.py runs to completion (exit 0) on a clone with no dataset/ directory
  • Smoke tests pass

Refs PyAutoLabs/autolens_workspace#108

🤖 Generated with Claude Code

…om disk

`welcome.py` previously loaded `dataset/example_1d/gaussian_x1/data.json`,
but `dataset/` is gitignored — so a fresh clone has nothing to load and
the script crashes with `FileNotFoundError`. Synthesise a noisy 1D
Gaussian inline with numpy instead, making the script self-contained
like the other workspace `welcome.py` files.

Refs PyAutoLabs/autolens_workspace#108

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Jammy2211 Jammy2211 added the pending-release PR queued for the next release build label Apr 30, 2026
@Jammy2211
Jammy2211 merged commit 620e6d5 into main Apr 30, 2026
1 of 5 checks passed
@Jammy2211
Jammy2211 deleted the feature/welcome-start-here-fixes branch April 30, 2026 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pending-release PR queued for the next release build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant