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

Fareto #390

Merged
merged 3 commits into from
Jun 12, 2024
Merged

Fareto #390

merged 3 commits into from
Jun 12, 2024

Conversation

mattwigway
Copy link
Contributor

This adds support for rfareto to r5r.

@mattwigway
Copy link
Contributor Author

If you'd prefer to integrate rfareto into r5r, that's okay with me as well.

@mattwigway
Copy link
Contributor Author

The mac build failures I believe are unrelated to this PR; it looks like an x86_64 java is getting installed on an arm64 mac.

@rafapereirabr
Copy link
Member

thanks so much, @mattwigway !

@rafapereirabr rafapereirabr merged commit a25bae5 into ipeaGIT:master Jun 12, 2024
7 of 9 checks passed
@mattwigway
Copy link
Contributor Author

@rafapereirabr are you interested in integrating rfareto into r5r? Alternately maybe we add usage/install instructions to the pareto vignette.

@rafapereirabr
Copy link
Member

Hi @mattwigway , I haven't thought about this carefully yet but it sounds like a good idea. Let me try to use it to see how it works and I'd be happy to discuss this after that

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