Sky

[Zini]

@scrawl: It seems you are adding a fair bit of script instructions. Could you please also update the scripting wiki page?

[scrawl]

Sure!

So the process is to update the wiki page as soon as these instructions are in a feature branch?

[Zini]

We never formally decided about this. But so far I have changed the status of an instruction as soon as it is in the repository.

[scrawl]

I'd call the sky finished for now. Please test and merge in next if you think its ready.

what's still missing:
- some sounds (rain, blight, ashstorm) - there is no loop option in Soundmanager::playSound
- rain, snow, blizzard, ashstorm effects (we do not support NIF particles currently)

[werdanith]

I'd call the sky finished for now. Please test and merge in next if you think its ready.

Did you add the initialization to NULL of that fader that caused me that crash?
(I still don't get why I got that crash and you didn't)

[scrawl]

I'd call the sky finished for now. Please test and merge in next if you think its ready.

Did you add the initialization to NULL of that fader that caused me that crash?
(I still don't get why I got that crash and you didn't)

yes, thats already merged in OpenEngine / next

[Zini]

Merged.

We have three sky/weather related issues on the tracker:

* Sky-Type Weather Elements
* Review Caelum
* Precipitation/Fog-Type Weather Elements

I think we can close the first two and keep the last. And we need a new issue for weather related sound effects. Do you agree?

[scrawl]

yep.

making the weather sound effects is not hard (actually there is already some code in there that tells which sound effects should be playing) but i didnt feel like messing with the sound implementation (which was missing a 'loop' option for 2d sounds)

[Zini]

Yeah. Actually, I think it would be best to do the sound manager cleanup task first, before extending the sound manager. Also, there are some changes in master, that might result in a rather nasty merge. So that will be a task for later.

[werdanith]

I may have gotten a little over excited.
http://openmw.org/wiki/index.php?title=Screenshots