Skip to content

Oil Peak and Tavern Optimization and Grouping - #285

Open
Phate4569 wants to merge 9 commits into
soolar:masterfrom
Phate4569:Oil-Peak-And-Tavern-Optimization-With-Grouping
Open

Oil Peak and Tavern Optimization and Grouping#285
Phate4569 wants to merge 9 commits into
soolar:masterfrom
Phate4569:Oil-Peak-And-Tavern-Optimization-With-Grouping

Conversation

@Phate4569

Copy link
Copy Markdown
Contributor

Replaces my Previous Pull Request for a similar issue. I re-re-did it since there have been so many changes

Added - Oil Peak and Tavern Optimization and Grouping

In testing saved 8 Adv on Oil Peak. The Tavern is harder to quantify.

Comment thread RELEASE/scripts/sl_ascend.ash Outdated
Comment thread RELEASE/scripts/sl_ascend.ash Outdated
Comment thread RELEASE/scripts/sl_ascend.ash
Comment thread RELEASE/scripts/sl_ascend.ash
rugrat added 2 commits July 26, 2019 12:58
…evaluate end of the Quest (Thanks jaspercb!)
…paro). Also removed useless DEBUG line I copied over from my test version.
@Phate4569

Phate4569 commented Jul 29, 2019

Copy link
Copy Markdown
Contributor Author

Closes #255 #257 #155

@jaspercb

Copy link
Copy Markdown
Contributor

https://help.github.com/en/articles/closing-issues-using-keywords for proper usage

Comment thread RELEASE/scripts/sl_ascend.ash Outdated
set_property("sl_oilpeak", "finished");

// Reset basline
set_property("sl_maximize_baseline", "");

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This property should only be touched by the user. I think you're looking for resetMaximize()

sl_change_mcd(11);

// Set baseline to force maximizer usage
set_property("sl_maximize_baseline", "1000ml 100max");

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Again, don't fiddle with this property automatically.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I added this one (and the one previously) because maximize didn't seem to be setting anything at all until I added a value.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

An addToMaximize call didn't work? Did you have sl_maximize_baseline set to disabled or something? Either way you can't do this, since it would override a user's personal settings, if any.

handleFamiliar("item");

// Reset baseline again, just in case
set_property("sl_maximize_baseline", "");

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Still no.

Comment thread RELEASE/scripts/sl_ascend.ash Outdated
Comment thread RELEASE/scripts/sl_ascend.ash Outdated
print("Checking to see if we should do the tavern while we are running high ML.", "green");
set_property("sl_forceTavern", true);

// Remove Driving Wastefully if we had it

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why bother? It's not like it stops us from getting another buff when we actually want to get one, and it's harmless.

@Phate4569 Phate4569 Aug 19, 2019

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Somehow it does, possibly the Asdon Buff function is broken? If I don't put this in it keeps driving Wastefully when it goes to the the next area. I found this while trying to figure out why it wasn't buffing Tavern properly.

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.

3 participants