Document ammeter and multi-axis simulation graph controls#770
Document ammeter and multi-axis simulation graph controls#770ShiboSoftwareDev wants to merge 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| referenceTo=".V1 > .pin2" | ||
| graphDisplayName="VOUT" | ||
| graphCenter={3.3} | ||
| graphOffsetDivs={-1} |
There was a problem hiding this comment.
This should just be graphVerticalOffset, not graphOffsetDivs
| graphDisplayName="VOUT" | ||
| graphCenter={3.3} | ||
| graphOffsetDivs={-1} | ||
| graphUnitsPerDiv={0.05} |
There was a problem hiding this comment.
ive already said this- REMOVE "UNITS", graphVoltagePerDiv.
This api is not clear!
seveibar
left a comment
seveibar
left a comment
There was a problem hiding this comment.
The ammeter api is also wrong
| graphDisplayName="IIN" | ||
| graphCenter={0.005} | ||
| graphOffsetDivs={1} | ||
| graphUnitsPerDiv={0.001} |
There was a problem hiding this comment.
| graphUnitsPerDiv={0.001} | |
| graphCurrentPerDiv={0.001} |
| color="#e05a00" | ||
| graphDisplayName="IIN" | ||
| graphCenter={0.005} | ||
| graphOffsetDivs={1} |
There was a problem hiding this comment.
graphVerticalOffset="1mA"
|
Also the user should NOT need to specify those, they should be automatically computed- they should not appear in a code example IMO because 99% of the time they should be autocomputed (graphVerticalOffset and graphXXXPerDiv) |
|
Also the graphCenter!!!!! The user will 99% of the time not specify them- please do nit include them in code samples!!!! They are an advanced almost entirely unused property that need their names fixed |
|
Nobody wants to manually configure that when it can be done automatically |
2 participants
No description provided.