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 irregular time series testing and randomly generated data to current tests. #55

Merged
merged 15 commits into from
Jan 10, 2023

Conversation

agneborn98
Copy link
Contributor

This pull request adds a generator for generating random test data and randomized irregular timestamps, new tests for testing irregular time series during compression, and a test to ensure that the selected model has the least amount of bytes. Additional fixes for the macOS and Windows testing environments have also been added to (hopefully) ensure that GitHub checks do not fail again on these platforms.

@agneborn98 agneborn98 self-assigned this Dec 21, 2022
@agneborn98 agneborn98 changed the title Dev/irregular ts unittesting Add irregular time series testing and randomly generated data to current tests. Dec 21, 2022
@CGodiksen CGodiksen requested review from CGodiksen and removed request for CGodiksen December 21, 2022 14:21
@agneborn98 agneborn98 requested a review from CGodiksen December 22, 2022 07:33
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/models/mod.rs Outdated Show resolved Hide resolved
@agneborn98 agneborn98 force-pushed the dev/irregular-ts-unittesting branch from 352bfaf to 7ee7526 Compare January 2, 2023 17:27
@agneborn98 agneborn98 requested a review from CGodiksen January 2, 2023 17:29
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
@agneborn98 agneborn98 force-pushed the dev/irregular-ts-unittesting branch from c46c9e5 to 112a00f Compare January 3, 2023 12:19
@agneborn98 agneborn98 requested a review from CGodiksen January 3, 2023 15:49
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
@agneborn98 agneborn98 requested a review from CGodiksen January 4, 2023 12:02
@skejserjensen skejserjensen removed the request for review from chrthomsen January 4, 2023 13:01
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/models/mod.rs Show resolved Hide resolved
server/tests/integration_test.rs Show resolved Hide resolved
server/src/compression.rs Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
server/src/compression.rs Outdated Show resolved Hide resolved
@agneborn98 agneborn98 force-pushed the dev/irregular-ts-unittesting branch from b260067 to 904841b Compare January 10, 2023 20:31
@skejserjensen skejserjensen merged commit 389cf6f into master Jan 10, 2023
@skejserjensen skejserjensen deleted the dev/irregular-ts-unittesting branch January 10, 2023 21:18
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