8000 Bad TARGETID output in csv · Issue #88 · desihub/prospect · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Bad TARGETID output in csv #88

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you 8000 account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
armengau opened this issue Jun 28, 2023 · 5 comments
Open

Bad TARGETID output in csv #88

armengau opened this issue Jun 28, 2023 · 5 comments
Assignees
@armengau

Comments

@armengau
Copy link
Collaborator

From Arjun Dey:
A few of the ODIN VI folks are having an unusual issue when saving the csv file from the prospect pages, where the TARGETID is recorded as a real valued number (and then loses digits) instead of a LONG64. At least one person is using Safari version 16.5. Has anyone seen this issue before?
The symptom is that the fields in the CSV file look like:
TARGETID,Spec_version,Redrock_version,Template_version,Redrock_spectype,Redrock_z,Redrock_deltachi2,VI_scanner,VI_quality,VI_issue,VI_z,VI_spectype,VI_comment
3.91E+16,0.57.0.dev7802,0.17.0.dev758,-1,GALAXY,0.8479,0.6,WSJ,0,RC,,,No significant line feature
3.91E+16,0.57.0.dev7802,0.17.0.dev758,-1,GALAXY,0.0051,83.3,WSJ,3,,3.1395,GALAXY,LAE; asymmetric Lya
3.91E+16,0.57.0.dev7802,0.17.0.dev758,-1,GALAXY,1.3514,6,WSJ,0,RC,,,No significant line feature
instead of
TARGETID,Spec_version,Redrock_version,Template_version,Redrock_spectype,Redrock_z,Redrock_deltachi2,VI_scanner,VI_quality,VI_issue,VI_z,VI_spectype,VI_comment
39089837499752854,0.57.0.dev7802,0.17.0.dev758,-1,GALAXY,0.8479,0.6,AJJ,2,,,,unsure what the features are
39089837499752864,0.57.0.dev7802,0.17.0.dev758,-1,GALAXY,0.0051,83.3,AJJ,3,,3.1400,GALAXY,LAE
39089837499752878,0.57.0.dev7802,0.17.0.dev758,-1,GALAXY,1.3514,6.0,AJJ,0,,,,
@armengau armengau added the bug Something isn't working label Jun 28, 2023
@armengau armengau self-assigned this Jun 28, 2023
@weaverba137
Copy link
Member

Could Arjun please provide some information about the version of Prospect being used and whether this is from a notebook or a static webpage?

The thing is, if this was an ongoing problem, it would have been noticed during the intense VI period during survey validation. In fact, I think it was noticed then and fixed. This is why I'm asking what version they are using.

@armengau
Copy link
Collaborator Author

Current status: we have no proof at this point that it's not due to conversions by an excel-like software.

In that case it's more a documentation issue. We already know since SV that one should not open prospect's csv outputs with excel, which by default seems to convert longs to floats.

@weaverba137
Copy link
Member

Ah, that sounds familiar too. I'll ping Arjun about this.

@armengau
Copy link
Collaborator Author
armengau commented Sep 6, 2023

Status after discussion at Durham: this is just a documentation issue (csv files should not be opened by excel)

@armengau armengau removed the bug Something isn't working label Sep 6, 2023
@weaverba137
Copy link
Member

@armengau, do we need to keep this open?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@armengau armengau

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@weaverba137 @armengau
0