Skip to content

Fix solution for exercise 5#256

Open
jsyzlbw wants to merge 3 commits into
rougier:masterfrom
jsyzlbw:master
Open

Fix solution for exercise 5#256
jsyzlbw wants to merge 3 commits into
rougier:masterfrom
jsyzlbw:master

Conversation

@jsyzlbw

@jsyzlbw jsyzlbw commented Jul 8, 2026

Copy link
Copy Markdown

Fixes #185.

The solution for exercise 5 currently uses %run with a shell command:

%run `python -c "import numpy; numpy.info(numpy.add)"`

In IPython/Jupyter, %run expects a Python file/script, so this can fail with File 'python.py' not found.
Since the exercise asks for a command-line solution, this PR changes it to:

python -c "import numpy; numpy.info(numpy.add)"

@rougier

rougier commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Thanks for the fix. Can you apply on kth files instead (they are used to generate others)

@jsyzlbw

jsyzlbw commented Jul 10, 2026

Copy link
Copy Markdown
Author

Applied the fix to source/exercises100.ktx and regenerated the Markdown files.
Thank you very much for review

@rougier

rougier commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Thanks. Just the kth file please, I'll regenerate other files.

@jsyzlbw

jsyzlbw commented Jul 10, 2026

Copy link
Copy Markdown
Author

OK👌

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.

about question 5

2 participants