Skip to content

Conversation

@agich073
Copy link
Contributor

@agich073 agich073 commented May 9, 2025

チケット: https://github.com/orgs/SSL-Roots/projects/7?pane=issue&itemId=108992680

追加したファイル

  • circular_move.py
    • 円軌道で移動するTacticを定義.
    • パラメータ
      • 中心座標 (center_x, center_y)
      • 中心座標 (x, y)
      • 半径 (radius)
      • 回転時間 (seconds)
      • 回転方向(cw→True: 時計回り, False: 半時計回り)
  • circular_plays.py
    • 円軌道で移動するPlayを定義.
  • playbook_circular.py
    • circular_plays.pyを動かすためのplaybook
  • composite_ball_centric_mover.py
    • CircularMoveを用いたボール中心に円運動するTactic.

Copy link
Contributor

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

動作OKです!
Lintで失敗しているので修正お願いします。 🙏

StopとAdjustPositionは使われてないので削除すれば治るはず。
https://github.com/SSL-Roots/consai_ros2/actions/runs/14925175393/job/41928323527?pr=488

  >>>
build/consai_game/test_results/consai_game/flake8.xunit.xml: 2 tests, 0 errors, 2 failures, 0 skipped
- consai_game.flake8 F401 (./consai_game/tactic/circular_move.py:21:1)
  <<< failure message
    'consai_game.utils.adjust_position_to_field.AdjustPosition' imported but unused:
    from consai_game.utils.adjust_position_to_field import AdjustPosition
  >>>
- consai_game.flake8 F401 (./consai_game/play/factory/circular_plays.py:24:1)
  <<< failure message
    'consai_game.tactic.stop.Stop' imported but unused:
    from consai_game.tactic.stop import Stop
  >>>

@agich073 agich073 requested review from ShotaAk, Yamam0t0, nabekuro0104, tilt-silvie and uchikun2493 and removed request for tilt-silvie May 15, 2025 09:32
@agich073
Copy link
Contributor Author

Lintのエラーを修正しました
原点中心からボール中心に回るtacticに変更、追加したファイルについて追記しました

# limitations under the License.

"""
条件に応じてディフェンス動作やキックやパスを切り替えるTactic
Copy link
Contributor

@ShotaAk ShotaAk May 15, 2025

Choose a reason for hiding this comment

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

コメントの修正お願いします!

Suggested change
条件に応じてディフェンス動作やキックやパスを切り替えるTactic
ボールを中心に回転するTactic

from consai_msgs.msg import MotionCommand


class BallCentricMover(TacticBase):
Copy link
Contributor

Choose a reason for hiding this comment

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

composite_ball_centric_mover.pyと同じ・・・?
多分、ファイルの消し忘れだと思います!

Copy link
Contributor

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

動きはOKです!

コメントの修正とファイルの削除(もしくは修正)をお願いします!

Copy link
Contributor

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

LGTM!!

@ShotaAk ShotaAk merged commit 16d88bf into main May 16, 2025
13 checks passed
@ShotaAk ShotaAk deleted the add_circular_move branch May 16, 2025 09:18
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.

3 participants