Skip to content
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 account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Column overrun on ADJ file #78

Closed
jameslevershadelwp opened this issue Nov 30, 2020 · 2 comments · Fixed by #83
Closed

Column overrun on ADJ file #78

jameslevershadelwp opened this issue Nov 30, 2020 · 2 comments · Fixed by #83

Comments

@jameslevershadelwp
Copy link
Contributor

On erroneous output in some columns, the data overruns into the next columns. This makes it extremely difficult in automatically parsing results. Can you truncate these before output, possibly go to scientific if it breaches 0 decimal places?

The adjustment sd is so large it runs into the residual columns when I parse the columns using the specified column details in the manual I get

240009879.931 and 2240009879.9310

rather than the actual

240009879.9312 and 2240009879.9310

image

Please see attached adj file, I had to change it to text to upload.

TP325001Y.simult.txt
@jameslevershadelwp
Copy link
Contributor Author

TP325001Y.stn.txt
TP325001Y.msr.txt

@stratiform
Copy link

stratiform commented Feb 18, 2021 •
edited
Loading

Additional data to test - contains overflows in COR file and ADJ file
690d1ab601ca44f18ec5e08bf1a1858c.phased.cor.txt

690d1ab601ca44f18ec5e08bf1a1858c.phased.adj.txt

Full adjustment data and inputs:
PS527037N.zip

rogerfraser added a commit to rogerfraser/DynAdjust that referenced this issue Feb 26, 2021
@rogerfraser
Initial changes to fix issue icsm-au#78
0f648e6
rogerfraser added a commit to rogerfraser/DynAdjust that referenced this issue Mar 2, 2021
@rogerfraser
Further code changes to address issue icsm-au#78
55dd973
rogerfraser added a commit to rogerfraser/DynAdjust that referenced this issue Mar 12, 2021
@rogerfraser
Further code changes to address issue icsm-au#78
215fb1c
@rogerfraser rogerfraser mentioned this issue Mar 13, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Minor changes to prevent column overrun in fixed-width output files rogerfraser/DynAdjust
2 participants
@stratiform @jameslevershadelwp