Skip to content

Conversation

@victoryforphil
Copy link
Contributor

Add Prefect tasks and a geo-sliced flow for VGGT processing to integrate VGGT into the existing workflow system, enabling scalable imagery processing.


Open in Cursor Open in Web

@cursor
Copy link

cursor bot commented Sep 19, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@victoryforphil victoryforphil marked this pull request as ready for review September 19, 2025 03:59
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

)

# If not using MinIO, assume items are local paths or strings
local_paths = [obj.object_name if hasattr(obj, "object_name") else str(obj) for obj in images]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: MinIO Path Misused as Local File Path

When minio_config is not provided, the task attempts to use MinioObject.object_name directly as a local file path. Since object_name contains the remote MinIO path, not a local one, downstream VGGT processing fails when trying to access these non-existent files.

Fix in Cursor Fix in Web

This commit introduces support for VGGT by adding necessary dependencies and updating the lock file.

Co-authored-by: victoryforphil <victoryforphil@gmail.com>
@cursor cursor bot force-pushed the cursor/implement-vggt-sliced-workflow-890d branch from f951a9e to 038c1a3 Compare September 20, 2025 03:32
Co-authored-by: victoryforphil <victoryforphil@gmail.com>
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.

3 participants