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

Document and test calculate param #673

Merged
merged 5 commits into from
Sep 16, 2024
Merged

Document and test calculate param #673

merged 5 commits into from
Sep 16, 2024

Conversation

bart1
Copy link
Collaborator

@bart1 bart1 commented Sep 13, 2024

Provides a work-around and example for issue #672

@bart1 bart1 requested a review from adokter September 13, 2024 09:03
@bart1
Copy link
Collaborator Author

bart1 commented Sep 16, 2024

@adokter I think this can be merged, all errors seem to be vpts related

@bart1 bart1 merged commit bf6a8c5 into master Sep 16, 2024
1 of 5 checks passed
@adokter
Copy link
Owner

adokter commented Sep 16, 2024

Correct, the test failures that appeared are likely related to a bug in readr, I reported the bug here: tidyverse/readr#1555

@bart1 bart1 deleted the issue_#672 branch September 16, 2024 14:02
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