Changeset 1199

Timestamp:

02/06/23 01:14:15 (22 months ago)

Author:

Maciej Komosinski

Message:

Cosmetic

Location:

framspy

Files:

• FramsticksLib.py (modified) (1 diff)
• frams.py (modified) (1 diff)
• gui.py (modified) (1 diff)

framspy/FramsticksLib.py

@@ -70 +70 @@
                         ec.close()
                         print(ec.messages) # output all caught messages
-                        assert ec.error_count._value() == 0, "Problem while importing file '%s'" % simfile # make missing files fatal because error messages are easy to overlook
+                        if ec.error_count._value() > 0:
+                                raise ValueError("Problem while importing file '%s'" % simfile) # make missing files or incorrect paths fatal because they would not stop Framsticks from further processing, and error messages are easy to overlook in output 
 
 

framspy/frams.py

@@ -310 +310 @@
         Python programs do not have to know the Framstics path but if they know, just pass the path as the first argument.
         Similarly '-dPATH' and '-DPATH' needed by Framsticks are optional and derived from the first path, unless they are specified as args in init().
-        '-LNAME' is the optional library name (full name including the file name extension), default is 'frams-objects.dll/.so' depending on the platform.
+        '-LNAME' is the optional library name (full name including the file name extension), default is 'frams-objects.dll/.so/.dylib' depending on the platform.
         All other arguments are passed to Framsticks and not interpreted by this function.
         """

framspy/gui.py

@@ -9 +9 @@
     args = parser.parse_args()
 
-    if args.server and args.library:
+    if args.library and args.server:
         print("Do not provide both the library path and the network server address.")
         sys.exit(2)