Skip to content

Conversation

@aadimaxi
Copy link

Added checks for input type and missing columns in inverse_transform method.

Fixes #
added error handling for inverse_transform()

Proposed Changes

  • Added error validation for inverse_transform to handle missing columns gracefully instead of crashing

Added checks for input type and missing columns in inverse_transform method.
@aadimaxi
Copy link
Author

👋, I’m really excited to have contributed to this project!
This is my first open-source PR, and I learned a lot while working on improving the inverse_transform method with validation checks.

@maintainers — whenever possible, please tag me as a contributor once it’s merged, I’d love to share this contribution and spread the word about the amazing work being done here. 🙌

@PaulWestenthanner
Copy link
Collaborator

PaulWestenthanner commented Oct 31, 2025

Hi @aadimaxi
I'll have a look at it. The pipeline is currently failing but this is probably on my end.

EDIT: It's not on my end, the indentation seems to be wrong in your code. Please make sure to test your code properly, also consider adding unit tests if you modify some functionality

@PaulWestenthanner
Copy link
Collaborator

I've updated the master branch with some small fixes to the CI pipeline. Please rebase you branch on master and try again. Thank you for your contribution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants