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

chore: revert s3methods back to v2.3.3 release #356

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

soumyaray
Copy link
Contributor

@soumyaray soumyaray commented Nov 19, 2024

Reverting breakage of S3 methods:

  • rhoC_AVE_pls_model -> rhoC_AVE.pls_model
  • rhoC_AVE_boot_seminr_model -> rhoC_AVE.boot_seminr_model
  • rhoC_AVE_cbsem_model -> rhoC_AVE.cbsem_model
  • rhoC_AVE_cfa_model -> rhoC_AVE.cfa_model
  • remove \r from line ends from files
  • clarify/rename as.reflective.matrix function in R/inspect_mmMatrix

@soumyaray soumyaray changed the base branch from master to develop November 19, 2024 02:49
@soumyaray
Copy link
Contributor Author

New warning from rcmdcheck appearing after upgrading R to 4.4.2 and clean reinstalling all packages:

W  checking S3 generic/method consistency ...
   rhoC_AVE:
     function(x, ...)
   rhoC_AVE.pls_model:
     function(pls_model, constructs)
   See section ‘Generic functions and methods’ in the ‘Writing R
   Extensions’ manual.

From https://cran.r-project.org/doc/manuals/R-exts.html#Generic-functions-and-methods:

A method must have all the arguments of the generic, including … if the generic does.

@soumyaray soumyaray changed the title chore: revert s3methods from prior to v2.3.3 release chore: revert s3methods back to v2.3.3 release Nov 19, 2024
@soumyaray
Copy link
Contributor Author

@NicholasDanks I think I'm done with the fixes for the S3 methods, and all tests and checks are passing. I also reverted the insertion of \r (CR) to the end of lines of several files. Let's discuss how to prevent these in future.

And I also want to discuss the intent of this function:

as.reflective.matrix <- function(x, ...) {

And then I think we're back on track!

@soumyaray soumyaray reopened this Nov 22, 2024
@soumyaray
Copy link
Contributor Author

@NicholasDanks I think you closed the PR without merging? Reopening it again but please let me know if you had another plan.

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