Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update main.py #29

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update main.py #29

wants to merge 2 commits into from

Conversation

mst-rajatmishra
Copy link

@mst-rajatmishra mst-rajatmishra commented Nov 8, 2024

Hi,
I tried making some improvments,
Added --verbose, --save_intermediate, --output_name to improve usability and customization.
Arguments like --do_texture_mapping and --save_memory were preserved.

The check_paths() function ensures that all the model files and configurations are valid before running the pipeline.
Descriptive error messages are shown if files or paths are missing.

tqdm could be integrated for tasks that have loops, but here it's mainly for checking and saving intermediate outputs.
Added --verbose flag for users to get step-by-step feedback in the terminal.

The output filenames are now dynamically named using the --output_name argument, allowing users to customize their output filenames for mesh files and gifs.
Intermediate images are saved in images subfolder, meshes in models, and gifs in renders.

Automatically creates necessary subfolders (images, models, renders) inside the output folder for better file organization.

The --verbose flag provides additional print statements, giving users progress updates.
Intermediate Results:

The pipeline now saves intermediate steps (images and views) if --save_intermediate is specified.
Thank You,
RAJAT MISHRA

@skypeandyk7
Copy link

Make it greater and greater!

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