Skip to content

fix: dataflow.yml in llm.md, Link to the requirement_llm.txt, and typo on yolov8.md - #59

Closed
sauhardh wants to merge 2 commits into
dora-rs:mainfrom
sauhardh:main
Closed

fix: dataflow.yml in llm.md, Link to the requirement_llm.txt, and typo on yolov8.md#59
sauhardh wants to merge 2 commits into
dora-rs:mainfrom
sauhardh:main

Conversation

@sauhardh

@sauhardh sauhardh commented Feb 28, 2026

Copy link
Copy Markdown

Solves #58

Changes and Improvements

on yolov8.md

  • .yaml -> .yml and then the link finally works.
  • dataflow.yaml to dataflow.yml because later in command line, it is being run using dora start dataflow.yml --attach

on llm.md

  • Added link to the requirement_llm.txt

  • Added Optimization for lower resource (Optional) section

  • Changes on keyboard nodes (This way, it solves the error it threw earlier.)
    (And, i suppose, it can't be operator as Node() is initialized on keyboard_op.py)

  ## Speech to text
  - id: keyboard
    custom:
      source: !Local
      path: keyboard_op.py
      outputs:
        - buffer
        - submitted
        - record
        - ask
        - send
        - change
      inputs:
        recording: whisper/text

Copilot AI review requested due to automatic review settings February 28, 2026 17:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the YOLOv8 getting-started guide so the referenced dataflow file link resolves correctly and matches the filename used in the run command.

Changes:

  • Fixes the wget URL to download dataflow.yml (instead of the non-existent dataflow.yaml) for dora v0.3.6.
  • Renames the inline filename label from dataflow.yaml to dataflow.yml to align with dora start dataflow.yml --attach.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sauhardh sauhardh changed the title fix: typo on getting-started yolov8 guide, link to the github file fix: dataflow.yml in llm.md, Link to the requirement_llm.txt, and typo on yolov8.md Mar 1, 2026
@haixuanTao haixuanTao closed this Mar 26, 2026
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