Skip to content

Commit

Permalink
Merge pull request #290 from ostephens/instructor-notes-fix
Browse files Browse the repository at this point in the history
Fixes to instructor notes in 06 and 11
  • Loading branch information
jas58 authored Jun 27, 2023
2 parents e1b462b + 3d9185d commit a7e7c8a
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 13 deletions.
14 changes: 9 additions & 5 deletions episodes/06-working-with-columns.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,15 @@ Unlike in Excel, 'Sorts' in OpenRefine are temporary - that is, if you remove th

You can sort on multiple columns at the same time by adding another sorted column (in the same way).

> \##Separator
> Do not rush these last two sentences. Repeat them slowly after a pause and allow learners to explore a moment, even see the growing undo list
> {:.instructor}
:::::::::::::::::::::::::::::::::::::::: instructor

## Sorting and Reorder Rows Permanently

Do not rush these last two sentences. Repeat them slowly after a pause and allow learners to explore how sorting works for a moment.

Although the "Undo/Redo" tab is not introduced until episode 9, it may be worth noting that applying a sort does not count as a change to the data because removing the sort will restore the data to its original order. However, once you select "Reorder Rows Permanently" this does count as a data change and adds an entry to the Undo/Redo history.

::::::::::::::::::::::::::::::::::::::::::::::::::

:::::::::::::::::::::::::::::::::::::::: keypoints

Expand All @@ -51,5 +57,3 @@ You can sort on multiple columns at the same time by adding another sorted colum
- The original order of rows in OpenRefine is maintained during a sort until you use the option to Reorder Rows Permanently from the Sort drop-down menu

::::::::::::::::::::::::::::::::::::::::::::::::::


22 changes: 14 additions & 8 deletions episodes/11-using-arrays-transformations.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,13 @@ exercises: 15

The following example is chosen to demonstrate how to go from a list with duplicated values to a list with each value just once by using an array in a transformation.

> ## Caution
>
> Ask the students what transformation means to them currently. Many may only know it from Excel to convert columns into rows or vice versa. Discuss how in OpenRefine, transformation is specifically the working window--these values are neither stored nor displayed in the cells or output.
> {:.instructor}
:::::::::::::::::::::::::::::::::::::::: instructor

## Different meanings of 'transformation'

Ask the students what transformation means to them currently. Many may only know it from Excel to convert columns into rows or vice versa. Discuss how in OpenRefine, transformation is specifically the working window--these values are neither stored nor displayed in the cells or output.

::::::::::::::::::::::::::::::::::::::::::::::::::

It does this using a function called uniques() which can be used to remove duplicates from an array. In this example we start with a list of subject words:

Expand Down Expand Up @@ -98,10 +101,13 @@ value.split("|").uniques().join("|")

Taking the same example again, this would result in a string with the subjects in alphabetical order, listed with commas between each subject.

> ## Caution
>
> Recall previous discussion of dangers of changing separators--especially commas. Possible question to pose: Which subject would be broken if a hyphen were used as a separator?
> {:.instructor}
::::::::::::::::::::::::::::::::::::::: instructor

## Recap on best practice for separators

Recall previous discussion of dangers of changing separators and ensuring you avoid using a separator character that is already used in the text. A possible question to pose to learners could be: Which subject would be broken if a hyphen were used as a separator?

::::::::::::::::::::::::::::::::::::::::::::::::::

::::::::::::::::::::::::::::::::::::::: checklist

Expand Down

0 comments on commit a7e7c8a

Please sign in to comment.