Skip to content
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

Team 4 Levels - General Levels Updates #200

Merged
merged 36 commits into from
Oct 1, 2023
Merged

Team 4 Levels - General Levels Updates #200

merged 36 commits into from
Oct 1, 2023

Conversation

max9753
Copy link
Contributor

@max9753 max9753 commented Sep 30, 2023

The pull request include several features of levels that have been created such as:

  • Levels begin with game. Spawn mobs after an interval (player gets time to setup stuff)
  • Levels end when all mobs are killed
  • Also basic service methods for UI team to implement a display for a player to see how long until mobs are spawned or a level starts etc. Also service methods for seeing how many mobs remain in a level.

samsully and others added 30 commits September 15, 2023 10:32
WaveManager
WaveService
WaveClass

Further implemtation required
Explains new functionality about spawning waves
…s remaining)

Need to set what happens when the level is completed, as game currently crashes due to index out of bounds error.
…eCount in the WaveTask.java when a new wave is started. This will be used hopefully for some UI showing the player what wave they are in.
…ves-max

# Conflicts:
#	source/core/src/main/com/csse3200/game/components/tasks/waves/WaveTask.java
#	source/core/src/main/com/csse3200/game/services/WaveService.java
bojyyy and others added 6 commits September 29, 2023 15:41
…ll be spawned in WaveService.java. Also made the wave start timestamp update in LevelWaves.java (the first time waves are made) & in WaveTask.java (For when a wave ends and a new one starts).
# Conflicts:
#	source/core/src/main/com/csse3200/game/components/tasks/waves/LevelWaves.java
Copy link
Contributor

@Mohamad11Dab Mohamad11Dab left a comment

Choose a reason for hiding this comment

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

LGTM !

Copy link
Contributor

@MiniSoda17 MiniSoda17 left a comment

Choose a reason for hiding this comment

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

Looks Good!

Copy link
Contributor

@AlasdairS4698737 AlasdairS4698737 left a comment

Choose a reason for hiding this comment

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

There appears to be no function for pausing the wave timer mid-game, only startWaveTimer() and stopWaveTimer() which appear to only work at the setup and cleanup of the stage.
I'd like for WaveTask to have some method of pausing the in-between waves timer that can I can access with the pause menu entity.

Copy link
Contributor

@shiv-0831 shiv-0831 left a comment

Choose a reason for hiding this comment

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

There are some minor javadoc errors in getNextWaveTime() and getWaveCount() methods in WaveService. Other than that, everything else looks good!

@max9753 max9753 merged commit 1a37f77 into main Oct 1, 2023
2 checks passed
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.

9 participants