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

Capital Module Rework/Minor edits to Sub-cap Modules #1883

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

taelon875
Copy link
Contributor

@taelon875 taelon875 commented Sep 5, 2024

Fixes:
Bulwark and Repair Field now don't loop to provide extra healing, They also fixed Visual Timers.

Changed:
All Capital Ship Modules lie in their own category in the Engineering Bench.

Targeting Array - No longer gives Attack Boost (see Weapons Computer) instead of evasion decrease it is now damage.

Advanced Thrusters - No Value Changes here, just a description edit.

Reinforced Plating - No longer gives evasion decrease, but does decrease shields instead.

Quad Lasers: Adjusted oversized Fire Range and increase capacitor costs.

Turbolasers: Increased Damage from Module Bonus.

Laser Cannon Battery: Adjusted Range and Damage. (Slight increase on Lower-End Boss Damages.)

Beam Cannon/Laser Cannon: Adjusted Ranges and Cap Costs.

Bulwark/Repair Field - Both now heal larger amounts to cover for single use fire mechanics opposed to original design.

Missile Weapons - Adjusted the "minimum shield value" to 10.

Capital Ship Configuration Changes -
Scaling for Hull/Shields Increased Overall.
Warship - No Longer gains Attack Bonus. Slight increase in Armor values and health boost. But Gains Accuracy Increase. Logistics - Small Shield Increase and Armor Increase. Skirmisher - Lost Accuracy Increase, Gained Attack Increase and larger shield Pool.

Added:
Weapons Computer - Increased Damage for Lower Accuracy Power Diverter - Lower Cap for Faster shield recharge E-War Module - Generates Enmity

https://docs.google.com/spreadsheets/d/1g0DB4i-zY4bQQzifilb1OrbiFn28FHK5Zg_29RxK4Yo/edit?usp=sharing

taelon875 and others added 5 commits September 5, 2024 00:15
Fixes:
Bulwark and Repair Field now don't loop to provide extra healing, They also fixed Visual Timers.

Changed:
All Capital Ship Modules lie in their own category in the Engineering Bench.

Targeting Array - No longer gives Damage Boost (see Weapons Computer) instead of evasion decrease it is now damage.

Advanced Thrusters -  No Value Changes here, just a description edit.

Reinforced Plating -  No longer gives evasion decrease, but does decrease shields instead.

Quad Lasers:  Adjusted oversized Fire Range and increase capacitor costs.

Turbolasers: Increased Damage from Module Bonus.

Laser Cannon Battery: Adjusted Range and Damage. (Slight increase on NPC Damages.

Beam Cannon/Laser Cannon: Adjusted Ranges and Cap Costs.

Bulwark/Repair Field - Both now heal larger amounts to cover for single use fire mechanics opposed to original design.

Missile Weapons - Adjusted the "minimum shield value" to 10.

Capital Ship Configuration Changes -
Scaling for Hull/Shields Increased Overall.
Warship - No Longer gains Damage Bonus.  Slight increase in Armor values and health boost. But Gains Accuracy Increase.
Logistics - Small Shield Increase and Armor Increase.
Skirmisher -  Lost Accuracy Increase, Gained Damage Increase and larger shield Pool.

Added:
Weapons Computer - Increased Damage for Lower Accuracy
Power Diverter  - Lower Cap for Faster shield recharge
E-War Module - Generates Enmity

https://docs.google.com/spreadsheets/d/1g0DB4i-zY4bQQzifilb1OrbiFn28FHK5Zg_29RxK4Yo/edit?usp=sharing
Changed from Damage to Attack to align with rest of modules.
Changed Desc for uniformity.
.Name(name)
.ShortName(shortName)
.Texture("iit_ess_252")
.Description("This weapons computer increases all Attack by 20 but reduce Accuracy by 10.")
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reduce -> reduces

@@ -130,6 +130,8 @@ public enum RecipeCategoryType
SpecialSubmissionItems = 60,
[RecipeCategory("Starship Ammo", true)]
StarshipAmmo = 61,
[RecipeCategory("Capital Module Recipes", true)]
CapitalShipModule =62,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

space between the equals and the 62

@@ -2502,6 +2502,11 @@ public enum RecipeType
CorvetteJehaveyFrigate = 4794,

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you don't need a gap of empty space here

public MigrationExecutionType ExecutionType => MigrationExecutionType.PostCacheLoad;
public void Migrate()
{
RecalculateAllShipStats();
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please confirm this recalc works and we won't run into the same problem as last time.

Copy link
Owner

@zunath zunath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting changes

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

Successfully merging this pull request may close these issues.

2 participants