Skip to content

added virus injection documents #351

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
Merged
merged 13 commits into from
May 14, 2025
Merged

added virus injection documents #351

merged 13 commits into from
May 14, 2025

Conversation

@msalvi1
Copy link
Collaborator

@msalvi1 msalvi1 commented May 13, 2025

virus_injection blank and schema documents

jesshaley and others added 13 commits April 27, 2025 21:20
@jesshaley
Adding unit test for oneepoch
3a5024a
@jesshaley
Merge pull request #338 from VH-Lab/main
d1a58bc 
Update to latest NDI ndr reader
@jesshaley
Added fixtures and helper functions
2053d38 
Still need to reorganize test structure.
@jesshaley
New functionality for OneEpochTest
47ab780 
Still finishing up documentation and organization of unittest
@jesshaley
Completed draft of OneEpochTest and its fixtures
12f3ab6
@ehennestad
Ensure the resulting path is a char, not a string
dd3f1a2
@stevevanhooser
Merge pull request #343 from VH-Lab/fix-ndi-pathconstants
1bc439a 
Fix bug in PathConstants when multiple calculator document locations are available
@jesshaley
Swapped verifyClass with verifyEqual
f5bc9b4
@stevevanhooser
Merge pull request #337 from VH-Lab/oneepoch-unittest
a707bc4 
Adding unit test for oneepoch
@ehennestad
Fix broken links
6ec0690
@ehennestad
Update oldREADME.md
26f7474
@stevevanhooser
Merge pull request #346 from VH-Lab/fix-documentation-hyperlinks
91a9e7f 
Fix broken links in documentation
@msalvi1
added virus injection documents
1517746
@msalvi1 msalvi1 requested a review from stevevanhooser May 13, 2025 17:20
@msalvi1 msalvi1 closed this May 13, 2025
@msalvi1 msalvi1 reopened this May 13, 2025
@stevevanhooser stevevanhooser changed the base branch from main to dabrowska May 14, 2025 01:26
stevevanhooser
stevevanhooser approved these changes May 14, 2025
View reviewed changes
Copy link
Contributor

@stevevanhooser stevevanhooser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Megha!

@stevevanhooser stevevanhooser merged commit 0790ae9 into dabrowska May 14, 2025
@stevevanhooser stevevanhooser deleted the virus_injection_docs branch May 14, 2025 15:21
@stevevanhooser
Copy link
Contributor

@msalvi1 Thanks! I merged this into branch dabrowska. Please checkout a different branch going forward because virus_injection_docs is now deleted on the GitHub site. (We delete branches when they are merged, otherwise we'd have tons of old branches)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stevevanhooser stevevanhooser stevevanhooser approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@msalvi1 @stevevanhooser @jesshaley @ehennestad