jrbudda / KerbalEngineer

Kerbal Engineer v1.0 (current in-development build).
242 stars 30 forks source link

Add check for DeltaVGlobals #93

Closed Lucaspec72 closed 6 months ago

Lucaspec72 commented 6 months ago

In order for the DeltaV Calcs to take into account Propellants ignored by stock deltaV Calcs.

Currently, there doesn't seem to be any way to make ressources act like electriccharge in deltaV calcs (ignore it completely). This should (not tested) fix this, by adding a check to the stock list PropellantsToIgnore from DeltaVGlobals, name self explainatory. Unsure if the ElectricCharge and IntakeAir checks would even be necessary, they might already be in PropellantsToIgnore, but i haven't taken that much time looking into every detail.

This is mainly to outline the fact this is not taken into account when calculating deltaV, and it should.

Lucaspec72 commented 6 months ago

Ressources are stored as hashes, not plain text names. will fix and make a new pull request