-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Fortran Parser using Codegen AST #17356
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
Conversation
* Added Fortran parser using lfortran * Created fortran parsing module * set up import_module for lfortran
* Modified parser to use codegen AST for output * Added sym_expr module * Added documentation refence for the module * Set up Travis for the parser
* Added documentation and examples for all the modules * Added tests for the parser and tested on Travis * Set up Travis and fix any build errors
|
✅ Hi, I am the SymPy bot (v147). I'm here to help you write a release notes entry. Please read the guide on how to write release notes. Your release notes are in good order. Here is what the release notes will look like:
This will be added to https://github.com/sympy/sympy/wiki/Release-Notes-for-1.5. Note: This comment will be updated with the latest check if you edit the pull request. You need to reload the page to see it. Click here to see the pull request description that was parsed.
Update The release notes on the wiki have been updated. |
|
Thanks @Sc0rpi0n101. If tests pass, then I think we can merge it, so that it's in, and we can improve upon it later. |
|
@certik Looks like all the tests passed. |
|
Thanks. I am going to merge this. We can now build upon this. Thanks for your work on this PR! |
Codecov Report
@@ Coverage Diff @@
## master #17356 +/- ##
=============================================
+ Coverage 66.257% 74.533% +8.275%
=============================================
Files 628 631 +3
Lines 162911 163076 +165
Branches 38231 38271 +40
=============================================
+ Hits 107941 121546 +13605
+ Misses 49623 36137 -13486
- Partials 5347 5393 +46 |
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #17356 +/- ##
=============================================
+ Coverage 66.257% 74.533% +8.275%
=============================================
Files 628 631 +3
Lines 162911 163076 +165
Branches 38231 38271 +40
=============================================
+ Hits 107941 121546 +13605
+ Misses 49623 36137 -13486
- Partials 5347 5393 +46 |
References to other Issues or PRs
Related Issue: #16563
Rebased version of #17171
Brief description of what is fixed or changed
Fortran parser for sympy using the codegen AST nodes
The parser can currently only parse variable and function definitions, assignments and basic binary operations for now due to the limitations in the lfortran ASR and codegen AST.
Other comments
Release Notes