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

fix: fixing proportion check for SMP mix calculations #1547

Merged
merged 3 commits into from
Aug 28, 2024

Conversation

mgaseta
Copy link
Collaborator

@mgaseta mgaseta commented Aug 26, 2024

Description

Closes #1541

Took more time testing than fixing, but seems to working right.

Changelog

New

  • Nothing :)

Changed

  • Changed an if >.<

Removed

  • Nothing :)

How was this tested?

  • 🧠 Not needed
  • 👀 Eyeball
  • 🤖 Added tests

What gif/image best describes this PR or how it makes you feel?


Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

@mgaseta mgaseta self-assigned this Aug 26, 2024
@mgaseta mgaseta linked an issue Aug 26, 2024 that may be closed by this pull request
3 tasks
Copy link
Contributor

@RMCampos RMCampos left a comment

Choose a reason for hiding this comment

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

LGTM

@RMCampos
Copy link
Contributor

RMCampos commented Aug 28, 2024

Cypress tests are still failing. Lots of them. Incredibly, everything is passing locally. We need to take closer look at cypress and the environment. Tests are failing only on github actions, where we need them to pass.

I'll merge this PR. An issue will address this bug: #1565

@RMCampos RMCampos merged commit 041409c into main Aug 28, 2024
23 of 25 checks passed
@RMCampos RMCampos deleted the fix/1541-fix-smp-mix-single-tree-submission branch August 28, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix SMP Mix single tree submission
2 participants