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

Figure out how to handle grad errors #11

Open
baileyparker opened this issue Feb 20, 2018 · 1 comment
Open

Figure out how to handle grad errors #11

baileyparker opened this issue Feb 20, 2018 · 1 comment
Assignees

Comments

@baileyparker
Copy link
Owner

No description provided.

@baileyparker baileyparker self-assigned this Feb 20, 2018
@baileyparker
Copy link
Owner Author

Currently the test assertion is that if the phase file has a line starting with error: , we expect stderr to start with error: as well. This allows grads to continue emitting errors without the test harness complaining that stderr doesn't exactly match. Ideally, we should be able to assert that in the grad case the correct number of errors are omitted. Additionally, it would be a nicety if we could have some sort of fuzzy matching on location (although this might be tricky, because I'm not convinced that every error has a only one valid location that can be reported).

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