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

Initial Capital Palace building yields #43

Closed
GrantSP opened this issue Dec 1, 2013 · 4 comments
Closed

Initial Capital Palace building yields #43

GrantSP opened this issue Dec 1, 2013 · 4 comments

Comments

@GrantSP
Copy link
Collaborator

GrantSP commented Dec 1, 2013

When you found your first city the Palace has a (2) next to it and 'some' of the yields shown are for multiples of the building.

Production shows 4 from buildings and the Palace is the only building that gives a yield, which is 2.
Gold shows 6 from buildings. Subtract the 2 from the Capital and the Palace again is giving double the Gold yield.
Science, like Gold shows 6 with the same circumstances as Gold.
Likewise Culture shows 6.

Is the Palace building somehow being duplicated in the Capital? I haven't yet founded a new city so I cannot say if it also does this in new cities.

I should have checked the Aztec capital earlier. I forgot that the Palace building also goes into captured cities if your total number of cities is less than 3.
So, the palace in Tenochtitlan is only giving yields for a single building and it doesn't have a (2) after it, so I can assume the building is working correctly here.

stackpoint added a commit that referenced this issue Dec 7, 2013
Removed the (2) from the capital building. See
#43 for more information.
@stackpoint
Copy link
Contributor

Found the source of this bug. Looks like the old Great Hall Building from GEM hasn't been completely removed. I've pushed a commit with the fix. Doesn't look like any yields were changed. May need further investigation.

@stackpoint
Copy link
Contributor

Whoops. It looks like I commented out the wrong capital reference. Back to the drawing board.

@stackpoint
Copy link
Contributor

Okay, I think I fixed it without breaking functionality this time: 9f4b85f

@GrantSP
Copy link
Collaborator Author

GrantSP commented Dec 10, 2013

Looks good.

@GrantSP GrantSP closed this as completed Dec 10, 2013
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