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.

Sign up for GitHub

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

Jump to bottom

Allow negative rewards/penalties #149

Closed
jrossignol opened this issue Mar 18, 2015 · 4 comments
Closed

Allow negative rewards/penalties #149

jrossignol opened this issue Mar 18, 2015 · 4 comments
Labels
enhancement
Milestone
1.9.6

Comments

@jrossignol
Copy link
Owner

Requested by Arachnidek (forum post)

Try removing the validations on rewards/penalties to see if it works. Otherwise need to manually account for this.
@jrossignol
Copy link
Owner Author

Ran some tests, KSP will clamp it to zero, unfortunately.

@jrossignol jrossignol added this to the 1.9.0 milestone Nov 29, 2015
@jrossignol jrossignol mentioned this issue Nov 29, 2015
Closed
@inigmatus inigmatus mentioned this issue Dec 1, 2015
Closed
@inigmatus inigmatus mentioned this issue Dec 10, 2015
Closed
@jrossignol jrossignol modified the milestones: 1.9.x, 1.10.0 Jan 15, 2016
@jrossignol jrossignol modified the milestones: 1.9.6, 1.9.x Feb 13, 2016
jrossignol added a commit that referenced this issue Feb 17, 2016
@jrossignol
[#149] Allow negtive rewards.
dc98081
@jrossignol
Copy link
Owner Author

Well, guess I put this one off for long enough that is solved itself... KSP no longer clamps the rewards to zero, so all I had to do was remove the validation. 👍

@inigmatus
Copy link
Contributor

looking forward to testing this. if it works, I'll be able to offer a store of sorts! :D

@inigmatus
Copy link
Contributor

IT WORKS! i could hug you. thank you. funds and science. i suppose rep too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
1.9.6
Development

No branches or pull requests
2 participants
@jrossignol @inigmatus