wpbonelli

Results 58 comments of wpbonelli

I'd be happy to help, particularly testing and bugfixes. I'd like to submit patches for #169 ~~and #170~~ with tests. Happy to work on converting to pytest as proposed in...

@hjia1005 apologies for the late reply here. I tried modifying a [test case](https://github.com/modflowpy/flopy/blob/4e1d53ac3685d9027b1ced45749c9e20d8b701d5/autotest/regression/test_mf6.py#L1346) to remove two different packages both of which were initialized with timeseries, and could not reproduce this....

Tentatively closing but please reopen if premature

I cannot reproduce the reported warning — however `HeadUFile.get_alldata()` fails on the previous line https://github.com/modflowpy/flopy/blob/4e1d53ac3685d9027b1ced45749c9e20d8b701d5/flopy/utils/datafile.py#L578 when called on `HeadUFile("/examples/data/unstructured/headu.githds")` due to array inhomogeneity ``` ValueError: setting an array element with...

Hi @giovannifi was this problem resolved for you? I [rechecked](https://github.com/w-bonelli/flopy/commit/6e65529ddfe8ee835a57da7df1c67faa0839fcf2) some of flopy's example `.hds` files and didn't encounter the issue. Wanted to confirm before closing as there is a...

@giovannifi if you're on mac and haven't previously installed certificates for your Python distribution, [this might help](https://stackoverflow.com/a/42098127/6514033)

I see the same on macOS 13.6.1, aarch64-apple-darwin22.3.0, GCC 13.2.0. It is indeed the `move` test that fails, in particular move to self: https://github.com/fortran-lang/stdlib/blob/b8fbb3ce1aaf2fd1729897739fcdae8f6b3c4ef4/test/string/test_string_intrinsic.f90#L717

It may be related to apple silicon — @Pranavchiku has not reported arch but presumably CI [would fail here](https://github.com/fortran-lang/stdlib/actions/runs/6631524902/job/18016039169#step:10:112) if it was universal on mac. I will investigate. **edit**: confirmed...

At this point it seems like a compiler problem. The issue does not reproduce in a minimal test https://github.com/wpbonelli/move-test on arm64. ~~My next step is to add a `test-drive` harness...

I think this can be closed, the deprecation of USG support was turned into an error with https://github.com/modflowpy/flopy/commit/696a209416f91bc4a9e61b4392f4218e9bd83408