-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Add several new R packages #2952
Conversation
depends_on('r-kernlab', type=('build', 'run')) | ||
depends_on('r-e1071', type=('build', 'run')) | ||
depends_on('r-glmnet', type=('build', 'run')) | ||
depends_on('r-xgboost', type=('build', 'run')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Concretizing this package is absolutely brutal:
$ time spack spec r-rminer
...
real 7m16.433s
user 7m9.730s
sys 0m4.468s
Why do the new packages subclass |
Hmm, that is a good question, seeing as how it correctly renamed them to |
@@ -249,6 +249,7 @@ def __init__(self, name, *args): | |||
|
|||
class RPackageTemplate(PackageTemplate): | |||
"""Provides appropriate overrides for R extensions""" | |||
base_class_name = 'RPackage' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Somehow I completely forgot this in #2761.
@adamjstewart: is this WIP? |
Well, I haven't yet tried to install any of them, but they shouldn't break anything at least. I was going to install them on Monday 😄 |
Ok I'll wait til Monday. |
Ok, this should be all of the packages I need for now. I was able to install all of them, but I wasn't able to activate them due to the bug I mentioned in #2951 (comment). |
* Add several new R packages * Add a few more R packages * Update more versions * Convert Package to RPackage * Add a few more packages * Add missing dependencies
* Add several new R packages * Add a few more R packages * Update more versions * Convert Package to RPackage * Add a few more packages * Add missing dependencies
* Add several new R packages * Add a few more R packages * Update more versions * Convert Package to RPackage * Add a few more packages * Add missing dependencies
Also updates to the latest version for several existing R packages.