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.

Sign up for GitHub

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

Jump to bottom

adding an atime test case for gshift as gforce optimized shift #5205 #6295

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

adding an atime test case for gshift as gforce optimized shift #5205 #6295

Changes from 11 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
0013772
adding an atime test case for gshift as gforce optimized shift #5205
DorisAmoakohene Jul 17, 2024
25e276f
Fixed some tense error in comments
DorisAmoakohene Jul 29, 2024
93396ca
Fixed typo
DorisAmoakohene Jul 30, 2024
2f4f570
Resolved conflicts between adding-a-test-for-regression-for-5205 and …
DorisAmoakohene Jul 30, 2024
bb10ce8
remove >> and == from script
DorisAmoakohene Jul 31, 2024
c899b9c
added NULL
DorisAmoakohene Jul 31, 2024
1f8dbbb
Towards resolving merge conflicts
Anirban166 Sep 13, 2024
6e79980
Merge branch 'master' into adding-a-test-for-regression-for-5205
Anirban166 Sep 13, 2024
1ed4d1a
Formatting
Anirban166 Sep 13, 2024
f394d91
Correct comment based on fix status
Anirban166 Sep 13, 2024
3224e7a
Try the last one in PR's range of commits (else simplify comment to u…
Anirban166 Sep 13, 2024
7a25d4f
Slow/Fast (Improvement case) over Regression/Fixed
Anirban166 Sep 16, 2024
187c79c
Merge branch 'master' into adding-a-test-for-regression-for-5205
MichaelChirico Sep 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions .ci/atime/tests.R
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -166,5 +166,16 @@ test.list <- atime::atime_test_list(
Slow = "a01f00f7438daf4612280d6886e6929fa8c8f76e", # Parent of the first commit (https://github.com/Rdatatable/data.table/commit/fc0c1e76408c34a8482f16f7421d262c7f1bde32) in the PR (https://github.com/Rdatatable/data.table/pull/6296/commits) that fixes the issue
Fast = "f248bbe6d1204dfc8def62328788eaadcc8e17a1"), # Merge commit of the PR (https://github.com/Rdatatable/data.table/pull/6296) that fixes the issue

# Issue reported in: https://github.com/Rdatatable/data.table/issues/1534
# Fixed in: https://github.com/Rdatatable/data.table/pull/5205
"DT shift improved in #5205" = atime::atime_test(
N = 10^seq(1, 7),
setup = {
DT = data.table(x = sample(N), y = sample(1e2, N, TRUE))
},
expr = data.table:::`[.data.table`(DT, , shift(x, 1, type = "lag"), y),
Regression = "7f0ce147eef1dd5cd5ff05dffc3c72f472bcde51", # Parent of the first commit (https://github.com/Rdatatable/data.table/commit/58135017a985f3cc2c6f0d091c4effaec4442f56) in the PR (https://github.com/Rdatatable/data.table/pull/5205/commits) that fixes the issue
Fixed = "5eec8f2d00521edb0a4a088ad154d065daa869c6"), # Last commit in the PR (https://github.com/Rdatatable/data.table/pull/5205/commits) that fixes the regression

NULL)
tdhock marked this conversation as resolved.
Show resolved Hide resolved
# nolint end: undesirable_operator_linter.
Loading