Add modern configuration options for SIN-4-RS-20 and SIN-4-RS-20_PRO

[ikruglov]

Proposed change

This PR adds the following configuration options for NodOn roller shutter SIN-4-RS-20 and PRO version

• calibration_vertical_run_time_up - Set vertical run time up of the roller shutter
• calibration_vertical_run_time_down - Set vertical run time down of the roller shutter
• calibration_rotation_run_time_up - Set rotation run time up of the roller shutter
• calibration_rotation_run_time_down - Set rotation run time down of the roller shutter

All these settings allow fine-tune calibration if necessary.

Additional information

This is a port of https://github.com/Koenkk/zigbee-herdsman-converters/blob/master/src/devices/nodon.ts#L16

Checklist

• The changes are tested and work correctly
• pre-commit checks pass / the code has been formatted using Black
• Tests have been added to verify that the new code works

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.99%. Comparing base (bd9ecbb) to head (42ce393).
Report is 10 commits behind head on dev.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #3901      +/-   ##
==========================================
+ Coverage   90.96%   90.99%   +0.02%     
==========================================
  Files         327      328       +1     
  Lines       10606    10637      +31     
==========================================
+ Hits         9648     9679      +31     
  Misses        958      958              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[TheJulianJES]

Seems good otherwise.

[TheJulianJES]

Is there a reason to not inherit from WindowCovering.AttributeDefs here?
If that's done, we could remove all the copied attributes from zigpy below.

[TheJulianJES]

Ah, the other attributes have the same attribute id. Hmm..
Does it not work if both are included? There should be a difference because of the manufacturer id. But it might not work with zigpy cache and some other zigpy parts.

[ikruglov]

Yes, the attributes have same IDs. Happy to inherit and override if there is an easy way.

[ikruglov]

No, it doesn't work of if I just inherit. It was my first approach and it didn't work.

[TheJulianJES]

I think we should removed initially_disabled=True everywhere.

[ikruglov]

Hi. I don't have a strong opinion but these parameters are to fine-tune or override settings after internal calibration procedure. So, they are not much useful by default. Hence, I think it makes sense to keep them disabled by default.