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 usage of if inside paste() which works differently from c(). #180

Merged
merged 1 commit into from
Mar 6, 2025

Conversation

astamm
Copy link
Owner

@astamm astamm commented Mar 6, 2025

After merging #176, package kergp was newly broken. Logs were:

Run revdepcheck::revdep_details(, "kergp") for more info

Newly broken

  • checking examples ... ERROR
    Running examples in ‘kergp-Ex.R’ failed
    The error most likely occurred in:
    
    > ### Name: optimMethods
    > ### Title: Optimization Methods (or Algorithms) for the 'mle' Method
    > ### Aliases: optimMethods
    > 
    > ### ** Examples
    > 
    > optimMethods()
    Error in data.frame(optimFun = c(rep("stats::optim", length(oMO)), rep("nloptr::nloptr",  : 
      arguments imply differing number of rows: 44, 43
    Calls: optimMethods -> data.frame
    Execution halted
    

It turns out that the use of if statement within the paste() function behaves differently from its use within c().

@astamm astamm merged commit 69e7077 into master Mar 6, 2025
10 checks passed
@astamm astamm deleted the fix-kergp branch March 6, 2025 16:53
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.

1 participant