Git Product home page Git Product logo

Comments (38)

zealvurte avatar zealvurte commented on June 2, 2024 4

T.MoreSpells afaik.

from ventureplansodmissions.

glassleo avatar glassleo commented on June 2, 2024 4

I don't know how to use/install the cumulative fixes branch. There's nothing to download or copy like the "latest release" button for Hythloday's page.

Go to Divergentcurl's repo page for the branch and click the green "Code" button in the top right corner and select "Download ZIP".

Note that the folder needs to be renamed to VenturePlanSoDMissions for the game to load it properly. (It will be named VenturePlanSoDMissions-cumulative-fixes in the ZIP archive).

from ventureplansodmissions.

Divergentcurl avatar Divergentcurl commented on June 2, 2024 4

@zealvurte Thanks for the swift update. I have updated my branch as well.

from ventureplansodmissions.

hythloday avatar hythloday commented on June 2, 2024 3

Thanks for this, I'm not playing at the moment but I'll get this merged when I have a moment.

@zealvurte could you message me on Discord? ItsJames#0297

from ventureplansodmissions.

Divergentcurl avatar Divergentcurl commented on June 2, 2024 3

i havent updated ventureplan in ages. where do i start with all the new updates

Yeah this is very all over the place and disorganized.

I was frustrated by this as well. That is why I have gathered all the fixes I know in my cumulative fixes branch. If anyone notices anything missing, please let me know and I will add it.

from ventureplansodmissions.

woefulwabbit avatar woefulwabbit commented on June 2, 2024 2

@hythloday You'll want to merge from https://github.com/Divergentcurl/VenturePlanSoDMissions/tree/cumulative-fixes too

from ventureplansodmissions.

zealvurte avatar zealvurte commented on June 2, 2024 2

There's been a hotfix to fix (most of) the issues I highlighted in the data. I've updated the original post with the changes.

from ventureplansodmissions.

teelolws avatar teelolws commented on June 2, 2024 2

Mikanikos was "nerfed" in 9.2.5. Bugfix to match its actual tooltip. Spell 85 modDamageTaken from -5000 to -50

from ventureplansodmissions.

solkarnar avatar solkarnar commented on June 2, 2024 1

Just dropping in to say thanks to everyone working on this!

from ventureplansodmissions.

LostTemple1990 avatar LostTemple1990 commented on June 2, 2024 1

sim error results and real logs.txt
enemy in slot 5 cast spell 363 to itself in turn 13
the value of key "target" to spell 363 may be friend-front-soft but not friend-front-hard

from ventureplansodmissions.

gaveer avatar gaveer commented on June 2, 2024

@zealvurte in which file i add these changes ?
in extra-vs-spells file there is 2 section :
T.MoreSpells and T.UnverifiedSpells

from ventureplansodmissions.

Johnydoe1977 avatar Johnydoe1977 commented on June 2, 2024

Hi, has the new 9.2 missions been included?

from ventureplansodmissions.

tgeisterfer avatar tgeisterfer commented on June 2, 2024

i havent updated ventureplan in ages. where do i start with all the new updates

from ventureplansodmissions.

glassleo avatar glassleo commented on June 2, 2024

i havent updated ventureplan in ages. where do i start with all the new updates

Yeah this is very all over the place and disorganized.

from ventureplansodmissions.

greghudson avatar greghudson commented on June 2, 2024

[352]={type="aura", modDamageTaken=30, target=4},
[353]={type="aura", modDamageDealt=20, target=1},

I think these need a duration=; otherwise I get lua errors predicting Automa Attack. I'm guessing the value should be 2 based on in-game tooltips, but I'm less certain about that.

from ventureplansodmissions.

zealvurte avatar zealvurte commented on June 2, 2024

Ah, I forgot I had to modify VP for those 2 special cases. Simulations showed these auras with no duration are never cast, so I've replaced them with nop for VP.

from ventureplansodmissions.

Briandris avatar Briandris commented on June 2, 2024

