-
Notifications
You must be signed in to change notification settings - Fork 30
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.
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
MLnz20 Original calibration - table and reference #2242
Conversation
WIP - will update with the station correction |
Need an entry in README |
5b63305
to
b98cf2c
Compare
MLnz20 entries Set Station Correction-calibration for MLNz20
b98cf2c
to
41d7913
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Barring one comment to consider and assuming the corrections are pulled straight from the Table S3.csv, LGTM.
just a comment as I bumped into this. In delta, we already have a |
@staylorofford Okay |
@staylorofford @Thomas-Benson Up we go . |
This is probably not how I would have laid out this file. I'd think: (a) Is the notes field really necessary? I'd much prefer a "lookup table" if there was something to say, free form fields are hard to validate, and is it needed. In the past there wasn't really any concept of a location code when it came to station corrections, do you therefore need to keep track of all variations, and magnitude differences related to boreholes would be very hard to isolate. Jury is out on this one, as it really limits when these corrections can be applied. |
Hello @ozym ! |
Provisional entry for magnitude calibration (MLnz20)