Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
1 task done
wiggywoosaa opened this issue Jan 29, 2024 · 7 comments
Closed
1 task done
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@wiggywoosaa
Copy link

K-Shake&Tune module branch

  • I confirm using the main 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

@wiggywoosaa wiggywoosaa added the bug Something isn't working label Jan 29, 2024
@dhruvinsh
Copy link

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
Copy link
Author

Worked perfect, thank you

@arsonists
Copy link

Good morning,

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

@wiggywoosaa
Copy link
Author

wiggywoosaa commented Feb 9, 2024

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
Copy link

thank you very much

@Frix-x
Copy link
Owner

Frix-x commented Feb 9, 2024

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 Frix-x added the duplicate This issue or pull request already exists label Feb 9, 2024
@Frix-x
Copy link
Owner

Frix-x commented Mar 4, 2024

I'll close this one in favor of #44

@Frix-x Frix-x closed this as completed Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

4 participants