Creating an NRaas project Potentially new Woohooer module

Talk nerdy to us here.
Post Reply
icarus_allsorts
Reactions:
Posts: 213
Joined: March 8th, 2012, 6:00 pm

Potentially new Woohooer module

Post by icarus_allsorts » August 14th, 2014, 1:53 am

Woah, it's been a while since I've posted here. Anyways, I was playing around with Woohooer and sort of put together a (seemingly) working module for a "new" woohoo location... wondering if it's okay to upload here for any curious guine.. err, testers out there?

Also while testing around I found that the GetRandomObject method in CommonWoohoo.PushWoohoo doesn't ignore objects that aren't in world (stumbled upon this when my test sims decided to do it in an invisible photo booth at the student union that only appears during orientation). I'm guessing the best (though most tedious) way to rectify this is to tweak TestUse methods in each LocationControl since doing it to the GetRandomObject method looks needlessly expensive in comparison. Wondering if there's a better fix though...

User avatar
Chain_Reaction
Site Admin
Reactions:
Posts: 7595
Joined: December 30th, 2011, 6:00 pm
Answers: 81
Contact:

Post by Chain_Reaction » August 14th, 2014, 4:35 pm

The source of the base mod is a bit behind as I'm still working out the kinks of those versions. I can't commit right this second as it's full of debugging code unfortunately. But if your module doesn't require changes to the base mod it should be okay to go for it. :)

I concur it looks like the only way to sort that is to change each TestUse method unless one loops through the list and checks them there. That does seem unnecessarily expensive and lazy which this game doesn't need more of. I'll change each one. :)

icarus_allsorts
Reactions:
Posts: 213
Joined: March 8th, 2012, 6:00 pm

Post by icarus_allsorts » August 15th, 2014, 12:46 am

Thanks Chain, I was gonna offer to change the TestUse methods since I had to change the WoohooLocation enums anyways, but I guess my changes will have to wait. I think the module should work as is if I disable the new LocationControl I made for it, if that works out I'll upload it for testing soon :)

User avatar
Chain_Reaction
Site Admin
Reactions:
Posts: 7595
Joined: December 30th, 2011, 6:00 pm
Answers: 81
Contact:

Post by Chain_Reaction » August 16th, 2014, 9:38 am

I have committed woohooer 127 to the source so feel free to build a new base mod if you need to :)

icarus_allsorts
Reactions:
Posts: 213
Joined: March 8th, 2012, 6:00 pm

Post by icarus_allsorts » August 16th, 2014, 11:30 am

Oh goodie :) Guess I'll upload a v127b as well then, unless you have plans to release one sometime soon?

User avatar
Chain_Reaction
Site Admin
Reactions:
Posts: 7595
Joined: December 30th, 2011, 6:00 pm
Answers: 81
Contact:

Post by Chain_Reaction » August 16th, 2014, 7:02 pm

Glad you went ahead with release as no immediate plans here. Looks great!

Post Reply