Redwall: Warlords

Discussion => Turbo Discussion => Topic started by: The Lady Shael on July 23, 2014, 11:32:02 PM

Title: Watch for broken stuff
Post by: The Lady Shael on July 23, 2014, 11:32:02 PM
I'm doing some major code refactoring in the RWL baseline (it's over 12 years old...it's way overdue). It's going pretty okay but I'm doing it little by little to make sure stuff doesn't get horribly broken.

Speaking of horribly broken, I just refactored a file that could cause some horrible breakage (the file that handles actions on a timer, like giving out turns, updating networth, emp tracking, etc). It's a lot of stuff and I did some preliminary testing to make sure that the file completes, but it'll take me a while to verify that everything being done was successful. So I will need everyone's help with this. If you see anything that is broken, please let me know immediately.

Thanks for your patience. =)
Title: Re: Watch for broken stuff
Post by: Shadow on July 24, 2014, 06:39:50 AM
thanks shael!

I'm back now and I'm always more motivated to write code when others are, so if I can help let me know.
Title: Re: Watch for broken stuff
Post by: Gen. Volkov on July 24, 2014, 12:56:20 PM
So yeah, Turbo is horribly broken. It still shows me as logged on, and I haven't gotten turns since I ran this morning, around 3 AM.
Title: Re: Watch for broken stuff
Post by: The Lady Shael on July 24, 2014, 07:28:14 PM
Okay, I think I fixed it. Turns are coming now, at least. Let me know if anything else seems weird.
Title: Re: Watch for broken stuff
Post by: Gen. Volkov on July 25, 2014, 07:42:17 AM
Will do.
Title: Re: Watch for broken stuff
Post by: Gen. Volkov on July 25, 2014, 02:04:33 PM
It seems kind of random when I get turns. I was at full, so I ran, and by the time I was done, 10 minutes had gone by and I got turns, and then I waited another 10 minutes, and I didn't get turns. Also, I can't sell anything on mercs.
Title: Re: Watch for broken stuff
Post by: Gen. Volkov on July 27, 2014, 10:15:11 AM
Still can't sell anything on mercs. Have full turns though.
Title: Re: Watch for broken stuff
Post by: Vulpes Jarr on July 28, 2014, 08:22:57 AM
YAY!
Title: Re: Watch for broken stuff
Post by: Gen. Volkov on August 01, 2014, 09:34:06 AM
Mercs still are not working. No new troops to buy are being generated, and you can't sell troops again once you've sold them.
Title: Re: Watch for broken stuff
Post by: Wolf Snare on August 02, 2014, 11:53:21 AM
Kyle do you have any idea what could be causing this mercs issue? It's got me at a standstill in game and I don't think Shael's been on the last few days. Please poke around if you feel comfortable. Thanks!
Title: Re: Watch for broken stuff
Post by: Shadow on August 02, 2014, 03:25:11 PM
I know where in the code that particular line is, but I know Shael's doing some reworking of the code-base and moving to a version control system. I don't know the details fully yet so I'm hesitant to mess with it before she says she's done.

I can try to figure out the new system this evening and see if I can find it, though. I'll have to learn it eventually anyway ^_^.

Update: I have the verson control system downloaded and the code in place. I will test some small change to see if I break anything and I will see if I can find out my mercs is broken when I'm confident I won't break things worse.

update 2: shael has been very busy - the turns script is now using an object oriented approach which I am unfamiliar with. It will take me a little while to parse through it. Bear with me. At first glance I can't see why it's not working, I can't see anything obviously out of place...

That being said, the new code looks beautiful and shael should be thanked profusely for doing it. Holy hell.

Update 3: found the problem, no idea how to fix it. Will try things at random. Stay tuned.
Title: Re: Watch for broken stuff
Post by: Shadow on August 03, 2014, 11:16:39 AM
The bug should be fixed. Please confirm.
Title: Re: Watch for broken stuff
Post by: Gen. Volkov on August 03, 2014, 08:58:04 PM
It appears to be fixed, thanks Shadow!

Also, thanks Shael for updating the code!
Title: Re: Watch for broken stuff
Post by: Wolf Snare on August 05, 2014, 11:28:27 PM
Thanks guys! The landfarm hasn't attempted scouting in the last 24 hours at least, can someone look into it please?
Title: Re: Watch for broken stuff
Post by: Wolf Snare on August 06, 2014, 08:09:01 PM
I sent him resources to get his butt moving again but still no sign of life...
Title: Re: Watch for broken stuff
Post by: Shadow on August 06, 2014, 09:49:01 PM
might have gotten dropped in the turns script rewrite, I'll poke it tomorrow.
Title: Re: Watch for broken stuff
Post by: Wolf Snare on August 06, 2014, 11:16:31 PM
Thanks! Renting skiffs is no longer possible:

Sending aid...
Renting skiffs...
Something Has Occurred
You must send at least 3490 Skiffs!
Title: Re: Watch for broken stuff
Post by: Shadow on August 07, 2014, 06:54:17 PM
made the land farm scout every 10 minutes instead of every hour. It worked once then turns stopped happening. Consider me confused...

I think I reverted to what it was before, please let me know if turns have stopped for no apparent reason.