Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Lyw unit test #146

Merged
merged 11 commits into from
Jul 10, 2023
Merged

Lyw unit test #146

merged 11 commits into from
Jul 10, 2023

Conversation

laywang142
Copy link
Collaborator

@laywang142 laywang142 commented Jul 7, 2023

I just made a few tests based on what was mentioned during the meeting, but I'm not sure if it's exactly what you're looking for. I would appreciate it if you could check them out and give me some feedback. I did the checks for the functions in lines 87-93 on the excel sheet. Thanks.
@harriscw

@laywang142 laywang142 requested a review from harriscw July 7, 2023 20:32
@harriscw
Copy link
Collaborator

Looks good thanks!

@harriscw harriscw merged commit 0056e2a into devel Jul 10, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants