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

Add test config for aatams_acoustic_qc #154

Merged
merged 1 commit into from
May 28, 2019

Conversation

jonescc
Copy link
Contributor

@jonescc jonescc commented May 24, 2019

Time zone processing issue with this harvester as well.

@jonescc
Copy link
Contributor Author

jonescc commented May 24, 2019

6-nec-hob test results: aatams_acoustic_qc.zip

@gsatimos
Copy link
Contributor

@jonescc
Noted, that there is time time zone issue, but the atf_acoustic_qc_detections_map table has a colour column that needs to be excluded.

@jonescc jonescc force-pushed the talend-test-aatams_acoustic_qc branch from e6463bf to bb69f9c Compare May 27, 2019 01:00
@jonescc
Copy link
Contributor Author

jonescc commented May 27, 2019

Hmm.. Looks like I'd already made the change but hadn't pushed it up. It's been pushed up now

@gsatimos gsatimos merged commit 5461ad4 into master May 28, 2019
@gsatimos gsatimos deleted the talend-test-aatams_acoustic_qc branch May 28, 2019 05:26
@lbesnard
Copy link
Contributor

@gsatimos not sure that we wanted to merge this one as yet until this PR aodn/harvesters#744 could be tested?

@gsatimos
Copy link
Contributor

@lbesnard we've marked it on our spreadsheet that we're keeping to track issues. We'll definitely be giving it a test once that issue is fixed and merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants