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

Editing the error message for the sampling_zero_paths argument #91

Open
ashenvi10 opened this issue Aug 19, 2022 · 0 comments
Open

Editing the error message for the sampling_zero_paths argument #91

ashenvi10 opened this issue Aug 19, 2022 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers Small job
Milestone

Comments

@ashenvi10
Copy link
Collaborator

In the EventTree class, the sampling_zero_paths argument error message should be changed to reflect the following:
[ ] You do not, in fact, need to end the tuple with a comma for it to work. This what the current message says and we should remove this.
[ ] The elements in the tuple for the path should match exactly (they are case sensitive) with the edge labels in the tree. The error message should ask the user to confirm this.

@ashenvi10 ashenvi10 added enhancement New feature or request Small job good first issue Good for newcomers labels Aug 19, 2022
@ashenvi10 ashenvi10 added this to the v1.1 milestone Aug 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers Small job
Projects
None yet
Development

No branches or pull requests

1 participant