-
Notifications
You must be signed in to change notification settings - Fork 2
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
Updates to simulations, external comfort, and plots #106
Conversation
@BHoMBot check required |
@jamesramsden-bh to confirm, the following actions are now queued:
There are 171 requests in the queue ahead of you. |
@FraserGreenroyd to confirm, the following actions are now queued:
|
@BHoMBot check required |
@jamesramsden-bh to confirm, the following actions are now queued:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have run through the code with @jamesramsden-bh on a call and all is in line with his testing of work done in collaboration with @tg359 - happy to approve to deploy to unblock them.
@jamesramsden-bh to confirm, the following actions are now queued:
|
Includes some refactoring off the back of several projects within the python src folder. Test against the usual test batch file within the Python folder.
Closes #98
Closes #100
Closes #104