Skip to content

Conversation

@yosh778
Copy link

@yosh778 yosh778 commented May 17, 2017

No description provided.

@dr1s
Copy link
Contributor

dr1s commented May 18, 2017

FYI this seems to include my scheduler implementation. If this gets merged please check the code again as I'm not sure how well it worked.

@yosh778
Copy link
Author

yosh778 commented May 18, 2017

Ok indeed, I saw many nice improvements on your master branch so I merged the latest with this master branch + some fixes I did.
The scheduler feature isn't enabled by default (just need to add a section to user.cfg), so it shouldn't be a big issue. I only tested the program in regular mode myself.
Thanks for the nice work btw ! Pretty cool

Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
Repository owner deleted a comment Jul 14, 2017
planet.loot = self.fleet_client.get_attack_loot(planet, target)

ordered_planets = sorted(self.planets, key=lambda x: x.loot, reverse=True)
maxLoot = max(self.planets, key=lambda x: x.loot)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this awesome work to you and Yoshi :)

@yosh778
Copy link
Author

yosh778 commented Jul 25, 2017

Ok so I made a lot of updates & improvements on my fork, stopped playing recently though cause it's not much fun anymore once you reach a certain level lol

Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Repository owner deleted a comment Jul 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants