calculate grade with possible values from radarValues, not from judgement counts from attempted steps
This commit is contained in:
@@ -522,7 +522,7 @@ void NoteDataUtil::LoadTransformedLights( const NoteData &in, NoteData &out, int
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
void NoteDataUtil::GetRadarValues( const NoteData &in, float fSongSeconds, RadarValues& out )
|
void NoteDataUtil::CalculateRadarValues( const NoteData &in, float fSongSeconds, RadarValues& out )
|
||||||
{
|
{
|
||||||
// The for loop and the assert are used to ensure that all fields of
|
// The for loop and the assert are used to ensure that all fields of
|
||||||
// RadarValue get set in here.
|
// RadarValue get set in here.
|
||||||
|
|||||||
@@ -31,7 +31,7 @@ namespace NoteDataUtil
|
|||||||
float GetFreezeRadarValue( const NoteData &in, float fSongSeconds );
|
float GetFreezeRadarValue( const NoteData &in, float fSongSeconds );
|
||||||
float GetChaosRadarValue( const NoteData &in, float fSongSeconds );
|
float GetChaosRadarValue( const NoteData &in, float fSongSeconds );
|
||||||
|
|
||||||
void GetRadarValues( const NoteData &in, float fSongSeconds, RadarValues& out );
|
void CalculateRadarValues( const NoteData &in, float fSongSeconds, RadarValues& out );
|
||||||
|
|
||||||
void RemoveHoldNotes( NoteData &inout, int iStartIndex = 0, int iEndIndex = MAX_NOTE_ROW );
|
void RemoveHoldNotes( NoteData &inout, int iStartIndex = 0, int iEndIndex = MAX_NOTE_ROW );
|
||||||
void ChangeRollsToHolds( NoteData &in, int iStartIndex, int iEndIndex );
|
void ChangeRollsToHolds( NoteData &in, int iStartIndex, int iEndIndex );
|
||||||
|
|||||||
@@ -51,7 +51,7 @@ void SMLoader::LoadFromSMTokens(
|
|||||||
RadarValues v;
|
RadarValues v;
|
||||||
FOREACH_RadarCategory(rc)
|
FOREACH_RadarCategory(rc)
|
||||||
v[rc] = strtof( saValues[rc], NULL );
|
v[rc] = strtof( saValues[rc], NULL );
|
||||||
out.SetRadarValues( v );
|
out.SetCachedRadarValues( v );
|
||||||
}
|
}
|
||||||
|
|
||||||
out.SetSMNoteData(sNoteData);
|
out.SetSMNoteData(sNoteData);
|
||||||
|
|||||||
@@ -9,6 +9,7 @@
|
|||||||
#include <float.h>
|
#include <float.h>
|
||||||
#include "GameState.h"
|
#include "GameState.h"
|
||||||
#include "Course.h"
|
#include "Course.h"
|
||||||
|
#include "Steps.h"
|
||||||
|
|
||||||
#define GRADE_PERCENT_TIER(i) THEME->GetMetricF("PlayerStageStats",ssprintf("GradePercent%s",GradeToString((Grade)i).c_str()))
|
#define GRADE_PERCENT_TIER(i) THEME->GetMetricF("PlayerStageStats",ssprintf("GradePercent%s",GradeToString((Grade)i).c_str()))
|
||||||
#define GRADE_TIER02_IS_ALL_PERFECTS THEME->GetMetricB("PlayerStageStats","GradeTier02IsAllPerfects")
|
#define GRADE_TIER02_IS_ALL_PERFECTS THEME->GetMetricB("PlayerStageStats","GradeTier02IsAllPerfects")
|
||||||
@@ -129,7 +130,14 @@ Grade PlayerStageStats::GetGrade() const
|
|||||||
|
|
||||||
/* XXX: This entire calculation should be in ScoreKeeper, but final evaluation
|
/* XXX: This entire calculation should be in ScoreKeeper, but final evaluation
|
||||||
* is tricky since at that point the ScoreKeepers no longer exist. */
|
* is tricky since at that point the ScoreKeepers no longer exist. */
|
||||||
float Possible = 0, Actual = 0;
|
float fPossible = 0;
|
||||||
|
FOREACH_CONST( Steps*, vpPossibleSteps, s )
|
||||||
|
{
|
||||||
|
fPossible += (*s)->GetRadarValues().m_Values.v.fNumTapsAndHolds * PREFSMAN->m_iGradeWeightMarvelous;
|
||||||
|
fPossible += (*s)->GetRadarValues().m_Values.v.fNumHolds * PREFSMAN->m_iGradeWeightOK;
|
||||||
|
}
|
||||||
|
|
||||||
|
float fActual = 0;
|
||||||
FOREACH_TapNoteScore( tns )
|
FOREACH_TapNoteScore( tns )
|
||||||
{
|
{
|
||||||
int iTapScoreValue;
|
int iTapScoreValue;
|
||||||
@@ -146,9 +154,7 @@ Grade PlayerStageStats::GetGrade() const
|
|||||||
case TNS_MARVELOUS: iTapScoreValue = PREFSMAN->m_iGradeWeightMarvelous; break;
|
case TNS_MARVELOUS: iTapScoreValue = PREFSMAN->m_iGradeWeightMarvelous; break;
|
||||||
default: FAIL_M( ssprintf("%i", tns) ); break;
|
default: FAIL_M( ssprintf("%i", tns) ); break;
|
||||||
}
|
}
|
||||||
Actual += iTapNoteScores[tns] * iTapScoreValue;
|
fActual += iTapNoteScores[tns] * iTapScoreValue;
|
||||||
if( tns != TNS_HIT_MINE )
|
|
||||||
Possible += iTapNoteScores[tns] * PREFSMAN->m_iGradeWeightMarvelous;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
FOREACH_HoldNoteScore( hns )
|
FOREACH_HoldNoteScore( hns )
|
||||||
@@ -161,14 +167,13 @@ Grade PlayerStageStats::GetGrade() const
|
|||||||
case HNS_OK: iHoldScoreValue = PREFSMAN->m_iGradeWeightOK; break;
|
case HNS_OK: iHoldScoreValue = PREFSMAN->m_iGradeWeightOK; break;
|
||||||
default: FAIL_M( ssprintf("%i", hns) ); break;
|
default: FAIL_M( ssprintf("%i", hns) ); break;
|
||||||
}
|
}
|
||||||
Actual += iHoldNoteScores[hns] * iHoldScoreValue;
|
fActual += iHoldNoteScores[hns] * iHoldScoreValue;
|
||||||
Possible += iHoldNoteScores[hns] * PREFSMAN->m_iGradeWeightOK;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
LOG->Trace( "GetGrade: Actual: %f, Possible: %f", Actual, Possible );
|
LOG->Trace( "GetGrade: fActual: %f, fPossible: %f", fActual, fPossible );
|
||||||
|
|
||||||
|
|
||||||
float fPercent = (Possible == 0) ? 0 : Actual / Possible;
|
float fPercent = (fPossible == 0) ? 0 : fActual / fPossible;
|
||||||
|
|
||||||
Grade grade = GetGradeFromPercent( fPercent );
|
Grade grade = GetGradeFromPercent( fPercent );
|
||||||
|
|
||||||
|
|||||||
@@ -60,7 +60,7 @@ void ScoreKeeperMAX2::Load(
|
|||||||
*/
|
*/
|
||||||
NoteDataUtil::TransformNoteData( nd, aa, pSteps->m_StepsType, pSong );
|
NoteDataUtil::TransformNoteData( nd, aa, pSteps->m_StepsType, pSong );
|
||||||
RadarValues rvPre;
|
RadarValues rvPre;
|
||||||
NoteDataUtil::GetRadarValues( nd, pSong->m_fMusicLengthSeconds, rvPre );
|
NoteDataUtil::CalculateRadarValues( nd, pSong->m_fMusicLengthSeconds, rvPre );
|
||||||
|
|
||||||
/* Apply user transforms to find out how the notes will really look.
|
/* Apply user transforms to find out how the notes will really look.
|
||||||
*
|
*
|
||||||
@@ -71,7 +71,7 @@ void ScoreKeeperMAX2::Load(
|
|||||||
* mean moving the queues in ScreenGameplay to GameState ... */
|
* mean moving the queues in ScreenGameplay to GameState ... */
|
||||||
NoteDataUtil::TransformNoteData( nd, m_pPlayerState->m_PlayerOptions, pSteps->m_StepsType );
|
NoteDataUtil::TransformNoteData( nd, m_pPlayerState->m_PlayerOptions, pSteps->m_StepsType );
|
||||||
RadarValues rvPost;
|
RadarValues rvPost;
|
||||||
NoteDataUtil::GetRadarValues( nd, pSong->m_fMusicLengthSeconds, rvPost );
|
NoteDataUtil::CalculateRadarValues( nd, pSong->m_fMusicLengthSeconds, rvPost );
|
||||||
|
|
||||||
iTotalPossibleDancePoints += this->GetPossibleDancePoints( rvPre, rvPost );
|
iTotalPossibleDancePoints += this->GetPossibleDancePoints( rvPre, rvPost );
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -2145,7 +2145,7 @@ void ScreenGameplay::SongFinished()
|
|||||||
* not for the percentages (RADAR_AIR). */
|
* not for the percentages (RADAR_AIR). */
|
||||||
RadarValues v;
|
RadarValues v;
|
||||||
|
|
||||||
NoteDataUtil::GetRadarValues( m_Player[p].m_NoteData, GAMESTATE->m_pCurSong->m_fMusicLengthSeconds, v );
|
NoteDataUtil::CalculateRadarValues( m_Player[p].m_NoteData, GAMESTATE->m_pCurSong->m_fMusicLengthSeconds, v );
|
||||||
STATSMAN->m_CurStageStats.m_player[p].radarPossible += v;
|
STATSMAN->m_CurStageStats.m_player[p].radarPossible += v;
|
||||||
|
|
||||||
NoteDataWithScoring::GetActualRadarValues( m_Player[p].m_NoteData, p, GAMESTATE->m_pCurSong->m_fMusicLengthSeconds, v );
|
NoteDataWithScoring::GetActualRadarValues( m_Player[p].m_NoteData, p, GAMESTATE->m_pCurSong->m_fMusicLengthSeconds, v );
|
||||||
|
|||||||
@@ -153,7 +153,7 @@ void Steps::TidyUpData()
|
|||||||
|
|
||||||
void Steps::CalculateRadarValues( float fMusicLengthSeconds )
|
void Steps::CalculateRadarValues( float fMusicLengthSeconds )
|
||||||
{
|
{
|
||||||
m_RadarValues = RadarValues();
|
m_CachedRadarValues = RadarValues();
|
||||||
|
|
||||||
// If we're autogen, don't calculate values. GetRadarValues will take from our parent.
|
// If we're autogen, don't calculate values. GetRadarValues will take from our parent.
|
||||||
if( parent != NULL )
|
if( parent != NULL )
|
||||||
@@ -165,7 +165,7 @@ void Steps::CalculateRadarValues( float fMusicLengthSeconds )
|
|||||||
|
|
||||||
NoteData tempNoteData;
|
NoteData tempNoteData;
|
||||||
this->GetNoteData( tempNoteData );
|
this->GetNoteData( tempNoteData );
|
||||||
NoteDataUtil::GetRadarValues( tempNoteData, fMusicLengthSeconds, m_RadarValues );
|
NoteDataUtil::CalculateRadarValues( tempNoteData, fMusicLengthSeconds, m_CachedRadarValues );
|
||||||
}
|
}
|
||||||
|
|
||||||
void Steps::Decompress() const
|
void Steps::Decompress() const
|
||||||
@@ -270,7 +270,7 @@ void Steps::DeAutogen()
|
|||||||
m_sDescription = Real()->m_sDescription;
|
m_sDescription = Real()->m_sDescription;
|
||||||
m_Difficulty = Real()->m_Difficulty;
|
m_Difficulty = Real()->m_Difficulty;
|
||||||
m_iMeter = Real()->m_iMeter;
|
m_iMeter = Real()->m_iMeter;
|
||||||
m_RadarValues = Real()->m_RadarValues;
|
m_CachedRadarValues = Real()->m_CachedRadarValues;
|
||||||
|
|
||||||
parent = NULL;
|
parent = NULL;
|
||||||
|
|
||||||
@@ -347,10 +347,10 @@ void Steps::SetMeter(int meter)
|
|||||||
m_iMeter = meter;
|
m_iMeter = meter;
|
||||||
}
|
}
|
||||||
|
|
||||||
void Steps::SetRadarValues( const RadarValues& v )
|
void Steps::SetCachedRadarValues( const RadarValues& v )
|
||||||
{
|
{
|
||||||
DeAutogen();
|
DeAutogen();
|
||||||
m_RadarValues = v;
|
m_CachedRadarValues = v;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -38,7 +38,7 @@ public:
|
|||||||
Difficulty GetDifficulty() const { return Real()->m_Difficulty; }
|
Difficulty GetDifficulty() const { return Real()->m_Difficulty; }
|
||||||
ProfileSlot GetLoadedFromProfile() const { return m_LoadedFromProfile; }
|
ProfileSlot GetLoadedFromProfile() const { return m_LoadedFromProfile; }
|
||||||
int GetMeter() const { return Real()->m_iMeter; }
|
int GetMeter() const { return Real()->m_iMeter; }
|
||||||
const RadarValues& GetRadarValues() const { return Real()->m_RadarValues; }
|
const RadarValues& GetRadarValues() const { return Real()->m_CachedRadarValues; }
|
||||||
|
|
||||||
void SetFile( CString fn );
|
void SetFile( CString fn );
|
||||||
void SetSavedToDisk( bool b ) { m_bSavedToDisk = b; }
|
void SetSavedToDisk( bool b ) { m_bSavedToDisk = b; }
|
||||||
@@ -50,7 +50,7 @@ public:
|
|||||||
|
|
||||||
void SetLoadedFromProfile( ProfileSlot slot ) { m_LoadedFromProfile = slot; }
|
void SetLoadedFromProfile( ProfileSlot slot ) { m_LoadedFromProfile = slot; }
|
||||||
void SetMeter(int meter);
|
void SetMeter(int meter);
|
||||||
void SetRadarValues( const RadarValues& v );
|
void SetCachedRadarValues( const RadarValues& v );
|
||||||
bool IsAutogen() const; // Was created by autogen?
|
bool IsAutogen() const; // Was created by autogen?
|
||||||
float PredictMeter() const;
|
float PredictMeter() const;
|
||||||
|
|
||||||
@@ -90,7 +90,7 @@ protected:
|
|||||||
CString m_sDescription; // Step author, edit name, or something meaningful
|
CString m_sDescription; // Step author, edit name, or something meaningful
|
||||||
Difficulty m_Difficulty; // difficulty classification
|
Difficulty m_Difficulty; // difficulty classification
|
||||||
int m_iMeter; // difficulty rating from MIN_METER to MAX_METER
|
int m_iMeter; // difficulty rating from MIN_METER to MAX_METER
|
||||||
RadarValues m_RadarValues;
|
RadarValues m_CachedRadarValues;
|
||||||
};
|
};
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
|||||||
@@ -75,14 +75,14 @@ RadarValues Trail::GetRadarValues() const
|
|||||||
NoteData nd;
|
NoteData nd;
|
||||||
pSteps->GetNoteData( nd );
|
pSteps->GetNoteData( nd );
|
||||||
RadarValues rv_orig;
|
RadarValues rv_orig;
|
||||||
NoteDataUtil::GetRadarValues( nd, e->pSong->m_fMusicLengthSeconds, rv_orig );
|
NoteDataUtil::CalculateRadarValues( nd, e->pSong->m_fMusicLengthSeconds, rv_orig );
|
||||||
PlayerOptions po;
|
PlayerOptions po;
|
||||||
po.FromString( e->Modifiers );
|
po.FromString( e->Modifiers );
|
||||||
if( po.ContainsTransformOrTurn() )
|
if( po.ContainsTransformOrTurn() )
|
||||||
NoteDataUtil::TransformNoteData( nd, po, pSteps->m_StepsType );
|
NoteDataUtil::TransformNoteData( nd, po, pSteps->m_StepsType );
|
||||||
NoteDataUtil::TransformNoteData( nd, e->Attacks, pSteps->m_StepsType, e->pSong );
|
NoteDataUtil::TransformNoteData( nd, e->Attacks, pSteps->m_StepsType, e->pSong );
|
||||||
RadarValues transformed_rv;
|
RadarValues transformed_rv;
|
||||||
NoteDataUtil::GetRadarValues( nd, e->pSong->m_fMusicLengthSeconds, transformed_rv );
|
NoteDataUtil::CalculateRadarValues( nd, e->pSong->m_fMusicLengthSeconds, transformed_rv );
|
||||||
rv += transformed_rv;
|
rv += transformed_rv;
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
|
|||||||
Reference in New Issue
Block a user