Skip to content

Refactor ASTx to Python Transpiler: Combine Node Handlers #282

Closed
nagajaideep wants to merge 22 commits intoarxlang:mainfrom
nagajaideep:main
Closed

Refactor ASTx to Python Transpiler: Combine Node Handlers #282
nagajaideep wants to merge 22 commits intoarxlang:mainfrom
nagajaideep:main

Conversation

@nagajaideep
Copy link
Contributor

Pull Request description

This PR resolves several failing tests in the ASTx Python transpiler and improves literal node handling.

Key changes:
-unified all redundant nodes(literalint,int etc)

  • Updated test assertions to match the new literal node handler outputs.
  • Added test_new_implementation.py to verify the new AST-based implementation.

How to test these changes

  • Run pytest in the libs/astx-transpilers/tests directory to ensure all tests pass.

Pull Request checklists

Note:

This PR is a:

  • bug-fix
  • new feature
  • maintenance

About this PR:

  • it includes tests.
  • the tests are executed on CI.
  • the tests generate log file(s) (path).
  • pre-commit hooks were executed locally.
  • this PR requires a project documentation update.

Author's checklist:

  • I have reviewed the changes and it contains no misspelling.
  • The code is well commented, especially in the parts that contain more
    complexity.
  • New and old tests passed locally.

Additional information

No screenshots are required as this PR focuses on backend logic and test fixes.


Reviewer's Checklist

  • I managed to reproduce the problem locally from the main branch
  • I managed to test the new changes locally
  • I confirm that the issues mentioned were

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@nagajaideep nagajaideep closed this by deleting the head repository Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants