Skip to content
This repository has been archived by the owner on Oct 21, 2024. It is now read-only.

Incorrect pickup #7

Closed
jarquafelmu opened this issue Apr 10, 2020 · 13 comments
Closed

Incorrect pickup #7

jarquafelmu opened this issue Apr 10, 2020 · 13 comments
Labels
bug Something isn't working

Comments

@jarquafelmu
Copy link

I am not sure what this alert is telling me, its taking things to the correct place

image

@raiguard
Copy link
Collaborator

Someone else also reported this, I'm looking into it. For now you can disable the popups in the mod settings. Thank you!

@raiguard raiguard added the bug Something isn't working label Apr 10, 2020
@raiguard
Copy link
Collaborator

I've released a patch, please try it now!

@jarquafelmu
Copy link
Author

I am still getting these alerts

image

@raiguard
Copy link
Collaborator

Could you please send me your save file? You can PM me on Discord or post a link here.

@raiguard
Copy link
Collaborator

Alternatively, you could type this command into the console and send me a screenshot of what it does:

/c __LtnManager__ game.print(serpent.block(global.data.alerts))

@jarquafelmu
Copy link
Author

I should have sent this before, I apologize for not doing so
https://drive.google.com/open?id=1jqr1qUfBQkbW_xGrKgo7Uzelu1SZA7sl

@raiguard
Copy link
Collaborator

raiguard commented Apr 10, 2020

It's fine. It's a good idea for the future though, if you're experiencing a bug, to post a link to the save file.

Thanks, I'll take a look and see what I can find!

@jarquafelmu
Copy link
Author

Yeah I do it for all of my reports on the mod portal, it just slipped my mind here

@raiguard
Copy link
Collaborator

So, the numbers actually are off. The pumps you're using are so fast that they're overfilling the fluid wagons so the numbers don't match. I didn't think of this when I made the alert.

What I'm going to do is if the actually shipped amount is within a reasonable margin of error (say, 1000), I'll let it slide. I'll also make this a configurable option so those like you with an insane amount of mods can make it work.

Thank you for the report!

@raiguard
Copy link
Collaborator

Well, the same thing happens with items. I'm just naiive and forgot that they'll always try to move a full stack if they can.

I'm just gonna completely disable this alert until I can figure out the best way to handle this.

@jarquafelmu
Copy link
Author

that sounds like a decent plan and it sounds that you have the right direction

@raiguard
Copy link
Collaborator

I pushed the patch. I also added amounts to the material button tooltips, for easy reference:

image

@jarquafelmu
Copy link
Author

perfect

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants