Skip to content

Better "hello world" example sequence - #4575

Draft
RHartung-ND wants to merge 4 commits into
cooperative-computing-lab:masterfrom
RHartung-ND:master
Draft

RHartung-ND wants to merge 4 commits into
cooperative-computing-lab:masterfrom
RHartung-ND:master

Conversation

@RHartung-ND

@RHartung-ND RHartung-ND commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

Proposed Changes

Updates to the TaskVine "hello world" example as per pr (#4562).

Updates to (quickstart.py)

  • Estimates pi with Monte Carlo sampling split across 16 PythonTasks
  • Uses port=0 and a per-user project name (taskvine-pi-<username>).
  • Creates the Manager and a local Factory with with blocks, so workers and the manager are cleaned up automatically.
  • Includes comments for new users (why imports go inside remote functions, what wait() returning None means, and what task.output holds).

New "Build on the Quick Start" section:

  • Unix command tasks (Task with an input buffer)
  • Python function tasks (PythonTask)
  • Basic resource requests (set_cores, set_memory)
  • Shared local file and URL inputs with cache="workflow"
  • Connecting two tasks through a temporary file (declare_temp)

Merge Checklist

  • make test Run local tests prior to pushing.
  • make format Format source code to comply with lint policies. Note that some lint errors can only be resolved manually (e.g., Python)
  • make lint Run lint on source code prior to pushing.
  • Manual Update: Update the manual to reflect user-visible changes.
  • Type Labels: Select a github label for the type: bugfix, enhancement, etc.
  • Product Labels: Select a github label for the product: TaskVine, Makeflow, etc.
  • PR RTM: Mark your PR as ready to merge.

@dthain

dthain commented Nov 10, 2026

Copy link
Copy Markdown
Member

Make a new PR against the taskvine-manual-restructure branch, please.

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.

2 participants