Page MenuHomeWildfire Games

Danubius Cinematic Path
AbandonedPublic

Authored by elexis on May 9 2017, 4:32 AM.

Details

Reviewers
vladislavbelov
Summary

The new feature to add cinematic paths from a trigger script might be nice for the Danubius map,
since that map has a number of point of interests (gaia stronghold, ships, ritual place).
If such a path is accepted, it should not reveal too much of the map area ideally but focus on eye candy.

The first iteration of the patch looks very crude. Feel free to commandeer the revision and upload a proper cinematic path.

Test Plan

Start the map.

Diff Detail

Repository
rP 0 A.D. Public Repository
Branch
/ps/trunk
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 1581
Build 2498: Vulcan BuildJenkins
Build 2497: arc lint + arc unit

Event Timeline

elexis created this revision.May 9 2017, 4:32 AM

When starting the map, this warning always appears: WARNING: Path with name 'danubius' doesn't exist

(From glancing quickly, I guess it's because it tries to delete a non-existing path).

Maybe it would be nice if the cinematic path went over the river.

Vulcan added a subscriber: Vulcan.May 9 2017, 12:12 PM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!

http://jw:8080/job/phabricator/1098/ for more details.

Yes, forgot to remove the DeletePath call. It was used so you can restart the path each time you save the triggers file without launcing the game again.
Also the path looks kind of crappy in my opinion :-/

elexis abandoned this revision.Jun 14 2017, 7:47 PM