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

[WIP] Autodoc software modules V2 #27623

Closed
wants to merge 1 commit into from
Closed

[WIP] Autodoc software modules V2 #27623

wants to merge 1 commit into from

Conversation

Firestorm01X2
Copy link
Contributor

@Firestorm01X2 Firestorm01X2 commented Jan 14, 2019

Implementing autodoc software modules. Special items that can be found and placed near autodoc to increase player chance to install CBMs.

Additionaly first module will be built in in all Autodocs by defult. So player get some bonus anyway. Installing CBMs will be easier by defualt. But only using autodoc.

Manual installation from mod and from NPC should be unaffected. Manual installation still well be disabled by default.

Summary

SUMMARY: Features "Autodoc software modules"

Purpose of change

Adding more depth to autodoc functionality.

Describe the solution

Implementing 3 levels of sofware modules.

  1. 1.0 "Basic Edition". Built in in all autodocs. Allow to install basic CBMs, like Alarm System CBM even for zero skill player with at least 50% chance.

  2. 2.0 "Deep Surgical Edition" . Should be found on external flash drive. Allow to instal more complex CMBs more easily.

  3. 3.0 "Military Edition". Rare module. Allow to install almost everything with hight chance even with no player expirience.

Modules should be placed near Autodoc or can be in player inventory. Just like anastetic kits.
Software module DOES NOT consumed via installation.

Describe alternatives you've considered

Additional context

[WIP] Autodoc software modules
@Firestorm01X2
Copy link
Contributor Author

@kevingranade Is it ok now?

@nexusmrsep
Copy link
Contributor

Will they be placed on SD cards throughout the world?

uilist amenu;
amenu.text = _( "Autodoc Mk. XI. Status: Online. Please choose operation." );
amenu.text = string_format(_( "Autodoc Mk. XI. Status: Online. Version: %s . Please choose operation." ));
Copy link
Contributor

Choose a reason for hiding this comment

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

Need to add the version string as a parameter to string_format:
string_format( _( "Autodoc Mk. XI. Status: Online. Version: %s . Please choose operation." ), version_string );

@kevingranade
Copy link
Member

You don't need NO rationale, you need a workable rationale.
"Adding more depth" is not a sufficient rationale, and neither is a link to a meme-laden reddit thread.
Any adjustment in this area needs a rationale and expected balance outcomes before it needs a mechanism to make the balance adjustments.

It looks like you're having the software act as a general-purpose bonus to success of the activity, which makes no sense in the context of the player programming the autodoc for a specific procedure.

@Firestorm01X2
Copy link
Contributor Author

Firestorm01X2 commented Jan 14, 2019

@kevingranade
You see. You don't need workable rationale.

You are just looking for reasons to close my PRs. This is way too obvious now.

@Firestorm01X2
Copy link
Contributor Author

Firestorm01X2 commented Jan 14, 2019

Too bad that project lead starts acting like that.

@Firestorm01X2
Copy link
Contributor Author

Firestorm01X2 commented Jan 15, 2019

Will they be placed on SD cards throughout the world?

It was planned as separate object.
But now, probably not)

@Firestorm01X2
Copy link
Contributor Author

Firestorm01X2 commented Jan 17, 2019

To get another "It makes no sense"?

@CleverRaven CleverRaven deleted a comment from SilearFlare Jan 17, 2019
@SilearFlare
Copy link

SilearFlare commented Jan 17, 2019

Maek modd
Distribute it with discourse as an unofficial mod
Make them fear your girth

@Firestorm01X2
Copy link
Contributor Author

Firestorm01X2 commented Jan 17, 2019

Maek modd
Distribute it with discourse as an unofficial mod

The point was to do in main repo. Otherwise it likely will be just lost in time.
Also changes like that reqiure noticable changes in the source code. So it will be not mod but separate game version. Additionally saves will not be compatible - because I need new items here.

Make them fear your girth

The only person I have problems here is Kevin. There is no "them".

@kevingranade
Copy link
Member

@tyrael93 stop spamming the issue tracker, this is your last warning.

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.

5 participants