SW radiation reversed in toposub runs
SW seems to be high on north face and low on south in toposub runs (here blue is low rad, greens/yellows high rad)

am investigating....
what is the colorscale on your plot? And I suspect those are DEM artefacts?

confirmed for the Alps (Kandersteg) This is Sw direct 2015-06-01 12:00. Values 0 (blue) - 529 (yellow) Wm-2. North face has higher SWdir than South face
I still dont understand l.149 in topo_param.py:
ds['aspect'] = (["y", "x"], np.deg2rad(aspect - 180))
why do we subtract 180 deg? doesnt this reverses the aspect?
This is indeed strange! Could it be to bring aspect in the same convention as solar geom or Horizon which assumes the south at 0, west in negative and east in positive?
Then this should be done otherwise as it clearly messes with the rest.
And if you remove that 180?
with GST results I just made I get cold N and warm S - just rerunning case above to see how SW is affected.....
.... with the 180 removed, that is.

removed the 180 ...looks a bit better :) this is Jan 31 2015 12:00 with similar color scale as above. S now has 400+ Wm2 and N 0-ish
much better!
shouldn't there be some diffuse SW reaching the north? it is not dark there
prob not steep north in January - I think its realistic. Better than before :)

23 Jan 2022 lighting at 12pm from GE - the steep north face here is the one that dominates the scene above
sorry misread your comment - this is just SW-direct component above.
pushed the commit - however I leave open as we still need to fully understand if there are any implications on eg solar geom or horizon as mentioned above (-180 convention)
Sorry for not following your discussions guys, but this sounds like an aspect convention issue to me. I would always recommend using Dozier's convention of south=0, east=pi/2 (90 deg), north=pi (180 deg), west=-pi/2 (-90 deg), that way we have a right handed coordinate system which is much easier to work with mathematically. This has to be used for everything, so we should just be sure that both the solar azimuth and slope aspect are defined with the same convention (that should be relatively easy to check, since we know roughly where the sun rises and sets and we know where the north and south facing slopes are). For visualization it's always possible to convert aspects to the more usual bearing convention in post-processing, but for the actual math I would always use Dozier's convention. I don't think that simply removing or adding 180 degrees is the right way to convert between these systems (sorry if you inherited this from my code, then that was wrong too), since that only works for half the circle.