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

Add unit tests #8

Open
1 of 2 tasks
wleoncio opened this issue Nov 24, 2021 · 0 comments
Open
1 of 2 tasks

Add unit tests #8

wleoncio opened this issue Nov 24, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@wleoncio
Copy link
Member

wleoncio commented Nov 24, 2021 •
edited
Loading

  • Add examples as unit tests
  • Get expectations from output of original EMJMCMC2016 repo
@wleoncio wleoncio added the enhancement New feature or request label Nov 24, 2021
wleoncio added a commit that referenced this issue Nov 26, 2021
@wleoncio
Added unit tests (#8)
6d49866
wleoncio added a commit that referenced this issue Nov 26, 2021
@wleoncio
Isolated tests for long functions (#8)
02adf18 
These functions (with the exception of `parall.gmj`) are explicitly
mentioned on README.md, and the execution of their examples is not
straightforward, so their test scripts are separated. Perhaps using the
function examples is not the way to go, and new, shorter (sub 5-second) test code needs
to be written before this issue is resolved.
This was referenced Nov 26, 2021
Closed
Merged
wleoncio added a commit that referenced this issue Nov 29, 2021
@wleoncio
Added linter to CI workflow (#8)
06e95b9
wleoncio added a commit that referenced this issue Nov 30, 2021
@wleoncio
Adding example of runemjmcmc as test prep (#8)
3b4cbf2
wleoncio added a commit that referenced this issue Nov 30, 2021
@wleoncio
Added expectations for runemjmcmc (#8)
d6e4103
wleoncio added a commit that referenced this issue Nov 30, 2021
@wleoncio
Added test seed, adjusted expectations (#8)
5198610
wleoncio added a commit that referenced this issue Nov 30, 2021
@wleoncio
Reduced test size, adjusted expectations (#8)
b440677
wleoncio added a commit that referenced this issue Nov 30, 2021
@wleoncio
Added code from README to test (#8)
5d063b4
wleoncio added a commit that referenced this issue Nov 30, 2021
@wleoncio
Added test for pinferunemjmcmc (#8)
6793ce6
wleoncio added a commit that referenced this issue Nov 30, 2021
@wleoncio
Sped up test (#8)
4c87bb6
wleoncio added a commit that referenced this issue Nov 30, 2021
@wleoncio
Suppressing messages during tests (#8)
acdd9ba
wleoncio added a commit that referenced this issue Apr 1, 2022
@wleoncio
Syntax improvements to estimate.gamma.cpen (#8)
b49dc6d 
This is related to issue #8 because it fixes unit tests.

Summary of changes:

- Automatic restyling
- Added alternative as `estimate.gamma.cpen_2` (to be merged in the
future, since they are quite similar)
- Incorporated functions from the `relat` argument
wleoncio added a commit that referenced this issue Feb 19, 2024
@wleoncio
Added unit tests for #8
5feb0c4
wleoncio added a commit that referenced this issue Feb 19, 2024
@wleoncio
Reduced test length (#8)
4c91632
wleoncio added a commit that referenced this issue Feb 19, 2024
@wleoncio
Resolved TODOs (#8)
24a6e30 
Didn't manage to run code on 1.4.3. however, the tests are generic enough to work anyway. We should do more tests to cover the values themselves at another point
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@wleoncio