'Sequence' object has no attribute 'gradient_waveforms' in Sequence.calculate_kspace

[fzimmermann89]

Hi,

pypulseq/pypulseq/Sequence/sequence.py

Line 214 in a18e34f

gw = self.gradient_waveforms()

results in 'Sequence' object has no attribute 'gradient_waveforms'
as gradient_waveforms has been removed in 22fb57c

Not sure why as I am not familiar with thecodebase and the commit message does not really explain it...

Cheers,
Felix

[sravan953]

Hello, can you share code to reproduce this bug?

[fzimmermann89]

Yes,
any call to calculate_kspace will cause the error. The minimal example is

import pypulseq as pp
pp.Sequence().calculate_kspace()

[btasdelen]

@sravan953 @fzimmermann89 I believe calculate_kspace is also non-functional in upstream Pulseq implementation. I am not sure why.

@sravan953 Was there a reason to remove gradient_waveforms function? There are still several cases where it is useful. We can also take it out from Sequence.m and put into another file, if complexity is the issue.

[fzimmermann89]

..and is calculate_kspacePP a drop-in replacement for calculate_kspace?

Nevermind, even though the description is the same, calculate_kspacePP uses piecewise polynomials.

[sravan953]

@bilal-tasdelen @fzimmermann89 calculate_kspacePP() is supposed to be the replacement for calculate_kspace(). Since gradient_waveforms() was only called by calculate_kspace(), I removed it!

[fzimmermann89]

As https://pypulseq.readthedocs.io/en/master/pypulseq.Sequence.html#pypulseq.Sequence.sequence.Sequence.calculate_kspace
calculate_kspace is still in the documentation, I would prefer to get this fixed again.

Sorry for dropping the ball here, but I did not use pypulseq for some time -- and today got hit again by the same issue.