Skip to content
This repository has been archived by the owner on Feb 22, 2024. It is now read-only.

Define a class for the for major objects and clarify this a bit in mans and vignette #26

Open
SinomeM opened this issue Aug 28, 2020 · 3 comments
Assignees

Comments

@SinomeM
Copy link
Owner

SinomeM commented Aug 28, 2020

No description provided.

@SinomeM SinomeM added this to the Bioconductor Submission milestone Aug 28, 2020
@SinomeM SinomeM self-assigned this Aug 28, 2020
@SinomeM
Copy link
Owner Author

SinomeM commented Sep 2, 2020

Also check that the major functions return objects of the correct classes

@SinomeM
Copy link
Owner Author

SinomeM commented Sep 28, 2020

there is also this R CMD check WARNING

* checking S3 generic/method consistency ... WARNING
summary:
  function(object, ...)
summary.CNVresults:
  function(object, sample_list, markers, plots_path)

See section ‘Generic functions and methods’ in the ‘Writing R
Extensions’ manual.

@SinomeM
Copy link
Owner Author

SinomeM commented Sep 30, 2020

Best thing is to define is.XXX and as.XXX functions

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

1 participant