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

Summer2023 func args #229

Draft
wants to merge 14 commits into
base: summer2023
Choose a base branch
from
Draft

Summer2023 func args #229

wants to merge 14 commits into from

Conversation

dbosk
Copy link
Owner

@dbosk dbosk commented Jul 21, 2023

Reworks the functions-variables section.

@dbosk dbosk changed the base branch from summer2023-initial-example to summer2023 July 21, 2023 19:21
@dbosk dbosk requested a review from celinasoori July 23, 2023 22:02
@dbosk
Copy link
Owner Author

dbosk commented Jul 23, 2023

@celinasoori still not sure if the fusion in the first example should be fusion or not. It's not really varying several aspects together. Or maybe it is. I'll think some more while I continue with other aspects.

One thing that I find is missing now is "starting with the undivided whole, focusing its parts (contrast and generalisation) and then fusing it back together".

- Removes fusion, those examples should be generalisation.
- Adds XXX about functions next.
@dbosk
Copy link
Owner Author

dbosk commented Jul 24, 2023

I decided that the fusion is actually generalisation.

@dbosk
Copy link
Owner Author

dbosk commented Jul 24, 2023

I added the first example of functions in relation to variables. Since I reuse the first example to introduce the contrast, I think we're getting a bit of the part-whole relationship.

@dbosk
Copy link
Owner Author

dbosk commented Jul 24, 2023

I also highlighted what corresponds. Maybe that's better in this kind of example? Since that's the relevant change, the others being simply consequences. What do you think @celinasoori ?

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.

1 participant