Typo fix in DataTransformers.ipynb

[data-hound]

Under Section 2, Multiple Inputs

class MultiOutputTransformer(BaseEstimator, TransformerMixin):
    ...

class MultiOutputClassifier(KerasClassifier):

    @property
    def feature_encoder(self):
        return MultiInputTransformer(...)

changed to

class MultiInputTransformer(BaseEstimator, TransformerMixin):
    ...


class MultiInputClassifier(KerasClassifier):

    @property
    def feature_encoder(self):
        return MultiInputTransformer(...)

[adriangb]

Thank you for the bug report! I will review your PR shortly.

[codecov-io]

Codecov Report

Merging #176 (3da1bc3) into master (7fa51a4) will decrease coverage by 0.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #176      +/-   ##
==========================================
- Coverage   99.55%   99.09%   -0.46%     
==========================================
  Files           6        5       -1     
  Lines         679      666      -13     
==========================================
- Hits          676      660      -16     
- Misses          3        6       +3     

Impacted Files	Coverage Δ
scikeras/_saving_utils.py	96.20% <0.00%> (-3.80%)	⬇️
scikeras/wrappers.py	99.44% <0.00%> (-0.01%)	⬇️
scikeras/__init__.py

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7fa51a4...3da1bc3. Read the comment docs.

[adriangb]

Are the changes here intentional?

[data-hound]

Nope. After I ran the pre-commit test, and everything was fine, I was asked to stage this file for commit as well, and so I did.

[adriangb]

Hmm, not sure what happened there. Would you mind reverting this change then?

[adriangb]

I'm actually in the middle of a pretty large overhaul of the docs in #174, the main goal being to make it easier to edit notebooks. Would you mind waiting until I merge #174 and then we can use this change as a case study for the effectiveness of the new system?

[data-hound]

I see the #174 has been merged. I will make the changes then.

[adriangb]

Awesome, thank you! Please let me know if you encounter any issues with the new setup.

[adriangb]

Hey @data-hound it looks like the way I set up CI won't work for forks 🤦 . I think I'll be able to get it fixed, but it might take a couple of days. That is to say, don't worry about the CI failure, and sorry for wasting your time with broken CI. In the meantime, could you please revert the change to .pre-commit-config.yaml ? Thanks!

[adriangb]

I was able to make the changes. Thank you very much @data-hound !

[data-hound]

I was able to make the changes. Thank you very much @data-hound !

Mention Not!
Seems you did the rectification for the pre-commit-config.yaml in the branch as well - Thanks for that!