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

Algebraic Splines package #688

Merged
merged 6 commits into from
Aug 25, 2017
Merged

Algebraic Splines package #688

merged 6 commits into from
Aug 25, 2017

Conversation

midipasq
Copy link
Contributor

I propose adding version 1.0 of the Algebraic Splines package to the Macaulay2 package repository. Programming for this package was begun at the 2015 Macaulay2 workshop in Boise by Michael DiPasquale, Gwyn Whieldon, Eliana Duarte, and Daniel Bernstein.

@DanGrayson
Copy link
Member

It would be good also to add the name of the package to the file containing the names of the packages to be distributed. See the README in the packages directory.

@midipasq
Copy link
Contributor Author

Sorry about that. The distributed-packages file is now updated.

------------------------------------------
------------------------------------------

if version#"VERSION" <= "1.4" then (
Copy link
Member

Choose a reason for hiding this comment

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

Compatibility with M2 1.4 seems no longer relevant.

Headline => "Package for computing topological boundary maps and piecewise continuous splines on polyhedral complexes.",
Configuration => {},
DebuggingMode => true,
if version#"VERSION" > "1.4" then PackageExports => {
Copy link
Member

Choose a reason for hiding this comment

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

Compatibility with M2 1.4 seems no longer relevant.

}
), x -> x =!= null)

if version#"VERSION" <= "1.4" then (
Copy link
Member

Choose a reason for hiding this comment

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

Compatibility with M2 1.4 seems no longer relevant.

},
Headline => "Package for computing topological boundary maps and piecewise continuous splines on polyhedral complexes.",
Configuration => {},
DebuggingMode => true,
Copy link
Member

Choose a reason for hiding this comment

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

Leave DebuggingMode false when distributing the package.

@DanGrayson
Copy link
Member

The examples run and the tests succeed, so that's good.

{Name => "Eliana Duarte", Email => "emduart2@illinois.edu", HomePage => "http://illinois.edu/~emduart2"},
{Name => "Daniel Irving Bernstein", Email=> "dibernst@ncsu.edu", HomePage =>"http://www4.ncsu.edu/~dibernst"}
},
Headline => "Package for computing topological boundary maps and piecewise continuous splines on polyhedral complexes.",
Copy link
Member

Choose a reason for hiding this comment

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

The prefix "Package for computing" seems redundant, since all packages are packages to compute something. Also, the headline would look better without the terminal period.

@DanGrayson
Copy link
Member

Okay, modulo those remarks, it looks good to me.

@midipasq
Copy link
Contributor Author

Ok, last pushed version addresses the above remarks. Thank you.

@DanGrayson DanGrayson merged commit 768890d into Macaulay2:master Aug 25, 2017
@DanGrayson
Copy link
Member

Thank you for the contribution!

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