Skip to content

Update financial.rst - original syntax throws KeyError#356

Open
itscarlo-nobnos wants to merge 1 commit into
spyder-ide:masterfrom
itscarlo-nobnos:patch-1
Open

Update financial.rst - original syntax throws KeyError#356
itscarlo-nobnos wants to merge 1 commit into
spyder-ide:masterfrom
itscarlo-nobnos:patch-1

Conversation

@itscarlo-nobnos

@itscarlo-nobnos itscarlo-nobnos commented Dec 10, 2023

Copy link
Copy Markdown

The text in https://docs.spyder-ide.org/current/workshops/financial.html related to Mean and Standard Deviation presents a KeyError.

Line 193 in the linked file available from the website is also affected.

Pull Request

Pull Request Checklist

  • Read and followed this repo's Contributing Guidelines
  • Based your PR on the latest version of the correct branch (master or 4.x)
  • Checked your writing carefully for correct English spelling, grammar, etc
  • Described your changes and the motivation for them below
  • Noted what issue(s) this pull request resolves, creating one if needed

Description of Changes

Issue(s) Resolved

Fixes #

The text in https://docs.spyder-ide.org/current/workshops/financial.html related to Mean and Standard Deviation presents a KeyError. 

Line 193 in the linked file available from the website is also affected.

@itscarlo-nobnos itscarlo-nobnos left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I tested the code but need someone to approve it for the workshop documentation.

@ccordoba12 ccordoba12 requested a review from CAM-Gerlach January 3, 2024 16:34
@ccordoba12

Copy link
Copy Markdown
Member

Thanks for your contribution @itscarlo-nobnos! We'll take a look at it soon.

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