Skip Menu |

This queue is for tickets about the Statistics-R CPAN distribution.

Report information
The Basics
Id: 63906
Status: resolved
Priority: 0/
Queue: Statistics-R

People
Owner: Nobody in particular
Requestors: florent.c.delmotte [...] bankofamerica.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: bug report
Date: Tue, 14 Dec 2010 11:20:33 -0500
To: bug-Statistics-R [...] rt.cpan.org
From: "Delmotte, Florent C" <florent.c.delmotte [...] bankofamerica.com>
Hello, I have found what I believe is a bug. I also have a question or suggestion. 1) a bug: I found that the extra arguments to R::new are not passed to the R::Bridge::new constructor. For example, you may ask: my $R = Statistics::R->new( log_dir => '/tmp/my_dir') ; but you end up with the default log_dir: ' $TMP_DIR/Statistics-R'. I believe the fix is: < $this->R; Show quoted text
> $this->R(@_);
on line 17 of Statistics/R.pm 2) a question: I've noticed that the $session->start modifies the current directory to the 'log_dir' defined above (grep chdir Statistics/R/Bridge.pm). Would it be possible to avoid it by using filenames with log_dir in the path? Regards, Florent. ---------------------------------------------------------------------- This message w/attachments (message) is intended solely for the use of the intended recipient(s) and may contain information that is privileged, confidential or proprietary. If you are not an intended recipient, please notify the sender, and then please delete and destroy all copies and attachments, and be advised that any review or dissemination of, or the taking of any action in reliance on, the information contained in or attached to this message is prohibited. Unless specifically indicated, this message is not an offer to sell or a solicitation of any investment products or other financial product or service, an official confirmation of any transaction, or an official statement of Sender. Subject to applicable law, Sender may intercept, monitor, review and retain e-communications (EC) traveling through its networks/systems and may produce any such EC to regulators, law enforcement, in litigation and as required by law. The laws of the country of each sender/recipient may impact the handling of EC, and EC may be archived, supervised and produced in countries other than the country in which you are located. This message cannot be guaranteed to be secure or free of errors or viruses. References to "Sender" are references to any subsidiary of Bank of America Corporation. Securities and Insurance Products: * Are Not FDIC Insured * Are Not Bank Guaranteed * May Lose Value * Are Not a Bank Deposit * Are Not a Condition to Any Banking Service or Activity * Are Not Insured by Any Federal Government Agency. Attachments that are part of this EC may have additional important disclosures and disclaimers, which you should read. This message is subject to terms available at the following link: http://www.bankofamerica.com/emaildisclaimer. By messaging with Sender you consent to the foregoing.
Subject: RE: [rt.cpan.org #63906] AutoReply: bug report
Date: Tue, 14 Dec 2010 12:56:35 -0500
To: bug-Statistics-R [...] rt.cpan.org
From: "Delmotte, Florent C" <florent.c.delmotte [...] bankofamerica.com>
Here are code changes to Statistics/R/Bridge.pm that seem to work and avoid chdir: 302c302 < my $cmd = "$this->{START_CMD} <start.r >output.log"; --- Show quoted text
> my $cmd = "$this->{START_CMD} <$this->{ START_R } >$this->{ OUTPUT_R }";
556c556,557 < PERLINPUTFILE <- paste("input.", PERLINPUTFILEX , ".r" , sep="") ; --- Show quoted text
> PERLINPUTFILE <- paste("$this->{LOG_DIR}/input.", > PERLINPUTFILEX , ".r" , sep="") ;
Regards. Show quoted text
-----Original Message----- From: Bugs in Statistics-R via RT [mailto:bug-Statistics-R@rt.cpan.org] Sent: Tuesday, December 14, 2010 11:21 AM To: Delmotte, Florent C Subject: [rt.cpan.org #63906] AutoReply: bug report Greetings, This message has been automatically generated in response to the creation of a trouble ticket regarding: "bug report", a summary of which appears below. There is no need to reply to this message right now. Your ticket has been assigned an ID of [rt.cpan.org #63906]. Your ticket is accessible on the web at: https://rt.cpan.org/Ticket/Display.html?id=63906 Please include the string: [rt.cpan.org #63906] in the subject line of all future correspondence about this issue. To do so, you may reply to this message. Thank you, bug-Statistics-R@rt.cpan.org ------------------------------------------------------------------------- Hello, I have found what I believe is a bug. I also have a question or suggestion. 1) a bug: I found that the extra arguments to R::new are not passed to the R::Bridge::new constructor. For example, you may ask: my $R = Statistics::R->new( log_dir => '/tmp/my_dir') ; but you end up with the default log_dir: ' $TMP_DIR/Statistics-R'. I believe the fix is: < $this->R;
> $this->R(@_);
on line 17 of Statistics/R.pm 2) a question: I've noticed that the $session->start modifies the current directory to the 'log_dir' defined above (grep chdir Statistics/R/Bridge.pm). Would it be possible to avoid it by using filenames with log_dir in the path? Regards, Florent. ---------------------------------------------------------------------- This message w/attachments (message) is intended solely for the use of the intended recipient(s) and may contain information that is privileged, confidential or proprietary. If you are not an intended recipient, please notify the sender, and then please delete and destroy all copies and attachments, and be advised that any review or dissemination of, or the taking of any action in reliance on, the information contained in or attached to this message is prohibited. Unless specifically indicated, this message is not an offer to sell or a solicitation of any investment products or other financial product or service, an official confirmation of any transaction, or an official statement of Sender. Subject to applicable law, Sender may intercept, monitor, review and retain e-communications (EC) traveling through its networks/systems and may produce any such EC to regulators, law enforcement, in litigation and as required by law. The laws of the country of each sender/recipient may impact the handling of EC, and EC may be archived, supervised and produced in countries other than the country in which you are located. This message cannot be guaranteed to be secure or free of errors or viruses. References to "Sender" are references to any subsidiary of Bank of America Corporation. Securities and Insurance Products: * Are Not FDIC Insured * Are Not Bank Guaranteed * May Lose Value * Are Not a Bank Deposit * Are Not a Condition to Any Banking Service or Activity * Are Not Insured by Any Federal Government Agency. Attachments that are part of this EC may have additional important disclosures and disclaimers, which you should read. This message is subject to terms available at the following link: http://www.bankofamerica.com/emaildisclaimer. By messaging with Sender you consent to the foregoing. ---------------------------------------------------------------------- This message w/attachments (message) is intended solely for the use of the intended recipient(s) and may contain information that is privileged, confidential or proprietary. If you are not an intended recipient, please notify the sender, and then please delete and destroy all copies and attachments, and be advised that any review or dissemination of, or the taking of any action in reliance on, the information contained in or attached to this message is prohibited. Unless specifically indicated, this message is not an offer to sell or a solicitation of any investment products or other financial product or service, an official confirmation of any transaction, or an official statement of Sender. Subject to applicable law, Sender may intercept, monitor, review and retain e-communications (EC) traveling through its networks/systems and may produce any such EC to regulators, law enforcement, in litigation and as required by law. The laws of the country of each sender/recipient may impact the handling of EC, and EC may be archived, supervised and produced in countries other than the country in which you are located. This message cannot be guaranteed to be secure or free of errors or viruses. References to "Sender" are references to any subsidiary of Bank of America Corporation. Securities and Insurance Products: * Are Not FDIC Insured * Are Not Bank Guaranteed * May Lose Value * Are Not a Bank Deposit * Are Not a Condition to Any Banking Service or Activity * Are Not Insured by Any Federal Government Agency. Attachments that are part of this EC may have additional important disclosures and disclaimers, which you should read. This message is subject to terms available at the following link: http://www.bankofamerica.com/emaildisclaimer. By messaging with Sender you consent to the foregoing.
Hey Florent, Thank you for your bug report. On Tue Dec 14 12:20:47 2010, florent.c.delmotte@bankofamerica.com wrote: Show quoted text
> 1) a bug: > > I found that the extra arguments to R::new are not passed to the > R::Bridge::new constructor. For example, you may ask: > > my $R = Statistics::R->new( log_dir => '/tmp/my_dir') ; > > but you end up with the default log_dir: ' $TMP_DIR/Statistics-R'. > > I believe the fix is: > > < $this->R;
> > $this->R(@_);
> > on line 17 of Statistics/R.pm
Yes. Exactly. I've applied this patch to the git source: https://github.com/bricas/statistics-r/commit/5806e9894d121ab2d2423d328a933a4526a463e7 Show quoted text
> 2) a question: > > I've noticed that the $session->start modifies the current directory > to > the 'log_dir' defined above (grep chdir Statistics/R/Bridge.pm). Would > it be possible to avoid it by using filenames with log_dir in the > path?
I very much long to do some hard refactoring on Bridge.pm. The issue you've mentioned is also found in an existing bug report: https://rt.cpan.org/Ticket/Display.html?id=6724 If you care to take a stab at patching the issue, I'd be very thankful if you would attach it to that ticket above. Otherwise it will have to wait until I get some 'round 'tuits. Thanks again, and please let me know if you find any other issues. Cheers, -Brian