CQUI-Org / cqui

Civilization 6 - Chao's Quick UI. Reduce clicks and manage your empire faster!
MIT License
318 stars 64 forks source link

BTS Summer 2017 #603

Closed astog closed 7 years ago

astog commented 7 years ago

Updates Better Trade Screen to v4.0.

Changelog for BTS v4.0

  1. Ports all changes made in Summer Patch update into BTS.
  2. Fixes issues brought about by the patch.
  3. Adds sorting by name (origin or destination) in TradeOverview
  4. Sorts the cities in TradeOriginChooser alphabetically.
  5. Significantly improves sorting performance, especially in TradeOverview with a group setting.
  6. Improves the performance in displaying routes in TradeOverview.
  7. Other minor performance improvements.
  8. Minor UI changes in trade screens. Better highlights selected routes.
  9. Adds a divider between groups to visually separate them.
  10. Bug fixes with how the screens were opened/closed
  11. Crash fix when cycling between traders.

Sorting performance tests

Huge map ~1200 routes, grouped routes by destination. Time in seconds

Yield Before After
Food 10.45 0.09
Production 8.41 0.011
Gold 5.76 0.01
Science 10.55 0.099
Culture 8.37 0.01
Faith 8.48 0.099

This is with sorting specifically after the routes table was built. Next patch aims to improve how the routes are built, since that is the slowest part now.

Overall this was a major update, so if you want to go through the changes in code specifically, I would recommend going to my repo and going through commits from (c6099ba) and after, since reading the diff from the first commit would be very difficult.

CQUI Specific Changes

  1. Added Trader Screen tab in CQUI setting panel.
  2. Added three different setting to said panel. I aim to add more in later updates like default sort option, color adjustments, etc.
chaorace commented 7 years ago

@astog Big thanks for helping port over these changes, you've been absolutely wonderful ❤️