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.

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

Update American Data Sets #18

Closed
rBatt opened this issue Nov 11, 2015 · 4 comments
Closed

Update American Data Sets #18

rBatt opened this issue Nov 11, 2015 · 4 comments

Comments

@rBatt
Copy link
Owner

rBatt commented Nov 11, 2015

these can probably just be taken from the recent Ocean Adapt udpate

@rBatt
Copy link
Owner Author

rBatt commented Dec 14, 2015

I noticed that, for GMEX, we previously had a file for CTD casts, and something else called GLFREC. I'm dropping these from the package. @mpinsky @JWMorley @bselden let me know if you have a need for them.

@rBatt
Copy link
Owner Author

rBatt commented Dec 14, 2015

@JWMorley for the seus file on ocean adapt, did you do any pre-processing? I don't see all the data files there, and the names that seem to match have smaller file sizes in your version. That doesn't seem right to me

@rBatt
Copy link
Owner Author

rBatt commented Dec 15, 2015

I don't think there were any updates to the wctri, so I didn't bother with that

rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
the files aren't the same; see question to @JWMorley on #18

will need to change read.trawl to match
rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
@mpinsky good call, datetime exists in new data set; @bselden might want to check with your sources, but this approach infers far less; datetime is specified
rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
@mpinsky
Copy link
Collaborator

mpinsky commented Dec 15, 2015

Correct. That hasn't been updated recently.

On Monday, December 14, 2015, Ryan Batt notifications@github.com wrote:

I don't think there were any updates to the wctri, so I didn't bother with
that


Reply to this email directly or view it on GitHub
#18 (comment).

Please excuse, sent from a device with tiny keys...

rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 15, 2015
rBatt added a commit that referenced this issue Dec 16, 2015
@rBatt rBatt closed this as completed in fc57d51 Dec 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants