Repository: manujosephv/pytorch_tabular (language: python)

Issue:
bumping PL versions & compatibility
let's bump the PL version to the latest and if possible release it...
Fixes #76


Discussion / hints:
this error seems to be unrelated to the PL, as I downgraded PL back to the original and still seeing it... :confused:
 in `examples/to_test_regression_custom_models.py`
```
omegaconf.errors.ConfigAttributeError: Missing key _model_name
    full_key: _model_name
    object_type=dict
```

**UPDATE: seems that in examples is lots of dead code which won't run :(**
to reveal it you just add `pytest` argument `--doctest-modules`
@manujosephv fyi, I am running all the tests in fork CI, and so far so good, except for the dumping callback, is it really needed to keep/preserve them for any reason? :rabbit: 
```
TypeError: cannot pickle '_thread.RLock' object
```

**UPDATE: this seems to be resolved with upgrade to the latest PL 1.8**
@Borda Thank you for taking the time to go through all of these.. I have been slacking off on the repo due to some other commitments. But since the last few weeks, I've been working on an pretty major update to the repo.. major refactoring and addition of functionalities etc. It's almost done and I'll push the new version in a week or so.. I would love it if we can take a look at all this after the new update? If that is too much re-work, we can connect briefly to figure out the best way to make the re-work as less as possible... 

I agree, the examples and some of the tutorials have dead code in them.. and currently re-vamping them as well..

Also, I'll try and bump PL version to the latest one this release.. 

Incidentally, the `_model_name` error I just fixed today :) 
> Also, I'll try and bump PL version to the latest one this release..

I think that most of the changes here shall be safe, so if possible I would be happier to get this merge before additional refactoring... as I made it during my holiday and sire how much time I would be able to spend later on it... :innocent: 
BTW, I can try to rever some changes which are not essential, to reduce eventual collisions...
@manujosephv rebased, could you approve CI run? :otter: 

Fix patch:
diff --git a/.github/dependabot.yml b/.github/dependabot.yml
new file mode 100644
index 00000000..2a86ac6a
--- /dev/null
+++ b/.github/dependabot.yml
@@ -0,0 +1,38 @@
+# Basic dependabot.yml file with minimum configuration for two package managers
+
+version: 2
+updates:
+  # Enable version updates for python
+  - package-ecosystem: "pip"
+    # Look for a `requirements` in the `root` directory
+    directory: "/"
+    # Check for updates once a week
+    schedule:
+      interval: "monthly"
+    # Labels on pull requests for version updates only
+    labels:
+      - "dependencies"
+    pull-request-branch-name:
+      # Separate sections of the branch name with a hyphen for example, `dependabot-npm_and_yarn-next_js-acorn-6.4.1`
+      separator: "-"
+    # Allow up to 5 open pull requests for pip dependencies
+    open-pull-requests-limit: 5
+    reviewers:
+      - "manujosephv"
+
+  # Enable version updates for GitHub Actions
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    # Check for updates once a week
+    schedule:
+      interval: "monthly"
+    # Labels on pull requests for version updates only
+    labels:
+      - "enhancement"
+    pull-request-branch-name:
+      # Separate sections of the branch name with a hyphen for example, `dependabot-npm_and_yarn-next_js-acorn-6.4.1`
+      separator: "-"
+    # Allow up to 5 open pull requests for GitHub Actions
+    open-pull-requests-limit: 5
+    reviewers:
+      - "manujosephv"


Test patch:
diff --git a/tests/test_categorical_embedding.py b/tests/test_categorical_embedding.py
index 67ec69e3..40a95121 100644
--- a/tests/test_categorical_embedding.py
+++ b/tests/test_categorical_embedding.py
@@ -59,7 +59,7 @@ def test_regression(
     # custom_loss,
     # custom_optimizer,
     custom_args,
-    custom_head_config
+    custom_head_config,
 ):
     (train, test, target) = regression_data
     (custom_metrics, custom_loss, custom_optimizer) = custom_args
@@ -212,15 +212,6 @@ def test_embedding_transformer(regression_data):
 
     transformer = CategoricalEmbeddingTransformer(tabular_model)
     train_transform = transformer.fit_transform(train)
-    embed_cols = [
-        col for col in train_transform.columns if "HouseAgeBin_embed_dim" in col
-    ]
-    assert len(train["HouseAgeBin"].unique()) + 1 == len(
-        transformer._mapping["HouseAgeBin"].keys()
-    )
-    assert all(
-        [
-            val.shape[0] == len(embed_cols)
-            for val in transformer._mapping["HouseAgeBin"].values()
-        ]
-    )
+    embed_cols = [col for col in train_transform.columns if "HouseAgeBin_embed_dim" in col]
+    assert len(train["HouseAgeBin"].unique()) + 1 == len(transformer._mapping["HouseAgeBin"].keys())
+    assert all([val.shape[0] == len(embed_cols) for val in transformer._mapping["HouseAgeBin"].values()])