Pat Erley

The current tunings were not done with finding oscillation and using Z-N calcs. That method is less optimal for thermal systems like this The actual limit to that method is...

The actual limit we ran into here was having to update the display in the same loop. It takes, about, 7ms to update the screen, so I chose 20ms for...

Forgot to mention, once we get the threading stuff done and screen display can be moved out, the PID can be massively re-worked to use something like 2 ms update...

Fair point, an ISR would probably be the way to go now and later. Would still have to deal with the occasional 7ms delay to update the screen (unless we...

Perfect, I'll switch over to a timer for the pid and crank it up to 100,000,000Hz. That or maybe my 500Hz I originally thought. If you have the 3/5/7 sample...

Perfect, I can make sure we're firing whenever the timer is running.

@forx157 Unfortunately `esp_ble_mesh_set_unprovisioned_device_name`, is insufficient. When you call `esp_ble_mesh_init` it leads to a call to `bt_mesh_proxy_server_init`, which explicitly sets the name to "ESP-BLE-MESH", and starts advertising. If an iOS devices...

For context, here's the change we made to our app when introducing this patch and it resolved seeing 'ESP-BLE-MESH' as the name: ``` + err = esp_ble_mesh_set_default_unprovisioned_device_name(deako_provisioning_get_serial()); + RETURN_ON_ESP_ERROR(err, TAG,...

> Currently this part of the functionality is already available, you can learn about the API: `esp_ble_mesh_set_unprovisioned_device_name`，Thank you for your contribution. Any chance you can comment on if there's a...

@forx157 Friendly ping. Can you review my comments here? The existing API is not sufficient for our needs for the reasons outlined.