Skip to content

Feature/issue 197 remove writer#213

Merged
bgoodri merged 16 commits into
developfrom
feature/issue-197-remove-writer
Dec 2, 2015
Merged

Feature/issue 197 remove writer#213
bgoodri merged 16 commits into
developfrom
feature/issue-197-remove-writer

Conversation

@syclik
Copy link
Copy Markdown
Member

@syclik syclik commented Sep 21, 2015

Summary:

This updates stan_fit.hpp to work with the latest develop branch of Stan.

Intended Effect:

With stan-dev/stan#1601, some of the header files were removed. These were moved to RStan.

How to Verify:

This should compile and work exactly like the version without it.

Side Effects:

@bgoodri, can you double check the changes to stan_fit.hpp? It looks like the merge brought back in the section for advi... If you want that removed, just say so and I'll remove the sections.

Documentation:

None required.

Reviewer Suggestions:

@bgoodri

@syclik
Copy link
Copy Markdown
Member Author

syclik commented Sep 24, 2015

@bgoodri, just FYI, the failure was a travis-ci thing. It timed out trying to get R.

This is to get RStan up to develop. I'd wait on merging until after 2.8.0-1 or whatever you're working on for release now.

@syclik syclik added this to the 2.8.0++ milestone Sep 24, 2015
@ariddell
Copy link
Copy Markdown

Is there some way to track these refactoring tasks centrally? Should there be a meta-task in the stan repo? It also occurred to me that having a stan-3-stage-1 milestone or something like that would be useful.

@syclik
Copy link
Copy Markdown
Member Author

syclik commented Sep 26, 2015

There is a meta-task in the repo. This fix is post 2.8.0, but not Stan 3.
Once you release 2.8.0, I have a pull request ready to go.

(When releasing 2.8.0, if you use the v2.8.0 tagged version of the Stan
library, you won't need this patch.)

On Sat, Sep 26, 2015 at 10:03 AM, Allen Riddell notifications@github.com
wrote:

Is there some way to track these refactoring tasks centrally? Should there
be a meta-task in the stan repo? It also occurred to me that having a
stan-3-stage-1 milestone or something like that would be useful.


Reply to this email directly or view it on GitHub
#213 (comment).

@ariddell
Copy link
Copy Markdown

ariddell commented Oct 2, 2015

PyStan 2.8.0.0 is out. Just waiting for the uploads of these monster OS
X wheels to make the announcement.

On 09/26, Daniel Lee wrote:

There is a meta-task in the repo. This fix is post 2.8.0, but not Stan 3.
Once you release 2.8.0, I have a pull request ready to go.

(When releasing 2.8.0, if you use the v2.8.0 tagged version of the Stan
library, you won't need this patch.)

On Sat, Sep 26, 2015 at 10:03 AM, Allen Riddell notifications@github.com
wrote:

Is there some way to track these refactoring tasks centrally? Should there
be a meta-task in the stan repo? It also occurred to me that having a
stan-3-stage-1 milestone or something like that would be useful.


Reply to this email directly or view it on GitHub
#213 (comment).


Reply to this email directly or view it on GitHub:
#213 (comment)

@bgoodri
Copy link
Copy Markdown
Contributor

bgoodri commented Nov 30, 2015

Okay to merge this? Anything else needed for 2.9?

@syclik syclik force-pushed the feature/issue-197-remove-writer branch from 0e03966 to 4d4a9df Compare December 1, 2015 18:55
bgoodri added a commit that referenced this pull request Dec 2, 2015
@bgoodri bgoodri merged commit 0b73888 into develop Dec 2, 2015
@maverickg
Copy link
Copy Markdown
Contributor

Some error messages cannot find their ways to the output; some units tests failed here: https://travis-ci.org/stan-dev/rstan/jobs/94535542

@maverickg
Copy link
Copy Markdown
Contributor

I think I fixed the error message issue in the unit tests in develop branch.

@syclik
Copy link
Copy Markdown
Member Author

syclik commented Dec 3, 2015

Hi Jiqiang, does that one fix actually work? I tried the same thing and it
didn't pass. If so, then great! (and sorry for not getting it right the
first time.)

On Thu, Dec 3, 2015 at 2:26 PM, maverickg notifications@github.com wrote:

I think I fixed the error message issue in the unit tests in develop
branch.


Reply to this email directly or view it on GitHub
#213 (comment).

@maverickg
Copy link
Copy Markdown
Contributor

@syclik It works.

@syclik syclik deleted the feature/issue-197-remove-writer branch August 29, 2016 14:10
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.

4 participants