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

start.json #13

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

start.json #13

wants to merge 2 commits into from

Conversation

princetoday
Copy link

Basic Information = Specifies the monster's name and total health.
Default States= Defines the monster's default states, including arriving (with animations and sounds), idle (with stats like armor, attack, defense, speed, and abilities), and a dying_state.
Phased States = Details how the monster transitions through three health phases:
1.0 Phase (100% Health) = Custom arrival animations and sounds while using default idle stats.
0.5 Phase (50% Health)= Introduces new arrival animations and an additional armor value.
0.25 Phase (25% Health)= Further customizes the arrival and significantly enhances attack and armor values, indicating the monster's heightened danger.

@Chovin
Copy link
Owner

Chovin commented Oct 2, 2024

sorry, let me rephrase, the #11 issue is to make phases work in the Invasion cog. I was just giving you an example format that could work for phases during our conversation. I'm pretty sure the only thing you'd need to alter is enemy.py to get phases working. Once you do get it working, you can test that they work by using [p]provoke after setting up a test monster in invasion/data/enemies/. please do make sure you follow the format described in here though (besides your changes for phases). For example, there's no abilities field or sound field

Copy link
Owner

@Chovin Chovin left a comment

Choose a reason for hiding this comment

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

Please see previous comment. This issue isn't for creating a new monster that has phases, it's to add the capability of having enemies with phases.

Copy link
Owner

@Chovin Chovin left a comment

Choose a reason for hiding this comment

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

please fix the commented sections.
also, it's alright since it's your first contribution, but in general you should only include the changes that implement the feature/fix in your PR. so for example all the formatting changes shouldn't be included in your commits

ARRIVING_STATE = "arriving"
WIN_STATE = "dying"
LOSE_STATE = "attacking"
def new_func():
Copy link
Owner

Choose a reason for hiding this comment

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

this function isn't needed, please remove this ❤️

"Divine"
"Angry", "Enraged", "Formidable", "Rampaging", "Ferocious",
"Epic", "Mythic", "Ancient", "Colossal", "Unstoppable",
"Apocalyptic", "Eternal", "Divine"
Copy link
Owner

Choose a reason for hiding this comment

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

let's keep these each on a single line like they were before. makes them easier to read and maintain

@@ -158,7 +146,6 @@ def title_prefix(self):
px = len(self._title_prefixes) - 1

return self._title_prefixes[
# map value from between mn and mx (exclusive) to between pn and px (inclusive)
Copy link
Owner

Choose a reason for hiding this comment

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

please put this comment back

@@ -182,60 +169,70 @@ def hurt(self, player: Member, dmg_type: str, damage: int):
return
is_bomb = dmg_type == BOMB_DMG_TYPE
pid = player.id
# dmg as an arg for purchasable bombs
Copy link
Owner

Choose a reason for hiding this comment

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

please put these comments back

else:
self.bombed_by[player.id] = self.bombed_by.get(player.id, 0) + 1
return dmg

def attack(self):
Copy link
Owner

Choose a reason for hiding this comment

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

please put this function back


if health_percentage > 0.75:
Copy link
Owner

Choose a reason for hiding this comment

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

the phases should be based on the configuration done in the stats.json file for each monster. please don't hard-code a single monster's configuration into the code

@princetoday
Copy link
Author

Ok I do again today

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.

2 participants