Skip to content

Conversation

@JaskRendix
Copy link
Contributor

@JaskRendix JaskRendix commented Feb 28, 2025

PR splits out the isometric part from #62 and adds unittest to check the two methods.

@bitcraft bitcraft merged commit a0936b2 into bitcraft:master Mar 23, 2025
4 checks passed
@bitcraft
Copy link
Owner

thanks for the update. ill probably mention this again, but its been my experience than using the [] and {} syntax compared to list(), dict(), etc doesn't produce a meaningful improvement in performance. micro benchmarks aside, its not going to make rendering any faster. im not going to ask you to change it, but if i do, understand that i think in many cases, its a style preference i have that makes the code more readable.

@JaskRendix JaskRendix deleted the isometric branch March 23, 2025 11:57
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