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

Package scaffolding #825

Open
nikosbosse opened this issue May 19, 2024 · 1 comment
Open

Package scaffolding #825

nikosbosse opened this issue May 19, 2024 · 1 comment

Comments

@nikosbosse
Copy link
Contributor

The Metrics R package doesn’t seem actively maintained and may present a risk to the long-term sustainability of the project. From what I can tell, we only use it for relatively straightforward operations so I wonder if it’s really worth depending on it. A potential strong argument for having such a dependency is if it provided a significant speed boost by using C/C++ or Rust, especially for operations used in loops. But Metrics uses pure R so provides limited value-add in our case.

Conversely, I would probably recommend depending on the MatrixStats package. It’s a well-established package by an active member of the R community. It simple, clearly-named, and more efficient (written in C) replacement for the various apply() calls throughout the codebase.

Originally posted by @Bisaloo in #791 (review)

@nikosbosse nikosbosse added this to the scoringutils-2.x milestone May 19, 2024
@seabbs
Copy link
Contributor

seabbs commented May 19, 2024

The other option we could consider here is finding a new maintainer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants