Egor Savkin esavkin
  • Joined on 2022-09-30
esavkin commented on issue M-Labs/artiq-zynq#204 2023-01-03 10:11:45 +08:00
ZC706+Evaluation Board Not Running Idle Kernels

RISC-V version doesn't require reboot. It needs to run experiment only

esavkin commented on pull request M-Labs/artiq-zynq#210 2023-01-03 09:56:38 +08:00
Fix idle_kernel typo in config

That's right, fixed

esavkin pushed to 204-idle-kernel-typo at esavkin/artiq-zynq 2023-01-03 09:56:07 +08:00
44ef13d1c0 Fix idle/startup_kernel typos in config
esavkin created pull request M-Labs/artiq-zynq#210 2022-12-23 16:16:02 +08:00
Fix idle_kernel typo in config
esavkin pushed to 204-idle-kernel-typo at esavkin/artiq-zynq 2022-12-23 16:15:32 +08:00
6b79b82dd5 Fix idle_kernel typo in config
esavkin created branch 204-idle-kernel-typo in esavkin/artiq-zynq 2022-12-23 16:15:32 +08:00
esavkin commented on issue M-Labs/artiq-zynq#204 2022-12-23 16:05:13 +08:00
ZC706+Evaluation Board Not Running Idle Kernels

Can confirm it reproduces as in the description. However, idle kernel can be run, but after board's reboot. Also, there is (possibly mistaken) difference from documentation and RISC-V ARTIQ - the…

esavkin commented on pull request M-Labs/thermostat#75 2022-12-23 15:18:29 +08:00
Fix failure on commands ending on whitespace

on the firmware side instead

I think it's better to keep both soulutions, since it is also covered by tests now:

$ cargo test --target x86_64-unknown-linux-gnu
   Compiling…
esavkin pushed to 63-fix-pid-error at esavkin/thermostat 2022-12-23 15:15:53 +08:00
081b461e31 Accept ending whitespace in commands
esavkin pushed to 69-fan_pwm at esavkin/thermostat 2022-12-23 13:23:28 +08:00
630635486e Polish minor issues
esavkin pushed to 69-fan_pwm at esavkin/thermostat 2022-12-23 13:13:43 +08:00
ea2eb51b27 Add fan warnings
esavkin commented on pull request M-Labs/thermostat#73 2022-12-22 17:28:57 +08:00
Support fan PWM settings

Moved to the separate file

esavkin pushed to 69-fan_pwm at esavkin/thermostat 2022-12-22 17:28:19 +08:00
58650d37f1 Refactor and coefficients implemented
esavkin commented on pull request M-Labs/thermostat#73 2022-12-22 12:29:27 +08:00
Support fan PWM settings

Also, that may be useful for GUI, as user could choose any 3 points on the fan graph.

esavkin commented on pull request M-Labs/thermostat#73 2022-12-22 11:35:25 +08:00
Support fan PWM settings

Oscilloscope shows similar values

esavkin commented on pull request M-Labs/thermostat#73 2022-12-21 17:18:57 +08:00
Support fan PWM settings

c would add ability to make it nearly linear

esavkin commented on pull request M-Labs/thermostat#73 2022-12-21 17:16:32 +08:00
Support fan PWM settings

Introduce a,b,c of quadratic equation coefficients?

esavkin commented on pull request M-Labs/thermostat#73 2022-12-21 17:13:24 +08:00
Support fan PWM settings

Mostly because I cannot guarantee it is precise and would not like to see how someone would tie something-very-important to its value.

esavkin commented on pull request M-Labs/thermostat#73 2022-12-21 17:09:03 +08:00
Support fan PWM settings

Yes, it is not tied to any channel. But it is logically similar to other PWM pins.

esavkin commented on pull request M-Labs/thermostat#73 2022-12-21 17:04:25 +08:00
Support fan PWM settings

I do not have exact numbers on pulses missing IRL. The tested cases are:

  1. Increasing PWM should lead to increased tacho number.
  2. Manually lowering the fan speed should lead to lower…