Repository navigation
Update DataMind for evaluation and add multimodal data tools - #37
Open
xuchangtolearn wants to merge 2 commits into
Open
xuchangtolearn wants to merge 2 commits into
xuchangtolearn wants to merge 2 commits into
Conversation
Use main at 36e189d as the implementation baseline for the research branch. Replace the old research runtime and branch-only artifacts with the current source tree before adding the reviewed data-tool changes.
Preserve the current implementation while removing deployment-specific configuration and paths. Add on-demand DataSpace execution, native SQLite and structured-table input, video evidence bundles, managed operations, and generic query and finalization contracts with offline regression coverage.
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR replaces the legacy
researchimplementation with DataMind based onmainat36e189d, together with the data-tool extensions described below. It adds multimodal data tools and validation of answer schemas and exported artifacts.Scope
This is an intentional replacement of the old research tree, including its agent/snapshot architecture, rather than a compatibility port into that implementation. Research-only content absent from the replacement, including
paper/, is removed.The changes are split into two commits:
4bc8b09— Alignresearchwith the implementation atmaincommit36e189d.bf8a362— Add the data-tool extensions, documentation, tests, and cleanup on that foundation (68 changed files).The complete PR changes 214 files relative to the target
researchbranch.Implementation
Cleanup
The PR includes source code, reusable examples, documentation, and tests. Real API configuration, credentials, personal deployment paths, runtime logs, downloaded models, generated databases, predictions, and build products are excluded. Configuration examples use portable paths and placeholders. New commits use a GitHub noreply address.
Gitleaks scans of the final tracked tree and both new commits reported no secrets. An additional scan found no private deployment paths, server address, or private email in the tracked files. Ignore rules cover local environment files, logs, and caches while retaining example configuration files.
Validation