search

MODFLOW-USGS / modflow6

USGS Modular Hydrologic Model
https://modflow6.readthedocs.io/
Other
253 stars 116 forks source link

binary exchange data #128

Closed jdhughes-usgs closed 5 months ago

jdhughes-usgs commented 5 years ago

evaluate option to specify exchange data as binary data

```
BEGIN EXCHANGEDATA
    <cellidm1> <cellidm2> <ihc> <cl1> <cl2> <hwva> [<aux(naux)>]
    <cellidm1> <cellidm2> <ihc> <cl1> <cl2> <hwva> [<aux(naux)>]
    ...
END EXCHANGEDATA
```

Support this option

```
BEGIN EXCHANGEDATA
    OPEN/CLOSE exchange.exc (BINARY)
END EXCHANGEDATA
```
langevin-usgs commented 10 months ago

@mjreno, is this supported now by using IDM (Input Data Model) loading routines for exchanges (#1505)?

mjreno commented 10 months ago

@langevin-usgs This hasn't been tested yet but should be close if not already done. I'll find or generate a suitable input and test and then will update this issue.

langevin-usgs commented 5 months ago

@mjreno, I suspect this is supported now. If so, can you close this PR?

mjreno commented 5 months ago

@langevin-usgs apologies for losing track of this one, will verify and close.

langevin-usgs commented 5 months ago

@langevin-usgs apologies for losing track of this one, will verify and close.

No probem! Was just looking through these issues and think some might be fixed now because of IDM.

mjreno commented 5 months ago

@langevin-usgs just thinking about this a bit- the struct array input reader already can read similar binary inputs. The current use case is for stress period data, however, so currently the control record is read and binary read interface invoked only in the dynamic loader. The change needed would be update the static loader to do the same.

If this sounds right to you it wouldn't be a lot of work on the MODFLOW 6 side- would FloPy also need to be updated to generate binary exchange list style input?

mjreno commented 5 months ago

Closing as this has been addressed with #1784 and #1783.