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

Small edits on grammar #33

Merged
merged 3 commits into from
Sep 2, 2024
Merged

Small edits on grammar #33

merged 3 commits into from
Sep 2, 2024

Conversation

ppxasjsm
Copy link
Contributor

@ppxasjsm ppxasjsm commented Sep 1, 2024

I have gone through the text and am making small suggestions for changes. Please have a look at them and let me know if you are happy to accept.

@agerada
Copy link
Owner

agerada commented Sep 2, 2024

Thank you @ppxasjsm, I agree with all the changes.

Furthermore I have made the following updates:

  1. keras and opencv references changed to hyperlinks since no DOI available
  2. Caught a few misuses of the term "test", where "assay" is a preferable term (previously flagged up by @gchure but a couple of uses were missed).

The tests are passing, from what I can see the failed checks are because the workflow doesn't have permission to publish the coverage comment, so I will go ahead and merge with my repo.

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.

2 participants