Skip to content
This repository was archived by the owner on Jun 4, 2025. It is now read-only.

Fix input/output shapes for exported samples#5

Merged
natuan merged 1 commit intomasterfrom
fix_exported_input_shapes
Jul 23, 2021
Merged

Fix input/output shapes for exported samples#5
natuan merged 1 commit intomasterfrom
fix_exported_input_shapes

Conversation

@natuan
Copy link
Copy Markdown

@natuan natuan commented Jul 23, 2021

Squeeze the export input/output samples to have shapes of (N,) instead of (1, N)

@natuan natuan requested review from a team, bfineran, markurtz and mgoin July 23, 2021 01:53
@natuan natuan merged commit 71f0032 into master Jul 23, 2021
@natuan natuan deleted the fix_exported_input_shapes branch July 23, 2021 13:18
bfineran pushed a commit that referenced this pull request Jun 5, 2024
* Cohere Model Release (#1)

Cohere Model Release

* Remove unnecessary files and code (#2)

Some cleanup

* Delete cohere-model directory (#3)

* Make Fix (#5)

* Pr fixes (#6)

* fixes for pr

* pr fixes for the format

* pr fixes for the format

* src/transformers/models/auto/tokenization_auto.py

* Tokenizer test (#8)

* tokenizer test

* format fix

* Adding Docs and other minor changes (#7)

* Add modeling tests (#9)

* Smol Fix (#11)

* tokenization tests are fixed

* format fixes

* fix pr doc tests

* fix pr doc tests

* fix pr doc tests

* fix pr style check

* small changes in cohere.md

* FIX: Address final comments for transformers integration (#13)

* fix modeling final nits and add proper test file

* for now leave empty tests

* add integration test

* push new test

* fix modeling cohere (#14)

* Update chat templates to use the new API (#15)

---------

Co-authored-by: ahmetustun <ahmetustun89@gmail.com>
Co-authored-by: Younes Belkada <49240599+younesbelkada@users.noreply.github.com>
Co-authored-by: Matt <Rocketknight1@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants