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

qs::c_qsave moved to the runner #14

Merged
merged 1 commit into from Aug 1, 2022
Merged

qs::c_qsave moved to the runner #14

merged 1 commit into from Aug 1, 2022

Conversation

FabrizioSandri
Copy link
Owner

@FabrizioSandri FabrizioSandri commented Aug 1, 2022

The problem that incorrect inputs were included in the result table is resolved by this pull request.

Fixes #13

@FabrizioSandri FabrizioSandri changed the title qs::c_qsave moved to the runner (#13) qs::c_qsave moved to the runner Aug 1, 2022
@FabrizioSandri
Copy link
Owner Author

The test harness input saving part has been moved from the generator to the runner test. In this way, for each test case, the .qs file is created only when the runner is executed. Every time a test is analyzed with RcppDeepState the associated .qs file is read, without needing to worry that the subsequent test execution would replace it.

@FabrizioSandri FabrizioSandri merged commit 900a03d into master Aug 1, 2022
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.

Wrong inputs column
1 participant