Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate underlying framework from TensorFlow to PyTorch #1266

Merged
merged 236 commits into from
Nov 3, 2021
Merged

Conversation

tgaddair
Copy link
Collaborator

Continuation of #1155.

tgaddair and others added 26 commits October 28, 2021 14:12
* refactor: correct data type for specifying input_size

* refactor: replace tf apis w/ torch, add input_size specification

* refactor: tensor manipulation and testing for expected shape

* refactor: partial working with attention reducer

* code: clean out tf commented out code

* debug: placehoder code

* refactor: use proper Torch container for dependency_reducers

* refactor: back-off special handling of attention reducer
This will need to revaluated becuase this reducer has
parameters that need special handling to support
attention reducer initialization.
* Updated GitHub actions

* Fixed check

* Updated AverageMeter to MeanMetric

* Lower bound torchmetrics
…assing test_experiment. (#1443)

* WIP for test_experiment.

* Fixes to StackedTransformer, image_utils, sequence_feature, and how images are read in test_experiment to address all test failures.

* Minor cleanup and test parameterization.
* refactor: removed deprecated sequence encoder test
mark decoder related test to be reworked at later
date.

* test: working version of sequence input feature test
* Remove keras/tf references from test_collect integration tests and detach tensors before serializing to numpy (required by torch).

* Fix (and simplify) implementation for collect_weights.

* Cleanup.

* Fix _get_layers() helper.

* Use torch.allclose() to compare tensor values.

* Simplify test_collect_activations.

* Use named children instead of layers.

* Remove import logging
- Removed per module regularization
- Added regularization at the ECD-model level
- Updated regularization tests to check for end-to-end regularization.
- Updated MeanMetric to return a tensor of dimension torch.Size([])
…raining as an additional parameter everywhere. (#1459)
@tgaddair tgaddair marked this pull request as ready for review November 3, 2021 05:19
@tgaddair tgaddair merged commit fe9524a into master Nov 3, 2021
@tgaddair tgaddair deleted the tf-to-torch branch November 3, 2021 05:23
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.

10 participants