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

Add initial support for rendering citizens and specialists #549

Merged
merged 1 commit into from
Feb 15, 2025

Conversation

TomWerner
Copy link
Contributor

This doesn't yet handle citizen moods (everyone is rendered as content), doesn't handle citizen gender, and doesn't handle the "region" of the civ. But it does allow us to see specialists, which will be important for tile assignment - we need to be able to turn a citizen into an entertainer to support some types of tile reassignment.

#541

Real game screenshot:
image

C7 screenshot of the same save game:
image

@TomWerner TomWerner force-pushed the twrner/render-citizens branch 5 times, most recently from 80a7a31 to fa6599e Compare February 15, 2025 16:36
Copy link
Contributor

@KulkoBSW KulkoBSW left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@TomWerner TomWerner force-pushed the twrner/render-citizens branch 2 times, most recently from 8f84cfe to 4c69ea7 Compare February 15, 2025 19:38
This doesn't yet handle citizen moods (everyone is rendered as content), doesn't handle citizen gender, and doesn't handle the "region" of the civ. But it does allow us to see specialists, which will be important for tile assignment - we need to be able to turn a citizen into an entertainer to support some types of tile reassignment.

#541
@TomWerner TomWerner force-pushed the twrner/render-citizens branch from 4c69ea7 to 96fa1fa Compare February 15, 2025 19:49
@TomWerner TomWerner merged commit 82acab2 into Development Feb 15, 2025
3 checks passed
@TomWerner TomWerner deleted the twrner/render-citizens branch February 15, 2025 19:51
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.

2 participants