-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[MATLAB] Add TimeUnit
and TimeZone
to the arrow.type.TimestampType
display
#36735
Comments
take |
kevingurney
pushed a commit
that referenced
this issue
Jul 26, 2023
…Type` display (#36871) ### Rationale for this change The `arrow.type.TimestampType` display should include the `TimeUnit` and `TimeZone` properties. Right now we only display the `ID` property: ```matlab >> type = arrow.type.timestamp(TimeUnit="Second", TimeZone="America/Anchorage") type = TimestampType with properties: ID: Timestamp ``` We should show the other two properties in the display. ### What changes are included in this PR? Modified the display of `TimestampType`: ```matlab >> type = arrow.type.timestamp(TimeUnit="Second", TimeZone="America/Anchorage") type = TimestampType with properties: ID: Timestamp TimeUnit: Second TimeZone: "America/Anchorage" ``` Now `TimeUnit` and `TimeZone` are included. ### Are these changes tested? Added a test case to `tTimestampType` called `Display`. It verifies `TimestampType` objects are displayed correctly in the Command Window. ### Are there any user-facing changes? Yes, users will see the new display. * Closes: #36735 Authored-by: Sarah Gilmore <sgilmore@mathworks.com> Signed-off-by: Kevin Gurney <kgurney@mathworks.com>
R-JunmingChen
pushed a commit
to R-JunmingChen/arrow
that referenced
this issue
Aug 20, 2023
…estampType` display (apache#36871) ### Rationale for this change The `arrow.type.TimestampType` display should include the `TimeUnit` and `TimeZone` properties. Right now we only display the `ID` property: ```matlab >> type = arrow.type.timestamp(TimeUnit="Second", TimeZone="America/Anchorage") type = TimestampType with properties: ID: Timestamp ``` We should show the other two properties in the display. ### What changes are included in this PR? Modified the display of `TimestampType`: ```matlab >> type = arrow.type.timestamp(TimeUnit="Second", TimeZone="America/Anchorage") type = TimestampType with properties: ID: Timestamp TimeUnit: Second TimeZone: "America/Anchorage" ``` Now `TimeUnit` and `TimeZone` are included. ### Are these changes tested? Added a test case to `tTimestampType` called `Display`. It verifies `TimestampType` objects are displayed correctly in the Command Window. ### Are there any user-facing changes? Yes, users will see the new display. * Closes: apache#36735 Authored-by: Sarah Gilmore <sgilmore@mathworks.com> Signed-off-by: Kevin Gurney <kgurney@mathworks.com>
loicalleyne
pushed a commit
to loicalleyne/arrow
that referenced
this issue
Nov 13, 2023
…estampType` display (apache#36871) ### Rationale for this change The `arrow.type.TimestampType` display should include the `TimeUnit` and `TimeZone` properties. Right now we only display the `ID` property: ```matlab >> type = arrow.type.timestamp(TimeUnit="Second", TimeZone="America/Anchorage") type = TimestampType with properties: ID: Timestamp ``` We should show the other two properties in the display. ### What changes are included in this PR? Modified the display of `TimestampType`: ```matlab >> type = arrow.type.timestamp(TimeUnit="Second", TimeZone="America/Anchorage") type = TimestampType with properties: ID: Timestamp TimeUnit: Second TimeZone: "America/Anchorage" ``` Now `TimeUnit` and `TimeZone` are included. ### Are these changes tested? Added a test case to `tTimestampType` called `Display`. It verifies `TimestampType` objects are displayed correctly in the Command Window. ### Are there any user-facing changes? Yes, users will see the new display. * Closes: apache#36735 Authored-by: Sarah Gilmore <sgilmore@mathworks.com> Signed-off-by: Kevin Gurney <kgurney@mathworks.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
Currently, the display for
arrow.type.TimestampType
does not show theTimeUnit
or theTimeZone
.We should modify the display to show both
TimeUnit
andTimeZone
.Component(s)
MATLAB
The text was updated successfully, but these errors were encountered: