Skip to content
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

Fix export_util.py to encode in the UTF-8 #554

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

antejavor
Copy link

@antejavor antejavor commented Mar 6, 2025

Description

The export_util can fail if special characters are not supported by default OS encoding. Specifying the version should help because UTF-8 should work universally on multiple systems.

Pull request type

  • Bugfix
  • Algorithm/Module
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Related issues

Closes: #553

######################################

Reviewer checklist (the reviewer checks this part)

Module/Algorithm

  • Core algorithm/module implementation
  • Query module implementation
  • Tests provided (unit / e2e)
  • Code documentation
  • README short description

Documentation checklist

  • Add the documentation label tag
  • Add the bug / feature label tag
  • Add the milestone for which this feature is intended
    • If not known, set for a later milestone
  • Write a release note, including added/changed clauses
    • **Set UTF-8 encoding in the export utility to support special characters **
  • Link the documentation PR here
    • [Documentation PR link]
  • Tag someone from docs team in the comments

Copy link

sonarqubecloud bot commented Mar 6, 2025

@antejavor antejavor self-assigned this Mar 6, 2025
@antejavor antejavor added bug bug Docs unnecessary Docs unnecessary labels Mar 6, 2025
@antejavor antejavor marked this pull request as ready for review March 6, 2025 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bug Docs unnecessary Docs unnecessary
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unicode error when exporting data with accents using csv_graph
1 participant