Conversation
hyphen-o
left a comment
hyphen-o
left a comment
There was a problem hiding this comment.
何個かコメントした！
ちゃんとプロジェクトの形に合わせてReadme変更してくれたり，prjmanにコメント追加してくれたりしてくれてるのは見やすくてめちゃくちゃありがたい！
Scratchのデータ構造理解するの難しいと思うけど，コメント内容踏まえて修正頑張ってほしい！
scratcher/app/sample/sample.py
Outdated
|
|
||
| # ブロック数を取得 | ||
| project_manager = ProjectManager(sample_id) | ||
| blocks_length = project_manager.get_blocks_length() |
There was a problem hiding this comment.
全部のブロックを取得するのはget_all_blocks_length()の方で取れるよ〜
scratcher/app/sample/sample.py
Outdated
| project_manager = ProjectManager(sample_id) | ||
| blocks_length = project_manager.get_blocks_length() | ||
| # ブロックを取得 | ||
| blockType = project_manager.get_blocks() |
There was a problem hiding this comment.
現状get_blocksは動かないからコメントアウトした方がいいのと，__blocksはブロックの種類を格納してるわけじゃ無いから新しくブロックの種類数をカウントするような実装が必要になりそう
scratcher/app/sample/sample.py
Outdated
| blocks_length = project_manager.get_blocks_length() | ||
| # ブロックを取得 | ||
| blockType = project_manager.get_blocks() | ||
| # スプライト数を取得 "isStage"の数がスプライト数？1つはステージなので-1する |
There was a problem hiding this comment.
そうやね，isStageの数がスプライトの数を表してる（厳密にはisStageを格納してる配列の数を数えてあげればいいだけ）のはあってる！
ただ，下にも書いてあるけど，現状の実装ではそれが実現できてなさそうやから答えになっちゃうけどself.__spritesの配列の長さをとってあげれば実現できると思う
scratcher/app/sample/sample.py
Outdated
| # スプライト数を取得 "isStage"の数がスプライト数？1つはステージなので-1する | ||
| sprits_length = project_manager.get_sprites_length() | ||
| # CTスコア合計点数を取得 | ||
| mastery = drscratch_analyzer.Mastery(sample_id) |
There was a problem hiding this comment.
drscratch_analyzerは分析先のprocess関数にファイル（作品のJSONファイル）のパスを渡して実行，analyze関数に出力先のJSONファイルのパスを渡して実行すると分析結果がファイルとして出力されるようになってるから，コンストラクタにIDを渡す必要はなくて，下記の実装にすると実行できると思う（パスは適宜変えてほしい）
mastery = Mastery()
mastery.process("../271001536.json")
mastery.analyze("./out.json")| self.__ID = id | ||
| self.__project = scratch_client.get_project(self.__ID) | ||
| self.__head_blocks = self.__project["targets"][1]["blocks"] | ||
| self.__start_splites = self.__project["targets"][0]["isStage"] |
There was a problem hiding this comment.
ここは先頭のスプライトがステージかどうかのbooleanしか格納できてないと思う！
|
Scratch3.0以降の作品を入力してください． このエラーの対処法わかりますでしょうか．．． |
縦軸名を正しく変更
各項目のup,down,uncの総数出力する
ヒストグラムを出力するように変更
2 participants
No description provided.