-
Notifications
You must be signed in to change notification settings - Fork 37
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
Analysis: Refactoring to improve readibility + efficiency. #1
base: main
Are you sure you want to change the base?
Conversation
Updating to wrap main script in if__name__=="main" statement. Worked with @Jennivine. |
Updating to simplify creation of results array in workflow1.py. Worked with @Jennivine. |
Simplifying calculation + printing of number of critical results. Worked with @Jennivine. |
Updating workflow2.py to import + use read_file function from workflow1.py. Worked with @Jennivine. |
Adding utils.py file to house functions used by both workflows. Worked with @Jennivine. |
Wrapping script in workflow2.py in if name == "main" statement. Worked with @Jennivine. |
Simplifying calculation of dsum with numpy in workflow2.py. Worked with @Jennivine. |
Creating + implementing read_files function in workflow1.py. Answers UCL-COMP0233-24-25/RSE-Classwork#42. Worked with @Jennivine.