Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 24, 2026

Thanks for the feedback on #8. I've created this new PR, which merges into #8, to address your comment. I will work on the changes and keep this PR's description up to date as I make progress.

Original PR: #8
Triggering comment (#8 (comment)):

@copilot The base branch hybrid-and-mcts-fix has a linking error that prevents successful builds:

undefined reference to `MetalFish::GPU::GPUTuningParams::select_strategy(int) const'
This is an LTO (Link-Time Optimization) issue where a function in gpu_nnue_integration.cpp is being optimized away during linking. I verified this exists in the base branch independently. The Lc0 NN work is complete and properly integrated, but the underlying build issue in the base branch needs to be fixed before the full test suite can run.

The merge conflicts are fully resolved and the code is ready for the base branch build issue to be addressed.

do we have this issue?


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Jan 24, 2026
@NripeshN NripeshN closed this Jan 24, 2026
Copilot AI requested a review from NripeshN January 24, 2026 14:45
Copilot stopped work on behalf of NripeshN due to an error January 24, 2026 14:45
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