Skip to content

Conversation

@tak-ka3
Copy link
Collaborator

@tak-ka3 tak-ka3 commented Aug 25, 2025

Description of changes・変更内容の説明

基本チュートリアルの修正をした。

Ticket link・チケットリンク

https://www.notion.so/acompany-ac/CC-883

Testing method・実施したテスト方法 (CI/Manual)

  • E2Eテストを実行し、正常に動作したことを確認。

Remarks・備考

Review guidelines

Generic PR Review guidelines・一般的な PR レビューのガイドライン

@tak-ka3 tak-ka3 requested a review from a team as a code owner August 25, 2025 07:00
@tak-ka3 tak-ka3 force-pushed the fix/hiraoka/CC-883/modify-jupyter-basic-tutorial branch from 5f6374d to 8086619 Compare August 25, 2025 07:04
@github-actions
Copy link

github-actions bot commented Aug 25, 2025

Ruff チェック結果

フォーマット

✅ フォーマットの問題はありませんでした。

3 files already formatted

Lint

✅ Lint の問題はありませんでした。

Copy link
Contributor

@sakamoto-souta sakamoto-souta left a comment

Choose a reason for hiding this comment

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

cross_table の列名 rename は 0-indexed なのが気になる
そもそも同じ名前の列があったら polars が自動で rename をしてくれるはずなので、rename やめてもいいかも
join はそうなっているはず

@sakamoto-souta
Copy link
Contributor

<apc_cli_version> を入力する箇所が二箇所あるから、バイナリダウンロードセルの先頭に一時変数として

APC_CLI_VERSION=

を置いてそれを参照するようにした方が見やすい

@tak-ka3
Copy link
Collaborator Author

tak-ka3 commented Aug 26, 2025

@sakamoto-souta

cross_table の列名 rename は 0-indexed なのが気になる そもそも同じ名前の列があったら polars が自動で rename をしてくれるはずなので、rename やめてもいいかも join はそうなっているはず

rename辞めても元々の出力とカラムの順番が変わるだけで、内容としては実質同じことも確認できたのと、たとえカラム名が同じだったとしても、被った方のカラム名を weight_rightのように変更してくれることを確認したので、renameはやめます。

b7ab636

  • old
number_of_rows,0:height,0:weight,1:dominant
2,180,60,left
3,180,60,right
2,170,60,right
  • new
number_of_rows,dominant,height,weight
3,right,180,60
2,left,180,60
2,right,170,60

@tak-ka3 tak-ka3 requested a review from sakamoto-souta August 26, 2025 02:28
@tak-ka3 tak-ka3 requested a review from a team August 26, 2025 02:43
Copy link
Contributor

@KotaTakahashi9320 KotaTakahashi9320 left a comment

Choose a reason for hiding this comment

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

実行はしてないですが、コードはLGTMです!

@tak-ka3 tak-ka3 requested a review from a team August 26, 2025 04:02
@tak-ka3 tak-ka3 merged commit 2f77c95 into main Aug 26, 2025
3 checks passed
come2ry pushed a commit that referenced this pull request Sep 9, 2025
…-jupyter-basic-tutorial

[CC-883] JupyterNotebookで書かれた基本チュートリアルの修正
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.

4 participants