Merging new main code into integration #3
Merged
Conversation
This file contains hidden or 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
* Impl function expander * Fix error message * fix flake8 errors * file doc * Suggestions * Fix docstring * Fix docstring * rename test methods * slight rephrasing * rm local var * Reformat import * suggestions Co-authored-by: Zibing Zhang <[email protected]>
Co-authored-by: Zibing Zhang <[email protected]>
* [add] configuration * [fix] variable name error * Apply suggestions from code review Co-authored-by: Yusuke Oda <[email protected]> * [doc] method docstring * [fix] fplake8 E501 line too long * Apply suggestions from code review Co-authored-by: Yusuke Oda <[email protected]> * [fix] documentation * [fix] direct import * lint Co-authored-by: Yusuke Oda <[email protected]> Co-authored-by: Yusuke Oda <[email protected]>
…#131) * keyword ast args * value -> n Co-authored-by: Zibing Zhang <[email protected]>
* docs(README.md): add nice badges to doc * docs(README.md): add python version badge * Update README.md Co-authored-by: Yusuke Oda <[email protected]> Co-authored-by: Yusuke Oda <[email protected]>
* all single arg fns in math * two argument fns * nested testing * tests * shorten line * suggestions * import * recursive case already handled Co-authored-by: Zibing Zhang <[email protected]>
* prefix trimmer * add test * add frontend flags
* fix * remove use_raw_function_name, apply IdentifierConverter to all identifiers * docstring * remove final
* Add basic matrix support * Fix generation function * Fix formatting and minor issues * Fix tests * refactor * fix Co-authored-by: Kshitij Sharma <[email protected]>
* Add basic matrix support * Fix generation function * Fix formatting and minor issues * Fix tests * refactor * fix * add ast_utils.parse_expr Co-authored-by: Kshitij Sharma <[email protected]>
* Add basic matrix support * Fix generation function * Fix formatting and minor issues * Fix tests * refactor * fix * add ast_utils.parse_expr * refactor Co-authored-by: Kshitij Sharma <[email protected]>
* refactoring * add test * fix 3.7 bugs
* revert BuiltinFnName * fix
* add mypy * revert BuiltinFnName * fix * fix * fix * fix
Co-authored-by: Zibing Zhang <[email protected]>
* modified files * minor style fix * refactoring Co-authored-by: hjiawei <[email protected]> Co-authored-by: odashi <[email protected]>
* wrote a basic test that calls visit_Match * added extra space to publish branch * implementation for MatchValue * merged match_case into Match (with juliawgraham) * 1. removed unused i; 2. added error handling for multiple statements in Match node; 3. added corresponding unit tests Co-authored-by: Lucybean-hi <[email protected]> Co-authored-by: juliawgraham <[email protected]> Co-authored-by: Erica Fu <[email protected]> * Removed unused _reduce_stop_parameter function Co-authored-by: Yuqi <[email protected]> Co-authored-by: Erica Fu <[email protected]> * updated unit test syntax * updated formatting using flake * updated unit test formatting after flake changes * reformatted files using black * refactoring Co-authored-by: juliawgraham <[email protected]> Co-authored-by: Yuqi Gong <[email protected]> Co-authored-by: Lucybean-hi <[email protected]> Co-authored-by: juliawgraham <[email protected]> Co-authored-by: Erica Fu <[email protected]> Co-authored-by: juliawgraham <[email protected]> Co-authored-by: Yuqi <[email protected]> Co-authored-by: odashi <[email protected]>
* wrote a basic test that calls visit_Match * added extra space to publish branch * implementation for MatchValue * partial implementation for MatchAs, only implemented for the wildcard case * changed test names for matchas tests * 1. removed unused i; 2. added error handling for multiple statements in Match node; 3. added corresponding unit tests Co-authored-by: Lucybean-hi <[email protected]> Co-authored-by: juliawgraham <[email protected]> Co-authored-by: Erica Fu <[email protected]> * updated syntax according to flake * updated syntax so both flake and unit tests pass * ran black reformatter * require python 3.10 or higher for all match tests * fix * refactoring * refactoring * refactoring Co-authored-by: juliawgraham <[email protected]> Co-authored-by: Yuqi Gong <[email protected]> Co-authored-by: Lucybean-hi <[email protected]> Co-authored-by: juliawgraham <[email protected]> Co-authored-by: Erica Fu <[email protected]> Co-authored-by: Yuqi <[email protected]> Co-authored-by: odashi <[email protected]>
* please work * moved back * same trick * move back * factor expr codegen out * add visit_, remove expr tests from func * suggestiosn * codegen test * using generic visit * extract method * only stmt codegen * reduce stop param * fix compatability issues * abc * Visit an XYZ node. * standard test method names * suggestions Co-authored-by: Zibing Zhang <[email protected]>
* modified files * minor style fix * refactoring Co-authored-by: hjiawei <[email protected]> Co-authored-by: odashi <[email protected]> Co-authored-by: Jiawei Hao <[email protected]> Co-authored-by: Zibing Zhang <[email protected]>
* init alg * comments and annotations * fix indentation * tests, procedure * forgot one * expose Style, tests * bug * rm visit_Expr from expr_codegen * rm line * inline some code * specify codegen * too long * suggestions * rm todo Co-authored-by: Zibing Zhang <[email protected]>
* move * move * move * move * fix * add test * move * fix * tree -> node * fix
Merging in new commits made one latexify main branch
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.
Updating integration branch with main changes