I'm confused and fairly new to Github. I really appreciate those of you who are keeping this addon functional for people like me who might as well be reading a foreign language.

Previously, I followed instructions for "Getting it Running" and "Hotfixing the Code" for extra companions. I installed hythloday's Ventureplansodmissions. It was working great until 9.2 hit this week.

I don't know how to use/install the cumulative fixes branch. There's nothing to download or copy like the "latest release" button for Hythloday's page.

I did see the added change for overideAA and put that in the ventureplan file.
I deleted the old VentureplanSoDMissions and did a fresh download and put it in the addons file
I copy/pasted the new spells kindly posted from @zealvurte.

When I load WoW and the open the adventure table, there are alerts on the missions Prison Break, Gatecrashers, Customary
Shakedown, Defeat the World Eaters and Piratical Problems. Could someone help me understand where I likely went wrong please?

from ventureplansodmissions.

Briandris avatar Briandris commented on June 2, 2024

@glassleo Fixed! Thanks so much!

from ventureplansodmissions.

siweia avatar siweia commented on June 2, 2024

For mission 2323, https://www.wowhead.com/mission=2323/joint-operations-the-hunt, there should be four rewards include follower XP. VP only displayed three reward icons and anchoring weried.
112

from ventureplansodmissions.

solkarnar avatar solkarnar commented on June 2, 2024

For mission 2323, https://www.wowhead.com/mission=2323/joint-operations-the-hunt, there should be four rewards include follower XP. VP only displayed three reward icons and anchoring weried. 112

Seeing that this mission requires 5 icons (1 exp + 4 rep), could it be due to an icon overflow problem? @zealvurte

from ventureplansodmissions.

zealvurte avatar zealvurte commented on June 2, 2024

Almost certainly. I noticed it, but didn't look into resolving it at the time or since, as it's a one off so far.

from ventureplansodmissions.

zealvurte avatar zealvurte commented on June 2, 2024

Verified and corrected, thanks - my logs clearly didn't encounter a case where it made a difference.

Also taking another look at my logs for spell 46, I have none that verify a difference between hard and soft either, so I've unverified that.

from ventureplansodmissions.

Barleduq avatar Barleduq commented on June 2, 2024

I don't know how to use/install the cumulative fixes branch. There's nothing to download or copy like the "latest release" button for Hythloday's page.

Go to Divergentcurl's repo page for the branch and click the green "Code" button in the top right corner and select "Download ZIP".

Note that the folder needs to be renamed to VenturePlanSoDMissions for the game to load it properly. (It will be named VenturePlanSoDMissions-cumulative-fixes in the ZIP archive).

Ok, so the zip file from the cumulative-fixes branch should replace the current version of VPSODM 0.10 ? Or is this more files that I have to insert in places? (I'm comfortable modifying files, but I don't always understand what's going on with repos and commits - my coding days were all in college, way before version control was a thing.)

from ventureplansodmissions.

glassleo avatar glassleo commented on June 2, 2024

Ok, so the zip file from the cumulative-fixes branch should replace the current version of VPSODM 0.10 ? Or is this more files that I have to insert in places? (I'm comfortable modifying files, but I don't always understand what's going on with repos and commits - my coding days were all in college, way before version control was a thing.)

I'm assuming you managed to download the ZIP file correctly, and also followed the instructions for modifying the VenturePlan addon already.

  1. Open your WoW AddOns folder (usually C:\Program Files (x86)\World of Warcraft\_retail_\Interface\AddOns if you are running Windows)
  2. Delete the folder named VenturePlanSoDMissions inside the AddOns folder if it exists
  3. Open up the ZIP file you downloaded from Diventcurl's GitHub
  4. Inside the ZIP, there will be a folder named VenturePlanSoDMissions-cumulative-fixes, drag this into your AddOns folder to extract it, then close the ZIP file
  5. Rename the folder VenturePlanSoDMissions-cumulative-fixes inside your AddOns folder to VenturePlanSoDMissions
  6. That's it

from ventureplansodmissions.

Barleduq avatar Barleduq commented on June 2, 2024

Yes, I've modified the Widgets file when that fix came out in August, and the vs.lua file just now. I downloaded VPSoDM from wowinterface.com. That folder only has 2 files in it, and the cum fixes folder has 2 folders and 3 files more. So I was making sure I was doing the correct thing with it.
I hope I can hear about those fixes being included in the main file (and, presumably uploaded to WoWI) so I can continue my usual addon installation habits.

Thank you for the fast reply!
-Barleduq

from ventureplansodmissions.

Jevnaker avatar Jevnaker commented on June 2, 2024

I tried to use this and now this addon (the ventureplan modifier one) thinks i havent modified the ventureplan files required.

from ventureplansodmissions.

Divergentcurl avatar Divergentcurl commented on June 2, 2024

Please have a look at https://github.com/Divergentcurl/VenturePlanSoDMissions/tree/cumulative-fixes#getting-it-running. It should help you get these fixes running. You only have to do this once.

from ventureplansodmissions.

Jevnaker avatar Jevnaker commented on June 2, 2024

ok ty

from ventureplansodmissions.

Jevnaker avatar Jevnaker commented on June 2, 2024

the issue i have is that i had already done that before implementing this fix thingy in this thread and now it believes i havent done it despite the correct things have been changed like in the link you shared with me.

from ventureplansodmissions.

Divergentcurl avatar Divergentcurl commented on June 2, 2024

I don't know how to help you further with the information you have given me. Maybe you could make a backup of both addons and try again with a fresh install? If that doesn't work out you can always restore the backups.

from ventureplansodmissions.

solkarnar avatar solkarnar commented on June 2, 2024

Mikanikos was "nerfed" in 9.2.5. Bugfix to match its actual tooltip. Spell 85 modDamageTaken from -5000 to -50

[85]={type="aura", target=3, duration=2, firstTurn=3, modDamageTaken=-50},

This should be the new value in the vs-spells.lua file right?

from ventureplansodmissions.

zeenk avatar zeenk commented on June 2, 2024

[85]={type="aura", target=3, duration=2, firstTurn=3, modDamageTaken=-50},

This should be the new value in the vs-spells.lua file right?

Yes

from ventureplansodmissions.

woefulwabbit avatar woefulwabbit commented on June 2, 2024

@hythloday Are you able to add a few collaborators to the project e.g. @Divergentcurl @zealvurte so they can merge updates and create releases?

from ventureplansodmissions.

Barleduq avatar Barleduq commented on June 2, 2024

Adding to the request at least at the point of creating releases such that the CurseBreaker addon can pick it up automagically. (Currently hitting 'code' button for more recent .zip file on Divergentcurl's fork, and installing it as instructed above, and hoping I'm doing all that needs doing.)

from ventureplansodmissions.

Divergentcurl avatar Divergentcurl commented on June 2, 2024

Thanks for reporting the Mikanikos change. I have updated my branch and have created a new project on WOWInterface to make it easier for everyone to update.

from ventureplansodmissions.

Barleduq avatar Barleduq commented on June 2, 2024

Wow, thanks! I've made a comment there for clarification, in case anyone just comes across it there and is confused.

from ventureplansodmissions.

rewzone avatar rewzone commented on June 2, 2024

Please have a look at https://github.com/Divergentcurl/VenturePlanSoDMissions/tree/cumulative-fixes#getting-it-running. It should help you get these fixes running. You only have to do this once.

I'm having an issue following your added instructions around overrideAA. That code doesn't seem to be in the vs.lua file for me. I also noticed I'm running 4.22 which is the current version of Venture Plan on Curse.

from ventureplansodmissions.

greghudson avatar greghudson commented on June 2, 2024

With Venture Plan 4.22 you shouldn't need the fixes from this addon any longer. (At least in my experience; I haven't failed a mission since I updated to 4.22.)

from ventureplansodmissions.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.