Chris Satterlee / IV Swinger 2 - History

Add commas to font list

No Issue filed for this. The list of fonts generated by clicking on the List button on the Plotting tab of Preferences had no delimiters, so it was impossible to tell where each font name started. This change adds a comma.

Fix import to instance (Issue #217)

import_results() method: If import is to an instance, set the copy destination to the import folder itself instead of its parent.
get_dest_dir() method: if destination is an instance, don't add the app name or source directory instance path.
display_copy_summary() method: display the correct path (don't append IV_Swinger2) if the destination is an instance

Remove redundant attributes/properties and more

This commit:
- Removes attributes and properties from the IV_Swinger2 class that are not necessary because they override the same-named attributes and properties in the parent IV_Swinger class, and they are identical
- Replaces several hardcoded values with constants
- Fixes Issue #214 the correct way

Switch to icecream for debug printing (Issue #216)

Removed the print_dbg_str function and added the import and configuration of icecream to all modules. Now I can just insert:
ic() on a line to print the file/linenumber when the line is executed, or can include a variable or expression for more fancy output.

Fix refurb FURB136 suggestions (Issue #215)

Fixed these:

csatt@Chriss-2021-MBP_209.17:04:08[GitHub/IV_Swinger/python3]=> refurb --disable-all --enable FURB136 *.py
IV_Swinger2_gui.py:9170:21 [FURB136]: Replace `x if x <= y else y` with `min(x, y)`
IV_Swinger_PV_model.py:234:16 [FURB136]: Replace `x if x < y else y` with `min(x, y)`
IV_Swinger_PV_model.py:257:20 [FURB136]: Replace `x if x < y else y` with `min(x, y)`
IV_Swinger_PV_model.py:278:20 [FURB136]: Replace `x if x < y else y` with `min(x, y)`
IV_Swinger_PV_model.py:310:20 [FURB136]: Replace `x if x < y else y` with `min(x, y)`
IV_Swinger_PV_model.py:334:20 [FURB136]: Replace `x if x < y else y` with `min(x, y)`
IV_Swinger_PV_model.py:582:46 [FURB136]: Replace `x if x > y else y` with `max(x, y)`

Fix refurb FURB123 suggestions (Issue #215)

Fixed these:

csatt@Chriss-2021-MBP_209.17:00:45[GitHub/IV_Swinger/python3]=> refurb --disable-all --enable FURB123 *.py
IV_Swinger2_gui.py:7964:31 [FURB123]: Replace `str(x)` with `x`
IV_Swinger2_gui.py:7965:37 [FURB123]: Replace `str(x)` with `x`
IV_Swinger2_gui.py:7966:28 [FURB123]: Replace `str(x)` with `x`
IV_Swinger2_gui.py:7970:30 [FURB123]: Replace `str(x)` with `x`
IV_Swinger2_gui.py:7971:26 [FURB123]: Replace `str(x)` with `x`
IV_Swinger2_gui.py:7972:26 [FURB123]: Replace `str(x)` with `x`
IV_Swinger2_gui.py:7973:32 [FURB123]: Replace `str(x)` with `x`
IV_Swinger2_gui.py:7974:31 [FURB123]: Replace `str(x)` with `x`
IV_Swinger2_gui.py:7975:32 [FURB123]: Replace `str(x)` with `x`

Fix refurb FURB108 suggestion (Issue #215)

Fixed this one:

csatt@Chriss-2021-MBP_209.16:57:52[GitHub/IV_Swinger/python3]=> refurb --disable-all --enable FURB108 *.py
IV_Swinger2_gui.py:6749:12 [FURB108]: Replace `x == y or z == y` with `y in (x, z)`

Fix refurb FURB125 suggestion (Issue #215)

Fixed this one:

csatt@Chriss-2021-MBP_209.16:56:42[GitHub/IV_Swinger/python3]=> refurb --disable-all --enable FURB125 *.py
IV_Swinger2_gui.py:5303:13 [FURB125]: Return is redundant here

Fix refurb FURB165 suggestion (Issue #215)

Fixed this one:

csatt@Chriss-2021-MBP_209.16:52:46[GitHub/IV_Swinger/python3]=> refurb --disable-all --enable FURB165 *.py
IV_Swinger2_gui.py:3367:24 [FURB165]: Replace `Path().home()` with `Path.home()`

Fix refurb FURB124 suggestions (Issue #215)

Fixed these:

csatt@Chriss-2021-MBP_209.16:49:19[GitHub/IV_Swinger/python3]=> refurb --disable-all --enable FURB124 *.py
IV_Swinger2_gui.py:3198:12 [FURB124]: Replace `x is y and z is y` with `x is y is z`
IV_Swinger2_gui.py:3910:16 [FURB124]: Replace `x is y and z is y` with `x is y is z`
IV_Swinger2_gui.py:9661:16 [FURB124]: Replace `x == y and z == y` with `x == y == z`