CaltechOpticalObservatories / NGPS

NGPS Software
2 stars 0 forks source link

Problem loading CSV file into GUI #43

Closed chazshapiro closed 9 months ago

chazshapiro commented 9 months ago

In NGPS-OBSERVE I am trying to import

/home/developer/Software/Python/OTM/inputCSV/sample_input_3.csv

but it is not being fully parsed. Many rows are missing, including first few. Is this not a valid CSV format for the GUI?

Screenshot 2023-09-25 at 10 02 17 AM
jenniferwillow commented 9 months ago

Hi Chaz, No resolution yet but I am looking at the problem now. I just wanted you to know I’m working on it. Sincerely, Jennifer

On Sep 25, 2023, at 10:01 AM, Chaz Shapiro @.**@.>> wrote:

Assigned #43https://github.com/CaltechOpticalObservatories/NGPS/issues/43 to @jenniferwillowhttps://github.com/jenniferwillow.

— Reply to this email directly, view it on GitHubhttps://github.com/CaltechOpticalObservatories/NGPS/issues/43#event-10465419421, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFCPAKDEGAO2KQFY23HHMDDX4G2FPANCNFSM6AAAAAA5GMYSBI. You are receiving this because you were assigned.Message ID: @.***>

jenniferwillow commented 9 months ago

Hi Chaz, The issue has been resolved. It turned out to be a vector index problem that got introduced to the code when I added the requested slit angle to the table (or changed its position). I have tested each of the 4 CSV files and all of them import without issue. I also found that the first time the OTM was run it got passed a 0.0 for the airmass limit and that caused an error. I've corrected both issues and I have deployed a new version. No change on your part to run the new code. Sincerely, Jennifer

jenniferwillow commented 9 months ago

Hi Chaz, I also deployed a new version of ngps-plan since the error would have effected both programs. I deployed the new ngps-plan and tested it with all 4 csv files. Good luck. Jennifer