mirror of
https://github.com/Krafpy/KSP-MGA-Planner.git
synced 2025-12-05 20:40:13 -08:00
Removed second leg arcs recalcuation.
For resonant swing bys, it looks like it makes the solver privilege orbits with high apoapsis. Looking for a way to fix it.
This commit is contained in:
@@ -125,11 +125,6 @@ class TrajectoryCalculator {
|
||||
infos.duration = Math.max(minLegDuration, infos.duration);
|
||||
}
|
||||
recomputeLegsSecondArcs() {
|
||||
for (let i = 0; i < this._secondArcsData.length; i++) {
|
||||
const data = this._secondArcsData[i];
|
||||
const step = this.steps[3 + i * 3];
|
||||
this._recomputeSecondArc(step, data);
|
||||
}
|
||||
}
|
||||
_recomputeSecondArc(lambertStep, arcData) {
|
||||
const fbBody = this.system[arcData.fbBodyId];
|
||||
|
||||
@@ -185,13 +185,16 @@ class TrajectoryCalculator {
|
||||
* Recomputes the second arc of a all legs accounting this time for SOI enter points,
|
||||
* thus providing a more precise visualization of the orbit.
|
||||
*/
|
||||
public recomputeLegsSecondArcs(){
|
||||
for(let i = 0; i < this._secondArcsData.length; i++){
|
||||
public recomputeLegsSecondArcs(){
|
||||
// FIX: This causes wrong arc orbits recalculations in the case of resonant
|
||||
// swing bys, because of the Lambert solver not considering multirevolutions...
|
||||
// I guess that's the problem ?
|
||||
/*for(let i = 0; i < this._secondArcsData.length; i++){
|
||||
const data = this._secondArcsData[i];
|
||||
const step = this.steps[3 + i*3];
|
||||
|
||||
this._recomputeSecondArc(step, data);
|
||||
}
|
||||
}*/
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
Reference in New Issue
Block a user