search

darsnack / FluxModels.jl

Standard (pre-trained) ML models written in Flux.jl
MIT License
3 stars 5 forks source link

added untrained model Inception v3 #19

Closed rishabhvarshney14 closed 3 years ago

rishabhvarshney14 commented 3 years ago

I have followed pytorch implementation of Inception3 (here).

Please let me know if this is not the correct way. Thank you

darsnack commented 3 years ago

Thanks for this! At first glance things took good. I will provide a more detailed review later. But can you add this to this PR instead?

How did you find this repo? I want to make sure future additions go to that PR.

rishabhvarshney14 commented 3 years ago

I found this from here. I did not the last comment sorry about that.

I will make a pr to that. Just a quick question should I make a pr in your fork or to the original one? I am not sure how I can make a pr to another pr. Thank You.

darsnack commented 3 years ago

I added you as a collaborator to my fork. You can make commits directly to the branch in the PR.

darsnack commented 3 years ago

Great to see someone seeing our tracker and contributing!