On 02/26/17 20:07, Utkarsh Ayachit wrote:
Gena,

Can you please try if the attached patch fixes the issue?
Yep! But it'll be only tomorrow..


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





locale.patch


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