{"action":"created","issue":{"url":"https://api.github.com/repos/BobPalmer/MKS/issues/436","labels_url":"https://api.github.com/repos/BobPalmer/MKS/issues/436/labels{/name}","comments_url":"https://api.github.com/repos/BobPalmer/MKS/issues/436/comments","events_url":"https://api.github.com/repos/BobPalmer/MKS/issues/436/events","html_url":"https://github.com/BobPalmer/MKS/pull/436","id":74388032,"number":436,"title":"quick fix for negative cost in VAB","user":{"login":"diomepa","id":10139769,"avatar_url":"https://avatars.githubusercontent.com/u/10139769?v=3","gravatar_id":"","url":"https://api.github.com/users/diomepa","html_url":"https://github.com/diomepa","followers_url":"https://api.github.com/users/diomepa/followers","following_url":"https://api.github.com/users/diomepa/following{/other_user}","gists_url":"https://api.github.com/users/diomepa/gists{/gist_id}","starred_url":"https://api.github.com/users/diomepa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/diomepa/subscriptions","organizations_url":"https://api.github.com/users/diomepa/orgs","repos_url":"https://api.github.com/users/diomepa/repos","events_url":"https://api.github.com/users/diomepa/events{/privacy}","received_events_url":"https://api.github.com/users/diomepa/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-08T14:48:58Z","updated_at":"2015-05-10T19:14:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/BobPalmer/MKS/pulls/436","html_url":"https://github.com/BobPalmer/MKS/pull/436","diff_url":"https://github.com/BobPalmer/MKS/pull/436.diff","patch_url":"https://github.com/BobPalmer/MKS/pull/436.patch"},"body":"I tried to fix negative costs, and \"quick fixed\" most of them by just increasing the base price. I have no clue what causes it in a first place, as most other parts I've seen add the resource cost on top, but not these. So they appear with huge price in the list, but don't cost anywhere near that when empty. I have also trouble fixing the radial container, as it has some funny business going on with tweak scale. The PDUs, somehow won't change the price either."},"comment":{"url":"https://api.github.com/repos/BobPalmer/MKS/issues/comments/100682467","html_url":"https://github.com/BobPalmer/MKS/pull/436#issuecomment-100682467","issue_url":"https://api.github.com/repos/BobPalmer/MKS/issues/436","id":100682467,"user":{"login":"AvitusNerva","id":7943714,"avatar_url":"https://avatars.githubusercontent.com/u/7943714?v=3","gravatar_id":"","url":"https://api.github.com/users/AvitusNerva","html_url":"https://github.com/AvitusNerva","followers_url":"https://api.github.com/users/AvitusNerva/followers","following_url":"https://api.github.com/users/AvitusNerva/following{/other_user}","gists_url":"https://api.github.com/users/AvitusNerva/gists{/gist_id}","starred_url":"https://api.github.com/users/AvitusNerva/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AvitusNerva/subscriptions","organizations_url":"https://api.github.com/users/AvitusNerva/orgs","repos_url":"https://api.github.com/users/AvitusNerva/repos","events_url":"https://api.github.com/users/AvitusNerva/events{/privacy}","received_events_url":"https://api.github.com/users/AvitusNerva/received_events","type":"User","site_admin":false},"created_at":"2015-05-10T19:14:37Z","updated_at":"2015-05-10T19:14:37Z","body":"I commented on issue #433 about how the cost seems to be calculated in the game. Here is a copy:\r\n\r\nAfter a little digging KSP seems to care about the wet cost (with all resources full, not the resource levels set in the config). It seems that KSP sets this total wet cost (TWC for brevity) like this:\r\n\r\nTWC = cost_from_config\r\n\r\nif( TWC <= cost_of_resources( when_full ) )\r\n{\r\nTWC = cost_of_resources( at_quantities_in_config )\r\n}\r\n\r\nThis therefore prevents TWC from being negative and could be a KSP bug because preventing the dry cost from being negative would make more sense. The listed cost is also this TWC."}}