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

Combinatorica as submodule #1226

Merged
merged 5 commits into from
Dec 17, 2024
Merged

Combinatorica as submodule #1226

merged 5 commits into from
Dec 17, 2024

Conversation

rocky
Copy link
Member

@rocky rocky commented Dec 15, 2024

Remove Combinatorica from this repository but include it as a git submodule from Mathics3-Combinatorical.

As a by-product, version 2.0.0 of Combinatorica is also now available. An additional CI job is set up to run the Combinatorica tests that we have (currently for v0.9). These will be expanded soon.
And possibly we may include v2.0.0 tests as well.

To be determined if we want the one CI just to run tests for other packages as well.

@rocky rocky force-pushed the combinatorica-as-submodule branch 3 times, most recently from 6c88696 to 47193d7 Compare December 15, 2024 03:11
@rocky rocky force-pushed the combinatorica-as-submodule branch from 47193d7 to 812c672 Compare December 15, 2024 03:13
@rocky rocky force-pushed the combinatorica-as-submodule branch from f702d5b to 43f7195 Compare December 15, 2024 03:19
@rocky rocky requested a review from mmatera December 15, 2024 03:27
@mmatera
Copy link
Contributor

mmatera commented Dec 17, 2024

is

The only question I have is if we want to run the Combinatorica tests here: combinatorica tests are about testing the package, or testing the support of Mathics-core to the package?

@rocky
Copy link
Member Author

rocky commented Dec 17, 2024

is

The only question I have is if we want to run the Combinatorica tests here: combinatorica tests are about testing the package, or testing the support of Mathics-core to the package?

The combinatoria tests test both the Mathics3 code we have and Mathics3's support of basic primitives.

Some examples to support Mathics3's support of basic primitives. The bugs uncovered and fixed have to do with Drop'ing and Take'ing element 0. These I guess are just plain bugs. Supporting Random while it is deprecated, still seems to be around in WMA, and probably because too many packages don't want to switch.

So here is how this PR proposes to handle this. There is a new CI job just for combinatorica (whether we expand this workflow to include other packages or make them separate workflows is up for discussion and experimentation).

This way, we don't have to test combinatorica on all of the different Python versions and OS''s; one Python version on one OS is probably fine for here. And this is done in parallel to the other tests.

Over in Mathics3-Combinatorica we can test many Python versions and OS's.

Your thoughts?

@mmatera
Copy link
Contributor

mmatera commented Dec 17, 2024

The combinatoria tests test both the Mathics3 code we have and Mathics3's support of basic primitives.

Some examples to support Mathics3's support of basic primitives. The bugs uncovered and fixed have to do with Drop'ing and Take'ing element 0. These I guess are just plain bugs. Supporting Random while it is deprecated, still seems to be around in WMA, and probably because too many packages don't want to switch.

So here is how this PR proposes to handle this. There is a new CI job just for combinatorica (whether we expand this workflow to include other packages or make them separate workflows is up for discussion and experimentation).

This way, we don't have to test combinatorica on all of the different Python versions and OS''s; one Python version on one OS is probably fine for here. And this is done in parallel to the other tests.

Over in Mathics3-Combinatorica we can test many Python versions and OS's.

This is a very good point

Your thoughts?

OK, go ahead then! LGTM

@rocky rocky merged commit e2ceab3 into master Dec 17, 2024
14 checks passed
@rocky rocky deleted the combinatorica-as-submodule branch December 17, 2024 10:49
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