-
Notifications
You must be signed in to change notification settings - Fork 13
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
Feat: Base Class Implemenations #17
base: dev
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
""" | ||
Build Class | ||
|
||
The build class is the top-level class encompassing all attributes and | ||
parameters defining a build. It is defined by a specific Tree and Player | ||
instance at time of evaluation. | ||
|
||
The intent is that there is only one Build for a character. There might be | ||
numerous Passive Trees (at various Player Levels, or various Cluster Jewels) | ||
associated with a Player. | ||
""" | ||
|
||
import Tree | ||
import Player | ||
|
||
|
||
class Build: | ||
def __init__(self, name: str = "temp") -> None: | ||
self.name = name | ||
self.tree = Tree.Tree() | ||
self.player = Player.Player() | ||
|
||
def __repr__(self) -> str: | ||
ret_str = f"[BUILD]: '{self.name}'\n" | ||
ret_str += f"{self.tree}" | ||
ret_str += f"{self.player}" | ||
return ret_str | ||
|
||
|
||
def test() -> None: | ||
build = Build() | ||
print(build) | ||
|
||
|
||
if __name__ == "__main__": | ||
test() |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
# Enumeration Data for Path of Exile constants | ||
|
||
from enum import Enum | ||
|
||
|
||
class PlayerClasses(Enum): | ||
SCION = 0 | ||
MARAUDER = 1 | ||
RANGER = 2 | ||
WITCH = 3 | ||
DUELIST = 4 | ||
TEMPLAR = 5 | ||
SHADOW = 6 | ||
|
||
|
||
class PlayerAscendancies(Enum): | ||
NONE = None |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
""" | ||
Player Class | ||
|
||
The Player represents an in-game character at a specific point in their | ||
leveling progress. | ||
|
||
A unique Player is defined by: level, class selection, ascendancy selection, | ||
skill selection, and itemization across the item sets. | ||
""" | ||
|
||
from Enumerations import PlayerClasses, PlayerAscendancies | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These enums are currently used in a single place. It's best to keep them as local as possible. When they are used by more than one module on the same package, put them in a separate module. When they are used by more than one package, consider reworking the packages or put them in their own package. |
||
|
||
|
||
class Player: | ||
def __init__( | ||
self, | ||
player_class: PlayerClasses = PlayerClasses.SCION, | ||
ascendancy: PlayerAscendancies = PlayerAscendancies.NONE, | ||
level: int = 1, | ||
) -> None: | ||
self.player_class = player_class | ||
self.ascendancy = ascendancy | ||
self.level = level | ||
self.stats = set() | ||
self.skills = list() | ||
self.item_sets = set() | ||
self.minions = set() | ||
|
||
def __repr__(self) -> str: | ||
ret_str = f"Level {self.level} {self.player_class.name}" | ||
ret_str += f" {self.ascendancy.value}\n" if self.ascendancy.value else "\n" | ||
return ret_str | ||
|
||
|
||
def test() -> None: | ||
player = Player() | ||
print(player) | ||
|
||
|
||
if __name__ == "__main__": | ||
test() |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
""" | ||
Tree Class | ||
|
||
This class represents an instance of the Passive Tree for a given Build. | ||
Multiple Trees can exist in a single Build (at various progress levels; | ||
at different Jewel/Cluster itemizations, etc.) | ||
|
||
A Tree is tied to a Version of the Tree as released by GGG and thus older Trees | ||
need to be supported for backwards compatibility reason. | ||
|
||
""" | ||
|
||
_VERSION_ = "3.17" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consider storing version numbers as tuples. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could you supply an example ? I'm too new to Python to understand what that means. |
||
|
||
|
||
class Tree: | ||
def __init__(self, version: str = _VERSION_) -> None: | ||
self.version = version | ||
self.tree_nodes = set() | ||
self.allocated_nodes = set() | ||
|
||
def __repr__(self) -> str: | ||
ret_str = f"[TREE]: version '{self.version}'\n" | ||
return ret_str | ||
|
||
|
||
def test() -> None: | ||
tree = Tree() | ||
print(tree) | ||
|
||
|
||
if __name__ == "__main__": | ||
test() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be a module-level docstring.