Updated semisupervised net + unit test for semi supervised - Githubissues

choderalab / pinot

Probabilistic Inference for NOvel Therapeutics

MIT License

15 stars 2 forks source link

Updated semisupervised net + unit test for semi supervised #81

Closed dnguyen1196 closed 4 years ago

dnguyen1196 commented 4 years ago

SemiSupeprvised will now take output_regressor and decoder as classes to be initialized as objects

class SemiSupervisedNet(pinot.Net):
    def __init__(self, representation,
            output_regressor=NeuralNetworkRegressor,
            decoder=DecoderNetwork,
            unsup_scale=1.,
            embedding_dim=64,
            generative_hidden_dim=64,
        ):

lgtm-com[bot] commented 4 years ago

This pull request introduces 7 alerts and fixes 1 when merging cf4d652ba60b2bf1bd8c88ab02404fbef0235ca0 into 9d9c2622b7bd37eb053710d4b8c71248f1d84eb8 - view on LGTM.com

new alerts:

2 for Module is imported with 'import' and 'import from'
1 for Module is imported more than once
1 for Unused local variable
1 for Redundant assignment
1 for Wrong name for an argument in a class instantiation
1 for Wrong number of arguments in a class instantiation

fixed alerts:

1 for Unused import

lgtm-com[bot] commented 4 years ago

This pull request introduces 5 alerts and fixes 1 when merging 6a86f092dbe6721deacb144eb62a1b924b30968a into 9d9c2622b7bd37eb053710d4b8c71248f1d84eb8 - view on LGTM.com

new alerts:

2 for Module is imported with 'import' and 'import from'
1 for Module is imported more than once
1 for Unused local variable
1 for Redundant assignment

fixed alerts:

1 for Unused import

lgtm-com[bot] commented 4 years ago

This pull request introduces 5 alerts and fixes 1 when merging d69f2cdc615346da428151d19c9a63f3ecfe3156 into 9d9c2622b7bd37eb053710d4b8c71248f1d84eb8 - view on LGTM.com

new alerts:

2 for Module is imported with 'import' and 'import from'
1 for Module is imported more than once
1 for Unused local variable
1 for Redundant assignment

fixed alerts:

1 for Unused import

lgtm-com[bot] commented 4 years ago

This pull request introduces 5 alerts and fixes 1 when merging 76c3600f4206e70a23b2aaa41b83d864a1187b9b into d01d3da9c6ec3f3a88a9101ada4da528ce5c6e2d - view on LGTM.com

new alerts:

2 for Module is imported with 'import' and 'import from'
1 for Module is imported more than once
1 for Unused local variable
1 for Redundant assignment

fixed alerts:

1 for Unused import

lgtm-com[bot] commented 4 years ago

This pull request introduces 5 alerts and fixes 1 when merging 82d37317d1c08e2da3001997a7d55dd605a0a141 into d01d3da9c6ec3f3a88a9101ada4da528ce5c6e2d - view on LGTM.com

new alerts:

2 for Module is imported with 'import' and 'import from'
1 for Module is imported more than once
1 for Unused local variable
1 for Redundant assignment

fixed alerts:

1 for Unused import