Skip to content

Merge 'feature/batch sogs vggt' major feature update into main#2

Open
Eecornwell wants to merge 24 commits intomainfrom
feature/batch-sogs-vggt
Open

Merge 'feature/batch sogs vggt' major feature update into main#2
Eecornwell wants to merge 24 commits intomainfrom
feature/batch-sogs-vggt

Conversation

@Eecornwell
Copy link
Contributor

Description of changes:
Enabled features: AWS Batch, AWS CodeBuild, Pipeline container updates

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Eric Cornwell and others added 18 commits October 17, 2025 05:44
…, enabled Spot Instances using Batch option, enabled option to render output video and thumbnail from splat using the input trajectories, added spherical pre-processing script from COLMAP, added object eraser component functionality, upgraded cuda and pytorch major versions, bumped colmap, glomap, ceres, spz versions, added version env vars for dockerfile dep versions, moved object eraser, background removal models to S3 during model deployment lambda, moved container main functions to utils, simplified redundant code around sanitizing strings in container
…moved vggt/map_anything due to cuda upgrade issues, added eval metrics, updated local build container doc, added coordinate system setting, Gradio interface overhaul
…ible with new dependency versions for using CUDA with SfM
Added security scanner tools links etc
@Eecornwell Eecornwell self-assigned this Feb 27, 2026
Updated security issues table with explanations for flagged errors.
@Eecornwell
Copy link
Contributor Author

CodeQL scanning results above are reporting false positives that are already contained and explained in the security table on readme.md

Error False Positive Codeguru OS command injection — generate_splat_gradio.py:115,116,117 Untrusted input used in system commands These lines set os.environ variables for AWS credentials provided by the user in the Gradio credentials tab. No subprocess or shell command is invoked with these values; they are passed to boto3.Session() which handles them safely.
Error False Positive Codeguru Path traversal — generate_splat_gradio.py (multiple lines) User-controlled input in file paths The flagged paths in generate_splat_gradio.py (lines 584, 1780, 1791, 1801, 1902, 3841) construct S3 key strings or presigned URL parameters, not local filesystem paths. S3 key construction does not carry path traversal risk.

Copy link
Contributor

@dzilbermanvmw dzilbermanvmw left a comment

Choose a reason for hiding this comment

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

Reviewed proposed code and documentation changes - all is approved

Eric Cornwell and others added 4 commits March 12, 2026 18:50
…d 3d-isp parameter, removed Gradio 3DModel view component
Added new section for 3D Image Signal Processing tools and updated security scanning tools used for codebase validation.
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