Skip to content

Change attacks to use damage and energy requirements from database#23

Merged
apmnt merged 2 commits intoapmnt:mainfrom
KFSPC8:main
Oct 27, 2025
Merged

Change attacks to use damage and energy requirements from database#23
apmnt merged 2 commits intoapmnt:mainfrom
KFSPC8:main

Conversation

@KFSPC8
Copy link
Contributor

@KFSPC8 KFSPC8 commented Oct 26, 2025

A huge step towards being fully compatible with the database file from deckgym. I think the only remaining field is the "is_ex" field. I originally wanted to complete this feature before creating this PR but since I'll be away the entire of next week, I'm making this PR now.

The original attacks are still in attack_common.py and should probably be cleaned-up since all the data will eventually be sourced from the database.json file.

@apmnt apmnt merged commit df75b6a into apmnt:main Oct 27, 2025
1 check passed
@apmnt
Copy link
Owner

apmnt commented Oct 27, 2025

Thanks for the work, looks good. Those shared references have definitely given me trouble before

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