-
Notifications
You must be signed in to change notification settings - Fork 390
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
Global PlantPipingSystemsMgr #8493
Global PlantPipingSystemsMgr #8493
Conversation
@mitchute any thoughts on what change is causing the big math diff before I review this? |
I was surprised by that one, but I haven't had a chance to dig into it yet. |
@Myoldmopar the diffs we're seeing across multiple branches from |
Awesome! OK, as soon as CI reports back, this can merge in. |
The diff is actually fixed in this branch now, so merging this in will clean that up for future runs. This is good to go. Thanks @mitchute ! |
Pull request overview
Pull Request Author
Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
Reviewer
This will not be exhaustively relevant to every PR.