stan-dev / stan-dev/cmdstan

use second evaluate of log density for time estimate; remove multiplier

Open
#675 0 comments 0 reactions 1 assignee View on GitHub

@bob-carpenter is already working on this.

Since Feb 26, 2019.

Feature user experience
Dominant language
C++
Stars
238
Forks
106
Avg merge
2h 29m
Merged PRs (30d)
6

Description

Summary:

If we use the first log density evaluation, it's going to include things like transfer costs of data for MPI. We'll get a more accurate measurement for running the program by taking a second evaluation.

Also, we want to get rid of the overall estimate and just go with the single evaluation time.

Current Version:

v2.18.1

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.