-
Notifications
You must be signed in to change notification settings - Fork 18
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
A major (performance) update on the submodule: srl-zoo #26
Closed
Closed
Changes from all commits
Commits
Show all changes
82 commits
Select commit
Hold shift + click to select a range
6ecfdbc
fix issue: image rotated by 90 degrees
ncble e37db43
default image processing should be 'tf' mode
ncble d11611f
fix preprocessing mode; add gan.py (not finished)
ncble 9489bd2
add img_shape to input; add gan.py (not finished yet)
ncble aa24811
fix issue (image rotated); img_shape
ncble 15323ed
30% training speed-up (for 'tf' preprocessing mode) by modifying few …
ncble 9f7e992
update all CNN-based models for any image shape (input)
ncble 7d5fc1e
fix plt slow down issue
ncble 30f2850
new features in plotting/representation_plot.py: plotImage (support m…
ncble ac9f59c
improve data_loader.py git add .!
ncble 0027c26
add monitor mode ('pbar' or 'loss')
ncble d35b908
update data_loader.py for test mode
ncble 11762a6
fix plotRepre; replace DISPLAY_PLOT by fidir as args (savefig or not)
ncble 096a15a
fix plot delay problem
ncble 1a0dba7
fix train.py --log-folder bug
ncble ef30fac
huge update: re-modulize the srl-zoo (module/model part)
ncble 676be83
tempo commit pb to be fixed: import error
ncble 70745e0
remove getStates (ambiguous) from AutoEncoder model
ncble 783ca2f
fix absolute/relative import issue
ncble 95316d4
update VAE to new format
ncble 00a177e
update srl module (forward/inverse loss)
ncble b645c83
fix image shape error
ncble cf8cfdc
add BaseTrainer
ncble 78d7a39
First release of ASRL
ncble 803cfda
First release of ASRL
ncble a601b42
first exp with gan
ncble 057f03f
temporary update
ncble 6b8b764
add CUDA VISIBLE DEVICES control
ncble 35d38b9
fix plotRepresentation bug
ncble 3915458
clean models
ncble 893f0a3
first result with adv_srl
ncble f3989e7
remove N_CHANNELS and getNChannels from preprocessing/preprocess.py
ncble 9314a85
default CUDA visible devices to 0
ncble 7c8b9aa
fix reward loss
ncble b967b2a
merge adv_srl to master
ncble c08b44f
merge adv_srl2 to master
ncble 8c61f2e
merge adv_srl2 to master
ncble 33fb207
add BasicTrainer for inverse/forward models
ncble b06325c
clean files
ncble 4d1fe1e
maintain consistency
ncble a467eb2
enable gan in enjoy_latent.py
ncble 6717bf4
fix GTC error
ncble 0852846
remove split module, integrate it into SRLModules
ncble eb36b52
autopep8
ncble 004fa65
division by zero pb (in inverse model init)
ncble b84bd06
change Reward/Inverse models
ncble a641ce0
clean comments
ncble d530bb1
new reward model
ncble 061561f
self-supervised srl split
ncble f704752
update coeff
ncble 4c0b9da
fix autoencoder activation
ncble 9337042
fix bug (autoencoder without activation): add activation tanh
ncble db583e8
Merge branch 'master' of https://github.com/ncble/srl-zoo
ncble 764dd57
fix bug: add tanh activation to vae, autoencoder
ncble b4d20b9
fix bug (add activation tanh) in autoencoder and vae
ncble 7484656
inverse label of rewardModel (-1,0,1) to (2,1,0) ignore 2
ncble 8cc599c
pass loss_weights to args
ncble 779ec54
fix issue #24 (VAE couldn't reconstruct image)
ncble 07600ac
(hack) enable partially no-split with inverse/forward
ncble 66ab7f2
quick fix
ncble 2599f3d
integrate gan to losses; add reward2 for split
ncble 682262e
tempo commit
ncble 9c7b696
update reward2 compatibility
ncble e55fb28
refactor gan
ncble 0067dfe
re-modular gan
ncble 2a8d517
(partial) resnet gan
ncble 8380c05
update gan.py
ncble 32e4a84
merge debug2_tempo
ncble d3c6710
quick fix (pytorch dataloader leaking FDs); move spcls_loss out of re…
ncble 61ffdd1
quick fix: num_classes for spcls_loss
ncble d0f91a3
fix spcls num_classes
ncble a3a53f2
gtc with labyrinth to be fixed
ncble b0c831e
add argument monitorGTC
ncble a041b34
clean ./preprocessing.data_loader.py
ncble 0a4d2fd
quick fix for spcls num_classes (we should use new class design)
ncble 7af3fd5
fix int64 can't save in json
ncble 8140ecc
add supervised learning ('supervised') to losses
ncble 20afc87
fix l2 loss in 'supervised' setting
ncble 341671f
clean code
ncble 0b62a6a
minor change to learner.py
ncble 28188eb
use png instead of jpg/jpeg extension for saving images
ncble cc209a2
clean description
ncble File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
from .models import CustomCNN | ||
from .modules import SRLModules, SRLModulesSplit | ||
from .priors import SRLConvolutionalNetwork, SRLDenseNetwork, SRLCustomCNN, Discriminator | ||
from .supervised import ConvolutionalNetwork, DenseNetwork, CustomCNN | ||
from .base_models import CustomCNN, UNet | ||
from .modules import SRLModules | ||
from .supervised import ConvolutionalNetwork, DenseNetwork | ||
from .autoencoders import LinearAutoEncoder, DenseAutoEncoder, CNNAutoEncoder | ||
from .vae import DenseVAE, CNNVAE | ||
|
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add docstrings for the loss methods in the following lines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, please reformat the file (to conform with PEP8 style).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. I used vscode autopep8.