search

herosheets / woin-starships

Starship Creator for WOIN
http://herosheets.github.io/woin-starships/
1 stars 2 forks source link

Crew discount for computers should not add up #65

Closed Morrus closed 8 years ago

Morrus commented 8 years ago

When multiple control computers are in use, the crew discount %'s should not be added together; instead the best one should be used.

jsomara commented 8 years ago

Is this true for multiple types of control computers, or also true for multiple quantities of the same computer?

IE, if I have 10x of Chemerkin-Liang Products LM-2 is my crew discount 10% or 1%?

Morrus commented 8 years ago

Just 1%. You only get one discount, once.

On 7 Jun 2016, at 05:32, Jordan OMara notifications@github.com wrote:

Is this true for multiple types of control computers, or also true for multiple quantities of the same computer?

IE, if I have 10x of Chemerkin-Liang Products LM-2 is my crew discount 10% or 1%?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

Morrus commented 8 years ago

This hasn't been implemented. I just created a ship and it applied the discounts every time a computer was added. Relevant section of the stat block below. As you see, I got it to go into negative crew.

It should apply the discount only once no matter how many computers are selected.

Hull Class XI (INIT -2d6) Traits Crew -400 (cost -80000cr/m); Troops 0; Passengers 0 (0 standard, 0 luxury) Command & Control Systems Computers 20x Newstellar EM-1H Command Computer (CPU cycles: 32; max FTL: 8; checks: +1d6) Sensors none (range 0; check 0)

seiyria commented 8 years ago

Whoops. There's some code there that has a multiplier. I'll remove that next time.

On Tue, Jun 7, 2016, 17:28 Morrus notifications@github.com wrote:

Reopened #65 https://github.com/herosheets/woin-starships/issues/65.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/herosheets/woin-starships/issues/65#event-685059892, or mute the thread https://github.com/notifications/unsubscribe/AAum2UZDOpM0GgscMFyHXettS7Nf2fq1ks5qJfCSgaJpZM4Ijp5B .

seiyria commented 8 years ago

The multiplier should only apply once now. If this isn't the case, I'll need more information (basically what you're doing and what you expect the result to be).