pycraft icon indicating copy to clipboard operation
pycraft copied to clipboard

Python Renderer Compatibility

Open olls opened this issue 7 years ago • 3 comments

Are we going to continue to maintain the Python renderer, if not should we remove it?

olls avatar Jan 23 '17 01:01 olls

I think it's better to have Python renderer, so new contributors can understand rendering system more easily.

Yardanico avatar Jan 24 '17 09:01 Yardanico

Yes, that's a good reason. At the moment the C renderer in #134 is probably easier to understand than the Python renderer. The Python renderer is an inefficient mess of spaghetti code at the moment and would not help anyone understand the rendering, so if we keep it we need to rewrite it to match the operation of the new C renderer (which is probably the correct answer...).

#130 is a good description about how the new lighting rendering system now works if you are interested. We should probably write up the rest of the renderer as well into our imaginary documentation!

olls avatar Jan 24 '17 11:01 olls

I vote for removal of the Python renderer at this point. There is no point in rewriting to match the C renderer just for the sake of it. And it's no use to anyone in its current state.

olls avatar Dec 13 '20 00:12 olls