-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow n-band radiative transfer #1014
base: main
Are you sure you want to change the base?
Conversation
ef3d4c1
to
fba4d2a
Compare
b6d5976
to
c4c1e23
Compare
115102c
to
9c371e2
Compare
Can anyone help me out with the GPU compatibility issue here? It's a bit odd to me because the same broadcast structure is used over |
I can help with the GPU compat, but I have a couple of other things that I need to finish first. |
I expect that somewhere in |
Is it possible to apply |
Possibly. I was originally doing this but had some issues handling it correctly when α/ρ/τ were tuple-valued-fields vs. plain tuples. But you're right this must be the issue. Thanks |
Purpose
This PR extends our radiative transfer functionality from 2 bands (PAR and NIR) to an n-band scheme. This will allow pairing with ClimaAtmos which provides 16 bands. Represents step one towards #876.
This PR must be a large number of lines due to updating of all of the experiments and tests.
To-do
windows-latest
to tests when workingContent