Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PDU encoder #7

Closed
bronte2k7 opened this issue Jul 12, 2020 · 3 comments
Closed

PDU encoder #7

bronte2k7 opened this issue Jul 12, 2020 · 3 comments
Labels
feature New feature

Comments

@bronte2k7
Copy link

really lacking

@alexpirine
Copy link
Member

Hi @bronte2k7 , could you please describe your use case and give an example of the API you wish you had?

@bronte2k7
Copy link
Author

bronte2k7 commented Jul 17, 2020

Hello
Your code is very neat. So my example will be in your style.

class PDU:
    def encode(data={}):
        return {'encoded_data': '', 'length': 'int'}  # perhaps some more necessary data

    def decode(data=None):
        return {'decoded_data': '', 'length': 'int'}  # perhaps some more necessary data

# p.s. I really hope that you understand me

Of all the libraries I've seen, only yours works with python 3, but lacks an encoder)))

@alexpirine alexpirine added the feature New feature label Mar 23, 2022
@alexpirine alexpirine changed the title [feature request] PDU encoder PDU encoder Mar 23, 2022
@alexpirine
Copy link
Member

Closing this for now, since the project is only capable of decoding right now.

If anyone is interested in a Python encoder, it would be helpful to present use cases to move forward. And even better, creating a PR :)

@alexpirine alexpirine closed this as not planned Won't fix, can't repro, duplicate, stale Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Development

No branches or pull requests

2 participants