Skip to content

fix spelling and grammar errors - #1406

Open
moazreyad wants to merge 2 commits into
apache:dev-postgresqlfrom
moazreyad:fix-typos
Open

moazreyad wants to merge 2 commits into
apache:dev-postgresqlfrom
moazreyad:fix-typos

Conversation

@moazreyad

Copy link
Copy Markdown
Contributor

No description provided.

@moazreyad moazreyad self-assigned this Jul 14, 2026

@lzjpaul lzjpaul left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

are all these comments and not affecting the code implementations?

@moazreyad

Copy link
Copy Markdown
Contributor Author

are all these comments and not affecting the code implementations?

The PR mainly corrects spelling and grammar in comments, documentation, CLI help, logs, warnings, and error messages. These edits do not change model calculations.

There are a few code/API-visible changes:

  • The C++ declaration Boradcast is corrected to Broadcast.
  • The SWIG-visible EnableLazyAlloc parameter changes from enbale to enable, which changes the keyword name exposed to Python callers.
  • The returned string "unkown type" is corrected to "unknown type".
  • The phase-one runner’s import was corrected. The PR imported controller.py, while the file was still named controler.py; I renamed the file to controller.py in
    the latest commit so the import resolves.

The remaining executable-file changes correct user-visible text, such as log messages, warnings, assertions, and CLI descriptions. So the PR is mostly text corrections, but it is not limited to comments. No model or training logic was changed.

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