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

Add ptAvatar.findBone method #972

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

dpogue
Copy link
Member

@dpogue dpogue commented Jul 25, 2021

This allows getting an avatar's bones using Python, which enables hackery with PtAttachObject to them

@dpogue dpogue added Enhancement Non-critical feature addition Python Issues stemming from the Python API files labels Jul 25, 2021
@Hoikas
Copy link
Member

Hoikas commented Jul 25, 2021

I think we need to spend some time designing a better system. PtAttachObject has no persistence and doesn't seem like the best method IMO.

@dpogue
Copy link
Member Author

dpogue commented Jul 25, 2021

"No persistence" meaning there's no state to be synched to newly joined players? hmm...

A visual issue like that wouldn't be a barrier for my particular use case, but does seem like something worth addressing.

@Hoikas
Copy link
Member

Hoikas commented Jul 25, 2021

Correct. To my mind, I think it would be better to have a more formalized inventory system for attaching objects to the avatar (this system probably involves avatar.sdl) with the ability to anchor objects to the left hip or off-hand built-in, for example. That way, we discourage bespoke solutions in each Age that results in an increased maintenance burden.

@dpogue
Copy link
Member Author

dpogue commented Jul 25, 2021

I pulled these comments into a discussion #974 so we can come up with a plan before doing any further implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Non-critical feature addition Python Issues stemming from the Python API files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants