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

Cyberware/bioware incompatibilities #724

Merged
merged 3 commits into from
Dec 27, 2023
Merged

Conversation

jdevnull
Copy link
Contributor

@jdevnull jdevnull commented Dec 17, 2023

Add guard for metabolic arrester incompatibilities. According to MM pg 67, the metabolic arrester is not compatible with the adrenal pump or suprathyroid gland.

Remove guard preventing datajack + eye datajack. Multiple datajacks is not unusual in canon (SR3 pg 298), and the get_datajack function is able to handle multiple datajacks.

@jdevnull jdevnull changed the title Metabolic arrester compatibilities Cyberware/bioware incompatibilities Dec 17, 2023
@luciensadi
Copy link
Owner

Sorry, just noticed this PR. I think the datajack code is worth keeping, because there's no coded reason to have multiple datajacks, so it will just risk costing essence when someone doesn't need to spend it. I don't want people picking up essence holes because they did their operations out of order.

@jdevnull
Copy link
Contributor Author

Technically, different datajacks are blocked by different types of clothing (e.g., helmets, gloves), so there can be a very minor benefit in ease of jacking in when switching out equipment configurations, in addition to RP support.

Though it looks like normal datajacks will still be incompatible with induction datajacks since they're both type cyb_datajack. I'll go ahead and revert that bit since it's incomplete anyway.

Copy link
Owner

@luciensadi luciensadi left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@luciensadi luciensadi merged commit 19d7c5e into luciensadi:master Dec 27, 2023
1 check passed
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