search for: isnamespaceloaded

We ran into a bug in our package that seems to boil down to isNamespaceLoaded() returning TRUE for namespaces that R is currently loading. We had something like this in an .onLoad function: if (isNamespaceLoaded("upstream")) { upstream::upstream_function() } Which seems OK, unless upstream (recursively) imports the package having this code. Should that happen,...

>>>>> Michael Lawrence <lawrence.michael at gene.com> >>>>> on Mon, 26 Jan 2015 05:12:55 -0800 writes: > A isNamespaceLoaded() function would be a useful thing to > have in general if we are interested in readable code. An > efficient implementation would be just a bonus. Good point (readability), and thank you for the support! Note one slight drawback with your name (which is clearly better than mine fir...

...on), the "base" test in "tests/Examples" fails, apparently because it depends on MASS. I'm citing from "base-Ex.Rout.fail": | > ## The string "foo" and the symbol 'foo' can be used interchangably here: | > stopifnot( identical(isNamespaceLoaded( "foo" ), FALSE), | + identical(isNamespaceLoaded(quote(foo)), FALSE), | + identical(isNamespaceLoaded(quote(stats)), statL)) | > | > hasM <- isNamespaceLoaded("MASS") # (to restore if needed) | > Mns <- asNamespace("...

...; test in > "tests/Examples" fails, apparently because it depends on MASS. I'm > citing from "base-Ex.Rout.fail": > | > ## The string "foo" and the symbol 'foo' can be used interchangably here: > | > stopifnot( identical(isNamespaceLoaded( "foo" ), FALSE), > | + identical(isNamespaceLoaded(quote(foo)), FALSE), > | + identical(isNamespaceLoaded(quote(stats)), statL)) > | > > | > hasM <- isNamespaceLoaded("MASS") # (to restore if needed) > | &gt...

...; test in > "tests/Examples" fails, apparently because it depends on MASS. I'm > citing from "base-Ex.Rout.fail": > > | > ## The string "foo" and the symbol 'foo' can be used interchangably here: > | > stopifnot( identical(isNamespaceLoaded( "foo" ), FALSE), > | + identical(isNamespaceLoaded(quote(foo)), FALSE), > | + identical(isNamespaceLoaded(quote(stats)), statL)) > | > > | > hasM <- isNamespaceLoaded("MASS") # (to restore if needed) > | > Mns...

...dressing this, Michael On Mon, Jan 26, 2015 at 5:51 AM, Martin Maechler < maechler at lynne.stat.math.ethz.ch> wrote: > >>>>> Michael Lawrence <lawrence.michael at gene.com> > >>>>> on Mon, 26 Jan 2015 05:12:55 -0800 writes: > > > A isNamespaceLoaded() function would be a useful thing to > > have in general if we are interested in readable code. An > > efficient implementation would be just a bonus. > > Good point (readability), and thank you for the support! > > Note one slight drawback with your name (which is...

>>>>> Winston Chang <winstonchang1 at gmail.com> >>>>> on Fri, 23 Jan 2015 10:15:53 -0600 writes: > I think you can simplify a little by replacing this: > pkg %in% loadedNamespaces() > with this: > .getNamespace(pkg) almost: It would be !is.null(.getNamespace(pkg)) > Whereas getNamespace(pkg) will load the

.../library/base/R/library.R#L967-L968 https://github.com/r-devel/r-svn/blob/4198a2941b702d965bb2374c2b908f48c369f40a/src/library/base/R/library.R#L684-L685 Shouldn't "Loading" be "Attaching" instead? My understanding is "a package is loaded" is equivalent to "isNamespaceLoaded()", i.e., loadNamespace() was run. And that "a package is attached" is equivalent to "pkg %in% .packages()", i.e., library(pkg) was run. Is the terminology not so precise? There's certainly ambiguity in the internal variable names referenced above -- in require, we se...

...ch(.) has always worked for data frames (already in pre-R S). Still, I agree (when teaching and otherwise), we should clearly distinguish the two notably in the context of packages and their namespaces. > My understanding is "a package is loaded" is equivalent to > "isNamespaceLoaded()", i.e., loadNamespace() was run. And that "a > package is attached" is equivalent to "pkg %in% .packages()", i.e., > library(pkg) was run. or (easier for teaching) that "package:<pkg>" %in% search() -- which is what .packages() does {by de...

A isNamespaceLoaded() function would be a useful thing to have in general if we are interested in readable code. An efficient implementation would be just a bonus. On Mon, Jan 26, 2015 at 3:36 AM, Martin Maechler <maechler at lynne.stat.math.ethz.ch> wrote: >>>>>> Winston Chang <winstonch...