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

test house-of-representative: reduce checking #208

Conversation

tikkss
Copy link
Contributor

@tikkss tikkss commented Sep 2, 2024

GitHub: GH-188

Because csv file is too big (10,779 rows and 41 columns).

Before this change:

$ time ruby test/run-test.rb -t HouseOfRepresentativeTest --verbose=important-only
Finished in 2.679117 seconds.
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m3.295s
user    0m3.031s
sys     0m0.152s

After this change:

$ time ruby test/run-test.rb -t HouseOfRepresentativeTest --verbose=important-only
Finished in 0.095506 seconds.
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m0.659s
user    0m0.465s
sys     0m0.103s

GitHub: red-data-toolsGH-188

Because csv file is too big (10,779 rows and 41 columns).

Before this change:

```console
$ time ruby test/run-test.rb -t HouseOfRepresentativeTest --verbose=important-only
Finished in 2.679117 seconds.
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m3.295s
user    0m3.031s
sys     0m0.152s
```

After this change:

```console
$ time ruby test/run-test.rb -t HouseOfRepresentativeTest --verbose=important-only
Finished in 0.095506 seconds.
1 tests, 1 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0
notifications

real    0m0.659s
user    0m0.465s
sys     0m0.103s
```
@kou kou merged commit f38bede into red-data-tools:master Sep 3, 2024
12 checks passed
@kou
Copy link
Member

kou commented Sep 3, 2024

Thanks.

@tikkss tikkss deleted the test-house-of-representative-reduce-checking branch September 3, 2024 03:17
@tikkss
Copy link
Contributor Author

tikkss commented Sep 3, 2024

Thanks for your review!

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.

None yet

2 participants