Skip to content

fix: Make sure strings can be parsed in the python interpreter#9908

Merged
erichare merged 4 commits into
release-1.6.0from
fix-python-interp
Sep 18, 2025
Merged

fix: Make sure strings can be parsed in the python interpreter#9908
erichare merged 4 commits into
release-1.6.0from
fix-python-interp

Conversation

@erichare
Copy link
Copy Markdown
Collaborator

This pull request updates the input handling for the PythonREPLComponent to better support multi-line Python code input. The main change is replacing the previous code input type with a more flexible multi-line input.

Input Handling Improvements:

  • Replaced CodeInput with MultilineInput for the python_code parameter in the PythonREPLComponent to support multi-line code execution.
  • Updated the import statement to use MultilineInput instead of CodeInput from langflow.io.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Sep 18, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-python-interp

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the bug Something isn't working label Sep 18, 2025
@github-actions github-actions Bot added the lgtm This PR has been approved by a maintainer label Sep 18, 2025
@github-actions github-actions Bot added bug Something isn't working and removed bug Something isn't working labels Sep 18, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 35.77%. Comparing base (75f34b3) to head (3a2f7f7).
⚠️ Report is 4 commits behind head on release-1.6.0.

❌ Your project status has failed because the head coverage (8.84%) is below the target coverage (10.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                @@
##           release-1.6.0    #9908      +/-   ##
=================================================
+ Coverage          35.76%   35.77%   +0.01%     
=================================================
  Files               1218     1218              
  Lines              58112    58112              
  Branches            5524     5524              
=================================================
+ Hits               20781    20787       +6     
+ Misses             37170    37164       -6     
  Partials             161      161              
Flag Coverage Δ
backend 55.66% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...langflow/components/processing/python_repl_core.py 27.27% <100.00%> (ø)

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions Bot added bug Something isn't working and removed bug Something isn't working labels Sep 18, 2025
@github-actions github-actions Bot added bug Something isn't working and removed bug Something isn't working labels Sep 18, 2025
@sonarqubecloud
Copy link
Copy Markdown

@erichare erichare merged commit 38c918f into release-1.6.0 Sep 18, 2025
71 of 72 checks passed
@erichare erichare deleted the fix-python-interp branch September 18, 2025 21:19
jordanrfrazier pushed a commit that referenced this pull request Sep 19, 2025
* fix: Execution of python interpreter

* Update test_python_repl_tool.py

* [autofix.ci] apply automated fixes

---------

Co-authored-by: Carlos Coelho <80289056+carlosrcoelho@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
github-merge-queue Bot pushed a commit that referenced this pull request Sep 19, 2025
* fix: Load flows autologin false (#9578)

* docs: update support documentation to reflect support rebranding (#9538)

* docs: update support documentation to reflect rebranding to IBM Elite Support for Langflow

* remove-info-tab

* Apply suggestions from code review

Co-authored-by: April I. Murphy <36110273+aimurphy@users.noreply.github.com>

---------

Co-authored-by: April I. Murphy <36110273+aimurphy@users.noreply.github.com>

* setup.py disable autologin check for loading flows

* [autofix.ci] apply automated fixes

* New base branch, updated files for release

---------

Co-authored-by: Mendon Kissling <59585235+mendonk@users.noreply.github.com>
Co-authored-by: April I. Murphy <36110273+aimurphy@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Jordan Frazier <jordan.frazier@datastax.com>

* fix: Move docling dependency into core dependencies instead of dev (#9906)

* fix: Make sure strings can be parsed in the python interpreter (#9908)

* fix: Execution of python interpreter

* Update test_python_repl_tool.py

* [autofix.ci] apply automated fixes

---------

Co-authored-by: Carlos Coelho <80289056+carlosrcoelho@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

* Remove docling dep from optional group since its in main dep group

* fix: Correct the docling dependencies and Chroma in particular (#9925)

fix: Correct docling dependencies

---------

Co-authored-by: Sebastián Estévez <estevezsebastian@gmail.com>
Co-authored-by: Mendon Kissling <59585235+mendonk@users.noreply.github.com>
Co-authored-by: April I. Murphy <36110273+aimurphy@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Eric Hare <ericrhare@gmail.com>
Co-authored-by: Carlos Coelho <80289056+carlosrcoelho@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants