Manual type promotion for arcsine distribution#1373
Manual type promotion for arcsine distribution#1373JacobHass8 wants to merge 7 commits intoboostorg:developfrom
Conversation
|
@mborland what do you think of this version? I made a number of actual changes to the code that I'd like to get your opinion on. I left comments below. On an unrelated note, I wasn't able to get |
It is setup to run, and I just tried it on head and the test ran fine. What's the issue? You can try |
|
I do think this approach looks better. Thanks! |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #1373 +/- ##
===========================================
+ Coverage 95.34% 95.35% +0.01%
===========================================
Files 825 825
Lines 68160 68175 +15
===========================================
+ Hits 64986 65010 +24
+ Misses 3174 3165 -9
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
There were just a ton of errors in my code. I missed a couple of closing brackets which caused a large amount of errors. |
This tries to rewrite #1299 in a more human readable way. See #1294 for the same issue with the logistic distribution.