psolymos / EDMAinR

Euclidean Distance Matrix Analysis in R
https://psolymos.github.io/EDMAinR
3 stars 2 forks source link

Create Euploid_Adults #7

Closed LME9517 closed 3 years ago

LME9517 commented 3 years ago

Hi Peter, hopefully everything is going well.

I have done the changes that you asked, could you check them and give me your opinion if they are well written. I would like to test the data in R with this data.

Thanks,

Luis Miguel Echeverry.


De: Peter Solymos notifications@github.com Enviado: viernes, 30 de octubre de 2020 9:44 a. m. Para: psolymos/EDMAinR EDMAinR@noreply.github.com Cc: LME9517 luis-mi-e-q@hotmail.com; Author author@noreply.github.com Asunto: Re: [psolymos/EDMAinR] Create Euploid_Adults (#7)

@psolymos requested changes on this pull request.

Thank you for your PR. Please make the following changes before merge:

Thanks.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/psolymos/EDMAinR/pull/7#pullrequestreview-520735938, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARINJU7GHEW3DICW3Y7O5WDSNLGLHANCNFSM4TEAJMRQ.

psolymos commented 3 years ago

Hi Luis, thanks. Are you trying to read in the data for analysis?

You can just use read_xyz("Euploid_Adults") to read in your data for analysis? It doesn't have to be part of the package. The system file thing in the examples is just the way I ship text files with the package as examples, but you can read files from anywhere in your file system. Either set working directory (setwd()) or use the path as part of the file argument.

LME9517 commented 3 years ago

Hi Peter,

Yes, i have been trying to read the data in the package "EDMAinR". However, i have tryied different ways to read it without success. I used the following script "file1 <- system.file("inst/extdata/EDMA_EU_DS/Euploid_Adults.xyz", package="EDMAinR"), but it did not work. Got the error message: "Error in character(nchar(h[2L])) : vector size can not be NA". Could you give me a hand? The two files that contain the data are attached in this mail.

Thank you very much for your help!!!


De: Peter Solymos notifications@github.com Enviado: sábado, 31 de octubre de 2020 3:17 p. m. Para: psolymos/EDMAinR EDMAinR@noreply.github.com Cc: LME9517 luis-mi-e-q@hotmail.com; Author author@noreply.github.com Asunto: Re: [psolymos/EDMAinR] Create Euploid_Adults (#7)

Hi Luis, thanks. Are you trying to read in the data for analysis?

You can just use read_xyz("Euploid_Adults") to read in your data for analysis? It doesn't have to be part of the package. The system file thing in the examples is just the way I ship text files with the package as examples, but you can read files from anywhere in your file system. Either set working directory (setwd()) or use the path as part of the file argument.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/psolymos/EDMAinR/pull/7#issuecomment-719982958, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARINJU57PNI6KFEMTYACZU3SNRWEJANCNFSM4TEAJMRQ.

psolymos commented 3 years ago

Sorry for the confusion. The system.file() piece is not needed, you only need to know where your file is located. E.g. if your file (assuming windows) is at 'c:/Users/User/Documents/Euploid_Adults.xyz' then you just need to do:

x <- read_xyz("c:/Users/User/Documents/Euploid_Adults.xyz")

The system.file is used to locate the text files that are distributed with the package itself (so that you don't have to download it first). Your file already exists and the read_xyz command should be able to read it.