Hi everyone! I've been exploring Metaflow and foun...
# dev-metaflow
b
Hi everyone! I've been exploring Metaflow and found issue #3017 (decompress_list crashes on empty string input). I've implemented and tested a fix for this. The Problem:
decompress_list("")
throws an IndexError instead of returning
[]
, breaking the invariant that
decompress_list(compress_list(x)) == x
. The Solution: Added a guard clause at the start of the function: Complete Implementation: https://github.com/debojit11070/metaflow/tree/fix/3017-decompress-empty-string Includes: • Fix to util.py • Comprehensive test suite with 7 test cases (all passing) • Ready to merge Since I'm a new contributor, I can't directly open a PR. I am getting the following message "An owner of this repository has limited the ability to open a pull request to users that have contributed to this repository in the past", but the code is solid and ready for review. Would a maintainer be able to review and merge this, or would you prefer I approach it differently? @wooden-dusk-90720 can you help me with this?