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

Please include CLCapture in the LogicAnalyser package #170

Open
UffeJakobsen opened this issue Dec 19, 2024 · 6 comments
Open

Please include CLCapture in the LogicAnalyser package #170

UffeJakobsen opened this issue Dec 19, 2024 · 6 comments

Comments

@UffeJakobsen
Copy link

Please include CLCapture in the LogicAnalyser package

It feels a little stupid to have 2 almost identical installations - just to have both GUI and CLI
Also it is a little cumbersome having to disect the 2 packages in order to merge them to one unified package

@gusmanb
Copy link
Owner

gusmanb commented Dec 19, 2024

Usually the users that need CLCapture do not use the GUI app and vice-versa, what is your use case that needs both apps?

@UffeJakobsen
Copy link
Author

For some projects this UI works - for other projects I need to feed the capture into PulseView in a automated manner and do my analysis there.

Imagine repetitive UI operations such at this: capture in one UI - export - (optional) preprocessing - import into PulseView - do some actual work.... after repeating this many times you wish for something more automated.

@UffeJakobsen
Copy link
Author

UffeJakobsen commented Dec 19, 2024

The last dissection of 5.1 packages came up with 5 files that needed to be copied from CLcapture and into the LogicAnalyser installation:

  • CLCapture.runtimeconfig.json
  • CLCapture.pdb
  • CLCapture.dll
  • CLCapture.deps.json
  • CLCapture

Their total size is around 400KB - so I guess that putting them into the LogicAnalyser package wouldn't be noticed by anyone not needing it - as the package already is huge (110MB unpacked)

@gusmanb
Copy link
Owner

gusmanb commented Dec 19, 2024

Have you tried V6.0? It includes all the analyzers from Sigrok. You can download the betas from the discussion sections.

Also CLCapture is being replaced with a much more advanced termial app, the triggers for V6.0 went so complex that is a pain to fill the command line, so the new app does a dual purpose, it can open a terminal GUI where to configure the captures, check the limits against devices, trigger captures, etc. or to if a configuration file is provided then it captures silently.

@UffeJakobsen
Copy link
Author

Have you tried V6.0? It includes all the analyzers from Sigrok. You can download the betas from the discussion sections.

No, I'm a little out of time for the moment...

PS: just downloaded 6.0.0-beta5 (I believe that is the latest) and gave it a quick run on the hardware available - and it instantly core dumped before presenting any UI - failing on to access /sys/bus/usb/devices/1-9/serial:

PPS: there is no pico attached to this system (I'm at $WORK) - but on 5.1 it was possible to work in the UI with SDL etc - without any actual hardware available.

Complete output:

Unhandled exception. System.IO.FileNotFoundException: Could not find file '/sys/bus/usb/devices/1-9/serial'.
File name: '/sys/bus/usb/devices/1-9/serial'
   at Interop.ThrowExceptionForIoErrno(ErrorInfo errorInfo, String path, Boolean isDirError)
   at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String path, OpenFlags flags, Int32 mode, Boolean failForSymlink, Boolean& wasSymlink, Func`4 createOpenException)
   at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, UnixFileMode openPermissions, Int64& fileLength, UnixFileMode& filePermissions, Boolean failForSymlink, Boolean& wasSymlink, Func`4 createOpenException)
   at System.IO.Strategies.OSFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
   at System.IO.StreamReader.ValidateArgsAndOpenPath(String path, Encoding encoding, Int32 bufferSize)
   at System.IO.File.ReadAllText(String path, Encoding encoding)
   at SharedDriver.DeviceDetector.DetectLinux() in F:\LogicAnalyzerRepo\Software\LogicAnalyzer\SharedDriver\DeviceDetector.cs:line 70
   at SharedDriver.DeviceDetector.Detect() in F:\LogicAnalyzerRepo\Software\LogicAnalyzer\SharedDriver\DeviceDetector.cs:line 40
   at LogicAnalyzer.MainWindow.RefreshPorts() in F:\LogicAnalyzerRepo\Software\LogicAnalyzer\LogicAnalyzer\MainWindow.axaml.cs:line 1292
   at LogicAnalyzer.MainWindow..ctor() in F:\LogicAnalyzerRepo\Software\LogicAnalyzer\LogicAnalyzer\MainWindow.axaml.cs:line 147
   at LogicAnalyzer.App.OnFrameworkInitializationCompleted() in F:\LogicAnalyzerRepo\Software\LogicAnalyzer\LogicAnalyzer\App.axaml.cs:line 24
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_0(Object state)
   at Avalonia.Threading.SendOrPostCallbackDispatcherOperation.InvokeCore()
   at Avalonia.Threading.DispatcherOperation.Execute()
   at Avalonia.Threading.Dispatcher.ExecuteJob(DispatcherOperation job)
   at Avalonia.Threading.Dispatcher.ExecuteJobsCore(Boolean fromExplicitBackgroundProcessingCallback)
   at Avalonia.Threading.Dispatcher.Signaled()
   at Avalonia.X11.X11PlatformThreading.CheckSignaled()
   at Avalonia.X11.X11PlatformThreading.RunLoop(CancellationToken cancellationToken)
   at Avalonia.Threading.DispatcherFrame.Run(IControlledDispatcherImpl impl)
   at Avalonia.Threading.Dispatcher.PushFrame(DispatcherFrame frame)
   at Avalonia.Threading.Dispatcher.MainLoop(CancellationToken cancellationToken)
   at Avalonia.Controls.ApplicationLifetimes.ClassicDesktopStyleApplicationLifetime.Start(String[] args)
   at Avalonia.ClassicDesktopStyleApplicationLifetimeExtensions.StartWithClassicDesktopLifetime(AppBuilder builder, String[] args, Action`1 lifetimeBuilder)
   at LogicAnalyzer.Program.Main(String[] args) in F:\LogicAnalyzerRepo\Software\LogicAnalyzer\LogicAnalyzer\Program.cs:line 22
zsh: IOT instruction (core dumped)  ./LogicAnalyzer
./LogicAnalyzer  1,20s user 0,15s system 95% cpu 1,419 total

@gusmanb
Copy link
Owner

gusmanb commented Dec 19, 2024

Thanks for the report, that's the new feature for autodetection and is not well tested under Linux. I will update the packages today.

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

No branches or pull requests

2 participants