-
Notifications
You must be signed in to change notification settings - Fork 2
adding api doc and cleanup of Shape and related concepts #94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -48,7 +48,7 @@ class Tensor[T <: Tuple: Labels, V] private[dimwit] ( | |
| s"TracerTensor(${shape.toString})" | ||
| case _ => jaxValue.toString() | ||
|
|
||
| def extent[L](axis: Axis[L])(using ev: AxisIndex[T, L]): AxisExtent[L] = | ||
| def extent[L: Label](axis: Axis[L])(using ev: AxisIndex[T, L]): AxisExtent[L] = | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Was label evidence added on purpose. I suggest removing it if not necessary in the method.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also here I thought that adding a label for documentation could help. It is not really needed but in most other method when we have this type parameter it also needs to be a label. so why not adding it here? Again, I have no strong opinion. |
||
| shape.extent(axis) | ||
|
|
||
| private val jaxTypeName: String = py.Dynamic.global.`type`(jaxValue).`__name__`.as[String] | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
L Label evidence added. See other comment.