Frix-x / klippain-shaketune

Klipper streamlined input shaper workflow and calibration tools
GNU General Public License v3.0
602 stars 65 forks source link

No CSV files found in the /tmp folder to create the vibration graphs! #56

Closed wiggywoosaa closed 5 months ago

wiggywoosaa commented 6 months ago

K-Shake&Tune module branch

Version

v2.5.0-0-g84c406b4

Describe the bug and expected behavior

After running Vibrations Calibrations or Axes Map Calibration I received the No CSV files found in the /tmp folder to create the vibration graphs! in the cli windows. I have checked the /tmp file and the files are indeed present

Additional information and klippy.log

No response

dhruvinsh commented 6 months ago

Patch

diff --git a/K-ShakeTune/scripts/is_workflow.py b/K-ShakeTune/scripts/is_workflow.py
index dde04f6..1ea631c 100755
--- a/K-ShakeTune/scripts/is_workflow.py
+++ b/K-ShakeTune/scripts/is_workflow.py
@@ -136,7 +136,7 @@ def create_vibrations_graph(axis_name, accel, chip_name, keep_csv):
     current_date = datetime.now().strftime('%Y%m%d_%H%M%S')
     lognames = []
-    globbed_files = glob.glob(f'/tmp/{chip_name}-*.csv')
+    globbed_files = glob.glob(f'/tmp/adxl345-{chip_name}-*.csv')
     if not globbed_files:
         print("No CSV files found in the /tmp folder to create the vibration graphs!")
         sys.exit(1)
wiggywoosaa commented 6 months ago

Worked perfect, thank you

arsonists commented 6 months ago

Good morning,

I have the same problem, how can I apply the update?

wiggywoosaa commented 6 months ago

find this file - K-ShakeTune/scripts/is_workflow.py

remove this line - globbed_files = glob.glob(f'/tmp/{chip_name}-.csv') replace it with this - globbed_files = glob.glob(f'/tmp/adxl345-{chip_name}-.csv')

reboot

arsonists commented 6 months ago

thank you very much

Frix-x commented 6 months ago

Patch

diff --git a/K-ShakeTune/scripts/is_workflow.py b/K-ShakeTune/scripts/is_workflow.py
index dde04f6..1ea631c 100755
--- a/K-ShakeTune/scripts/is_workflow.py
+++ b/K-ShakeTune/scripts/is_workflow.py
@@ -136,7 +136,7 @@ def create_vibrations_graph(axis_name, accel, chip_name, keep_csv):
     current_date = datetime.now().strftime('%Y%m%d_%H%M%S')
     lognames = []
-    globbed_files = glob.glob(f'/tmp/{chip_name}-*.csv')
+    globbed_files = glob.glob(f'/tmp/adxl345-{chip_name}-*.csv')
     if not globbed_files:
         print("No CSV files found in the /tmp folder to create the vibration graphs!")
         sys.exit(1)

Hello, thanks for sharing this patch. In fact it was like this before and I had to change it since it break the compatibility with LIS2DW and other kind of sensors. This issue is in fact a duplicate of #44 .

I'll work on a proper fix to be able to get it work with all kind of sensors and with configuration where they have "custom names". In the meantime, avoid using [adxl345 my_custom_name] for everything to work correctly :)

Frix-x commented 5 months ago

I'll close this one in favor of #44