Skip to content

Comments

change rise/drop time definition#28

Open
jingqiangye wants to merge 4 commits intomasterfrom
risedroptime
Open

change rise/drop time definition#28
jingqiangye wants to merge 4 commits intomasterfrom
risedroptime

Conversation

@jingqiangye
Copy link
Contributor

In this update, we changed the definition of rise and drop.

  • before: rise time is time between 10% and 50% height, similar for drop time
  • now: between 10% and 90% height

In this case we increased rise and drop time, which will benefit small signals where rise/drop time is 0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant