pyfa-org / Pyfa

Python fitting assistant, cross-platform fitting tool for EVE Online
GNU General Public License v3.0
1.61k stars 406 forks source link

Citadel Pre-Release: Siege/Triage/Bastion crash (singularity-1027729) #558

Closed JoshuaNotowitz closed 8 years ago

JoshuaNotowitz commented 8 years ago

Siege and Triage mods appear to be broken and break fits when you attempt to fit them.

blitzmann commented 8 years ago

Hrm, yeah, I guess I should at least fit them next time to make sure they don't break.

Attached is a zip containing fixed effect files. Go ahead and copy these into the pyfa/library.zip/eos/effects/ directory and allow it to overwrite the ones there. I'm going to leave this open so that others can find the fix. Thanks for letting me know! effect_files.zip

JoshuaNotowitz commented 8 years ago

Siege isn't affecting phoenix range atm. it should be giving it a 150% bonus

blitzmann commented 8 years ago

Removing fixed label until I can take a look.

blitzmann commented 8 years ago

I can only confirm for Torps, not XL torps or cruise missiles. Is this the case ingame/sisi?

Hoerks commented 8 years ago

The networked sensor array also broke fits, I attempted to fix it.

Disclaimer: I have no actual idea how this code (or any other code, for that matter) works, but by replicating the changes you did to the bastion file I think I was able to unbreak it. Monkey see, monkey do. Use at your own discretion. Also, it's a passive mod in pyfa, while it is an active one on sisi.

networkedsensorarray.zip

P.S. at least the Nidhoggur has the old, TQ Powergrid&CPU stats, not the new ones, that are on SiSi.

blitzmann commented 8 years ago

The SISI dump that the prerelease uses is not about a week old, so it's very possible some stats (such as cpu, etc) are incorrect to latest SISI.

I will take a look at the Network module and report back =)

blitzmann commented 8 years ago

Yep, that worked. Fixed on my branch =) Thanks!

hipplewm commented 8 years ago

Where do you put this file on a Mac?

JoshuaNotowitz commented 8 years ago

hipplewm show package contents on pyfa, then screen shot 2016-04-22 at 10 51 48 am is the path

hipplewm commented 8 years ago

I don't seem to have as many folders as you do.......I have show invisible files on

screen shot 2016-04-22 at 5 03 06 pm
blitzmann commented 8 years ago

@hipplewm the mac is a bit different. The source files are included with other python packages instead of separated as they wre in the windows build. So the path you are looking for is:

pyfa.app\Contents\Resources\lib\python2.7\site-packages.zip\eos\effects

I believe that you can't natively copy a file into a zip on OSX, so you'll have to extract site-packages.zip, moke the modifications, and zip it back up. =)

@JoshuaNotowitz is still using the deprecated version it seems (why? are there still major bugs with the wx3 update? I honestly haven't looked at the MAC build in months <_<)

hipplewm commented 8 years ago

Hmm, well, I did what you said, worked fine, but now the program won't even start...Grrr

I'll just wait for the patch....

blitzmann commented 8 years ago

What do you get when you run pyfa.app/MacOS/pyfa directly from the terminal?

hipplewm commented 8 years ago

I get no such file or directory i changed the directory to applications and still get the same thing, do I have to change the "MacOS" to my drive name? - edit, that didn't seem to work either

hipplewm commented 8 years ago

Hmm, ok, may have a bigger problem, even if i swap back to original sitepackages.zip, won't work, but I updated java after i ran it last.......wonder if that messed with something. I running 8.91 now i think

hipplewm commented 8 years ago

OK, it didn't revert correctly, when I deleted the new zip and replaced with the old zip, it runs fine now

blitzmann commented 8 years ago

You may even be able to just unzip it and keep it unzipped as long as you get rid of the previous zip file. Also, make sure it's named site-packages and not sitepackages like your comment implies. =) If you can't get it to work, jut wait for the release next week

hipplewm commented 8 years ago

OK, unzipped the file, left it unzipped, swapped out the 3 files you made and the one file for the networksensor array and all works good, just left it unzipped. Thanks

JoshuaNotowitz commented 8 years ago

@blitzmann i still use the deprecated version because I've had a couple problems with the new one, nothing major, just a little bit of overall performance but enough to make a difference

blitzmann commented 8 years ago

@JoshuaNotowitz what kind of performance issues? Also, are you using 10.11? Because I know theres a few graphical bugs with wx3 and 10.11 due to wxpython/wxwidgets itself... =(

hipplewm commented 8 years ago

10.10.5 here on a 2011 MBP

Runs just fine, no issues, but not a heavy user

JoshuaNotowitz commented 8 years ago

yea I'm on 10.11 Everything just seems to run slower, searching for fits, for mods, putting mods on, than on the deprecated version.

blitzmann commented 8 years ago

Marking this as fixed again as all items in here should have been addressed