Skip to content

Independent FullForm#1316

Merged
mmatera merged 7 commits into
masterfrom
makeboxes_refactor_2025_step_0p5
Dec 26, 2025
Merged

Independent FullForm#1316
mmatera merged 7 commits into
masterfrom
makeboxes_refactor_2025_step_0p5

Conversation

@mmatera

@mmatera mmatera commented Jan 24, 2025

Copy link
Copy Markdown
Contributor

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.

@rocky

rocky commented Jan 24, 2025

Copy link
Copy Markdown
Member

Let's defer this until after a release, which I think may happen this weekend.

@mmatera

mmatera commented Jan 24, 2025

Copy link
Copy Markdown
Contributor Author

Sure. I just put this here in relation with the other PR.

@mmatera
mmatera force-pushed the makeboxes_refactor_2025_step_0p5 branch from 3b54e1b to 74bb87c Compare February 9, 2025 22:11
Comment thread mathics/eval/makeboxes/formatvalues.py Outdated
@rocky

rocky commented Sep 7, 2025

Copy link
Copy Markdown
Member

It would be very useful to add some tests that exercise and show in isolation what is going on here.

@rocky

rocky commented Dec 25, 2025

Copy link
Copy Markdown
Member

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?

@rocky

rocky commented Dec 25, 2025

Copy link
Copy Markdown
Member

LGTM

@mmatera

mmatera commented Dec 25, 2025

Copy link
Copy Markdown
Contributor Author

Yep, it is just a way to decouple Full form of the other formatting. This helps to debug the other changes.

@mmatera
mmatera merged commit 0f46422 into master Dec 26, 2025
17 checks passed
@mmatera
mmatera deleted the makeboxes_refactor_2025_step_0p5 branch December 26, 2025 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants