Skip to content

Feature/67 make map size available for strategy - #73

Merged
DivineThreepwood merged 10 commits into
devfrom
feature/67-make-map-size-available-for-strategy
Sep 10, 2026
Merged

Feature/67 make map size available for strategy#73
DivineThreepwood merged 10 commits into
devfrom
feature/67-make-map-size-available-for-strategy

Conversation

@DivineThreepwood

Copy link
Copy Markdown
Member

📜 Description

Changes proposed in this pull request:

  • Introduce level size.
  • Implement proper tower interface.
  • Make tower available at top level of the strategy.
  • Introduce Unknown tower type and make type not nullable.
  • Implement scan action and make it available for strategy
  • Implement scan animation.
Screenshot 2026-07-07 at 23 58 18

@DivineThreepwood DivineThreepwood linked an issue Jul 7, 2026 that may be closed by this pull request

@tommy33790 tommy33790 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In addition to the comments / questions, we should check for @Deprecated methods for the tower (i.e. AgentLegacyInterface.seeTower()).
Don't know if such polishing should go into a seperate issue / PR, @DivineThreepwood ?

Yeah, it took a while to review...

Comment thread src/main/kotlin/org/openbase/planetsudo/level/levelobjects/TowerInterface.kt Outdated
Comment thread src/main/kotlin/org/openbase/planetsudo/view/level/levelobjects/TowerTopPanel.kt Outdated

@tommy33790 tommy33790 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looking excellent

@DivineThreepwood
DivineThreepwood merged commit f044dc1 into dev Sep 10, 2026
2 checks passed
@DivineThreepwood
DivineThreepwood deleted the feature/67-make-map-size-available-for-strategy branch September 10, 2026 21:19
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.

Make map size available for strategy

2 participants