Should I add some checks to make sure it is a int that is in the valid range 0x00000000-0x7FFFFFFF ?

If you're up to, otherwise just merge it as is

@eigenein Something like this ?
I wanted to make custom exceptions first but then I thought this was generic enough to use the built-ins.

Please, add/update requirements for custom sequence generator to README as potentially breaking changes

@podshumok I understand the part about adding information about it on the README but why are you mentionning breaking changes ?

I take that back, I thought you create other implementation instance if it is passed

confusing part is that start_sequence will not work with custom sequence generator

So I think we don't need this argument in Client constructor. You can create SimpleSequenceGgenerator instance with desired start_sequence and pass this instance to the client's __init__

I'm okay with either option. One seems less DRY, another is more friendly

I vote for SimpleSequenceGenerator, ask for some documentation and/or tests (but I don't insist), and vote against start_sequence in Client

@podshumok @eigenein Made some changes, let me know if that's ok now.