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

Make it possible for calculation of MathNode #2173

Closed
HK-SHAO opened this issue Apr 12, 2021 · 3 comments
Closed

Make it possible for calculation of MathNode #2173

HK-SHAO opened this issue Apr 12, 2021 · 3 comments

Comments

@HK-SHAO
Copy link
Contributor

HK-SHAO commented Apr 12, 2021

Mathjs is so powerful!
I'm using it to do something I'm very interested in, a full platform computing APP.
I use the symbol '$' to keep an AST from being calculated.
It's like the Hold function in Mathematica or the hold function in MATLAB.
It would be interesting if a MathNode could also be computed!
It's like

$(sin(x))+$(cos(x))==$(sin(x)+cos(x))

image
image
image

@cshaa
Copy link
Collaborator

cshaa commented Apr 12, 2021

Personally, I would love to have more operations for Nodes, see a related issue #1732. However the current dependency system isn't quite ready for all the circular dependencies this would bring (add(Node, Node) depends on Node, which depends on add(number, number)), see PR #1747 for the problem, and issue #1975 for a lengthy discussion on the possible fix of it.

If you have a proposal on how to implement this in a reasonable way that would work with the current dependency system, I'd love to hear it. Comments in the linked issues are also welcome :)

@josdejong
Copy link
Owner

Thanks for your suggestions @HK-SHAO , and good find of the open discussion/POC on this topic @m93a 👍

I'll close this PR in favor of #1732 to keep the discussion focused, please continue there.

@HK-SHAO
Copy link
Contributor Author

HK-SHAO commented Apr 14, 2021

Thanks for your suggestions @HK-SHAO , and good find of the open discussion/POC on this topic @m93a 👍

I'll close this PR in favor of #1732 to keep the discussion focused, please continue there.

Thank you for your attention 😆

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

3 participants