-
Notifications
You must be signed in to change notification settings - Fork 2
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
Merge ast and parser from forward #3
Merged
Merged
Changes from all commits
Commits
Show all changes
68 commits
Select commit
Hold shift + click to select a range
53844ed
Update poetry configuration
Legotier e6a9c16
Add testreport to gitignore
Legotier 0e4ffaa
Merge new stuff from forward into ast.py
Legotier 7fbc8bd
Minimal changes to make the tests pass
Legotier 4fc36c1
Rename float types to real types
Legotier c296998
Update docs to include new classes
Legotier 9d167e3
Use autodata instead of autoclass for union types
Legotier 7adbe0f
changed workflow to python version 3.9
LKlinke 7021d76
Enable type aliases for union types in the docs
Legotier 816dc31
Merge branch 'brand-new' of https://github.com/Philipp15b/probably in…
Legotier 697e1d1
Remove forward type references in ast.py
Legotier b2ede13
resolved dep issues
1d5af7a
Disallow edges to None in ControlFlowGraph.to_networkx()
Legotier 6eb0df9
Merge parser from forward
Legotier b8f7cf3
A helpful comment so people don't have to spend hours figuring out wh…
Legotier 15f59e7
Update test_uniform_checks
Legotier 9012370
Fix precedence of likelyhood operator
Legotier 077a5eb
Add test for basic operator precedence
Legotier f8b3e08
Revert "Add test for basic operator precedence"
Legotier 9cc3a75
Add test for basic operator precedence
Legotier 8833b7f
Test for correct parsing of distributions
Legotier 3835e4d
Test for using minus with likely
Legotier 2afe578
Only allow natural numbers and parameters in distribution expressions
Legotier 7912125
Revert "Only allow natural numbers and parameters in distribution exp…
Legotier ce569fb
Minor fixes
Legotier c179a25
Revert to older version (2.5) of networkx
Legotier 5de84e6
Remove test for parser check that is no longer done
Legotier a3a3246
Clear parameter dict for every new program that is parsed
Legotier fe6feab
Fix almost all pylint problems
Legotier 017ad52
Split ast module into several files
Legotier 1bfd89c
I forgot to add the new files to git
Legotier 9fd0257
Move documentation of ast module to __init__.py
Legotier 8557c91
Fix pylint cyclic import warnings
Legotier e300fd3
Fix mypy errors
Legotier b22519b
Update mypy to newer version
Legotier 9189730
Move Bounds class to types file
Legotier 6ec6e8a
Move unused import to doctest
Legotier 5bdf111
Merge typechecker from forward
Legotier 3e25603
Add tests for some of the typechecking
Legotier 965da72
Revert "Remove test for parser check that is no longer done"
Legotier 8404efa
Fix bug in typechecking test
Legotier 02dc6dd
Remove 'parameters' parameter from Program.from_parse
Legotier 9943cb9
Remove VarExpr.is_parameter
Legotier a1f93d9
Fix unsafe return type of get_type for iid expressions
Legotier 8cc2a56
Prettify linear checks
Legotier 2aa93a3
Fix unsafe default value warning
Legotier f96efa4
Add some old checks from the parser to the typechecker
Legotier ecd08e9
Fix precedence of modulo operator
Legotier 476940a
Fix (suppressed) cyclic dependency warnings
Legotier 16e31c4
Fix typechecking for IidSampleExpr
Legotier 5ab560c
Give some binary operators clearer names
Legotier 0da4e94
Replace not_a_variable and add a warning to the typechecker
Legotier 6158b37
Make it more explicit that iid type checking is unsafe
Legotier 7e8c180
Logger warning for typechecking of iid exprs
Legotier 6247646
Update parameter in syntax module
Legotier bb071e1
Fix warnings
Legotier 579fdf6
Disallow constants in programs in the syntax module
Legotier 0a598c2
Make some variable names (true, false) illegal
Legotier 12167a5
Eliminate true and false from pgcl grammar
Legotier e606806
Revert "Eliminate true and false from pgcl grammar"
Legotier 4d4c322
Fix error message in check_is_linear_expr
Legotier 21ae331
Add assertion for illegal keyword to constructor of VarExpr
Legotier f1d923a
Better error message for usage of true or false in plot instructions
Legotier 79991b9
Allow variables in distribution parameters
Legotier 1777e88
Remove program config
Legotier 3af0bf5
Add typecheck for TickInstr and TickExpr
Legotier 2dd1aa3
Auto format
Legotier a2adf9f
Add format target to makefile
Legotier File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -49,6 +49,7 @@ coverage.xml | |
*.cover | ||
.hypothesis/ | ||
.pytest_cache/ | ||
testreport.xml | ||
|
||
# Translations | ||
*.mo | ||
|
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isort
is not a dev-dependency, does this actually work?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It runs on my machine... Maybe some other package depends on
isort
? In any case, if this should stay here then we should probably add it to the dependencies