Skip to content

docs: fix remaining language errors - #1597

Merged
marcopeix merged 1 commit into
mainfrom
fix/remaining-language-errors
Aug 24, 2026
Merged

docs: fix remaining language errors#1597
marcopeix merged 1 commit into
mainfrom
fix/remaining-language-errors

Conversation

@loama

@loama loama commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

What

Fix eight objective spelling errors in the published NeuralForecast documentation and model parameter text.

Why

These mistakes are visible on the generated Nixtlaverse pages. The corrections preserve the technical meaning and change only misspelled words.

How

  1. Correct three Prerequisites headings.
  2. Correct the descriptions for cutoff timestamps and RevIn.
  3. Correct spelling in the TFT and TimesNet model pages.

Testing

  1. Parsed all four modified notebooks as valid JSON.
  2. Confirmed the reported misspellings are absent from the affected files.
  3. Reviewed the complete diff.
  4. Confirmed no secrets, credentials, or generated binaries are included.

Checklist

  1. The title follows the conventional commit format.
  2. The pull request is focused on one concern.
  3. The complete diff was reviewed.
  4. No secrets, credentials, or personal data are included.

@loama loama self-assigned this Aug 23, 2026
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@CLAassistant

CLAassistant commented Aug 23, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@loama
loama requested review from deven367 and marcopeix and removed request for deven367 August 23, 2026 08:55
@loama
loama enabled auto-merge (squash) August 23, 2026 08:58
@deven367
deven367 disabled auto-merge August 23, 2026 14:06
@marcopeix
marcopeix merged commit 6c4f3e5 into main Aug 24, 2026
21 checks passed
@marcopeix
marcopeix deleted the fix/remaining-language-errors branch August 24, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants