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

simulation failing with upcoming release #894

Closed
jashapiro opened this issue Nov 19, 2024 · 1 comment · Fixed by #898
Closed

simulation failing with upcoming release #894

jashapiro opened this issue Nov 19, 2024 · 1 comment · Fixed by #898
Assignees

Comments

@jashapiro
Copy link
Member

When preparing for the 2024-11-25 data release, the workflow for simulating data failed. I suspect this had to do with genes that now have all zero values, but I will have to investigate and update the workflow to handle this error.

The error message is listed below.

Command executed:

  mkdir SCPCS000257
  simulate-sce.R       --sample_dir input       --metadata_file single_cell_metadata.tsv       --output_dir SCPCS000257

Command exit status:
  1

Command output:
  Error in computing default starting values.

Command error:
  Error in `map()`:
  ℹ In index: 3.
  Caused by error in `manageparam()`:
  ! Error in if (any(x < 0)) stop("values must be positive to fit an gamma  distribution") : 
    missing value where TRUE/FALSE needed
  Backtrace:
       ▆
    1. ├─purrr::walk(...)
    2. │ └─purrr::map(.x, .f, ..., .progress = .progress)
    3. │   └─purrr:::map_("list", .x, .f, ..., .progress = .progress)
    4. │     ├─purrr:::with_indexed_errors(...)
    5. │     │ └─base::withCallingHandlers(...)
    6. │     ├─purrr:::call_with_cleanup(...)
    7. │     └─global .f(.x[[i]], ...)
    8. │       └─global simulate_sce(real_sce, opts$ncells, replacement_metadata, processed = is_processed)
    9. │         ├─splatter::simpleEstimate(as.matrix(counts(sce_sim)))
   10. │         └─splatter:::simpleEstimate.matrix(as.matrix(counts(sce_sim))) at splatter/R/simple-estimate.R:28:5
   11. │           └─fitdistrplus::fitdist(means, "gamma", method = "mme") at splatter/R/simple-estimate.R:53:5
   12. │             └─fitdistrplus:::manageparam(...)
   13. │               └─base::stop(trystart)
   14. └─base::.handleSimpleError(...)
   15.   └─purrr (local) h(simpleError(msg, call))
   16.     └─cli::cli_abort(...)
   17.       └─rlang::abort(...)
\
@jashapiro
Copy link
Member Author

closed by #898

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 a pull request may close this issue.

1 participant