Details
-
Type:
Bug
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: SUIT
-
Labels:
-
Story Points:2
-
Epic Link:
-
Sprint:SUIT Sprint 2017-2
-
Team:Science User Interface
Description
IpacTableFromSource processor does not return the last row of the attached table. (This table does not have the newline after the last row.)
To test, put the attached a.tbl to /hydra/workarea/firefly/temp_files and the attached test.html into /hydra/server/tomcat/webapps/firefly/demo. Use http://localhost:8080/firefly/demo/test.html to see the table.
- Notice the number of rows is 157, the last row seems to be lost.
- Sort on any column. Notice the number of rows is 158.
We noticed this problem in LC phase folded table (the one that is being uploaded to the server). The attached file is a copy of the uploaded phase-folded table.
Attachments
Activity
This ticket was fixed by DM-7971, https://github.com/Caltech-IPAC/firefly/pull/300.
I tested it, the problem is no longer there after Loi's ticket was merged.
The last row missing bug is fixed in DM-7971 In IpacTableUtil and IpacTableParser.
The class IpacTableFromSource is just a caller. Three public/protected methods take TableServerRequest and return either File or unique id. Making a test unit for this class doesn't gain too much. Skip it for now.
Yi Mei [X] (Inactive)
added a comment - The last row missing bug is fixed in DM-7971 In IpacTableUtil and IpacTableParser.
The class IpacTableFromSource is just a caller. Three public/protected methods take TableServerRequest and return either File or unique id. Making a test unit for this class doesn't gain too much. Skip it for now. 
This affects also Gator/Atlas when dong multiple target search uploading a table with this characteristic.