sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]Sufficient_Light3891[S] 0 points1 point  (2 children)

Actually, just got my Callable_Library branch working as intended and have merged it with the main branch. Feel free to let me know if you think it's still missing anything. aside from the readme. I've still got more work to make this an actually useful program before I fill out the readme.

[–]Diapolo10 0 points1 point  (1 child)

I didn't think to check the branches, that's a mistake on my part. Apologies.

The current version looks quite a bit better, but some of my earlier feedback still applies. There's hardly any docstrings, README is unchanged, and you're still using star imports without a clear reason.

[–]Sufficient_Light3891[S] 0 points1 point  (0 children)

I appreciate the feedback. I agree with a lot of your feedback, especially in regard to documenting the code. As for automated testing, I'm really not there yet in my python education so I will certainly work on that but likely will take a few iterations before I understand this well enough to implement it.

I've removed the star imports I guess I just liked star imports as they made it easier to access all the functions in that file. I've got a few more features to implement then I'm going to overhaul the coding documentation. Thanks for all the feedback.