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

Make sure crow vtps reader can handle csv files with duplicate timestamps #179

Open
stijnvanhoey opened this issue Jan 23, 2023 · 3 comments
Assignees
Milestone

Comments

@stijnvanhoey
Copy link
Collaborator

See enram/vptstools#22, we have to check if this works and take first one when it occurs.

@peterdesmet
Copy link
Member

@stijnvanhoey the application works with duplicate timestamps, but I think the last, rather than the first row is used.

@peterdesmet peterdesmet added this to the Aloft milestone Apr 11, 2023
@peterdesmet
Copy link
Member

Todo:

  • Use first row
  • Add a test for this

@stijnvanhoey
Copy link
Collaborator Author

I would convert (or add to) the txt-files in the tests to the corresponding CSV version in these tests:

This would check the usage of the first occurrence.

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

No branches or pull requests

2 participants