Gena,

Can you please try if the attached patch fixes the issue?

Thanks
Utkarsh

On Fri, Feb 24, 2017 at 5:27 AM, Gena Bug <[email protected]> wrote:
> Hi, Utkarsh
>
> I've just tried RC2 and saw that plots looks much better, thanks!
>
> I also noticed another issues:
> dot and comma: https://gitlab.kitware.com/paraview/paraview/issues/17225
> rescale: https://gitlab.kitware.com/paraview/paraview/issues/17226
>
> However, they both concern my regional settings (ru). After switching to C
> they are gone.
>
>
> Debian testing amd64, kf5, plasma5
>
>
> On 20.02.2017 23:48, Utkarsh Ayachit wrote:
>>
>> Gena,
>>
>> I can certainly reproduce this issue. Let me see what's going on. I've
>> reported the issue here:
>> https://gitlab.kitware.com/paraview/paraview/issues/17202
>>
>> On Sat, Feb 18, 2017 at 10:02 AM, Gena Bug via ParaView
>> <[email protected]> wrote:
>>>
>>> Hi,
>>>
>>> I noticed a difference in the plot rendering (Line Chart View) -- in the
>>> 5.3RC1 lines aren't smooth as it were in 5.2.
>>>
>>> Another difference is that in 5.3RC1 you can't set real numbers in the
>>> axis
>>> custom range fields using . (dot), only with , (comma).
>>>
>>> Debian testing amd64, nvidia, fxaa disabled.
>>>
>>>
>>> On 13.02.2017 19:49, Cory Quammen wrote:
>>>>
>>>>
>>>> Folks,
>>>>
>>>> Binaries and source code zip files and tar balls for ParaView 5.3.0
>>>> Release Candidate 1 are now available for download from
>>>>
>>>> http://www.paraview.org/download/
>>>>
>>>> Please let us know if you run into any problems with this release
>>>> candidate.
>>>>
>>>> Sincerely,
>>>> Cory
>>>>
>>>
>>> _______________________________________________
>>> Powered by www.kitware.com
>>>
>>> Visit other Kitware open-source projects at
>>> http://www.kitware.com/opensource/opensource.html
>>>
>>> Please keep messages on-topic and check the ParaView Wiki at:
>>> http://paraview.org/Wiki/ParaView
>>>
>>> Search the list archives at: http://markmail.org/search/?q=ParaView
>>>
>>> Follow this link to subscribe/unsubscribe:
>>> http://public.kitware.com/mailman/listinfo/paraview
>>>
>>
>
diff --git a/CMake/branded_paraview_main.cxx.in b/CMake/branded_paraview_main.cxx.in
index 9af8446..b965f19 100644
--- a/CMake/branded_paraview_main.cxx.in
+++ b/CMake/branded_paraview_main.cxx.in
@@ -38,6 +38,7 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 #include <QApplication>
 #include <QStyleFactory>
+#include <QLocale>
 
 #include "pq@[email protected]"
 #include "vtkPVConfig.h"
@@ -104,10 +105,8 @@ int main(int argc, char* argv[])
 #endif
 #endif
 
-  // We force LC_NUMERIC to C - it is important as Qt sets locale to the user's
-  // environment locale value and this might break some basic features like
-  // the calls to atof() that are locale dependent.
-  vtksys::SystemTools::PutEnv("LC_NUMERIC=C");
+  std::setlocale(LC_NUMERIC, LC_CTYPE); 
+  QLocale::setDefault(QLocale::c());
 
   QApplication qtapp(argc, argv);
 
_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the ParaView Wiki at: 
http://paraview.org/Wiki/ParaView

Search the list archives at: http://markmail.org/search/?q=ParaView

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/paraview

Reply via email to