You are viewing a single comment's thread from:

RE: My experiment drawing Mexico flag with thousands pictures

in #python5 years ago (edited)

Thank you for your contribution. This is a nice idea.

  1. I notice that the code is not on an independent repro, instead there are some other irrelevant project(s), this makes it harder to see the changes log in the future.
  2. The code demonstrates well the idea, but it makes it hardly practical for general use. Expecially that not every people knows how to put the image paths into the temporal database.
  3. Also, directories and filenames are hard-coded in the code, you might want to improve the utility by accepting command line options.
  4. You might want to extract the functionalities as a few funtions/modules, so that you can add unit tests.
  5. Almost no comments, you might want to comment more - and in English :)
  6. And please also add more documentation in the README.md
  7. You might want to use Pull Request to group your commits.

Lookingforward to your next contribution.

According to Utopian policies and guidelines, here are two important ones that you will not want to miss:

To be considered for potential reward, Bug Fixes and New Features should be submitted via Pull Requests. The Pull Request should have been merged within the past 14 days.

Updates on Own Projects may be committed directly, without a Pull Request. To be considered for potential reward, commits should not be older than 14 days.

Happy New Year!


Need help? Write a ticket on https://support.utopian.io/.
Chat with us on Discord.
[utopian-moderator]

Sort:  

Thank you for your review, @justyy! Keep up the good work!

Coin Marketplace

STEEM 0.30
TRX 0.11
JST 0.031
BTC 68643.75
ETH 3779.37
USDT 1.00
SBD 3.67