#12 Support for MIT Physiobank 212, 310, 311 data format

open
None
3
2011-12-27
2011-11-21
Tom Schorpp
No

SigViewer trunk crashes reproducible in libbiosig trunk sopen() opening MIT-BIH hea files.
Reproduction:
1. Make debug build of libbiosig.so and SigViewer using the attached patches (and possibly more manual override tricks I forgot to document since QT4 debug config support seems broken in debian squeeze, or broken qmake project config *.pro)
2. Compile+Link SigViever NOT against ./extern but against the /usr installed trunk debug libbiosig.so and headers
3. Launch SigViewer and open a MIT-BIH hea file from the File->Open GUI-dialog, should segfault in libbiosig->sopen(), see attached gdb output.

Discussion

  • Tom Schorpp

    Tom Schorpp - 2011-11-21
    • assigned_to: nobody --> schloegl
    • priority: 5 --> 7
     
  • Tom Schorpp

    Tom Schorpp - 2011-11-21

    Priority raised - crashes are blockers.

     
  • Alois Schloegl

    Alois Schloegl - 2011-12-27

    Several bugs have been fixed:
    - rare header configuration
    - memory allocation error introduced in Jun 2011
    - fix when length is not defined in header
    - add support for new data formats (24 and 32)

    These fixes make libbiosig/sigviewer usable to most data of physiobank.

    Known issues are that 212, 310, and 311 data format is not supported yet. However, these cases are handled gracefully and do not crash. Therefore, I'll close this bug and change this into feature request for support of 212, 310 and 311 pyhsiobank dataformat .

     
  • Alois Schloegl

    Alois Schloegl - 2011-12-27
    • priority: 7 --> 3
    • labels: 1145032 -->
     
  • Alois Schloegl

    Alois Schloegl - 2011-12-27
    • summary: trunk libbiosig sopen() crashes opening MIT-BIH hea --> Support for MIT Physiobank 212, 310, 311 data format
     
  • Alois Schloegl

    Alois Schloegl - 2011-12-27

    adapt title

     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks