8,981
edits
Talk:Modifying an existing level: Difference between revisions
Talk:Modifying an existing level (view source)
Revision as of 21:12, 18 April 2026
, Yesterday at 21:12no edit summary
Paradox-01 (talk | contribs) mNo edit summary |
Paradox-01 (talk | contribs) mNo edit summary |
||
| Line 208: | Line 208: | ||
* A complete dev env should contain files to test every possible conversion including options. | * A complete dev env should contain files to test every possible conversion including options. | ||
* Keep test data out of original AE directory. When debugging multiple codebases, you don't want them to crash into one another. | * Keep test data out of original AE directory. When debugging multiple codebases, you don't want them to crash into one another. | ||
* To | * To set up advanced agents - that might even include (interesting but tricky) self-improvement - look at the web for [examples https://david.coffee/vibe-coding-part-1/]. | ||
=====Roundtripping===== | =====Roundtripping===== | ||
| Line 249: | Line 249: | ||
Read test data from ...\Oni\AE\GameDataFolder if needed, but write outputs to github-ignoreable subfolders of the individual's local OniSplit codebase to avoid conflicts with the source code and to keep test outputs organized and separate from source files. | Read test data from ...\Oni\AE\GameDataFolder if needed, but write outputs to github-ignoreable subfolders of the individual's local OniSplit codebase to avoid conflicts with the source code and to keep test outputs organized and separate from source files. | ||
Use .test-output for outputs and comparisons to reach hash-based conclusions about parity between | Use .test-output for outputs and comparisons to reach hash-based conclusions about parity between C# and VB builds. | ||
If you compare over 100 files at once, do it sequentially in steps of 100 so the consoles do not overflow or hang. If using powershell or other terminals, don't lose connection and report | If you compare over 100 files at once, do it sequentially in steps of 100 so the consoles do not overflow or hang. If using powershell or other terminals, don't lose connection and report every time a batch finished, so you can continue with your remaining work after all batches finished. | ||
If roundtripping is requested, get specified files (or naming pattern) from active user prompt, export oni files from dat if necessary, then extract oni files to xml and create oni again. Analyze diff in xml and hash-diffs of old and new oni files. Patch | If roundtripping is requested, get specified files (or naming pattern) from active user prompt, export oni files from dat if necessary, then extract oni files to xml and create oni again. Analyze diff in xml and hash-diffs of old and new oni files. Patch VB codebase to make roundtripping work and reach oni-hash-parity: Ignore-exception: Descriptors IDs like in TRAC id="6787" are not preserved because irrelevant and will be reset to 0. Log example of probably unimportant mismatches though. And show values of one mismatch example in console output: What do they represent. | ||
When conversions with this | When conversions with this VB-based application fail, test same command with the C#-based Onisplit.exe to check if the issue is VB-specific or shared. | ||
If shared, then it's not a parity issue and should be fixed in both builds together. Skip (but document) shared issues for now and focus on | If shared, then it's not a parity issue and should be fixed in both builds together. Skip (but document) shared issues for now and focus on VB-only issues. | ||
If vb-specific, then it's a parity issue and should be fixed in the | If vb-specific, then it's a parity issue and should be fixed in the VB codebase to match the c# behavior. | ||
</pre> | </pre> | ||