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

Task 50 #65

Merged
merged 39 commits into from
Mar 8, 2024
Merged

Task 50 #65

merged 39 commits into from
Mar 8, 2024

Conversation

seanbsu
Copy link
Collaborator

@seanbsu seanbsu commented Mar 7, 2024

implements #50

seanbsu and others added 28 commits February 24, 2024 17:04
…d first 2 tests to match what the function expects to be passed to it based on the Matlab implementation.
…tract method implementation in the test concrete class for compiling
… matlab, access the struct like how we will access it in the port(a list of dictionaries), use np dot() to multiply matrices instead of @ operator that was introduced in python 3.5
…nctionality after researching MatLab's left divide operator
… test.sh for ubuntu, trying to load mat files in tester to set up input struct
…ng a 2nd iteration like matlab but the first works
@seanbsu seanbsu requested a review from paulvxx March 7, 2024 18:24
Copy link
Collaborator

@paulvxx paulvxx left a comment

Choose a reason for hiding this comment

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

You can sometimes do for a in array: instead of for i in range(len(array))): where you only need to iterate over a single list (it often simplifies matters).

@seanbsu seanbsu merged commit d38f9b8 into main Mar 8, 2024
1 check failed
@seanbsu seanbsu deleted the task_50 branch March 8, 2024 17:41
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