Tam Pham

Hi @danibene > I defined x_new a bit differently than you suggested: oh no it's entirely my bad (please excuse my midnight brain). Given that we are interpolating the rri,...

Hi @noa7291 Thank you so much for highlighting the issue to us. To help us quickly reproduce and diagnose the issue, could you share your current processing script (the one...

Dear @noa7291 Upon reading the documentation of [Kubios](https://www.kubios.com/downloads/Kubios_HRV_Users_Guide.pdf), I do recognize that a lot of the calculations of the HRV parameters are being done on the detrended IBI, except the...

hey @LaxmanSinghTomar I would like to clarify a little bit about the code you shared. Could you share the exact input for us (i.e. `list(chain(*total.iloc[0, 1:2].values))`) so that we can...

hey @LaxmanSinghTomar Would appreciate it if you can give us the exact reproducible piece of code so that replicate your exact output and sort out what's wrong. Specifically, how's your...

Hi @FrancisThibaultNRC Sorry for this very delayed reply. And yes, the `np.seterr()` function should be placed before and not after the division by zero. I will make a quick fix...

Hi @TiagoTostas Could you help us to look at this out-of-bound error? It seems to be related to #465 but the error is at stage 1 now. https://github.com/neuropsychology/NeuroKit/blob/c1104386655724a5c624e740abf651c939fc5e48/neurokit2/ecg/ecg_findpeaks.py#L983 Would it...

Thanks @raphaelvallat for very detailed answer and the comparison script. Recently, I have been looking at the paper here: [Shi (2017)](https://www.mdpi.com/1099-4300/19/10/568/htm) and the paper seems to suggest that:  for...

By the way, we might look into implementing a function to optimize r for each signal, based on this paper [Lu (2008)](https://biosignal.uconn.edu/wp-content/uploads/sites/2503/2018/09/06_Lu_2008_TBME.pdf). True to what you say, since r can...