Skip to content

fix: spelling errors, broken dependencies, and broken doc link#6

Open
Kishan0703 wants to merge 1 commit into
In-Network-Machine-Learning:mainfrom
Kishan0703:main
Open

fix: spelling errors, broken dependencies, and broken doc link#6
Kishan0703 wants to merge 1 commit into
In-Network-Machine-Learning:mainfrom
Kishan0703:main

Conversation

@Kishan0703

Copy link
Copy Markdown

What this PR does

Fixes three categories of issues found during onboarding — typos, broken dependencies, and a broken doc link. No logic changes.

Why it matters

  • Broken dependencies (multiprocess=0.70.12, pytorch) block fresh installs
  • Outdated pins (numpy==1.19.2) fail on Python 3.10+
  • Typos in CLI output confuse users
  • Broken doc link hurts new contributors trying to set up P4Pi on BMv2

Changes

Area Fixes
Spelling (60+ fixes) Verification seccesssuccess, falesfails, varifcationverification, paraemtersparameters, funtionsfunctions, tunningtuning, wirtewrite, etc.
requirements_pip3.txt Remove duplicate category_encoders, remove invalid pytorch (keep torch), fix scikit_learnscikit-learn, update numpy and xgboost for Python 3.10+
requirements_sudo_pip3.txt Fix multiprocess=0.70.12==, fix pytorchtorch, fix scikit_learnscikit-learn, trim leading whitespace
Envronment_Issues.md Fix broken relative link: ./src/help/...../Throughput_Test/...

What's NOT changed

  • Zero logic or functionality changes
  • No file renames

Checklist

  • Spelling fixes — comments and strings only
  • Dependency files — syntax and compatibility fixes
  • Doc link — verified correct relative path
  • No new warnings introduced

@Kishan0703

Copy link
Copy Markdown
Author

This address the #5

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.

1 participant