Full Implementation of the Captum-Optim Module#1545
Open
ProGamerGov wants to merge 908 commits intometa-pytorch:masterfrom
Open
Full Implementation of the Captum-Optim Module#1545ProGamerGov wants to merge 908 commits intometa-pytorch:masterfrom
ProGamerGov wants to merge 908 commits intometa-pytorch:masterfrom
Conversation
…o ImageTensor (meta-pytorch#839) * Add better colorspace support, image grids, & user agent to ImageTensor * Added color space support to `save_tensor_as_image` & `ImageTensor.export`. * Added image grid creation support to `ImageTensor.export` , `ImageTensor.show` , `show` & `save_tensor_as_image` via a new `make_grid_image` function. * Added user agent to `ImageTensor.open` as sites like Wikipedia require user agents. * Add description to make_grid_image tests * `nrow` -> `images_per_row` * Remove test description It's no longer required now that the images_per_row variable was renamed. * Add missing tests * Fix test
* Ensure testing coverage is as high as possible. * Simplified code with new `rmodule_op` function. * Removed the NumPy import from loss testing.
* Fix duplicated target bug * Fix duplicated target bug in `sum_loss_list` & `collect_activations` * Add ToDo comment for target handling
* Wrap all remaining `torch.__version__` calls in `version.parse`. * Remove unused version check in `typing.py`. * Expose `MaxPool2dRelaxed` to users so that tutorials using it work. * Expose `dataset` module to users. * Fixed `show` & `save_tensor_as_image` docs.
* Improve efficiency of the `FacetLoss` objective.
* Add missing docs. * Fix errors in existing docs.
Contributor
|
Hi @ProGamerGov, I wanted to leave a quick comment - thank you for bringing this project back to light! This is a big and exciting contribution, but will definitely take us some time to review it thoroughly. It's busy right now, but I will discuss this with the team and plan how we will deliver a timely review. We appreciate your patience and contributions. |
Contributor
Author
|
@craymichael Awesome, thank you for the update! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
So this project got left on the back burner for a while but I've brought it back. The original plan was to either merge it into Captum or spin it out as a separate library under the PyTorch umbrella. The project was almost ready for release before it was paused, and I have since pushed it closer to full release quality.
The ReadMe for the module can be found here: https://github.com/ProGamerGov/captum/tree/master-optim/captum/optim
This PR contains all the following PR merged together:
#983
#968
#967
#966
#965
#961
#960
#957
#955
#955
#951
#949
#946
#945
#943
#927
#828
The module was developed based on this initial starting PR: #412
The last PR reviewed by the Captum team, was this one: #935
As part of this project, numerous improvements were made to Captum and the libraries that it is dependent on:
CONTRIBUTING.mdpytorch/pytorch#83536, Add docstring type guidelines for list & tuple toCONTRIBUTING.mdpytorch/pytorch#83634