NTM-One-Shot-TF icon indicating copy to clipboard operation
NTM-One-Shot-TF copied to clipboard

Generalized code + Saving options

Open dhruvramani opened this issue 6 years ago • 2 comments

dhruvramani avatar Jun 14 '18 22:06 dhruvramani

Hi Dhruv!

Thanks for the input. Can you tell me why did you add input_ph and target_ph definitions as a part of the commit? I don't find any actual usage of them in the code. Beside them the code seem to be fine.

hmishra2250 avatar Jul 09 '18 08:07 hmishra2250

Hey!

I had a modified a little bit of the code for personal usage in a separate repo, maybe that got carried forward. I’ll modify it and put another commit. Thank you! Dhruv

On Mon, Jul 9, 2018 at 2:14 PM Himadri Mishra [email protected] wrote:

Hi Dhruv!

Thanks for the input. Can you tell me why did you add input_ph and target_ph definitions as a part of the commit? I don't find any actual usage of them in the code. Beside them the code seem to be fine.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/hmishra2250/NTM-One-Shot-TF/pull/8#issuecomment-403405384, or mute the thread https://github.com/notifications/unsubscribe-auth/AIGeKr0lv3fyIncnWOk8_8SB5N4UVFMhks5uExfVgaJpZM4UovL6 .

dhruvramani avatar Jul 10 '18 07:07 dhruvramani