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
adding edfWrite? #4
Comments
Hi,
First, I am sorry for this extreme late response.
I agree that it would be nice to extent the edfReader package to an edfReaderWriter package.
However I don’t have the time to write it.
Being retired, I am too often claimed by my (grand)kids and I am currently working at another package more in line which my actual research interest.
Nevertheless, if I could be of any assistant for somebody else who wants to do the job, I will be glad to do so.
Best regards
Pisca46
… On 30 Nov 2018, at 10:38, Bruno Nicenboim ***@***.***> wrote:
Hi, Thanks for the useful package!
I know it's edfReader, but it would be nice if the package would allow to write back edf files after modifying the edf objects that edfReader produces.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <#4>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AIBWXhzBo4XSq9u75iPksEwsqPZIbi3nks5u0PyAgaJpZM4Y7UhG>.
|
@Pisca46 I recently wrote a pure-Python EDF+ writer for getting EDFs out of Python and into R, and I feel pretty confident the code wouldn't be too difficult to bring over to R, so I'd be willing to take a crack at a pull request if you're interested! @bnicenboim What were you imagining the API to be like for this function? I'm imagining a function that takes a dataframe and uses the data and column names to generate the file, with extra arguments for annotations and for specifying file metadata (could be an edfHeader object or a list). That would mean you could import (or generate) EEG data from other formats/packages in R and still be able to write them out to EDF. |
yeah that would be great, I guess it should take the metadata as list, and probably a matrix (to speed things up) with the eeg data. I'm actually storing the EEG data in |
Hi,
First, I am sorry again for my extreme late response.
Thank you for your offer “to take a crack at a pull request “.
However, as explained in my mail to @bnicenboim <https://github.com/bnicenboim> dd Feb 26, 2019, I don’t have the time to work on an edfWriter package. Moreover such a package is not in my personal interest too.
Best regards
Piska46
PS
I am not sure whether this is of intrest for you, but storing EDF signals in a single data frame may be be cause problems as each EDF signal may have a different sample rate.
… On 26 Nov 2020, at 22:42, Austin Hurst ***@***.*** ***@***.***>> wrote:
@Pisca46 <https://github.com/Pisca46> I recently wrote a pure-Python EDF+ writer for getting EDFs out of Python and into R, and I feel pretty confident the code wouldn't be too difficult to bring over to R, so I'd be willing to take a crack at a pull request if you're interested!
@bnicenboim <https://github.com/bnicenboim> What were you imagining the API to be like for this function? I'm imagining a function that takes a dataframe and uses the data and column names to generate the file, with extra arguments for annotations and for specifying file metadata (could be an edfHeader object or a list). That would mean you could import (or generate) EEG data from other formats/packages in R and still be able to write them out to EDF.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#4 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACAFMXRZW747VIAKDTQFZ63SR3DUTANCNFSM4GHNJBDA>.
|
@Pisca46 Thank you for your response! I fully understand no longer having time or interest in maintaining a package you no longer need yourself. To make sure your hard work on edfReader is preserved for future versions of R, would you be open to transferring maintenance of the package to someone else? All that's required of you is sending a one-sentence email to CRAN-submissions@R-project.org (from the email address you use for the package), containing the name and email address of the person you're transferring maintainership to. CRAN will take care of the rest. If either @craddm (of the eegUtils project) or @bnicenboim are interested I'll defer to them, but if not I'd be happy to do it myself. I've taken over maintenance of another abandoned CRAN pacakge before so I know what's involved! |
@a-hurst, I'll be glad if you take over! |
@a-hurst <https://github.com/a-hurst>, First my best wishes for 2021 and, again, my apologies for my late response.
Thank you for your offer to maintain the edfReader package. However I have trouble in doing it myself for the next few years. Its a full implementation (with no outstanding requests for corrections or new features) and also R is quite stable.
Best regards,
Piska46
… On 20 Dec 2020, at 20:50, Austin Hurst ***@***.*** ***@***.***>> wrote:
@Pisca46 <https://github.com/Pisca46> Thank you for your response! I fully understand no longer having time or interest in maintaining a package you no longer need yourself.
To make sure your hard work on edfReader is preserved for future versions of R, would you be open to transferring maintenance of the package to someone else? All that's required of you is sending a one-sentence email to ***@***.*** ***@***.***> (from the email address you use for the package), containing the name and email address of the person you're transferring maintainership to. CRAN will take care of the rest.
If either @craddm <https://github.com/craddm> (of the eegUtils project) or @bnicenboim <https://github.com/bnicenboim> are interested I'll defer to them, but if not I'd be happy to do it myself. I've taken over maintenance of another abandoned CRAN pacakge before so I know what's involved!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#4 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACAFMXR2QA2XI66OI63CU6DSVZIRLANCNFSM4GHNJBDA>.
|
Hi, Thanks for the useful package!
I know it's edfReader, but it would be nice if the package would allow to write back edf files after modifying the edf objects that edfReader produces.
The text was updated successfully, but these errors were encountered: