-
-
Notifications
You must be signed in to change notification settings - Fork 62
Independent FullForm #1316
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
base: master
Are you sure you want to change the base?
Independent FullForm #1316
Conversation
fullform
|
Let's defer this until after a release, which I think may happen this weekend. |
|
Sure. I just put this here in relation with the other PR. |
3b54e1b to
74bb87c
Compare
|
It would be very useful to add some tests that exercise and show in isolation what is going on here. |
@mmatera Is it correct that there is no user-facing change that is observed, but instead these changes are purely organizational and internal? |
|
LGTM |
|
Yep, it is just a way to decouple Full form of the other formatting. This helps to debug the other changes. |
This is useful to handle the FullForm outside the evaluation process, for example, in debugging, or as a last resource with the standard evaluation process fails.