dselivanov / dselivanov/rsparse

Make RhpcBLASctl Suggests?

Open
#82 5 comments 0 reactions 0 assignees View on GitHub
Dominant language
R
Stars
180
Forks
31
PR merge metrics
No merged PRs in 30d

Description

{RhpcBLASctl} is AGPL-licensed, which makes it invalid for import at many orgs, Google included:

https://opensource.google/documentation/reference/using/agpl-policy

The RhpcBLASctl author confirmed this choice is intentional, but it makes using downstream packages like {rsparse} difficult as well.

To use it, I've patched out the requirement, which AFAICT is about supporting parallelism for performance.

Because it will just make the code slower, I think making {RhpcBLASctl} optional makes sense to more readily support users who can't install it.

Would you accept a PR?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.