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

Assets mapping to wrong Stations #27

Closed
clstb opened this issue May 16, 2016 · 9 comments
Closed

Assets mapping to wrong Stations #27

clstb opened this issue May 16, 2016 · 9 comments
Assignees
Labels

Comments

@clstb
Copy link

clstb commented May 16, 2016

Hello,

For me LMEVE is mapping the Assets to a wrong Station. The Assets are fine but located under the wrong Station. Also is SOV calculated in Fuelblockconsumption ?

@roxlukas
Copy link
Owner

Thanks for your bug report.

To investigate I would need the asset data from your instance (raw xml from
api cache under lmeve/var/ and apiassets table from mysql so I can compare
them and verify if the api poller loads the data correctly to db). Are you
willing to share this information?

Fuelblock consumption doesn't take into account TCU bonus. It is however
rather simple feature to add, so I've added it to the ToDo list.
On May 16, 2016 3:54 PM, "ClaasStoertenbecker" notifications@github.com
wrote:

Hello,

For me LMEVE is mapping the Assets to a wrong Station. The Assets are fine
but located under the wrong Station. Also is SOV calculated in
Fuelblockconsumption ?


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#27

@clstb
Copy link
Author

clstb commented May 17, 2016

Because your tool is so amazing im surely willing to provide you the information you need to fix the bug. Just tell me on what platform you want me to send it to you because i dont want it to share that openly.

Btw. can i somehow contact you on a easier way ? GitHub sure is great for bug reports but since your tool isnt that well documented it is quite hard to get into it for new users (besides the installation, it is well documented if you know how2linux).

cheers Claas Störtenbecker

@roxlukas
Copy link
Owner

Please send it to my e-mail at pozniak(dot)lukasz(at)gmail(dot)com

Thanks!

2016-05-17 16:09 GMT+02:00 ClaasStoertenbecker notifications@github.com:

Because your tool is so amazing im surely willing to provide you the
information you need to fix the bug. Just tell me on what platform you want
me to send it to you because i dont want it to share that openly.

Btw. can i somehow contact you on a easier way ? GitHub sure is great for
bug reports but since your tool isnt that well documented it is quite hard
to get into it for new users (besides the installation, it is well
documented if you know how2linux).

cheers Claas Störtenbecker


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#27 (comment)

@roxlukas roxlukas added the bug label May 17, 2016
@roxlukas roxlukas self-assigned this May 17, 2016
@clstb
Copy link
Author

clstb commented May 17, 2016

Done

@clstb
Copy link
Author

clstb commented May 24, 2016

Have you got my E-Mail? Im just curious because there wasnt any reply.

@roxlukas
Copy link
Owner

I did yes, but been very busy and didnt look into it yet.

I'll get back to you as soon as I find the cause

Best regards,
Łukasz Poźniak

On 24 maj 2016, at 17:18, ClaasStoertenbecker notifications@github.com wrote:

Have you got my E-Mail? Im just curious because there wasnt any reply.


You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub

@clstb
Copy link
Author

clstb commented May 27, 2016

I checked how lmeve.apiassets is mapped to eve_cit_dbo.staStations and found no problem there. So i think either the API somehow messes up in pulling the right locationID or the eve dump is wrong. The both stations are btw. only 1 digit apart. (60014943 and 60014944).

@roxlukas
Copy link
Owner

Bug confirmed. Fix included in the upcoming patch.

roxlukas added a commit that referenced this issue Jun 1, 2016
Implements the following enhancements and fixes the following issues:

#33 Cannot set price and cost index solar systems bug
#32 Task tracking. bug
#30 Enhancement Suggestion - Table crestindustrysystems enhancement
#29 POS Silos enhancement
#27 Assets mapping to wrong Stations bug
@roxlukas
Copy link
Owner

roxlukas commented Jun 1, 2016

Fixed in d132dd5

@roxlukas roxlukas closed this as completed Jun 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants