PDA

View Full Version : [Update] SRL-OSR: October 2nd, 2013 ~ Smart 8.3 & Simba 1.00 Support



Olly
10-02-2013, 09:38 PM
Hello!

After months, and months SRL-OSR will now run with the SMART packaged with Simba (v8.3), with that being said SRL-OSR has also received some updates so it can now correctly run with Simba 1.00 +.

All this means you should be able to download (or update) Simba the latest version, simply go to the SRL-OSR repo (https://github.com/SRL/SRL-OSR) and place it in your includes (re naming the folder to srl-osr) and all (updated) scripts should work. :)

Everything should work correctly, but if you find any bugs feel free to report them here.

If your script used any smart functions they will be to be changed to be compatible with 8.3, pretty much every first parameter is "target" which is our current smart target, some examples:


SmartSetDebug(true) > smartSetDebug(smartCurrentTarget, true)
SmartGetFieldInt(10, 20) > SmartGetFieldInt(smartCurrentTarget, 10, 20))



Olly (1):
Updated to only run on SMART 8.3 (packaged with Simba)

JohnPeel (1):
Simba 1.0 Support



Commits (https://github.com/SRL/SRL-OSR/commits/master)

YoHoJo
10-02-2013, 09:44 PM
Wow awesome!
So basically this update is telling us that SRL-OSR at https://github.com/SRL/SRL-OSR not works with the latest SMART/Simba?!

So just DL the zip, rename from SRL-OSR Master to just SRL-OSR and plop it into Includes exactly like it was before huh?

What does this mean about all of the current OSR scripts? Do they work out of the box, or will scripters need to update/tweak their scripts to work with the new stuff?

It seems like you and JohnPeel single handedly (maybe with some help?) converted this all?
Great job guys!!!

Olly
10-03-2013, 12:02 AM
Yea it should all work like you said, scripts shouldn't need updating maybe some smart functions if they use them (ie smartSetDebug(true) > smartSetDebug(smartCurrentTarget, true);

Flight
10-03-2013, 01:20 AM
Ouch...

YoHoJo
10-03-2013, 01:41 AM
Ouch...
Ouch because you did a lot too? :-)

Ashaman88
10-03-2013, 01:57 AM
Thanks Olly :)

Flight
10-03-2013, 02:26 AM
Ouch because you did a lot too? :-)

In this particular release, none of it. But I did indeed have a working, updated SRL-OSR released a while back; unfortunately it didn't receive much attention.

YoHoJo
10-03-2013, 02:33 AM
In this particular release, none of it. But I did indeed have a working, updated SRL-OSR released a while back; unfortunately it didn't receive much attention.

I guess the best way to go about it is work with the actual GitHub repo.
Don't worry, your work around here is appreciated and you know it!

Flight
10-03-2013, 03:27 AM
I guess the best way to go about it is work with the actual GitHub repo.
Don't worry, your work around here is appreciated and you know it!

I guess that's the truth.

@OP: There's pairing issues with SMART in this include. If you load up SMART with one script that SMART will only work with that script, any other script will fail to pair with your existing SMART. I even tried reloading Simba, opening the same script, and running on an existing (free) SMART client; Simba says it successfully paired but just instantly terminates.

Why didn't you guys adopt Brandon's style of handling SMART? It's done right, in the correct Lape format, and it's much more organized.

Olly
10-03-2013, 10:43 AM
I guess that's the truth.

@OP: There's pairing issues with SMART in this include. If you load up SMART with one script that SMART will only work with that script, any other script will fail to pair with your existing SMART. I even tried reloading Simba, opening the same script, and running on an existing (free) SMART client; Simba says it successfully paired but just instantly terminates.

Why didn't you guys adopt Brandon's style of handling SMART? It's done right, in the correct Lape format, and it's much more organized.

I only did this as a somewhat temporary fix to keep the include working in pascalscript while we figure something out for a new include. I do respect your work you did but that requires lape, I thought it would be better for now to get srl-osr working with newest simba and such, in pascalscript. :)

e: I dont get your problem with smart only pairing/working for one script :s

Hoodz
10-03-2013, 12:28 PM
Poor flight :(

Olly
10-03-2013, 03:56 PM
Lol, I didn't mean to annoy anyone, but the pure fact is flight's was lape and srl-osr is currently supported by pascal script.

Coh3n
10-03-2013, 04:04 PM
Why didn't you guys adopt Brandon's style of handling SMART? It's done right, in the correct Lape format, and it's much more organized.You mean his smart.simba? We had our new one written ages before his, and ours worked, so there was no point in switching. That being said, it was never really discussed or brought up until now.

Brandon
10-03-2013, 10:19 PM
You mean his smart.simba? We had our new one written ages before his, and ours worked, so there was no point in switching. That being said, it was never really discussed or brought up until now.


No sire Coh3n, that is Impossibru! http://villavu.com/forum/showthread.php?t=105548&p=1262830#post1262830

I'm always the very first to write anything Smart related; I stalk Benland and his github ;)
Still, nice work Olly.

Coh3n
10-03-2013, 11:43 PM
No sire Coh3n, that is Impossibru! http://villavu.com/forum/showthread.php?t=105548&p=1262830#post1262830

I'm always the very first to write anything Smart related; I stalk Benland and his github ;)
Still, nice work Olly.I wasn't talking about that. I meant when we rewrote SRL-5's smart.simba for SRL-6 (which was long before SMART 8.3). It was rewritten then just updated later to work with SMART 8.3.

MariusK
10-22-2013, 06:38 PM
Thanks a lot!