Skip to content

Conversation

ricopicone
Copy link

… an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (673,) + inhomogeneous part.

Read more about this error here: https://stackoverflow.com/questions/67183501/setting-an-array-element-with-a-sequence-requested-array-has-an-inhomogeneous-sh

… an array element with a sequence. The requested array has an inhomogeneous shape after 1 dimensions. The detected shape was (673,) + inhomogeneous part.
@jankoslavic
Copy link
Contributor

Dear @ricopicone thank you for the PR. Looks great. Can you please add a small test for this PR? I suggest adding a test where a sample file would be read and would raise error if this PR is not implemented and would work properly if it is implemented.
Would this be doable from your side?

@ricopicone
Copy link
Author

Thanks @jankoslavic. Yes, I can write a test. It will probably be about a week before I can get it it, but I'll do it ASAP! Thanks for writing this package—it's saving me lots of time!

@jankoslavic
Copy link
Contributor

jankoslavic commented Sep 30, 2025

Thank you for the feedback @ricopicone ! It is nice to hear it saved you time. this is why I love open source :)
The test should be very simple, e.g.:

  1. test file similar to this: https://github.com/ladisk/lvm_read/blob/master/data/short.lvm
  2. and the just adding a read function to all tests: https://github.com/ladisk/lvm_read/blob/master/tests/test_all.py

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