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

example runs on windows, fixed dependencies and API_TOKEN handling #19

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bmerkle
Copy link

@bmerkle bmerkle commented Dec 11, 2024

example runs on windows, fixed dependencies and API_TOKEN handling

  • added missing pip dependencies
  • changed pip to %pip and not -q (output)
  • using HUGGINGFACE_HUB_TOKEN and commandline
  • changed linux ls command to python portable version
  • convert int32 tensors to int64

- changed pip to %pip and not -q (output)
- using HUGGINGFACE_HUB_TOKEN and commandline
- changed linux ls command to python portable version
- convert int32 tensors to int64
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@bmerkle
Copy link
Author

bmerkle commented Dec 11, 2024

@merveenoyan please note that my githook removed the notebook output and I also cleared output before commit
Not sure how you want to handling this in your project... (some developers want to commit with notebook output)

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.

1 participant