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

85 cpp tests #117

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

85 cpp tests #117

wants to merge 15 commits into from

Conversation

dimalvovs
Copy link
Contributor

@dimalvovs dimalvovs commented Sep 30, 2024

  • fix outdated catch version;
  • reverse the test compilation condition (make ON default, used to be OFF);
  • add mechanics of calling cpp tests from within R;
  • add possibility to use tags to call tests (for example to call groups of cpp tests);
  • fix a bug that prevented configure to parse version from DESCRIPTION

@dimalvovs dimalvovs linked an issue Sep 30, 2024 that may be closed by this pull request
@dimalvovs dimalvovs marked this pull request as ready for review November 22, 2024 18:00
@dimalvovs
Copy link
Contributor Author

Interestingly, there is a strong execution time dependency on whether the package was bullt or loaded with load_all()

devtools::load():

On branch master
Your branch is up to date with 'origin/master'.

# A tibble: 2 × 13
  expression      min median `itr/sec` mem_alloc `gc/sec` n_itr  n_gc total_time
  <bch:expr> <bch:tm> <bch:>     <dbl> <bch:byt>    <dbl> <int> <dbl>   <bch:tm>
1 dense         8.86s  8.91s     0.112    1.59MB   0.0280     4     1      35.7s
2 sparse        3.91s  3.95s     0.254    2.07MB   0          5     0      19.7s
On branch 85-cpp-tests
This is CoGAPS version 3.25.1 
Running Standard CoGAPS on GIST.data_frame (1363 genes and 9 samples)
# A tibble: 2 × 13
  expression      min median `itr/sec` mem_alloc `gc/sec` n_itr  n_gc total_time
  <bch:expr> <bch:tm> <bch:>     <dbl> <bch:byt>    <dbl> <int> <dbl>   <bch:tm>
1 dense         8.95s  9.02s     0.111    1.59MB   0          5     0      45.2s
2 sparse        3.96s     4s     0.247    2.07MB   0.0617     4     1      16.2s

Built and Installed:

On branch master
# A tibble: 2 × 13
  expression      min median `itr/sec` mem_alloc `gc/sec` n_itr  n_gc total_time
  <bch:expr> <bch:tm> <bch:>     <dbl> <bch:byt>    <dbl> <int> <dbl>   <bch:tm>
1 dense         533ms  534ms      1.86     1.4MB        0     5     0      2.69s
2 sparse        915ms  921ms      1.09     1.2MB        0     5     0       4.6s

On branch 85-cpp-tests

# A tibble: 2 × 13
  expression      min median `itr/sec` mem_alloc `gc/sec` n_itr  n_gc total_time
  <bch:expr> <bch:tm> <bch:>     <dbl> <bch:byt>    <dbl> <int> <dbl>   <bch:tm>
1 dense         533ms  539ms      1.86     1.4MB        0     5     0      2.69s
2 sparse        915ms  920ms      1.08     1.2MB        0     5     0      4.61s

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.

Cpp tests
2 participants