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

ba_trials regression #76

Closed
mverouden opened this issue Sep 13, 2019 · 1 comment
Closed

ba_trials regression #76

mverouden opened this issue Sep 13, 2019 · 1 comment

Comments

@mverouden
Copy link
Collaborator

The revision of the ba_trials call caused a regression with respect to the BrAPI specifications. On the test-server the argument active can have three values:

  • all results for active = true and active = false are shown
    true only results where active = true are shown
    false, where only results when active = false are shown

By setting the argument in the ba_trials to only logical the BrAPI R package does not allow for displaying all results. For that reason active = "any" was introduced.

@mverouden
Copy link
Collaborator Author

Resolved the issue by setting active = NA as default setting

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

No branches or pull requests

1 participant