Bug 603311 - Review Request: R-caTools - Tools: moving window statistics, gif, base64, roc auc...
Summary: Review Request: R-caTools - Tools: moving window statistics, gif, base64, ro...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Chen Lei
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 585817 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-06-12 10:29 UTC by Pierre-YvesChibon
Modified: 2010-07-26 17:09 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-07-26 17:09:10 UTC
Type: ---
Embargoed:
supercyper1: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Pierre-YvesChibon 2010-06-12 10:29:23 UTC
Spec URL: 
http://pingou.fedorapeople.org/RPMs/R-caTools.spec
SRPM URL: 
http://pingou.fedorapeople.org/RPMs/R-caTools-1.10-3.fc13.src.rpm
Description: 
Contains several basic utility functions including: moving (rolling,
running) window statistic functions, read/write for GIF and ENVI binary
files, fast calculation of AUC, LogitBoost classifier, base64
encoder/decoder, round-off error free sum and cumsum, etc.

Comment 1 Pierre-YvesChibon 2010-06-12 10:31:05 UTC
*** Bug 585817 has been marked as a duplicate of this bug. ***

Comment 2 Chen Lei 2010-06-13 10:20:58 UTC
The package is already reviewed by the submitter, spec file is petty file.

rpmlint only showes some harmless warnings which can be ignored.

rpmlint R-caTools-*
R-caTools.x86_64: W: spelling-error Summary(en_US) gif -> fig, gig, if
R-caTools.x86_64: W: spelling-error Summary(en_US) roc -> cor, rock, Kroc
R-caTools.x86_64: W: spelling-error Summary(en_US) auc -> aux, ac, arc
R-caTools.x86_64: W: summary-ended-with-dot C Tools: moving window statistics, gif, base64, roc auc...
R-caTools.x86_64: W: spelling-error %description -l en_US cumsum -> cum sum, cum-sum, cums um

License field matches the upstream license.

The source used to build the package matches the upstream source,
as provided in the spec URL.
<<md5sum checksum>> 19c479fa167ffba886764c0059fc6824

This package is approved.

Comment 3 Pierre-YvesChibon 2010-06-13 10:28:01 UTC
Thanks for the review.


New Package CVS Request
=======================
Package Name: R-caTools
Short Description: Tools: moving window statistics, gif, base64, roc auc...
Owners: pingou
Branches: F-13 F-12
InitialCC:

Comment 4 Kevin Fenzi 2010-06-14 04:47:48 UTC
CVS done (by process-cvs-requests.py).

Comment 5 Pierre-YvesChibon 2010-06-27 15:00:28 UTC
hm I can't build it since R-bitops is still in updates-testing.

I have added a 'works for me' on koji, let's see if the maintainer reacts.

Comment 6 Chen Lei 2010-06-27 16:16:05 UTC
(In reply to comment #5)
> hm I can't build it since R-bitops is still in updates-testing.
> I have added a 'works for me' on koji, let's see if the maintainer reacts.    

Hi Pierre-YvesChibon,

Why not request for a membership in provenpackager? Things will be easier if you can commit all R-related packages.
See http://fedoraproject.org/wiki/Provenpackager_policy

Comment 7 Pierre-YvesChibon 2010-06-27 17:37:00 UTC
I actually asked once and got refused for valid reasons.

Comment 8 Chen Lei 2010-06-27 18:50:14 UTC
(In reply to comment #7)
> I actually asked once and got refused for valid reasons.    

The policy is changed, now you just need 3 positive votes from existed sponsors to become a provenpackager. Is their any active provenpackgers in R SIG?

Comment 9 Pierre-YvesChibon 2010-06-28 06:14:09 UTC
(In reply to comment #8)
> Is their any active provenpackgers in R SIG?    

Spot is the R maintainer


Note You need to log in before you can comment on or make changes to this bug.