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

OpenRefine lesson updates #3205

Closed

Conversation

wetneb
Copy link

@wetneb wetneb commented Mar 14, 2024

Hello, thank you so much for maintaining this fantastic OpenRefine lesson, which is still a reference.

I went through the tutorial and fixed a few inconsistencies I found, mostly updating for changes that we have made in OpenRefine itself.

I haven't touched the Spanish and Portuguese translations because I don't master those languages but I could update the links already if that's welcome.

Checklist

  • Assign yourself in the "Assignees" menu
  • Add the appropriate "Label"
  • If this PR closes an Issue, add the phrase Closes #ISSUENUMBER to your summary above
  • Ensure the status checks pass: if you have difficulty fixing build errors, please contact our Publishing Manager @anisa-hawes
  • Check the Netlify Preview: navigate to netlify/ph-preview/deploy-preview and click 'details' (at right)
  • Assign at least one individual or team to "Reviewers"
  • if the text needs to be translated, please follow the translation request guidelines, then assign the relevant language team(s) as "Reviewers" and tag both the team as well as the managing editor in your PR.

I went through the tutorial and fixed a few inconsistencies I found.
meaning to *OpenRefine*. Additionally, select the 'Parse cell text into
numbers, dates, ...' checkbox to let OpenRefine automatically detect
meaning to *OpenRefine*. Additionally, select the 'Attempt to parse cell
text into numbers' checkbox to let OpenRefine automatically detect
Copy link
Author

Choose a reason for hiding this comment

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

This reflects an update that we made in the tool, changing the name of this option to match its actual behavior (it doesn't actually parse dates).

@@ -292,7 +292,7 @@ cells**, **Join multi-valued cells**, **OK**. Choose the pipe character (|) as a
separator. The rows now look like before, with a multi-valued Categories
field.

### Applying ad-hoc transformations through the use of regular expressions
### Applying ad-hoc transformations through the use of GREL expressions
Copy link
Author

Choose a reason for hiding this comment

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

This section does not use regular expressions at all, only GREL expressions, so I think it's worth avoiding this confusion.

@@ -379,7 +379,7 @@ Once your data has been cleaned, you can take the next step and explore
other exciting features of *OpenRefine*. The user community of
*OpenRefine* has developed two particularly interesting extensions which
allow you to link your data to data that has already been published on
the Web. The [RDF Refine extension][] transforms plaintext keywords into
the Web. The [RDF Transform extension][] transforms plaintext keywords into
Copy link
Author

Choose a reason for hiding this comment

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

The RDF Transform is the successor of this extension. It should work with recent versions of OpenRefine.

@wetneb wetneb changed the title Openrefine lesson updates OpenRefine lesson updates Mar 14, 2024
@anisa-hawes
Copy link
Contributor

Hello Antonin Delpeuch @wetneb.

Many thanks for making contact, and offering to support an update of the lesson cleaning-data-with-openrefine and its subsequent translations.

We actually prefer outside contributors not to create direct Pull Requests from their forks of our repo. Updates to our site are so regular that knots in forks are very frequent.

I'll transfer this to an Issue, create a branch on our side and replicate the changes you suggest. I'll then seek the support of our ES and PT teams to update their translations, and ask our Managing Editors to review and approve these changes.

Thank you for sharing your expertise ✨

@anisa-hawes
Copy link
Contributor

The updates advised by @wetneb are now being prepared in #3216.

Thank you again, Antonin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants