Skip to content

Conversation

@devatcadam
Copy link
Contributor

@devatcadam devatcadam commented Oct 20, 2025

Description

Note

Please provide a description of the work completed in this PR below

Resolve error on Windows with pipeline.draw("pipeline.html") and charmap lib :
File "e:\Codes\PS-CEA-kg-building.venv\Lib\site-packages\neo4j_graphrag\experimental\pipeline\pipeline.py", line 207, in draw
f.write(G.render().data)
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.12_3.12.2800.0_x64__qbz5n2kfra8p0\Lib\encodings\cp1252.py", line 19, in encode
return codecs.charmap_encode(input,self.errors,encoding_table)[0]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
UnicodeEncodeError: 'charmap' codec can't encode character '\u0100' in position 34971: character maps to

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Documentation update
  • Project configuration change

Complexity

Note

Please provide an estimated complexity of this PR of either Low, Medium or High

Complexity: Low

How Has This Been Tested?

  • Unit tests
  • E2E tests
  • Manual tests

Checklist

The following requirements should have been met (depending on the changes in the branch):

  • Documentation has been updated
  • Unit tests have been updated
  • E2E tests have been updated
  • Examples have been updated
  • New files have copyright header
  • CLA (https://neo4j.com/developer/cla/) has been signed
  • CHANGELOG.md updated if appropriate

@devatcadam devatcadam requested a review from a team as a code owner October 20, 2025 09:48
@stellasia
Copy link
Contributor

Hey @devatcadam , we've pushed a fix to main for the E2E test CI, if you rebase your branch they should now run without disk space error.

Copy link
Contributor

@stellasia stellasia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just realized I can update the branch myself, so ignore my previous comment. Thank you for contributing this fix! PR will be merged as soon as CI passes.

@stellasia stellasia merged commit c4a21ae into neo4j:main Dec 9, 2025
9 checks passed
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.

2 participants