These forums have been archived and are now read-only.

The new forums are live and can be found at https://forums.eveonline.com/

EVE Technology Lab

 
  • Topic is locked indefinitely.
 

jEveAssets 4.1.2 (2017-06-07)

First post
Author
Golden Gnu
Lobach Inc.
#401 - 2013-02-23 13:14:49 UTC
@Tonto Auri
There is no need to copy anything from any dialog...
Just send me the jeveassets.log file and I will have anything I need...
Please read the link I posted...

Creator of jEveAssets - the asset manager

"Download is the meaning of life, upload is the meaning of intelligent life"

Tonto Auri
Vhero' Multipurpose Corp
#402 - 2013-03-05 17:18:17 UTC
Okay, I sorted the issue.
However, for the future, PLEASE separate API verification errors from API communication errors.
In my case, it was the latter. But I've had no way to tell, until I stumbled upon API proxy setting in application options.
Also, the application REQUIRES access to Account Status, but does not suggest to include it in the predefined key, nor tolerate it's absence (it simply crash upon update, if access to account status is not granted.)

Two most common elements in the universe are hydrogen and stupidity. -- Harlan Ellison

Golden Gnu
Lobach Inc.
#403 - 2013-03-06 00:39:26 UTC
@Tonto Auri
AFAIK Account Status is always granted, for any API Key?
How on earth did you manage to exclude it?
Reproduction steps would be very much appreciated...

The account import is:
o l d - c o d e™
AKA it could probably be done better, but, it works (except for you).
Do not fix, what is not broken...
New code = new bugs
And this is a critical part of the program, that if it breaks, will render jEveAssets useless (as you know first hand)

tl:dr
When I'm able to reproduce it, I will be able to fix it...

Creator of jEveAssets - the asset manager

"Download is the meaning of life, upload is the meaning of intelligent life"

Anele Sunder
One Nerd Army
#404 - 2013-03-06 00:40:06 UTC
This program is incredible...

I'm kicking myself for never using it in the past.

Your work is much appreciated Big smile
Golden Gnu
Lobach Inc.
#405 - 2013-03-06 00:41:43 UTC
@Anele Sunder
Thank you very much! :)

Happy users FTW!!!

Creator of jEveAssets - the asset manager

"Download is the meaning of life, upload is the meaning of intelligent life"

Tonto Auri
Vhero' Multipurpose Corp
#406 - 2013-03-06 13:33:42 UTC
Golden Gnu wrote:
@Tonto Auri
AFAIK Account Status is always granted, for any API Key?

No. Private information, the third checkbox.

Quote:
How on earth did you manage to exclude it?
Reproduction steps would be very much appreciated...

Question is not "how I manged to exclude it", question is "why application crash on user input".

Quote:
The account import is:
o l d - c o d e™
AKA it could probably be done better, but, it works (except for you).
Do not fix, what is not broken...
New code = new bugs
And this is a critical part of the program, that if it breaks, will render jEveAssets useless (as you know first hand)

tl:dr
When I'm able to reproduce it, I will be able to fix it...

For account status, I told ya.

For API communication issues:
[x] Enable API proxy
API proxy address: http://api.eveonline.com/

It claim any api key invalig, instead of telling that there was an issue communicating with API. I would've probably deduced it sooner (from the log file, the error now makes sense), if I saw/remembered that setting.

Two most common elements in the universe are hydrogen and stupidity. -- Harlan Ellison

Desmont McCallock
#407 - 2013-03-06 19:09:20 UTC  |  Edited by: Desmont McCallock
@Golden Gnu
Tonto has years of EVE related 3rd party apps software design consulting as he's an honored 'consultant' member of the EVEMon Development Team (he even predates me on the team). Even when he gets 'irritating' sometimes (that's just his style), he always has a point. We may not always agree but I come out wiser (software wise) from every of our discussions.
Golden Gnu
Lobach Inc.
#408 - 2013-03-06 20:24:02 UTC
@Tonto Auri
1) I'm able to reproduce: Not able to add API Key, with bad API proxy - Great reproduction steps: thank you! :)

2) I'm not able to reproduce any crash - can you give me the reproduction steps?

3) I do not need Account Status on my API Keys for anything to work and jEveAssets only calls the APIKeyInfo API never the AccountStatus API AKA I'm not able to reproduce it. again: reproduction steps

I'm sorry you have to go through fire to get this fixed Sad

@Desmont McCallock
It's a tricky bug, as Tonto Auri pointed out, the log was not very helpful. if the log is bogus, I need to be able to reproduce it locally, to debug it or I can not fix it, sadly...

Creator of jEveAssets - the asset manager

"Download is the meaning of life, upload is the meaning of intelligent life"

Tonto Auri
Vhero' Multipurpose Corp
#409 - 2013-03-06 21:37:21 UTC
Ok, regarding account status: I probably have one more idea, that may help, but first I'll try and get some more debugging locally. A loud say "debugging", though, more like "messing around with buttons", but let's see what comes out of it. Since I was fighting two issues in one, there may be some environmental effects. I was overly excited by my find of that API proxy setting, I didn't though of saving the last crashlog :/ my bad.

Two most common elements in the universe are hydrogen and stupidity. -- Harlan Ellison

