Skip to content

化石ポケモンの色違い厳選(画像認識)#10

Merged
KawaSwitch merged 14 commits intoKawaSwitch:masterfrom
Moi-poke:new_commands
Jan 24, 2020
Merged

化石ポケモンの色違い厳選(画像認識)#10
KawaSwitch merged 14 commits intoKawaSwitch:masterfrom
Moi-poke:new_commands

Conversation

@Moi-poke
Copy link
Copy Markdown
Contributor

@Moi-poke Moi-poke commented Jan 22, 2020

いつもお世話になります。早速ですが概要です。

化石ポケモンの色違い厳選コマンド

画像認識必須
前準備として以下の状態でレポートを書いた上でスタートする

  • 厳選したい化石ポケモンのカセキを最低30個、残り全種のカセキを最低1個所持していること
  • カセキ復元のウカッツの目の前にいる状態
    2020-01-23_02-39-31
  • 手持ちが6体全て埋まっていて、ボックスを開いたとき、空ボックスであること
    また、プラスボタンで技構成のあるステータス画面を表示するようにしておく
    2020-01-23_03-52-35
  • メニューを開いたときのカーソル位置とポケモンの位置が写真の位置にあること
    2020-01-23_02-09-37
    これは自分がアイコン位置を変更していること。また、厳選の過程でポケモンの再起動処理が入る際にカーソル位置が左上にリセットされるようであるため。
    デフォルトのアイコン位置を忘れてしまったので、デフォルトのアイコン位置に合わせてコマンドを修正したほうが良いかもしれません。

動作ですが、

1ボックス分化石ポケモン受け取る
ボックスを開いて色違いがいるかどうか確認し、居なければリセット。居たら終了。

を繰り返します。(1ループ約10分です)
逃がす処理は行わないので、終了時はこんな感じです。

手元では動いたので動くとは思うのですが、一旦draftです。
プログラム初心者なので若干変なコマンドの実装になっていると思います。

@Moi-poke
無限羽回収を追加
2305564 
ランクマバグを使用した無限羽回収
5番道路の預け屋前からスタート
メニューを開いた際にマップが選択されている必要がある
@Moi-poke
無限羽回収コマンド修正
f5ec2b2 
古いコマンドを使用していたので修正
checkIfAliveを追加
@Moi-poke
Merge branch 'master' of https://github.com/KawaSwitch/Poke-Controller …
55a7f8b 
…into new_commands
@Moi-poke
カセキ色厳選
cf2f09a 
各種カセキポケモンの色違い厳選コマンドを追加
雑めの実装になっているのでどうにかしたい
@Moi-poke
add OP.png
85cb70f 
上げ忘れ
@Moi-poke
まだ手元で確認中の関数を使っていたので修正
ff6bf5a
@KawaSwitch KawaSwitch self-requested a review January 23, 2020 11:36
KawaSwitch
KawaSwitch reviewed Jan 23, 2020
View reviewed changes
KawaSwitch
KawaSwitch reviewed Jan 23, 2020
View reviewed changes
KawaSwitch
KawaSwitch approved these changes Jan 23, 2020
View reviewed changes
Copy link
Copy Markdown
Owner

@KawaSwitch KawaSwitch left a comment

Choose a reason for hiding this comment

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

細かいですが継承しているのでメンバメソッドの書き方へ
4匹の選び方は後から実装予定のコマンドオプション機能等(まだGUIの概要は決まっていませんが...)で1コマンドに統一するかもしれませんが, 現状はこの実装で大丈夫です
fossil_loopメソッドの実装助かります

@KawaSwitch
Copy link
Copy Markdown
Owner

KawaSwitch commented Jan 23, 2020

これは自分がアイコン位置を変更していること。また、厳選の過程でポケモンの再起動処理が入る際にカーソル位置が左上にリセットされるようであるため。
デフォルトのアイコン位置を忘れてしまったので、デフォルトのアイコン位置に合わせてコマンドを修正したほうが良いかもしれません。

デフォルトでも"ポケモン"の位置は上段の左から2番目なので特に問題は無いかと思います

Moi-poke and others added 6 commits January 24, 2020 00:44
@Moi-poke @KawaSwitch
メンバメソッドの修正
d78db22 
Co-Authored-By: KawaSwitch <41296626+KawaSwitch@users.noreply.github.com>
@Moi-poke @KawaSwitch
メンバメソッドの修正
056f76d 
Co-Authored-By: KawaSwitch <41296626+KawaSwitch@users.noreply.github.com>
@Moi-poke @KawaSwitch
メンバメソッドの修正
4ae05fd 
Co-Authored-By: KawaSwitch <41296626+KawaSwitch@users.noreply.github.com>
@Moi-poke @KawaSwitch
メンバメソッドの修正
8afe2c3 
Co-Authored-By: KawaSwitch <41296626+KawaSwitch@users.noreply.github.com>
@Moi-poke @KawaSwitch
色カセキコマンド修正
0ea04ec 
OP認識後のコマンド修正

Co-Authored-By: KawaSwitch <41296626+KawaSwitch@users.noreply.github.com>
@Moi-poke @KawaSwitch
色カセキ厳選コマンド修正
cc1afd6 
ラグ対策でwait修正

Co-Authored-By: KawaSwitch <41296626+KawaSwitch@users.noreply.github.com>
KawaSwitch
KawaSwitch reviewed Jan 23, 2020
View reviewed changes
@Moi-poke
細々と色カセキ厳選修正
ffcf26e 
過剰なB連打をひかえめした
コメントアウト部分を修正
ループのインクリメント位置をループの最後に。それに合わせてprintも修正
@Moi-poke
Merge branch 'new_commands' of https://github.com/Moi-poke/Poke-Contr…
1e78a9f 
…oller into new_commands
@Moi-poke
Copy link
Copy Markdown
Contributor Author

Moi-poke commented Jan 24, 2020

返信遅くなりすみません。
レビューありがとうございました。ラグの調整や、特にメソッドの書き方など勉強になります。
Network...pngの画像認識後コマンドが入るタイミングが早すぎるとコマンドを認識しないことがあったので、waitを追加しています。

@Moi-poke Moi-poke marked this pull request as ready for review January 24, 2020 09:46
@KawaSwitch KawaSwitch merged commit 55d66f8 into KawaSwitch:master Jan 24, 2020
@KawaSwitch KawaSwitch added the new command New command added label Jan 24, 2020
@Moi-poke Moi-poke changed the title 【wip】化石ポケモンの色違い厳選(画像認識) 化石ポケモンの色違い厳選(画像認識) Jan 24, 2020
@KawaSwitch
Copy link
Copy Markdown
Owner

KawaSwitch commented Jan 24, 2020

お疲れ様です。いつもありがとうございます。
化石ポケモンの色違い厳選実装の提供、大変ありがとうございます！

画像認識後のwaitの件について了解いたしました。
こちらの環境でも動作を確認いたしましたのでマージさせて頂きました。
後ほどWikiの方へ説明を追加いたします。

@xv13 xv13 mentioned this pull request Feb 10, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KawaSwitch KawaSwitch KawaSwitch approved these changes

Assignees

No one assigned

Labels

new command New command added

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Moi-poke @KawaSwitch