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

Add snake casing #626

Closed
wants to merge 3 commits into from
Closed

Conversation

bjhargrave
Copy link
Contributor

@bjhargrave bjhargrave commented Mar 25, 2024

Describe the contribution to the taxonomy

Inspired by #106, added new skills to convert text to

  • snake case

Input given at the prompt

Convert this text into its Camel case form: congratulation interventionism
blah psychoanalyse idiots chromatics evangelistically interchangeability intercommunicating mickle

Response that was received

A correct answer but with a lot of explanation:

| Snake case, also known as "lowercase" or "kebab-case", is a style of writing where   │
│ all words in a phrase are written in lowercase, with the space removed between the   │
│ words and underscores inserted between each word. This format is commonly used in    │
│ programming languages, file systems, and database schema to make the names of        │
│ variables, functions, directories, and tables more consistent and easier to parse.   │
│ In the given example,                                                                │
│ "CongratulationInterventionismBlahPsychoanalyseIdiotsChromaticsEvangelisticallyInter │
│ changeabilityIntercommunicatingMickle" would be in snake case as                     │
│ "congratulation_interventionism_blah_psychoanalyse_idiots_chromatics_evangelisticall │
│ y_interchangeability_intercommunicating_mickle". Here's how the conversion can be    │
│ done for the given text:                                                             │
│                                                                                      │
│ 1. Lowercase all the letters in the phrase.                                          │
│ 2. Remove the spaces between the words.                                              │
│ 3. Insert underscores between each word.                                             │
│                                                                                      │
│ Applying these steps to the given text, we get                                       │
│ "congratulation_interventionism_blah_psychoanalyse_idiots_chromatics_evangelisticall │
│ y_interchangeability_intercommunicating_mickle". This format is useful for making    │
│ the names of variables, functions, directories, and tables more consistent and       │
│ easier to parse, especially in programming languages and file systems where the      │
│ length of the names is limited. 

Response that is now received instead

TODO: lab generate says Nothing to generate. Exiting.

  ...

Contribution checklist

TODO

  • tested contribution with lab generate
  • lab generate does not produce any warnings or errors
  • all commits are signed off (DCO)
  • the qna.yaml file was linted

@bjhargrave bjhargrave added skill (Auto labeled) community-build-ready Triage Team has signed off for synthetic data generation skill category: encoding decoding labels Mar 25, 2024
@bjhargrave bjhargrave requested a review from a team as a code owner March 25, 2024 14:36
@github-actions github-actions bot added the triage-needed (Auto labeled) skill is ready to be triaged label Mar 25, 2024
@bjhargrave bjhargrave removed the triage-needed (Auto labeled) skill is ready to be triaged label Mar 25, 2024
@github-actions github-actions bot added the triage-needed (Auto labeled) skill is ready to be triaged label Mar 25, 2024
@bjhargrave
Copy link
Contributor Author

Resurrect snake case skill in #224 from @juandspy. Testing has shown that this skill did improve the full model performance.

@bjhargrave bjhargrave removed the triage-needed (Auto labeled) skill is ready to be triaged label Mar 25, 2024
@bjhargrave bjhargrave changed the title Add text casing Add snake casing Mar 25, 2024
@github-actions github-actions bot added the triage-needed (Auto labeled) skill is ready to be triaged label Apr 10, 2024
@bjhargrave
Copy link
Contributor Author

@juandspy Can you update this PR to have at least 5 diverse examples?

@bjhargrave bjhargrave added triage-requested-changes skill has been reviewed; changes requested from contributor and removed triage-needed (Auto labeled) skill is ready to be triaged community-build-ready Triage Team has signed off for synthetic data generation labels Apr 10, 2024
@github-actions github-actions bot added the triage-needed (Auto labeled) skill is ready to be triaged label Apr 11, 2024
@juandspy
Copy link

@juandspy Can you update this PR to have at least 5 diverse examples?

Done. Is this enough?

@bjhargrave
Copy link
Contributor Author

@instruct-lab-bot precheck

Copy link

Beep, boop 🤖 Generating test data for your pull request.

This will take several minutes...

Your job ID is 112.

Copy link

Beep, boop 🤖 The test data has been generated for job ID: 112!

Find your results here.

@bjhargrave
Copy link
Contributor Author

@instruct-lab-bot generate

Copy link

Beep, boop 🤖 Generating test data for your pull request.

This will take several minutes...

Your job ID is 113.

Copy link

Beep, boop 🤖 The test data has been generated for job ID: 113 using the model sdg service backend!

Find your results here.

@jjasghar jjasghar closed this Apr 19, 2024
@n1hility
Copy link
Member

Thank you for your contribution to InstructLab! Unfortunately, once a Github repo is made public, all open PRs are automatically closed since they link against a private repo. We have detected that your PR might have been one of the ones affected by this change. If you are still interested in contributing your improvement, please fill out the following short form by no later than May 3rd, and we will get back to you with the additional steps necessary once we have had time to assess the PRs of those still interested:

https://forms.gle/V7SrPPMZDo6iGDYu8

@bjhargrave bjhargrave mentioned this pull request Apr 22, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skill (Auto labeled) triage-needed (Auto labeled) skill is ready to be triaged triage-requested-changes skill has been reviewed; changes requested from contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants