Long time coder here with a STEM degree, but not a CS degree. I’ve mostly used programming as a tool to help me with my job rather than as my job, so I’m conscious that I have some gaps in my programming skillset.

To close a couple of those gaps, I’m trying become competent at Github and take my Python skills to the next level.

If you kind people could provide suggestions on improvements I could make to this repo and the code in it I’d be ever so grateful. :)

It’s a bot to run Lemmy posts and comments through the pretrained Detoxify transformer model and to report toxic comments for the Mods to action.

  • Oscar@programming.dev
    link
    fedilink
    English
    arrow-up
    1
    ·
    edit-2
    1 year ago

    I haven’t gone though it in detail but something that stood out to me is the complexity of process_content().

    If you at some point end up with a large function, or if you have deeply nested blocks, it can help readability to split it up into smaller functions with more clear goals, even if they are only called once. In your case you could keep process_content() as a sort of parent function for calling smaller ones.

    I’m guilty of large functions too because it’s easier to just add stuff to a single function while developing and debugging, but before I submit stuff I tend to go through and clean up by doing this.

    Though I guess this is sort of opinionated too!