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

Integrated Infiltration in autopilot update #286

Open
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

Arthan-the-one
Copy link
Contributor

@Arthan-the-one Arthan-the-one commented Apr 22, 2023

Updated copy of otto-kirchheims PR for infiltration integration

fixes #238

relevant info from #239 discussion:

  • there is no reason to do casino and Infiltration both, as infiltration is vastly faster (unless SF8)
    • either infiltration or casino should always be turned on to amass starting cash
  • infiltration earnings are relevant to ascension calcs in autopilot
  • infiltration scales negatively with combat stats (more stats => fewer rewards)
  • infiltration does not scale upwards with the exception of a single augment (reward*1.5)

further info:

  • infiltration tops out around 20b, however is extremely useful for grinding reputation (330k)
    • it actually depends on the stat and exp multipliers for combat and charisma: 1 2
      • this implies that rewards should usually not change without installing augments
  • added button to overview to stop infiltrations from autostarting
  • infiltrations are a player skill/timing game, which would be an issue for people who are just bad at this (like me), or who have some sort of disability
  • automation stops infiltrating for money at 20t, at that point the earnings are irrelevant (<0.1%of money)
  • it will get money for traveling by infiltrating a local business (if money multiplier > 0.1)

@Arthan-the-one
Copy link
Contributor Author

Arthan-the-one commented Apr 23, 2023

This needs a way for the player to stop the repeated infiltrations, maybe a button can be added in the overview.
((Also has issues with current implementation of #107 ))

@otto-kirchheim : the check of !options["getMoney"] is true for getmoney = "", which the next bit of code explicity uses ("" is falsy), working on it.

Also working on the button, but UI-Handling must happen in Overview, as everything else disappears, and it's kinda ugly right now.

@alainbryden alainbryden added the work-in-progress not ready for merge label Apr 24, 2023
@alainbryden
Copy link
Owner

As mentioned in #238, I will leave this open. That said, I will close #239 and #173 in favour of this more recent PR to focus future development.

@Kenshij
Copy link

Kenshij commented May 6, 2023

There are two errors in the scripts.
autopilot.js the variable ranGetMoney is not defined, fixed with add line 47 let ranGetMoney = false;
infiltrator.js line 166 on bnMults will be null and the log throws error, fixed with bnMults?.InfiltrationRep and bnMults?.InfiltrationMoney

and even after enabling infiltration ['disable-Infiltration', false] the infiltrator.js script won't be started from autostart.js
problem is tryGetBitNodeMultipliers again it is null and to run the script it needs a value of > 0.5
quick fix autopilot.js line 496 if (bitnodeMults?.InfiltrationRep === undefined) bitnodeMults = { "InfiltrationRep": 1 }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
work-in-progress not ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Infiltration automation
4 participants