Thank you for your contribution.

- I will not suggest adding code that use steemdata because it just doesn't work, I don't think dead code should be in the code repro for any reasons.
- While you start building the library, I would suggest you should also start adding unit tests, which gives the confidence while you make changes.
- the code is not very much commented and I have seen the commented-out code as well.
- You shouldn't set default parameters e.g. something like id='dapeng', if the id is not given, you should expect the function to throw exception
- the database URI should be defined as a constant but should not appear in your code base more than once.
- Your hourrose method, as you said you make some changes from the original copy, it will be better to provide the reference to the original code. And BTW, there are more than 400 lines of code for this function, which is un-maintainable.

Your contribution has been evaluated according to [Utopian policies and guidelines](https://join.utopian.io/guidelines), as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, [click here](https://review.utopian.io/result/3/2333223).

---- 
Need help? Write a ticket on https://support.utopian.io/. 
Chat with us on [Discord](https://discord.gg/uTyJkNm). 
[[utopian-moderator]](https://join.utopian.io/)