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.

Sign up for GitHub

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

Jump to bottom

LambdaExpressionExtensions sample notebook #191

Merged
merged 7 commits into from
Oct 19, 2022
Merged

LambdaExpressionExtensions sample notebook #191

merged 7 commits into from
Oct 19, 2022

Conversation

ElisendaGascon
Copy link
Contributor

No description provided.

@jamesbroome
Copy link

My overriding comment on this one is I'm not sure under what conditions I'd need to use it. The samples are easy to understand in the abstract, but I can't think of a use case where I'd need to do this. Are we able to flesh this one out a bit more with something real - i.e. do we have a specific usage scenario that would translate into the notebook?

@idg10 idg10 added the no_release Suppresses auto_release functionality label Oct 3, 2022
@jesshill99
Copy link
Contributor

My overriding comment on this one is I'm not sure under what conditions I'd need to use it. The samples are easy to understand in the abstract, but I can't think of a use case where I'd need to do this. Are we able to flesh this one out a bit more with something real - i.e. do we have a specific usage scenario that would translate into the notebook?

Updated the notebook to include examples of specific usage scenarios.

@Lmooney25 Lmooney25 merged commit 8c28851 into main Oct 19, 2022
@Lmooney25 Lmooney25 deleted the lambda-expresion-sample-notebook branch October 19, 2022 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamesbroome jamesbroome jamesbroome left review comments

@jesshill99 jesshill99 jesshill99 left review comments

@idg10 idg10 idg10 approved these changes

Assignees
No one assigned
Labels
no_release Suppresses auto_release functionality
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ElisendaGascon @jamesbroome @jesshill99 @idg10 @Lmooney25