Skip to content

Pandas入門の解答 #5

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Pandas入門の解答 #5

wants to merge 3 commits into from

Conversation

xiaoyouyou3
Copy link
Owner

No description provided.

Copy link

@09-sturnus 09-sturnus left a comment

Choose a reason for hiding this comment

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

コンフリクトが起きています。
このままではマージできないので修正をお願いします。

🔴の部分は修正してください。
🟢の部分は修正は必須ではありませんが、見て下さると幸いです。
確認お願いします。

問3
🟢
これでも大丈夫ですが、len()を使った方がはるかに高速になります。

問5
🔴
これでは平均より売り上げの多いレシートが出力されてしまっています。

問6
🔴
df_customer_only_womenは顧客ごとにデータが並んでいるのに対して、df_receiptは決済ごとにデータが並んでいます。これらを直接マージするのはよくないかと思います

問7
🔴
2015/09/05のデータが、1970/01/01 00:00:00.020150905に変換されてしまっています

問11
🔴
四捨五入が実装されていません。丸め込みなどで検索すれば出てくるかと思います。

Copy link

@09-sturnus 09-sturnus left a comment

Choose a reason for hiding this comment

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

レビューしました!
確認お願いします。

問6
🔴
1回目のgroupbyで顧客ごとにデータがまとめられているので2回目のgroupbyの行は必要ないかなとおもいました。

問11
🔴
欠損値の穴埋めの処理は適切ですが、欠損値がないことの確認は不適切です(fillnaでは列数は変わらない)。

Copy link

@09-sturnus 09-sturnus left a comment

Choose a reason for hiding this comment

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

レビューしました。
問に関してはLGTMです。
コンフリクトを解消してから、Slackの方で一報ください。

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