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

Updated mathematica files with restrictions of r and theta #4

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Samrfgh
Copy link
Contributor

@Samrfgh Samrfgh commented Aug 20, 2023

Hi Nike, I've updated the files with the restriction that all the bond angles and bond lengths must be positive, however I did not seem to get any simplified result that I haven't encountered before. Maybe I'm doing something wrong, and someone more experienced can improve upon these files?

The previous version only had 3 outputs because I commented out the rest of the code (I didn't want previous simplifications to keep outputting as I'm working with other equations). Each block has a "(" and a corresponding ")" at the beginning and the end, removing those will cause the simplified equations of that block to be shown in the output.

@ndattani
Copy link
Member

Thanks @Samrfgh. I'm not sure to what extent public users without a Mathematica license can remove characters like ( and ). Some people may have a "read-only" type of experience, but I'll take a look at your updated files as soon as I get a chance :)

Copy link
Member

@ndattani ndattani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are there no meaningful changes to Simplify.cdf, only to Simplify.nb?

@ndattani
Copy link
Member

I do not appear to be able to remove the ( and ) character in the CDF player. I cannot run commands, or edit anything.

@Samrfgh
Copy link
Contributor Author

Samrfgh commented Aug 24, 2023

Apologies, seems I got confused and didn't update the .cdf file before. I have now updated both cdf and nb files to remove the comments and show all outputs.

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