Changeset 012e397 in sasview for src/sas/sasgui/perspectives/fitting/basepage.py

Timestamp:

Nov 7, 2016 5:58:15 AM (8 years ago)

Author:

wojciech

Branches:

master, ESS_GUI, ESS_GUI_Docs, ESS_GUI_batch_fitting, ESS_GUI_bumps_abstraction, ESS_GUI_iss1116, ESS_GUI_iss879, ESS_GUI_iss959, ESS_GUI_opencl, ESS_GUI_ordering, ESS_GUI_sync_sascalc, costrafo411, magnetic_scatt, release-4.1.1, release-4.1.2, release-4.2.2, ticket-1009, ticket-1094-headless, ticket-1242-2d-resolution, ticket-1243, ticket-1249, ticket885, unittest-saveload

Children:

bf44249e

Parents:

e0d9ed2

Message:

Keeping time delay for Windows as it seems to be necessary with the current design of SasView?

File:

• src/sas/sasgui/perspectives/fitting/basepage.py (modified) (1 diff)

src/sas/sasgui/perspectives/fitting/basepage.py

@@ -1456 +1456 @@
                 self.state_change = True
                 self._draw_model()
-                #Time delay introduced to prevent double compilation on Windows
-                #It seems to be threading related issue that triggers
-                #compilation twice at the same time
-                if ON_MAC == False:
+                # Time delay has been introduced to prevent _handle error
+                # on Windows
+                # This part of code is executed when model is selected and
+                # it's parameters are changed (with respect to previously
+                # selected model). There are two evaluations of Iq occuring one
+                # after another and therefore there may be compilation error
+                # if model is calculted for the first time.
+                # The proper solution (other than time delay) requires more
+                # fundemental code refatoring
+                # Wojtek P. Nov 7, 2016
+                if not ON_MAC:
                     time.sleep(0.1)
                 self.Refresh()