1
0
mirror of synced 2025-02-25 22:28:08 +01:00

converters.py: Fix bug when computing the second

This bug wasn't caught because `measureDurationPrev` generally equals `measureDuration` for the first two measures.

However, if there is a BPM change within the first 3 measures, then the wrong `measureDuration` will be used. So, we need to explicitly specify `measureDurationPrev`
This commit is contained in:
Viv 2023-07-01 11:44:23 -04:00
parent 1b4a938a5e
commit e1b089a595

View File

@ -138,7 +138,7 @@ def convertTJAToFumen(tja):
# Compute the 1st measure's offset by subtracting the 2nd measure's duration from the tjaOffset
if idx_m == 1:
tjaOffset = float(tja['metadata']['offset']) * 1000 * -1
tjaConverted['measures'][-1]['fumenOffset'] = tjaOffset - measureDuration
tjaConverted['measures'][-1]['fumenOffset'] = tjaOffset - measureDurationPrev
# Use the previous measure's offset plus the previous duration to compute the current measure's offset
measureOffsetPrev = tjaConverted['measures'][-1]['fumenOffset']
measureFumen['fumenOffset'] = measureOffsetPrev + measureDurationPrev