Set a basic property

[Avionyx]

Hi All,

Trying my first venture in to nasal, I'd like to set a property for the radio power to be on if 2 values are true and one false.

Could someone help please - I'd love to learn.

Code: Select all

#Radio Power Logic
var radiopower = {
   update : func {
        var batswitch = getprop("/controls/electric/directbat-switch");
        var generator = getprop("/controls/electric/engine/generator");
        var running = getprop("/engines/engine/running");
        if ((batswitch != false) and (generator != true) and (running != false) {
      setprop("/controls/electric/radiopower", "true");}}
}

Thanks in advance!


[Necolatis]

This line:

if ((batswitch != false) and (generator != true) and (running != false) {

is missing a parentess at the end.

"true" should not be in quotation marks, then nasal thinks its a string. In fact its better to use just 1 or 0 for true and false.

You probably also want an else statement. So that if the above is not the case then turn the radio off.

Code: Select all

#Radio Power Logic
var radiopower = {
   update : func {
        var batswitch = getprop("/controls/electric/directbat-switch");
        var generator = getprop("/controls/electric/engine/generator");
        var running = getprop("/engines/engine/running");
        if (batswitch == 1 and generator == 0 and running == 1) {
            setprop("/controls/electric/radiopower", 1);
        } else {
            setprop("/controls/electric/radiopower", 0);
        }
    },
}


[Avionyx]

Hi Necolatis,

Thank's for such a fast reply.
I've given that a go and I'm getting no errors anymore but it's also not setting the properties even though the criteria has been met.
To keep it simple I stripped it back to 2 properties and tried that but again the same results, no errors and no properties set.

Here's the snippet I'm using and a screenshot from in sim with the Nasal Console open.

Code: Select all

#Radio Power Logic
var radiopower = {
   update : func {
        var batswitch = getprop("/controls/electric/directbat-switch");
        var generator = getprop("/controls/electric/engine/generator");
        if (batswitch == 1 and generator == 1) {
            setprop("/controls/electric/radiopower", 1);
        } else {
            setprop("/controls/electric/radiopower", 0);
        }
    },
}



[code]
(Thumbnail, click to view larger size)


I really appreciate any help with this, I'd absolutely love to harness the power of Nasal.

Alex

[Necolatis]

That looks good.
Might be a stupid question, but are calling this in a loop, with a timer or listener?

[Avionyx]

This is probably where my lack of knowledge isn't helping.
I'd assumed it would need to be a listener but I'm not sure how to set it properly in this instance.

Let's say I wanted to (rather wastefully) set this as a standalone .nas file.
What would I need to include to get the above code operating as expected? A listener on the property I'm setting or one for each that we're using in the AND statement?

Many thanks for your help so far.

Alex

[Necolatis]

Code: Select all

#Radio Power Logic
var radiopower = {
   update : func {
        var batswitch = getprop("/controls/electric/directbat-switch");
        var generator = getprop("/controls/electric/engine/generator");
        if (batswitch == 1 and generator == 1) {
            setprop("/controls/electric/radiopower", 1);
        } else {
            setprop("/controls/electric/radiopower", 0);
        }
    },
}

setlistener("/controls/electric/directbat-switch", func {radiopower.update();});
setlistener("/controls/electric/engine/generator", func {radiopower.update();});


You will also need to add the nas file to the aircraft, you know how to do that?

[Avionyx]

I do , thank you!

Should this work in the nasal console? I'm getting a parse error for:

Code: Select all

var radiopower = {

[Necolatis]

Oh, do a ; after the ending bracket, sorry.

[Avionyx]

Huge thanks for your help my friend.
All working and now I'm ready to tackle more Nasal.

Really appreciate the time you put into this for me.

All the best,
Alex