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

csv export to use right locale. use comma instead of point in decimals #2072

Closed
mihkal opened this issue Jan 19, 2019 · 7 comments
Closed

csv export to use right locale. use comma instead of point in decimals #2072

mihkal opened this issue Jan 19, 2019 · 7 comments
Assignees
Labels
Export Fix Committed Wishlist New features (not a bug)
Milestone

Comments

@mihkal
Copy link

mihkal commented Jan 19, 2019

Expected Behavior

when exporting csv file gephi does not respect locale for decimal numbers
here it is , instead of .

Current Behavior

image

Possible Solution

Steps to Reproduce

Context

Your Environment

  • Version used: Gephi 0.9.2
  • Java version:
  • Operating System:
@eduramiba
Copy link
Member

Hi,
As this CSV data is later to be parsed and auto-recognized as a number by Gephi in any computer in any locale, we use the kind-of-standard dot for decimal separator.

Anyway I think localizing CSV is a bad idea as it makes it not portable, which is not good since it is not normally written by humans with a text editor, but instead read and written by programs (like GEXF is).

@eduramiba eduramiba self-assigned this Jan 19, 2019
@mihkal
Copy link
Author

mihkal commented Jan 19, 2019

then add an option to export with right delimiter. it is really a lot work to make all decimals work in office applications like excel. this approch is NOT standard.

@eduramiba
Copy link
Member

Right, we can add an option to override default settings.

@eduramiba eduramiba added Wishlist New features (not a bug) and removed Won't Fix labels Jan 19, 2019
@eduramiba eduramiba reopened this Jan 19, 2019
@mihkal
Copy link
Author

mihkal commented Jan 19, 2019

excellent 👍

@mihkal
Copy link
Author

mihkal commented Sep 23, 2019

Hello. Doing anything to this?

@eduramiba eduramiba added this to the 0.9.3 milestone Sep 23, 2019
@eduramiba
Copy link
Member

Adding to 0.9.3 milestone

@wetneb wetneb added the Export label Dec 2, 2021
@mbastian mbastian modified the milestones: 0.9.3, 0.9.4 Mar 1, 2022
@mbastian mbastian modified the milestones: 0.9.4, 0.10.0 Apr 11, 2022
@mbastian mbastian assigned mbastian and unassigned eduramiba Sep 11, 2022
@mbastian
Copy link
Member

Thanks @mihkal! I added an option to CSV export to configure the decimal separator. In the long run, we could maybe have a global option to configure the Locale we use for all the numbers - also for import. For now, we prefer to stick with a single Locale (Locale.ENGLISH) instead of taking the system default locale as explicit is better in my opinion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Export Fix Committed Wishlist New features (not a bug)
Projects
None yet
Development

No branches or pull requests

4 participants