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

Installable tools #86

Merged
merged 3 commits into from
Dec 16, 2024
Merged

Installable tools #86

merged 3 commits into from
Dec 16, 2024

Conversation

Waino
Copy link
Collaborator

@Waino Waino commented Dec 2, 2024

Three scripts formerly in the tools directory have been promoted to mammoth.bin and made into installable scripts.

  • mammoth_config_config: Your trusted config swiss army knife.
  • mammoth_iterate_tasks: Helper for iterating over all tasks in a config.
  • mammoth_generate_synth_data: Toy data used in quickstart.

@Waino Waino requested a review from jrvc December 2, 2024 14:11
@Waino Waino force-pushed the chore/update-documentation branch from 41d1573 to d32140e Compare December 9, 2024 10:54
Three scripts formerly in the tools directory have been promoted to
mammoth.bin and made into installable scripts.
- mammoth_config_config: Your trusted config swiss army knife.
- mammoth_iterate_tasks: Helper for iterating over all tasks in a
  config.
- mammoth_generate_synth_data: Toy data used in quickstart.
@Waino Waino force-pushed the installable-tools branch from a19b967 to 6271534 Compare December 9, 2024 10:55
@Waino Waino requested a review from TimotheeMickus December 9, 2024 12:19
Copy link
Collaborator

@TimotheeMickus TimotheeMickus left a comment

Choose a reason for hiding this comment

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

LGTM. I assume this has been tested.

@Waino Waino mentioned this pull request Dec 9, 2024
@Waino
Copy link
Collaborator Author

Waino commented Dec 9, 2024

LGTM. I assume this has been tested.

The tools have been tested, they work after installing from the repo with pip install -e .

In the documentation, we use pip install mammoth-nlp. This doesn't work yet, but will work once a new version has been released.

@Waino Waino merged commit eca322f into chore/update-documentation Dec 16, 2024
2 checks passed
@Waino Waino deleted the installable-tools branch December 16, 2024 08:20
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.

2 participants