Tony Hallam

If you reply to Github issues via email you can't add attachments. Perhaps better to have the discussion here https://swung.slack.com/archives/C01472D5QGP It's not clear to me why `VSP` raises a `NameError`...

**bump** @salsa360

So I've downloaded your data and tried to run an import export workflow. The following appears to work for me ``` seisnc_vol = segy_loader("09-02_crs_PostSTM_fisc.sgy.val", cdp=21, cdpx=181, cdpy=185) segy_writer(seisnc_vol, "test_out2d_2.sgy.val", {'cdp':21,...

Ok, Sorry, I didn't see the notebook. This cell is your issue ``` data['cdp_x'] = np.around(x_3034, 1) data['cdp_y'] = np.around(y_3034, 1) ``` When you assign numpy arrays back to and...

@musiczhzhao I'm sorry but this isn't a priority for me right now. I wasn't actually aware anyone was using the ZGY functionality in Segysak. I am happy to accept pull...

@da-wad , that's cool! I didn't know about this package. Definitely happy to devolve zgy handling to `pyzgy` at some point. Probably just needs some drop in changes to the...

Sounds good. Just a thought, `mmap` is meant to improve performance in segyio when reading large datasets. Maybe this function could be used to set the cache size with a...

Hi @musiczhzhao ZGY was a bit of an after thought so it is in it's own area at the moment. The fact that `pyzgy` mirrors `segyio` though is an opportunity...

Hi Zhao, I would simply start by replacing statements that say ``` import segyio ``` with ``` import pyzgy as segyio ``` and see what works and what breaks, are...

@da-wad does `pyzgy` support writing to `ZGY` at this time? I couldn't see any write methods in the source.