-
Notifications
You must be signed in to change notification settings - Fork 15
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
6.3 Deployment #3178
6.3 Deployment #3178
Conversation
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.
Reviewed with the Lighting_Engine test methods, all passed.
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.
Tested Environment_Engine using BHoM_6.3.230919.beta.0.msi and all tests passed
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.
Approving:
- Diffing_Engine with 230915 6.3 installer and related test procedure
- Analytical_Engine (Graph) with 230915 6.3 installer and related test procedure
@bhombot-ci[bot] just to let you know, I have provided a |
Tested MEP_Engine 20/9 Passed. |
Tested using Facade_Engine procedure. Passed |
@bhombot-ci[bot] just to let you know, I have provided a |
@BHoMBot check versioning |
@FraserGreenroyd to confirm, the following actions are now queued:
|
@FraserGreenroyd to confirm, the following actions are now queued:
|
Please be advised that the check with reference 16981029710 has more than 50 annotations of notes. API limitations restrict annotations to 50. You may need to rerun this check to obtain the next set when you make changes. At the time of reporting this check, there are 29 additional annotations waiting, made up of 0 errors and 29 warnings. |
@FraserGreenroyd to confirm, the following actions are now queued:
There are 18 requests in the queue ahead of you. |
The check |
The check |
The check |
@bhombot-ci[bot] just to let you know, I have provided a |
@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: beta-code-compliance, beta-documentation-compliance |
@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results. |
@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: code-compliance, documentation-compliance |
@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results. |
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.
PR Checked by DevOps following Testing Sprint. Everything is satisfactory, DevOps clear to merge.
@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: ready-to-merge |
@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results. |
BHoM_Engine
Feature
PadFoundation
andPileFoundation
to work withIElement2D
andIElementM
respectively;Create
methods forPadFoundation
,PileFoundation
andPileGroup
IsNull
methods to include new foundation objects;GrahamScan
method to determine the perimeter of aList<Point>
which is used in theCreate
methods for aPileFoundation
.Bug
Poyline
was returning morePoint
s than were being input.FromJson()
to Trim the text before attempting to read itCompliance
Test-script
Documentation
Uncategorised