Skip to content

Feat: hello world example - #3

Merged
jhcipar merged 3 commits into
mainfrom
jhcipar/alpha-flash-examples-01
Nov 17, 2025
Merged

jhcipar merged 3 commits into
mainfrom
jhcipar/alpha-flash-examples-01

Conversation

@jhcipar

@jhcipar jhcipar commented Nov 17, 2025

Copy link
Copy Markdown
Contributor

This PR introduces the first runnable Flash example under 01_getting_started/01_hello_world, giving both CPU and GPU worker implementations plus a simple entrypoint.

It adds a .flashignore tailored for the example’s artifacts, a main.py that spins up the example with routing and matching worker packages (workers/cpu and workers/gpu) with stub init.pys and concrete endpoint.py handlers so developers can see parallel CPU/GPU patterns side by side.

@jhcipar
jhcipar requested a review from deanq November 17, 2025 15:32

# Scaling behavior is controlled by configuration passed to the `CpuLiveServerless` class.
# Here, we'll define several variables that change the default behavior of our serverless endpoint.
# `workersMin` sets our endpoint to scale to 0 active containers; `workersMax` will allow our endpoint

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nitpicky, but this comment wraps at 102 characters, and the one on line 12 doesn't wrap and is 123 characters.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 probably time to have an actual strategy for comment wrapping, instead of the mad max approach

@deanq

deanq commented Nov 17, 2025

Copy link
Copy Markdown
Member

@jhcipar I've thought about keeping the first example to its simplest form, and that we should just focus on having a GPU worker. This way, we can have a separate one just for CPU worker. The rest would go like:

  • 02_cpu_worker
  • 03_mixed_workers
  • 04_dependencies

@deanq deanq left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's focus on GPU workers to keep the first example small. See my comment in general. Also the general 01 readme https://github.com/runpod/flash-examples/tree/main/01_getting_started

@jhcipar
jhcipar force-pushed the jhcipar/alpha-flash-examples-01 branch from 3f46613 to 9235dde Compare November 17, 2025 20:27
@jhcipar
jhcipar force-pushed the jhcipar/alpha-flash-examples-01 branch from 9235dde to db2d60b Compare November 17, 2025 20:31
@jhcipar
jhcipar merged commit 890bf44 into main Nov 17, 2025
6 checks passed
@jhcipar
jhcipar deleted the jhcipar/alpha-flash-examples-01 branch November 17, 2025 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants