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

Add toString to Filter model #710

Open
QubitPi opened this issue Jun 5, 2018 · 1 comment
Open

Add toString to Filter model #710

QubitPi opened this issue Jun 5, 2018 · 1 comment

Comments

@QubitPi
Copy link
Contributor

QubitPi commented Jun 5, 2018

It will become super useful during testing and debugging when Filter are serialized properly, otherwise,
people only see memory addresses which give them no idea about which filter are they looking at

@cdeszaq
Copy link
Collaborator

cdeszaq commented Jun 7, 2018

In general, there's little reason for any class to use the default toString method, so this is definitely a good idea!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants