Skip to content

Conversation

@PabloRuizCuevas
Copy link
Contributor

@PabloRuizCuevas PabloRuizCuevas commented Oct 26, 2025

I am submitting

  • a new package
  • an update for a package
  • Added Matrix determinant for nxn matrix
  • Added test and updated documentation

@typst-package-check typst-package-check bot changed the title typst:0.1.0 numty:0.1.0 Oct 26, 2025
@typst-package-check typst-package-check bot added the update A package update. label Oct 26, 2025
@elegaanz elegaanz self-assigned this Oct 28, 2025
Copy link
Member

@elegaanz elegaanz left a comment

Choose a reason for hiding this comment

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

I have a few suggestions to make the package better, but overall it looks good to me. Thank you for this update.

A library for performing mathematical operations on n-dimensional matrices, vectors/arrays, and numbers in Typst, with support for broadcasting and handling NaN values. Numty’s broadcasting rules and API are inspired by NumPy.

```typ
#import "numty.typ" as nt
Copy link
Member

Choose a reason for hiding this comment

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

It would be better to show how to import the package from Universe, with "@preview/numty:0.1.0".

Comment on lines +66 to +67
#import "numty.typ" as nt
#import "numty.typ": c, r
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure it is relevant to copy this file here, it seems to be only useful for development purpose.

entrypoint = "lib.typ"
authors = ["Pablo Ruiz Cuevas"]
license = "MIT"
description = "Numeric Typst"
Copy link
Member

Choose a reason for hiding this comment

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

I think the description could be more complete. Rephrasing your README, it could say something along the lines of "Perform mathematical operations on n-dimensional matrices, vectors/arrays, and numbers".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

update A package update.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants