Change employment overlaps to informational only
- Rename "Employment Overlap" to "Concurrent Employment" - Remove score penalty for overlaps (often legitimate: part-time, consulting, transitions) - Change severity from Warning/Critical to Info - Update scoring explanation to note overlaps are not penalised Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -397,22 +397,17 @@ public sealed class ProcessCVCheckJob
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
// Penalty for overlaps (only if > 2 months)
|
// Note overlaps - these are often legitimate (part-time, consulting, transitions)
|
||||||
|
// Only flag as informational, no score penalty
|
||||||
foreach (var overlap in timeline.Overlaps)
|
foreach (var overlap in timeline.Overlaps)
|
||||||
{
|
{
|
||||||
var excessMonths = Math.Max(0, overlap.Months - 2); // Allow 2 month transition, prevent negative
|
|
||||||
var overlapPenalty = excessMonths * OverlapMonthPenalty;
|
|
||||||
score -= overlapPenalty;
|
|
||||||
|
|
||||||
var severity = overlap.Months >= 6 ? FlagSeverity.Critical : FlagSeverity.Warning;
|
|
||||||
|
|
||||||
flags.Add(new FlagResult
|
flags.Add(new FlagResult
|
||||||
{
|
{
|
||||||
Category = FlagCategory.Timeline.ToString(),
|
Category = FlagCategory.Timeline.ToString(),
|
||||||
Severity = severity.ToString(),
|
Severity = FlagSeverity.Info.ToString(),
|
||||||
Title = "Employment Overlap",
|
Title = "Concurrent Employment",
|
||||||
Description = $"{overlap.Months} month overlap between '{overlap.Company1}' and '{overlap.Company2}' ({overlap.OverlapStart:MMM yyyy} to {overlap.OverlapEnd:MMM yyyy})",
|
Description = $"Worked at both '{overlap.Company1}' and '{overlap.Company2}' simultaneously for {overlap.Months} months ({overlap.OverlapStart:MMM yyyy} to {overlap.OverlapEnd:MMM yyyy})",
|
||||||
ScoreImpact = -overlapPenalty
|
ScoreImpact = 0
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -200,7 +200,7 @@
|
|||||||
<li>Director claims not verified: <strong>-20 pts</strong> each</li>
|
<li>Director claims not verified: <strong>-20 pts</strong> each</li>
|
||||||
<li>Employment before company existed: <strong>-20 pts</strong></li>
|
<li>Employment before company existed: <strong>-20 pts</strong></li>
|
||||||
<li>Employment gaps: <strong>-1 to -10 pts</strong> per gap</li>
|
<li>Employment gaps: <strong>-1 to -10 pts</strong> per gap</li>
|
||||||
<li>Excessive overlaps: <strong>-2 pts</strong> per month</li>
|
<li>Concurrent employment is noted but not penalised</li>
|
||||||
</ul>
|
</ul>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
Reference in New Issue
Block a user