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

Some expected outputs are zero length #1

Open
fragglet opened this issue Jun 11, 2015 · 1 comment
Open

Some expected outputs are zero length #1

fragglet opened this issue Jun 11, 2015 · 1 comment

Comments

@fragglet
Copy link
Member

We paint all demos in the Compet-N with a broad brushstroke and play them back according to the name of the directory they're located in. But not all demos should be played back with the same game version / IWAD.

This is evident in the output/ directory where there are a bunch of .txt files that are of zero size. Some of these are legitimate: for example, for ExM8 levels, no statistics data gets generated. However, there are some that should not be zero, and that are only zero because they've been generated from the wrong configuration. Examples are:

  • Doom 1 demos that should be played back with the registered version IWAD, or with a different version to Doom 1.9.
  • Final Doom demos recorded with the "alternate" version of the .exe that was included with the Id Anthology. This has different teleport behavior.

We need a separate configuration file with individual "overrides" for these misconfigured demos, so that we can play them back properly and test these different versions.

@fragglet
Copy link
Member Author

There are also some, like tnt/nm100s/es18-301.zip, that are Final Doom demos recorded with Doom II's doom2.exe. Why? Who knows.

fragglet added a commit that referenced this issue Jun 13, 2015
Some demo outputs were zero length because they were recorded with
the "alternate" version of Final Doom included with the id anthology.
So add an exceptions file that lists these, and fix the expected
outputs for these files.

This is the first step towards a fix for #1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant