Conversation
Per NEXT_STEPS.md Priority 1 (Week 2), investigated signal power calculation chain to address daytime E-layer mode errors and target >80% validation pass rate. ## Key Findings - **Baseline: 72.2% pass rate** (156/216 tests passing) - **Nighttime predictions work well**: ~7 dB deviation (acceptable) - **Daytime above-MUF predictions fail**: 30-60 dB SNR errors - **Root cause identified**: xls penalty calculation for above-MUF operation ## Investigation Results 1. **XLS Penalty Analysis**: When operating above MUF (e.g., 25.90 MHz vs 19.1 MHz MUF), the xls penalty is very large (87+ dB), causing SNR predictions to be 30-60 dB too low compared to VOACAP. 2. **Attempted Fix**: Removed secant multiplication from xls calculation - Result: **Made it worse** (67.6% pass rate vs 72.2%) - Conclusion: Formula is correct as coded 3. **Root Cause**: VOACAP likely uses different algorithm for: - MUF probability calculation (calc_muf_prob) - XLS penalty clamping/limits - Or considers additional propagation modes ## Files Added - `SIGNAL_POWER_INVESTIGATION.md` - Detailed technical analysis with test cases - `debug_signal_power.py` - Debug script showing all loss components - `INVESTIGATION_SUMMARY_2025-11-14.md` - Executive summary and recommendations ## Next Steps To reach >80% pass rate: 1. Investigate calc_muf_prob function vs FORTRAN SIGDIS.FOR 2. Test xls penalty clamping (e.g., max 50 dB) 3. Expand validation test suite per NEXT_STEPS Priority 2 ## Status - Current pass rate: 72.2% (operational and usable) - Engine is fundamentally sound - Issue isolated to daytime/above-MUF scenarios - Ready for Week 3-4 work (test expansion and systematic validation)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Per NEXT_STEPS.md Priority 1 (Week 2), investigated signal power calculation chain to address daytime E-layer mode errors and target
Key Findings
Investigation Results
XLS Penalty Analysis: When operating above MUF (e.g., 25.90 MHz vs 19.1 MHz MUF), the xls penalty is very large (87+ dB), causing SNR predictions to be 30-60 dB too low compared to VOACAP.
Attempted Fix: Removed secant multiplication from xls calculation
Root Cause: VOACAP likely uses different algorithm for:
Files Added
SIGNAL_POWER_INVESTIGATION.md- Detailed technical analysis with test casesdebug_signal_power.py- Debug script showing all loss componentsINVESTIGATION_SUMMARY_2025-11-14.md- Executive summary and recommendationsNext Steps
To reach >80% pass rate:
Status