Bug 879

Summary: CSV file import causes application termination
Product: PulseView Reporter: Ian <ianpegg9>
Component: File handlingAssignee: Nobody <nobody>
Status: RESOLVED WORKSFORME    
Severity: critical CC: Gerhard.Sittig, soeren, uwe
Priority: Normal    
Version: unreleased development snapshot   
Target Milestone: ---   
Hardware: All   
OS: All   
Attachments: Screenshot

Description Ian 2016-12-21 03:26:33 CET
Created attachment 276 [details]
Screenshot

I am using the import comma separated values function. A screenshot of the error is attached. There is a significant spike in CPU usage, but none in memory usage. The first few lines of the file are copied below. The file has 1048577 lines. Please let me know if you would like me to send you the csv file. I could only attach one file.

I do understand that I am probably using the import wrong.

"Time","ALE","A","D","RAM /CE"," ROM /CE","/G (PERIPHERAL DATA BUFFER)","/OE","/WR","/G (INTERRUPT BUFFER)","/IR","PD","INTR","/INTA","IO/M","S1","S0"
-1.048578000 ms,0,0058,FB,1,0,1,0,1,1,7,FF,0,1,0,1,1
-1.048576000 ms,0,0058,FB,1,0,1,0,1,1,7,FF,0,1,0,1,1
-1.048574000 ms,0,0058,FB,1,0,1,0,1,1,7,FF,0,1,0,1,1
-1.048572000 ms,0,0058,FB,1,0,1,0,1,1,7,FF,0,1,0,1,1
-1.048570000 ms,0,0058,FB,1,0,1,0,1,1,7,FF,0,1,0,1,1
-1.048568000 ms,0,0058,FB,1,0,1,0,1,1,7,FF,0,1,0,1,1
Comment 1 Gerhard Sittig 2017-06-05 19:27:43 CEST
You run the software on Windows.  Does the last line of the CSV file miss 
the line termination perhaps, which seems to be rather popular there?  In 
that case the report is a duplicate of bug 635, and a fix will be available 
soon.  Does the file have the (unnecessary) BOM perhaps?  In that case the 
report would be a duplicate of bug 756 (soon to get fixed, too).  Did you 
tick the "Header" checkbox in the CSV input configuration dialog?

It's probably best if you could provide a short file that reproduces the 
issue.  Thank you!
Comment 2 Uwe Hermann 2017-06-07 00:33:15 CEST
The sample provided has a few more complications, e.g. analog/float value as first column, various logic columns, two hex-byte columns ("FF", "FB"). Not sure if all of that is supported, yet.
Comment 3 Soeren Apel 2018-10-21 13:28:30 CEST
Ian, does this still happen with a current PV nightly build?
Comment 4 Gerhard Sittig 2019-10-13 10:37:56 CEST
Closing due to user inactivity. It's been a year without an answer.
Comment 5 Gerhard Sittig 2019-10-18 19:57:56 CEST
really close, feel free to re-open when the issue persists, but do provide 
feedback and details when asked
Comment 6 Gerhard Sittig 2020-07-25 15:40:40 CEST
Did a quick re-test with recent software, works as expected. Command line 
options were: -I csv:column_formats=-,l,x16,x8,6l,x4,x8,5l (guessed).