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

Values for LSL, USL, target labels and unit tests updates #333

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

JTPetter
Copy link
Contributor

  • Added the specified value to the USL, LSL and target labels in the process capability plot.

@JTPetter JTPetter requested a review from vandenman September 26, 2024 13:20
@vandenman
Copy link
Contributor

can you resolve the conflicts? you probably need to rebase and recreate the two figures

@JTPetter
Copy link
Contributor Author

JTPetter commented Oct 9, 2024

@vandenman I rebased. The unit tests are fine for me locally, not sure why they fail on GitHub. There is also this message "There was a problem generating the rebase commit". Would you happen to know how to fix this?

@JTPetter
Copy link
Contributor Author

JTPetter commented Oct 9, 2024

Okay "Squash and merge" would work, if we would go ahead despite the unit tests

@JTPetter
Copy link
Contributor Author

Fixed the unit tests, so I will merge this

@JTPetter JTPetter merged commit b3204f7 into jasp-stats:master Oct 16, 2024
5 checks passed
@JTPetter JTPetter deleted the pcsChartLabels branch October 16, 2024 13:36
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.

2 participants