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

Missing icon files #80

Closed
blitzmann opened this issue Apr 22, 2014 · 2 comments
Closed

Missing icon files #80

blitzmann opened this issue Apr 22, 2014 · 2 comments

Comments

@blitzmann
Copy link
Collaborator

This has been bugging me. Market browser has a few missing icons. In the eve.db file, the bad icon file is usually listed as res:/UI..., which leads me to think that there is a conversion issue from eve data -> pyfa data, although I haven't looked into this at all.

Some missing icons:


rigs

drones

sub

@blitzmann
Copy link
Collaborator Author

It seems that even CCP can't get their shit together. You can see missing icons eve on EVElopedia. It's kinda a mess


For missing rig icons

Most icons in the EVE Client and database dump point to an iconFile that looks like this: 34_65. These are fairly easily translated into the official icon pack dump. However, some icons pointed to EVE Client files such as res:/UI/Texture/Icons/123_124_23.png. Not exactly sure why CCP has two different formats.

I'm going to make a pull request that may or may not be the best way, or most future proof way, to fix this. Please refer to said pull request for pros/cons

A proper and sexier fix would be to create a script that parses through everything, and pulls the icons from the official release and auto-converts it and everything. It seems that all icon file management in Pyfa is done manually. But this is pretty low priority methinks as icons don't change often.


For missing racial subsystem icons

Same deal as above. However, the icons that are CCP designated for these market groups look odd. The icons we currently use for race selector on ship browser look much better and will probably be used in release. See below for comparison:

CCP release:
actual

Change to:

better


For drone icons

The market groups and drones themselves simply don't have an icon assigned to them. Looking into a fix -- possibly a an override in the icon fetching code

@blitzmann
Copy link
Collaborator Author

After discussion with Kadesh, merged the pull request into master. As for drones, we'll be leaving them as-is for now. Drones in the client don't have an icon, they have a render. Still looks weird in pyfa as there is a space for the missing icon -- that's a separate issue that may or may not be fixed...

Ebag333 added a commit to Ebag333/Pyfa that referenced this issue May 11, 2017
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

No branches or pull requests

1 participant