search

AlexIIL / LargeCurvedRails

whats that? a recent commit?
5 stars 1 forks source link

Models of curved tracks #6

Closed AlexIIL closed 8 years ago

AlexIIL commented 8 years ago

An automatic generator that can provide:

AlexIIL commented 8 years ago

Technically implemented but the curved paths are still wrong.

NitroxydeX commented 8 years ago

It's a bit early but I think questions and feedback is a good thing. I grabbed the latest commit to look at curved tracks.

Maybe this is not final or maybe it is but I think putting a diagonal track between two curves is not the best way. It looks in my opinion a little bit weird if there is a little kink in the curve. See attached photos.

Maybe this is only the way for 45° curves and you implement 90° curves too. Then it's okay.

2016-02-08_22 00 04 2016-02-08_22 00 12 2016-02-08_22 03 59

AlexIIL commented 8 years ago

Yeah, that was the first thing I noticed when joining them :) (its was mathematically horrible to try and get 45 degree curves to work at all, so I don't want to waste another week modifying them).

Adding 90 degree turns is the easiest way to fix this though, so I might implement it tomorrow.

AlexIIL commented 8 years ago

(The commit you used is AFTER I fixed curved tracks BTW). I'm continuing 90 degree curves in #12.

NitroxydeX commented 8 years ago

I think there is still something wrong with the paths. Train always derail at the second curve. No matter how long the diagonal track is between them

2016-02-09_15 14 34

Even if you start it on the diagonal track it directly derails on the curve.

2016-02-09_15 18 33

And there is a problem with the curved 8 track.

http://pastebin.com/gBRpsHm3

It's placed after reloading but Minecraft crashs on placement

AlexIIL commented 8 years ago

I'm just about to push a breaking commit (that breaks the way curved tracks work), but it does also fix the 8-wide curves crashing