Owen Vallis

Results 78 comments of Owen Vallis

So I've been working on this and the TFRecordDatasetSampler is a little tricky. Ideally we would like to pass a set of TFRecord datasets to [choose_from_datasets](https://www.tensorflow.org/api_docs/python/tf/data/Dataset#choose_from_datasets), but this turns out...

Support for setting the number of repeats was just added in PR #196

Thanks for adding this! I'll review the PR shortly.

Thanks for offering to work on the PR! This loss is a little older and it comes before Triplet-Loss and the more recent variations on other metric losses. What would...

Sounds good, thanks. Let's add the PR to the development branch and then we can merge it in to master along with the other recent updates.

@dewball345 We've been pushing a bunch of updates to the branches, so looks like you'll need to rebase the changes into your branch

We just pushed 0.15 to the main branch and removed the augmentation arg from all the tfsim.architectures. This is a breaking change but should resolve some of these issues.

Hi @yonigottesman, we're actually working on this right now and plan to provide benchmarks for all the losses this quarter. Additionally, I've personally reproduced (or as close as I could)...

Hi @Wansh619, let me look into it and see how we might either add it to the existing Contrastive model or if we might need a separate model. At first...

Typically we recommend that you fork the development branch and share a design proposal in the issue or submit a PR. See issue #92 for a good example that implemented...