Skip to content

Simplify debugging for potential parser issue #115

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

Closed
rok-cesnovar opened this issue Jan 27, 2020 · 5 comments
Closed

Simplify debugging for potential parser issue #115

rok-cesnovar opened this issue Jan 27, 2020 · 5 comments
Labels
feature New feature or request

Comments

@rok-cesnovar
Copy link
Member

rok-cesnovar commented Jan 27, 2020

Cmdstan 2.22 uses the new parser (stanc3) and we should add a simple interface to switch to the older parser for debugging purposes.

That would simplify debugging issues like stan-dev/math#1643 that turned out to not be related to the parser.

compile() should accept something like (use_stanc2 = TRUE), that would be turned off by default. This should only be allowed for 2.22 and up until he version where the existing compiler will be phased out.

@mitzimorris
Copy link
Member

great idea - filed stan-dev/cmdstanpy#202 - can argument be simply stanc2 instead of use_stanc2?

@rok-cesnovar
Copy link
Member Author

Yeah, definitely. Also think this should print a warning that the flag is on and should only be used to debug.

@mitzimorris
Copy link
Member

this isn't feasible as a flag on the compile function given current CmdStan makefiles - right now CmdStan assumes that there is only one compiler - bin/stanc - and it builds it either from Stanc2 sources or uses the release or nightly build binary.
putting this flag on the compile function assumes that both compilers are available. will file an issue on CmdStan to make this happen.

@mitzimorris
Copy link
Member

stan-dev/cmdstan#819

@rok-cesnovar
Copy link
Member Author

I am going to close this one for two reasons:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants