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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix command for installing pre-commit hooks. #229

Merged
merged 1 commit into from
Oct 30, 2023

Conversation

carschno
Copy link
Collaborator

@carschno carschno commented Oct 30, 2023

make pre-commit-install does not exist any longer, make install-dev seems to be the right choice now.

Type of Change

  • 馃摎 Examples / docs / tutorials / dependencies update

Checklist

  • I've read the CODE_OF_CONDUCT.md document.
  • I've read the CONTRIBUTING.md guide.
  • I've successfully run the style checks using make fix-style.
  • I've written tests for all new methods and classes that I created and successfully ran make test.
  • I've written the docstring in Google format for all the methods and classes that I used.

`make pre-commit-install` does not exist any longer, `make install-dev` seems to be the right choice now.
@gsarti gsarti merged commit 8183185 into inseq-team:main Oct 30, 2023
4 checks passed
gsarti added a commit that referenced this pull request Oct 30, 2023
* origin/main:
  Fix command for installing pre-commit hooks. (#229)
@carschno carschno deleted the patch-1 branch October 30, 2023 10:42
gsarti added a commit that referenced this pull request Oct 30, 2023
* origin/main:
  Attributed behavior for contrastive step functions (#228)
  Fix command for installing pre-commit hooks. (#229)
  Remove `max_input_length` from `model.encode` (#227)
  Migrate to `ruff format` (#225)
  Remove contrast_target_prefixes from contrastive step functions (#224)
  Step functions fixes, add `in_context_pvi` (#223)
  Format fixes, add Attanasio et al. (2023) to readme
  Add Sequential IG method (#222)
  Fix LIME and Occlusion outputs (#220)
  Update citation information
  Bump dependencies
  Add end_pos for contrast_targets_alignments
  Fix dummy output viz in console
  Minor fixes
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.

None yet

2 participants