I think nobody would expect its ship to be blocked by a small floating barrel.
And the current pathfinder is quite bad and sometime ships are blocked in-between two nearby such debris.
Details
- Reviewers
elexis - Commits
- rP21349: Small floating debris should not block ship movement.
agree or not with the change
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 5171 Build 8846: Vulcan Build Jenkins Build 8845: arc lint + arc unit
Event Timeline
Those are only 3 barrels, very small, so overlapping of the models isn't really noticeable.
If you rename that file to not have shipwreck in the name, i.e. to barrels_floating or the like I wouldn't disagree.
It's the same model as food_barrels_buried.
The other shipwrecks should probably keep it, though woulnd't raise a concern if collision is removed there.
(also please have a look at D1326 in case it's not already on your table)
yes, it looks fine
If you rename that file to not have shipwreck in the name, i.e. to barrels_floating or the like I wouldn't disagree.
It's the same model as food_barrels_buried.
We'd have to also change all maps which have it. I'd rather not do that now.
The other shipwrecks should probably keep it, though woulnd't raise a concern if collision is removed there.
agree that the other ones should keep it.
(also please have a look at D1326 in case it's not already on your table)
ok
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/86/display/redirect