We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
waitForConfig()
As discussed in micro-manager/pymmcore#90 there seems to be unexpected behaviour in waitForConfig(), where errors are logged but ignored:
void CMMCore::waitForConfig(const char* group, const char* configName) throw (CMMError) { CheckConfigGroupName(group); CheckConfigPresetName(configName); Configuration cfg = getConfigData(group, configName); try { for(size_t i=0; i<cfg.size(); i++) waitForDevice(cfg.getSetting(i).getDeviceLabel().c_str()); } catch (CMMError& err) { // trap MM exceptions and keep quiet - this is not a good time to blow up logError("waitForConfig", err.getMsg().c_str()); } }
For comparison, in waitForSystem() errors like e.g. a device timeout error are propagated from waitForDevice()->waitForDeviceType()->waitForSystem().
waitForSystem()
waitForDevice()
waitForDeviceType()
As by @marktsuchida, it's been this way since 2007, so changing this would probably break existing scripts.
The text was updated successfully, but these errors were encountered:
Thanks for creating the issue. We might be able to slowly migrate this to throwing using the new "feature" facility (#394/#399).
Sorry, something went wrong.
No branches or pull requests
As discussed in micro-manager/pymmcore#90 there seems to be unexpected behaviour in
waitForConfig()
, where errors are logged but ignored:For comparison, in
waitForSystem()
errors like e.g. a device timeout error are propagated fromwaitForDevice()
->waitForDeviceType()
->waitForSystem()
.As by @marktsuchida, it's been this way since 2007, so changing this would probably break existing scripts.
The text was updated successfully, but these errors were encountered: