G-Node / G-Node/nixpy

Extend 'units.sanitizer' function to replace omega character

Open
#384 0 comments 0 reactions 0 assignees View on GitHub
discussion
Dominant language
Python
Stars
22
Forks
29
PR merge metrics
No merged PRs in 30d

Description

The current implementation of the `units.sanitizer` function includes the 'micro' and 'mu' characters and replaces them with the 'u' character.

Since the large omega (Ω) is the symbol of the derived SI unit ohm, it might make sense to include it in the `sanitizer` and replace it with `Ohm`.

If not escaped, this breaks an import from a UTF-8 odML file containing the Ω character in a `Property.unit` to NIX using Python 2 with a `UnicodeDecodeError`.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the `units.sanitizer` implementation and review how it handles the existing `micro` and `mu` characters. Verify that an odML import with Ω in `Property.unit` no longer raises a Python 2 `UnicodeDecodeError`, and that the character is converted to `Ohm`.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.