Skip to content
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

vignette: number of platforms for the GSE781 dataset #12

Closed
dalloliogm opened this issue Apr 21, 2015 · 3 comments
Closed

vignette: number of platforms for the GSE781 dataset #12

dalloliogm opened this issue Apr 21, 2015 · 3 comments

Comments

@dalloliogm
Copy link
Contributor

In the vignette, in the section Converting a GSE to an expressionset, you mention that all the GSM arrays are from the GPL5 platform. However if I run the code on my computer, I get a list of GPL96 and GPL97 arrays:

> gse <- getGEO(filename=system.file("extdata/GSE781_family.soft.gz",package="GEOquery"))
> gsmplatforms <- lapply(GSMList(gse),function(x) {Meta(x)$platform})
> gsmplatforms
$GSM11805
[1] "GPL96"

$GSM11810
[1] "GPL97"

$GSM11814
[1] "GPL96"

This is also evident in the bioconductor page of GEOquery: http://www.bioconductor.org/packages/release/bioc/vignettes/GEOquery/inst/doc/GEOquery.html#converting-gse-to-an-expressionset

Maybe this file has been changed since the last documentation update?

dalloliogm added a commit to dalloliogm/GEOquery that referenced this issue Apr 21, 2015
Fixed a minor typo. However the example here must be checked. See seandavi#12
@seandavi
Copy link
Owner

Nice catch, @dalloliogm. Thanks. Should be fixed.

@dalloliogm
Copy link
Contributor Author

Thanks for fixing it! It is also good to have an example of the Filter function.

@seandavi
Copy link
Owner

Yes, Filter is not used often enough (by me).

On Tue, Apr 21, 2015 at 10:28 AM, dalloliogm notifications@github.com
wrote:

Thanks for fixing it! It is also good to have an example of the Filter
function.


Reply to this email directly or view it on GitHub
#12 (comment).

seandavi pushed a commit that referenced this issue May 8, 2016
Commit id: 0b9de3e

    Merge pull request #13 from dalloliogm/patch-1

    Update GEOquery.Rmd

Commit id: 99c7d30

    Update GEOquery.Rmd

    Fixed a minor typo. However the example here must be checked. See #12


git-svn-id: https://hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/GEOquery@102877 bc3139a8-67e5-0310-9ffc-ced21a209358
seandavi pushed a commit that referenced this issue May 8, 2016
Commit id: 6807735

    Merge branch 'master' of github.com:seandavi/GEOquery


Commit id: bb27ec8

    Few vignette changes

    * fixes #12
    * default HTML vignette
    


git-svn-id: https://hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/GEOquery@102883 bc3139a8-67e5-0310-9ffc-ced21a209358
seandavi added a commit that referenced this issue Sep 23, 2017
Commit id: 0b9de3e

    Merge pull request #13 from dalloliogm/patch-1

    Update GEOquery.Rmd

Commit id: 99c7d30

    Update GEOquery.Rmd

    Fixed a minor typo. However the example here must be checked. See #12


git-svn-id: file:///home/git/hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/GEOquery@102877 bc3139a8-67e5-0310-9ffc-ced21a209358
seandavi added a commit that referenced this issue Sep 23, 2017
Commit id: 6807735

    Merge branch 'master' of github.com:seandavi/GEOquery


Commit id: bb27ec8

    Few vignette changes

    * fixes #12
    * default HTML vignette
    


git-svn-id: file:///home/git/hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/GEOquery@102883 bc3139a8-67e5-0310-9ffc-ced21a209358
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants