CubicChunks

CubicChunks

840k Downloads

Add vertical view distance

NotStirred opened this issue ยท 15 comments

commented
commented

I'm working on this now.

commented

This is not as easy as it may seem. CubeDistanceGraph doesn't really work that way.

commented

What would be the best way to store a vertical view distance client side? Forge config file? options.txt?

commented

1.12.2 had it in config/cubicchunks.cfg as I:verticalCubeLoadDistance

commented

This is quite a lot of work to do for vertical view to work, good luck...

commented

Ok, I'll do that for now.

commented

Would it be a bad idea to just use the EarlyConfig for now?

commented

you can do, I'll be moving that stuff around later anyway

commented

The target is forge config file but there is a problem with that:

in 1.12.2 I could modify forge config from code and store it back. So I could have the vertical view distance slider modify the config value and store it.

Modifying config values from code is no longer possible. So right now just make some dummy setter and getter for it in the config object and leave it to figure out later.

commented

Correction: I read the code now and apparently sometime between last time I looked at it and now an option to set config values programatically has been added.

commented

Well I suppose I'll create the config.

commented

And no you shouldn't use EarlyConfig for that. EarlyConfig has one purpose: configuring stuff for mixin and stuff that needs to be loaded arbitrarily early. Vertical view distance can be just standard forge client config.

commented

image
Okay, I've got vertical view & render distance working. Sort of. This is a hack if you've ever seen one, so be aware, but I've got a prototype.

Commit: AidanLovelace@ed9d112

commented

wait im an idiot

commented

updated and merged in d41a712