Navigation Menu

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

Jump to bottom

town builds non-connectible roads at hills #1207

Closed
DorpsGek opened this issue Sep 5, 2007 · 10 comments
Closed

town builds non-connectible roads at hills #1207

DorpsGek opened this issue Sep 5, 2007 · 10 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

DorpsGek commented Sep 5, 2007

SmatZ opened the ticket and wrote:

r11047

With 2x2 or 3x3 grid town layout, some roads are built inconnectible to existing roads.

Attachments

Reported version: trunk
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/1207
@DorpsGek
Copy link
Member Author

DorpsGek commented Sep 5, 2007

skidd13 wrote:

The problem is caused by a template structure in the algorithm which is calulating the grid. This structure will be replaced by a more dynamic solution. #1090 fixes this already, but it does a lot more. After the rewrite of the town growth (#1161) the problem should be solved.

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207#comment2068

@DorpsGek
Copy link
Member Author

DorpsGek commented Sep 6, 2007

skidd13 wrote:

The current patch of #1161 should fix the major problem of really inconnective road tiles. There appear still some unaesthetic road junctions. But therefore #1090 exists.

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207#comment2079

@DorpsGek
Copy link
Member Author

DorpsGek commented Sep 7, 2007

skidd13 wrote:

So reconvert from #1161 to the old procedure is done. This one should fix it.

Attachments

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207#comment2082

@DorpsGek
Copy link
Member Author

Rubidium wrote:

#1161 does not seem to "solve" this issue and #1090 doesn't apply anymore without a lot of rejects.

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207#comment2122

@DorpsGek
Copy link
Member Author

skidd13 wrote:

#1161 prevents only the generation of new inconnectible roads. I'm already at the rewrite of #1090.

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207#comment2124

@DorpsGek
Copy link
Member Author

skidd13 wrote:

This patch will fix the unconnectable roads. It took me a while till I noticed that bypassing the templates solved the problem easily.

Patch against: r11127

Attachments

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207#comment2149

@DorpsGek
Copy link
Member Author

skidd13 wrote:

Damn forget to remove some unneeded stuff. Sorry.

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207#comment2151

@DorpsGek
Copy link
Member Author

skidd13 wrote:

And now I missed to add the file :(

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207#comment2152

@DorpsGek
Copy link
Member Author

skidd13 wrote:

Hmm, the file does not like to be uploaded. Or flyspray is blocking me.... strange. So next try

Attachments

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207#comment2153

@DorpsGek
Copy link
Member Author

Rubidium closed the ticket.

Reason for closing: Fixed

In r11130

This comment was imported from FlySpray: https://bugs.openttd.org/task/1207

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DorpsGek