Citadel fitting view crashes when slot colors are enabled

[gscottnelson]

Good morning.

When trying to create a citadel fitting, by clicking on any of them. The app gets extremely slow and seems to hang for several seconds. Once the fitting screen opens after click on any Citadel, Moving to the market section or anywhere doesn't seem to work and the application seems to hang. When I resize the window, the window goes black like it can't redraw as if its waiting for some other process to complete.

This on an iMac 27" with core i7 running @3.50 Ghz - Mac OS X 10.11.6

[ppfeufer]

Similar issue on linux, but it just stops working at all.

Traceback (most recent call last):
File "/home/ppfeufer/applications/pyfa/gui/builtinViews/fittingView.py", line 463, in fitChanged
self.refresh(self.mods)
File "/home/ppfeufer/applications/pyfa/gui/builtinViews/fittingView.py", line 590, in refresh
self.SetItemBackgroundColour(i, self.slotColour(mod.slot))
File "/home/ppfeufer/applications/pyfa/gui/builtinViews/fittingView.py", line 560, in slotColour
return self.slotColourMap[slot] or self.GetBackgroundColour()
KeyError: 8

[blitzmann]

@gscottnelson I will check, but I don't have easy access to OS X anymore. =/

@ppfeufer I am hoping it is as simple as this, however someone included a fix for exactly this issue: #700 Weird

[blitzmann]

Oh motherfucker... that fix went into the citadel branch and was never merged back into master. x_x Will fix and test tonight

[gscottnelson]

Thank Ryan –

I appreciate all the work you put in. When the pre-release gets available, I will download and see how it responds.

Thanks again
Scooter

From: Ryan Holmes [email protected]
Reply-To: pyfa-org/Pyfa [email protected]
Date: Monday, August 29, 2016 at 4:19 PM
To: pyfa-org/Pyfa [email protected]
Cc: Scott Nelson [email protected], Mention [email protected]
Subject: Re: [pyfa-org/Pyfa] 1.23.0 - When fitting Citidel (#711)

Oh motherfucker... that fix went into the citadel branch and was never merged back into master. x_x Will fix and test tonight

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com//issues/711#issuecomment-243259923, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJjBBbz7-5yF9L0xYqlUwo9Ta-Zi16g6ks5qk0zrgaJpZM4JvlQB.

---

This message is intended only for the use of the individual(s) or entity to which it is addressed and may contain information that is privileged, confidential, and/or proprietary to RealPage and its affiliated companies. If the reader of this message is not the intended recipient, you are hereby notified that any dissemination, distribution, forwarding or copying of this communication is prohibited without the express permission of the sender. If you have received this communication in error, please notify the sender immediately and delete the original message.

[blitzmann]

The fix has been merged into master, and will be available in the next point release (soonish)