Date: Wed, 11 Apr 2007 16:53:55 -0400
From: "James W. MacDonald" <jmacdon at med.umich.edu>
Subject: Re: [Bioc-devel] RFC: xy2i and i2xy in *cdf packages
To: Seth Falcon <sfalcon at fhcrc.org>
Cc: bioc-devel at stat.math.ethz.ch
Message-ID: <461D4AE3.2070501 at med.umich.edu>
Content-Type: text/plain; charset="utf-8"; format=flowed
Seth Falcon wrote:
Wolfgang Huber <huber at ebi.ac.uk> writes:
Hi,
Rather than make up these new dimension objects in the CDF package,
why not teach the affy:indices2xy function to reach into the
appropriate CDF package and grab the *unexported*, existing i2xy
function in order to do its thing, and the affy:xy2indices to grab the
*unexported* xy2i function? Seems like the least intrusive way to do
things, and more flexible.
I don't think we should change the visibility of the xy2i functions
without going through the deprecation cycle.
But in general, I like your suggestion. Teaching affy to grab the
appropriate function seems sensible. Personally, I think it is fine
to have these functions exported. You can still get the right one
when using affy::xy2indices. Symbol masking is a fact of R.
But having the dimensions accessible in the CDF package also makes
sense to me and is easy so I'm still in favor of adding hgu95av2dim.
And the xy2i and i2xy functions should use it...
The functions in affy don't care what happens with xy2i and i2xy since
they don't use them (they get dimensions directly from the AffyBatch).
The only functions that care are those that don't have an AffyBatch to
query.
So, unless there are any objections*, I will add the little dim env to
the cdf packages any any functions that need this info (and don't have
an AffyBatch to query) will have to learn to use it.
*By objections, I mean 'This will completely break my code and destroy
my life'. The cdf packages are nearing the end of their shelf lives,
soon to be replaced with SQLite packages, so arguments about stylistic
differences and 'How things should be done' are counterproductive,
especially given how close we are to release. Also, remember that xy2i
and i2xy are only being _deprecated_. Anybody who uses these functions
has > 6 months from now to make changes.
Best,
Jim
--
James W. MacDonald, M.S.
Biostatistician
Affymetrix and cDNA Microarray Core
University of Michigan Cancer Center
1500 E. Medical Center Drive
7410 CCGC
Ann Arbor MI 48109
734-647-5623
**********************************************************
Electronic Mail is not secure, may not be read every day, and should not be used for urgent or sensitive issues.
------------------------------