-
Notifications
You must be signed in to change notification settings - Fork 48
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
Error when no Trademin present #27
Comments
Is this still a problem? |
As far as I know, assuming no one has addressed it. I havn't been actively playing for awhile so I'd have to do some updates and testing to give you a definitive answer one way or t'other. ChrisEveryone thinks of changing the world, but no one thinks of changing himself.
|
You get the same problem with a Space Station (which never has any waste). I've a minimal fix for it. |
…54d6a880c Divide by zero errors - Issue #27
I fixed the absence of "sum_keys" while implementing other_upgrades (see pull #49) |
Setup a new colony (gas giant), but had not built a TradeMin as of yet. When Governor.pl runs, I get this error (last 2 lines of output prior to exit):
.oO( Can't push from here without a Trade Ministry )
Illegal division by zero at /usr/local/share/perl/5.10.1/Games/Lacuna/Client/Governor.pm line 830.
Presumably each time the script is doing this, it is trying to do a full cache, ending up blowing through the allotted RPC calls after a few runs.
The final error before actually running out of RPC calls was this:
oO( Priority: storage_crisis )
<!> Storage crisis detected for ORE: Only remain until full, less than 1:03:00 threshhold.
Undefined subroutine &Games::Lacuna::Client::Governor::sum_keys called at /usr/local/share/perl/5.10.1/Games/Lacuna/Client/Governor.pm line 1079.
I'll look and see if I can see what to edit on the client to skip those parts if there is no Trademin, but thought I would put this up here just in case.
The text was updated successfully, but these errors were encountered: