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

Fixed int * float multiplication error #61

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Toradus
Copy link

@Toradus Toradus commented Mar 10, 2024

Fixed len call throwing error when attribute "sample_weights" is provided. Since "num_transitions" is an int value, multiplying it with a float threw an error. I simply switched it around, since "sample_weights" is either int / float and "num_transitions" is always int.

Fixed len call throwing error when attribute "sample_weights" is provided. Since "num_transitions" is an int value, multiplying it with a float threw an error. I simply switched it around, since "sample_weights" is either int / float and "num_transitions" is always int.
Toradus added a commit to mbreuss/oxe_torch_dataloader that referenced this pull request Mar 10, 2024
WenchangGaoT pushed a commit to WenchangGaoT/octo1 that referenced this pull request May 10, 2024
* adds gradient clipping

* add bfloat16 to optim
Toradus and others added 2 commits May 20, 2024 16:02
…ty to load from arbitrary data_dirs (adjustable in OXE_DATASET_CONFIGS now)
@Toradus
Copy link
Author

Toradus commented May 20, 2024

Further added the possibility to load OXE data from arbitrary data_paths. Simply add a "data_path" field to the OXE_DATASET_CONFIGS dataset.
Can also be a local path ("~" will be converted to home_dir), to allow self made tfds builds.

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