search

narc0tiq / YARM

Yet another variant of the Resource Monitor for Factorio
https://mods.factorio.com/mod/YARM
MIT License
54 stars 42 forks source link

fix an ore per minute calculation error (fixes #118) #148

Closed wchristian closed 1 year ago

wchristian commented 1 year ago

Made an oversight when i originally wrote this, that kept YARM from recognizing "no change in ore amount" properly. This change fixes that.

narc0tiq commented 1 year ago

Thanks, I appreciate the fix!

wchristian commented 1 year ago

Well, i did perpetrate the bug. :)

Looking at the forums for the mod, you might wanna add some kind of filtering to make any speed smaller than 0.1 be 0, or maybe i can do it this afternoon.

narc0tiq commented 1 year ago

Sure, I'll hit "merge" on a PR, no problem :)

wchristian commented 1 year ago

Well, you got it in #149 now :)