Ryan Malloy
579bad9921
Add Carryout and Trav'ler Pro to firmware variant table, SK-1000 physical specs, error messages, calibration procedures, emergency stow notes, motor behavior details, and Gpredict setup config. Note leap-frog bug is also present in Pro repo.
46 lines
1.6 KiB
Markdown
46 lines
1.6 KiB
Markdown
# Known Bugs in Original Code
|
|
|
|
## Leap-Frog Elevation Bug
|
|
|
|
**Location:** `Trav-ler-Rotor-For-HAL-2.05/travler_rotor.py` lines 98-105
|
|
|
|
**Issue:** The elevation delta section of the leap-frog algorithm modifies
|
|
`target_az` instead of `target_el`. This is a copy-paste error from the
|
|
azimuth section above it.
|
|
|
|
**Original code (lines 90-105):**
|
|
```python
|
|
# Azimuth compensation (correct)
|
|
if target_az - current_az > 2:
|
|
target_az+=1
|
|
elif target_az - current_az < -2:
|
|
target_az-=1
|
|
elif target_az - current_az > 1:
|
|
target_az+=0.5
|
|
elif target_az - current_az < -1:
|
|
target_az-=0.5
|
|
|
|
# Elevation compensation (BUG: modifies target_az instead of target_el)
|
|
if target_el - current_el > 2:
|
|
target_az+=1 # <-- should be target_el
|
|
elif target_el - current_el < -2:
|
|
target_az-=1 # <-- should be target_el
|
|
elif target_el - current_el > 1:
|
|
target_az+=0.5 # <-- should be target_el
|
|
elif target_el - current_el < -1:
|
|
target_az-=0.5 # <-- should be target_el
|
|
```
|
|
|
|
**Impact:**
|
|
- Elevation leap-frog compensation was never applied, so the dish would
|
|
lag behind in elevation during fast satellite passes
|
|
- Azimuth received double compensation (its own delta + the elevation delta),
|
|
causing over-correction on the azimuth axis
|
|
|
|
**Affected repos:**
|
|
- `saveitforparts/Trav-ler-Rotor-For-HAL-2.05` — `travler_rotor.py` lines 98-105
|
|
- `saveitforparts/Travler-Pro-Rotor` — same bug copy-pasted into `travler_pro_rotor.py`
|
|
|
|
**Fix:** In `travler_rotor.leapfrog.apply_leapfrog()`, the elevation
|
|
compensation correctly modifies `target_el`.
|