Skip to content
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

Node consistency 81 90 #11141

Merged
merged 21 commits into from
Oct 6, 2020

Conversation

martinstacey
Copy link
Contributor

Purpose

The following PR addresses nodes' names and descriptions to make them more descriptive for beginners.

81) NODE: List.Deconstruct
-CHANGES:
Output description:
first: type: var[]..[] (first item in the list)
rest: type: var[]..[] (rest of the list)

82) NODE: List.Map
-CHANGES:
Input: fx -> function
Output: mapped -> list
Output description list: type: var[]..[] (mapped list)

83) NODE: List.Cycle
-CHANGES:
Output description list: type: var[]..[] (list of repeated lists)

84) NODE: Sequence
-CHANGES:
Input description: Amount of numbers or letters in the sequence
Output seq to list
Output description list: type: var[]..[] (New list)

85) NODE: Range
-CHANGES:
Output seq to list
Output description list: type: var[]..[] (New list)

86) NODE: TimeSpan.Components
-CHANGES:
Output description:
days: type: int (amount of days)
hours: type: int (amount of hours)
minutes: type: int (amount of minutes)
seconds: type: int (amount of seconds)
milliseconds: type: int (amount of milliseconds)

87) NODE: Object.IsNull
-CHANGES:
Description: Determines if the given object is null.

88) NODE: Object.Identity
-CHANGES:
Input: obj to object
Output : var[]..[] to object
Output object description: Same object

89) NODE: DateTime.TimeOfDay
-CHANGES:
Output: var to TimeSpan
Output Description: TimeSpan since midnight

90) NODE: DateTime.Date
-CHANGES:
Output: var to date
Output Description: date from DateTime

image

Reviewers

@SHKnudsen
@Amoursol
@QilongTang
@aparajit-pratap

FYIs

-No FYI

Master Branch Updated from DynamoDS/master
**NODE: Sequence**
-CHANGES:
Input description: Amount of numbers or letters in the sequence
Output seq to sequence

**NODE: Range**
-CHANGES:
Output seq to sequence
**NODE: TimeSpan.Components**
-CHANGES:
Output description:
days: Amount of days on TimeSpan
hours: Amount of hours on TimeSpan
minutes: Amount of minutes on TimeSpan
seconds: Amount of seconds on TimeSpan
milliseconds: Amount of milliseconds on TimeSpan
**NODE: Object.IsNull**
-CHANGES:
Description: Determines if the given object is null.
Input: obj to object
Output : bool to boolean
**NODE: Object.Identity**
-CHANGES:
Input: obj to object
Output : var[]..[] to object
**NODE: DateTime.TimeOfDay**
-CHANGES:
Output: var to TimeSpan
Output Description: TimeSpan since midnight
**NODE: DateTime.Date**
-CHANGES:
Output: var to date
Output Description: date from DateTime
new search options: Date, DataTime.Date
f(x) to function
Range:
Output: seq to sequence
Description: Amount of numbers or letters in the sequence

Sequence:
Output: seq to sequence
Node name erased from search, Node name is not required for search
outputs descriptions updated to include datatype
Sequence and Range:
output name: list
output list description : type: var[]..[] (New list)
output name mapped -> list
output description: type: var[]..[] (mapped list)
output first description: type: var[]..[] (first item in the list)
output rest description: type: var[]..[] (rest of the list)
Update changes from DynamoDS master branch 2020-09-23 09-52
@Amoursol
Copy link
Contributor

@martinstacey Are the outputs descriptions in this format: type: var[]..[] (New list) ? If so, please inverse it to New list of type: var[]..[]

Copy link
Contributor

@aparajit-pratap aparajit-pratap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@martinstacey please address review comments before we can merge this PR. Thanks.

@aparajit-pratap aparajit-pratap merged commit fddfe07 into DynamoDS:master Oct 6, 2020
@martinstacey martinstacey deleted the Node-Consistency-81-90 branch December 3, 2020 12:28
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.

3 participants