Conversation
|
failing because of stanio? |
|
where are you seeing failures from stanio? |
Codecov Report
@@ Coverage Diff @@
## develop #707 +/- ##
===========================================
- Coverage 79.72% 79.60% -0.13%
===========================================
Files 25 50 +25
Lines 3793 7590 +3797
===========================================
+ Hits 3024 6042 +3018
- Misses 769 1548 +779 see 33 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
not a problem - I was concerned it might be because running mypy locally produces these errors: maybe my local setup is defective somehow? |
|
Windows tests were broken by this release of the Github Actions OS images. They bumped Mingw from 11.2 to 12.2, so we need similar flags set as RTools 43. I have a temporary fix in #691 but it probably deserves a closer look |
|
regarding RTools, |
|
We should probably do that at some point, I'm hoping we can figure out a way that doesn't require this extra config first (stan-dev/math#2948) |
WardBrian
left a comment
There was a problem hiding this comment.
Thanks for this, a lot of much-needed cleanups
|
ready for re-review. |
|
We know the failures aren't from this PR so I think it's all safe to merge |

Submission Checklist
Summary
Added references to Pathfinder throughout docs and docstrings in code.
(Note: Sphinx documentation builds, but page
api.rsttriggers complaint about bad anonymous reference. cannot track down the problem, but the documentation looks good on my machine. discussion on Sphinx issue tracker says basically "too much preprocessing going on". sphinx-doc/sphinx#6036 (comment))Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses: