Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

seatExternalCmd not recognized in HasCrewCapacity parameter #505

Closed
inigmatus opened this issue Apr 30, 2016 · 4 comments
Closed

seatExternalCmd not recognized in HasCrewCapacity parameter #505

inigmatus opened this issue Apr 30, 2016 · 4 comments

Comments

@inigmatus
Copy link
Contributor

From: http://forum.kerbalspaceprogram.com/index.php?/topic/91625-112-contract-configurator-v1113-2016-04-30/&do=findComment&comment=2551580

@inigmatus inigmatus changed the title seatExternalCmd not recognized in seat count seatExternalCmd not recognized in HasCrewCapacity parameter Apr 30, 2016
@jrossignol
Copy link
Owner

Response from the thread:

Checked, and all Contract Configurator does here is tally up the part.CrewCapacity values. For a stock command seat that will be zero (and as far as I can tell always has been). If TakeCommand isn't setting that value when it adds itself then I'd take the issue over there.

I don't think that there's anything wrong with what Contract Configurator is doing here, this is just how stock KSP treats seats (and one of the things that TakeCommand changes/fixes).

@inigmatus
Copy link
Contributor Author

What is HasCrewCapacity counting specifically from KSP that I need to bring back to TakeCommand to have Sean confirm it is making the seat available to CC for counting?

@jrossignol
Copy link
Owner

Part.CrewCapacity

@jrossignol
Copy link
Owner

Not sure if this is fixed in TakeCommand yet, but going to close out this issue. Can re-open if it turns out there is something for CC to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants