[pavanpannati]

Closes #523

Fixes the nonexistent cd python/pandas in the pandas setup instructions: the setup now enters fundamentals/pandas, so the unchanged Run commands work from a fresh checkout.

[yeongseon]

Thanks for picking up #523, @pavanpannati. The broken cd python/pandas does need fixing, but this version introduces two new problems:

1. The Run section stops working. It still says python3 01_read_sql.py and so on, which only works from inside fundamentals/pandas. With this change the reader stays at the repository root, so every Run command fails with "No such file or directory".
2. cd cubrid-cookbook-python/ assumes where the reader is. It works only from the folder that contains the clone. #523 asks for commands that work from the repository root.

The smallest fix that keeps every later step working is to change only the first line:

cd fundamentals/pandas   # from the repository root
python3 -m venv .venv
source .venv/bin/activate
pip install -r requirements.txt

Two smaller things:

• The PR title check requires a Conventional Commits title, for example docs(fundamentals): fix the pandas setup directory.
• Please add Closes #523 to the description so the issue closes on merge.

I have approved the CI runs for this PR, so you will see the results after your next push.

[pavanpannati]

Thanks for picking up #523, @pavanpannati. The broken cd python/pandas does need fixing, but this version introduces two new problems:

1. The Run section stops working. It still says python3 01_read_sql.py and so on, which only works from inside fundamentals/pandas. With this change the reader stays at the repository root, so every Run command fails with "No such file or directory".
2. cd cubrid-cookbook-python/ assumes where the reader is. It works only from the folder that contains the clone. bug: Fix the nonexistent pandas directory in setup instructions #523 asks for commands that work from the repository root.

The smallest fix that keeps every later step working is to change only the first line:

cd fundamentals/pandas   # from the repository root
python3 -m venv .venv
source .venv/bin/activate
pip install -r requirements.txt

Two smaller things:

• The PR title check requires a Conventional Commits title, for example docs(fundamentals): fix the pandas setup directory.
• Please add Closes #523 to the description so the issue closes on merge.

I have approved the CI runs for this PR, so you will see the results after your next push.

I hope issue #543 closes look into it, notify me if there any other issues

[yeongseon]

[P2] Keep the Run commands consistent with the new working directory

The Setup block now leaves the reader at the repository root (cd cubrid-cookbook-python/), but the Run block still uses python3 01_read_sql.py through 06_export_csv.py. Those files live under fundamentals/pandas/, so following Setup and Run in sequence fails with a file-not-found error.

Please add cd fundamentals/pandas before the Run commands, or prefix all six script paths with fundamentals/pandas/. Validate the two blocks together from a fresh checkout; the current green documentation checks did not catch this mismatch.

[yeongseon]

Thanks for this, @pavanpannati. To finish it, I pushed the single-line change suggested in the review: cd fundamentals/pandas from the repository root, keeping pip install -r requirements.txt so the Run commands work. I also updated the title and set Closes #523 in the description.

The requested one-line fix, title and Closes #523 are applied in 3841fb7.

[yeongseon]

Maintainer review (delegated): the diff is now only cd fundamentals/pandas from the repository root; that directory holds requirements.txt and the six Run scripts, so setup and Run work together. My earlier change request is addressed and dismissed. CI Gate, docs-sync and PR title pass. Merging.

[pavanpannati]

Thank you @yeongseon