Confusing `rafx_visibility` and `rafx_framework::visiblity` exported as `rafx::visibility` about rafx HOT 3 CLOSED

Going to close for the time being, I want to delay final decisions for naming/crate layout until later on when we've been able to do more road-testing of these systems.

from rafx.

The history of it is that there used to be a rafx_visibility crate, then I pulled it into the rafx_framework crate to reduce modules, and then @DVD added the crate back when he implemented an actual visibility algorithm.

@DVD do you have any thoughts or ideas for how we could name this. I personally don't feel we should bother changing this unless we have an idea we're pretty happy with. Long-term, both these crates/modules will be internal details that few people will ever touch.

from rafx.

If this is all in flux and going to change later anyway, I'm not opposed to leaving it how it is. I'm always pushing towards very tidy and easy-to-understand documentation and organization, but I totally get it if this is all work-in-progress and it's not worth spending time on this yet.

from rafx.