Tonto Auri
Vhero' Multipurpose Corp
#410 - 2013-03-06 21:55:36 UTC  |  Edited by: Tonto Auri
Ok, it was probably due to corrupted settings file. I can make application do all the weird things with a settings file, that is missing certain parts of key identity. The key does not appear in key management window, but will allow updating (eventually, it MAY update... or not. Or crash afterward... depends on the state of the key it read from file.)
I don't think it's really applicable. I mean, applicable for fixing. Wrong input is wrong, however you input it.
Probably a simple sanity check around settings file reader would do just fine.

And you may add a note on API verification dialogue to check for network/API proxy settings, if any errors occur, that you can't detect as CCP API messages.

Two most common elements in the universe are hydrogen and stupidity. -- Harlan Ellison

Golden Gnu
Lobach Inc.
#411 - 2013-03-06 22:10:43 UTC
@Tonto Auri
if you send me the corrupted settings file(s) or an example that makes jEveAssets crash (if it's a short file you can just copy 'n' paste it here) - I'll be able to make a test case and it should be super easy to fix :)

Yes, more info is always good - I will:
-Add a paragraph to the readme file about this issue
-Separate EVE API errors and "Real" errors in import dialog (and link the readme file)
All this will take some time, but, I'll try to get it done sooner than soon™ ;)

I hope this is good enough? If you have more ideas, please let me know! :)
Thank you very much for your feedback! \o/

Creator of jEveAssets - the asset manager

"Download is the meaning of life, upload is the meaning of intelligent life"

Tonto Auri
Vhero' Multipurpose Corp
#412 - 2013-03-06 22:36:14 UTC  |  Edited by: Tonto Auri
I don't see any glaring issues from my side, other than what has been mentioned.
It works fine (even better, than I remember it), though it took some time to understand some of the new columns.
The one problem i've is with "Price % Reprocessed" column.
I'd make it just "% Reprocessed" and calcualte it as "ROUND(100 * Reprocessed value % Sell value)"
Then, values less than "100" would mean it's not worth reprocessing. And column header would be shorter... (for some odd reason, column autoresizing works off header width, instead of content width, as it should.)
Because, in my opinion, the most often asked question is "how much I'd gain, if I reprocess it, rather than sell as is", not "how much an assembled item is more valuable, than a pile of minerals".

Two most common elements in the universe are hydrogen and stupidity. -- Harlan Ellison

Golden Gnu
Lobach Inc.
#413 - 2013-03-08 10:47:16 UTC
@Tonto Auri
Auto resize works on both content and header - I like to be able to read the header...
I'll rename the column to make it shorter.
It just work the other way around: if it's bellow 100 it should be reprocessed if it's above 100 it's should be sold...
Also note that you can disable auto resize and it will save the column width you set...

Thank you for all your feedback! :)

Creator of jEveAssets - the asset manager

"Download is the meaning of life, upload is the meaning of intelligent life"

Tonto Auri
Vhero' Multipurpose Corp
#414 - 2013-03-08 12:17:11 UTC
The problem with the way it works now, is that reprocessed materials are not what is used to manufacture the item. Yet you comparing the value of an item to reprocessed materials and tell, how much more valuable the item is... Which is not what I want to know, when reprocessing it.

Two most common elements in the universe are hydrogen and stupidity. -- Harlan Ellison

Golden Gnu
Lobach Inc.
#415 - 2013-03-08 16:24:06 UTC
@Tonto Auri
I'm sorry, I do not understand what you mean Sad

Creator of jEveAssets - the asset manager

"Download is the meaning of life, upload is the meaning of intelligent life"

Tonto Auri
Vhero' Multipurpose Corp
#416 - 2013-03-08 16:41:14 UTC  |  Edited by: Tonto Auri
The column answers the question "how the item is more profitable, than a pile of minerals it can be reprocessed into", not "how much I gain, if I reprocess the item and sell minerals".
The first answer is pretty much useless, as the results swing wildly (I have values ranging from 0% to hundreds of thousands of percents, which makes the column nearly useless).
With the latter, you'll rarely get answers over 100%, and almost never will they exceed 3 digits, not to say, spring to multiple thousands. Which will make a column more readable, and more useful. IMO.

Two most common elements in the universe are hydrogen and stupidity. -- Harlan Ellison

Golden Gnu
Lobach Inc.
#417 - 2013-03-08 16:47:37 UTC
@Tonto Auri
I agree, it needs to be changed.

I think there is some kind of language barrier and it's frustrating when I can not understand what you write *sigh*
So: Thank you for explaining it to me! :)

Creator of jEveAssets - the asset manager

"Download is the meaning of life, upload is the meaning of intelligent life"

Tonto Auri
Vhero' Multipurpose Corp
#418 - 2013-03-08 18:18:20 UTC  |  Edited by: Tonto Auri
Question. Does it support corp API keys?... nvm was environmental issue. Working now.

Two most common elements in the universe are hydrogen and stupidity. -- Harlan Ellison

Golden Gnu
Lobach Inc.
#419 - 2013-03-08 18:21:52 UTC
@Tonto Auri
Yes...

Creator of jEveAssets - the asset manager

"Download is the meaning of life, upload is the meaning of intelligent life"

themaker1971
Republic University
Minmatar Republic
#420 - 2013-03-11 23:20:45 UTC
Great program.
I get minerals in a location where i have none showing up and affecting the grand total.
Its only happened in the last few months.
Any idea?