{"action":"created","issue":{"url":"https://api.github.com/repos/BobPalmer/UmbraSpaceIndustries/issues/37","labels_url":"https://api.github.com/repos/BobPalmer/UmbraSpaceIndustries/issues/37/labels{/name}","comments_url":"https://api.github.com/repos/BobPalmer/UmbraSpaceIndustries/issues/37/comments","events_url":"https://api.github.com/repos/BobPalmer/UmbraSpaceIndustries/issues/37/events","html_url":"https://github.com/BobPalmer/UmbraSpaceIndustries/issues/37","id":123318628,"number":37,"title":"Average pulse drive thrust is unaffected by the throttle position","user":{"login":"tsbockman","id":1332026,"avatar_url":"https://avatars.githubusercontent.com/u/1332026?v=3","gravatar_id":"","url":"https://api.github.com/users/tsbockman","html_url":"https://github.com/tsbockman","followers_url":"https://api.github.com/users/tsbockman/followers","following_url":"https://api.github.com/users/tsbockman/following{/other_user}","gists_url":"https://api.github.com/users/tsbockman/gists{/gist_id}","starred_url":"https://api.github.com/users/tsbockman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tsbockman/subscriptions","organizations_url":"https://api.github.com/users/tsbockman/orgs","repos_url":"https://api.github.com/users/tsbockman/repos","events_url":"https://api.github.com/users/tsbockman/events{/privacy}","received_events_url":"https://api.github.com/users/tsbockman/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-12-21T17:07:58Z","updated_at":"2015-12-21T18:51:52Z","closed_at":null,"body":"My testing indicates that the Orion drive always produces the same amount of thrust (averaged over a few cycles), even if I set the throttle as low as 1/3.\r\n\r\nBased on the source code, it appears that the plug-in currently responds to reduced throttle by simply lowering the frequency of the roughly sinusoidal thrust curve (and the animation). This has no affect on the average thrust at all; the longer low-thrust periods are exactly cancelled out by the longer high-thrust periods. (It also makes no sense physically.)\r\n\r\nThis also means that the `maxPulseTime` and `minPulseTime` *cfg* parameters are purely cosmetic, whereas I had expected the thrust to be scaled by `1.0 / minPulseTime`.\r\n\r\nPossible fixes:\r\n* Don't support throttling, except as on/off. This would be entirely reasonable, given how an NPE works.\r\n* Make each pulse last `minPulseTime`, but after each pulse zero out the thrust completely for a throttle-dependant delay before the next one.\r\n* Instead of messing with the timing, Use the throttle to select the NPU type - for example, 100% throttle for NPU-2500, vs 20% throttle for NPU-250.\r\n* Or, Just directly scale the thrust - chalk it up to [variable yield](https://en.wikipedia.org/wiki/Variable_yield) NPUs. Efficiency (effective specific impulse) would be lower when throttling.\r\n\r\nPersonally, I'd like to see throttle-based selection of the NPU type, combined with a magazine UI that enables easy mix-and-match of sizes. I think it's the most realistic option, and shouldn't be too hard to implement."},"comment":{"url":"https://api.github.com/repos/BobPalmer/UmbraSpaceIndustries/issues/comments/166388334","html_url":"https://github.com/BobPalmer/UmbraSpaceIndustries/issues/37#issuecomment-166388334","issue_url":"https://api.github.com/repos/BobPalmer/UmbraSpaceIndustries/issues/37","id":166388334,"user":{"login":"SixDasher","id":16107012,"avatar_url":"https://avatars.githubusercontent.com/u/16107012?v=3","gravatar_id":"","url":"https://api.github.com/users/SixDasher","html_url":"https://github.com/SixDasher","followers_url":"https://api.github.com/users/SixDasher/followers","following_url":"https://api.github.com/users/SixDasher/following{/other_user}","gists_url":"https://api.github.com/users/SixDasher/gists{/gist_id}","starred_url":"https://api.github.com/users/SixDasher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SixDasher/subscriptions","organizations_url":"https://api.github.com/users/SixDasher/orgs","repos_url":"https://api.github.com/users/SixDasher/repos","events_url":"https://api.github.com/users/SixDasher/events{/privacy}","received_events_url":"https://api.github.com/users/SixDasher/received_events","type":"User","site_admin":false},"created_at":"2015-12-21T18:51:52Z","updated_at":"2015-12-21T18:51:52Z","body":":+1:  I didn't look that far into it, just that it was off. I haven't really used the mod, as applications are limited."}}