Skip to content

remove a redundant line#14

Open
sytuannguyen wants to merge 1 commit intochavinlo:mainfrom
sytuannguyen:patch-1
Open

remove a redundant line#14
sytuannguyen wants to merge 1 commit intochavinlo:mainfrom
sytuannguyen:patch-1

Conversation

@sytuannguyen
Copy link
Copy Markdown

No description provided.

@chavinlo
Copy link
Copy Markdown
Owner

chavinlo commented Nov 30, 2024 via email

@larpedd
Copy link
Copy Markdown

larpedd commented Nov 30, 2024

why would we import torch 2 times when we can import it once

@chavinlo
Copy link
Copy Markdown
Owner

chavinlo commented Nov 30, 2024 via email

@larpedd
Copy link
Copy Markdown

larpedd commented Nov 30, 2024

The code is most likely outdated and meta released the official trainer a year ago...

On Sat, Nov 30, 2024, 3:08 PM bulieme0 @.> wrote: why would we import torch 2 times when we can import it once — Reply to this email directly, view it on GitHub <#14 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUNQL6ZEVDFWKQUEFFXFS7D2DILKBAVCNFSM6AAAAABKYDH2P6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBZGE4DKNZUGQ . You are receiving this because you commented.Message ID: @.>

well ok, at least it worked 🤷

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.

3 participants