(Prepare Dataset) List indices must be intergers or slices, not dict #335

New Issue

MrMustachio43 · 2023-08-22T20:36:25Z

MrMustachio43 commented

2023-08-22 20:36:25 +00:00

I don't know why this isn't working or what's wrong.
First two images is the error, then the last is the files I'm using to clone.

I don't know why this isn't working or what's wrong. First two images is the error, then the last is the files I'm using to clone.

Screenshot 2023-08-22 212550.png

103 KiB

Screenshot 2023-08-22 212628.png

69 KiB

Screenshot 2023-08-22 213408.png

3.7 KiB

👍 1

mrq commented

2023-08-23 14:54:43 +00:00

What's odd is that I did use the openai/whisper backend the other day with no apparent issues, but regardless I'll take a look at it when I get a chance.

What's odd is that I did use the `openai/whisper` backend the other day with no apparent issues, but regardless I'll take a look at it when I get a chance.

👍 1

mrq referenced this issue

2023-08-23 21:01:56 +00:00

Vall-E Backend Training: "list indices must be integers or slices, not dict" #339

mrq commented

2023-08-23 21:03:52 +00:00

I haven't gotten a chance to actually look at it, but from the other issue I referenced to, you might need to chick Slice Segments, and set your offsets to -0.05 and 0.2 for safety. I think I did actually get an issue when not having slicing enabled and I forgot to look into it the other day.

But again when I get a chance to actually look and experiment and debug, I will.

I haven't gotten a chance to actually look at it, but from the other issue I referenced to, you *might* need to chick `Slice Segments`, and set your offsets to `-0.05` and `0.2` for safety. I think I did actually get an issue when not having slicing enabled and I forgot to look into it the other day. But again when I get a chance to actually look and experiment and debug, I will.

MrMustachio43 commented

2023-08-23 21:39:41 +00:00

The fix worked perfectly. Odd though, as I never needed to before haha. Thank you for your assistance :)

mrq commented

2023-08-23 21:46:02 +00:00

Odd though, as I never needed to before haha.

I'm very, very certain it was a regression I've introduced when committing the changes from my training rig back to this repo (commit 72a38ff2fc), as I'm very very messy and forgetful when I do work on that machine.

Although, looking at it now, I think I see the issue. Should be fixed in commit 29290f574e. I haven't gotten a chance to test it, but running the logic in my head, it looks sound.

> Odd though, as I never needed to before haha. I'm very, very certain it was a regression I've introduced when committing the changes from my training rig back to this repo (commit 72a38ff2fc6aeb8b8184bb8e8103497a37ce151d), as I'm very very messy and forgetful when I do work on that machine. Although, looking at it now, I think I see the issue. Should be fixed in commit 29290f574eb2f8aa9c2cf19b9aaf131a2fd1c3ff. I haven't gotten a chance to test it, but running the logic in my head, it looks sound.

MrMustachio43 commented

2023-08-24 14:15:01 +00:00

Yes, it's been fixed! I shall close this issue, thank you :)

MrMustachio43 closed this issue

2023-08-24 14:15:01 +00:00

MrMustachio43 commented

2023-08-24 14:15:03 +00:00

Yes, it's been fixed! I shall close this issue, thank you :)

Sign in to join this conversation.