-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Preservation of story nuggets #51
Comments
Fair enough. I think we have some time on this one - I hadn't been planning to remove them immediately. I don't know enough to have an opinion here. I think @ChristopherA would be the one to convince. |
You did however commit them to the deprecation list which is how I found out about this. Thing is if they still work I'd like them kept around as a second option. If anything I think they just need to be better documented. |
At the moment I'm taking documentation from an external site, written by other folks, and committing it where I think it goes. Similarly, the information on Regency clothing that I dropped into the gables_game repo wasn't written by me -- I'm just putting stuff that currently isn't in the repos, into the repos. |
My hope is that a lot of the disparate external sources (e.g. the Builders section of the TWiki) can be merged into a central location, which will let us figure out what's correct, what's out of date and what can just go in terms of documentation. The deprecation list came from HackMD. |
The problem is that StoryNuggets doesn’t work in stock Gables. There is some specific code in Marrach that must not exist in our codebase. If you want to write a new SkotOS plugin to make that feature available in the future, that would be great. However anything that is not functional should be removed from base SkotOS. In addition almost everything not necessary for almost all games or a makes for great demo should be converted to optional plugins. Either way StoryNuggets needs to be deprecated from base SkotOS. We have backups in our repos now should someone want to try to make it work in the future. |
That's surprising to me ChrisA, I was under the impression that StoryNuggets worked just fine in that character creation page that let you redo your attributes, unless I'm horribly confused. Is it possible that it was one of the things that got broken by the cold boot process and only worked during a live instance of Gables? |
Any idea if it works on the current gables.chattheatre.com? That isn't the cold-boot code (yet, we'll get there). Though it's pretty different code in general. |
I just remember using the character creation page to reset my attributes (eye color, hair, nose, etc) and I was sure that it used StoryNuggets |
It does look like the current character editor uses story nuggets in some capacity: https://github.com/ChatTheatre/SkotOS/blob/master/skoot/data/vault/Theatre/Theatres/Tavern.xml |
We should ask @saraht45 — I don’t believe it works with current start story. |
I personally don't use story nuggets as I feel it is too limited. The current Gables start story that I wrote does not use it, but that's not to say there aren't other references to it floating around. I know there is an "edit character" page but I think the idea is to use +traits/@traits to modify chatter bodies. My only concern would be if it's hardwired into SkotOS then removing it might break something on a lower level. My vote is to get rid of it. As Chris said, it wasn't working right on Gables anyway. |
If Sarah says it wasn't working right on Gables, that means it's also not working right on prod Gables - so that means she's not referring to a cold-boot or CTSkotOS problem, since that's not the codebase she's using. |
If I'm not mistaken this XML dump cites storynuggets as a component of the theatre object if the tags are anything to go by, since the tag types reference the SIDs being used to parse the xml. |
So for the moment this does imply that existing code would indeed be broken if we got rid of it. I'm not really clean to get delete happy with the code unless it's broken in a way that can't be avoided. |
You're correct -- if we were going to remove story nuggets, that would require first changing the start story in CTSkotOS, which uses them. |
I also would opine that theatres are important in their own right for thematic reasons. I'm just not really feeling comfortable just gutting stuff, especially if things change later that make it hard to reintegrate it later once it's fixed. |
There is nothing gutted. It all exists in GitHub archival repos now, and can be easily brought back into a SkotOS instance if needed. But it if isn't needed for root SkotOS repo, it shouldn't be there — it should either be in its own archival repo, or if important enough, in a repo that supports @noahgibbs new plugin architecture so that it can be easily added to a SkotOS instance. |
I would like to lobby for the preservation of storynuggets as a character generation interface, I think it was used on marrach if I'm not mistaken unless I misread something, and if they still work I don't think they should be removed.
Though by contrast I agree bilbo needs to go.
The text was updated successfully, but these errors were encountered: