-
Notifications
You must be signed in to change notification settings - Fork 62
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
Small errors with new REViSE, CCHVAE, and CRUD code #184
Comments
Hi,
but I'm not sure if that's the problem that you mean.
|
Is what is described for 1. a good solution for this issue? And 2. is being fixed in PR 187. |
Hi,
from carla.data.catalog.online_catalog import OnlineCatalog
from carla.models.catalog import MLModelCatalog
from carla.models.negative_instances import predict_negative_instances
from carla.recourse_methods.catalog import GrowingSpheres
# load a catalog dataset
data_name = "adult"
dataset = OnlineCatalog(data_name)
# load artificial neural network from catalog
model = MLModelCatalog(dataset, "ann", "tensorflow")
# get factuals from the data to generate counterfactual examples
factuals = predict_negative_instances(model, dataset.df)
test_factual = factuals.iloc[:5]
# load a recourse model and pass black box model
gs = GrowingSpheres(model)
# generate counterfactual examples
counterfactuals = gs.get_counterfactuals(test_factual)
hyperparams = {
"data_name": dataset.name,
"n_search_samples": 100,
"p_norm": 1,
"step": 0.1,
"max_iter": 1000,
"clamp": True,
"binary_cat_features": False,
"vae_params": {
"layers": [len(ml_model.feature_input_order), 512, 256, 8],
"train": True,
"lambda_reg": 1e-6,
"epochs": 5,
"lr": 1e-3,
"batch_size": 32,
},
} should be changed to hyperparams = {
"data_name": dataset.name,
"n_search_samples": 100,
"p_norm": 1,
"step": 0.1,
"max_iter": 1000,
"clamp": True,
"binary_cat_features": False,
"vae_params": {
"layers": [sum(model.get_mutable_mask()), 512, 256, 8],
"train": True,
"lambda_reg": 1e-6,
"epochs": 5,
"lr": 1e-3,
"batch_size": 32,
},
}
|
|
Great! Looking forward to see all the changes :) |
Hi,
Since the latest release of CARLA, I think there are some errors that have popped up.
The biggest problem is that (I believe) the dimension of the input layer of the VAE has to be adjusted for REViSE, CCHVAE, and CRUD if the immutable_mask contains at least one TRUE.
In addition, when running the methods on a GPU, there is some code that has to be adjusted (so far, I've only found problems for these three methods but I haven't tested all of them).
I'm not an expert on these methods, but can I do a pull request where I make changes to these methods such that they run again (on GPU)?
Thanks,
Annabelle
The text was updated successfully, but these errors were encountered: