Skip to content

[TASK] Cpython (https://github.com/SENATOROVAI/intro-cs/issues/4)#624

Open
YurGusev wants to merge 1 commit into
SENATOROVAI:mainfrom
YurGusev:Cpython
Open

[TASK] Cpython (https://github.com/SENATOROVAI/intro-cs/issues/4)#624
YurGusev wants to merge 1 commit into
SENATOROVAI:mainfrom
YurGusev:Cpython

Conversation

@YurGusev

@YurGusev YurGusev commented Jun 23, 2026

Copy link
Copy Markdown
Member

Closes https://github.com/SENATOROVAI/intro-cs/issues/4

Describe your change:

Посмотрел видео Интерпритатор Cpython и PYPY и ответил на вопросы.

Fixes #4

  • Add an algorithm?
  • Fix a bug or typo in an existing algorithm?
  • Add or change doctests? -- Note: Please avoid changing both code and tests in a single pull request.
  • Documentation change?

Checklist:

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized.
  • I know that pull requests will not be merged if they fail the automated tests.
  • This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
  • All new Python files are placed inside an existing directory.
  • All filenames are in all lowercase characters with no spaces or dashes.
  • All functions and variable names follow Python naming conventions.
  • All function parameters and return values are annotated with Python type hints.
  • All functions have doctests that pass the automated testing.
  • All new algorithms include at least one URL that points to Wikipedia or another similar explanation.
  • If this pull request resolves one or more open issues then the description above includes the issue number(s) with a closing keyword: "Fixes #2 #4".

@YurGusev YurGusev left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

сабмичу

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 71f81f6f21

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread python/cpython.py
Comment on lines +76 to +77
# Ответ: интерпретатор - это программа, которая читает и запускает код
# построчно.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Correct the CPython execution model

For Q10, saying CPython reads and runs code “line by line” gives learners the wrong model: CPython parses/compiles the source to bytecode first, then executes that bytecode in the virtual machine. Since this file is the lesson answer key for the CPython task, please describe the compile-to-bytecode step instead of presenting execution as purely line-by-line.

Useful? React with 👍 / 👎.

Comment thread python/cpython.py
Comment on lines +154 to +155
# Ответ: PyPy работает быстрее CPython. Главное отличие для задания - PyPy
# быстрее, но подходит не для всех проектов.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Explain PyPy's speedup mechanism

Q19 asks why PyPy can run faster, but this answer only repeats that it is faster and not suitable everywhere. That leaves the requested explanation missing for the PyPy section; please mention the JIT/tracing optimizer compiling hot paths to machine code, otherwise the lesson does not answer the question.

Useful? React with 👍 / 👎.

Comment thread python/cpython.py
Comment on lines +236 to +238
# Ответ: для сравнения нужно запустить один и тот же `performance_test.py` сначала
# через CPython, затем через PyPy и сравнить строку `Execution time`. PyPy должен работать быстрее CPython, но использовать его можно не для всех
# проектов.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Include actual performance comparison results

In practical task 5, the answer only says how to run performance_test.py, but the task is to compare CPython and PyPy. Without the measured Execution time output for both interpreters and a conclusion, the submission does not demonstrate that the practical task was completed or give readers anything to verify.

Useful? React with 👍 / 👎.

@ruslansenatorov

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

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