Skip to content

Simplify tier study block counting#147

Open
Interzoneism wants to merge 1 commit into
masterfrom
codex/remove-book-counting-from-tierstudy
Open

Simplify tier study block counting#147
Interzoneism wants to merge 1 commit into
masterfrom
codex/remove-book-counting-from-tierstudy

Conversation

@Interzoneism

Copy link
Copy Markdown
Owner

Summary

  • count TierStudy blocks once when building clusters instead of summing study item scores
  • remove the study score plumbing and inventory scanning that attempted to total books
  • update the default tierStudy config to omit unused score values

Testing

  • dotnet build SafeHaven/SafeHaven.sln -nologo -clp:Summary -warnaserror

https://chatgpt.com/codex/tasks/task_e_6906766a9ed0832badef128b87d6b58d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant