The Alembic don't support Chinese path #49112

New Issue

Leroy · 2016-08-19T05:08:22+02:00

Leroy commented

2016-08-19 05:08:22 +02:00

System Information
windows 7 64bit sp1

Blender Version
Broken: 2.77.3 1fe9d67
Worked: (optional)

The Alembic don't support Chinese path, Whether import or export.
test.zip

Exact steps for others to reproduce the error
Download the test file, it's have contain one folder and named with Chinese, whether import or export, all not work, export not report error, but no file in the folder. Even check the International Fonts in the User Preferences, still not work. this not a big problem, just want the developer know it, thanks :D

**System Information** windows 7 64bit sp1 **Blender Version** Broken: 2.77.3 1fe9d67 Worked: (optional) The Alembic don't support Chinese path, Whether import or export. [test.zip](https://archive.blender.org/developer/F338940/test.zip) **Exact steps for others to reproduce the error** Download the test file, it's have contain one folder and named with Chinese, whether import or export, all not work, export not report error, but no file in the folder. Even check the International Fonts in the User Preferences, still not work. this not a big problem, just want the developer know it, thanks :D

Leroy commented

2016-08-19 05:08:22 +02:00

Changed status to: 'Open'

Leroy commented

2016-08-19 05:08:22 +02:00

Added subscriber: @Leroy-Xie

Kévin Dietrich commented

2016-08-19 07:04:37 +02:00

Added subscribers: @kjym3, @kevindietrich

Kévin Dietrich commented

2016-08-19 07:04:37 +02:00

I can't reproduce this here on Linux (both import and export work fine), maybe this a Windows path issue (or character encoding shenanigan)?

@kjym3 can you try to reproduce this?

I can't reproduce this here on Linux (both import and export work fine), maybe this a Windows path issue (or character encoding shenanigan)? @kjym3 can you try to reproduce this?

Brecht Van Lommel commented

2016-08-19 10:09:53 +02:00

Added subscriber: @brecht

Brecht Van Lommel commented

2016-08-19 10:09:53 +02:00

For unicode paths to work on Windows you need to use specific functions to open files, like _wfopen with wchar_t or std::ifstream with std::wstring. I don't see anything like that happening in the Alembic library itself or the Blender code using it.

If we could pass a file stream rather than a filename to Alembic that would make it possible to solve this on the Blender side, but I don't see a way to do that (not sure it's impossible though, not really familiar with the Alembic API).

It may well be that the Alembic library needs to be fixed and that other applications have the same limitation.

For unicode paths to work on Windows you need to use specific functions to open files, like `_wfopen` with `wchar_t` or `std::ifstream` with `std::wstring`. I don't see anything like that happening in the Alembic library itself or the Blender code using it. If we could pass a file stream rather than a filename to Alembic that would make it possible to solve this on the Blender side, but I don't see a way to do that (not sure it's impossible though, not really familiar with the Alembic API). It may well be that the Alembic library needs to be fixed and that other applications have the same limitation.

Bastien Montagne commented

2016-08-19 12:46:19 +02:00

Added subscriber: @mont29

Bastien Montagne commented

2016-08-19 12:46:19 +02:00

@kevindietrich please double check whether we can do that or not with current alembic, if not possible best is to close this report and 'forward' it to alembic devs I guess?

Kévin Dietrich commented

2016-08-20 03:05:54 +02:00

I thought using wstring & co was discouraged on Windows (or is it just the UTF-8 guys saying so?). Anyway, it should be doable to use streams instead of passing the file name.

Brecht Van Lommel commented

2016-08-20 03:43:32 +02:00

I'm not sure why it would be discouraged. We use UTF-8 as much as possible in Blender, but at some point we need to convert to UTF-16, as far as I know there is no way around that on Windows.

Kévin Dietrich commented

2016-08-20 04:40:31 +02:00

I uploaded a patch to make use of streams (D2160), maybe people using Windows around it can test/review it before committing to master?

I uploaded a patch to make use of streams ([D2160](https://archive.blender.org/developer/D2160)), maybe people using Windows around it can test/review it before committing to master?

blender-admin commented

2016-08-25 05:19:41 +02:00

This issue was referenced by 3c533be77e

This issue was referenced by 3c533be77ea8f6c88b188eb870bb46ba972d1372

Kévin Dietrich commented

2016-08-25 05:20:50 +02:00

Changed status from 'Open' to: 'Resolved'

Kévin Dietrich closed this issue

2016-08-25 05:20:50 +02:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

The Alembic don't support Chinese path #49112