BCG support proposal

Question

BCG support proposal

gcathelain opened this issue 6 years ago · 10 comments

Dear Carlos Carreiras,

I appreciate the code that you provide with this signal processing library, especially the tools.
I am a PhD student working in Ballistocardiography (cardiac mechanical activity monitoring).

Would you like me to contribute and create a BCG processing section ? I found no other open source Python library in this domain.

Sincerely yours,

Guillaume Cathelain

Answer 1 · 2019-04-19T10:45:08.000Z

Hi @guillaumeCat

Thank you for your interest in BioSPPy and your enthusiasm in contributing to this project.
Indeed, it would be interesting to have a module for BCG processing. I think it makes sense to have a separate module for BCG, as there is for ECG and other kinds of signals, following the same structure.

When your code is ready, please submit a PR and we'll review it and add your name to the list of contributors. If your code is based on published research (your own or by others), please don't forget to cite them in the module's docstrings (take a look in the source for the ECG module to see how we've been doing it)

Also, if you can, please add a sample BCG signal that we can add to the examples folder, so that we can test your code.

Kind regards,
Carlos

Answer 2 · 2019-04-19T18:00:47.000Z

Dear Carlos,

Thanks for your support
I am on it !

Best

Answer 3 · 2019-05-06T08:25:29.000Z

Dear Carlos,

I opened a pull request for the BCG section. We can have a video call if you want.

Kind regards

Guillaume

Answer 4 · 2019-05-06T09:40:25.000Z

Hi @guillaumeCat

Thanks for your contribution.
We'll review your code as soon as possible.
How would you like to be cited in the Authors file?

Answer 5 · 2019-05-09T12:32:17.000Z

Hi @capcarr

You're welcome.
I am new to this and am asking support to my university. What is your proposal ?
Sincerely yours

Answer 6 · 2019-05-21T07:17:27.000Z

Dear Carlos,

You can cite me as Guillaume Cathelain, PhD student, CHArt laboratory (Cognition Humaine et Artificielle), EPHE school (École Pratique des Hautes Etudes), PSL University (Paris Sciences Lettres).

Best

Answer 7 · 2020-03-02T15:16:59.000Z

Hello!
Is there any news about this PR?

Answer 8 · 2020-03-07T06:22:30.000Z

Hi, I couldn't find time to update it, sorry about that Le lun. 2 mars 2020 à 16:17, eladwass <notifications@github.com> a écrit :

…

Hello! Is there any news about this PR? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#56?email_source=notifications&email_token=AG3F7MJYYNJ5GUXOZDGRP4TRFPEWZA5CNFSM4HHCWGZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENPWA2Q#issuecomment-593453162>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG3F7MORLDUQFYXG3OINMLLRFPEWZANCNFSM4HHCWGZQ> .

Answer 9 · 2021-05-28T13:56:21.000Z

Done in c5bf34b.

Answer 10 · 2021-06-29T09:02:47.000Z

Some issues were found on this module and it was removed from the master branch, 6977d10. This module is available in its own branch bcg, c8559c5, and it will be merged back once the highlighted TODO issues are fixed.