New cel commands for Celestia 1.6.0

[duds26]

Will the integration of these features mean that Celestia-ED will be discontinued?
(I mean the adjusted version of Celestia, not the Educational Activities.)

-------------------
And users of Celestia-ED will be directed to Celestia 1.6?
And will the educational activities adjusted for Celestia 1.6?
???

And what formats are supported?
wma, (mp3 probably not), flac, ogg (vorbis), spx (speex codec for speech), ape, others???
(Please ogg codecs: flac, ogg, spx)
(libs for all three available on the Xiph.org Website)

[Adirondack]

Vincent,

this new CEL features are great!

Nevertheless, I have an objection concerning the setgalaxylightgain-command:
Shouldn't we implement 'setgalaxylightgain' to the global set-command in order to correspond to the standards?
Instead of setgalaxylightgain {gain 0.5} we should use set {name "GalaxyLightgain" value 0.5}
Just like we set the FOV, AmbientLightLevel and so on.

Adirondack

[Vincent]

Shouldn't we implement 'setgalaxylightgain' to the global set-command in order to correspond to the standards?
Instead of setgalaxylightgain {gain 0.5} we should use set {name "GalaxyLightgain" value 0.5}
Just like we set the FOV, AmbientLightLevel and so on.

Hi Ulrich,

The setambientlight and setfaintestautomag45deg commands are not part
of the global set command:

Code: Select all

setambientlight { brightness 0.2 }
setfaintestautomag45deg { magnitude 7.5 }

That's why I decided to not include setgalaxylightgain in the global set command either.
The use of the GalaxyLightgain field within the global set command looks quite deprecated to me.

[Adirondack]

The setambientlight and setfaintestautomag45deg commands are not part
of the global set command:

Code: Select all

setambientlight { brightness 0.2 }

Yes, but AFAIK setambientlight { brightness 0.2 } has become obsolete today
and is replaced with set {name "AmbientLightLevel" value 0.2 }.

Just my two cents. You are the man.

Adirondack

[selden]

Personally I think this was a mistaken oversight.
I think it would be best to have all of the "set" commands handled the same way. Having arbitrary exceptions like these will cause coding errors and frustration.

[Vincent]

Yes, but AFAIK setambientlight { brightness 0.2 } has become obsolete today
and is replaced with set {name "AmbientLightLevel" value 0.2 }.

Just my two cents. You are the man.

I afraid I'm not your Cel man anymore here...
I've not been using Cel scripting for monthes (years?).

Personally I think this was a mistaken oversight.
I think it would be best to have all of the "set" commands handled the same way. Having arbitrary exceptions like these will cause coding errors and frustration.

I agree and I'll make the change.
But we need to decide first what will be this 'same way', i.e., if we shall use the global set command or not...