Shadows

Everything about development and the OpenMW source code.
User avatar
Capostrophic
Posts: 794
Joined: 22 Feb 2016, 20:32

Re: Shadows

Post by Capostrophic »

u srs
User avatar
psi29a
Posts: 5355
Joined: 29 Sep 2011, 10:13
Location: Belgium
Gitlab profile: https://gitlab.com/psi29a/
Contact:

Re: Shadows

Post by psi29a »

no, absolutely not

I know where at the point that we're discussing 'style' and best practices instead of further technical work... but still. Let's keep to the plan please. :)
User avatar
lysol
Posts: 1513
Joined: 26 Mar 2013, 01:48
Location: Sweden

Re: Shadows

Post by lysol »

But but... shadows r hoodge! And will need testing for weeks!
User avatar
psi29a
Posts: 5355
Joined: 29 Sep 2011, 10:13
Location: Belgium
Gitlab profile: https://gitlab.com/psi29a/
Contact:

Re: Shadows

Post by psi29a »

lysol wrote: 31 Jan 2019, 20:36 But but... shadows r hoodge! And will need testing for weeks!
Perfect for 0.46
User avatar
AnyOldName3
Posts: 2667
Joined: 26 Nov 2015, 03:25

Re: Shadows

Post by AnyOldName3 »

I've already merged parts of 0.46 into my PR, so it would be a massive mess to even try and get it into 0.45, regardless of it also being a bad idea.
User avatar
Br0ken
Posts: 243
Joined: 02 Apr 2012, 05:54
Location: Siberia

Re: Shadows

Post by Br0ken »

So, is it ready to merge?
User avatar
AnyOldName3
Posts: 2667
Joined: 26 Nov 2015, 03:25

Re: Shadows

Post by AnyOldName3 »

It's time for general code review and thorough testing and bug hunting. If nothing comes up, it should be merged very soon. I think one of the settings doesn't do quite what it's supposed to, though, so there's probably something that needs changing.
User avatar
psi29a
Posts: 5355
Joined: 29 Sep 2011, 10:13
Location: Belgium
Gitlab profile: https://gitlab.com/psi29a/
Contact:

Re: Shadows

Post by psi29a »

A general round of reviews and testing, when we're comfortable, we'll merge.

Then come the nightlies and more testing for those living on the edge.
User avatar
lysol
Posts: 1513
Joined: 26 Mar 2013, 01:48
Location: Sweden

Re: Shadows

Post by lysol »

psi29a wrote: 31 Jan 2019, 20:38
lysol wrote: 31 Jan 2019, 20:36 But but... shadows r hoodge! And will need testing for weeks!
Perfect for 0.46
Just to clarify, yes, this is what I meant too.

Anyway, will the shadow settings get ingame adjustable settings now or in a later PR?
User avatar
AnyOldName3
Posts: 2667
Joined: 26 Nov 2015, 03:25

Re: Shadows

Post by AnyOldName3 »

Akortunov started a PR against my fork for that, but it'll mostly need redoing as the set of available settings has changed.
Post Reply