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

Don't force QLlama to have >2 input dims @Jeadie #320

Merged
merged 3 commits into from
May 16, 2024

Conversation

Jeadie
Copy link
Contributor

@Jeadie Jeadie commented May 16, 2024

Fix for #319

Copy link

Code Metrics Report
  ===============================================================================
 Language            Files        Lines         Code     Comments       Blanks
===============================================================================
 Dockerfile              1           34           25            0            9
 Happy                   1          442          369            0           73
 JSON                    5            9            9            0            0
 Python                 21          741          622           21           98
 TOML                   16          420          380            1           39
-------------------------------------------------------------------------------
 Jupyter Notebooks       1            0            0            0            0
 |- Markdown             1           60           30           22            8
 |- Python               1           96           87            1            8
 (Total)                            156          117           23           16
-------------------------------------------------------------------------------
 Markdown               16         1026            0          758          268
 |- BASH                 6          205          192            0           13
 |- Python               6          121          110            0           11
 |- Rust                 3          185          172            9            4
 (Total)                           1537          474          767          296
-------------------------------------------------------------------------------
 Rust                   81        26574        24448          337         1789
 |- Markdown            38          375            0          370            5
 (Total)                          26949        24448          707         1794
===============================================================================
 Total                 143        29246        25853         1117         2276
===============================================================================
  

@EricLBuehler
Copy link
Owner

EricLBuehler commented May 16, 2024

@Jeadie, it looks like formating and linting are failing. Thanks for fixing this, I'll merge as soon as those pass.

I think #317 will have a permanent fix.

@Jeadie
Copy link
Contributor Author

Jeadie commented May 16, 2024

Fixed

@EricLBuehler EricLBuehler merged commit d78313c into EricLBuehler:master May 16, 2024
11 checks passed
@EricLBuehler
Copy link
Owner

EricLBuehler commented May 16, 2024

Thank you!

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

3 participants