Skip to content

Divide download and main function into smaller pieces #7

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.

Sign up for GitHub

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

Jump to bottom
Open
egrace479 opened this issue Jun 5, 2024 · 1 comment
Open

Divide download and main function into smaller pieces #7

egrace479 opened this issue Jun 5, 2024 · 1 comment
Assignees
@Dahlializi
Labels
structure Refactoring or architecture, general code organization

Comments

@egrace479
Copy link
Member

The download_images and main() functions are both rather long. It would be good to try to break up for improved readability. #6 may help with this as well.
@egrace479 egrace479 added the structure Refactoring or architecture, general code organization label Jun 5, 2024
@egrace479
Copy link
Member Author

Start with pulling download_images into a download.py file to break up the large main file. Note there are some constants at the top of the file, imports will need adjusting, and tests may need adjusting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Dahlializi Dahlializi

Labels
structure Refactoring or architecture, general code organization
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@egrace479 @Dahlializi