-
Notifications
You must be signed in to change notification settings - Fork 228
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
Missing documentation nodes in Macaulay2Doc #1668
Comments
@DanGrayson could you document the 11 missing nodes at the top? I tried looking at a the source code for few of them, but I have no idea what most of them are intended to do. |
Sure, but not soon. |
@DanGrayson when the items above are documented, we can switch to raising an error for items with missing documentation so that going forward pull requests are forced to document everything. Otherwise the list slowly grows. |
Yes, indeed, we should do that. |
@mikestillman should document "columnRankProfile" -- I don't know anything about it. |
@mikestillman should document "rowRankProfile" -- I don't know anything about it. |
@mikestillman -- just tell me what those two functions do and I'll document them today. |
@mikestillman -- never mind, I found it online. |
@mikestillman -- shouldn't the row rank equal the column rank? i30 : f = mutableMatrix {{1,2,3}, {3,4,5}, {4,6,8.} }
o30 = | 1 2 3 |
| 3 4 5 |
| 4 6 8 |
o30 : MutableMatrix
i31 : rowRankProfile f
o31 = {0, 1, 2}
o31 : List
i32 : columnRankProfile f
o32 = {0, 1}
o32 : List |
@antonleykin -- can you tell me what "toricIdealPartials" in Dmodules does? I'll document it for you today. |
@mikestillman @luisgarciapuente -- if you tell me what globalMarkovStmts does in the package Markov, I can document it for you. |
The package Markov should no longer be used since GraphicalModels has superseded it for more than a decade. But in any case, globalMarkovStmts is now called globalMarkov which is documented |
Maybe we should move Markov to undistributedpackages? Or legacy packages? Or remove it entirely? I do have some old code that uses that package, so I might prefer one of the first 2 options, although I can try to get my old code working with the latest version (if all functionality appears in your "new" package. I checked at one point, but forget what I did, or noticed). |
In deed, even I have some old code that uses Markov that I have not taken the time to update to GraphicalModels. I also know that some people elsewhere still use it. So I also agree to move it to legacy packages but not to remove it. But definitely at this point GraphicalModels has all the exported functionality of Markov and more. Also, through the years, we have fixed some bugs in GraphicalModels and some of those were legacy bugs from Markov. Maybe we should add a note on Markov that users are strongly encouraged to use GraphicalModels instead? |
Well, if we're not going to remove the package "Markov", and we're going to keep distributing it, then we should at least make sure it works. (I'm working on making it an error for a package to have missing documentation nodes.) The symbol "globalMarkov" is in a different package, so even though it's documented, that doesn't help document globalMarkovStmts in this package. We don't have a class of packages called "legacy packages". I think we don't need one, either. Just add a statement to the documentation of Markov that there is a modern replacement available. |
This seems to create a toric ideal describing the image of the given monomial map.
The exponents of the monomials in the monomial map here are the columns of |
Since We (Avi Steiner, @denham0 ,and @ggsmith) also anticipate making a pull request for a major revision of the |
@antonleykin -- thanks. I've added this: document {
Key => { toricIdealPartials, (toricIdealPartials, Matrix, PolynomialRing) },
Headline => "image of a monomial map",
Usage => "toricIdealPartials(A,D)",
Inputs => {
"A" => Matrix => { "a matrix over ZZ whose columns give the coefficients of a monomial map" },
"D" => PolynomialRing => { "the corresponding Weyl algebra" }
},
Outputs => {
Ideal => { "the toric ideal of the image the monomial map described by ", TT "A", " formed in the
polynomial ring generated by the differential variables of ", TT "D", "." }
},
EXAMPLE lines ///
A = matrix{{1,1,1},{0,1,2}}
D = makeWA(QQ[x_1..x_3])
toricIdealPartials(A,D)
///
} |
Ah, I'll fix that by changing
Okay. |
That's been done in #2433 -- enjoy! |
Ouch. Yes!
…Sent from my iPhone
On Apr 12, 2022, at 4:24 PM, Daniel R. Grayson ***@***.***> wrote:
@mikestillman -- shouldn't the row rank equal the column rank?
i30 : f = mutableMatrix {{1,2,3}, {3,4,5}, {4,6,8.} }
o30 = | 1 2 3 |
| 3 4 5 |
| 4 6 8 |
o30 : MutableMatrix
i31 : rowRankProfile f
o31 = {0, 1, 2}
o31 : List
i32 : columnRankProfile f
o32 = {0, 1}
o32 : List
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.
|
@pzinn can you think of a way to either automatically document these or mark them as undocumented? o48 = {0 => (**, Expression, TensorProduct) }
{1 => (**, TensorProduct, Expression) }
{2 => (**, TensorProduct, TensorProduct)}
{3 => (*, Expression, Product) }
{4 => (*, Product, Expression) }
{5 => (*, Product, Product) }
{6 => (++, DirectSum, DirectSum) }
{7 => (++, DirectSum, Expression) }
{8 => (++, Expression, DirectSum) }
{9 => (+, Expression, Sum) }
{10 => (+, Sum, Expression) }
{11 => (+, Sum, Sum) }
{12 => (-, ZeroExpression) }
{13 => (==, Equation, Equation) }
{14 => (==, Equation, Expression) }
{15 => (==, Expression, Equation) }
{16 => (expression, Expression) }
{17 => (hold, Expression) } (The list is just everything without documentation that inherits from an Expression). Currently some methods like this are manually marked as undocumented, e.g: M2/M2/Macaulay2/packages/Macaulay2Doc/doc12.m2 Lines 284 to 291 in ec9e9ac
But maintaining this by hand is hard, and I want something straightforward like this: M2/M2/Macaulay2/packages/Macaulay2Doc/doc12.m2 Lines 293 to 299 in ec9e9ac
|
yeah that shouldn't be too hard. |
After #1666 is merged, there will be 214 warnings left, and 1370 node are not subnodes, which would be good to improve.
missing nodes:
missing methods:
sources of missing citations:
The text was updated successfully, but these errors were encountered: