feat: ajout d'un fichier README_FR.md et mise à jour du README.md avec un lien de parrainage

README.md

@@ -28,11 +28,10 @@ Fabrication Toolkit is distributed with the official releases of KiCad 6+. Open
 Download the [latest release](https://github.com/bennymeg/JLC-Plugin-for-KiCad/releases) ZIP file. Open the "Plugin and Content Manager" from the KiCads main window and install the ZIP file via "Install from File".
 
 ## Usage
-Click on the Fabrication Toolkit <img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/resources/icon.png?raw=true" style="margin-bottom: 8px;" alt="Logo" height=24> button on the top tool box inside KiCad pcb editor (pcbnew).
+Click on the Fabrication Toolkit <img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/resources/icon.png?raw=true" style="margin-bottom: 8px;" alt="Logo" height=24> button on the top tool box inside KiCad pcb editor (pcbnew), choose the required options and click on 'Generate'.
 
-**⊛** Ensure your board is synchronized before invoking this addon [**F8**].
-
-**⊛** The `User_1` layer in internally defined as a **V-Cuts** layer, please avoid using it for anything else (unless disabled). <span style="text-color: light-grey !important;">_(since v3.0.0)_.</span>
+> [!IMPORTANT]  
+> Ensure your board is synchronized before invoking this addon [**F8**].
 
 ## Options
 
@@ -44,10 +43,17 @@ Options can be set in the dialog that appears when the plugin is invoked. They a
 ☑ __Additional layers__: Comma-separated list of additional layers to include in the gerber archive.</br>
 ☑ __Plot all active layers__: Whether to include all layers, instead of just the layers required by JLCPCB.</br>
 ☑ __Set User.1 as V-Cut layer__: Merge User.1 layer with the Edge-Cut layer in production.</br>
-☑ __Use User.2 for an alternative Edge-Cut layer__: Use the User.2 instead of the Edge-Cut layer for the board outline in production. This is useful if you need process edges or panelization during production but still want to keep the individual outline for prototyping, 3D model exports, or similar purposes.</br>
+☑ __Use User.2 for an alternative Edge-Cut layer__: Use the User.2 instead of the Edge-Cut layer for the board outline in production. 
+    This is useful if you need process edges or panelization during production but still want to keep the individual outline for prototyping, 3D model exports, or similar purposes.</br>
 ☑ __Apply automatic translations__: Apply known translation fixes for common components.</br>
 ☑ __Apply automatic fill for all zones__: Refill all zones before generation production files.</br>
 ☑ __Exclude DNP components from BOM__: Exclude components the had been set a DNP from th BOM.</br>
+☑ __Generate Backups__: Generate Backup zip for the production files.</br>
+
+## Attributes
+
+> [!NOTE]  
+> All the attributes are entered in the Schematic Editor, make sure to update your PCB [**F8**] with the changes made to the schematic before generating a new set of production files, otherwise your changes won't be reflected.
 
 ### ① Include Component Part Number in Production Files
 Add an 'LCSC Part #'* field with the LCSC component part number to the symbol's fields property.
@@ -91,7 +97,18 @@ If the JLC preview shows a footprint like this, enter a rotation offset of -90 t
 
 <img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/rotation.png?raw=true" height=420>
 
-As the rotation offset is entered in the Schematic Editor, make sure to update your PCB with the changes made to the schematic before generating a new set of production files, otherwise your changes won't be reflected.
+Use following table to quickly find out to which coordinate enter the correction based on JLC arrows clicks - depending on footprint rotation in KiCad PCB Editor status bar:
+|KiCad footprint deg | x | y|
+|----|----|----|
+|0 deg, Front | right arrow | up arrow |
+|0 deg, Back | left arrow | down arrow |
+|180 deg, Front | left arrow | down arrow |
+|180 deg, Back | right arrow | up arrow |
+|90 deg, Front or Back | up arrow | left arrow |
+|-90 deg, Front or Back | down arrow | right arrow |
+
+> [!TIP]  
+> Single arrow press in JLC is 0.0635mm (= 1/400in) shift.
 
 #### Primary Fields*:
 | 'FT Rotation Offset' |
@@ -110,22 +127,10 @@ _The fields will be queried in the order denoted above._
 ### ④ Offset Component Position
 The position of components in KiCad Footprints does not always match the orientation in the JLC library because KiCad and JLCPCB used different variation of the same standard. To the exception cases: add an 'FT Position Offset'* field with an comma separated x,y position offset to correct it.
 
-Use following table to quickly find out to which coordinate enter the correction based on JLC arrows clicks - depending on footprint rotation in KiCad PCB Editor status bar:
-|KiCad footprint deg | x | y|
-|----|----|----|
-|0 deg, Front | right arrow | up arrow |
-|0 deg, Back | left arrow | down arrow |
-|180 deg, Front | left arrow | down arrow |
-|180 deg, Back | right arrow | up arrow |
-|90 deg, Front or Back | up arrow | left arrow |
-|-90 deg, Front or Back | down arrow | right arrow |
-
-For custom angles it's best to place also a temporary straight symbol to perform alignment.
-Single arrow press in JLC is 0.0635mm (= 1/400in) shift.
-
 <img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/position.png?raw=true" height=420>
 
-As the position offset is entered in the Schematic Editor, make sure to update your PCB with the changes made to the schematic before generating a new set of production files, otherwise your changes won't be reflected.
+> [!TIP]  
+> Single arrow press in JLC equals to 0.0635mm (= 1/400in) shift.
 
 #### Primary Fields*:
 | 'FT Position Offset' |
@@ -216,3 +221,5 @@ options:
 ## Author
 
 Benny Megidish
+
+<iframe src="https://github.com/sponsors/electron-rare/card" title="Sponsor electron-rare" height="225" width="600" style="border: 0;"></iframe>

README_FR.md

@@ -0,0 +1,225 @@
+<img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/logo.svg?raw=true"
+    style="display:block margin-left: auto; margin-right: auto;" alt="JLC PCB Plug-in for KiCad">
+
+<div align="center">
+
+| **JLC PCB Plug-in for KiCad** |
+|:-----:|
+
+[![Sponsor](https://img.shields.io/static/v1?label=Sponsor&message=%E2%9D%A4&logo=GitHub&color=%23fe8e86)](https://github.com/sponsors/bennymeg)
+![GitHub Downloads (all assets, all releases)](https://img.shields.io/github/downloads/bennymeg/fabrication-toolkit/total)
+
+</div>
+
+## Features
+1.	Generates gerber files in correct format for production
+2.	Generates BOM file in correct format for production
+3.	Generates Pick and Place file in correct format for assembly
+4.	Generates IPC netlist file
+
+## Installation
+
+### Official Installation
+Fabrication Toolkit is distributed with the official releases of KiCad 6+. Open the "Plugin and Content Manager" from the KiCad main menu and install the "<ins>Fabrication Toolkit</ins>" plugin from the selection list.
+
+<img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/manager.png?raw=true" height=275>
+
+### Manual installation
+Download the [latest release](https://github.com/bennymeg/JLC-Plugin-for-KiCad/releases) ZIP file. Open the "Plugin and Content Manager" from the KiCads main window and install the ZIP file via "Install from File".
+
+## Usage
+Click on the Fabrication Toolkit <img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/resources/icon.png?raw=true" style="margin-bottom: 8px;" alt="Logo" height=24> button on the top tool box inside KiCad pcb editor (pcbnew), choose the required options and click on 'Generate'.
+
+> [!IMPORTANT]  
+> Ensure your board is synchronized before invoking this addon [**F8**].
+
+## Options
+
+Options can be set in the dialog that appears when the plugin is invoked. They are saved in a file called `fabrication-toolkit-options.json` in the project directory so that they are remembered between invocations of the plugin.
+
+<img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/options.png?raw=true" height=275>
+
+☑ __Archive name__: Name of the archive file to be generated. Can include text variables, such as e.g. `${TITLE}_${REVISION}`</br>
+☑ __Additional layers__: Comma-separated list of additional layers to include in the gerber archive.</br>
+☑ __Plot all active layers__: Whether to include all layers, instead of just the layers required by JLCPCB.</br>
+☑ __Set User.1 as V-Cut layer__: Merge User.1 layer with the Edge-Cut layer in production.</br>
+☑ __Use User.2 for an alternative Edge-Cut layer__: Use the User.2 instead of the Edge-Cut layer for the board outline in production. 
+    This is useful if you need process edges or panelization during production but still want to keep the individual outline for prototyping, 3D model exports, or similar purposes.</br>
+☑ __Apply automatic translations__: Apply known translation fixes for common components.</br>
+☑ __Apply automatic fill for all zones__: Refill all zones before generation production files.</br>
+☑ __Exclude DNP components from BOM__: Exclude components the had been set a DNP from th BOM.</br>
+☑ __Generate Backups__: Generate Backup zip for the production files.</br>
+
+## Attributes
+
+> [!NOTE]  
+> All the attributes are entered in the Schematic Editor, make sure to update your PCB [**F8**] with the changes made to the schematic before generating a new set of production files, otherwise your changes won't be reflected.
+
+### ① Include Component Part Number in Production Files
+Add an 'LCSC Part #'* field with the LCSC component part number to the symbol's fields property.
+
+<img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/mpn.png?raw=true" height=420>
+
+#### Primary Fields*:
+| 'LCSC Part #' | 'LCSC Part' | 'LCSC PN' | 'LCSC P/N' | 'LCSC Part No.' | 'LCSC Part Number' | 'JLCPCB Part #' | 'JLCPCB Part' | 'JLCPCB PN' | 'JLCPCB P/N' | 'JLCPCB Part No.' | 'JLCPCB Part Number' |
+| --- | --- | --- | --- | --- | --- | --- | --- | --- | --- | --- | --- |
+
+_The fields will be queried in the order denoted above._
+
+#### Fallback Fields*:
+| 'LCSC' | 'JLC' | 'MPN' | 'Mpn' | 'mpn' |
+| --- | --- | --- | --- | --- |
+
+_The fields will be queried in the order denoted above._
+
+Empty fields are skipped so, for example, a 'MPN' field with a value will be used if
+all earlier fields are missing or empty.
+
+---
+
+### ② Ignore Footprint in Production Files
+Select 'Exclude from board' or 'Exclude from BOM' in the symbol's attributes property in order to ignore the footprint from the relevant file.
+
+<img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/attributes.png?raw=true" height=420>
+
+Select 'Exclude from position files' or 'Exclude from BOM' in the footprint's fabrication attributes property in order to ignore the footprint from the relevant file.
+
+<img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/fabrication.png?raw=true" height=505>
+
+---
+
+### ③ Offset Component Rotation
+The rotation of components in KiCad Footprints does not always match the orientation in the JLC library because KiCad and JLC PCB used different variation of the same standard. Most of the rotations may be corrected by the `rotations.cf` definitions. To the exception cases: add an 'FT Rotation Offset'* field - with positive values indicating counter-clockwise orientation offset in degrees.
+
+<img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/rotation-jlc.png?raw=true" height=164>
+
+If the JLC preview shows a footprint like this, enter a rotation offset of -90 to rotate pin 1 to the lower right corner.
+
+<img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/rotation.png?raw=true" height=420>
+
+Use following table to quickly find out to which coordinate enter the correction based on JLC arrows clicks - depending on footprint rotation in KiCad PCB Editor status bar:
+|KiCad footprint deg | x | y|
+|----|----|----|
+|0 deg, Front | right arrow | up arrow |
+|0 deg, Back | left arrow | down arrow |
+|180 deg, Front | left arrow | down arrow |
+|180 deg, Back | right arrow | up arrow |
+|90 deg, Front or Back | up arrow | left arrow |
+|-90 deg, Front or Back | down arrow | right arrow |
+
+> [!TIP]  
+> Single arrow press in JLC is 0.0635mm (= 1/400in) shift.
+
+#### Primary Fields*:
+| 'FT Rotation Offset' |
+| --- |
+
+_The fields will be queried in the order denoted above._
+
+#### Fallback Fields*:
+| 'Rotation Offset' | 'RotOffset' |
+| --- | --- |
+
+_The fields will be queried in the order denoted above._
+
+---
+
+### ④ Offset Component Position
+The position of components in KiCad Footprints does not always match the orientation in the JLC library because KiCad and JLCPCB used different variation of the same standard. To the exception cases: add an 'FT Position Offset'* field with an comma separated x,y position offset to correct it.
+
+<img src="https://github.com/bennymeg/JLC-Plugin-for-KiCad/blob/master/assets/position.png?raw=true" height=420>
+
+> [!TIP]  
+> Single arrow press in JLC equals to 0.0635mm (= 1/400in) shift.
+
+#### Primary Fields*:
+| 'FT Position Offset' |
+| --- |
+
+_The fields will be queried in the order denoted above._
+
+#### Fallback Fields*:
+| 'Position Offset' | 'PosOffset' |
+| --- | --- |
+
+_The fields will be queried in the order denoted above._
+
+### ⑤ Override Component Origin
+The Fabrication Toolkit reports the position of each component based on an automatically selected point of reference. This default behavior can be overridden by adding an 'FT Origin'* field to the component.
+
+#### Primary Fields*:
+| 'FT Origin' |
+| --- |
+
+_The fields will be queried in the order denoted above._
+
+#### Fallback Fields*:
+| 'Origin' |
+| --- |
+
+The **Origin** field supports the following values:
+
+- `Anchor` - Uses the footprint's anchor point, which can be modified in KiCad's footprint editor.
+- `Center` - Uses the center of the bounding box formed by the footprint's pads.
+
+### ⑥ Override Component Layer
+Some footprints may have their components defined on the opposite layer to there actual footprints. In these instances you can override mount side by adding an 'FT Layer Override'* field to the component.
+
+The **Layer Override** field supports the following values:
+- `top`, `t` - Override footprint layer to the top.
+- `bottom`, `b` - Override footprint layer to the bottom.
+
+#### Primary Fields*:
+| 'FT Layer Override' |
+| --- |
+
+_The fields will be queried in the order denoted above._
+
+#### Fallback Fields*:
+| 'Layer Override' | 'LayerOverride' |
+| --- | --- |
+
+_The fields will be queried in the order denoted above._
+
+
+## CLI
+The plugin can also be used via the linux command line. This can be particularly useful if you need to embed the plugin into an automation pipeline or environment.
+The plugin can be called with the command below:
+```
+python3 -m plugins.cli -p /myProject/myBoard.kicad_pcb
+```
+
+All the options from the GUI are also available via the cli interface:
+```
+python3 -m plugins.cli -h
+
+usage: Fabrication Toolkit [-h] --path PATH [--additionalLayers LAYERS] [--user1VCut] [--user2AltVCut]
+                           [--autoTranslate] [--autoFill] [--excludeDNP] [--allActiveLayers] [--archiveName NAME]
+                           [--openBrowser] [--noBackup]
+
+Generates JLCPCB production files from a KiCAD board file
+
+options:
+  -h, --help            show this help message and exit
+  --path PATH, -p PATH  Path to KiCAD board file
+  --additionalLayers LAYERS, -aL LAYERS
+                        Additional layers(comma-separated)
+  --user1VCut, -u1      Set User.1 as V-Cut layer
+  --user2AltVCut, -u2   Use User.2 for alternative Edge-Cut layer
+  --autoTranslate, -t   Apply automatic position/rotation translations
+  --autoFill, -f        Apply automatic fill for all zones
+  --excludeDNP, -e      Exclude DNP components from BOM
+  --allActiveLayers, -aaL
+                        Export all active layers instead of only commonly used ones
+  --archiveName NAME, -aN NAME
+                        Name of the generated archives
+  --openBrowser, -b     Open web browser with directory file overview after generation
+  --noBackup, -nB       Do not create a backup of the project before generation
+```
+
+
+## Author
+
+Benny Megidish
+
+<iframe src="https://github.com/sponsors/electron-rare/card" title="Sponsor electron-rare" height="225" width="600" style="border: 0;"></iframe>

plugins/cli.py

@@ -31,7 +31,7 @@ if __name__ == '__main__':
     options[ALL_ACTIVE_LAYERS_OPT] = args.allActiveLayers
     options[ARCHIVE_NAME] = args.archiveName
     options[EXTRA_LAYERS] = args.additionalLayers
-    options[NO_BACKUP_OPT] = args.noBackup
+    options[BACKUP_OPT] = not args.noBackup
     
     openBrowser = args.openBrowser
     nonInteractive = args.nonInteractive

plugins/options.py

@@ -8,4 +8,4 @@ ALL_ACTIVE_LAYERS_OPT = "ALL_ACTIVE_LAYERS"
 OPEN_BROWSER_OPT = "OPEN BROWSER"
 ARCHIVE_NAME = "ARCHIVE_NAME"
 EXTRA_LAYERS = "EXTRA_LAYERS"
-NO_BACKUP_OPT = "NO_BACKUP_OPT"
+BACKUP_OPT = "BACKUP_OPT"

plugins/plugin.py

@@ -4,7 +4,7 @@ import pcbnew  # type: ignore
 
 from .thread import ProcessThread
 from .events import StatusEvent
-from .options import AUTO_FILL_OPT, AUTO_TRANSLATE_OPT, EXCLUDE_DNP_OPT, EXTEND_EDGE_CUT_OPT, ALTERNATIVE_EDGE_CUT_OPT, EXTRA_LAYERS, ALL_ACTIVE_LAYERS_OPT, ARCHIVE_NAME, OPEN_BROWSER_OPT, NO_BACKUP_OPT
+from .options import AUTO_FILL_OPT, AUTO_TRANSLATE_OPT, EXCLUDE_DNP_OPT, EXTEND_EDGE_CUT_OPT, ALTERNATIVE_EDGE_CUT_OPT, EXTRA_LAYERS, ALL_ACTIVE_LAYERS_OPT, ARCHIVE_NAME, OPEN_BROWSER_OPT, BACKUP_OPT
 from .utils import load_user_options, save_user_options, get_layer_names
 
 
@@ -37,7 +37,7 @@ class KiCadToJLCForm(wx.Frame):
             AUTO_FILL_OPT: True,
             EXCLUDE_DNP_OPT: False,
             OPEN_BROWSER_OPT: True,
-            NO_BACKUP_OPT: False,
+            BACKUP_OPT: True,
         })
 
         self.mOptionsLabel = wx.StaticText(self, label='Options:')
@@ -67,8 +67,8 @@ class KiCadToJLCForm(wx.Frame):
         self.mExcludeDnpCheckbox.SetValue(userOptions[EXCLUDE_DNP_OPT])
         self.mOpenBrowserCheckbox = wx.CheckBox(self, label='Open browser after generation')
         self.mOpenBrowserCheckbox.SetValue(userOptions[OPEN_BROWSER_OPT])
-        self.mNoBackupCheckbox = wx.CheckBox(self, label='Do not create backup files')
-        self.mNoBackupCheckbox.SetValue(userOptions[NO_BACKUP_OPT])
+        self.mBackupCheckbox = wx.CheckBox(self, label='Generate backup files')
+        self.mBackupCheckbox.SetValue(userOptions[BACKUP_OPT])
 
         self.mGaugeStatus = wx.Gauge(
             self, wx.ID_ANY, 100, wx.DefaultPosition, wx.Size(600, 20), wx.GA_HORIZONTAL)
@@ -91,7 +91,7 @@ class KiCadToJLCForm(wx.Frame):
         boxSizer.Add(self.mAutomaticFillCheckbox, 0, wx.ALL, 5)
         boxSizer.Add(self.mExcludeDnpCheckbox, 0, wx.ALL, 5)
         boxSizer.Add(self.mOpenBrowserCheckbox, 0, wx.ALL, 5)
-        boxSizer.Add(self.mNoBackupCheckbox, 0, wx.ALL, 5)
+        boxSizer.Add(self.mBackupCheckbox, 0, wx.ALL, 5)
         boxSizer.Add(self.mGaugeStatus, 0, wx.ALL, 5)
         boxSizer.Add(self.mGenerateButton, 0, wx.ALL, 5)
 
@@ -122,7 +122,7 @@ class KiCadToJLCForm(wx.Frame):
         options[AUTO_FILL_OPT] = self.mAutomaticFillCheckbox.GetValue()
         options[EXCLUDE_DNP_OPT] = self.mExcludeDnpCheckbox.GetValue()
         options[OPEN_BROWSER_OPT] = self.mOpenBrowserCheckbox.GetValue()
-        options[NO_BACKUP_OPT] = self.mNoBackupCheckbox.GetValue()
+        options[BACKUP_OPT] = self.mBackupCheckbox.GetValue()
 
         save_user_options(options)
 
@@ -136,6 +136,7 @@ class KiCadToJLCForm(wx.Frame):
         self.mAutomaticFillCheckbox.Hide()
         self.mExcludeDnpCheckbox.Hide()
         self.mOpenBrowserCheckbox.Hide()
+        self.mBackupCheckbox.Hide()
         self.mGenerateButton.Hide()
         self.mGaugeStatus.Show()
 

plugins/process.py

@@ -347,7 +347,6 @@ class ProcessManager:
                     rowNum = rowNum + 1
                     # First row is header row, skip.
                     if rowNum == 1:
-                        skipFirst = False
                         continue
 
                     # If there was too many fields, throw an exception.
@@ -532,7 +531,7 @@ class ProcessManager:
             try:
                 offset = offset.split(",")
                 return (float(offset[0]), float(offset[1]))
-            except Exception as e:
+            except Exception:
                 raise RuntimeError("Position offset of {} is not a valid pair of numbers".format(footprint.GetReference()))
 
     def _get_origin_from_footprint(self, footprint) -> float:

plugins/thread.py

@@ -162,12 +162,15 @@ class ProcessThread(Thread):
         os.rename(temp_file, os.path.join(temp_dir, gerberArchiveName))
 
         if self.options[ARCHIVE_NAME]:
-            os.rename(os.path.join(temp_dir, designatorsFileName), os.path.join(temp_dir, ProcessManager.normalize_filename("_".join((baseName.strip() + '_designators.csv').split()))))
-            os.rename(os.path.join(temp_dir, placementFileName), os.path.join(temp_dir, ProcessManager.normalize_filename("_".join((baseName.strip() + '_positions.csv').split()))))
-            os.rename(os.path.join(temp_dir, bomFileName), os.path.join(temp_dir, ProcessManager.normalize_filename("_".join((baseName.strip() + '_bom.csv').split()))))
+            if os.path.exists(os.path.join(temp_dir, designatorsFileName)):
+                os.rename(os.path.join(temp_dir, designatorsFileName), os.path.join(temp_dir, ProcessManager.normalize_filename("_".join((baseName.strip() + '_designators.csv').split()))))
+            if os.path.exists(os.path.join(temp_dir, placementFileName)):
+                os.rename(os.path.join(temp_dir, placementFileName), os.path.join(temp_dir, ProcessManager.normalize_filename("_".join((baseName.strip() + '_positions.csv').split()))))
+            if os.path.exists(os.path.join(temp_dir, bomFileName)):
+                os.rename(os.path.join(temp_dir, bomFileName), os.path.join(temp_dir, ProcessManager.normalize_filename("_".join((baseName.strip() + '_bom.csv').split()))))
 
-        # Make a backup as long as the NO_BACKUP_OPT flag isn't set.
-        if not self.options[NO_BACKUP_OPT]:
+        # Make a backup as long as the BACKUP_OPT flag is set.
+        if not self.options[BACKUP_OPT]:
             timestamp = datetime.datetime.now().strftime('%Y-%m-%d %H-%M-%S')
             backup_name = ProcessManager.normalize_filename("_".join(("{} {}".format(baseName, timestamp).strip()).split()))
             shutil.make_archive(os.path.join(output_path, 'backups', backup_name), 'zip', temp_dir)
@@ -178,7 +181,7 @@ class ProcessThread(Thread):
             if self.openBrowser:
                 webbrowser.open("file://%s" % (output_path))
             shutil.rmtree(temp_dir)
-        except Exception as e:
+        except Exception:
             if self.openBrowser:
                 webbrowser.open("file://%s" % (temp_dir))