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

Only support otx log sematics #115

Merged
merged 2 commits into from
Aug 16, 2023
Merged

Conversation

stronnag
Copy link
Collaborator

Regrettably, the EdgeTX changes to log name semantics have proved to be an ongoing problem for the INAV Lua widget's log replay capability; in particular:

  • The need to use the dir() function to enumerate log files leaves the script open to "CPU limit" forced termination when the user has numerous log files.

  • The need to use table.sort() to prune the list of log files meant that log replay was not available on small screen / B&W radios.

    The replay function has therefore been changed to only support OpenTX file names. Logs may still be replayed on EdgeTX by renaming the required file (MODEL-YYYY-MM-DD_HHMMSS.csv to MODEL-YYYY-MM-DD.csv (i.e. without the time part).

Fixes #101 (again).

@stronnag stronnag merged commit a38930b into master Aug 16, 2023
4 checks passed
@stronnag stronnag deleted the only_support_OTX_log_sematics branch August 16, 2023 13:10
@rickgode
Copy link

I know many might not find this worthwhile, but I really like it and am glad the functionality can work if we rename the file. I renamed the file and took out the time, but my playback log is still greyed out. Am I missing something? I just updated to EdgeTX 2.10.

Thank you

@stronnag
Copy link
Collaborator Author

Works OK here in ETX 2.10. Note that:

  • The log finder only looks back 15 days in order to conserve resources.
  • The log must start with the current model's name.
$ ls LOGS/
'Mr Spaced Out-2023-08-15-165737.csv'  'Mr Spaced Out-2024-05-12.csv'

So for "Mr Spaced Out", there is one valid log on 2024-05-12 (and until 2024-05-27).

image

@stronnag
Copy link
Collaborator Author

And the LOGS directory is read once, at startup.

@rickgode
Copy link

Thank you! It was the 15 days. I thought it was only the naming convention.

@rickgode
Copy link

One last question. Now that I have the inav playback working. What step am I missing to run this in companion. Specifically, I was wanting to screen record the inav telemetry script running the log in companion so I could overlay it on my flight footage. When I simulate my radio in companion and scroll over to my inav screen, there is nothing there.

Thanks Again.

@stronnag
Copy link
Collaborator Author

One last question. Now that I have the inav playback working. What step am I missing to run this in companion. Specifically, I was wanting to screen record the inav telemetry script running the log in companion so I could overlay it on my flight footage. When I simulate my radio in companion and scroll over to my inav screen, there is nothing there.

Thanks Again.

No idea, the image I posted was from companion, and here companion plays the log file correctly (& thusly, a few moments on from the first image):
image

@rickgode
Copy link

Thank you again. It helps to read... I needed the LuaTelemetry v2.2.4_lua.zip : Source LUA, compiles on the device, may be required in the simulator. I was using my SD card of my transmitter.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Too many SD logs causes cpulimit error
2 participants