Anonymous Login
2018-02-23 23:10 CET

View Issue Details Jump to Notes ]
IDProjectCategoryView StatusLast Update
0001979OpenClonkObjectspublic2018-02-15 13:06
ReporterLuchs 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
StatusnewResolutionopen 
Product Versiongit master 
Target Version8.1Fixed in Version 
Summary0001979: Some buildings have badly placed vertices
DescriptionSee attached screenshot.
TagsNo tags attached.
Attached Files

-Relationships
+Relationships

-Notes

~0006073

Clonkonaut (developer)

What is the exact problem? The straight line of vertices? That buildings sometimes sit on top of the landscape?
And what is your proposal?

~0006074

Caesar (developer)

@Clonkonaut: I'm guessing it's that on the roofs, they don't really correspond to where the building ends…

(If that's the case I think we could ignore this bug for 8.0.)

~0006075

Clonkonaut (developer)

Ah. I didn't see those up there!

~0006078

Luchs (developer)

Yeah, it's mostly the roof vertices that don't fit. However, especially the steam engine only has a couple of vertices in the middle and tends to fall through the landscape when digging at the wrong spot.

I'm fine with ignoring the other buildings for 8.0. The top vertices usually only matter for building placement anyways, I think?

~0006080

Luchs (developer)

https://github.com/openclonk/openclonk/commit/9eef4e2a9f6d356d32b930c487162e18db7ae6c2 - probably fixed now?

~0006081

Clonkonaut (developer)

I didn't fix all buildings. I left out those affected by 0001980. So this should stay open as a reminder.

~0006084

Zapper (developer)

@Clonkonaut
0001980 is fixed now. Only the foundry is now flippable and wasn't before. What to do now?

~0006085

Clonkonaut (developer)

@Zapper
Make the vertices of the foundry more centralised, ideally symmetrical. Have a critical look if the bottom vertices are sufficient to keep the foundry at its place unless a good deal of earth was dug away.

~0006109

occ (reporter)

Hi! There's been a check-in that references this bug. For more information you can visit the repository browser at this address:
https://git.openclonk.org/openclonk.git/commitdiff/9e01a861ce4322df8a63f1dcb84cd49ca76d37b9

Changeset 9e01a86 by Maikel de Vries <mdt.maikel@gmail.com>
fix some more structure vertices, also for asymmetric ones when flipped (0001979)

~0006110

Maikel (developer)

Did some more fixes and I think there is only one problem left.

Namely the windmill and its wings which do not have the vertices in sync and at the right place. But I don't understand the undocumented code there...

~0006113

Clonkonaut (developer)

I don't think it is that important to have these vertices in sync with the wings.

~0006114

Maikel (developer)

It looks really stupid if the wings are stuck mid-air. But we can postpone this until 9.0 or even later, or a separate report.

~0006133

occ (reporter)

Hi! There's been a check-in that references this bug. For more information you can visit the repository browser at this address:
https://git.openclonk.org/openclonk.git/commitdiff/c46b66002c43dc46f1e7dafd116f5836d2f0c787

Changeset c46b660 by Maikel de Vries <mdt.maikel@gmail.com>
fix some more structure vertices, also for asymmetric ones when flipped (0001979)

+Notes

-Issue History
Date Modified Username Field Change
2018-01-16 23:03 Luchs New Issue
2018-01-16 23:03 Luchs File Added: building vertices.png
2018-01-16 23:24 Clonkonaut Note Added: 0006073
2018-01-16 23:38 Caesar Note Added: 0006074
2018-01-16 23:42 Clonkonaut Note Added: 0006075
2018-01-17 00:25 Luchs Note Added: 0006078
2018-01-19 09:12 Luchs Note Added: 0006080
2018-01-19 10:30 Clonkonaut Note Added: 0006081
2018-01-19 17:19 Zapper Note Added: 0006084
2018-01-19 18:05 Clonkonaut Note Added: 0006085
2018-01-31 10:56 Caesar Target Version 8.0 => 8.1
2018-01-31 20:00 occ Note Added: 0006109
2018-01-31 20:01 Maikel Note Added: 0006110
2018-02-05 12:14 Clonkonaut Note Added: 0006113
2018-02-05 14:03 Maikel Note Added: 0006114
2018-02-15 13:06 occ Note Added: 0006133
+Issue History