Skip to content
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

280: Automatic Lights requires Custom Distance which requires EEVEE #564

Closed
tngreene opened this issue May 15, 2020 · 1 comment
Closed
Assignees
Labels
Bug priority high WYSIWYG Lights For the many ~problems~ challenges related to the WYSIWYG Lights

Comments

@tngreene
Copy link
Contributor

I picked Custom Distance and didn't realize that Cycles doesn't use that. In 2.80 EEVEE is the default, in 2.81 it is Cycles, and in 2.82 it appears they went back. Regardless of the default, people still have to be warned

-if they're in Cycles and they're in Lights Properties Tab and the light is a Spot and the light name has a size parameter

that they can't change the SIZE

And now here's the tough decision:

Should we give people an error for using the "wrong" rendering engine? Do we just use whatever setting the put in as long as they did it once?

My intuition says "don't have hidden UI silently work or fail", but, if people tell me they like switching in and out of EEVEE......... I still don't like it. We don't have an official rendering engine for XPlane2Blender but maybe if we one day get to material nodes we'll have to make a choice.

For now I think it'll be warning label and an error to stop people from being lost? We can always remove it and be more flexible later.

@tngreene tngreene self-assigned this May 15, 2020
@tngreene tngreene added Bug in review priority high WYSIWYG Lights For the many ~problems~ challenges related to the WYSIWYG Lights labels May 15, 2020
@tngreene tngreene added this to the Blender 2.80-beta.2 milestone May 21, 2020
@tngreene
Copy link
Contributor Author

tngreene commented Jun 2, 2020

This was solved by re-using Custom Light's size for size. See #568

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug priority high WYSIWYG Lights For the many ~problems~ challenges related to the WYSIWYG Lights
Projects
None yet
Development

No branches or pull requests

1 participant