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.

Sign up for GitHub

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

Jump to bottom

Ssld: Modifie condition for column I #1108

Merged
merged 3 commits into from
Dec 10, 2024
Merged

Ssld: Modifie condition for column I #1108

merged 3 commits into from
Dec 10, 2024

Conversation

TruongQuangSB
Copy link
Contributor

  • Fix wrong determine distance between the start signal and the target FMA
  • The target FMA must have same direction like start Signal
  • When the distance between the start signal and the target FMA longer than the Dweg Is-Length or shorter than the Dweg Soll-Length, then take Ist-Length of the Dweg

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 28, 2024 •
edited
Loading

Test Results

121 tests   121 ✅  17s ⏱️
 28 suites    0 💤
 28 files      0 ❌

Results for commit b838c37.

♻️ This comment has been updated with latest results.

@github-actions github-actions bot mentioned this pull request Nov 28, 2024
Closed
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 9, 2024

License summary

❌ Not yet vetted dependencies:

Dependency License Status Ticket
maven/mavencentral/org.hamcrest/hamcrest/2.2 None restricted #17677
  • Committers can request a license review via by commenting /request-license-review.
  • After all reviews have concluded, Committers can re-run the license-vetting check by commenting /license-check

Workflow run (with attached summary files):
https://github.com/eclipse-set/set/actions/runs/12237983272

@mariusheine mariusheine merged commit bc26b13 into main Dec 10, 2024
11 of 12 checks passed
@mariusheine mariusheine deleted the fix-dweg-length branch December 10, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mariusheine mariusheine mariusheine approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TruongQuangSB @mariusheine