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

Move forward in going over operator examples #84

Merged
merged 3 commits into from
Nov 19, 2024

Conversation

rocky
Copy link
Member

@rocky rocky commented Nov 7, 2024

@mmatera So you know what's up, here is what I have been trying to go over in my spare time with respect to operator examples in the operator YML file.

Previously I had been trying to generate this automatically from a CSV, but there are too many changes that should be made and it is too cumbersome. So instead I am just working off of the YAML.

At some point in the future, I may convert this back to CSV.

A change will probably need to be made in mathics-core as for generation of the YAML file.

@rocky rocky requested a review from mmatera November 7, 2024 00:14
@rocky rocky marked this pull request as draft November 7, 2024 00:15
@mmatera
Copy link
Contributor

mmatera commented Nov 7, 2024

@rocky , OK, this looks like a huge amount of work. Thanks for doing it! Something that could be done is to write a program in WolframScript to check the compatibility of the examples from the table. I can give it a try when I have some time.

@rocky rocky marked this pull request as ready for review November 19, 2024 02:29
@rocky
Copy link
Member Author

rocky commented Nov 19, 2024

This has been hanging out for a while. While it is a step in the right direction, there are more examples to go over.

Rather than let this hang out more and get stale, I have commented out the "usage:" field for now. I will come back to this later after going over operators with no meaning.

@rocky rocky merged commit 051cb0c into master Nov 19, 2024
12 checks passed
@rocky rocky deleted the box-operator-example-improvements2 branch November 19, 2024 02:33
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