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

Create Archetype Starter Gear #20

Closed
whitenoiseoss opened this issue Jul 5, 2015 · 12 comments
Closed

Create Archetype Starter Gear #20

whitenoiseoss opened this issue Jul 5, 2015 · 12 comments

Comments

@whitenoiseoss
Copy link
Contributor

Write and create Starter Gear for each Archetype. This can be a joint effort between a builder and a coder.

On the builder side, what is needed:
Item short and long descriptions (i.e., "This breastplate is made of a fine..." and "a fine, leather breastplate")
What slots the item will take up
Any effects of the item (is it cursed? does it boost a stat? does it give health regen?)

On the coder side:
Create a Typeclass for each abstracted type of item (a Breastplate typeclass for example, from Armor, that is coded to only take up a 'torso' slot)--then all things that only take up the Torso can be Breastplates.
Add to core typeclasses if necessary for decided item effects
Write Spawner/Prototype code to generate these items

@stefanludlow
Copy link
Contributor

@Nadorrano
Copy link
Contributor

The equip handler is ok with hosting objects that take up more than one slot, however commands have to be reworked a little bit to do that.

Edit: now it is supposed one slot one item, we can use Feet for boots so we don't lose the possibility to have different earrings, rings etc. These items should have slot = ['left ear', 'right ear'], with the meaning "take the first empty slot among these"

@whitenoiseoss
Copy link
Contributor Author

@kaine904 that is a good start yes

@Nadorrano
Copy link
Contributor

@ergodicbreak @whitenoiseoss

Would you like this approach:

slots = ('head', 'torso', 'feet', 'legs', 'blah', 'blah')
limbs = {'head': 'head'
              'left foot': 'feet',
              'right foot': 'feet'
              'etc': 'etc'}

It will allow to print pretty much every possible combination of limbs/slots vs items

@ergodicbreak
Copy link
Contributor

@Nadorrano @whitenoiseoss I'm going to split this slots/limbs discussion off into a new issue at #24.

@ergodicbreak
Copy link
Contributor

Per #24, we'll have slots and limbs where limbs basically are hittable slots. Eq can take up multiple slots. I think we may want a max number on slots, so for example you could wear multiple necklaces.

@ergodicbreak
Copy link
Contributor

@kaine904 would you like to continue this as well?

@stefanludlow
Copy link
Contributor

Let's touch base on general status.

Sent from my iPhone

On Aug 2, 2015, at 3:54 PM, George Oliver notifications@github.com wrote:

@kaine904 would you like to continue this as well?


Reply to this email directly or view it on GitHub.

@feend78
Copy link
Member

feend78 commented Nov 6, 2015

Hey, I'm interested in helping out with Ainneve, and have been looking at this issue and reading the OA manuals over the past couple days. Is anyone still actively working on this?

Between the information documented in this issue, the associated EquipHandler and slots discussion referenced above, the wiki, and the OA manuals, I think I could make a stab at it.

@ergodicbreak
Copy link
Contributor

@feend78 there hasn't been much Ainneve development lately, but I and probably @whitenoiseoss are available to work with you on this. We would definitely appreciate your contribution.

@Nadorrano
Copy link
Contributor

Hi, happy to see that someone is still partecipating in this project! If you are working on this, @feend78, I have a request. That slot_operator attribute on items is ugly (is my code), it could be better refactored to self.db.multi_slot = True/False or similar. Then line 188 of equip.py should be changed to:

if obj.db.multi_slot:

Would you make these changes for me?
thx and good coding

Edit: only if you think it's a better solution, of course.

@feend78
Copy link
Member

feend78 commented Nov 8, 2015

Hi @Nadorrano, I like that solution, and have changed it and updated the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants