Skip to content

Commit

Permalink
Fix for #161, no freq in csv
Browse files Browse the repository at this point in the history
  • Loading branch information
@amanusk
amanusk committed Aug 15, 2020
1 parent 108de4b commit cd52917
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion s_tui/helper_functions.py
View file
Expand Up @@ -93,7 +93,11 @@ def output_to_csv(sources, csv_writeable_file):
csv_dict.update({'Time': time.strftime("%Y-%m-%d_%H:%M:%S")})
summaries = [val for key, val in sources.items()]
for summarie in summaries:
csv_dict.update(summarie.source.get_sensors_summary())
update_dict = dict()
for prob, val in summarie.source.get_sensors_summary().items():
prob = summarie.source.get_source_name() + ":" + prob
update_dict[prob] = val
csv_dict.update(update_dict)

fieldnames = [key for key, val in csv_dict.items()]
writer = csv.DictWriter(csvfile, fieldnames=fieldnames)
Expand Down

3 comments on commit cd52917

@ajohnnyz
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just wondering how I use this fix. s-tui is still not showing freq in CSV for me.
Also how can do I change the default temp threshold from 80?

@ajohnnyz
Copy link

@ajohnnyz ajohnnyz commented on cd52917 Oct 19, 2020 •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I got the fix installed. I just need to know how to get the CPU to go above the default threshold, where do I change that.

Also I dot have any voltage graphs showing?

Thanks, John

@amanusk
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, John.
Did you manage to run the fix by running the source? I didn't publish a new version yet, so this is the only way for now.
You can change the default threshold by passing the -tt flag when you run s-tui. For example -tt 50.
Saving the setting will also save this setting for future runs

Please sign in to comment.