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

Environment_Engine: Hard coded default acceptOnEdges to true #2872

Merged
merged 1 commit into from
Jun 20, 2022

Conversation

jamesramsden-bh
Copy link
Contributor

Closes #2871

Turns out that the two bugs were related - fixing one fixed the other.

Test files

Test via sections 1 and 3 of the Environment_Engine test procedure.

@jamesramsden-bh
Copy link
Contributor Author

@BHoMBot check required

@bhombot-ci
Copy link

bhombot-ci bot commented Jun 17, 2022

@jamesramsden-bh to confirm, the following checks are now queued:

  • code-compliance
  • documentation-compliance
  • project-compliance
  • core
  • null-handling
  • serialisation
  • versioning
  • installer

@jamesramsden-bh
Copy link
Contributor Author

@BHoMBot check dataset-compliance
@BHoMBot check copyright-compliance
@BHoMBot check ready-to-merge

@bhombot-ci
Copy link

bhombot-ci bot commented Jun 17, 2022

@jamesramsden-bh to confirm, the following checks are now queued:

  • dataset-compliance
  • copyright-compliance
  • ready-to-merge

There are 9 requests in the queue ahead of you.

@jamesramsden-bh jamesramsden-bh self-assigned this Jun 17, 2022
@jamesramsden-bh jamesramsden-bh added the type:bug Error or unexpected behaviour label Jun 17, 2022
@jamesramsden-bh
Copy link
Contributor Author

@BHoMBot check ready-to-merge

@bhombot-ci
Copy link

bhombot-ci bot commented Jun 17, 2022

@jamesramsden-bh to confirm, the following checks are now queued:

  • ready-to-merge

Copy link
Contributor

@FraserGreenroyd FraserGreenroyd left a comment

Choose a reason for hiding this comment

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

Changes match what was agreed and tested display working functionality akin to the previous milestone.

@FraserGreenroyd
Copy link
Contributor

@BHoMBot check ready-to-merge

@bhombot-ci
Copy link

bhombot-ci bot commented Jun 20, 2022

@FraserGreenroyd to confirm, the following checks are now queued:

  • ready-to-merge

@FraserGreenroyd FraserGreenroyd merged commit d29a9e1 into main Jun 20, 2022
@FraserGreenroyd FraserGreenroyd deleted the BHoM_Engine-#2871-ToleranceBugs branch June 20, 2022 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Error or unexpected behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Environment_Engine: Bugs relating to tolerance
2 participants