-
Notifications
You must be signed in to change notification settings - Fork 107
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
Make DumpMachine/DumpRegister output more compact and reduce use of Unicode symbols #1699
Comments
Point number 1 around removing DumpMachine is certainly doable and a likely improvement, but number 2 introduces an inconsistency that we are undecided on. We think it makes more sense to either leave unicode in or remove all unicode. |
Not all Unicode is equal :-) There is a lot of value in using ⟩ instead of > for ket, but having i in italic is not important, so I'm willing to trade off some non-uniformity for better usability |
I think the concern is more about what precisely the usability improvement is. As far as I understand, having a unicode string is of usability level X, and having a non-unicode string is of usability level Y. Removing some but not all unicode does not change the usability level from X to Y, as any external tools you're using still have to deal with a unicode string. I understand the sentiment that the italic 𝒊 is less important, but I mean...if we are already outputting strings of usability level X, we might as well take full advantage of this and provide the most prettified output possible, right? |
It does actually change usability, since I need to make a decision about what to do with each type of character individually. Reducing the number of characters to think about improves usability |
One more odd thing I noticed was that the minus signs in the second and the last columns were different, one was highlighted as special by VS Code. That didn't cause any rendering issues with my particular platform, but looked odd |
Looks like #1885 addresses at least the first point, removing the titles from the output. |
Is your feature request related to a problem? Please describe.
Describe the solution you'd like
The text was updated successfully, but these errors were encountered: