-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
Feature/first working version #1
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…rface/command_test.cpp
…interface/fixed_param_test.cpp
…c/test/interface/optimization_output_test.cpp
…ace/print_test.cpp
…p to src/test/interface/gm/argument_configuration_test.cpp
…rint_uninitialized_test.cpp and csv_header_consistency_test.cpp
2 tasks
yizhang-yiz
pushed a commit
to metrumresearchgroup/cmdstan
that referenced
this pull request
Apr 19, 2021
14e1050caf Merge branch 'master' into torsten-develop 7663b6a587 Merge pull request stan-dev#537 from rybern/loop-ix-fix 710f8f4a66 Changed the type of loop indices from size_t to int 7c20c088e2 Merge pull request stan-dev#493 from nhuurre/array-init-dummyvar aaefd626e5 Merge branch 'master' into array-init-dummyvar c0281e80b6 Merge pull request stan-dev#521 from nhuurre/bugfix/out-of-bounds-check 850cd41852 Merge branch 'master' into bugfix/out-of-bounds-check 22390c7927 Merge branch 'master' into array-init-dummyvar f556d0df24 Merge pull request #510 from stan-dev/bugfix/issue-497-offset-multiplier 034a7dd60b Real 0.0 to Int 0 and dune promote 71983b82d9 dune promote 02594422e7 Merge branch 'master' into bugfix/issue-497-offset-multiplier 3dcd7d8a84 more out-of-bounds checks 1e4c2fb9c4 Merge pull request stan-dev#520 from nhuurre/bugfix/param-decls-sized e895a8b213 array requires sized decl 97df934d4d Merge pull request stan-dev#515 from nhuurre/bugfix/foreach-slicing 6857b44d49 Merge pull request stan-dev#519 from nhuurre/reduce_sum-arg-swap bc519cd5df Update src/stan_math_backend/Statement_gen.ml db8c75dd63 Merge branch 'master' into array-init-dummyvar ecf2fae4bf Merge pull request stan-dev#518 from nhuurre/bugfix/transform-inits-constrained 004e730cc4 dune promote 9ff33e83c7 rename file ce78f28c2e _lpdf-compatible reduce_sum 7be86f60ca Merge pull request stan-dev#464 from rybern/keyword-var-warning ef7842d7c9 Merge master, which meant updating all the parser.message error numbers via menhir --update-error fdfebc529d fix transform_inits for constrained types 3397311bf9 Merge pull request stan-dev#512 from rybern/transform-ordering 1f0a49db4a evaluate the iteratee before foreach loop dbf144f259 Merge pull request stan-dev#508 from stan-dev/bugfix/issue-504-spurious-integer-warning 3e46293a3e dune promote b665b34804 Added test for <upper, lower> transformation a51ac010fc Fixed wording error in parser message ecfb812a51 Added reverse-ordered offset/multiplier and lower/upper to the parser, plus complete error message abd00b3fb0 change offset to 3 to make it easier to check tests ca4bdc0962 fix the offset multiplier bug 3aff87223c Merge pull request stan-dev#451 from bstatcomp/parallel_reduce 7a0e718cf5 consistent naming, is_reduce_sum_fn helper, clearer names, cleanup and format c87a179900 change warning message and dune runtest 71390194ea Merge branch 'master' into bugfix/issue-504-spurious-integer-warning 147eb99aae remove int,real, divide and change the error message 35ed016536 Merge pull request stan-dev#507 from stan-dev/bugfix/filename_startswith_number 400561bb75 add code to the test model f149bc06b2 dont use the entire path 7b29d45f9c check model name 8ec4f4a20e fix off-by-one error in the error message d3032f262c Merge pull request stan-dev#502 from rybern/include-format cc7b140d95 cleanup 315f7dbcba move to sets and make sure functor declarations have a trailing newline f377675d77 remove the use of string literals 9a0f0e72cd nhuures fix for find_functors f206b2cc6f add recursive call for all patterns in find_functors 5697210202 dune promote 50d3e36edd extend the test model so a function is only used in compound functions 9a058254cd change the error message 0076b2a78c dont allow user defined functions named reduce_sum or reduce_sum_static, fix checks 9b4c1bc932 format 33b61b0564 changed arg match check, changed test, simplified pattern to match c59bcdbdf4 cleanup matching, change allowed_slice_type name 9cf693b39c Added specifying include files with < > brackets in addition to quotes 611403361c update test models 6590fae234 format fd5120c841 add support for recursion and sliced types bf77a475dc add more thorough test models 77f742d66a use stan::math::fill 84e2722363 add dots to the error message 590e6f6d71 add error message for additiona arg mismatch 4af3ae4200 cleanup 7fd279c5a4 formatting 971a7a75e3 split the generic error message case 871696d532 handle return type separately eecc0e5a59 add tests for reduce_sum_static and a new way of printing errors 144210ab55 add reduce_sum_static 825e3f3607 Merge branch 'master' into parallel_reduce 6acb8290a6 dune promote 5a45b0b6d5 improve array initialization eaf1f960db Merge pull request stan-dev#492 from stan-dev/bugfix/add_using_std_pow 8acea361be dune promote 3d6c3e81ae add using std::pow b463824c5d Fix for Jenkinsfile test binaries 4fc4e46f17 Added checkout logic to stanc3 test binaries Jenkinsfile f2b16d3131 Merge branch 'master' into parallel_reduce 69586e602b semantic check (with the same error message) done, added tests for bad behavior 2bace2d0ba Merge pull request stan-dev#486 from stan-dev/jenkinsfile-test-binaries c8794bc27f Removed stash commands 0ad1ce30c7 Added missing archive artifacts 4ce1f06a94 Added Jenkinsfile used for building binaries on demand eef5a5ddda dune promote 80e150e551 test two deep use of reduce_sum 454eca0dac cleanup semantic_check function 928118a582 add missing slice type 97ed338099 cleanup 0b414a4eff fix the bug where functions only used inside functions were not labeled as used in reduce_sums d1c76101bc Merge branch 'master' into keyword-var-warning 21cd5cca56 Merge remote-tracking branch 'bstatcomp/master' into parallel_reduce 6d86b35696 Added exhaustive list of keyword tokens to keyword parser errors e166c5b7ad skip semantic check on reduce_sum fdcf2c5b52 more cleanup d795755a13 format 2874578f66 remove the code that used Any ff8b3732da Merge branch 'master' into parallel_reduce 7e5b369f94 expand test 8f9f3d7f1d added for up to 8 arguments 34134ae872 try adding Any type f49d7100a7 proper handling of creating rsfunctors for funs used in reduce_sum f712de6f51 move back to only 3 arguments to speedup dune runtest 4ef08cf9a3 support up to 5 arguments 5312a599b1 Merge pull request stan-dev#1 from rybern/parallel_reduce 8997183183 Refactored parallel reduce code for scalability cc3b774962 only add the rsfunctor if first two arguments are ints 77a9f2e421 Revert "added reduce sum semantic error" 6405cd37ea Merge remote-tracking branch 'origin/master' into parallel_reduce fa375483b4 added reduce sum semantic error b27fcc8bc7 concat with ^ 949a6960ac remove unused expected files ba4e0eec05 remove hpp files 1d3f47f8ba try to squash the indexing bug 5b124be85b dune promote e9fec03bc8 remove duplicated signature, nicer code 28ec23a484 dune promote 83da2e3992 changed to rsfunctor b0f18713d0 fix unwanted change in order 110c0c69fa remove commented out code 9ae8d04253 comment f3edf9ccc4 slighly nicer creation of duplicate functors 549f72aa5f dune promote c98ff7d68b fix the original functor args order d406f582fa test 0 to 3 real args 516d7f23d0 format code ced01706a0 after cleanup 977b695ece cleanup d842ca93a5 brute force version one done ad67e35b7e promote test 975fad6b9c parallel_reduce testing 4c7064d8ca basic version b18bd88dc1 promote 3d2c15f075 added normal a45c715a1c fixed order of arguments 818349d99a first silly prototype 55bf838d3d hierachiral reduce git-subtree-dir: stanc3 git-subtree-split: 14e1050caf538d0047171b854849b0c011b15e10
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
This is a working version of CmdStan. It currently relies on Stan's feature/issue-405-refactor-command.
This should be merged at the same time feature/issue-405-refactor-command is merged.