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

Automatic crash dump retrieval #921

Merged
merged 23 commits into from
Nov 29, 2023
Merged
Show file tree
Hide file tree
Changes from 17 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -185,6 +185,7 @@ The following environment variables can be set to alter default behavior of the
| DEPTHAI_BOOTLOADER_BINARY_ETH | Overrides device Network Bootloader binary. Mostly for internal debugging purposes. |
| DEPTHAI_ALLOW_FACTORY_FLASHING | Internal use only |
| DEPTHAI_LIBUSB_ANDROID_JAVAVM | JavaVM pointer that is passed to libusb for rootless Android interaction with devices. Interpreted as decimal value of uintptr_t |
| DEPTHAI_CRASHDUMP | Directory in which to save the crashdump. |

## Running tests

Expand Down
2 changes: 1 addition & 1 deletion cmake/Depthai/DepthaiDeviceSideConfig.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
set(DEPTHAI_DEVICE_SIDE_MATURITY "snapshot")

# "full commit hash of device side binary"
set(DEPTHAI_DEVICE_SIDE_COMMIT "cae51725e78d8128ee7324c13f74648bcc59addc")
set(DEPTHAI_DEVICE_SIDE_COMMIT "82adb2d2dac9fe8b2f0ef6ebd4594b01ce47e557")

# "version if applicable"
set(DEPTHAI_DEVICE_SIDE_VERSION "")
8 changes: 6 additions & 2 deletions include/depthai/device/DeviceBase.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -304,7 +304,7 @@ class DeviceBase {
*
* @param devInfo DeviceInfo which specifies which device to connect to
*/
DeviceBase(const DeviceInfo& devInfo);
explicit DeviceBase(const DeviceInfo& devInfo);

/**
* Connects to any available device with a DEFAULT_SEARCH_TIME timeout.
Expand Down Expand Up @@ -376,8 +376,9 @@ class DeviceBase {
* @param config Config with which the device will be booted with
* @param devInfo DeviceInfo which specifies which device to connect to
* @param pathToCmd Path to custom device firmware
* @param dumpOnly If true only the minimal connection is established to retrieve the crash dump
*/
DeviceBase(Config config, const DeviceInfo& devInfo, const dai::Path& pathToCmd);
DeviceBase(Config config, const DeviceInfo& devInfo, const dai::Path& pathToCmd, bool dumpOnly = false);

/**
* Device destructor
Expand Down Expand Up @@ -964,5 +965,8 @@ class DeviceBase {

// Device config
Config config;

dai::Path firmwarePath;
bool dumpOnly = false;
};
} // namespace dai
Loading
Loading